Content
86%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, executable setup guide: correct bundle reference, complete settings snippets, merge-safe instructions, and a verification test. The only improvements are collapsing the duplicated JSON block and adding a failure-path for the verification step.
Suggestions
Show the project settings JSON once and state the global variant differs only in the command path ('~/.claude/hooks/block-dangerous-git.sh'), saving ~18 lines.
Add one line of error recovery to Step 5, e.g. 'If it does not block, check the script is executable and the settings JSON is valid.'
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes competence — no explanation of what git or hooks are — but the project and global settings JSON blocks are near-identical duplicates differing only in the command path, which is trimmable token cost. Efficient with minor instances that could be tightened, matching anchor 4 rather than the 'every token earns its place' of 5. | 4 / 5 |
Actionability | Fully executable guidance throughout: exact copy destinations, chmod, two copy-paste-ready settings JSON blocks (with the documented "$CLAUDE_PROJECT_DIR" pattern), a merge instruction to avoid clobbering existing settings, and a concrete verification command with expected exit code 2 and stderr message. The only placeholder, <path-to-script>, is self-explanatory. Matches anchor 5's copy-paste-ready coverage of common cases. | 5 / 5 |
Workflow Clarity | Five clearly sequenced steps with an explicit validation step ("Should exit with code 2 and print a BLOCKED message to stderr"), so the destructive/batch cap of 3 does not apply. It stops short of anchor 5 because there is no feedback loop — no guidance on what to do if the verification test fails to block. | 4 / 5 |
Progressive Disclosure | Verified against the actual bundle: the single referenced file, [scripts/block-dangerous-git.sh](scripts/block-dangerous-git.sh), exists, is correctly linked, one level deep, and holds exactly the content that belongs outside SKILL.md (the pattern-matching logic). The body is a well-sectioned overview (blocked list, steps, verify) with no content that should be split out. Clear overview with a well-signaled one-level reference matches anchor 5. | 5 / 5 |
Total | 18 / 20 Passed |