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 copy-paste commands, a complete worked example, and a genuine error-recovery feedback loop, and it correctly references real bundle files. Its weakest point is conciseness: substantial generic governance boilerplate and a self-contradicting validation section add tokens without PDF-specific value.
Suggestions
Remove or collapse the generic governance boilerplate (When Not to Use, Required Inputs, Output Contract, Validation and Safety Rules, Failure Handling, Quick Validation, Deterministic Output Rules, Completion Checklist) that is not specific to HTML→PDF conversion.
Resolve the contradiction between the 'Validation Shortcut' (run scripts/validate_skill.py --help) and the 'Quick Validation' section claiming 'No local script validation step is required'.
Consider moving the full 44-line single-page HTML example into a references/ file (e.g. examples/single-page.html) referenced from the body to tighten SKILL.md further.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The PDF-specific core (CLI table, RTL handling, page-fit rule, verification loop) is lean and assumes competence, but ~8 generic governance sections (When Not to Use, Output Contract, Validation and Safety Rules, Deterministic Output Rules, Completion Checklist, etc.) are template boilerplate, and the 'Quick Validation' section self-contradicts ('No local script validation step is required' despite a Validation Shortcut that runs the validator). | 3 / 5 |
Actionability | Seven numbered copy-paste-ready commands cover install, local→PDF, URL→PDF, RTL, stdin, executable-path, and a complete single-page A4 HTML example, plus a CLI parameter table with defaults — fully executable and covering the common cases. | 5 / 5 |
Workflow Clarity | The 'Verification workflow' provides an explicit 5-attempt feedback loop (default → reduce scale → reduce margin → both → fix HTML/CSS) with visual validation checks, but the primary 'Recommended Workflow' uses vague validation ('Run a final validation pass') and the contradictory 'Quick Validation' section leaves a minor checkpoint gap. | 4 / 5 |
Progressive Disclosure | Well-organized into clear section headers, and the heavy implementation is appropriately externalized to real, correctly-referenced bundle files (assets/html-to-pdf.js, scripts/validate_skill.py) at one level deep; minor gaps are the inlined full HTML example and governance boilerplate that could be trimmed or split. | 4 / 5 |
Total | 16 / 20 Passed |