@magnusekdahl/parallix 1.2.0 → 1.3.0

package/lib/review/review-commands.js

@@ -1,1471 +1,1318 @@
+"use strict";
 /**
  * Review Commands Module
  * Extracted from parallix/lib/review.js for task-1201
  * Handles command dispatching and CLI command implementations.
  */
-
-const fs = require('fs');
-const path = require('path');
-const os = require('os');
-const fmt = require('../core/fmt');
-const { run, getCurrentBranch } = require('../core/git');
-const { findMissionDir, findMissionArea, resolveWorktree, inferSlug, findCheckpoints, missionBranchName, missionBaseDir } = require('../core/mission-utils');
-const { resolveTaskFile, getTaskStatus, getAcceptanceCriteria, getTaskAssignee, getTaskImplementer, reportTaskResolution, transitionTask } = require('../tools/backlog');
-const { toVirtual } = require('../core/state-map');
-const { getPrStatus, readToken, postComment, createPr, getComments, closePr, resolveReviewUser, isProviderEnabled } = require('./review-adapter');
-const { buildAutonomousReviewMatrix, formatMatrixSummary } = require('../core/runtime-matrix');
-
-const { readReviewState, writeReviewState, resolveReviewIdentity } = require('./review-state');
-const { createEvent, importAllLegacyArtifacts, ALL_EVENT_TYPES,
-        isValidEventType, shouldMirrorToProvider, readAllEvents } = require('./review-events');
-const { startAgent } = require('../agents/agents');
-const { performHandoff } = require('../commands/handoff');
-const { formatVerificationCommand, runVerificationGate } = require('../core/verification');
-const { bootstrapReviewSurface } = require('../tools/setup-review');
-const { resolveReviewAdapter } = require('../core/product-config');
-
-// Import from review-artifacts module
-const { buildMetadataFooter, reviewArtifactPath, postWorkflowComment, postWorkflowReview, consumeReviewerArtifacts } = require('./review-artifacts');
-
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.flagValue = flagValue;
+exports.readTextFlag = readTextFlag;
+exports.formatStaticReviewFindings = formatStaticReviewFindings;
+exports.formatStaticReviewSuccess = formatStaticReviewSuccess;
+exports.postStaticReviewComment = postStaticReviewComment;
+exports.performStaticReview = performStaticReview;
+exports.verifyReview = verifyReview;
+exports.submitForReview = submitForReview;
+exports.readComments = readComments;
+exports.pushRound = pushRound;
+exports.showReviewStatus = showReviewStatus;
+exports.commentRound = commentRound;
+exports.consumeArtifacts = consumeArtifacts;
+exports.submitReviewRound = submitReviewRound;
+exports.closeMissionPr = closeMissionPr;
+exports.createEventHandler = createEventHandler;
+exports.importLegacyHandler = importLegacyHandler;
+exports.review = review;
+const fs = __importStar(require("fs"));
+const path = __importStar(require("path"));
+const os = __importStar(require("os"));
+const fmt = __importStar(require("../core/fmt.js"));
+const git_js_1 = require("../core/git.js");
+const mission_utils_js_1 = require("../core/mission-utils.js");
+const backlog_js_1 = require("../tools/backlog.js");
+const state_map_js_1 = require("../core/state-map.js");
+const review_adapter_js_1 = require("./review-adapter.js");
+const runtime_matrix_js_1 = require("../core/runtime-matrix.js");
+const review_state_js_1 = require("./review-state.js");
+const review_events_js_1 = require("./review-events.js");
+const agents_js_1 = require("../agents/agents.js");
+const verification_js_1 = require("../core/verification.js");
+const setup_review_js_1 = require("../tools/setup-review.js");
+const product_config_js_1 = require("../core/product-config.js");
+const review_artifacts_js_1 = require("./review-artifacts.js");
+const review_loop_js_1 = require("./review-loop.js");
+/** Lazily loaded handoff module. */
+let _handoff = null;
+async function getHandoff() {
+    if (!_handoff) {
+        _handoff = await Promise.resolve().then(() => __importStar(require('../commands/handoff.js')));
+    }
+    return _handoff;
+}
 const DEFAULT_MAX_ATTEMPTS = 5;
-
 // ============================================================================
 // Internal Helpers
 // ============================================================================
-
-/** @param {string[]} args @param {string} flag */
 function flagValue(args, flag) {
-  const idx = args.indexOf(flag);
-  if (idx === -1) {return null;}
-  const val = args[idx + 1];
-  if (!val || val.startsWith('--')) {return null;}
-  return val;
+    const idx = args.indexOf(flag);
+    if (idx === -1) {
+        return null;
+    }
+    const val = args[idx + 1];
+    if (!val || val.startsWith('--')) {
+        return null;
+    }
+    return val;
 }
-
-/**
- * @param {string[]} args
- * @param {string} inlineFlag
- * @param {string} fileFlag
- * @param {string} label
- * @param {{readFileSync?: Function, error?: Function, exit?: Function}} [options]
- * @returns {string|null}
- */
 function readTextFlag(args, inlineFlag, fileFlag, label, options = {}) {
-  const readFileSync = options.readFileSync || fs.readFileSync;
-  const error = options.error || fmt.log.plainError;
-  const exit = options.exit || process.exit;
-  const filePath = flagValue(args, fileFlag);
-  if (filePath) {
-    try {
-      return readFileSync(filePath, 'utf8').trimEnd();
-    } catch (err) {
-      error(`Could not read ${label} from ${fmt.path(filePath)}: ${/** @type {Error} */(err).message}`);
-      exit(1);
-      return null;
-    }
-  }
-
-  return flagValue(args, inlineFlag);
+    const readFileSync = options.readFileSync || fs.readFileSync;
+    const error = options.error || fmt.log.plainError;
+    const exit = options.exit || process.exit;
+    const filePath = flagValue(args, fileFlag);
+    if (filePath) {
+        try {
+            return readFileSync(filePath, 'utf8').trimEnd();
+        }
+        catch (err) {
+            error(`Could not read ${label} from ${fmt.path(filePath)}: ${err.message}`);
+            exit(1);
+            return null;
+        }
+    }
+    return flagValue(args, inlineFlag);
 }
-
-/** @param {string[]} findings */
 function formatStaticReviewFindings(findings) {
-  const lines = [
-    'Static review found the following issue(s) before autonomous review:',
-    ''
-  ];
-  findings.forEach((finding, index) => {
-    lines.push(`${index + 1}. ${finding}`);
-  });
-  lines.push('', 'Auto-launching the act-on-review loop for follow-up.');
-  return lines.join('\n');
+    const lines = [
+        'Static review found the following issue(s) before autonomous review:',
+        ''
+    ];
+    findings.forEach((finding, index) => {
+        lines.push(`${index + 1}. ${finding}`);
+    });
+    lines.push('', 'Auto-launching the act-on-review loop for follow-up.');
+    return lines.join('\n');
 }
-
-/** @param {string} slug */
 function formatStaticReviewSuccess(slug) {
-  return [
-    `Static review for ${slug} found zero issues.`,
-    '',
-    'Checked:',
-    '- mission diff against the primary branch',
-    '- checkpoint presence',
-    '- final checkpoint Goal Check evidence',
-    '',
-    'Mission remains in `review` status awaiting an actual autonomous or peer review verdict.'
-  ].join('\n');
+    return [
+        `Static review for ${slug} found zero issues.`,
+        '',
+        'Checked:',
+        '- mission diff against the primary branch',
+        '- checkpoint presence',
+        '- final checkpoint Goal Check evidence',
+        '',
+        'Mission remains in `review` status awaiting an actual autonomous or peer review verdict.'
+    ].join('\n');
 }
-
-/**
- * @param {string} slug
- * @param {{log?: Function, error?: Function, getTaskStatusFn?: Function, resolveTaskFileFn?: Function, transitionTaskFn?: Function, rootDir?: string}} [options]
- * @returns {{repaired: boolean, skipped?: boolean, currentStatus?: string}}
- */
 function repairStaleActiveTaskAfterReview(slug, options = {}) {
-  const log = options.log || fmt.log.plain;
-  const error = options.error || fmt.log.plainError;
-  const getTaskStatusFn = options.getTaskStatusFn || getTaskStatus;
-  const resolveTaskFileFn = options.resolveTaskFileFn || resolveTaskFile;
-  const transitionTaskFn = options.transitionTaskFn || transitionTask;
-  const rootDir = options.rootDir || process.cwd();
-
-  const taskResolution = resolveTaskFileFn(slug, rootDir);
-  if (!taskResolution.ok) {
-    return { repaired: false, skipped: true };
-  }
-
-  const currentStatus = getTaskStatusFn(taskResolution.taskFile);
-  if (toVirtual(currentStatus) !== 'active') {
-    return { repaired: false, skipped: true, currentStatus };
-  }
-
-  if (!transitionTaskFn(slug, 'review', { rootDir, log })) {
-    error(fmt.status('WARN', `Could not transition backlog task ${slug} to review after recording the review outcome.`));
-    return { repaired: false, skipped: false, currentStatus };
-  }
-
-  return { repaired: true, currentStatus };
+    const log = options.log || fmt.log.plain;
+    const error = options.error || fmt.log.plainError;
+    const getTaskStatusFn = options.getTaskStatusFn || backlog_js_1.getTaskStatus;
+    const resolveTaskFileFn = options.resolveTaskFileFn || backlog_js_1.resolveTaskFile;
+    const transitionTaskFn = options.transitionTaskFn || backlog_js_1.transitionTask;
+    const rootDir = options.rootDir || process.cwd();
+    const taskResolution = resolveTaskFileFn(slug, rootDir);
+    if (!taskResolution.ok) {
+        return { repaired: false, skipped: true };
+    }
+    const currentStatus = getTaskStatusFn(taskResolution.taskFile);
+    if ((0, state_map_js_1.toVirtual)(currentStatus || '') !== 'active') {
+        return { repaired: false, skipped: true, currentStatus: currentStatus ?? undefined };
+    }
+    if (!transitionTaskFn(slug, 'review', { rootDir, log })) {
+        error(fmt.status('WARN', `Could not transition backlog task ${slug} to review after recording the review outcome.`));
+        return { repaired: false, skipped: false, currentStatus: currentStatus ?? undefined };
+    }
+    return { repaired: true, currentStatus: currentStatus ?? undefined };
 }
-
-/**
- * @param {string} slug
- * @param {{worktree?: string, taskFile?: string|null, log?: Function, error?: Function}} options
- * @returns {Promise<{ok: boolean, dirty?: boolean, unsafe?: boolean}>}
- */
 async function commitPersistedReviewOutputs(slug, options = {}) {
-  const { commitSafeMissionArtifacts } = require('./review-loop');
-  return commitSafeMissionArtifacts(slug, options.worktree || process.cwd(), {
-    taskFile: options.taskFile || null,
-    log: options.log || fmt.log.plain,
-    error: options.error || fmt.log.plainError,
-  });
+    return (0, review_loop_js_1.commitSafeMissionArtifacts)(slug, options.worktree || process.cwd(), {
+        taskFile: options.taskFile || null,
+        log: options.log || fmt.log.plain,
+        error: options.error || fmt.log.plainError,
+    });
 }
-
-/**
- * @param {string} slug
- * @param {string} message
- * @param {{log?: Function, error?: Function, resolveTaskFileFn?: Function, getTaskImplementerFn?: Function, resolveWorktreeFn?: Function, readTokenFn?: Function, postCommentFn?: Function, resolveReviewUserFn?: Function, resolveForgejoUserFn?: Function, readReviewStateFn?: Function, buildMetadataFooterFn?: Function, rootDir?: string}} [options]
- * @returns {{ok: boolean, error?: string}}
- */
 function postStaticReviewComment(slug, message, options = {}) {
-  const log = options.log || fmt.log.plain;
-  const error = options.error || fmt.log.plainError;
-  const resolveTaskFileFn = options.resolveTaskFileFn || resolveTaskFile;
-  const getTaskImplementerFn = options.getTaskImplementerFn || getTaskImplementer;
-  const resolveWorktreeFn = options.resolveWorktreeFn || resolveWorktree;
-  const readTokenFn = options.readTokenFn || readToken;
-  const postCommentFn = options.postCommentFn || postComment;
-  const resolveReviewUserFn = options.resolveReviewUserFn || options.resolveForgejoUserFn || resolveReviewUser;
-  const readReviewStateFn = options.readReviewStateFn || readReviewState;
-  const buildMetadataFooterFn = options.buildMetadataFooterFn || buildMetadataFooter;
-
-  const rootDir = options.rootDir || resolveWorktreeFn(slug) || process.cwd();
-  const branch = missionBranchName(slug, rootDir);
-  const { identityUser } = resolveReviewIdentity(slug, rootDir, {
-    readReviewStateFn,
-  });
-  let resolvedUser = identityUser;
-
-  if (!resolvedUser) {
-    const taskResolution = resolveTaskFileFn(slug, rootDir);
-    if (taskResolution.ok) {
-      resolvedUser = getTaskImplementerFn(taskResolution.taskFile);
-    }
-  }
-  if (!resolvedUser) {
-    error('Cannot determine review identity. Set FORGEJO_USER, persist review-state.json, or assign the task implementer.');
-    return { ok: false, error: 'missing-user' };
-  }
-  resolvedUser = resolveReviewUserFn(resolvedUser);
-  const token = readTokenFn(resolvedUser, { rootDir });
-  if (!token) {
-    error(`No Forgejo token found for user "${resolvedUser}". Cannot post static review comment.`);
-    return { ok: false, error: 'missing-token' };
-  }
-
-  const taggedMessage = message + buildMetadataFooterFn(slug, rootDir);
-  log(`Posting static review comment on ${fmt.branch(branch)} as ${resolvedUser}...`);
-  const result = postCommentFn(branch, token, taggedMessage, { reviewIdentity: resolvedUser, forgejoUser: resolvedUser });
-
-  if (!result.ok) {
-    error(`Could not post static review comment: ${result.error || 'API error'}`);
+    const log = options.log || fmt.log.plain;
+    const error = options.error || fmt.log.plainError;
+    const resolveTaskFileFn = options.resolveTaskFileFn || backlog_js_1.resolveTaskFile;
+    const getTaskImplementerFn = options.getTaskImplementerFn || backlog_js_1.getTaskImplementer;
+    const resolveWorktreeFn = options.resolveWorktreeFn || mission_utils_js_1.resolveWorktree;
+    const readTokenFn = options.readTokenFn || review_adapter_js_1.readToken;
+    const postCommentFn = options.postCommentFn || review_adapter_js_1.postComment;
+    const resolveReviewUserFn = options.resolveReviewUserFn || options.resolveForgejoUserFn || review_adapter_js_1.resolveReviewUser;
+    const readReviewStateFn = options.readReviewStateFn || review_state_js_1.readReviewState;
+    const buildMetadataFooterFn = options.buildMetadataFooterFn || review_artifacts_js_1.buildMetadataFooter;
+    const rootDir = options.rootDir || resolveWorktreeFn(slug) || process.cwd();
+    const branch = (0, mission_utils_js_1.missionBranchName)(slug, rootDir);
+    const { identityUser } = (0, review_state_js_1.resolveReviewIdentity)(slug, rootDir, {
+        readReviewStateFn,
+    });
+    let resolvedUser = identityUser;
+    if (!resolvedUser) {
+        const taskResolution = resolveTaskFileFn(slug, rootDir);
+        if (taskResolution.ok) {
+            resolvedUser = getTaskImplementerFn(taskResolution.taskFile);
+        }
+    }
+    if (!resolvedUser) {
+        error('Cannot determine review identity. Set FORGEJO_USER, persist review-state.json, or assign the task implementer.');
+        return { ok: false, error: 'missing-user' };
+    }
+    resolvedUser = resolveReviewUserFn(resolvedUser);
+    const token = readTokenFn(resolvedUser, { rootDir });
+    if (!token) {
+        error(`No Forgejo token found for user "${resolvedUser}". Cannot post static review comment.`);
+        return { ok: false, error: 'missing-token' };
+    }
+    const taggedMessage = message + buildMetadataFooterFn(slug, rootDir);
+    log(`Posting static review comment on ${fmt.branch(branch)} as ${resolvedUser}...`);
+    const result = postCommentFn(branch, token, taggedMessage, { reviewIdentity: resolvedUser, forgejoUser: resolvedUser });
+    if (!result.ok) {
+        error(`Could not post static review comment: ${result.error || 'API error'}`);
+        return result;
+    }
+    log(fmt.status('PASS', `Static review comment posted on PR for ${fmt.branch(branch)}.`));
     return result;
-  }
-
-  log(fmt.status('PASS', `Static review comment posted on PR for ${fmt.branch(branch)}.`));
-  return result;
 }
