Content
92%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 strong, mature skill body: a clearly staged five-step workflow with genuine validation gates, copy-paste-ready commands and artifacts, and clean one-level-deep disclosure into an existing, well-populated references/ bundle. The only real cost is length — a modest amount of duplicated cross-referencing and worked-example material could be trimmed or pushed to references to tighten token usage.
Suggestions
Trim the top 'Cross-references' paragraph, which largely duplicates the Step 4 handoff table and the 'Not for' list; keep one canonical routing table and drop the rest.
Reduce the Step 5 verdict examples from two to one per verdict (keep the most illustrative, e.g. the CC8.1 branch-protection confirmed case) and move additional exemplars into references/findings-format.md.
Consider moving the sample-size and test-method tables (Step 3) into the framework reference files, keeping only a pointer plus the one-line rule that inquiry alone never evidences a control.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~290-line body is dense and almost entirely domain-specific (scope tables, sample-size tables, fingerprint rules, verdict exemplars) with essentially no explanation of concepts Claude already knows, so it sits above anchor 3. It falls short of anchor 5's 'every token earns its place' only in minor places: the cross-reference paragraph near the top duplicates the Step 4 handoff table, and the full report template plus two worked examples per verdict could be trimmed by roughly a fifth without losing guidance. | 4 / 5 |
Actionability | Guidance is fully executable throughout: exact commands (`gh api repos/<owner>/<repo>/branches/main/protection`, `grep -rnE "required_approving_review_count|enforce_admins" --include='*.tf' .`), named output files with field-by-field tables (`evidence-requests.md`, `control-matrix.md`), literal fingerprint formats with five examples, and a runnable validator (`node references/validate-findings.mjs findings.json`). Anchor 5 ('copy-paste ready; specific examples cover the common cases') fits; anchor 4 would require minor gaps in the concrete detail, and none stand out. | 5 / 5 |
Workflow Clarity | Five explicitly sequenced steps (Scope → inventory → evidence requests → test/hand off → verify then record) with real validation checkpoints: "run the refutation pass in references/verification.md before anything is confirmed" and "Validate it and fix every error: node references/validate-findings.mjs findings.json", plus a re-run feedback loop (match by fingerprint, re-sample per new period). Anchor 5 ('clear sequence with explicit validation steps; feedback loops') matches; the destructive/batch cap does not apply because validation is present. | 5 / 5 |
Progressive Disclosure | Framework bulk is externalized one level deep into three clearly linked reference files (soc2.md, iso27001.md, nist-800-53.md, all present in references/), shared formats live in findings-format.md / verification.md / severity-calibration.md / validate-findings.mjs / to-sarif.mjs, all of which exist in the bundle, and the body keeps only the overview, workflow and output spec. Anchor 5 ('clear overview with well-signaled one-level-deep references; content appropriately split') fits; there is no multi-level nesting and no buried or inlined content that clearly belongs in a reference file. | 5 / 5 |
Total | 19 / 20 Passed |