Use node: for Node.js builtin modules

Use the the `node:` protocol when importing Node.js builtin modules
and enable rule in eslint to enforce.

The `node:` protocol for Node.js builtin modules has been available
since Node.js v14.

See: https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/enforce-node-protocol-usage.md
(cherry picked from commit af71c152c1)

.eslintrc.json

@@ -253,6 +253,10 @@
             "forceSingleLine": true
           }
         ],
+        "import/enforce-node-protocol-usage": [
+          "error",
+          "always"
+        ],
 
         "unused-imports/no-unused-imports": "error",
         "lodash/import-scope": [

cypress/plugins/index.ts

@@ -1,4 +1,4 @@
-const fs = require('fs');
+const fs = require('node:fs');
 
 // These two global variables are used to store information about the REST API used
 // by these e2e tests. They are filled out prior to running any tests in the before()

lint/generate-docs.ts

@@ -12,8 +12,8 @@ import {
   readFileSync,
   rmSync,
   writeFileSync,
-} from 'fs';
+} from 'node:fs';
-import { join } from 'path';
+import { join } from 'node:path';
 
 import { default as htmlPlugin } from './src/rules/html';
 import { default as tsPlugin } from './src/rules/ts';

lint/src/util/theme-support.ts

@@ -6,10 +6,11 @@
  * http://www.dspace.org/license/
  */
 
+import { readFileSync } from 'node:fs';
+import { basename } from 'node:path';
+
 import { TSESTree } from '@typescript-eslint/utils';
 import { RuleContext } from '@typescript-eslint/utils/ts-eslint';
-import { readFileSync } from 'fs';
-import { basename } from 'path';
 import ts, { Identifier } from 'typescript';
 
 import {

src/app/core/services/server-xhr.service.ts

@@ -6,13 +6,14 @@
  * http://www.dspace.org/license/
  */
 
-import { XhrFactory } from '@angular/common';
-import { Injectable } from '@angular/core';
 import {
   Agent as HttpAgent,
   AgentOptions as HttpAgentOptions,
-} from 'http';
+} from 'node:http';
-import { Agent as HttpsAgent } from 'https';
+import { Agent as HttpsAgent } from 'node:https';
+
+import { XhrFactory } from '@angular/common';
+import { Injectable } from '@angular/core';
 import {
   prototype,
   XMLHttpRequest,

src/backend/api.ts

@@ -8,7 +8,7 @@ import ITEMS from './data/items.json';
 import { fakeDataBase } from './db';
 
 const { Router } = require('express');
-const util = require('util');
+const util = require('node:util');
 
 // you would use cookies/token etc
 const USER_ID = 'f9d98cf1-1b96-464e-8755-bcc2a5c09077'; // hardcoded as an example

src/config/config.server.ts

@@ -1,16 +1,17 @@
+import {
+  existsSync,
+  readFileSync,
+  writeFileSync,
+} from 'node:fs';
+import { join } from 'node:path';
+
 import {
   blue,
   bold,
   green,
   red,
 } from 'colors';
-import {
-  existsSync,
-  readFileSync,
-  writeFileSync,
-} from 'fs';
 import { load } from 'js-yaml';
-import { join } from 'path';
 
 import { isNotEmpty } from '../app/shared/empty.util';
 import { AppConfig } from './app-config.interface';

src/ngx-translate-loaders/translate-server.loader.ts

@@ -1,6 +1,7 @@
+import { readFileSync } from 'node:fs';
+
 import { TransferState } from '@angular/core';
 import { TranslateLoader } from '@ngx-translate/core';
-import { readFileSync } from 'fs';
 import {
   Observable,
   of,