-
 // ============================================================================
 // Command: performStaticReview
 // ============================================================================
-
-/**
- * Perform a static review of the mission branch when no PR exists.
- * Inspects git diff for changed files, checks the final checkpoint for
- * a Goal Check table, and returns findings.
- */
-/**
- * @param {string} slug
- * @param {{log?: Function, findMissionDir?: Function, findCheckpoints?: Function, readFileSync?: Function, run?: Function, resolveWorktree?: Function, missionPath?: string, rootDir?: string}} [options]
- * @returns {{ok: boolean, findings: string[]}}
- */
 function performStaticReview(slug, options = {}) {
-  const log = options.log || fmt.log.plain;
-  const findMissionDirFn = options.findMissionDir || findMissionDir;
-  const findCheckpointsFn = options.findCheckpoints || findCheckpoints;
-  const readFileSyncFn = options.readFileSync || fs.readFileSync;
-  const runFn = options.run || run;
-  const resolveWorktreeFn = options.resolveWorktree || resolveWorktree;
-  const findings = [];
-
-  log(`Performing static review for mission: ${fmt.slug(slug)}`);
-
-  // Resolve the worktree root early so mission dir and checkpoint lookups use the same base
-  const worktreeRoot = resolveWorktreeFn(slug);
-  const rootDir = worktreeRoot || process.cwd();
-
-  // Check if mission directory exists (with optional --mission path override)
-  const missionDir = findMissionDirFn(slug, rootDir, { missionPath: options.missionPath });
-  if (!missionDir) {
-    findings.push(`Mission directory not found for slug: ${slug}`);
-    return { ok: false, findings };
-  }
-  log(fmt.status('PASS', `Mission directory found: ${fmt.path(missionDir)}`));
-
-  // Check if checkpoint documents exist
-  const checkpoints = findCheckpointsFn(missionDir);
-  if (checkpoints.length === 0) {
-    findings.push('No checkpoint documents found. Implementation evidence is required.');
-    return { ok: false, findings };
-  }
-  log(fmt.status('PASS', `Found ${checkpoints.length} checkpoint document(s).`));
-
-  // Check the final checkpoint for a Goal Check table
-  const finalCheckpoint = checkpoints[checkpoints.length - 1];
-  try {
-    const checkpointContent = readFileSyncFn(finalCheckpoint, 'utf8');
-    const goalCheckMatch = checkpointContent.match(/^## Goal Check(?: Table)?\s*$/m);
-    if (!goalCheckMatch) {
-      findings.push(`Final checkpoint ${path.basename(finalCheckpoint)} is missing a "## Goal Check" section.`);
-    } else {
-      log(fmt.status('PASS', 'Final checkpoint contains "## Goal Check" section.'));
-
-      // Verify goal-check table has at least one evidence row
-      const afterHeader = checkpointContent.slice(goalCheckMatch.index + goalCheckMatch[0].length);
-      const separatorPattern = /^\|\s*:?-+:?\s*\|/;
-      const headerPattern = /^\| .+\| .+\| .+\|$/;
-      const evidenceLinePattern = /^\| .+\| .+\| .+\|$/;
-      const linesAfterHeader = afterHeader.split('\n');
-      let foundEvidence = false;
-      let pastHeader = false;
-      for (const line of linesAfterHeader) {
-        const trimmed = line.trim();
-        if (trimmed === '') {continue;}
-        if (!pastHeader && headerPattern.test(trimmed)) {
-          pastHeader = true;
-          continue;
+    const log = options.log || fmt.log.plain;
+    const findMissionDirFn = options.findMissionDir || mission_utils_js_1.findMissionDir;
+    const findCheckpointsFn = options.findCheckpoints || mission_utils_js_1.findCheckpoints;
+    const readFileSyncFn = options.readFileSync || fs.readFileSync;
+    const runFn = options.run || git_js_1.run;
+    const resolveWorktreeFn = options.resolveWorktree || mission_utils_js_1.resolveWorktree;
+    const findings = [];
+    log(`Performing static review for mission: ${fmt.slug(slug)}`);
+    // Resolve the worktree root early so mission dir and checkpoint lookups use the same base
+    const worktreeRoot = resolveWorktreeFn(slug);
+    const rootDir = worktreeRoot || process.cwd();
+    // Check if mission directory exists (with optional --mission path override)
+    const missionDir = findMissionDirFn(slug, rootDir, { missionPath: options.missionPath });
+    if (!missionDir) {
+        findings.push(`Mission directory not found for slug: ${slug}`);
+        return { ok: false, findings };
+    }
+    log(fmt.status('PASS', `Mission directory found: ${fmt.path(missionDir)}`));
+    // Check if checkpoint documents exist
+    const checkpoints = findCheckpointsFn(missionDir);
+    if (checkpoints.length === 0) {
+        findings.push('No checkpoint documents found. Implementation evidence is required.');
+        return { ok: false, findings };
+    }
+    log(fmt.status('PASS', `Found ${checkpoints.length} checkpoint document(s).`));
+    // Check the final checkpoint for a Goal Check table
+    const finalCheckpoint = checkpoints[checkpoints.length - 1];
+    try {
+        const checkpointContent = readFileSyncFn(finalCheckpoint, 'utf8');
+        const goalCheckMatch = checkpointContent.match(/^## Goal Check(?: Table)?\s*$/m);
+        if (!goalCheckMatch) {
+            findings.push(`Final checkpoint ${path.basename(finalCheckpoint)} is missing a "## Goal Check" section.`);
         }
-        if (separatorPattern.test(trimmed)) {continue;}
-        if (pastHeader && evidenceLinePattern.test(trimmed)) {
-          foundEvidence = true;
-          break;
+        else {
+            log(fmt.status('PASS', 'Final checkpoint contains "## Goal Check" section.'));
+            // Verify goal-check table has at least one evidence row
+            const afterHeader = checkpointContent.slice(goalCheckMatch.index + goalCheckMatch[0].length);
+            const separatorPattern = /^\|\s*:?-+:?\s*\|/;
+            const headerPattern = /^\| .+\| .+\| .+\|$/;
+            const evidenceLinePattern = /^\| .+\| .+\| .+\|$/;
+            const linesAfterHeader = afterHeader.split('\n');
+            let foundEvidence = false;
+            let pastHeader = false;
+            for (const line of linesAfterHeader) {
+                const trimmed = line.trim();
+                if (trimmed === '') {
+                    continue;
+                }
+                if (!pastHeader && headerPattern.test(trimmed)) {
+                    pastHeader = true;
+                    continue;
+                }
+                if (separatorPattern.test(trimmed)) {
+                    continue;
+                }
+                if (pastHeader && evidenceLinePattern.test(trimmed)) {
+                    foundEvidence = true;
+                    break;
+                }
+                break;
+            }
+            if (!foundEvidence) {
+                findings.push(`Final checkpoint ${path.basename(finalCheckpoint)} has "## Goal Check" section but no evidence rows.`);
+            }
+            else {
+                log(fmt.status('PASS', 'Goal Check table contains evidence rows.'));
+            }
         }
-        break;
-      }
-      if (!foundEvidence) {
-        findings.push(`Final checkpoint ${path.basename(finalCheckpoint)} has "## Goal Check" section but no evidence rows.`);
-      } else {
-        log(fmt.status('PASS', 'Goal Check table contains evidence rows.'));
-      }
-    }
-  } catch (err) {
-    findings.push(`Could not read final checkpoint ${path.basename(finalCheckpoint)}: ${/** @type {Error} */(err).message}`);
-  }
-
-  // Inspect git diff for changed files
-  const baseBranch = require('../core/mission-utils').getPrimaryBranch(rootDir);
-  const diffResult = runFn('git', ['diff', `${baseBranch}..HEAD`, '--name-only'], { cwd: rootDir });
-  if (diffResult.status === 0 && diffResult.stdout) {
-    const changedFiles = diffResult.stdout.trim().split('\n').filter((/** @type {string} */ f) => f.trim());
-    const missionDirPrefix = path.relative(rootDir, missionBaseDir(rootDir)).split(path.sep).join('/') + '/';
-    log(`Changed files in branch: ${changedFiles.join(', ')}`);
-    // Check for unexpected areas — missions may legitimately touch many surfaces.
-    // Derive allowed set from known workflow areas plus common repo structures,
-    // plus any file with a recognized extension or inside the mission directory.
-    const knownAreas = [
-      'parallix/', 'docs/', 'scripts/', 'config/', 'backlog/', 'forgejo/',
-      '.agents/', '.github/', '.vscode/', '.graphifyignore',
-    ];
-    const knownExtensions = ['.sh', '.csv', '.json', '.yaml', '.yml', '.toml', '.lock', '.cfg', '.ini', '.env', '.txt', '.properties', '.sql', '.css', '.html', '.xml', '.png', '.jpg', '.jpeg', '.gif', '.svg', '.ico', '.pdf', '.tar', '.gz', '.zip'];
-      const unexpectedFiles = changedFiles.filter((/** @type {string} */ f) =>
-      !knownAreas.some((/** @type {string} */ area) => f.startsWith(area)) &&
-      !f.startsWith(missionDirPrefix) &&
-      !f.endsWith('.md') &&
-      !knownExtensions.some((/** @type {string} */ ext) => f.toLowerCase().endsWith(ext))
-    );
-    if (unexpectedFiles.length > 0) {
-      log(fmt.status('WARN', `Changed files outside known areas (may be intentional): ${unexpectedFiles.join(', ')}`));
-    } else {
-      log(fmt.status('PASS', 'All changed files are within expected areas.'));
-    }
-  } else {
-    log(fmt.status('WARN', `git diff ${baseBranch}..HEAD returned no output or failed — branch may be up to date with ${baseBranch}.`));
-  }
-
-  return { ok: findings.length === 0, findings };
+    }
+    catch (err) {
+        findings.push(`Could not read final checkpoint ${path.basename(finalCheckpoint)}: ${err.message}`);
+    }
+    // Inspect git diff for changed files
+    const baseBranch = (0, mission_utils_js_1.getPrimaryBranch)(rootDir);
+    const diffResult = runFn('git', ['diff', `${baseBranch}..HEAD`, '--name-only'], { cwd: rootDir });
+    if (diffResult.status === 0 && diffResult.stdout) {
+        const changedFiles = diffResult.stdout.trim().split('\n').filter((f) => f.trim());
+        const missionDirPrefix = path.relative(rootDir, (0, mission_utils_js_1.missionBaseDir)(rootDir)).split(path.sep).join('/') + '/';
+        log(`Changed files in branch: ${changedFiles.join(', ')}`);
+        // Check for unexpected areas — missions may legitimately touch many surfaces.
+        // Derive allowed set from known workflow areas plus common repo structures,
+        // plus any file with a recognized extension or inside the mission directory.
+        const knownAreas = [
+            'parallix/', 'docs/', 'scripts/', 'config/', 'backlog/', 'forgejo/',
+            '.agents/', '.github/', '.vscode/', '.graphifyignore',
+        ];
+        const knownExtensions = ['.sh', '.csv', '.json', '.yaml', '.yml', '.toml', '.lock', '.cfg', '.ini', '.env', '.txt', '.properties', '.sql', '.css', '.html', '.xml', '.png', '.jpg', '.jpeg', '.gif', '.svg', '.ico', '.pdf', '.tar', '.gz', '.zip'];
+        const unexpectedFiles = changedFiles.filter((f) => !knownAreas.some((area) => f.startsWith(area)) &&
+            !f.startsWith(missionDirPrefix) &&
+            !f.endsWith('.md') &&
+            !knownExtensions.some((ext) => f.toLowerCase().endsWith(ext)));
+        if (unexpectedFiles.length > 0) {
+            log(fmt.status('WARN', `Changed files outside known areas (may be intentional): ${unexpectedFiles.join(', ')}`));
+        }
+        else {
+            log(fmt.status('PASS', 'All changed files are within expected areas.'));
+        }
+    }
+    else {
+        log(fmt.status('WARN', `git diff ${baseBranch}..HEAD returned no output or failed — branch may be up to date with ${baseBranch}.`));
+    }
+    return { ok: findings.length === 0, findings };
 }
-
 // ============================================================================
 // Command: verifyReview
 // ============================================================================
-
-/**
- * @param {string} slug
- * @param {boolean|string} skipGate
- * @param {{log?: Function, error?: Function, exit?: Function, resolveWorktreeFn?: Function, findMissionDirFn?: Function, getCurrentBranchFn?: Function, resolveTaskFileFn?: Function, getPrStatusFn?: Function, getTaskStatusFn?: Function, toVirtualFn?: Function, findMissionAreaFn?: Function, runFn?: Function, getAcceptanceCriteriaFn?: Function, formatMatrixSummaryFn?: Function, buildAutonomousReviewMatrixFn?: Function, readReviewStateFn?: Function, isReviewProviderEnabledFn?: Function, isForgejoReviewEnabledFn?: Function, cwdFn?: Function, missionPath?: string, skipGate?: boolean}} [options]
- * @returns {void}
- */
 function verifyReview(slug, skipGate, options = {}) {
-  const log = options.log || fmt.log.plain;
-  const error = options.error || fmt.log.plainError;
-  const exit = options.exit || process.exit;
-  const resolveWorktreeFn = options.resolveWorktreeFn || resolveWorktree;
-  const findMissionDirFn = options.findMissionDirFn || findMissionDir;
-  const getCurrentBranchFn = options.getCurrentBranchFn || getCurrentBranch;
-  const resolveTaskFileFn = options.resolveTaskFileFn || resolveTaskFile;
-  const getPrStatusFn = options.getPrStatusFn || getPrStatus;
-  const getTaskStatusFn = options.getTaskStatusFn || getTaskStatus;
-  const toVirtualFn = options.toVirtualFn || toVirtual;
-  const findMissionAreaFn = options.findMissionAreaFn || findMissionArea;
-  const runFn = options.runFn || run;
-  const getAcceptanceCriteriaFn = options.getAcceptanceCriteriaFn || getAcceptanceCriteria;
-  const formatMatrixSummaryFn = options.formatMatrixSummaryFn || formatMatrixSummary;
-  const buildAutonomousReviewMatrixFn = options.buildAutonomousReviewMatrixFn || buildAutonomousReviewMatrix;
-  const readReviewStateFn = options.readReviewStateFn || readReviewState;
-  const isReviewProviderEnabledFn = options.isReviewProviderEnabledFn || options.isForgejoReviewEnabledFn || isProviderEnabled;
-  const cwdFn = options.cwdFn || (() => process.cwd());
-
-  const worktree = resolveWorktreeFn(slug);
-  const rootDir = worktree || cwdFn();
-
-  const missionDir = findMissionDirFn(slug, rootDir, { missionPath: options.missionPath });
-  const branch = missionBranchName(slug, rootDir);
-  const current = getCurrentBranchFn(rootDir);
-  const taskResolution = resolveTaskFileFn(slug, rootDir);
-  const providerEnabled = isReviewProviderEnabledFn(rootDir);
-  const pr = providerEnabled ? getPrStatusFn(branch, rootDir) : { exists: false };
-  const failures = [];
-  const warnings = [];
-
-  log(`Reviewer verification for mission: ${fmt.slug(slug)}`);
-  if (worktree) {
-    log(`Found dedicated worktree: ${fmt.path(worktree)}`);
-  } else {
-    log('Using current directory as mission root.');
-  }
-
-  if (!missionDir) {
-    failures.push('mission-dir');
-    log(fmt.status('FAIL', `Mission directory not found for slug: ${fmt.slug(slug)}`));
-  } else {
-    log(fmt.status('PASS', `Mission doc: ${fmt.path(path.join(missionDir, 'MISSION.md'))}`));
-  }
-
-  if (current !== branch) {
-    failures.push('branch');
-    log(fmt.status('FAIL', `Branch: current branch is ${fmt.branch(current)}, expected ${fmt.branch(branch)}`));
-  } else {
-    log(fmt.status('PASS', `Branch: ${fmt.branch(current)}`));
-  }
-
-  let taskStatus = null;
-  let virtualStatus = null;
-  if (!taskResolution.ok) {
-    failures.push('task');
-    reportTaskResolution(taskResolution, slug, log);
-  } else {
-    taskStatus = getTaskStatusFn(taskResolution.taskFile);
-    virtualStatus = toVirtualFn(taskStatus);
-    if (taskStatus === 'review' || virtualStatus === 'approved') {
-      log(fmt.status('PASS', `Backlog task: ${path.basename(taskResolution.taskFile)} (${taskStatus})`));
-    } else if (taskStatus === 'done') {
-      failures.push('task-status');
-      log(fmt.status('FAIL', 'Backlog task: task is already done/integrated'));
-    } else if (taskStatus === 'active' || virtualStatus === 'active') {
-      warnings.push('task-still-active');
-      log(fmt.status('WARN', `Backlog task: ${path.basename(taskResolution.taskFile)} is still ${taskStatus}`));
-    } else {
-      failures.push('task-status');
-      log(fmt.status('FAIL', `Backlog task: unexpected status ${taskStatus}`));
-    }
-  }
-
-  if (providerEnabled) {
-    if (pr.exists && pr.state === 'open' && !pr.merged) {
-      log(fmt.status('PASS', `Review PR: PR #${pr.number} is open`));
-    } else if (pr.exists) {
-      failures.push('pr-state');
-      log(fmt.status('FAIL', `Review PR: expected an open PR, got state=${pr.state} merged=${pr.merged}`));
-    } else {
-      // No PR exists - check if task is in implementation phase
-      if (taskResolution.ok) {
-        const isImplementationPhase = taskStatus === 'active' || virtualStatus === 'active';
-        if (isImplementationPhase) {
-          // Task is still in implementation - emit warning instead of failure
-          warnings.push('no-pr-yet');
-          log(fmt.status('WARN', `Review PR: no PR found for ${branch}. Task is still ${taskStatus} — complete implementation and submit first: px review ${slug} --push`));
-        } else {
-          // Task is in post-implementation state or ambiguous - hard fail
-          failures.push('pr-missing');
-          log(fmt.status('FAIL', `Review PR: ${pr.raw || 'no PR found'}`));
+    const log = options.log || fmt.log.plain;
+    const error = options.error || fmt.log.plainError;
+    const exit = options.exit || process.exit;
+    const resolveWorktreeFn = options.resolveWorktreeFn || mission_utils_js_1.resolveWorktree;
+    const findMissionDirFn = options.findMissionDirFn || mission_utils_js_1.findMissionDir;
+    const getCurrentBranchFn = options.getCurrentBranchFn || git_js_1.getCurrentBranch;
+    const resolveTaskFileFn = options.resolveTaskFileFn || backlog_js_1.resolveTaskFile;
+    const getPrStatusFn = options.getPrStatusFn || review_adapter_js_1.getPrStatus;
+    const getTaskStatusFn = options.getTaskStatusFn || backlog_js_1.getTaskStatus;
+    const toVirtualFn = options.toVirtualFn || state_map_js_1.toVirtual;
+    const findMissionAreaFn = options.findMissionAreaFn || mission_utils_js_1.findMissionArea;
+    const runFn = options.runFn || git_js_1.run;
+    const getAcceptanceCriteriaFn = options.getAcceptanceCriteriaFn || backlog_js_1.getAcceptanceCriteria;
+    const formatMatrixSummaryFn = options.formatMatrixSummaryFn || runtime_matrix_js_1.formatMatrixSummary;
+    const buildAutonomousReviewMatrixFn = options.buildAutonomousReviewMatrixFn || runtime_matrix_js_1.buildAutonomousReviewMatrix;
+    const readReviewStateFn = options.readReviewStateFn || review_state_js_1.readReviewState;
+    const isReviewProviderEnabledFn = options.isReviewProviderEnabledFn || options.isForgejoReviewEnabledFn || review_adapter_js_1.isProviderEnabled;
+    const cwdFn = options.cwdFn || (() => process.cwd());
+    const worktree = resolveWorktreeFn(slug);
+    const rootDir = worktree || cwdFn();
+    const missionDir = findMissionDirFn(slug, rootDir, { missionPath: options.missionPath });
+    const branch = (0, mission_utils_js_1.missionBranchName)(slug, rootDir);
+    const current = getCurrentBranchFn(rootDir);
+    const taskResolution = resolveTaskFileFn(slug, rootDir);
+    const providerEnabled = isReviewProviderEnabledFn(rootDir);
+    const pr = providerEnabled ? getPrStatusFn(branch, rootDir) : { exists: false };
+    const failures = [];
+    const warnings = [];
+    log(`Reviewer verification for mission: ${fmt.slug(slug)}`);
+    if (worktree) {
+        log(`Found dedicated worktree: ${fmt.path(worktree)}`);
+    }
+    else {
+        log('Using current directory as mission root.');
+    }
+    if (!missionDir) {
+        failures.push('mission-dir');
+        log(fmt.status('FAIL', `Mission directory not found for slug: ${fmt.slug(slug)}`));
+    }
+    else {
+        log(fmt.status('PASS', `Mission doc: ${fmt.path(path.join(missionDir, 'MISSION.md'))}`));
+    }
+    if (current !== branch) {
+        failures.push('branch');
+        log(fmt.status('FAIL', `Branch: current branch is ${fmt.branch(current)}, expected ${fmt.branch(branch)}`));
+    }
+    else {
+        log(fmt.status('PASS', `Branch: ${fmt.branch(current)}`));
+    }
+    let taskStatus = null;
+    let virtualStatus = null;
+    if (!taskResolution.ok) {
+        failures.push('task');
+        (0, backlog_js_1.reportTaskResolution)(taskResolution, slug, log);
+    }
+    else {
+        taskStatus = getTaskStatusFn(taskResolution.taskFile);
+        virtualStatus = toVirtualFn(taskStatus || '');
+        if (taskStatus === 'review' || virtualStatus === 'approved') {
+            log(fmt.status('PASS', `Backlog task: ${path.basename(taskResolution.taskFile)} (${taskStatus})`));
         }
-      } else {
-        // Cannot determine task status - safe default to hard fail
-        failures.push('pr-missing');
-        log(fmt.status('FAIL', `Review PR: ${pr.raw || 'no PR found'}`));
-      }
-    }
-  } else {
-    log(fmt.status('INFO', 'Forgejo PR: skipped (review provider is not forgejo).'));
-  }
-
-  if (missionDir) {
-    const area = findMissionAreaFn(missionDir);
-    const skipGateFlag = Boolean(skipGate || options.skipGate);
-    if (skipGateFlag) {
-      log(fmt.status('WARN', `Verification gate skipped (--no-gate) for area ${area}`));
-    } else {
-      log(`Running reviewer gate: ${fmt.command(formatVerificationCommand(area, process.cwd()))}`);
-      const verifyResult = runVerificationGate(area, { rootDir: process.cwd(), stdio: 'inherit', runFn });
-      if (verifyResult.status !== 0) {
-        failures.push('gate');
-        log(fmt.status('FAIL', 'Reviewer gate failed.'));
-      } else {
-        log(fmt.status('PASS', 'Reviewer gate passed.'));
-      }
-    }
-  }
-
-  if (taskResolution.ok) {
-    const acceptanceCriteria = getAcceptanceCriteriaFn(taskResolution.taskFile);
-    log(fmt.status('INFO', 'Acceptance evidence checklist:'));
-    if (acceptanceCriteria.length === 0) {
-      log('  - No Acceptance Criteria found on the Backlog task.');
-    } else {
-      acceptanceCriteria.forEach((/** @type {string} */ line) => log(`  ${line}`));
-    }
-  }
-
-  log(fmt.status('INFO', 'Autonomous review runtime matrix:'));
-    formatMatrixSummaryFn(buildAutonomousReviewMatrixFn()).forEach((/** @type {string} */ line) => log(line));
-
-  // Show persisted reviewer state if present
-  const persisted = readReviewStateFn(slug);
-  if (persisted) {
-    log(`Persisted reviewer state: reviewer=${fmt.agent(persisted.reviewer)} implementer=${fmt.agent(persisted.implementer)} round=${persisted.round} startedAt=${persisted.startedAt}`);
-  }
-
-  if (warnings.length > 0) {
-    log(fmt.status('WARN', `Review verification warnings: ${warnings.join(', ')}`));
-  }
-
-  if (failures.length > 0) {
-    error('\n' + fmt.status('INFO', 'Review verification failed. Resolve the blockers above before starting review.'));
-    exit(1);
-    return;
-  }
-
-  log('\n' + fmt.status('PASS', 'Review verification complete.'));
+        else if (taskStatus === 'done') {
+            failures.push('task-status');
+            log(fmt.status('FAIL', 'Backlog task: task is already done/integrated'));
+        }
+        else if (taskStatus === 'active' || virtualStatus === 'active') {
+            warnings.push('task-still-active');
+            log(fmt.status('WARN', `Backlog task: ${path.basename(taskResolution.taskFile)} is still ${taskStatus}`));
+        }
+        else {
+            failures.push('task-status');
+            log(fmt.status('FAIL', `Backlog task: unexpected status ${taskStatus}`));
+        }
+    }
+    const prAny = pr;
+    if (providerEnabled) {
+        if (prAny.exists && prAny.state === 'open' && !prAny.merged) {
+            log(fmt.status('PASS', `Review PR: PR #${prAny.number} is open`));
+        }
+        else if (prAny.exists) {
+            failures.push('pr-state');
+            log(fmt.status('FAIL', `Review PR: expected an open PR, got state=${prAny.state} merged=${prAny.merged}`));
+        }
+        else {
+            // No PR exists - check if task is in implementation phase
+            if (taskResolution.ok) {
+                const isImplementationPhase = taskStatus === 'active' || virtualStatus === 'active';
+                if (isImplementationPhase) {
+                    // Task is still in implementation - emit warning instead of failure
+                    warnings.push('no-pr-yet');
+                    log(fmt.status('WARN', `Review PR: no PR found for ${branch}. Task is still ${taskStatus} — complete implementation and submit first: px review ${slug} --push`));
+                }
+                else {
+                    // Task is in post-implementation state or ambiguous - hard fail
+                    failures.push('pr-missing');
+                    log(fmt.status('FAIL', `Review PR: ${prAny.raw || 'no PR found'}`));
+                }
+            }
+            else {
+                // Cannot determine task status - safe default to hard fail
+                failures.push('pr-missing');
+                log(fmt.status('FAIL', `Review PR: ${prAny.raw || 'no PR found'}`));
+            }
+        }
+    }
+    else {
+        log(fmt.status('INFO', 'Forgejo PR: skipped (review provider is not forgejo).'));
+    }
+    if (missionDir) {
+        const area = findMissionAreaFn(missionDir);
+        const skipGateFlag = Boolean(skipGate || options.skipGate);
+        if (skipGateFlag) {
+            log(fmt.status('WARN', `Verification gate skipped (--no-gate) for area ${area}`));
+        }
+        else {
+            log(`Running reviewer gate: ${fmt.command((0, verification_js_1.formatVerificationCommand)(area, process.cwd()))}`);
+            const verifyResult = (0, verification_js_1.runVerificationGate)(area, { rootDir: process.cwd(), stdio: 'inherit', runFn });
+            if (verifyResult.status !== 0) {
+                failures.push('gate');
+                log(fmt.status('FAIL', 'Reviewer gate failed.'));
+            }
+            else {
+                log(fmt.status('PASS', 'Reviewer gate passed.'));
+            }
+        }
+    }
+    if (taskResolution.ok) {
+        const acceptanceCriteria = getAcceptanceCriteriaFn(taskResolution.taskFile);
+        log(fmt.status('INFO', 'Acceptance evidence checklist:'));
+        if (acceptanceCriteria.length === 0) {
+            log('  - No Acceptance Criteria found on the Backlog task.');
+        }
+        else {
+            acceptanceCriteria.forEach((line) => log(`  ${line}`));
+        }
+    }
+    log(fmt.status('INFO', 'Autonomous review runtime matrix:'));
+    formatMatrixSummaryFn(buildAutonomousReviewMatrixFn()).forEach((line) => log(line));
+    // Show persisted reviewer state if present
+    const persisted = readReviewStateFn(slug);
+    if (persisted) {
+        log(`Persisted reviewer state: reviewer=${fmt.agent(persisted.reviewer ?? '')} implementer=${fmt.agent(persisted.implementer ?? '')} round=${persisted.round} startedAt=${persisted.startedAt}`);
+    }
+    if (warnings.length > 0) {
+        log(fmt.status('WARN', `Review verification warnings: ${warnings.join(', ')}`));
+    }
+    if (failures.length > 0) {
+        error('\n' + fmt.status('INFO', 'Review verification failed. Resolve the blockers above before starting review.'));
+        exit(1);
+        return;
+    }
+    log('\n' + fmt.status('PASS', 'Review verification complete.'));
 }
-
 // ============================================================================
 // Command: submitForReview
 // ============================================================================
-
-/**
- * @param {string} slug
- * @param {boolean|string} skipGate
- * @param {{exit?: Function, resolveTaskFileFn?: Function, getTaskImplementerFn?: Function, resolveWorktreeFn?: Function, performHandoffFn?: Function, readReviewStateFn?: Function, isReviewProviderEnabledFn?: Function, isForgejoReviewEnabledFn?: Function, log?: Function}} [options]
- * @returns {Promise<void>}
- */
 async function submitForReview(slug, skipGate, options = {}) {
-  const exit = options.exit || process.exit;
-  const resolveTaskFileFn = options.resolveTaskFileFn || resolveTaskFile;
-  const getTaskImplementerFn = options.getTaskImplementerFn || getTaskImplementer;
-  const resolveWorktreeFn = options.resolveWorktreeFn || resolveWorktree;
-  const performHandoffFn = options.performHandoffFn || performHandoff;
-  const readReviewStateFn = options.readReviewStateFn || readReviewState;
-  const isReviewProviderEnabledFn = options.isReviewProviderEnabledFn || options.isForgejoReviewEnabledFn || isProviderEnabled;
-  const log = options.log || fmt.log.plain;
-
-  const worktree = resolveWorktreeFn(slug) || process.cwd();
-  const providerEnabled = isReviewProviderEnabledFn(worktree);
-
-  const { identityUser: reviewStateUser } = resolveReviewIdentity(slug, worktree, {
-    readReviewStateFn,
-  });
-  let reviewIdentity = reviewStateUser;
-
-  // 2. Check backlog task assignee
-  if (!reviewIdentity) {
-    const taskResolution = resolveTaskFileFn(slug, worktree);
-    if (taskResolution.ok) {
-      reviewIdentity = getTaskImplementerFn(taskResolution.taskFile);
+    const exit = options.exit || process.exit;
+    const resolveTaskFileFn = options.resolveTaskFileFn || backlog_js_1.resolveTaskFile;
+    const getTaskImplementerFn = options.getTaskImplementerFn || backlog_js_1.getTaskImplementer;
+    const resolveWorktreeFn = options.resolveWorktreeFn || mission_utils_js_1.resolveWorktree;
+    const performHandoffFn = options.performHandoffFn || (await getHandoff()).performHandoff;
+    const readReviewStateFn = options.readReviewStateFn || review_state_js_1.readReviewState;
+    const isReviewProviderEnabledFn = options.isReviewProviderEnabledFn || options.isForgejoReviewEnabledFn || review_adapter_js_1.isProviderEnabled;
+    const log = options.log || fmt.log.plain;
+    const worktree = resolveWorktreeFn(slug) || process.cwd();
+    const providerEnabled = isReviewProviderEnabledFn(worktree);
+    const { identityUser: reviewStateUser } = (0, review_state_js_1.resolveReviewIdentity)(slug, worktree, {
+        readReviewStateFn,
+    });
+    let reviewIdentity = reviewStateUser;
+    // 2. Check backlog task assignee
+    if (!reviewIdentity) {
+        const taskResolution = resolveTaskFileFn(slug, worktree);
+        if (taskResolution.ok) {
+            reviewIdentity = getTaskImplementerFn(taskResolution.taskFile);
+        }
+    }
+    // 3. Mode-specific final fallback: named identity (provider-backed) vs "autonomous" (provider=none) (SC 6)
+    if (!reviewIdentity) {
+        if (providerEnabled) {
+            log(fmt.status('FAIL', `No review identity resolved for ${slug}. Persist review-state.json or set the task implementer before submitting for review.`));
+            exit(1);
+            return;
+        }
+        else {
+            reviewIdentity = 'autonomous';
+            log(fmt.status('WARN', `No reviewer/implementer identity resolved for ${slug}; defaulting to "autonomous"`));
+        }
+    }
+    const result = await performHandoffFn(slug, { skipGate, reviewIdentity, forgejoUser: reviewIdentity, worktree });
+    if (!result.ok) {
+        exit(1);
     }
-  }
-
-  // 3. Mode-specific final fallback: named identity (provider-backed) vs "autonomous" (provider=none) (SC 6)
-  if (!reviewIdentity) {
-    if (providerEnabled) {
-      log(fmt.status('FAIL', `No review identity resolved for ${slug}. Persist review-state.json or set the task implementer before submitting for review.`));
-      exit(1);
-      return;
-    } else {
-      reviewIdentity = 'autonomous';
-      log(fmt.status('WARN', `No reviewer/implementer identity resolved for ${slug}; defaulting to "autonomous"`));
-    }
-  }
-
-  const result = await performHandoffFn(slug, { skipGate, reviewIdentity, forgejoUser: reviewIdentity, worktree });
-  if (!result.ok) {
-    exit(1);
-  }
 }
-
 // ============================================================================
 // Command: readComments
 // ============================================================================
-
-/**
- * @param {string} slug
- * @param {{log?: Function, error?: Function, exit?: Function, readTokenFn?: Function, readReviewStateFn?: Function, resolveReviewUserFn?: Function, resolveForgejoUserFn?: Function, getCommentsFn?: Function, readAllEventsFn?: Function, isReviewProviderEnabledFn?: Function, isForgejoReviewEnabledFn?: Function, worktree?: string}} [options]
- * @returns {Promise<void>}
- */
 async function readComments(slug, options = {}) {
-  const log = options.log || fmt.log.plain;
-  const error = options.error || fmt.log.plainError;
-  const exit = options.exit || process.exit;
-  const readTokenFn = options.readTokenFn || readToken;
-  const readReviewStateFn = options.readReviewStateFn || readReviewState;
-  const resolveReviewUserFn = options.resolveReviewUserFn || options.resolveForgejoUserFn || resolveReviewUser;
-  const getCommentsFn = options.getCommentsFn || getComments;
-  const readAllEventsFn = options.readAllEventsFn || readAllEvents;
-  const isReviewProviderEnabledFn = options.isReviewProviderEnabledFn || options.isForgejoReviewEnabledFn || isProviderEnabled;
-  const worktree = options.worktree || resolveWorktree(slug) || process.cwd();
-  const branch = missionBranchName(slug, worktree);
-
-  // Provider-disabled fallback: read persisted local review events instead of polling the PR.
-  // Avoids resolving a provider token (which would FAIL/exit) when the provider is off.
-  if (!isReviewProviderEnabledFn(worktree)) {
-    log(fmt.status('INFO', `Review provider disabled; reading local review events for ${slug}...`));
-    const events = readAllEventsFn(slug, { rootDir: worktree, log, error });
-    if (!events || events.length === 0) {
-      log('no comments');
-      return;
-    }
-    for (const e of events) {
-      let label = e.event_type || 'event';
-      if (e.round !== null) {label += ` round ${e.round}`;}
-      log(`--- ${label} | ${e.actor || 'unknown'} (${e.timestamp || e.fileCreated || ''}) ---`);
-      log(e.content || '(no body)');
-      log('');
-    }
-    return;
-  }
-
-  let reviewIdentity = resolveReviewIdentity(slug, worktree, {
-    readReviewStateFn,
-  }).identityUser;
-
-  if (!reviewIdentity) {
-    error(fmt.status('FAIL', `Cannot determine review identity for ${slug}. Persist review-state.json or set FORGEJO_USER.`));
-    exit(1);
-    return;
-  }
-  reviewIdentity = resolveReviewUserFn(reviewIdentity);
-
-  const token = readTokenFn(reviewIdentity, { rootDir: worktree });
-  if (!token) {
-    error(fmt.status('FAIL', `No Forgejo token found for user "${reviewIdentity}".`));
-    exit(1);
-    return;
-  }
-
-  log(fmt.status('INFO', `Reading PR comments on ${branch} as ${reviewIdentity}...`));
-  const comments = await getCommentsFn(branch, token);
-
-  if (comments === null) {
-    error(fmt.status('FAIL', `Could not fetch PR comments for ${branch}. The review provider may be unreachable or the PR is missing.`));
-    exit(1);
-    return;
-  }
-
-  if (comments.length === 0) {
-    log('no comments');
-    return;
-  }
-
-  for (const c of comments) {
-    let label = c.kind;
-    if (c.location) {label += ` ${c.location}`;}
-    log(`--- ${label} | ${c.user} (${c.created}) ---`);
-    log(c.body || '(no body)');
-    log('');
-  }
+    const log = options.log || fmt.log.plain;
+    const error = options.error || fmt.log.plainError;
+    const exit = options.exit || process.exit;
+    const readTokenFn = options.readTokenFn || review_adapter_js_1.readToken;
+    const readReviewStateFn = options.readReviewStateFn || review_state_js_1.readReviewState;
+    const resolveReviewUserFn = options.resolveReviewUserFn || options.resolveForgejoUserFn || review_adapter_js_1.resolveReviewUser;
+    const getCommentsFn = options.getCommentsFn || review_adapter_js_1.getComments;
+    const readAllEventsFn = options.readAllEventsFn || review_events_js_1.readAllEvents;
+    const isReviewProviderEnabledFn = options.isReviewProviderEnabledFn || options.isForgejoReviewEnabledFn || review_adapter_js_1.isProviderEnabled;
+    const worktree = options.worktree || (0, mission_utils_js_1.resolveWorktree)(slug) || process.cwd();
+    const branch = (0, mission_utils_js_1.missionBranchName)(slug, worktree);
+    // Provider-disabled fallback: read persisted local review events instead of polling the PR.
+    // Avoids resolving a provider token (which would FAIL/exit) when the provider is off.
+    if (!isReviewProviderEnabledFn(worktree)) {
+        log(fmt.status('INFO', `Review provider disabled; reading local review events for ${slug}...`));
+        const events = readAllEventsFn(slug, { rootDir: worktree, error });
+        if (!events || events.length === 0) {
+            log('no comments');
+            return;
+        }
+        for (const e of events) {
+            let label = e.event_type || 'event';
+            if (e.round !== null) {
+                label += ` round ${e.round}`;
+            }
+            log(`--- ${label} | ${e.actor || 'unknown'} (${e.timestamp || e.fileCreated || ''}) ---`);
+            log(e.content || '(no body)');
+            log('');
+        }
+        return;
+    }
+    let reviewIdentity = (0, review_state_js_1.resolveReviewIdentity)(slug, worktree, {
+        readReviewStateFn,
+    }).identityUser;
+    if (!reviewIdentity) {
+        error(fmt.status('FAIL', `Cannot determine review identity for ${slug}. Persist review-state.json or set FORGEJO_USER.`));
+        exit(1);
+        return;
+    }
+    reviewIdentity = resolveReviewUserFn(reviewIdentity);
+    const token = readTokenFn(reviewIdentity, { rootDir: worktree });
+    if (!token) {
+        error(fmt.status('FAIL', `No Forgejo token found for user "${reviewIdentity}".`));
+        exit(1);
+        return;
+    }
+    log(fmt.status('INFO', `Reading PR comments on ${branch} as ${reviewIdentity}...`));
+    const comments = await getCommentsFn(branch, token);
+    if (comments === null) {
+        error(fmt.status('FAIL', `Could not fetch PR comments for ${branch}. The review provider may be unreachable or the PR is missing.`));
+        exit(1);
+        return;
+    }
+    if (comments.length === 0) {
+        log('no comments');
+        return;
+    }
+    for (const c of comments) {
+        let label = c.kind;
+        if (c.location) {
+            label += ` ${c.location}`;
+        }
+        log(`--- ${label} | ${c.user} (${c.created}) ---`);
+        log(c.body || '(no body)');
+        log('');
+    }
 }
-
 // ============================================================================
 // Command: pushRound
 // ============================================================================
-
-/**
- * @param {string} slug
- * @param {{log?: Function, error?: Function, exit?: Function, resolveWorktreeFn?: Function, resolveTaskFileFn?: Function, getTaskImplementerFn?: Function, transitionTaskFn?: Function, resolveReviewUserFn?: Function, resolveForgejoUserFn?: Function, isReviewProviderEnabledFn?: Function, isForgejoReviewEnabledFn?: Function, readTokenFn?: Function, readReviewStateFn?: Function, createPrFn?: Function, bootstrapReviewSurfaceFn?: Function, resolveReviewAdapterFn?: Function, cwdFn?: Function, force?: boolean}} [options]
- * @returns {Promise<void>}
- */
 async function pushRound(slug, options = {}) {
-  const log = options.log || fmt.log.plain;
-  const error = options.error || fmt.log.plainError;
-  const exit = options.exit || process.exit;
-  const resolveWorktreeFn = options.resolveWorktreeFn || resolveWorktree;
-  const resolveTaskFileFn = options.resolveTaskFileFn || resolveTaskFile;
-  const getTaskImplementerFn = options.getTaskImplementerFn || getTaskImplementer;
-  const transitionTaskFn = options.transitionTaskFn || transitionTask;
-  const resolveReviewUserFn = options.resolveReviewUserFn || options.resolveForgejoUserFn || resolveReviewUser;
-  const isReviewProviderEnabledFn = options.isReviewProviderEnabledFn || options.isForgejoReviewEnabledFn || isProviderEnabled;
-  const readTokenFn = options.readTokenFn || readToken;
-  const readReviewStateFn = options.readReviewStateFn || readReviewState;
-  const createPrFn = options.createPrFn || createPr;
-  const bootstrapReviewSurfaceFn = options.bootstrapReviewSurfaceFn || bootstrapReviewSurface;
-  const resolveReviewAdapterFn = options.resolveReviewAdapterFn || resolveReviewAdapter;
-  const cwdFn = options.cwdFn || (() => process.cwd());
-  const force = options.force || false;
-  const worktree = resolveWorktreeFn(slug);
-  const rootDir = worktree || cwdFn();
-  const branch = missionBranchName(slug, rootDir);
-  const providerEnabled = isReviewProviderEnabledFn(rootDir);
-
-  const { identityUser: reviewStateUser } = resolveReviewIdentity(slug, rootDir, {
-    readReviewStateFn,
-  });
-  let reviewIdentity = reviewStateUser;
-
-  if (!reviewIdentity) {
-    // 1. Check backlog task assignee
-    const taskResolution = resolveTaskFileFn(slug, rootDir);
-    if (taskResolution.ok) {
-      reviewIdentity = getTaskImplementerFn(taskResolution.taskFile);
-    }
-  }
-
-  // 2. Mode-specific final fallback: named identity (provider-backed) vs "autonomous" (provider=none) (SC 6)
-  if (!reviewIdentity) {
-    if (providerEnabled) {
-      error(fmt.status('FAIL', `No review identity resolved for --push on ${slug}. Persist review-state.json or set the task implementer.`));
-      exit(1);
-      return;
-    } else {
-      reviewIdentity = 'autonomous';
-      log(fmt.status('WARN', `No reviewer/implementer identity resolved for --push; defaulting to "autonomous"`));
-    }
-  }
-
-  if (!reviewIdentity || (providerEnabled && reviewIdentity === 'autonomous')) {
-    error(fmt.status('FAIL', 'No review identity resolved for push.'));
-    exit(1);
-    return;
-  }
-
-  reviewIdentity = resolveReviewUserFn(reviewIdentity);
-  const token = readTokenFn(reviewIdentity, { rootDir: worktree });
-  if (!token) {
-    error(fmt.status('FAIL', `No Forgejo token found for user "${reviewIdentity}".`));
-    exit(1);
-    return;
-  }
-
-  // Transition to review before pushing so the state change is included in the PR update
-  transitionTaskFn(slug, 'review', { rootDir, log });
-
-  log(fmt.status('INFO', `Pushing ${branch} to the review provider as ${reviewIdentity}...${force ? ' (force-with-lease)' : ''}`));
-  if (worktree) {
-    log(fmt.status('INFO', `Found dedicated worktree: ${worktree}`));
-  }
-
-  let result = createPrFn(branch, reviewIdentity, token, { rootDir, forceWithLease: true });
-  if (!result.ok && /Repository not found/i.test(result.error || '')) {
-    const reviewAdapter = resolveReviewAdapterFn(rootDir);
-    const ownerLogin = (reviewAdapter.repo && reviewAdapter.repo.split('/')[0]) || 'magnus';
-    const bootstrap = await bootstrapReviewSurfaceFn(rootDir, {
-      baseUrl: reviewAdapter.baseUrl,
-      repo: reviewAdapter.repo,
-      ownerLogin,
-      ownerPassword: '',
-      agentPasswords: [],
-    }, {
-      interactive: false,
-      log,
-      error,
+    const log = options.log || fmt.log.plain;
+    const error = options.error || fmt.log.plainError;
+    const exit = options.exit || process.exit;
+    const resolveWorktreeFn = options.resolveWorktreeFn || mission_utils_js_1.resolveWorktree;
+    const resolveTaskFileFn = options.resolveTaskFileFn || backlog_js_1.resolveTaskFile;
+    const getTaskImplementerFn = options.getTaskImplementerFn || backlog_js_1.getTaskImplementer;
+    const transitionTaskFn = options.transitionTaskFn || backlog_js_1.transitionTask;
+    const resolveReviewUserFn = options.resolveReviewUserFn || options.resolveForgejoUserFn || review_adapter_js_1.resolveReviewUser;
+    const isReviewProviderEnabledFn = options.isReviewProviderEnabledFn || options.isForgejoReviewEnabledFn || review_adapter_js_1.isProviderEnabled;
+    const readTokenFn = options.readTokenFn || review_adapter_js_1.readToken;
+    const readReviewStateFn = options.readReviewStateFn || review_state_js_1.readReviewState;
+    const createPrFn = options.createPrFn || review_adapter_js_1.createPr;
+    const bootstrapReviewSurfaceFn = options.bootstrapReviewSurfaceFn || setup_review_js_1.bootstrapReviewSurface;
+    const resolveReviewAdapterFn = options.resolveReviewAdapterFn || product_config_js_1.resolveReviewAdapter;
+    const cwdFn = options.cwdFn || (() => process.cwd());
+    const force = options.force || false;
+    const worktree = resolveWorktreeFn(slug);
+    const rootDir = worktree || cwdFn();
+    const branch = (0, mission_utils_js_1.missionBranchName)(slug, rootDir);
+    const providerEnabled = isReviewProviderEnabledFn(rootDir);
+    const { identityUser: reviewStateUser } = (0, review_state_js_1.resolveReviewIdentity)(slug, rootDir, {
+        readReviewStateFn,
     });
-    if (bootstrap.ok) {
-      result = createPrFn(branch, reviewIdentity, token, { rootDir, forceWithLease: true });
-    }
-  }
-
-  if (!result.ok) {
-    error(fmt.status('FAIL', `Push to review provider failed: ${result.error}`));
-    exit(1);
-    return;
-  }
-
-  log(fmt.status('PASS', `Branch pushed and PR updated for ${branch}.`));
+    let reviewIdentity = reviewStateUser;
+    if (!reviewIdentity) {
+        // 1. Check backlog task assignee
+        const taskResolution = resolveTaskFileFn(slug, rootDir);
+        if (taskResolution.ok) {
+            reviewIdentity = getTaskImplementerFn(taskResolution.taskFile);
+        }
+    }
+    // 2. Mode-specific final fallback: named identity (provider-backed) vs "autonomous" (provider=none) (SC 6)
+    if (!reviewIdentity) {
+        if (providerEnabled) {
+            error(fmt.status('FAIL', `No review identity resolved for --push on ${slug}. Persist review-state.json or set the task implementer.`));
+            exit(1);
+            return;
+        }
+        else {
+            reviewIdentity = 'autonomous';
+            log(fmt.status('WARN', `No reviewer/implementer identity resolved for --push; defaulting to "autonomous"`));
+        }
+    }
+    if (!reviewIdentity || (providerEnabled && reviewIdentity === 'autonomous')) {
+        error(fmt.status('FAIL', 'No review identity resolved for push.'));
+        exit(1);
+        return;
+    }
+    reviewIdentity = resolveReviewUserFn(reviewIdentity);
+    const token = readTokenFn(reviewIdentity, { rootDir: worktree || undefined });
+    if (!token) {
+        error(fmt.status('FAIL', `No Forgejo token found for user "${reviewIdentity}".`));
+        exit(1);
+        return;
+    }
+    // Transition to review before pushing so the state change is included in the PR update
+    transitionTaskFn(slug, 'review', { rootDir, log });
+    log(fmt.status('INFO', `Pushing ${branch} to the review provider as ${reviewIdentity}...${force ? ' (force-with-lease)' : ''}`));
+    if (worktree) {
+        log(fmt.status('INFO', `Found dedicated worktree: ${worktree}`));
+    }
+    let result = createPrFn(branch, reviewIdentity, token, { rootDir, forceWithLease: true });
+    if (!result.ok && /Repository not found/i.test(result.error || '')) {
+        const reviewAdapter = resolveReviewAdapterFn(rootDir);
+        const ownerLogin = (reviewAdapter.repo && reviewAdapter.repo.split('/')[0]) || 'magnus';
+        const bootstrap = await bootstrapReviewSurfaceFn(rootDir, {
+            baseUrl: reviewAdapter.baseUrl,
+            repo: reviewAdapter.repo,
+            ownerLogin,
+            ownerPassword: '',
+            agentPasswords: [],
+        }, {
+            interactive: false,
+            log,
+            error,
+        });
+        if (bootstrap.ok) {
+            result = createPrFn(branch, reviewIdentity, token, { rootDir, forceWithLease: true });
+        }
+    }
+    if (!result.ok) {
+        error(fmt.status('FAIL', `Push to review provider failed: ${result.error}`));
+        exit(1);
+        return;
+    }
+    log(fmt.status('PASS', `Branch pushed and PR updated for ${branch}.`));
 }
-
 // ============================================================================
 // Command: showReviewStatus
 // ============================================================================
-
-/**
- * @param {string} slug
- * @param {{log?: Function, readReviewStateFn?: Function, resolveWorktreeFn?: Function}} [options]
- * @returns {void}
- */
 function showReviewStatus(slug, options = {}) {
-  const log = options.log || fmt.log.plain;
-  const readReviewStateFn = options.readReviewStateFn || readReviewState;
-  const resolveWorktreeFn = options.resolveWorktreeFn || resolveWorktree;
-
-  const worktree = resolveWorktreeFn(slug) || process.cwd();
-  const state = readReviewStateFn(slug, worktree);
-
-  log(fmt.status('INFO', `Review status for mission: ${fmt.slug(slug)}`));
-
-  if (!state) {
-    log(fmt.status('INFO', 'No persisted review state found.'));
-    return;
-  }
-
-  log(`  Round:       ${state.round}`);
-  log(`  Phase:       ${state.phase}`);
-  log(`  Reviewer:    ${fmt.agent(state.reviewer)}`);
-  log(`  Implementer: ${fmt.agent(state.implementer)}`);
-  log(`  Started at:  ${state.startedAt}`);
-  if (state.disposition) {
-    log(`  Disposition: ${state.disposition}`);
-  }
-  if (state.reviewerRetryCount > 0) {
-    log(`  Reviewer retries:    ${state.reviewerRetryCount}`);
-  }
-  if (state.implementerRetryCount > 0) {
-    log(`  Implementer retries: ${state.implementerRetryCount}`);
-  }
+    const log = options.log || fmt.log.plain;
+    const readReviewStateFn = options.readReviewStateFn || review_state_js_1.readReviewState;
+    const resolveWorktreeFn = options.resolveWorktreeFn || mission_utils_js_1.resolveWorktree;
+    const worktree = resolveWorktreeFn(slug) || process.cwd();
+    const state = readReviewStateFn(slug, worktree);
+    log(fmt.status('INFO', `Review status for mission: ${fmt.slug(slug)}`));
+    if (!state) {
+        log(fmt.status('INFO', 'No persisted review state found.'));
+        return;
+    }
+    log(`  Round:       ${state.round}`);
+    log(`  Phase:       ${state.phase}`);
+    log(`  Reviewer:    ${fmt.agent(state.reviewer ?? '')}`);
+    log(`  Implementer: ${fmt.agent(state.implementer ?? '')}`);
+    log(`  Started at:  ${state.startedAt}`);
+    if (state.disposition) {
+        log(`  Disposition: ${state.disposition}`);
+    }
+    if (state.reviewerRetryCount > 0) {
+        log(`  Reviewer retries:    ${state.reviewerRetryCount}`);
+    }
+    if (state.implementerRetryCount > 0) {
+        log(`  Implementer retries: ${state.implementerRetryCount}`);
+    }
 }
-
 // ============================================================================
 // Command: commentRound
 // ============================================================================
-
-/**
- * @param {string} slug
- * @param {string} message
- * @param {{log?: Function, error?: Function, exit?: Function, readReviewStateFn?: Function, writeReviewStateFn?: Function, resolveReviewUserFn?: Function, resolveForgejoUserFn?: Function, rootDir?: string, readTokenFn?: Function, postCommentFn?: Function, buildMetadataFooterFn?: Function}} [options]
- * @returns {void}
- */
 function commentRound(slug, message, options = {}) {
-  const log = options.log || fmt.log.plain;
-  const error = options.error || fmt.log.plainError;
-  const exit = options.exit || process.exit;
-  const readReviewStateFn = options.readReviewStateFn || readReviewState;
-  const writeReviewStateFn = options.writeReviewStateFn || writeReviewState;
-  const resolveReviewUserFn = options.resolveReviewUserFn || options.resolveForgejoUserFn || resolveReviewUser;
-  const rootDir = options.rootDir || resolveWorktree(slug) || process.cwd();
-  
-  let reviewIdentity = resolveReviewIdentity(slug, rootDir, {
-    readReviewStateFn,
-  }).identityUser;
-
-  if (!reviewIdentity) {
-    error(fmt.status('FAIL', `Cannot determine review identity for ${slug}. Persist review-state.json or set FORGEJO_USER.`));
-    exit(1);
-    return;
-  }
-  reviewIdentity = resolveReviewUserFn(reviewIdentity);
-  const result = postWorkflowComment(slug, message, {
-    rootDir,
-    reviewIdentity: reviewIdentity || undefined,
-    readTokenFn: options.readTokenFn,
-    postCommentFn: options.postCommentFn,
-    buildMetadataFooterFn: options.buildMetadataFooterFn,
-    log,
-    error
-  });
-
-  if (!result.ok) {
-    exit(1);
-    return;
-  }
-
-  const currentState = readReviewStateFn(slug, rootDir);
-  if (currentState) {
-    writeReviewStateFn(slug, currentState, rootDir);
-  }
+    const log = options.log || fmt.log.plain;
+    const error = options.error || fmt.log.plainError;
+    const exit = options.exit || process.exit;
+    const readReviewStateFn = options.readReviewStateFn || review_state_js_1.readReviewState;
+    const writeReviewStateFn = options.writeReviewStateFn || review_state_js_1.writeReviewState;
+    const resolveReviewUserFn = options.resolveReviewUserFn || options.resolveForgejoUserFn || review_adapter_js_1.resolveReviewUser;
+    const rootDir = options.rootDir || (0, mission_utils_js_1.resolveWorktree)(slug) || process.cwd();
+    let reviewIdentity = (0, review_state_js_1.resolveReviewIdentity)(slug, rootDir, {
+        readReviewStateFn,
+    }).identityUser;
+    if (!reviewIdentity) {
+        error(fmt.status('FAIL', `Cannot determine review identity for ${slug}. Persist review-state.json or set FORGEJO_USER.`));
+        exit(1);
+        return;
+    }
+    reviewIdentity = resolveReviewUserFn(reviewIdentity);
+    const result = (0, review_artifacts_js_1.postWorkflowComment)(slug, message, {
+        rootDir,
+        reviewIdentity: reviewIdentity || undefined,
+        readTokenFn: options.readTokenFn,
+        postCommentFn: options.postCommentFn,
+        buildMetadataFooterFn: options.buildMetadataFooterFn,
+        log,
+        error
+    });
+    if (!result.ok) {
+        exit(1);
+        return;
+    }
+    const currentState = readReviewStateFn(slug, rootDir);
+    if (currentState) {
+        writeReviewStateFn(slug, currentState, rootDir);
+    }
 }
-
 // ============================================================================
 // Command: consumeArtifacts
 // ============================================================================
-
-/**
- * @param {string} slug
- * @param {{log?: Function, error?: Function, resolveWorktreeFn?: Function, transitionTaskFn?: Function, consumeReviewerArtifactsFn?: Function, resolveTaskFileFn?: Function, getTaskAssigneeFn?: Function, getTaskStatusFn?: Function, resolveArtifactDirFn?: Function, readReviewStateFn?: Function, createEventFn?: Function, readArtifactFn?: Function, deleteArtifactFn?: Function}} [options]
- * @returns {Promise<{ok: boolean, consumed: boolean, reviewState?: string|null}>}
- */
 async function consumeArtifacts(slug, options = {}) {
-  const log = options.log || fmt.log.plain;
-  const error = options.error || fmt.log.plainError;
-  const resolveWorktreeFn = options.resolveWorktreeFn || resolveWorktree;
-  const transitionTaskFn = options.transitionTaskFn || transitionTask;
-  const consumeReviewerArtifactsFn = options.consumeReviewerArtifactsFn || consumeReviewerArtifacts;
-  const resolveTaskFileFn = options.resolveTaskFileFn || resolveTaskFile;
-  const getTaskAssigneeFn = options.getTaskAssigneeFn || getTaskAssignee;
-  const getTaskStatusFn = options.getTaskStatusFn || getTaskStatus;
-  const resolveArtifactDirFn = options.resolveArtifactDirFn || require('./review-artifacts').resolveArtifactDir;
-
-  const worktree = resolveWorktreeFn(slug) || process.cwd();
-  const rootDir = worktree;
-  const taskResolution = resolveTaskFileFn(slug, rootDir);
-
-  // Resolve artifact directory
-  const artifactDir = resolveArtifactDirFn(rootDir);
-  log(fmt.status('INFO', `Consuming reviewer artifacts for ${slug} from ${artifactDir}`));
-
-  // Determine reviewer identity from review-state first, then task assignee.
-  const { identityUser: stateReviewer } = resolveReviewIdentity(slug, worktree, {
-    readReviewStateFn: options.readReviewStateFn || readReviewState,
-  });
-  let reviewer = stateReviewer;
-  if (!reviewer && taskResolution.ok) {
-    reviewer = getTaskAssigneeFn(taskResolution.taskFile);
-  }
-  if (!reviewer) {
-    reviewer = 'autonomous';
-    log(fmt.status('WARN', `No reviewer identity resolved; defaulting to "${reviewer}"`));
-  }
-
-  // Consume artifacts - this will create reviewer_findings and reviewer_outcome events
-  const result = await consumeReviewerArtifactsFn(slug, reviewer, {
-    rootDir,
-    worktree,
-    tmpDir: artifactDir,
-    log,
-    error,
-    providerEnabled: false,
-    createEventFn: options.createEventFn,
-    readArtifactFn: options.readArtifactFn,
-    deleteArtifactFn: options.deleteArtifactFn,
-  });
-
-  if (!result.consumed) {
-    log(fmt.status('WARN', `No reviewer artifact files found at ${artifactDir} for ${slug}.`));
-    return { ok: false, consumed: false };
-  }
-
-  if (!result.ok) {
-    error(fmt.status('FAIL', `Failed to consume reviewer artifacts for ${slug}: ${result.ok === false ? 'missing required fields (findings, outcome, verdict)' : 'unknown failure'}`));
-    return { ok: false, consumed: true };
-  }
-
-  // Persist the artifact location to review-state.json if it doesn't exist yet
-  const persisted = readReviewState(slug, worktree);
-  if (!persisted) {
-    const { ReviewState } = require('./review-state');
-    writeReviewState(slug, new ReviewState(slug, {
-      reviewer,
-      round: 1,
-      phase: 'reviewing',
-    }), worktree);
-    log(fmt.status('INFO', 'Created review-state.json for artifact consumption.'));
-  }
-
-  // Transition backlog task to review status
-  if (taskResolution.ok) {
-    const currentStatus = getTaskStatusFn ? getTaskStatusFn(taskResolution.taskFile) : null;
-    if (!currentStatus || currentStatus !== 'review') {
-      transitionTaskFn(slug, 'review', { rootDir, log });
-    } else {
-      log(fmt.status('INFO', `Backlog task for ${slug} already at review status.`));
-    }
-  }
-
-  const cleanup = await commitPersistedReviewOutputs(slug, {
-    worktree,
-    taskFile: taskResolution.ok ? taskResolution.taskFile : null,
-    log,
-    error
-  });
-  if (!cleanup.ok) {
-    error(fmt.status('FAIL', `Consumed reviewer artifacts for ${slug}, but could not commit the persisted mission artifacts.`));
-    return { ok: false, consumed: true };
-  }
-
-  log(fmt.status('PASS', `Reviewer artifacts consumed for ${slug}. Backlog task set to review.`));
-  return { ok: true, consumed: true, reviewState: result.reviewState };
+    const log = options.log || fmt.log.plain;
+    const error = options.error || fmt.log.plainError;
+    const resolveWorktreeFn = options.resolveWorktreeFn || mission_utils_js_1.resolveWorktree;
+    const transitionTaskFn = options.transitionTaskFn || backlog_js_1.transitionTask;
+    const consumeReviewerArtifactsFn = options.consumeReviewerArtifactsFn || review_artifacts_js_1.consumeReviewerArtifacts;
+    const resolveTaskFileFn = options.resolveTaskFileFn || backlog_js_1.resolveTaskFile;
+    const getTaskAssigneeFn = options.getTaskAssigneeFn || backlog_js_1.getTaskAssignee;
+    const getTaskStatusFn = options.getTaskStatusFn || backlog_js_1.getTaskStatus;
+    const resolveArtifactDirFn = options.resolveArtifactDirFn || review_artifacts_js_1.resolveArtifactDir;
+    const worktree = resolveWorktreeFn(slug) || process.cwd();
+    const rootDir = worktree;
+    const taskResolution = resolveTaskFileFn(slug, rootDir);
+    // Resolve artifact directory
+    const artifactDir = resolveArtifactDirFn(rootDir);
+    log(fmt.status('INFO', `Consuming reviewer artifacts for ${slug} from ${artifactDir}`));
+    // Determine reviewer identity from review-state first, then task assignee.
+    const { identityUser: stateReviewer } = (0, review_state_js_1.resolveReviewIdentity)(slug, worktree, {
+        readReviewStateFn: options.readReviewStateFn || review_state_js_1.readReviewState,
+    });
+    let reviewer = stateReviewer;
+    if (!reviewer && taskResolution.ok) {
+        reviewer = getTaskAssigneeFn(taskResolution.taskFile);
+    }
+    if (!reviewer) {
+        reviewer = 'autonomous';
+        log(fmt.status('WARN', `No reviewer identity resolved; defaulting to "${reviewer}"`));
+    }
+    // Consume artifacts - this will create reviewer_findings and reviewer_outcome events
+    const result = await consumeReviewerArtifactsFn(slug, reviewer, {
+        worktree,
+        tmpDir: artifactDir,
+        log,
+        error,
+        providerEnabled: false,
+        createEventFn: options.createEventFn,
+        readArtifactFn: options.readArtifactFn,
+        deleteArtifactFn: options.deleteArtifactFn,
+    });
+    if (!result.consumed) {
+        log(fmt.status('WARN', `No reviewer artifact files found at ${artifactDir} for ${slug}.`));
+        return { ok: false, consumed: false };
+    }
+    if (!result.ok) {
+        error(fmt.status('FAIL', `Failed to consume reviewer artifacts for ${slug}: ${result.ok === false ? 'missing required fields (findings, outcome, verdict)' : 'unknown failure'}`));
+        return { ok: false, consumed: true };
+    }
+    // Persist the artifact location to review-state.json if it doesn't exist yet
+    const persisted = (0, review_state_js_1.readReviewState)(slug, worktree);
+    if (!persisted) {
+        (0, review_state_js_1.writeReviewState)(slug, new review_state_js_1.ReviewState(slug, {
+            reviewer,
+            round: 1,
+            phase: 'reviewing',
+        }), worktree);
+        log(fmt.status('INFO', 'Created review-state.json for artifact consumption.'));
+    }
+    // Transition backlog task to review status
+    if (taskResolution.ok) {
+        const currentStatus = getTaskStatusFn ? getTaskStatusFn(taskResolution.taskFile) : null;
+        if (!currentStatus || currentStatus !== 'review') {
+            transitionTaskFn(slug, 'review', { rootDir, log });
+        }
+        else {
+            log(fmt.status('INFO', `Backlog task for ${slug} already at review status.`));
+        }
+    }
+    const cleanup = await commitPersistedReviewOutputs(slug, {
+        worktree,
+        taskFile: taskResolution.ok ? taskResolution.taskFile : null,
+        log,
+        error
+    });
+    if (!cleanup.ok) {
+        error(fmt.status('FAIL', `Consumed reviewer artifacts for ${slug}, but could not commit the persisted mission artifacts.`));
+        return { ok: false, consumed: true };
+    }
+    log(fmt.status('PASS', `Reviewer artifacts consumed for ${slug}. Backlog task set to review.`));
+    return { ok: true, consumed: true, reviewState: result.reviewState };
 }
-
 // ============================================================================
 // Command: submitReviewRound
 // ============================================================================
-
-/**
- * @param {string} slug
- * @param {string} outcome
- * @param {string} message
- * @param {{log?: Function, error?: Function, exit?: Function, transitionTaskFn?: Function, readReviewStateFn?: Function, writeReviewStateFn?: Function, isReviewProviderEnabledFn?: Function, isForgejoReviewEnabledFn?: Function, resolveReviewUserFn?: Function, resolveForgejoUserFn?: Function, resolveTaskFileFn?: Function, getTaskStatusFn?: Function, worktree?: string, readTokenFn?: Function, postReviewFn?: Function, getPrAuthorFn?: Function, createEventFn?: Function, buildMetadataFooterFn?: Function}} [options]
- * @returns {void}
- */
 function submitReviewRound(slug, outcome, message, options = {}) {
-  const log = options.log || fmt.log.plain;
-  const error = options.error || fmt.log.plainError;
-  const exit = options.exit || process.exit;
-  const transitionTaskFn = options.transitionTaskFn || transitionTask;
-  const readReviewStateFn = options.readReviewStateFn || readReviewState;
-  const writeReviewStateFn = options.writeReviewStateFn || writeReviewState;
-  const isReviewProviderEnabledFn = options.isReviewProviderEnabledFn || options.isForgejoReviewEnabledFn || isProviderEnabled;
-  const resolveReviewUserFn = options.resolveReviewUserFn || options.resolveForgejoUserFn || resolveReviewUser;
-  const resolveTaskFileFn = options.resolveTaskFileFn || resolveTaskFile;
-  const getTaskStatusFn = options.getTaskStatusFn || getTaskStatus;
-  const VALID_OUTCOMES = ['approve', 'request-changes', 'comment'];
-  if (!VALID_OUTCOMES.includes(outcome)) {
-    error(fmt.status('FAIL', `Unknown review outcome "${outcome}". Valid: ${VALID_OUTCOMES.join(', ')}.`));
-    exit(1);
-    return;
-  }
-
-  const worktree = options.worktree || resolveWorktree(slug) || process.cwd();
-  const providerEnabled = isReviewProviderEnabledFn(worktree);
-
-  // For provider=none (standalone), skip provider posting and only update review-state
-  if (!providerEnabled) {
-    log(fmt.status('INFO', `Review provider is none — skipping provider posting, updating review-state only for ${slug}.`));
-    const { ReviewState } = require('./review-state');
-    const transitionTaskFn = options.transitionTaskFn || transitionTask;
+    const log = options.log || fmt.log.plain;
+    const error = options.error || fmt.log.plainError;
+    const exit = options.exit || process.exit;
+    const transitionTaskFn = options.transitionTaskFn || backlog_js_1.transitionTask;
+    const readReviewStateFn = options.readReviewStateFn || review_state_js_1.readReviewState;
+    const writeReviewStateFn = options.writeReviewStateFn || review_state_js_1.writeReviewState;
+    const isReviewProviderEnabledFn = options.isReviewProviderEnabledFn || options.isForgejoReviewEnabledFn || review_adapter_js_1.isProviderEnabled;
+    const resolveReviewUserFn = options.resolveReviewUserFn || options.resolveForgejoUserFn || review_adapter_js_1.resolveReviewUser;
+    const resolveTaskFileFn = options.resolveTaskFileFn || backlog_js_1.resolveTaskFile;
+    const getTaskStatusFn = options.getTaskStatusFn || backlog_js_1.getTaskStatus;
+    const VALID_OUTCOMES = ['approve', 'request-changes', 'comment'];
+    if (!VALID_OUTCOMES.includes(outcome)) {
+        error(fmt.status('FAIL', `Unknown review outcome "${outcome}". Valid: ${VALID_OUTCOMES.join(', ')}.`));
+        exit(1);
+        return;
+    }
+    const worktree = options.worktree || (0, mission_utils_js_1.resolveWorktree)(slug) || process.cwd();
+    const providerEnabled = isReviewProviderEnabledFn(worktree);
+    // For provider=none (standalone), skip provider posting and only update review-state
+    if (!providerEnabled) {
+        log(fmt.status('INFO', `Review provider is none — skipping provider posting, updating review-state only for ${slug}.`));
+        const currentState = readReviewStateFn(slug, worktree);
+        let stateToWrite;
+        if (currentState) {
+            stateToWrite = currentState;
+            if (outcome === 'approve') {
+                stateToWrite.disposition = 'APPROVED';
+                try {
+                    stateToWrite.transitionTo('approved');
+                }
+                catch (_) { /* ignore */ }
+            }
+            else if (outcome === 'request-changes') {
+                stateToWrite.disposition = 'REQUEST_CHANGES';
+                try {
+                    stateToWrite.transitionTo('fixing');
+                }
+                catch (_) { /* ignore */ }
+            }
+        }
+        else {
+            // No existing state, create minimal state for tracking
+            const phaseForOutcome = outcome === 'approve' ? 'approved' : 'fixing';
+            stateToWrite = new review_state_js_1.ReviewState(slug, {
+                disposition: outcome === 'approve' ? 'APPROVED' : outcome === 'request-changes' ? 'REQUEST_CHANGES' : undefined,
+                reviewer: 'autonomous',
+                implementer: process.env.WORKFLOW_AGENT || 'autonomous',
+                round: 1,
+                phase: phaseForOutcome,
+            });
+        }
+        writeReviewStateFn(slug, stateToWrite, worktree);
+        // Also transition the backlog task for provider=none so integrate preflight passes
+        const backlogStatusMap = {
+            'approve': 'approved',
+            'request-changes': 'review',
+            'comment': 'review'
+        };
+        const backlogStatus = backlogStatusMap[outcome];
+        if (backlogStatus && !transitionTaskFn(slug, backlogStatus, { rootDir: worktree, log })) {
+            log(fmt.status('WARN', `Could not transition backlog task ${slug} to ${backlogStatus}.`));
+        }
+        log(fmt.status('PASS', `Review outcome "${outcome}" recorded locally for ${slug}.`));
+        return;
+    }
+    // For provider-backed reviews, post through the adapter. Review-state is the normal source.
+    let reviewIdentity = (0, review_state_js_1.resolveReviewIdentity)(slug, worktree, {
+        readReviewStateFn,
+    }).identityUser;
+    if (!reviewIdentity) {
+        error(fmt.status('FAIL', `Cannot determine review identity for ${slug}. Persist review-state.json or set FORGEJO_USER.`));
+        exit(1);
+        return;
+    }
+    reviewIdentity = resolveReviewUserFn(reviewIdentity);
+    const result = (0, review_artifacts_js_1.postWorkflowReview)(slug, outcome, message, {
+        worktree,
+        reviewIdentity: reviewIdentity || undefined,
+        readTokenFn: options.readTokenFn,
+        postReviewFn: options.postReviewFn,
+        getPrAuthorFn: options.getPrAuthorFn,
+        writeReviewStateFn: options.writeReviewStateFn,
+        createEventFn: options.createEventFn,
+        readReviewStateFn: options.readReviewStateFn,
+        buildMetadataFooterFn: options.buildMetadataFooterFn,
+        log,
+        error
+    });
+    // Self-author skip: postWorkflowReview intentionally did not POST to the provider
+    // (reviewer == PR author) and already persisted the verdict locally. This is
+    // a legitimate same-agent-reviewer fallback, not a failure — do NOT exit(1).
+    if (result.ok && result.skipped) {
+        repairStaleActiveTaskAfterReview(slug, {
+            rootDir: worktree,
+            resolveTaskFileFn,
+            getTaskStatusFn,
+            transitionTaskFn,
+            log,
+            error
+        });
+        log(fmt.status('WARN', `Review outcome "${outcome}" recorded locally for ${slug} (self-approval POST skipped). A different agent or a human must post the formal provider approval.`));
+        return;
+    }
+    if (!result.ok) {
+        exit(1);
+        return;
+    }
     const currentState = readReviewStateFn(slug, worktree);
-
-    let stateToWrite;
     if (currentState) {
-      stateToWrite = currentState;
-      if (outcome === 'approve') {
-        stateToWrite.disposition = 'APPROVED';
-        try { stateToWrite.transitionTo('approved'); } catch (_) {}
-      } else if (outcome === 'request-changes') {
-        stateToWrite.disposition = 'REQUEST_CHANGES';
-        try { stateToWrite.transitionTo('fixing'); } catch (_) {}
-      }
-    } else {
-      // No existing state, create minimal state for tracking
-      const phaseForOutcome = outcome === 'approve' ? 'approved' : 'fixing';
-      stateToWrite = new ReviewState(slug, {
-        disposition: outcome === 'approve' ? 'APPROVED' : outcome === 'request-changes' ? 'REQUEST_CHANGES' : undefined,
-        reviewer: 'autonomous',
-        implementer: process.env.WORKFLOW_AGENT || 'autonomous',
-        round: 1,
-        phase: phaseForOutcome,
-      });
-    }
-    writeReviewStateFn(slug, stateToWrite, worktree);
-
-    // Also transition the backlog task for provider=none so integrate preflight passes
-    const backlogStatusMap = {
-      'approve': 'approved',
-      'request-changes': 'review',
-      'comment': 'review'
-    };
-    const backlogStatus = backlogStatusMap[/** @type {keyof typeof backlogStatusMap} */ (outcome)];
-    if (backlogStatus && !transitionTaskFn(slug, backlogStatus, { rootDir: worktree, log })) {
-      log(fmt.status('WARN', `Could not transition backlog task ${slug} to ${backlogStatus}.`));
-    }
-    
-    log(fmt.status('PASS', `Review outcome "${outcome}" recorded locally for ${slug}.`));
-    return;
-  }
-  
-  // For provider-backed reviews, post through the adapter. Review-state is the normal source.
-  let reviewIdentity = resolveReviewIdentity(slug, worktree, {
-    readReviewStateFn,
-  }).identityUser;
-  
-  if (!reviewIdentity) {
-    error(fmt.status('FAIL', `Cannot determine review identity for ${slug}. Persist review-state.json or set FORGEJO_USER.`));
-    exit(1);
-    return;
-  }
-  reviewIdentity = resolveReviewUserFn(reviewIdentity);
-  const result = postWorkflowReview(slug, outcome, message, {
-    worktree,
-    reviewIdentity: reviewIdentity || undefined,
-    readTokenFn: options.readTokenFn,
-    postReviewFn: options.postReviewFn,
-    getPrAuthorFn: options.getPrAuthorFn,
-    writeReviewStateFn: options.writeReviewStateFn,
-    createEventFn: options.createEventFn,
-    readReviewStateFn: options.readReviewStateFn,
-    buildMetadataFooterFn: options.buildMetadataFooterFn,
-    log,
-    error
-  });
-
-  // Self-author skip: postWorkflowReview intentionally did not POST to the provider
-  // (reviewer == PR author) and already persisted the verdict locally. This is
-  // a legitimate same-agent-reviewer fallback, not a failure — do NOT exit(1).
-  if (result.ok && result.skipped) {
-    repairStaleActiveTaskAfterReview(slug, {
-      rootDir: worktree,
-      resolveTaskFileFn,
-      getTaskStatusFn,
-      transitionTaskFn,
-      log,
-      error
-    });
-    log(fmt.status('WARN', `Review outcome "${outcome}" recorded locally for ${slug} (self-approval POST skipped). A different agent or a human must post the formal provider approval.`));
-    return;
-  }
-
-  if (!result.ok) {
-    exit(1);
-    return;
-  }
-
-  const currentState = readReviewStateFn(slug, worktree);
-  if (currentState) {
+        if (outcome === 'approve') {
+            currentState.disposition = 'APPROVED';
+            try {
+                currentState.transitionTo('approved');
+            }
+            catch (_) { /* ignore */ }
+        }
+        else if (outcome === 'request-changes') {
+            currentState.disposition = 'REQUEST_CHANGES';
+            try {
+                currentState.transitionTo('fixing');
+            }
+            catch (_) { /* ignore */ }
+        }
+        writeReviewStateFn(slug, currentState, worktree);
+    }
+    const taskResolution = resolveTaskFileFn(slug, worktree);
+    const currentStatus = taskResolution.ok ? getTaskStatusFn(taskResolution.taskFile) : null;
+    let backlogStatus = null;
     if (outcome === 'approve') {
-      currentState.disposition = 'APPROVED';
-      try { currentState.transitionTo('approved'); } catch (_) {}
-    } else if (outcome === 'request-changes') {
-      currentState.disposition = 'REQUEST_CHANGES';
-      try { currentState.transitionTo('fixing'); } catch (_) {}
-    }
-    writeReviewStateFn(slug, currentState, worktree);
-  }
-
-  const taskResolution = resolveTaskFileFn(slug, worktree);
-  const currentStatus = taskResolution.ok ? getTaskStatusFn(taskResolution.taskFile) : null;
-  let backlogStatus = null;
-
-  if (outcome === 'approve') {
-    backlogStatus = currentStatus === 'active' ? 'review' : 'approved';
-  } else if (outcome === 'request-changes' || outcome === 'comment') {
-    backlogStatus = 'review';
-  }
-
-  if (backlogStatus && !transitionTaskFn(slug, backlogStatus, { rootDir: worktree, log })) {
-    log(fmt.status('WARN', `Could not transition backlog task ${slug} to ${backlogStatus}.`));
-  }
+        backlogStatus = currentStatus === 'active' ? 'review' : 'approved';
+    }
+    else if (outcome === 'request-changes' || outcome === 'comment') {
+        backlogStatus = 'review';
+    }
+    if (backlogStatus && !transitionTaskFn(slug, backlogStatus, { rootDir: worktree, log })) {
+        log(fmt.status('WARN', `Could not transition backlog task ${slug} to ${backlogStatus}.`));
+    }
 }
-
 // ============================================================================
 // Command: closeMissionPr
 // ============================================================================
-
-/**
- * @param {string} slug
- * @param {{log?: Function, error?: Function, exit?: Function, readTokenFn?: Function, readReviewStateFn?: Function, resolveReviewUserFn?: Function, resolveForgejoUserFn?: Function, closePrFn?: Function, worktree?: string}} [options]
- * @returns {Promise<void>}
- */
 async function closeMissionPr(slug, options = {}) {
-  const log = options.log || fmt.log.plain;
-  const error = options.error || fmt.log.plainError;
-  const exit = options.exit || process.exit;
-  const readTokenFn = options.readTokenFn || readToken;
-  const readReviewStateFn = options.readReviewStateFn || readReviewState;
-  const resolveReviewUserFn = options.resolveReviewUserFn || options.resolveForgejoUserFn || resolveReviewUser;
-  const closePrFn = options.closePrFn || closePr;
-  const worktree = options.worktree || resolveWorktree(slug) || process.cwd();
-  const branch = missionBranchName(slug, worktree);
-  
-  let reviewIdentity = resolveReviewIdentity(slug, worktree, {
-    readReviewStateFn,
-  }).identityUser;
-
-  if (!reviewIdentity) {
-    error(fmt.status('FAIL', `Cannot determine review identity for ${slug}. Persist review-state.json or set FORGEJO_USER.`));
-    exit(1);
-    return;
-  }
-  reviewIdentity = resolveReviewUserFn(reviewIdentity);
-
-  const token = readTokenFn(reviewIdentity, { rootDir: worktree });
-  if (!token) {
-    error(fmt.status('FAIL', `No Forgejo token found for user "${reviewIdentity}".`));
-    exit(1);
-    return;
-  }
-
-  log(fmt.status('INFO', `Closing PR for ${branch} as ${reviewIdentity}...`));
-  const result = await closePrFn(branch, token, reviewIdentity);
-  
-  if (!result.ok) {
-    exit(1);
-  }
+    const log = options.log || fmt.log.plain;
+    const error = options.error || fmt.log.plainError;
+    const exit = options.exit || process.exit;
+    const readTokenFn = options.readTokenFn || review_adapter_js_1.readToken;
+    const readReviewStateFn = options.readReviewStateFn || review_state_js_1.readReviewState;
+    const resolveReviewUserFn = options.resolveReviewUserFn || options.resolveForgejoUserFn || review_adapter_js_1.resolveReviewUser;
+    const closePrFn = options.closePrFn || review_adapter_js_1.closePr;
+    const worktree = options.worktree || (0, mission_utils_js_1.resolveWorktree)(slug) || process.cwd();
+    const branch = (0, mission_utils_js_1.missionBranchName)(slug, worktree);
+    let reviewIdentity = (0, review_state_js_1.resolveReviewIdentity)(slug, worktree, {
+        readReviewStateFn,
+    }).identityUser;
+    if (!reviewIdentity) {
+        error(fmt.status('FAIL', `Cannot determine review identity for ${slug}. Persist review-state.json or set FORGEJO_USER.`));
+        exit(1);
+        return;
+    }
+    reviewIdentity = resolveReviewUserFn(reviewIdentity);
+    const token = readTokenFn(reviewIdentity, { rootDir: worktree });
+    if (!token) {
+        error(fmt.status('FAIL', `No Forgejo token found for user "${reviewIdentity}".`));
+        exit(1);
+        return;
+    }
+    log(fmt.status('INFO', `Closing PR for ${branch} as ${reviewIdentity}...`));
+    const result = await closePrFn(branch, token, reviewIdentity);
+    if (!result.ok) {
+        exit(1);
+    }
 }
-
 // ============================================================================
 // Event CLI Handlers
 // ============================================================================
-
-/**
- * @param {string} slug
- * @param {string[]} args
- * @param {{log?: Function, error?: Function, exit?: Function, resolveWorktreeFn?: Function, readReviewStateFn?: Function}} [options]
- * @returns {void}
- */
 function createEventHandler(slug, args, options = {}) {
-  const log = options.log || fmt.log.plain;
-  const error = options.error || fmt.log.plainError;
-  const exit = options.exit || process.exit;
-  const resolveWorktreeFn = options.resolveWorktreeFn || resolveWorktree;
-
-  const eventType = flagValue(args, '--type');
-  const inputFile = flagValue(args, '--input-file');
-  const actor = flagValue(args, '--actor');
-  const roundRaw = flagValue(args, '--round');
-  const phase = flagValue(args, '--phase');
-  const disposition = flagValue(args, '--disposition');
-  const verdict = flagValue(args, '--verdict');
-
-  if (!eventType) {
-    error(fmt.status('FAIL', '--create-event requires --type <classification>'));
-    exit(1);
-    return;
-  }
-
-  // Validate event type first
-  if (!isValidEventType(eventType)) {
-    error(fmt.status('FAIL', `Invalid event type "${eventType}". Valid: ${ALL_EVENT_TYPES.join(', ')}`));
-    exit(1);
-    return;
-  }
-
-  // Read content from input file or stdin
-  let content = '';
-  if (inputFile) {
-    try {
-      content = fs.readFileSync(inputFile, 'utf8');
-      log(fmt.status('INFO', `Read event content from: ${inputFile}`));
-    } catch (err) {
-      error(fmt.status('FAIL', `Failed to read input file: ${/** @type {Error} */(err).message}`));
-      exit(1);
-      return;
-    }
-  }
-
-  const round = roundRaw ? parseInt(roundRaw, 10) : undefined;
-  if (roundRaw && (isNaN(/** @type {number} */ (round))) ) {
-    error(fmt.status('FAIL', `--round must be a number, got "${roundRaw}"`));
-    exit(1);
-    return;
-  }
-
-  const worktree = resolveWorktreeFn(slug) || process.cwd();
-
-  // Build params
-  /** @type {{content: string, round?: number, phase?: string, actor?: string, disposition?: string, verdict?: string, fixedItems?: string[], pushedBackItems?: string[], parkedItems?: string[], blockedReason?: string}} */
-  const params = { content };
-  if (round !== undefined) {params.round = round;}
-  if (phase) {params.phase = phase;}
-  if (actor) {params.actor = actor;}
-  if (disposition) {params.disposition = disposition;}
-  if (verdict) {params.verdict = verdict;}
-
-  const readReviewStateFn = options.readReviewStateFn || readReviewState;
-  const { identityUser: stateReviewIdentity } = resolveReviewIdentity(slug, worktree, {
-    readReviewStateFn,
-  });
-  const reviewIdentity = actor || stateReviewIdentity;
-
-  // SC 4: For mirrored event types, a provider identity is required before creating the event.
-  if (shouldMirrorToProvider(eventType) && !reviewIdentity) {
-    error(fmt.status('FAIL', 'Cannot determine review identity for a mirrored event. Validate review-state.json or use --actor.'));
-    exit(1);
-    return;
-  }
-
-  // Extract fields from content if structured
-  if (content.includes('fixed_items:') || content.includes('fixedItems:')) {
-    try {
-      const frontmatterMatch = content.match(/fixed_items:\s*(\[[^\]]*\])/i);
-      if (frontmatterMatch) {
-        params.fixedItems = JSON.parse(frontmatterMatch[1]);
-      }
-    } catch (_) {}
-    try {
-      const frontmatterMatch = content.match(/pushed_back_items:\s*(\[[^\]]*\])/i);
-      if (frontmatterMatch) {
-        params.pushedBackItems = JSON.parse(frontmatterMatch[1]);
-      }
-    } catch (_) {}
-    try {
-      const frontmatterMatch = content.match(/parked_items:\s*(\[[^\]]*\])/i);
-      if (frontmatterMatch) {
-        params.parkedItems = JSON.parse(frontmatterMatch[1]);
-      }
-    } catch (_) {}
-    try {
-      const frontmatterMatch = content.match(/blocked_reason:\s*"([^"]*)"/i);
-      if (frontmatterMatch) {
-        params.blockedReason = frontmatterMatch[1];
-      }
-    } catch (_) {}
-  }
-
-  // Create the event
-  const result = createEvent(slug, eventType, params, {
-    worktree,
-    skipGit: false,
-    log,
-    error
-  });
-
-  if (!result.ok) {
-    error(fmt.status('FAIL', `Could not create event: ${result.error}`));
-    exit(1);
-    return;
-  }
-
-  log(fmt.status('PASS', `Created review event at: ${result.path}`));
+    const log = options.log || fmt.log.plain;
+    const error = options.error || fmt.log.plainError;
+    const exit = options.exit || process.exit;
+    const resolveWorktreeFn = options.resolveWorktreeFn || mission_utils_js_1.resolveWorktree;
+    const eventType = flagValue(args, '--type');
+    const inputFile = flagValue(args, '--input-file');
+    const actor = flagValue(args, '--actor');
+    const roundRaw = flagValue(args, '--round');
+    const phase = flagValue(args, '--phase');
+    const disposition = flagValue(args, '--disposition');
+    const verdict = flagValue(args, '--verdict');
+    if (!eventType) {
+        error(fmt.status('FAIL', '--create-event requires --type <classification>'));
+        exit(1);
+        return;
+    }
+    // Validate event type first
+    if (!(0, review_events_js_1.isValidEventType)(eventType)) {
+        error(fmt.status('FAIL', `Invalid event type "${eventType}". Valid: ${review_events_js_1.ALL_EVENT_TYPES.join(', ')}`));
+        exit(1);
+        return;
+    }
+    // Read content from input file or stdin
+    let content = '';
+    if (inputFile) {
+        try {
+            content = fs.readFileSync(inputFile, 'utf8');
+            log(fmt.status('INFO', `Read event content from: ${inputFile}`));
+        }
+        catch (err) {
+            error(fmt.status('FAIL', `Failed to read input file: ${err.message}`));
+            exit(1);
+            return;
+        }
+    }
+    const round = roundRaw ? parseInt(roundRaw, 10) : undefined;
+    if (roundRaw && isNaN(round)) {
+        error(fmt.status('FAIL', `--round must be a number, got "${roundRaw}"`));
+        exit(1);
+        return;
+    }
+    const worktree = resolveWorktreeFn(slug) || process.cwd();
+    const params = { content };
+    if (round !== undefined) {
+        params.round = round;
+    }
+    if (phase) {
+        params.phase = phase;
+    }
+    if (actor) {
+        params.actor = actor;
+    }
+    if (disposition) {
+        params.disposition = disposition;
+    }
+    if (verdict) {
+        params.verdict = verdict;
+    }
+    const readReviewStateFn = options.readReviewStateFn || review_state_js_1.readReviewState;
+    const { identityUser: stateReviewIdentity } = (0, review_state_js_1.resolveReviewIdentity)(slug, worktree, {
+        readReviewStateFn,
+    });
+    const reviewIdentity = actor || stateReviewIdentity;
+    // SC 4: For mirrored event types, a provider identity is required before creating the event.
+    if ((0, review_events_js_1.shouldMirrorToProvider)(eventType) && !reviewIdentity) {
+        error(fmt.status('FAIL', 'Cannot determine review identity for a mirrored event. Validate review-state.json or use --actor.'));
+        exit(1);
+        return;
+    }
+    // Extract fields from content if structured
+    if (content.includes('fixed_items:') || content.includes('fixedItems:')) {
+        try {
+            const frontmatterMatch = content.match(/fixed_items:\s*(\[[^\]]*\])/i);
+            if (frontmatterMatch) {
+                params.fixedItems = JSON.parse(frontmatterMatch[1]);
+            }
+        }
+        catch (_) { /* ignore */ }
+        try {
+            const frontmatterMatch = content.match(/pushed_back_items:\s*(\[[^\]]*\])/i);
+            if (frontmatterMatch) {
+                params.pushedBackItems = JSON.parse(frontmatterMatch[1]);
+            }
+        }
+        catch (_) { /* ignore */ }
+        try {
+            const frontmatterMatch = content.match(/parked_items:\s*(\[[^\]]*\])/i);
+            if (frontmatterMatch) {
+                params.parkedItems = JSON.parse(frontmatterMatch[1]);
+            }
+        }
+        catch (_) { /* ignore */ }
+        try {
+            const frontmatterMatch = content.match(/blocked_reason:\s*"([^"]*)"/i);
+            if (frontmatterMatch) {
+                params.blockedReason = frontmatterMatch[1];
+            }
+        }
+        catch (_) { /* ignore */ }
+    }
+    // Create the event
+    const result = (0, review_events_js_1.createEvent)(slug, eventType, params, {
+        worktree,
+        skipGit: false,
+        log,
+        error
+    });
+    if (!result.ok) {
+        error(fmt.status('FAIL', `Could not create event: ${result.error}`));
+        exit(1);
+        return;
+    }
+    log(fmt.status('PASS', `Created review event at: ${result.path}`));
 }
