feat(agents): dogfood dev-agent MCP tools in Claude agents

[prosdev]

Summary

• Gives 8 of our 9 Claude agents access to dev-agent's own MCP tools (search, refs, map, patterns, status)
• Each agent gets the tools most relevant to its workflow — not a blanket "give everything to everyone"
• Adds an Agent → MCP Tool Matrix to CLAUDE.md for quick reference

Agent → MCP Tool Matrix

┌───────────────────┬────────────┬──────────┬─────────┬──────────────┬────────────┬────────────┐
│       Agent       │ dev_search │ dev_refs │ dev_map │ dev_patterns │ dev_status │ dev_health │
├───────────────────┼────────────┼──────────┼─────────┼──────────────┼────────────┼────────────┤
│ bug-investigator  │     ★      │    ★     │    ●    │              │            │            │
│ quick-scout       │     ★      │    ★     │    ●    │              │            │            │
│ research-planner  │     ★      │    ●     │    ★    │      ★       │            │            │
│ logic-reviewer    │     ●      │    ★     │         │      ●       │            │            │
│ security-reviewer │     ★      │    ●     │         │      ★       │            │            │
│ quality-reviewer  │     ●      │          │         │      ★       │            │            │
│ plan-reviewer     │            │    ★     │    ★    │      ●       │            │            │
│ pr-composer       │            │          │    ●    │              │     ●      │            │
│ code-reviewer     │            │          │         │              │            │            │
└───────────────────┴────────────┴──────────┴─────────┴──────────────┴────────────┴────────────┘

★ = high impact, ● = useful

Changes

Agents (.claude/agents/)

• quick-scout — dev_search, dev_refs, dev_map. Rewrote workflow: semantic search first, ref tracing for usages, map for structure
• bug-investigator — dev_search, dev_refs, dev_map. New MCP Tools section, integrated into data flow tracing and root cause phases
• research-planner — dev_search, dev_refs, dev_map, dev_patterns. New MCP Tools section, rewrote "Map the territory" workflow step
• logic-reviewer — dev_search, dev_refs, dev_patterns. Added guidance to Cross-Package Data Flow section
• security-reviewer — dev_search, dev_refs, dev_patterns. Added MCP usage guidance before the security checklist
• quality-reviewer — dev_search, dev_patterns. Added guidance to Readability & Simplification section
• plan-reviewer — dev_refs, dev_map, dev_patterns. Integrated into Pass 1 (Engineer Review) verification steps
• pr-composer — dev_map, dev_status. Added to diff analysis step
• code-reviewer — Unchanged (orchestrator, delegates to sub-reviewers)

CLAUDE.md

• Added Agent → MCP Tool Matrix under the Agents section

Test plan

• Run quick-scout on a conceptual query — verify it uses dev_search instead of just Grep
• Run bug-investigator on a known issue — verify it uses dev_refs for tracing
• Run code-reviewer on a branch diff — verify sub-reviewers pick up MCP tools
• Verify agents still work when MCP server is not running (graceful fallback)

Generated with Claude Code