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.
Highly actionable content: concrete commands, verbatim prompts, exact output formats, and validation checkpoints with error-recovery paths for a mutating self-upgrade. It is lean overall with only minor trimmable asides and one version-pinned external reference that is not bundled.
Suggestions
Trim or relocate the always-on-daemon/autopilot parenthetical under 'mode = auto' and de-duplicate the Contract section against the flow below it to tighten conciseness.
Replace the version-pinned `skills/migrations/v0.46.3.0.md` path with a stable location (e.g., a `references/` migration index or a glob like 'latest v* migration playbook') so the reference does not rot when new versions ship.
Consider adding a brief post-upgrade verification step (e.g., re-run `gbrain self-upgrade --check-only` or a version print) to close the loop after an applied upgrade.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and imperative with no padding of known concepts, but the always-on-daemon/autopilot parenthetical and the Contract section restate details covered by the flow, and the version-pinned `skills/migrations/v0.46.3.0.md` reference is mildly time-sensitive without being in a deprecated/old-patterns section. | 4 / 5 |
Actionability | Every step is copy-paste ready: `gbrain config get self_upgrade.mode`, `gbrain self-upgrade --check-only --json`, `gbrain config set self_upgrade.mode auto|off`, a verbatim 4-option operator prompt, and exact one-line report formats for each outcome. | 5 / 5 |
Workflow Clarity | The mode-branched sequence (read mode, then off/auto/notify paths) has explicit validation checkpoints — `--check-only --json` must report `update_available: true` before prompting, the version is regex-validated, failed versions are never retried, and doctor `self_upgrade_health` failures are surfaced — giving a genuine validate/recover feedback loop for a mutating operation. | 5 / 5 |
Progressive Disclosure | A single-file skill with well-organized sections and one clearly signaled one-level reference ('open `skills/migrations/v0.46.3.0.md` and follow that playbook'); the minor gap is that the referenced playbook is not part of the bundle and its version-pinned filename is likely to rot across releases. | 4 / 5 |
Total | 18 / 20 Passed |