@engineeros/connector 0.4.1 → 0.4.4

package/README.md

@@ -36,3 +36,14 @@ Credentials are stored per workspace under `~/.engineeros/connectors` with owner
 Keep the connector online to receive Goals assigned from EngineerOS. Each Goal runs in an isolated worktree below `~/.engineeros/runs`. Cancellation stops Codex. The connector returns changed paths, a bounded diff, and the exact repository ZIP; a human still performs independent attestation.
 
 Requirements: Node.js 22 or newer, Git, and an authenticated Codex CLI (`codex login`).
+
+## Codex CLI compatibility
+
+The connector prints the exact Codex CLI version it will use before connecting. If the
+configured model requires a newer CLI, update Codex and restart the connector:
+
+```sh
+npm install -g @openai/codex@latest
+codex --version
+npx @engineeros/connector start --workspace .
+```

package/bin/engineeros-connector.mjs

@@ -10,9 +10,11 @@ import {
   workspaceUrl,
 } from "../src/config.mjs";
 import {
+  assessmentProgressMessage,
   executeAssignment,
   executeConnectedPrompt,
   executeWorkspaceAssessment,
+  inspectCodexCli,
   stopProcess,
   workspaceSnapshot,
 } from "../src/runner.mjs";
@@ -73,6 +75,17 @@ if (command === "pair") {
   fail("Use `engineeros-connector pair`, `start`, or `status`.");
 }
 
+let codexCli;
+try {
+  codexCli = await inspectCodexCli(config.workspace);
+} catch (error) {
+  fail(error instanceof Error ? error.message : String(error));
+}
+console.log(`Using ${codexCli.version}.`);
+if (firstMessage.type === "pair") {
+  firstMessage.capabilities.codex_cli_version = codexCli.version;
+}
+
 let stopped = false;
 let active = null;
 const available = [];
@@ -375,9 +388,8 @@ async function executeAssessment(assignment) {
         if (active?.runId === assessmentId) active.child = child;
       },
       onEvent: (event) => {
-        const message =
-          event.message || event.item?.text || event.type || "Codex is assessing the workspace";
-        reportProgress(message);
+        const message = assessmentProgressMessage(event);
+        if (message) reportProgress(message);
       },
     });
     const response = await fetch(

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@engineeros/connector",
-  "version": "0.4.1",
+  "version": "0.4.4",
   "description": "Connect a local Codex CLI workspace to EngineerOS over an outbound WebSocket.",
   "private": false,
   "type": "module",

package/src/runner.mjs

@@ -1,6 +1,13 @@
 import { createHash } from "node:crypto";
 import { spawn } from "node:child_process";
-import { lstat, mkdir, readFile, readdir, stat, writeFile } from "node:fs/promises";
+import {
+  lstat,
+  mkdir,
+  readFile,
+  readdir,
+  stat,
+  writeFile,
+} from "node:fs/promises";
 import os from "node:os";
 import path from "node:path";
 import { promisify } from "node:util";
@@ -114,14 +121,30 @@ export async function executeAssignment(assignment, config, callbacks) {
   };
 }
 
