chore: beautify error message

package.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://raw.githubusercontent.com/wzhudev/squirrel/master/src/schema/package.schema.json",
   "name": "@wendellhu/redi",
-  "version": "0.15.2",
+  "version": "0.15.3",
   "description": "A dependency library for TypeScript and JavaScript, along with a binding for React.",
   "scripts": {
     "test": "vitest run",

src/dependencyCollection.ts

@@ -48,11 +48,9 @@ export class DependencyNotFoundForModuleError extends RediError {
     const msg = `Cannot find "${prettyPrintIdentifier(id)}" registered by any injector. It is the ${index}th param of "${isIdentifierDecorator(toInstantiate)
       ? prettyPrintIdentifier(toInstantiate)
       : (toInstantiate as Ctor<any>).name
-      }". The stack of dependencies is: "${ResolvingStack.map((id) => prettyPrintIdentifier(id)).join(' -> ')}".`
+      }".`
 
     super(msg)
-
-    clearResolvingStack();
   }
 }
 
@@ -63,6 +61,8 @@ export class DependencyNotFoundError extends RediError {
     const msg = `Cannot find "${prettyPrintIdentifier(id)}" registered by any injector. The stack of dependencies is: "${ResolvingStack.map((id) => prettyPrintIdentifier(id)).join(' -> ')}".`
 
     super(msg)
+
+    clearResolvingStack();
   }
 }
 

src/injector.ts

@@ -309,20 +309,12 @@ export class Injector {
   ): T[] | T | null {
     this._ensureInjectorNotDisposed();
 
-    try {
-      const newResult = this._get(id, quantityOrLookup, lookUp)
-      if ((Array.isArray(newResult) && newResult.some((r) => isAsyncHook(r))) || isAsyncHook(newResult)) {
-        throw new GetAsyncItemFromSyncApiError(id)
-      }
-
-      return newResult as T | T[] | null
-    } catch (e: unknown) {
-      if (e instanceof DependencyNotFoundError) {
-        clearResolvingStack();
-      }
-
-      throw e;
+    const newResult = this._get(id, quantityOrLookup, lookUp)
+    if ((Array.isArray(newResult) && newResult.some((r) => isAsyncHook(r))) || isAsyncHook(newResult)) {
+      throw new GetAsyncItemFromSyncApiError(id)
     }
+
+    return newResult as T | T[] | null
   }
 
   private _get<T>(

test/core.spec.ts

@@ -34,7 +34,7 @@ describe('core', () => {
     class D { constructor(@Inject(C) private c: C) { } }
 
     const j = new Injector([[B], [C], [D]]);
-    expectToThrow(() => j.get(D), 'Cannot find "A" registered by any injector. It is the 0th param of "B". The stack of dependencies is: "D -> C -> B -> A"');
+    expectToThrow(() => j.get(D), 'Cannot find "A" registered by any injector. It is the 0th param of "B".');
   });
 
   describe('basics', () => {
@@ -434,7 +434,7 @@ describe('core', () => {
         }
 
         const j = new Injector([[B]])
-        expect(() => { j.get(B) }).toThrow('[redi]: Cannot find "A" registered by any injector. It is the 1th param of "B". The stack of dependencies is: "B -> A".');
+        expect(() => { j.get(B) }).toThrow('[redi]: Cannot find "A" registered by any injector. It is the 1th param of "B".');
       })
     })
 
@@ -492,7 +492,7 @@ describe('core', () => {
         ])
         expectToThrow(() => {
           j.get(b)
-        }, '[redi]: Cannot find "A" registered by any injector. It is the 0th param of "b". The stack of dependencies is: "b -> A".')
+        }, '[redi]: Cannot find "A" registered by any injector. It is the 0th param of "b".')
       })
     })
 
@@ -963,7 +963,7 @@ describe('core', () => {
 
       expectToThrow(
         () => child.get(bI),
-        '[redi]: Cannot find "cI" registered by any injector. It is the 1th param of "bI". The stack of dependencies is: "bI".'
+        '[redi]: Cannot find "cI" registered by any injector. It is the 1th param of "bI".'
       )
     })