Content
67%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 strong, immediately usable instruction-only skill: exact CLI syntax with realistic examples, a verify-first setup flow, and an explicit recall-before/store-immediately operating loop. The main costs are internal redundancy (store categories vs. best practices repeat each other, one bullet duplicated verbatim) and a missing post-store verification step.
Suggestions
Merge the 'When to Store Memories' category taxonomy with the 'Best Practices' list — they overlap heavily and 'Architecture decisions and their rationale' appears verbatim under two headings.
Trim the retain examples from six to two or three (one plain, one --context, one transcript) and consider moving the category taxonomy to a references/ file to slim the always-loaded body.
Add a brief post-store checkpoint, e.g. what a successful retain prints and what to do on failure (retry, check config), to close the validation loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient — the 'How Hindsight Works' section teaches genuinely non-obvious server behavior (retain runs an extraction pipeline, so pass raw content), and commands come with tight, varied examples. But there is clear tightening headroom: the 'When to Store Memories' category list and the 'Best Practices' list repeat each other ('Attribute individual preferences', 'Include outcomes', rich-context advice appear twice), and 'Architecture decisions and their rationale' is literally duplicated under both 'Learnings from Tasks' and 'Team Knowledge'. Six retain examples plus a transcript example is more than needed. Matches anchor 3 (mostly efficient, some unnecessary explanation / could be tightened); not 2 because nothing explains concepts Claude already knows, and the padding is duplication rather than tutorial prose. | 3 / 5 |
Actionability | Guidance is concrete and near copy-paste ready: exact bash commands for retain/recall/reflect with realistic fully-written payloads, the --context flag demonstrated with real values ('procedures', 'learnings', 'preferences'), and a complete setup recipe (curl install, heredoc config with chmod 600, bank-ID step). Matches anchor 4 (mostly executable, minor gaps): the <bank-id> placeholder must be substituted, and there is no indication of what recall/reflect output looks like or how to interpret it — small gaps that keep it from anchor 5. | 4 / 5 |
Workflow Clarity | The core loop is clearly sequenced with an explicit validation checkpoint in setup ('cat ~/.hindsight/config' → 'If the file doesn't exist or is missing credentials, help the user set it up' → install → create config → get bank ID), and the operating rhythm is explicit: 'Always recall before' a enumerated list of situations, 'Store immediately' on discovery. Matches anchor 4 (clear sequence, most checkpoints present, minor validation gaps). Not 5: there is no feedback loop after retain (no check that the store succeeded or what the CLI prints on failure) and no recovery guidance if recall returns nothing; not 3 because validation is not absent — setup is explicitly verify-first. | 4 / 5 |
Progressive Disclosure | No bundle files exist, so everything is inline in a single ~150-line SKILL.md with clear, well-ordered sections (Setup → How it works → Commands → When to store → When to recall → Best practices) and no nested or dead references. This is good structure for an instruction-only skill (per the rubric's code_vs_instruction note, absence of external files is not penalized per se), though the long taxonomy of memory categories, the six retain examples, and the transcript example are exactly the kind of content that could be offloaded to a reference file to slim the always-loaded body. Matches anchor 4 (good structure, most content appropriately placed, minor organization gaps); not 5 because the file carries reference-grade detail inline. | 4 / 5 |
Total | 15 / 20 Passed |