Merge pull request #13370 from nestjs/revert-13056-perf/12914-readabl…

…e-error-on-undefined-token

Revert "perf(common): Improve error handling for undefined tokens"

packages/common/decorators/core/inject.decorator.ts

@@ -39,12 +39,6 @@ export function Inject<T = any>(
   return (target: object, key: string | symbol | undefined, index?: number) => {
     const type = token || Reflect.getMetadata('design:type', target, key);
 
-    if (!type) {
-      throw new Error(`Token is undefined at index: ${index}. This often occurs due to circular dependencies.
-Ensure there are no circular dependencies in your files or barrel files. 
-For more details, refer to https://trilon.io/blog/avoiding-circular-dependencies-in-nestjs.`);
-    }
-
     if (!isUndefined(index)) {
       let dependencies =
         Reflect.getMetadata(SELF_DECLARED_DEPS_METADATA, target) || [];

packages/common/test/decorators/inject.decorator.spec.ts

@@ -22,14 +22,4 @@ describe('@Inject', () => {
     ];
     expect(metadata).to.be.eql(expectedMetadata);
   });
-
-  it('should throw an error when token is undefined', () => {
-    const defineInvalidClass = () => {
-      class Test {
-        constructor(@Inject(undefined) invalidParam) {}
-      }
-    };
-
-    expect(defineInvalidClass).to.throw(/^Token is undefined/);
-  });
 });