Skip to content

docs(roadmap): add #417 — mcp show nonexistent returns found:false + status:ok + exit 0; contradictory#2961

Closed
code-yeongyu wants to merge 1 commit into
mainfrom
docs/roadmap-417-mcp-show-found-false-status-ok
Closed

docs(roadmap): add #417 — mcp show nonexistent returns found:false + status:ok + exit 0; contradictory#2961
code-yeongyu wants to merge 1 commit into
mainfrom
docs/roadmap-417-mcp-show-found-false-status-ok

Conversation

@code-yeongyu
Copy link
Copy Markdown
Collaborator

Pinpoint #417

mcp show <nonexistent> --output-format json returns found:false + status:"ok" + exit 0.

Evidence:

{"action":"show","config_load_error":null,"found":false,"kind":"mcp","message":"server `nonexistent` is not configured","server_name":"nonexistent","status":"ok","working_directory":"..."}

Exit code: 0

Automation checking status == "ok" will false-positive on every missing server query. Distinct from #102 (liveness probe gap) — this is the simpler semantic contradiction when server is not configured at all.

ROADMAP-only PR. No source/test changes.

@Yeachan-Heo
Copy link
Copy Markdown
Contributor

Review verdict: REQUEST_CHANGES

I inspected the exact diff against origin/main: docs-only ROADMAP.md | 2 ++; CI is green and merge state is CLEAN, so runtime risk is low.

Blocker: the new #417 materially duplicates existing ROADMAP #347. #347 already covers mcp show <missing-server> --output-format json returning found:false with status:"ok", the automation ambiguity, and the required typed not_found/error status plus exit/status contract alignment.

#417 adds sharper explicit exit-0 evidence, but that should be folded into #347 rather than creating a new numbered item.

Requested change: replace #417 with an amendment/fresh-run note on #347 adding the explicit exit-code evidence and any desired acceptance wording around nonzero/typed not-found behavior.

Summary:


[repo owner's gaebal-gajae (clawdbot) 🦞]

@code-yeongyu
Copy link
Copy Markdown
Collaborator Author

Closing — duplicate of ROADMAP #347 which already documents mcp show <missing> --output-format json returning found:false + status:"ok". Thanks gaebal-gajae for catching in review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants