Summary

Manufact's readiness review flagged annotations.title missing across the external MCP surface. An earlier round of EXTERNAL_TOOL_METADATA assumed only the top-level title field mattered , the code even had a comment saying so ("annotations.title is a legacy fallback the SDK also accepts but reviewers don't read"). This new finding shows that assumption was wrong.

Fix

Rather than duplicating the title string across EXTERNAL_TOOL_METADATA's 25 entries, fixed it at each registration site:

• server.ts's registerFamilyToolOnServer now derives annotations.title from the same metadata.title at the one shared call site, so the two can never drift for any family tool.
• The four standalone registrations (load-skill-tool.ts, cedar-staff-debug-tool.ts, load-brain-skills-tool.ts, deprecated-record-shims.ts) each set annotations.title to the same literal string as their existing top-level title.

Tests

Added a regression check in external-mcp.e2e.test.ts asserting every tool in a real tools/list response carries a non-empty top-level title AND a matching annotations.title , covers the family loop, load-skill, cedar-staff-debug, and both deprecated record shims in one pass, plus a second targeted check for load-brain-skills (registered by its own standalone function, not covered by the first session's granted scopes).

Verification

• tsc -b clean
• pnpm deps:check clean (no dependency violations, 1845 modules cruised)
• Scoped vitest across external-mcp.e2e.test.ts, tool-hints-match-behavior.test.ts, server.test.ts: 67/67 passing

🤖 Generated with Claude Code

Confidence Score: 5/5

The PR appears safe to merge, with title metadata consistently propagated through every changed registration path.

Summary

The PR consistently exposes MCP tool titles in both supported metadata locations.

• Family tools now derive annotations.title from the same metadata value as their top-level title.
• Standalone tools now provide matching annotation titles.
• End-to-end coverage checks the shared family registration path and each standalone registration path.

_{Reviews (1) · Last reviewed commit: "fix(mcp): add annotations.title alongsid..."}