@parall/codex-agent 1.30.0 → 1.32.0

package/src/workspace.ts

@@ -1,6 +1,6 @@
-import * as fs from "node:fs";
-import * as path from "node:path";
-import { parse as parseToml } from "smol-toml";
+import * as fs from 'node:fs';
+import * as path from 'node:path';
+import { parse as parseToml } from 'smol-toml';
 import {
   BRIDGE_WORKSPACE_INSTRUCTIONS,
   PRLL_BEHAVIOR,
@@ -8,8 +8,8 @@ import {
   buildIdentity,
   buildSkillReferences,
   writeSkillFiles,
-} from "@parall/agent-core";
-import type { AgentIdentity } from "@parall/agent-core";
+} from '@parall/agent-core';
+import type { AgentIdentity } from '@parall/agent-core';
 
 /**
  * Ensure the workspace directory is marked as trusted in the global Codex
@@ -26,15 +26,15 @@ export function ensureWorkspaceTrusted(
   workspaceDir: string,
   log?: { warn: (msg: string) => void },
 ): void {
-  const configPath = path.join(codexHome, "config.toml");
+  const configPath = path.join(codexHome, 'config.toml');
   const normalizedPath = path.resolve(workspaceDir);
 
   try {
-    let content = "";
+    let content = '';
     try {
-      content = fs.readFileSync(configPath, "utf8");
+      content = fs.readFileSync(configPath, 'utf8');
     } catch (err) {
-      if ((err as NodeJS.ErrnoException).code !== "ENOENT") throw err;
+      if ((err as NodeJS.ErrnoException).code !== 'ENOENT') throw err;
     }
 
     // Structured read — know exactly what state we're in.
@@ -44,16 +44,18 @@ export function ensureWorkspaceTrusted(
         parsed = parseToml(content) as Record<string, unknown>;
       } catch {
         // File is already broken TOML — don't make it worse.
-        log?.warn(`Codex config.toml is not valid TOML; skipping trust write for ${normalizedPath}`);
+        log?.warn(
+          `Codex config.toml is not valid TOML; skipping trust write for ${normalizedPath}`,
+        );
         return;
       }
     }
 
     const projects = parsed?.projects as Record<string, Record<string, unknown>> | undefined;
-    if (projects?.[normalizedPath]?.trust_level === "trusted") return;
+    if (projects?.[normalizedPath]?.trust_level === 'trusted') return;
 
     // TOML basic-string keys require backslash and double-quote escaping.
-    const escapedPath = normalizedPath.replace(/\\/g, "\\\\").replace(/"/g, '\\"');
+    const escapedPath = normalizedPath.replace(/\\/g, '\\\\').replace(/"/g, '\\"');
     const sectionHeader = `[projects."${escapedPath}"]`;
     const trustLine = 'trust_level = "trusted"';
 
@@ -63,7 +65,7 @@ export function ensureWorkspaceTrusted(
       // within the section (up to the next `[` header or EOF) for an
       // existing trust_level key — it may not be the first line after
       // the header if the user added comments or other keys.
-      const headerLineEnd = content.indexOf("\n", headerIdx);
+      const headerLineEnd = content.indexOf('\n', headerIdx);
       if (headerLineEnd === -1) {
         content = `${content}\n${trustLine}\n`;
       } else {
@@ -76,24 +78,29 @@ export function ensureWorkspaceTrusted(
           const matchEnd = matchStart + trustMatch[0].length;
           content = content.substring(0, matchStart) + trustLine + content.substring(matchEnd);
         } else {
-          content = content.substring(0, afterHeader) + trustLine + "\n" + content.substring(afterHeader);
+          content =
+            content.substring(0, afterHeader) + trustLine + '\n' + content.substring(afterHeader);
         }
       }
-      fs.writeFileSync(configPath, content, "utf8");
+      fs.writeFileSync(configPath, content, 'utf8');
     } else if (projects?.[normalizedPath] !== undefined) {
       // smol-toml found the section but indexOf missed it — the header
       // uses non-canonical TOML formatting. Appending would create a
       // duplicate table. Skip rather than corrupt the file.
-      log?.warn(`Codex config.toml has non-canonical header for ${normalizedPath}; skipping trust write`);
-    } else if (projects && !content.includes("[projects.")) {
+      log?.warn(
+        `Codex config.toml has non-canonical header for ${normalizedPath}; skipping trust write`,
+      );
+    } else if (projects && !content.includes('[projects.')) {
       // `projects` exists in parsed output but no `[projects.` table
       // headers in the raw text — it's an inline table. Appending a
       // standard table header would produce invalid TOML.
-      log?.warn(`Codex config.toml uses inline table for projects; skipping trust write for ${normalizedPath}`);
+      log?.warn(
+        `Codex config.toml uses inline table for projects; skipping trust write for ${normalizedPath}`,
+      );
     } else {
       // Section doesn't exist — append.
       fs.mkdirSync(codexHome, { recursive: true });
-      fs.appendFileSync(configPath, `\n${sectionHeader}\n${trustLine}\n`, "utf8");
+      fs.appendFileSync(configPath, `\n${sectionHeader}\n${trustLine}\n`, 'utf8');
     }
   } catch (err) {
     log?.warn(`failed to write Codex project trust for ${normalizedPath}: ${String(err)}`);
@@ -106,8 +113,8 @@ export function ensureWorkspaceTrusted(
  * whether OPENAI_BASE_URL points to the Parall API.
  */
 export function isParallProxyMode(env: NodeJS.ProcessEnv = process.env): boolean {
-  const baseUrl = env.OPENAI_BASE_URL?.trim()?.replace(/\/+$/, "");
-  const apiUrl = env.PRLL_API_URL?.trim()?.replace(/\/+$/, "");
+  const baseUrl = env.OPENAI_BASE_URL?.trim()?.replace(/\/+$/, '');
+  const apiUrl = env.PRLL_API_URL?.trim()?.replace(/\/+$/, '');
   if (!baseUrl || !apiUrl) return false;
   return baseUrl === apiUrl || baseUrl.startsWith(`${apiUrl}/`);
 }
