@magnusekdahl/parallix 1.2.1 → 1.3.1

package/lib/review/review-state.ts

@@ -0,0 +1,353 @@
+/**
+ * Reviewer-family persistence for the autonomous review loop.
+ *
+ * State is stored under the adapter-resolved mission directory on the mission branch.
+ * Writing and committing state before each round ensures session-restart safety.
+ *
+ * Owned by the Node workflow harness (ADR 0037 / task-089).
+ */
+
+import * as fs from 'fs';
+import * as path from 'path';
+import { git } from '../core/git.js';
+import { findMissionDir, resolveWorktree } from '../core/mission-utils.js';
+import { log } from '../core/fmt.js';
+
+/**
+ * Return the path to the review-state file for a given slug.
+ * Resolves using the same mission-dir discovery as other workflow commands.
+ *
+ * @param {string} slug
+ * @param {string} [rootDir]
+ * @returns {string|null}  Absolute path, or null if mission dir not found
+ */
+export function reviewStateFile(slug: string, rootDir = process.cwd()): string | null {
+  const missionDir = findMissionDir(slug, rootDir);
+  if (!missionDir) { return null; }
+  return path.join(missionDir, 'review-state.json');
+}
+
+/**
+ * Read the persisted review state for a mission.
+ *
+ * @param {string} slug
+ * @param {string} [rootDir]  Directory to resolve the mission from (defaults to process.cwd())
+ * @returns {ReviewState|null}
+ */
+export function readReviewState(slug: string, rootDir = process.cwd()): ReviewState | null {
+  const statePath = reviewStateFile(slug, rootDir);
+  if (!statePath || !fs.existsSync(statePath)) { return null; }
+
+  try {
+    const raw = fs.readFileSync(statePath, 'utf8');
+    const parsed = JSON.parse(raw);
+    if (parsed && parsed.reviewer && parsed.implementer) {
+      return new ReviewState(slug, parsed);
+    }
+    return null;
+  } catch {
+    return null;
+  }
+}
+
+/**
+ * Resolve the review-provider identity for a review workflow path.
+ *
+ * The normal contract is review-state-backed so standard review/handoff/rebase
+ * paths do not require a provider identity env var to be exported.
+ *
+ * @param {string} slug
+ * @param {string} [rootDir]
+ * @param {{readReviewStateFn?: Function}} [options]
+ * @returns {{ identityUser: string|null, commentIdentityUser: string|null, forgejoUser: string|null, commentForgejoUser: string|null, reviewState: ReviewState|null, source: 'review-state'|null }}
+ */
+export function resolveReviewIdentity(
+  slug: string,
+  rootDir = process.cwd(),
+  options: { readReviewStateFn?: (s: string, r: string) => ReviewState | null } = {}
+): {
+  identityUser: string | null;
+  commentIdentityUser: string | null;
+  forgejoUser: string | null;
+  commentForgejoUser: string | null;
+  reviewState: ReviewState | null;
+  source: 'review-state' | null;
+} {
+  const readReviewStateFn = options.readReviewStateFn || readReviewState;
+  const reviewState = readReviewStateFn(slug, rootDir);
+  const reviewerUser = reviewState ? (reviewState.reviewer || reviewState.implementer || null) : null;
+  const implementerUser = reviewState ? (reviewState.implementer || reviewState.reviewer || null) : null;
+
+  return {
+    identityUser: reviewerUser || null,
+    commentIdentityUser: implementerUser || null,
+    forgejoUser: reviewerUser || null,
+    commentForgejoUser: implementerUser || null,
+    reviewState,
+    source: reviewerUser ? 'review-state' : null
+  };
+}
+
+/**
+ * Manager class for mission review state.
+ * Expanded to include phase, disposition, and metadata for canonical state ownership.
+ */
+export const VALID_PHASES = ['reviewing', 'fixing', 'pending-approval', 'approved'] as const;
+const PHASE_ALIASES = new Map<string, string>([
+  ['review', 'reviewing'],
+  ['rewiewing', 'reviewing'],
+  ['fix', 'fixing'],
+  ['pending_approval', 'pending-approval'],
+  ['pending approval', 'pending-approval']
+]);
+
+const PHASE_TRANSITIONS: Record<string, string[]> = {
+  'reviewing': ['fixing', 'approved'],
+  'fixing': ['reviewing', 'pending-approval'],
+  'pending-approval': ['reviewing'],
+  'approved': []
+};
+
+/** @param {string} disposition */
+function inferPhaseFromDisposition(disposition: string): string {
+  switch (String(disposition || '').trim().toUpperCase()) {
+  case 'APPROVED':
+    return 'approved';
+  case 'REQUEST_CHANGES':
+  case 'COMMENT':
+  case 'PUSHBACK_ALL':
+  case 'BLOCKED':
+  case 'PARKED':
+    return 'fixing';
+  case 'CHANGES_MADE':
+    return 'reviewing';
+  default:
+    return 'reviewing';
+  }
+}
+
+/** @param {string} phase @param {string} disposition */
+export function normalizeReviewPhase(phase: string, disposition: string): { phase: string; original: string | null; normalized: boolean } {
+  if (!phase) {
+    return { phase: 'reviewing', original: null, normalized: false };
+  }
+
+  const raw = String(phase).trim();
+  const canonical = raw.toLowerCase().replace(/[_\s]+/g, '-');
+  if ((VALID_PHASES as readonly string[]).includes(canonical)) {
+    return { phase: canonical, original: raw, normalized: canonical !== raw };
+  }
+
+  const alias = PHASE_ALIASES.get(raw.toLowerCase()) || PHASE_ALIASES.get(canonical);
+  if (alias) {
+    return { phase: alias, original: raw, normalized: true };
+  }
+
+  return {
+    phase: inferPhaseFromDisposition(disposition),
+    original: raw,
+    normalized: true
+  };
+}
+
+/** @typedef {{reviewer?: string, implementer?: string, round?: number, startedAt?: string, phase?: string, disposition?: string | null, reviewerRetryCount?: number, implementerRetryCount?: number, metadata?: {[key: string]: unknown}}} ReviewStateData */
+export interface ReviewStateData {
+  reviewer?: string;
+  implementer?: string;
+  round?: number;
+  startedAt?: string;
+  phase?: string;
+  disposition?: string | null;
+  reviewerRetryCount?: number;
+  implementerRetryCount?: number;
+  metadata?: Record<string, unknown>;
+}
+
+export class ReviewState {
+  slug: string;
+  reviewer?: string;
+  implementer?: string;
+  round: number;
+  startedAt: string;
+  phase: string;
+  disposition: string | null;
+  reviewerRetryCount: number;
+  implementerRetryCount: number;
+  metadata: Record<string, unknown>;
+  phaseOriginal: string | null;
+
+  /**
+   * @param {string} slug
+   * @param {ReviewStateData} [data]
+   */
+  constructor(slug: string, data: ReviewStateData = {}) {
+    const phaseInfo = normalizeReviewPhase(data.phase || '', data.disposition || '');
+    this.slug = slug;
+    this.reviewer = data.reviewer;
+    this.implementer = data.implementer;
+    this.round = data.round || 1;
+    this.startedAt = data.startedAt || new Date().toISOString();
+    this.phase = phaseInfo.phase;
+    this.disposition = data.disposition || null;
+    this.reviewerRetryCount = data.reviewerRetryCount || 0;
+    this.implementerRetryCount = data.implementerRetryCount || 0;
+    this.metadata = data.metadata || {};
+    this.phaseOriginal = phaseInfo.normalized ? phaseInfo.original : null;
+  }
+
+  /**
+   * @param {string} slug
+   * @param {ReviewState|ReviewStateData} dataOrInstance
+   * @returns {ReviewState}
+   */
+  static from(slug: string, dataOrInstance: ReviewState | ReviewStateData): ReviewState {
+    if (dataOrInstance instanceof ReviewState) {
+      if (dataOrInstance.slug !== slug) {
+        throw new Error(`ReviewState slug mismatch: expected "${slug}", got "${dataOrInstance.slug}"`);
+      }
+      return dataOrInstance;
+    }
+    return new ReviewState(slug, dataOrInstance || {});
+  }
+
+  /**
+   * @param {string} nextPhase
+   * @returns {ReviewState}
+   */
+  transitionTo(nextPhase: string): ReviewState {
+    if (!(VALID_PHASES as readonly string[]).includes(nextPhase)) {
+      throw new Error(`Invalid phase: "${nextPhase}". Valid: ${(VALID_PHASES as readonly string[]).join(', ')}`);
+    }
+    const allowedArr = PHASE_TRANSITIONS[this.phase];
+    if (!allowedArr || !allowedArr.includes(nextPhase)) {
+      throw new Error(`Cannot transition from "${this.phase}" to "${nextPhase}". Allowed: ${(allowedArr || []).join(', ') || 'none'}`);
+    }
+    this.phase = nextPhase;
+    return this;
+  }
+
+  advanceRound(): ReviewState {
+    this.round += 1;
+    this.phase = 'reviewing';
+    this.disposition = null;
+    this.reviewerRetryCount = 0;
+    this.implementerRetryCount = 0;
+    this.startedAt = new Date().toISOString();
+    return this;
+  }
+
+  /**
+   * @returns {{reviewer?: string, implementer?: string, round?: number, startedAt?: string, phase?: string, disposition?: string, reviewerRetryCount?: number, implementerRetryCount?: number, metadata?: {[key: string]: unknown}}}
+   */
+  toJSON(): {
+    reviewer?: string;
+    implementer?: string;
+    round?: number;
+    startedAt?: string;
+    phase?: string;
+    disposition?: string;
+    reviewerRetryCount?: number;
+    implementerRetryCount?: number;
+    metadata?: Record<string, unknown>;
+  } {
+    const payload: {
+      reviewer?: string;
+      implementer?: string;
+      round?: number;
+      startedAt?: string;
+      phase?: string;
+      disposition?: string;
+      reviewerRetryCount?: number;
+      implementerRetryCount?: number;
+      metadata?: Record<string, unknown>;
+    } = {
+      reviewer: this.reviewer,
+      implementer: this.implementer,
+      round: this.round,
+      startedAt: this.startedAt,
+      phase: this.phase,
+      disposition: this.disposition || undefined
+    };
+
+    if (this.reviewerRetryCount > 0) { payload.reviewerRetryCount = this.reviewerRetryCount; }
+    if (this.implementerRetryCount > 0) { payload.implementerRetryCount = this.implementerRetryCount; }
+    if (Object.keys(this.metadata || {}).length > 0) { payload.metadata = this.metadata; }
+
+    return payload;
+  }
+
+  save(
+    worktree = resolveWorktree(this.slug) || process.cwd(),
+    gitFn = git
+  ): boolean {
+    const statePath = reviewStateFile(this.slug, worktree);
+    if (!statePath) { return false; }
+
+    const payload = this.toJSON();
+    fs.writeFileSync(statePath, JSON.stringify(payload, null, 2) + '\n', 'utf8');
+
+    const relPath = path.relative(worktree, statePath);
+    const result = gitFn(['-C', worktree, 'add', relPath]);
+    const msg = `review-state(${this.slug}): round ${this.round} (${this.phase}) [${this.reviewer} -> ${this.implementer}]${this.disposition ? ` disposition=${this.disposition}` : ''}`;
+    const commitResult = gitFn(['-C', worktree, 'commit', '-m', msg]);
+
+    if (commitResult.status !== 0) {
+      const statusResult = gitFn(['-C', worktree, 'status', '--porcelain', relPath]);
+      if (statusResult.stdout.trim() === '') {
+        return true;
+      }
+      log.warn(`Failed to commit review state update: ${commitResult.stderr}`);
+    }
+
+    return true;
+  }
+}
+
+/**
+ * Write and commit the review state for a mission.
+ * Commits to the current branch so the state is visible after session restarts.
+ *
+ * @param {string} slug
+ * @param {ReviewState|object} state
+ * @returns {boolean}  true if written successfully
+ */
+export function writeReviewState(
+  slug: string,
+  state: ReviewState | Record<string, unknown>,
+  worktree = resolveWorktree(slug) || process.cwd(),
+  gitFn = git
+): boolean {
+  const instance = state instanceof ReviewState ? state : new ReviewState(slug, state as ReviewStateData);
+  return instance.save(worktree, gitFn);
+}
+
+/**
+ * Delete the review state file for a mission (used by --reset).
+ * Commits the deletion if the file was tracked.
+ *
+ * @param {string} slug
+ * @returns {boolean}  true if a file was deleted
+ */
+export function resetReviewState(
+  slug: string,
+  worktree = resolveWorktree(slug) || process.cwd(),
+  gitFn = git
+): boolean {
+  const statePath = reviewStateFile(slug, worktree);
+  if (!statePath || !fs.existsSync(statePath)) { return false; }
+
+  fs.unlinkSync(statePath);
+
+  const relPath = path.relative(worktree, statePath);
+  try {
+    gitFn(['-C', worktree, 'rm', '--cached', relPath]);
+    const result = gitFn(['-C', worktree, 'commit', '-m', `review-state(${slug}): reset (--reset flag)`]);
+    if (result.status !== 0) {
+      log.warn(`Failed to commit review state reset: ${result.stderr}`);
+    }
+  } catch {
+    // file may not have been tracked yet
+  }
+
+  return true;
+}

