Skip to content

feat(routing): add KiloClaw as a named routing mode#1669

Merged
chrarnoldus merged 8 commits intomainfrom
session/agent_2fbfe5db-0794-450e-a989-239956724738
Mar 27, 2026
Merged

feat(routing): add KiloClaw as a named routing mode#1669
chrarnoldus merged 8 commits intomainfrom
session/agent_2fbfe5db-0794-450e-a989-239956724738

Conversation

@kilo-code-bot
Copy link
Copy Markdown
Contributor

@kilo-code-bot kilo-code-bot bot commented Mar 27, 2026

Summary

  • Added kiloclaw as an explicit routing mode in both FRONTIER_MODE_TO_MODEL and BALANCED_MODE_TO_MODEL in describeRouting
  • For Frontier: routes to Claude Opus with reasoning enabled and high verbosity (same config as plan)
  • For Balanced: routes to Kimi with reasoning enabled (same config as plan)
  • Updated applyResolvedAutoModel to pass 'kiloclaw' directly as the mode instead of mapping it to 'plan', so KiloClaw now appears in the routing description

Verification

  • Reviewed the diff manually; the change is additive and consistent with existing mode configurations
  • Typecheck could not be run due to missing tsgo binary in the environment

Visual Changes

N/A

Reviewer Notes

Previously, featureHeader === 'kiloclaw' was mapped to 'plan' before lookup, so KiloClaw was silently sharing plan's routing without appearing in describeRouting. Now it has its own named entry, making the routing description accurate and allowing independent tuning in future.

@kilo-code-bot
Copy link
Copy Markdown
Contributor Author

kilo-code-bot bot commented Mar 27, 2026

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
src/lib/kilo-auto-model.ts 266 Passing mixed-case KiloClaw still bypasses the resolver lookup because x-kilocode-mode is not lowercased upstream, so KiloClaw requests fall back to the default code model
Other Observations (not in diff)

N/A

Fix these issues in Kilo Cloud

Files Reviewed (2 files)
  • src/lib/kilo-auto-model.ts - 1 issue
  • src/tests/openrouter-models-sorting.approved.json - 0 issues

@chrarnoldus
Copy link
Copy Markdown
Contributor

so it shows up in the description

@chrarnoldus chrarnoldus merged commit d396711 into main Mar 27, 2026
11 checks passed
@chrarnoldus chrarnoldus deleted the session/agent_2fbfe5db-0794-450e-a989-239956724738 branch March 27, 2026 19:54
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