@engineeros/connector 0.10.2 → 0.10.3

package/package.json

@@ -1,40 +1,40 @@
-{
-  "name": "@engineeros/connector",
-  "version": "0.10.2",
-  "description": "Connect a local coding agent to EngineerOS, using ACP when supported.",
-  "private": false,
-  "type": "module",
-  "license": "UNLICENSED",
-  "files": [
-    "bin",
-    "src",
-    "README.md"
-  ],
-  "bin": {
-    "engineeros-connector": "bin/engineeros-connector.mjs"
-  },
-  "scripts": {
-    "start": "node ./bin/engineeros-connector.mjs",
-    "prepublishOnly": "node ./scripts/release.mjs check",
-    "release:patch": "node ./scripts/release.mjs patch",
-    "test": "node --test --test-concurrency=1",
-    "type-check": "node --check ./bin/engineeros-connector.mjs && node --check ./scripts/release.mjs && node --check ./src/acp-client.mjs && node --check ./src/agent-harness.mjs && node --check ./src/agent-registry.mjs && node --check ./src/capabilities.mjs && node --check ./src/cli-args.mjs && node --check ./src/config.mjs && node --check ./src/connection.mjs && node --check ./src/mcp-server.mjs && node --check ./src/runner.mjs"
-  },
-  "engines": {
-    "node": ">=22"
-  },
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/vinpuli/engineerosv2.git",
-    "directory": "packages/engineeros-connector"
-  },
-  "bugs": {
-    "url": "https://github.com/vinpuli/engineerosv2/issues"
-  },
-  "publishConfig": {
-    "access": "public"
-  },
-  "dependencies": {
-    "@agentclientprotocol/sdk": "1.3.0"
-  }
-}
+{
+  "name": "@engineeros/connector",
+  "version": "0.10.3",
+  "description": "Connect a local coding agent to EngineerOS, using ACP when supported.",
+  "private": false,
+  "type": "module",
+  "license": "UNLICENSED",
+  "files": [
+    "bin",
+    "src",
+    "README.md"
+  ],
+  "bin": {
+    "engineeros-connector": "bin/engineeros-connector.mjs"
+  },
+  "scripts": {
+    "start": "node ./bin/engineeros-connector.mjs",
+    "prepublishOnly": "node ./scripts/release.mjs check",
+    "release:patch": "node ./scripts/release.mjs patch",
+    "test": "node --test --test-concurrency=1",
+    "type-check": "node --check ./bin/engineeros-connector.mjs && node --check ./scripts/release.mjs && node --check ./src/acp-client.mjs && node --check ./src/agent-harness.mjs && node --check ./src/agent-registry.mjs && node --check ./src/capabilities.mjs && node --check ./src/cli-args.mjs && node --check ./src/codex-app-server.mjs && node --check ./src/config.mjs && node --check ./src/connection.mjs && node --check ./src/mcp-server.mjs && node --check ./src/runner.mjs"
+  },
+  "engines": {
+    "node": ">=22"
+  },
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/vinpuli/engineerosv2.git",
+    "directory": "packages/engineeros-connector"
+  },
+  "bugs": {
+    "url": "https://github.com/vinpuli/engineerosv2/issues"
+  },
+  "publishConfig": {
+    "access": "public"
+  },
+  "dependencies": {
+    "@agentclientprotocol/sdk": "1.3.0"
+  }
+}

package/src/acp-client.mjs

@@ -58,11 +58,50 @@ export function launchAcpAgent(
   };
 }
 
