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.
A lean, highly actionable instruction-only skill: every step names concrete tools, filters, and config keys, with explicit failure handling for incomplete setup. The main deductions are mild redundancy in step 2's prose restatement and unspecified details (which tool to read notes with, and the permalink format) in steps 4-6.
Suggestions
Collapse the step 2 prose that restates the bullet rules ('Always query... Never run an unscoped coding-session query...') into one or two clauses, since the bullets already encode the rules.
Name the note-reading tool and show a one-line permalink example in step 4 so 'read the highest-signal hits' and 'include permalinks' are fully executable.
Trim the intro paragraph, which repeats the frontmatter description almost verbatim, to a single sentence distinguishing this from the SessionStart hook briefing.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense but nearly every token carries real, non-inferable constraints (hook config precedence, 'external_id' UUID handling, the lifecycle-trace exclusion). It is not 5 because step 2 states the coding-session rules twice — once in the bullet list and again in prose ('Always query... include coding_session... Never run an unscoped...') — and the intro paragraph re-states the description, which could be trimmed. | 4 / 5 |
Actionability | Concrete, executable guidance throughout: the exact tool call 'search_notes' with literal 'metadata_filters' JSON per query, the six config keys, and explicit precedence rules. It is not 5 because step 4 says 'read the highest-signal hits' without naming the read tool or the expected permalink format, leaving small execution gaps. | 4 / 5 |
Workflow Clarity | A clear five-step sequence with explicit error paths — 'if the repository is missing, report that setup is incomplete', 'mention that setup has not been run', and surfacing 'any missing setup or ambiguous project mapping' in the output. It is not 5 because there is no validate→fix→retry analogue (e.g., re-checking the merged checkpoint list), and not 3 because checkpoints are explicit rather than implicit. | 4 / 5 |
Progressive Disclosure | The skill is short (~40 lines of prose) with no bundle files and nothing that clearly belongs in a separate file; the title, intro, and '## Steps' sections are well-organized and self-contained. Per the simple-skill scoring note ('under 50 lines with no need for external references... can score 5 with just well-organized sections'), this fits the top anchor. | 5 / 5 |
Total | 17 / 20 Passed |