-
-/**
- * @param {string} slug
- * @param {string[]} args
- * @param {{log?: Function, error?: Function, exit?: Function, resolveWorktreeFn?: Function}} [options]
- * @returns {void}
- */
 function importLegacyHandler(slug, args, options = {}) {
-  const log = options.log || fmt.log.plain;
-  const error = options.error || fmt.log.plainError;
-  const exit = options.exit || process.exit;
-  const resolveWorktreeFn = options.resolveWorktreeFn || resolveWorktree;
-
-  const tmpDir = flagValue(args, '--tmp-dir') || process.env.WORKFLOW_TMP_DIR || os.tmpdir();
-  const worktree = resolveWorktreeFn(slug) || process.cwd();
-
-  const result = importAllLegacyArtifacts(slug, { tmpDir, worktree, log, error });
-  
-  const errors = result.errors || [];
-  if (!result.ok) {
-    error(fmt.status('FAIL', `Legacy import failed: ${errors.join(', ')}`));
-    exit(1);
-    return;
-  }
-
-  log(fmt.status('PASS', `Imported ${result.imported ? result.imported.length : 0} legacy artifacts for ${slug}`));
+    const log = options.log || fmt.log.plain;
+    const error = options.error || fmt.log.plainError;
+    const exit = options.exit || process.exit;
+    const resolveWorktreeFn = options.resolveWorktreeFn || mission_utils_js_1.resolveWorktree;
+    const tmpDir = flagValue(args, '--tmp-dir') || process.env.WORKFLOW_TMP_DIR || os.tmpdir();
+    const worktree = resolveWorktreeFn(slug) || process.cwd();
+    const result = (0, review_events_js_1.importAllLegacyArtifacts)(slug, { tmpDir, worktree, log, error });
+    const errors = result.errors || [];
+    if (!result.ok) {
+        error(fmt.status('FAIL', `Legacy import failed: ${errors.join(', ')}`));
+        exit(1);
+        return;
+    }
+    log(fmt.status('PASS', `Imported ${result.imported ? result.imported.length : 0} legacy artifacts for ${slug}`));
 }
