@openai-lite/codex-feishu 0.1.5 → 0.1.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openai-lite/codex-feishu",
-  "version": "0.1.5",
+  "version": "0.1.7",
   "description": "Feishu bridge for Codex with dual-end synchronized conversations.",
   "type": "module",
   "bin": {

package/src/commands/daemon.js

@@ -5944,6 +5944,13 @@ export async function runDaemon() {
       signal: info.signal ?? null,
     });
   });
+  app.on("error", async (err) => {
+    await appendEvent(store, {
+      source: "app_server",
+      type: "error",
+      error: err?.message ?? String(err),
+    });
+  });
 
   try {
     await app.ensureStarted();

package/src/lib/app_server_client.js

@@ -1,7 +1,59 @@
 import { EventEmitter, once } from "node:events";
+import fs from "node:fs";
 import { spawn } from "node:child_process";
 import readline from "node:readline";
 
+function normalizeExecutable(raw) {
+  const text = String(raw ?? "").trim();
+  if (!text) {
+    return text;
+  }
+  const quoted =
+    (text.startsWith('"') && text.endsWith('"')) || (text.startsWith("'") && text.endsWith("'"));
+  return quoted ? text.slice(1, -1).trim() : text;
+}
+
+function quoteForCmd(arg) {
+  const text = String(arg ?? "");
+  if (text.length === 0) {
+    return '""';
+  }
+  if (!/[\s"&|<>^()]/.test(text)) {
+    return text;
+  }
+  return `"${text.replace(/"/g, '""')}"`;
+}
+
+function sanitizeSpawnCwd(cwd) {
+  if (typeof cwd !== "string" || !cwd.trim()) {
+    return process.cwd();
+  }
+  const normalized = cwd.trim();
+  if (process.platform === "win32") {
+    try {
+      if (!fs.existsSync(normalized)) {
+        return process.cwd();
+      }
+    } catch {
+      return process.cwd();
+    }
+  }
+  return normalized;
+}
+
+function spawnCodex(codexBin, args, options = {}) {
+  const bin = normalizeExecutable(codexBin);
+  const spawnOptions = {
+    ...options,
+    cwd: sanitizeSpawnCwd(options.cwd),
+  };
+  if (process.platform !== "win32") {
+    return spawn(bin, args, spawnOptions);
+  }
+  const command = [bin, ...args].map(quoteForCmd).join(" ");
+  return spawn("cmd.exe", ["/d", "/s", "/c", command], spawnOptions);
+}
+
 function normalizeJsonRpcPayload(payload) {
   if (!payload || typeof payload !== "object") {
     return null;
@@ -77,7 +129,7 @@ async function probeSubcommand(codexBin, args, timeoutMs = 2_500) {
     let stdout = "";
     let stderr = "";
     let timer = null;
-    const child = spawn(codexBin, args, {
+    const child = spawnCodex(codexBin, args, {
       stdio: ["ignore", "pipe", "pipe"],
       env: process.env,
     });
@@ -135,7 +187,11 @@ async function probeSubcommand(codexBin, args, timeoutMs = 2_500) {
 export class AppServerClient extends EventEmitter {
   constructor(options = {}) {
     super();
-    this.codexBin = options.codexBin || process.env.CODEX_BIN || "codex";
+    this.codexBin = normalizeExecutable(
+      options.codexBin ||
+      process.env.CODEX_BIN ||
+      (process.platform === "win32" ? "codex.cmd" : "codex"),
+    );
     this.transport = options.transport || process.env.CODEX_FEISHU_TRANSPORT || "auto";
     this.protoCwd = options.protoCwd || process.env.CODEX_FEISHU_CWD || process.cwd();
     this.protoModel = options.protoModel || process.env.CODEX_FEISHU_MODEL || "gpt-5.3-codex";
@@ -236,7 +292,7 @@ export class AppServerClient extends EventEmitter {
   }
 
   async startLegacyAppServer() {
-    const child = spawn(this.codexBin, ["app-server", "--listen", "stdio://"], {
+    const child = spawnCodex(this.codexBin, ["app-server", "--listen", "stdio://"], {
       stdio: ["pipe", "pipe", "pipe"],
       env: process.env,
       cwd: this.protoCwd,
@@ -309,7 +365,7 @@ export class AppServerClient extends EventEmitter {
   }
 
   async startProto() {
-    const child = spawn(this.codexBin, ["proto"], {
+    const child = spawnCodex(this.codexBin, ["proto"], {
       stdio: ["pipe", "pipe", "pipe"],
       env: process.env,
       cwd: this.protoCwd,

package/src/lib/daemon_control.js

@@ -56,6 +56,22 @@ async function isDaemonRpcResponsive(timeoutMs = 1000) {
   }
 }
 
+async function readLogTail(logPath, maxLines = 60) {
+  try {
+    const text = await readTextIfExists(logPath);
+    if (!text) {
+      return "";
+    }
+    const lines = text
+      .split(/\r?\n/)
+      .filter((line) => line.trim().length > 0)
+      .slice(-maxLines);
+    return lines.join("\n");
+  } catch {
+    return "";
+  }
+}
+
 async function stopByPid(pid, timeoutMs = 3000) {
   if (!isPidAlive(pid)) {
     return { action: "already_stopped", pid };
@@ -201,7 +217,7 @@ export async function restartDaemonDetached() {
     for (const cliEntry of candidateEntries) {
       attempts.push([process.execPath, [cliEntry, "daemon"]]);
     }
-    attempts.push(["codex-feishu", ["daemon"]]);
+    attempts.push(["cmd.exe", ["/d", "/s", "/c", "codex-feishu daemon"]]);
   } else {
     attempts.push(["codex-feishu", ["daemon"]]);
     for (const cliEntry of candidateEntries) {
@@ -234,8 +250,10 @@ export async function restartDaemonDetached() {
     failedAttempts.push(`${cmd} ${args.join(" ")} => ${startResult.error}`);
   }
   if (!startResult.ok) {
+    const logTail = await readLogTail(logPath, 80);
     const details = failedAttempts.length > 0 ? `; attempts: ${failedAttempts.join(" | ")}` : "";
-    throw new Error(`failed to start daemon in background: ${startResult.error}${details}`);
+    const logHint = logTail ? `; daemon.log tail:\n${logTail}` : "";
+    throw new Error(`failed to start daemon in background: ${startResult.error}${details}${logHint}`);
   }
 
   if (startResult.pid) {