perf: use node: prefix to bypass require.cache call for builtins (#125

)

README.md

@@ -51,8 +51,8 @@ Examples
 * Parsing (multipart) with default options:
 
 ```javascript
-const http = require('http');
-const { inspect } = require('util');
+const http = require('node:http');
+const { inspect } = require('node:util');
 const Busboy = require('busboy');
 
 http.createServer((req, res) => {
@@ -103,10 +103,10 @@ http.createServer((req, res) => {
 * Save all incoming files to disk:
 
 ```javascript
-const http = require('http');
-const path = require('path');
-const os = require('os');
-const fs = require('fs');
+const http = require('node:http');
+const path = require('node:path');
+const os = require('node:os');
+const fs = require('node:fs');
 
 const Busboy = require('busboy');
 
@@ -133,8 +133,8 @@ http.createServer(function(req, res) {
 * Parsing (urlencoded) with default options:
 
 ```javascript
-const http = require('http');
-const { inspect } = require('util');
+const http = require('node:http');
+const { inspect } = require('node:util');
 
 const Busboy = require('busboy');
 

bench/dicer/multiparty-bench-multipart-parser.js

@@ -1,4 +1,4 @@
-var assert = require('assert'),
+var assert = require('node:assert'),
     Form = require('multiparty').Form,
     boundary = '-----------------------------168072824752491622650073',
     mb = 100,

bench/dicer/parted-bench-multipart-parser.js

@@ -2,7 +2,7 @@
 // because otherwise it attempts to do some things above and beyond just parsing
 // -- like saving to disk and whatnot
 
-var assert = require('assert');
+var assert = require('node:assert');
 var Parser = require('./parted-multipart'),
     boundary = '-----------------------------168072824752491622650073',
     parser = new Parser('boundary=' + boundary),

bench/dicer/parted-multipart.js

@@ -4,10 +4,10 @@
  * Copyright (c) 2011, Christopher Jeffrey. (MIT Licensed)
  */
 
-var fs = require('fs')
-  , path = require('path')
-  , EventEmitter = require('events').EventEmitter
-  , StringDecoder = require('string_decoder').StringDecoder
+var fs = require('node:fs')
+  , path = require('node:path')
+  , EventEmitter = require('node:events').EventEmitter
+  , StringDecoder = require('node:string_decoder').StringDecoder
   , set = require('qs').set
   , each = Array.prototype.forEach;
 

benchmarks/common/resultsCombinator.js

@@ -1,5 +1,5 @@
-const fs = require('fs')
-const path = require('path')
+const fs = require('node:fs')
+const path = require('node:path')
 const getopts = require('getopts')
 const systemInformation = require('systeminformation')
 const { loadResults } = require('photofinish')

deps/dicer/lib/Dicer.js

@@ -1,7 +1,7 @@
 'use strict'
 
-const WritableStream = require('stream').Writable
-const inherits = require('util').inherits
+const WritableStream = require('node:stream').Writable
+const inherits = require('node:util').inherits
 
 const StreamSearch = require('../../streamsearch/sbmh')
 

deps/dicer/lib/HeaderParser.js

@@ -1,7 +1,7 @@
 'use strict'
 
-const EventEmitter = require('events').EventEmitter
-const inherits = require('util').inherits
+const EventEmitter = require('node:events').EventEmitter
+const inherits = require('node:util').inherits
 const getLimit = require('../../../lib/utils/getLimit')
 
 const StreamSearch = require('../../streamsearch/sbmh')

deps/dicer/lib/PartStream.js

@@ -1,7 +1,7 @@
 'use strict'
 
-const inherits = require('util').inherits
-const ReadableStream = require('stream').Readable
+const inherits = require('node:util').inherits
+const ReadableStream = require('node:stream').Readable
 
 function PartStream (opts) {
   ReadableStream.call(this, opts)

deps/streamsearch/sbmh.js

@@ -26,8 +26,8 @@
  * Based heavily on the Streaming Boyer-Moore-Horspool C++ implementation
  * by Hongli Lai at: https://github.com/FooBarWidget/boyer-moore-horspool
  */
-const EventEmitter = require('events').EventEmitter
-const inherits = require('util').inherits
+const EventEmitter = require('node:events').EventEmitter
+const inherits = require('node:util').inherits
 
 function SBMH (needle) {
   if (typeof needle === 'string') {

lib/main.js

@@ -1,7 +1,7 @@
 'use strict'
 
-const WritableStream = require('stream').Writable
-const { inherits } = require('util')
+const WritableStream = require('node:stream').Writable
+const { inherits } = require('node:util')
 const Dicer = require('../deps/dicer/lib/Dicer')
 
 const MultipartParser = require('./types/multipart')

lib/types/multipart.js

@@ -7,8 +7,8 @@
 //  * support limits.fieldNameSize
 //     -- this will require modifications to utils.parseParams
 
-const { Readable } = require('stream')
-const { inherits } = require('util')
+const { Readable } = require('node:stream')
+const { inherits } = require('node:util')
 
 const Dicer = require('../../deps/dicer/lib/Dicer')
 

test/dicer-multipart-extra-trailer.spec.js

@@ -1,8 +1,8 @@
 const Dicer = require('../deps/dicer/lib/Dicer')
-const assert = require('assert')
-const fs = require('fs')
-const path = require('path')
-const inspect = require('util').inspect
+const assert = require('node:assert')
+const fs = require('node:fs')
+const path = require('node:path')
+const inspect = require('node:util').inspect
 
 const FIXTURES_ROOT = path.join(__dirname, 'fixtures/')
 

test/dicer-multipart-nolisteners.spec.js

@@ -1,8 +1,8 @@
 const Dicer = require('../deps/dicer/lib/Dicer')
-const assert = require('assert')
-const fs = require('fs')
-const path = require('path')
-const inspect = require('util').inspect
+const assert = require('node:assert')
+const fs = require('node:fs')
+const path = require('node:path')
+const inspect = require('node:util').inspect
 
 const FIXTURES_ROOT = path.join(__dirname, 'fixtures/')
 

test/dicer-multipart.spec.js

@@ -1,8 +1,8 @@
 const Dicer = require('../deps/dicer/lib/Dicer')
-const assert = require('assert')
-const fs = require('fs')
-const path = require('path')
-const inspect = require('util').inspect
+const assert = require('node:assert')
+const fs = require('node:fs')
+const path = require('node:path')
+const inspect = require('node:util').inspect
 
 const FIXTURES_ROOT = path.join(__dirname, 'fixtures/')
 

test/multipart-stream-pause.spec.js

@@ -1,4 +1,4 @@
-const { inspect } = require('util')
+const { inspect } = require('node:util')
 const { assert } = require('chai')
 const Busboy = require('..')
 

test/parse-params.spec.js

@@ -1,4 +1,4 @@
-const { inspect } = require('util')
+const { inspect } = require('node:util')
 const { assert } = require('chai')
 const parseParams = require('../lib/utils/parseParams')
 

test/types-multipart.spec.js

@@ -1,6 +1,6 @@
 const Busboy = require('..')
 
-const { inspect } = require('util')
+const { inspect } = require('node:util')
 const { assert } = require('chai')
 
 const EMPTY_FN = function () {

test/types-urlencoded.spec.js

@@ -1,4 +1,4 @@
-const { inspect } = require('util')
+const { inspect } = require('node:util')
 const { assert } = require('chai')
 const Busboy = require('..')