Content
70%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 a well-sequenced, highly actionable planning workflow with strong validation gates and error-recovery loops, and its script/reference links mostly resolve. Its weaknesses are repetition and inline version/date noise that bloat the token budget, and a broken templates/ reference in the central getting-started path that undermines the bundle's navigation.
Suggestions
Ship the templates/ directory (task_plan.md, findings.md, progress.md) or replace the dead template links with the init-session.sh invocation and an inline minimal skeleton, since both the body and init-session.sh's TEMPLATE_DIR expect those files.
Consolidate the repeated security-boundary statements into the single Security Boundary section and move inline version numbers, issue references, and dated examples into a changelog or references file.
Unify the skill-directory environment variable (PWF_SKILL_DIR vs SKILL_DIR) across the creation contract and the parallel-workflow example so commands are copy-paste consistent.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient — tables, terse rules, and copy-paste commands with little explanation of concepts Claude already knows — but it could be tightened: security boundary statements are repeated in at least three places, 'planning files go in your project root' is stated twice, and inline time-sensitive markers ('v2.36.1', 'v2.37.0', 'issue #237', example dates like '2026-09-13') are not confined to a changelog/deprecated section. | 3 / 5 |
Actionability | Guidance is largely executable: exact commands ("sh \"$PWF_SKILL_DIR/scripts/init-session.sh\"", "check-complete.sh --validate path/to/task_plan.md", "set-active-plan.sh --list", "attest-plan.sh --show"), a precise plan-format contract ("### Phase N: Name" with exactly one **Status:** line), a concrete parallel-workflow example, and an error-table format. Minor gaps: the env var for the skill directory is PWF_SKILL_DIR in one section and SKILL_DIR in another, and several commands rely on placeholders the reader must resolve. | 4 / 5 |
Workflow Clarity | Multi-step processes are clearly sequenced with explicit validation checkpoints and feedback loops: 'FIRST: Restore Project State' before anything else, an init-then-validate gate ('Validation must succeed before substantive work and after changing phase structure'), the 3-Strike protocol as an error-recovery loop with user escalation, and checklists (Read vs Write decision matrix, 5-Question Reboot Test). Even the semi-destructive archive operation is guarded with preview/--yes/--restore and an explicit 'Never archive on age or completion alone' rule. | 5 / 5 |
Progressive Disclosure | References are one level deep and clearly signaled (references/reference.md, references/examples.md, references/plan-inventory.md all exist in the bundle, and scripts/ matches every script named in the body), but the core 'Templates' section links templates/task_plan.md, templates/findings.md, and templates/progress.md and instructs 'Copy these templates to start' — and no templates/ directory ships in the bundle, nor do the referenced paths exist. The primary getting-started path therefore dead-ends, and 270+ lines of body content mix advanced material (attestation, archiving, parallel workflows) that could partly live in references. | 3 / 5 |
Total | 15 / 20 Passed |