Content
93%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-crafted instruction skill body: lean, concrete, and sequenced, with real one-level-deep reference files and a clear when-to-read table. The single weakness is the absence of an explicit verification/recovery loop in the Capture phase for this batch-style exhaustive crawl.
Suggestions
Add an explicit per-route verification checkpoint in the Capture Phase, e.g. "Before moving to the next route, confirm every applicable state in capture-checklist.md has a captured screenshot" — this closes the validate loop for the batch capture operation.
Add brief error-recovery guidance for common capture failures (route fails to load, a state can't be reached, screenshot comes back blank) so the workflow has a feedback loop rather than only happy-path instructions.
State upfront what to do when discovery finds far more routes than the chosen scope covers (e.g., re-confirm scope with the user vs. capture everything), since the exhaustive default can silently turn into a multi-hour run.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense operational guidance throughout — exact viewport numbers ("Pin 1440×900 as baseline… capture 768 and 375"), state/trigger tables, path conventions — with no explanation of concepts Claude already knows. Only two or three short rhetorical flourishes ("One 3-second GIF is worth a paragraph of description") and each justifies a behavior. Not 4: there is no substantive passage to trim. | 5 / 5 |
Actionability | Fully concrete and executable: a copy-paste command ("img-process batch ./screenshots --action optimise --max-width 1440"), exact output paths and a file-tree for "docs/ux-extracts/<app-name>/", a 16-row state/trigger capture table, verbatim copy examples, and interaction-capture fields (trigger/target/feedback/reversibility). Not 4: the common cases are covered end-to-end from setup to output. | 5 / 5 |
Workflow Clarity | Setup → Discovery → Capture → Dedup → Output is clearly sequenced, with per-route checkpoints ("Run at the end of each route's capture, or across the whole screenshots/ folder at the end") and a referenced per-pattern checklist plus incremental building ("open the library file at the start, append as you go"). Not 5: there is no explicit validate-and-retry loop — e.g. confirming every applicable checklist state was captured per route before moving on, or what to do when a route fails to load or a state can't be reached. Not 3: checkpoints and a checklist instrument are present and wired into the phases. | 4 / 5 |
Progressive Disclosure | The body stays an operational overview while detail lives one level deep in real, verified files: references/capture-checklist.md and references/pattern-library-template.md both exist and match their claimed purpose, linked both contextually ("See references/capture-checklist.md for the full per-pattern checklist") and in a "Reference files" when/read table; the third link targets sibling skill ux-audit and is clearly labeled. Not 4: no reference material is inlined that belongs in the bundle files — the body tables are summaries and the files hold the full checklists/template. | 5 / 5 |
Total | 19 / 20 Passed |