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 well-structured, highly actionable instruction skill with real bundle files, explicit guards, and a clearly sequenced three-phase workflow. Its weaknesses are moderate redundancy (duplicated reference pointers and stream-naming rules, a largely inferable usage-examples section) and the absence of an in-body post-write validation step.
Suggestions
Remove the duplicate reference pointers: keep either the inline 'See references/reference.md' note in Phase 3 or the final References section, not both, and state the stream-naming rule once.
Trim or cut the 'Usage Examples' section (~34 lines) — the file paths it lists are already specified in the Workflow and References, so the section mostly re-derives inferable outcomes.
Add an explicit post-write validation checkpoint in Phase 3 (e.g., verify the written CONTEXT file is within the 1200-1500 token budget and that upsert-index.sh reported UPDATED/APPENDED before proceeding to archival).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient instructions (phases, guards, anti-patterns), but there is noticeable tightening opportunity: the final "References" section (lines 154-156) duplicates the pointer already given in Phase 3 (line 64), stream-naming rules appear in both Phase 1 and Phase 3, and the 34-line "Usage Examples" section largely restates file outcomes already specified in the Workflow. This fits "mostly efficient but includes some unnecessary explanation or could be tightened" better than the 4 anchor's 'minor instances', while being well above the verbose padding of a 2. | 3 / 5 |
Actionability | Concrete, executable guidance dominates: exact shell invocations ("rtk ls -t .context/session/CONTEXT-*llm.md", "mkdir -p .context/session/done && mv ..."), a stream-name regex, a real upsert script with documented arguments, and deferred-but-real templates in references/reference.md. It is not a 5 because Phase 2 synthesis guidance and the AskUserQuestion fallback are directive rather than copy-paste executable, and the archive-exceptions (baseline stream) live only in the reference file — minor gaps per the 4 anchor. | 4 / 5 |
Workflow Clarity | A clear three-phase sequence with most checkpoints present: the AskUserQuestion empty-answer guard with an explicit wait-for-reply loop, read-before-overwrite in the anti-patterns, and a status condition gating the destructive mv archive (only done/parked, with an anti-pattern warning against premature archival). It falls short of 5 because there is no post-write verification (e.g., confirming the file stayed within the 1500-token budget — the self-check exists only in the reference), leaving a minor validation gap per the 4 anchor. | 4 / 5 |
Progressive Disclosure | Good structure against the actual bundle: the body is a workflow overview, and both referenced paths are real, one-level-deep files clearly described (references/reference.md holds the template, status mapping, self-check, and upsert details; scripts/upsert-index.sh is a working script). It is not a 5 because the body runs ~145 lines with some content (usage examples, duplicated reference pointers, stream-naming repetition) that overlaps or could live in the reference, and the reference pointer is stated twice rather than cleanly signaled once. | 4 / 5 |
Total | 15 / 20 Passed |