lerna.json

@@ -3,5 +3,5 @@
   "packages": [
     "packages/*"
   ],
-  "version": "7.0.10"
+  "version": "7.0.11"
 }

packages/common/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nestjs/common",
-  "version": "7.0.10",
+  "version": "7.0.11",
   "description": "Nest - modern, fast, powerful node.js web framework (@common)",
   "author": "Kamil Mysliwiec",
   "homepage": "https://nestjs.com",

packages/core/errors/exceptions/invalid-module.exception.ts

@@ -2,8 +2,7 @@ import { INVALID_MODULE_MESSAGE } from '../messages';
 import { RuntimeException } from './runtime.exception';
 
 export class InvalidModuleException extends RuntimeException {
-  constructor(trace: any[]) {
-    const scope = (trace || []).map(module => module.name).join(' -> ');
-    super(INVALID_MODULE_MESSAGE`${scope}`);
+  constructor(parentModule: any, index: number, scope: any[]) {
+    super(INVALID_MODULE_MESSAGE(parentModule, index, scope));
   }
 }

packages/core/errors/exceptions/undefined-forwardref.exception.ts

@@ -0,0 +1,9 @@
+import { UNDEFINED_FORWARDREF_MESSAGE } from '../messages';
+import { RuntimeException } from './runtime.exception';
+import { Type } from '@nestjs/common';
+
+export class UndefinedForwardRefException extends RuntimeException {
+  constructor(scope: Type<any>[]) {
+    super(UNDEFINED_FORWARDREF_MESSAGE(scope));
+  }
+}

packages/core/errors/exceptions/undefined-module.exception.ts

@@ -0,0 +1,8 @@
+import { RuntimeException } from './runtime.exception';
+import { UNDEFINED_MODULE_MESSAGE } from '../messages';
+
+export class UndefinedModuleException extends RuntimeException {
+  constructor(parentModule: any, index: number, scope: any[]) {
+    super(UNDEFINED_MODULE_MESSAGE(parentModule, index, scope));
+  }
+}

packages/core/errors/messages.ts

@@ -37,6 +37,9 @@ const getDependencyName = (dependency: InjectorDependency): string =>
 const getModuleName = (module: Module) =>
   (module && getInstanceName(module.metatype)) || 'current';
 
+const stringifyScope = (scope: any[]): string =>
+  (scope || []).map(getInstanceName).join(' -> ');
+
 export const UNKNOWN_DEPENDENCIES_MESSAGE = (
   type: string | symbol,
   unknownDependencyContext: InjectorDependencyContext,
@@ -84,15 +87,43 @@ export const INVALID_MIDDLEWARE_MESSAGE = (
   name: string,
 ) => `The middleware doesn't provide the 'use' method (${name})`;
 
-export const INVALID_MODULE_MESSAGE = (
-  text: TemplateStringsArray,
-  scope: string,
-) =>
-  `Nest cannot create the module instance. Often, this is because of a circular dependency between modules. Use forwardRef() to avoid it.
+export const UNDEFINED_FORWARDREF_MESSAGE = (
+  scope: Type<any>[],
+) => `Nest cannot create the module instance. Often, this is because of a circular dependency between modules. Use forwardRef() to avoid it.
 
 (Read more: https://docs.nestjs.com/fundamentals/circular-dependency)
-Scope [${scope}]
-`;
+Scope [${stringifyScope(scope)}]
+  `;
+
+export const INVALID_MODULE_MESSAGE = (
+  parentModule: any,
+  index: number,
+  scope: any[],
+) => {
+  const parentModuleName = parentModule?.name || 'module';
+
+  return `Nest cannot create the ${parentModuleName} instance.
+Received an unexpected value at index [${index}] of the ${parentModuleName} "imports" array. 
+
+Scope [${stringifyScope(scope)}]`;
+};
+
+export const UNDEFINED_MODULE_MESSAGE = (
+  parentModule: any,
+  index: number,
+  scope: any[],
+) => {
+  const parentModuleName = parentModule?.name || 'module';
+
+  return `Nest cannot create the ${parentModuleName} instance.
+The module at index [${index}] of the ${parentModuleName} "imports" array is undefined.
+
+Potential causes:
+- A circular dependency between modules. Use forwardRef() to avoid it. Read more: https://docs.nestjs.com/fundamentals/circular-dependency
+- The module at index [${index}] is of type "undefined". Check your import statements and the type of the module.
+
+Scope [${stringifyScope(scope)}]`;
+};
 
 export const UNKNOWN_EXPORT_MESSAGE = (token = 'item', module: string) => {
   return `Nest cannot export a provider/module that is not a part of the currently processed module (${module}). Please verify whether the exported ${token} is available in this particular context.

packages/core/injector/container.ts

@@ -4,7 +4,6 @@ import { Injectable } from '@nestjs/common/interfaces/injectable.interface';
 import { Type } from '@nestjs/common/interfaces/type.interface';
 import { ApplicationConfig } from '../application-config';
 import { CircularDependencyException } from '../errors/exceptions/circular-dependency.exception';
-import { InvalidModuleException } from '../errors/exceptions/invalid-module.exception';
 import { UnknownModuleException } from '../errors/exceptions/unknown-module.exception';
 import { ExternalContextCreator } from '../helpers/external-context-creator';
 import { HttpAdapterHost } from '../helpers/http-adapter-host';
@@ -16,6 +15,7 @@ import { InternalProvidersStorage } from './internal-providers-storage';
 import { Module } from './module';
 import { ModuleTokenFactory } from './module-token-factory';
 import { ModulesContainer } from './modules-container';
+import { UndefinedForwardRefException } from '../errors/exceptions/undefined-forwardref.exception';
 
 export class NestContainer {
   private readonly globalModules = new Set<Module>();
@@ -55,8 +55,10 @@ export class NestContainer {
     metatype: Type<any> | DynamicModule | Promise<DynamicModule>,
     scope: Type<any>[],
   ): Promise<Module> {
+    // In DependenciesScanner#scanForModules we already check for undefined or invalid modules
+    // We sill need to catch the edge-case of `forwardRef(() => undefined)`
     if (!metatype) {
-      throw new InvalidModuleException(scope);
+      throw new UndefinedForwardRefException(scope);
     }
     const { type, dynamicMetadata, token } = await this.moduleCompiler.compile(
       metatype,

packages/core/injector/instance-wrapper.ts

@@ -256,9 +256,11 @@ export class InstanceWrapper<T = any> {
   ): boolean {
     const isDependencyTreeStatic = this.isDependencyTreeStatic();
 
-    return ((!isDependencyTreeStatic &&
+    return (
+      !isDependencyTreeStatic &&
       contextId !== STATIC_CONTEXT &&
-      (!this.isTransient || (this.isTransient && inquirer))) as any) as boolean;
+      (!this.isTransient || (this.isTransient && !!inquirer))
+    );
   }
 
   public isLazyTransient(
@@ -299,7 +301,8 @@ export class InstanceWrapper<T = any> {
     return (
       this.isDependencyTreeStatic() &&
       contextId === STATIC_CONTEXT &&
-      (!this.isTransient || (isStaticTransient && !!inquirer))
+      (!this.isTransient ||
+        (isStaticTransient && !!inquirer && !inquirer.isTransient))
     );
   }
 

packages/core/injector/module-ref.ts

@@ -1,5 +1,6 @@
 import { Type } from '@nestjs/common';
 import { UnknownElementException } from '../errors/exceptions/unknown-element.exception';
+import { getClassScope } from '../helpers/get-class-scope';
 import { NestContainer } from './container';
 import { ContainerScanner } from './container-scanner';
 import { Injector } from './injector';
@@ -38,8 +39,8 @@ export abstract class ModuleRef {
     const wrapper = new InstanceWrapper({
       name: type && type.name,
       metatype: type,
-      instance: undefined,
       isResolved: false,
+      scope: getClassScope(type),
       host: moduleRef,
     });
     return new Promise<T>(async (resolve, reject) => {

packages/core/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nestjs/core",
-  "version": "7.0.10",
+  "version": "7.0.11",
   "description": "Nest - modern, fast, powerful node.js web framework (@core)",
   "author": "Kamil Mysliwiec",
   "license": "MIT",
@@ -36,7 +36,7 @@
     "uuid": "8.0.0"
   },
   "devDependencies": {
-    "@nestjs/common": "7.0.10"
+    "@nestjs/common": "7.0.11"
   },
   "peerDependencies": {
     "@nestjs/common": "^7.0.0",

packages/core/scanner.ts

@@ -43,6 +43,8 @@ import { NestContainer } from './injector/container';
 import { InstanceWrapper } from './injector/instance-wrapper';
 import { Module } from './injector/module';
 import { MetadataScanner } from './metadata-scanner';
+import { InvalidModuleException } from './errors/exceptions/invalid-module.exception';
+import { UndefinedModuleException } from './errors/exceptions/undefined-module.exception';
 
 interface ApplicationProviderWrapper {
   moduleKey: string;
@@ -90,7 +92,14 @@ export class DependenciesScanner {
           ...((module as DynamicModule).imports || []),
         ];
 
-    for (const innerModule of modules) {
+    for (const [index, innerModule] of modules.entries()) {
+      // In case of a circular dependency (ES module system), JavaScript will resolve the type to `undefined`.
+      if (innerModule === undefined) {
+        throw new UndefinedModuleException(module, index, scope);
+      }
+      if (!innerModule) {
+        throw new InvalidModuleException(module, index, scope);
+      }
       if (ctxRegistry.includes(innerModule)) {
         continue;
       }