bun-workspaces 1.1.1 → 1.1.2

package/README.md

@@ -65,9 +65,6 @@ bw run lint my-workspace # Run for a single workspace
 bw run lint my-workspace-a my-workspace-b # Run for multiple workspaces
 bw run lint my-alias-a my-alias-b # Run by alias (set by optional config)
 
-bw run lint "my-workspace-*" # Run for matching workspace names
-bw run lint "alias:my-alias-pattern-*" "path:my-glob/**/*" # Use matching specifiers
-
 # A workspace's script will wait until any workspaces it depends on have completed
 # Similar to Bun's --filter behavior
 bw run lint --dep-order
@@ -75,6 +72,9 @@ bw run lint --dep-order
 # Continue running scripts even if a dependency fails
 bw run lint --dep-order --ignore-dep-failure
 
+bw run lint "my-workspace-*" # Run for matching workspace names
+bw run lint "alias:my-alias-pattern-*" "path:my-glob/**/*" # Use matching specifiers
+
 bw run lint --args="--my-appended-args" # Add args to each script call
 bw run lint --args="--my-arg=<workspaceName>" # Use the workspace name in args
 
@@ -140,14 +140,22 @@ const runSingleScript = async () => {
   const { output, exit } = project.runWorkspaceScript({
     workspaceNameOrAlias: "my-workspace",
     script: "my-script",
-    args: "--my --appended --args", // optional, arguments to add to the command
+
+    // Optional. Arguments to add to the command
+    // Can be a string or an array of strings
+    // If string, the argv will be parsed POSIX-style
+    args: ["--my", "--appended", "--args"],
+
+    // Optional. Whether to ignore all output from the script.
+    // This saves memory when you don't need script output.
+    ignoreOutput: false,
   });
 
   // Get a stream of the script subprocess's output
   for await (const { chunk, metadata } of output.text()) {
-    // console.log(chunk); // the content (string)
-    // console.log(metadata.streamName); // "stdout" or "stderr"
-    // console.log(metadata.workspace); // the workspace that the output came from
+    // console.log(chunk); // The output chunk's content (string)
+    // console.log(metadata.streamName); // The output stream, "stdout" or "stderr"
+    // console.log(metadata.workspace); // The target Workspace
   }
 
   // Get data about the script execution after it exits
@@ -173,8 +181,8 @@ const runManyScripts = async () => {
     // Required. The package.json "scripts" field name to run
     script: "my-script",
 
-    // Optional. Arguments to add to the command
-    args: "--my --appended --args",
+    // Optional. Arguments to add to the command (same as for runWorkspaceScript)
+    args: ["--my", "--appended", "--args"],
 
     // Optional. Whether to run the scripts in parallel (default: true)
     parallel: true,
@@ -187,6 +195,10 @@ const runManyScripts = async () => {
     // continue running scripts even if a dependency fails
     ignoreDependencyFailure: false,
 
+    // Optional. Whether to ignore all output from the scripts.
+    // This saves memory when you don't need script output.
+    ignoreOutput: false,
+
     // Optional, callback when script starts, skips, or exits
     onScriptEvent: (event, { workspace, exitResult }) => {
       // event: "start", "skip", "exit"
@@ -195,9 +207,9 @@ const runManyScripts = async () => {
 
   // Get a stream of script output
   for await (const { chunk, metadata } of output.text()) {
-    // console.log(chunk); // the content (string)
+    // console.log(chunk); // the output chunk's content (string)
     // console.log(metadata.streamName); // "stdout" or "stderr"
-    // console.log(metadata.workspace); // the workspace that the output came from
+    // console.log(metadata.workspace); // the Workspace that the output came from
   }
 
   // Get final summary data and script exit details after all scripts have completed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bun-workspaces",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "description": "A monorepo management tool for Bun, with a CLI and API to enhance Bun's native workspaces.",
   "license": "MIT",
   "main": "src/index.mjs",
@@ -32,6 +32,7 @@
     }
   },
   "dependencies": {
-    "commander": "^12.1.0"
+    "commander": "^12.1.0",
+    "shell-quote": "^1.8.3"
   }
 }

package/src/cli/commands/runScript/handleRunScript.mjs

@@ -58,7 +58,7 @@ const runScript = handleProjectCommand(
       process.exit(1);
     }
     const scriptArgs = postTerminatorArgs.length
-      ? postTerminatorArgs.join(" ")
+      ? postTerminatorArgs
       : options.args;
     if (positionalWorkspacePatterns.length && options.workspacePatterns) {
       logger.error(

package/src/project/implementations/fileSystemProject.d.ts

@@ -39,8 +39,8 @@ export type RunWorkspaceScriptOptions = {
   script: string;
   /** Whether to run the script as an inline command */
   inline?: boolean | InlineScriptOptions;
-  /** The arguments to append to the script command */
-  args?: string;
+  /** The arguments to append to the script command. If passed as a string, the argv will be parsed POSIX-style */
+  args?: string | string[];
   /** Set to `true` to ignore all output from the script. This saves memory when you don't need script output. */
   ignoreOutput?: boolean;
 };
@@ -89,8 +89,8 @@ export type RunScriptAcrossWorkspacesOptions = {
   script: string;
   /** Whether to run the script as an inline command */
   inline?: boolean | InlineScriptOptions;
-  /** The arguments to append to the script command. `<workspaceName>` will be replaced with the workspace name */
-  args?: string;
+  /** The arguments to append to the script command. If passed as a string, the argv will be parsed POSIX-style */
+  args?: string | string[];
   /** Whether to run the scripts in parallel (default: `true`). Pass `false` to run in series. */
   parallel?: ParallelOption;
   /** When `true`, run scripts so that dependent workspaces run only after their dependencies */

package/src/project/implementations/fileSystemProject.mjs

@@ -1,11 +1,15 @@
 import fs from "fs";
 import path from "path";
+import { parse, quote } from "shell-quote/";
 import { loadRootConfig } from "../../config/index.mjs";
 import { getUserEnvVar } from "../../config/userEnvVars/index.mjs";
 import {
   DEFAULT_TEMP_DIR,
+  IS_WINDOWS,
+  InvalidJSTypeError,
   expandHomePath,
   isPlainObject,
+  validateJSArray,
   validateJSTypes,
 } from "../../internal/core/index.mjs";
 import { logger } from "../../internal/logger/index.mjs";
@@ -27,6 +31,7 @@ import {
   resolveWorkspacePath,
 } from "./projectBase.mjs"; // CONCATENATED MODULE: external "fs"
 // CONCATENATED MODULE: external "path"
+// CONCATENATED MODULE: external "shell-quote/"
 // CONCATENATED MODULE: external "../../config/index.mjs"
 // CONCATENATED MODULE: external "../../config/userEnvVars/index.mjs"
 // CONCATENATED MODULE: external "../../internal/core/index.mjs"
@@ -40,6 +45,41 @@ import {
 // CONCATENATED MODULE: external "./projectBase.mjs"
 // CONCATENATED MODULE: ./src/project/implementations/fileSystemProject.ts
 
+const quoteArg = (arg, shell) =>
+  IS_WINDOWS && shell === "system"
+    ? `"${arg.replace(/"/g, '""')}"`
+    : quote([arg]);
+const serializeArgs = (args, metadata, shell) => {
+  if (!args || args.length === 0) return "";
+  if (Array.isArray(args)) {
+    return args
+      .map((arg) =>
+        quoteArg(interpolateScriptRuntimeMetadata(arg, metadata, shell), shell),
+      )
+      .join(" ");
+  }
+  const interpolated = interpolateScriptRuntimeMetadata(args, metadata, shell);
+  // Escape backslashes in interpolated values before POSIX parse on Windows,
+  // so that path separators survive parse's escape processing (\\→\)
+  const parseInput =
+    IS_WINDOWS && shell === "system"
+      ? interpolated.replace(/\\/g, "\\\\")
+      : interpolated;
+  return parse(parseInput)
+    .flatMap((entry) => {
+      if (typeof entry === "string") {
+        return [quoteArg(entry, shell)];
+      }
+      if ("comment" in entry) {
+        return [];
+      }
+      if ("pattern" in entry) {
+        return [entry.pattern];
+      }
+      return [entry.op];
+    })
+    .join(" ");
+};
 class _FileSystemProject extends ProjectBase {
   rootDirectory;
   workspaces;
@@ -122,11 +162,6 @@ class _FileSystemProject extends ProjectBase {
           typeofName: ["boolean", "object"],
           optional: true,
         },
-        "args option": {
-          value: options.args,
-          typeofName: "string",
-          optional: true,
-        },
         "ignoreOutput option": {
           value: options.ignoreOutput,
           typeofName: "boolean",
@@ -137,6 +172,23 @@ class _FileSystemProject extends ProjectBase {
         throw: true,
       },
     );
+    if (options.args !== undefined) {
+      if (typeof options.args !== "string" && !Array.isArray(options.args)) {
+        throw new InvalidJSTypeError(
+          `Type error: args option expects type string | string[], received ${typeof options.args}`,
+        );
+      }
+      if (Array.isArray(options.args)) {
+        const argsError = validateJSArray({
+          value: options.args,
+          valueLabel: "args option",
+          itemOptions: {
+            typeofName: "string",
+          },
+        });
+        if (argsError) throw argsError;
+      }
+    }
     if (isPlainObject(options.inline)) {
       validateJSTypes(
         {
@@ -185,11 +237,7 @@ class _FileSystemProject extends ProjectBase {
       workspaceName: workspace.name,
       scriptName: options.inline ? inlineScriptName : options.script,
     };
-    const args = interpolateScriptRuntimeMetadata(
-      options.args ?? "",
-      scriptRuntimeMetadata,
-      shell,
-    );
+    const args = serializeArgs(options.args, scriptRuntimeMetadata, shell);
     const script = options.inline
       ? interpolateScriptRuntimeMetadata(
           options.script,
@@ -243,11 +291,6 @@ class _FileSystemProject extends ProjectBase {
           typeofName: ["boolean", "object"],
           optional: true,
         },
-        "args option": {
-          value: options.args,
-          typeofName: "string",
-          optional: true,
-        },
         "parallel option": {
           value: options.parallel,
           typeofName: ["boolean", "object"],
@@ -297,6 +340,23 @@ class _FileSystemProject extends ProjectBase {
         },
       );
     }
+    if (options.args !== undefined) {
+      if (typeof options.args !== "string" && !Array.isArray(options.args)) {
+        throw new InvalidJSTypeError(
+          `Type error: args option expects type string | string[], received ${typeof options.args}`,
+        );
+      }
+      if (Array.isArray(options.args)) {
+        const argsError = validateJSArray({
+          value: options.args,
+          valueLabel: "args option",
+          itemOptions: {
+            typeofName: "string",
+          },
+        });
+        if (argsError) throw argsError;
+      }
+    }
     if (isPlainObject(options.parallel)) {
       validateJSTypes(
         {
@@ -384,11 +444,7 @@ class _FileSystemProject extends ProjectBase {
           workspaceName: workspace.name,
           scriptName: options.inline ? inlineScriptName : options.script,
         };
-        const args = interpolateScriptRuntimeMetadata(
-          options.args ?? "",
-          scriptRuntimeMetadata,
-          shell,
-        );
+        const args = serializeArgs(options.args, scriptRuntimeMetadata, shell);
         const script = options.inline
           ? interpolateScriptRuntimeMetadata(
               options.script,