Content
72%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 lean, mostly actionable rule list with concrete commands, files, and template syntax, weakened by an implicit workflow order (usage rules before the config setup they depend on) and vague bundle navigation that leaves the config-format details undiscoverable. The usage path is well served; the first-time setup path is not.
Suggestions
Reorder (or number) the workflow so configuration precedes usage: author .worktree-env.json → set the slot variable in a worktree-local env file → start via the root `bun dev*` script → verify with `bun dev:info --json` before accessing apps.
Replace the generic "bundled in `scripts/`" pointer with named files and roles, e.g. `scripts/worktree-runtime.ts` (resolver) and `scripts/worktree-env.ts` (command wrapper).
Reference `assets/worktree-env.schema.json` where `exports`, app offsets, and range stride are mentioned, or inline a minimal `.worktree-env.json` example so the config-format rules are self-contained.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Ten lean imperative bullets with zero padding — "Run `bun dev:info --json` before accessing local apps. Use the returned app URLs; never assume a port" — and no explanation of concepts Claude already knows. Every token earns its place; nothing could be cut without losing information. | 5 / 5 |
Actionability | Mostly concrete and executable: named commands (`bun dev:info --json`), named files (`.env.worktree.local`, `.env.local`, `.worktree-env.json`), and exact template syntax (`{apps.<name>.url}`). Gaps keep it below 5: "Keep app offsets unique and below the configured range stride" and "Declare every variable an app needs under its `exports`" never say where these fields live, and no example `.worktree-env.json` is shown. | 4 / 5 |
Workflow Clarity | The steps are present but ordered thematically rather than by execution — usage rules ("Run `bun dev:info --json` before accessing") precede the config setup (slot variable, `exports`, templates) that must actually come first, so the sequence is implicit. The sole checkpoint is the dev:info pre-access rule; there is no error-recovery guidance (e.g., what to do on a port conflict or unassigned slot), matching anchor 3 rather than anchor 4's "most checkpoints present". | 3 / 5 |
Progressive Disclosure | Implementation is appropriately split out (the resolver and wrapper do live in `scripts/`), but the pointer "The generic resolver and command wrapper are bundled in `scripts/`" names no files or their roles, and `assets/worktree-env.schema.json` — which documents the `exports`, offsets, and range-stride fields half the bullets rely on — is never referenced at all. References are present but weakly signaled with a real navigation gap, matching anchor 3; it is not anchor 4 because the omissions exceed minor organization gaps. | 3 / 5 |
Total | 15 / 20 Passed |