@flowscripter/dynamic-cli-framework 2.3.0 → 2.3.1

package/dist/src/service/spawn/DefaultSpawnService.d.ts

@@ -1,4 +1,10 @@
 import type { PrinterService, ShutdownService, SpawnOptions, SpawnResult, SpawnService } from "@flowscripter/dynamic-cli-framework-api";
+/**
+ * On Windows, batch-file shims (e.g. `npm.cmd`, `bun.cmd`) cannot be launched directly via
+ * `CreateProcess` - they must be run through `cmd.exe /c`. Resolve the command through the shell
+ * in that case; leave it untouched on other platforms.
+ */
+export declare function resolveForPlatform(command: ReadonlyArray<string>): string[];
 export default class DefaultSpawnService implements SpawnService {
     #private;
     setDependencies(printerService: PrinterService, shutdownService: ShutdownService): void;

package/dist/src/service/spawn/DefaultSpawnService.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"DefaultSpawnService.d.ts","sourceRoot":"","sources":["../../../../src/service/spawn/DefaultSpawnService.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,cAAc,EACd,eAAe,EACf,YAAY,EACZ,WAAW,EACX,YAAY,EACb,MAAM,yCAAyC,CAAC;AAOjD,MAAM,CAAC,OAAO,OAAO,mBAAoB,YAAW,YAAY;;IAIvD,eAAe,CAAC,cAAc,EAAE,cAAc,EAAE,eAAe,EAAE,eAAe,GAAG,IAAI;IAkCjF,KAAK,CAChB,OAAO,EAAE,aAAa,CAAC,MAAM,CAAC,EAC9B,OAAO,GAAE,YAAiB,GACzB,OAAO,CAAC,WAAW,CAAC;CAiExB"}
+{"version":3,"file":"DefaultSpawnService.d.ts","sourceRoot":"","sources":["../../../../src/service/spawn/DefaultSpawnService.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EACV,cAAc,EACd,eAAe,EACf,YAAY,EACZ,WAAW,EACX,YAAY,EACb,MAAM,yCAAyC,CAAC;AAOjD;;;;GAIG;AACH,wBAAgB,kBAAkB,CAAC,OAAO,EAAE,aAAa,CAAC,MAAM,CAAC,GAAG,MAAM,EAAE,CAa3E;AAED,MAAM,CAAC,OAAO,OAAO,mBAAoB,YAAW,YAAY;;IAIvD,eAAe,CAAC,cAAc,EAAE,cAAc,EAAE,eAAe,EAAE,eAAe,GAAG,IAAI;IAkCjF,KAAK,CAChB,OAAO,EAAE,aAAa,CAAC,MAAM,CAAC,EAC9B,OAAO,GAAE,YAAiB,GACzB,OAAO,CAAC,WAAW,CAAC;CAiExB"}

package/dist/src/service/spawn/DefaultSpawnService.js

@@ -1,6 +1,26 @@
+import process from "node:process";
 import getLogger from "../../util/logger.js";
 const logger = getLogger("DefaultSpawnService");
 const SHUTDOWN_GRACE_PERIOD_MS = 5000;
+/**
+ * On Windows, batch-file shims (e.g. `npm.cmd`, `bun.cmd`) cannot be launched directly via
+ * `CreateProcess` - they must be run through `cmd.exe /c`. Resolve the command through the shell
+ * in that case; leave it untouched on other platforms.
+ */
+export function resolveForPlatform(command) {
+    if (process.platform !== "win32") {
+        return [...command];
+    }
+    const [bin, ...args] = command;
+    if (bin === undefined) {
+        return [...command];
+    }
+    const resolved = Bun.which(bin) ?? bin;
+    if (!/\.(cmd|bat)$/i.test(resolved)) {
+        return [resolved, ...args];
+    }
+    return ["cmd.exe", "/d", "/s", "/c", resolved, ...args];
+}
 export default class DefaultSpawnService {
     #printerService;
     #shutdownService;
@@ -46,7 +66,7 @@ export default class DefaultSpawnService {
         }
         let proc;
         try {
-            proc = Bun.spawn([...command], {
+            proc = Bun.spawn(resolveForPlatform(command), {
                 cwd: options.cwd,
                 stdin: stdio === "wrapped" ? "ignore" : "inherit",
                 stdout: stdio === "wrapped" ? "pipe" : "inherit",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@flowscripter/dynamic-cli-framework",
-  "version": "2.3.0",
+  "version": "2.3.1",
   "description": "A framework for developing CLI applications which supports dynamic discovery and installation of new commands",
   "keywords": [
     "bun",
@@ -49,7 +49,7 @@
   },
   "dependencies": {
     "@flowscripter/dynamic-cli-framework-api": "^1.2.0",
-    "@flowscripter/dynamic-plugin-framework": "1.11.0",
+    "@flowscripter/dynamic-plugin-framework": "1.11.1",
     "emphasize": "^7.0.0",
     "fastest-levenshtein": "^1.0.16",
     "figlet": "^1.11.0",

package/src/service/spawn/DefaultSpawnService.ts

@@ -1,3 +1,4 @@
+import process from "node:process";
 import type {
   PrinterService,
   ShutdownService,
@@ -11,6 +12,26 @@ const logger = getLogger("DefaultSpawnService");
 
 const SHUTDOWN_GRACE_PERIOD_MS = 5000;
 
+/**
+ * On Windows, batch-file shims (e.g. `npm.cmd`, `bun.cmd`) cannot be launched directly via
+ * `CreateProcess` - they must be run through `cmd.exe /c`. Resolve the command through the shell
+ * in that case; leave it untouched on other platforms.
+ */
+export function resolveForPlatform(command: ReadonlyArray<string>): string[] {
+  if (process.platform !== "win32") {
+    return [...command];
+  }
+  const [bin, ...args] = command;
+  if (bin === undefined) {
+    return [...command];
+  }
+  const resolved = Bun.which(bin) ?? bin;
+  if (!/\.(cmd|bat)$/i.test(resolved)) {
+    return [resolved, ...args];
+  }
+  return ["cmd.exe", "/d", "/s", "/c", resolved, ...args];
+}
+
 export default class DefaultSpawnService implements SpawnService {
   #printerService: PrinterService | undefined;
   #shutdownService: ShutdownService | undefined;
@@ -68,7 +89,7 @@ export default class DefaultSpawnService implements SpawnService {
 
     let proc;
     try {
-      proc = Bun.spawn([...command], {
+      proc = Bun.spawn(resolveForPlatform(command), {
         cwd: options.cwd,
         stdin: stdio === "wrapped" ? "ignore" : "inherit",
         stdout: stdio === "wrapped" ? "pipe" : "inherit",