Content
85%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-engineered orchestrator body: fully executable commands, explicit validation and repair loops, a resume table, and clean one-level-deep reference signaling. The two real weaknesses are token redundancy at the top (routing blockquote duplicating the description, maintainer notes inline) and a bundle that ships only the references/ third of the files the body points to, leaving the agents/, phases/, and categories/ detail paths unresolved.
Suggestions
Ship the referenced detail files in the bundle (agents/director.md, agents/builder.md, agents/finalize.md, phases/source/guide.md + resolve.mjs, categories/<id>/module.md, catalog-map.md) or inline their minimal content — most 'Detailed flow' pointers currently dead-end.
Drop the opening routing blockquote (it duplicates the frontmatter description nearly verbatim, which is loaded anyway) and move the maintainer 'Design notes' section to a separate non-execution file so the always-loaded SKILL.md carries only execution-relevant tokens.
Replace the Step 2 'illustrative' resolve command with the real invocation once phases/source/resolve.mjs is present, so every Bash step is copy-paste executable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and operational throughout — phase tables, category tables, exact bash, and a maintainer section explicitly marked "execution does not read this" — with zero explanation of concepts Claude already knows. It falls short of anchor 5 ("every token earns its place") due to redundancy: the opening routing blockquote ("A longer, multi-scene, or narrated treatment → /general-video; a narrated video of a website → /website-to-video...") nearly duplicates the frontmatter description that loads alongside it, the phase table overlaps the Flow steps it summarizes, and the ~20-line maintainer "Design notes" section spends always-loaded tokens on non-execution content. | 4 / 5 |
Actionability | Copy-paste-ready commands for every Bash phase — init ("npx hyperframes init \"$PROJECT_DIR\" --non-interactive --example=blank"), render ("npx hyperframes render . --skill=motion-graphics -q draft -o ./renders/video.mp4"), verify ("npx hyperframes lint . && npx hyperframes inspect ."), preview — plus a one-line validation ("[ -s \"$PROJECT_DIR/shot-plan.json\" ] && echo ok || echo missing"), an enforced subshell convention ("every Bash command... is a (cd \"$PROJECT_DIR\" && ...) subshell"), and exact subagent prompt composition. The single illustrative command in Step 2 is explicitly flagged ("# illustrative — see phases/source/guide.md"), which the guidelines accept as justified flexibility. Common cases (asset-free vs. sourced, mp4 vs. overlay) are each covered. | 5 / 5 |
Workflow Clarity | Steps 0-6 are clearly sequenced with a conditional branch ("If asset_needs is empty, skip to Step 3"), explicit validation checkpoints at each handoff (the shot-plan existence check, lint/inspect with "exit 0 → done"), an error-recovery feedback loop ("On lint/inspect errors, dispatch the repair subagent... one in-place fix pass + re-render"), a graceful-degradation path for missing providers, and a resume table keyed on artifact state ("no shot-plan.json → Step 1", "renders/video.mp4 exists → Report + stop"). This matches anchor 5's feedback loops and checkpoints. | 5 / 5 |
Progressive Disclosure | The body's structure and signaling are strong — the phase table's "Detailed flow" column, "See phases/source/guide.md", and per-category "categories/<id>/module.md" pointers are one level deep and clearly labeled, and the three files under references/ (motion-vocabulary.md, shot-plan-ir.md, builder-contract.md) do exist and match their citations. However, scored against the actual bundle: the majority of referenced detail files — agents/director.md, agents/builder.md, agents/finalize.md, phases/source/guide.md, phases/source/resolve.mjs, categories/<id>/module.md, and catalog-map.md — are absent from the provided bundle, so most navigation pointers dead-end. That is more than the "minor organization gaps" of anchor 4 but well above the inlining/buried-reference problems of anchor 2, leaving anchor 3 (structure present, organization incomplete). | 3 / 5 |
Total | 17 / 20 Passed |