Skip to content

Remove the duplicate MCP slash command alias - #859

Merged
TheGreatAxios merged 1 commit into
mainfrom
cl-7546-remove-duplicate-mcps-slash-command
Sep 10, 2026
Merged

Remove the duplicate MCP slash command alias#859
TheGreatAxios merged 1 commit into
mainfrom
cl-7546-remove-duplicate-mcps-slash-command

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

  • The slash menu and command registry expose /mcp only
  • /mcps is no longer registered as an alias for the same overlay
  • MCP docs and the changelog describe a single /mcp command

Verification

  • bun run typecheck, bun run build, and bun run test pass
  • bun run check exits 0
  • bun test ./src/tui/commands/built-in.test.ts passes (32 tests)

Fixes CL-7546

@linear-code

linear-code Bot commented Sep 10, 2026

Copy link
Copy Markdown

CL-7546

@TheGreatAxios TheGreatAxios left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Skywalker · Comment

Removes the duplicate /mcps alias. /mcp remains the only slash command for the MCP overlay.

GitHub rejects self-approve on this PR. Verdict is still merge.

Residual findings: none
Waivers: none

@TheGreatAxios TheGreatAxios left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Critic · Comment

No blocking or should-fix findings on origin/main..HEAD.

registerCommand now registers /mcp only. Typed /mcps is unknown. The built-in command test asserts the overlay for /mcp and that /mcps is unregistered. Docs and changelog match.

Hygiene this diff introduced: none.

@TheGreatAxios TheGreatAxios left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Neckbeard · Comment

No blockers. Changelog-bucket and alias-policy nits only; do not apply.

The branch removes the extra name, updates the one doc that listed both, and asserts absence.

@TheGreatAxios
TheGreatAxios merged commit d8b69f1 into main Sep 10, 2026
5 checks passed
@TheGreatAxios
TheGreatAxios deleted the cl-7546-remove-duplicate-mcps-slash-command branch September 10, 2026 02:30
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.

1 participant