reviewflow 3.12.0 → 3.14.0

package/dist/modules/worktree-management/interface-adapters/gateways/worktree.fileSystem.gateway.js

@@ -0,0 +1,67 @@
+import { existsSync, mkdirSync, readdirSync, statSync } from 'node:fs';
+import { join } from 'node:path';
+import { WORKTREE_BASE_DIR } from '../../../../shared/services/daemonPaths.js';
+import { createWorktreePath, deriveWorktreePath, parseWorktreeDirectoryName, } from '../../../../modules/worktree-management/entities/worktree/worktree.js';
+import { ensureWorktree } from '../../../../modules/worktree-management/usecases/ensureWorktree.usecase.js';
+import { removeWorktree } from '../../../../modules/worktree-management/usecases/removeWorktree.usecase.js';
+import { writeWorktreeSettings } from '../../../../modules/worktree-management/services/worktreeSettingsWriter.js';
+export class WorktreeFileSystemGateway {
+    executor;
+    baseDirectory;
+    constructor(deps) {
+        this.executor = deps.executor;
+        this.baseDirectory = deps.baseDirectory ?? WORKTREE_BASE_DIR;
+    }
+    async ensure(request) {
+        mkdirSync(this.baseDirectory, { recursive: true });
+        return ensureWorktree({
+            identity: request.identity,
+            sourceBranch: request.sourceBranch,
+            source: request.source,
+            sourceCheckoutPath: request.sourceCheckoutPath,
+        }, {
+            executor: this.executor,
+            worktreeExists: async (path) => existsSync(path),
+            writeWorktreeSettings,
+        });
+    }
+    async remove(request) {
+        try {
+            return await removeWorktree({ identity: request.identity, sourceCheckoutPath: request.sourceCheckoutPath }, {
+                executor: this.executor,
+                worktreeExists: async (path) => existsSync(path),
+            });
+        }
+        catch (error) {
+            const message = error instanceof Error ? error.message : String(error);
+            return { status: 'failed', warning: message };
+        }
+    }
+    async exists(identity) {
+        return existsSync(deriveWorktreePath(identity));
+    }
+    async list() {
+        if (!existsSync(this.baseDirectory)) {
+            return [];
+        }
+        const directories = readdirSync(this.baseDirectory, { withFileTypes: true });
+        const entries = [];
+        for (const dirent of directories) {
+            if (!dirent.isDirectory())
+                continue;
+            const identity = parseWorktreeDirectoryName(dirent.name);
+            if (identity === null)
+                continue;
+            const path = createWorktreePath(join(this.baseDirectory, dirent.name));
+            try {
+                const stats = statSync(path);
+                entries.push({ identity, path, mtime: stats.mtime });
+            }
+            catch {
+                // ignore worktrees whose stat fails (transient FS errors, race with removal)
+            }
+        }
+        return entries;
+    }
+}
+//# sourceMappingURL=worktree.fileSystem.gateway.js.map

package/dist/modules/worktree-management/interface-adapters/gateways/worktree.fileSystem.gateway.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"worktree.fileSystem.gateway.js","sourceRoot":"","sources":["../../../../../src/modules/worktree-management/interface-adapters/gateways/worktree.fileSystem.gateway.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,SAAS,EAAE,WAAW,EAAE,QAAQ,EAAE,MAAM,SAAS,CAAC;AACvE,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EAAE,iBAAiB,EAAE,MAAM,kCAAkC,CAAC;AACrE,OAAO,EACL,kBAAkB,EAClB,kBAAkB,EAClB,0BAA0B,GAC3B,MAAM,6DAA6D,CAAC;AAarE,OAAO,EAAE,cAAc,EAAE,MAAM,kEAAkE,CAAC;AAClG,OAAO,EAAE,cAAc,EAAE,MAAM,kEAAkE,CAAC;AAClG,OAAO,EAAE,qBAAqB,EAAE,MAAM,kEAAkE,CAAC;AAQzG,MAAM,OAAO,yBAAyB;IACnB,QAAQ,CAAqB;IAC7B,aAAa,CAAS;IAEvC,YAAY,IAA2C;QACrD,IAAI,CAAC,QAAQ,GAAG,IAAI,CAAC,QAAQ,CAAC;QAC9B,IAAI,CAAC,aAAa,GAAG,IAAI,CAAC,aAAa,IAAI,iBAAiB,CAAC;IAC/D,CAAC;IAED,KAAK,CAAC,MAAM,CAAC,OAA8B;QACzC,SAAS,CAAC,IAAI,CAAC,aAAa,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QACnD,OAAO,cAAc,CACnB;YACE,QAAQ,EAAE,OAAO,CAAC,QAAQ;YAC1B,YAAY,EAAE,OAAO,CAAC,YAAY;YAClC,MAAM,EAAE,OAAO,CAAC,MAAM;YACtB,kBAAkB,EAAE,OAAO,CAAC,kBAAkB;SAC/C,EACD;YACE,QAAQ,EAAE,IAAI,CAAC,QAAQ;YACvB,cAAc,EAAE,KAAK,EAAC,IAAI,EAAC,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC;YAC9C,qBAAqB;SACtB,CACF,CAAC;IACJ,CAAC;IAED,KAAK,CAAC,MAAM,CAAC,OAA8B;QACzC,IAAI,CAAC;YACH,OAAO,MAAM,cAAc,CACzB,EAAE,QAAQ,EAAE,OAAO,CAAC,QAAQ,EAAE,kBAAkB,EAAE,OAAO,CAAC,kBAAkB,EAAE,EAC9E;gBACE,QAAQ,EAAE,IAAI,CAAC,QAAQ;gBACvB,cAAc,EAAE,KAAK,EAAC,IAAI,EAAC,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC;aAC/C,CACF,CAAC;QACJ,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,MAAM,OAAO,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;YACvE,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,OAAO,EAAE,OAAO,EAAE,CAAC;QAChD,CAAC;IACH,CAAC;IAED,KAAK,CAAC,MAAM,CAAC,QAA0B;QACrC,OAAO,UAAU,CAAC,kBAAkB,CAAC,QAAQ,CAAC,CAAC,CAAC;IAClD,CAAC;IAED,KAAK,CAAC,IAAI;QACR,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,aAAa,CAAC,EAAE,CAAC;YACpC,OAAO,EAAE,CAAC;QACZ,CAAC;QACD,MAAM,WAAW,GAAG,WAAW,CAAC,IAAI,CAAC,aAAa,EAAE,EAAE,aAAa,EAAE,IAAI,EAAE,CAAC,CAAC;QAC7E,MAAM,OAAO,GAAoB,EAAE,CAAC;QACpC,KAAK,MAAM,MAAM,IAAI,WAAW,EAAE,CAAC;YACjC,IAAI,CAAC,MAAM,CAAC,WAAW,EAAE;gBAAE,SAAS;YACpC,MAAM,QAAQ,GAAG,0BAA0B,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;YACzD,IAAI,QAAQ,KAAK,IAAI;gBAAE,SAAS;YAChC,MAAM,IAAI,GAAG,kBAAkB,CAAC,IAAI,CAAC,IAAI,CAAC,aAAa,EAAE,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC;YACvE,IAAI,CAAC;gBACH,MAAM,KAAK,GAAG,QAAQ,CAAC,IAAI,CAAC,CAAC;gBAC7B,OAAO,CAAC,IAAI,CAAC,EAAE,QAAQ,EAAE,IAAI,EAAE,KAAK,EAAE,KAAK,CAAC,KAAK,EAAE,CAAC,CAAC;YACvD,CAAC;YAAC,MAAM,CAAC;gBACP,6EAA6E;YAC/E,CAAC;QACH,CAAC;QACD,OAAO,OAAO,CAAC;IACjB,CAAC;CACF"}