-export async function executeWorkspaceAssessment(assignment, config, callbacks) {
+export async function executeWorkspaceAssessment(
+  assignment,
+  config,
+  callbacks,
+) {
   const execution = connectorExecution(assignment);
-  const revision = await run(
+  const startingRevision = await run(
     "git",
     ["rev-parse", "HEAD"],
     config.workspace,
     { allowFailure: true },
   );
+  const startingHead =
+    startingRevision.code === 0
+      ? startingRevision.stdout.trim().slice(0, 128)
+      : null;
+  if (
+    assignment.target_head_revision &&
+    startingHead !== assignment.target_head_revision
+  ) {
+    throw new Error(
+      "This workspace is no longer at the commit inventoried by EngineerOS. Refresh the workspace inventory before assessing it.",
+    );
+  }
   const controller = launchCodexProcess(
     config.workspace,
     execution.prompt,
@@ -132,10 +155,24 @@ export async function executeWorkspaceAssessment(assignment, config, callbacks)
   const completed = await controller.completed;
   const report = completed.finalMessage.trim();
   if (!report) throw new Error("Codex completed without returning a response.");
+  const endingRevision = await run(
+    "git",
+    ["rev-parse", "HEAD"],
+    config.workspace,
+    { allowFailure: true },
+  );
+  const endingHead =
+    endingRevision.code === 0
+      ? endingRevision.stdout.trim().slice(0, 128)
+      : null;
+  if (startingHead !== endingHead) {
+    throw new Error(
+      "The Git commit changed during assessment. Refresh the workspace inventory and assess the new commit.",
+    );
+  }
   return {
     report_markdown: report,
-    observed_head_revision:
-      revision.code === 0 ? revision.stdout.trim().slice(0, 128) : null,
+    observed_head_revision: endingHead,
   };
 }
 
@@ -156,6 +193,70 @@ export async function executeConnectedPrompt(assignment, config, callbacks) {
   return { content, model: "codex-cli" };
 }
 
+export async function inspectCodexCli(workspace = process.cwd()) {
+  const command =
+    process.env.CODEX_BIN ||
+    (process.platform === "win32" ? "codex.cmd" : "codex");
+  const result = await runCodexCommand(command, ["--version"], workspace);
+  if (result.code !== 0) {
+    throw new Error(
+      "Codex CLI is unavailable. Install it with `npm install -g @openai/codex@latest`, run `codex login`, then restart this connector.",
+    );
+  }
+  const version = result.stdout.trim().slice(0, 100);
+  if (!version) {
+    throw new Error(
+      "Codex CLI returned no version. Reinstall @openai/codex, then restart this connector.",
+    );
+  }
+  return { command, version };
+}
+
+export function codexFailureMessage(output, code) {
+  if (/requires a newer version of Codex/i.test(output)) {
+    return (
+      "The configured model requires a newer Codex CLI. " +
+      "Run `npm install -g @openai/codex@latest`, verify with `codex --version`, " +
+      "then restart the EngineerOS connector and retry the assessment."
+    );
+  }
+  if (/not logged in|login required|authentication required/i.test(output)) {
+    return "Codex CLI is not authenticated. Run `codex login`, then restart the EngineerOS connector.";
+  }
+  return `Codex exited with code ${code}. ${output.slice(-1_000)}`;
+}
+
+export function assessmentProgressMessage(event) {
+  if (!event || typeof event !== "object") return null;
+  if (event.type === "turn.started")
+    return "Codex started analyzing repository evidence";
+  if (event.type === "item.started") {
+    if (event.item?.type === "command_execution") {
+      const command = compactCommand(event.item.command);
+      return command
+        ? `Inspecting with ${command}`
+        : "Inspecting repository files";
+    }
+    if (event.item?.type === "mcp_tool_call")
+      return "Consulting a connected analysis tool";
+    if (event.item?.type === "web_search")
+      return "Checking an external technical reference";
+    return null;
+  }
+  if (event.type === "item.completed" && event.item?.type === "agent_message") {
+    return "Assessment report prepared";
+  }
+  return null;
+}
+
+function compactCommand(command) {
+  const value = Array.isArray(command)
+    ? command.join(" ")
+    : String(command || "");
+  const compact = value.replace(/\s+/g, " ").trim();
+  return compact.length > 96 ? `${compact.slice(0, 93)}...` : compact;
+}
+
 export function connectorExecution(assignment) {
   const prompt = assignment?.prompt_markdown;
   if (typeof prompt !== "string" || !prompt.trim()) {
@@ -292,15 +393,7 @@ function launchCodexProcess(workspace, prompt, sandbox, callbacks) {
   const command =
     process.env.CODEX_BIN ||
     (process.platform === "win32" ? "codex.cmd" : "codex");
-  const args = [
-    "exec",
-    "--json",
-    "--sandbox",
-    sandbox,
-    "-C",
-    workspace,
-    "-",
-  ];
+  const args = ["exec", "--json", "--sandbox", sandbox, "-C", workspace, "-"];
   const child = spawn(command, args, {
     cwd: workspace,
     env: process.env,
@@ -349,15 +442,31 @@ function launchCodexProcess(workspace, prompt, sandbox, callbacks) {
     child.once("error", reject);
     child.once("close", (code) => {
       if (code === 0) resolve({ output: output.slice(-20_000), finalMessage });
-      else
-        reject(
-          new Error(`Codex exited with code ${code}. ${output.slice(-1_000)}`),
-        );
+      else reject(new Error(codexFailureMessage(output, code)));
     });
   });
   return { child, completed };
 }
 
+function runCodexCommand(command, args, cwd) {
+  return new Promise((resolve, reject) => {
+    const child = spawn(command, args, {
+      cwd,
+      env: process.env,
+      shell: process.platform === "win32",
+      windowsHide: true,
+    });
+    let stdout = "";
+    let stderr = "";
+    child.stdout.setEncoding("utf8");
+    child.stderr.setEncoding("utf8");
+    child.stdout.on("data", (chunk) => (stdout += chunk));
+    child.stderr.on("data", (chunk) => (stderr += chunk));
+    child.once("error", reject);
+    child.once("close", (code) => resolve({ code: code ?? 1, stdout, stderr }));
+  });
+}
+
 async function changedFilePaths(workspace) {
   const tracked = await run(
     "git",
@@ -452,7 +561,12 @@ async function workspaceFiles(root, current = root) {
 
 function isShareablePath(relative) {
   const normalized = normalizePath(relative);
-  if (!normalized || normalized === ".." || normalized.startsWith("../") || path.isAbsolute(normalized)) {
+  if (
+    !normalized ||
+    normalized === ".." ||
+    normalized.startsWith("../") ||
+    path.isAbsolute(normalized)
+  ) {
     return false;
   }
   const parts = normalized.split("/");
@@ -482,7 +596,9 @@ function isShareablePath(relative) {
 
 function isCodeBearing(relative) {
   const name = path.posix.basename(relative).toLowerCase();
-  return CODE_MARKERS.has(name) || CODE_EXTENSIONS.has(path.posix.extname(name));
+  return (
+    CODE_MARKERS.has(name) || CODE_EXTENSIONS.has(path.posix.extname(name))
+  );
 }
 
 async function worktreeBase(workspace, requested) {