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.
A well-structured, dense attack playbook with copy-paste payloads, concrete HTTP examples, and a clear map→attack→evidence workflow. Weaknesses are minor: some editorializing and known-concept restatement, placeholder-heavy request examples, no explicit false-positive/impact validation step, and everything inlined in one file rather than splitting reference material out.
Suggestions
Add an explicit validation step to the PoC pattern (e.g., 'confirm a finding by completing the token exchange in the attacker session before reporting; discard deviations that do not reproduce'), which would also address false-positive risk from batch Intruder fuzzing.
Trim editorial framing ('the single highest-paying ATO vector...') and the restated definitions of state/PKCE; the exemplar list and severity table could move to a references/ file to slim the main SKILL.md.
Provide one fully-populated end-to-end example (authorize URL → token POST with real-shaped values) instead of only placeholder request bodies.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and efficient — payload tables, terse attack sections, and concrete HTTP examples dominate, with no concept tutorials. Minor trimmable content exists: editorializing ('the single highest-paying ATO vector'), brief restatements of known concepts (what state binding and PKCE are), and mild redundancy between the ATO chain in section 2.9 and the PoC pattern. Not 5 (a few tokens don't earn their place), not 3 (padding is minor, not a pattern). | 4 / 5 |
Actionability | Mostly executable guidance: a runnable curl command for .well-known/openid-configuration, a copy-paste redirect_uri bypass payload table, concrete POST /token request bodies, and a systematic PoC procedure naming the exact parameters to fuzz. Minor gaps: several examples use template placeholders (client_id=..., code=<stolen_code>) and there is no complete end-to-end exchange example. Not 5, not 3 — well above 'pseudocode' level. | 4 / 5 |
Workflow Clarity | A clear sequence is present: map the flow (section 1) → attack surface (section 2) → a numbered 4-step PoC pattern with an evidence-capture checkpoint ('on any deviation ... capture as evidence') → severity calibration (section 5). Not 5 because there is no explicit validation step for confirming impact or ruling out false positives before assigning severity; not 3 because checkpoints are present, not merely implicit. | 4 / 5 |
Progressive Disclosure | Good structure: numbered sections, a dedicated Cross-references section with clearly signaled one-level-deep pointers, and a Tools section. Not 5 because it is a single ~157-line monolithic file (above the under-50-line simple-skill exception) with content such as the severity table and known-exemplar lists that could live in a reference file, and the cross-referenced paths point outside this bundle (no references/, scripts/, or assets/ exist). Not 3: structure and navigation are solid. | 4 / 5 |
Total | 16 / 20 Passed |