Content
77%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.
Highly actionable with excellent workflow gating and payload coverage, but noticeably padded with known concepts and narrative scenarios, and monolithically inlined where a payload library and citation list would benefit from separate reference files.
Suggestions
Trim the 'Real Impact Examples' narratives and 'Common Root Causes' to essentials, or move them to a reference file, to reduce padding Claude doesn't need.
Split the payload library and disclosed-report citations into reference files under references/ and link to them one level deep, improving progressive disclosure.
Promote the buried docs/verification/phase2g-saml-mfa-xxe.md reference into a clearly signaled 'Verification notes' section so the lxml evidence is discoverable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Core material (payloads, parser-ecosystem matrix, methodology, gates) is signal-dense, but several sections are padded with concepts Claude already knows — the bounty-dollar preamble, three narrative 'Real Impact Examples', and 'Common Root Causes' (e.g., copy-paste from StackOverflow). | 3 / 5 |
Actionability | Copy-paste-ready XML payloads, curl commands, grep patterns, concrete URL/header patterns, and specific fingerprint signals cover the common XXE cases fully. | 5 / 5 |
Workflow Clarity | A 10-step methodology is clearly sequenced with explicit validation checkpoints — the Pre-Severity inline-entity probe, the OOB-Or-It-Didn't-Happen feedback loop (no reflection → pivot OOB → error-based fallback), and the 3-question Gate 0 checklist before reporting. | 5 / 5 |
Progressive Disclosure | No bundle files exist (references/scripts/assets empty), yet the single ~415-line SKILL.md inlines content that belongs in separate files (full payload library, 6 report citations, parser matrix); the one external doc reference (docs/verification/phase2g-saml-mfa-xxe.md) is buried in a table cell rather than clearly signaled. | 3 / 5 |
Total | 16 / 20 Passed |