Content
80%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, highly actionable skill body: concrete commands, exact paths, and excellent progressive disclosure with one verified, clearly signaled reference file. Its weaknesses are a redundant PARA-rules section that repeats the directory tree, and an implicit maintenance workflow lacking validation checkpoints.
Suggestions
Remove or collapse the 'PARA rules' section, since the directory tree's inline comments already state what each PARA category means; keep only the non-obvious rule ('Move to archives when complete').
Add a short numbered maintenance loop with checkpoints, e.g. heartbeat: scan session -> extract durable facts to items.yaml -> verify entity exists (create if 3+ mentions) -> bump access metadata; and weekly: rewrite summary.md -> verify cold facts remain in items.yaml.
State explicitly what to do when qmd is unavailable or the index is stale (fallback to grepping $AGENT_HOME), so recall does not silently fail.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean with concrete paths, a compact directory tree, and punchy rules ('Memory does not survive session restarts. Files do.'). However, the 'PARA rules' section repeats the folder-tree comments almost verbatim (projects=active work, areas=ongoing, resources=reference, archives=inactive), a redundancy that keeps it below anchor 5. | 4 / 5 |
Actionability | Fully executable guidance throughout: copy-paste-ready qmd commands with examples ('qmd query "what happened at Christmas"', 'qmd index $AGENT_HOME'), concrete supersession mechanics ('status: superseded', add 'superseded_by'), explicit entity-creation thresholds (mentioned 3+ times), and exact file paths. Specific examples cover the common cases. | 5 / 5 |
Workflow Clarity | A temporal sequence is present (write daily notes continuously, extract durable facts during heartbeats, rewrite summary.md weekly) but it is implicit rather than clearly sequenced, and there are no validation checkpoints for heartbeat extraction or weekly synthesis. The Planning section's staleness check is the only feedback loop, matching anchor 3 rather than 4. | 3 / 5 |
Progressive Disclosure | The body is a clean overview with a single well-signaled, one-level-deep reference ('For the atomic fact YAML schema and memory decay rules, see references/schemas.md'), which was verified to exist and contains exactly the promised schema and decay rules. Content is appropriately split with easy navigation. | 5 / 5 |
Total | 17 / 20 Passed |