Content
81%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 highly actionable, well-sequenced gate procedure with excellent validation feedback loops and concrete recovery guidance for every failure mode. Its main weakness is redundancy — pronunciation/brand-rule details stated twice and the motivating example repeated across sections — which inflates token cost without adding information.
Suggestions
State the --pronunciations / brand-rules.json semantics once: the Contract section and 'Brand names and confirmed pronunciations' both explain the file format and the 'missing file is an ERROR (exit 3)' behavior — consolidate into one section and cross-reference it.
Trim 'Why this exists' to a sentence or two: the 'human-vetted' → 'human witted' example and the silent-audio failure are already covered by the description and the 'What still FAILS' table.
Resolve the 'tests/test_review_render.py' reference — the file is absent from this skill's bundle, so either ship the tests directory or point at wherever the test actually lives.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every line carries skill-specific knowledge Claude does not already have (canonicalization rules, alias guards), but there is real duplication: the `--pronunciations` flag and brand-rules.json format are described in full in both the Contract section and "Brand names and confirmed pronunciations" ("A missing file is an ERROR (exit 3)" appears twice), and the "Why this exists" narrative repeats the "human-vetted" spoken as "human witted" example already in the description and failure table. This fits 'Mostly efficient but includes some unnecessary explanation or could be tightened' rather than 4, where over-explanation would be only minor. | 3 / 5 |
Actionability | The contract gives a copy-paste-ready command with flags and output paths, and the exit-code contract maps each outcome to a concrete action ("exit 2 → FAIL — do NOT pin it. Read the report, fix, re-run"). The failure table maps exact report lines to root causes and fixes, and a runnable test command is included — fully executable guidance covering common cases. | 5 / 5 |
Workflow Clarity | The sequence is explicit and ordered (persist "the exact approved spoken lines ... to working/approved-script.txt" before render, run after the master render exists and before pinning), with "Re-run after every fix / re-roll until it PASSES" as an explicit feedback loop and a per-exit-code recovery path. This matches the 5 anchor: clear sequence, explicit validation steps, and error-recovery guidance via the failure table. | 5 / 5 |
Progressive Disclosure | Good section structure with the heavy implementation correctly split into `scripts/review_render.py` (which exists in the bundle) one level deep, and cross-skill references (create-vo-elevenlabs, seedance failure modes) clearly named. It falls short of 5 because the canonicalization/failure tables could live in a reference file, and the referenced "tests/test_review_render.py" does not exist in this skill's bundle — minor organization gaps per the 4 anchor. | 4 / 5 |
Total | 17 / 20 Passed |