package/dist/modules/worktree-management/services/worktreeSettingsWriter.d.ts

@@ -0,0 +1,4 @@
+import type { WorktreePath } from '../../../modules/worktree-management/entities/worktree/worktree.schema.js';
+import type { WorktreeSettingsWriteResult } from '../../../modules/worktree-management/usecases/ensureWorktree.usecase.js';
+export declare function writeWorktreeSettings(worktreePath: WorktreePath): Promise<WorktreeSettingsWriteResult>;
+//# sourceMappingURL=worktreeSettingsWriter.d.ts.map

package/dist/modules/worktree-management/services/worktreeSettingsWriter.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"worktreeSettingsWriter.d.ts","sourceRoot":"","sources":["../../../../src/modules/worktree-management/services/worktreeSettingsWriter.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oEAAoE,CAAC;AACvG,OAAO,KAAK,EAAE,2BAA2B,EAAE,MAAM,kEAAkE,CAAC;AAIpH,wBAAsB,qBAAqB,CACzC,YAAY,EAAE,YAAY,GACzB,OAAO,CAAC,2BAA2B,CAAC,CAUtC"}

package/dist/modules/worktree-management/services/worktreeSettingsWriter.js

@@ -0,0 +1,16 @@
+import { mkdirSync, writeFileSync } from 'node:fs';
+import { join } from 'node:path';
+const settingsContent = JSON.stringify({ worktree: { bgIsolation: 'none' } }, null, 2);
+export async function writeWorktreeSettings(worktreePath) {
+    try {
+        const claudeDirectory = join(worktreePath, '.claude');
+        mkdirSync(claudeDirectory, { recursive: true });
+        writeFileSync(join(claudeDirectory, 'settings.json'), settingsContent);
+        return { status: 'ok' };
+    }
+    catch (error) {
+        const message = error instanceof Error ? error.message : String(error);
+        return { status: 'failed', reason: message };
+    }
+}
+//# sourceMappingURL=worktreeSettingsWriter.js.map

package/dist/modules/worktree-management/services/worktreeSettingsWriter.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"worktreeSettingsWriter.js","sourceRoot":"","sources":["../../../../src/modules/worktree-management/services/worktreeSettingsWriter.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,aAAa,EAAE,MAAM,SAAS,CAAC;AACnD,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AAIjC,MAAM,eAAe,GAAG,IAAI,CAAC,SAAS,CAAC,EAAE,QAAQ,EAAE,EAAE,WAAW,EAAE,MAAM,EAAE,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC;AAEvF,MAAM,CAAC,KAAK,UAAU,qBAAqB,CACzC,YAA0B;IAE1B,IAAI,CAAC;QACH,MAAM,eAAe,GAAG,IAAI,CAAC,YAAY,EAAE,SAAS,CAAC,CAAC;QACtD,SAAS,CAAC,eAAe,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAChD,aAAa,CAAC,IAAI,CAAC,eAAe,EAAE,eAAe,CAAC,EAAE,eAAe,CAAC,CAAC;QACvE,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;IAC1B,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,MAAM,OAAO,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;QACvE,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,OAAO,EAAE,CAAC;IAC/C,CAAC;AACH,CAAC"}

package/dist/modules/worktree-management/usecases/ensureWorktree.usecase.d.ts

@@ -0,0 +1,19 @@
+import type { EnsureResult, MrSource, WorktreeIdentity, WorktreePath } from '../../../modules/worktree-management/entities/worktree/worktree.schema.js';
+import type { GitCommandExecutor } from '../../../modules/worktree-management/entities/gitCommand/gitCommand.gateway.js';
+export interface EnsureWorktreeInput {
+    identity: WorktreeIdentity;
+    sourceBranch: string;
+    source: MrSource;
+    sourceCheckoutPath: string;
+}
+export interface WorktreeSettingsWriteResult {
+    status: 'ok' | 'failed';
+    reason?: string;
+}
+export interface EnsureWorktreeDependencies {
+    executor: GitCommandExecutor;
+    worktreeExists: (path: WorktreePath) => Promise<boolean>;
+    writeWorktreeSettings: (path: WorktreePath) => Promise<WorktreeSettingsWriteResult>;
+}
+export declare function ensureWorktree(input: EnsureWorktreeInput, deps: EnsureWorktreeDependencies): Promise<EnsureResult>;
+//# sourceMappingURL=ensureWorktree.usecase.d.ts.map