package/lib/review/review.js

@@ -1,97 +1,76 @@
+"use strict";
 /**
  * Review Module - Main Entry Point
  * Refactored for task-1201: complexity reduction via service extraction
- * 
+ *
  * This file now serves as a re-export hub for the extracted sub-modules:
- * - review-commands.js (command dispatcher and CLI handlers)
- * - review-loop.js (autonomous review loop orchestration)
- * - review-artifacts.js (artifact handling and posting)
- * - review-polling.js (polling utilities)
+ * - review-commands.ts (command dispatcher and CLI handlers)
+ * - review-loop.ts (autonomous review loop orchestration)
+ * - review-artifacts.ts (artifact handling and posting)
+ * - review-polling.ts (polling utilities)
  */
-
-// Re-export from review-polling module
-const { POLL_TIMEOUT, delay, resolvePollIntervalMs, resolvePollTimeoutMs, formatElapsed, isPollTimeout, pollForReview, pollForDisposition } = require('./review-polling');
-
-// Re-export from review-artifacts module
-const { buildMetadataFooter, reviewArtifactPath, readArtifactFile, deleteArtifactFile, normalizeReviewVerdict, normalizeDisposition, postWorkflowComment, postWorkflowReview, consumeReviewerArtifacts, consumeImplementerArtifacts } = require('./review-artifacts');
-
-// Re-export from review-loop module
-const { maybeUpdateGraphifyBeforeReview, commitSafeMissionArtifacts, rebaseBeforeReviewRound, applyAgentFallback, persistNormalizedPhaseRepair, startReviewLoop, recordStageStatsSafe } = require('./review-loop');
-
-// Re-export from review-commands module
-const {
-  flagValue,
-  readTextFlag,
-  formatStaticReviewFindings,
-  formatStaticReviewSuccess,
-  postStaticReviewComment,
-  performStaticReview,
-  verifyReview,
-  submitForReview,
-  readComments,
-  pushRound,
-  showReviewStatus,
-  commentRound,
-  submitReviewRound,
-  closeMissionPr,
-  createEventHandler,
-  importLegacyHandler,
-  review
-} = require('./review-commands');
-
+// Import from review-polling module
+const review_polling_js_1 = require("./review-polling.js");
+// Import from review-artifacts module
+const review_artifacts_js_1 = require("./review-artifacts.js");
+// Import from review-loop module
+const review_loop_js_1 = require("./review-loop.js");
+// Import from review-commands module
+const review_commands_js_1 = require("./review-commands.js");
+// Import from review-state module
+const review_state_js_1 = require("./review-state.js");
 // ============================================================================
 // Module Exports
 // ============================================================================
