Content
81%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 strong, high-signal audit playbook: every section carries concrete patterns, exact checks, executable search commands, and a working Foundry PoC template, with a clear four-step audit workflow and explicit validation recipe. The main gaps are the undefined foundry_inflation_test pseudo-invocation, placeholder values in the PoC, and a monolithic layout that could offload the PoC template and exemplars to reference files.
Suggestions
Define or implement foundry_inflation_test (e.g., as a scripts/ tool) or replace it with the concrete Foundry test code it stands for, since the current invocation has no backing in the bundle.
Move the PoC template and known-exemplars list into a references/ file (e.g., references/poc-template.md and references/exemplars.md), keeping SKILL.md as a tighter overview with clearly signaled one-level-deep links.
Tighten the intro and OPSEC sections: drop the one-line explanation of what a flash loan is and reduce the watcher/dashboard narration to the operative instruction (use a forked anvil / private mempool for PoCs).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and efficient — attack patterns, checks, and a PoC template with almost no padding of concepts Claude already knows. It misses anchor 5 on minor points: the opening line explains what a flash loan is, and the OPSEC section spends a few lines on narration ('this is on-chain', listing watcher dashboards) that could be trimmed to a single directive. | 4 / 5 |
Actionability | Mostly executable guidance: a concrete grep command ('grep -rn 'executeOperation\|flashLoan\|onFlashLoan...' src/'), per-handler check items with exact expressions ('amounts[i] + premiums[i] is repaid'), a Foundry PoC template, and a severity table. It stops short of anchor 5 because 'foundry_inflation_test(vault_address="...", ...)' is an undefined invocation with no backing script in the bundle, and the PoC template uses placeholders (TARGET = 0x..., fork URL placeholder, WETH never declared/imported). | 4 / 5 |
Workflow Clarity | The audit workflow is a clear sequence — locate integrations via grep, check each handler against an explicit checklist, delegate price functions, then validate share math — and includes a fully specified validation procedure with expected outcomes ('Victim deposits 1e18 → gets 0 shares', 'Attacker withdraws 1 share → gets 100% of pool') plus a severity calibration table for triage. This matches anchor 5's clear sequence, explicit validation steps, and checklists; the skill is analysis/PoC work, so no destructive-operation cap applies. | 5 / 5 |
Progressive Disclosure | Good structure: well-organized sections, and cross-references to sibling skills ('see oracle-manipulation/SKILL.md') are clearly signaled and one level deep. It does not reach anchor 5 because at ~180 lines the skill is monolithic — the ~50-line PoC template and the exemplars list could live in reference files, keeping SKILL.md as a tighter overview — and no bundle files exist to split content across. | 4 / 5 |
Total | 17 / 20 Passed |