Content
60%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 highly actionable — dense with executable, copy-paste-ready git worktree commands and clearly sequenced workflows with checklists and troubleshooting — but it is a monolithic 1,360-line file with no bundle structure, inlining three full command-suite specifications that belong in separate reference files and duplicating cherry-pick/merge/checkout guidance across sections. Much of the padding re-explains basic git and dependency-installation behavior Claude already knows.
Suggestions
Split the three '/worktrees' command specifications (compare, create, merge) into separate reference files (e.g. references/compare.md, references/create.md, references/merge.md) and keep SKILL.md as a concise overview with one-level-deep, clearly signaled links to them.
Deduplicate repeated content: the interactive patch-mode key table, selective-merge/cherry-pick strategies, and 'Branch is already checked out' troubleshooting each appear 2-3 times across the main body and the appended command sections — state each once and cross-reference.
Cut explanations of concepts Claude already knows — how git merge/diff/cherry-pick work, and lockfile-based package-manager detection (bun.lockb -> bun install etc.) — and trim to the worktree-specific knowledge (branch lock, prune, repair, sibling-directory conventions, cleanup gotchas) that is the skill's actual value.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is ~1,360 lines / 40KB with heavy redundancy and padding: the interactive patch keys (y/n/s/e) are explained twice, "Branch is already checked out" troubleshooting appears in three places, and the appended '/worktrees' sections re-explain cherry-pick, selective checkout, and merge strategies already covered above. It also extensively explains concepts Claude already knows — basic git merge/diff/cherry-pick behavior and lockfile-to-package-manager dependency detection ("bun.lockb -> Use bun install", "package-lock.json or default -> Use npm install") — fitting the 'noticeably verbose, several unnecessary explanations or padded sections' anchor rather than the wholly-beginner-explanation anchor of 1, since the core worktree material (branch lock, prune, repair, lock/unlock) is genuinely skill-specific. | 2 / 5 |
Actionability | Nearly every section is copy-paste-ready executable bash with realistic, complete arguments — e.g. "git worktree add -b hotfix-456 ../project-hotfix origin/main", "git worktree add --track -b feature ../feature origin/feature", "git cherry-pick abc1234^..def5678" — including worked end-to-end patterns (fetch PR ref, add worktree, review, remove) that cover the common cases. | 5 / 5 |
Workflow Clarity | The three command workflows use explicit numbered sequences starting with state checks ("Run git worktree list ... and git status to verify the repository state is clean"), include a Verification Checklist organized by phase, and have error-recovery troubleshooting sections (dependency-install failure recovery, conflict resolution with "<<<<<<<" markers). Minor validation gaps keep it below 5: the create workflow warns about a dirty state but defines no follow-up action, and force-removal ("discards uncommitted changes") relies on a checklist reminder rather than an in-workflow checkpoint. | 4 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, or assets/), so everything — including three complete ~250-line command-suite specifications ("How to Compare Worktrees", "How to Create Worktree", "How to Merge Worktree") that clearly belong in separate reference files — is inlined in one monolithic SKILL.md. This matches the anchor 'content that clearly belongs in separate files is inlined'; the internal section headers prevent the lowest score but no progressive disclosure structure exists at all. | 2 / 5 |
Total | 13 / 20 Passed |