Skip to content
This repository was archived by the owner on Oct 28, 2025. It is now read-only.

chore: deprecate tools with warning#198

Merged
brandonspark merged 3 commits intomainfrom
brandon/deprecate
Sep 29, 2025
Merged

chore: deprecate tools with warning#198
brandonspark merged 3 commits intomainfrom
brandon/deprecate

Conversation

@brandonspark
Copy link
Collaborator

@brandonspark brandonspark commented Sep 29, 2025

This PR makes it so users of the MCP will know that we are changing the modality, and gives advice on how to transition to the new semgrep mcp server.

Test plan:

image

@brandonspark brandonspark requested a review from nmote September 29, 2025 20:04
Copy link
Contributor

@nmote nmote left a comment

Choose a reason for hiding this comment

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

Nice! Have you tested this to see how agents react?

@brandonspark
Copy link
Collaborator Author

Nice! Have you tested this to see how agents react?

i've witnessed that the agent typically will call deprecation notice and notice that the server is deprecated:
image

i think i'd like this to be a bit more explicit. i think i'll tell the agent to explicitly signal to the user that they should switch their mcp.json

@brandonspark brandonspark merged commit 30fc3d8 into main Sep 29, 2025
9 checks passed
@brandonspark brandonspark deleted the brandon/deprecate branch September 29, 2025 20:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants