Content
92%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 a tight, fully operational runbook: concrete commands for environment detection, merging, pushing, and cleanup, with explicit validation gates and confirmation loops around every destructive action, plus a rationalizations table that preempts drift. The only gaps are minor: the PR-creation step and the merged-result test command are left as flexible placeholders rather than concrete examples.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence: no explanation of what git, worktrees, or PRs are; every section is operational ('Run the project's full test suite', 'git worktree remove "$WORKTREE_PATH"', 'Present the menu exactly as written'). Comments in the code blocks carry sequencing rationale ('Capture now, while still inside the workspace — Step 5 changes directory'), and the 'Common Rationalizations' table maps one line per real failure mode. This matches anchor 5 ('lean and efficient; every token earns its place') rather than 4, where noticeable trimmable explanations would remain. | 5 / 5 |
Actionability | Nearly everything is executable: concrete detection commands ('git rev-parse --git-common-dir'), copy-paste merge/push/delete sequences, verbatim menus, and a refusal-handling flow with a status command. It falls just short of anchor 5 ('fully executable; specific examples cover the common cases') because two paths stay generic: PR creation is delegated to 'the forge's tooling — its CLI if one is available, or the creation URL' without a concrete example command, and the merged-result verification is the literal placeholder '<test command>'. Comfortably above anchor 3, since the guidance is real commands, not pseudocode. | 4 / 5 |
Workflow Clarity | The six steps are explicitly sequenced with validation checkpoints at every risky point: tests must pass before the menu ('the menu comes after a green suite'), the merged result must be green before any cleanup ('Once the merged result is green'), discard requires a typed confirmation ('Type \'discard\' to confirm'), and worktree removal refusal triggers a show-and-ask loop with three concrete recovery options. This matches anchor 5 ('explicit validation steps; feedback loops for error recovery') — the destructive-operation cap of 3 does not apply because validation and confirmation are pervasive. | 5 / 5 |
Progressive Disclosure | No bundle files exist and the skill needs none: every part of the body is operational content used on each invocation (state detection table, menus, cleanup rules, quick-reference table), so nothing belongs in a separate reference file. The single file is well-sectioned with a clear overview line ('Verify tests → Detect environment → Present options → Execute choice → Clean up') and a Quick Reference table for navigation, with zero nested or buried references. This satisfies anchor 5's 'easy navigation / content appropriately split' — here, appropriately split means one self-contained file — and the rubric's simple-skill exception confirms well-organized sections with no external references can score 5. | 5 / 5 |
Total | 19 / 20 Passed |