Content
63%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 a well-structured overview with genuinely actionable tool-selection and vetting guidance, and it correctly delegates deep parameter detail to a canonical reference. Its weaknesses are internal duplication (local-vs-Glean point and filter criteria stated twice) and references that point to files absent from the bundle.
Suggestions
Delete the 'Key differentiator' section (or fold its three example questions into 'When to reach for this') — the opening paragraph already makes the local-vs-cross-repo distinction, and the section repeats it verbatim.
Remove the redundant 'Filter out:' sentence in the Vetting section — the Quality/Recency/Relevance tests and the section's first line already cover those criteria.
Ship the referenced files (reference/exploration.md, reference/plan-prep.md) in the bundle or inline their one-line summaries, so the clearly-signaled links are not dangling.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient with no over-explanation of known concepts, but it could be tightened: the local-vs-Glean distinction is stated in the opening paragraph ("Local tools (`Grep`, `Glob`, `Read`) see only the current repo") and repeated almost verbatim in the "Key differentiator" section, and the Vetting section's "Filter out:" line duplicates the criteria already given in the Quality/Recency/Relevance tests and the section's opening sentence. Not 4 because the duplication is noticeable across three sections, not minor trimming. | 3 / 5 |
Actionability | Concrete guidance throughout: exact tool names (`code_search`, `search`, `employee_search`, `read_document`) in a selection table keyed to user intent, vetting criteria with specific thresholds ("Commits in past 3 months", "12+ months"), and specific presentation rules (full URL, last-update date, group by repo, flag drift). Per the code_vs_instruction note, absent code is not penalized since the guidance is actionable; it falls short of 5 only because no example `code_search` invocation or concrete search pattern appears inline — full param shape is delegated to the reference file. | 4 / 5 |
Workflow Clarity | The sections read in workflow order (when to reach for it → tool selection → vetting → output expectations) and vetting functions as an explicit validation checkpoint with pass/fail tiers (GOOD/ACCEPTABLE/POOR, ACTIVE/SLOWING/STALE, RELEVANT/RELATED/TANGENTIAL) before surfacing results. Not 5 because the search → vet → present sequence is implicit rather than stated as steps, and there is no error-recovery guidance (e.g., what to do when a search returns nothing or only POOR matches). Not a destructive/batch skill, so no validation cap applies. | 4 / 5 |
Progressive Disclosure | References are clearly signaled, one level deep, with descriptive labels, and the body stays an overview — but the bundle contains no files at all, so the promised `reference/exploration.md` and `reference/plan-prep.md` (plus the cross-skill `../using-glean/reference/...` paths) are dangling. Not 4 because missing own-bundle reference files are more than a minor organization gap: navigation to the referenced workflow material fails as shipped. | 3 / 5 |
Total | 14 / 20 Passed |