Content
77%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.
An unusually actionable and well-sequenced orchestration skill with strong validation loops, escalation rules, and a per-finding fix dispatch table. Its main weaknesses are token efficiency and file structure: the Mode A checks are restated as invariants in the same body, and the invariants reference plus caps table should be moved to reference files with the body keeping a summary and links.
Suggestions
Move the full Invariants reference section (~140 lines) into a references/ file (e.g. references/graph-invariants-summary.md) and keep only the invariant IDs and one-line descriptions in SKILL.md, since the body already declares graph-invariants.md as the source of truth.
De-duplicate the Mode A "8 quick-check items" against the Invariants reference by describing each check once and cross-referencing its invariant ID (e.g. "Dead links (E-1)"), cutting the body by roughly 80-100 lines.
Move the Caps table (a declared mirror of project-conventions' canonical specs) into a reference file and have the script's ARTIFACT_CAPS documented there, keeping only the tolerance rule and the link in SKILL.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is ~640 lines with no padding about concepts Claude already knows, but it duplicates content: the Mode A "8 quick-check items" are restated nearly verbatim in the Invariants reference (e.g. dead links / E-1, backlog completeness / E-12, ADR references / E-3), and the Caps table is an acknowledged mirror of another file ("the table below is a mirror and must be kept hand-aligned"). The templated console-report block with {n} placeholders could also be tightened. | 3 / 5 |
Actionability | Guidance is fully concrete and executable: exact glob patterns for node discovery ("_devprocess/requirements/epics/EPIC-*.md"), exact frontmatter fields to parse ("feature-refs, adr-refs, fix-refs, imp-refs"), a per-finding-type fix dispatch table, exact marker syntax ("FIXME(stub): <reason> -- see FIX-{ee}-{ff}-{nn}"), and copy-paste commands ("bash <DIA-checkout>/tools/install-git-hooks.sh"). | 5 / 5 |
Workflow Clarity | The six-step Workflow is clearly sequenced with an abort condition if structure detection fails, and Mode C is a genuine feedback loop: load -> triage -> per-finding fix -> re-check (Mode A re-run) -> commit confirmation -> follow-up, with AskUserQuestion checkpoints at every step. Validation for risky batch behavior is explicit (escalation on shared-infrastructure or substance-deleting fixes, non-auto-fixable findings left for human triage) and a quality-gates checklist guards completion. | 5 / 5 |
Progressive Disclosure | Section structure is clear, but content that clearly belongs in reference files is inlined: the full ~140-line invariants reference (whose stated source of truth is skills/project-conventions/references/graph-invariants.md) and the mirrored caps table both live in the SKILL.md body. External file pointers are plain code paths rather than clearly signaled navigable references, and no bundle files (references/, scripts/, assets/) exist to offload this detail. | 3 / 5 |
Total | 16 / 20 Passed |