Content
65%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The skill is highly actionable with executable code throughout and a clear six-step workflow, but it is held back by verbosity (the Key Concepts table), absent validation checkpoints, and a monolithic structure that ignores its own bundle files. Tightening inline reference material and linking the existing api-reference.md and agent.py would lift conciseness and progressive disclosure.
Suggestions
Add explicit validation/verification checkpoints to the workflow — e.g., after Step 3 confirm the decoded command set against Ghidra disassembly, and after Step 6 test Suricata rules against the source PCAP before finalizing.
Reference the existing bundle files from the body (references/api-reference.md for scapy/dpkt/tshark/JA3 API detail, scripts/agent.py for the beacon-detection implementation) instead of inlining all of that material.
Trim the Key Concepts table to only C2-specific, non-obvious definitions and consider moving the framework-signature and Tools & Systems tables into a reference file to reduce the inline token load.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient operational code, but the 'Key Concepts' table explains largely known terms (beaconing, jitter, malleable C2, domain fronting) and inline reference material could be tightened; not the lean level-3 ideal yet above verbose level 1. | 2 / 3 |
Actionability | Provides fully executable, copy-paste-ready code (scapy beacon analysis, dpkt HTTP decoding, requests-based Shodan/VirusTotal enrichment, CobaltStrikeParser usage, and complete Suricata rules), matching the executable-and-complete anchor. | 3 / 3 |
Workflow Clarity | Steps 1–6 are clearly sequenced and a numbered scenario approach exists, but there are no explicit validation/verification checkpoints (e.g., confirming a decoded protocol against disassembly or verifying a signature fires), so it sits at the sequence-present-but-checkpoints-implicit level rather than level 3. | 2 / 3 |
Progressive Disclosure | Sections are reasonably organized, but the ~370-line body is a monolithic wall with content that belongs in references, and the existing bundle files (references/api-reference.md, scripts/agent.py) are never referenced from the body — references present but not clearly signaled. | 2 / 3 |
Total | 9 / 12 Passed |