handle powershell stripping commas (#10288)

* handle powershell stripping commas

* remove console log

* changelog entry

* add ai suggestion

---------

Co-authored-by: Joe Hanley <[email protected]>

Author: aalej

CHANGELOG.md

@@ -1,5 +1,6 @@
 - Added Enterprise Edition support to the Firestore emulator. Configure it by setting `firebase.json#firestore.edition` or `firebase.json#emulators.firestore.edition`.
 - Fixed an issue where functions deployments would silently fail (#6989)
+- Fixed issue where the CLI isn't able to correctly parse command arguments on PowerShell (#7506)
 - Add support for Next.js 16 middleware (`proxy.ts`/`proxy.js`) (#9631)
 - Updates the default region for new App Hosting backends to us-east4 (#10271)
 - Fix Next.js image optimization detection in client components (#10228)

src/command.spec.ts

@@ -170,6 +170,60 @@ describe("Command", () => {
       });
     });
   });
+
+  it("should handle comma separated values in 'only' options", async () => {
+    const run = command
+      .action((options) => {
+        return {
+          only: options.only,
+        };
+      })
+      .runner();
+
+    const result = await run({
+      only: "firestore,hosting,auth",
+    });
+
+    expect(result).to.deep.eq({
+      only: "firestore,hosting,auth",
+    });
+  });
+
+  it("should normalize space separated values in 'only' options", async () => {
+    const run = command
+      .action((options) => {
+        return {
+          only: options.only,
+        };
+      })
+      .runner();
+
+    const result = await run({
+      only: "firestore hosting auth",
+    });
+
+    expect(result).to.deep.eq({
+      only: "firestore,hosting,auth",
+    });
+  });
+
+  it("should normalize space and commas separated values in 'only' options", async () => {
+    const run = command
+      .action((options) => {
+        return {
+          only: options.only,
+        };
+      })
+      .runner();
+
+    const result = await run({
+      only: "firestore, hosting,  auth",
+    });
+
+    expect(result).to.deep.eq({
+      only: "firestore,hosting,auth",
+    });
+  });
 });
 
 describe("validateProjectId", () => {

src/command.ts

@@ -342,6 +342,16 @@ export class Command {
       options.configPath = getInheritedOption(options, "config");
     }
 
+    const onlyOption = getInheritedOption(options, "only");
+    if (onlyOption) {
+      // Handle cases where PowerShell replaces commas with spaces.
+      // see https://github.com/firebase/firebase-tools/issues/7506
+      options.only = onlyOption
+        .split(/[\s,]+/)
+        .filter(Boolean)
+        .join(",");
+    }
+
     try {
       options.config = Config.load(options);
     } catch (e: any) {