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 content is actionable and well-sequenced with concrete CLI commands and validation checkpoints, but it is a monolithic wall of text with no bundle files to offload detail, and some sections could be tightened.
Suggestions
Move the full CLI reference, hook-integration JSON, and agent-teams pipeline into separate reference files under references/ and link to them one level deep, turning SKILL.md into a lean overview.
Trim the agent-teams pipeline section, which largely restates the earlier 'Before Any Code Change' workflow.
Add a single end-to-end worked example (create intent → record symbols → drift check) to close the minor actionability gap.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~320-line body is mostly dense reference material and assumes Claude's intelligence (it does not belabor what SQLite/AST/CFG are), but sections like the agent-teams pipeline restate the earlier workflow and some tables could be trimmed, matching the 'mostly efficient but could be tightened' anchor 3 rather than 4. | 3 / 5 |
Actionability | It provides concrete, copy-paste-ready CLI commands ('icpg init', 'icpg query prior "<goal>"'), JSON hook configs, and example outputs covering common cases — mostly executable with only minor gaps, fitting anchor 4; not 5 because no single end-to-end create→record→drift example ties them together. | 4 / 5 |
Workflow Clarity | The 'Before Any Code Change' workflow is an explicit 8-step numbered sequence with validation checkpoints (DRIFT CHECK, VERIFY) and a feedback notion ('resolve or create new intents'), matching anchor 4; not 5 because error-recovery detail for failed drift checks is brief. | 4 / 5 |
Progressive Disclosure | No bundle files exist (references/scripts/assets directories are absent), so the body is a monolithic single file with clear section headers but content that could live in separate files (full CLI reference, agent-teams integration, hook configs) is inlined, fitting anchor 3; in-path references like './scripts/icpg' point to nonexistent files. | 3 / 5 |
Total | 14 / 20 Passed |