Content
71%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-organized router-style body with a genuinely useful renderer selection table, concrete code-fence rules, and an explicit syntax-validation step. Its main weaknesses are the unresolved upstream references (no paths, and no such files in the bundle), no example artifact to anchor output format, and minor redundancy between Workflow and Delivery.
Suggestions
Name the upstream skill files explicitly (e.g., "read `markdown-viewer/skills/uml/SKILL.md`") or link their actual paths so step 3 is resolvable without guessing.
Include one minimal example artifact per renderer family (a small plantuml block and a vega-lite block) so the output format is anchored even when the pack is not installed locally.
Add an error-recovery branch to the workflow (what to do when the syntax check fails) and trim the redundant Delivery list, which restates the Workflow's final steps.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient — a selection table, terse output rules, and a 6-step workflow with no explanation of concepts Claude already knows. Not 5 because a few lines justify rather than instruct ("The value of this skill is that the output stays editable, reviewable, and renderable from Markdown", "Prefer a clear, accurate first version over decorative complexity") and the Delivery section partially restates the Workflow. | 4 / 5 |
Actionability | Mostly executable guidance: a concrete install command (`npx skills add markdown-viewer/skills`), an explicit goal→renderer mapping table, and an exact code-fence-to-renderer mapping. Not 5 because the core instruction — read the upstream skill for syntax — never shows even one example artifact block, and the upstream skill paths are never named, so a fresh agent can't execute step 3 without external installation; well above the 3 anchor's pseudocode-level guidance. | 4 / 5 |
Workflow Clarity | A clear 6-step sequence with a validation checkpoint: "Check syntax before delivery: matching fences, valid JSON/YAML where required, PlantUML starts and ends correctly, and labels are readable". Not 5 because there is no error-recovery loop (what to do when the check fails) and the delivery reporting list is a checklist without verification; not 3 because the checkpoint is explicit and concrete, unlike the 3 anchor's merely-implied validation. | 4 / 5 |
Progressive Disclosure | Sections are well-organized, but the body's key reference — "read the corresponding upstream `SKILL.md` for exact syntax and pitfalls" — points at files that do not exist in this bundle (no references/, scripts/, or assets/ directories) and gives no paths or links, so navigation to the detailed material is not actionable. This matches the 3 anchor ("references present but not clearly signaled") and is below 4, where references would be mostly clear and resolvable. | 3 / 5 |
Total | 15 / 20 Passed |