Content
85%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 highly actionable, well-orchestrated skill body: every step is executable with concrete commands, validation checkpoints are explicit, and the multi-file bundle is cleanly structured and easy to navigate. The one real weakness is token efficiency — the epistemic posture is repeated across three body sections that also duplicate guidance from the opening blockquote and references/epistemics.md.
Suggestions
Collapse "Mindset", "Operating Posture", and "Core Epistemics" into a single short epistemics section (one summary plus the pointer to references/epistemics.md) — principles like "evidence before narrative", hedging, and "name the gaps" are currently stated three times in the body.
Trim the opening ported-from blockquote to provenance only; its tokensave-first and MCP-mapping guidance is repeated nearly verbatim in Step 2 and the Discovery subsection — state each point once where it's operationally used.
Consider moving the per-category roster detail (what each of the 7 investigators physically contains and surfaces) into references/source-playbook.md or the per-source playbooks, keeping only the spawn/skip decision rules in the body.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient operational content, but the same epistemic principles (evidence before narrative, hedging, naming gaps, no intent from code shape) are stated three times across "Mindset", "Operating Posture", and "Core Epistemics" before the pointer to references/epistemics.md, and the opening blockquote's tokensave/MCP-mapping guidance is repeated in Step 2 and the Discovery subsection. Not anchor 2 because the majority of the 378 lines — tool commands, roster, skip rules, output format, failure modes — is dense non-redundant guidance, not padding. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready guidance: exact tokensave tool signatures (tokensave_blame/log/diff/changelog with parameters), concrete git fallbacks (git blame -L <start>,<end>, git log --follow -p), the full gh pr view --json field list, subagent config (subagent_type, model guidance), and a worked example of the Sources Consulted output lines. | 5 / 5 |
Workflow Clarity | Steps 1–5 (Understand → Code Anchor → Spawn Investigators → Synthesize → Present) are clearly sequenced, with an explicit validation checkpoint in the Verification section ("spot-check at least one citation... Run this the same way you'd run a test before shipping") and feedback loops via the Troubleshooting and Common Failure Modes sections covering contradiction handling, empty results, and misapplied skips. | 5 / 5 |
Progressive Disclosure | The body holds orchestration while details are split one level deep into verified, well-signaled files: epistemics.md (confidence framework), investigator-prompt.md and synthesizer-prompt.md (templates), and a per-category playbook index (source-playbook.md → sources/*.md, which SKILL.md also references directly), with a Reference Files section mapping each file to its purpose. All referenced paths exist in the bundle. | 5 / 5 |
Total | 18 / 20 Passed |