Repository navigation
fix(opencode): seat plain gemini-4-argon in argon-plan - #1530
Conversation
The contributor spelling (gemini-4-argon-contributor, from models.dev's listing) is absent from the Go workspaces' live catalog as the serves resolve it: with only contributor leaves, argon-plan expanded to zero leaves and vanished from the deployed serves (verified live after the #1520 deploy — 21 of 22 routes resolved). Plain gemini-4-argon is served on all three workspaces; seat it instead. The contributor spelling returns if the live catalog gains it.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Update Argon route targets .opencode/opencode.jsonc |
The three provider targets now use gemini-4-argon instead of gemini-4-argon-contributor. The comments describe which model names appear in the live Go catalogs. |
Priority:
Estimated code review effort: 2 (Simple) | ~8 minutes
Change: Bug fix
Possibly related issues
- Issue 1460: The route uses the model name that the live catalog resolves, matching the issue’s objective to keep route-pool seats usable when providers are enabled.
Merge Risk: 
Minimal · up to 506ef
-
The route now targets the plain Argon model name across all three Go workspaces. No supported defect blocks merging after normal checks.
Pre-merge checks: 
5
-
Full details: Title check
-
Explanation
The title names the actual change: the argon-plan route now uses the plain gemini-4-argon model instead of gemini-4-argon-contributor. The raw summary confirms the route now targets gemini-4-argon in the Personal, Lab, and base OpenCode Go providers. The title is specific, short, and describes the primary change from the developer's perspective. It uses the conventional fix(scope) prefix.
Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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 @coderabbitai help to get the list of available commands.
Follow-up to #1520, found during its deploy verification.
#1520 seated
gemini-4-argon-contributor×3 in argon-plan (the spelling models.dev lists). The deployed serves' live catalogs do not carry that spelling — with only contributor leaves, argon-plan expanded to zero leaves and silently vanished: the background serve resolved 21 of 22 routes after the deploy, andopencode modelsshows plaingemini-4-argonon all three Go workspaces but no contributor variant anywhere.Fix: seat plain
gemini-4-argon×3, and record the finding in the route comment (the contributor spelling returns if the live catalog gains it). Specialist/summit were degraded gracefully in the interim (their ordered chains skipped the empty argon-plan reference); this restores their #2 seat.Verified before opening: every other new leaf from #1520 resolves in the live serve catalog (Go DeepSeek Pro/MiMo/Flash/Haiku/Beam/Luna, Command Code DeepSeek/MiMo) — Argon-contributor was the only miss.
Need help on this PR? Tag
@codesmith-botwith what you need.Summary by CodeRabbit