package/dist/modules/worktree-management/usecases/ensureWorktree.usecase.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"ensureWorktree.usecase.d.ts","sourceRoot":"","sources":["../../../../src/modules/worktree-management/usecases/ensureWorktree.usecase.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EACV,YAAY,EACZ,QAAQ,EACR,gBAAgB,EAChB,YAAY,EACb,MAAM,oEAAoE,CAAC;AAC5E,OAAO,KAAK,EACV,kBAAkB,EAEnB,MAAM,yEAAyE,CAAC;AAEjF,MAAM,WAAW,mBAAmB;IAClC,QAAQ,EAAE,gBAAgB,CAAC;IAC3B,YAAY,EAAE,MAAM,CAAC;IACrB,MAAM,EAAE,QAAQ,CAAC;IACjB,kBAAkB,EAAE,MAAM,CAAC;CAC5B;AAED,MAAM,WAAW,2BAA2B;IAC1C,MAAM,EAAE,IAAI,GAAG,QAAQ,CAAC;IACxB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,0BAA0B;IACzC,QAAQ,EAAE,kBAAkB,CAAC;IAC7B,cAAc,EAAE,CAAC,IAAI,EAAE,YAAY,KAAK,OAAO,CAAC,OAAO,CAAC,CAAC;IACzD,qBAAqB,EAAE,CAAC,IAAI,EAAE,YAAY,KAAK,OAAO,CAAC,2BAA2B,CAAC,CAAC;CACrF;AAMD,wBAAsB,cAAc,CAClC,KAAK,EAAE,mBAAmB,EAC1B,IAAI,EAAE,0BAA0B,GAC/B,OAAO,CAAC,YAAY,CAAC,CAyDvB"}

package/dist/modules/worktree-management/usecases/ensureWorktree.usecase.js

@@ -0,0 +1,53 @@
+import { deriveFetchRef, deriveWorktreePath } from '../../../modules/worktree-management/entities/worktree/worktree.js';
+function isFetchFailure(result) {
+    return result.exitCode !== 0;
+}
+export async function ensureWorktree(input, deps) {
+    const targetPath = deriveWorktreePath(input.identity);
+    const fetchRef = deriveFetchRef(input.source, input.sourceBranch, input.identity.mrNumber);
+    await deps.executor.execute({
+        kind: 'worktree-prune',
+        args: ['worktree', 'prune'],
+        cwd: input.sourceCheckoutPath,
+    });
+    const alreadyExists = await deps.worktreeExists(targetPath);
+    if (alreadyExists) {
+        const fetchInsideResult = await deps.executor.execute({
+            kind: 'fetch',
+            args: ['fetch', fetchRef.remote, fetchRef.refspec],
+            cwd: targetPath,
+        });
+        if (isFetchFailure(fetchInsideResult)) {
+            return { status: 'failed', reason: 'branch-not-found' };
+        }
+        const resetResult = await deps.executor.execute({
+            kind: 'reset-hard',
+            args: ['reset', '--hard', fetchRef.worktreeRef],
+            cwd: targetPath,
+        });
+        if (resetResult.exitCode !== 0) {
+            return { status: 'failed', reason: 'reset-failed' };
+        }
+        return { status: 'reused', path: targetPath };
+    }
+    const fetchResult = await deps.executor.execute({
+        kind: 'fetch',
+        args: ['fetch', fetchRef.remote, fetchRef.refspec],
+        cwd: input.sourceCheckoutPath,
+    });
+    if (isFetchFailure(fetchResult)) {
+        return { status: 'failed', reason: 'branch-not-found' };
+    }
+    const addResult = await deps.executor.execute({
+        kind: 'worktree-add',
+        args: ['worktree', 'add', targetPath, fetchRef.worktreeRef],
+        cwd: input.sourceCheckoutPath,
+    });
+    if (addResult.exitCode !== 0) {
+        return { status: 'failed', reason: 'worktree-add-failed' };
+    }
+    const settingsResult = await deps.writeWorktreeSettings(targetPath);
+    const settingsWarning = settingsResult.status === 'failed' ? settingsResult.reason ?? 'unknown' : null;
+    return { status: 'created', path: targetPath, settingsWarning };
+}
+//# sourceMappingURL=ensureWorktree.usecase.js.map

package/dist/modules/worktree-management/usecases/ensureWorktree.usecase.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"ensureWorktree.usecase.js","sourceRoot":"","sources":["../../../../src/modules/worktree-management/usecases/ensureWorktree.usecase.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,kBAAkB,EAAE,MAAM,6DAA6D,CAAC;AA8BjH,SAAS,cAAc,CAAC,MAAwB;IAC9C,OAAO,MAAM,CAAC,QAAQ,KAAK,CAAC,CAAC;AAC/B,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,cAAc,CAClC,KAA0B,EAC1B,IAAgC;IAEhC,MAAM,UAAU,GAAG,kBAAkB,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;IACtD,MAAM,QAAQ,GAAG,cAAc,CAAC,KAAK,CAAC,MAAM,EAAE,KAAK,CAAC,YAAY,EAAE,KAAK,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC;IAE3F,MAAM,IAAI,CAAC,QAAQ,CAAC,OAAO,CAAC;QAC1B,IAAI,EAAE,gBAAgB;QACtB,IAAI,EAAE,CAAC,UAAU,EAAE,OAAO,CAAC;QAC3B,GAAG,EAAE,KAAK,CAAC,kBAAkB;KAC9B,CAAC,CAAC;IAEH,MAAM,aAAa,GAAG,MAAM,IAAI,CAAC,cAAc,CAAC,UAAU,CAAC,CAAC;IAE5D,IAAI,aAAa,EAAE,CAAC;QAClB,MAAM,iBAAiB,GAAG,MAAM,IAAI,CAAC,QAAQ,CAAC,OAAO,CAAC;YACpD,IAAI,EAAE,OAAO;YACb,IAAI,EAAE,CAAC,OAAO,EAAE,QAAQ,CAAC,MAAM,EAAE,QAAQ,CAAC,OAAO,CAAC;YAClD,GAAG,EAAE,UAAU;SAChB,CAAC,CAAC;QACH,IAAI,cAAc,CAAC,iBAAiB,CAAC,EAAE,CAAC;YACtC,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,kBAAkB,EAAE,CAAC;QAC1D,CAAC;QAED,MAAM,WAAW,GAAG,MAAM,IAAI,CAAC,QAAQ,CAAC,OAAO,CAAC;YAC9C,IAAI,EAAE,YAAY;YAClB,IAAI,EAAE,CAAC,OAAO,EAAE,QAAQ,EAAE,QAAQ,CAAC,WAAW,CAAC;YAC/C,GAAG,EAAE,UAAU;SAChB,CAAC,CAAC;QACH,IAAI,WAAW,CAAC,QAAQ,KAAK,CAAC,EAAE,CAAC;YAC/B,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,cAAc,EAAE,CAAC;QACtD,CAAC;QAED,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,IAAI,EAAE,UAAU,EAAE,CAAC;IAChD,CAAC;IAED,MAAM,WAAW,GAAG,MAAM,IAAI,CAAC,QAAQ,CAAC,OAAO,CAAC;QAC9C,IAAI,EAAE,OAAO;QACb,IAAI,EAAE,CAAC,OAAO,EAAE,QAAQ,CAAC,MAAM,EAAE,QAAQ,CAAC,OAAO,CAAC;QAClD,GAAG,EAAE,KAAK,CAAC,kBAAkB;KAC9B,CAAC,CAAC;IACH,IAAI,cAAc,CAAC,WAAW,CAAC,EAAE,CAAC;QAChC,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,kBAAkB,EAAE,CAAC;IAC1D,CAAC;IAED,MAAM,SAAS,GAAG,MAAM,IAAI,CAAC,QAAQ,CAAC,OAAO,CAAC;QAC5C,IAAI,EAAE,cAAc;QACpB,IAAI,EAAE,CAAC,UAAU,EAAE,KAAK,EAAE,UAAU,EAAE,QAAQ,CAAC,WAAW,CAAC;QAC3D,GAAG,EAAE,KAAK,CAAC,kBAAkB;KAC9B,CAAC,CAAC;IACH,IAAI,SAAS,CAAC,QAAQ,KAAK,CAAC,EAAE,CAAC;QAC7B,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,qBAAqB,EAAE,CAAC;IAC7D,CAAC;IAED,MAAM,cAAc,GAAG,MAAM,IAAI,CAAC,qBAAqB,CAAC,UAAU,CAAC,CAAC;IACpE,MAAM,eAAe,GACnB,cAAc,CAAC,MAAM,KAAK,QAAQ,CAAC,CAAC,CAAC,cAAc,CAAC,MAAM,IAAI,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC;IAEjF,OAAO,EAAE,MAAM,EAAE,SAAS,EAAE,IAAI,EAAE,UAAU,EAAE,eAAe,EAAE,CAAC;AAClE,CAAC"}

