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.
A well-structured, largely actionable instruction skill with a clear restore-then-plan workflow and a real error-escalation feedback loop. Its weaknesses are repeated error-handling guidance across four sections, unwired validation steps, and bundle navigation that breaks: most referenced files (templates, references.md, examples.md) are absent from the bundle.
Suggestions
Consolidate the four overlapping error-handling sections (Critical Rules 5-6, 3-Strike protocol, Anti-Patterns rows) into one, and drop the trivial 'if action_failed' pseudocode block.
Wire validation into the workflow explicitly, e.g. add 'run scripts/check-complete.sh after each phase; only mark complete when it passes' as a numbered Quick Start step, and give a concrete resolve-plan-dir.sh example invocation.
Ship the referenced files (templates/task_plan.md, findings.md, progress.md, references.md, examples.md) or remove/inline the links so navigation in the bundle does not dead-end.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient — tables for file purposes, the decision matrix, and anti-patterns are token-dense — but error handling is explained four separate times (Critical Rules 5 and 6, the 3-Strike Error Protocol, the Read/Write matrix rows, and two Anti-Patterns rows), and the 'Never Repeat Failures' section reduces to trivial pseudocode ('if action_failed: next_action != same_action') that Claude needs no help deriving. This matches 'Mostly efficient but includes some unnecessary explanation or could be tightened', not 4 — the repetition is more than minor trimming. | 3 / 5 |
Actionability | Concrete, runnable guidance dominates: an exact session-catchup.py invocation with $(command -v python3 ...) substitution, 'run scripts/init-session.sh "Task Name"', 'sh "<skill-dir>/scripts/set-active-plan.sh" --list' with the PowerShell equivalent, and named scripts (resolve-plan-dir.sh, check-complete.sh) that all exist in scripts/. Minor gaps keep it from 5: the resolve-plan-dir.sh usage is described ('with the host's PLAN_ID and PWF_PLAN_ROOT') without a copy-paste example, and check-complete.sh is listed but never wired into the workflow with an example invocation. | 4 / 5 |
Workflow Clarity | A clear ordered sequence exists — restore state (resolve plan dir, read the three files, git diff --stat), then Quick Start steps 1-4, with explicit correction handling ('If an explicit selector is rejected... correct the pin and do not fall back') and the 3-Strike protocol providing a genuine diagnose → alternative → rethink → escalate feedback loop. It is not 5 because validation is described but not checkpointed into the main flow: check-complete.sh is only listed under Scripts rather than placed as an explicit verify step after phases, and the checkpoint placement is implicit rather than sequenced. | 4 / 5 |
Progressive Disclosure | The body is well-sectioned and clearly signals one-level-deep references (Templates, Scripts, Advanced Topics pointing to references.md/examples.md), and the scripts/ paths it cites all exist in the bundle. However, scoring against the actual bundle, the majority of outbound references are dangling: templates/ and its three template files, references.md, and examples.md do not exist in the bundle, so following the navigation fails. This sits between 'references present but not clearly signaled' (3) and broken navigation (below), landing at 3 with the missing files as the dominant defect. | 3 / 5 |
Total | 14 / 20 Passed |