Content
92%Weight 40%Scale 1-3Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A high-quality, actionable skill with an excellent detect-first workflow and concrete executable commands, assuming Claude's competence without padding basics. The one weaker spot is progressive disclosure: detailed edge-case and per-harness guidance lives inline in a single long file rather than being split into signaled reference files.
Suggestions
Move the detailed PR-checkout variants (fork-safe detached-then-`gh pr checkout`, the already-checked-out edge cases) and the per-harness blocking-question tool list into a reference file (e.g. references/edge-cases.md), keeping SKILL.md as a tighter overview that links to it.
Pull the 'Other worktree operations' and 'Troubleshooting' sections into a separate reference so the main body stays focused on the detect -> native-tool -> git-fallback core flow.
Signal each extracted reference explicitly in-place (e.g. '**PR checkout edge cases**: See [references/edge-cases.md]') so navigation is one level deep and clearly marked.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean and information-dense with no padding or explanation of basics Claude already knows; every extended passage addresses a non-obvious gotcha (resolved-path comparison, trailing-slash check-ignore, detached FETCH_HEAD orphaning commits). It is dense, but each token earns its place. | 3 / 3 |
Actionability | Provides fully executable, copy-paste-ready commands throughout — `git rev-parse --absolute-git-dir`, `git worktree add -b <branch-name> .worktrees/<branch-name> origin/<from-branch>`, `git fetch origin pull/<n>/head:pr-<n>` — with concrete mode-specific variants rather than pseudocode. | 3 / 3 |
Workflow Clarity | Clear detect -> native-tool -> git-fallback sequence with explicit validation checkpoints (resolved-path isolation check, `git check-ignore` before creation, non-fatal fetch handling, already-checked-out rule, blocking user decision on permission failure) and error-recovery feedback loops for a risky operation. | 3 / 3 |
Progressive Disclosure | Well-sectioned and self-contained, but at ~87 lines it exceeds the simple-skill (<50 line) carve-out and keeps detailed edge-case content inline (fork-safe PR checkout variants, per-harness blocking-tool enumeration, troubleshooting) that could instead be split into one-level-deep reference files. It is not merely an overview pointing to deeper materials. | 2 / 3 |
Total | 11 / 12 Passed |