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

…5026)

.github/scripts/lint-ecosystem.js

@@ -1,8 +1,8 @@
 'use strict'
 
-const path = require('path')
-const fs = require('fs')
-const readline = require('readline')
+const path = require('node:path')
+const fs = require('node:fs')
+const readline = require('node:readline')
 
 const basePathEcosystemDocFile = path.join('docs', 'Guides', 'Ecosystem.md')
 const ecosystemDocFile = path.join(__dirname, '..', '..', basePathEcosystemDocFile)

build/build-error-serializer.js

@@ -2,8 +2,8 @@
 'use strict'
 
 const FJS = require('fast-json-stringify')
-const path = require('path')
-const fs = require('fs')
+const path = require('node:path')
+const fs = require('node:fs')
 
 const code = FJS({
   type: 'object',

build/build-validation.js

@@ -2,8 +2,8 @@
 
 const AjvStandaloneCompiler = require('@fastify/ajv-compiler/standalone')
 const { _ } = require('ajv')
-const fs = require('fs')
-const path = require('path')
+const fs = require('node:fs')
+const path = require('node:path')
 
 const factory = AjvStandaloneCompiler({
   readMode: false,

build/sync-version.js

@@ -1,7 +1,7 @@
 'use strict'
 
-const fs = require('fs')
-const path = require('path')
+const fs = require('node:fs')
+const path = require('node:path')
 
 // package.json:version -> fastify.js:VERSION
 const { version } = JSON.parse(fs.readFileSync(path.join(__dirname, '..', 'package.json')).toString('utf8'))

docs/Guides/Database.md

@@ -275,7 +275,7 @@ CREATE TABLE IF NOT EXISTS users (
 ```javascript
 const pg = require('pg')
 const Postgrator = require('postgrator')
-const path = require('path')
+const path = require('node:path')
 
 async function migrate() {
   const client = new pg.Client({

docs/Guides/Delay-Accepting-Requests.md

@@ -135,7 +135,7 @@ follows:
 
 ```js
 const { fetch } = require('undici')
-const { setTimeout } = require('timers/promises')
+const { setTimeout } = require('node:timers/promises')
 
 const MAGIC_KEY = '12345'
 
@@ -249,7 +249,7 @@ server.listen({ port: '1234' })
 
 ```js
 const { fetch } = require('undici')
-const { setTimeout } = require('timers/promises')
+const { setTimeout } = require('node:timers/promises')
 
 const MAGIC_KEY = '12345'
 

docs/Reference/HTTP2.md

@@ -15,8 +15,8 @@ HTTP2 is supported in all modern browsers __only over a secure connection__:
 ```js
 'use strict'
 
-const fs = require('fs')
-const path = require('path')
+const fs = require('node:fs')
+const path = require('node:path')
 const fastify = require('fastify')({
   http2: true,
   https: {
@@ -42,8 +42,8 @@ box:
 ```js
 'use strict'
 
-const fs = require('fs')
-const path = require('path')
+const fs = require('node:fs')
+const path = require('node:path')
 const fastify = require('fastify')({
   http2: true,
   https: {

docs/Reference/Hooks.md

@@ -840,7 +840,7 @@ tools first" fashion.
 
 ```js
 const tracer = /* retrieved from elsewhere in the package */
-const dc = require('diagnostics_channel')
+const dc = require('node:diagnostics_channel')
 const channel = dc.channel('fastify.initialization')
 const spans = new WeakMap()
 

docs/Reference/Middleware.md

@@ -54,7 +54,7 @@ the first parameter to `use` and you are done!
 `/user/:id/comments`) and wildcards are not supported in multiple paths.*
 
 ```js
-const path = require('path')
+const path = require('node:path')
 const serveStatic = require('serve-static')
 
 // Single path

docs/Reference/Reply.md

@@ -242,7 +242,7 @@ reply.trailer('server-timing', function() {
   return 'db;dur=53, app;dur=47.2'
 })
 
-const { createHash } = require('crypto')
+const { createHash } = require('node:crypto')
 // trailer function also receive two argument
 // @param {object} reply fastify reply
 // @param {string|Buffer|null} payload payload that already sent, note that it will be null when stream is sent
@@ -668,7 +668,7 @@ fastify.get('/json', options, function (request, reply) {
 `'application/octet-stream'`.
 ```js
 fastify.get('/streams', function (request, reply) {
-  const fs = require('fs')
+  const fs = require('node:fs')
   const stream = fs.createReadStream('some-file', 'utf8')
   reply.header('Content-Type', 'application/octet-stream')
   reply.send(stream)
@@ -677,7 +677,7 @@ fastify.get('/streams', function (request, reply) {
 When using async-await you will need to return or await the reply object:
 ```js
 fastify.get('/streams', async function (request, reply) {
-  const fs = require('fs')
+  const fs = require('node:fs')
   const stream = fs.createReadStream('some-file', 'utf8')
   reply.header('Content-Type', 'application/octet-stream')
   return reply.send(stream)
@@ -690,7 +690,7 @@ fastify.get('/streams', async function (request, reply) {
 If you are sending a buffer and you have not set a `'Content-Type'` header,
 *send* will set it to `'application/octet-stream'`.
 ```js
-const fs = require('fs')
+const fs = require('node:fs')
 fastify.get('/streams', function (request, reply) {
   fs.readFile('some-file', (err, fileBuffer) => {
     reply.send(err || fileBuffer)
@@ -700,7 +700,7 @@ fastify.get('/streams', function (request, reply) {
 
 When using async-await you will need to return or await the reply object:
 ```js
-const fs = require('fs')
+const fs = require('node:fs')
 fastify.get('/streams', async function (request, reply) {
   fs.readFile('some-file', (err, fileBuffer) => {
     reply.send(err || fileBuffer)
@@ -715,7 +715,7 @@ fastify.get('/streams', async function (request, reply) {
 `send` manages TypedArray and sets the `'Content-Type'=application/octet-stream'`
 header if not already set.
 ```js
-const fs = require('fs')
+const fs = require('node:fs')
 fastify.get('/streams', function (request, reply) {
   const typedArray = new Uint16Array(10)
   reply.send(typedArray)
@@ -842,7 +842,7 @@ Fastify natively handles promises and supports async-await.
 
 *Note that in the following examples we are not using reply.send.*
 ```js
-const { promisify } = require('util')
+const { promisify } = require('node:util')
 const delay = promisify(setTimeout)
 
 fastify.get('/promises', options, function (request, reply) {

docs/Reference/Server.md

@@ -635,7 +635,7 @@ You can also use Fastify's default parser but change some handling behavior,
 like the example below for case insensitive keys and values:
 
 ```js
-const querystring = require('querystring')
+const querystring = require('node:querystring')
 const fastify = require('fastify')({
   querystringParser: str => querystring.parse(str.toLowerCase())
 })
@@ -1892,7 +1892,7 @@ The properties that can currently be exposed are:
 - http2SessionTimeout
 
 ```js
-const { readFileSync } = require('fs')
+const { readFileSync } = require('node:fs')
 const Fastify = require('fastify')
 
 const fastify = Fastify({

examples/benchmark/parser.js

@@ -5,7 +5,7 @@ const fastify = require('../../fastify')({
 })
 
 const jsonParser = require('fast-json-body')
-const querystring = require('querystring')
+const querystring = require('node:querystring')
 
 // Handled by fastify
 // curl -X POST -d '{"hello":"world"}' -H'Content-type: application/json' http://localhost:3000/

examples/http2.js

@@ -1,7 +1,7 @@
 'use strict'
 
-const fs = require('fs')
-const path = require('path')
+const fs = require('node:fs')
+const path = require('node:path')
 const fastify = require('../fastify')({
   http2: true,
   https: {

examples/https.js

@@ -1,7 +1,7 @@
 'use strict'
 
-const fs = require('fs')
-const path = require('path')
+const fs = require('node:fs')
+const path = require('node:path')
 const fastify = require('../fastify')({
   https: {
     key: fs.readFileSync(path.join(__dirname, '../test/https/fastify.key')),

examples/parser.js

@@ -2,7 +2,7 @@
 
 const fastify = require('../fastify')({ logger: true })
 const jsonParser = require('fast-json-body')
-const querystring = require('querystring')
+const querystring = require('node:querystring')
 
 // Handled by fastify
 // curl -X POST -d '{"hello":"world"}' -H'Content-type: application/json' http://localhost:3000/

examples/simple-stream.js

@@ -4,7 +4,7 @@ const fastify = require('../fastify')({
   logger: false
 })
 
-const Readable = require('stream').Readable
+const Readable = require('node:stream').Readable
 
 fastify
   .get('/', function (req, reply) {

fastify.js

@@ -3,7 +3,7 @@
 const VERSION = '4.22.2'
 
 const Avvio = require('avvio')
-const http = require('http')
+const http = require('node:http')
 let lightMyRequest
 
 const {
@@ -496,7 +496,7 @@ function fastify (options) {
   server.on('clientError', options.clientErrorHandler.bind(fastify))
 
   try {
-    const dc = require('diagnostics_channel')
+    const dc = require('node:diagnostics_channel')
     const initChannel = dc.channel('fastify.initialization')
     if (initChannel.hasSubscribers) {
       initChannel.publish({ fastify })

lib/contentTypeParser.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const { AsyncResource } = require('async_hooks')
+const { AsyncResource } = require('node:async_hooks')
 const { Fifo } = require('toad-cache')
 const { safeParse: safeParseContentType, defaultContentType } = require('fast-content-type-parse')
 const secureJson = require('secure-json-parse')

lib/error-handler.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const statusCodes = require('http').STATUS_CODES
+const statusCodes = require('node:http').STATUS_CODES
 const wrapThenable = require('./wrapThenable')
 const {
   kReplyHeaders,

lib/pluginUtils.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const semver = require('semver')
-const assert = require('assert')
+const assert = require('node:assert')
 const kRegisteredPlugins = Symbol.for('registered-plugin')
 const {
   kTestInternals

lib/reply.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const eos = require('stream').finished
+const eos = require('node:stream').finished
 
 const {
   kFourOhFourContext,

lib/server.js

@@ -1,8 +1,8 @@
 'use strict'
 
-const http = require('http')
-const https = require('https')
-const dns = require('dns')
+const http = require('node:http')
+const https = require('node:https')
+const dns = require('node:dns')
 
 const warnings = require('./warnings')
 const { kState, kOptions, kServerBindings } = require('./symbols')
@@ -426,7 +426,7 @@ function logServerAddress (server, listenTextResolver) {
 
 function http2 () {
   try {
-    return require('http2')
+    return require('node:http2')
   } catch (err) {
     throw new FST_ERR_HTTP2_INVALID_VERSION()
   }

test/404s.test.js

@@ -1031,7 +1031,7 @@ test('setNotFoundHandler should not suppress duplicated routes checking', t => {
 test('log debug for 404', t => {
   t.plan(1)
 
-  const Writable = require('stream').Writable
+  const Writable = require('node:stream').Writable
 
   const logStream = new Writable()
   logStream.logs = []

test/als.test.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const { AsyncLocalStorage } = require('async_hooks')
+const { AsyncLocalStorage } = require('node:async_hooks')
 const t = require('tap')
 const Fastify = require('..')
 const sget = require('simple-get').concat

test/async-await.test.js

@@ -6,7 +6,7 @@ const sget = require('simple-get').concat
 const Fastify = require('..')
 const split = require('split2')
 const pino = require('pino')
-const statusCodes = require('http').STATUS_CODES
+const statusCodes = require('node:http').STATUS_CODES
 const sleep = ms => new Promise(resolve => setTimeout(resolve, ms))
 
 const opts = {

test/bodyLimit.test.js

@@ -2,7 +2,7 @@
 
 const Fastify = require('..')
 const sget = require('simple-get').concat
-const zlib = require('zlib')
+const zlib = require('node:zlib')
 const t = require('tap')
 const test = t.test
 

test/build/error-serializer.test.js

@@ -2,8 +2,8 @@
 
 const t = require('tap')
 const test = t.test
-const fs = require('fs')
-const path = require('path')
+const fs = require('node:fs')
+const path = require('node:path')
 
 const { code } = require('../../build/build-error-serializer')
 

test/build/version.test.js

@@ -1,7 +1,7 @@
 'use strict'
 
-const fs = require('fs')
-const path = require('path')
+const fs = require('node:fs')
+const path = require('node:path')
 const t = require('tap')
 const test = t.test
 const fastify = require('../../fastify')()

test/bundler/webpack/webpack.config.js

@@ -1,4 +1,4 @@
-const path = require('path')
+const path = require('node:path')
 
 module.exports = {
   entry: { success: './src/index.js', failPlugin: './src/fail-plugin-version.js' },

test/client-timeout.test.js

@@ -2,7 +2,7 @@
 
 const { test } = require('tap')
 const fastify = require('..')({ requestTimeout: 5, http: { connectionsCheckingInterval: 1000 } })
-const { connect } = require('net')
+const { connect } = require('node:net')
 
 test('requestTimeout should return 408', t => {
   t.plan(1)

test/close.test.js

@@ -1,7 +1,7 @@
 'use strict'
 
-const net = require('net')
-const http = require('http')
+const net = require('node:net')
+const http = require('node:http')
 const { test } = require('tap')
 const Fastify = require('..')
 const { Client } = require('undici')