Content
88%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.
The body is a strong operational skill: an explicit done/NOT-done contract, a six-phase workflow with validation and rollback at every mutation boundary, a concrete editorial checklist, and named failure modes with recovery paths. The only real costs are mild repetition of the copy-not-move and lint-rationale points, and a single-file layout that inlines a checklist which could be a reference file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with non-obvious, repo-specific guidance (path-confinement, rollback on lint match, slug collision handling) and teaches nothing Claude already knows. It falls short of 5 because a few points are repeated — 'harvest is a copy, not a move' appears in Anti-Patterns, Phase 5, and 'What harvest does NOT do', and the dry-run warning and lint-vs-editorial-pass rationale each appear twice — but these are minor trims, matching the 4 anchor ('minor instances of over-explanation that could be trimmed'). | 4 / 5 |
Actionability | Guidance is fully executable: copy-paste CLI invocations for dry-run, real harvest, and the bypass case; concrete verification commands (`bun test test/skills-conformance.test.ts`, `gbrain skillpack check --strict`, `gbrain skillpack list`); a Phase 3 checklist with specific before→after replacements ('garry → the user', 'Real people → placeholder slugs (alice-example, acme-example)'); and enumerated outcome states with a defined retry path. This matches the 5 anchor: copy-paste ready commands covering the common cases. | 5 / 5 |
Workflow Clarity | Six phases are clearly sequenced with explicit validation checkpoints and a real feedback loop: dry-run preview (Phase 2) before the mutating copy, privacy lint with rollback + non-zero exit (Phase 4), a lint_failed → 'Go back to Phase 3, scrub the host file, retry' recovery loop, conformance testing, drift check, and a hard stop for explicit user approval before commit. This is the 5 anchor: clear sequence, explicit validation, error-recovery loops, and a checklist for the complex editorial pass. | 5 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are all absent), so this is a single-file skill; structure comes from well-organized, consistently headed sections (Contract, Output Format, Anti-Patterns, Preconditions, Phases, Bypass, Files touched) with pointers like `src/core/skillpack/harvest-lint.ts` and `../_brain-filing-rules.md` kept one level deep. It does not reach 5 because the skill exceeds the under-50-line simple case and inlines ~245 lines where the Phase 3 genericization checklist could plausibly live in a reference file; it stays above 3 because nothing is buried or nested and every section is clearly signaled. | 4 / 5 |
Total | 18 / 20 Passed |