Content
53%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-written overview in isolation — clean routing table, tight invariants, and a genuine verification checklist — but it is wired to a bundle that doesn't match it: all five workflow runbook links are dead, and most of the actual reference material is orphaned with no entry point from SKILL.md. The skill functions as a directory to a building whose rooms are all locked and unlisted.
Suggestions
Fix the five dead runbook links: either create the workflows/*.md files or re-point the triage matrix rows at the existing reference files that cover them (e.g. workflows/file.md → references/classification.md + folder-conventions.md; workflows/reconcile.md → references/revisions.md; workflows/archive.md → references/lifecycle.md; workflows/navigate.md → references/navigation.md; workflows/import-records.md → references/source-records.md).
Link the remaining orphaned references (vault-conventions.md, and the ones not covered above) from an explicit 'References' section so every bundle file is reachable from SKILL.md in one hop.
Collapse the duplicated routing — the ASCII lifecycle tree and the triage table convey the same five workflow mappings; keep the table (it adds the intent-matching and output focus) and trim the tree or reduce it to the three phase headings.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is compact (~55 lines), teaches nothing Claude already knows, and uses dense invariants ("evaluate in strict order: Projects (active, finite) → Areas → Resources → Archives") and specific field names ("status, completed, archived-from"). It matches the 'efficient; minor instances that could be trimmed' anchor: the lifecycle ASCII tree and the triage table both map the same five workflows, and the top scope directive partially repeats the frontmatter's negative boundaries. Not 3 because there is no real over-explanation, only this duplicated routing. | 4 / 5 |
Actionability | The triage matrix and invariants are concrete, but the primary executable paths — every "Target Runbook" link ([`workflows/file.md`](workflows/file.md) through [`workflows/import-records.md`](workflows/import-records.md)) — point to files that do not exist in the bundle, so Claude cannot actually execute the routed task. This matches the 'some concrete guidance but incomplete' anchor. Not 4 ('mostly executable with minor gaps'): the gap is not minor — all five runbooks are dead links, and the real detail files that do exist (e.g. revisions.md, source-records.md) are never surfaced. | 3 / 5 |
Workflow Clarity | Routing-level sequencing is clear (three lifecycle phases, intent-to-runbook table) and the Verification section supplies explicit checkpoints for a destructive/batch skill, so the validation cap is not triggered. However, the actual step-by-step workflows live exclusively behind the five nonexistent workflows/*.md links, so the detailed sequences and their per-task checkpoints are unreachable — matching 'steps listed but gaps; checkpoints missing or implicit' at the level where work is actually done. Not 4: a minor validation gap would still leave executable runbooks behind the links. | 3 / 5 |
Progressive Disclosure | Scored against the actual bundle per the guidelines: the body links to 5 workflows/*.md files that do not exist, while 7 of the 8 real reference files (classification, folder-conventions, lifecycle, navigation, revisions, source-records, vault-conventions) are never referenced from SKILL.md — they are buried and undiscoverable. Only references/frontmatter-schemas.md is both real and linked. This matches the 'references are buried' anchor. Not 3: the problem is not inlining — it is that the declared structure does not match the bundle and most detail content is unreachable. Not 1 only because the SKILL.md body itself is cleanly sectioned and one-level linking is attempted; vault-conventions.md's hop to ../../manage-file-operations/core.md is a cross-skill dependency rather than gratuitous nesting. | 2 / 5 |
Total | 12 / 20 Passed |