Content
81%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 content is an excellent, tight operational guide: a clearly sequenced loop with explicit validation/repair checkpoints, copy-paste commands, and hard-won pitfalls that only project experience could supply. The main gaps are the absence of an inline minimal ctx.prove flow example, a few rhetorical flourishes that could be trimmed, and references that point to repo files rather than skill-bundle files.
Suggestions
Inline a minimal 3-4 line ctx.prove flow example (claim, voiceover, action, assert, screenshot) so the headline API is executable without opening evals/flows/.
Trim rhetorical flourishes (e.g. "It is the thing a human looks at (and listens to)...", "Pitfalls (learned the hard way)", "you have not framed the experience yet") to tighten token efficiency.
Move the ctx.* helper reference and flow patterns into skill-local files under references/ so the skill remains self-contained if the repo layout changes.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with project-specific facts Claude cannot know (CDP ports, __ipolloworkControl readiness, voiceover drift checks) and wastes no space on concepts Claude already knows. It falls short of a 5 due to rhetorical flourish ("It is the thing a human looks at (and listens to) to understand, at a glance...", "Pitfalls (learned the hard way)", "If you cannot say which of the two your fraimz is, you have not framed the experience yet") that could be trimmed without losing information. | 4 / 5 |
Actionability | Concrete, executable commands appear throughout: 'IPOLLOWORK_ELECTRON_REMOTE_DEBUG_PORT=9826 pnpm dev', 'pnpm fraimz --flow <id> --cdp-url http://127.0.0.1:9826', 'pnpm fraimz scaffold <id>', plus the full ctx.* helper signatures and copy-paste bash blocks. Not a 5 because no complete flow example is inlined (usage is delegated to evals/flows/ files) and placeholders like <electron-cdp-url> and <printed-electron-cdp-url> are not resolvable without external docs. | 4 / 5 |
Workflow Clarity | The loop is a clearly sequenced 6-step process with an explicit validation-and-repair feedback cycle ("Validate, repair, repeat... fix the visible state or the code and rerun until every claim has a passing assertion"), an explicit verdict gate ("Report Passed only when fraimz exists and every claim is backed by an observable assertion; otherwise Incomplete / Failed"), and a pitfalls section covering error recovery (stale dialogs, idempotency, readiness waits). This matches the anchor-5 pattern of clear sequence + explicit validation + error-recovery feedback loops. | 5 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are absent), and the body handles this well: it is a well-sectioned ~140-line overview whose 'Source of truth' section points one level deep to clearly labeled destinations (evals/README.md for the full reference, evals/flows/ for examples, evals/voiceovers/ for scripts). Not a 5 because the ctx.* helpers section partially duplicates the full reference in evals/README.md, and the referenced files live in the repo rather than a skill-local references/ file, so the skill is not self-contained if moved. | 4 / 5 |
Total | 17 / 20 Passed |