@esmx/import 3.0.0-rc.31 → 3.0.0-rc.33

package/dist/error.d.ts

@@ -8,9 +8,7 @@ export declare class ModuleLoadingError extends Error {
 }
 export declare class CircularDependencyError extends ModuleLoadingError {
     constructor(message: string, moduleIds: string[], targetModule: string);
-    toString(): string;
 }
 export declare class FileReadError extends ModuleLoadingError {
     constructor(message: string, moduleIds: string[], targetModule: string, originalError?: Error);
-    toString(): string;
 }

package/dist/error.mjs

@@ -19,14 +19,14 @@ const getRelativeFromCwd = (filePath) => {
 };
 export const formatCircularDependency = (moduleIds, targetModule) => {
   const fullChain = [...moduleIds, targetModule];
-  return `${colorize(colorize("Circular dependency:", Colors.BOLD), Colors.RED)}
+  return `${colorize(colorize("Module dependency chain (circular reference found):", Colors.BOLD), Colors.RED)}
 ${fullChain.map((module, index) => {
     const isLastModule = index === fullChain.length - 1;
     const prefix = index === 0 ? "\u250C\u2500 " : index === fullChain.length - 1 ? "\u2514\u2500 " : "\u251C\u2500 ";
     const displayPath = getRelativeFromCwd(module);
     const isCircularModule = fullChain.filter((m) => m === module).length > 1;
     const coloredFile = isCircularModule ? colorize(colorize(displayPath, Colors.BOLD), Colors.RED) : colorize(displayPath, Colors.CYAN);
-    const suffix = isLastModule ? ` ${colorize("\u2190 circular", Colors.YELLOW)}` : "";
+    const suffix = isLastModule ? ` ${colorize("\u{1F504} Creates circular reference", Colors.YELLOW)}` : "";
     return `${colorize(prefix, Colors.GRAY)}${coloredFile}${suffix}`;
   }).join("\n")}`;
 };
@@ -37,21 +37,21 @@ export const formatModuleChain = (moduleIds, targetModule, originalError) => {
     result = `${colorize("Failed to load:", Colors.CYAN)} ${colorize(displayPath, Colors.RED)}`;
   } else {
     const chain = [...moduleIds, targetModule];
-    result = `${colorize(colorize("Import chain:", Colors.BOLD), Colors.CYAN)}
+    result = `${colorize(colorize("Module loading path:", Colors.BOLD), Colors.CYAN)}
 ${chain.map((module, index) => {
       const indent = "  ".repeat(index);
       const connector = index === 0 ? "" : "\u2514\u2500 ";
       const displayPath = getRelativeFromCwd(module);
       const isFailedFile = index === chain.length - 1;
       const coloredFile = isFailedFile ? colorize(colorize(displayPath, Colors.BOLD), Colors.RED) : colorize(displayPath, Colors.CYAN);
-      const status = isFailedFile ? ` ${colorize(colorize("\u2717 FAILED", Colors.BOLD), Colors.RED)}` : "";
+      const status = isFailedFile ? ` ${colorize(colorize("\u274C Loading failed", Colors.BOLD), Colors.RED)}` : "";
       return `${colorize(indent + connector, Colors.GRAY)}${coloredFile}${status}`;
     }).join("\n")}`;
   }
   if (originalError) {
     result += `
 
-${colorize("Cause:", Colors.YELLOW)} ${originalError.message}`;
+${colorize("Error details:", Colors.YELLOW)} ${originalError.message}`;
   }
   return result;
 };
@@ -62,27 +62,43 @@ export class ModuleLoadingError extends Error {
     this.targetModule = targetModule;
     this.originalError = originalError;
     this.name = "ModuleLoadingError";
+    Object.defineProperty(this, "moduleIds", {
+      value: moduleIds,
+      writable: false,
+      enumerable: false,
+      configurable: true
+    });
+    Object.defineProperty(this, "targetModule", {
+      value: targetModule,
+      writable: false,
+      enumerable: false,
+      configurable: true
+    });
+    if (originalError) {
+      Object.defineProperty(this, "originalError", {
+        value: originalError,
+        writable: false,
+        enumerable: false,
+        configurable: true
+      });
+    }
   }
 }
 export class CircularDependencyError extends ModuleLoadingError {
   constructor(message, moduleIds, targetModule) {
     super(message, moduleIds, targetModule);
     this.name = "CircularDependencyError";
-  }
-  toString() {
-    return `${this.name}: ${this.message}
+    this.stack = `${this.name}: ${message}
 
-${formatCircularDependency(this.moduleIds, this.targetModule)}`;
+${formatCircularDependency(moduleIds, targetModule)}`;
   }
 }
 export class FileReadError extends ModuleLoadingError {
   constructor(message, moduleIds, targetModule, originalError) {
     super(message, moduleIds, targetModule, originalError);
     this.name = "FileReadError";
-  }
-  toString() {
-    return `${this.name}: ${this.message}
+    this.stack = `${this.name}: ${message}
 
-${formatModuleChain(this.moduleIds, this.targetModule, this.originalError)}`;
+${formatModuleChain(moduleIds, targetModule, originalError)}`;
   }
 }

package/dist/error.test.mjs

@@ -31,6 +31,13 @@ describe("Module Loading Errors", () => {
       );
       expect(error.name).toBe("CircularDependencyError");
       expect(error.message).toBe("Test circular dependency");
+      expect(error.stack).toContain("Test circular dependency");
+      expect(error.stack).toContain(
+        "Module dependency chain (circular reference found):"
+      );
+      expect(error.stack).toContain("\u250C\u2500");
+      expect(error.stack).toContain("\u2514\u2500");
+      expect(error.stack).toContain("\u{1F504} Creates circular reference");
       expect(error.moduleIds).toEqual(moduleIds);
       expect(error.targetModule).toBe(targetModule);
       expect(error instanceof ModuleLoadingError).toBe(true);
@@ -44,14 +51,8 @@ describe("Module Loading Errors", () => {
         targetModule
       );
       const formatted = error.toString();
-      const relativeA = path.relative(process.cwd(), "/src/A.js");
-      const relativeB = path.relative(process.cwd(), "/src/B.js");
       expect(formatted).toContain("CircularDependencyError");
       expect(formatted).toContain("Circular dependency detected");
-      expect(formatted).toContain(`\u250C\u2500 ${relativeA}`);
-      expect(formatted).toContain(`\u251C\u2500 ${relativeB}`);
-      expect(formatted).toContain(`\u2514\u2500 ${relativeA}`);
-      expect(formatted).toContain("circular");
     });
   });
   describe("FileReadError", () => {
@@ -69,6 +70,14 @@ describe("Module Loading Errors", () => {
       );
       expect(error.name).toBe("FileReadError");
       expect(error.message).toBe("Failed to read module");
+      expect(error.stack).toContain("Failed to read module");
+      expect(error.stack).toContain("Module loading path:");
+      expect(error.stack).toContain("main.js");
+      expect(error.stack).toContain("App.js");
+      expect(error.stack).toContain("missing.js");
+      expect(error.stack).toContain("\u274C Loading failed");
+      expect(error.stack).toContain("Error details:");
+      expect(error.stack).toContain("ENOENT");
       expect(error.moduleIds).toEqual(moduleIds);
       expect(error.targetModule).toBe(targetModule);
       expect(error.originalError).toBe(originalError);
@@ -89,13 +98,6 @@ describe("Module Loading Errors", () => {
       const formatted = error.toString();
       expect(formatted).toContain("FileReadError");
       expect(formatted).toContain("Failed to read module");
-      expect(formatted).toContain("Import chain:");
-      expect(formatted).toContain("main.js");
-      expect(formatted).toContain("App.js");
-      expect(formatted).toContain("missing.js");
-      expect(formatted).toContain("\u2717 FAILED");
-      expect(formatted).toContain("Cause:");
-      expect(formatted).toContain("ENOENT");
     });
   });
   describe("Formatting Functions", () => {
@@ -106,12 +108,14 @@ describe("Module Loading Errors", () => {
       const relativeA = path.relative(process.cwd(), "/src/A.js");
       const relativeB = path.relative(process.cwd(), "/src/B.js");
       const relativeC = path.relative(process.cwd(), "/src/C.js");
-      expect(formatted).toContain("Circular dependency:");
+      expect(formatted).toContain(
+        "Module dependency chain (circular reference found):"
+      );
       expect(formatted).toContain(`\u250C\u2500 ${relativeA}`);
       expect(formatted).toContain(`\u251C\u2500 ${relativeB}`);
       expect(formatted).toContain(`\u251C\u2500 ${relativeC}`);
       expect(formatted).toContain(`\u2514\u2500 ${relativeA}`);
-      expect(formatted).toContain("circular");
+      expect(formatted).toContain("\u{1F504} Creates circular reference");
     });
     it("should format module chain correctly", () => {
       const moduleIds = ["/src/main.js", "/src/app.js"];
@@ -122,12 +126,12 @@ describe("Module Loading Errors", () => {
         targetModule,
         originalError
       );
-      expect(formatted).toContain("Import chain:");
+      expect(formatted).toContain("Module loading path:");
       expect(formatted).toContain("main.js");
       expect(formatted).toContain("app.js");
       expect(formatted).toContain("missing.js");
-      expect(formatted).toContain("\u2717 FAILED");
-      expect(formatted).toContain("Cause:");
+      expect(formatted).toContain("\u274C Loading failed");
+      expect(formatted).toContain("Error details:");
       expect(formatted).toContain("File not found");
     });
     it("should handle empty moduleIds in formatModuleChain", () => {
@@ -155,7 +159,9 @@ describe("Module Loading Errors", () => {
       expect(formatted).toContain(`  \u2514\u2500 ${relativeB}`);
       expect(formatted).toContain(`    \u2514\u2500 ${relativeC}`);
       expect(formatted).toContain(`      \u2514\u2500 ${relativeD}`);
-      expect(formatted).toContain(`        \u2514\u2500 ${relativeE} \u2717 FAILED`);
+      expect(formatted).toContain(
+        `        \u2514\u2500 ${relativeE} \u274C Loading failed`
+      );
     });
   });
 });

package/package.json

@@ -33,7 +33,7 @@
     },
     "devDependencies": {
         "@biomejs/biome": "1.9.4",
-        "@esmx/lint": "3.0.0-rc.31",
+        "@esmx/lint": "3.0.0-rc.33",
         "@types/node": "^24.0.0",
         "@vitest/coverage-v8": "3.2.4",
         "stylelint": "16.21.0",
@@ -41,7 +41,7 @@
         "unbuild": "3.5.0",
         "vitest": "3.2.4"
     },
-    "version": "3.0.0-rc.31",
+    "version": "3.0.0-rc.33",
     "type": "module",
     "private": false,
     "exports": {
@@ -60,5 +60,5 @@
         "template",
         "public"
     ],
-    "gitHead": "d472904500f5d697cd61b99b012570f6b063c580"
+    "gitHead": "b38d6e27a1e1df3e4a11474e91e5838ddf26a3c4"
 }

package/src/error.test.ts

@@ -36,7 +36,16 @@ describe('Module Loading Errors', () => {
             );
 
             expect(error.name).toBe('CircularDependencyError');
+            // Message is now clean and simple
             expect(error.message).toBe('Test circular dependency');
+            // Formatted content is in stack property
+            expect(error.stack).toContain('Test circular dependency');
+            expect(error.stack).toContain(
+                'Module dependency chain (circular reference found):'
+            );
+            expect(error.stack).toContain('┌─');
+            expect(error.stack).toContain('└─');
+            expect(error.stack).toContain('🔄 Creates circular reference');
             expect(error.moduleIds).toEqual(moduleIds);
             expect(error.targetModule).toBe(targetModule);
             expect(error instanceof ModuleLoadingError).toBe(true);
@@ -54,16 +63,9 @@ describe('Module Loading Errors', () => {
 
             const formatted = error.toString();
 
-            // Calculate expected relative paths
-            const relativeA = path.relative(process.cwd(), '/src/A.js');
-            const relativeB = path.relative(process.cwd(), '/src/B.js');
-
             expect(formatted).toContain('CircularDependencyError');
             expect(formatted).toContain('Circular dependency detected');
-            expect(formatted).toContain(`┌─ ${relativeA}`);
-            expect(formatted).toContain(`├─ ${relativeB}`);
-            expect(formatted).toContain(`└─ ${relativeA}`);
-            expect(formatted).toContain('circular');
+            // toString() now uses default Error behavior, so it only shows name and message
         });
     });
 
@@ -83,7 +85,17 @@ describe('Module Loading Errors', () => {
             );
 
             expect(error.name).toBe('FileReadError');
+            // Message is now clean and simple
             expect(error.message).toBe('Failed to read module');
+            // Formatted content is in stack property
+            expect(error.stack).toContain('Failed to read module');
+            expect(error.stack).toContain('Module loading path:');
+            expect(error.stack).toContain('main.js');
+            expect(error.stack).toContain('App.js');
+            expect(error.stack).toContain('missing.js');
+            expect(error.stack).toContain('❌ Loading failed');
+            expect(error.stack).toContain('Error details:');
+            expect(error.stack).toContain('ENOENT');
             expect(error.moduleIds).toEqual(moduleIds);
             expect(error.targetModule).toBe(targetModule);
             expect(error.originalError).toBe(originalError);
@@ -108,13 +120,7 @@ describe('Module Loading Errors', () => {
 
             expect(formatted).toContain('FileReadError');
             expect(formatted).toContain('Failed to read module');
-            expect(formatted).toContain('Import chain:');
-            expect(formatted).toContain('main.js');
-            expect(formatted).toContain('App.js');
-            expect(formatted).toContain('missing.js');
-            expect(formatted).toContain('✗ FAILED');
-            expect(formatted).toContain('Cause:');
-            expect(formatted).toContain('ENOENT');
+            // toString() now uses default Error behavior, so it only shows name and message
         });
     });
 
@@ -130,12 +136,14 @@ describe('Module Loading Errors', () => {
             const relativeB = path.relative(process.cwd(), '/src/B.js');
             const relativeC = path.relative(process.cwd(), '/src/C.js');
 
-            expect(formatted).toContain('Circular dependency:');
+            expect(formatted).toContain(
+                'Module dependency chain (circular reference found):'
+            );
             expect(formatted).toContain(`┌─ ${relativeA}`);
             expect(formatted).toContain(`├─ ${relativeB}`);
             expect(formatted).toContain(`├─ ${relativeC}`);
             expect(formatted).toContain(`└─ ${relativeA}`);
-            expect(formatted).toContain('circular');
+            expect(formatted).toContain('🔄 Creates circular reference');
         });
 
         it('should format module chain correctly', () => {
@@ -149,12 +157,12 @@ describe('Module Loading Errors', () => {
                 originalError
             );
 
-            expect(formatted).toContain('Import chain:');
+            expect(formatted).toContain('Module loading path:');
             expect(formatted).toContain('main.js');
             expect(formatted).toContain('app.js');
             expect(formatted).toContain('missing.js');
-            expect(formatted).toContain('✗ FAILED');
-            expect(formatted).toContain('Cause:');
+            expect(formatted).toContain('❌ Loading failed');
+            expect(formatted).toContain('Error details:');
             expect(formatted).toContain('File not found');
         });
 
@@ -190,7 +198,9 @@ describe('Module Loading Errors', () => {
             expect(formatted).toContain(`  └─ ${relativeB}`);
             expect(formatted).toContain(`    └─ ${relativeC}`);
             expect(formatted).toContain(`      └─ ${relativeD}`);
-            expect(formatted).toContain(`        └─ ${relativeE} ✗ FAILED`);
+            expect(formatted).toContain(
+                `        └─ ${relativeE} ❌ Loading failed`
+            );
         });
     });
 });

package/src/error.ts

@@ -38,7 +38,7 @@ export const formatCircularDependency = (
 ): string => {
     const fullChain = [...moduleIds, targetModule];
 
-    return `${colorize(colorize('Circular dependency:', Colors.BOLD), Colors.RED)}\n${fullChain
+    return `${colorize(colorize('Module dependency chain (circular reference found):', Colors.BOLD), Colors.RED)}\n${fullChain
         .map((module, index) => {
             const isLastModule = index === fullChain.length - 1;
             const prefix =
@@ -59,7 +59,7 @@ export const formatCircularDependency = (
                 : colorize(displayPath, Colors.CYAN);
 
             const suffix = isLastModule
-                ? ` ${colorize('← circular', Colors.YELLOW)}`
+                ? ` ${colorize('🔄 Creates circular reference', Colors.YELLOW)}`
                 : '';
 
             return `${colorize(prefix, Colors.GRAY)}${coloredFile}${suffix}`;
@@ -80,7 +80,7 @@ export const formatModuleChain = (
         result = `${colorize('Failed to load:', Colors.CYAN)} ${colorize(displayPath, Colors.RED)}`;
     } else {
         const chain = [...moduleIds, targetModule];
-        result = `${colorize(colorize('Import chain:', Colors.BOLD), Colors.CYAN)}\n${chain
+        result = `${colorize(colorize('Module loading path:', Colors.BOLD), Colors.CYAN)}\n${chain
             .map((module, index) => {
                 const indent = '  '.repeat(index);
                 const connector = index === 0 ? '' : '└─ ';
@@ -92,7 +92,7 @@ export const formatModuleChain = (
                     : colorize(displayPath, Colors.CYAN);
 
                 const status = isFailedFile
-                    ? ` ${colorize(colorize('✗ FAILED', Colors.BOLD), Colors.RED)}`
+                    ? ` ${colorize(colorize('❌ Loading failed', Colors.BOLD), Colors.RED)}`
                     : '';
 
                 return `${colorize(indent + connector, Colors.GRAY)}${coloredFile}${status}`;
@@ -101,7 +101,7 @@ export const formatModuleChain = (
     }
 
     if (originalError) {
-        result += `\n\n${colorize('Cause:', Colors.YELLOW)} ${originalError.message}`;
+        result += `\n\n${colorize('Error details:', Colors.YELLOW)} ${originalError.message}`;
     }
 
     return result;
@@ -117,6 +117,30 @@ export class ModuleLoadingError extends Error {
     ) {
         super(message);
         this.name = 'ModuleLoadingError';
+
+        // Hide auxiliary properties from enumeration to avoid cluttering error display
+        Object.defineProperty(this, 'moduleIds', {
+            value: moduleIds,
+            writable: false,
+            enumerable: false,
+            configurable: true
+        });
+
+        Object.defineProperty(this, 'targetModule', {
+            value: targetModule,
+            writable: false,
+            enumerable: false,
+            configurable: true
+        });
+
+        if (originalError) {
+            Object.defineProperty(this, 'originalError', {
+                value: originalError,
+                writable: false,
+                enumerable: false,
+                configurable: true
+            });
+        }
     }
 }
 
@@ -125,10 +149,9 @@ export class CircularDependencyError extends ModuleLoadingError {
     constructor(message: string, moduleIds: string[], targetModule: string) {
         super(message, moduleIds, targetModule);
         this.name = 'CircularDependencyError';
-    }
 
-    toString(): string {
-        return `${this.name}: ${this.message}\n\n${formatCircularDependency(this.moduleIds, this.targetModule)}`;
+        // Custom stack for clean error display
+        this.stack = `${this.name}: ${message}\n\n${formatCircularDependency(moduleIds, targetModule)}`;
     }
 }
 
@@ -142,9 +165,8 @@ export class FileReadError extends ModuleLoadingError {
     ) {
         super(message, moduleIds, targetModule, originalError);
         this.name = 'FileReadError';
-    }
 
-    toString(): string {
-        return `${this.name}: ${this.message}\n\n${formatModuleChain(this.moduleIds, this.targetModule, this.originalError)}`;
+        // Custom stack for clean error display
+        this.stack = `${this.name}: ${message}\n\n${formatModuleChain(moduleIds, targetModule, originalError)}`;
     }
 }