Content
67%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 a genuinely actionable overview: executable commands, a clear mode-selection decision table, a numbered wait-mode workflow, and a real one-level-deep reference to an existing script. The main drag is the Preferences section, where two ASCII box tables and a circular 'Extension Support' pointer burn tokens restating what a bash snippet already shows.
Suggestions
Replace the two ASCII box-drawing tables in the Preferences section with the bash check plus two short bullets (paths and found/not-found actions), cutting ~16 padded lines that duplicate existing content.
Delete the 'Extension Support' section, which only re-points to the Preferences section, or merge it into the Preferences heading as a single line.
Make the wait-mode stdin step concrete, e.g. show how to send the newline (printf '\n' | npx -y bun ${SKILL_DIR}/scripts/main.ts <url> --wait or an interactive example), closing the last executable gap.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Most sections are tight (options, capture modes, and env vars are compact tables; troubleshooting is one dense line), but the two ASCII box-drawing tables (~16 padded lines) in the Preferences section restate information the preceding bash block and one-line bullet already convey, and the 'Extension Support' section merely points back to the Preferences section. This matches 'mostly efficient but includes some unnecessary explanation or could be tightened' rather than the minor-trim-only 4 anchor. | 3 / 5 |
Actionability | Fully concrete commands are provided ('npx -y bun ${SKILL_DIR}/scripts/main.ts <url> -o output.md') with a script-reference table, an options table, and a 3-step wait-mode workflow; the ${SKILL_DIR} placeholder is explicitly resolved in the Script Directory instructions. Below 5 only because the wait-mode step 'Send newline to stdin to trigger capture' lacks a concrete example of how to do that (e.g. the exact shell incantation), leaving one executable gap. | 4 / 5 |
Workflow Clarity | Usage modes are clearly sequenced with a decision table ('Auto ... Public pages, static content' vs 'Wait ... Login-required, lazy loading, paywalls'), the wait workflow is a numbered 3-step sequence, and there is error-recovery guidance (conflict resolution via timestamp suffix; troubleshooting line for Chrome-not-found and timeout). It is not a destructive/batch operation, so the validation cap does not apply; it stops short of 5 because there is no explicit verification step that the output file was written correctly. | 4 / 5 |
Progressive Disclosure | The body is a well-organized overview whose only bundle reference is 'scripts/main.ts', which exists in the bundle (scripts/ contains main.ts, cdp.ts, constants.ts, html-to-markdown.ts, paths.ts) — a single-level, clearly signaled reference with the supporting scripts correctly kept out of view. The redundant 'Extension Support' section and the inlined ASCII preference tables (which could be trimmed or moved to an EXTEND.md-style reference) keep it at the minor-organization-gaps 4 anchor rather than 5. | 4 / 5 |
Total | 15 / 20 Passed |