lib: replace var with let/const

PR-URL: #30440
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

Author: khoumani

lib/internal/modules/cjs/loader.js

@@ -153,7 +153,7 @@ Module.builtinModules = builtinModules;
 Module._cache=Object.create(null);
 Module._pathCache=Object.create(null);
 Module._extensions=Object.create(null);
-varmodulePaths=[];
+letmodulePaths=[];
 Module.globalPaths=[];
 
 letpatched=false;
@@ -341,7 +341,7 @@ function toRealPath(requestPath) {
 
 // Given a path, check if the file exists with any of the set extensions
 functiontryExtensions(p,exts,isMain){
-for(vari=0;i<exts.length;i++){
+for(leti=0;i<exts.length;i++){
 constfilename=tryFile(p+exts[i],isMain);
 
 if(filename){
@@ -474,22 +474,22 @@ Module._findPath = function(request, paths, isMain) {
 if(entry)
 returnentry;
 
-varexts;
-vartrailingSlash=request.length>0&&
+letexts;
+lettrailingSlash=request.length>0&&
 request.charCodeAt(request.length-1)===CHAR_FORWARD_SLASH;
 if(!trailingSlash){
 trailingSlash=/(?:^|\/)\.?\.$/.test(request);
 }
 
 // For each path
-for(vari=0;i<paths.length;i++){
+for(leti=0;i<paths.length;i++){
 // Don't search further if path doesn't exist
 constcurPath=paths[i];
 if(curPath&&stat(curPath)<1)continue;
-varbasePath=resolveExports(curPath,request,absoluteRequest);
-varfilename;
+constbasePath=resolveExports(curPath,request,absoluteRequest);
+letfilename;
 
-varrc=stat(basePath);
+constrc=stat(basePath);
 if(!trailingSlash){
 if(rc===0){// File.
 if(!isMain){
@@ -556,9 +556,7 @@ if (isWindows) {
 return[from+'node_modules'];
 
 constpaths=[];
-varp=0;
-varlast=from.length;
-for(vari=from.length-1;i>=0;--i){
+for(leti=from.length-1,p=0,last=from.length;i>=0;--i){
 constcode=from.charCodeAt(i);
 // The path segment separator check ('\' and '/') was used to get
 // node_modules path for every path segment.
@@ -597,9 +595,7 @@ if (isWindows) {
 // to be absolute. Doing a fully-edge-case-correct path.split
 // that works on both Windows and Posix is non-trivial.
 constpaths=[];
-varp=0;
-varlast=from.length;
-for(vari=from.length-1;i>=0;--i){
+for(leti=from.length-1,p=0,last=from.length;i>=0;--i){
 constcode=from.charCodeAt(i);
 if(code===CHAR_FORWARD_SLASH){
 if(p!==nmLen)
@@ -744,7 +740,7 @@ Module._resolveFilename = function(request, parent, isMain, options) {
 returnrequest;
 }
 
-varpaths;
+letpaths;
 
 if(typeofoptions==='object'&&options!==null){
 if(Array.isArray(options.paths)){
@@ -760,12 +756,12 @@ Module._resolveFilename = function(request, parent, isMain, options) {
 
 paths=[];
 
-for(vari=0;i<options.paths.length;i++){
+for(leti=0;i<options.paths.length;i++){
 constpath=options.paths[i];
 fakeParent.paths=Module._nodeModulePaths(path);
 constlookupPaths=Module._resolveLookupPaths(request,fakeParent);
 
-for(varj=0;j<lookupPaths.length;j++){
+for(letj=0;j<lookupPaths.length;j++){
 if(!paths.includes(lookupPaths[j]))
 paths.push(lookupPaths[j]);
 }
@@ -784,7 +780,7 @@ Module._resolveFilename = function(request, parent, isMain, options) {
 constfilename=Module._findPath(request,paths,isMain);
 if(!filename){
 constrequireStack=[];
-for(varcursor=parent;
+for(letcursor=parent;
 cursor;
 cursor=cursor.parent){
 requireStack.push(cursor.filename||cursor.id);
@@ -794,7 +790,7 @@ Module._resolveFilename = function(request, parent, isMain, options) {
 message=message+'\nRequire stack:\n- '+requireStack.join('\n- ');
 }
 // eslint-disable-next-line no-restricted-syntax
-varerr=newError(message);
+consterr=newError(message);
 err.code='MODULE_NOT_FOUND';
 err.requireStack=requireStack;
 throwerr;
@@ -858,7 +854,7 @@ Module.prototype.require = function(id) {
 
 // Resolved path to process.argv[1] will be lazily placed here
 // (needed for setting breakpoint when called with --inspect-brk)
-varresolvedArgv;
+letresolvedArgv;
 lethasPausedEntry=false;
 
 // Run the file contents in the correct scope or sandbox. Expose
@@ -928,7 +924,7 @@ Module.prototype._compile = function(content, filename) {
 compiledWrapper=compiled.function;
 }
 
-varinspectorWrapper=null;
+letinspectorWrapper=null;
 if(getOptionValue('--inspect-brk')&&process._eval==null){
 if(!resolvedArgv){
 // We enter the repl if we're not given a filename argument.
@@ -947,7 +943,7 @@ Module.prototype._compile = function(content, filename) {
 }
 constdirname=path.dirname(filename);
 constrequire=makeRequireFunction(this,redirects);
-varresult;
+letresult;
 constexports=this.exports;
 constthisValue=exports;
 constmodule=this;
@@ -1090,26 +1086,16 @@ function createRequire(filename) {
 Module.createRequire=createRequire;
 
 Module._initPaths=function(){
-varhomeDir;
-varnodePath;
-if(isWindows){
-homeDir=process.env.USERPROFILE;
-nodePath=process.env.NODE_PATH;
-}else{
-homeDir=safeGetenv('HOME');
-nodePath=safeGetenv('NODE_PATH');
-}
+consthomeDir=isWindows ? process.env.USERPROFILE : safeGetenv('HOME');
+constnodePath=isWindows ? process.env.NODE_PATH : safeGetenv('NODE_PATH');
 
-// $PREFIX/lib/node, where $PREFIX is the root of the Node.js installation.
-varprefixDir;
 // process.execPath is $PREFIX/bin/node except on Windows where it is
-// $PREFIX\node.exe.
-if(isWindows){
-prefixDir=path.resolve(process.execPath,'..');
-}else{
-prefixDir=path.resolve(process.execPath,'..','..');
-}
-varpaths=[path.resolve(prefixDir,'lib','node')];
+// $PREFIX\node.exe where $PREFIX is the root of the Node.js installation.
+constprefixDir=isWindows ?
+path.resolve(process.execPath,'..') :
+path.resolve(process.execPath,'..','..');
+
+letpaths=[path.resolve(prefixDir,'lib','node')];
 
 if(homeDir){
 paths.unshift(path.resolve(homeDir,'.node_libraries'));
@@ -1143,7 +1129,7 @@ Module._preloadModules = function(requests) {
 throwe;
 }
 }
-for(varn=0;n<requests.length;n++)
+for(letn=0;n<requests.length;n++)
 parent.require(requests[n]);
 };