fix: harden discussion codex repo checks

Author: 777genius

__tests__/unit/providers/codex-provider.test.ts

@@ -73,15 +73,16 @@ describe('CodexProvider', () => {
 
     expect(args).toEqual(
       expect.arrayContaining([
+        '--skip-git-repo-check',
         '--disable',
         'shell_tool',
         'unified_exec',
         'browser_use',
         'computer_use',
         'plugins',
-        '--skip-git-repo-check',
       ])
     );
+    expect(args.indexOf('--skip-git-repo-check')).toBe(1);
   });
 
   it('sanitizes spawned Codex environment', () => {

dist/index.js

@@ -13296,6 +13296,9 @@ var CodexProvider = class extends Provider {
       "--output-last-message",
       options.outputLastMessageFile
     ];
+    if (options.skipGitRepoCheck) {
+      args.splice(1, 0, "--skip-git-repo-check");
+    }
     if (options.disableTools) {
       args.push(
         "--disable",
@@ -13316,9 +13319,6 @@ var CodexProvider = class extends Provider {
         "plugins"
       );
     }
-    if (options.skipGitRepoCheck) {
-      args.push("--skip-git-repo-check");
-    }
     if (options.outputSchemaFile) {
       args.push("--output-schema", options.outputSchemaFile);
     }
@@ -27768,6 +27768,8 @@ function sanitizeError(error2) {
 var fs14 = __toESM(require("fs/promises"));
 var os6 = __toESM(require("os"));
 var path13 = __toESM(require("path"));
+var import_child_process7 = require("child_process");
+var import_util5 = require("util");
 var INTENTS = [
   "question",
   "disagreement",
@@ -27780,6 +27782,7 @@ var SUGGESTED_ACTIONS = [
   "suggest_rr_skip",
   "ask_for_details"
 ];
+var execFileAsync = (0, import_util5.promisify)(import_child_process7.execFile);
 var CodexDiscussionResponder = class {
   constructor(model, timeoutMs) {
     this.model = model;
@@ -27792,6 +27795,7 @@ var CodexDiscussionResponder = class {
     });
     const cwd = await fs14.mkdtemp(path13.join(os6.tmpdir(), "review-router-chat-"));
     try {
+      await initializeEmptyGitRepository(cwd);
       const content = await provider.runStructuredPrompt(
         this.buildPrompt(context),
         this.buildSchema(),
@@ -27931,6 +27935,12 @@ function redactSecrets(value) {
 function escapeAttr(value) {
   return value.replace(/&/g, "&").replace(/"/g, """);
 }
+async function initializeEmptyGitRepository(cwd) {
+  try {
+    await execFileAsync("git", ["init", "-q"], { cwd, timeout: 5e3 });
+  } catch {
+  }
+}
 
 // src/main.ts
 function syncEnvFromInputs() {

dist/index.js.map

@@ -1,6 +1,8 @@
 import * as fs from 'fs/promises';
 import * as os from 'os';
 import * as path from 'path';
+import { execFile } from 'child_process';
+import { promisify } from 'util';
 import { CodexProvider } from '../providers/codex';
 import {
   DiscussionIntent,
@@ -23,6 +25,7 @@ const SUGGESTED_ACTIONS: DiscussionSuggestedAction[] = [
   'suggest_rr_skip',
   'ask_for_details',
 ];
+const execFileAsync = promisify(execFile);
 
 export class CodexDiscussionResponder implements DiscussionResponder {
   constructor(
@@ -38,6 +41,7 @@ export class CodexDiscussionResponder implements DiscussionResponder {
     const cwd = await fs.mkdtemp(path.join(os.tmpdir(), 'review-router-chat-'));
 
     try {
+      await initializeEmptyGitRepository(cwd);
       const content = await provider.runStructuredPrompt(
         this.buildPrompt(context),
         this.buildSchema(),
@@ -212,3 +216,12 @@ function redactSecrets(value: string): string {
 function escapeAttr(value: string): string {
   return value.replace(/&/g, '&').replace(/"/g, '"');
 }
+
+async function initializeEmptyGitRepository(cwd: string): Promise<void> {
+  try {
+    await execFileAsync('git', ['init', '-q'], { cwd, timeout: 5000 });
+  } catch {
+    // Codex also receives --skip-git-repo-check. The empty repo is a CI
+    // compatibility layer, not a hard dependency for discussion replies.
+  }
+}

src/discussion/codex-responder.ts

@@ -220,6 +220,10 @@ export class CodexProvider extends Provider {
       options.outputLastMessageFile,
     ];
 
+    if (options.skipGitRepoCheck) {
+      args.splice(1, 0, '--skip-git-repo-check');
+    }
+
     if (options.disableTools) {
       args.push(
         '--disable',
@@ -241,10 +245,6 @@ export class CodexProvider extends Provider {
       );
     }
 
-    if (options.skipGitRepoCheck) {
-      args.push('--skip-git-repo-check');
-    }
-
     if (options.outputSchemaFile) {
       args.push('--output-schema', options.outputSchemaFile);
     }