Content
60%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 well-structured, action-dense skill body with genuinely strong sequencing and validation checkpoints, undermined by a broken bundle: the four referenced files it depends on (including the schema.sql the mandatory entry step requires) are missing, and the core safety rule is stated three times. Fixing the bundle and de-duplicating the guardrails would move this to the top band.
Suggestions
Add the missing bundle files (schema.sql, COMMANDS.md, EXAMPLES.md, .gitattributes) or rewrite the Files section and the mandatory Step 1 so no reference points at a file that does not exist — the required 'read schema.sql first' step is currently unexecutable.
Consolidate the three repetitions of the never-destroy-the-DB rule (entry-point preamble, FORBIDDEN ACTIONS table, and DB-First Discipline) into one authoritative section, and remove the duplicated git-setup/csv-export command listings.
Move the detailed SQL snippets, merge-strategy table, and validation severity rules into the referenced COMMANDS.md/EXAMPLES.md files to shrink the ~370-line body toward a navigable overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly dense, well-compressed tables with no explanations of concepts Claude already knows, but the same safety material is repeated three times (the 'MANDATORY ENTRY POINT' preamble, the 'FORBIDDEN ACTIONS' table, and the 'DB-First Discipline' section all restate 'never destroy the existing DB'), and commands like git-setup and csv-export appear twice. This matches 'mostly efficient but includes some ... could be tightened' better than the 2 anchor, since the repetition is deliberate guardrail reinforcement rather than padding of known concepts. | 3 / 5 |
Actionability | Nearly everything is executable: exact bash invocations ('python .claude/skills/audit-flow/scripts/audit.py list'), concrete SQL INSERT examples with column lists, a full command table, and a validate command. It stops short of 5 because the mandated first step 'cat .claude/skills/audit-flow/schema.sql' and the Files-section references (COMMANDS.md, EXAMPLES.md, schema.sql) point at files that are not in the bundle, so the entry workflow cannot be executed as written, and CLI paths assume one fixed install location. | 4 / 5 |
Workflow Clarity | The sequence is clear and well-checkpointed: numbered entry steps, session-start questions, granularity choice, 'Every 5 tuples -> audit.py show', 'Run ... validate <session> before export' with an error/warning table, and a completion checklist — this satisfies the validation requirement for DB/batch operations. It misses 5 because step 1 of the mandated entry workflow (reading schema.sql) references a file absent from the bundle, so the first checkpoint fails before the feedback loops are reached. | 4 / 5 |
Progressive Disclosure | The Files section signals one-level-deep references, but scored against the actual bundle, 4 of the 5 referenced files (COMMANDS.md, EXAMPLES.md, schema.sql, .gitattributes) do not exist — only scripts/audit.py is present. Meanwhile ~370 lines of detailed SQL snippets, merge strategy, and validation rules that the missing COMMANDS.md/EXAMPLES.md were meant to hold are inlined in SKILL.md. This matches 'references are buried/broken and content that should be separate is inline' (2) rather than 3, where references would at least resolve. | 2 / 5 |
Total | 13 / 20 Passed |