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.
A tight, high-signal procedural skill: sequenced steps with real commands, explicit abort/warn checkpoints, OSS-mode failure fallback, and a strong Do-not list guarding known failure modes. The two soft spots are the env-copy recipe being external to the file and the enterprise subsection carrying detail that belongs in its referenced companion skill.
Suggestions
Inline the fast `git ls-files` env-copy recipe (or at least the `--lite` core-apps variant) in step 3 so the copy step is executable without first loading `nv-worktree-commands`.
Trim the `--enterprise` subsection to the trigger, preference order, and OSS-mode fallback, moving the candidate-clone list and HTTPS/submodule commands into the referenced `enterprise-submodule` material.
Clarify in the Example section that the shown output is what the agent should report at the Confirm step, tying the example directly to step 6.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every line is operational with no padding or explanation of concepts Claude already knows — e.g. "Abort if the target path already exists" and "Do **not** run `setup-env-files.js` — it regenerates encryption keys". Rationale-bearing parentheticals ("walks `node_modules`/`.nx`, ~10s") convey failure modes and costs rather than background, fitting the anchor 'lean and efficient; every token earns its place'. | 5 / 5 |
Actionability | Concrete, copy-paste-ready commands cover preflight (`git rev-parse --show-toplevel && git worktree list && git status --porcelain`), creation (`git worktree add -b <branch> <worktree-path> [<base-ref>]`), and submodule init, plus a worked `/worktree` example and a specific handoff tool (`move_agent_to_root`). It falls short of 5 because the env-copy step's actual recipe ("Use the fast `git ls-files` recipe in `nv-worktree-commands`") is deferred to another skill rather than shown, leaving a minor executable gap in step 3. | 4 / 5 |
Workflow Clarity | Six clearly sequenced steps with explicit validation checkpoints and feedback loops: preflight abort ("Abort if the target path already exists"), a dirty-tree confirmation ask, enterprise skip conditions and failure fallback ("If enterprise setup fails, warn and continue in OSS mode — do not block the worktree"), and a final confirm checklist of what to report. This matches the top anchor; the operations are also non-destructive, so the missing-validation cap does not apply. | 5 / 5 |
Progressive Disclosure | Well-organized sections with clearly signaled, one-level-deep references ("For git command details and path naming, read `nv-worktree-commands`"; "For cleanup, read `nv-worktree-cleanup`"). Not a 5 because this bundle has no reference files of its own and the dense `--enterprise` subsection (candidate clones, HTTPS fallback, symlink refresh) largely duplicates material that could live in the referenced `enterprise-submodule` skill. | 4 / 5 |
Total | 18 / 20 Passed |