Content
82%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 body built almost entirely from executable gh commands, with an explicit ordered PR workflow, duplicate-PR validation, and an error-recovery table. The main improvement opportunities are consolidating the three overlapping PR-creation sections and adding a pre-merge CI verification step.
Suggestions
Consolidate 'Full PR creation workflow', 'Create a PR (minimal form)', and 'Capturing PR URL' into one section — all three repeat 'gh pr create' with the same --title/--base/--body pattern, which is the main conciseness drag.
Before the destructive 'gh pr merge --squash --delete-branch' step, require verification that CI is green (e.g. run 'gh pr checks' and only merge when all pass) — this closes the workflow-clarity validation gap and addresses the destructive-operation feedback-loop note.
Trim the 'Shell escaping rule' to the single directive ('Always use single-quoted <<'EOF'') with one correct example — the detailed WRONG-variant walkthrough re-teaches heredoc quoting Claude already knows.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with executable commands and tables and almost never explains concepts Claude already knows; the 'Why gh over MCP' section is brief policy justification, not padding. Minor over-explanation remains — e.g. the 'Shell escaping rule' teaches heredoc quoting Claude already knows, and the token-cost figures — so it sits at anchor 4 ('efficient; minor instances of over-explanation') rather than the fully lean anchor 5. | 4 / 5 |
Actionability | Nearly every section is copy-paste-ready: complete 'gh pr create' with heredoc body, 'gh pr view --json ... --jq ...' filters, 'gh pr checks --json name,state,conclusion', 'gh pr merge <number-or-url> --squash --delete-branch', plus a per-error fix table. This matches anchor 5 ('fully executable; copy-paste ready code or commands; specific examples cover the common cases'). | 5 / 5 |
Workflow Clarity | The 'Full PR creation workflow' gives a numbered, ordered sequence ('Always follow this order — never skip steps') with a real validation checkpoint (checking for an existing PR before creating one) and an error-handling table that acts as a feedback loop (auth error → gh auth login; branch not pushed → push first). It falls short of anchor 5 only because the destructive 'gh pr merge --squash --delete-branch' step says just 'when authorized' without requiring CI checks to be green first — a minor validation gap consistent with anchor 4. | 4 / 5 |
Progressive Disclosure | The skill has no bundle files and organizes everything into clearly headed sections (workflow, core commands, conventions tables, error handling) that are easy to navigate — matching anchor 4's 'good structure; most content is appropriately placed'. It misses anchor 5 partly because at ~200 lines some material (e.g. the branch/title/body-template conventions) could live in a reference file, and there is duplication across the three PR-creation sections ('Full PR creation workflow', 'Create a PR (minimal form)', 'Capturing PR URL') plus repeated auth guidance between the 'Authentication' section and the error table. | 4 / 5 |
Total | 17 / 20 Passed |