Follow-up from the Phase 7 (PR #2) merge review.
Gap
Phase 7 spec (docs/roadmap.md) states as a goal: "can drill into details when one fails", and stopping criteria: "Selecting an entry expands its details" / "Panel keybindings: arrow nav, Enter to expand, Escape to close."
Currently mcp/handle-key: Enter on a requires-auth row dispatches connect; Enter on every other row (including failed) is a no-op. There is no expansion state, and stored detail fields (command, args, url, prompts, resources from MCPServerUpdatedParams) are never rendered. A failed server cannot be drilled into at all — the headline "drill into details" goal is unmet.
Scope
Acceptance
- Enter on a
failed/running row expands to show command/args/url/tool names.
- Enter again (or Escape) collapses.
- Regression test in
test/eca_cli/mcp_test.clj.
Follow-up from the Phase 7 (PR #2) merge review.
Gap
Phase 7 spec (
docs/roadmap.md) states as a goal: "can drill into details when one fails", and stopping criteria: "Selecting an entry expands its details" / "Panel keybindings: arrow nav, Enter to expand, Escape to close."Currently
mcp/handle-key: Enter on arequires-authrow dispatches connect; Enter on every other row (includingfailed) is a no-op. There is no expansion state, and stored detail fields (command,args,url,prompts,resourcesfromMCPServerUpdatedParams) are never rendered. A failed server cannot be drilled into at all — the headline "drill into details" goal is unmet.Scope
:mcppicker (which row is expanded).render-mcp-panel-linesrenders the expanded row's detail fields.requires-authrows (keep connect onrequires-auth, or reconcile the two — see /mcp panel: requires-auth hint should name the login flow #9).Acceptance
failed/runningrow expands to show command/args/url/tool names.test/eca_cli/mcp_test.clj.