CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

auto-review-loop

Autonomous multi-round research review loop. In Copilot CLI it defaults to the native complementary rubber-duck subagent with host-event model evidence; elsewhere it uses Codex, while explicit external reviewer overrides remain available. Implements fixes and re-reviews until a policy-approved positive assessment or max rounds is reached.

Quality

62%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Passed

No findings from the security scan

Fix and improve this skill with Tessl

tessl review fix ./skills/auto-review-loop/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

52%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

The body specifies an elaborate, fail-closed multi-backend review loop with genuinely concrete commands, schemas, and validation gates, but as a ~1,200-line monolith it duplicates prompt templates and routing rules verbatim multiple times and inlines everything rather than splitting it across referenced files (which are not shipped in the bundle). It is actionable and well-validated yet token-inefficient and hard to navigate.

Suggestions

Extract the verbatim-repeated 'SCOPE LIMITS' block (repeated 5 times) and the backend calling conventions into a single referenced file (e.g. references/review-prompts.md) and include each prompt template exactly once.

Trim toward ~500 lines by moving the REVIEW_STATE/ACQUITTAL schemas, the review_gate bash block, and the Stop-Gate State-Transition Tests list into references/ or into the helper scripts they document.

Delete the internal correction history in POSITIVE_THRESHOLD ('Earlier wording here used `or` and a stale verdict set ...') and state only the authoritative rule; also ship or inline the ../shared-references/ files the workflow depends on so the skill is self-contained.

DimensionReasoningScore

Conciseness

A ~1,200-line monolith with substantial verbatim duplication: the 25-line 'SCOPE LIMITS' block appears five times across prompt templates, the copilot routing contract is stated twice (Reviewer Calling Convention and Phase A), and the ACQUITTAL_LOG.jsonl receipt schema appears twice; the body also carries self-referential correction history ('Earlier wording here used `or` and a stale verdict set ... that was an internal inconsistency'). Not 1 because it does not explain concepts Claude already knows — the verbosity is duplication and spec sprawl, not padding with known material.

2 / 5

Actionability

Highly concrete: exact MCP call signatures with config JSON, a copy-paste bash block resolving and invoking review_gate.py, exact artifact naming patterns ('<run_id>_r<round>_review_<8-random-hex>'), and exact JSONL schemas. Not 5 because several commands depend on helper scripts (review_gate.py, copilot_native_evidence.py, save_trace.sh) and shared-references files that are not part of this bundle, and the review prompts remain placeholder templates ('<path(s)>', '[paste executor's rebuttal]').

4 / 5

Workflow Clarity

Explicit sequencing (Initialization -> Step -1/0 -> Phases A-E -> Termination) with genuine validation checkpoints (fail-closed REVIEW_UNAVAILABLE, native-evidence verification, gate recomputation after rebuttals) and feedback loops (review -> fix -> re-review, validate -> fix -> re-validate). Not 5 because the state-transition logic is fragmented across Constants, Step 0 snapshots, Phases B.5.1 and E, with round_backend vs reviewer_backend bookkeeping intricate enough that the document itself acknowledges a prior internal inconsistency in the stop condition.

4 / 5

Progressive Disclosure

No references/, scripts/, or assets/ directories exist in the bundle; full prompt templates, bash scripts, JSON state schemas, and a state-transition test list are all inlined in one 1,200-line file, and the linked files (../shared-references/reviewer-routing.md, integration-contract.md, reviewer-backends.md, review-tracing.md, output-versioning.md, output-manifest.md, output-language.md, external-cadence.md) live outside the skill directory and are not present. Not 1 because section headers and clearly labeled markdown links do provide structure, but the dominant trait is that large blocks which clearly belong in separate reference files are inlined.

2 / 5

Total

12

/

20

Passed

Description

58%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

The description communicates a concrete and fairly distinctive capability (an autonomous multi-round review-fix-re-review loop driven by an independent cross-model reviewer) but omits any explicit when-to-use trigger guidance and leans heavily on internal ecosystem jargon ('rubber-duck subagent', 'host-event model evidence', 'policy-approved') instead of natural user terms. It answers 'what' clearly while leaving 'when' unstated.

Suggestions

Add an explicit 'Use when...' clause naming natural triggers, e.g. 'Use when the user wants an independent model to critique research work, verify claims against code and results, or iterate fixes until a review passes.'

Replace internal jargon ('native complementary rubber-duck subagent with host-event model evidence', 'policy-approved positive assessment') with user-facing language and keep backend-routing detail in the body where it belongs.

Include common synonyms users would naturally say — 'review my paper/repo', 'critique', 'adversarial review', 'reviewer loop', 'improve until it passes' — so the description matches real trigger phrasing.

DimensionReasoningScore

Specificity

Lists several concrete actions ('Implements fixes and re-reviews until a policy-approved positive assessment or max rounds is reached') plus backend-specific behavior, but coverage is diluted by internal jargon like 'native complementary rubber-duck subagent with host-event model evidence'. Not 5 because that jargon obscures the full action inventory; not 3 because it goes well beyond 1-2 concrete actions.

4 / 5

Completeness

Has a clear 'what' (autonomous review -> implement fixes -> re-review loop with an explicit stop condition) but no 'Use when...' clause or equivalent explicit trigger guidance, which caps completeness at 3 per the rubric guideline. Not 4/5 because 'when' is entirely absent rather than merely implicit.

3 / 5

Trigger Term Quality

Contains relevant keywords ('research review loop', 're-reviews', 'reviewer', 'fixes') but misses the natural phrases a user would actually say ('review my paper', 'critique my work', 'adversarial review'), and much of the text is backend vocabulary ('Codex', 'Copilot CLI', 'host-event model evidence') rather than trigger terms. Not 4 because common user variations and synonyms are largely absent.

3 / 5

Distinctiveness Conflict Risk

'Autonomous multi-round research review loop' with cross-model independent-reviewer routing is a fairly distinct niche with distinct triggers, carrying only minor overlap risk with generic review/critique skills. Not 5 because without an explicit 'when' clause, distinctiveness rests on ecosystem jargon rather than clearly stated trigger context.

4 / 5

Total

14

/

20

Passed

Validation

75%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 12 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

skill_md_line_count

SKILL.md is long (1212 lines); consider splitting into references/ and linking

Warning

allowed_tools_field

'allowed-tools' contains unusual tool name(s)

Warning

frontmatter_unknown_keys

Unknown frontmatter key(s) found; consider removing or moving to metadata

Warning

relative_links

Relative link issues: 6 suspicious

Warning

Total

12

/

16

Passed

Repository

wanshuiyin/Auto-claude-code-research-in-sleep

Path

skills/auto-review-loop/SKILL.md

Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.