fix(test): stabilize flaky Windows dlx binary tests

Consolidate three near-identical Windows script extension tests
(.cmd, .bat, .ps1) into a single loop. Race the spawnPromise with
a 5s timeout to prevent hangs when shell execution stalls on CI
(e.g. PowerShell execution policy prompts causing the .ps1 test
to exceed vitest's 10s default timeout).

Author: jdalton

test/unit/dlx/binary.test.mts

@@ -1190,102 +1190,54 @@ describe.sequential('dlx-binary', () => {
   describe('Windows-specific behavior', () => {
     const originalPlatform = process.platform
 
-    it.skipIf(process.platform !== 'win32')(
-      'should handle .cmd files with shell on Windows',
-      async () => {
-        await runWithTempDir(async tmpDir => {
-          const restoreHome = mockHomeDir(tmpDir)
-
-          try {
-            // Mock Windows platform
-            Object.defineProperty(process, 'platform', {
-              configurable: true,
-              value: 'win32',
-            })
-
-            const url = `${httpBaseUrl}/binary-windows.cmd`
-
-            const result = await dlxBinary(['--version'], {
-              name: 'test.cmd',
-              url,
-            })
-
-            expect(result.binaryPath).toContain('.cmd')
-            await result.spawnPromise.catch(() => {})
-          } finally {
-            restoreHome()
-            Object.defineProperty(process, 'platform', {
-              configurable: true,
-              value: originalPlatform,
-            })
-          }
-        }, 'dlxBinary-windows-cmd-')
-      },
-    )
-
-    it.skipIf(process.platform !== 'win32')(
-      'should handle .bat files with shell on Windows',
-      async () => {
-        await runWithTempDir(async tmpDir => {
-          const restoreHome = mockHomeDir(tmpDir)
-
-          try {
-            Object.defineProperty(process, 'platform', {
-              configurable: true,
-              value: 'win32',
-            })
-
-            const url = `${httpBaseUrl}/binary-windows.bat`
-
-            const result = await dlxBinary(['--version'], {
-              name: 'test.bat',
-              url,
-            })
-
-            expect(result.binaryPath).toContain('.bat')
-            await result.spawnPromise.catch(() => {})
-          } finally {
-            restoreHome()
-            Object.defineProperty(process, 'platform', {
-              configurable: true,
-              value: originalPlatform,
-            })
-          }
-        }, 'dlxBinary-windows-bat-')
-      },
-    )
-
-    it.skipIf(process.platform !== 'win32')(
-      'should handle .ps1 files with shell on Windows',
-      async () => {
-        await runWithTempDir(async tmpDir => {
-          const restoreHome = mockHomeDir(tmpDir)
-
-          try {
-            Object.defineProperty(process, 'platform', {
-              configurable: true,
-              value: 'win32',
-            })
-
-            const url = `${httpBaseUrl}/binary-windows.ps1`
-
-            const result = await dlxBinary(['--version'], {
-              name: 'test.ps1',
-              url,
-            })
-
-            expect(result.binaryPath).toContain('.ps1')
-            await result.spawnPromise.catch(() => {})
-          } finally {
-            restoreHome()
-            Object.defineProperty(process, 'platform', {
-              configurable: true,
-              value: originalPlatform,
-            })
-          }
-        }, 'dlxBinary-windows-ps1-')
-      },
-    )
+    // Windows script extensions that require shell: true in spawn.
+    // Each test verifies dlxBinary downloads the script and resolves
+    // the correct binaryPath. The spawnPromise is raced with a timeout
+    // to avoid hanging on CI when shell execution stalls (e.g. ps1
+    // execution policy prompts).
+    const windowsScriptExts = ['.cmd', '.bat', '.ps1'] as const
+
+    for (const ext of windowsScriptExts) {
+      it.skipIf(process.platform !== 'win32')(
+        `should handle ${ext} files with shell on Windows`,
+        async () => {
+          await runWithTempDir(
+            async tmpDir => {
+              const restoreHome = mockHomeDir(tmpDir)
+
+              try {
+                Object.defineProperty(process, 'platform', {
+                  configurable: true,
+                  value: 'win32',
+                })
+
+                const url = `${httpBaseUrl}/binary-windows${ext}`
+
+                const result = await dlxBinary(['--version'], {
+                  name: `test${ext}`,
+                  url,
+                })
+
+                expect(result.binaryPath).toContain(ext)
+                // Race the spawn with a 5s timeout to prevent hangs on CI
+                // (e.g. PowerShell execution policy prompts).
+                await Promise.race([
+                  result.spawnPromise.catch(() => {}),
+                  new Promise(resolve => setTimeout(resolve, 5_000)),
+                ])
+              } finally {
+                restoreHome()
+                Object.defineProperty(process, 'platform', {
+                  configurable: true,
+                  value: originalPlatform,
+                })
+              }
+            },
+            `dlxBinary-windows-${ext.slice(1)}-`,
+          )
+        },
+      )
+    }
   })
 
   describe('edge cases', () => {