@nestjs/core 10.4.11 → 11.0.0-next.2

package/nest-application-context.js

@@ -9,7 +9,6 @@ const exceptions_1 = require("./errors/exceptions");
 const context_id_factory_1 = require("./helpers/context-id-factory");
 const hooks_1 = require("./hooks");
 const abstract_instance_resolver_1 = require("./injector/abstract-instance-resolver");
-const compiler_1 = require("./injector/compiler");
 const injector_1 = require("./injector/injector");
 const instance_links_host_1 = require("./injector/instance-links-host");
 /**
@@ -34,8 +33,8 @@ class NestApplicationContext extends abstract_instance_resolver_1.AbstractInstan
         });
         this.shouldFlushLogsOnOverride = false;
         this.activeShutdownSignals = new Array();
-        this.moduleCompiler = new compiler_1.ModuleCompiler();
         this.injector = new injector_1.Injector();
+        this.moduleCompiler = container.getModuleCompiler();
         if (this.appOptions.preview) {
             this.printInPreviewModeWarning();
         }
@@ -48,18 +47,26 @@ class NestApplicationContext extends abstract_instance_resolver_1.AbstractInstan
      * Allows navigating through the modules tree, for example, to pull out a specific instance from the selected module.
      * @returns {INestApplicationContext}
      */
