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.
An exemplary operational skill body: executable commands throughout, an explicit verify-before-merge workflow with a 4-point validation checklist and a symptom/cause/fix recovery table, and dense non-obvious gotchas (symlink trap, EROFS, warmup-swallow) instead of padding. Minor room remains in trimming a few wordy passages and offloading the per-CLI reference table and tmux recipe to bundle files.
Suggestions
Move the per-CLI headless invocation table and the full tmux execution recipe into a references/ file (e.g., references/cli-invocation.md), keeping SKILL.md to the core model and decision guidance.
Tighten the multi-clause parenthetical sentences in Primitive 4 and the 'When to use this vs. the alternatives' section — the same facts can be stated in fewer tokens.
Reconcile the CLI list between the description (codex/gemini/copilot/claude) and the body's invocation table (codex/agy/copilot/claude/qwen/opencode) so users aren't misled about supported tools.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean in the ways that matter — it explains no basic concepts (no 'what is a git worktree' filler) and every section delivers non-obvious operational knowledge like the '.gitignore with node_modules/ (trailing slash) matches a directory, not a symlink' trap and the 'git index lives in the main repo's .git/worktrees/<name>/, which is outside the worktree sandbox' EROFS explanation. A few long parenthetical passages (e.g., the multi-clause sentences in Primitive 4 and the alternatives section) could be tightened without losing content, matching anchor 4 rather than the every-token-earns-its-place of 5. | 4 / 5 |
Actionability | Guidance is fully executable and copy-paste ready: the complete 'quintet worktrees' CLI block with status/merge/abort subcommands, concrete tmux commands ('tmux new-window -t "=$sess" -n "wt1-codex" -c "$WT1" "codex exec -C '$WT1' -s workspace-write ... 2>&1 | tee '$WT1/.agent.log'"'), 'git worktree add -b <branch> <dir> <base>' setup, and the brief-file pattern ('cp brief.txt "<dir>/.brief.txt"'). A per-CLI invocation table with exact headless forms and auto-approve flags covers all common cases, matching anchor 5. | 5 / 5 |
Workflow Clarity | The sequence is explicit (dispatch with numbered steps 1-3 in the first-class CLI section, then verify-and-merge) and validation is emphatic and itemized: 'Never trust a worker's exit 0' with a 4-point in-order verification checklist (commit scope, leak scan, build green, 'grep the build output for secret values'). Feedback loops are present throughout — the 'abort and clean up in case of failure' command, 'amend out any leak' recovery, and the failure-mode quick-reference table mapping symptoms to fixes — matching anchor 5. | 5 / 5 |
Progressive Disclosure | Structure is good: a clear overview ('The core model' diagram, 'When to use this vs. the alternatives'), well-labeled sections (Primitive 1-8), and navigable tables. But the ~156-line body keeps everything inline with no bundle files — the per-CLI headless invocation table and the tmux execution recipe are reference-style material that could be split into a references/ file, which is the minor organization gap that holds this at anchor 4 rather than a 5 with appropriately split content. | 4 / 5 |
Total | 18 / 20 Passed |