Content
71%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.
An authoritative, highly actionable schema document weakened by carrying too much inline: nearly everything a maintainer needs is copy-paste ready, but the 685-line body inlines content that belongs in the (underused) references directory, leaves two of three reference files unlinked, and cites a script that is not in the bundle. Internal structure and sequencing are clear throughout.
Suggestions
Move the Page Template, Paper Deep-Dive Template, confidence formula/source-quality table, and Importance Tiering sections into a reference file (e.g. references/SCHEMA.md) and link them one level deep, keeping SKILL.md to the architecture, principles, and config-resolution core — this addresses both the conciseness and progressive-disclosure gaps.
Fix the dangling and unused bundle references: link `references/karpathy-pattern.md` (the pattern this skill is 'Based on Andrej Karpathy's LLM Wiki architecture'), reference `references/WRITING.md` from the Writing Profile Resolution section, and either ship `scripts/manifest.py` or remove the `scripts/manifest.py migrate` invocation from the Source key contract section.
Deduplicate: delete the repeated `CODE_UNDERSTANDING_CODEGRAPH_BIN` env-var entry and consolidate the provenance-marker rules stated in three places (Provenance Markers, Page Template, Core Principles #4) into the single canonical section.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~40KB body is dense project-specific schema knowledge, not padding about concepts Claude already knows, but there is clear tightening available: `CODE_UNDERSTANDING_CODEGRAPH_BIN` appears twice verbatim, the extracted/inferred/ambiguous rules are restated across the Provenance, Page Template, and Core Principles sections, and manifest-migration edge cases run long. Mostly efficient with some removable redundancy — anchor 3, not 4. | 3 / 5 |
Actionability | Fully executable guidance throughout: complete shell functions (`obsidian_wiki_config_dir`, `find_config`), exact commands (`obsidian-wiki memory sync INGEST source="papers/attention.pdf" pages_created=3`, `scripts/manifest.py migrate <vault> --dry-run --from-root <old-vault-root>`), copy-paste page and paper templates, a concrete confidence formula with a scoring-bucket table, and a full env-var table. No meaningful gaps; exceeds anchor 4. | 5 / 5 |
Workflow Clarity | The Config Resolution Protocol is a well-sequenced 0→3 process with explicit failure handling ("tell the user it doesn't exist and list the available vaults... then stop"), and the lifecycle state machine names illegal transitions and how `obsidian-wiki lint` reports them. Not 5 because there is no consolidated end-to-end workflow with validate→fix→retry checkpoints — operational sequencing defers entirely to companion skills; not 3 because the sequences present carry explicit checkpoints. | 4 / 5 |
Progressive Disclosure | `references/MEMORY.md` is real, one level deep, and well signaled, but scored against the actual bundle: `references/WRITING.md` and `references/karpathy-pattern.md` are never referenced from the body (the description cites Karpathy yet never links his pattern file), `scripts/manifest.py` is invoked but absent, and ~450 lines of template/formula/tiering detail that belongs in reference files are inlined. Some structure with real gaps — anchor 3. | 3 / 5 |
Total | 15 / 20 Passed |