Content
81%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 highly actionable, well-sequenced editorial workflow with executable commands, explicit validation checkpoints, and error-recovery loops. Its weaknesses are redundant restatements of the same points (copy-not-move stated three times, lint-vs-editorial-pass framing stated twice) and a monolithic ~250-line body whose only references point outside the skill directory.
Suggestions
State 'harvest is a copy, not a move' once (e.g., in 'What harvest does NOT do') and drop the restatements in Anti-Patterns and Phase 5.
Merge the 'editorial pass is the primary defense / linter is the safety net' framing from Anti-Patterns and the Bypass section into a single place.
Move the privacy-lint pattern details (built-in defaults, harvest-private-patterns.txt, harvest-lint.ts) into a reference file inside the skill bundle to slim the main body and make the skill's references internal.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient domain-specific content (phases, commands, outcomes, checklists) that assumes Claude's competence on general concepts, but with repeated restatements: 'harvest is a copy, not a move' appears in Anti-Patterns ('Moving files instead of copying... Don't rm -rf the source'), Phase 5 ('harvest is a copy, not a move'), and 'What harvest does NOT do' ('It does NOT move files (it copies)'); the 'editorial pass is the primary defense / linter is the safety net' framing appears in both Anti-Patterns ('Trusting the linter alone... Phase 3 (the editorial pass) is the primary defense') and the Bypass section ('The privacy linter is the safety net. The editorial pass is the primary defense'). Not 4+: these duplications could be tightened into single statements. Not 2: there is no padded explanation of concepts Claude already knows. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready commands cover the common cases: 'gbrain skillpack harvest <slug> --from <host-repo-root> --dry-run', 'gbrain skillpack harvest <slug> --from <host-repo-root>', 'gbrain skillpack harvest <slug> --from <host-repo-root> --no-lint', 'bun test test/skills-conformance.test.ts', 'gbrain skillpack check --strict', 'gbrain skillpack list', 'cd <gbrainRoot> && git diff -- skills/<slug>/'. Concrete scrub examples ('alice-example', 'acme-example', 'fund-a', 'example@example.com', '#some-channel') and specific paths ('~/.gbrain/harvest-private-patterns.txt', 'src/core/skillpack/harvest-lint.ts') leave no key details missing. | 5 / 5 |
Workflow Clarity | Six clearly sequenced phases (Plan → Dry-run → Genericization checklist → Real harvest → Verify → Announce) with explicit validation checkpoints (dry-run preview, privacy lint with rollback on match, conformance test, --strict drift check, user diff review and explicit commit approval in the Contract) and feedback loops for error recovery ('lint_failed — privacy linter caught something. Go back to Phase 3, scrub the host file, retry'). Outcomes and failure modes are enumerated (harvested, lint_failed, slug_collision, drift). This matches the anchor for clear sequence, explicit validation, and checklists. | 5 / 5 |
Progressive Disclosure | A single-file body with well-organized sections and two clearly signaled references ([_brain-filing-rules.md](../_brain-filing-rules.md), [agent-operator-protocol.md](../conventions/agent-operator-protocol.md)). No bundle files exist (no references/, scripts/, or assets/ directories), and the two links point outside the skill directory ('../') to unverifiable paths; the ~250-line body also inlines material (the Phase 3 scrub checklist details, lint pattern internals) that could partly live in a reference file to slim the main skill. Good structure with minor organization gaps — anchor 4, not 5. | 4 / 5 |
Total | 17 / 20 Passed |