feat(mcp): consolidated 6 mode-driven tools alongside deprecated 34

Plan Phase 2 — collapse the MCP surface so agents see a navigable set
instead of 34 narrow tools.

New tools (all read-only, all delegate to existing handlers — surface
change only, no query-layer rewrite):

  graph_summary       overview | categories | capabilities | provenance
  find_in_graph       nodes | edges | text | fuzzy | by_file | by_endpoint
  inspect_node        neighbors | ego | evidence | source
  trace_relationships callers | consumers | producers | dependencies |
                      dependents | shortest_path
  analyze_impact      blast_radius | trace | cycles | circular_deps |
                      dead_code | dead_services | bottlenecks
  topology_view       summary | service | service_deps |
                      service_dependents | flow

run_cypher stays as the escape hatch (unchanged).
review_changes lands in Phase 3.

The 34 deprecated tools remain wired for one release for back-compat
with agents pinned to old names. Each consolidated handler delegates
to the deprecated tool's handler via a synthesized params object, so
behavior stays in lockstep — no logic forks.

Tests:
- TestRegisterConsolidated_AllSixToolsLand
- TestConsolidatedTool_UnknownModeRejected (all 6 reject bogus mode
  with INVALID_INPUT envelope)
- TestGraphSummary_DefaultModeIsOverview

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: aksOps

go/internal/cli/mcp.go

@@ -162,6 +162,10 @@ func registerAllTools(srv *mcp.Server, d *mcp.Deps) error {
 	if err := mcp.RegisterIntelligence(srv, d); err != nil {
 		return fmt.Errorf("register intelligence tools: %w", err)
 	}
+	// Plan §2 — consolidated tools alongside the deprecated 34.
+	if err := mcp.RegisterConsolidated(srv, d); err != nil {
+		return fmt.Errorf("register consolidated tools: %w", err)
+	}
 	for _, hook := range optionalRegisterHooks {
 		if hook == nil {
 			continue

go/internal/mcp/tools_consolidated_test.go

@@ -0,0 +1,73 @@
+package mcp
+
+import (
+	"context"
+	"encoding/json"
+	"strings"
+	"testing"
+)
+
+// TestRegisterConsolidated_AllSixToolsLand verifies the six new tool names
+// appear in the server registry after RegisterConsolidated runs.
+func TestRegisterConsolidated_AllSixToolsLand(t *testing.T) {
+	srv, err := NewServer(ServerOptions{Name: "test", Version: "0.0.0"})
+	if err != nil {
+		t.Fatal(err)
+	}
+	if err := RegisterConsolidated(srv, &Deps{}); err != nil {
+		t.Fatalf("RegisterConsolidated: %v", err)
+	}
+	names := srv.Registry().Names()
+	want := []string{
+		"graph_summary",
+		"find_in_graph",
+		"inspect_node",
+		"trace_relationships",
+		"analyze_impact",
+		"topology_view",
+	}
+	for _, w := range want {
+		found := false
+		for _, n := range names {
+			if n == w {
+				found = true
+				break
+			}
+		}
+		if !found {
+			t.Errorf("missing tool %q in registry; got %v", w, names)
+		}
+	}
+}
+
+// TestConsolidatedTool_UnknownModeRejected — each consolidated tool returns
+// a CodeInvalidInput envelope when the mode is unrecognized.
+func TestConsolidatedTool_UnknownModeRejected(t *testing.T) {
+	d := &Deps{}
+	for _, build := range []func(*Deps) Tool{
+		toolGraphSummary, toolFindInGraph, toolInspectNode,
+		toolTraceRelationships, toolAnalyzeImpact, toolTopologyView,
+	} {
+		tool := build(d)
+		params, _ := json.Marshal(map[string]string{"mode": "bogus"})
+		got, _ := tool.Handler(context.Background(), params)
+		gotJSON, _ := json.Marshal(got)
+		if !strings.Contains(string(gotJSON), "INVALID_INPUT") {
+			t.Errorf("%s with bogus mode: expected INVALID_INPUT envelope, got %s", tool.Name, gotJSON)
+		}
+	}
+}
+
+// TestGraphSummary_DefaultModeIsOverview — mode unset routes to get_stats.
+func TestGraphSummary_DefaultModeIsOverview(t *testing.T) {
+	d := &Deps{}
+	tool := toolGraphSummary(d)
+	got, _ := tool.Handler(context.Background(), json.RawMessage(`{}`))
+	gotJSON, _ := json.Marshal(got)
+	// With no Stats wired, overview falls into the "stats service not wired"
+	// envelope. Any other path would have been routed to capabilities or
+	// provenance with a different envelope wording.
+	if !strings.Contains(string(gotJSON), "stats service not wired") {
+		t.Errorf("default mode did not route to overview: %s", gotJSON)
+	}
+}