Content
73%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 operational skill: the workflow is clearly sequenced with real validation gates and a completion checklist, and commands are specific and executable. Weaknesses are rhetorical padding in the companion-flow and dispatch sections, oversized inline paragraphs, and a missing referenced bundle file (`sub-agents/frame-worker.md`) that breaks the dispatch role-assembly step as bundled.
Suggestions
Add the missing `sub-agents/frame-worker.md` (referenced by both the section 5 dispatch step and scripts/frame-packets.mjs) or remove the reference, since the worker-role assembly currently fails as bundled.
Move the section 5 step 1 plan-stage policy (catalog search rules, storyboard block format, citation rules) into a reference file and keep a short summary inline.
Trim the companion-flow rhetoric ("Arrive as the director, not the contractor", "The ceiling belongs to the concept, not the toolbox") to directive statements — the principles are useful but stated at essay length.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes competence (no explaining of what video editing or HTML is), but several sections are noticeably padded: the companion-flow prose ("A user who chose companion chose involvement and quality; the honest response is the best version you can design, not the smallest one you can defend", "The ceiling belongs to the concept, not the toolbox") is motivational rhetoric rather than instruction, and step 1 of section 5 is a single enormous paragraph inlining catalog-search policy, storyboard blocks, and citation rules that belong in a reference. This fits "mostly efficient but includes some unnecessary explanation or could be tightened" rather than the 4 anchor's "minor instances". | 3 / 5 |
Actionability | Commands are concrete and mostly copy-paste ready with defined placeholders: `npx hyperframes init "videos/<project>" --non-interactive --example=blank --skill=general-video`, `node <SKILL_DIR>/scripts/frame-packets.mjs --project "$PROJECT_DIR" --storyboard ...`, `npx hyperframes check`, plus exact data attributes (`data-media-start`, `data-duration`, `data-has-audio="true"`). It stops short of the 5 anchor because several operational paths are outside this bundle and unverifiable, and the dispatch workflow's `sub-agents/frame-worker.md` — cited as "this skill's" file — is absent, so the pipeline as bundled is not fully executable. | 4 / 5 |
Workflow Clarity | The process is explicitly sequenced with validation checkpoints and feedback loops: the section 2 state table routes entry, section 5 gives a numbered 8-stage dependency order, lint is positioned as fast feedback with check as the final gate ("it reruns lint internally, so do not run a redundant standalone lint"), failures are surfaced ("Surface an update failure instead of continuing from memory"), and section 7 is a completion checklist with a kept approval question before render. This matches the top anchor with explicit validation, error-recovery handling, and a checklist. | 5 / 5 |
Progressive Disclosure | The body acts as a routing overview with clearly signaled, one-level-deep references, mostly via condition→read tables ("Any composition HTML or scene layout → /hyperframes-core"), and its own script bundle exists with tests. It does not reach 5 because one intra-bundle reference is broken (`sub-agents/frame-worker.md` is cited in both the body and scripts/frame-packets.mjs's `deltaPath` but does not exist in this bundle) and long policy runs like the step-1 plan paragraph are inlined rather than split into a reference. | 4 / 5 |
Total | 16 / 20 Passed |