Content
77%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is highly actionable with a well-sequenced, gated three-phase workflow and concrete code. Its weaknesses are token efficiency (heavy duplication of the same hardening examples and some concept explanation) and a monolithic structure with no progressive disclosure of the large pattern catalogs.
Suggestions
De-duplicate the per-language hardening code: keep one canonical catalog (e.g. Vulnerable vs. Secure Examples) and have the Patterns and Execution sections reference it rather than re-listing the same Java DOM / lxml / .NET / PHP snippets.
Move the large language-by-language pattern lists into a references file (e.g. references/xxe-patterns.md) and have SKILL.md summarize parser-default behavior and link out, so the main body stays a lean overview.
Trim the 'What is XXE' conceptual explanation to a one-line definition plus the core pattern, since Claude already understands the XXE attack class.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The same hardening code recurs roughly three times (Patterns That Prevent XXE, Vulnerable vs. Secure Examples, and the Execution recon list — e.g. Java DOM, lxml, .NET, and PHP each repeated), and the 'What is XXE' section explains a concept Claude largely already knows; not the verbose-walled level 1, but not the lean level 3 either. | 2 / 3 |
Actionability | Provides executable per-language code with exact feature flags and copy-paste-ready curl payloads in the output templates, fully matching the 'executable, copy-paste ready' anchor. | 3 / 3 |
Workflow Clarity | Three phases are explicitly sequenced with an between-phase validation gate (zero-findings early exit writes 'No vulnerabilities found.'), parallel batching rules, merge-after-all-complete ordering, and an 'Important Reminders' checklist; the batch operation includes validation so the level-2 cap does not apply. | 3 / 3 |
Progressive Disclosure | No bundle files exist and the ~550-line body is monolithic, with large language-by-language pattern catalogs that could be split into reference files; sections are well-organized (not level 1) but content that should be separate is inline (not level 3), and the under-50-line exception does not apply. | 2 / 3 |
Total | 10 / 12 Passed |