RT-171/172/183: mcp tools heal-pair repo lookup, daemon error explain

[m4ttheweric]

• RT-171: resolveRepoIdentity shares reverseLookupByName (lib/repo-name-lookup.ts) with the CLI's tryResolveRepoArg for heal-pair collapse, instead of a name-search duplicated in both places.
• lib/explain-error.ts: shared daemon-error explainer used by the mcp tools layer.
• e2e: round-trips gate_ask against the test daemon.
• skills/rt-chat/SKILL.md: fixes CLI-first residues (verb-table row, viewer note) per the SKILLS-71 follow-up.

🤖 Generated with Claude Code

Summary by CodeRabbit

• New Features

  • MCP tools now provide clearer explanations for recognized worktree and repository errors, including recovery guidance.
  • Repository lookups support exact identities and report ambiguous or unknown repositories more clearly.
  • Missing gates now return an explicit error.
  • MCP workflows can submit gate requests and verify their open status.

• Documentation

  • Updated chat tool guidance to explain returned message IDs and how to construct message links. CLI output continues to include the link.

[coderabbitai]

Warning

Review limit reached

Next included review available in 32 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available. Your 80 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 3de9373f-29f2-44e5-b80c-54e3a40fcfbb

📥 Commits

Reviewing files that changed from the base of the PR and between 3cdd96f86735eb965acf13f9fcba1f52ab0c9c4f and 63ec360.

📒 Files selected for processing (6)

• commands/worktree.ts
• e2e/tests/mcp-serve.test.ts
• lib/explain-error.ts
• lib/mcp/__tests__/tools.test.ts
• lib/mcp/tools.ts
• skills/rt-chat/SKILL.md

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: cb242e61-4c89-4e90-a1da-b19f03784d91

📥 Commits

Reviewing files that changed from the base of the PR and between facced7 and 3cdd96f86735eb965acf13f9fcba1f52ab0c9c4f.

📒 Files selected for processing (6)

• commands/worktree.ts
• e2e/tests/mcp-serve.test.ts
• lib/explain-error.ts
• lib/mcp/__tests__/tools.test.ts
• lib/mcp/tools.ts
• skills/rt-chat/SKILL.md

Included review availability: 0 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 2 reviews per hour.

---

📝 Walkthrough

Walkthrough

The change centralizes worktree error explanation, updates MCP repository and gate handling, expands gate end-to-end coverage, and documents chat_post message-link behavior.

Changes

MCP handling and gate coverage

Layer / File(s)	Summary
Shared error formatting lib/explain-error.ts, commands/worktree.ts, lib/mcp/tools.ts	explainError now lives in a shared module and remains available from commands/worktree.ts. MCP responses format recognized daemon errors through the shared mapper.
Repository and gate handling lib/mcp/tools.ts, lib/mcp/__tests__/tools.test.ts, e2e/tests/mcp-serve.test.ts	Repository identity resolution preserves exact identities and detects ambiguity. Missing gates return no gate <id>. Tests cover repository deduplication, unknown repositories, gate_ask, and gate_list.

Chat message-link documentation

Layer / File(s)	Summary
chat_post return and link guidance skills/rt-chat/SKILL.md	The documentation states that chat_post returns a message ID and explains how MCP callers construct the message URL. CLI link output remains documented.

Priority: ⬇️ Low

Estimated code review effort: 3 (Moderate) | ~25 minutes

Change: Bug fix

Sequence Diagram(s)

sequenceDiagram
  participant MCPServeE2E
  participant MCPServer
  participant Daemon
  MCPServeE2E->>MCPServer: gate_ask for test:mcp-e2e
  MCPServer->>Daemon: create gate
  Daemon-->>MCPServer: gate ID and subject
  MCPServeE2E->>MCPServer: gate_list with open true
  MCPServer->>Daemon: list open gates
  Daemon-->>MCPServer: created gate
  MCPServer-->>MCPServeE2E: gate appears in list

Loading

Merge Risk: ⚪ Minimal · up to 3cdd9

The updated MCP and documentation paths have no identified actionable merge risk.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 42.86% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 7 functions across 5 files. (1 skipped: 1…	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately identifies the main changes: MCP tool repository lookup and shared daemon error explanations. It is concise and specific enough for the changeset.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

Docstring coverage is 42.86% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 7 functions across 5 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch mcp-tools

---

_{Comment @coderabbitai help to get the list of available commands.}