package/dist/modules/worktree-management/usecases/removeWorktree.usecase.d.ts

@@ -0,0 +1,12 @@
+import type { RemoveResult, WorktreeIdentity, WorktreePath } from '../../../modules/worktree-management/entities/worktree/worktree.schema.js';
+import type { GitCommandExecutor } from '../../../modules/worktree-management/entities/gitCommand/gitCommand.gateway.js';
+export interface RemoveWorktreeInput {
+    identity: WorktreeIdentity;
+    sourceCheckoutPath: string;
+}
+export interface RemoveWorktreeDependencies {
+    executor: GitCommandExecutor;
+    worktreeExists: (path: WorktreePath) => Promise<boolean>;
+}
+export declare function removeWorktree(input: RemoveWorktreeInput, deps: RemoveWorktreeDependencies): Promise<RemoveResult>;
+//# sourceMappingURL=removeWorktree.usecase.d.ts.map

package/dist/modules/worktree-management/usecases/removeWorktree.usecase.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"removeWorktree.usecase.d.ts","sourceRoot":"","sources":["../../../../src/modules/worktree-management/usecases/removeWorktree.usecase.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EACV,YAAY,EACZ,gBAAgB,EAChB,YAAY,EACb,MAAM,oEAAoE,CAAC;AAC5E,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,yEAAyE,CAAC;AAElH,MAAM,WAAW,mBAAmB;IAClC,QAAQ,EAAE,gBAAgB,CAAC;IAC3B,kBAAkB,EAAE,MAAM,CAAC;CAC5B;AAED,MAAM,WAAW,0BAA0B;IACzC,QAAQ,EAAE,kBAAkB,CAAC;IAC7B,cAAc,EAAE,CAAC,IAAI,EAAE,YAAY,KAAK,OAAO,CAAC,OAAO,CAAC,CAAC;CAC1D;AAED,wBAAsB,cAAc,CAClC,KAAK,EAAE,mBAAmB,EAC1B,IAAI,EAAE,0BAA0B,GAC/B,OAAO,CAAC,YAAY,CAAC,CA4BvB"}

package/dist/modules/worktree-management/usecases/removeWorktree.usecase.js

@@ -0,0 +1,26 @@
+import { deriveWorktreePath } from '../../../modules/worktree-management/entities/worktree/worktree.js';
+export async function removeWorktree(input, deps) {
+    const targetPath = deriveWorktreePath(input.identity);
+    await deps.executor.execute({
+        kind: 'worktree-prune',
+        args: ['worktree', 'prune'],
+        cwd: input.sourceCheckoutPath,
+    });
+    const exists = await deps.worktreeExists(targetPath);
+    if (!exists) {
+        return { status: 'absent' };
+    }
+    const removeResult = await deps.executor.execute({
+        kind: 'worktree-remove',
+        args: ['worktree', 'remove', '--force', targetPath],
+        cwd: input.sourceCheckoutPath,
+    });
+    if (removeResult.exitCode !== 0) {
+        return {
+            status: 'failed',
+            warning: removeResult.stderr || 'git worktree remove failed',
+        };
+    }
+    return { status: 'removed' };
+}
+//# sourceMappingURL=removeWorktree.usecase.js.map

package/dist/modules/worktree-management/usecases/removeWorktree.usecase.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"removeWorktree.usecase.js","sourceRoot":"","sources":["../../../../src/modules/worktree-management/usecases/removeWorktree.usecase.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,kBAAkB,EAAE,MAAM,6DAA6D,CAAC;AAkBjG,MAAM,CAAC,KAAK,UAAU,cAAc,CAClC,KAA0B,EAC1B,IAAgC;IAEhC,MAAM,UAAU,GAAG,kBAAkB,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;IAEtD,MAAM,IAAI,CAAC,QAAQ,CAAC,OAAO,CAAC;QAC1B,IAAI,EAAE,gBAAgB;QACtB,IAAI,EAAE,CAAC,UAAU,EAAE,OAAO,CAAC;QAC3B,GAAG,EAAE,KAAK,CAAC,kBAAkB;KAC9B,CAAC,CAAC;IAEH,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,cAAc,CAAC,UAAU,CAAC,CAAC;IACrD,IAAI,CAAC,MAAM,EAAE,CAAC;QACZ,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,CAAC;IAC9B,CAAC;IAED,MAAM,YAAY,GAAG,MAAM,IAAI,CAAC,QAAQ,CAAC,OAAO,CAAC;QAC/C,IAAI,EAAE,iBAAiB;QACvB,IAAI,EAAE,CAAC,UAAU,EAAE,QAAQ,EAAE,SAAS,EAAE,UAAU,CAAC;QACnD,GAAG,EAAE,KAAK,CAAC,kBAAkB;KAC9B,CAAC,CAAC;IAEH,IAAI,YAAY,CAAC,QAAQ,KAAK,CAAC,EAAE,CAAC;QAChC,OAAO;YACL,MAAM,EAAE,QAAQ;YAChB,OAAO,EAAE,YAAY,CAAC,MAAM,IAAI,4BAA4B;SAC7D,CAAC;IACJ,CAAC;IAED,OAAO,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC;AAC/B,CAAC"}

