feat(cli): remove is-builtin-module

packages/@sanity/cli/package.config.ts

@@ -1,11 +1,11 @@
 /* eslint-disable no-sync */
 import fs from 'node:fs'
+import {isBuiltin} from 'node:module'
 import path from 'node:path'
 
 import baseConfig from '@repo/package.config'
 import {nodeResolve} from '@rollup/plugin-node-resolve'
 import {defineConfig} from '@sanity/pkg-utils'
-import isBuiltin from 'is-builtin-module'
 
 const workersDir = path.join(__dirname, 'src', 'workers')
 
@@ -17,21 +17,14 @@ const workerNames = fs
 export default defineConfig({
   ...baseConfig,
 
-  // This is a workaround for an issue where the `readable-stream` package imports
-  // from `string_decoder/` instead of `string_decoder` (note the trailing slash),
-  // which _should_ tell rollup not to use the built-in `string_decoder` module,
-  // which it unfortunately does not. Hoping we can get rid of this hack soon:
-  // https://github.com/rollup/plugins/issues/1211
-  // https://github.com/sindresorhus/is-builtin-module/issues/7
-  // https://github.com/sindresorhus/builtin-modules/pull/17
   rollup: {
     plugins: (plugins) => [
       ...plugins.filter((plugin) => plugin.name !== 'node-resolve'),
       nodeResolve({
         browser: false,
         extensions: ['.cjs', '.mjs', '.js', '.jsx', '.json', '.node'],
         preferBuiltins: false,
-        resolveOnly: (moduleName) => moduleName === 'string_decoder' || !isBuiltin(moduleName),
+        resolveOnly: (moduleName) => !isBuiltin(moduleName),
       }),
     ],
   },

packages/@sanity/cli/package.json

@@ -111,7 +111,6 @@
     "get-latest-version": "^5.0.0",
     "git-user-info": "^2.0.3",
     "inquirer": "^6.0.0",
-    "is-builtin-module": "^3.2.0",
     "is-installed-globally": "^0.4.0",
     "leven": "^3.1.0",
     "lodash": "^4.17.21",