Content
78%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 dense, highly actionable hunting playbook whose commands, PoCs, and patterns are copy-paste ready. Its main structural weakness is that it is monolithic — report template, tools table, and per-class detail would earn their keep as separate reference files — and it lacks explicit verification checkpoints in the hunt workflow.
Suggestions
Split the report template (§10) and tools reference (§12) into one-level-deep reference files (e.g. references/report-template.md, references/tools.md) and link them from a concise SKILL.md overview, trimming the SKILL.md body accordingly.
Add an explicit verification step to the hunt loop — e.g. 'confirm the DNS/HTTP callback (or log line) proves execution before writing the report' — to turn the checklist into a validated workflow.
Trim the 'What It Is' / 'Why It's Critical' prose blocks (§2, §3, §5) to one-line reminders; the vulnerable/safe pattern pairs already convey the mechanism.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The bulk is high-signal commands, grep patterns, PoC payloads, and vulnerable/safe YAML pairs, but a few blocks restate knowledge Claude already has — e.g. "What It Is: GitHub Actions exposes PR/issue data as context variables" and the "Why It Matters" / "Why It's Critical" paragraphs. Not 5 because those over-explanatory prose blocks could be trimmed; not 3 because they are minor relative to the dense actionable majority. | 4 / 5 |
Actionability | Fully executable throughout: copy-paste grep commands ("grep -rn '\${{.*github\.event\.(pull_request\|issue\|comment\|review\|discussion)' .github/workflows/"), concrete PoC payloads, side-by-side vulnerable/safe YAML, a ready-made report template with CVSS string, and exact tool invocations. Not 4 because the common cases (injection, pull_request_target, secrets-in-logs, self-hosted runners, OIDC) each have specific runnable detection code with no gaps. | 5 / 5 |
Workflow Clarity | There is a clear sequence — the numbered "QUICK KILL CHECKLIST" front-loads the hunt order, each bug class has a detection-then-PoC flow, and the report template closes the loop; scope notes provide explicit safety checkpoints ("Never trigger a workflow that could affect production infrastructure without explicit written permission"). Not 5 because there are no explicit verification checkpoints in the hunt loop (e.g. confirming the DNS/HTTP callback landed before drafting the report), leaving validation mostly implicit. | 4 / 5 |
Progressive Disclosure | The 12 numbered sections with headers make the file navigable, and no bundle files exist (references/, scripts/, assets/ are absent), so everything is inlined in one ~380-line SKILL.md. Not 4 because content that clearly belongs in separate one-level-deep reference files — the report template, the tools reference table, and the per-bug-class deep dives — is all inline; not 2 because section structure is genuinely good and the top checklist serves as an effective overview. | 3 / 5 |
Total | 16 / 20 Passed |