@rely-ai/caliber 1.48.0 → 1.48.2

package/dist/bin.js

@@ -276,6 +276,21 @@ var init_resolve_caliber = __esm({
   }
 });
 
+// src/utils/windows.ts
+function quoteForWindows(arg) {
+  if (!arg) return '""';
+  if (!/[ \t\n\v"]/.test(arg)) return arg;
+  return '"' + arg.replace(/(\\*)"/g, '$1$1\\"').replace(/(\\+)$/, "$1$1") + '"';
+}
+function bashPath(p) {
+  return p.replace(/\\/g, "/");
+}
+var init_windows = __esm({
+  "src/utils/windows.ts"() {
+    "use strict";
+  }
+});
+
 // src/llm/types.ts
 var types_exports = {};
 __export(types_exports, {
@@ -961,10 +976,20 @@ function openDiffsInEditor(editor, files) {
     try {
       const leftPath = file.originalPath ?? getEmptyFilePath(file.proposedPath);
       if (IS_WINDOWS5) {
-        const quote = (s) => `"${s}"`;
-        spawn4([cmd, "--diff", quote(leftPath), quote(file.proposedPath)].join(" "), { shell: true, stdio: "ignore", detached: true }).unref();
+        spawn4(
+          [
+            quoteForWindows(cmd),
+            "--diff",
+            quoteForWindows(leftPath),
+            quoteForWindows(file.proposedPath)
+          ].join(" "),
+          { shell: true, stdio: "ignore", detached: true }
+        ).unref();
       } else {
-        spawn4(cmd, ["--diff", leftPath, file.proposedPath], { stdio: "ignore", detached: true }).unref();
+        spawn4(cmd, ["--diff", leftPath, file.proposedPath], {
+          stdio: "ignore",
+          detached: true
+        }).unref();
       }
     } catch {
       continue;
@@ -975,6 +1000,7 @@ var IS_WINDOWS5, DIFF_TEMP_DIR;
 var init_editor = __esm({
   "src/utils/editor.ts"() {
     "use strict";
+    init_windows();
     IS_WINDOWS5 = process.platform === "win32";
     DIFF_TEMP_DIR = path25.join(os6.tmpdir(), "caliber-diff");
   }
@@ -2622,12 +2648,8 @@ function estimateTokens(text) {
   return Math.ceil(text.length / 4);
 }
 
-// src/utils/windows.ts
-function quoteForWindows(arg) {
-  if (!arg) return '""';
-  if (!/[ \t\n\v"]/.test(arg)) return arg;
-  return '"' + arg.replace(/(\\*)"/g, '$1$1\\"').replace(/(\\+)$/, "$1$1") + '"';
-}
+// src/llm/cursor-acp.ts
+init_windows();
 
 // src/lib/subprocess-sentinel.ts
 var CALIBER_SUBPROCESS_ENV = "CALIBER_SUBPROCESS";
@@ -2977,6 +2999,7 @@ function isCursorLoggedIn() {
 // src/llm/claude-cli.ts
 import fs7 from "fs";
 import { spawn as spawn2, execSync as execSync7, execFileSync as execFileSync2 } from "child_process";
+init_windows();
 var DEFAULT_TIMEOUT_MS3 = 10 * 60 * 1e3;
 var IS_WINDOWS2 = process.platform === "win32";
 function candidateClaudePaths() {
@@ -3027,12 +3050,18 @@ function cleanClaudeEnv() {
 function spawnClaude(args) {
   const bin = resolveClaudeBin();
   const env = withCaliberSubprocessEnv(cleanClaudeEnv());
-  return IS_WINDOWS2 ? spawn2([bin, ...args].join(" "), {
-    cwd: process.cwd(),
-    stdio: ["pipe", "pipe", "pipe"],
-    env,
-    shell: true
-  }) : spawn2(bin, args, {
+  return IS_WINDOWS2 ? (
+    // Windows path: shell:true skips Node's exe quoting, so paths like
+    // `C:\Program Files\caliber\claude.cmd` (with spaces) would be parsed by
+    // cmd.exe as multiple words. Quote each piece explicitly. Mirrors the
+    // pattern used in cursor-acp.ts and the learn-finalize background spawn.
+    spawn2([quoteForWindows(bin), ...args.map(quoteForWindows)].join(" "), {
+      cwd: process.cwd(),
+      stdio: ["pipe", "pipe", "pipe"],
+      env,
+      shell: true
+    })
+  ) : spawn2(bin, args, {
     cwd: process.cwd(),
     stdio: ["pipe", "pipe", "pipe"],
     env
@@ -4607,6 +4636,7 @@ function getCursorConfigDir() {
 
 // src/lib/hooks.ts
 init_resolve_caliber();
+init_windows();
 import fs11 from "fs";
 import path10 from "path";
 import { execSync as execSync10 } from "child_process";
@@ -4788,28 +4818,58 @@ var installNotificationHook = notificationHook.install;
 var removeNotificationHook = notificationHook.remove;
 var PRECOMMIT_START = "# caliber:pre-commit:start";
 var PRECOMMIT_END = "# caliber:pre-commit:end";
+function tryWindowsDirectNodeInvocation(cmd) {
+  if (process.platform !== "win32") return null;
+  if (!/\.cmd$/i.test(cmd)) return null;
+  const npmDir = path10.dirname(cmd);
+  const binJs = path10.join(npmDir, "node_modules", "@rely-ai", "caliber", "dist", "bin.js");
+  if (!fs11.existsSync(binJs)) return null;
+  let nodePath;
+  try {
+    const out = execSync10("where node", {
+      encoding: "utf-8",
+      stdio: ["pipe", "pipe", "pipe"]
+    }).trim();
+    nodePath = pickExecutable(out);
+    if (!nodePath) return null;
+  } catch {
+    return null;
+  }
+  const fwdNode = nodePath.replace(/\\/g, "/");
+  const fwdBin = binJs.replace(/\\/g, "/");
+  return `"${fwdNode}" "${fwdBin}"`;
+}
 function getPrecommitBlock() {
   const cmd = resolveCaliber();
   const npx = isNpxResolution();
   let guard;
   let invoke;
   if (npx) {
-    const npxBin = cmd.split(" ")[0];
-    if (npxBin.startsWith("/")) {
+    const npxBinRaw = cmd.split(" ")[0];
+    const npxBin = bashPath(npxBinRaw);
+    if (path10.isAbsolute(npxBinRaw)) {
       guard = `[ -x "${npxBin}" ]`;
-      const npxArgs = cmd.slice(npxBin.length);
+      const npxArgs = cmd.slice(npxBinRaw.length);
       invoke = `"${npxBin}"${npxArgs}`;
     } else {
       guard = "command -v npx >/dev/null 2>&1";
       invoke = cmd;
     }
   } else {
-    if (cmd.startsWith("/")) {
-      guard = `[ -x "${cmd}" ]`;
+    const directNode = tryWindowsDirectNodeInvocation(cmd);
+    if (directNode) {
+      const nodeBin = directNode.match(/^"([^"]+)"/)?.[1] ?? "";
+      guard = `[ -x "${nodeBin}" ]`;
+      invoke = directNode;
     } else {
-      guard = `[ -x "${cmd}" ] || command -v "${cmd}" >/dev/null 2>&1`;
+      const cmdBash = bashPath(cmd);
+      if (path10.isAbsolute(cmd)) {
+        guard = `[ -x "${cmdBash}" ]`;
+      } else {
+        guard = `[ -x "${cmdBash}" ] || command -v "${cmdBash}" >/dev/null 2>&1`;
+      }
+      invoke = `"${cmdBash}"`;
     }
-    invoke = `"${cmd}"`;
   }
   return `${PRECOMMIT_START}
 if ${guard}; then
@@ -13454,6 +13514,7 @@ async function configCommand() {
 }
 
 // src/commands/learn.ts
+init_windows();
 import fs48 from "fs";
 import path39 from "path";
 import chalk23 from "chalk";
@@ -15223,10 +15284,9 @@ function parseAgentOption(value) {
     )
   ];
   if (agents.length === 0) {
-    console.error(
+    throw new Error(
       `Invalid agent "${value}". Choose from: claude, cursor, codex, opencode, github-copilot (comma-separated for multiple)`
     );
-    process.exit(1);
   }
   return agents;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rely-ai/caliber",
-  "version": "1.48.0",
+  "version": "1.48.2",
   "description": "AI context infrastructure for coding agents — keeps CLAUDE.md, Cursor rules, and skills in sync as your codebase evolves",
   "type": "module",
   "bin": {