-// Set module.exports to the main review function (for backward compatibility)
+// Set the module exports to the main review function (for backward compatibility)
 // then add all other exports as properties
-
-module.exports = review;
-
+const _review = review_commands_js_1.review;
 // From review-polling
-module.exports.POLL_TIMEOUT = POLL_TIMEOUT;
-module.exports.delay = delay;
-module.exports.resolvePollIntervalMs = resolvePollIntervalMs;
-module.exports.resolvePollTimeoutMs = resolvePollTimeoutMs;
-module.exports.formatElapsed = formatElapsed;
-module.exports.isPollTimeout = isPollTimeout;
-module.exports.pollForReview = pollForReview;
-module.exports.pollForDisposition = pollForDisposition;
-
+_review.POLL_TIMEOUT = review_polling_js_1.POLL_TIMEOUT;
+_review.delay = review_polling_js_1.delay;
+_review.resolvePollIntervalMs = review_polling_js_1.resolvePollIntervalMs;
+_review.resolvePollTimeoutMs = review_polling_js_1.resolvePollTimeoutMs;
+_review.formatElapsed = review_polling_js_1.formatElapsed;
+_review.isPollTimeout = review_polling_js_1.isPollTimeout;
+_review.pollForReview = review_polling_js_1.pollForReview;
+_review.pollForDisposition = review_polling_js_1.pollForDisposition;
 // From review-artifacts
