docs(auto-model): clarify KiloClaw plan-mode routing#1666
Closed
kilo-code-bot[bot] wants to merge 1 commit intomainfrom
Closed
docs(auto-model): clarify KiloClaw plan-mode routing#1666kilo-code-bot[bot] wants to merge 1 commit intomainfrom
kilo-code-bot[bot] wants to merge 1 commit intomainfrom
Conversation
| ); | ||
| // KiloClaw is an autonomous cloud agent without user-selected editor modes, so we | ||
| // force mode to 'plan' to route it to the highest-capability reasoning model | ||
| // (Opus for frontier, Kimi for balanced) with reasoning enabled and high verbosity. |
Contributor
Author
There was a problem hiding this comment.
SUGGESTION: The comment overstates the balanced plan route
BALANCED_MODE_TO_MODEL.plan enables reasoning, but it does not set verbosity, so "with reasoning enabled and high verbosity" reads as if both the frontier and balanced KiloClaw paths get high verbosity. Rephrasing this to scope high verbosity to the frontier route would keep the documentation aligned with the implementation.
Contributor
Author
Code Review SummaryStatus: 1 Issue Found | Recommendation: Address before merge Overview
Issue Details (click to expand)SUGGESTION
Other Observations (not in diff)Issues found in unchanged code that cannot receive inline comments: N/A Files Reviewed (1 files)
Fix these issues in Kilo Cloud Reviewed by gpt-5.4-20260305 · 155,341 tokens |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
src/lib/kilo-auto-model.tsexplaining how KiloClaw requests are routed through the auto-model system. KiloClaw forces the mode to'plan'(overriding the client-sent mode header) to ensure it always resolves to the highest-capability reasoning model: Claude Opus for frontier, Kimi for balanced, both with reasoning enabled.effectiveModevariable for readability, and added cross-reference comments on theplanentries in bothFRONTIER_MODE_TO_MODELandBALANCED_MODE_TO_MODELrouting tables so readers understand these entries serve KiloClaw in addition to the regularplaneditor mode.Verification
node_modules/tsgoin the environment, but the change is purely additive comments and a trivial refactor (extracting an inline expression to a named const).Visual Changes
N/A
Reviewer Notes
The key insight documented here:
applyResolvedAutoModeloverrides the mode to'plan'whenfeatureHeader === 'kiloclaw'because KiloClaw is an autonomous cloud agent that doesn't use interactive editor modes. This was previously undocumented and non-obvious to anyone reading the routing tables or the resolve logic.