Skip to content

chore: delete MCP package#1117

Merged
brendan-kellam merged 2 commits intomainfrom
bkellam/deprecate-mcp-package
Apr 15, 2026
Merged

chore: delete MCP package#1117
brendan-kellam merged 2 commits intomainfrom
bkellam/deprecate-mcp-package

Conversation

@brendan-kellam
Copy link
Copy Markdown
Contributor

@brendan-kellam brendan-kellam commented Apr 15, 2026 •
edited by coderabbitai bot
Loading

As @sourcebot/mcp is deprecated in favor of the streamable http transport, we can delete the package from the repo.

Summary by CodeRabbit

  • Revert
    • Removed the entire MCP package, including its server implementation, configuration files, and supporting documentation.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 15, 2026 •
edited
Loading

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ff0c5b23-61ae-475c-8589-55c7728b478e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Walkthrough

This PR deletes the entire packages/mcp directory, removing the standalone MCP (Model Context Protocol) server package including its source code, configuration files, and documentation. The package previously contained server implementation, client utilities, type definitions, schemas, and supporting configuration.

Changes

Cohort / File(s) Summary
Configuration & Build Setup
packages/mcp/package.json, packages/mcp/tsconfig.json, packages/mcp/Dockerfile, packages/mcp/smithery.yaml		 Removed package manifest, TypeScript configuration, Docker build setup, and Smithery project configuration for the MCP server.
Version Control & NPM Ignore Rules
packages/mcp/.gitignore, packages/mcp/.npmignore		 Removed Git and npm ignore patterns that excluded dist/ and node_modules/ from version control and npm packages.
Documentation
packages/mcp/README.md, packages/mcp/CHANGELOG.md		 Removed user-facing documentation covering setup, environment configuration, MCP tools with parameter descriptions, supported code hosts, and changelog history across 18 releases.
Core Server & Type Definitions
packages/mcp/src/index.ts, packages/mcp/src/schemas.ts, packages/mcp/src/types.ts		 Removed MCP server entrypoint with tool registrations (search_code, list_commits, list_repos, read_file, list_tree, list_language_models, ask_codebase) and comprehensive Zod schema/type definitions for request/response validation.
Client & Utilities
packages/mcp/src/client.ts, packages/mcp/src/env.ts, packages/mcp/src/utils.ts		 Removed HTTP client implementation for Sourcebot API calls, environment configuration parsing, and utility functions for error handling, path normalization, and tree traversal.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

  • PR #976: Implements equivalent MCP server functionality and utilities under packages/web/src/features/mcp, suggesting a migration of the MCP implementation from this standalone package to the web application.
  • PR #292: Introduced the entire packages/mcp package that is now being removed in this PR.
  • PR #429: Modified packages/mcp/src/client.ts and packages/mcp/src/index.ts which are now being deleted.

Suggested labels

sourcebot-team

Suggested reviewers

  • msukkari
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: delete MCP package' accurately and concisely describes the primary change: removal of the entire MCP package from the repository.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bkellam/deprecate-mcp-package

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.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 15, 2026 •
edited
Loading

License Audit

❌ Status: FAIL

Metric Count
Total packages 2150
Resolved (non-standard) 8
Unresolved 3
Strong copyleft 0
Weak copyleft 39

Fail Reasons

  • 3 packages have unresolvable licenses: @react-grab/cli (0.1.23), @react-grab/cli (0.1.29), @react-grab/mcp (0.1.29)

Unresolved Packages

Package Version License Reason
@react-grab/cli 0.1.23 UNKNOWN No repository or homepage URL in package metadata; npm registry entry for this version contains no license field; GitHub organization react-grab has no public repositories
@react-grab/cli 0.1.29 UNKNOWN No repository or homepage URL in package metadata; npm registry entry for this version contains no license field; GitHub organization react-grab has no public repositories
@react-grab/mcp 0.1.29 UNKNOWN No repository or homepage URL in package metadata; npm registry entry for this version contains no license field; GitHub organization react-grab has no public repositories

Weak Copyleft Packages (informational)