-module.exports.buildMetadataFooter = buildMetadataFooter;
-module.exports.reviewArtifactPath = reviewArtifactPath;
-module.exports.readArtifactFile = readArtifactFile;
-module.exports.deleteArtifactFile = deleteArtifactFile;
-module.exports.normalizeReviewVerdict = normalizeReviewVerdict;
-module.exports.normalizeDisposition = normalizeDisposition;
-module.exports.postWorkflowComment = postWorkflowComment;
-module.exports.postWorkflowReview = postWorkflowReview;
-module.exports.consumeReviewerArtifacts = consumeReviewerArtifacts;
-module.exports.consumeImplementerArtifacts = consumeImplementerArtifacts;
-
+_review.buildMetadataFooter = review_artifacts_js_1.buildMetadataFooter;
+_review.reviewArtifactPath = review_artifacts_js_1.reviewArtifactPath;
+_review.readArtifactFile = review_artifacts_js_1.readArtifactFile;
+_review.deleteArtifactFile = review_artifacts_js_1.deleteArtifactFile;
+_review.normalizeReviewVerdict = review_artifacts_js_1.normalizeReviewVerdict;
+_review.normalizeDisposition = review_artifacts_js_1.normalizeDisposition;
+_review.postWorkflowComment = review_artifacts_js_1.postWorkflowComment;
+_review.postWorkflowReview = review_artifacts_js_1.postWorkflowReview;
+_review.consumeReviewerArtifacts = review_artifacts_js_1.consumeReviewerArtifacts;
+_review.consumeImplementerArtifacts = review_artifacts_js_1.consumeImplementerArtifacts;
 // From review-loop
