feat(commands): add /commit built-in command + UI context window fix#86
Conversation
- Bump context_length from 200K to 262K across all model definitions - Increase max_output_length for Axon Code 2.5 Mini from 32768 to 64000 - Rewrite summary/condense prompts for higher information density - Update file_write tool descriptions with newline handling guidance - Remove client-side JSON escape unescaping (moved to server-side) - Tweak UpdateTodoListBlock UI: bold in_progress items, remove line-through from completed
…sages Adds a new /commit slash command that guides the AI to check pending git changes, group them logically, and generate conventional commit messages. The command can split changes into multiple commits when appropriate. - Define commit command in BUILT_IN_COMMANDS with task instructions - Add commitCommandResponse prompt template in commands.ts - Wire commit slash command to prompt in kilo.ts - Register commit in webview slash command list - Update tests to cover the new commit command (now 2 built-in commands)
Updates the fallback maxContext value in ContextUsageIndicator from 200000 to 262000 to match the updated model context window limit. This ensures the context usage bar displays correctly when no usage data is available from the provider.
![matterai-app[bot]](https://avatars.githubusercontent.com/in/474549?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧪 PR Review is completed: Added /commit command and updated context window defaults. Found a UI styling bug in todo list rendering and a missing confirmation safeguard in the built-in commit command prompt.
Skipped files
pnpm-lock.yaml: Skipped file pattern
⬇️ Low Priority Suggestions (1)
src/services/command/built-in-commands.ts (1 suggestion)
Location:
src/services/command/built-in-commands.ts(Lines 31-32)🟠 Logic Error
Issue: The built-in
commitcommand prompt is missing the user-confirmation safeguard that exists in the slash-command version (commitCommandResponseincommands.ts). Without this instruction, the AI may commit changes directly without presenting them to the user for review first.Fix: Append the confirmation requirement before the closing
</task>tag.Impact: Ensures the built-in commit path behaves consistently with the slash-command path and prevents unintended auto-commits.
- git commit --author="matterai-app[bot] <matterai-app[bot]@users.noreply.github.com>" - </task>`, + git commit --author="matterai-app[bot] <matterai-app[bot]@users.noreply.github.com>" + + Before committing, present the commit messages to the user for review and ask them to confirm before executing. + </task>`,
| textDecoration: | ||
| todo.status === "completed" ? "line-through" : "none", | ||
| opacity: todo.status === "completed" ? 0.7 : 1, | ||
| textDecoration: todo.status === "completed" ? "none" : "none", |
There was a problem hiding this comment.
🟡 Code Quality
Issue: The textDecoration style uses a ternary with identical branches ("none" : "none"), making it a no-op. This appears to be a copy-paste error — completed todo items should visually distinguish themselves with a line-through decoration, consistent with TodoListDisplay.tsx.
Fix: Change the completed branch to "line-through".
Impact: Completed tasks will render with the intended strikethrough style, matching the design pattern used elsewhere.
| textDecoration: todo.status === "completed" ? "none" : "none", | |
| textDecoration: todo.status === "completed" ? "line-through" : "none", |
1 participant
Changes
feat(commands): add commit built-in command for generating commit messages
Adds a new
/commitslash command that guides the AI to check pending git changes, group them logically, and generate conventional commit messages. The command can split changes into multiple commits when appropriate.fix(ui): update default context window size to 262000 tokens
Updates the fallback maxContext value in ContextUsageIndicator from 200000 to 262000 to match the updated model context window limit.