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 a strong, contract-driven workflow: precise artifact list, bounded checkup loop with human escalation, and safety rules around destructive output. Its weaknesses are redundancy from versioned editing (the brief-only boundary repeated four times, v0.6.4 vs 1.1.1 version confusion), no complete example command in the body, and several cited bundle paths (adapters/, registry/, SPEC.md, docs/) that don't exist in this skill's directory.
Suggestions
State the brief-only boundary once (invariant 1) and prune its restatements from the Positioning, OPC, and rule-4 sections; fold the 'formerly the QA loop' and other version-history asides into a release-notes reference file.
Reconcile the version references: the body hard-codes 'v0.6.4' in two headings and five invariants while the frontmatter says 'version: 1.1.1' — either update the invariants to be version-neutral or move the per-version notes to a deprecated/old-patterns section.
Fix or remove the broken bundle references (adapters/ppt-master-bridge-notes.md, registry/renderer_registry.json, SPEC.md, docs/index.md, docs/local-demo.md), and add one complete example invocation (e.g. a full `python scripts/humanize_ppt.py --source ... --out ...` brief-mode and a `--qa-from` checkup command) inline so the CLI surface is usable without the missing docs file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly information-dense (no explanations of concepts Claude already knows), but noticeably tightened-by-editing residue: the brief-only boundary is restated at least four times ('never copies a downstream skill's template... and never post-processes rendered HTML or PPTX' appears in Positioning, the OPC P block, invariant 1, and rule 4's postMessage clause), version noise ('v0.6.4' in two headings and five invariants contradicts the frontmatter's 'version: 1.1.1', and 'formerly the QA loop, CLI flag still --qa-from') is history, not instruction, and rule 12's dense WorkBuddy packaging paragraph duplicates references/workbuddy-team-packaging-and-video-materials.md. It could be tightened but is not padded — fits anchor 3 rather than 2. | 3 / 5 |
Actionability | Concrete, executable surface throughout: an 18-artifact output contract with exact filenames and JSON shapes ('slide_plan.json... per-page media: {image, diagram, video} decision and layout_hint'), named CLI flags (--qa-from, --style-gallery, --renderer ppt-master, --out with a precise auto-wipe/refusal rule and --force), and named scripts. The gap keeping it below 5: not a single complete example invocation appears in the body, and the pointer for them ('Full CLI examples... live in docs/local-demo.md') targets a file that does not exist in this bundle. | 4 / 5 |
Workflow Clarity | The O→P→Q→C workflow is a clear end-to-end sequence with explicit validation checkpoints and a bounded feedback loop: two human review gates before rendering (outline preview, style gallery), QA iterations that 'write qa_report.md, fix_prompt.md, and qa_iteration.json', a hard cap ('The presentation checkup caps at 3 iterations. Round 4 with remaining fail findings is needs-human'), and a destructive-output safeguard (--out refuses to wipe unless it is empty, missing, or a prior Humanize run, else requires --force). This matches the 5 anchor: explicit validation, error-recovery loop, and escalation to a human. | 5 / 5 |
Progressive Disclosure | The 'Operational references' section is well-signaled with one-line annotations and the references/ files it cites all exist, but five paths the body points to are absent from the bundle: 'adapters/ppt-master-bridge-notes.md', 'registry/renderer_registry.json' (cited as the machine-readable source of truth for support levels), 'SPEC.md', 'docs/index.md' ('Full annotated index of all references'), and 'docs/local-demo.md' (which holds all CLI examples). Combined with inline content that duplicates an existing reference (rule 12), navigation is partly broken — fits anchor 3 ('references present but... content that should be separate is inline') better than 4's 'minor organization gaps'. | 3 / 5 |
Total | 15 / 20 Passed |