-    select(moduleType) {
+    select(moduleType, selectOptions) {
         const modulesContainer = this.container.getModules();
         const contextModuleCtor = this.contextModule.metatype;
         const scope = this.scope.concat(contextModuleCtor);
         const moduleTokenFactory = this.container.getModuleTokenFactory();
         const { type, dynamicMetadata } = this.moduleCompiler.extractMetadata(moduleType);
-        const token = moduleTokenFactory.create(type, dynamicMetadata);
+        const token = dynamicMetadata
+            ? moduleTokenFactory.createForDynamic(type, dynamicMetadata, moduleType)
+            : moduleTokenFactory.createForStatic(type, moduleType);
         const selectedModule = modulesContainer.get(token);
         if (!selectedModule) {
             throw new exceptions_1.UnknownModuleException(type.name);
         }
-        return new NestApplicationContext(this.container, this.appOptions, selectedModule, scope);
+        const options = typeof selectOptions?.abortOnError !== 'undefined'
+            ? {
+                ...this.appOptions,
+                ...selectOptions,
+            }
+            : this.appOptions;
+        return new NestApplicationContext(this.container, options, selectedModule, scope);
     }
     /**
      * Retrieves an instance (or a list of instances) of either injectable or controller, otherwise, throws exception.
@@ -97,6 +104,7 @@ class NestApplicationContext extends abstract_instance_resolver_1.AbstractInstan
         if (this.isInitialized) {
             return this;
         }
+        /* eslint-disable-next-line no-async-promise-executor */
         this.initializationPromise = new Promise(async (resolve, reject) => {
             try {
                 await this.callInitHook();
@@ -239,7 +247,9 @@ class NestApplicationContext extends abstract_instance_resolver_1.AbstractInstan
      * modules and its children.
      */
     async callDestroyHook() {
-        const modulesSortedByDistance = this.getModulesToTriggerHooksOn();
+        const modulesSortedByDistance = [
+            ...this.getModulesToTriggerHooksOn(),
+        ].reverse();
         for (const module of modulesSortedByDistance) {
             await (0, hooks_1.callModuleDestroyHook)(module);
         }

package/nest-application.js

@@ -92,7 +92,7 @@ class NestApplication extends nest_application_context_1.NestApplicationContext
             return this;
         }
         this.applyOptions();
-        await this.httpAdapter?.init();
+        await this.httpAdapter?.init?.();
         const useBodyParser = this.appOptions && this.appOptions.bodyParser !== false;
         useBodyParser && this.registerParserMiddleware();
         await this.registerModules();
@@ -154,7 +154,7 @@ class NestApplication extends nest_application_context_1.NestApplicationContext
         }
         const [parserType, ...otherArgs] = args;
         const rawBody = !!this.appOptions.rawBody;
-        this.httpAdapter.useBodyParser(...[parserType, rawBody, ...otherArgs]);
+        this.httpAdapter.useBodyParser?.(...[parserType, rawBody, ...otherArgs]);
         return this;
     }
     enableCors(options) {

package/nest-factory.d.ts

@@ -1,8 +1,9 @@
-import { INestApplication, INestApplicationContext, INestMicroservice } from '@nestjs/common';
+import { DynamicModule, ForwardReference, INestApplication, INestApplicationContext, INestMicroservice, Type } from '@nestjs/common';
 import { NestMicroserviceOptions } from '@nestjs/common/interfaces/microservices/nest-microservice-options.interface';
 import { NestApplicationContextOptions } from '@nestjs/common/interfaces/nest-application-context-options.interface';
 import { NestApplicationOptions } from '@nestjs/common/interfaces/nest-application-options.interface';
 import { AbstractHttpAdapter } from './adapters/http-adapter';
+type IEntryNestModule = Type<any> | DynamicModule | ForwardReference | Promise<IEntryNestModule>;
 /**
  * @publicApi
  */
@@ -19,7 +20,7 @@ export declare class NestFactoryStatic {
      * @returns A promise that, when resolved,
      * contains a reference to the NestApplication instance.
      */
-    create<T extends INestApplication = INestApplication>(module: any, options?: NestApplicationOptions): Promise<T>;
+    create<T extends INestApplication = INestApplication>(module: IEntryNestModule, options?: NestApplicationOptions): Promise<T>;
     /**
      * Creates an instance of NestApplication with the specified `httpAdapter`.
      *
@@ -31,7 +32,7 @@ export declare class NestFactoryStatic {
      * @returns A promise that, when resolved,
      * contains a reference to the NestApplication instance.
      */
-    create<T extends INestApplication = INestApplication>(module: any, httpAdapter: AbstractHttpAdapter, options?: NestApplicationOptions): Promise<T>;
+    create<T extends INestApplication = INestApplication>(module: IEntryNestModule, httpAdapter: AbstractHttpAdapter, options?: NestApplicationOptions): Promise<T>;
     /**
      * Creates an instance of NestMicroservice.
      *
@@ -41,7 +42,7 @@ export declare class NestFactoryStatic {
      * @returns A promise that, when resolved,
      * contains a reference to the NestMicroservice instance.
      */
-    createMicroservice<T extends object>(moduleCls: any, options?: NestMicroserviceOptions & T): Promise<INestMicroservice>;
+    createMicroservice<T extends object>(moduleCls: IEntryNestModule, options?: NestMicroserviceOptions & T): Promise<INestMicroservice>;
     /**
      * Creates an instance of NestApplicationContext.
      *
@@ -51,7 +52,7 @@ export declare class NestFactoryStatic {
      * @returns A promise that, when resolved,
      * contains a reference to the NestApplicationContext instance.
      */
-    createApplicationContext(moduleCls: any, options?: NestApplicationContextOptions): Promise<INestApplicationContext>;
+    createApplicationContext(moduleCls: IEntryNestModule, options?: NestApplicationContextOptions): Promise<INestApplicationContext>;
     private createNestInstance;
     private initialize;
     private handleInitializationError;
@@ -79,3 +80,4 @@ export declare class NestFactoryStatic {
  * @publicApi
  */
 export declare const NestFactory: NestFactoryStatic;
+export {};

package/nest-factory.js

@@ -100,7 +100,7 @@ class NestFactoryStatic {
         const dependenciesScanner = new scanner_1.DependenciesScanner(container, metadataScanner, graphInspector, config);
         container.setHttpAdapter(httpServer);
         const teardown = this.abortOnError === false ? rethrow_1.rethrow : undefined;
-        await httpServer?.init();
+        await httpServer?.init?.();
         try {
             this.logger.log(constants_1.MESSAGES.APPLICATION_START);
             await exceptions_zone_1.ExceptionsZone.asyncRun(async () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nestjs/core",
-  "version": "10.4.11",
+  "version": "11.0.0-next.2",
   "description": "Nest - modern, fast, powerful node.js web framework (@core)",
   "author": "Kamil Mysliwiec",
   "license": "MIT",
@@ -31,12 +31,12 @@
     "@nuxtjs/opencollective": "0.3.2",
     "fast-safe-stringify": "2.1.1",
     "iterare": "1.2.1",
-    "path-to-regexp": "3.3.0",
+    "path-to-regexp": "8.2.0",
     "tslib": "2.7.0",
     "uid": "2.0.2"
   },
   "devDependencies": {
-    "@nestjs/common": "10.4.11"
+    "@nestjs/common": "^11.0.0-next.2"
   },
   "peerDependencies": {
     "@nestjs/common": "^10.0.0",

package/pipes/pipes-context-creator.d.ts

@@ -7,7 +7,7 @@ export declare class PipesContextCreator extends ContextCreator {
     private readonly container;
     private readonly config?;
     private moduleContext;
-    constructor(container: NestContainer, config?: ApplicationConfig);
+    constructor(container: NestContainer, config?: ApplicationConfig | undefined);
     create(instance: Controller, callback: (...args: unknown[]) => unknown, moduleKey: string, contextId?: import("../injector/instance-wrapper").ContextId, inquirerId?: string): PipeTransform[];
     createConcreteContext<T extends any[], R extends any[]>(metadata: T, contextId?: import("../injector/instance-wrapper").ContextId, inquirerId?: string): R;
     getPipeInstance(pipe: Function | PipeTransform, contextId?: import("../injector/instance-wrapper").ContextId, inquirerId?: string): PipeTransform | null;

package/pipes/pipes-context-creator.js

@@ -23,11 +23,11 @@ class PipesContextCreator extends context_creator_1.ContextCreator {
         return (0, iterare_1.iterate)(metadata)
             .filter((pipe) => pipe && (pipe.name || pipe.transform))
             .map(pipe => this.getPipeInstance(pipe, contextId, inquirerId))
-            .filter(pipe => pipe && pipe.transform && (0, shared_utils_1.isFunction)(pipe.transform))
+            .filter(pipe => !!pipe && pipe.transform && (0, shared_utils_1.isFunction)(pipe.transform))
             .toArray();
     }
     getPipeInstance(pipe, contextId = constants_2.STATIC_CONTEXT, inquirerId) {
-        const isObject = pipe.transform;
+        const isObject = !!pipe.transform;
         if (isObject) {
             return pipe;
         }

package/repl/assign-to-object.util.d.ts

@@ -2,4 +2,4 @@
  * Similar to `Object.assign` but copying properties descriptors from `source`
  * as well.
  */
-export declare function assignToObject<T, U>(target: T, source: U): T & U;
+export declare function assignToObject<T, U extends object>(target: T, source: U): T & U;

package/router/interfaces/route-params-factory.interface.d.ts

@@ -4,5 +4,5 @@ export interface IRouteParamsFactory {
         req: TRequest;
         res: TResponse;
         next: Function;
-    }): TResult;
+    }): TResult | null;
 }

package/router/legacy-route-converter.d.ts

@@ -0,0 +1,13 @@
+export declare class LegacyRouteConverter {
+    private static readonly logger;
+    /**
+     * Convert legacy routes to the new format (syntax).
+     * path-to-regexp used by Express>=v5 and @fastify/middie>=v9 no longer support unnamed wildcards.
+     * This method attempts to convert the old syntax to the new one, and logs an error if it fails.
+     * @param route The route to convert.
+     * @returns The converted route, or the original route if it cannot be converted.
+     */
+    static tryConvert(route: string): string;
+    static printError(route: string): void;
+    static printWarning(route: string): void;
+}

package/router/legacy-route-converter.js

@@ -0,0 +1,48 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.LegacyRouteConverter = void 0;
+const common_1 = require("@nestjs/common");
+const UNSUPPORTED_PATH_MESSAGE = (text, route) => `Unsupported route path: "${route}". In previous versions, the symbols ?, *, and + were used to denote optional or repeating path parameters. The latest version of "path-to-regexp" now requires the use of named parameters. For example, instead of using a route like /users/* to capture all routes starting with "/users", you should use /users/*path. For more details, refer to the migration guide.`;
+class LegacyRouteConverter {
+    /**
+     * Convert legacy routes to the new format (syntax).
+     * path-to-regexp used by Express>=v5 and @fastify/middie>=v9 no longer support unnamed wildcards.
+     * This method attempts to convert the old syntax to the new one, and logs an error if it fails.
+     * @param route The route to convert.
+     * @returns The converted route, or the original route if it cannot be converted.
+     */
+    static tryConvert(route) {
+        // Normalize path to eliminate additional conditions.
+        const routeWithLeadingSlash = route.startsWith('/') ? route : `/${route}`;
+        const normalizedRoute = route.endsWith('/')
+            ? routeWithLeadingSlash
+            : `${routeWithLeadingSlash}/`;
+        if (normalizedRoute.endsWith('/(.*)/')) {
+            // Skip printing warning for the "all" wildcard.
+            if (normalizedRoute !== '/(.*)/') {
+                this.printWarning(route);
+            }
+            return route.replace('(.*)', '*path');
+        }
+        if (normalizedRoute.endsWith('/*/')) {
+            // Skip printing warning for the "all" wildcard.
+            if (normalizedRoute !== '/*/') {
+                this.printWarning(route);
+            }
+            return route.replace('*', '*path');
+        }
+        if (normalizedRoute.endsWith('/+/')) {
+            this.printWarning(route);
+            return route.replace('/+', '/*path');
+        }
+        return route;
+    }
+    static printError(route) {
+        this.logger.error(UNSUPPORTED_PATH_MESSAGE `${route}`);
+    }
+    static printWarning(route) {
+        this.logger.warn(UNSUPPORTED_PATH_MESSAGE `${route}` + ' Attempting to convert...');
+    }
+}
+exports.LegacyRouteConverter = LegacyRouteConverter;
+LegacyRouteConverter.logger = new common_1.Logger(LegacyRouteConverter.name);

package/router/request/request-providers.js

@@ -3,7 +3,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.requestProvider = void 0;
 const common_1 = require("@nestjs/common");
 const request_constants_1 = require("./request-constants");
-// eslint-disable-next-line @typescript-eslint/no-empty-function
 const noop = () => { };
 exports.requestProvider = {
     provide: request_constants_1.REQUEST,

package/router/route-params-factory.d.ts

@@ -1,9 +1,9 @@
 import { RouteParamtypes } from '@nestjs/common/enums/route-paramtypes.enum';
 import { IRouteParamsFactory } from './interfaces/route-params-factory.interface';
 export declare class RouteParamsFactory implements IRouteParamsFactory {
-    exchangeKeyForValue<TRequest extends Record<string, any> = any, TResponse = any, TResult = any>(key: RouteParamtypes | string, data: string | object | any, { req, res, next }: {
+    exchangeKeyForValue<TRequest extends Record<string, any> = any, TResponse = any, TResult = any>(key: RouteParamtypes | string, data: string, { req, res, next }: {
         req: TRequest;
         res: TResponse;
         next: Function;
-    }): TResult;
+    }): TResult | null;
 }

package/router/route-params-factory.js

@@ -18,6 +18,7 @@ class RouteParamsFactory {
             case route_paramtypes_enum_1.RouteParamtypes.PARAM:
                 return data ? req.params[data] : req.params;
             case route_paramtypes_enum_1.RouteParamtypes.HOST:
+                /* eslint-disable-next-line no-case-declarations */
                 const hosts = req.hosts || {};
                 return data ? hosts[data] : hosts;
             case route_paramtypes_enum_1.RouteParamtypes.QUERY:

package/router/route-path-factory.d.ts

@@ -6,7 +6,7 @@ export declare class RoutePathFactory {
     private readonly applicationConfig;
     constructor(applicationConfig: ApplicationConfig);
     create(metadata: RoutePathMetadata, requestMethod?: RequestMethod): string[];
-    getVersion(metadata: RoutePathMetadata): VersionValue;
+    getVersion(metadata: RoutePathMetadata): VersionValue | undefined;
     getVersionPrefix(versioningOptions: VersioningOptions): string;
     appendToAllIfDefined(paths: string[], fragmentToAppend: string | string[] | undefined): string[];
     isExcludedFromGlobalPrefix(path: string, requestMethod?: RequestMethod, versionOrVersions?: VersionValue, versioningOptions?: VersioningOptions): boolean;

package/router/router-exception-filters.d.ts

@@ -9,6 +9,6 @@ export declare class RouterExceptionFilters extends BaseExceptionFilterContext {
     private readonly config;
     private readonly applicationRef;
     constructor(container: NestContainer, config: ApplicationConfig, applicationRef: HttpServer);
-    create(instance: Controller, callback: RouterProxyCallback, moduleKey: string, contextId?: import("../injector/instance-wrapper").ContextId, inquirerId?: string): ExceptionsHandler;
+    create(instance: Controller, callback: RouterProxyCallback, moduleKey: string | undefined, contextId?: import("../injector/instance-wrapper").ContextId, inquirerId?: string): ExceptionsHandler;
     getGlobalMetadata<T extends unknown[]>(contextId?: import("../injector/instance-wrapper").ContextId, inquirerId?: string): T;
 }

package/router/router-exception-filters.js

@@ -3,10 +3,10 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.RouterExceptionFilters = void 0;
 const constants_1 = require("@nestjs/common/constants");
 const shared_utils_1 = require("@nestjs/common/utils/shared.utils");
+const iterare_1 = require("iterare");
 const base_exception_filter_context_1 = require("../exceptions/base-exception-filter-context");
 const exceptions_handler_1 = require("../exceptions/exceptions-handler");
 const constants_2 = require("../injector/constants");
-const iterare_1 = require("iterare");
 class RouterExceptionFilters extends base_exception_filter_context_1.BaseExceptionFilterContext {
     constructor(container, config, applicationRef) {
         super(container);

package/router/router-execution-context.d.ts

@@ -45,10 +45,10 @@ export declare class RouterExecutionContext {
         data: unknown;
     }, pipes: PipeTransform[]): Promise<unknown>;
     isPipeable(type: number | string): boolean;
-    createGuardsFn<TContext extends string = ContextType>(guards: CanActivate[], instance: Controller, callback: (...args: any[]) => any, contextType?: TContext): (args: any[]) => Promise<void> | null;
+    createGuardsFn<TContext extends string = ContextType>(guards: CanActivate[], instance: Controller, callback: (...args: any[]) => any, contextType?: TContext): ((args: any[]) => Promise<void>) | null;
     createPipesFn(pipes: PipeTransform[], paramsOptions: (ParamProperties & {
         metatype?: any;
-    })[]): <TRequest, TResponse>(args: any[], req: TRequest, res: TResponse, next: Function) => Promise<void>;
+    })[]): (<TRequest, TResponse>(args: any[], req: TRequest, res: TResponse, next: Function) => Promise<void>) | null;
     createHandleResponseFn(callback: (...args: unknown[]) => unknown, isResponseHandled: boolean, redirectResponse?: RedirectResponse, httpStatusCode?: number): HandleResponseFn;
     private isResponseHandled;
 }

package/router/router-execution-context.js

@@ -108,7 +108,7 @@ class RouterExecutionContext {
             }
             const numericType = Number(type);
             const extractValue = (req, res, next) => this.paramsFactory.exchangeKeyForValue(numericType, data, {
-                req,
+                req: req,
                 res,
                 next,
             });

package/router/router-explorer.js

@@ -6,7 +6,7 @@ const enums_1 = require("@nestjs/common/enums");
 const exceptions_1 = require("@nestjs/common/exceptions");
 const logger_service_1 = require("@nestjs/common/services/logger.service");
 const shared_utils_1 = require("@nestjs/common/utils/shared.utils");
-const pathToRegexp = require("path-to-regexp");
+const path_to_regexp_1 = require("path-to-regexp");
 const unknown_request_mapping_exception_1 = require("../errors/exceptions/unknown-request-mapping.exception");
 const guards_1 = require("../guards");
 const context_id_factory_1 = require("../helpers/context-id-factory");
@@ -102,7 +102,10 @@ class RouterExplorer {
                     },
                 };
                 this.copyMetadataToCallback(targetCallback, routeHandler);
-                routerMethodRef(path, routeHandler);
+                const normalizedPath = router.normalizePath
+                    ? router.normalizePath(path)
+                    : path;
+                routerMethodRef(normalizedPath, routeHandler);
                 this.graphInspector.insertEntrypointDefinition(entrypointDefinition, instanceWrapper.id);
             });
             const pathsToLog = this.routePathFactory.create({
@@ -127,9 +130,18 @@ class RouterExplorer {
         const httpAdapterRef = this.container.getHttpAdapterRef();
         const hosts = Array.isArray(host) ? host : [host];
         const hostRegExps = hosts.map((host) => {
-            const keys = [];
-            const regexp = pathToRegexp(host, keys);
-            return { regexp, keys };
+            if (typeof host === 'string') {
+                try {
+                    return (0, path_to_regexp_1.pathToRegexp)(host);
+                }
+                catch (e) {
+                    if (e instanceof TypeError) {
+                        this.logger.error(`Unsupported host "${host}" syntax. In past releases, ?, *, and + were used to denote optional or repeating path parameters. The latest version of "path-to-regexp" now requires the use of named parameters. For example, instead of using a route like /users/* to capture all routes starting with "/users", you should use /users/*path. Please see the migration guide for more information.`);
+                    }
+                    throw e;
+                }
+            }
+            return { regexp: host, keys: [] };
         });
         const unsupportedFilteringErrorMessage = Array.isArray(host)
             ? `HTTP adapter does not support filtering on hosts: ["${host.join('", "')}"]`

package/router/router-proxy.d.ts

@@ -1,6 +1,6 @@
 import { ExceptionsHandler } from '../exceptions/exceptions-handler';
-export type RouterProxyCallback = <TRequest, TResponse>(req?: TRequest, res?: TResponse, next?: () => void) => void;
+export type RouterProxyCallback = <TRequest, TResponse>(req: TRequest, res: TResponse, next: () => void) => void | Promise<void>;
 export declare class RouterProxy {
-    createProxy(targetCallback: RouterProxyCallback, exceptionsHandler: ExceptionsHandler): <TRequest, TResponse>(req: TRequest, res: TResponse, next: () => void) => Promise<TResponse>;
-    createExceptionLayerProxy(targetCallback: <TError, TRequest, TResponse>(err: TError, req: TRequest, res: TResponse, next: () => void) => void, exceptionsHandler: ExceptionsHandler): <TError, TRequest, TResponse>(err: TError, req: TRequest, res: TResponse, next: () => void) => Promise<TResponse>;
+    createProxy(targetCallback: RouterProxyCallback, exceptionsHandler: ExceptionsHandler): <TRequest, TResponse>(req: TRequest, res: TResponse, next: () => void) => Promise<TResponse | undefined>;
+    createExceptionLayerProxy(targetCallback: <TError, TRequest, TResponse>(err: TError, req: TRequest, res: TResponse, next: () => void) => void | Promise<void>, exceptionsHandler: ExceptionsHandler): <TError, TRequest, TResponse>(err: TError, req: TRequest, res: TResponse, next: () => void) => Promise<TResponse | undefined>;
 }

package/router/sse-stream.js

@@ -49,7 +49,7 @@ class SseStream extends stream_1.Transform {
                 // NGINX support https://www.nginx.com/resources/wiki/start/topics/examples/x-accel/#x-accel-buffering
                 'X-Accel-Buffering': 'no',
             });
-            destination.flushHeaders();
+            destination.flushHeaders?.();
         }
         destination.write('\n');
         return super.pipe(destination, options);

package/router/utils/flatten-route-paths.util.d.ts

@@ -1,2 +1,6 @@
+import { Type } from '@nestjs/common';
 import { Routes } from '../interfaces/routes.interface';
-export declare function flattenRoutePaths(routes: Routes): any[];
+export declare function flattenRoutePaths(routes: Routes): {
+    module: Type;
+    path: string;
+}[];

package/scanner.d.ts

@@ -46,8 +46,8 @@ export declare class DependenciesScanner {
     calculateModulesDistance(): void;
     insertImport(related: any, token: string, context: string): Promise<void>;
     isCustomProvider(provider: Provider): provider is ClassProvider | ValueProvider | FactoryProvider | ExistingProvider;
-    insertProvider(provider: Provider, token: string): string | symbol | Function | InstanceWrapper<unknown>;
-    insertInjectable(injectable: Type<Injectable> | object, token: string, host: Type<Injectable>, subtype: EnhancerSubtype, methodKey?: string): InstanceWrapper<any>;
+    insertProvider(provider: Provider, token: string): string | symbol | Function | InstanceWrapper<unknown> | undefined;
+    insertInjectable(injectable: Type<Injectable> | object, token: string, host: Type<Injectable>, subtype: EnhancerSubtype, methodKey?: string): InstanceWrapper<any> | undefined;
     insertExportedProvider(exportedProvider: Type<Injectable> | ForwardReference, token: string): void;
     insertController(controller: Type<Controller>, token: string): void;
     private insertOrOverrideModule;

package/scanner.js

@@ -29,9 +29,9 @@ class DependenciesScanner {
             overrides: options?.overrides,
         });
         await this.scanModulesForDependencies();
-        this.calculateModulesDistance();
         this.addScopedEnhancersMetadata();
         this.container.bindGlobalScope();
+        this.calculateModulesDistance();
     }
     async scanForModules({ moduleDefinition, lazy, scope = [], ctxRegistry = [], overrides = [], }) {
         const { moduleRef: moduleInstance, inserted: moduleInserted } = (await this.insertOrOverrideModule(moduleDefinition, overrides, scope)) ??
@@ -198,19 +198,19 @@ class DependenciesScanner {
         const modulesGenerator = this.container.getModules().values();
         // Skip "InternalCoreModule" from calculating distance
         modulesGenerator.next();
-        const modulesStack = [];
-        const calculateDistance = (moduleRef, distance = 1) => {
-            if (!moduleRef || modulesStack.includes(moduleRef)) {
+        const calculateDistance = (moduleRef, distance = 1, modulesStack = []) => {
+            const localModulesStack = [...modulesStack];
+            if (!moduleRef || localModulesStack.includes(moduleRef)) {
                 return;
             }
-            modulesStack.push(moduleRef);
+            localModulesStack.push(moduleRef);
             const moduleImports = moduleRef.imports;
             moduleImports.forEach(importedModuleRef => {
                 if (importedModuleRef) {
                     if (distance > importedModuleRef.distance) {
                         importedModuleRef.distance = distance;
                     }
-                    calculateDistance(importedModuleRef, distance + 1);
+                    calculateDistance(importedModuleRef, distance + 1, localModulesStack);
                 }
             });
         };

package/services/reflector.service.d.ts

@@ -66,7 +66,7 @@ export declare class Reflector {
      * @param targets context (decorated objects) to retrieve metadata from
      *
      */
-    getAll<T extends ReflectableDecorator<any>>(decorator: T, targets: (Type<any> | Function)[]): T extends ReflectableDecorator<infer R> ? R extends Array<any> ? R : R[] : unknown;
+    getAll<TParam = any, TTransformed = TParam>(decorator: ReflectableDecorator<TParam, TTransformed>, targets: (Type<any> | Function)[]): TTransformed extends Array<any> ? TTransformed : TTransformed[];
     /**
      * Retrieve metadata for a specified key for a specified set of targets.
      *
@@ -82,7 +82,7 @@ export declare class Reflector {
      * @param targets context (decorated objects) to retrieve metadata from
      *
      */
-    getAllAndMerge<T extends ReflectableDecorator<any>>(decorator: T, targets: (Type<any> | Function)[]): T extends ReflectableDecorator<infer R> ? R : unknown;
+    getAllAndMerge<TParam = any, TTransformed = TParam>(decorator: ReflectableDecorator<TParam, TTransformed>, targets: (Type<any> | Function)[]): TTransformed extends Array<any> ? TTransformed : TTransformed extends object ? TTransformed : TTransformed[];
     /**
      * Retrieve metadata for a specified key for a specified set of targets and merge results.
      *
@@ -98,7 +98,7 @@ export declare class Reflector {
      * @param targets context (decorated objects) to retrieve metadata from
      *
      */
-    getAllAndOverride<T extends ReflectableDecorator<any>>(decorator: T, targets: (Type<any> | Function)[]): T extends ReflectableDecorator<infer R> ? R : unknown;
+    getAllAndOverride<TParam = any, TTransformed = TParam>(decorator: ReflectableDecorator<TParam, TTransformed>, targets: (Type<any> | Function)[]): TTransformed;
     /**
      * Retrieve metadata for a specified key for a specified set of targets and return a first not undefined value.
      *

package/services/reflector.service.js

@@ -60,6 +60,13 @@ class Reflector {
         if ((0, shared_utils_1.isEmpty)(metadataCollection)) {
             return metadataCollection;
         }
+        if (metadataCollection.length === 1) {
+            const value = metadataCollection[0];
+            if ((0, shared_utils_1.isObject)(value)) {
+                return value;
+            }
+            return metadataCollection;
+        }
         return metadataCollection.reduce((a, b) => {
             if (Array.isArray(a)) {
                 return a.concat(b);