Removes Ably MCP and positions Ably CLI as Public Preview#138
Removes Ably MCP and positions Ably CLI as Public Preview#138umair-ably merged 1 commit intomainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
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.
WalkthroughThis 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
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
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. Comment |
e571f35 to
1e74e43
Compare
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 mcpcommand group, the MCP server implementation undersrc/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/sdkdependency, removing MCP-specific ESLint/coverage exclusions, and adjusting web/interactive-mode command restriction logic/tests now thatmcp*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
Documentation
Chores