@nestjs/core 11.1.5 → 11.1.7

package/Readme.md

@@ -8,7 +8,7 @@
   <p align="center">A progressive <a href="https://nodejs.org" target="_blank">Node.js</a> framework for building efficient and scalable server-side applications.</p>
     <p align="center">
 <a href="https://www.npmjs.com/~nestjscore" target="_blank"><img src="https://img.shields.io/npm/v/@nestjs/core.svg" alt="NPM Version" /></a>
-<a href="https://www.npmjs.com/~nestjscore" target="_blank"><img src="https://img.shields.io/npm/l/@nestjs/core.svg" alt="Package License" /></a>
+<a href="https://github.com/nestjs/nest/blob/master/LICENSE" target="_blank"><img src="https://img.shields.io/npm/l/@nestjs/core.svg" alt="Package License" /></a>
 <a href="https://www.npmjs.com/~nestjscore" target="_blank"><img src="https://img.shields.io/npm/dm/@nestjs/common.svg" alt="NPM Downloads" /></a>
 <a href="https://circleci.com/gh/nestjs/nest" target="_blank"><img src="https://img.shields.io/circleci/build/github/nestjs/nest/master" alt="CircleCI" /></a>
 <a href="https://discord.gg/G7Qnnhy" target="_blank"><img src="https://img.shields.io/badge/discord-online-brightgreen.svg" alt="Discord"/></a>
@@ -78,10 +78,7 @@ Nest is an MIT-licensed open source project. It can grow thanks to the sponsors
 </tr>
 <tr>
 <td><a href="https://snyk.co/nestjs" target="_blank"><img src="https://nestjs.com/img/logos/snyk-logo-black.png" width="185" valign="middle" /></a></td>
-<td><a href="https://fuseautotech.com/" target="_blank"><img src="https://nestjs.com/img/logos/fuse-logo.svg" width="105" valign="middle" /></a></td>
-<td><a href="https://ridicorp.com/career/" target="_blank"><img src="https://nestjs.com/img/logos/ridi-logo.svg" width="105" valign="middle" /></a></td>
 <td><a href="https://www.movavi.com/imovie-for-windows.html" target="_blank"><img src="https://nestjs.com/img/logos/movavi-logo.svg" width="105" valign="middle" /></a></td>
-<td><a href="https://skunk.team" target="_blank"><img src="https://nestjs.com/img/logos/skunk-logo.png" height="60" valign="middle" /></a></td>
 </tr>
 </table>
 
@@ -91,10 +88,10 @@ Nest is an MIT-licensed open source project. It can grow thanks to the sponsors
 <tr>
 <td><a href="https://www.mercedes-benz.com/" target="_blank"><img src="https://nestjs.com/img/logos/mercedes-logo.png" width="100" valign="middle" /></a></td>
 <td><a href="https://www.dinii.jp/" target="_blank"><img src="https://nestjs.com/img/logos/dinii-logo.png" width="65" valign="middle" /></a></td>
-<td><a href="https://bloodycase.com/?promocode=NEST" target="_blank"><img src="https://nestjs.com/img/logos/bloodycase-logo.png" width="65" valign="middle" /></a></td>
 <td><a href="https://handsontable.com/docs/react-data-grid/?utm_source=NestJS_GH&utm_medium=sponsorship&utm_campaign=library_sponsorship_2024" target="_blank"><img src="https://nestjs.com/img/logos/handsontable-dark-logo.svg#2" width="150" valign="middle" /></a></td>
 <td align="center" valign="middle"><a href="https://www.itflashcards.com/" target="_blank"><img src="https://nestjs.com/img/logos/it_flashcards-logo.png" width="170" valign="middle" /></a></td>
 <td align="center" valign="middle"><a href="https://arcjet.com/?ref=nestjs" target="_blank"><img src="https://nestjs.com/img/logos/arcjet-logo.svg" width="170" valign="middle" /></a></td>
+<td align="center" valign="middle"><a href="https://crawljobs.com" target="_blank"><img src="https://nestjs.com/img/logos/crawljobs-logo.svg" width="130" valign="middle" /></a></td>
 </tr>
 </table>
 
@@ -118,19 +115,16 @@ Nest is an MIT-licensed open source project. It can grow thanks to the sponsors
 <td align="center" valign="middle"><a href="https://triplecore.io" target="_blank"><img src="https://nestjs.com/img/logos/triplecore-logo.svg" width="50" valign="middle" /></a></td>
 </tr>
 <tr>