Package Version License
@img/sharp-libvips-darwin-arm64 1.0.4 LGPL-3.0-or-later
@img/sharp-libvips-darwin-arm64 1.2.4 LGPL-3.0-or-later
@img/sharp-libvips-darwin-x64 1.0.4 LGPL-3.0-or-later
@img/sharp-libvips-darwin-x64 1.2.4 LGPL-3.0-or-later
@img/sharp-libvips-linux-arm 1.0.5 LGPL-3.0-or-later
@img/sharp-libvips-linux-arm 1.2.4 LGPL-3.0-or-later
@img/sharp-libvips-linux-arm64 1.0.4 LGPL-3.0-or-later
@img/sharp-libvips-linux-arm64 1.2.4 LGPL-3.0-or-later
@img/sharp-libvips-linux-ppc64 1.2.4 LGPL-3.0-or-later
@img/sharp-libvips-linux-riscv64 1.2.4 LGPL-3.0-or-later
@img/sharp-libvips-linux-s390x 1.0.4 LGPL-3.0-or-later
@img/sharp-libvips-linux-s390x 1.2.4 LGPL-3.0-or-later
@img/sharp-libvips-linux-x64 1.0.4 LGPL-3.0-or-later
@img/sharp-libvips-linux-x64 1.2.4 LGPL-3.0-or-later
@img/sharp-libvips-linuxmusl-arm64 1.0.4 LGPL-3.0-or-later
@img/sharp-libvips-linuxmusl-arm64 1.2.4 LGPL-3.0-or-later
@img/sharp-libvips-linuxmusl-x64 1.0.4 LGPL-3.0-or-later
@img/sharp-libvips-linuxmusl-x64 1.2.4 LGPL-3.0-or-later
@img/sharp-wasm32 0.33.5 Apache-2.0 AND LGPL-3.0-or-later AND MIT
@img/sharp-wasm32 0.34.5 Apache-2.0 AND LGPL-3.0-or-later AND MIT
@img/sharp-win32-arm64 0.34.5 Apache-2.0 AND LGPL-3.0-or-later
@img/sharp-win32-ia32 0.33.5 Apache-2.0 AND LGPL-3.0-or-later
@img/sharp-win32-ia32 0.34.5 Apache-2.0 AND LGPL-3.0-or-later
@img/sharp-win32-x64 0.33.5 Apache-2.0 AND LGPL-3.0-or-later
@img/sharp-win32-x64 0.34.5 Apache-2.0 AND LGPL-3.0-or-later
axe-core 4.10.3 MPL-2.0
dompurify 3.3.1 (MPL-2.0 OR Apache-2.0)
lightningcss 1.32.0 MPL-2.0
lightningcss-android-arm64 1.32.0 MPL-2.0
lightningcss-darwin-arm64 1.32.0 MPL-2.0
lightningcss-darwin-x64 1.32.0 MPL-2.0
lightningcss-freebsd-x64 1.32.0 MPL-2.0
lightningcss-linux-arm-gnueabihf 1.32.0 MPL-2.0
lightningcss-linux-arm64-gnu 1.32.0 MPL-2.0
lightningcss-linux-arm64-musl 1.32.0 MPL-2.0
lightningcss-linux-x64-gnu 1.32.0 MPL-2.0
lightningcss-linux-x64-musl 1.32.0 MPL-2.0
lightningcss-win32-arm64-msvc 1.32.0 MPL-2.0
lightningcss-win32-x64-msvc 1.32.0 MPL-2.0
Resolved Packages (8)
Package Version Original Resolved Source
codemirror-lang-elixir 4.0.0 UNKNOWN Apache-2.0 GitHub repo: https://github.com/livebook-dev/codemirror-lang-elixir/blob/main/LICENSE
element-source 0.0.3 UNKNOWN MIT GitHub repo: https://github.com/aidenybai/element-source/blob/main/LICENSE
lezer-elixir 1.1.2 UNKNOWN Apache-2.0 GitHub repo: https://github.com/livebook-dev/lezer-elixir/blob/main/LICENSE
map-stream 0.1.0 UNKNOWN MIT GitHub repo: https://github.com/dominictarr/map-stream/blob/master/LICENCE
memorystream 0.3.1 UNKNOWN MIT npm registry: license field is [{"type":"MIT","url":"http://github.com/JSBizon/node-memorystream/raw/master/LICENSE"}] — extracted type field
pause-stream 0.0.11 ["MIT","Apache2"] MIT AND Apache-2.0 extracted from license array ["MIT","Apache2"] in oss-licenses.json
posthog-js 1.345.5 SEE LICENSE IN LICENSE Apache-2.0 GitHub repo: https://github.com/PostHog/posthog-js/blob/main/LICENSE (primary license is Apache-2.0; some bundled third-party files are MIT)
valid-url 1.0.9 UNKNOWN MIT GitHub repo: https://github.com/ogt/valid-url/blob/master/LICENSE

@brendan-kellam brendan-kellam merged commit c7202bf into main Apr 15, 2026
7 of 8 checks passed
@brendan-kellam brendan-kellam deleted the bkellam/deprecate-mcp-package branch April 15, 2026 00:20
@github-actions github-actions bot mentioned this pull request Apr 15, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@brendan-kellam