Content
88%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 content is a well-engineered operational runbook: exact commands, a complete classification table, safety rules, ordering constraints, and anti-patterns that anticipate real failure modes (squash-merge detection, branch-before-worktree deletion order). Its only weaknesses are minor redundancy in the repeated safety emphasis and a body long enough that some illustrative material could be offloaded to a reference file.
Suggestions
Deduplicate the force-removal rule stated in Hard rule 1, step 5's inline comments, and the first anti-pattern into a single authoritative statement.
Move the Example session (and optionally the base-branch fallback chain) into a references/ file, keeping SKILL.md focused on the workflow itself.
Trim the repeated explanation of why `-d` refuses squash-merged branches (it appears in the classification table, step 5, and an anti-pattern) to one location.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes git competence — no concept explanations, dense classification table, purposeful Anti-patterns section — but safety guidance repeats across 'Never `--force` remove… without explicit user confirmation' (Hard rule 1), the step-5 comments, and the first anti-pattern. That trimmable redundancy fits the 'efficient; minor instances… could be trimmed' anchor (4) rather than the every-token-earns-its-place anchor (5). | 4 / 5 |
Actionability | Fully executable throughout: `git worktree list --porcelain`, `git symbolic-ref refs/remotes/origin/HEAD`, `git merge-base --is-ancestor <branch> origin/<base>`, `gh pr list --state merged --head <branch> --json number`, and the exact remove/branch-delete commands with when-to-use-`-D` conditions. Commands are copy-paste ready and the classification table gives detection criteria for every case — matches the 5 anchor. | 5 / 5 |
Workflow Clarity | A clear six-step sequence with explicit validation checkpoints throughout a destructive batch operation: `git fetch --prune origin` before merge checks, per-worktree classification before any action, one consolidated `AskQuestion` approval, safe `git branch -d` that 'refuses if unmerged' as a built-in guard, and a closing report of removals, skips, and failures. Validation is present and explicit, so the destructive-batch cap of 3 does not apply; the error-recovery reporting matches the 5 anchor. | 5 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ absent) and the skill is a single well-sectioned document with clear headers (Hard rules, Workflow steps 1–6, Anti-patterns, Example session) — appropriate for a ~110-line procedural skill. Not a 5: at this length, the Example session and the base-branch fallback details are content that could live in a separate reference file, leaving SKILL.md a tighter overview. | 4 / 5 |
Total | 18 / 20 Passed |