@magnusekdahl/parallix 1.2.0 → 1.3.0

package/lib/review/rebase.js

@@ -1,3 +1,4 @@
+"use strict";
 /**
  * Rebase helpers for pre-review workflow.
  *
@@ -6,15 +7,50 @@
  *
  * Both handoff.js and review-loop.js import from this module instead.
  */
-
-const path = require('path');
-const fmt = require('../core/fmt');
-const { git } = require('../core/git');
-const { resolveWorktree } = require('../core/mission-utils');
-const { isProviderEnabled } = require('./review-adapter');
-
+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.commitSafeMissionArtifacts = commitSafeMissionArtifacts;
+exports.rebaseBeforeReviewRound = rebaseBeforeReviewRound;
+const path = __importStar(require("path"));
+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 review_adapter_js_1 = require("./review-adapter.js");
+const child_process_1 = require("child_process");
+const _spawnSync = child_process_1.spawnSync;
 const SHARED_FILE_REBASE_CONFLICT_RE = /shared file(?:\(s\))? require agent-assisted resolution|shared-file-conflicts/i;
-
 /**
  * Auto-commit safe mission artifacts before rebase.
  * Only touches mission-generated files, task files, and stats CSV.
@@ -23,149 +59,110 @@ const SHARED_FILE_REBASE_CONFLICT_RE = /shared file(?:\(s\))? require agent-assi
  * @param {{taskFile?: string|null, gitFn?: Function, log?: Function, error?: Function, isMissionArtifactFn?: Function, isWorkflowGeneratedArtifactFn?: Function, resolveStatsRelPathFn?: Function}} [options]
  * @returns {Promise<{ok: boolean, dirty: boolean, unsafe?: boolean}>}
  */
