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.
A well-engineered, highly actionable skill body: tables carry the detail, every usage context has a copy-paste command, and the fix/re-test validation contract is explicit throughout. Its main costs are minor redundancy (exit-code and log-path repetition) and a contributor-oriented section that most users will never need inline.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dominated by dense tables, concrete commands, and actionable rules with almost no padding of concepts Claude already knows. Minor trimming opportunities exist: exit-code semantics appear in Contract, "When it fails", and "Output Format", and the log path is repeated three times. Not 5 due to that repetition plus the contributor-focused section adding length for a subset of readers. | 4 / 5 |
Actionability | Invocation is copy-paste ready across all contexts ("gbrain smoke-test", the startup-hook redirect, "exec: gbrain smoke-test"), the custom-test drop-in is a complete working example, and failure handling gives specific next actions (run "gbrain db-repair" and re-test; name the missing key). Placeholders in the contributor template and the OpenClaw path are explicitly pattern/path-justified rather than gaps. | 5 / 5 |
Workflow Clarity | The single action (run the smoke test, then interpret results) is unambiguous, and the mutating auto-fix workflow is built around explicit validation: the test-first/re-test-after-fix contract, skip-vs-fail semantics ("Exit code = count of real failures, not skipped checks"), and "never report all-green when a test was skipped" provide the feedback loops the cap rule demands. Not 4 because checkpoints are explicit, not implicit. | 5 / 5 |
Progressive Disclosure | The single-file body is well sectioned (Contract, Built-in Tests, Usage, Custom Tests, Env Vars, Known Issues, Failure protocol, Anti-patterns, Output) with one clearly signaled one-level reference to ../conventions/agent-operator-protocol.md. Not 5 because no bundle exists to split into, and the contributor "Adding Built-in Tests" section plus Known Issues are inline candidates for a references file at this length (~170 lines). | 4 / 5 |
Total | 18 / 20 Passed |