@nestjs/core 11.1.7 → 11.1.9

package/Readme.md

@@ -1,5 +1,5 @@
 <p align="center">
-  <a href="https://nestjs.com/" target="blank"><img src="https://nestjs.com/img/logo-small.svg" width="120" alt="Nest Logo" /></a>
+  <a href="https://nestjs.com/" target="_blank"><img src="https://nestjs.com/img/logo-small.svg" width="120" alt="Nest Logo" /></a>
 </p>
 
 [circleci-image]: https://img.shields.io/circleci/build/github/nestjs/nest/master?token=abc123def456
@@ -49,7 +49,7 @@ Please make sure to read the [Issue Reporting Checklist](https://github.com/nest
 
 ## Consulting
 
-With official support, you can get expert help straight from Nest core team. We provide dedicated technical support, migration strategies, advice on best practices (and design decisions), PR reviews, and team augmentation. Read more about [support here](https://enterprise.nestjs.com).
+With official support, you can get expert help straight from the Nest core team. We provide dedicated technical support, migration strategies, advice on best practices (and design decisions), PR reviews, and team augmentation. Read more about [support here](https://enterprise.nestjs.com).
 
 ## Support
 
@@ -92,6 +92,8 @@ Nest is an MIT-licensed open source project. It can grow thanks to the sponsors
 <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><tr>
+<td align="center" valign="middle"><a href="https://pandektes.com" target="_blank"><img src="https://nestjs.com/img/logos/pandektes-logo.png" width="65" valign="middle" /></a></td>
 </tr>
 </table>
 

package/injector/abstract-instance-resolver.js

@@ -8,7 +8,8 @@ class AbstractInstanceResolver {
         const instanceLinkOrArray = this.instanceLinksHost.get(typeOrToken, options);
         const pluckInstance = ({ wrapperRef }) => {
             if (wrapperRef.scope === common_1.Scope.REQUEST ||
-                wrapperRef.scope === common_1.Scope.TRANSIENT) {
+                wrapperRef.scope === common_1.Scope.TRANSIENT ||
+                !wrapperRef.isDependencyTreeStatic()) {
                 throw new exceptions_1.InvalidClassScopeException(typeOrToken);
             }
             return wrapperRef.instance;

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) {
@@ -448,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 }) => ({
@@ -466,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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nestjs/core",
-  "version": "11.1.7",
+  "version": "11.1.9",
   "description": "Nest - modern, fast, powerful node.js web framework (@core)",
   "author": "Kamil Mysliwiec",
   "license": "MIT",
@@ -39,7 +39,7 @@
     "uid": "2.0.2"
   },
   "devDependencies": {
-    "@nestjs/common": "11.1.7"
+    "@nestjs/common": "11.1.9"
   },
   "peerDependencies": {
     "@nestjs/common": "^11.0.0",