docs(roadmap): add #419 — mcp unknown subcommand returns action:help exit 0 not error#2963
docs(roadmap): add #419 — mcp unknown subcommand returns action:help exit 0 not error#2963code-yeongyu wants to merge 1 commit intomainfrom
Conversation
…exit 0 instead of error
|
Review verdict: APPROVE Inspected exact diff against The #419 pinpoint is concrete and actionable: it names Non-duplicative: distinct from #108 (top-level unknown subcommands falling into prompt dispatch), #346 ( Merge risk: very low. Blockers: none. Minor note only: the entry says dogfooded 2026-05-01 while this review ran 2026-04-30 UTC, likely local-time provenance and not a blocker. — |
Pinpoint #419
mcp <unknown-subcommand> --output-format jsonreturnsaction:"help"+ exit 0 instead of an error.Evidence (both
addandremove):{"action":"help","kind":"mcp","unexpected":"add","usage":{"direct_cli":"claw mcp [list|show <server>|help]","slash_command":"/mcp [list|show <server>|help]","sources":[...]}}Exit code: 0
Distinct from #108 (unknown top-level subcommands fall through to LLM prompt). This is MCP-specific: recognized but unimplemented subcommands return help with exit 0, making them look like successful no-ops.
ROADMAP-only PR. No source/test changes.