@open-code-review/cli 1.10.1 → 1.10.3

package/dist/dashboard/server.js

@@ -22752,7 +22752,7 @@ async function defaultBrowserId() {
   if (process4.platform !== "darwin") {
     throw new Error("macOS only");
   }
-  const { stdout } = await execFileAsync2("defaults", ["read", "com.apple.LaunchServices/com.apple.launchservices.secure", "LSHandlers"]);
+  const { stdout } = await execFileAsync("defaults", ["read", "com.apple.LaunchServices/com.apple.launchservices.secure", "LSHandlers"]);
   const match = /LSHandlerRoleAll = "(?!-)(?<id>[^"]+?)";\s+?LSHandlerURLScheme = (?:http|https);/.exec(stdout);
   const browserId = match?.groups.id ?? "com.apple.Safari";
   if (browserId === "com.apple.safari") {
@@ -22760,17 +22760,17 @@ async function defaultBrowserId() {
   }
   return browserId;
 }
-var execFileAsync2;
+var execFileAsync;
 var init_default_browser_id = __esm({
   "../../node_modules/.pnpm/default-browser-id@5.0.1/node_modules/default-browser-id/index.js"() {
-    execFileAsync2 = promisify2(execFile2);
+    execFileAsync = promisify2(execFile2);
   }
 });
 
 // ../../node_modules/.pnpm/run-applescript@7.1.0/node_modules/run-applescript/index.js
 import process5 from "node:process";
 import { promisify as promisify3 } from "node:util";
