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 a well-organized, actionable catalog: every pattern pairs a tight breach summary with concrete audit questions and specific named checks, a clear usage workflow, a defined output format, and explicit boundaries. Weaknesses are moderate: duplicated source lists across two sections, a few checks that stay high-level or defer entirely to sibling skills, and a monolithic ~230-line structure with no progressive disclosure to reference files.
Suggestions
Deduplicate the post-mortem source lists — 'Good post-mortem sources' and 'References' both list the Verizon DBIR, MITRE ATT&CK reports, Krebs on Security, and Risky.Biz; keep one list and drop the other.
Tighten the handful of high-level checks ('SSRF defences on every URL-taking endpoint', 'CI network egress monitored where feasible') into concrete, verifiable checks like the IMDSv2 and CISA KEV entries, or state exactly which sibling-skill check to run.
Consider moving the per-pattern detail or the reference/source material into a references/ file so SKILL.md stays a lean overview with one-level-deep pointers, which would also shrink the ~230-line inline body.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and functional: each breach gets a one-paragraph summary, bulleted audit questions, and bulleted checks with specifics (CVE numbers, '169.254.169.254', 'MetadataOptions.HttpTokens: required'). It stays above the 3 anchor because padding is minimal and Claude's competence is mostly assumed, but below 5 due to real redundancy — the post-mortem source list (DBIR, MITRE ATT&CK, Krebs, Risky.Biz) appears in both 'Patterns to add' and 'References', and phrases like 'Among the goodies' are chatty. | 4 / 5 |
Actionability | For an instruction-only skill the guidance is highly actionable: named controls (FIDO2/WebAuthn, number matching, SLSA/sigstore, HIBP monitoring), concrete config values and CVEs, specific metadata endpoints, explicit cross-references to the audit skill that owns each deep dive, and a copy-paste output template. Minor gaps keep it at 4 — a few checks remain high-level ('SSRF defences on every URL-taking endpoint', 'CI network egress monitored where feasible') and no runnable verification commands are given inline. | 4 / 5 |
Workflow Clarity | A clear four-step sequence (Read → Ask → Map → Decide disposition) with three named dispositions acting as a decision checkpoint, a defined output artifact, and an explicit escalation boundary ('if a check shows the pattern is happening now, stop and switch to incident-triage'). Not a 5 because there is no validation/verification step for the produced coverage document and no guidance on re-running or verifying that a check's 'Clean' status is actually evidence-based. | 4 / 5 |
Progressive Disclosure | No bundle files exist, so everything is inline, but the body is well-sectioned with a consistent per-pattern structure (What happened / Audit questions / Check) that is easy to navigate, and cross-references to sibling audit skills are clearly signaled at each check. It sits at 4 rather than 5 because the ten inline patterns plus the duplicated source lists make a ~230-line file where some material (per-pattern detail or reference sources) could be split out for a leaner overview. | 4 / 5 |
Total | 16 / 20 Passed |