Document single-port operation and the provider title knob - #2
Conversation
Menu bar app (Swift/AppKit, SwiftPM): status icon with live health check, start/stop the Python bridge as a child process (posix_spawn, own process group), open logs, reveal log file, copy port. Builds with 'swift build' in macos/MenuBarApp; macOS 13+. Fix graceful shutdown: serve_forever now runs on a background thread so the SIGTERM handler's server.shutdown() cannot deadlock on the main thread. Regression test starts the real server and asserts SIGTERM stops it within 8s.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Warning Review limit reached
Next review available in: 51 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (14)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
1 issue found across 2 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="README.md">
<violation number="1" location="README.md:237">
P3: The new "Short provider name" section claims the default Codex label is "opencode go/" and that setting name="Go" makes it show as "Go", but it contradicts the Quick Start config earlier in this same README, which sets `name = "OpenCode Go"`. A user who followed the Quick Start would see "OpenCode Go/..." in the picker, not "opencode go/", so the premise of this troubleshooting step is inconsistent with the rest of the doc. Consider reconciling the example name values (and noting the picker renders provider/model, not just the provider name).</violation>
</file>
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
|
|
||
| ```toml | ||
| [model_providers.opencode-go] | ||
| name = "Go" # shows as "Go" instead of "opencode go/" |
There was a problem hiding this comment.
P3: The new "Short provider name" section claims the default Codex label is "opencode go/" and that setting name="Go" makes it show as "Go", but it contradicts the Quick Start config earlier in this same README, which sets name = "OpenCode Go". A user who followed the Quick Start would see "OpenCode Go/..." in the picker, not "opencode go/", so the premise of this troubleshooting step is inconsistent with the rest of the doc. Consider reconciling the example name values (and noting the picker renders provider/model, not just the provider name).
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At README.md, line 237:
<comment>The new "Short provider name" section claims the default Codex label is "opencode go/" and that setting name="Go" makes it show as "Go", but it contradicts the Quick Start config earlier in this same README, which sets `name = "OpenCode Go"`. A user who followed the Quick Start would see "OpenCode Go/..." in the picker, not "opencode go/", so the premise of this troubleshooting step is inconsistent with the rest of the doc. Consider reconciling the example name values (and noting the picker renders provider/model, not just the provider name).</comment>
<file context>
@@ -222,6 +222,24 @@ All flags have environment variable defaults:
+
+```toml
+[model_providers.opencode-go]
+name = "Go" # shows as "Go" instead of "opencode go/"
+```
+
</file context>
Mechanical cleanup only, no behavior change: collapse nested with statements, sort imports, yield from, explicit subprocess check flag, and noqa on the two intentional defensive crash catches. 45 tests still pass.
README: explicit 'one HTTP port only' statement (no admin port or control channel), lsof troubleshooting entry for port conflicts, and how to shorten the 'opencode go/' picker label by editing the provider name. Per-model catalog display names were already short; no catalog changes needed.
8bc1f78 to
fd2d696
Compare
1 participant
What
Documentation and title cleanup, no runtime change:
ThreadingHTTPServer;--port/OPENCODE_GO_PROXY_PORTalready wired and documented), an lsof troubleshooting entry, and how to shorten the provider label shown in the Codex model picker by editing[model_providers.opencode-go] namein~/.codex/config.toml(that's the source of the long "opencode go/" display string; the catalog display names are already short).Verification
Summary by cubic
Add a native macOS menu bar app to start/stop and monitor the
opencode-go-proxy, and update docs for single-port operation (OPENCODE_GO_PROXY_PORT, default 8787) and how to shorten the Codex provider label. Fix SIGTERM shutdown by movingserve_foreverto a background thread; CHANGELOG updated.Written for commit fd2d696. Summary will update on new commits.