Change pure require property access and node require calls

jdalton · jdalton · commit 6142e70e2577 · 2025-08-09T11:40:43.000-04:00
diff --git a/packages/npm/deep-equal/index.cjs b/packages/npm/deep-equal/index.cjs
@@ -1,7 +1,11 @@
 'use strict'
 
-const { deepEqual: assertIsDeepEqual } = require('node:assert')
-const { isDeepStrictEqual } = require('node:util')
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeAssert = /*@__PURE__*/ require('assert')
+const assertIsDeepEqual = nodeAssert.deepEqual
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeUtil = /*@__PURE__*/ require('util')
+const isDeepStrictEqual = nodeUtil.isDeepStrictEqual
 
 module.exports = function deepEqual(value1, value2, options) {
   try {
diff --git a/packages/npm/is-arguments/index.cjs b/packages/npm/is-arguments/index.cjs
@@ -1,3 +1,5 @@
 'use strict'
 
-module.exports = require('node:util/types').isArgumentsObject
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeUtilTypes = /*@__PURE__*/ require('util/types')
+module.exports = nodeUtilTypes.isArgumentsObject
diff --git a/packages/npm/is-array-buffer/index.cjs b/packages/npm/is-array-buffer/index.cjs
@@ -1,3 +1,5 @@
 'use strict'
 
-module.exports = require('node:util/types').isArrayBuffer
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeUtilTypes = /*@__PURE__*/ require('util/types')
+module.exports = nodeUtilTypes.isArrayBuffer
diff --git a/packages/npm/is-bigint/index.cjs b/packages/npm/is-bigint/index.cjs
@@ -1,6 +1,8 @@
 'use strict'
 
-const { isBigIntObject } = require('node:util/types')
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeUtilTypes = /*@__PURE__*/ require('util/types')
+const isBigIntObject = nodeUtilTypes.isBigIntObject
 
 module.exports = function isBigInt(value) {
   return typeof value === 'bigint' || isBigIntObject(value)
diff --git a/packages/npm/is-boolean-object/index.cjs b/packages/npm/is-boolean-object/index.cjs
@@ -1,6 +1,8 @@
 'use strict'
 
-const { isBooleanObject } = require('node:util/types')
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeUtilTypes = /*@__PURE__*/ require('util/types')
+const isBooleanObject = nodeUtilTypes.isBooleanObject
 
 module.exports = function isBoolean(value) {
   return typeof value === 'boolean' || isBooleanObject(value)
diff --git a/packages/npm/is-core-module/index.cjs b/packages/npm/is-core-module/index.cjs
@@ -1,6 +1,8 @@
 'use strict'
 
-const { isBuiltin } = require('node:module')
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeModule = /*@__PURE__*/ require('module')
+const isBuiltin = nodeModule.isBuiltin
 
 module.exports = function isCore(moduleName, nodeVersion) {
   if (typeof nodeVersion === 'string') {
diff --git a/packages/npm/is-date-object/index.cjs b/packages/npm/is-date-object/index.cjs
@@ -1,3 +1,5 @@
 'use strict'
 
-module.exports = require('node:util/types').isDate
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeUtilTypes = /*@__PURE__*/ require('util/types')
+module.exports = nodeUtilTypes.isDate
diff --git a/packages/npm/is-generator-function/index.cjs b/packages/npm/is-generator-function/index.cjs
@@ -1,3 +1,5 @@
 'use strict'
 
-module.exports = require('node:util/types').isGeneratorFunction
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeUtilTypes = /*@__PURE__*/ require('util/types')
+module.exports = nodeUtilTypes.isGeneratorFunction
diff --git a/packages/npm/is-map/index.cjs b/packages/npm/is-map/index.cjs
@@ -1,3 +1,5 @@
 'use strict'
 
-module.exports = require('node:util/types').isMap
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeUtilTypes = /*@__PURE__*/ require('util/types')
+module.exports = nodeUtilTypes.isMap
diff --git a/packages/npm/is-number-object/index.cjs b/packages/npm/is-number-object/index.cjs
@@ -1,3 +1,5 @@
 'use strict'
 
-module.exports = require('node:util/types').isNumberObject
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeUtilTypes = /*@__PURE__*/ require('util/types')
+module.exports = nodeUtilTypes.isNumberObject
diff --git a/packages/npm/is-regex/index.cjs b/packages/npm/is-regex/index.cjs
@@ -1,3 +1,5 @@
 'use strict'
 
-module.exports = require('node:util/types').isRegExp
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeUtilTypes = /*@__PURE__*/ require('util/types')
+module.exports = nodeUtilTypes.isRegExp
diff --git a/packages/npm/is-set/index.cjs b/packages/npm/is-set/index.cjs
@@ -1,3 +1,5 @@
 'use strict'
 
-module.exports = require('node:util/types').isSet
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeUtilTypes = /*@__PURE__*/ require('util/types')
+module.exports = nodeUtilTypes.isSet
diff --git a/packages/npm/is-shared-array-buffer/index.cjs b/packages/npm/is-shared-array-buffer/index.cjs
@@ -1,3 +1,5 @@
 'use strict'
 
-module.exports = require('node:util/types').isSharedArrayBuffer
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeUtilTypes = /*@__PURE__*/ require('util/types')
+module.exports = nodeUtilTypes.isSharedArrayBuffer
diff --git a/packages/npm/is-string/index.cjs b/packages/npm/is-string/index.cjs
@@ -1,6 +1,8 @@
 'use strict'
 
-const { isStringObject } = require('node:util/types')
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeUtilTypes = /*@__PURE__*/ require('util/types')
+const isStringObject = nodeUtilTypes.isStringObject
 
 module.exports = function isString(value) {
   return typeof value === 'string' || isStringObject(value)
diff --git a/packages/npm/is-symbol/index.cjs b/packages/npm/is-symbol/index.cjs
@@ -1,6 +1,8 @@
 'use strict'
 
-const { isSymbolObject } = require('node:util/types')
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeUtilTypes = /*@__PURE__*/ require('util/types')
+const isSymbolObject = nodeUtilTypes.isSymbolObject
 
 module.exports = function isSymbol(value) {
   return typeof value === 'symbol' || isSymbolObject(value)
diff --git a/packages/npm/is-typed-array/index.cjs b/packages/npm/is-typed-array/index.cjs
@@ -1,6 +1,8 @@
 'use strict'
 
-const { isSymbolObject } = require('node:util/types')
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeUtilTypes = /*@__PURE__*/ require('util/types')
+const isSymbolObject = nodeUtilTypes.isSymbolObject
 
 module.exports = function isSymbol(value) {
   return typeof value === 'symbol' || isSymbolObject(value)
diff --git a/packages/npm/is-weakmap/index.cjs b/packages/npm/is-weakmap/index.cjs
@@ -1,3 +1,5 @@
 'use strict'
 
-module.exports = require('node:util/types').isWeakMap
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeUtilTypes = /*@__PURE__*/ require('util/types')
+module.exports = nodeUtilTypes.isWeakMap
diff --git a/packages/npm/is-weakset/index.cjs b/packages/npm/is-weakset/index.cjs
@@ -1,3 +1,4 @@
 'use strict'
 
-module.exports = require('node:util/types').isWeakSet
+const nodeUtilTypes = /*@__PURE__*/ require('node:util/types')
+module.exports = nodeUtilTypes.isWeakSet
diff --git a/packages/npm/util.promisify/implementation.js b/packages/npm/util.promisify/implementation.js
@@ -1,17 +1,18 @@
 'use strict'
 
-const {
-  promisify: UtilPromisify,
-  promisify: { custom }
-  // Use non-'node:' prefixed require to avoid Webpack errors.
-  // eslint-disable-next-line n/prefer-node-protocol
-} = require('util')
+// Use non-'node:' prefixed require to avoid Webpack errors.
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeUtil = /*@__PURE__*/ require('util')
+const UtilPromisify = nodeUtil.promisify
+const { custom } = UtilPromisify
 
-const customPromisifyArgs = Object.getOwnPropertySymbols(
-  // Use non-'node:' prefixed require to avoid Webpack errors.
-  // eslint-disable-next-line n/prefer-node-protocol
-  require('fs').read
-).find(s => s.description === 'customPromisifyArgs')
+// Use non-'node:' prefixed require to avoid Webpack errors.
+// eslint-disable-next-line n/prefer-node-protocol
+const nodeFs = /*@__PURE__*/ require('fs')
+const read = nodeFs.read
+const customPromisifyArgs = Object.getOwnPropertySymbols(read).find(
+  s => s.description === 'customPromisifyArgs'
+)
 
 module.exports = Object.assign(
   function promisify(original) {
diff --git a/packages/npm/yocto-spinner/index.cjs b/packages/npm/yocto-spinner/index.cjs
@@ -96,7 +96,8 @@ function stripVTControlCharacters(string) {
   if (_stripVTControlCharacters === undefined) {
     // Use non-'node:' prefixed require to avoid Webpack errors.
     // eslint-disable-next-line n/prefer-node-protocol
-    _stripVTControlCharacters = require('util').stripVTControlCharacters
+    const nodeUtil = /*@__PURE__*/ require('util')
+    _stripVTControlCharacters = nodeUtil.stripVTControlCharacters
   }
   return _stripVTControlCharacters(string)
 }
diff --git a/registry/index.js b/registry/index.js
@@ -5,8 +5,8 @@ let _PackageURL
 function getPackageURL() {
   if (_PackageURL === undefined) {
     // The 'packageurl-js' package is browser safe.
-    _PackageURL =
-      /*@__PURE__*/ require('./external/@socketregistry/packageurl-js').PackageURL
+    const packageUrlJs = /*@__PURE__*/ require('./external/@socketregistry/packageurl-js')
+    _PackageURL = packageUrlJs.PackageURL
   }
   return _PackageURL
 }
diff --git a/registry/lib/constants/abort-signal.js b/registry/lib/constants/abort-signal.js
@@ -1,6 +1,7 @@
 'use strict'
 
-const abortSignal = /*@__PURE__*/ require('./abort-controller').signal
+const abortController = /*@__PURE__*/ require('./abort-controller')
+const abortSignal = abortController.signal
 
 // By manually setting `kMaxEventTargetListeners` on `abortSignal` we avoid:
 //   TypeError [ERR_INVALID_ARG_TYPE]: The "emitter" argument must be an
diff --git a/registry/lib/debug.js b/registry/lib/debug.js
@@ -10,7 +10,8 @@ let _debugJs
 function getDebugJs() {
   if (_debugJs === undefined) {
     // The 'debug' package is browser safe.
-    _debugJs = /*@__PURE__*/ require('../external/debug').default
+    const debugExport = /*@__PURE__*/ require('../external/debug')
+    _debugJs = debugExport.default
   }
   return _debugJs
 }
diff --git a/registry/lib/logger.js b/registry/lib/logger.js
@@ -12,7 +12,8 @@ function constructConsole(...args) {
   if (_Console === undefined) {
     // Use non-'node:' prefixed require to avoid Webpack errors.
     // eslint-disable-next-line n/prefer-node-protocol
-    _Console = /*@__PURE__*/ require('console').Console
+    const nodeConsole = /*@__PURE__*/ require('console')
+    _Console = nodeConsole.Console
   }
   return ReflectConstruct(_Console, args)
 }
diff --git a/registry/lib/packages.js b/registry/lib/packages.js
@@ -385,8 +385,8 @@ let _PackageURL
 function getPackageURL() {
   if (_PackageURL === undefined) {
     // The 'packageurl-js' package is browser safe.
-    _PackageURL =
-      /*@__PURE__*/ require('../external/@socketregistry/packageurl-js').PackageURL
+    const packageUrlJs = /*@__PURE__*/ require('../external/@socketregistry/packageurl-js')
+    _PackageURL = packageUrlJs.PackageURL
   }
   return _PackageURL
 }
diff --git a/registry/lib/prompts.js b/registry/lib/prompts.js
@@ -1,10 +1,5 @@
 'use strict'
 
-const {
-  Separator,
-  default: selectRaw
-} = /*@__PURE__*/ require('../external/@inquirer/select')
-
 /*@__NO_SIDE_EFFECTS__*/
 function wrapPrompt(inquirerPrompt) {
   return async (...args) => {
@@ -39,16 +34,23 @@ function wrapPrompt(inquirerPrompt) {
   }
 }
 
-const confirm = /*@__PURE__*/ wrapPrompt(
-  require('../external/@inquirer/confirm')
-)
-const input = /*@__PURE__*/ wrapPrompt(require('../external/@inquirer/input'))
-const password = /*@__PURE__*/ wrapPrompt(
-  require('../external/@inquirer/password')
-)
-const search = /*@__PURE__*/ wrapPrompt(
-  require('../external/@inquirer/search').default
-)
+const selectExport = /*@__PURE__*/ require('../external/@inquirer/select')
+const selectRaw = selectExport.default
+const Separator = selectExport.Separator
+
+const confirmRaw = /*@__PURE__*/ require('../external/@inquirer/confirm')
+const confirm = /*@__PURE__*/ wrapPrompt(confirmRaw)
+
+const inputRaw = /*@__PURE__*/ require('../external/@inquirer/input')
+const input = /*@__PURE__*/ wrapPrompt(inputRaw)
+
+const passwordRaw = /*@__PURE__*/ require('../external/@inquirer/password')
+const password = /*@__PURE__*/ wrapPrompt(passwordRaw)
+
+const searchExport = /*@__PURE__*/ require('../external/@inquirer/search')
+const searchRaw = searchExport.default
+
+const search = /*@__PURE__*/ wrapPrompt(searchRaw)
 const select = /*@__PURE__*/ wrapPrompt(selectRaw)
 
 module.exports = {
diff --git a/scripts/constants.js b/scripts/constants.js
diff --git a/scripts/lib/biome.js b/scripts/lib/biome.js
diff --git a/scripts/lib/git.js b/scripts/lib/git.js
diff --git a/scripts/lib/templates.js b/scripts/lib/templates.js
