Skip to content

Conversation

@jonathannorris
Copy link
Member

@jonathannorris jonathannorris commented Oct 7, 2025

Removes commented-out tool registrations and unused handler functions from the MCP codebase.

Changes

  • Deleted environmentTools.ts (all tools were disabled)
  • Deleted customPropertiesTools.ts (unused, not imported anywhere)
  • Removed disabled tool registrations from projectTools.ts and selfTargetingTools.ts
  • Cleaned up unused handlers and imports
  • Fixed linter warnings (anyunknown)

Copilot AI review requested due to automatic review settings October 7, 2025 19:31
@jonathannorris jonathannorris requested a review from a team as a code owner October 7, 2025 19:31
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 7, 2025

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
devcycle-mcp-server d32a0bb Oct 07 2025, 07:33 PM

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes disabled MCP tool registrations and unused handler functions from the MCP codebase to improve maintainability.

  • Deleted entire files containing only disabled/unused tools
  • Removed commented-out tool registrations from active files
  • Fixed linter warnings by changing any to unknown types

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/mcp/tools/selfTargetingTools.ts Removed unused import, deleted disabled handler function, and updated type annotations
src/mcp/tools/projectTools.ts Removed unused imports and handler functions, cleaned up disabled tool registrations
src/mcp/tools/index.ts Removed imports and registration calls for deleted tool files
src/mcp/tools/environmentTools.ts Completely deleted file containing only disabled tools
src/mcp/tools/customPropertiesTools.ts Completely deleted file containing unused tools

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jonathannorris jonathannorris merged commit 690c379 into main Oct 8, 2025
8 checks passed
@jonathannorris jonathannorris deleted the chore-cleanup-disabled-mcp-tools branch October 8, 2025 18:28
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.

3 participants