Content
68%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 compact, well-structured skill body that leads with non-obvious rationale and gives concrete ripgrep commands plus a sensible first-time setup flow. Its main weakness is the absence of an explicit validation checkpoint for the batch sync step, which caps workflow clarity.
Suggestions
Add an explicit verification step after the sync (e.g., check that `~/repo-mirror/repos` is non-empty or that the sync script exited 0) before running search commands, to satisfy the batch-operation validation expectation.
Replace the `bash <script-path>` placeholder with a concrete one-liner that captures the found path into a variable and reuses it, making the workflow copy-paste ready.
Confirm the referenced `private-github-search-sync.sh` is actually bundled (e.g., under `scripts/`) and reference it by a direct relative path so the bundle structure matches the body.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence (no explanation of what ripgrep or mirroring is); the opening paragraph justifies the approach with non-obvious facts about GitHub's search engines, which earns its place, with only minor wording that could be trimmed — efficient but not perfectly lean. | 4 / 5 |
Actionability | Provides concrete, executable commands (`find ~/.claude ~/.codex -name ...`, `rg -il 'pattern' ~/repo-mirror/repos`) and a clear first-time setup sequence; falls short of score 5 because `bash <script-path>` and `'pattern'` are placeholders rather than fully copy-paste-ready examples. | 4 / 5 |
Workflow Clarity | The refresh-then-search sequence and first-time setup are clearly ordered, but the batch sync operation (cloning all repos) lacks an explicit validation/verification checkpoint that it succeeded before searching; per the rubric's batch-operation cap, workflow clarity cannot exceed 3. | 3 / 5 |
Progressive Disclosure | Well-organized into clearly signaled sections (Searching, First-time setup, Caveats) with a single one-level-deep reference to a bundled sync script; not a 5 because the script is located dynamically via `find` and no bundle directory is present to confirm the referenced file, leaving a minor organization gap. | 4 / 5 |
Total | 15 / 20 Passed |