Content
52%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 genuinely actionable workflows with explicit validation gates and error-recovery loops, and its script commands are concrete and executable. However, it is bloated with changelog-style version history and design rationale, and its progressive disclosure is broken in practice: seven referenced markdown/template files are missing from the bundle while deep reference material is inlined.
Suggestions
Ship the referenced files or fix the links: templates/task_plan.md, templates/findings.md, templates/progress.md, templates/loop.md, reference.md, examples.md, and plan-inventory.md are all cited but absent from the bundle.
Move the v3 hardening, host capability tiers, unchanged-view suppression, parallel-write guard, and turn-loop integration sections into a separate reference file, leaving SKILL.md a lean overview with one-level-deep pointers.
Strip version numbers and issue/PR history ("v2.42.0 clarification", "issue #178", "PR #180 lesson") from the instructional text, or consolidate them into a dedicated changelog/deprecated section, and collapse the four overlapping plan-restoration explanations into one canonical sequence.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~500-line body is noticeably verbose: it is littered with version numbers and changelog references ("v3.24.0", "v2.42.0 clarification", "issue #237", "PR #180 lesson") and long design-rationale essays ("This is the lesson from issue #178: an incomplete plan is a normal state...") that are history, not instruction. Core sections like File Purposes and Critical Rules are efficient, keeping it above the severely-padded anchor. | 2 / 5 |
Actionability | Guidance is mostly executable: exact commands ("sh \"$PWF_SKILL_DIR/scripts/init-session.sh\"", "session-catchup.py --metadata \"$(pwd)\""), a 6-step manual /plan-goal procedure, and a precise phase format contract ("exactly one **Status:** pending... line"). The minor gap is that the templates the body tells the agent to copy are not shipped in this bundle. | 4 / 5 |
Workflow Clarity | Multi-step processes are clearly sequenced with real checkpoints: numbered restore steps, "Validation must succeed before substantive work and after changing phase structure", the 3-Strike Error Protocol as an error-recovery feedback loop, and check-complete.sh verification. It falls short of the top anchor because the mainline workflow is scattered across overlapping sections (plan restoration is re-explained in at least four places). | 4 / 5 |
Progressive Disclosure | Checked against the actual bundle: the templates/ directory does not exist, so the links to templates/task_plan.md, templates/findings.md, templates/progress.md, and templates/loop.md are dead, as are the reference.md, examples.md, and plan-inventory.md links — only the scripts/ references resolve. Meanwhile ~300 lines of v3 hardening, host-capability-tier, and turn-loop integration detail that clearly belongs in separate reference files are inlined in SKILL.md. | 2 / 5 |
Total | 12 / 20 Passed |