Skip to content

docs(roadmap): add #420 — plugins help json returns mutation shape, not help envelope like mcp/agents/skills#2964

Open
code-yeongyu wants to merge 1 commit intomainfrom
docs/roadmap-420-plugins-help-mutation-shape-vs-help-envelope
Open

docs(roadmap): add #420 — plugins help json returns mutation shape, not help envelope like mcp/agents/skills#2964
code-yeongyu wants to merge 1 commit intomainfrom
docs/roadmap-420-plugins-help-mutation-shape-vs-help-envelope

Conversation

@code-yeongyu
Copy link
Copy Markdown
Collaborator

Pinpoint #420

plugins help --output-format json returns the mutation response shape instead of the help envelope used by all sibling subgroups.

plugins help (wrong — mutation shape):

{"action":"help","kind":"plugin","message":"Unknown /plugins action 'help'. Use list...","reload_runtime":false,"target":null}

mcp/agents/skills help (correct — help envelope):

{"action":"help","kind":"mcp","unexpected":null,"usage":{"direct_cli":"claw mcp [list|show <server>|help]","slash_command":"/mcp [...]","sources":[...]}}

Automation checking usage.direct_cli fails with KeyError for plugins while succeeding for all sibling subgroups.

ROADMAP-only PR. No source/test changes.

@Yeachan-Heo
Copy link
Copy Markdown
Contributor

Review verdict: APPROVE

Inspected exact diff against origin/main: docs-only ROADMAP.md change (+2), no source/test/schema/runtime contract files touched. CI is green and merge state is CLEAN.

The #420 pinpoint is concrete and actionable: it names plugins help --output-format json, shows the mutation-shaped payload (message, reload_runtime, target), compares it to sibling help envelopes (mcp, agents, skills with usage.direct_cli/slash_command/sources), and states the automation failure mode.

Non-duplicative:

Merge risk: low. Blockers: none. Minor note only: the added text says dogfooded 2026-05-01 while this review ran 2026-04-30 UTC; likely KST/local-time provenance and not a blocker.


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

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