Content
75%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.
Highly actionable content with a complete reviewer prompt, exact tool contracts, and concrete report templates, organized into a clear four-step workflow with validation gates. The main weaknesses are redundancy (duplicated calling-convention blocks) and dense, confusing meta-blockquotes that could be trimmed without losing function.
Suggestions
State the manual-review calling convention once (in the Reviewer Calling Convention section) and have Step 2 reference it, removing the near-verbatim duplicate config blocks.
Move the 'Reviewer-Model: <exact-model-id>' manual-response requirement out of Step 1 (artifact collection) into the manual backend branch where it belongs, and simplify the opening Selectable Reviewers blockquote to drop undefined concepts like 'slot's tier' and 'source/image inputs'.
Trim the Integration with Other Skills section to what this skill produces (EXPERIMENT_AUDIT.json and integrity_status) and drop the pseudocode describing /result-to-claim and /paper-write internal logic, which belongs in those skills.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient (exact reviewer prompt, report templates, checklist), but includes unnecessary material: the manual-review calling convention and config block appear twice nearly verbatim (Reviewer Calling Convention section and Step 2), the opening meta-blockquotes reference undefined concepts ('this slot's tier', 'source/image inputs and deterministic gates'), and the Integration section documents other skills' (/result-to-claim, /paper-write) internal behavior. Fits anchor 3 ('could be tightened') rather than 2, since no section is pure padding. | 3 / 5 |
Actionability | Fully executable guidance: a complete copy-paste-ready reviewer prompt with placeholder slots, exact tool names and configs (mcp__codex__codex with 'model_reasoning_effort': 'ultra', sandbox read-only), the full manual-review config, and complete EXPERIMENT_AUDIT.md/.json templates plus a printed-summary format. Matches anchor 5. | 5 / 5 |
Workflow Clarity | Four clearly sequenced, role-labeled steps (Collect Artifacts → Send to Reviewer → Parse/Write Report → Print Summary) with validation gates (REVIEW_UNAVAILABLE on missing/same-family reviewer identity, stop-and-print-install-command when the manual MCP is unavailable) and PASS/WARN/FAIL verdict branching. Not 5 because the reviewer-identity requirement blockquote is misplaced inside Step 1 (artifact collection) and the dense header routing note muddies the workflow entry point. | 4 / 5 |
Progressive Disclosure | The body is well-sectioned with clearly signaled one-level-deep references (shared-references/reviewer-routing.md, review-tracing.md, integration-contract.md, external-cadence.md), and inlining the ~70-line audit prompt is justified since it must be passed verbatim. No bundle files exist (references/, scripts/, assets/ absent) and everything lives in one ~320-line file, so it does not reach anchor 5's clean overview-plus-split structure; minor organization gaps keep it at 4. | 4 / 5 |
Total | 16 / 20 Passed |