chore: remove workaround for npm scripts argument parsing

Refs nodejs/node@f99aa74

Author: deepak1556

.github/workflows/pr-win32-test.yml

@@ -121,7 +121,7 @@ jobs:
         run: |
           for ($i = 1; $i -le 3; $i++) {
             try {
-              npm exec -- -- npm-run-all -lp "electron ${{ env.VSCODE_ARCH }}" "playwright-install"
+              npm exec -- npm-run-all -lp "electron ${{ env.VSCODE_ARCH }}" "playwright-install"
               break
             }
             catch {
@@ -230,19 +230,19 @@ jobs:
         if: ${{ inputs.electron_tests }}
         timeout-minutes: 20
         shell: pwsh
-        run: npm run smoketest-no-compile -- -- --tracing
+        run: npm run smoketest-no-compile -- --tracing
 
       - name: 🧪 Run smoke tests (Browser, Chromium)
         if: ${{ inputs.browser_tests }}
         timeout-minutes: 20
         shell: pwsh
-        run: npm run smoketest-no-compile -- -- --web --tracing --headless
+        run: npm run smoketest-no-compile -- --web --tracing --headless
 
       - name: 🧪 Run smoke tests (Remote)
         if: ${{ inputs.remote_tests }}
         timeout-minutes: 20
         shell: pwsh
-        run: npm run smoketest-no-compile -- -- --remote --tracing
+        run: npm run smoketest-no-compile -- --remote --tracing
 
       - name: Diagnostics after smoke test run
         shell: pwsh

build/azure-pipelines/win32/product-build-win32-test.yml

@@ -11,8 +11,7 @@ parameters:
     type: string
 
 steps:
-  # Additional "--" needed to workaround https://github.com/npm/cli/issues/7375
-  - powershell: npm exec -- -- npm-run-all -lp "electron $(VSCODE_ARCH)" "playwright-install"
+  - powershell: npm exec -- npm-run-all -lp "electron $(VSCODE_ARCH)" "playwright-install"
     env:
       GITHUB_TOKEN: "$(github-distro-mixin-password)"
     displayName: Download Electron and Playwright
@@ -22,14 +21,12 @@ steps:
     - powershell: .\scripts\test.bat --build --tfs "Unit Tests"
       displayName: 🧪 Run unit tests (Electron)
       timeoutInMinutes: 15
-    # Additional "--" needed to workaround https://github.com/npm/cli/issues/7375
-    - powershell: npm run test-node -- -- --build
+    - powershell: npm run test-node -- --build
       displayName: 🧪 Run unit tests (node.js)
       timeoutInMinutes: 15
 
   - ${{ if eq(parameters.VSCODE_RUN_BROWSER_TESTS, true) }}:
-    # Additional "--" needed to workaround https://github.com/npm/cli/issues/7375
-    - powershell: npm run test-browser-no-install -- -- --build --browser chromium --tfs "Browser Unit Tests"
+    - powershell: npm run test-browser-no-install -- --build --browser chromium --tfs "Browser Unit Tests"
       displayName: 🧪 Run unit tests (Browser, Chromium)
       timeoutInMinutes: 20
 
@@ -122,22 +119,19 @@ steps:
     condition: succeededOrFailed()
 
   - ${{ if eq(parameters.VSCODE_RUN_ELECTRON_TESTS, true) }}:
-    # Additional "--" needed to workaround https://github.com/npm/cli/issues/7375
-    - powershell: npm run smoketest-no-compile -- -- --tracing --build "$(agent.builddirectory)\test\VSCode-win32-$(VSCODE_ARCH)"
+    - powershell: npm run smoketest-no-compile -- --tracing --build "$(agent.builddirectory)\test\VSCode-win32-$(VSCODE_ARCH)"
       displayName: 🧪 Run smoke tests (Electron)
       timeoutInMinutes: 20
 
   - ${{ if eq(parameters.VSCODE_RUN_BROWSER_TESTS, true) }}:
-    # Additional "--" needed to workaround https://github.com/npm/cli/issues/7375
-    - powershell: npm run smoketest-no-compile -- -- --web --tracing --headless
+    - powershell: npm run smoketest-no-compile -- --web --tracing --headless
       env:
         VSCODE_REMOTE_SERVER_PATH: $(agent.builddirectory)\test\vscode-server-win32-$(VSCODE_ARCH)-web
       displayName: 🧪 Run smoke tests (Browser, Chromium)
       timeoutInMinutes: 20
 
   - ${{ if eq(parameters.VSCODE_RUN_REMOTE_TESTS, true) }}:
-    # Additional "--" needed to workaround https://github.com/npm/cli/issues/7375
-    - powershell: npm run smoketest-no-compile -- -- --tracing --remote --build "$(agent.builddirectory)\test\VSCode-win32-$(VSCODE_ARCH)"
+    - powershell: npm run smoketest-no-compile -- --tracing --remote --build "$(agent.builddirectory)\test\VSCode-win32-$(VSCODE_ARCH)"
       env:
         VSCODE_REMOTE_SERVER_PATH: $(agent.builddirectory)\test\vscode-server-win32-$(VSCODE_ARCH)
       displayName: 🧪 Run smoke tests (Remote)

build/azure-pipelines/win32/product-build-win32.yml

@@ -256,12 +256,11 @@ steps:
         echo "##vso[task.setvariable variable=EsrpCliDllPath]$Version\net6.0\esrpcli.dll"
       displayName: Find ESRP CLI
 
-    # Additional "--" needed to workaround https://github.com/npm/cli/issues/7375
     - powershell: |
         . build/azure-pipelines/win32/exec.ps1
         $ErrorActionPreference = "Stop"
         mkdir -Force .build/node-cpuprofile
-        exec { npx deemon --detach --wait -- -- npx zx build/azure-pipelines/win32/codesign.js }
+        exec { npx deemon --detach --wait -- npx zx build/azure-pipelines/win32/codesign.js }
       env:
         SYSTEM_ACCESSTOKEN: $(System.AccessToken)
         NODE_DEBUG: "net,child_process"
@@ -278,11 +277,10 @@ steps:
         VSCODE_TEST_ARTIFACT_NAME: ${{ parameters.VSCODE_TEST_ARTIFACT_NAME }}
 
   - ${{ if ne(parameters.VSCODE_CIBUILD, true) }}:
-    # Additional "--" needed to workaround https://github.com/npm/cli/issues/7375
     - powershell: |
         . build/azure-pipelines/win32/exec.ps1
         $ErrorActionPreference = "Stop"
-        exec { npx deemon --attach -- -- npx zx build/azure-pipelines/win32/codesign.js }
+        exec { npx deemon --attach -- npx zx build/azure-pipelines/win32/codesign.js }
       condition: succeededOrFailed()
       env:
         NODE_DEBUG: "net,child_process"