@engineeros/connector 0.15.2 → 0.16.0

package/package.json

@@ -1,41 +1,41 @@
-{
-  "name": "@engineeros/connector",
-  "version": "0.15.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/assessment-spool.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",
-    "zod": "3.25.76"
-  }
-}
+{
+  "name": "@engineeros/connector",
+  "version": "0.16.0",
+  "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/assessment-spool.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",
+    "zod": "3.25.76"
+  }
+}

package/src/acp-client.mjs

@@ -115,10 +115,10 @@ class AcpRuntime {
     this.onClose = onClose;
     this.sessions = new Map();
     this.turns = new Map();
-    this.stderr = "";
-    this.disposed = false;
-    this.disposePromise = null;
-    this.idleTimer = null;
+    this.stderr = "";
+    this.disposed = false;
+    this.disposePromise = null;
+    this.idleTimer = null;
     this.child = spawn(
       config.agent_command,
       Array.isArray(config.agent_args) ? config.agent_args : [],
@@ -159,13 +159,13 @@ class AcpRuntime {
       )
       .connect(stream);
     this.context = this.connection.agent;
-    this.ready = this.initialize();
-    void this.ready.catch(() => {
-      void this.dispose();
-    });
-    this.child.stdout.once("end", () => this.closed());
-    this.child.stdout.once("close", () => this.closed());
-    this.child.once("exit", () => this.closed());
+    this.ready = this.initialize();
+    void this.ready.catch(() => {
+      void this.dispose();
+    });
+    this.child.stdout.once("end", () => this.closed());
+    this.child.stdout.once("close", () => this.closed());
+    this.child.once("exit", () => this.closed());
     void this.connection.closed.then(
       () => this.closed(),
       () => this.closed(),
@@ -201,19 +201,20 @@ class AcpRuntime {
     this.clearIdleTimer();
     const initialized = await this.ready;
     let session = this.sessions.get(sessionKey);
-    if (!session) {
-      session = await this.openSession(
+    if (!session) {
+      session = await this.openSession(
         sessionKey,
         previousSessionId,
         sandbox,
         profile,
       );
-      callbacks.onEvent?.({
-        type: "agent.connected",
-        message: `Agent connected with protocol ${initialized.protocolVersion}`,
-      });
-    }
-    if (this.turns.has(session.sessionId)) {
+      callbacks.onEvent?.({
+        type: "agent.connected",
+        message: `Agent connected with protocol ${initialized.protocolVersion}`,
+      });
+    }
+    await callbacks.onSession?.(session.sessionId);
+    if (this.turns.has(session.sessionId)) {
       throw new Error(
         "The agent is already answering another prompt in this session.",
       );
@@ -361,17 +362,17 @@ class AcpRuntime {
     });
   }
 
-  isRunning() {
-    return (
-      !this.disposed &&
-      this.childIsRunning() &&
-      !this.connection.signal.aborted
-    );
-  }
-
-  childIsRunning() {
-    return this.child.exitCode === null && this.child.signalCode === null;
-  }
+  isRunning() {
+    return (
+      !this.disposed &&
+      this.childIsRunning() &&
+      !this.connection.signal.aborted
+    );
+  }
+
+  childIsRunning() {
+    return this.child.exitCode === null && this.child.signalCode === null;
+  }
 
   scheduleIdleDisposal() {
     if (this.turns.size || this.disposed) return;
@@ -393,44 +394,44 @@ class AcpRuntime {
     );
   }
 
-  closed() {
-    void this.dispose();
-  }
-
-  async dispose() {
-    if (this.disposePromise) return this.disposePromise;
-    this.disposed = true;
-    this.clearIdleTimer();
-    this.onClose?.();
-    this.disposePromise = this.releaseResources();
-    return this.disposePromise;
-  }
-
-  async releaseResources() {
-    try {
-      this.connection.close();
-    } catch {
-      // The ACP transport may already be closed.
-    }
-    if (this.childIsRunning()) {
-      this.child.kill("SIGTERM");
-      await Promise.race([
-        new Promise((resolve) => this.child.once("exit", resolve)),
-        new Promise((resolve) => setTimeout(resolve, 500)),
-      ]);
-    }
-    if (this.childIsRunning()) {
-      this.child.kill("SIGKILL");
-      await Promise.race([
-        new Promise((resolve) => this.child.once("exit", resolve)),
-        new Promise((resolve) => setTimeout(resolve, 500)),
-      ]);
-    }
-    this.child.stdin.destroy();
-    this.child.stdout.destroy();
-    this.child.stderr.destroy();
-  }
-}
+  closed() {
+    void this.dispose();
+  }
+
+  async dispose() {
+    if (this.disposePromise) return this.disposePromise;
+    this.disposed = true;
+    this.clearIdleTimer();
+    this.onClose?.();
+    this.disposePromise = this.releaseResources();
+    return this.disposePromise;
+  }
+
+  async releaseResources() {
+    try {
+      this.connection.close();
+    } catch {
+      // The ACP transport may already be closed.
+    }
+    if (this.childIsRunning()) {
+      this.child.kill("SIGTERM");
+      await Promise.race([
+        new Promise((resolve) => this.child.once("exit", resolve)),
+        new Promise((resolve) => setTimeout(resolve, 500)),
+      ]);
+    }
+    if (this.childIsRunning()) {
+      this.child.kill("SIGKILL");
+      await Promise.race([
+        new Promise((resolve) => this.child.once("exit", resolve)),
+        new Promise((resolve) => setTimeout(resolve, 500)),
+      ]);
+    }
+    this.child.stdin.destroy();
+    this.child.stdout.destroy();
+    this.child.stderr.destroy();
+  }
+}
 
 export function normalizeTokenUsage(value) {
   if (!value || typeof value !== "object") return null;