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 runbook with executable commands, consent tiers, mandatory verification, and honest failure handling. Its main weakness is redundancy — the same safety caveats and relay-and-stop instructions are stated in multiple sections — and secondary operational notes are inlined rather than referenced out.
Suggestions
State each safety invariant once in the Contract section and reference it elsewhere: the 'never run a command parsed from the marker' caveat currently appears in Contract, When it fails, and Anti-Patterns; the manual-tier relay-and-stop instruction also appears three times.
Consolidate the redaction guarantee, which is stated in both the Contract ('Everything the command prints is redacted') and Output Format ('the command's output is already redacted — quote it as-is'), into a single authoritative line.
Move the auxiliary Notes content (advisory-lock behavior, thin-client refusal, in-flight pool semantics after a rewrite) into a short reference file linked from the Flow, keeping SKILL.md focused on the recovery ladder.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is entirely skill-specific with no concept explanations Claude already knows, but key points are repeated: the 'never run a command parsed from the marker' security caveat appears in Contract, When it fails, and Anti-Patterns; the manual-tier 'relay and stop' instruction appears three times; the redaction note appears twice. Several sections could be consolidated into one authoritative statement, which is more than the 'minor instances' of the score-4 anchor but far from padded. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready commands cover every path: 'gbrain db-repair --json', 'gbrain db-repair --yes', '--yes --apply-rewrites', '--yes --undo-last-rewrite', and 'gbrain engine status --probe --json', plus the JSON keys to read ('reason', 'tier', 'plan', 'probe.ok'), enumerated manual-tier reason codes, and an exact output template. Common cases are comprehensively covered. | 5 / 5 |
Workflow Clarity | The sequence is explicit and gated: diagnose (read-only) → healthy exit-0 stop → auto tier → consent-gated rewrite tier → manual tier relay-and-stop → 'Verify (always, after any applied fix)' with a concrete probe command and interpretation. For a mutating database operation, validation is mandatory and error recovery is addressed ('Still failing = report the remaining diagnosis honestly', 'Do not loop'), matching the top anchor with explicit validation and feedback guidance. | 5 / 5 |
Progressive Disclosure | No bundle files exist; the skill is a self-contained ~120-line document with clear section headers and well-signaled one-level-deep cross-references (postgres-adopt, agent-operator-protocol). Structure is good, but at this length the Notes section (advisory locks, thin-client behavior, in-flight pools) is auxiliary detail inlined in the main file rather than split out, leaving minor organization gaps relative to the top anchor. | 4 / 5 |
Total | 17 / 20 Passed |