-<td align="center" valign="middle"><a href="https://thecasinowizard.com/bonuses/no-deposit-bonuses/" target="_blank"><img src="https://nestjs.com/img/logos/casinowizard-logo.png" width="120" valign="middle" /></a></td>
 <td align="center" valign="middle"><a href="https://polygon-software.ch/" target="_blank"><img src="https://nestjs.com/img/logos/polygon-logo.svg" width="120" valign="middle" /></a></td>
 <td align="center" valign="middle"><a href="https://boringowl.io/" target="_blank"><img src="https://nestjs.com/img/logos/boringowl-logo.svg" width="120" valign="middle" /></a></td>
 <td align="center" valign="middle"><a href="https://nordbot.app/" target="_blank"><img src="https://nestjs.com/img/logos/nordbot-logo.png" width="120" valign="middle" /></a></td>
 <td align="center" valign="middle"><a href="https://doppio.sh/" target="_blank"><img src="https://nestjs.com/img/logos/dopiosh-logo.png" width="50" valign="middle" /></a></td>
 <td align="center" valign="middle"><a href="https://www.hingehealth.com/" target="_blank"><img src="https://nestjs.com/img/logos/hinge-health-logo.svg" width="100" valign="middle" /></a></td>
+<td align="center" valign="middle"><a href="https://www.tripoffice.com/" target="_blank"><img src="https://nestjs.com/img/logos/tripoffice-logo.png" width="140" valign="middle" /></a></td>
 </tr>
 <tr>
-<td align="center" valign="middle"><a href="https://julienferand.dev/" target="_blank"><img src="https://nestjs.com/img/logos/julienferand-logo.jpeg" width="55" valign="middle" /></a></td>
-<td align="center" valign="middle"><a href="https://www.tripoffice.com/" target="_blank"><img src="https://nestjs.com/img/logos/tripoffice-logo.png" width="140" valign="middle" /></a></td>
 <td align="center" valign="middle"><a href="https://solcellsforetag.se/" target="_blank"><img src="https://nestjs.com/img/logos/solcellsforetag-logo.svg" width="140" valign="middle" /></a></td>
 <td align="center" valign="middle"><a href="https://www.route4me.com/" target="_blank"><img src="https://nestjs.com/img/logos/route4me-logo.svg" width="100" valign="middle" /></a></td>
-<td align="center" valign="middle"><a href="https://www.slotsup.com/" target="_blank"><img src="https://nestjs.com/img/logos/slotsup-logo.png" width="60" valign="middle" /></a></td>
 </tr>
 </table>
 

package/errors/messages.js

@@ -48,10 +48,31 @@ const UNKNOWN_DEPENDENCIES_MESSAGE = (type, unknownDependencyContext, moduleRef)
     const { index, name, dependencies, key } = unknownDependencyContext;
     const moduleName = getModuleName(moduleRef);
     const dependencyName = getDependencyName(name, 'dependency');
-    const potentialSolutions = 
-    // If module's name is well defined
-    moduleName !== 'current'
-        ? `\n
+    const isImportTypeIssue = !(0, shared_utils_1.isNil)(index) &&
+        dependencies &&
+        (dependencies[index] === undefined ||
+            dependencies[index] === Object ||
+            (typeof dependencies[index] === 'function' &&
+                dependencies[index].name === 'Object'));
+    let potentialSolutions;
+    if (isImportTypeIssue) {
+        potentialSolutions = `\n
+Potential solutions:
+- The dependency at index [${index}] appears to be undefined at runtime
+- This commonly occurs when using 'import type' instead of 'import' for injectable classes
+- Check your imports and change:
+  ❌ import type { SomeService } from './some.service';
+  ✅ import { SomeService } from './some.service';
+- Ensure the imported class is decorated with @Injectable() or is a valid provider
+- If using dynamic imports, ensure the class is available at runtime, not just for type checking
+
+For more common dependency resolution issues, see: https://docs.nestjs.com/faq/common-errors`;
+    }
+    else {
+        potentialSolutions =
+            // If module's name is well defined
+            moduleName !== 'current'
+                ? `\n
 Potential solutions:
 - Is ${moduleName} a valid NestJS module?
 - If ${dependencyName} is a provider, is it part of the current ${moduleName}?
@@ -59,15 +80,18 @@ Potential solutions:
   @Module({
     imports: [ /* the Module containing ${dependencyName} */ ]
   })
-`
-        : `\n
+
+For more common dependency resolution issues, see: https://docs.nestjs.com/faq/common-errors`
+                : `\n
 Potential solutions:
 - If ${dependencyName} is a provider, is it part of the current Module?
 - If ${dependencyName} is exported from a separate @Module, is that module imported within Module?
   @Module({
     imports: [ /* the Module containing ${dependencyName} */ ]
   })
-`;
+
+For more common dependency resolution issues, see: https://docs.nestjs.com/faq/common-errors`;
+    }
     let message = `Nest can't resolve dependencies of the ${type.toString()}`;
     if ((0, shared_utils_1.isNil)(index)) {
         message += `. Please make sure that the "${key.toString()}" property is available in the current context.${potentialSolutions}`;
@@ -77,7 +101,7 @@ Potential solutions:
     dependenciesName[index] = '?';
     message += ` (`;
     message += dependenciesName.join(', ');
-    message += `). Please make sure that the argument ${dependencyName} at index [${index}] is available in the ${moduleName} context.`;
+    message += `). Please make sure that the argument ${isImportTypeIssue ? 'dependency' : dependencyName} at index [${index}] is available in the ${isImportTypeIssue ? 'current' : moduleName} context.`;
     message += potentialSolutions;
     return message;
 };
