Content
85%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-structured, code-forward skill with a genuinely instructive worked example and correct use of a one-level-deep reference file for advanced @page geometry. The two weaknesses are moderate: the overview steps duplicate the detailed sections, and the PDF-inspection helper functions the assertions depend on are undefined.
Suggestions
Define or stub the PDF-inspection helpers (getPdfPageCount, extractTextPerPage, getPdfDimensions, renderPdfPage) once — e.g. a short setup snippet naming the library or the pdf-snapshot-tester functions they map to — so the assertion code is copy-paste ready.
Collapse the 7-item 'How to use' list into a 2–3 sentence overview or fold it into the Step headings, removing the duplication with the detailed Step 1–6 sections.
Consider moving the anti-patterns table's cross-engine row detail into references/page-geometry.md alongside the other pdf-snapshot-tester pairing notes to keep the main file lean.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Prose is tight and limited to non-obvious facts ("preferCSSPageSize: true lets CSS @page { size: A4 } win over the API format option"), but the 7-item "How to use" list substantially duplicates the six detailed Step sections that follow. Matches 4 (efficient with minor trimming possible); not 5 because of that duplication, not 3 because padding is limited. | 4 / 5 |
Actionability | The Playwright structure is fully executable (goto, emulateMedia, page.pdf options, expect assertions), but the PDF-inspection helpers used by the assertions (getPdfPageCount, extractTextPerPage, renderPdfPage, hasBrandColorAtTop) are never defined or imported — only "Use a PDF inspector lib or pair with pdf-snapshot-tester". Matches 4 (mostly executable, minor gaps); not 3 since the core code is real and runnable, not 5 since the page-level assertions are not copy-paste ready. | 4 / 5 |
Workflow Clarity | Clear numbered sequence (How to use 1–7, then Steps 1–6) where every step ends in an assertion, and the Worked example demonstrates an explicit feedback loop: baseline count assertion fails → extractTextPerPage pinpoints the totals table straddling pages 2/3 → re-add break-inside: avoid → re-run passes. Matches 5; not 4 because error recovery is shown end-to-end. | 5 / 5 |
Progressive Disclosure | Verified against the bundle: references/page-geometry.md exists and is clearly signaled ("Advanced @page geometry ... covered in [references/page-geometry.md])") exactly one level deep, with the advanced pseudo-class/margin material appropriately split out of SKILL.md. Matches 5 (clear overview, well-signaled one-level-deep references); not 4 because there are no organization gaps. | 5 / 5 |
Total | 18 / 20 Passed |