@wrongstack/acp 1.0.26 → 1.0.27

package/dist/agent.js

@@ -1691,6 +1691,16 @@ import { treeKill } from "@wrongstack/core/utils/tree-kill";
 var DEFAULT_MAX_FRAME_CHARS = 20 * 1024 * 1024;
 var DEFAULT_MAX_QUEUED_MESSAGES = 1e3;
 var DEFAULT_MAX_QUEUED_CHARS = 32 * 1024 * 1024;
+function takeLines(pending, chunk) {
+  const lines = [];
+  let start = 0;
+  for (let nl = chunk.indexOf("\n"); nl !== -1; nl = chunk.indexOf("\n", start)) {
+    lines.push(pending + chunk.slice(start, nl));
+    pending = "";
+    start = nl + 1;
+  }
+  return { lines, pending: pending + chunk.slice(start) };
+}
 function positiveLimit(value, fallback) {
   return value !== void 0 && Number.isFinite(value) && value > 0 ? Math.floor(value) : fallback;
 }
@@ -1778,9 +1788,8 @@ var StdioTransport = class {
     this.claimHandlers.clear();
   }
   onData(chunk) {
-    this.buffer += chunk;
-    const lines = this.buffer.split("\n");
-    this.buffer = lines.pop() ?? "";
+    const { lines, pending } = takeLines(this.buffer, chunk);
+    this.buffer = pending;
     if (this.buffer.length > this.maxFrameChars) {
       this.stderr.write(
         `[wstack-acp frame error] pending frame exceeds ${this.maxFrameChars} characters

package/dist/client.js

@@ -11,6 +11,16 @@ import {
 var DEFAULT_MAX_FRAME_CHARS = 20 * 1024 * 1024;
 var DEFAULT_MAX_QUEUED_MESSAGES = 1e3;
 var DEFAULT_MAX_QUEUED_CHARS = 32 * 1024 * 1024;
+function takeLines(pending, chunk) {
+  const lines = [];
+  let start = 0;
+  for (let nl = chunk.indexOf("\n"); nl !== -1; nl = chunk.indexOf("\n", start)) {
+    lines.push(pending + chunk.slice(start, nl));
+    pending = "";
+    start = nl + 1;
+  }
+  return { lines, pending: pending + chunk.slice(start) };
+}
 function positiveLimit(value, fallback) {
   return value !== void 0 && Number.isFinite(value) && value > 0 ? Math.floor(value) : fallback;
 }
@@ -178,9 +188,8 @@ var ClientTransport = class {
     this.child = null;
   }
   onChildData(chunk) {
-    this.buffer += chunk;
-    const lines = this.buffer.split("\n");
-    this.buffer = lines.pop() ?? "";
+    const { lines, pending } = takeLines(this.buffer, chunk);
+    this.buffer = pending;
     if (this.buffer.length > this.maxFrameChars) {
       writeErr(`[acp-child pending frame exceeds ${this.maxFrameChars} characters]
 `);
@@ -1263,11 +1272,30 @@ function makePermissionPolicy(decide) {
 // src/client/terminal-server.ts
 import { spawn } from "node:child_process";
 import { randomBytes as randomBytes2 } from "node:crypto";
-import { realpathSync as realpathSync2 } from "node:fs";
+import { accessSync, realpathSync as realpathSync2 } from "node:fs";
 import * as path2 from "node:path";
 import { buildChildEnv } from "@wrongstack/core/utils";
 import { treeKill as treeKill2 } from "@wrongstack/core/utils/tree-kill";
 var EMPTY_BUFFER = Buffer.alloc(0);
+function win32BatchTarget(command) {
+  if (/\.(?:cmd|bat)$/i.test(command)) return command;
+  if (/[\\/]/.test(command) || path2.extname(command)) return void 0;
+  const exts = (process.env["PATHEXT"] ?? ".COM;.EXE;.BAT;.CMD").toLowerCase().split(";");
+  for (const dir of (process.env["PATH"] ?? "").split(path2.delimiter)) {
+    if (!dir) continue;
+    for (const ext of exts) {
+      if (!ext) continue;
+      const full = path2.join(dir, `${command}${ext}`);
+      try {
+        accessSync(full);
+      } catch {
+        continue;
+      }
+      return ext === ".cmd" || ext === ".bat" ? full : void 0;
+    }
+  }
+  return void 0;
+}
 var DEBUG_DISPOSE = typeof process !== "undefined" && !!process.env?.WRONGSTACK_DEBUG && process.env.WRONGSTACK_DEBUG !== "0" && process.env.WRONGSTACK_DEBUG !== "false";
 var TerminalServer = class {
   terminals = /* @__PURE__ */ new Map();
@@ -1318,11 +1346,14 @@ var TerminalServer = class {
       Math.max(1, this.clampFiniteInt(params.outputByteLimit, this.outputByteLimit)),
       this.maxOutputByteLimit
     );
-    const proc = spawn(params.command, params.args ?? [], {
+    const batch = process.platform === "win32" ? win32BatchTarget(params.command) : void 0;
+    const shim = batch ? buildWin32CmdShimInvocation(batch, params.args ?? []) : void 0;
+    const proc = spawn(shim?.command ?? params.command, shim?.args ?? params.args ?? [], {
       cwd,
       env: this.buildEnv(params.env),
       stdio: ["ignore", "pipe", "pipe"],
-      windowsHide: true
+      windowsHide: true,
+      ...shim ? { windowsVerbatimArguments: shim.windowsVerbatimArguments } : {}
       // shell: false on purpose. The terminal server is invoked with
       // the agent's explicit argv; turning on shell-mode would make
       // the command a single shell-parsed string, which breaks

package/dist/index.js

@@ -991,6 +991,16 @@ import {
 var DEFAULT_MAX_FRAME_CHARS = 20 * 1024 * 1024;
 var DEFAULT_MAX_QUEUED_MESSAGES = 1e3;
 var DEFAULT_MAX_QUEUED_CHARS = 32 * 1024 * 1024;
+function takeLines(pending, chunk) {
+  const lines = [];
+  let start = 0;
+  for (let nl = chunk.indexOf("\n"); nl !== -1; nl = chunk.indexOf("\n", start)) {
+    lines.push(pending + chunk.slice(start, nl));
+    pending = "";
+    start = nl + 1;
+  }
+  return { lines, pending: pending + chunk.slice(start) };
+}
 function positiveLimit(value, fallback) {
   return value !== void 0 && Number.isFinite(value) && value > 0 ? Math.floor(value) : fallback;
 }
@@ -1078,9 +1088,8 @@ var StdioTransport = class {
     this.claimHandlers.clear();
   }
   onData(chunk) {
-    this.buffer += chunk;
-    const lines = this.buffer.split("\n");
-    this.buffer = lines.pop() ?? "";
+    const { lines, pending } = takeLines(this.buffer, chunk);
+    this.buffer = pending;
     if (this.buffer.length > this.maxFrameChars) {
       this.stderr.write(
         `[wstack-acp frame error] pending frame exceeds ${this.maxFrameChars} characters
@@ -1320,9 +1329,8 @@ var ClientTransport = class {
     this.child = null;
   }
   onChildData(chunk) {
-    this.buffer += chunk;
-    const lines = this.buffer.split("\n");
-    this.buffer = lines.pop() ?? "";
+    const { lines, pending } = takeLines(this.buffer, chunk);
+    this.buffer = pending;
     if (this.buffer.length > this.maxFrameChars) {
       writeErr(`[acp-child pending frame exceeds ${this.maxFrameChars} characters]
 `);
@@ -2786,11 +2794,30 @@ function makePermissionPolicy(decide) {
 // src/client/terminal-server.ts
 import { spawn } from "node:child_process";
 import { randomBytes as randomBytes2 } from "node:crypto";
-import { realpathSync as realpathSync2 } from "node:fs";
+import { accessSync, realpathSync as realpathSync2 } from "node:fs";
 import * as path3 from "node:path";
 import { buildChildEnv } from "@wrongstack/core/utils";
 import { treeKill as treeKill2 } from "@wrongstack/core/utils/tree-kill";
 var EMPTY_BUFFER = Buffer.alloc(0);
+function win32BatchTarget(command) {
+  if (/\.(?:cmd|bat)$/i.test(command)) return command;
+  if (/[\\/]/.test(command) || path3.extname(command)) return void 0;
+  const exts = (process.env["PATHEXT"] ?? ".COM;.EXE;.BAT;.CMD").toLowerCase().split(";");
+  for (const dir of (process.env["PATH"] ?? "").split(path3.delimiter)) {
+    if (!dir) continue;
+    for (const ext of exts) {
+      if (!ext) continue;
+      const full = path3.join(dir, `${command}${ext}`);
+      try {
+        accessSync(full);
+      } catch {
+        continue;
+      }
+      return ext === ".cmd" || ext === ".bat" ? full : void 0;
+    }
+  }
+  return void 0;
+}
 var DEBUG_DISPOSE = typeof process !== "undefined" && !!process.env?.WRONGSTACK_DEBUG && process.env.WRONGSTACK_DEBUG !== "0" && process.env.WRONGSTACK_DEBUG !== "false";
 var TerminalServer = class {
   terminals = /* @__PURE__ */ new Map();
@@ -2841,11 +2868,14 @@ var TerminalServer = class {
       Math.max(1, this.clampFiniteInt(params.outputByteLimit, this.outputByteLimit)),
       this.maxOutputByteLimit
     );
-    const proc = spawn(params.command, params.args ?? [], {
+    const batch = process.platform === "win32" ? win32BatchTarget(params.command) : void 0;
+    const shim = batch ? buildWin32CmdShimInvocation(batch, params.args ?? []) : void 0;
+    const proc = spawn(shim?.command ?? params.command, shim?.args ?? params.args ?? [], {
       cwd,
       env: this.buildEnv(params.env),
       stdio: ["ignore", "pipe", "pipe"],
-      windowsHide: true
+      windowsHide: true,
+      ...shim ? { windowsVerbatimArguments: shim.windowsVerbatimArguments } : {}
       // shell: false on purpose. The terminal server is invoked with
       // the agent's explicit argv; turning on shell-mode would make
       // the command a single shell-parsed string, which breaks

package/dist/wrongstack-acp-agent.js

@@ -988,6 +988,16 @@ import { treeKill } from "@wrongstack/core/utils/tree-kill";
 var DEFAULT_MAX_FRAME_CHARS = 20 * 1024 * 1024;
 var DEFAULT_MAX_QUEUED_MESSAGES = 1e3;
 var DEFAULT_MAX_QUEUED_CHARS = 32 * 1024 * 1024;
+function takeLines(pending, chunk) {
+  const lines = [];
+  let start = 0;
+  for (let nl = chunk.indexOf("\n"); nl !== -1; nl = chunk.indexOf("\n", start)) {
+    lines.push(pending + chunk.slice(start, nl));
+    pending = "";
+    start = nl + 1;
+  }
+  return { lines, pending: pending + chunk.slice(start) };
+}
 function positiveLimit(value, fallback) {
   return value !== void 0 && Number.isFinite(value) && value > 0 ? Math.floor(value) : fallback;
 }
@@ -1075,9 +1085,8 @@ var StdioTransport = class {
     this.claimHandlers.clear();
   }
   onData(chunk) {
-    this.buffer += chunk;
-    const lines = this.buffer.split("\n");
-    this.buffer = lines.pop() ?? "";
+    const { lines, pending } = takeLines(this.buffer, chunk);
+    this.buffer = pending;
     if (this.buffer.length > this.maxFrameChars) {
       this.stderr.write(
         `[wstack-acp frame error] pending frame exceeds ${this.maxFrameChars} characters

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wrongstack/acp",
-  "version": "1.0.26",
+  "version": "1.0.27",
   "license": "MIT",
   "description": "ACP (Agent Client Protocol) integration for WrongStack — client + agent support",
   "keywords": [
@@ -52,8 +52,8 @@
   ],
   "dependencies": {
     "@agentclientprotocol/sdk": "^1.4.0",
-    "@wrongstack/core": "1.0.26",
-    "@wrongstack/primitives": "1.0.26"
+    "@wrongstack/core": "1.0.27",
+    "@wrongstack/primitives": "1.0.27"
   },
   "devDependencies": {
     "@types/node": "^26.5.1",