-export async function disposeAcpRuntimes() {
+export async function disposeAcpRuntimes() {
   const active = [...runtimes.values()];
   runtimes.clear();
   await Promise.all(active.map((runtime) => runtime.dispose()));
-}
+}
+
+export async function inspectAcpExecutionProfiles(workspace, config) {
+  const runtime = new AcpRuntime(workspace, config, () => {});
+  try {
+    await runtime.ready;
+    const response = await runtime.context.request(acp.methods.agent.session.new, {
+      cwd: workspace,
+      mcpServers: [],
+    });
+    return executionProfilesFromConfigOptions(response.configOptions);
+  } finally {
+    await runtime.dispose();
+  }
+}
+
+export function executionProfilesFromConfigOptions(configOptions = []) {
+  const selectOption = (category) =>
+    configOptions.find((candidate) => candidate.category === category && candidate.type === "select");
+  const selectValues = (option) =>
+    (option?.options || []).flatMap((candidate) =>
+      Array.isArray(candidate.options) ? candidate.options : [candidate],
+    );
+  const modelOption = selectOption("model");
+  const effortOption = selectOption("thought_level");
+  const models = selectValues(modelOption);
+  const efforts = selectValues(effortOption);
+  return {
+    model_selection: models.length > 0,
+    model_profiles: models.map((model) => ({
+      id: model.value,
+      name: model.name || model.value,
+      description: model.description || "",
+      is_default: model.value === modelOption.currentValue,
+      default_reasoning_effort: effortOption?.currentValue || null,
+      reasoning_efforts: efforts.map((effort) => effort.value),
+    })),
+    reasoning_efforts: efforts.map((effort) => effort.value),
+  };
+}
 
 export function activeAcpRuntimeCount() {
   return runtimes.size;

package/src/capabilities.mjs

@@ -1,29 +1,24 @@
-import path from "node:path";
-import { agentHarnessCapabilities } from "./agent-harness.mjs";
-
+import path from "node:path";
+import { agentHarnessCapabilities } from "./agent-harness.mjs";
+
 export function advertisedCapabilities(config, codingAgent) {
-  const configuredModels = String(
-    process.env.ENGINEEROS_AGENT_MODELS || "gpt-5.6-sol,gpt-5.6-terra",
-  )
-    .split(",")
-    .map((model) => model.trim())
-    .filter(Boolean);
-  return {
-    agent_protocols: [codingAgent.protocol],
-    coding_agent: true,
-    codex_cli: codingAgent.protocol === "codex",
-    platform: process.platform,
-    workspace_name: path.basename(config.workspace),
-    agent_name: codingAgent.name,
-    agent_version: codingAgent.version,
-    ...agentHarnessCapabilities(),
-    execution_profiles: {
-      model_selection: codingAgent.protocol === "codex",
-      models: codingAgent.protocol === "codex" ? configuredModels : [],
-      reasoning_efforts:
-        codingAgent.protocol === "codex"
-          ? ["low", "medium", "high", "xhigh"]
-          : [],
-    },
+  const executionProfiles = codingAgent.executionProfiles || {
+    model_selection: false,
+    model_profiles: [],
+    reasoning_efforts: [],
   };
-}
+  return {
+    agent_protocols: [codingAgent.protocol],
+    coding_agent: true,
+    codex_cli: codingAgent.protocol === "codex",
+    platform: process.platform,
+    workspace_name: path.basename(config.workspace),
+    agent_name: codingAgent.name,
+    agent_version: codingAgent.version,
+    ...agentHarnessCapabilities(),
+    execution_profiles: {
+      ...executionProfiles,
+      models: (executionProfiles.model_profiles || []).map((model) => model.id),
+    },
+  };
+}

package/src/codex-app-server.mjs

@@ -1,5 +1,84 @@
 import { spawn } from "node:child_process";
-import readline from "node:readline";
+import readline from "node:readline";
+
+export async function inspectCodexExecutionProfiles({
+  workspace,
+  command = process.env.CODEX_BIN || (process.platform === "win32" ? "codex.cmd" : "codex"),
+  spawnProcess = spawn,
+  timeoutMs = 15_000,
+}) {
+  const child = spawnProcess(command, ["app-server", "--stdio"], {
+    cwd: workspace,
+    env: process.env,
+    shell: process.platform === "win32",
+    windowsHide: true,
+    stdio: ["pipe", "pipe", "pipe"],
+  });
+  const pending = new Map();
+  let requestId = 0;
+  let stderr = "";
+  const request = (method, params) => new Promise((resolve, reject) => {
+    const id = ++requestId;
+    pending.set(id, { resolve, reject });
+    child.stdin.write(`${JSON.stringify({ id, method, params })}\n`);
+  });
+  const lines = readline.createInterface({ input: child.stdout });
+  lines.on("line", (line) => {
+    try {
+      const message = JSON.parse(line);
+      const waiter = pending.get(message.id);
+      if (!waiter) return;
+      pending.delete(message.id);
+      if (message.error) waiter.reject(new Error(message.error.message || "Codex model discovery failed."));
+      else waiter.resolve(message.result);
+    } catch {
+      // Ignore non-protocol output.
+    }
+  });
+  child.stderr.setEncoding("utf8");
+  child.stderr.on("data", (chunk) => { stderr = `${stderr}${chunk}`.slice(-4_000); });
+  const rejectPending = (message) => {
+    for (const waiter of pending.values()) waiter.reject(new Error(message));
+    pending.clear();
+  };
+  child.once("error", (error) => rejectPending(error.message));
+  child.once("close", (code) => {
+    if (pending.size) rejectPending(`Codex model discovery stopped with code ${code ?? 1}. ${stderr}`.trim());
+  });
+  const timeout = setTimeout(() => {
+    rejectPending("Codex model discovery timed out.");
+    child.kill();
+  }, timeoutMs);
+  try {
+    await request("initialize", {
+      clientInfo: { name: "engineeros-connector", title: "EngineerOS Connector", version: "0.10.3" },
+    });
+    child.stdin.write(`${JSON.stringify({ method: "initialized" })}\n`);
+    const models = [];
+    let cursor = null;
+    do {
+      const result = await request("model/list", { cursor, includeHidden: false });
+      models.push(...(Array.isArray(result?.data) ? result.data : []));
+      cursor = result?.nextCursor || null;
+    } while (cursor);
+    return models.map((item) => ({
+      id: item.model || item.id,
+      name: item.displayName || item.model || item.id,
+      description: item.description || "",
+      is_default: item.isDefault === true,
+      default_reasoning_effort: item.defaultReasoningEffort || null,
+      reasoning_efforts: (item.supportedReasoningEfforts || [])
+        .map((option) => option.reasoningEffort)
+        .filter(Boolean),
+    })).filter((item) => item.id);
+  } catch (error) {
+    const detail = error instanceof Error ? error.message : String(error);
+    throw new Error(`${detail}${stderr ? ` ${stderr}` : ""}`.trim());
+  } finally {
+    clearTimeout(timeout);
+    child.kill();
+  }
+}
 
 export function launchCodexAppServer({
   workspace,
@@ -89,7 +168,7 @@ export function launchCodexAppServer({
     void (async () => {
       try {
         await request("initialize", {
-          clientInfo: { name: "engineeros-connector", title: "EngineerOS Connector", version: "0.10.1" },
+          clientInfo: { name: "engineeros-connector", title: "EngineerOS Connector", version: "0.10.3" },
         });
         child.stdin.write(`${JSON.stringify({ method: "initialized" })}\n`);
         const threadResult = previousSessionId

package/src/runner.mjs

@@ -11,8 +11,11 @@ import os from "node:os";
 import path from "node:path";
 import { promisify } from "node:util";
 import { deflateRaw } from "node:zlib";
-import { launchAcpAgent } from "./acp-client.mjs";
-import { launchCodexAppServer } from "./codex-app-server.mjs";
+import { inspectAcpExecutionProfiles, launchAcpAgent } from "./acp-client.mjs";
+import {
+  inspectCodexExecutionProfiles,
+  launchCodexAppServer,
+} from "./codex-app-server.mjs";
 import {
   buildAgentHarnessPrompt,
   normalizeAgentStructuredOutput,
@@ -526,22 +529,34 @@ export async function executeConnectedPrompt(assignment, config, callbacks) {
   };
 }
 
-export async function inspectCodingAgent(config, workspace = process.cwd()) {
+export async function inspectCodingAgent(config, workspace = process.cwd()) {
   if (config.agent_protocol === "acp") {
     if (!config.agent_command) {
       throw new Error(
         "ACP requires --agent-command when pairing the connector.",
       );
     }
-    return {
-      protocol: "acp",
+    const executionProfiles = await inspectAcpExecutionProfiles(workspace, config);
+    return {
+      protocol: "acp",
       name: config.agent_name || path.basename(config.agent_command),
-      version: config.agent_version || "ACP v1",
+      version: config.agent_version || "ACP v1",
+      executionProfiles,
     };
   }
-  const codex = await inspectCodexCli(workspace);
-  return { protocol: "codex", name: "Codex CLI", version: codex.version };
-}
+  const codex = await inspectCodexCli(workspace);
+  const modelProfiles = await inspectCodexExecutionProfiles({ workspace, command: codex.command });
+  return {
+    protocol: "codex",
+    name: "Codex CLI",
+    version: codex.version,
+    executionProfiles: {
+      model_selection: modelProfiles.length > 0,
+      model_profiles: modelProfiles,
+      reasoning_efforts: [...new Set(modelProfiles.flatMap((model) => model.reasoning_efforts))],
+    },
+  };
+}
 
 export async function inspectCodexCli(workspace = process.cwd()) {
   const command =
@@ -727,10 +742,10 @@ export function connectorExecution(assignment, options = {}) {
   const model =
     typeof rawProfile.model === "string" ? rawProfile.model.trim() : "";
   const reasoningEffort = rawProfile.reasoning_effort;
-  if (
-    reasoningEffort &&
-    !new Set(["low", "medium", "high", "xhigh"]).has(reasoningEffort)
-  ) {
+  if (
+    reasoningEffort &&
+    !new Set(["minimal", "low", "medium", "high", "xhigh", "max", "ultra"]).has(reasoningEffort)
+  ) {
     throw new Error(
       "EngineerOS assignment has an unsupported reasoning effort.",
     );

package/src/skills/change-planning/SKILL.md

@@ -1,12 +1,12 @@
----
-name: change-planning
-description: Produce a decision-complete implementation plan grounded in the connected repository.
----
-
-# Change Planning
-
-1. Inspect current behavior, ownership boundaries, call paths, tests, and project instructions before proposing changes.
-2. Define the intended outcome, affected components, non-goals, risks, and proof of completion.
-3. Resolve questions answerable from the repository. Label only genuinely unavailable facts as assumptions.
-4. Sequence bounded implementation steps with the files or symbols each step affects and the verification it requires.
-5. Stay read-only and hand off a plan that an implementation Agent can execute without rediscovering the problem.
+---
+name: change-planning
+description: Produce a decision-complete implementation plan grounded in the connected repository.
+---
+
+# Change Planning
+
+1. Inspect current behavior, ownership boundaries, call paths, tests, and project instructions before proposing changes.
+2. Define the intended outcome, affected components, non-goals, risks, and proof of completion.
+3. Resolve questions answerable from the repository. Label only genuinely unavailable facts as assumptions.
+4. Sequence bounded implementation steps with the files or symbols each step affects and the verification it requires.
+5. Stay read-only and hand off a plan that an implementation Agent can execute without rediscovering the problem.

package/src/skills/change-verification/SKILL.md

@@ -1,12 +1,12 @@
----
-name: change-verification
-description: Independently verify a completed change against its explicit proof contract.
----
-
-# Change Verification
-
-1. Inspect the exact supplied revision, changed paths, Goal boundaries, constraints, and every Proof item.
-2. Run or inspect each check independently; do not rely on the implementation Agent's claims.
-3. Stay read-only. Do not repair failures, edit files, install dependencies, commit, or push.
-4. Mark a check passed only when directly observed evidence matches its expected result.
-5. Return the requested structured verification report with concise commands, exit codes, and evidence.
+---
+name: change-verification
+description: Independently verify a completed change against its explicit proof contract.
+---
+
+# Change Verification
+
+1. Inspect the exact supplied revision, changed paths, Goal boundaries, constraints, and every Proof item.
+2. Run or inspect each check independently; do not rely on the implementation Agent's claims.
+3. Stay read-only. Do not repair failures, edit files, install dependencies, commit, or push.
+4. Mark a check passed only when directly observed evidence matches its expected result.
+5. Return the requested structured verification report with concise commands, exit codes, and evidence.

package/src/skills/codebase-research/SKILL.md

@@ -1,12 +1,12 @@
----
-name: codebase-research
-description: Investigate a connected repository and answer from directly observed evidence.
----
-
-# Codebase Research
-
-1. Define the exact question and inspect the smallest relevant surface first.
-2. Trace callers, dependencies, data flow, configuration, and tests when they materially affect the answer.
-3. Separate observed facts from inferences and cite concrete repository paths for important claims.
-4. Stay read-only. Do not install, generate, edit, delete, commit, or start long-running services.
-5. Return the answer first, followed by supporting evidence and unresolved gaps only when they matter.
+---
+name: codebase-research
+description: Investigate a connected repository and answer from directly observed evidence.
+---
+
+# Codebase Research
+
+1. Define the exact question and inspect the smallest relevant surface first.
+2. Trace callers, dependencies, data flow, configuration, and tests when they materially affect the answer.
+3. Separate observed facts from inferences and cite concrete repository paths for important claims.
+4. Stay read-only. Do not install, generate, edit, delete, commit, or start long-running services.
+5. Return the answer first, followed by supporting evidence and unresolved gaps only when they matter.

package/src/skills/goal-execution/SKILL.md

@@ -1,12 +1,12 @@
----
-name: goal-execution
-description: Implement one bounded EngineerOS Goal and prove the resulting behavior.
----
-
-# Goal Execution
-
-1. Read the complete Goal packet, repository instructions, current Git state, and relevant implementation paths.
-2. Implement the smallest coherent change that satisfies the included outcome and constraints.
-3. Preserve unrelated user changes and stay inside the stated boundary; stop only for a genuine missing authority or prerequisite.
-4. Run focused checks while working, then the proportionate final tests, lint, type checks, or build required by the Goal.
-5. Do not commit or push. EngineerOS creates the isolated run commit after the implementation completes.
+---
+name: goal-execution
+description: Implement one bounded EngineerOS Goal and prove the resulting behavior.
+---
+
+# Goal Execution
+
+1. Read the complete Goal packet, repository instructions, current Git state, and relevant implementation paths.
+2. Implement the smallest coherent change that satisfies the included outcome and constraints.
+3. Preserve unrelated user changes and stay inside the stated boundary; stop only for a genuine missing authority or prerequisite.
+4. Run focused checks while working, then the proportionate final tests, lint, type checks, or build required by the Goal.
+5. Do not commit or push. EngineerOS creates the isolated run commit after the implementation completes.