fs: lazily load ReadFileContext

gurgunday · ruyadorno · commit 3a55bd9448e0 · 2025-01-05T12:57:30.000-05:00
PR-URL: #55998 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: LiviaMedeiros <livia@cirno.name> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Jason Zhang <xzha4350@gmail.com>
diff --git a/lib/fs.js b/lib/fs.js
@@ -162,6 +162,7 @@ let WriteStream;
 let rimraf;
 let rimrafSync;
 let kResistStopPropagation;
+let ReadFileContext;
 
 // These have to be separate because of how graceful-fs happens to do it's
 // monkeypatching.
@@ -363,7 +364,7 @@ function readFile(path, options, callback){
 callback ||= options;
 validateFunction(callback, 'cb');
 options = getOptions(options,{flag: 'r' });
- const ReadFileContext = require('internal/fs/read/context');
+ ReadFileContext ??= require('internal/fs/read/context');
 const context = new ReadFileContext(callback, options.encoding);
 context.isUserFd = isFd(path); // File descriptor ownership
 

-Original file line number
+Diff line change
 letrimraf;
 letrimrafSync;
 letkResistStopPropagation;
 +letReadFileContext;
 // These have to be separate because of how graceful-fs happens to do it's
 // monkeypatching.
 callback||=options;
 validateFunction(callback,'cb');
 options=getOptions(options,{flag: 'r'});
 -constReadFileContext=require('internal/fs/read/context');
 +ReadFileContext??=require('internal/fs/read/context');
 constcontext=newReadFileContext(callback,options.encoding);
 context.isUserFd=isFd(path);// File descriptor ownership