Content
73%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.
The body is a tight, executable workflow with strong validation gates appropriate to its destructive operations. Its main weakness is redundancy between the 'Common Mistakes' and 'Red Flags' sections plus a filler 'When to Use' line that add tokens without adding guidance.
Suggestions
Merge 'Common Mistakes' and 'Red Flags' into a single 'Rules' section (or drop one) — they restate the same verify-tests / confirm-discard / cleanup-for-1-and-4 guidance.
Replace the filler 'When to Use' line with concrete trigger conditions (e.g. 'Use after implementation is complete and all tests pass, when deciding between merge, PR, keep, or discard') or remove it.
Trim the 'Announce at start' line unless announcement behavior is a required part of the skill contract; it adds tokens for marginal guidance value.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly lean and command-driven, but 'Common Mistakes' and 'Red Flags' restate the same rules (verify tests, confirm discard, cleanup for options 1 & 4), and the 'When to Use' section is filler ('applicable to execute the workflow or actions described in the overview'). Not a 4 because the redundancy and filler are noticeable rather than minor. | 3 / 5 |
Actionability | Provides concrete, executable git/gh commands for every option (checkout, pull, merge, push, gh pr create, branch -d/-D, worktree remove) with a real PR heredoc template; minor gaps are the unsubstituted placeholders (<base-branch>, <test command>) which are appropriate for a template. | 4 / 5 |
Workflow Clarity | Clear 5-step sequence (verify tests → base branch → present options → execute → cleanup) with explicit validation checkpoints: a test-pass gate that halts before options, re-verification on the merged result, and a typed 'discard' confirmation for the destructive option. The destructive-operation validation cap does not apply because validation is present. | 5 / 5 |
Progressive Disclosure | A self-contained single SKILL.md with no bundle files, organized into clear headed sections (Process, Quick Reference, Common Mistakes, Red Flags, Integration); good structure with a minor organization gap from the duplicated Common Mistakes/Red Flags sections. Not a 5 because the redundancy keeps navigation from being fully clean. | 4 / 5 |
Total | 16 / 20 Passed |