Content
27%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is an encyclopedic reference dump — verbose, largely conceptual, and monolithic with broken wiki references — containing only scattered executable snippets and partial workflows rather than a lean, actionable, well-split skill. The description is strong, but the content wastes the context window on knowledge Claude already has.
Suggestions
Replace the ~1760-line reference exposition with a concise overview and split the PCIe/IOMMU/FPGA/pcileech details into separate reference files under references/, linking one level deep instead of inlining everything in SKILL.md.
Remove or relocate concepts Claude already knows (PCIe TLP format, CAM/ECAM, MSI/MSI-X basics, PCR allocation) and keep only the domain-specific, non-obvious guidance and executable commands.
Verify the referenced wiki paths (wiki/index.md, wiki/AGENTS.md, wiki/overviews/dma-attack.md) actually exist as bundle files, or drop the dangling references; consolidate the executable commands and snippets into a single actionable quick-start section.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~1760-line body is dense reference exposition — PCIe protocol tables, capability offsets, IOMMU/VT-d internals, PCR allocations — largely concepts Claude already knows and padded with explanatory prose, matching the verbose/explains-known-concepts anchor rather than the mostly-efficient anchor at 2. | 1 / 3 |
Actionability | Scattered concrete items exist (a pcileech_read_phys snippet, a TranslateVA C function, `lspci -d [VID:DID] -vvv -xxxx`, `setpci -s <BDF> ECAP_AER+0x4.L`, archive URL formats), but they are sparse islands in mostly descriptive prose and not copy-paste ready as a whole, fitting the some-concrete-guidance-but-incomplete anchor. | 2 / 3 |
Workflow Clarity | Some sequenced pipelines with checkpoints appear (Layered Detection Pipeline, Multi-Signal Correlation, "containment protects the live match"), but large sections are flat catalogs (16-row bypass catalog, firmware tier table) with no sequence or validation, so checkpoints are present in places and missing in others. | 2 / 3 |
Progressive Disclosure | It is a single monolithic SKILL.md with no bundle files (references/scripts/assets absent) and inline reference content that should be split out; the referenced `wiki/index.md`, `wiki/AGENTS.md`, and `wiki/overviews/dma-attack.md` paths do not exist as actual bundle files, matching the monolithic-wall-of-text anchor rather than the some-structure anchor at 2. | 1 / 3 |
Total | 6 / 12 Passed |