Skip to content

fix(tui): add MCP list and remove slash entries#337

Open
OnlyYu1996 wants to merge 1 commit into
CortexLM:mainfrom
OnlyYu1996:fix-tui-mcp-slash-aliases
Open

fix(tui): add MCP list and remove slash entries#337
OnlyYu1996 wants to merge 1 commit into
CortexLM:mainfrom
OnlyYu1996:fix-tui-mcp-slash-aliases

Conversation

@OnlyYu1996
Copy link
Copy Markdown

Fixes PlatformNetwork/bounty-challenge#53252.
Fixes PlatformNetwork/bounty-challenge#53255.

Summary

  • register first-class /mcp-list with /mcp-ls alias for configured server discovery
  • register first-class /mcp-remove with /mcp-rm alias next to the existing MCP slash commands
  • route the new slash entries to the MCP manager, matching existing /mcp, /mcp-tools, and /mcp-auth behavior
  • add registry and executor coverage for the new entries

Validation

  • rustfmt +stable --check src/cortex-tui/src/commands/executor/dispatch.rs src/cortex-tui/src/commands/executor/tests.rs src/cortex-tui/src/commands/registry/builtin.rs src/cortex-tui/src/commands/registry/mod.rs
  • cargo +stable test --package cortex-tui commands::registry::tests::test_specific_commands_registered
  • cargo +stable test --package cortex-tui test_mcp_subcommands
  • git diff --check

Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant