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 is highly actionable with real, executable script commands and explicit validation checkpoints, and the core workflow (restore, create, validate, update) is easy to follow. However, it is overlong for a SKILL.md: host-adapter internals and padded heuristic sections belong in reference files, and half of its file references point to files missing from the shipped bundle, breaking navigation. Fixing the bundle layout would lift both conciseness and progressive disclosure.
Suggestions
Ship the referenced files or remove the links: templates/task_plan.md, templates/findings.md, templates/progress.md, reference.md, examples.md, and plan-inventory.md are cited as core steps ('Use templates/task_plan.md as reference') but are absent from the bundle.
Move the Hermes Notes section (hook behavior, max_verify_nudges defaults, Windows path resolution, plugin install commands) into a separate host-integration reference file, keeping only a one-line pointer in SKILL.md.
Drop or compress low-value sections such as 'The Core Pattern' RAM/Disk analogy and the 5-Question Reboot Test, which restate guidance already covered by the Read vs Write matrix and Critical Rules.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly operational, but the Hermes Notes section carries dense host-adapter implementation detail (pre_llm_call/post_tool_call hook behavior, agent.max_verify_nudges defaults, Windows home-path resolution), and padded sections like 'Context Window = RAM' and the 5-Question Reboot Test could be tightened or moved to a reference file. | 3 / 5 |
Actionability | Provides copy-paste-ready bash and PowerShell blocks with Hermes-home auto-detection, concrete validation commands ('sh "$PWF_SKILL_DIR/scripts/check-complete.sh" --validate path/to/task_plan.md'), and archive/restore flags; all referenced scripts exist in scripts/. Falls short of 5 because several guidance steps point to template and reference files that are absent from the bundle. | 4 / 5 |
Workflow Clarity | Clear restore-first sequence with explicit validation checkpoints ('Validation must succeed before substantive work and after changing phase structure') and a 3-Strike error feedback loop. Minor gaps: the parallel root-plan vs .planning/<plan>/ paths and tool-vs-script initialization options are interleaved rather than presented as one decision tree. | 4 / 5 |
Progressive Disclosure | Sections are well-organized and references are clearly signaled, but six referenced files (templates/task_plan.md, templates/findings.md, templates/progress.md, reference.md, examples.md, plan-inventory.md) do not exist in the bundle, and dense host-specific content (Hermes Notes, install instructions, security internals) is inlined instead of split into reference files. | 3 / 5 |
Total | 14 / 20 Passed |