fs: make process.binding('fs') internal

Refs: #22160

PR-URL: #22478
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Denys Otrishko <[email protected]>

Author: Masashi Hirano

lib/fs.js

@@ -42,7 +42,7 @@ const {
 const { _extend } = require('util');
 const pathModule = require('path');
 const { isArrayBufferView } = require('internal/util/types');
-const binding = process.binding('fs');
+const binding = internalBinding('fs');
 const { Buffer, kMaxLength } = require('buffer');
 const errors = require('internal/errors');
 const {

lib/internal/bootstrap/loaders.js

@@ -81,6 +81,7 @@
     'constants',
     'contextify',
     'crypto',
+    'fs',
     'fs_event_wrap',
     'http_parser',
     'icu',

lib/internal/fs/promises.js

@@ -7,7 +7,7 @@ const {
   S_IFMT,
   S_IFREG
 } = internalBinding('constants').fs;
-const binding = process.binding('fs');
+const binding = internalBinding('fs');
 const { Buffer, kMaxLength } = require('buffer');
 const {
   ERR_FS_FILE_TOO_LARGE,

lib/internal/fs/read_file_context.js

@@ -1,7 +1,8 @@
 'use strict';
 
 const { Buffer } = require('buffer');
-const { FSReqCallback, close, read } = process.binding('fs');
+
+const { FSReqCallback, close, read } = internalBinding('fs');
 
 const kReadFileBufferLength = 8 * 1024;
 

lib/internal/fs/streams.js

@@ -3,7 +3,7 @@
 const {
   FSReqCallback,
   writeBuffers
-} = process.binding('fs');
+} = internalBinding('fs');
 const {
   ERR_INVALID_ARG_TYPE,
   ERR_OUT_OF_RANGE

lib/internal/fs/watchers.js

@@ -4,7 +4,7 @@ const errors = require('internal/errors');
 const {
   kFsStatsFieldsNumber,
   StatWatcher: _StatWatcher
-} = process.binding('fs');
+} = internalBinding('fs');
 const { FSEvent } = internalBinding('fs_event_wrap');
 const { UV_ENOSPC } = internalBinding('uv');
 const { EventEmitter } = require('events');

lib/internal/http2/core.js

@@ -118,7 +118,7 @@ const {
 } = require('internal/timers');
 const { isArrayBufferView } = require('internal/util/types');
 
-const { FileHandle } = process.binding('fs');
+const { FileHandle } = internalBinding('fs');
 const binding = internalBinding('http2');
 const {
   ShutdownWrap,

lib/internal/modules/cjs/loader.js

@@ -33,7 +33,7 @@ const { URL } = require('url');
 const {
   internalModuleReadJSON,
   internalModuleStat
-} = process.binding('fs');
+} = internalBinding('fs');
 const { safeGetenv } = internalBinding('util');
 const {
   makeRequireFunction,

lib/internal/net.js

@@ -1,7 +1,7 @@
 'use strict';
 
 const Buffer = require('buffer').Buffer;
-const { writeBuffer } = process.binding('fs');
+const { writeBuffer } = internalBinding('fs');
 const errors = require('internal/errors');
 
 // IPv4 Segment

src/node_file.cc

@@ -2236,4 +2236,4 @@ void Initialize(Local<Object> target,
 
 }  // end namespace node
 
-NODE_BUILTIN_MODULE_CONTEXT_AWARE(fs, node::fs::Initialize)
+NODE_MODULE_CONTEXT_AWARE_INTERNAL(fs, node::fs::Initialize)