Content
75%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A highly actionable, well-structured skill with excellent executable examples and real, clearly signaled reference files. Its main weakness is token efficiency — a large inline gh command reference that duplicates what Claude already knows (and partially duplicates the wrapper section) plus low-value 'Model Guidance'/'Summary' sections — followed by minor validation gaps around destructive operations like merges.
Suggestions
Move the 'Commands (Direct gh Usage)' section into a references file (e.g., references/gh-commands.md) and keep only the read/write split guidance and a few representative commands in SKILL.md, since Claude already knows standard gh subcommands.
Trim the 'Model Guidance', 'Configuration', and 'Summary' sections — the quick-start steps in the summary repeat content already covered, and configuration via 'gh config' is gh knowledge Claude already has.
Add an explicit post-merge/post-close verification step (e.g., '$SKILL_DIR/scripts/github.py prs view 456' to confirm merge state) to the destructive-operation examples, and a short error-recovery note for failed auth or CI reruns.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly terse command blocks rather than explanatory prose, but it carries noticeable bulk Claude does not need: the entire 'Commands (Direct gh Usage)' section (~70 lines) restates gh subcommands and manual links that Claude already knows, the wrapper-script listing and direct-gh listing duplicate the same read operations, and the 'Model Guidance', 'Configuration', and 'Summary' sections restate or pad content. This fits the 'mostly efficient but could be tightened' anchor — noticeably above level 2's padded-prose profile but short of level 4 because whole sections could be cut. | 3 / 5 |
Actionability | Nearly everything is copy-paste executable: wrapper invocations with $SKILL_DIR paths and flags, complete action commands ('gh issue create --title ... --body ... --label bug', 'gh pr merge 456 --squash'), composed pipelines ('gh run watch $(gh run list --limit 1 --json databaseId --jq ...)', jq/@csv exports), and gh api POST examples. Common cases are covered end-to-end with concrete arguments, matching the fully-executable anchor. | 5 / 5 |
Workflow Clarity | Multi-step workflows (Daily PR Review, Create Issue and Link PR, Monitor CI) have a clear sequence and explicit verification comments ('# Verify approval was recorded', '# Verify: note the issue number from output', '# Verify rerun started'), plus a Troubleshooting section. It falls short of level 5 because error-recovery feedback loops are thin and some risky operations (e.g., 'gh pr merge 456 --squash') appear without a post-action verification step — matching the 'most checkpoints present, minor validation gaps' anchor. | 4 / 5 |
Progressive Disclosure | SKILL.md works as an overview with well-signaled, real, one-level-deep references: 'See [permissions.md](references/permissions.md) for read/write classification' and 'See [common-workflows.md](references/common-workflows.md) for more examples', both of which exist and carry substantive content, plus the wrapper in scripts/github.py. The gap versus level 5 is the ~70-line inline direct-gh command reference (and Advanced Usage material) that could itself live in a reference file, keeping the main body closer to a true overview. | 4 / 5 |
Total | 16 / 20 Passed |