package/dist/modules/worktree-management/usecases/sweepStaleWorktrees.usecase.d.ts

@@ -0,0 +1,23 @@
+import type { RemoveResult, WorktreeEntry, WorktreeIdentity } from '../../../modules/worktree-management/entities/worktree/worktree.schema.js';
+import type { TrackedMr } from '../../../modules/tracking/entities/tracking/trackedMr.js';
+export interface SweepTrackingGateway {
+    getById(projectPath: string, mrId: string): TrackedMr | null;
+}
+export interface SweepRepository {
+    localPath: string;
+    enabled: boolean;
+}
+export interface SweepStaleWorktreesDependencies {
+    listEntries: () => Promise<WorktreeEntry[]>;
+    removeWorktree: (identity: WorktreeIdentity) => Promise<RemoveResult>;
+    trackingGateway: SweepTrackingGateway;
+    getRepositories: () => SweepRepository[];
+    now: () => Date;
+}
+export interface SweepSummary {
+    inspected: number;
+    removed: number;
+    failures: number;
+}
+export declare function sweepStaleWorktrees(deps: SweepStaleWorktreesDependencies): Promise<SweepSummary>;
+//# sourceMappingURL=sweepStaleWorktrees.usecase.d.ts.map

package/dist/modules/worktree-management/usecases/sweepStaleWorktrees.usecase.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"sweepStaleWorktrees.usecase.d.ts","sourceRoot":"","sources":["../../../../src/modules/worktree-management/usecases/sweepStaleWorktrees.usecase.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,YAAY,EACZ,aAAa,EACb,gBAAgB,EACjB,MAAM,oEAAoE,CAAC;AAC5E,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,mDAAmD,CAAC;AAKnF,MAAM,WAAW,oBAAoB;IACnC,OAAO,CAAC,WAAW,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,SAAS,GAAG,IAAI,CAAC;CAC9D;AAED,MAAM,WAAW,eAAe;IAC9B,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,OAAO,CAAC;CAClB;AAED,MAAM,WAAW,+BAA+B;IAC9C,WAAW,EAAE,MAAM,OAAO,CAAC,aAAa,EAAE,CAAC,CAAC;IAC5C,cAAc,EAAE,CAAC,QAAQ,EAAE,gBAAgB,KAAK,OAAO,CAAC,YAAY,CAAC,CAAC;IACtE,eAAe,EAAE,oBAAoB,CAAC;IACtC,eAAe,EAAE,MAAM,eAAe,EAAE,CAAC;IACzC,GAAG,EAAE,MAAM,IAAI,CAAC;CACjB;AAED,MAAM,WAAW,YAAY;IAC3B,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,MAAM,CAAC;CAClB;AAiCD,wBAAsB,mBAAmB,CACvC,IAAI,EAAE,+BAA+B,GACpC,OAAO,CAAC,YAAY,CAAC,CAkCvB"}

package/dist/modules/worktree-management/usecases/sweepStaleWorktrees.usecase.js

@@ -0,0 +1,63 @@
+const ONE_DAY_MS = 24 * 60 * 60 * 1000;
+const STALE_THRESHOLD_MS = 7 * ONE_DAY_MS;
+function buildTrackedMrId(identity) {
+    return `${identity.platform}-${identity.projectPath}-${identity.mrNumber}`;
+}
+function findTrackedMr(identity, trackingGateway, repositories) {
+    const mrId = buildTrackedMrId(identity);
+    for (const repository of repositories) {
+        if (!repository.enabled)
+            continue;
+        const tracked = trackingGateway.getById(repository.localPath, mrId);
+        if (tracked)
+            return tracked;
+    }
+    return null;
+}
+function isClosedOrMergedOverThreshold(tracked, now) {
+    if (tracked.state !== 'merged' && tracked.state !== 'closed')
+        return false;
+    const mergedAt = tracked.mergedAt;
+    if (mergedAt === null)
+        return true;
+    const mergedTime = Date.parse(mergedAt);
+    if (!Number.isFinite(mergedTime))
+        return true;
+    return now.getTime() - mergedTime > ONE_DAY_MS;
+}
+function isStale(entry, now) {
+    return now.getTime() - entry.mtime.getTime() > STALE_THRESHOLD_MS;
+}
+export async function sweepStaleWorktrees(deps) {
+    const entries = await deps.listEntries();
+    const repositories = deps.getRepositories();
+    const now = deps.now();
+    let removedCount = 0;
+    let failures = 0;
+    for (const entry of entries) {
+        const tracked = findTrackedMr(entry.identity, deps.trackingGateway, repositories);
+        const shouldRemove = tracked === null
+            || isClosedOrMergedOverThreshold(tracked, now)
+            || isStale(entry, now);
+        if (!shouldRemove)
+            continue;
+        try {
+            const result = await deps.removeWorktree(entry.identity);
+            if (result.status === 'removed') {
+                removedCount += 1;
+            }
+            else if (result.status === 'failed') {
+                failures += 1;
+            }
+        }
+        catch {
+            failures += 1;
+        }
+    }
+    return {
+        inspected: entries.length,
+        removed: removedCount,
+        failures,
+    };
+}
+//# sourceMappingURL=sweepStaleWorktrees.usecase.js.map

