@cccarv82/freya 3.8.1 → 3.8.2

package/cli/retroactive-ingest.js

@@ -34,14 +34,16 @@ function run(cmd, args, cwd, extraEnv) {
     const env = extraEnv ? { ...process.env, ...extraEnv } : process.env;
     try {
       if (process.platform === 'win32') {
-        const escapedArgs = args.map(a => {
-          const escaped = String(a).replace(/'/g, "''");
-          return `'${escaped}'`;
-        });
-        const psCommand = `& '${cmd}' ${escapedArgs.join(' ')}`;
-        child = spawn('powershell.exe', [
-          '-NoLogo', '-Command', psCommand
-        ], { cwd, env, windowsHide: true });
+        // If we have a full path to the executable, use it directly with spawn
+        // (no shell needed — avoids PowerShell argument mangling).
+        // Only fall back to cmd.exe for bare command names.
+        if (path.isAbsolute(cmd) || cmd.includes('\\') || cmd.includes('/')) {
+          child = spawn(cmd, args, { cwd, env, windowsHide: true });
+        } else {
+          // Bare command name — use cmd.exe to resolve from PATH
+          const comspec = process.env.ComSpec || 'cmd.exe';
+          child = spawn(comspec, ['/d', '/s', '/c', cmd, ...args], { cwd, shell: false, env });
+        }
       } else {
         child = spawn(cmd, args, { cwd, shell: true, env });
       }

package/cli/web.js

@@ -971,17 +971,16 @@ function run(cmd, args, cwd, extraEnv, stdinData) {
 
     try {
       if (process.platform === 'win32') {
-        // Use PowerShell — fnm/nvm set PATH only in PowerShell profile,
-        // so cmd.exe (shell:true) can't find npm global .cmd shims like copilot.
-        // Do NOT use -NoProfile: the profile is what loads fnm/nvm PATH entries.
-        const escapedArgs = args.map(a => {
-          const escaped = String(a).replace(/'/g, "''");
-          return `'${escaped}'`;
-        });
-        const psCommand = `& '${cmd}' ${escapedArgs.join(' ')}`;
-        child = spawn('powershell.exe', [
-          '-NoLogo', '-Command', psCommand
-        ], { cwd, env, windowsHide: true });
+        // getCopilotCmd() resolves the full path via PowerShell/where.
+        // Once we have the full path, spawn directly — no shell needed.
+        // This avoids PowerShell argument mangling issues.
+        if (path.isAbsolute(cmd) || cmd.includes('\\') || cmd.includes('/')) {
+          child = spawn(cmd, args, { cwd, env, windowsHide: true });
+        } else {
+          // Bare command name — use cmd.exe to resolve from PATH
+          const comspec = process.env.ComSpec || 'cmd.exe';
+          child = spawn(comspec, ['/d', '/s', '/c', cmd, ...args], { cwd, shell: false, env });
+        }
       } else {
         child = spawn(cmd, args, { cwd, shell: true, env });
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cccarv82/freya",
-  "version": "3.8.1",
+  "version": "3.8.2",
   "description": "Personal AI Assistant with local-first persistence",
   "scripts": {
     "health": "node scripts/validate-data.js && node scripts/validate-structure.js",