Content
77%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 highly actionable, well-sequenced autonomous workflow with real validation checkpoints and state recovery — workflow clarity and actionability are exemplary. Its weaknesses are token efficiency (verbatim duplicated SCOPE LIMITS blocks and polling instructions, editorial commentary in constants) and progressive disclosure (a long monolithic file whose prompt templates and scope rules should live in reference files).
Suggestions
Factor the ~20-line SCOPE LIMITS block, currently duplicated verbatim in the Phase A and Round 2+ prompt templates, into a single named constant or a referenced file to cut ~20 lines of pure duplication.
Move the full Phase A and Round 2+ prompt templates into a references/ file (e.g. references/prompts.md), keeping only the tool names and polling contract inline in SKILL.md.
Delete the editorial meta-commentary from the POSITIVE_THRESHOLD constant ("Earlier wording used 'or' + a stale verdict set...") — it documents revision history, not the rule itself.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly instruction-dense rather than explanatory, but it carries real padding: the ~20-line SCOPE LIMITS block is duplicated verbatim in both the Phase A and Round 2+ templates, the MCP polling instructions are repeated word-for-word, and the POSITIVE_THRESHOLD constant embeds editorial meta-commentary ("Earlier wording used 'or' + a stale verdict set; the AND form is authoritative"). It is not 2 because nearly all content is genuine operational guidance rather than concepts Claude already knows, and not 4 because the duplicated blocks are substantial removable tokens. | 3 / 5 |
Actionability | Fully executable throughout: exact MCP tool names (mcp__gemini-review__review_start / review_status / review_reply_start) with jobId/threadId handling, a concrete JSON schema for REVIEW_STATE.json, a copy-paste markdown template for documenting rounds, a literal human-checkpoint message template, and explicit input-parsing rules ("go"/"continue"/"skip 1,3"/"stop"). This matches the 5 anchor — copy-paste-ready commands and templates covering the common cases. | 5 / 5 |
Workflow Clarity | The sequence is explicit — Initialization (with resume/stale-state validation branches) → Loop Phases A-E → Termination — with an exact STOP CONDITION (score >= 6 AND verdict in {"ready","almost"}), a state file written every Phase E for recovery, and a true feedback loop (review → implement fixes → re-review). This matches the 5 anchor: clear sequencing with explicit validation checkpoints and error-recovery handling. | 5 / 5 |
Progressive Disclosure | The body is well-sectioned with clear headers, but it is a single ~300-line file with no bundle files (references/, scripts/, assets/ absent), and large blocks that belong in a reference are inlined — the duplicated SCOPE LIMITS prompt text and both full prompt templates. The Output Protocols links to ../../shared-references/*.md are one level deep but those files are not present in this bundle, so navigation cannot be verified. It is not 2 because structure and signaling are decent, and not 4 because significant content that should be split out is inline. | 3 / 5 |
Total | 16 / 20 Passed |