Content
70%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 delivers an unusually rigorous, actionable audit workflow: sequenced phases, calibrated severities, an extensive false-positive filter, independent verification, and a precise report schema with trend tracking. Its weaknesses are structural: ~150 lines of framework boilerplate dilute token efficiency, and the whole 15-phase methodology lives in one monolithic file with no progressive disclosure via reference files.
Suggestions
Move the framework boilerplate (preamble plumbing, Voice/persona, telemetry onboarding, effort tables, contributor mode, plan-status footer) out of SKILL.md or into a shared base file so the skill's context budget is spent on audit methodology.
Split reference-worthy content into one-level-deep bundle files — e.g., references/owasp-checks.md for Phase 9 category details, references/fp-rules.md for the 22 hard exclusions and precedents, and references/report-schema.md for the Phase 14 JSON schema — keeping SKILL.md as a phase-by-phase overview with clear pointers.
Tighten Phase 9–11 actionability by attaching concrete Grep patterns or commands to each prose question (e.g., the rate-limiting and session-management checks), matching the specificity the other phases already have.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The 15 audit phases themselves are dense with non-obvious operational knowledge (FP rules, severity calibration, 22 hard exclusions, precedents) rather than concepts Claude already knows, but roughly 150 lines of gstack framework boilerplate (preamble session/telemetry plumbing, Voice persona, onboarding prompts, effort tables, contributor mode) pad the file without advancing the security task. This lands on the 3 anchor — mostly efficient with unnecessary sections that could be trimmed — rather than 2, because the core audit content explains nothing Claude already knows. | 3 / 5 |
Actionability | Mostly executable: copy-paste git-log/grep/find commands per phase, verbatim sub-agent verifier prompts, exact severity mappings, output templates, and a complete JSON report schema. It is not a 5 because phases 9–11 pose prose questions ("Rate limits on authentication endpoints?", "Is sensitive data encrypted at rest and in transit?") without concrete search patterns or commands, and Phase 1 counting instructions rely on adaptation rather than ready commands. | 4 / 5 |
Workflow Clarity | The 15 phases are explicitly sequenced 0–14 with a mode-resolution ruleset (mutually exclusive scope flags error out, always-run phases named, diff-mode behavior specified), and Phase 12 is a dedicated validation gate with feedback loops: confidence-gate filtering, active verification, variant analysis, parallel independent re-verification with explicit discard thresholds, and escalation after 3 failed attempts. This matches the 5 anchor — clear sequence, explicit validation, error-recovery loops. | 5 / 5 |
Progressive Disclosure | Scored against the actual bundle: there are no references/, scripts/, or assets/ files, so everything — OWASP per-category details, the 22-item hard-exclusion list, the report JSON schema, the incident-response playbook — is inlined in one ~900-line SKILL.md. Internal structure is genuinely good (well-labeled phase sections make it navigable, above the 2 anchor's 'minimal structure'), but content that clearly belongs in one-level-deep reference files is fully inlined, matching the 3 anchor's 'content that should be separate is inline'. | 3 / 5 |
Total | 15 / 20 Passed |