Content
86%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, actionable retrieval workflow with executable commands, a fallback for sandbox lock errors, and a clear output format. The only trim opportunity is the repeated MDIR env-expansion prefix in the 'When unsure' section, and the workflow lacks an explicit validation feedback loop though its read-only nature makes that a minor gap.
Suggestions
Factor the repeated `MDIR="${MEMSEARCH_DIR:-$(git rev-parse --show-toplevel 2>/dev/null || pwd)/.memsearch}"` prefix into a single assignment in the 'When unsure what to search' section so the three example commands share it instead of repeating it verbatim.
Add an explicit checkpoint after the Evaluate step (e.g., 'if fewer than 2 results survive, broaden the query and re-search') to give the workflow a concrete feedback loop rather than only an expand-failure fallback.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Largely lean with no concept over-explanation, but the 'When unsure what to search' section repeats the verbatim `MDIR="${MEMSEARCH_DIR:-...}"` prefix three times across consecutive commands — a minor instance of padding that could be factored out; not verbose enough to drop to 3. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready commands with specific flags throughout — `memsearch search "<query>" --top-k 5 --json-output --collection <collection>`, `memsearch expand <chunk_hash> --collection ...`, `bash __INSTALL_DIR__/scripts/parse-rollout.sh <rollout_path>`, plus a fallback `cat <source_file>` — covering the common cases. | 5 / 5 |
Workflow Clarity | Clear numbered sequence (Project Collection → Search → Evaluate → Expand → Deep drill → Return) with an explicit fallback for expand failures, but lacks a formal validate-then-retry feedback loop; the read-only retrieval nature means the destructive/batch cap does not apply, placing this above 3 but below 5. | 4 / 5 |
Progressive Disclosure | Self-contained body under 50 lines with no local bundle files (the `__INSTALL_DIR__/scripts/*` paths are install-dir scripts, not bundle files) and well-organized sections (Project Collection, Steps, When unsure, Output Format), matching the simple-skills exception for clear organization without external references. | 5 / 5 |
Total | 18 / 20 Passed |