Content
78%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 highly actionable with concrete executable code and commands, and the workflow is a clear four-step sequence with useful error-recovery guidance. Its main weakness is progressive disclosure: the SKILL.md imports from and links to helper/scratchpad files that are not present in the bundle, leaving the skill non-functional as packaged.
Suggestions
Ship the referenced helper files (parseSnapshot.ts, streamSnapshot.mjs, compareSnapshots.ts, findRetainers.ts) in a helpers/ bundle directory so the import examples actually resolve.
Add an explicit validation checkpoint to the workflow (e.g., verify the parsed graph/edge counts against snapshot.meta before running comparison or retainer analysis) to support a verify-fix-retry loop.
Tighten the 'Useful tricks' bullet list and the approximation-boundaries discussion, or move that detail into a reference file that is actually bundled, to improve token efficiency.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient and specialized (V8 format, VS Code retention patterns, false retainers) without explaining basics Claude already knows, but the 10-bullet 'Useful tricks' list and the approximation-boundaries discussion are instances that could be trimmed, keeping it just below the lean/5 anchor. | 4 / 5 |
Actionability | Provides copy-paste-ready code for parseSnapshot/compareSnapshots/findRetainerPaths/streamSnapshot with concrete parameters (maxPaths: 5, maxDepth: 25) and exact bash commands with flags (node --max-old-space-size=24576), covering the common analysis cases. | 5 / 5 |
Workflow Clarity | A clear four-step sequence (Parse → Compare → Find Retainers → Write Scripts) with error-recovery guidance (ERR_FS_FILE_TOO_LARGE → switch to streaming), but no explicit validation checkpoints or verify-fix-retry feedback loops in the main workflow, capping it below 5. | 4 / 5 |
Progressive Disclosure | Sections are well-organized and references are clearly signaled with markdown links, but every referenced bundle path (helpers/parseSnapshot.ts, streamSnapshot.mjs, compareSnapshots.ts, findRetainers.ts, scratchpad/) is absent from the bundle, so the one-level-deep references do not resolve and the skill is incomplete as shipped; substantial specialized content is also inlined. | 3 / 5 |
Total | 16 / 20 Passed |