Content
82%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 concrete commands, pinned versions, and worked provider examples, plus a clearly sequenced install workflow with validation checkpoints. Minor gaps are some repeated verbosity in env-var semantics and a broken link to REMOVE.md.
Suggestions
Add the missing REMOVE.md bundle file (or remove the `[REMOVE.md](REMOVE.md)` link) so the one external reference resolves.
Factor the repeated 'Anything but a positive integer is logged and treated as unset' caveat into a single shared note rather than restating it per env var.
Make the install failure-recovery loop explicit (e.g. 'If a check fails: fix the reported error and re-run that check before proceeding') to reach the top workflow-clarity anchor.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Dense and project-specific with no padding of concepts Claude already knows, though the repeated 'Anything but a positive integer is logged and treated as unset' phrasing across several env vars could be trimmed. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready guidance: exact version pin `@opencode-ai/sdk@1.4.17`, concrete `nc:copy`/`nc:append`/`nc:dep` blocks, and complete `onecli secrets create` commands with worked DeepSeek, OpenRouter, Anthropic, and Zen examples. | 5 / 5 |
Workflow Clarity | Clear five-step install sequence with explicit validation commands (tsc, vitest, build.sh) and a 'must be clean before proceeding' gate, but the error-recovery loop on failure is implicit rather than spelled out. | 4 / 5 |
Progressive Disclosure | Well-organized into clear sections (Install, Configuration, Operational notes, Next steps) with one clearly signaled one-level reference, though that reference ([REMOVE.md](REMOVE.md)) does not exist in the bundle. | 4 / 5 |
Total | 17 / 20 Passed |