Content
63%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.
The skill body is exceptionally actionable - every step is executable copy-paste code with validation checkpoints and error recovery - but it is a 60KB monolith with no progressive disclosure. Subcommand manuals and repeated boilerplate belong in separate reference files, and the interpreter-resolution preamble is duplicated across nearly a dozen code blocks.
Suggestions
Move the subcommand manuals (query, path, explain, add, watch, git commit hook, CLAUDE.md integration) into separate reference files (e.g. references/query.md, references/add.md) and keep only a one-line pointer plus the common pipeline in SKILL.md, so 60KB is not loaded into context for every invocation.
Deduplicate the repeated 'load extraction/analysis -> build_from_json -> communities/labels' preamble that appears in Steps 4, 5, 6, 6b, 7, 7b, and 7c - factor it into one script (e.g. scripts/rebuild.py) invoked with arguments, or state the preamble once and reference it.
Fix the ordering confusion in the --update section (the 'cp graphify-out/graph.json graphify-out/.graphify_old.json' backup instruction appears after the code block that reads it) and reconcile the inconsistent Part/Step naming ('Step 3B' vs 'Part B2', inner 'Step 1/Step 2' inside Step 2.5).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is ~1400 lines / 60KB with substantial padding: roughly ten code blocks repeat the same 'resolve interpreter -> load extraction/analysis -> rebuild graph' preamble nearly verbatim (Steps 4, 5, 6, 6b, 7, 7b, 7c all reload and rebuild the same objects), and the usage block lists 28 flag variants inline. It avoids explaining concepts Claude already knows (so not 1), but the duplicated boilerplate and inlined subcommand manuals are noticeably verbose per the anchor 2 example. | 2 / 5 |
Actionability | Fully executable guidance throughout: copy-paste bash blocks with explicit placeholder-substitution instructions ('Replace INPUT_PATH with the actual path', 'Replace LABELS_DICT with the actual dict'), an exact verbatim subagent prompt with a complete JSON output schema, and concrete error-handling paths for every failure mode. This matches the anchor for copy-paste ready commands covering common cases. | 5 / 5 |
Workflow Clarity | Steps 0-9 are explicitly sequenced ('Follow these steps in order. Do not skip steps.') with real validation and feedback loops: empty-graph abort, missing-chunk detection with re-run guidance, a corpus-size confirmation gate that waits for the user, and extraction-cache checks. However, there are noticeable sequencing quirks: the 'save the old graph before merge' instruction appears after the code block that consumes it, Step 2.5 contains its own inner 'Step 1/Step 2' labels, naming is inconsistent ('Step 3B' vs 'Part B2'), and the cross-repo merge flow leaves post-merge outputs undefined. This lands above the midpoint but below the anchor-5 standard of fully coherent checkpoints. | 4 / 5 |
Progressive Disclosure | There is genuine in-file structure (numbered steps, clear ## section headers, a usage block), so navigation is possible, but no bundle files exist at all - references/, scripts/, and assets/ are absent - and hundreds of lines of subcommand manuals (query, path, explain, add, watch, git hook, CLAUDE.md integration) are inlined in the single SKILL.md where on-demand reference files belong. This matches anchor 3: some structure, but content that should be separate is inline. | 3 / 5 |
Total | 14 / 20 Passed |