package/dist/modules/worktree-management/usecases/sweepStaleWorktrees.usecase.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"sweepStaleWorktrees.usecase.js","sourceRoot":"","sources":["../../../../src/modules/worktree-management/usecases/sweepStaleWorktrees.usecase.ts"],"names":[],"mappings":"AAOA,MAAM,UAAU,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,CAAC;AACvC,MAAM,kBAAkB,GAAG,CAAC,GAAG,UAAU,CAAC;AAyB1C,SAAS,gBAAgB,CAAC,QAA0B;IAClD,OAAO,GAAG,QAAQ,CAAC,QAAQ,IAAI,QAAQ,CAAC,WAAW,IAAI,QAAQ,CAAC,QAAQ,EAAE,CAAC;AAC7E,CAAC;AAED,SAAS,aAAa,CACpB,QAA0B,EAC1B,eAAqC,EACrC,YAA+B;IAE/B,MAAM,IAAI,GAAG,gBAAgB,CAAC,QAAQ,CAAC,CAAC;IACxC,KAAK,MAAM,UAAU,IAAI,YAAY,EAAE,CAAC;QACtC,IAAI,CAAC,UAAU,CAAC,OAAO;YAAE,SAAS;QAClC,MAAM,OAAO,GAAG,eAAe,CAAC,OAAO,CAAC,UAAU,CAAC,SAAS,EAAE,IAAI,CAAC,CAAC;QACpE,IAAI,OAAO;YAAE,OAAO,OAAO,CAAC;IAC9B,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAED,SAAS,6BAA6B,CAAC,OAAkB,EAAE,GAAS;IAClE,IAAI,OAAO,CAAC,KAAK,KAAK,QAAQ,IAAI,OAAO,CAAC,KAAK,KAAK,QAAQ;QAAE,OAAO,KAAK,CAAC;IAC3E,MAAM,QAAQ,GAAG,OAAO,CAAC,QAAQ,CAAC;IAClC,IAAI,QAAQ,KAAK,IAAI;QAAE,OAAO,IAAI,CAAC;IACnC,MAAM,UAAU,GAAG,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;IACxC,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,UAAU,CAAC;QAAE,OAAO,IAAI,CAAC;IAC9C,OAAO,GAAG,CAAC,OAAO,EAAE,GAAG,UAAU,GAAG,UAAU,CAAC;AACjD,CAAC;AAED,SAAS,OAAO,CAAC,KAAoB,EAAE,GAAS;IAC9C,OAAO,GAAG,CAAC,OAAO,EAAE,GAAG,KAAK,CAAC,KAAK,CAAC,OAAO,EAAE,GAAG,kBAAkB,CAAC;AACpE,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,mBAAmB,CACvC,IAAqC;IAErC,MAAM,OAAO,GAAG,MAAM,IAAI,CAAC,WAAW,EAAE,CAAC;IACzC,MAAM,YAAY,GAAG,IAAI,CAAC,eAAe,EAAE,CAAC;IAC5C,MAAM,GAAG,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;IAEvB,IAAI,YAAY,GAAG,CAAC,CAAC;IACrB,IAAI,QAAQ,GAAG,CAAC,CAAC;IAEjB,KAAK,MAAM,KAAK,IAAI,OAAO,EAAE,CAAC;QAC5B,MAAM,OAAO,GAAG,aAAa,CAAC,KAAK,CAAC,QAAQ,EAAE,IAAI,CAAC,eAAe,EAAE,YAAY,CAAC,CAAC;QAClF,MAAM,YAAY,GAChB,OAAO,KAAK,IAAI;eACb,6BAA6B,CAAC,OAAO,EAAE,GAAG,CAAC;eAC3C,OAAO,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC;QAEzB,IAAI,CAAC,YAAY;YAAE,SAAS;QAE5B,IAAI,CAAC;YACH,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,cAAc,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;YACzD,IAAI,MAAM,CAAC,MAAM,KAAK,SAAS,EAAE,CAAC;gBAChC,YAAY,IAAI,CAAC,CAAC;YACpB,CAAC;iBAAM,IAAI,MAAM,CAAC,MAAM,KAAK,QAAQ,EAAE,CAAC;gBACtC,QAAQ,IAAI,CAAC,CAAC;YAChB,CAAC;QACH,CAAC;QAAC,MAAM,CAAC;YACP,QAAQ,IAAI,CAAC,CAAC;QAChB,CAAC;IACH,CAAC;IAED,OAAO;QACL,SAAS,EAAE,OAAO,CAAC,MAAM;QACzB,OAAO,EAAE,YAAY;QACrB,QAAQ;KACT,CAAC;AACJ,CAAC"}

package/dist/shared/services/daemonPaths.d.ts

@@ -1,4 +1,5 @@
 export declare const PID_FILE_PATH: string;
 export declare const LOG_FILE_PATH: string;
 export declare const LOG_DIR: string;
+export declare const WORKTREE_BASE_DIR: string;
 //# sourceMappingURL=daemonPaths.d.ts.map

package/dist/shared/services/daemonPaths.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"daemonPaths.d.ts","sourceRoot":"","sources":["../../../src/shared/services/daemonPaths.ts"],"names":[],"mappings":"AAKA,eAAO,MAAM,aAAa,QAAqC,CAAC;AAChE,eAAO,MAAM,aAAa,QAA6C,CAAC;AACxE,eAAO,MAAM,OAAO,QAA2B,CAAC"}
+{"version":3,"file":"daemonPaths.d.ts","sourceRoot":"","sources":["../../../src/shared/services/daemonPaths.ts"],"names":[],"mappings":"AAKA,eAAO,MAAM,aAAa,QAAqC,CAAC;AAChE,eAAO,MAAM,aAAa,QAA6C,CAAC;AACxE,eAAO,MAAM,OAAO,QAA2B,CAAC;AAEhD,eAAO,MAAM,iBAAiB,QAA8C,CAAC"}

package/dist/shared/services/daemonPaths.js

@@ -4,4 +4,5 @@ const CONFIG_DIR = join(homedir(), '.config', 'reviewflow');
 export const PID_FILE_PATH = join(CONFIG_DIR, 'reviewflow.pid');
 export const LOG_FILE_PATH = join(CONFIG_DIR, 'logs', 'reviewflow.log');
 export const LOG_DIR = join(CONFIG_DIR, 'logs');
+export const WORKTREE_BASE_DIR = join(homedir(), '.reviewflow', 'worktrees');
 //# sourceMappingURL=daemonPaths.js.map

package/dist/shared/services/daemonPaths.js.map

