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.
The body is highly actionable with an exemplary validated workflow (dry-run gates, lint, verify, failure recovery). Its weaknesses are redundancy between the Outputs and Contract sections and time-sensitive version detail inline, plus inlined reference material and unresolvable convention paths given the empty bundle.
Suggestions
Merge the Outputs and Contract sections into one, keeping the single fuller treatment of pack file, audit row, backfill, trust, and atomicity to remove the duplication.
Move the JSON output examples and the error-code list into a references/ file (e.g. references/outputs.md) and link to it one level deep, which also gives the conventions and sibling-skill paths a real bundle to resolve against.
Relocate version-specific history (v0.40.6.0 wiring, cross-process invalidation gap) into an 'old patterns' / 'version notes' section or the referenced conventions doc, and use one canonical path for schema-evolution.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is command-dense and assumes competence, but the Outputs and Contract sections duplicate each other (both enumerate the pack file, audit row, and backfilled pages.type), and version-history asides ("added in v0.40.6.0 — pre-v0.40.6 brains silently ignored custom expert-routed types") are time-sensitive content placed outside any old-patterns section. | 3 / 5 |
Actionability | Every phase gives copy-paste-ready commands with concrete flags (gbrain schema add-type researcher --primitive entity --prefix people/researchers/ --extractable --expert, lint --with-db, sync --apply) plus a worked batched-mutation JSONL example and a git commit sequence, fully covering the common cases. | 5 / 5 |
Workflow Clarity | Seven clearly sequenced phases embed explicit validation checkpoints for the batch/destructive operations: lint --with-db before write, sync dry-run with would_apply counts and sample slugs before --apply, post-apply verification via stats and whoknows, and a Failure modes section with recovery steps per error code. | 5 / 5 |
Progressive Disclosure | Section structure is good and conventions are referenced one level deep, but no bundle files exist, the referenced paths (../conventions/schema-evolution.md, also inconsistently cited as skills/conventions/schema-evolution.md) cannot be verified, and ~40 lines of JSON output/error-code reference material are inlined in the SKILL.md rather than split into a references file. | 3 / 5 |
Total | 16 / 20 Passed |