Content
70%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 delivers excellent workflow discipline — a gated 5-step pipeline, concrete executable commands, explicit validation checkpoints and feedback loops — but it is undermined by a bloated, repetitive prose style and by core routing artifacts (CATALOG.md, dna/, themes/) that are referenced throughout yet missing from the bundle. Strongest on process rigor, weakest on token efficiency and bundle navigation integrity.
Suggestions
Ship CATALOG.md, dna/README.md, and themes/README.md inside the bundle (or inline a minimal identity-routing table in SKILL.md) so the 'pick ONE identity' step and every CATALOG.md link resolve — currently the routing surface the whole workflow depends on is absent.
Deduplicate the rail-default / embed-scarcity rule, which is stated in the intro, Caption model, Step 0, and Non-negotiables; state it once in Non-negotiables and reference it elsewhere, cutting the body's length and reading load.
Break up the dense multi-clause parentheticals (e.g. the Theme-mode PLATE exception and the hero-lockup description) into separate bullets or move the detail into the existing references, keeping SKILL.md at overview altitude.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with domain-specific, non-obvious rules (no time wasted explaining things Claude already knows), but it is noticeably padded by repetition: "embedding every word is the common mistake" appears three times (intro, Caption model, Non-negotiables), the rail-first default is restated in at least four places, and long nested parentheticals (e.g. the Theme-mode exception aside) force re-reading. Anchor 3 fits — mostly efficient but could be tightened by deduplication; not 2 because the padding is repetition of load-bearing rules rather than generic explanation. | 3 / 5 |
Actionability | Highly concrete and executable: pinned install commands (sharp@0.35.3 puppeteer@25.8.0 gsap@3.15.0), exact probe commands (ffprobe, the 1fps contact-sheet ffmpeg line), full pipeline invocations (bash scripts/prepare.sh, node scripts/make-cinematic.cjs, render-theme.sh), and exact thresholds (80ms timing tolerance, luma 60/180/150, ≥0.5s captions). It misses anchor 5 because the single most-cited artifact — "pick ONE identity from CATALOG.md" (~8 references, "single source of truth for routing"), plus dna/README.md and themes/README.md — is not in the bundle, so the identity-pick step cannot be executed as written; not 3 because the commands present are copy-paste ready, not pseudocode. | 4 / 5 |
Workflow Clarity | A clear 5-step pipeline with a decision gate that runs first, explicit validation at every stage (timing/occlusion/overflow gates, check-timing.cjs --strict, verbatim-completeness gate at compile time), a preview-before-render checkpoint, and explicit feedback loops: "Apply fixes in cinematic.json / theme.json, recompile, re-preview — each loop costs seconds. Render once, when the previews pass", plus recovery paths (WHISPER_MODEL=medium retry, else refuse). This matches anchor 5's sequence + validation + feedback loops + checklists (5 failure checks, 5 positive checks). | 5 / 5 |
Progressive Disclosure | Structure is good in form — a Shared knowledge table of 14 reference docs (all present in references/), clearly signaled one-level-deep links, and "Skim by need" guidance. But scoring against the actual bundle: the three most load-bearing references — CATALOG.md, dna/README.md, and themes/README.md — are repeatedly cited yet absent from the bundle, and links also point outside it (../hyperframes/references/plugin-installation.md, brief-contract.md), so navigation breaks exactly where routing depends on it. That is beyond anchor 4's "minor organization gaps" and fits anchor 3's incomplete reference structure. | 3 / 5 |
Total | 15 / 20 Passed |