Skip to content

Removes Ably MCP and positions Ably CLI as Public Preview#138

Merged
umair-ably merged 1 commit intomainfrom
removeMCP
Feb 25, 2026
Merged

Removes Ably MCP and positions Ably CLI as Public Preview#138
umair-ably merged 1 commit intomainfrom
removeMCP

Conversation

@umair-ably
Copy link
Contributor

@umair-ably umair-ably commented Feb 25, 2026

Removes Ably MCP and positions Ably CLI as Public Preview

supersedes #137


Note

Low Risk
Mostly deletions of an experimental feature plus documentation/config cleanup; limited risk to core CLI behavior beyond potential help/command filtering edge cases.

Overview
Removes the experimental Model Context Protocol (MCP) server feature entirely: deletes the ably mcp command group, the MCP server implementation under src/mcp/, related type shims, and all MCP-focused unit tests.

Updates docs and project rules to position this as Ably CLI (not “CLI and MCP server”), adds a Public Preview note in README.md, and cleans up references to removed docs (docs/Product-Requirements.md, docs/TODO.md) and MCP in contribution/workflow guidance.

Cleans up tooling/config by dropping the @modelcontextprotocol/sdk dependency, removing MCP-specific ESLint/coverage exclusions, and adjusting web/interactive-mode command restriction logic/tests now that mcp* is no longer a special case.

Written by Cursor Bugbot for commit 1e74e43. This will update automatically on new commits. Configure here.

Summary by CodeRabbit

Release Notes

  • Removed Features

    • Removed Model Context Protocol (MCP) server functionality and all associated commands.
  • Documentation

    • Updated CLI branding to focus on "Ably CLI."
    • Added Public Preview status indicator to CLI documentation.
    • Streamlined documentation to reflect current CLI capabilities.
  • Chores

    • Removed MCP-related dependencies.

@vercel
Copy link

vercel bot commented Feb 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cli-web-cli Ready Ready Preview, Comment Feb 25, 2026 0:47am

Request Review

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

This is the final PR Bugbot will review for you during this billing cycle

Your free Bugbot reviews will reset on March 4

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@coderabbitai
Copy link

coderabbitai bot commented Feb 25, 2026

Walkthrough

This PR removes all Model Context Protocol (MCP) server functionality from the codebase, including the MCP command implementation, server logic, type definitions, dependencies, and associated tests and documentation references.

Changes

Cohort / File(s) Summary
Configuration & Build
.nycrc.json, eslint.config.js, package.json
Removed MCP-specific coverage exclusions, ESLint rule overrides, and the @modelcontextprotocol/sdk dependency.
Documentation
README.md, docs/Product-Requirements.md, docs/Project-Structure.md, docs/TODO.md
Removed all MCP-related sections, usage examples, server descriptions, and TODO items; updated CLI branding and table of contents.
MCP Command Layer
src/commands/mcp/index.ts, src/commands/mcp/start-server.ts
Deleted MCP command topic class and start-server command implementation.
MCP Server Implementation
src/mcp/index.ts, src/mcp/mcp-server.ts
Removed MCP server class (1291 lines) including Ably integration, resource/tool provisioning, and command execution logic.
Type Definitions
src/types/modelcontextprotocol.d.ts
Deleted ambient module declarations for MCP SDK types and factory functions.
Core CLI Configuration
src/base-command.ts, src/help.ts
Removed MCP from restricted/unsuitable command lists and cleaned up MCP-specific error handling branches.
Unit & Integration Tests
test/unit/commands/mcp/mcp.test.ts, test/unit/base/base-command-enhanced.test.ts, test/unit/base/base-command.test.ts, test/integration/interactive-mode.test.ts, test/unit/commands/interactive-autocomplete.test.ts, test/unit/help/interactive-help.test.ts, test/unit/help/web-cli-help.test.ts
Removed MCP test cases, command restrictions assertions, and test data references.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 Farewell, dear MCP, once shiny and bright,
We've swept out your files and removed you from sight,
The server's now gone, the commands have fled,
Our CLI springs lighter, with fewer threads spread,
Now hopping forward with cleaner code instead! 🌿✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main changes: removal of MCP functionality and positioning CLI as Public Preview.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch removeMCP

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@umair-ably umair-ably requested a review from AndyTWF February 25, 2026 12:09
@umair-ably umair-ably merged commit 040e7df into main Feb 25, 2026
11 of 12 checks passed
@umair-ably umair-ably deleted the removeMCP branch February 25, 2026 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants