cli-wechat-bridge 1.1.1 → 1.1.3

package/dist/wechat/channel-config.js

@@ -18,6 +18,34 @@ export const SYNC_BUF_FILE = path.join(CHANNEL_DATA_DIR, "sync_buf.txt");
 export const CONTEXT_CACHE_FILE = path.join(CHANNEL_DATA_DIR, "context_tokens.json");
 export const BRIDGE_STATE_FILE = path.join(CHANNEL_DATA_DIR, "bridge-state.json");
 export const BRIDGE_LOG_FILE = path.join(CHANNEL_DATA_DIR, "bridge.log");
+// Hard cap for bridge.log. Long-running daemons otherwise grow it without bound.
+export const BRIDGE_LOG_MAX_BYTES = 5 * 1024 * 1024; // 5 MiB
+/**
+ * Append a line to a log file, trimming it down to the tail half whenever it
+ * exceeds BRIDGE_LOG_MAX_BYTES. Trimming is best-effort under concurrent
+ * writers (a few lines may be lost during a trim) but keeps the file bounded.
+ */
+export function appendBoundedLog(filePath, line) {
+    try {
+        const stat = fs.statSync(filePath);
+        if (stat.size > BRIDGE_LOG_MAX_BYTES) {
+            const keepSize = Math.floor(BRIDGE_LOG_MAX_BYTES / 2);
+            const fd = fs.openSync(filePath, "r");
+            try {
+                const tail = Buffer.alloc(keepSize);
+                const bytesRead = fs.readSync(fd, tail, 0, keepSize, stat.size - keepSize);
+                fs.writeFileSync(filePath, tail.subarray(0, bytesRead));
+            }
+            finally {
+                fs.closeSync(fd);
+            }
+        }
+    }
+    catch {
+        // File missing or unreadable: fall through to a plain append.
+    }
+    fs.appendFileSync(filePath, line);
+}
 export const BRIDGE_LOCK_FILE = path.join(CHANNEL_DATA_DIR, "bridge.lock.json");
 export const DAEMON_ENDPOINT_FILE = path.join(CHANNEL_DATA_DIR, "daemon-endpoint.json");
 export const CODEX_PANEL_ENDPOINT_FILE = path.join(CHANNEL_DATA_DIR, "codex-panel-endpoint.json");

package/dist/wechat/setup.js

@@ -169,12 +169,14 @@ function printPostLoginHelp(log) {
     log("  wechat-codex-start");
     log("  wechat-claude-start");
     log("  wechat-opencode-start");
+    log("  wechat-pi-start");
     log("  wechat-bridge-shell");
     log("");
     log("Manual two-terminal mode is also available:");
     log("  wechat-bridge-codex  +  wechat-codex");
     log("  wechat-bridge-claude +  wechat-claude");
     log("  wechat-bridge-opencode + wechat-opencode");
+    log("  wechat-bridge-pi + wechat-pi");
     log("");
     log("Run wechat-setup again any time you need to refresh the login.");
 }

package/dist/wechat/wechat-transport.js

@@ -306,7 +306,12 @@ export function classifyWechatTransportError(error) {
 }
 function writeJsonFile(filePath, value) {
     ensureChannelDataDir();
-    fs.writeFileSync(filePath, JSON.stringify(value, null, 2), "utf-8");
+    const data = JSON.stringify(value, null, 2);
+    // Atomic write via temp file + rename, so a crash mid-write cannot leave a
+    // truncated/half-written JSON that would break readers on the next launch.
+    const tempPath = `${filePath}.${process.pid}.tmp`;
+    fs.writeFileSync(tempPath, data, "utf-8");
+    fs.renameSync(tempPath, filePath);
 }
 function randomWechatUin() {
     const uint32 = crypto.randomBytes(4).readUInt32BE(0);
@@ -445,7 +450,15 @@ async function getUploadUrl(account, params) {
         token: account.token,
         timeoutMs: SEND_TIMEOUT_MS,
     });
-    return JSON.parse(raw);
+    try {
+        return JSON.parse(raw);
+    }
+    catch (error) {
+        // A 200-OK but non-JSON body (e.g. an HTML error page, empty body) would
+        // otherwise throw an unclassified SyntaxError that bypasses transport error
+        // classification/retry. Surface it as a meaningful error instead.
+        throw new Error(`getUploadUrl returned a non-JSON response (${raw.length} bytes): ${String(error)}`, { cause: error });
+    }
 }
 function buildCdnUploadUrl(cdnBaseUrl, uploadParam, filekey) {
     return `${cdnBaseUrl}/upload?encrypted_query_param=${encodeURIComponent(uploadParam)}&filekey=${encodeURIComponent(filekey)}`;
@@ -678,11 +691,25 @@ export function buildInboundMessageClaimPath(messageKey, claimsDir = INBOUND_MES
     return path.join(claimsDir, fileName);
 }
 export function clearInboundMessageClaims(claimsDir = INBOUND_MESSAGE_CLAIMS_DIR) {
+    // Delete claim files individually from a directory snapshot rather than
+    // `rmSync(claimsDir, { recursive })`. A recursive rmdir can remove a file a
+    // concurrent process just created via tryClaimInboundMessage, silently
+    // invalidating its claim and letting the same message be double-processed.
+    // Per-file removal only deletes files present at snapshot time.
+    let entries;
     try {
-        fs.rmSync(claimsDir, { recursive: true, force: true });
+        entries = fs.readdirSync(claimsDir);
     }
     catch {
-        // Best effort cleanup.
+        return; // directory does not exist; nothing to clear
+    }
+    for (const entry of entries) {
+        try {
+            fs.rmSync(path.join(claimsDir, entry), { force: true });
+        }
+        catch {
+            // best effort
+        }
     }
 }
 export function tryClaimInboundMessage(messageKey, options = {}) {
@@ -719,7 +746,11 @@ export function tryClaimInboundMessage(messageKey, options = {}) {
             ? error.code
             : "";
         if (code !== "EEXIST") {
-            return true;
+            // Real persistence failure (permissions, disk full, missing dir, ...).
+            // Returning true here would claim success WITHOUT writing the claim file,
+            // letting concurrent processes all think they own this message and
+            // double-process it. Fail closed so the message is retried instead.
+            return false;
         }
     }
     try {
@@ -849,23 +880,28 @@ export class WeChatTransport {
             if (!extracted.text && extracted.attachments.length === 0) {
                 continue;
             }
-            const messageKey = buildMessageKey(rawMessage);
-            if (!this.rememberMessage(messageKey)) {
-                continue;
-            }
-            if (!tryClaimInboundMessage(buildScopedMessageClaimKey(account.accountId, messageKey))) {
-                continue;
-            }
             const senderId = rawMessage.from_user_id ?? "unknown";
             if (rawMessage.context_token) {
                 this.cacheContextToken(senderId, rawMessage.context_token);
             }
+            // Filter pre-start backlog BEFORE claiming/remembering the message so a
+            // skipped message is not permanently locked away from other processes.
+            // A missing create_time_ms is treated as fresh rather than dropped.
             const createdAtMs = rawMessage.create_time_ms ?? 0;
             if (typeof options.minCreatedAtMs === "number" &&
-                (!Number.isFinite(createdAtMs) || createdAtMs < options.minCreatedAtMs)) {
+                Number.isFinite(createdAtMs) &&
+                createdAtMs > 0 &&
+                createdAtMs < options.minCreatedAtMs) {
                 ignoredBacklogCount += 1;
                 continue;
             }
+            const messageKey = buildMessageKey(rawMessage);
+            if (!this.rememberMessage(messageKey)) {
+                continue;
+            }
+            if (!tryClaimInboundMessage(buildScopedMessageClaimKey(account.accountId, messageKey))) {
+                continue;
+            }
             const { attachments, failureLines } = await this.downloadInboundAttachments(extracted.attachments, rawMessage);
             const text = appendInboundAttachmentFailureText(extracted.text, failureLines);
             messages.push({

package/package.json

@@ -1,100 +1,120 @@
-{
-  "name": "cli-wechat-bridge",
-  "version": "1.1.1",
-  "description": "CLI WeChat Bridge for Codex, Claude Code, OpenCode, and PowerShell over WeChat",
-  "type": "module",
-  "files": [
-    "bin/",
-    "dist/",
-    "README.md",
-    "LICENSE.txt"
-  ],
-  "keywords": [
-    "wechat",
-    "cli",
-    "bridge",
-    "codex",
-    "claude",
-    "opencode",
-    "terminal"
-  ],
-  "author": "UNLINEARITY",
-  "license": "AGPL-3.0-or-later",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/UNLINEARITY/CLI-WeChat-Bridge.git"
-  },
-  "bugs": {
-    "url": "https://github.com/UNLINEARITY/CLI-WeChat-Bridge/issues"
-  },
-  "homepage": "https://github.com/UNLINEARITY/CLI-WeChat-Bridge#readme",
-  "publishConfig": {
-    "access": "public"
-  },
-  "bin": {
-    "wechat-bridge": "bin/wechat-bridge.mjs",
-    "wechat-bridge-codex": "bin/wechat-bridge-codex.mjs",
-    "wechat-codex": "bin/wechat-codex.mjs",
-    "wechat-codex-start": "bin/wechat-codex-start.mjs",
-    "wechat-claude": "bin/wechat-claude.mjs",
-    "wechat-claude-start": "bin/wechat-claude-start.mjs",
-    "wechat-bridge-claude": "bin/wechat-bridge-claude.mjs",
-    "wechat-bridge-shell": "bin/wechat-bridge-shell.mjs",
-    "wechat-bridge-opencode": "bin/wechat-bridge-opencode.mjs",
-    "wechat-opencode": "bin/wechat-opencode.mjs",
-    "wechat-opencode-start": "bin/wechat-opencode-start.mjs",
-    "wechat-daemon": "bin/wechat-daemon.mjs",
-    "wechat-setup": "bin/wechat-setup.mjs",
-    "wechat-check-update": "bin/wechat-check-update.mjs"
-  },
-  "scripts": {
-    "clean": "node -e \"require('node:fs').rmSync('dist', { recursive: true, force: true })\"",
-    "build": "npm run clean && tsc -p tsconfig.build.json",
-    "prepack": "npm run build",
-    "lint": "eslint bin src test",
-    "lint:fix": "eslint bin src test --fix",
-    "typecheck:src": "tsc -p tsconfig.typecheck.src.json --noEmit --pretty false",
-    "quality": "npm run lint && npm run typecheck:src && bun test test && npm run build",
-    "smoke:global": "node scripts/smoke-global-install.mjs",
-    "publish:dual": "node scripts/publish-dual.mjs",
-    "setup": "node --no-warnings --experimental-strip-types src/wechat/setup.ts",
-    "start": "node --no-warnings --experimental-strip-types src/wechat/wechat-channel.ts",
-    "check": "node --no-warnings --experimental-strip-types src/wechat/wechat-channel.ts --check",
-    "bridge": "node --no-warnings --experimental-strip-types src/bridge/wechat-bridge.ts",
-    "daemon": "node --no-warnings --experimental-strip-types src/daemon/wechat-daemon.ts",
-    "bridge:codex": "node --no-warnings --experimental-strip-types src/bridge/wechat-bridge.ts --adapter codex",
-    "codex:panel": "node --no-warnings --experimental-strip-types src/companion/codex-remote-client.ts",
-    "codex:start": "node --no-warnings --experimental-strip-types src/companion/local-companion-start.ts",
-    "claude:start": "node --no-warnings --experimental-strip-types src/companion/local-companion-start.ts --adapter claude",
-    "claude:companion": "node --no-warnings --experimental-strip-types src/companion/local-companion.ts --adapter claude",
-    "bridge:claude": "node --no-warnings --experimental-strip-types src/bridge/wechat-bridge.ts --adapter claude",
-    "bridge:shell": "node --no-warnings --experimental-strip-types src/bridge/wechat-bridge.ts --adapter shell",
-    "bridge:opencode": "node --no-warnings --experimental-strip-types src/bridge/wechat-bridge.ts --adapter opencode",
-    "opencode:panel": "node --no-warnings --experimental-strip-types src/companion/local-companion.ts --adapter opencode",
-    "opencode:start": "node --no-warnings --experimental-strip-types src/companion/local-companion-start.ts --adapter opencode",
-    "bridge:bun": "bun src/bridge/wechat-bridge.ts",
-    "test": "bun test test",
-    "test:bridge": "bun test test/bridge",
-    "test:companion": "bun test test/companion",
-    "test:wechat": "bun test test/wechat",
-    "test:watch": "bun test --watch test"
-  },
-  "dependencies": {
-    "@modelcontextprotocol/sdk": "^1.27.1",
-    "@opencode-ai/sdk": "^1.3.2",
-    "node-pty": "^1.1.0",
-    "qrcode-terminal": "^0.12.0"
-  },
-  "devDependencies": {
-    "@eslint/js": "^10.0.1",
-    "@types/bun": "latest",
-    "@types/node": "^24.12.4",
-    "eslint": "^10.3.0",
-    "typescript": "^6.0.2",
-    "typescript-eslint": "^8.59.3"
-  },
-  "engines": {
-    "bun": ">=1.0.0",
-    "node": ">=24.0.0"
-  }
-}
+{
+  "name": "cli-wechat-bridge",
+  "version": "1.1.3",
+  "description": "CLI WeChat Bridge for Codex, Claude Code, OpenCode, Pi, and PowerShell over WeChat",
+  "type": "module",
+  "files": [
+    "bin/",
+    "dist/",
+    "scripts/ensure-node-pty-permissions.mjs",
+    "README.md",
+    "LICENSE.txt"
+  ],
+  "keywords": [
+    "wechat",
+    "cli",
+    "bridge",
+    "codex",
+    "claude",
+    "opencode",
+    "pi",
+    "terminal"
+  ],
+  "author": "UNLINEARITY",
+  "license": "AGPL-3.0-or-later",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/UNLINEARITY/CLI-WeChat-Bridge.git"
+  },
+  "bugs": {
+    "url": "https://github.com/UNLINEARITY/CLI-WeChat-Bridge/issues"
+  },
+  "homepage": "https://github.com/UNLINEARITY/CLI-WeChat-Bridge#readme",
+  "publishConfig": {
+    "access": "public"
+  },
+  "bin": {
+    "wechat-bridge": "bin/wechat-bridge.mjs",
+    "wechat-bridge-codex": "bin/wechat-bridge-codex.mjs",
+    "wechat-codex": "bin/wechat-codex.mjs",
+    "wechat-codex-start": "bin/wechat-codex-start.mjs",
+    "wechat-claude": "bin/wechat-claude.mjs",
+    "wechat-claude-start": "bin/wechat-claude-start.mjs",
+    "wechat-bridge-claude": "bin/wechat-bridge-claude.mjs",
+    "wechat-bridge-shell": "bin/wechat-bridge-shell.mjs",
+    "wechat-bridge-opencode": "bin/wechat-bridge-opencode.mjs",
+    "wechat-opencode": "bin/wechat-opencode.mjs",
+    "wechat-opencode-start": "bin/wechat-opencode-start.mjs",
+    "wechat-bridge-pi": "bin/wechat-bridge-pi.mjs",
+    "wechat-pi": "bin/wechat-pi.mjs",
+    "wechat-pi-start": "bin/wechat-pi-start.mjs",
+    "wechat-daemon": "bin/wechat-daemon.mjs",
+    "wechat-setup": "bin/wechat-setup.mjs",
+    "wechat-check-update": "bin/wechat-check-update.mjs"
+  },
+  "scripts": {
+    "postinstall": "node scripts/ensure-node-pty-permissions.mjs",
+    "clean": "node -e \"require('node:fs').rmSync('dist', { recursive: true, force: true })\"",
+    "build": "npm run clean && tsc -p tsconfig.build.json",
+    "prepack": "npm run build",
+    "lint": "eslint bin src test",
+    "lint:fix": "eslint bin src test --fix",
+    "typecheck:src": "tsc -p tsconfig.typecheck.src.json --noEmit --pretty false",
+    "quality": "npm run lint && npm run typecheck:src && bun test test && npm run build",
+    "smoke:global": "node scripts/smoke-global-install.mjs",
+    "publish:dual": "node scripts/publish-dual.mjs",
+    "setup": "node --no-warnings --experimental-strip-types src/wechat/setup.ts",
+    "start": "node --no-warnings --experimental-strip-types src/wechat/wechat-channel.ts",
+    "check": "node --no-warnings --experimental-strip-types src/wechat/wechat-channel.ts --check",
+    "bridge": "node --no-warnings --experimental-strip-types src/bridge/wechat-bridge.ts",
+    "daemon": "node --no-warnings --experimental-strip-types src/daemon/wechat-daemon.ts",
+    "bridge:codex": "node --no-warnings --experimental-strip-types src/bridge/wechat-bridge.ts --adapter codex",
+    "codex:panel": "node --no-warnings --experimental-strip-types src/companion/codex-remote-client.ts",
+    "codex:start": "node --no-warnings --experimental-strip-types src/companion/local-companion-start.ts",
+    "claude:start": "node --no-warnings --experimental-strip-types src/companion/local-companion-start.ts --adapter claude",
+    "claude:companion": "node --no-warnings --experimental-strip-types src/companion/local-companion.ts --adapter claude",
+    "bridge:claude": "node --no-warnings --experimental-strip-types src/bridge/wechat-bridge.ts --adapter claude",
+    "bridge:shell": "node --no-warnings --experimental-strip-types src/bridge/wechat-bridge.ts --adapter shell",
+    "bridge:opencode": "node --no-warnings --experimental-strip-types src/bridge/wechat-bridge.ts --adapter opencode",
+    "opencode:panel": "node --no-warnings --experimental-strip-types src/companion/local-companion.ts --adapter opencode",
+    "opencode:start": "node --no-warnings --experimental-strip-types src/companion/local-companion-start.ts --adapter opencode",
+    "bridge:pi": "node --no-warnings --experimental-strip-types src/bridge/wechat-bridge.ts --adapter pi",
+    "pi:companion": "node --no-warnings --experimental-strip-types src/companion/local-companion.ts --adapter pi",
+    "pi:start": "node --no-warnings --experimental-strip-types src/companion/local-companion-start.ts --adapter pi",
+    "bridge:bun": "bun src/bridge/wechat-bridge.ts",
+    "test": "bun test test",
+    "test:bridge": "bun test test/bridge",
+    "test:companion": "bun test test/companion",
+    "test:wechat": "bun test test/wechat",
+    "test:watch": "bun test --watch test"
+  },
+  "dependencies": {
+    "@modelcontextprotocol/sdk": "^1.30.0",
+    "@opencode-ai/sdk": "^1.18.0",
+    "node-pty": "^1.1.0",
+    "qrcode-terminal": "^0.12.0"
+  },
+  "overrides": {
+    "@hono/node-server": "2.1.1",
+    "body-parser": "2.3.0",
+    "brace-expansion": "5.0.9",
+    "express-rate-limit": "8.6.2",
+    "fast-uri": "3.1.5",
+    "hono": "4.13.3",
+    "ip-address": "10.5.0",
+    "path-to-regexp": "8.4.2",
+    "qs": "6.15.3"
+  },
+  "devDependencies": {
+    "@eslint/js": "^10.0.1",
+    "@types/bun": "latest",
+    "@types/node": "^24.12.4",
+    "eslint": "^10.3.0",
+    "typescript": "^6.0.2",
+    "typescript-eslint": "^8.59.3"
+  },
+  "engines": {
+    "bun": ">=1.0.0",
+    "node": ">=24.0.0"
+  }
+}

package/scripts/ensure-node-pty-permissions.mjs

@@ -0,0 +1,73 @@
+#!/usr/bin/env node
+
+// Ensure the bundled node-pty `spawn-helper` is executable.
+//
+// node-pty ships its prebuilt `spawn-helper` binaries with mode 0644 (no exec
+// bit) in the published tarball. On macOS/Linux that makes `posix_spawnp`
+// fail when node-pty tries to fork a PTY, so the bridge silently drops to the
+// non-PTY fallback. The Claude adapter relies on PTY interactive mode, so the
+// fallback surfaces to users as:
+//
+//   Error: Input must be provided ... when using --print
+//
+// Restoring the exec bit after install fixes PTY spawning. This runs on
+// postinstall and is idempotent; it never fails the install.
+
+import fs from "node:fs";
+import path from "node:path";
+import { createRequire } from "node:module";
+
+if (process.platform === "win32") {
+  process.exit(0);
+}
+
+const require = createRequire(import.meta.url);
+
+let nodePtyDir;
+try {
+  nodePtyDir = path.dirname(require.resolve("node-pty/package.json"));
+} catch {
+  // node-pty is optional in unsupported environments; nothing to fix.
+  process.exit(0);
+}
+
+const candidates = [
+  "prebuilds/darwin-arm64/spawn-helper",
+  "prebuilds/darwin-x64/spawn-helper",
+  "prebuilds/linux-x64/spawn-helper",
+  "prebuilds/linux-arm64/spawn-helper",
+  "build/Release/spawn-helper",
+];
+
+let fixed = 0;
+for (const relative of candidates) {
+  const helperPath = path.join(nodePtyDir, relative);
+  let stats;
+  try {
+    stats = fs.statSync(helperPath);
+  } catch {
+    continue;
+  }
+  if (!stats.isFile()) {
+    continue;
+  }
+  if ((stats.mode & 0o111) !== 0) {
+    continue;
+  }
+  try {
+    fs.chmodSync(helperPath, stats.mode | 0o755);
+    fixed += 1;
+  } catch (error) {
+    console.warn(
+      `[cli-wechat-bridge] Could not make node-pty spawn-helper executable: ${helperPath}\n` +
+        `  ${error instanceof Error ? error.message : String(error)}\n` +
+        "  PTY mode may be unavailable; run --doctor for details.",
+    );
+  }
+}
+
+if (fixed > 0) {
+  console.log(
+    `[cli-wechat-bridge] Restored executable bit on ${fixed} node-pty spawn-helper binary(ies).`,
+  );
+}