Skip to content

Commit a58de38

Browse files
committed
add README for current state of src mcp experiment
1 parent 7393a82 commit a58de38

File tree

1 file changed

+27
-0
lines changed

1 file changed

+27
-0
lines changed

internal/mcp/README.md

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,27 @@
1+
# `src mcp` Experiment
2+
3+
> **Status:** Development paused. The Sourcegraph MCP with DCR is preferred.
4+
5+
## Motivation
6+
7+
We added functionality to the `src` CLI to dynamically expose Sourcegraph MCP tools as CLI commands. The aim was that exposing these commands as skills to agents would help them interact with the Sourcegraph instance more easily and use that knowledge to perform their tasks locally.
8+
9+
## Findings
10+
11+
Once `src mcp` was exposed to agents with skills, the results were hit and miss.
12+
13+
* Some agents like Claude became overzealous in reaching out to Sourcegraph and would use `src mcp` over grepping the source code locally.
14+
* In Amp, you really had to explicitly invoke `src mcp` — it wouldn't organically use it on its own. Sometimes it would, but that was the exception. The only time it would really pick `src mcp` as the first choice was if your query was extremely obvious that it should use Sourcegraph e.g. "Find all changes made by William across the Sourcegraph organization repositories".
15+
* By having the Sourcegraph MCP *and* the `src mcp` skill, the agent picked the MCP all the time and under the right circumstances.
16+
* For example, when asked "When and why was gitserver refactored to use grpc?" with the MCP installed, the agent immediately reached for `sg_deepsearch`.
17+
* With only the `src mcp` skill, it never used deepsearch unless explicitly told — defaulting to keyword search and commit search instead.
18+
19+
Through my own use, there was no compelling reason to have `src mcp` over the normal Sourcegraph MCP, especially now that the Sourcegraph MCP supports DCR (dynamic client registration).
20+
21+
## Decision
22+
23+
We're stopping additional development on `src mcp` as the usefulness of the experiment is just not there yet. We might pick it up in the future but for now further investment is not worth it.
24+
25+
The command isn't advertised in help anywhere so there is little to no risk in stopping development. The only way to know the command exists is to read the code.
26+
27+
We will keep the code as it is dynamic and adapts to what the Sourcegraph instance exposes so maintenance should be low.

0 commit comments

Comments
 (0)