@@ -125,6 +149,8 @@ const UNKNOWN_EXPORT_MESSAGE = (token = 'item', module) => {
 
 Possible Solutions:
 - Is ${token} part of the relevant providers/imports within ${module}?
+
+For more common dependency resolution issues, see: https://docs.nestjs.com/faq/common-errors
 `;
 };
 exports.UNKNOWN_EXPORT_MESSAGE = UNKNOWN_EXPORT_MESSAGE;

package/injector/injector.js

@@ -11,11 +11,11 @@ const exceptions_1 = require("../errors/exceptions");
 const runtime_exception_1 = require("../errors/exceptions/runtime.exception");
 const undefined_dependency_exception_1 = require("../errors/exceptions/undefined-dependency.exception");
 const unknown_dependencies_exception_1 = require("../errors/exceptions/unknown-dependencies.exception");
+const barrier_1 = require("../helpers/barrier");
 const constants_2 = require("./constants");
 const inquirer_1 = require("./inquirer");
 const instance_wrapper_1 = require("./instance-wrapper");
 const settlement_signal_1 = require("./settlement-signal");
-const barrier_1 = require("../helpers/barrier");
 class Injector {
     constructor(options) {
         this.options = options;
@@ -111,7 +111,7 @@ class Injector {
         return settlementSignal;
     }
     async resolveConstructorParams(wrapper, moduleRef, inject, callback, contextId = constants_2.STATIC_CONTEXT, inquirer, parentInquirer) {
-        let inquirerId = this.getInquirerId(inquirer);
+        const inquirerId = this.getInquirerId(inquirer);
         const metadata = wrapper.getCtorMetadata();
         if (metadata && contextId !== constants_2.STATIC_CONTEXT) {
             const deps = await this.loadCtorMetadata(metadata, contextId, inquirer, parentInquirer);
@@ -133,8 +133,10 @@ class Injector {
                     return parentInquirer && parentInquirer.instance;
                 }
                 if (inquirer?.isTransient && parentInquirer) {
-                    inquirer = parentInquirer;
-                    inquirerId = this.getInquirerId(parentInquirer);
+                    // When `inquirer` is transient too, inherit the parent inquirer
+                    // This is required to ensure that transient providers are only resolved
+                    // when requested
+                    inquirer.attachRootInquirer(parentInquirer);
                 }
                 const paramWrapper = await this.resolveSingleParam(wrapper, param, { index, dependencies }, moduleRef, contextId, inquirer, index);
                 /*
@@ -315,8 +317,11 @@ class Injector {
             const inquirerId = this.getInquirerId(inquirer);
             const instanceHost = instanceWrapperRef.getInstanceByContextId(this.getContextId(contextId, instanceWrapperRef), inquirerId);
             if (!instanceHost.isResolved && !instanceWrapperRef.forwardRef) {
-                wrapper.settlementSignal?.insertRef(instanceWrapperRef.id);
-                await this.loadProvider(instanceWrapperRef, relatedModule, contextId, wrapper);
+                /*
+                 * Provider will be loaded shortly in resolveComponentHost() once we pass the current
+                 * Barrier. We cannot load it here because doing so could incorrectly evaluate the
+                 * staticity of the dependency tree and lead to undefined / null injection.
+                 */
                 break;
             }
         }
@@ -412,11 +417,13 @@ class Injector {
                 ? Object.assign(instanceHost.instance, new metatype(...instances))
                 : new metatype(...instances);
             instanceHost.instance = this.instanceDecorator(instanceHost.instance);
+            instanceHost.isConstructorCalled = true;
         }
         else if (isInContext) {
             const factoryReturnValue = targetMetatype.metatype(...instances);
             instanceHost.instance = await factoryReturnValue;
             instanceHost.instance = this.instanceDecorator(instanceHost.instance);
+            instanceHost.isConstructorCalled = true;
         }
         instanceHost.isResolved = true;
         return instanceHost.instance;

package/injector/instance-wrapper.d.ts

@@ -25,6 +25,7 @@ export interface InstancePerContext<T> {
     isResolved?: boolean;
     isPending?: boolean;
     donePromise?: Promise<unknown>;
+    isConstructorCalled?: boolean;
 }
 export interface PropertyMetadata {
     key: symbol | string;
@@ -51,6 +52,11 @@ export declare class InstanceWrapper<T = any> {
     private transientMap?;
     private isTreeStatic;
     private isTreeDurable;
+    /**
+     * The root inquirer reference. Present only if child instance wrapper
+     * is transient and has a parent inquirer.
+     */
+    private rootInquirer;
     constructor(metadata?: Partial<InstanceWrapper<T>> & Partial<InstancePerContext<T>>);
     get id(): string;
     set instance(value: T);
@@ -80,6 +86,8 @@ export declare class InstanceWrapper<T = any> {
     isLazyTransient(contextId: ContextId, inquirer: InstanceWrapper | undefined): boolean;
     isExplicitlyRequested(contextId: ContextId, inquirer?: InstanceWrapper): boolean;
     isStatic(contextId: ContextId, inquirer: InstanceWrapper | undefined): boolean;
+    attachRootInquirer(inquirer: InstanceWrapper): void;
+    getRootInquirer(): InstanceWrapper | undefined;
     getStaticTransientInstances(): (InstancePerContext<T> | undefined)[];
     mergeWith(provider: Provider): void;
     private isNewable;

package/injector/instance-wrapper.js

@@ -242,10 +242,22 @@ class InstanceWrapper {
     isStatic(contextId, inquirer) {
         const isInquirerRequestScoped = inquirer && !inquirer.isDependencyTreeStatic();
         const isStaticTransient = this.isTransient && !isInquirerRequestScoped;
+        const rootInquirer = inquirer?.getRootInquirer();
         return (this.isDependencyTreeStatic() &&
             contextId === constants_1.STATIC_CONTEXT &&
             (!this.isTransient ||
-                (isStaticTransient && !!inquirer && !inquirer.isTransient)));
+                (isStaticTransient && !!inquirer && !inquirer.isTransient) ||
+                (isStaticTransient && !!rootInquirer && !rootInquirer.isTransient)));
+    }
+    attachRootInquirer(inquirer) {
+        if (!this.isTransient) {
+            // Only attach root inquirer if the instance wrapper is transient
+            return;
+        }
+        this.rootInquirer = inquirer.getRootInquirer() ?? inquirer;
+    }
+    getRootInquirer() {
+        return this.rootInquirer;
     }
     getStaticTransientInstances() {
         if (!this.transientMap) {
@@ -254,7 +266,11 @@ class InstanceWrapper {
         const instances = [...this.transientMap.values()];
         return (0, iterare_1.iterate)(instances)
             .map(item => item.get(constants_1.STATIC_CONTEXT))
-            .filter(item => !!item)
+            .filter(item => {
+            // Only return items where constructor has been actually called
+            // This prevents calling lifecycle hooks on non-instantiated transient services
+            return !!(item && item.isConstructorCalled);
+        })
             .toArray();
     }
     mergeWith(provider) {

package/nest-factory.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.NestFactory = exports.NestFactoryStatic = void 0;
+const console_logger_service_1 = require("@nestjs/common/services/console-logger.service");
 const logger_service_1 = require("@nestjs/common/services/logger.service");
 const load_package_util_1 = require("@nestjs/common/utils/load-package.util");
 const shared_utils_1 = require("@nestjs/common/utils/shared.utils");
@@ -154,10 +155,16 @@ class NestFactoryStatic {
         if (!options) {
             return;
         }
-        const { logger, bufferLogs, autoFlushLogs } = options;
+        const { logger, bufferLogs, autoFlushLogs, forceConsole } = options;
         if (logger !== true && !(0, shared_utils_1.isNil)(logger)) {
             logger_service_1.Logger.overrideLogger(logger);
         }
+        else if (forceConsole) {
+            // If no custom logger is provided but forceConsole is true,
+            // create a ConsoleLogger with forceConsole option
+            const consoleLogger = new console_logger_service_1.ConsoleLogger({ forceConsole: true });
+            logger_service_1.Logger.overrideLogger(consoleLogger);
+        }
         if (bufferLogs) {
             logger_service_1.Logger.attachBuffer();
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nestjs/core",
-  "version": "11.1.5",
+  "version": "11.1.7",
   "description": "Nest - modern, fast, powerful node.js web framework (@core)",
   "author": "Kamil Mysliwiec",
   "license": "MIT",
@@ -34,12 +34,12 @@
     "@nuxt/opencollective": "0.4.1",
     "fast-safe-stringify": "2.1.1",
     "iterare": "1.2.1",
-    "path-to-regexp": "8.2.0",
+    "path-to-regexp": "8.3.0",
     "tslib": "2.8.1",
     "uid": "2.0.2"
   },
   "devDependencies": {
-    "@nestjs/common": "11.1.5"
+    "@nestjs/common": "11.1.7"
   },
   "peerDependencies": {
     "@nestjs/common": "^11.0.0",