-import { execFile as execFile3, execFileSync as execFileSync4 } from "node:child_process";
+import { execFile as execFile3, execFileSync as execFileSync2 } from "node:child_process";
 async function runAppleScript(script, { humanReadableOutput = true, signal } = {}) {
   if (process5.platform !== "darwin") {
     throw new Error("macOS only");
@@ -22780,13 +22780,13 @@ async function runAppleScript(script, { humanReadableOutput = true, signal } = {
   if (signal) {
     execOptions.signal = signal;
   }
-  const { stdout } = await execFileAsync3("osascript", ["-e", script, outputArguments], execOptions);
+  const { stdout } = await execFileAsync2("osascript", ["-e", script, outputArguments], execOptions);
   return stdout.trim();
 }
-var execFileAsync3;
+var execFileAsync2;
 var init_run_applescript = __esm({
   "../../node_modules/.pnpm/run-applescript@7.1.0/node_modules/run-applescript/index.js"() {
-    execFileAsync3 = promisify3(execFile3);
+    execFileAsync2 = promisify3(execFile3);
   }
 });
 
@@ -22804,7 +22804,7 @@ var init_bundle_name = __esm({
 // ../../node_modules/.pnpm/default-browser@5.5.0/node_modules/default-browser/windows.js
 import { promisify as promisify4 } from "node:util";
 import { execFile as execFile4 } from "node:child_process";
-async function defaultBrowser(_execFileAsync = execFileAsync4) {
+async function defaultBrowser(_execFileAsync = execFileAsync3) {
   const { stdout } = await _execFileAsync("reg", [
     "QUERY",
     " HKEY_CURRENT_USER\\Software\\Microsoft\\Windows\\Shell\\Associations\\UrlAssociations\\http\\UserChoice",
@@ -22822,10 +22822,10 @@ async function defaultBrowser(_execFileAsync = execFileAsync4) {
   const baseIdByHyphen = hyphenIndex === -1 ? void 0 : id.slice(0, hyphenIndex);
   return windowsBrowserProgIds[id] ?? windowsBrowserProgIds[baseIdByDot] ?? windowsBrowserProgIds[baseIdByHyphen] ?? { name: id, id };
 }
-var execFileAsync4, windowsBrowserProgIds, _windowsBrowserProgIdMap, UnknownBrowserError;
+var execFileAsync3, windowsBrowserProgIds, _windowsBrowserProgIdMap, UnknownBrowserError;
 var init_windows = __esm({
   "../../node_modules/.pnpm/default-browser@5.5.0/node_modules/default-browser/windows.js"() {
-    execFileAsync4 = promisify4(execFile4);
+    execFileAsync3 = promisify4(execFile4);
     windowsBrowserProgIds = {
       MSEdgeHTM: { name: "Edge", id: "com.microsoft.edge" },
       // The missing `L` is correct.
@@ -22862,7 +22862,7 @@ async function defaultBrowser2() {
     return { name, id };
   }
   if (process6.platform === "linux") {
-    const { stdout } = await execFileAsync5("xdg-mime", ["query", "default", "x-scheme-handler/http"]);
+    const { stdout } = await execFileAsync4("xdg-mime", ["query", "default", "x-scheme-handler/http"]);
     const id = stdout.trim();
     const name = titleize(id.replace(/.desktop$/, "").replace("-", " "));
     return { name, id };
@@ -22872,13 +22872,13 @@ async function defaultBrowser2() {
   }
   throw new Error("Only macOS, Linux, and Windows are supported");
 }
-var execFileAsync5, titleize;
+var execFileAsync4, titleize;
 var init_default_browser = __esm({
   "../../node_modules/.pnpm/default-browser@5.5.0/node_modules/default-browser/index.js"() {
     init_default_browser_id();
     init_bundle_name();
     init_windows();
-    execFileAsync5 = promisify5(execFile5);
+    execFileAsync4 = promisify5(execFile5);
     titleize = (string) => string.toLowerCase().replaceAll(/(?:^|\s|-)\S/g, (x) => x.toUpperCase());
   }
 });
@@ -24901,8 +24901,35 @@ function createStatsRouter(db) {
 // src/server/routes/commands.ts
 var import_express8 = __toESM(require_express2(), 1);
 
+// ../shared/platform/src/index.ts
+import {
+  execFile,
+  execFileSync,
+  spawn
+} from "node:child_process";
+import { promisify } from "node:util";
+var execFilePromise = promisify(execFile);
+var isWindows = process.platform === "win32";
+function execBinary(binary, args, opts) {
+  return execFileSync(binary, args, {
+    ...opts,
+    shell: isWindows
+  });
+}
+async function execBinaryAsync(binary, args, opts) {
+  return execFilePromise(binary, args, {
+    ...opts,
+    shell: isWindows
+  });
+}
+function spawnBinary(binary, args, opts) {
+  return spawn(binary, args, {
+    ...opts,
+    ...isWindows && { shell: true, windowsHide: true }
+  });
+}
+
 // src/server/socket/command-runner.ts
-import { spawn as spawn3 } from "node:child_process";
 import { readFileSync as readFileSync4 } from "node:fs";
 import { dirname as dirname6, join as join8 } from "node:path";
 
@@ -24910,10 +24937,6 @@ import { dirname as dirname6, join as join8 } from "node:path";
 import { readFileSync as readFileSync3 } from "node:fs";
 import { join as join6 } from "node:path";
 
-// src/server/services/ai-cli/claude-adapter.ts
-import { execFileSync } from "node:child_process";
-import { spawn } from "node:child_process";
-
 // src/server/services/ai-cli/helpers.ts
 import { tmpdir } from "node:os";
 import { join as join5 } from "node:path";
@@ -24992,7 +25015,7 @@ var ClaudeCodeAdapter = class {
   binary = "claude";
   detect() {
     try {
-      const output = execFileSync("claude", ["--version"], {
+      const output = execBinary("claude", ["--version"], {
         encoding: "utf-8",
         timeout: 5e3,
         stdio: ["ignore", "pipe", "pipe"]
@@ -25021,7 +25044,7 @@ var ClaudeCodeAdapter = class {
     if (opts.resumeSessionId) {
       flags.push("--resume", opts.resumeSessionId);
     }
-    const proc = spawn("claude", flags, {
+    const proc = spawnBinary("claude", flags, {
       cwd: opts.cwd,
       env: cleanEnv(),
       detached: isWorkflow,
@@ -25091,7 +25114,6 @@ var ClaudeCodeAdapter = class {
 };
 
 // src/server/services/ai-cli/opencode-adapter.ts
-import { execFileSync as execFileSync2, spawn as spawn2 } from "node:child_process";
 function capitalize(s) {
   return s.charAt(0).toUpperCase() + s.slice(1);
 }
@@ -25100,7 +25122,7 @@ var OpenCodeAdapter = class {
   binary = "opencode";
   detect() {
     try {
-      const output = execFileSync2("opencode", ["--version"], {
+      const output = execBinary("opencode", ["--version"], {
         encoding: "utf-8",
         timeout: 5e3,
         stdio: ["ignore", "pipe", "pipe"]
@@ -25126,7 +25148,7 @@ var OpenCodeAdapter = class {
     if (opts.resumeSessionId) {
       args.push("--session", opts.resumeSessionId, "--continue");
     }
-    const proc = spawn2("opencode", args, {
+    const proc = spawnBinary("opencode", args, {
       cwd: opts.cwd,
       env: cleanEnv(),
       detached: isWorkflow,
@@ -25467,10 +25489,10 @@ function registerCommandHandlers(io2, socket, db, ocrDir, aiCliService) {
 function spawnCliCommand(io2, db, ocrDir, executionId, baseCommand, subArgs, entry) {
   const localCli = resolveLocalCli();
   const repoRoot = dirname6(ocrDir);
-  const proc = localCli ? spawn3("node", [localCli, baseCommand, ...subArgs], {
+  const proc = localCli ? spawnBinary("node", [localCli, baseCommand, ...subArgs], {
     cwd: repoRoot,
     env: cleanEnv()
-  }) : spawn3("ocr", [baseCommand, ...subArgs], {
+  }) : spawnBinary("ocr", [baseCommand, ...subArgs], {
     cwd: repoRoot,
     env: cleanEnv()
   });
@@ -25811,7 +25833,6 @@ function createCommandsRouter(db) {
 // src/server/routes/config.ts
 var import_express9 = __toESM(require_express2(), 1);
 import { readFileSync as readFileSync5, writeFileSync as writeFileSync3 } from "node:fs";
-import { execFileSync as execFileSync3 } from "node:child_process";
 import { join as join9, dirname as dirname7, basename } from "node:path";
 var VALID_IDES = ["vscode", "cursor", "windsurf", "jetbrains", "sublime"];
 function detectIde() {
@@ -25846,7 +25867,7 @@ function resolveIde(ocrDir) {
 }
 function detectGitBranch(cwd) {
   try {
-    return execFileSync3("git", ["rev-parse", "--abbrev-ref", "HEAD"], {
+    return execBinary("git", ["rev-parse", "--abbrev-ref", "HEAD"], {
       cwd,
       timeout: 3e3,
       encoding: "utf-8"
@@ -27891,13 +27912,10 @@ function cleanupAllChats() {
 }
 
 // src/server/socket/post-handler.ts
-import { execFile } from "node:child_process";
 import { existsSync as existsSync9, mkdirSync as mkdirSync3, readFileSync as readFileSync10, unlinkSync, writeFileSync as writeFileSync4 } from "node:fs";
 import { tmpdir as tmpdir2 } from "node:os";
 import { join as join13, dirname as dirname10, isAbsolute } from "node:path";
 import { randomUUID as randomUUID2 } from "node:crypto";
-import { promisify } from "node:util";
-var execFileAsync = promisify(execFile);
 function resolveSessionDir(sessionDir, ocrDir) {
   if (isAbsolute(sessionDir)) return sessionDir;
   return join13(dirname10(ocrDir), sessionDir);
@@ -27928,10 +27946,10 @@ async function findPrForBranch(branch, env, cwd) {
   }
   for (const candidate of candidates) {
     try {
-      const { stdout } = await execFileAsync(
+      const { stdout } = await execBinaryAsync(
         "gh",
         ["pr", "list", "--head", candidate, "--json", "number,url", "--limit", "1"],
-        { env, cwd }
+        { env, cwd, encoding: "utf-8" }
       );
       const prs = JSON.parse(stdout);
       if (prs.length > 0 && prs[0]) {
@@ -27971,7 +27989,7 @@ function registerPostHandlers(io2, socket, db, ocrDir, aiCliService) {
       const branch = session.branch;
       const repoRoot = dirname10(ocrDir);
       try {
-        await execFileAsync("gh", ["auth", "status"], { env: cleanEnv(), cwd: repoRoot });
+        await execBinaryAsync("gh", ["auth", "status"], { env: cleanEnv(), cwd: repoRoot, encoding: "utf-8" });
       } catch {
         socket.emit("post:gh-result", {
           authenticated: false,
@@ -28275,10 +28293,10 @@ function registerPostHandlers(io2, socket, db, ocrDir, aiCliService) {
         writeFileSync4(tmpFile, content, { mode: 384 });
         const repoRoot = dirname10(ocrDir);
         try {
-          const { stdout } = await execFileAsync(
+          const { stdout } = await execBinaryAsync(
             "gh",
             ["pr", "comment", String(prNumber), "--body-file", tmpFile],
-            { env: cleanEnv(), cwd: repoRoot }
+            { env: cleanEnv(), cwd: repoRoot, encoding: "utf-8" }
           );
           const urlMatch = stdout.match(/(https:\/\/github\.com\S+)/)?.[0] ?? null;
           tracker.appendOutput(`\u2713 Posted to PR #${prNumber}${urlMatch ? ` \u2014 ${urlMatch}` : ""}
@@ -28353,6 +28371,9 @@ if (process.env.NODE_ENV !== "production") {
     next();
   });
 }
+app.get("/api/health", (_req, res) => {
+  res.json({ status: "ok" });
+});
 app.use("/api", (req, res, next) => {
   const authHeader = req.headers.authorization;
   if (!authHeader || !authHeader.startsWith("Bearer ") || authHeader.slice(7) !== AUTH_TOKEN) {
@@ -28379,9 +28400,6 @@ if (process.env.NODE_ENV !== "production") {
     res.json({ token: AUTH_TOKEN });
   });
 }
-app.get("/api/health", (_req, res) => {
-  res.json({ status: "ok" });
-});
 async function startServer(options = {}) {
   const port = options.port ?? parseInt(process.env.PORT ?? "4173", 10);
   const ocrDir = resolveOcrDir();
@@ -28389,7 +28407,12 @@ async function startServer(options = {}) {
   const db = await openDb(ocrDir);
   const dataDir = join14(ocrDir, "data");
   const pidFilePath = join14(dataDir, "dashboard.pid");
+  const portFilePath = join14(dataDir, "server-port");
   mkdirSync4(dataDir, { recursive: true });
+  try {
+    unlinkSync2(portFilePath);
+  } catch {
+  }
   if (existsSync10(pidFilePath)) {
     try {
       const oldPid = parseInt(readFileSync11(pidFilePath, "utf-8").trim(), 10);
@@ -28575,7 +28598,6 @@ async function startServer(options = {}) {
   if (actualPort !== port) {
     console.log(`  Note: using port ${actualPort} (${port} was in use)`);
   }
-  const portFilePath = join14(dataDir, "server-port");
   writeFileSync5(portFilePath, String(actualPort), { mode: 384 });
   console.log(`  Server:            http://localhost:${actualPort}`);
   console.log(`  OCR directory:     ${shortenPath(ocrDir)}`);

package/dist/index.js

@@ -20894,10 +20894,29 @@ ${hint}
 }
 
 // src/lib/version.ts
-var CLI_VERSION = true ? "1.10.1" : createRequire(import.meta.url)("../../package.json").version;
+var CLI_VERSION = true ? "1.10.3" : createRequire(import.meta.url)("../../package.json").version;
+
+// ../shared/platform/src/index.ts
+import { pathToFileURL } from "node:url";
+import {
+  execFile,
+  execFileSync,
+  spawn as spawn2
+} from "node:child_process";
+import { promisify } from "node:util";
+var execFilePromise = promisify(execFile);
+var isWindows = process.platform === "win32";
+async function importModule(absolutePath) {
+  return import(pathToFileURL(absolutePath).href);
+}
+function execBinary(binary, args, opts) {
+  return execFileSync(binary, args, {
+    ...opts,
+    shell: isWindows
+  });
+}
 
 // src/lib/deps.ts
-import { execFileSync } from "node:child_process";
 var CATEGORY_ORDER = ["core", "ai-cli", "github"];
 var CATEGORY_INFO = {
   core: { label: "Core", hint: "" },
@@ -20946,7 +20965,7 @@ function parseVersion(output) {
 }
 function checkBinary(spec) {
   try {
-    const output = execFileSync(spec.binary, ["--version"], {
+    const output = execBinary(spec.binary, ["--version"], {
       encoding: "utf-8",
       timeout: 5e3,
       stdio: ["ignore", "pipe", "pipe"]
@@ -21441,7 +21460,7 @@ var STR_CLOSE = "close";
 var EMPTY_FN = () => {
 };
 var pl = process.platform;
-var isWindows = pl === "win32";
+var isWindows2 = pl === "win32";
 var isMacos = pl === "darwin";
 var isLinux = pl === "linux";
 var isFreeBSD = pl === "freebsd";
@@ -21814,7 +21833,7 @@ var setFsWatchListener = (path2, fullPath, options, handlers) => {
       const broadcastErr = fsWatchBroadcast.bind(null, fullPath, KEY_ERR);
       if (cont)
         cont.watcherUnusable = true;
-      if (isWindows && error.code === "EPERM") {
+      if (isWindows2 && error.code === "EPERM") {
         try {
           const fd = await open(path2, "r");
           await fd.close();
@@ -22587,7 +22606,7 @@ var FSWatcher = class extends EventEmitter {
     if (this.closed)
       return;
     const opts = this.options;
-    if (isWindows)
+    if (isWindows2)
       path2 = sysPath2.normalize(path2);
     if (opts.cwd)
       path2 = sysPath2.relative(opts.cwd, path2);
@@ -22956,7 +22975,7 @@ var isSharedWorker = typeof SharedWorkerGlobalScope !== "undefined" && globalThi
 var isServiceWorker = typeof ServiceWorkerGlobalScope !== "undefined" && globalThis instanceof ServiceWorkerGlobalScope;
 var platform = globalThis.navigator?.userAgentData?.platform;
 var isMacOs = platform === "macOS" || globalThis.navigator?.platform === "MacIntel" || globalThis.navigator?.userAgent?.includes(" Mac ") === true || globalThis.process?.platform === "darwin";
-var isWindows2 = platform === "Windows" || globalThis.navigator?.platform === "Win32" || globalThis.process?.platform === "win32";
+var isWindows3 = platform === "Windows" || globalThis.navigator?.platform === "Win32" || globalThis.process?.platform === "win32";
 var isLinux2 = platform === "Linux" || globalThis.navigator?.platform?.startsWith("Linux") === true || globalThis.navigator?.userAgent?.includes(" Linux ") === true || globalThis.process?.platform === "linux";
 var isIos = platform === "iOS" || globalThis.navigator?.platform === "MacIntel" && globalThis.navigator?.maxTouchPoints > 1 || /iPad|iPhone|iPod/.test(globalThis.navigator?.platform);
 var isAndroid = platform === "Android" || globalThis.navigator?.platform === "Android" || globalThis.navigator?.userAgent?.includes(" Android ") === true || globalThis.process?.platform === "android";
@@ -22967,7 +22986,7 @@ var OSC = "\x1B]";
 var BEL = "\x07";
 var SEP = ";";
 var isTerminalApp = !isBrowser && process11.env.TERM_PROGRAM === "Apple_Terminal";
-var isWindows3 = !isBrowser && process11.platform === "win32";
+var isWindows4 = !isBrowser && process11.platform === "win32";
 var isTmux = !isBrowser && (process11.env.TERM?.startsWith("screen") || process11.env.TERM?.startsWith("tmux") || process11.env.TMUX !== void 0);
 var cwdFunction = isBrowser ? () => {
   throw new Error("`process.cwd()` only works in Node.js, not the browser.");
@@ -23037,7 +23056,7 @@ var scrollDown = ESC2 + "T";
 var clearScreen = "\x1Bc";
 var clearViewport = `${eraseScreen}${ESC2}H`;
 var isOldWindows = () => {
-  if (isBrowser || !isWindows3) {
+  if (isBrowser || !isWindows4) {
     return false;
   }
   const parts = os3.release().split(".");
@@ -25869,7 +25888,7 @@ var dashboardCommand = new Command("dashboard").description("Start the OCR dashb
     console.log(source_default.bold("  OCR Dashboard"));
     console.log();
     try {
-      const { startServer } = await import(serverPath);
+      const { startServer } = await importModule(serverPath);
       await startServer({ port, open: options.open });
     } catch (err) {
       console.error(source_default.red("Error: Failed to start dashboard server."));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-code-review/cli",
-  "version": "1.10.1",
+  "version": "1.10.3",
   "description": "CLI for Open Code Review - Multi-environment setup and progress tracking",
   "type": "module",
   "bin": {
@@ -50,12 +50,13 @@
     "ora": "^8.1.1",
     "socket.io": "^4.8",
     "sql.js": "^1.14.1",
-    "@open-code-review/agents": "1.10.1"
+    "@open-code-review/agents": "1.10.3"
   },
   "publishConfig": {
     "access": "public"
   },
   "devDependencies": {
-    "@types/sql.js": "^1.4.9"
+    "@types/sql.js": "^1.4.9",
+    "@open-code-review/platform": "0.0.0"
   }
 }