Content
86%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 lean, highly actionable runbook: concrete commands, clear sequencing, and real validation checkpoints around destructive operations (force-push, branch deletion). The main gaps are the undefined `committer` tool and the absence of failure-path guidance if the gate, rebase, or final merge verification fails.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is telegraphic — 16 numbered steps of pure commands and variable assignments with zero padding, no git-concept explanations, and no filler. Every line instructs an action, fully matching 'lean and efficient; assumes Claude's competence'. | 5 / 5 |
Actionability | Nearly every step is a copy-paste-ready `gh`/`git` command with exact --jq filters and variable captures. It falls short of level 5 only because `committer` is an undefined external tool (no signature, no fallback shown) and placeholders like `<summary>` and `$contrib` require unstated context. | 4 / 5 |
Workflow Clarity | The 16 steps are explicitly ordered with a stated goal (PR must end MERGED), a pre-commit gate (step 9), guarded force-push (--force-with-lease), and a final state verification (step 15) — so validation is present despite destructive operations. It misses level 5 because there is no error-recovery loop: what to do when the gate fails, conflicts are unresolvable, or the merge state check fails is left implicit. | 4 / 5 |
Progressive Disclosure | This is a ~50-line single-purpose skill with no bundle files (references/, scripts/, assets/ are absent) and no content that warrants offloading. Per the simple-skill exception, the clean single numbered list with a goal header is appropriately organized for a skill of this size. | 5 / 5 |
Total | 18 / 20 Passed |