Content
96%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 tightly written, highly actionable workflow skill with excellent command-level guidance, explicit gates, and thorough error/fallback handling. The only weakness is that its progressive-disclosure references point to sibling skills/files that are not present in this skill's own bundle.
Suggestions
Co-locate or vendor the critical sibling references this skill depends on at runtime (../audit/references/security.md, ../audit/references/review-levels.md, ../hyperflow/DOCTRINE.md, ../hyperflow/git-workflow.md) so the skill is navigable as a standalone unit, or add a one-line 'Requirements' note stating these must be present in the plugin tree.
For the merge-method inference rule, inline the brief decision logic (linear history → --rebase, merge commits present → --merge, squash-dominant → --squash) as a tiny checklist rather than only prose, so the gate's status line can be generated deterministically.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean and command-dense throughout — it assumes Claude knows what a PR, gh, and git are, and devotes prose only to substantive rules (the untrusted-code boundary, gate etiquette) rather than concept explanations. | 5 / 5 |
Actionability | Fully executable guidance with real commands and exact argument shapes — 'gh pr view <n> --json title,body,…', 'git fetch origin pull/<n>/head:pr-<n>', 'gh api repos/{owner}/{repo}/pulls/<n>/reviews', 'git push origin pr-<n>:<headRefName>' — covering the common cases copy-paste ready modulo parameter substitution. | 5 / 5 |
Workflow Clarity | Steps 0–5 are explicitly sequenced with validation checkpoints (preflight auth check, SECURITY_VIOLATION halt, posting gate, fix gate, merge gate) and a feedback table covering fetch-failure fallback, unauthenticated mode, and headless behavior — exactly the validate→fix→retry structure the rubric rewards for destructive/batched operations. | 5 / 5 |
Progressive Disclosure | The body is a clear overview with well-signaled, one-level-deep references to sibling materials, but every cross-skill link (../audit/SKILL.md, ../audit/references/security.md, ../audit/references/review-levels.md, ../dispatch/SKILL.md, ../hyperflow/DOCTRINE.md, ../hyperflow/git-workflow.md, ../issue/SKILL.md) was verified missing from the skill's bundle, leaving minor navigation gaps. | 4 / 5 |
Total | 19 / 20 Passed |