Content
77%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 an unusually actionable and validation-heavy instruction set — exact commands, numeric thresholds, compile/render gates, and a preview-before-render QA loop with checklists — with excellent reference-table navigation in principle. Its two real weaknesses are redundancy (the pipeline, decision-gate probes, and core warnings are each stated multiple times, and the DNA registry is inlined despite having its own file) and bundle integrity: the most-cited files (CATALOG.md, themes/README.md, dna/README.md) are absent from the bundle, leaving the skill's central routing and schema steps unresolvable.
Suggestions
Merge 'Operational flow (TL;DR)' and 'Pipeline — 5 steps' into one canonical numbered sequence, state each load-bearing rule once (the 'embedding every word is the common mistake' warning appears four times), and drop either the inline 10-row DNA table or the dna/README.md pointer so the registry lives in exactly one place.
Ship CATALOG.md, themes/README.md, and dna/README.md in the bundle (or inline their essential content): they are referenced as the 'single source of truth for routing', the exact theme.json schema, and the DNA decision rule, yet do not exist — the skill's step 1 and both mode-authoring steps currently dead-end.
Deduplicate the pre-flight material between 'Decision gate — RUN FIRST' and 'Pre-flight probes' into a single probe list, and move the per-identity register/scene-fit details of the DNA table into dna/README.md, keeping only the pick rule inline.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with unique domain knowledge (no filler, no explanations of concepts Claude already knows), but it is noticeably duplicated rather than lean: the 5-step pipeline is stated twice ('Operational flow (TL;DR)' and 'Pipeline — 5 steps'), the decision-gate probes appear in two sections, 'embedding every word is the common mistake' is repeated four times, and a 10-row DNA registry table is inlined alongside a dedicated dna/README.md. This fits anchor 3 ('mostly efficient but could be tightened') better than 4, where only minor trimming would be needed — merging the duplicated sections would cut substantial length. It is above anchor 2 because the verbosity is structural repetition of load-bearing rules, not padded or generic explanation. | 3 / 5 |
Actionability | Fully executable throughout: copy-paste commands ('bash scripts/prepare.sh <project>', 'node scripts/preview-frames.cjs <project>', exact ffprobe/ffmpeg probe commands with flags, pinned 'npm install ... sharp@0.35.3 puppeteer@25.8.0 gsap@3.15.0'), exact numeric thresholds (luma > 150, ≥30% face uncovered per 0.3s, 80ms timing tolerance, 0.5s minimum on-screen), and pointers to exact schemas ('Schema: scripts/make-cinematic.cjs header'). Matches the anchor-5 pattern of copy-paste-ready commands covering the common cases; anchor 4's 'minor gaps' does not apply. | 5 / 5 |
Workflow Clarity | The sequence is explicit and numbered (init → prepare → author JSON per mode → preview QA → render, with a 'Decision gate — RUN FIRST' before either mode), validation is explicit at every stage (compile-time verbatim gate, check-timing.cjs --strict, render gates for timing/occlusion/overflow/hand-off, a 5-point Visual QA failure checklist plus 5 positive checks), and there is a genuine feedback loop ('Apply fixes in plan.json / theme.json, recompile, re-preview ... Render once, when the previews pass', plus the fresh-eyes subagent review). This matches anchor 5's 'explicit validation steps; feedback loops for error recovery; checklists'; anchor 4 is ruled out because no checkpoint is missing. | 5 / 5 |
Progressive Disclosure | Structure is genuinely good — a Shared knowledge table mapping 14 reference docs, per-need pointers ('skim by need', 'read before embedding') — but scored against the actual bundle: the three most load-bearing targets are missing. CATALOG.md (referenced ~10 times, called 'single source of truth for routing'), themes/README.md ('read FIRST — ... the exact theme.json schema'), and dna/README.md ('has the decision rule') do not exist in the bundle, so the routing and theme-schema steps dead-end; the inline 10-row DNA table also duplicates the (missing) registry file. This is worse than anchor 4's 'minor organization gaps' but better than anchors 1–2 (no deep nesting, no monolith; references are clearly signaled), so anchor 3 fits. | 3 / 5 |
Total | 16 / 20 Passed |