Content
67%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 well-structured, highly actionable body with concrete queries, tool tables, and a clear tiered workflow. The main weaknesses are a padded acronym-glossary diagram that explains concepts Claude already knows, and CodeQL examples that are not fully self-contained.
Suggestions
Cut the ASCII box defining AST/CFG/CDG/DDG/PDG — Claude already knows these acronyms — and drop or compress the Purpose paragraph that repeats the frontmatter description.
Make the second CodeQL example executable: add the missing import and replace the undefined "RedirectSink" with a real sink class or a comment noting it must be defined per codebase.
Embed the CPG-status check as an explicit validation step inside the combined workflow (e.g., "0. VERIFY: get_cpg_status before any Tier 2 query") rather than only mentioning it in the anti-patterns table.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient (dense tables, concrete queries), but includes unnecessary explanation Claude already knows — the ASCII box defining "AST = Abstract Syntax Tree (structure)", "CFG = Control Flow Graph (execution paths)", etc. — and a Purpose paragraph that restates the frontmatter. This matches anchor 3's "mostly efficient but includes some unnecessary explanation or could be tightened"; it is not 4 because the glossary diagram and purpose paragraph are clearly trimmable padding. | 3 / 5 |
Actionability | Concrete, executable guidance throughout: copy-paste CPGQL queries ("cpg.method(\"login\").callOut.code.l"), full CodeQL taint queries, MCP config JSON, and install commands ("~/.claude/install-graph-tools.sh --joern"). It falls short of anchor 5 because of minor gaps — the second CodeQL example omits its import and uses an undefined "RedirectSink" class, so it is not directly executable as written. | 4 / 5 |
Workflow Clarity | The combined workflow gives a clear five-step sequence (SCOPE → STRUCTURE → FLOW → SECURITY → REPORT) with per-tier tools, and the anti-patterns table supplies checkpoints like "Always verify CPG is built and current before querying" (get_cpg_status) and "scope with Tier 1 first". Not anchor 5 because validation is stated as guidance rather than embedded as explicit validation steps in the workflow itself — minor gaps matching anchor 4. | 4 / 5 |
Progressive Disclosure | The body is well-organized with clear sections (tier selection, per-tier tool tables, queries, combined workflow, anti-patterns), and no bundle files exist so all content lives inline appropriately for a ~220-line skill. It matches anchor 4's "good structure; most content is appropriately placed; minor organization gaps" — the query catalogs and MCP config snippets could arguably be split into reference files, keeping it below anchor 5. | 4 / 5 |
Total | 15 / 20 Passed |