Skip to content

docs(roadmap): add #351 — plugins disable json error is stderr-only#2892

Merged
Yeachan-Heo merged 1 commit intomainfrom
docs/roadmap-351-plugins-disable-stderr-only
Apr 29, 2026
Merged

docs(roadmap): add #351 — plugins disable json error is stderr-only#2892
Yeachan-Heo merged 1 commit intomainfrom
docs/roadmap-351-plugins-disable-stderr-only

Conversation

@Yeachan-Heo
Copy link
Copy Markdown
Contributor

@Yeachan-Heo Yeachan-Heo commented Apr 29, 2026

Summary

  • Add exactly one ROADMAP.md item, 合影 #351, from direct claw-code dogfood.
  • Pinpoints that plugins disable does-not-exist --output-format json returns exit 1 with stdout empty and the JSON error envelope on stderr only, using generic kind:"unknown" / type:"error" instead of a plugin-specific not-found contract on a predictable stream.
  • Records rebuilt-binary provenance (git_sha 0f9e891), a prompt plugins list sanity check, and replaces invalid hang PR docs(roadmap): add #351 — plugins disable missing target hangs #2891.

Verification

  • cargo run --manifest-path rust/Cargo.toml --bin claw -- version --output-format json
  • repeated timeout 8 ./rust/target/debug/claw plugins disable does-not-exist --output-format json
  • timeout 8 ./rust/target/debug/claw plugins list --output-format json
  • git diff --check
  • scripts/fmt.sh --check


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

Constraint: ROADMAP-only dogfood follow-up for 22:30 nudge on rebuilt claw git_sha 0f9e891

Rejected: implementation change to plugin lifecycle mutation; request was one concrete follow-up if no backlog item
Confidence: high after repeated bounded samples plus prompt list sanity check
Scope-risk: narrow
Directive: Replaces invalid hang PR #2891 with verified stderr-only JSON-mode gap
Tested: cargo run --manifest-path rust/Cargo.toml --bin claw -- version --output-format json; repeated timeout 8 ./rust/target/debug/claw plugins disable does-not-exist --output-format json; timeout 8 ./rust/target/debug/claw plugins list --output-format json; git diff --check; scripts/fmt.sh --check
Not-tested: runtime behavior change, because this commit only documents the gap
@Yeachan-Heo Yeachan-Heo force-pushed the docs/roadmap-351-plugins-disable-stderr-only branch from e652018 to 3b03375 Compare April 29, 2026 22:32
@Yeachan-Heo Yeachan-Heo changed the title docs(roadmap): add #351 — plugins disable json is stderr-only on missing target docs(roadmap): add #351 — plugins disable json error is stderr-only Apr 29, 2026
@Yeachan-Heo
Copy link
Copy Markdown
Contributor Author

OMX review session claw-code-pr-2892-review inspected the exact PR diff and CI state.

Verdict: APPROVE / merge-ready.

Scope verified:

  • ROADMAP.md only
  • 1 insertion, 0 deletions
  • adds 合影 #351 plugins disable <missing-plugin> --output-format json routing the JSON error envelope to stderr only with generic kind:"unknown" / type:"error"
  • no implementation/source/config/test files changed

Verification:

  • gh pr diff 2892 --repo ultraworkers/claw-code --name-only
  • gh pr diff 2892 --repo ultraworkers/claw-code --patch --color never
  • gh pr checks 2892 --repo ultraworkers/claw-code
  • CI 4/4 green: docs source-of-truth, cargo fmt, cargo clippy --workspace, cargo test --workspace

Risk: very low; documentation-only roadmap addition. No blockers found.


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

@Yeachan-Heo Yeachan-Heo merged commit 5eb1d7d into main Apr 29, 2026
4 checks passed
@Yeachan-Heo Yeachan-Heo deleted the docs/roadmap-351-plugins-disable-stderr-only branch April 29, 2026 23:00
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.

1 participant