@@ -131,47 +138,47 @@ export function ensureParallProvider(
   apiUrl: string,
   log?: { warn: (msg: string) => void },
 ): void {
-  const configPath = path.join(codexHome, "config.toml");
-  const baseUrl = apiUrl.replace(/\/$/, "") + "/api/llm/v1";
+  const configPath = path.join(codexHome, 'config.toml');
+  const baseUrl = apiUrl.replace(/\/$/, '') + '/api/llm/v1';
 
   try {
-    let content = "";
+    let content = '';
     try {
-      content = fs.readFileSync(configPath, "utf8");
+      content = fs.readFileSync(configPath, 'utf8');
     } catch (err) {
-      if ((err as NodeJS.ErrnoException).code !== "ENOENT") throw err;
+      if ((err as NodeJS.ErrnoException).code !== 'ENOENT') throw err;
     }
 
-    const sectionHeader = "[model_providers.parall]";
-    const authHeader = "[model_providers.parall.auth]";
+    const sectionHeader = '[model_providers.parall]';
+    const authHeader = '[model_providers.parall.auth]';
     const providerBlock = [
       sectionHeader,
       'name = "Parall Proxy"',
       `base_url = "${baseUrl}"`,
       'wire_api = "responses"',
-      "supports_websockets = false",
-      "requires_openai_auth = false",
-      "",
+      'supports_websockets = false',
+      'requires_openai_auth = false',
+      '',
       authHeader,
       'command = "printenv"',
       'args = ["OPENAI_API_KEY"]',
-    ].join("\n");
+    ].join('\n');
 
     const headerIdx = content.indexOf(sectionHeader);
     if (headerIdx !== -1) {
       let blockEnd = findSectionEnd(content, headerIdx + sectionHeader.length + 1);
       let authIdx = content.indexOf(authHeader, blockEnd);
-      while (authIdx !== -1 && content.substring(blockEnd, authIdx).trim() === "") {
+      while (authIdx !== -1 && content.substring(blockEnd, authIdx).trim() === '') {
         blockEnd = findSectionEnd(content, authIdx + authHeader.length + 1);
         authIdx = content.indexOf(authHeader, blockEnd);
       }
       content = content.substring(0, headerIdx) + providerBlock + content.substring(blockEnd);
     } else {
-      content = content.trimEnd() + "\n\n" + providerBlock + "\n";
+      content = content.trimEnd() + '\n\n' + providerBlock + '\n';
     }
 
     fs.mkdirSync(codexHome, { recursive: true });
-    fs.writeFileSync(configPath, content, "utf8");
+    fs.writeFileSync(configPath, content, 'utf8');
   } catch (err) {
     throw new Error(`failed to write Parall provider config: ${String(err)}`);
   }
@@ -180,7 +187,7 @@ export function ensureParallProvider(
 function findSectionEnd(content: string, fromIndex: number): number {
   // A TOML section ends at the next `[` that starts a new table header.
   // Match `[` at the beginning of a line (not inside a value).
-  const nextHeader = content.indexOf("\n[", fromIndex);
+  const nextHeader = content.indexOf('\n[', fromIndex);
   return nextHeader === -1 ? content.length : nextHeader;
 }
 
@@ -195,18 +202,18 @@ export function ensureCodexWorkspace(
     PRLL_BEHAVIOR,
     PRLL_REFERENCE_GUIDE,
     buildSkillReferences(workspaceDir),
-  ].join("\n\n");
+  ].join('\n\n');
 
   fs.mkdirSync(workspaceDir, { recursive: true });
 
-  const parallDir = path.join(workspaceDir, ".parall");
+  const parallDir = path.join(workspaceDir, '.parall');
   fs.mkdirSync(parallDir, { recursive: true });
-  fs.writeFileSync(path.join(parallDir, "system-prompt.md"), systemPrompt, "utf8");
+  fs.writeFileSync(path.join(parallDir, 'system-prompt.md'), systemPrompt, 'utf8');
 
-  writeSkillFiles(path.join(parallDir, "skills"));
+  writeSkillFiles(path.join(parallDir, 'skills'));
 
-  const codexConfigDir = path.join(workspaceDir, ".codex");
+  const codexConfigDir = path.join(workspaceDir, '.codex');
   fs.mkdirSync(codexConfigDir, { recursive: true });
-  const toml = `developer_instructions = """\n${systemPrompt.replace(/\\/g, "\\\\").replace(/"""/g, '\\"""')}\n"""\n`;
-  fs.writeFileSync(path.join(codexConfigDir, "config.toml"), toml, "utf8");
+  const toml = `developer_instructions = """\n${systemPrompt.replace(/\\/g, '\\\\').replace(/"""/g, '\\"""')}\n"""\n`;
+  fs.writeFileSync(path.join(codexConfigDir, 'config.toml'), toml, 'utf8');
 }