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 exceptionally actionable, well-sequenced pipeline skill with real validation feedback loops, undermined by monolithic structure and notable repetition of the paging and path-scoping warnings. The non-obvious Basic Memory operational knowledge (page-size caps, .txt vs .md line offsets, grep row limits) earns its tokens; the duplication and inlined schema boilerplate do not.
Suggestions
Move the six full schema write_note definitions (~170 lines of Phase 0) into a references/schemas.md file and keep only one exemplar schema plus a summary table inline, adding pointers like "See [schemas.md](references/schemas.md)".
Extract the jq chapter-inventory coverage script and the bm find/grep semantics details into a references/validation.md or references/bm-queries.md, leaving the workflow and the two headline rules in the body.
Deduplicate the "--page-size caps at 200 / --name cannot combine with --meta / positional path scoping" material, which appears nearly verbatim in both the Tools section and Coverage Checks — state it once in Tools and cross-reference from Coverage Checks.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient, high-value operational knowledge (paging caps, POSIX read verbs, offset maps), but the "--page-size 200" warning recurs four times (Tools, Phase 3, Coverage Checks, Phase 5) and the "--name cannot combine with --meta" / path-boundary-scoping explanation is repeated nearly verbatim in the Tools section and Coverage Checks. Anchor 4 requires only minor trimming; this level of duplication keeps it at 3. | 3 / 5 |
Actionability | Fully executable guidance throughout: complete write_note/edit_note templates with metadata and content, concrete bm CLI invocations ("bm cat ... --lines 4200-4890 --plain"), and a copy-paste-ready bash+jq pagination loop that walks pages until has_more is false and reports missing/duplicates/out_of_range. Placeholders like <Character Name> are inherent to a template for any literary work, not gaps. | 5 / 5 |
Workflow Clarity | A clearly sequenced Phase 0-5 pipeline with a dedicated validation phase (schema_validate, schema_diff, coverage checks, sequence-gap detection with feedback loops: "fix the spelling or add the alias, then re-run the chapter"; "Fix issues found") and an explicit "Validate periodically... after each batch" guideline. For a batch operation this matches the anchor-5 example with checkpoints and error-recovery loops. | 5 / 5 |
Progressive Disclosure | The body is well-sectioned with clear headers and a coherent phase flow, but there are no bundle files at all (no references/, scripts/, or assets/ directories exist) and ~740 lines are inlined monolithically: six full schema write_note definitions (~170 lines) and the long jq coverage-check script clearly belong in separate reference files. This matches anchor 3 — structure present, but content that should be separate is inline. | 3 / 5 |
Total | 16 / 20 Passed |