fix(qa): accept pnpm separator for lab up (#96246)

vincentkoc · web-flow · commit d42b86421969 · 2026-06-24T10:22:56.000+08:00
diff --git a/scripts/qa-lab-up.ts b/scripts/qa-lab-up.ts
@@ -33,8 +33,9 @@ Options:
 }
 
 function parseQaLabUpArgs(argv: readonly string[]) {
+  const args = argv[0] === "--" ? argv.slice(1) : argv;
   return parseArgs({
-    args: [...argv],
+    args: [...args],
     options,
     allowPositionals: false,
   }).values;
diff --git a/test/scripts/qa-lab-up.test.ts b/test/scripts/qa-lab-up.test.ts
@@ -34,6 +34,19 @@ describe("scripts/qa-lab-up", () => {
     );
   });
 
+  it("accepts the pnpm run argument separator", async () => {
+    const runQaDockerUpCommand = vi.fn(async () => {});
+    const loadRuntime = vi.fn(async () => ({ runQaDockerUpCommand }));
+
+    await expect(
+      qaLabUpTesting.runQaLabUp(["--", "--gateway-port", "4100"], { loadRuntime }),
+    ).resolves.toBe(0);
+
+    expect(runQaDockerUpCommand).toHaveBeenCalledWith(
+      expect.objectContaining({ gatewayPort: 4100 }),
+    );
+  });
+
   it("accepts the maximum TCP port before loading the Docker runtime", async () => {
     const runQaDockerUpCommand = vi.fn(async () => {});
     const loadRuntime = vi.fn(async () => ({ runQaDockerUpCommand }));
