Content
67%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-organized operational skill with concrete templates, a thorough error-handling table, and validation on destructive operations. It loses points for redundant overview/flow/instructions sections and two broken references — the DOCTRINE.md link path and the missing scripts/memory-index.py — which hurt both actionability and navigation.
Suggestions
Collapse the `Flow`, `Instructions`, and `Overview` sections into a single lifecycle description to remove the duplicated 4-step parse/validate/execute/print list and the repeated subcommand enumeration.
Fix the DOCTRINE.md reference: change `[DOCTRINE.md](../hyperflow/DOCTRINE.md)` to `[DOCTRINE.md](references/DOCTRINE.md)` so the link resolves to the bundled file.
Resolve the `scripts/memory-index.py` reference — either add the script to a `scripts/` directory or reword the `add` section so it does not instruct running a script that is not shipped with the skill.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Largely efficient (terse tables, no concept padding), but the `Flow`, `Instructions`, and `Overview` sections redundantly restate the same parse→validate→execute→print lifecycle and re-list subcommands already in the table — noticeable duplication beyond 'minor'. Confirmed below 4 because whole duplicated sections, not just trimmable phrases; confirmed above 2 because the bulk is tight, not padded with explanations Claude already knows. | 3 / 5 |
Actionability | Concrete and mostly executable: the `add` entry template (`### [YYYY-MM-DD] <title>` with What/Why/Evidence), `grep/ripgrep` commands, exact archive paths, and the 8-step compact flow are copy-paste ready. Confirmed below 5 because `scripts/memory-index.py` is referenced as `python3 scripts/memory-index.py .hyperflow` but no `scripts/` directory exists, so that instruction cannot run as written; confirmed above 3 because the guidance is real commands/templates, not pseudocode. | 4 / 5 |
Workflow Clarity | Destructive/batch ops carry validation: `clear` requires AskUserQuestion confirmation and backs up to `cleared-<timestamp>.md`; `prune` uses `test -f` before removal; `compact` has a Dedup Reviewer plus checksum refresh; the Error Handling table gives recovery behaviors. Confirmed below 5 because the top-level `Flow` is a generic 4-step list ('execute subcommand') whose detail lives elsewhere; confirmed above 3 because validation checkpoints are present, so the destructive-cap-at-3 rule does not apply. | 4 / 5 |
Progressive Disclosure | SKILL.md acts as an overview with one-level-deep references to real files (`references/memory-system.md`, `references/compaction.md`, `references/output-style.md`) and a Resources section with descriptions. Confirmed below 5 because navigation is impaired by two broken references: `[DOCTRINE.md](../hyperflow/DOCTRINE.md)` points to a non-existent path (the file is at `references/DOCTRINE.md`) and `scripts/memory-index.py` has no corresponding directory; confirmed above 3 because most references resolve and structure is genuinely one-level-deep. | 4 / 5 |
Total | 15 / 20 Passed |