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

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

See fastify/fastify-static#407

* try to fix tests

---------

Co-authored-by: Uzlopak <aras.abbasi@googlemail.com>

Author: Fdawgs

index.js

@@ -2,11 +2,11 @@
 
 const spawn = require('cross-spawn')
 const which = require('which')
-const path = require('path')
-const util = require('util')
-const tty = require('tty')
+const path = require('node:path')
+const util = require('node:util')
+const tty = require('node:tty')
 
-const fs = require('fs')
+const fs = require('node:fs')
 
 /**
  * Representation of a hook runner.

install.js

@@ -3,9 +3,9 @@
 //
 // Compatibility with older node.js as path.exists got moved to `fs`.
 //
-const fs = require('fs')
-const path = require('path')
-const os = require('os')
+const fs = require('node:fs')
+const path = require('node:path')
+const os = require('node:os')
 const hook = path.join(__dirname, 'hook')
 const root = path.resolve(__dirname, '..', '..', '..')
 const exists = fs.existsSync || path.existsSync

test.js

@@ -1,7 +1,7 @@
 'use strict'
 const t = require('tap')
 const Hook = require('./')
-const tty = require('tty')
+const tty = require('node:tty')
 const ttySupportColor = tty.isatty(process.stdout.fd)
 
 const proxyquire = require('proxyquire')
@@ -147,7 +147,7 @@ t.test('pre-commit', function (t) {
       t.plan(1)
 
       const Hook = proxyquire('.', {
-        fs: {
+        'node:fs': {
           existsSync () {
             return true
           },
@@ -168,7 +168,7 @@ t.test('pre-commit', function (t) {
       t.plan(4)
 
       let Hook = proxyquire('.', {
-        fs: {
+        'node:fs': {
           existsSync () {
             return true
           },
@@ -181,7 +181,7 @@ t.test('pre-commit', function (t) {
       hook = new Hook(exit)
 
       Hook = proxyquire('.', {
-        fs: {
+        'node:fs': {
           existsSync () { return true },
           readFileSync () {
             return Buffer.from('{ "bad": [json }')

uninstall.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 exists = fs.existsSync || path.existsSync
 const root = path.resolve(__dirname, '..', '..', '..')
 let git = path.resolve(root, '.git')