@@ -1 +1 @@
-{"version":3,"file":"daemonPaths.js","sourceRoot":"","sources":["../../../src/shared/services/daemonPaths.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EAAE,OAAO,EAAE,MAAM,SAAS,CAAC;AAElC,MAAM,UAAU,GAAG,IAAI,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,YAAY,CAAC,CAAC;AAE5D,MAAM,CAAC,MAAM,aAAa,GAAG,IAAI,CAAC,UAAU,EAAE,gBAAgB,CAAC,CAAC;AAChE,MAAM,CAAC,MAAM,aAAa,GAAG,IAAI,CAAC,UAAU,EAAE,MAAM,EAAE,gBAAgB,CAAC,CAAC;AACxE,MAAM,CAAC,MAAM,OAAO,GAAG,IAAI,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC"}
+{"version":3,"file":"daemonPaths.js","sourceRoot":"","sources":["../../../src/shared/services/daemonPaths.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EAAE,OAAO,EAAE,MAAM,SAAS,CAAC;AAElC,MAAM,UAAU,GAAG,IAAI,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,YAAY,CAAC,CAAC;AAE5D,MAAM,CAAC,MAAM,aAAa,GAAG,IAAI,CAAC,UAAU,EAAE,gBAAgB,CAAC,CAAC;AAChE,MAAM,CAAC,MAAM,aAAa,GAAG,IAAI,CAAC,UAAU,EAAE,MAAM,EAAE,gBAAgB,CAAC,CAAC;AACxE,MAAM,CAAC,MAAM,OAAO,GAAG,IAAI,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC;AAEhD,MAAM,CAAC,MAAM,iBAAiB,GAAG,IAAI,CAAC,OAAO,EAAE,EAAE,aAAa,EAAE,WAAW,CAAC,CAAC"}

package/dist/tests/acceptance/169-migrate-claude-invocation-to-bg-mode.acceptance.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=169-migrate-claude-invocation-to-bg-mode.acceptance.test.d.ts.map

package/dist/tests/acceptance/169-migrate-claude-invocation-to-bg-mode.acceptance.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"169-migrate-claude-invocation-to-bg-mode.acceptance.test.d.ts","sourceRoot":"","sources":["../../../src/tests/acceptance/169-migrate-claude-invocation-to-bg-mode.acceptance.test.ts"],"names":[],"mappings":""}

package/dist/tests/acceptance/169-migrate-claude-invocation-to-bg-mode.acceptance.test.js

@@ -0,0 +1,238 @@
+import { describe, it, expect, beforeEach, vi } from 'vitest';
+import { runClaudeReviewJob } from '../../modules/claude-invocation/usecases/runClaudeReviewJob.usecase.js';
+import { StubClaudeSessionGateway } from '../../tests/stubs/claudeSession.stub.js';
+import { StubMcpCompletionBridge } from '../../tests/stubs/mcpCompletion.stub.js';
+import { StubReviewReportGateway } from '../../tests/stubs/reviewReport.stub.js';
+import { StubBillingStateGateway } from '../../tests/stubs/billingState.stub.js';
+import { StubSupervisorHealthGateway } from '../../tests/stubs/supervisorHealth.stub.js';
+import { StubEnvironmentGateway } from '../../tests/stubs/environment.stub.js';
+import { auditBilling } from '../../modules/claude-invocation/usecases/auditBilling.usecase.js';
+import { checkSupervisorHealth } from '../../modules/claude-invocation/usecases/checkSupervisorHealth.usecase.js';
+import { parseSessionId } from '../../modules/claude-invocation/entities/claudeSession/claudeSession.schema.js';
+function createContext() {
+    const sessionGateway = new StubClaudeSessionGateway();
+    const completionBridge = new StubMcpCompletionBridge();
+    const reportGateway = new StubReviewReportGateway();
+    const billingState = new StubBillingStateGateway();
+    const supervisorHealth = new StubSupervisorHealthGateway();
+    const environment = new StubEnvironmentGateway();
+    const deps = {
+        sessionGateway,
+        completionBridge,
+        reportGateway,
+        billingState,
+        environment,
+        now: () => new Date('2026-05-22T10:00:00Z'),
+        timeoutMs: 15 * 60 * 1000,
+        pollIntervalMs: 30_000,
+    };
+    return {
+        sessionGateway,
+        completionBridge,
+        reportGateway,
+        billingState,
+        supervisorHealth,
+        environment,
+        deps,
+    };
+}
+const baseInput = {
+    jobId: 'gitlab:owner/repo:42',
+    jobType: 'review',
+    prompt: '/review-front 42',
+    flags: {
+        model: 'claude-opus-4-7',
+        mcpConfigJson: '{}',
+        systemPrompt: 'system',
+        allowedTools: 'Read,Bash',
+        disallowedTools: 'EnterPlanMode',
+        permissionMode: 'auto',
+    },
+    localPath: '/tmp/project',
+    mergeRequestId: 'gitlab-owner/repo-42',
+    mergeRequestNumber: 42,
+    attempt: 0,
+};
+describe('SPEC-169: Migrate Claude invocation to --bg mode (acceptance)', () => {
+    beforeEach(() => {
+        vi.useFakeTimers();
+        vi.setSystemTime(new Date('2026-05-22T10:00:00Z'));
+    });
+    describe('Scenario 1: Webhook triggers review dispatched via --bg', () => {
+        it('spawns claude --bg with the configured flags and captures the session ID', async () => {
+            const context = createContext();
+            context.sessionGateway.setDispatchResult({ status: 'dispatched', sessionId: parseSessionId('7c5dcf5d') });
+            context.completionBridge.scheduleCompletion(baseInput.jobId, {
+                source: 'mcp',
+                outcome: 'completed',
+                reason: null,
+            });
+            context.reportGateway.setReport({
+                content: '# Review report',
+                path: '/tmp/project/.claude/reviews/2026-05-22-MR-42-review.md',
+            });
+            const runPromise = runClaudeReviewJob(baseInput, context.deps);
+            await vi.runAllTimersAsync();
+            const result = await runPromise;
+            expect(result.status).toBe('completed');
+            const calls = context.sessionGateway.dispatchCalls;
+            expect(calls).toHaveLength(1);
+            expect(calls[0]?.flags.permissionMode).toBe('auto');
+            expect(JSON.stringify(calls[0])).not.toContain('-p ');
+            expect(JSON.stringify(calls[0])).not.toContain('--print');
+        });
+    });
+    describe('Scenario 2: Completion via MCP primary signal', () => {
+        it('reads the report from the conventional path, then stops and removes the session', async () => {
+            const context = createContext();
+            context.sessionGateway.setDispatchResult({ status: 'dispatched', sessionId: parseSessionId('abc12345') });
+            context.completionBridge.scheduleCompletion(baseInput.jobId, {
+                source: 'mcp',
+                outcome: 'completed',
+                reason: null,
+            });
+            context.reportGateway.setReport({
+                content: '# Review content',
+                path: '/tmp/project/.claude/reviews/2026-05-22-MR-42-review.md',
+            });
+            const runPromise = runClaudeReviewJob(baseInput, context.deps);
+            await vi.runAllTimersAsync();
+            const result = await runPromise;
+            expect(result.status).toBe('completed');
+            if (result.status === 'completed') {
+                expect(result.content).toBe('# Review content');
+            }
+            expect(context.sessionGateway.stopCalls).toContain('abc12345');
+            expect(context.sessionGateway.removeCalls).toContain('abc12345');
+        });
+    });
+    describe('Scenario 3: Completion via polling fallback', () => {
+        it('detects completion through claude agents --json when MCP stays silent', async () => {
+            const context = createContext();
+            context.sessionGateway.setDispatchResult({ status: 'dispatched', sessionId: parseSessionId('poll0001') });
+            context.sessionGateway.scheduleAgentCompletion('poll0001', 'completed', 1);
+            context.reportGateway.setReport({
+                content: '# Polled report',
+                path: '/tmp/project/.claude/reviews/2026-05-22-MR-42-review.md',
+            });
+            const runPromise = runClaudeReviewJob(baseInput, context.deps);
+            await vi.runAllTimersAsync();
+            const result = await runPromise;
+            expect(result.status).toBe('completed');
+            expect(context.sessionGateway.stopCalls).toContain('poll0001');
+            expect(context.sessionGateway.removeCalls).toContain('poll0001');
+        });
+    });
+    describe('Scenario 4: Hard timeout reached without completion signal', () => {
+        it('marks the job failed with reason "timeout" after 15 minutes', async () => {
+            const context = createContext();
+            context.sessionGateway.setDispatchResult({ status: 'dispatched', sessionId: parseSessionId('time0001') });
+            let nowMs = new Date('2026-05-22T10:00:00Z').getTime();
+            const deps = { ...context.deps, now: () => new Date(nowMs) };
+            const runPromise = runClaudeReviewJob(baseInput, deps);
+            await Promise.resolve();
+            await Promise.resolve();
+            await Promise.resolve();
+            nowMs += 16 * 60 * 1000;
+            await vi.advanceTimersByTimeAsync(16 * 60 * 1000);
+            const result = await runPromise;
+            expect(result.status).toBe('failed');
+            if (result.status === 'failed') {
+                expect(result.reason).toBe('timeout');
+            }
+            expect(context.sessionGateway.stopCalls).toContain('time0001');
+            expect(context.sessionGateway.removeCalls).toContain('time0001');
+        });
+    });
+    describe('Scenario 5: Report file missing after completion', () => {
+        it('fails the job with reason "report-missing" and still runs cleanup', async () => {
+            const context = createContext();
+            context.sessionGateway.setDispatchResult({ status: 'dispatched', sessionId: parseSessionId('rep00001') });
+            context.completionBridge.scheduleCompletion(baseInput.jobId, {
+                source: 'mcp',
+                outcome: 'completed',
+                reason: null,
+            });
+            context.reportGateway.setReport(null);
+            const runPromise = runClaudeReviewJob(baseInput, context.deps);
+            await vi.runAllTimersAsync();
+            const result = await runPromise;
+            expect(result.status).toBe('failed');
+            if (result.status === 'failed') {
+                expect(result.reason).toBe('report-missing');
+            }
+            expect(context.sessionGateway.stopCalls).toContain('rep00001');
+        });
+    });
+    describe('Scenario 6: Rate limit error on dispatch', () => {
+        it('returns a retry signal with exponential backoff', async () => {
+            const context = createContext();
+            context.sessionGateway.setDispatchResult({ status: 'rate-limited', rawStderr: '429 Too Many Requests' });
+            const result = await runClaudeReviewJob(baseInput, context.deps);
+            expect(result.status).toBe('retry');
+            if (result.status === 'retry') {
+                expect(result.delayMs).toBe(60_000);
+                expect(result.attempt).toBe(1);
+            }
+        });
+    });
+    describe('Scenario 7: Supervisor down detection', () => {
+        it('records supervisor down status when daemon is unreachable', async () => {
+            const context = createContext();
+            context.sessionGateway.setDaemonStatus({ reachable: false, reason: 'connection refused' });
+            const health = await checkSupervisorHealth({
+                sessionGateway: context.sessionGateway,
+                supervisorHealthGateway: context.supervisorHealth,
+                now: () => new Date('2026-05-22T10:00:00Z'),
+            });
+            expect(health.status).toBe('down');
+            expect(context.supervisorHealth.read().status).toBe('down');
+        });
+    });
+    describe('Scenario 8: Billing regression detected pre-dispatch', () => {
+        it('aborts dispatch when ANTHROPIC_API_KEY is present in the environment', async () => {
+            const context = createContext();
+            context.environment.setHasAnthropicApiKey(true);
+            const result = await runClaudeReviewJob(baseInput, context.deps);
+            expect(result.status).toBe('failed');
+            if (result.status === 'failed') {
+                expect(result.reason).toBe('billing-regression-prevented');
+            }
+            expect(context.sessionGateway.dispatchCalls).toHaveLength(0);
+        });
+    });
+    describe('Scenario 9: Periodic billing audit detects API consumption', () => {
+        it('pauses the dispatch queue when usage indicates API-pool consumption', async () => {
+            const context = createContext();
+            context.sessionGateway.setUsage({ usesApiPool: true, raw: 'API tokens used: 12345' });
+            const result = await auditBilling({
+                sessionGateway: context.sessionGateway,
+                billingStateGateway: context.billingState,
+                now: () => new Date('2026-05-22T10:00:00Z'),
+            });
+            expect(result.regression).toBe(true);
+            expect(context.billingState.read().dispatchPaused).toBe(true);
+        });
+    });
+    describe('Scenario 10: Followup job uses same --bg dispatch path', () => {
+        it('runs the followup through the same use case with jobType="followup"', async () => {
+            const context = createContext();
+            context.sessionGateway.setDispatchResult({ status: 'dispatched', sessionId: parseSessionId('foll0001') });
+            context.completionBridge.scheduleCompletion(baseInput.jobId, {
+                source: 'mcp',
+                outcome: 'completed',
+                reason: null,
+            });
+            context.reportGateway.setReport({
+                content: '# Followup report',
+                path: '/tmp/project/.claude/reviews/2026-05-22-MR-42-followup.md',
+            });
+            const runPromise = runClaudeReviewJob({ ...baseInput, jobType: 'followup' }, context.deps);
+            await vi.runAllTimersAsync();
+            const result = await runPromise;
+            expect(result.status).toBe('completed');
+            expect(context.reportGateway.lastReadJobType).toBe('followup');
+        });
+    });
+});
+//# sourceMappingURL=169-migrate-claude-invocation-to-bg-mode.acceptance.test.js.map