Skip to content

fix: allow normal messages starting with /#466

Merged
stdrc merged 2 commits intomainfrom
rc/fix-double-slash
Dec 12, 2025
Merged

fix: allow normal messages starting with /#466
stdrc merged 2 commits intomainfrom
rc/fix-double-slash

Conversation

@stdrc
Copy link
Copy Markdown
Contributor

@stdrc stdrc commented Dec 12, 2025

Fix #442

Signed-off-by: Richard Chien <stdrc@outlook.com>
Copilot AI review requested due to automatic review settings December 12, 2025 09:04
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes issue #442 by allowing normal messages that start with "/" to be sent to the agent instead of being incorrectly treated as meta commands. The fix validates slash-prefixed input against registered meta commands before processing, and only treats it as a meta command if it matches a valid command name.

Key Changes:

  • Modified the meta command check to validate command names using get_meta_command() before treating input as a meta command
  • Refactored _run_meta_command() to accept a MetaCommand object and args list instead of parsing internally
  • Added CHANGELOG entry documenting the fix

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/kimi_cli/ui/shell/init.py Refactored slash-prefix handling to validate command names before treating as meta commands, allowing invalid meta commands to fall through as normal messages
CHANGELOG.md Added entry documenting the fix for normal messages starting with /

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: Richard Chien <stdrc@outlook.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@stdrc stdrc merged commit 5de8b6d into main Dec 12, 2025
11 checks passed
@stdrc stdrc deleted the rc/fix-double-slash branch December 12, 2025 09:26
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.

Meta command // not found

2 participants