Content
77%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, an exact plan format contract, validation gates, and a well-designed error-escalation protocol. Its weaknesses are redundancy — the same read/write and persistence rules are restated across Critical Rules, the decision matrix, the reboot test, and anti-patterns — and dead references: most linked markdown files (templates, reference.md, examples.md, plan-inventory.md, docs/opencode.md) are missing from the bundle even though all referenced scripts exist.
Suggestions
Ship or fix the referenced files — templates/task_plan.md, templates/findings.md, templates/progress.md, reference.md, examples.md, plan-inventory.md, and docs/opencode.md — since the body directs readers to copy the templates and consult these docs, and none are present in the bundle.
Consolidate the overlapping guidance sections: Critical Rules 3/4, the Read vs Write Decision Matrix, the 5-Question Reboot Test, and the Anti-Patterns table each restate "re-read the plan before deciding" and "persist to files"; merging them into one section would cut tokens without losing clarity.
Move the OpenCode-specific installation and plugin details into the referenced docs/opencode.md and keep a one-line pointer in the body, since they are only relevant to a subset of hosts.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient — tables, terse rules, exact commands — but the same guidance is stated three or more times across sections: "Read Before Decide" (Critical Rules 3) reappears as the decision-matrix row "Starting new phase → Read plan/findings" and again as the anti-pattern "State goals once and forget → Re-read plan before decisions"; the RAM/Disk core pattern is restated by "Stuff everything in context → Store large content in files". Consolidating Critical Rules, the Read vs Write matrix, the 5-Question Reboot Test, and Anti-Patterns would remove noticeable redundancy. Not 4 because the repetition is across multiple full sections, not minor instances. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready guidance in both bash and PowerShell: the session-catchup discovery loop, `sh "$PWF_SKILL_DIR/scripts/init-session.sh"`, `check-complete.sh --validate path/to/task_plan.md`, `set-active-plan.sh --list` / `--archive <id>` / `--restore <id>`, all of which reference scripts that actually exist in the bundle's scripts/ directory. The plan format is pinned exactly ("### Phase N: Name with exactly one **Status:** pending/in_progress/complete line"), covering creation, validation, and archival cases. | 5 / 5 |
Workflow Clarity | Clear sequence with explicit validation gates and feedback loops: restore-before-create ordering in FIRST and Quick Start steps 1–4, the creation contract requiring "Validation must succeed before substantive work and after changing phase structure", the 3-Strike error protocol with escalation after 3 failures, and the semi-destructive archive flow gated by user approval with a preview (`--archive` previews, `--yes` applies). This matches the anchor for explicit validation steps plus error-recovery feedback loops. | 5 / 5 |
Progressive Disclosure | References are clearly signaled and one level deep ("See [reference.md](reference.md)", "See [examples.md](examples.md)", templates listed under Templates), but the referenced files do not exist in the bundle: templates/task_plan.md, templates/findings.md, templates/progress.md, reference.md, examples.md, plan-inventory.md, and docs/opencode.md are all absent — only the scripts/ directory ships. A reader following "Copy these templates to start" or "See reference.md" hits dead paths, which is more than the "minor organization gaps" of level 4. Not 2 because the in-body structure and signaling are good and the scripts that are referenced do exist. | 3 / 5 |
Total | 16 / 20 Passed |