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.
A well-structured, highly actionable skill body: executable commands, validated workflows with safety gates around destructive cleanup, and properly split one-level-deep references. The main costs are redundancy across the three overlapping checklist/pitfall sections and a broken README.md link in the References section.
Suggestions
Consolidate "Validation Checklist", "Operational Checklist", and the overlapping "Common Pitfalls"/"Best Practices"/"Failure Recovery" sections into a single checklist plus a short recovery section; ".worktree-ports.json" verification is repeated three times.
Remove the broken "[README.md](README.md)" link from the References section (no README.md exists in the bundle) or add the file.
Show the expected JSON schema for the `--input config.json` / stdin automation mode so the JSON example is executable end-to-end.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body never explains concepts Claude already knows (no 'what is a worktree' padding), but it is noticeably redundant across sections: "Validation Checklist", "Operational Checklist", "Common Pitfalls", "Best Practices", and "Failure Recovery" cover overlapping ground, and "Verify `.worktree-ports.json` exists" appears three separate times. This fits the 3 anchor (mostly efficient but could be tightened) rather than 4, since the overlap is more than minor trimming; it is not 2 because there is no filler or over-explanation of known concepts. | 3 / 5 |
Actionability | Concrete, copy-paste-ready commands are given for both create and cleanup flows with full flags (e.g. `python scripts/worktree_manager.py --repo . --branch feature/new-auth --name wt-auth --base-branch main --install-deps --format text`), plus stdin/`--input` JSON modes and real script paths. It stops short of 5 because the JSON automation example is not executable end-to-end: the expected structure of `config.json` is never shown, and no example output (text or JSON) is displayed so results can't be verified against expectations. | 4 / 5 |
Workflow Clarity | Create, parallel-session, and cleanup workflows are clearly sequenced, and the destructive cleanup path is well gated: "Detect stale worktrees and uncommitted changes before cleanup", "Remove only merged + clean worktrees, or force explicitly", and "Never force-remove dirty worktrees". The Validation and Operational checklists plus the Failure Recovery section (error -> corrective action, e.g. "If port allocation collides with external service: rerun with adjusted base ports") provide the explicit checkpoints and feedback loops of the 5 anchor, so the destructive-operation cap does not apply. | 5 / 5 |
Progressive Disclosure | The body is a clear overview with well-signaled, one-level-deep references (docker-compose-patterns.md and port-allocation-strategy.md, both verified present in the bundle) linked at point of use and again in a References section, and detail is appropriately split out. It misses 5 because the References section links "[README.md](README.md) for quick start and installation details" and no README.md exists in the bundle, a broken navigation pointer; it stays above 3 because structure, signaling, and content split are otherwise solid. | 4 / 5 |
Total | 16 / 20 Passed |