Skip to content

fix(cli): allow openrouter models in routed model resolution - #12430

Merged
chrarnoldus merged 1 commit into
mainfrom
fix/routed-model-openrouter
Jul 21, 2026
Merged

fix(cli): allow openrouter models in routed model resolution#12430
chrarnoldus merged 1 commit into
mainfrom
fix/routed-model-openrouter

Conversation

@chrarnoldus

Copy link
Copy Markdown
Collaborator

No description provided.

@chrarnoldus chrarnoldus self-assigned this Jul 21, 2026
@kilo-code-bot

kilo-code-bot Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Small, well-scoped one-line fix confined to packages/opencode/src/kilocode/ (no fork-hygiene concerns). The condition now also treats openrouter/-prefixed model IDs as auto-routed, which matches the PR's intent.

Two minor, non-blocking notes:

  • No test was added covering the new openrouter/ prefix case in session-routed-model.test.ts (existing tests cover kilo-auto/ and unrelated provider IDs, but not openrouter/).
  • No changeset was added; this changes user-visible routed-model label/footer behavior for openrouter/-prefixed models.
Files Reviewed (1 file)
  • packages/opencode/src/kilocode/session/routed-model.ts

Reviewed by claude-sonnet-5 · Input: 20 · Output: 3.7K · Cached: 393.6K

Review guidance: REVIEW.md from base branch main

@chrarnoldus
chrarnoldus merged commit 8465c1d into main Jul 21, 2026
29 checks passed
@chrarnoldus
chrarnoldus deleted the fix/routed-model-openrouter branch July 21, 2026 13:15
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
…router

fix(cli): allow openrouter models in routed model resolution
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants