Content
85%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-sequenced, lean operational runbook with genuine validation checkpoints (impl-validator, unmigrated-vault stop). Its weaknesses are the single-file monolith — Setup Mode platform recipes and the QMD boilerplate should live in reference files — and minor executability gaps such as the undefined `obsidian_wiki_config_dir` helper.
Suggestions
Move Setup Mode's platform-specific install recipes (launchd plist, systemd service/timer, crontab, shell-rc notification wiring) into a references/setup.md file and keep a short platform-dispatch summary in SKILL.md.
Define or inline the resolution of `obsidian_wiki_config_dir` (the XDG-style vs legacy path logic described in Setup Mode Step 5) where it is first used in 'Before You Start', since the state-dir derivation currently depends on an undefined helper.
Extract the generic 'QMD Refresh After Vault Writes' block into a shared reference file (it appears to be cross-skill boilerplate) and reference it from SKILL.md to cut duplicated context.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean, imperative runbook: no concept explanations, no library tutorials, every section is operational (commands, state-file writes, platform dispatch). Guardrail notes like "Do not enumerate pages with `find` and edit the index by hand" earn their tokens; every token earns its place. | 5 / 5 |
Actionability | Mostly copy-paste-ready commands (`obsidian-wiki memory index/hot`, state-file writes, launchd/systemd/cron install snippets, impl-validator spec), but with minor gaps: `obsidian_wiki_config_dir` is invoked without being defined or resolved anywhere in the body, and `echo "<stale_count>"` is a placeholder rather than a computed value. Level 4 'concrete commands with minor gaps', not the fully executable level 5. | 4 / 5 |
Workflow Clarity | Run Mode is a clearly sequenced Steps 1–7 with explicit validation: the impl-validator subagent checklist (timestamp within 60s, non-negative integer, hot.md updated field), "Apply any FAILs before logging", and a stop-and-escalate path if the vault is unmigrated. The batch-operation validation cap does not apply because feedback loops are present — matches the level-5 anchor with checklist and error recovery. | 5 / 5 |
Progressive Disclosure | There are no bundle reference files at all; everything is inlined in a ~250-line SKILL.md. Content that belongs in separate files is inline: the Setup Mode's three platform-specific install recipes (launchd plist, systemd units, crontab), the shell-rc notification wiring, and the generic QMD Refresh block. Some structure (clear headers, two modes), but the split is missing — level 3, not 4. | 3 / 5 |
Total | 17 / 20 Passed |