Add support for Bun dependency updates and improve test coverage

Author: LabinatorSolutions

index.js

@@ -1,7 +1,7 @@
 let { execSync } = require('child_process')
 let escalade = require('escalade/sync')
 let { existsSync, readFileSync, writeFileSync } = require('fs')
-let { join } = require('path')
+let { dirname, join } = require('path')
 let pico = require('picocolors')
 
 let { detectEOL, detectIndent } = require('./utils')
@@ -264,6 +264,55 @@ function updatePackageManually(print, lock, latest) {
   execSync(del + ' caniuse-lite baseline-browser-mapping')
 }
 
+/**
+ * Bun cannot update a transitive dependency by name: `bun update caniuse-lite`
+ * adds caniuse-lite to package.json as a new direct dependency at the latest
+ * version and leaves every nested copy - the ones that actually get resolved at
+ * runtime - on the old version. A temporary `overrides` entry reaches those,
+ * and the resolutions survive once it is taken back out.
+ */
+function updateBun(print, lock, latest) {
+  let pkgFile = join(dirname(lock.file), 'package.json')
+  let original = readFileSync(pkgFile)
+  let pkg = JSON.parse(original.toString())
+  let withOverride = {
+    ...pkg,
+    overrides: {
+      ...pkg.overrides,
+      'baseline-browser-mapping': 'latest',
+      'caniuse-lite': latest.version
+    }
+  }
+
+  // The file is restored byte for byte below, so its formatting does not matter.
+  writeFileSync(pkgFile, JSON.stringify(withOverride, null, 2) + '\n')
+
+  print(
+    'Updating caniuse-lite version\n' +
+      pico.yellow('$ bun install') +
+      ' (with a temporary caniuse-lite override)\n'
+  )
+  try {
+    execSync('bun install')
+  } catch (e) /* c8 ignore start */ {
+    writeFileSync(pkgFile, original)
+    print(pico.red(e.stdout.toString()))
+    print(
+      pico.red(
+        '\n' +
+          e.stack +
+          '\n\n' +
+          'Problem with `bun install` call. ' +
+          'Run it manually.\n'
+      )
+    )
+    process.exit(1)
+  } /* c8 ignore end */
+
+  writeFileSync(pkgFile, original)
+  updateWith(print, 'bun install', 'Removing the temporary override')
+}
+
 function updateWith(print, cmd, message = 'Updating caniuse-lite version') {
   print(message + '\n' + pico.yellow('$ ' + cmd) + '\n')
   try {
@@ -308,7 +357,7 @@ module.exports = function updateDB(print = defaultPrint) {
       : 'caniuse-lite'
     updateWith(print, 'pnpm up --depth=Infinity --no-save ' + packages)
   } else if (lock.mode === 'bun') {
-    updateWith(print, 'bun update caniuse-lite baseline-browser-mapping')
+    updateBun(print, lock, latest)
   } else if (lock.mode === 'deno') {
     updateWith(print, 'deno add npm:caniuse-lite npm:baseline-browser-mapping')
     updateWith(

package.json

@@ -42,8 +42,8 @@
   },
   "devDependencies": {
     "@logux/oxc-configs": "^1.1.0",
-    "actions-up": "^1.16.0",
-    "browserslist": "^4.28.7",
+    "actions-up": "^1.17.0",
+    "browserslist": "^4.28.8",
     "c8": "^12.0.0",
     "eslint-plugin-prefer-let": "^4.2.2",
     "oxlint": "^1.77.0",

pnpm-lock.yaml

@@ -311,19 +311,37 @@ test('updates caniuse-lite for pnpm', async () => {
 
 if (bunInstalled) {
   test('updates caniuse-lite for bun', async () => {
-    await chdir('update-bun', 'package.json', 'bun.lockb')
+    let dir = await chdir('update-bun', 'package.json', 'bun.lockb')
+    let pkgBefore = (await readFile(join(dir, 'package.json'))).toString()
+
     match(
       runUpdate(),
       `Latest version:     ${caniuse.version}\n` +
         'Updating caniuse-lite version\n' +
-        '$ bun update caniuse-lite baseline-browser-mapping\n' +
+        '$ bun install (with a temporary caniuse-lite override)\n' +
+        'Removing the temporary override\n' +
+        '$ bun install\n' +
         'caniuse-lite has been successfully updated\n'
     )
 
     let dependencies = execSync('bun pm ls --all', {
       env: { ...process.env, FORCE_COLOR: '0', NO_COLOR: '1' }
     }).toString()
-    ok(dependencies.includes(`caniuse-lite@${caniuse.version}`))
+    let versions = [...dependencies.matchAll(/caniuse-lite@(\S+)/g)].map(
+      i => i[1]
+    )
+
+    // Every copy has to move, including the nested ones. `bun update
+    // caniuse-lite` only ever added a new hoisted copy at the latest version
+    // and left those behind on the old one.
+    ok(versions.length > 0)
+    equal(
+      versions.filter(i => i !== caniuse.version),
+      []
+    )
+
+    // The override is temporary and must not survive in package.json.
+    equal((await readFile(join(dir, 'package.json'))).toString(), pkgBefore)
   })
 }