Content
81%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 well-structured, highly actionable workflow with strong validation gates and explicit handling of destructive operations. The main improvement opportunities are deduplicating the Common Mistakes and Red Flags sections and reducing placeholder commands.
Suggestions
Merge the overlapping "Common Mistakes" and "Red Flags" sections (worktree-removal-from-inside, provenance cleanup, and option-2 preservation each appear twice) into one checklist.
Replace the "npm test / cargo test / pytest / go test ./..." line and "<test command>" placeholder with explicit guidance to detect and reuse the project's existing test command.
Consider moving the Common Mistakes detail into a references file to slim the main SKILL.md body.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and command-driven with no padding explaining concepts Claude already knows, but the "Common Mistakes" and "Red Flags" sections substantially duplicate each other (worktree-removal-from-inside, provenance rules, and cleanup options appear in both), which could be trimmed. It is above the 3 anchor because the excess is redundancy rather than unnecessary explanation. | 4 / 5 |
Actionability | Concrete, mostly copy-paste-ready bash is given throughout (git rev-parse environment detection, git worktree remove, a complete gh pr create heredoc). Minor gaps remain: "npm test / cargo test / pytest / go test ./..." and "<test command>" are placeholders rather than executable commands, keeping it below the fully-executable 5 anchor. | 4 / 5 |
Workflow Clarity | A six-step sequence with explicit validation checkpoints ("Stop. Don't proceed to Step 2", "Verify tests on merged result", "Only after merge succeeds"), a feedback loop for failing tests, typed "discard" confirmation for the destructive option, and Always/Never checklists. Destructive operations are well validated, so the cap does not apply, and the explicit gates match the 5 anchor rather than the 4 anchor's 'minor validation gaps'. | 5 / 5 |
Progressive Disclosure | Well-organized single-file structure (Overview, numbered Process, Quick Reference table, Common Mistakes, Red Flags) with no nested or buried references, and the skill has no bundle files. It does not reach 5 because at ~250 lines the overlapping mistakes/red-flags material could be consolidated or split into a reference file, leaving minor organization gaps. | 4 / 5 |
Total | 17 / 20 Passed |