-
 // ============================================================================
 // Main Dispatcher
 // ============================================================================
-
-/**
- * @param {string[]} args
- * @param {{inferSlugFn?: Function, log?: Function, error?: Function, exit?: Function, verifyReviewFn?: Function, submitForReviewFn?: Function, consumeArtifactsFn?: Function, pushRoundFn?: Function, readCommentsFn?: Function, commentRoundFn?: Function, submitReviewRoundFn?: Function, closeMissionPrFn?: Function, startReviewLoopFn?: Function, recordStageStatsSafeFn?: Function, startAgentFn?: Function, resolveTaskFileFn?: Function, getTaskImplementerFn?: Function, getPrStatusFn?: Function, readReviewStateFn?: Function, performStaticReviewFn?: Function, postStaticReviewCommentFn?: Function, resolveWorktreeFn?: Function, run?: Function, missionPath?: string}} [options]
- * @returns {Promise<void>}
- */
 async function review(args, options = {}) {
-  const inferSlugFn = options.inferSlugFn || inferSlug;
-  const log = options.log || fmt.log.plain;
-  const error = options.error || fmt.log.plainError;
-  const exit = options.exit || process.exit;
-  const verifyReviewFn = options.verifyReviewFn || verifyReview;
-  const submitForReviewFn = options.submitForReviewFn || submitForReview;
-  const consumeArtifactsFn = options.consumeArtifactsFn || consumeArtifacts;
-  const pushRoundFn = options.pushRoundFn || pushRound;
-  const readCommentsFn = options.readCommentsFn || readComments;
-  const commentRoundFn = options.commentRoundFn || commentRound;
-  const submitReviewRoundFn = options.submitReviewRoundFn || submitReviewRound;
-  const closeMissionPrFn = options.closeMissionPrFn || closeMissionPr;
-  const startReviewLoopFn = options.startReviewLoopFn || require('./review-loop').startReviewLoop;
-const recordStageStatsSafeFn = options.recordStageStatsSafeFn || require('./review-loop').recordStageStatsSafe;
-  const startAgentFn = options.startAgentFn || startAgent;
-  const resolveTaskFileFn = options.resolveTaskFileFn || resolveTaskFile;
-  const getTaskImplementerFn = options.getTaskImplementerFn || getTaskImplementer;
-  const getPrStatusFn = options.getPrStatusFn || getPrStatus;
-  const readReviewStateFn = options.readReviewStateFn || readReviewState;
-  const performStaticReviewFn = options.performStaticReviewFn || performStaticReview;
-  const postStaticReviewCommentFn = options.postStaticReviewCommentFn || postStaticReviewComment;
-  const resolveWorktreeFn = options.resolveWorktreeFn || resolveWorktree;
-  const runFn = options.run || run;
-
-  const flags = args.filter(a => a.startsWith('--'));
-  const params = args.filter(a => !a.startsWith('--'));
-
-  const explicitSlug = params[0];
-  const slug = inferSlugFn(explicitSlug);
-  const isSubmit      = flags.includes('--submit');
-  const isVerify      = flags.includes('--verify');
-  const isStart       = flags.includes('--start');
-  const isContinue    = flags.includes('--continue');
-  const isPush        = flags.includes('--push');
-  const isForce       = flags.includes('--force');
-  const isComment     = flags.includes('--comment') || flags.includes('--comment-file');
-  const isComments    = flags.includes('--comments');
-  const isSubmitReview = flags.includes('--submit-review');
-  const isClose       = flags.includes('--close');
-  const isStatus      = flags.includes('--status');
-  const skipGate      = flags.includes('--no-gate');
-  const isDryRun      = flags.includes('--dry-run');
-  const isReset       = flags.includes('--reset');
-  const isCreateEvent = flags.includes('--create-event');
-  const isImportLegacy = flags.includes('--import-legacy');
-  const isConsumeArtifacts = flags.includes('--consume-artifacts');
-  const missionPath = flagValue(args, '--mission');
-
-  if (!slug) {
-    error('Usage: px review [<slug>] [--verify] [--submit] [--push] [--force] [--start] [--continue] [--no-gate] [--status] [--comments] [--comment "<msg>"|--comment-file <path>] [--submit-review <outcome> [--message "<summary>"|--message-file <path>] [--close] [--create-event --type <classification> [--input-file <path>] [--actor <name>] [--round <n>] [--phase <phase>] [--mission <path>]] [--import-legacy [--tmp-dir <dir>]] [--consume-artifacts]');
-    exit(1);
-    return;
-  }
-
-  if (isStatus) {
-    showReviewStatus(slug, { ...options, readReviewStateFn });
-    return;
-  } else if (isVerify) {
-    verifyReviewFn(slug, skipGate, { ...options, missionPath });
-  } else if (isSubmit) {
-    // Pre-check: inspect the configured artifact directory for unprocessed files
-    // and emit a warning if artifacts are found but --submit-review was not used.
-    const artifactDir = require('./review-artifacts').resolveArtifactDir(resolveWorktreeFn(slug) || process.cwd());
-    const findingsPath = reviewArtifactPath(slug, 'review-findings.md', artifactDir);
-    const outcomePath = reviewArtifactPath(slug, 'review-outcome.md', artifactDir);
-    const verdictPath = reviewArtifactPath(slug, 'review-verdict.txt', artifactDir);
-    if (fs.existsSync(findingsPath) || fs.existsSync(outcomePath) || fs.existsSync(verdictPath)) {
-      log(fmt.status('WARN', `Unprocessed review artifacts found at ${artifactDir} for ${slug}. Consider using --consume-artifacts to persist them before handoff, or use --submit-review to post a verdict.`));
-    }
-    await submitForReviewFn(slug, skipGate, options);
-  } else if (isConsumeArtifacts) {
-    await consumeArtifactsFn(slug, options);
-  } else if (isPush) {
-    await pushRoundFn(slug, { ...options, force: isForce });
-  } else if (isComments) {
-    await readCommentsFn(slug, options);
-  } else if (isComment) {
-    const message = readTextFlag(args, '--comment', '--comment-file', 'comment', options);
-    if (!message) {
-      error(fmt.status('FAIL', '--comment requires text via --comment "<text>" or --comment-file <path>.'));
-      exit(1);
-      return;
-    }
-    commentRoundFn(slug, message, options);
-  } else if (isSubmitReview) {
-    const outcome = flagValue(args, '--submit-review');
-    const message = readTextFlag(args, '--message', '--message-file', 'review message', options) || '';
-    if (!outcome) {
-      error(fmt.status('FAIL', '--submit-review requires an outcome: px review <slug> --submit-review <approve|request-changes|comment> [--message "<summary>"|--message-file <path>]'));
-      exit(1);
-      return;
-    }
-    submitReviewRoundFn(slug, outcome, message, options);
-  } else if (isClose) {
-    await closeMissionPrFn(slug, options);
-  } else if (isCreateEvent) {
-    createEventHandler(slug, args, options);
-    return;
-  } else if (isImportLegacy) {
-    importLegacyHandler(slug, args, options);
-    return;
-  } else if (isStart || isContinue) {
-    const implementer = flagValue(args, '--implementer');
-    const reviewer    = flagValue(args, '--reviewer');
-    const focus       = flagValue(args, '--focus') || 'all';
-    const maxAttempts = parseInt(flagValue(args, '--max-attempts') || String(DEFAULT_MAX_ATTEMPTS), 10);
-    const verbose     = flags.includes('--verbose');
-    const pollTimeoutRaw = flagValue(args, '--poll-timeout-seconds');
-    const pollTimeoutSeconds = pollTimeoutRaw ? parseInt(pollTimeoutRaw, 10) : null;
-    await startReviewLoopFn(slug, {
-      implementer,
-      reviewer,
-      focus,
-      maxAttempts,
-      dryRun: isDryRun,
-      reset: isReset,
-      isContinue,
-      verbose,
-      pollTimeoutSeconds,
-      missionPath,
-      recordStageStatsSafeFn
-    });
-  } else {
-    const pr = getPrStatusFn(missionBranchName(slug, process.cwd()), process.cwd());
-    log(fmt.status('INFO', `Review status for mission: ${fmt.slug(slug)}`));
-    if (pr.exists) {
-      log(pr.raw);
-    } else {
-      log(fmt.status('INFO', 'No active PR found for this mission.'));
-      // Static review: when branch exists but no PR is open, inspect the diff
-      // and check checkpoint evidence. Auto-trigger review loop if findings exist.
-      const worktreeForStatic = resolveWorktreeFn(slug) || process.cwd();
-      const staticResult = performStaticReviewFn(slug, { log, error, findMissionDir: findMissionDir, findCheckpoints: findCheckpoints, readFileSync: fs.readFileSync, run: runFn, resolveWorktree: resolveWorktreeFn, rootDir: worktreeForStatic, missionPath });
-      if (staticResult.findings && staticResult.findings.length > 0) {
-        // Trivial structural findings (missing Goal Check, no evidence rows, mission
-        // dir absent) are self-fixable, so re-launch the implementer with a targeted
-        // prompt instead of burning a full reviewer round via the review loop.
-        const taskResolution = resolveTaskFileFn(slug, worktreeForStatic);
-        const implementer = taskResolution && taskResolution.taskFile
-          ? getTaskImplementerFn(taskResolution.taskFile)
-          : null;
-        if (!implementer) {
-          log(fmt.status('WARN', `Static review found ${staticResult.findings.length} finding(s) but the implementer could not be resolved for ${fmt.slug(slug)} — not re-launching the implementer and not starting the review loop.`));
-        } else {
-          log(`\nStatic review found ${staticResult.findings.length} finding(s). Re-launching implementer (${fmt.agent(implementer)}) with a targeted fix prompt...`);
-          const findingLines = staticResult.findings.map((/** @type {string} */ f) => `- ${f}`).join('\n');
-          const prompt = `Static review of your mission branch found the following issue(s). Fix them and commit, then stop:\n${findingLines}`;
-          await startAgentFn('active', { prompt, worktree: worktreeForStatic, agent: implementer, slug });
+    const inferSlugFn = options.inferSlugFn || mission_utils_js_1.inferSlug;
+    const log = options.log || fmt.log.plain;
+    const error = options.error || fmt.log.plainError;
+    const exit = options.exit || process.exit;
+    const verifyReviewFn = options.verifyReviewFn || verifyReview;
+    const submitForReviewFn = options.submitForReviewFn || submitForReview;
+    const consumeArtifactsFn = options.consumeArtifactsFn || consumeArtifacts;
+    const pushRoundFn = options.pushRoundFn || pushRound;
+    const readCommentsFn = options.readCommentsFn || readComments;
+    const commentRoundFn = options.commentRoundFn || commentRound;
+    const submitReviewRoundFn = options.submitReviewRoundFn || submitReviewRound;
+    const closeMissionPrFn = options.closeMissionPrFn || closeMissionPr;
+    const startReviewLoopFn = options.startReviewLoopFn || review_loop_js_1.startReviewLoop;
+    const recordStageStatsSafeFn = options.recordStageStatsSafeFn || review_loop_js_1.recordStageStatsSafe;
+    const startAgentFn = options.startAgentFn || agents_js_1.startAgent;
+    const resolveTaskFileFn = options.resolveTaskFileFn || backlog_js_1.resolveTaskFile;
+    const getTaskImplementerFn = options.getTaskImplementerFn || backlog_js_1.getTaskImplementer;
+    const getPrStatusFn = options.getPrStatusFn || review_adapter_js_1.getPrStatus;
+    const readReviewStateFn = options.readReviewStateFn || review_state_js_1.readReviewState;
+    const performStaticReviewFn = options.performStaticReviewFn || performStaticReview;
+    const postStaticReviewCommentFn = options.postStaticReviewCommentFn || postStaticReviewComment;
+    const resolveWorktreeFn = options.resolveWorktreeFn || mission_utils_js_1.resolveWorktree;
+    const runFn = options.run || git_js_1.run;
+    const flags = args.filter(a => a.startsWith('--'));
+    const params = args.filter(a => !a.startsWith('--'));
+    const explicitSlug = params[0];
+    const slug = inferSlugFn(explicitSlug);
+    const isSubmit = flags.includes('--submit');
+    const isVerify = flags.includes('--verify');
+    const isStart = flags.includes('--start');
+    const isContinue = flags.includes('--continue');
+    const isPush = flags.includes('--push');
+    const isForce = flags.includes('--force');
+    const isComment = flags.includes('--comment') || flags.includes('--comment-file');
+    const isComments = flags.includes('--comments');
+    const isSubmitReview = flags.includes('--submit-review');
+    const isClose = flags.includes('--close');
+    const isStatus = flags.includes('--status');
+    const skipGate = flags.includes('--no-gate');
+    const isDryRun = flags.includes('--dry-run');
+    const isReset = flags.includes('--reset');
+    const isCreateEvent = flags.includes('--create-event');
+    const isImportLegacy = flags.includes('--import-legacy');
+    const isConsumeArtifacts = flags.includes('--consume-artifacts');
+    const missionPath = flagValue(args, '--mission');
+    if (!slug) {
+        error('Usage: px review [<slug>] [--verify] [--submit] [--push] [--force] [--start] [--continue] [--no-gate] [--status] [--comments] [--comment "<msg>"|--comment-file <path>] [--submit-review <outcome> [--message "<summary>"|--message-file <path>] [--close] [--create-event --type <classification> [--input-file <path>] [--actor <name>] [--round <n>] [--phase <phase>] [--mission <path>]] [--import-legacy [--tmp-dir <dir>]] [--consume-artifacts]');
+        exit(1);
+        return;
+    }
+    if (isStatus) {
+        showReviewStatus(slug, { ...options, readReviewStateFn });
+        return;
+    }
+    else if (isVerify) {
+        verifyReviewFn(slug, skipGate, { ...options, missionPath: missionPath || undefined });
+    }
+    else if (isSubmit) {
+        // Pre-check: inspect the configured artifact directory for unprocessed files
+        // and emit a warning if artifacts are found but --submit-review was not used.
+        const artifactDir = (0, review_artifacts_js_1.resolveArtifactDir)(resolveWorktreeFn(slug) || process.cwd());
+        const findingsPath = (0, review_artifacts_js_1.reviewArtifactPath)(slug, 'review-findings.md', artifactDir);
+        const outcomePath = (0, review_artifacts_js_1.reviewArtifactPath)(slug, 'review-outcome.md', artifactDir);
+        const verdictPath = (0, review_artifacts_js_1.reviewArtifactPath)(slug, 'review-verdict.txt', artifactDir);
+        if (fs.existsSync(findingsPath) || fs.existsSync(outcomePath) || fs.existsSync(verdictPath)) {
+            log(fmt.status('WARN', `Unprocessed review artifacts found at ${artifactDir} for ${slug}. Consider using --consume-artifacts to persist them before handoff, or use --submit-review to post a verdict.`));
+        }
+        await submitForReviewFn(slug, skipGate, options);
+    }
+    else if (isConsumeArtifacts) {
+        await consumeArtifactsFn(slug, options);
+    }
+    else if (isPush) {
+        await pushRoundFn(slug, { ...options, force: isForce });
+    }
+    else if (isComments) {
+        await readCommentsFn(slug, options);
+    }
+    else if (isComment) {
+        const message = readTextFlag(args, '--comment', '--comment-file', 'comment', options);
+        if (!message) {
+            error(fmt.status('FAIL', '--comment requires text via --comment "<text>" or --comment-file <path>.'));
+            exit(1);
+            return;
         }
-      } else if (staticResult.ok) {
-        log(fmt.status('INFO', 'Static review passed — no findings. Mission branch is clean.'));
-        const prStatus = getPrStatusFn(missionBranchName(slug, process.cwd()), process.cwd());
-        if (!prStatus.exists || prStatus.state !== 'open') {
-          log(fmt.status('INFO', 'No open PR found — submitting for review before finalizing static review...'));
-          await submitForReviewFn(slug, true, options);
+        commentRoundFn(slug, message, options);
+    }
+    else if (isSubmitReview) {
+        const outcome = flagValue(args, '--submit-review');
+        const message = readTextFlag(args, '--message', '--message-file', 'review message', options) || '';
+        if (!outcome) {
+            error(fmt.status('FAIL', '--submit-review requires an outcome: px review <slug> --submit-review <approve|request-changes|comment> [--message "<summary>"|--message-file <path>]'));
+            exit(1);
+            return;
         }
-        postStaticReviewCommentFn(
-          slug,
-          formatStaticReviewSuccess(slug),
-          { ...options, rootDir: worktreeForStatic, log, error }
-        );
-      }
+        submitReviewRoundFn(slug, outcome, message, options);
     }
-    const persisted = readReviewStateFn(slug);
-    if (persisted) {
-      log(fmt.status('INFO', `Persisted reviewer state: reviewer=${fmt.agent(persisted.reviewer)} implementer=${fmt.agent(persisted.implementer)} round=${persisted.round}`));
+    else if (isClose) {
+        await closeMissionPrFn(slug, options);
+    }
+    else if (isCreateEvent) {
+        createEventHandler(slug, args, options);
+        return;
+    }
+    else if (isImportLegacy) {
+        importLegacyHandler(slug, args, options);
+        return;
+    }
+    else if (isStart || isContinue) {
+        const implementer = flagValue(args, '--implementer');
+        const reviewer = flagValue(args, '--reviewer');
+        const focus = flagValue(args, '--focus') || 'all';
+        const maxAttempts = parseInt(flagValue(args, '--max-attempts') || String(DEFAULT_MAX_ATTEMPTS), 10);
+        const verbose = flags.includes('--verbose');
+        const pollTimeoutRaw = flagValue(args, '--poll-timeout-seconds');
+        const pollTimeoutSeconds = pollTimeoutRaw ? parseInt(pollTimeoutRaw, 10) : null;
+        await startReviewLoopFn(slug, {
+            implementer: implementer || undefined,
+            reviewer: reviewer || undefined,
+            focus,
+            maxAttempts,
+            dryRun: isDryRun,
+            reset: isReset,
+            isContinue,
+            verbose,
+            pollTimeoutSeconds,
+            missionPath: missionPath || undefined,
+            recordStageStatsSafeFn
+        });
+    }
+    else {
+        const pr = getPrStatusFn((0, mission_utils_js_1.missionBranchName)(slug, process.cwd()), process.cwd());
+        log(fmt.status('INFO', `Review status for mission: ${fmt.slug(slug)}`));
+        if (pr.exists) {
+            log(pr.raw);
+        }
+        else {
+            log(fmt.status('INFO', 'No active PR found for this mission.'));
+            // Static review: when branch exists but no PR is open, inspect the diff
+            // and check checkpoint evidence. Auto-trigger review loop if findings exist.
+            const worktreeForStatic = resolveWorktreeFn(slug) || process.cwd();
+            const staticResult = performStaticReviewFn(slug, { log, findMissionDir: mission_utils_js_1.findMissionDir, findCheckpoints: mission_utils_js_1.findCheckpoints, readFileSync: fs.readFileSync, run: runFn, resolveWorktree: resolveWorktreeFn, rootDir: worktreeForStatic, missionPath: missionPath || undefined });
+            if (staticResult.findings && staticResult.findings.length > 0) {
+                // Trivial structural findings (missing Goal Check, no evidence rows, mission
+                // dir absent) are self-fixable, so re-launch the implementer with a targeted
+                // prompt instead of burning a full reviewer round via the review loop.
+                const taskResolution = resolveTaskFileFn(slug, worktreeForStatic);
+                const implementer = taskResolution && taskResolution.taskFile
+                    ? getTaskImplementerFn(taskResolution.taskFile)
+                    : null;
+                if (!implementer) {
+                    log(fmt.status('WARN', `Static review found ${staticResult.findings.length} finding(s) but the implementer could not be resolved for ${fmt.slug(slug)} — not re-launching the implementer and not starting the review loop.`));
+                }
+                else {
+                    log(`\nStatic review found ${staticResult.findings.length} finding(s). Re-launching implementer (${fmt.agent(implementer)}) with a targeted fix prompt...`);
+                    const findingLines = staticResult.findings.map((f) => `- ${f}`).join('\n');
+                    const prompt = `Static review of your mission branch found the following issue(s). Fix them and commit, then stop:\n${findingLines}`;
+                    await startAgentFn('active', { prompt, worktree: worktreeForStatic, agent: implementer, slug });
+                }
+            }
+            else if (staticResult.ok) {
+                log(fmt.status('INFO', 'Static review passed — no findings. Mission branch is clean.'));
+                const prStatus = getPrStatusFn((0, mission_utils_js_1.missionBranchName)(slug, process.cwd()), process.cwd());
+                if (!prStatus.exists || prStatus.state !== 'open') {
+                    log(fmt.status('INFO', 'No open PR found — submitting for review before finalizing static review...'));
+                    await submitForReviewFn(slug, true, options);
+                }
+                postStaticReviewCommentFn(slug, formatStaticReviewSuccess(slug), { ...options, rootDir: worktreeForStatic, log, error });
+            }
+        }
+        const persisted = readReviewStateFn(slug);
+        if (persisted) {
+            log(fmt.status('INFO', `Persisted reviewer state: reviewer=${fmt.agent(persisted.reviewer ?? '')} implementer=${fmt.agent(persisted.implementer ?? '')} round=${persisted.round}`));
+        }
     }
-  }
 }
-
-// ============================================================================
-// Module Exports
-// ============================================================================
-
-module.exports = {
-  // Internal helpers
-  flagValue,
-  readTextFlag,
-  formatStaticReviewFindings,
-  formatStaticReviewSuccess,
-  postStaticReviewComment,
-  performStaticReview,
-  // Commands
-  verifyReview,
-  submitForReview,
-  consumeArtifacts,
-  readComments,
-  pushRound,
-  showReviewStatus,
-  commentRound,
-  submitReviewRound,
-  closeMissionPr,
-  // Event handlers
-  createEventHandler,
-  importLegacyHandler,
-  // Main dispatcher
-  review
-};