Context
The MCP server (internal/mcp/server.go) has inline implementations of findDeadFunctions and findAffected that do naive graph traversal (same approach the CLI commands used before #23 and #25). Now that client.DeadCode() and client.Impact() exist, the MCP tools should use them.
Current state
dead_code MCP tool: inline graph edge counting (checks calls/contains_call edges)
blast_radius MCP tool: inline reverse BFS on imports edges
Proposed change
dead_code tool → call client.DeadCode(), return rich results (confidence, line, reason)
blast_radius tool → call client.Impact(), return risk scores, affected files, entry points
Dependencies
Notes
- VSA: changes are isolated to
internal/mcp/server.go
- The MCP tool schemas (
inputSchema) will need updating to expose new params (min_confidence, limit, diff)
Context
The MCP server (
internal/mcp/server.go) has inline implementations offindDeadFunctionsandfindAffectedthat do naive graph traversal (same approach the CLI commands used before #23 and #25). Now thatclient.DeadCode()andclient.Impact()exist, the MCP tools should use them.Current state
dead_codeMCP tool: inline graph edge counting (checkscalls/contains_calledges)blast_radiusMCP tool: inline reverse BFS onimportsedgesProposed change
dead_codetool → callclient.DeadCode(), return rich results (confidence, line, reason)blast_radiustool → callclient.Impact(), return risk scores, affected files, entry pointsDependencies
client.DeadCode()✅ mergedclient.Impact()(pending merge)Notes
internal/mcp/server.goinputSchema) will need updating to expose new params (min_confidence,limit,diff)