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

)

* perf: use `node:` prefix to bypass require.cache call for builtins

See fastify/fastify-static#407

* docs: use `node:` prefix to bypass require.cache call for builtins

README.md

@@ -254,7 +254,7 @@ To use include files please extend your template options as follows:
 
 ```js
 // get a reference to resolve
-const resolve = require("path").resolve;
+const resolve = require('node:path').resolve;
 // other code ...
 // in template engine options configure how to resolve templates folder
 options: {
@@ -344,7 +344,7 @@ To configure liquid you need to pass the engine instance as engine option:
 
 ```js
 const { Liquid } = require("liquidjs");
-const path = require("path");
+const path = require('node:path');
 
 const engine = new Liquid({
   root: path.join(__dirname, "templates"),
@@ -370,7 +370,7 @@ This behaviour is recommended by the doT team [here](https://github.com/olado/do
 To make it possible it is necessary to provide a `root` or `templates` option with the path to the template directory.
 
 ```js
-const path = require("path");
+const path = require('node:path');
 
 fastify.register(require("@fastify/view"), {
   engine: {

examples/example-async.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const { promisify } = require('util')
+const { promisify } = require('node:util')
 const sleep = promisify(setTimeout)
 const templates = 'templates'
 

examples/example-ejs-with-some-options.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const fastify = require('fastify')()
-const resolve = require('path').resolve
+const resolve = require('node:path').resolve
 const templatesFolder = 'templates'
 const data = { text: 'Hello from EJS Templates' }
 

index.js

@@ -1,14 +1,14 @@
 'use strict'
 
 const fp = require('fastify-plugin')
-const readFile = require('fs').readFile
-const accessSync = require('fs').accessSync
-const existsSync = require('fs').existsSync
-const mkdirSync = require('fs').mkdirSync
-const readdirSync = require('fs').readdirSync
-const resolve = require('path').resolve
-const join = require('path').join
-const { basename, dirname, extname } = require('path')
+const readFile = require('node:fs').readFile
+const accessSync = require('node:fs').accessSync
+const existsSync = require('node:fs').existsSync
+const mkdirSync = require('node:fs').mkdirSync
+const readdirSync = require('node:fs').readdirSync
+const resolve = require('node:path').resolve
+const join = require('node:path').join
+const { basename, dirname, extname } = require('node:path')
 const HLRU = require('hashlru')
 const supportedEngines = ['ejs', 'nunjucks', 'pug', 'handlebars', 'mustache', 'art-template', 'twig', 'liquid', 'dot', 'eta']
 

test/helper.js

@@ -3,7 +3,7 @@
 const POV = require('..')
 const Fastify = require('fastify')
 const minifier = require('html-minifier')
-const fs = require('fs')
+const fs = require('node:fs')
 const dot = require('dot')
 const handlebars = require('handlebars')
 const { Liquid } = require('liquidjs')

test/test-art-template.js

@@ -4,7 +4,7 @@ const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
 const Fastify = require('fastify')
-const path = require('path')
+const path = require('node:path')
 
 test('reply.view with art-template engine and custom templates folder', t => {
   t.plan(6)

test/test-dot.js

@@ -4,8 +4,8 @@ const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
 const Fastify = require('fastify')
-const { existsSync, rmdirSync } = require('fs')
-const { join } = require('path')
+const { existsSync, rmdirSync } = require('node:fs')
+const { join } = require('node:path')
 const pino = require('pino')
 const split = require('split2')
 const compileOptions = {

test/test-ejs-async.js

@@ -4,7 +4,7 @@ const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
 const Fastify = require('fastify')
-const fs = require('fs')
+const fs = require('node:fs')
 
 test('reply.view with ejs engine and async: true (global option)', t => {
   t.plan(6)

test/test-ejs-with-snapshot.js

@@ -4,7 +4,7 @@ const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
 const Fastify = require('fastify')
-const path = require('path')
+const path = require('node:path')
 const ejs = require('ejs')
 const templatesFolder = 'templates'
 const options = {

test/test-ejs.js

@@ -4,8 +4,8 @@ const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
 const Fastify = require('fastify')
-const fs = require('fs')
-const path = require('path')
+const fs = require('node:fs')
+const path = require('node:path')
 const minifier = require('html-minifier')
 const minifierOpts = {
   removeComments: true,
@@ -789,7 +789,7 @@ test('reply.view with ejs engine, template folder specified, include files (ejs
   t.plan(7)
   const fastify = Fastify()
   const ejs = require('ejs')
-  const resolve = require('path').resolve
+  const resolve = require('node:path').resolve
   const templatesFolder = 'templates'
   const options = {
     filename: resolve(templatesFolder), // needed for include files to be resolved in include directive ...
@@ -839,7 +839,7 @@ test('reply.view with ejs engine, templates with folder specified, include files
   t.plan(7)
   const fastify = Fastify()
   const ejs = require('ejs')
-  const resolve = require('path').resolve
+  const resolve = require('node:path').resolve
   const templatesFolder = 'templates'
   const options = {
     filename: resolve(templatesFolder),
@@ -888,7 +888,7 @@ test('reply.view with ejs engine, templates with folder specified, include files
   t.plan(7)
   const fastify = Fastify()
   const ejs = require('ejs')
-  const resolve = require('path').resolve
+  const resolve = require('node:path').resolve
   const templatesFolder = 'templates'
   const options = {
     filename: resolve(templatesFolder),
@@ -936,7 +936,7 @@ test('reply.view with ejs engine, templates with folder specified, include files
   t.plan(7)
   const fastify = Fastify()
   const ejs = require('ejs')
-  const resolve = require('path').resolve
+  const resolve = require('node:path').resolve
   const templatesFolder = 'templates'
   const options = {
     filename: resolve(templatesFolder),
@@ -986,7 +986,7 @@ test('reply.view with ejs engine, templates with folder specified, include files
   t.plan(7)
   const fastify = Fastify()
   const ejs = require('ejs')
-  const resolve = require('path').resolve
+  const resolve = require('node:path').resolve
   const templatesFolder = 'templates'
   const options = {
     filename: resolve(templatesFolder),
@@ -1035,7 +1035,7 @@ test('reply.view with ejs engine, templates with folder specified, include files
   t.plan(7)
   const fastify = Fastify()
   const ejs = require('ejs')
-  const resolve = require('path').resolve
+  const resolve = require('node:path').resolve
   const templatesFolder = 'templates'
   const options = {
     filename: resolve(templatesFolder),

test/test-eta.js

@@ -4,8 +4,8 @@ const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
 const Fastify = require('fastify')
-const fs = require('fs')
-const path = require('path')
+const fs = require('node:fs')
+const path = require('node:path')
 
 const pointOfView = require('../index')
 const { Eta } = require('eta')

test/test-handlebars.js

@@ -4,8 +4,8 @@ const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
 const Fastify = require('fastify')
-const fs = require('fs')
-const { join } = require('path')
+const fs = require('node:fs')
+const { join } = require('node:path')
 const proxyquire = require('proxyquire')
 
 require('./helper').handleBarsHtmlMinifierTests(t, true)
@@ -513,7 +513,7 @@ test('reply.view with ejs engine, template folder specified, include files (ejs
   t.plan(7)
   const fastify = Fastify()
   const ejs = require('ejs')
-  const resolve = require('path').resolve
+  const resolve = require('node:path').resolve
   const templatesFolder = 'templates'
   const options = {
     filename: resolve(templatesFolder), // needed for include files to be resolved in include directive ...
@@ -563,7 +563,7 @@ test('reply.view with ejs engine, templates with folder specified, include files
   t.plan(7)
   const fastify = Fastify()
   const ejs = require('ejs')
-  const resolve = require('path').resolve
+  const resolve = require('node:path').resolve
   const templatesFolder = 'templates'
   const options = {
     filename: resolve(templatesFolder),

test/test-mustache.js

@@ -4,7 +4,7 @@ const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
 const Fastify = require('fastify')
-const fs = require('fs')
+const fs = require('node:fs')
 const minifier = require('html-minifier')
 const proxyquire = require('proxyquire')
 const minifierOpts = {

test/test-nunjucks.js

@@ -4,7 +4,7 @@ const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
 const Fastify = require('fastify')
-const path = require('path')
+const path = require('node:path')
 
 require('./helper').nunjucksHtmlMinifierTests(t, true)
 require('./helper').nunjucksHtmlMinifierTests(t, false)

test/test-pug.js

@@ -4,7 +4,7 @@ const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
 const Fastify = require('fastify')
-const fs = require('fs')
+const fs = require('node:fs')
 const proxyquire = require('proxyquire')
 
 require('./helper').pugHtmlMinifierTests(t, true)

test/test.js

@@ -1,11 +1,11 @@
 'use strict'
 
-const os = require('os')
+const os = require('node:os')
 const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
-const path = require('path')
-const fs = require('fs')
+const path = require('node:path')
+const fs = require('node:fs')
 const Fastify = require('fastify')
 
 test('fastify.view exist', t => {