Content
61%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-structured, honest spec that is strong on per-document requirements and repo-convention respect, with genuinely actionable capture lists. Its weaknesses are repetition (the /derive-tests provenance stated three times), an implicit rather than explicit end-to-end workflow across the sibling commands, and no output-validation checkpoint.
Suggestions
State the tests.md / /derive-tests provenance once (in its doc entry) and drop the repeats in 'How the set is organized' and Notes to reclaim tokens.
Add a short ordered pipeline section (e.g. /document-app produces docs 1-4 and applicable conditional docs → /derive-tests produces tests.md → /ship-check produces the handoff context) so the end-to-end sequence is explicit rather than scattered.
Add a lightweight validation checkpoint after writing (e.g. 're-read each doc's Must-capture list and confirm every item is present, and confirm each conditional doc either exists or has its one-line absence note') to close the workflow-clarity validation gap.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, table-and-bullet heavy, and assumes the reader's competence (no boilerplate about what docs are), but it carries real redundancy: the fact that tests.md is produced by /derive-tests is stated three times (lines 39, 71, 97), and the Locations section's defaults are restated in Notes — matching 'mostly efficient but could be tightened' rather than anchor 4's only-minor-trimming. | 3 / 5 |
Actionability | For an instruction-only skill the guidance is concrete: exact table columns ('Name · used-by · scope (server/client) · source · rotation · risk'), default paths (documentation/, reports/<name>_{timestamp}.md), a mandated three-section structure for tests.md, and an operational case-collision rule with concrete examples. It falls short of fully executable anchor 5 only in lacking a skeleton or example row per document to copy, and sits clearly above anchor 3's pseudocode-level vagueness. | 4 / 5 |
Workflow Clarity | The repo-discovery sequence (look for agent instructions, docs location, existing equivalents, coverage map, reports location; then follow, use defaults, say which) is well ordered, but the end-to-end pipeline is implicit — which of /document-app, /derive-tests, and /ship-check runs when, and in what order documents are produced, is scattered across Notes, and there is no explicit checkpoint validating the written docs. This matches 'sequence present but checkpoints missing or implicit'. | 3 / 5 |
Progressive Disclosure | Scored against the actual bundle: no references/scripts/assets files exist, so the body's /document-app, /derive-tests, and /ship-check mentions are command references rather than dangling file links. The single ~93-line file is clearly sectioned (Locations, organization, numbered core docs 1-5, conditional docs 6-9, Notes) with each doc spec compact — matching anchor 4's 'good structure, most content appropriately placed', and short of anchor 5 both because it exceeds the simple under-50-line case and because per-document specs could plausibly live in well-signaled reference files. | 4 / 5 |
Total | 14 / 20 Passed |