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, highly actionable skill file: concrete commands, explicit preflight validation, user-confirmation checkpoints, and graceful fallback behavior. Its main weakness is that the env-copy rsync recipe, path sanitization rules, and submodule troubleshooting all live in files outside this bundle, so those steps cannot be executed from the bundle alone.
Suggestions
Inline the .env* copy command (or a one-line rsync) in step 3 so the copy step is executable without nv-worktree-commands.
State the sanitized-name rule for the worktree path inline (or as a short formula), since step 2's <worktree-path> otherwise depends on an external file.
If nv-worktree-commands and nv-worktree-cleanup are meant to be part of this skill's bundle, include them under references/; if they are separate skills, note that explicitly so navigation expectations are unambiguous.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence — a terse goal statement, an input table, and step commands with no git-concept explanations; warnings carry real signal ('it regenerates encryption keys'). Every token earns its place, matching the 5 anchor rather than 4 because there is no over-explanation left to trim. | 5 / 5 |
Actionability | Mostly executable: 'git worktree add -b <branch> <worktree-path> [<base-ref>]', the preflight command trio, the submodule block, and 'pnpm symlink:submodules || echo "WARN: ..."' are concrete. Two executable details — the .env* 'rsync recipe' and the sanitized-name path rule — are deferred to nv-worktree-commands rather than given inline, so it is not fully copy-paste ready from this file alone. | 4 / 5 |
Workflow Clarity | Six clearly sequenced steps with explicit validation ('abort if the target path already exists', dirty-tree warning with user confirmation), error-recovery fallbacks ('warn and continue in OSS mode — do not block the worktree'), and a final confirm checklist. This matches the 5 anchor; it is above 4 because checkpoints are explicit and complete rather than mostly present. | 5 / 5 |
Progressive Disclosure | The SKILL.md is a well-structured overview that defers detail one level deep with clearly signaled references ('For git command details and path naming, read nv-worktree-commands. For cleanup, read nv-worktree-cleanup.'). However, none of the referenced files (nv-worktree-commands, nv-worktree-cleanup, enterprise-submodule) exist in this bundle — there is no references/ directory — so the deferred details are not reachable from the bundle itself, a minor organization gap that fits the 4 anchor. | 4 / 5 |
Total | 18 / 20 Passed |