@nestjs/core 11.1.6 → 11.1.8

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.d.ts

@@ -82,6 +82,14 @@ export declare class Injector {
     loadCtorMetadata(metadata: InstanceWrapper<any>[], contextId: ContextId, inquirer?: InstanceWrapper, parentInquirer?: InstanceWrapper): Promise<any[]>;
     loadPropertiesMetadata(metadata: PropertyMetadata[], contextId: ContextId, inquirer?: InstanceWrapper): Promise<PropertyDependency[]>;
     private getInquirerId;
+    /**
+     * For nested TRANSIENT dependencies (TRANSIENT -> TRANSIENT) in non-static contexts,
+     * returns parentInquirer to ensure each parent TRANSIENT gets its own instance.
+     * This is necessary because in REQUEST/DURABLE scopes, the same TRANSIENT wrapper
+     * can be used by multiple parents, causing nested TRANSIENTs to be shared incorrectly.
+     * For non-TRANSIENT -> TRANSIENT, returns inquirer (current wrapper being created).
+     */
+    private getEffectiveInquirer;
     private resolveScopedComponentHost;
     private isInquirerRequest;
     private isInquirer;

package/injector/injector.js

@@ -111,7 +111,6 @@ class Injector {
         return settlementSignal;
     }
     async resolveConstructorParams(wrapper, moduleRef, inject, callback, contextId = constants_2.STATIC_CONTEXT, inquirer, parentInquirer) {
-        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);
@@ -145,8 +144,9 @@ class Injector {
                  * and result in undefined / null injection.
                  */
                 await paramBarrier.signalAndWait();
-                const paramWrapperWithInstance = await this.resolveComponentHost(moduleRef, paramWrapper, contextId, inquirer);
-                const instanceHost = paramWrapperWithInstance.getInstanceByContextId(this.getContextId(contextId, paramWrapperWithInstance), inquirerId);
+                const effectiveInquirer = this.getEffectiveInquirer(paramWrapper, inquirer, parentInquirer, contextId);
+                const paramWrapperWithInstance = await this.resolveComponentHost(moduleRef, paramWrapper, contextId, effectiveInquirer);
+                const instanceHost = paramWrapperWithInstance.getInstanceByContextId(this.getContextId(contextId, paramWrapperWithInstance), this.getInquirerId(effectiveInquirer));
                 if (!instanceHost.isResolved && !paramWrapperWithInstance.forwardRef) {
                     isResolved = false;
                 }
@@ -357,12 +357,12 @@ class Injector {
                  * and result in undefined / null injection.
                  */
                 await propertyBarrier.signalAndWait();
-                const paramWrapperWithInstance = await this.resolveComponentHost(moduleRef, paramWrapper, contextId, inquirer);
+                const effectivePropertyInquirer = this.getEffectiveInquirer(paramWrapper, inquirer, parentInquirer, contextId);
+                const paramWrapperWithInstance = await this.resolveComponentHost(moduleRef, paramWrapper, contextId, effectivePropertyInquirer);
                 if (!paramWrapperWithInstance) {
                     return undefined;
                 }
-                const inquirerId = this.getInquirerId(inquirer);
-                const instanceHost = paramWrapperWithInstance.getInstanceByContextId(this.getContextId(contextId, paramWrapperWithInstance), inquirerId);
+                const instanceHost = paramWrapperWithInstance.getInstanceByContextId(this.getContextId(contextId, paramWrapperWithInstance), this.getInquirerId(effectivePropertyInquirer));
                 return instanceHost.instance;
             }
             catch (err) {
@@ -417,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;
@@ -446,8 +448,11 @@ class Injector {
     }
     async loadCtorMetadata(metadata, contextId, inquirer, parentInquirer) {
         const hosts = await Promise.all(metadata.map(async (item) => this.resolveScopedComponentHost(item, contextId, inquirer, parentInquirer)));
-        const inquirerId = this.getInquirerId(inquirer);
-        return hosts.map(item => item?.getInstanceByContextId(this.getContextId(contextId, item), inquirerId).instance);
+        return hosts.map((item, index) => {
+            const dependency = metadata[index];
+            const effectiveInquirer = this.getEffectiveInquirer(dependency, inquirer, parentInquirer, contextId);
+            return item?.getInstanceByContextId(this.getContextId(contextId, item), this.getInquirerId(effectiveInquirer)).instance;
+        });
     }
     async loadPropertiesMetadata(metadata, contextId, inquirer) {
         const dependenciesHosts = await Promise.all(metadata.map(async ({ wrapper: item, key }) => ({
@@ -464,10 +469,25 @@ class Injector {
     getInquirerId(inquirer) {
         return inquirer ? inquirer.id : undefined;
     }
+    /**
+     * For nested TRANSIENT dependencies (TRANSIENT -> TRANSIENT) in non-static contexts,
+     * returns parentInquirer to ensure each parent TRANSIENT gets its own instance.
+     * This is necessary because in REQUEST/DURABLE scopes, the same TRANSIENT wrapper
+     * can be used by multiple parents, causing nested TRANSIENTs to be shared incorrectly.
+     * For non-TRANSIENT -> TRANSIENT, returns inquirer (current wrapper being created).
+     */
+    getEffectiveInquirer(dependency, inquirer, parentInquirer, contextId) {
+        return dependency?.isTransient &&
+            inquirer?.isTransient &&
+            parentInquirer &&
+            contextId !== constants_2.STATIC_CONTEXT
+            ? parentInquirer
+            : inquirer;
+    }
     resolveScopedComponentHost(item, contextId, inquirer, parentInquirer) {
         return this.isInquirerRequest(item, parentInquirer)
             ? parentInquirer
-            : this.resolveComponentHost(item.host, item, contextId, inquirer);
+            : this.resolveComponentHost(item.host, item, contextId, this.getEffectiveInquirer(item, inquirer, parentInquirer, contextId));
     }
     isInquirerRequest(item, parentInquirer) {
         return item.isTransient && item.name === inquirer_1.INQUIRER && parentInquirer;

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;

package/injector/instance-wrapper.js

@@ -266,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.6",
+  "version": "11.1.8",
   "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.6"
+    "@nestjs/common": "11.1.8"
   },
   "peerDependencies": {
     "@nestjs/common": "^11.0.0",