Content
92%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.
An excellent single-file operational skill: dense, executable, and safety-gated, with a real decision procedure rather than vague advice. The only defect is two stray '</content></invoke>' wrapper tags left at the end of the file.
Suggestions
Remove the stray '</content>' and '</invoke>' lines (75-76) at the end of SKILL.md — they are generation artifacts that waste tokens and could confuse parsing.
The 'Related Skills' links point outside the bundle (../git/SKILL.md, ../standalone-commits/SKILL.md, ../pull-request/SKILL.md); either verify these paths exist in the deployment context or drop the ones that don't.
The audit loop's 'git cherry ... | grep -c ^+' output is used only for the unmerged label; a one-line note on interpreting 'ahead' counts for stacked branches would tighten the reap decision further.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's git competence throughout — no space is spent explaining what worktrees or merge-base are ('Four harnesses (warp, codex, opencode, conductor) mint worktrees here and nothing reaps them', 'lockfile-only churn is throwaway'). Not a 5 only because the file ends with two stray wrapper tags ('</content>', '</invoke>' at lines 75-76) that are pure noise and could be trimmed. | 4 / 5 |
Actionability | Fully executable guidance: a copy-paste audit pipeline ('git worktree list --porcelain | awk ... | while read b; do ... git cherry "$base" "$b" ... git merge-base --is-ancestor "$b" "$base"'), and a decision table with exact commands ('git worktree remove <path>; git branch -d/-D <branch>', 'git -C <path> switch -c holding/<name>'). Not below 5: placeholders are appropriate parameterization, not pseudocode. | 5 / 5 |
Workflow Clarity | Destructive/batch skill with explicit validation checkpoints and feedback loops: fetch-first, self-run audit ('Never reap on a self-reported "it's merged"; run the audit yourself'), an approval gate before any destructive command, and a Final Checks section ('Re-run the audit; confirm reaped paths are gone', 'git worktree prune', 'Confirm any anchored holding/* branches still hold their edits'). This clears the destructive-operations cap comfortably. | 5 / 5 |
Progressive Disclosure | A single-file skill (no references/, scripts/, or assets/ directories exist, and none are needed at this size) with well-organized, navigable sections (reap signal, audit procedure, decision table, gotchas, invariant, final checks) and one clearly-signaled Related Skills pointer. Content is appropriately placed; nothing is inlined that belongs in a separate file. | 5 / 5 |
Total | 19 / 20 Passed |