Content
57%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 dense, information-rich schema/theory document with strong concrete templates and executable config-resolution code, but it is over-long for a SKILL.md overview, inlines material that belongs in reference files, and lacks validation checkpoints for the destructive operations it describes. Tightening and splitting into the existing references/ folder would materially improve it.
Suggestions
Move the Page Template, Paper Deep-Dive Template, confidence formula/source-quality tables, and the source-key contract into reference files under references/, leaving SKILL.md a concise overview with clearly signaled links (e.g. "See references/page-templates.md").
Add explicit validation/verification checkpoints to the Rebuild/Restore workflow description (archive-then-verify, dry-run before clear) so the destructive mode is not described without a feedback loop.
Trim redundancy such as the duplicated `CODE_UNDERSTANDING_CODEGRAPH_BIN` entry and condense the env-var catalog to the required variable plus a pointer to `.env.example`.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Most content is framework-specific and earns its place (provenance markers, confidence formula, source-key contract), but the ~670-line body has noticeable slack: a duplicated `CODE_UNDERSTANDING_CODEGRAPH_BIN` entry and several sections that over-explain. Not 4 because the padding is more than minor; not 2 because the bulk is genuinely non-obvious domain detail rather than concepts Claude already knows. | 3 / 5 |
Actionability | Provides concrete, executable guidance: copy-paste shell functions (`obsidian_wiki_config_dir`, `find_config`), full page and paper-deep-dive templates, the confidence formula with a source-quality bucket table, and retrieval-primitive commands (`Grep -A <n> -B <n>`). Not 5 because some detail is delegated to companion skills (manifest schema, lint invocation) leaving minor gaps. | 4 / 5 |
Workflow Clarity | The Config Resolution Protocol is a clear 0→1→2→3 sequence, but the destructive Rebuild mode (archive + clear + reprocess) is described without validation checkpoints in this skill, and most operational workflows are offloaded to companion skills. Capped at 3 per the destructive/batch-operations-without-validation rule; not 2 because the config sequence is genuinely coherent. | 3 / 5 |
Progressive Disclosure | Section structure is good, but substantial content that could live in reference files is inlined (both page templates, the confidence formula + bucket table, the source-key contract, config-resolution shell functions, the env-var catalog), and the existing bundle files (references/karpathy-pattern.md, references/WRITING.md) are not clearly signaled or linked from the body. Not 4 because the references are not well-signaled and too much bulk is inline. | 3 / 5 |
Total | 13 / 20 Passed |