Fix plannotator availability check (--version → --help)#12
Merged
Conversation
Replace `plannotator --version` with `plannotator --help` in research, plan-tasks, and implement skills. The --version flag exits 1 because plannotator treats unrecognized invocations as hook handler calls that expect JSON on stdin. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
bgerstle
requested review from
Copilot and
ericjohnolson
and removed request for
ericjohnolson
There was a problem hiding this comment.
Pull request overview
Updates praxis skills to use a reliable plannotator availability check and bumps plugin/marketplace versions accordingly, avoiding the non-zero exit behavior triggered by plannotator --version in this environment.
Changes:
- Replace
plannotator --versionwithplannotator --helpin the research, plan-tasks, and implement skills. - Bump praxis plugin version to
2.1.1. - Bump marketplace metadata and praxis plugin entry version to
2.1.1.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| plugins/praxis/skills/research/SKILL.md | Switches the availability check command to plannotator --help in the research flow. |
| plugins/praxis/skills/plan-tasks/SKILL.md | Switches the availability check command to plannotator --help in the planning review step. |
| plugins/praxis/skills/implement/SKILL.md | Switches the availability check command to plannotator --help in the final verification/review step. |
| plugins/praxis/.claude-plugin/plugin.json | Bumps praxis plugin version from 2.1.0 to 2.1.1. |
| .claude-plugin/marketplace.json | Bumps marketplace metadata and praxis plugin entry versions from 2.1.0 to 2.1.1. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
plannotator --versionwithplannotator --helpin three praxis skills (research, plan-tasks, implement)Root cause:
plannotator --versionexits 1 because plannotator treats invocations without a recognized subcommand as a hook handler expecting JSON on stdin, producing:Failed to parse hook event from stdin: JSON Parse error: Unexpected EOF. The--helpflag exits 0 and correctly signals availability.Test plan
plannotator --helplocally to confirm exit 0plannotator --versionexits non-zero to validate the fix is necessarybash tests/local/validate-skills.shto confirm structure checks pass🤖 Generated with Claude Code