Content
96%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.
The body is lean, highly actionable, and sequenced with explicit validation checkpoints — an instruction-only skill done right. The only structural weakness is the unverified/missing TROUBLESHOOTING.md reference and unlinked 'See also' entries.
Suggestions
Ensure TROUBLESHOOTING.md actually ships in the skill bundle, or inline its fallback guidance in the Troubleshooting section, since the current link is dangling in this bundle.
Make the 'See also' entries resolvable (e.g., relative links or skill paths) so the related-skill pointers can be navigated rather than merely named.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every section adds non-obvious information: exact tool calls ("memory_sessions { ... }", "memory_recall { ... }"), the directory-boundary rule, pagination handling, and provenance checking — no explanation of what sessions or memory are. The boundary-check point recurs in Why/Workflow/Anti-patterns/Checklist, but each occurrence serves a distinct function (rationale, procedure, contrast example, verification) rather than padding, so it fits the score-5 anchor ("every token earns its place") better than score 4's "minor instances of over-explanation". | 5 / 5 |
Actionability | Guidance is fully executable: concrete JSON tool-call examples, exact match expression ("session.cwd === projectPath || session.cwd.startsWith(projectPath + sep)"), explicit path resolution ("path.resolve(process.cwd(), $ARGUMENTS)"), pagination instructions ("follow nextCursor ... Preserve filters between pages"), and a concrete expected-output sample. This matches the score-5 anchor (copy-paste ready, covers common cases); score 4's "minor gaps" does not apply. | 5 / 5 |
Workflow Clarity | Five clearly sequenced steps with explicit checkpoints: provenance validation in step 4 ("Check each result's session or project provenance before including it"), failure handling in step 2 ("No match: report that this project has no matching session and stop"), plus a Checklist section for verification. The operation is read-only (no destructive/batch cap applies), matching the score-5 anchor's explicit validation and verification steps; score 4 would require a missing checkpoint, and none is missing. | 5 / 5 |
Progressive Disclosure | Sections are well organized and the one external reference is clearly signaled and one level deep ("See [TROUBLESHOOTING.md](TROUBLESHOOTING.md) if memory_sessions or memory_recall is not available"), fitting score 4's "good structure; references mostly clear". It falls short of score 5 because the referenced TROUBLESHOOTING.md is not present in the bundle (no references/, scripts/, assets/, or TROUBLESHOOTING.md exists alongside SKILL.md), leaving a dangling link, and the "See also" skills ("recap, session-history, recall") are named without paths or links. | 4 / 5 |
Total | 19 / 20 Passed |