Content
75%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 well-structured, highly actionable instruction skill: sequenced steps, concrete commands and slash-invocation examples, explicit error-recovery gotchas, and a genuinely useful one-level-deep reference file. The main gaps are the missing scripts/retrospect-load-sessions.sh in the bundle (referenced in both Steps and Gotchas), minor duplication between the body's Metrics Summary Format and the reference's report template, and the absence of an output-verification checkpoint.
Suggestions
Ship or locate `scripts/retrospect-load-sessions.sh` — it is invoked in step 1 and gotchas but does not exist in the skill bundle; if it lives at the plugin root, state that path explicitly so the invocation is verifiable.
Replace the in-body 'Metrics Summary Format' section with a pointer to the report template in references/reference.md, which already contains the identical format, to remove duplication.
Add a final verification step, e.g., 'Confirm the report file exists at .retro/insights/collab/{PERIOD}.md and that the Sessions count in the Metrics Summary equals the number of session paths from step 1'.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and operational (steps, commands, gotchas) with no explanations of concepts Claude already knows, but the "Metrics Summary Format" block duplicates the Metrics Summary already present in reference.md's report template, and the gotcha about testing the loader repeats step 1. This matches 'efficient; minor instances that could be trimmed' rather than score 5's 'every token earns its place', and is well above score 3's noticeable padding. | 4 / 5 |
Actionability | Concrete copy-paste guidance throughout: `bash ${CLAUDE_PLUGIN_ROOT}/scripts/retrospect-load-sessions.sh $@`, exact slash-command examples for every argument mode, a concrete output path with example filename, and a filled-in metrics example. Not score 5 because the referenced loader script (`scripts/retrospect-load-sessions.sh`) does not exist in the skill's bundle (only `references/reference.md` is present), leaving a minor executability gap; far above score 3's pseudocode/incomplete level. | 4 / 5 |
Workflow Clarity | Steps 1-7 are clearly sequenced with concrete commands, and the Gotchas section supplies error-recovery loops (skip duration metrics when `duration_seconds` is absent, emit a partial report when session files are unreadable, parse the PERIOD header carefully, test the loader directly). Not score 5 because there is no explicit checkpoint verifying the report was written correctly or that the analyzed session count matches the loader output; this is not a destructive/batch-write operation, so the score-3 cap does not apply. | 4 / 5 |
Progressive Disclosure | Good structure against the actual bundle: the body is an overview with well-labeled sections and a clearly signaled, real one-level-deep reference ([Reference](references/reference.md) — 'scoring rubrics, metric definitions, and report format'). Not score 5 because the body references a `scripts/` directory that is absent from the bundle, and it inlines the Metrics Summary format and analysis dimension lists that partially duplicate reference.md content rather than deferring them. | 4 / 5 |
Total | 16 / 20 Passed |