Skip to content

Conversation

@batuhan
Copy link
Member

@batuhan batuhan commented Feb 1, 2026

No description provided.

Introduces ai.tpl.yaml for AI bridge network configuration and adds 'beeper-ai' to the list of websocket bridges in bridgeutil.go.
This commit introduces support for the 'ai' bridge across build, configuration, and utility files. It updates bridge lists, repository URLs, and configuration templates to properly recognize and handle the AI bridge.
Moved 'ai' bridge type to its own case in runBridge, with specific logic for binary selection and compilation. Removed 'ai' from shared bridge lists in both build.go and run.go to improve maintainability and clarity.
@coderabbitai
Copy link

coderabbitai bot commented Feb 1, 2026

📝 Walkthrough

Summary by CodeRabbit

  • New Features
    • Added support for configuring and running AI bridge instances with customizable network settings, provider configurations, and model caching options.

✏️ Tip: You can customize this high-level summary in your review settings.

Walkthrough

Introduces support for a new "ai" bridge type to the bbctl command-line tool, consisting of a YAML template configuration, registration in the bridges registry, and build/runtime logic with custom repository mapping and binary resolution.

Changes

Cohort / File(s) Summary
AI Bridge Configuration
bridgeconfig/ai.tpl.yaml
New YAML template file that defines AI bridge configuration with network settings templated from HungryAddress and ProvisioningSecret credentials, multiple provider models, system prompt, cache duration, and command prefix.
Bridge Registration & Runtime Support
cmd/bbctl/bridgeutil.go, cmd/bbctl/run.go
Registered "ai" as a new official bridge type in the bridges registry and websocket bridges map; implemented compilation and runtime support including custom repository mapping to ai-bridge, binary resolution logic (data directory or local build), local-dev build script support, and Go compilation in compile mode.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ❓ Inconclusive No description was provided by the author, making it impossible to evaluate whether it relates to the changeset. Add a pull request description explaining the purpose and scope of adding AI bridge support.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Add AI bridge' clearly summarizes the main change - adding support for a new AI bridge type across configuration and code.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch batuhan/ai

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant