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.
Strong operational content: the run commands, mapping table, gating checks, and QC checklist make the skill fully executable with excellent validation feedback loops. Weaknesses are noticeable repetition of the credibility-guard rules throughout, and a bundle mismatch — the referenced project/ Remotion tree is absent while the skill claims it is bundled.
Suggestions
State each craft rule once (in 'Craft rules') and remove the duplicate 'never AI-rendered / never stretched / gradient divs' passages from the intro and beat descriptions.
Fix the bundle mismatch: either ship the referenced project/ tree or clearly mark where it comes from, since props.ts, anim.ts, and the scenes are cited as load-bearing paths that don't exist in the bundle.
Move the config→props mapping table to a reference file (e.g. references/props-mapping.md) and link to it, keeping SKILL.md as a lean overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient — the beat spine, mapping table, and craft rules are load-bearing — but there is noticeable redundancy: 'the ONLY composited bitmap is the REAL product photo (objectFit:contain, never stretched)' and 'never AI-rendered text' each appear verbatim in the intro, the beats section, AND the Craft rules; 'never stretched' appears four times. The 'Choices' section also restates that decisions come from the recipe twice. Fits the 3 anchor ('could be tightened') better than 4, where redundancy would be only minor. | 3 / 5 |
Actionability | Fully executable: the Run section gives copy-paste commands ('cd project && npm install', 'python3 scripts/render.py --config path/to/config.json --work-dir <work> --out <work>/master') with expected outputs named, the config→props mapping table tells exactly which key binds to which prop and scene, and a worked example config (scripts/config.example.json) is shipped. Gating checks state precise pass/fail vocabulary. Matches the 5 anchor; not 4 because no key execution detail is missing. | 5 / 5 |
Workflow Clarity | Clear sequence with explicit validation checkpoints and feedback loops: two gating checks run pre-render and 'render.py rejects' on failure (validate→fix→retry), install is auto-run when node_modules is absent, and a post-render 'watch' QC checklist ('re-confirm both gating checks visually') closes the loop. This exceeds the 4 anchor, which allows minor validation gaps — none are present. | 5 / 5 |
Progressive Disclosure | The body is well-sectioned and points to real bundle files (scripts/render.py, scripts/config.example.json — both verified present), but it repeatedly references a bundled 'project/' tree ('project/src/props.ts', 'project/src/lib/anim.ts', 'project/src/scenes/Scene01–04.tsx', 'project/package.json') that does not exist in the bundle, so those references are broken. Additionally, the 15-row config→props mapping table is inline reference material that belongs in a separate reference file. Fits the 3 anchor ('content that should be separate is inline'; references not all resolving) better than 4, where references would be mostly clear. | 3 / 5 |
Total | 16 / 20 Passed |