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.
A highly actionable, well-sequenced setup runbook with executable commands, explicit status-block parsing, and strong verification/feedback loops. Its few weaknesses are a broken cross-reference ('continue to 4b' in the Docker branch), some duplicated credential-path options, and inline troubleshooting content that could be offloaded to a reference file.
Suggestions
Fix the 3a-docker branch pointer: 'DOCKER=running → continue to 4b' should read 'continue to 3c', since 4b is the Apple Container credential path and the Docker flow proceeds to build and test in 3c.
Deduplicate the credential section: state the Dashboard-vs-CLI AskUserQuestion once with a VALUE placeholder instead of repeating near-identical option blocks for both the subscription and API-key paths.
Move the Troubleshooting section into a reference file (e.g. references/troubleshooting.md) and ensure diagnostics.md ships in the bundle so the step-9 reference resolves.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, imperative, and free of explanations of concepts Claude already knows, but includes some avoidable duplication: the Dashboard/CLI AskUserQuestion options are repeated nearly verbatim in both the subscription and API-key paths, and the Troubleshooting section partially re-covers error handling already specified in steps 7 and 8. | 4 / 5 |
Actionability | Commands are copy-paste ready throughout — 'bash setup.sh', 'npx tsx setup/index.ts --step <name>', 'curl -fsSL onecli.sh/install | sh', the PATH-persistence grep/echo blocks, the setfacl fixes, and the systemd heredoc — with branching keyed to concrete parsed status fields (NODE_OK, DOCKER, BUILD_OK, SERVICE_LOADED). | 5 / 5 |
Workflow Clarity | Steps 0-9 are clearly sequenced with explicit validation checkpoints and feedback loops (step 8 Verify with per-failure remediation, 'onecli secrets list' re-verification, 'git remote -v' checks), but the 3a-docker branch says 'DOCKER=running → continue to 4b' — 4b is the Apple Container credential path, so this pointer is wrong (should be 3c) and could derail the flow. | 4 / 5 |
Progressive Disclosure | Good structure with numbered sections and sound offloading of channel-specific and conversion logic to sibling skills, but it is a long single file whose inline Troubleshooting section could live in a reference, and its one file reference (.claude/skills/setup/diagnostics.md) is not part of the bundle and thus unverifiable. | 4 / 5 |
Total | 17 / 20 Passed |