Skip to content

fix: add mcp-list slash command#287

Open
SlaVKsVolks wants to merge 1 commit into
CortexLM:mainfrom
SlaVKsVolks:codex/mcp-list-slash
Open

fix: add mcp-list slash command#287
SlaVKsVolks wants to merge 1 commit into
CortexLM:mainfrom
SlaVKsVolks:codex/mcp-list-slash

Conversation

@SlaVKsVolks
Copy link
Copy Markdown

Summary

  • adds /mcp-list as a dedicated MCP slash command
  • routes it to the existing MCP manager, matching the other MCP slash-command entry points
  • covers registry discovery and command execution with focused tests

Fixes PlatformNetwork/bounty-challenge#53252.

Verification

  • rustfmt --edition 2024 --check src/cortex-tui/src/commands/registry/builtin.rs src/cortex-tui/src/commands/registry/mod.rs src/cortex-tui/src/commands/executor/dispatch.rs src/cortex-tui/src/commands/executor/tests.rs
  • cargo test -p cortex-tui commands::registry::tests::test_specific_commands_registered
  • cargo test -p cortex-tui commands::executor::tests::test_mcp_list_opens_mcp_manager
  • cargo check -p cortex-tui
  • git diff --check

Note: full cargo fmt --check --package cortex-tui is currently blocked by a pre-existing formatting diff in src/cortex-tui/src/question.rs, outside this PR.

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

Development

Successfully merging this pull request may close these issues.

[BUG][v0.0.7] TUI slash registry exposes mcp-auth but not a dedicated mcp-list alias for cortex mcp list

1 participant