Content
56%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This skill is remarkably thorough and highly actionable — every step has complete, executable code with proper error handling and validation checkpoints. However, it is severely over-long for a single SKILL.md file, with ~800+ lines of inline implementation that should be split across multiple referenced files (subagent prompts, subcommand handlers, export formats). The workflow is well-sequenced but the monolithic structure makes it difficult to navigate and wastes context window budget.
Suggestions
Split subcommand implementations (query, path, explain, add, --update, --watch, --cluster-only) into separate referenced files (e.g., QUERY.md, UPDATE.md, EXPORTS.md) and link from the main SKILL.md with one-line descriptions
Extract the subagent extraction prompt (Step B2) into a separate file like SUBAGENT_PROMPT.md — it's ~80 lines that rarely change and dominates the skill body
Move export format handlers (Neo4j, SVG, GraphML, MCP, wiki) into an EXPORTS.md reference file since they are conditional/optional steps
Reduce repeated boilerplate across code blocks — the pattern of loading JSON files, building graph, reading analysis is repeated 10+ times and could reference a shared setup snippet
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The skill is extremely long (~800+ lines) with massive amounts of inline code that could be in separate files. Many code blocks repeat the same boilerplate (loading JSON, building graph, reading analysis). The subagent prompt alone is ~80 lines embedded inline. While the content is mostly non-redundant in terms of concepts, the sheer volume of inline implementation detail is far beyond what belongs in a single SKILL.md. | 2 / 5 |
Actionability | Every step has fully executable, copy-paste-ready code blocks with specific Python imports, function calls, file paths, and error handling. The subagent prompt is complete with exact JSON schema. Flag handling, edge cases (empty graph, code-only corpus, large corpus warnings), and error recovery are all concretely specified. | 5 / 5 |
Workflow Clarity | The multi-step pipeline is clearly sequenced (Steps 0-9) with explicit validation checkpoints (empty graph check after Step 4, cache checks before extraction, graph existence checks before queries). There are feedback loops for error recovery (subagent failure handling, re-validation). Minor gap: no explicit validation after the merge step in --update mode to confirm the merged graph is consistent, and the Step 2 detection says 'proceed to Step 2.5 if video files were detected, or Step 3 if not' but earlier says 'proceed directly to Step 2' which is slightly confusing. | 4 / 5 |
Progressive Disclosure | This is a monolithic wall of content with no references to external files despite being ~800+ lines. The subagent prompt, all subcommand implementations (query, path, explain, add, watch), export variants (Neo4j, SVG, GraphML, MCP), and incremental update logic are all inlined. No bundle files are provided. Content like the subagent extraction prompt, the query/path/explain subcommands, and export format handlers clearly belong in separate referenced files. | 1 / 5 |
Total | 12 / 20 Passed |