-module.exports.recordStageStatsSafe = recordStageStatsSafe;
-module.exports.maybeUpdateGraphifyBeforeReview = maybeUpdateGraphifyBeforeReview;
-module.exports.commitSafeMissionArtifacts = commitSafeMissionArtifacts;
-module.exports.rebaseBeforeReviewRound = rebaseBeforeReviewRound;
-module.exports.applyAgentFallback = applyAgentFallback;
-module.exports.persistNormalizedPhaseRepair = persistNormalizedPhaseRepair;
-module.exports.startReviewLoop = startReviewLoop;
-
+_review.recordStageStatsSafe = review_loop_js_1.recordStageStatsSafe;
+_review.maybeUpdateGraphifyBeforeReview = review_loop_js_1.maybeUpdateGraphifyBeforeReview;
+_review.commitSafeMissionArtifacts = review_loop_js_1.commitSafeMissionArtifacts;
+_review.rebaseBeforeReviewRound = review_loop_js_1.rebaseBeforeReviewRound;
+_review.applyAgentFallback = review_loop_js_1.applyAgentFallback;
+_review.persistNormalizedPhaseRepair = review_loop_js_1.persistNormalizedPhaseRepair;
+_review.startReviewLoop = review_loop_js_1.startReviewLoop;
 // From review-commands