-async function commitSafeMissionArtifacts(slug, worktree, {
-  taskFile = null,
-  gitFn = git,
-  log = fmt.log.plain,
-  error = fmt.log.plainError,
-  isMissionArtifactFn = require('../core/mission-utils').isMissionArtifact,
-  isWorkflowGeneratedArtifactFn = require('../core/mission-utils').isWorkflowGeneratedArtifact,
-  resolveStatsRelPathFn = () => null
-} = {}) {
-  const rootDir = worktree || process.cwd();
-  const statusResult = gitFn(['-C', rootDir, 'status', '--porcelain=v1', '-z']);
-  if (statusResult.status !== 0 || !statusResult.stdout) {
-    return { ok: true, dirty: false };
-  }
-
-  const parsePorcelainZ = (/** @type {string|Buffer} */ stdout) => {
-    const entries = String(stdout).split('\0').filter(Boolean);
-    const dirty = [];
-    for (let i = 0; i < entries.length; i += 1) {
-      const entry = entries[i];
-      const xy = entry.slice(0, 2);
-      const file = entry.slice(3);
-      /** @type {{xy: string, file: string, paths: string[], source?: string}} */
-      const record = { xy, file, paths: [file] };
-      if ((xy[0] === 'R' || xy[0] === 'C') && i + 1 < entries.length) {
-        const source = entries[i + 1];
-        record.source = source;
-        record.paths.push(source);
-        i += 1;
-      }
-      dirty.push(record);
+async function commitSafeMissionArtifacts(slug, worktree, { taskFile = null, gitFn = git_js_1.git, log = fmt.log.plain, error = fmt.log.plainError, isMissionArtifactFn = mission_utils_js_1.isMissionArtifact, isWorkflowGeneratedArtifactFn = mission_utils_js_1.isWorkflowGeneratedArtifact, resolveStatsRelPathFn = () => null } = {}) {
+    const rootDir = worktree || process.cwd();
+    const statusResult = gitFn(['-C', rootDir, 'status', '--porcelain=v1', '-z']);
+    if (statusResult.status !== 0 || !statusResult.stdout) {
+        return { ok: true, dirty: false };
+    }
+    const parsePorcelainZ = (stdout) => {
+        const entries = String(stdout).split('\0').filter(Boolean);
+        const dirty = [];
+        for (let i = 0; i < entries.length; i += 1) {
+            const entry = entries[i];
+            const xy = entry.slice(0, 2);
+            const file = entry.slice(3);
+            const record = { xy, file, paths: [file] };
+            if ((xy[0] === 'R' || xy[0] === 'C') && i + 1 < entries.length) {
+                const source = entries[i + 1];
+                record.source = source;
+                record.paths.push(source);
+                i += 1;
+            }
+            dirty.push(record);
+        }
+        return dirty;
+    };
+    const dirtyFiles = parsePorcelainZ(statusResult.stdout);
+    const unmerged = dirtyFiles.filter(f => ['DD', 'AU', 'UD', 'UA', 'DU', 'AA', 'UU'].includes(f.xy));
+    if (unmerged.length > 0) {
+        error(fmt.status('FAIL', `Cannot auto-commit: unmerged/conflicting files detected:\n${unmerged.map((f) => `       - ${f.file}`).join('\n')}`));
+        return { ok: false, dirty: true, unsafe: true };
+    }
+    const resolvedTaskFile = taskFile ? path.relative(rootDir, taskFile) : null;
+    const statsRelPath = resolveStatsRelPathFn(rootDir);
+    const isSafeToCommit = (file) => isWorkflowGeneratedArtifactFn(file)
+        || isMissionArtifactFn(file, slug, rootDir)
+        || !!(resolvedTaskFile && file === resolvedTaskFile)
+        || !!(statsRelPath && file === statsRelPath);
+    const unsafeFiles = dirtyFiles
+        .flatMap((f) => f.paths)
+        .filter((file) => !isSafeToCommit(file));
+    if (unsafeFiles.length > 0) {
+        error(fmt.status('FAIL', `Cannot auto-commit: dirty files include non-mission paths:`));
+        unsafeFiles.forEach((file) => error(`       - ${file}`));
+        return { ok: false, dirty: true, unsafe: true };
+    }
+    log(`Auto-committing safe mission artifacts for ${fmt.branch(`mission/${slug}`)} before rebase...`);
+    dirtyFiles.forEach((f) => {
+        log(`       - ${f.file}`);
+        gitFn(['-C', rootDir, 'add', '--', f.file]);
+    });
+    const commitRes = gitFn(['-C', rootDir, 'commit', '-m', `workflow(${slug}): auto-commit mission artifacts before pre-review rebase`]);
+    if (commitRes.status === 0) {
+        log(fmt.status('PASS', 'Mission artifacts committed.'));
+        return { ok: true, dirty: true };
+    }
+    else {
+        const failureMsg = [commitRes.stderr, commitRes.stdout].filter(Boolean).join('\n').trim();
+        error(fmt.status('FAIL', `Failed to commit mission artifacts: ${failureMsg}`));
+        return { ok: false, dirty: true };
     }
-    return dirty;
-  };
-
-  const dirtyFiles = parsePorcelainZ(statusResult.stdout);
-
-  const unmerged = dirtyFiles.filter(f => ['DD', 'AU', 'UD', 'UA', 'DU', 'AA', 'UU'].includes(f.xy));
-  if (unmerged.length > 0) {
-    error(fmt.status('FAIL', `Cannot auto-commit: unmerged/conflicting files detected:\n${unmerged.map(f => `       - ${f.file}`).join('\n')}`));
-    return { ok: false, dirty: true, unsafe: true };
-  }
-
-  const resolvedTaskFile = taskFile ? path.relative(rootDir, taskFile) : null;
-  const statsRelPath = resolveStatsRelPathFn(rootDir);
-  const isSafeToCommit = (/** @type {string} */ file) =>
-    isWorkflowGeneratedArtifactFn(file)
-    || isMissionArtifactFn(file, slug, rootDir)
-    || (resolvedTaskFile && file === resolvedTaskFile)
-    || (statsRelPath && file === statsRelPath);
-
-  const unsafeFiles = dirtyFiles
-    .flatMap(f => f.paths)
-    .filter((/** @type {string} */ file) => !isSafeToCommit(file));
-  if (unsafeFiles.length > 0) {
-    error(fmt.status('FAIL', `Cannot auto-commit: dirty files include non-mission paths:`));
-    unsafeFiles.forEach(file => error(`       - ${file}`));
-    return { ok: false, dirty: true, unsafe: true };
-  }
-
-  log(`Auto-committing safe mission artifacts for ${fmt.branch(`mission/${slug}`)} before rebase...`);
-  dirtyFiles.forEach(f => {
-    log(`       - ${f.file}`);
-    gitFn(['-C', rootDir, 'add', '--', f.file]);
-  });
-
-  const commitRes = gitFn(['-C', rootDir, 'commit', '-m', `workflow(${slug}): auto-commit mission artifacts before pre-review rebase`]);
-  if (commitRes.status === 0) {
-    log(fmt.status('PASS', 'Mission artifacts committed.'));
-    return { ok: true, dirty: true };
-  } else {
-    const failureMsg = [commitRes.stderr, commitRes.stdout].filter(Boolean).join('\n').trim();
-    error(fmt.status('FAIL', `Failed to commit mission artifacts: ${failureMsg}`));
-    return { ok: false, dirty: true };
-  }
 }
-
+/** @typedef {{ok: boolean, sharedFileConflicts: boolean}} RebaseResult */
 /**
  * Rebase the mission branch onto the latest primary/main branch before launching a reviewer.
- *
- * Always commits safe worktree state first, then rebases via the workflow CLI.
- * In standalone (provider-disabled) mode, skips the rebase after committing.
  * @param {string} slug
  * @param {{runFn?: Function, gitFn?: Function, taskFile?: string|null, worktree?: string, log?: Function, error?: Function, isReviewProviderEnabledFn?: Function|null, legacyIsForgejoReviewEnabledFn?: Function|null, isForgejoReviewEnabledFn?: Function|null}} [options]
  * @returns {Promise<{ok: boolean, sharedFileConflicts: boolean}>}
  */
-async function rebaseBeforeReviewRound(slug, {
-  runFn = require('child_process').spawnSync,
-  gitFn = git,
-  taskFile = null,
-  worktree = resolveWorktree(slug) || process.cwd(),
-  log = fmt.log.plain,
-  error = fmt.log.plainError,
-  isReviewProviderEnabledFn = undefined,
-  legacyIsForgejoReviewEnabledFn = null,
-  isForgejoReviewEnabledFn = null
-} = {}) {
-  const cleanup = await commitSafeMissionArtifacts(slug, worktree, { taskFile, gitFn, log, error });
-  if (!cleanup.ok) {
-    error(fmt.status('WARN', 'Worktree is dirty with unsafe or conflicted files. Rebase may fail.'));
-    return { ok: false, sharedFileConflicts: false };
-  }
-
-  const forgejoEnabledFn = isReviewProviderEnabledFn
-    || legacyIsForgejoReviewEnabledFn
-    || isForgejoReviewEnabledFn
-    || isProviderEnabled;
-  const forgejoEnabled = forgejoEnabledFn(worktree);
-  if (!forgejoEnabled) {
-    log(fmt.status('INFO', `Review provider disabled; committed worktree state and skipping pre-review rebase for ${fmt.branch(`mission/${slug}`)}.`));
-    return { ok: true, sharedFileConflicts: false };
-  }
-
-  const workflowCli = path.resolve(__dirname, '..', 'index.js');
-
-  log(`Rebasing ${fmt.branch(`mission/${slug}`)} onto the latest primary branch before reviewer launch...`);
-
-  const result = runFn(process.execPath, [workflowCli, 'rebase', slug, '--push'], {
-    cwd: worktree,
-    encoding: 'utf8',
-    stdio: ['ignore', 'pipe', 'pipe']
-  });
-  const output = [result.stdout, result.stderr].filter(Boolean).join('\n').trim();
-
-  if (result.status === 0) {
-    log(fmt.status('PASS', `Pre-review rebase completed for ${fmt.branch(`mission/${slug}`)}.`));
-    return { ok: true, sharedFileConflicts: false };
-  }
-
-  if (output) {
-    error(output);
-  }
-  const sharedFileConflicts = SHARED_FILE_REBASE_CONFLICT_RE.test(output);
-  if (sharedFileConflicts) {
-    error(fmt.status('FAIL', 'Shared-file rebase conflicts detected. Autonomous review loop cannot continue safely.'));
-    log(fmt.status('INFO', `Resolve the conflicts in the worktree, then re-run: px review ${slug} --start`));
-  } else {
-    error(fmt.status('FAIL', `Rebase failed before launching reviewer for ${fmt.branch(`mission/${slug}`)}.`));
-  }
-  return { ok: false, sharedFileConflicts };
+async function rebaseBeforeReviewRound(slug, { runFn = _spawnSync, gitFn = git_js_1.git, taskFile = null, worktree = (0, mission_utils_js_1.resolveWorktree)(slug) || process.cwd(), log = fmt.log.plain, error = fmt.log.plainError, isReviewProviderEnabledFn = undefined, legacyIsForgejoReviewEnabledFn = null, isForgejoReviewEnabledFn = null } = {}) {
+    const cleanup = await commitSafeMissionArtifacts(slug, worktree, { taskFile, gitFn, log, error });
+    if (!cleanup.ok) {
+        error(fmt.status('WARN', 'Worktree is dirty with unsafe or conflicted files. Rebase may fail.'));
+        return { ok: false, sharedFileConflicts: false };
+    }
+    const forgejoEnabledFn = isReviewProviderEnabledFn
+        || legacyIsForgejoReviewEnabledFn
+        || isForgejoReviewEnabledFn
+        || review_adapter_js_1.isProviderEnabled;
+    const forgejoEnabled = forgejoEnabledFn(worktree);
+    if (!forgejoEnabled) {
+        log(fmt.status('INFO', `Review provider disabled; committed worktree state and skipping pre-review rebase for ${fmt.branch(`mission/${slug}`)}.`));
+        return { ok: true, sharedFileConflicts: false };
+    }
+    const workflowCli = path.resolve(__dirname, '..', 'index.js');
+    log(`Rebasing ${fmt.branch(`mission/${slug}`)} onto the latest primary branch before reviewer launch...`);
+    const result = runFn(process.execPath, [workflowCli, 'rebase', slug, '--push'], {
+        cwd: worktree,
+        encoding: 'utf8',
+        stdio: ['ignore', 'pipe', 'pipe']
+    });
+    const output = [result.stdout, result.stderr].filter(Boolean).join('\n').trim();
+    if (result.status === 0) {
+        log(fmt.status('PASS', `Pre-review rebase completed for ${fmt.branch(`mission/${slug}`)}.`));
+        return { ok: true, sharedFileConflicts: false };
+    }
+    if (output) {
+        error(output);
+    }
+    const sharedFileConflicts = SHARED_FILE_REBASE_CONFLICT_RE.test(output);
+    if (sharedFileConflicts) {
+        error(fmt.status('FAIL', 'Shared-file rebase conflicts detected. Autonomous review loop cannot continue safely.'));
+        log(fmt.status('INFO', `Resolve the conflicts in the worktree, then re-run: px review ${slug} --start`));
+    }
+    else {
+        error(fmt.status('FAIL', `Rebase failed before launching reviewer for ${fmt.branch(`mission/${slug}`)}.`));
+    }
+    return { ok: false, sharedFileConflicts };
 }
-
-module.exports = {
-  commitSafeMissionArtifacts,
-  rebaseBeforeReviewRound,
-};

package/lib/review/rebase.ts

@@ -0,0 +1,188 @@
+/**
+ * Rebase helpers for pre-review workflow.
+ *
+ * Extracted from review-loop.js to break the circular dependency:
+ *   handoff.js -> review-loop.js -> handoff.js
+ *
+ * Both handoff.js and review-loop.js import from this module instead.
+ */
+
+import * as path from 'path';
+import * as fmt from '../core/fmt.js';
+import { git } from '../core/git.js';
+import { resolveWorktree, isMissionArtifact, isWorkflowGeneratedArtifact } from '../core/mission-utils.js';
+import { isProviderEnabled } from './review-adapter.js';
+import { spawnSync } from 'child_process';
+
+type RunFn = (cmd: string, args: string[], opts: { cwd?: string; encoding?: string; stdio?: unknown[] }) => { status: number | null; stdout?: string; stderr?: string };
+
+const _spawnSync = spawnSync as unknown as RunFn;
+
+const SHARED_FILE_REBASE_CONFLICT_RE = /shared file(?:\(s\))? require agent-assisted resolution|shared-file-conflicts/i;
+
+/**
+ * Auto-commit safe mission artifacts before rebase.
+ * Only touches mission-generated files, task files, and stats CSV.
+ * @param {string} slug
+ * @param {string} worktree
+ * @param {{taskFile?: string|null, gitFn?: Function, log?: Function, error?: Function, isMissionArtifactFn?: Function, isWorkflowGeneratedArtifactFn?: Function, resolveStatsRelPathFn?: Function}} [options]
+ * @returns {Promise<{ok: boolean, dirty: boolean, unsafe?: boolean}>}
+ */
+export async function commitSafeMissionArtifacts(slug: string, worktree: string, {
+  taskFile = null,
+  gitFn = git,
+  log = fmt.log.plain,
+  error = fmt.log.plainError,
+  isMissionArtifactFn = isMissionArtifact,
+  isWorkflowGeneratedArtifactFn = isWorkflowGeneratedArtifact,
+  resolveStatsRelPathFn = () => null
+}: {
+  taskFile?: string | null;
+  gitFn?: typeof git;
+  log?: (msg: string) => void;
+  error?: (msg: string) => void;
+  isMissionArtifactFn?: (file: string, slug: string, rootDir: string) => boolean;
+  isWorkflowGeneratedArtifactFn?: (file: string) => boolean;
+  resolveStatsRelPathFn?: (rootDir: string) => string | null;
+} = {}): Promise<{ ok: boolean; dirty: boolean; unsafe?: boolean }> {
+  const rootDir = worktree || process.cwd();
+  const statusResult = gitFn(['-C', rootDir, 'status', '--porcelain=v1', '-z']);
+  if (statusResult.status !== 0 || !statusResult.stdout) {
+    return { ok: true, dirty: false };
+  }
+
+  const parsePorcelainZ = (stdout: string | Buffer) => {
+    const entries = String(stdout).split('\0').filter(Boolean);
+    const dirty: { xy: string; file: string; paths: string[]; source?: string }[] = [];
+    for (let i = 0; i < entries.length; i += 1) {
+      const entry = entries[i];
+      const xy = entry.slice(0, 2);
+      const file = entry.slice(3);
+      const record: { xy: string; file: string; paths: string[]; source?: string } = { xy, file, paths: [file] };
+      if ((xy[0] === 'R' || xy[0] === 'C') && i + 1 < entries.length) {
+        const source = entries[i + 1];
+        record.source = source;
+        record.paths.push(source);
+        i += 1;
+      }
+      dirty.push(record);
+    }
+    return dirty;
+  };
+
+  const dirtyFiles = parsePorcelainZ(statusResult.stdout);
+
+  const unmerged = dirtyFiles.filter(f => ['DD', 'AU', 'UD', 'UA', 'DU', 'AA', 'UU'].includes(f.xy));
+  if (unmerged.length > 0) {
+    error(fmt.status('FAIL', `Cannot auto-commit: unmerged/conflicting files detected:\n${unmerged.map((f: { file: string }) => `       - ${f.file}`).join('\n')}`));
+    return { ok: false, dirty: true, unsafe: true };
+  }
+
+  const resolvedTaskFile = taskFile ? path.relative(rootDir, taskFile) : null;
+  const statsRelPath = resolveStatsRelPathFn(rootDir);
+  const isSafeToCommit = (file: string) =>
+    isWorkflowGeneratedArtifactFn(file)
+    || isMissionArtifactFn(file, slug, rootDir)
+    || !!(resolvedTaskFile && file === resolvedTaskFile)
+    || !!(statsRelPath && file === statsRelPath);
+
+  const unsafeFiles = dirtyFiles
+    .flatMap((f: { paths: string[] }) => f.paths)
+    .filter((file: string) => !isSafeToCommit(file));
+  if (unsafeFiles.length > 0) {
+    error(fmt.status('FAIL', `Cannot auto-commit: dirty files include non-mission paths:`));
+    unsafeFiles.forEach((file: string) => error(`       - ${file}`));
+    return { ok: false, dirty: true, unsafe: true };
+  }
+
+  log(`Auto-committing safe mission artifacts for ${fmt.branch(`mission/${slug}`)} before rebase...`);
+  dirtyFiles.forEach((f: { file: string }) => {
+    log(`       - ${f.file}`);
+    gitFn(['-C', rootDir, 'add', '--', f.file]);
+  });
+
+  const commitRes = gitFn(['-C', rootDir, 'commit', '-m', `workflow(${slug}): auto-commit mission artifacts before pre-review rebase`]);
+  if (commitRes.status === 0) {
+    log(fmt.status('PASS', 'Mission artifacts committed.'));
+    return { ok: true, dirty: true };
+  } else {
+    const failureMsg = [commitRes.stderr, commitRes.stdout].filter(Boolean).join('\n').trim();
+    error(fmt.status('FAIL', `Failed to commit mission artifacts: ${failureMsg}`));
+    return { ok: false, dirty: true };
+  }
+}
+
+/** @typedef {{ok: boolean, sharedFileConflicts: boolean}} RebaseResult */
+
+/**
+ * Rebase the mission branch onto the latest primary/main branch before launching a reviewer.
+ * @param {string} slug
+ * @param {{runFn?: Function, gitFn?: Function, taskFile?: string|null, worktree?: string, log?: Function, error?: Function, isReviewProviderEnabledFn?: Function|null, legacyIsForgejoReviewEnabledFn?: Function|null, isForgejoReviewEnabledFn?: Function|null}} [options]
+ * @returns {Promise<{ok: boolean, sharedFileConflicts: boolean}>}
+ */
+export async function rebaseBeforeReviewRound(slug: string, {
+  runFn = _spawnSync,
+  gitFn = git,
+  taskFile = null,
+  worktree = resolveWorktree(slug) || process.cwd(),
+  log = fmt.log.plain,
+  error = fmt.log.plainError,
+  isReviewProviderEnabledFn = undefined,
+  legacyIsForgejoReviewEnabledFn = null,
+  isForgejoReviewEnabledFn = null
+}: {
+  runFn?: (cmd: string, args: string[], opts: { cwd?: string; encoding?: string; stdio?: unknown[] }) => { status: number | null; stdout?: string; stderr?: string };
+  gitFn?: typeof git;
+  taskFile?: string | null;
+  worktree?: string;
+  log?: (msg: string) => void;
+  error?: (msg: string) => void;
+  isReviewProviderEnabledFn?: ((wt: string) => boolean) | undefined | null;
+  legacyIsForgejoReviewEnabledFn?: ((wt: string) => boolean) | null;
+  isForgejoReviewEnabledFn?: ((wt: string) => boolean) | null;
+} = {}): Promise<{ ok: boolean; sharedFileConflicts: boolean }> {
+  const cleanup = await commitSafeMissionArtifacts(slug, worktree, { taskFile, gitFn, log, error });
+  if (!cleanup.ok) {
+    error(fmt.status('WARN', 'Worktree is dirty with unsafe or conflicted files. Rebase may fail.'));
+    return { ok: false, sharedFileConflicts: false };
+  }
+
+  const forgejoEnabledFn = isReviewProviderEnabledFn
+    || legacyIsForgejoReviewEnabledFn
+    || isForgejoReviewEnabledFn
+    || isProviderEnabled;
+  const forgejoEnabled = forgejoEnabledFn(worktree);
+  if (!forgejoEnabled) {
+    log(fmt.status('INFO', `Review provider disabled; committed worktree state and skipping pre-review rebase for ${fmt.branch(`mission/${slug}`)}.`));
+    return { ok: true, sharedFileConflicts: false };
+  }
+
+  const workflowCli = path.resolve(__dirname, '..', 'index.js');
+
+  log(`Rebasing ${fmt.branch(`mission/${slug}`)} onto the latest primary branch before reviewer launch...`);
+
+  const result = runFn(process.execPath, [workflowCli, 'rebase', slug, '--push'], {
+    cwd: worktree,
+    encoding: 'utf8',
+    stdio: ['ignore', 'pipe', 'pipe']
+  });
+  const output = [result.stdout, result.stderr].filter(Boolean).join('\n').trim();
+
+  if (result.status === 0) {
+    log(fmt.status('PASS', `Pre-review rebase completed for ${fmt.branch(`mission/${slug}`)}.`));
+    return { ok: true, sharedFileConflicts: false };
+  }
+
+  if (output) {
+    error(output);
+  }
+  const sharedFileConflicts = SHARED_FILE_REBASE_CONFLICT_RE.test(output);
+  if (sharedFileConflicts) {
+    error(fmt.status('FAIL', 'Shared-file rebase conflicts detected. Autonomous review loop cannot continue safely.'));
+    log(fmt.status('INFO', `Resolve the conflicts in the worktree, then re-run: px review ${slug} --start`));
+  } else {
+    error(fmt.status('FAIL', `Rebase failed before launching reviewer for ${fmt.branch(`mission/${slug}`)}.`));
+  }
+  return { ok: false, sharedFileConflicts };
+}
+