1---2name: research-review-loop3description: Run iterative adversarial review over research plans, experiment outputs, and drafts with claim ledgers, issue tracking, evidence checks, and explicit closure criteria. Use when asked to red-team a research artifact across multiple rounds, maintain issue state across revisions, or pressure-test whether revised results and prose actually support a claim. Prefer `research-paper-review` for an initial single-paper critique or OCR/extraction workflow, and prefer `research-rebuttal` when concrete external reviewer comments already exist and the task is to draft a venue response.4---56# Research Review Loop78## Quick start9101. Identify the target artifact, stakes, intended audience, and requested evidence class.112. Initialize review state with `scripts/init_review_loop.py` unless a review pack already exists. Use `--from-paper-review <review_dir-or-final_issues.json>` when a first-pass paper-review bundle is available.123. Build a claim ledger and a predecessor-failure ledger before issuing conclusions.134. Review for internal consistency, evidence quality, external verifiability, provenance, non-vacuity, complete outcome accounting, and real rather than nominal independence.145. Preserve previous rounds and append or version new outputs instead of overwriting prior review artifacts.156. Update the latest `REVIEW_STATE.json`, `AUTO_REVIEW.md`, and `NARRATIVE_REPORT.md` after each round only after preserving prior versions.167. If a `research-paper-review` bundle exists, import its issue IDs, quotes, ratings, and summary path rather than re-summarizing or downgrading the first-pass critique.178. Infer the provenance of existing review artifacts before reorganizing them: upstream Claude/OpenAIReview bundles, Codex-native paper-review bundles, and review-loop hybrids are all valid inputs.189. If the user asks for project-level sequencing, current-state inspection, or coordination across multiple research stages, invoke `research-pipeline-planner` first instead of treating the review loop as the whole task.1910. Use `../research-pipeline-planner/references/epistemic-assurance-contract.md` whenever a round is expected to authorize evidence promotion or route advancement.2021## Relationship to sibling skills2223- `research-paper-review` should usually run before this skill when the artifact is a paper that has not yet received a first-pass critique.24- `research-review-loop` owns tracked iterative review: issue carry-forward, resolution checks, accepted risks, and round discipline.25- `research-results-auditor` owns detailed validity checks over concrete result artifacts; invoke it when a route decision turns on experiment outputs.26- `research-rebuttal` owns responses to external reviewer comments and venue-constrained discussion artifacts.27- `adversarial-doc-review` is broader and lighter-weight; use it for one-off document critique without a tracked research review state.2829## Input contract3031- Minimum:32 - one concrete artifact under review33- Prefer:34 - an existing review pack or tracked issue state35 - upstream Claude/OpenAIReview or Codex-adapted paper-review artifacts such as `summary.md`, `final_issues.json`, `review_summary.json`, `overall_assessment.txt`, `metadata.json`, `comments/`, `context/`, and `sections/`36 - revision diffs or an explicit statement of what changed since the last round37 - experiment plans, result artifacts, decision logs, failure reviews, and provenance records38 - current and requested evidence class plus the claimed independence arrangement3940## Output contract4142- For future research-review-loop runs: preserve previous rounds and append/version new outputs instead of overwriting prior review artifacts.43- Preserve prior rounds. Before writing a new round, copy, archive, or write into a round-specific directory so earlier `REVIEW_STATE.json`, `AUTO_REVIEW.md`, and `NARRATIVE_REPORT.md` artifacts remain available.44- Primary tracked artifacts:45 - `REVIEW_STATE.json`46 - `AUTO_REVIEW.md`47 - `NARRATIVE_REPORT.md`48- If upstream `paper-review` artifacts exist, keep explicit references to their file paths in the round state rather than rewriting the whole first-pass critique from scratch.49- Preserve `impact_rating`, `confidence_rating`, `severity`, `quote`, `source_section`, and `related_sections` from `paper-review/final_issues.json` when importing first-pass paper-review issues. Map `impact_rating >= 4` to major-tracked issues, `impact_rating == 3` to moderate tracked issues, and `impact_rating <= 2` to minor tracked issues; if only upstream `severity` is present, preserve it as the tracked severity and leave numeric ratings null.50- If the first-pass bundle lacks a root `review_summary.json` but has `metadata.json.round_summaries` or `round-N/review_summary.json`, use the latest round summary as the numeric/currentness summary and record the exact path in `source_artifacts`.51- For any round that authorizes advancement, record the bounded verdict, current evidence class, requested evidence class, actual independence dimensions, unresolved predecessor failures, and the exact evidence supporting each closure.5253## Hard stops5455- Do not authorize confirmatory or high-stakes advancement solely because all required files exist, validators pass, or the artifacts agree internally.56- Do not mark a material assurance issue resolved by renaming a role, adding a field, changing a flag, replacing an artifact, or reclassifying the route without new evidence.57- Do not call a review independent when the same effective process controls context, hidden data, implementation, evaluation, and advancement authority.58- If a prior round identified hidden-truth leakage, vacuous comparisons, incomplete loss accounting, omitted failures, or self-attested controls, carry the issue forward until evidence resolves it or the claim is explicitly narrowed.59- Continue useful exploratory review when stronger assurance is unavailable, but do not silently promote the evidence class.6061## Workflow6263### 1) Treat review as an iterative stateful process6465- Carry unresolved, resolved, and accepted issues across rounds.66- Never collapse multiple review rounds into one untracked summary.67- Never overwrite earlier round artifacts without first preserving them in a versioned or timestamped location.68- Require each issue to have severity or impact rating, confidence, status, evidence, and a concrete fix or follow-up.69- If `paper-review/final_issues.json` exists, initialize the first tracked issue set from that file instead of inventing a new initial ledger. Keep the paper-review issue title, quote, explanation, source section, and ratings traceable in the state.70- Before deciding where to write the next round, inspect `metadata.json`, `artifact-index.md`, and existing `round-N/` folders. Continue the latest round number instead of flattening root paper-review files into a round folder.71- Record which agent, model, prompt/context, code path, data access, and authority produced each stage-critical artifact when that provenance is available.7273### 2) Build the claim and failure ledgers first7475- Extract definitions, assumptions, factual claims, quantitative claims, causal claims, speculative claims, control claims, and route-authorization claims.76- Keep each claim traceable to a file location, section, figure, table, result artifact, or decision log.77- Distinguish unsupported from false, and ambiguous from misleading.78- For every material predecessor failure, record:79 - original failure and evidence80 - affected claim or route81 - current status82 - new evidence, accepted-risk rationale, or improper reclassification83- Reclassification, replacement, or omission is not resolution.8485### 3) Review in three passes8687- Internal pass:88 - consistency89 - terminology90 - logic91 - methodology92 - claim-to-evidence alignment93- Assurance pass:94 - evidence class and selection history95 - complete decision or loss contract96 - non-vacuity and discriminating cases97 - hidden-information controls98 - successes, errors, omissions, skips, nulls, retries, timeouts, and initial failures99 - property-versus-label checks for locks, isolation, replay, role independence, and validators100 - actual independence across context, data, implementation, evaluation, and advancement authority101- External pass:102 - time-sensitive facts103 - citations104 - benchmark claims105 - standards and rules106- If external verification is unavailable, mark items as unverified and say what evidence would resolve them.107108### 4) Force closure discipline109110- Do not mark a major issue resolved without new evidence, a revised artifact, or an explicit accepted risk.111- Treat “future work” as a resolution only when the claim has been narrowed accordingly.112- Do not resolve an imported paper-review issue by paraphrase alone; cite the revision diff, new analysis, narrowed claim, or accepted-risk rationale that changes its status.113- Do not resolve an assurance issue because a self-attested field now says the desired property holds; inspect the mechanism and evidence.114- An accepted risk must identify the unresolved failure, justify acceptance, narrow the affected claim or route, and remain visible in later rounds.115- Prefer fewer high-signal issues over long undifferentiated lists.116117### 5) Authorize advancement conservatively118119- Separate structural validity, internal consistency, exploratory usefulness, confirmatory support, and independent verification.120- Require `research-results-auditor` when advancement depends on concrete experimental outputs that have not received a validity audit.121- State the strongest remaining objection before the route decision.122- Use `proceed`, `revise`, `narrow evidence class`, or `stop`; never use an unqualified “passed” when the bounded meaning matters.123- If the runtime cannot provide material independence, record self-review and limit the verdict accordingly.124125### 6) Keep the report actionable126127- Use `references/report-template.md` for each round.128- Separate major issues from minor issues and open questions.129- Include targeted rewrites when wording is the real problem.130- Record what changed since the prior round and why each formerly-open issue is now resolved, deferred, accepted, narrowed, or still open.131- List the minimum evidence needed to reach the next stronger evidence class.132133## References134135- `references/review-checklist.md`136- `references/report-template.md`137- `references/review-state-schema.md`138- `references/tabmol-ddi-ood-adapter.md`139- `../research-pipeline-planner/references/review-stage-contract.md`140- `../research-pipeline-planner/references/epistemic-assurance-contract.md`141142## Script143144- `scripts/init_review_loop.py`: create deterministic state and report scaffolds for repeated review rounds; optionally seed open issues from `research-paper-review` output with `--from-paper-review`.