-module.exports.flagValue = flagValue;
-module.exports.readTextFlag = readTextFlag;
-module.exports.formatStaticReviewFindings = formatStaticReviewFindings;
-module.exports.formatStaticReviewSuccess = formatStaticReviewSuccess;
-module.exports.postStaticReviewComment = postStaticReviewComment;
-module.exports.performStaticReview = performStaticReview;
-module.exports.verifyReview = verifyReview;
-module.exports.submitForReview = submitForReview;
-module.exports.readComments = readComments;
-module.exports.pushRound = pushRound;
-module.exports.showReviewStatus = showReviewStatus;
-module.exports.commentRound = commentRound;
-module.exports.submitReviewRound = submitReviewRound;
-module.exports.closeMissionPr = closeMissionPr;
-module.exports.createEventHandler = createEventHandler;
-module.exports.importLegacyHandler = importLegacyHandler;
+_review.flagValue = review_commands_js_1.flagValue;
+_review.readTextFlag = review_commands_js_1.readTextFlag;
+_review.formatStaticReviewFindings = review_commands_js_1.formatStaticReviewFindings;
+_review.formatStaticReviewSuccess = review_commands_js_1.formatStaticReviewSuccess;
+_review.postStaticReviewComment = review_commands_js_1.postStaticReviewComment;
+_review.performStaticReview = review_commands_js_1.performStaticReview;
+_review.verifyReview = review_commands_js_1.verifyReview;
+_review.submitForReview = review_commands_js_1.submitForReview;
+_review.readComments = review_commands_js_1.readComments;
+_review.pushRound = review_commands_js_1.pushRound;
+_review.showReviewStatus = review_commands_js_1.showReviewStatus;
+_review.commentRound = review_commands_js_1.commentRound;
+_review.submitReviewRound = review_commands_js_1.submitReviewRound;
+_review.closeMissionPr = review_commands_js_1.closeMissionPr;
+_review.createEventHandler = review_commands_js_1.createEventHandler;
+_review.importLegacyHandler = review_commands_js_1.importLegacyHandler;
+// From review-state
+_review.reviewStateFile = review_state_js_1.reviewStateFile;
+_review.ReviewState = review_state_js_1.ReviewState;
+module.exports = _review;

package/lib/review/review.ts

