Summary

Manufact's readiness review flagged the same issue on these two no-op shims twice in a row now (most recently: "Add a type to this parameter: telemetry"). Instead of continuing to patch cosmetic schema findings on tools that exist purely for backward compat, checked whether anything actually still needs them.

Evidence

Queried Axiom (cedar-prod-logs) for external-mcp.tool_call events with toolId in (record-message, record-tool-call) over the trailing 90 days: zero calls, from either tool, from any caller. The public cedar-plugin repo's hooks that used to call them were already stripped when the intent-capture mechanism replaced them (2026-07-21), and no external MCP directory listing existed yet for any client to have a stale cached tools/list referencing them. There's no live caller left to protect.

Change

• Deleted deprecated-record-shims.ts
• Removed its import/registration call in server.ts
• Removed the now-stale toolNames/schema assertions in external-mcp.e2e.test.ts
• Added a dated note to docs/design/mcp-observability-and-plugin.md's existing resolution entry for these tools, recording why and when they were finally removed

Built on top of #2837 (fix/annotations-title), which merged to staging while this PR was in progress , retargeted at staging directly since that branch no longer exists. This diff contains only the new commit; #2837's changes are already on staging.

Verification

• tsc -b clean
• pnpm deps:check clean (no dependency violations, 1844 modules cruised)
• Scoped vitest across 10 affected/adjacent test files: 312/312 passing

🤖 Generated with Claude Code

Confidence Score: 4/5

The code change appears safe to merge, with a non-blocking documentation inconsistency that should be corrected.

Findings

1.  Impersonation Documentation Is Stale ▶

### Issue 1
docs/design/mcp-observability-and-plugin.md:357-361
This note says both deprecated tools are gone, but `docs/design/mcp-impersonation.md` still describes `record-message` and `record-tool-call` as active no-ops in impersonation mode. Update that section to describe the current `ephemeral` persistence behavior without relying on tools that no longer exist, so future maintainers are not misled.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Summary

This PR removes the deprecated record-message and record-tool-call compatibility shims from the external MCP server and updates discovery tests and observability documentation accordingly.

• Deletes the no-op shim implementation and registration.
• Updates the exact external MCP tool-list expectation.
• Records the production-usage rationale for removing the compatibility layer.
• Leaves one related impersonation design statement stale.

_{Reviews (1) · Last reviewed commit: "fix(mcp): remove deprecated record-messa..."}