Enable bun on Windows (#1401)

package.json

@@ -30,7 +30,7 @@
     "lint:lockfile": "lockfile-lint",
     "lint:markdown": "markdownlint \"**/*.md\" --ignore \"**/node_modules/**/*.md\" --ignore build --config .markdownlint.js",
     "lint:src": "eslint --cache --cache-location node_modules/.cache/.eslintcache --ignore-path .gitignore --report-unused-disable-directives .",
-    "prepare": "src/scripts/install-hooks && test/bun-setup.sh",
+    "prepare": "src/scripts/install-hooks",
     "prepublishOnly": "npm run build",
     "prettier": "prettier . --check",
     "test": "npm run test:unit && npm run test:e2e",
@@ -80,6 +80,7 @@
     "@types/update-notifier": "^6.0.8",
     "@typescript-eslint/eslint-plugin": "^7.4.0",
     "@typescript-eslint/parser": "^7.4.0",
+    "bun": "^1.1.4",
     "camelcase": "^6.3.0",
     "chai": "^4.3.10",
     "chai-as-promised": "^7.1.1",

src/index.ts

@@ -140,7 +140,7 @@ const install = async (
 
     pkgsNormalized.forEach(async pkgFile => {
       const packageManager = await getPackageManagerForInstall(options, pkgFile)
-      const cmd = packageManager + (process.platform === 'win32' ? '.cmd' : '')
+      const cmd = packageManager + (process.platform === 'win32' && packageManager !== 'bun' ? '.cmd' : '')
       const cwd = options.cwd || path.resolve(pkgFile, '..')
       let stdout = ''
       try {

src/package-managers/bun.ts

@@ -1,7 +1,6 @@
 import path from 'path'
 import spawn from 'spawn-please'
 import keyValueBy from '../lib/keyValueBy'
-import programError from '../lib/programError'
 import { Index } from '../types/IndexType'
 import { NpmOptions } from '../types/NpmOptions'
 import { Options } from '../types/Options'
@@ -15,12 +14,6 @@ async function spawnBun(
   spawnPleaseOptions: SpawnPleaseOptions = {},
   spawnOptions: Index<any> = {},
 ): Promise<{ stdout: string; stderr: string }> {
-  // Bun not yet supported on Windows.
-  // @see https://github.com/oven-sh/bun/issues/43
-  if (process.platform === 'win32') {
-    programError(npmOptions, 'Bun not yet supported on Windows')
-  }
-
   const fullArgs = [
     ...(npmOptions.global ? ['--global'] : []),
     ...(npmOptions.prefix ? [`--prefix=${npmOptions.prefix}`] : []),

test/doctor.test.ts

@@ -483,7 +483,6 @@ else {
     testFail({ packageManager: 'yarn' })
   })
 
-  // Bun not yet supported on Windows
   // const describeSkipWindows = os.platform() === 'win32' ? describe.skip : describe
   // TODO: Works locally, but not in GitHub action.
   describe.skip('bun', () => {

test/package-managers/bun/index.test.ts

@@ -1,12 +1,9 @@
-import os from 'os'
 import * as bun from '../../../src/package-managers/bun'
 import chaiSetup from '../../helpers/chaiSetup'
 
 chaiSetup()
 
-// Bun not yet supported on Windows
-const describeSkipWindows = os.platform() === 'win32' ? describe.skip : describe
-describeSkipWindows('bun', function () {
+describe('bun', function () {
   it('list', async () => {
     const result = await bun.list({ cwd: __dirname })
     result.should.have.property('ncu-test-v2')