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.
A highly actionable, well-sequenced skill body: every phase has copy-paste-ready commands, explicit validation gates, and safety contracts (.bak, dry-run, confirm-before-fix). Its weaknesses are token efficiency (narrative/history padding and a duplicated trigger-words section) and progressive disclosure — it is a ~230-line monolith whose prevention guide and output-format reference material belong in separate one-level-deep reference files.
Suggestions
Move the "Prevention — Writing Valid Frontmatter" section and the full JSON envelope example into a references/ file (e.g. references/prevention.md), leaving a one-line pointer in SKILL.md.
Delete the body-level "Trigger words" section — it duplicates the frontmatter `triggers` field verbatim and adds token cost with no new information.
Trim the "Why This Exists" narrative and the pre-v0.37.5.0 validator history to a sentence each (or fold the version note into a short 'historical note' under the arrays subsection), keeping only the actionable guidance.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient — the validation-classes table, phases, and output rules are dense and non-obvious — but there is noticeable padding: the "Why This Exists" narrative, the historical pre-v0.37.5.0 bug story ("One brain saw 6,981 of these on a single doctor run"), and a body-level "Trigger words" section that duplicates the frontmatter triggers. Not 4 because several sections could be trimmed without losing actionable content; not 2 because almost nothing explains concepts Claude already knows — the detail is tool-specific and unfamiliar. | 3 / 5 |
Actionability | Fully executable throughout: copy-paste-ready commands for every phase ("gbrain frontmatter audit --json", "gbrain frontmatter validate <path> --fix", "gbrain frontmatter install-hook"), explicit exit-code semantics ("Exit code 0 = clean; 1 = errors found"), a concrete correct/broken YAML example set, and realistic sample outputs covering the common cases. Not 4 because there are no gaps — commands, flags, and expected outputs are all specified. | 5 / 5 |
Workflow Clarity | Clear sequence with explicit validation checkpoints and feedback loops for destructive/batch operations: audit first ("Always run `gbrain frontmatter audit --json` first; never assume a brain is clean"), `--dry-run` preview ("Use this before applying fixes in batch"), `.bak` backups before every mutation, count-and-confirm before `--fix`, and exit-code validation for CI. The destructive-operation cap at 3 does not apply because verification steps are explicitly present. Not 4 because checkpoints are explicit and the anti-patterns section documents error-recovery behavior for every failure path. | 5 / 5 |
Progressive Disclosure | The single-file body has good header structure, but reference-style material is inlined that belongs in a separate file — the ~60-line "Prevention — Writing Valid Frontmatter" section (canonical YAML forms, quoting rules, the LLM-trap JS snippet) and the full JSON envelope example read like a reference doc, and there are no bundle files at all (no references/, scripts/, or assets/). It matches anchor 3 ('content that should be separate is inline'); not 4 because the long inline reference material is exactly what the level-4/5 anchors expect split out, and not 2 because the section structure and navigation within the file are clear. | 3 / 5 |
Total | 16 / 20 Passed |