small changes

Author: FogelAI

src/barrel.js

@@ -91,7 +91,6 @@ class BarrelFile {
       this.exportMapping = {};
       this.defaultPatternExport = new DefaultPatternExport();
       this.importMapping = {};
-      this.defaultPatternExport.numOfDefaultPatternUsed = 0;
     }
 
     handleExportNamedDeclaration(node) {
@@ -222,7 +221,6 @@ class BarrelFile {
         if (this.defaultPatternExport.numOfDefaultPatternUsed === 1) {
           this.exportMapping[this.defaultPatternExport.firstSpecifier.exportedName] = this.defaultPatternExport.firstSpecifier;
           this.defaultPatternExport = new DefaultPatternExport();
-          this.defaultPatternExport.isDefaultPatternCreated = true;
         }
         delete this.defaultPatternExport.numOfDefaultPatternUsed;
         delete this.defaultPatternExport.firstSpecifier;

src/path.js

@@ -66,18 +66,10 @@ class PathFunctions {
     }
 
     static removeLastSegment(path) {
-      // path.split("\\").slice(0,-1).join("\\");
-      // substring is more performant than split, in a half of the time.
       return path.substring(0, ospath.normalize(path).lastIndexOf(ospath.sep));
     }    
 
     static getAbsolutePath(path, from=process.cwd()) {
-      // https://github.com/nodejs/node/blob/main/doc/api/modules.md#loading-from-node_modules-folders
-      // solution for require function for ES modules
-      // https://stackoverflow.com/questions/54977743/do-require-resolve-for-es-modules
-      // https://stackoverflow.com/a/50053801
-      // import { createRequire } from "module";
-      // const require = createRequire(import.meta.url);
       if (ospath.isAbsolute(path)) return path;
       let currentDir = from;
       if (!PathFunctions.isNodeModule(path)) return ospath.join(currentDir, path);

src/resolver.js

@@ -26,9 +26,6 @@ class Resolver {
     }
 
     getTargetPathType(target, from) {
-      // if(isFromLocal && isToPackage) isCJS
-      // if(isFromLocal && isToLocal) isESM
-      // if(isFromPackage && isToPackage) isESM
       if (this.isExtensionFile(target, "cjs") || (!PathFunctions.isNodeModule(from) && PathFunctions.isNodeModule(target) && !this.isExtensionFile(target, "mjs"))) {
         return "CJS";
       } else {
@@ -67,7 +64,6 @@ class Resolver {
 
     getFilePathWithExtension(path) {
       const ext = this.extensions.find((ext)=> PathFunctions.fileExists(path + ext));
-      // can't use "if (!ext)" because ext can be equal to empty string that in this case should be true, but for javascript it's false
       if (ext === undefined) return null;
       const resolvedPath = path + ext;
       return resolvedPath;