Content
75%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 is highly actionable with executable templates, concrete numbers, checklists, and a repair loop, organized into clear sections with a well-signaled one-level reference. Its weaknesses are repetition of the height-match rule across multiple sections and overlap between inline examples and the reference file.
Suggestions
State the height-calc comment / h-[Npx] match rule once (in Hard rules) and reference it elsewhere instead of repeating it in the HTML section, the example block, and the hard-rules list.
Move the bad-pattern examples and one of the duplicated complete HTML examples into references/chart.md, keeping only a minimal template inline in SKILL.md.
Define or inline the "layout width/height self-check" referenced in calculation step 10, or point to the exact section of the layout skill that specifies it, to close the workflow validation gap.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body avoids explaining concepts Claude already knows, but the comment/frame height-match rule is restated at least four times (e.g. "The final number in the comment and h-[Npx] MUST match", "Do NOT write a comment that ends with one number and a frame height that uses another number", and again in Hard rules), and three near-complete HTML example blocks repeat the same template. This fits the mostly-efficient-but-could-be-tightened anchor rather than anchor 2, since the redundancy is domain-specific rule reinforcement rather than generic padding. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready guidance: complete PPT.createChart script with DOMContentLoaded wrapper, exact height-calc comment template with the @ppt-chart-height=N marker, concrete pixel arithmetic (p-6=48, gap-6=24, 24-40px reserve), a complete working example, and a diagnostic repair checklist. This clearly matches the fully-executable anchor with common cases covered. | 5 / 5 |
Workflow Clarity | A 9-step ordered decision checklist, a 10-step height calculation procedure, and a failure-repair sequence provide a clear sequence with most checkpoints and an error-recovery loop. It falls short of anchor 5 because step 10 references a "layout width/height self-check" that is never defined in this skill, and there is no explicit post-creation validation step. | 4 / 5 |
Progressive Disclosure | The single reference (references/chart.md) is real, one level deep, and clearly signaled at the top with its contents enumerated (chart selection, frame height guide, category axis patterns, layout integration tips). It misses anchor 5 because ~220 lines of inlined material, including bad-pattern examples and a complete working example, overlap sections that also live in the reference file and could be split out. | 4 / 5 |
Total | 16 / 20 Passed |