Content
76%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.
An unusually actionable single-file workflow: every step pairs a concrete search with concrete, executable fixes, and closes with a fixed-vs-remaining report. The two real gaps are the missing verification loop after batch edits, and a monolithic length that could be slimmed via a reference file.
Suggestions
Add a verification step after the fixes: re-run the Step 2–6 greps expecting zero hits, re-run `pnpm audit --audit-level=high`, and confirm the app builds — with an instruction to loop back and fix anything still found.
Move the per-pattern fix recipes (DOMPurify, Zod, setTimeout, header config details) into a `references/fixes.md` and keep SKILL.md as the step-by-step search-and-decide workflow, which would improve both conciseness and progressive disclosure.
Tighten Step 5: the Atlas/agent bullet list is terse to the point of being decodable only with outside context ('5 / max 50, cached') — either expand these into concrete checks or drop them.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is largely lean — grep commands, a pattern/action table, before/after code snippets, and terse fix instructions with no padding about what XSS or CSP are. A few tokens could be trimmed (the repeated '### How to fix' scaffolding, and reminders of things Claude already knows like 'Never pass user-controlled strings to code evaluation'), which keeps it at 4 rather than 5. | 4 / 5 |
Actionability | Every step ships copy-paste-ready material: three complete grep command blocks, a DOMPurify before/after JSX example, a setTimeout string-to-arrow rewrite, a full Zod schema with safeParse, a literal `server.headers` config block, and exact pnpm commands. Together these cover the common cases for each step with no pseudocode. | 5 / 5 |
Workflow Clarity | Steps 1–9 are clearly numbered and sequenced with an attack-surface map up front and a fixed/remaining report at the end, which is above the level-2 anchor. However this is a batch operation that rewrites code across the whole app, and there is no verification loop — the skill never instructs re-running the greps or `pnpm audit` to confirm zero remaining findings, or building/testing after the edits, so the batch-operation cap of 3 applies. | 3 / 5 |
Progressive Disclosure | A single-file skill with clean section structure, well-anchored headers, and no external references — and the bundle has no reference files to navigate, so no references are buried or nested. It stays at 4 rather than 5 because the ~240-line body could split its per-pattern fix recipes (Steps 2–7) into a reference file, keeping SKILL.md as the workflow skeleton. | 4 / 5 |
Total | 16 / 20 Passed |