@@ -0,0 +1,138 @@
+/**
+ * Review Module - Main Entry Point
+ * Refactored for task-1201: complexity reduction via service extraction
+ *
+ * This file now serves as a re-export hub for the extracted sub-modules:
+ * - review-commands.ts (command dispatcher and CLI handlers)
+ * - review-loop.ts (autonomous review loop orchestration)
+ * - review-artifacts.ts (artifact handling and posting)
+ * - review-polling.ts (polling utilities)
+ */
+
+// Import from review-polling module
+import {
+  POLL_TIMEOUT,
+  delay,
+  resolvePollIntervalMs,
+  resolvePollTimeoutMs,
+  formatElapsed,
+  isPollTimeout,
+  pollForReview,
+  pollForDisposition
+} from './review-polling.js';
+
+// Import from review-artifacts module
+import {
+  buildMetadataFooter,
+  reviewArtifactPath,
+  readArtifactFile,
+  deleteArtifactFile,
+  normalizeReviewVerdict,
+  normalizeDisposition,
+  postWorkflowComment,
+  postWorkflowReview,
+  consumeReviewerArtifacts,
+  consumeImplementerArtifacts
+} from './review-artifacts.js';
+
+// Import from review-loop module
+import {
+  maybeUpdateGraphifyBeforeReview,
+  commitSafeMissionArtifacts,
+  rebaseBeforeReviewRound,
+  applyAgentFallback,
+  persistNormalizedPhaseRepair,
+  startReviewLoop,
+  recordStageStatsSafe
+} from './review-loop.js';
+
+// Import from review-commands module
+import {
+  flagValue,
+  readTextFlag,
+  formatStaticReviewFindings,
+  formatStaticReviewSuccess,
+  postStaticReviewComment,
+  performStaticReview,
+  verifyReview,
+  submitForReview,
+  readComments,
+  pushRound,
+  showReviewStatus,
+  commentRound,
+  submitReviewRound,
+  closeMissionPr,
+  createEventHandler,
+  importLegacyHandler,
+  review
+} from './review-commands.js';
+
+// Import from review-state module
+import {
+  reviewStateFile,
+  ReviewState
+} from './review-state.js';
+
+// ============================================================================
+// Module Exports
+// ============================================================================
+// Set the module exports to the main review function (for backward compatibility)
+// then add all other exports as properties
+
+const _review = review as any;
+
+// From review-polling
+_review.POLL_TIMEOUT = POLL_TIMEOUT;
+_review.delay = delay;
+_review.resolvePollIntervalMs = resolvePollIntervalMs;
+_review.resolvePollTimeoutMs = resolvePollTimeoutMs;
+_review.formatElapsed = formatElapsed;
+_review.isPollTimeout = isPollTimeout;
+_review.pollForReview = pollForReview;
+_review.pollForDisposition = pollForDisposition;
+
+// From review-artifacts
+_review.buildMetadataFooter = buildMetadataFooter;
+_review.reviewArtifactPath = reviewArtifactPath;
+_review.readArtifactFile = readArtifactFile;
+_review.deleteArtifactFile = deleteArtifactFile;
+_review.normalizeReviewVerdict = normalizeReviewVerdict;
+_review.normalizeDisposition = normalizeDisposition;
+_review.postWorkflowComment = postWorkflowComment;
+_review.postWorkflowReview = postWorkflowReview;
+_review.consumeReviewerArtifacts = consumeReviewerArtifacts;
+_review.consumeImplementerArtifacts = consumeImplementerArtifacts;
+
+// From review-loop
+_review.recordStageStatsSafe = recordStageStatsSafe;
+_review.maybeUpdateGraphifyBeforeReview = maybeUpdateGraphifyBeforeReview;
+_review.commitSafeMissionArtifacts = commitSafeMissionArtifacts;
+_review.rebaseBeforeReviewRound = rebaseBeforeReviewRound;
+_review.applyAgentFallback = applyAgentFallback;
+_review.persistNormalizedPhaseRepair = persistNormalizedPhaseRepair;
+_review.startReviewLoop = startReviewLoop;
+
+// From review-commands
+_review.flagValue = flagValue;
+_review.readTextFlag = readTextFlag;
+_review.formatStaticReviewFindings = formatStaticReviewFindings;
+_review.formatStaticReviewSuccess = formatStaticReviewSuccess;
+_review.postStaticReviewComment = postStaticReviewComment;
+_review.performStaticReview = performStaticReview;
+_review.verifyReview = verifyReview;
+_review.submitForReview = submitForReview;
+_review.readComments = readComments;
+_review.pushRound = pushRound;
+_review.showReviewStatus = showReviewStatus;
+_review.commentRound = commentRound;
+_review.submitReviewRound = submitReviewRound;
+_review.closeMissionPr = closeMissionPr;
+_review.createEventHandler = createEventHandler;
+_review.importLegacyHandler = importLegacyHandler;
+
+// From review-state
+_review.reviewStateFile = reviewStateFile;
+_review.ReviewState = ReviewState;
+
+// Export the review function as the module exports (CJS-compatible)
+export = _review;