Content
63%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 delivers strongly actionable, executable guidance with explicit validation checkpoints and error-recovery protocols, and it is well-sectioned. Its weaknesses are redundancy across the restore/create/quick-start sections, a lifecycle spread over multiple sections instead of one ordered flow, and a reference layer pointing at files that are not present in the provided bundle.
Suggestions
Ship the referenced files (templates/task_plan.md, templates/findings.md, templates/progress.md, references.md, examples.md, plan-inventory.md) in the bundle or remove/inline those links — currently most referenced navigation targets resolve to nothing.
Consolidate the repeated restore/create guidance from "FIRST: Restore Project State" and "Quick Start" into a single lifecycle section, and trim the "5-Question Reboot Test" and inferable rows of the "Read vs Write Decision Matrix".
Add a copy-paste example invocation for `resolve-plan-dir.sh` (with PLAN_ID/PWF_PLAN_ROOT) and replace the hardcoded `.factory/skills/...` path in the session-catchup command with the `$PWF_SKILL_DIR` pattern used elsewhere.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The many tables are tight, but guidance repeats across sections ("Restore before creating" appears in both "FIRST: Restore Project State" and "Quick Start"), and the "5-Question Reboot Test" and "Read vs Write Decision Matrix" restate inferences Claude can make on its own ("Error occurred → Read relevant file — Need current state to fix"). Not 2 because there is no concept-explanation padding; not 4 because the repetition and inferable matrix rows could be trimmed. | 3 / 5 |
Actionability | Concrete executable commands appear throughout: `sh "$PWF_SKILL_DIR/scripts/init-session.sh"`, `sh "$PWF_SKILL_DIR/scripts/check-complete.sh" --validate path/to/task_plan.md`, `sh "<skill-dir>/scripts/set-active-plan.sh" --list` with `--archive <id>`/`--yes`/`--restore <id>`, PowerShell equivalents, and the exact `### Phase N: Name` / `**Status:**` marker format. Not 5 because the session-catchup command hardcodes the `.factory/skills/...` install path and `resolve-plan-dir.sh` is described without a copy-paste example invocation. | 4 / 5 |
Workflow Clarity | The lifecycle is clear with an explicit validation gate ("Validation must succeed before substantive work and after changing phase structure"), a feedback loop (the 3-Strike protocol escalating to the user), and preview/confirm gating for the destructive archive operation. Not 5 because the sequence is distributed across "FIRST: Restore Project State", "New plan creation contract", "Quick Start", and "Critical Rules" rather than one coherent ordered flow, leaving the reader to assemble the full lifecycle. | 4 / 5 |
Progressive Disclosure | References are clearly signaled and one level deep ([templates/task_plan.md], [references.md](./references.md), [examples.md](./examples.md), [the inventory schema and worker handoff](plan-inventory.md)), and referenced scripts all exist in the bundle's scripts/ directory. However, scored against the actual bundle, four of the referenced navigation targets (templates/, templates/task_plan.md, references.md, examples.md, plan-inventory.md) are absent, so the navigation structure cannot be relied on. Not 4 because missing reference targets exceed "minor organization gaps". | 3 / 5 |
Total | 14 / 20 Passed |