@fluojs/di 1.0.0-beta.3 → 1.0.0-beta.5

package/README.ko.md

@@ -145,6 +145,7 @@ const service = await container.resolve(DataService);
 | `resolve<T>(token)` | 토큰을 인스턴스로 비동기 해석합니다. |
 | `createRequestScope()` | 요청 스코프 의존성을 위한 자식 컨테이너를 생성합니다. |
 | `has(token)` | 컨테이너나 부모에 토큰이 등록되어 있는지 확인합니다. |
+| `hasRequestScopedDependency(token)` | 토큰 해석 시 provider 그래프에 request-scoped 의존성이나 순환이 있어 request-scope 컨테이너가 필요할 수 있는지 확인합니다. |
 
 ## 관련 패키지
 

package/README.md

@@ -145,6 +145,7 @@ Ensure all required providers are registered in the container. If you use `creat
 | `resolve<T>(token)` | Asynchronously resolves a token to an instance. |
 | `createRequestScope()` | Creates a child container for request-scoped dependencies. |
 | `has(token)` | Checks if a token is registered in the container or its parents. |
+| `hasRequestScopedDependency(token)` | Checks whether resolving a token may require a request-scope container because its provider graph contains request-scoped dependencies or is cyclic. |
 
 ## Related Packages
 

package/dist/container.d.ts

@@ -9,16 +9,17 @@ export declare class Container {
     private readonly registrations;
     private readonly multiRegistrations;
     private readonly multiOverriddenTokens;
-    private readonly requestCache;
-    private readonly multiRequestCache;
+    private requestCache;
+    private multiRequestCache;
     private readonly multiSingletonCache;
     private readonly staleDisposalTasks;
     private readonly staleDisposalErrors;
     private readonly singletonCache;
     private readonly forwardRefTokenCache;
-    private readonly childScopes;
+    private childScopes;
     private disposePromise;
     private disposed;
+    private trackedByRoot;
     constructor(parent?: Container | undefined, requestScopeEnabled?: boolean, singletonCache?: Map<Token, Promise<unknown>>);
     /**
      * Registers providers in the current container scope.
@@ -53,6 +54,13 @@ export declare class Container {
      * @returns `true` when a single or multi provider exists for the token.
      */
     has(token: Token): boolean;
+    /**
+     * Returns whether resolving a token may require a request-scope container.
+     *
+     * @param token Provider token to inspect through aliases, multi providers, and dependencies.
+     * @returns `true` when the provider graph contains request-scoped dependencies or is cyclic.
+     */
+    hasRequestScopedDependency(token: Token): boolean;
     /**
      * Creates a child request-scope container that shares root singleton cache.
      *
@@ -79,6 +87,7 @@ export declare class Container {
      */
     dispose(): Promise<void>;
     private disposeAll;
+    private isDisposedInHierarchy;
     private hasMulti;
     private assertNoRegistrationConflict;
     private hasAncestorSingleRegistration;
@@ -86,6 +95,10 @@ export declare class Container {
     private hasAncestorMultiRegistration;
     private hasMultiRegistration;
     private collectMultiProviders;
+    private providerGraphRequiresRequestScope;
+    private unregisteredClassRequiresRequestScope;
+    private normalizedProviderRequiresRequestScope;
+    private dependencyEntryRequiresRequestScope;
     private resolveWithChain;
     private resolveFromRegisteredProviders;
     private requireProvider;
@@ -101,6 +114,9 @@ export declare class Container {
     private resolveDepToken;
     private withTokenInChain;
     private root;
+    private ensureTrackedRequestScope;
+    private requestCacheForWrite;
+    private multiRequestCacheForWrite;
     private lookupProvider;
     /**
      * Resolve the cache map that should hold the instance for `provider`.

package/dist/container.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"container.d.ts","sourceRoot":"","sources":["../src/container.ts"],"names":[],"mappings":"AAAA,OAAO,EAAmC,KAAK,KAAK,EAAE,MAAM,cAAc,CAAC;AAW3E,OAAO,KAAK,EASV,QAAQ,EAET,MAAM,YAAY,CAAC;AAuIpB;;GAEG;AACH,qBAAa,SAAS;IAgBlB,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAC;IACxB,OAAO,CAAC,QAAQ,CAAC,mBAAmB;IAhBtC,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAwC;IACtE,OAAO,CAAC,QAAQ,CAAC,kBAAkB,CAA0C;IAC7E,OAAO,CAAC,QAAQ,CAAC,qBAAqB,CAAoB;IAC1D,OAAO,CAAC,QAAQ,CAAC,YAAY,CAAsC;IACnE,OAAO,CAAC,QAAQ,CAAC,iBAAiB,CAAmD;IACrF,OAAO,CAAC,QAAQ,CAAC,mBAAmB,CAAmD;IACvF,OAAO,CAAC,QAAQ,CAAC,kBAAkB,CAA4B;IAC/D,OAAO,CAAC,QAAQ,CAAC,mBAAmB,CAAiB;IACrD,OAAO,CAAC,QAAQ,CAAC,cAAc,CAA+B;IAC9D,OAAO,CAAC,QAAQ,CAAC,oBAAoB,CAAsC;IAC3E,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAwB;IACpD,OAAO,CAAC,cAAc,CAA4B;IAClD,OAAO,CAAC,QAAQ,CAAS;gBAGN,MAAM,CAAC,EAAE,SAAS,YAAA,EAClB,mBAAmB,UAAQ,EAC5C,cAAc,CAAC,EAAE,GAAG,CAAC,KAAK,EAAE,OAAO,CAAC,OAAO,CAAC,CAAC;IAK/C;;;;;;;;;OASG;IACH,QAAQ,CAAC,GAAG,SAAS,EAAE,QAAQ,EAAE,GAAG,IAAI;IAyCxC;;;;;;;;;;;;;OAaG;IACH,QAAQ,CAAC,GAAG,SAAS,EAAE,QAAQ,EAAE,GAAG,IAAI;IA6BxC;;;;;OAKG;IACH,GAAG,CAAC,KAAK,EAAE,KAAK,GAAG,OAAO;IAI1B;;;;;OAKG;IACH,kBAAkB,IAAI,SAAS;IAa/B;;;;;;;;;OASG;IACG,OAAO,CAAC,CAAC,EAAE,KAAK,EAAE,KAAK,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,CAAC,CAAC;IAW7C;;;;;OAKG;IACG,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC;YAiBhB,UAAU;IAgBxB,OAAO,CAAC,QAAQ;IAMhB,OAAO,CAAC,4BAA4B;IAsBpC,OAAO,CAAC,6BAA6B;IAIrC,OAAO,CAAC,qBAAqB;IAM7B,OAAO,CAAC,4BAA4B;IAIpC,OAAO,CAAC,oBAAoB;IAM5B,OAAO,CAAC,qBAAqB;YAgBf,gBAAgB;YAehB,8BAA8B;IAqC5C,OAAO,CAAC,eAAe;YAgBT,kBAAkB;IAMhC,OAAO,CAAC,mCAAmC;YAoB7B,6BAA6B;YAc7B,4BAA4B;IA4B1C,OAAO,CAAC,6BAA6B;YAQvB,gCAAgC;IAuB9C,OAAO,CAAC,kCAAkC;IAY1C,OAAO,CAAC,qBAAqB;IAI7B,OAAO,CAAC,kCAAkC;YAI5B,eAAe;YAwBf,gBAAgB;IAiB9B,OAAO,CAAC,IAAI;IAIZ,OAAO,CAAC,cAAc;IAUtB;;;;;;;;;;OAUG;IACH,OAAO,CAAC,QAAQ;IAuBhB,OAAO,CAAC,aAAa;IAuBrB,OAAO,CAAC,qBAAqB;IAI7B,OAAO,CAAC,oBAAoB;YAkBd,YAAY;YAaZ,0BAA0B;YA0B1B,8BAA8B;IAc5C,OAAO,CAAC,mBAAmB;YAWb,yBAAyB;IAMvC,OAAO,CAAC,qBAAqB;IAoB7B,OAAO,CAAC,mBAAmB;IAU3B,OAAO,CAAC,YAAY;YAIN,WAAW;IA+BzB,OAAO,CAAC,+BAA+B;IAsBvC,OAAO,CAAC,wBAAwB;IA6BhC,OAAO,CAAC,8BAA8B;IAYtC,OAAO,CAAC,sBAAsB;YAUhB,mBAAmB;IAUjC,OAAO,CAAC,qBAAqB;CA6C9B"}
+{"version":3,"file":"container.d.ts","sourceRoot":"","sources":["../src/container.ts"],"names":[],"mappings":"AAAA,OAAO,EAAmC,KAAK,KAAK,EAAE,MAAM,cAAc,CAAC;AAW3E,OAAO,KAAK,EASV,QAAQ,EAET,MAAM,YAAY,CAAC;AAuIpB;;GAEG;AACH,qBAAa,SAAS;IAiBlB,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAC;IACxB,OAAO,CAAC,QAAQ,CAAC,mBAAmB;IAjBtC,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAwC;IACtE,OAAO,CAAC,QAAQ,CAAC,kBAAkB,CAA0C;IAC7E,OAAO,CAAC,QAAQ,CAAC,qBAAqB,CAAoB;IAC1D,OAAO,CAAC,YAAY,CAA2C;IAC/D,OAAO,CAAC,iBAAiB,CAAwD;IACjF,OAAO,CAAC,QAAQ,CAAC,mBAAmB,CAAmD;IACvF,OAAO,CAAC,QAAQ,CAAC,kBAAkB,CAA4B;IAC/D,OAAO,CAAC,QAAQ,CAAC,mBAAmB,CAAiB;IACrD,OAAO,CAAC,QAAQ,CAAC,cAAc,CAA+B;IAC9D,OAAO,CAAC,QAAQ,CAAC,oBAAoB,CAAsC;IAC3E,OAAO,CAAC,WAAW,CAA6B;IAChD,OAAO,CAAC,cAAc,CAA4B;IAClD,OAAO,CAAC,QAAQ,CAAS;IACzB,OAAO,CAAC,aAAa,CAAS;gBAGX,MAAM,CAAC,EAAE,SAAS,YAAA,EAClB,mBAAmB,UAAQ,EAC5C,cAAc,CAAC,EAAE,GAAG,CAAC,KAAK,EAAE,OAAO,CAAC,OAAO,CAAC,CAAC;IAK/C;;;;;;;;;OASG;IACH,QAAQ,CAAC,GAAG,SAAS,EAAE,QAAQ,EAAE,GAAG,IAAI;IAyCxC;;;;;;;;;;;;;OAaG;IACH,QAAQ,CAAC,GAAG,SAAS,EAAE,QAAQ,EAAE,GAAG,IAAI;IA6BxC;;;;;OAKG;IACH,GAAG,CAAC,KAAK,EAAE,KAAK,GAAG,OAAO;IAI1B;;;;;OAKG;IACH,0BAA0B,CAAC,KAAK,EAAE,KAAK,GAAG,OAAO;IAIjD;;;;;OAKG;IACH,kBAAkB,IAAI,SAAS;IAW/B;;;;;;;;;OASG;IACG,OAAO,CAAC,CAAC,EAAE,KAAK,EAAE,KAAK,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,CAAC,CAAC;IAW7C;;;;;OAKG;IACG,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC;YAiBhB,UAAU;IAiBxB,OAAO,CAAC,qBAAqB;IAQ7B,OAAO,CAAC,QAAQ;IAMhB,OAAO,CAAC,4BAA4B;IAsBpC,OAAO,CAAC,6BAA6B;IAIrC,OAAO,CAAC,qBAAqB;IAM7B,OAAO,CAAC,4BAA4B;IAIpC,OAAO,CAAC,oBAAoB;IAM5B,OAAO,CAAC,qBAAqB;IAgB7B,OAAO,CAAC,iCAAiC;IAyBzC,OAAO,CAAC,qCAAqC;IAc7C,OAAO,CAAC,sCAAsC;IAY9C,OAAO,CAAC,mCAAmC;YAa7B,gBAAgB;YAehB,8BAA8B;IAqC5C,OAAO,CAAC,eAAe;YAgBT,kBAAkB;IAMhC,OAAO,CAAC,mCAAmC;YAoB7B,6BAA6B;YAc7B,4BAA4B;IA4B1C,OAAO,CAAC,6BAA6B;YAQvB,gCAAgC;IAuB9C,OAAO,CAAC,kCAAkC;IAY1C,OAAO,CAAC,qBAAqB;IAI7B,OAAO,CAAC,kCAAkC;YAI5B,eAAe;YAwBf,gBAAgB;IAiB9B,OAAO,CAAC,IAAI;IAIZ,OAAO,CAAC,yBAAyB;IAWjC,OAAO,CAAC,oBAAoB;IAM5B,OAAO,CAAC,yBAAyB;IAMjC,OAAO,CAAC,cAAc;IAUtB;;;;;;;;;;OAUG;IACH,OAAO,CAAC,QAAQ;IAuBhB,OAAO,CAAC,aAAa;IAuBrB,OAAO,CAAC,qBAAqB;IAI7B,OAAO,CAAC,oBAAoB;YAkBd,YAAY;YAaZ,0BAA0B;YA0B1B,8BAA8B;IAc5C,OAAO,CAAC,mBAAmB;YAWb,yBAAyB;IAMvC,OAAO,CAAC,qBAAqB;IAoB7B,OAAO,CAAC,mBAAmB;IAU3B,OAAO,CAAC,YAAY;YAIN,WAAW;IA+BzB,OAAO,CAAC,+BAA+B;IAsBvC,OAAO,CAAC,wBAAwB;IA6BhC,OAAO,CAAC,8BAA8B;IAYtC,OAAO,CAAC,sBAAsB;YAUhB,mBAAmB;IAUjC,OAAO,CAAC,qBAAqB;CAmD9B"}

package/dist/container.js

@@ -119,16 +119,17 @@ export class Container {
   registrations = new Map();
   multiRegistrations = new Map();
   multiOverriddenTokens = new Set();
-  requestCache = new Map();
-  multiRequestCache = new Map();
+  requestCache;
+  multiRequestCache;
   multiSingletonCache = new Map();
   staleDisposalTasks = new Set();
   staleDisposalErrors = [];
   singletonCache;
   forwardRefTokenCache = new WeakMap();
-  childScopes = new Set();
+  childScopes;
   disposePromise;
   disposed = false;
+  trackedByRoot = false;
   constructor(parent, requestScopeEnabled = false, singletonCache) {
     this.parent = parent;
     this.requestScopeEnabled = requestScopeEnabled;
@@ -146,7 +147,7 @@ export class Container {
    * @throws {InvalidProviderError} When a provider definition is structurally invalid.
    */
   register(...providers) {
-    if (this.disposed) {
+    if (this.isDisposedInHierarchy()) {
       throw new ContainerResolutionError('Container has been disposed and can no longer register providers.', {
         hint: 'Ensure providers are registered before calling container.dispose().'
       });
@@ -190,7 +191,7 @@ export class Container {
    * @throws {InvalidProviderError} When a provider definition is structurally invalid.
    */
   override(...providers) {
-    if (this.disposed) {
+    if (this.isDisposedInHierarchy()) {
       throw new ContainerResolutionError('Container has been disposed and can no longer override providers.', {
         hint: 'Ensure overrides are applied before calling container.dispose().'
       });
@@ -222,6 +223,16 @@ export class Container {
     return this.lookupProvider(token) !== undefined || this.hasMulti(token);
   }
 
+  /**
+   * Returns whether resolving a token may require a request-scope container.
+   *
+   * @param token Provider token to inspect through aliases, multi providers, and dependencies.
+   * @returns `true` when the provider graph contains request-scoped dependencies or is cyclic.
+   */
+  hasRequestScopedDependency(token) {
+    return this.providerGraphRequiresRequestScope(token, new Set());
+  }
+
   /**
    * Creates a child request-scope container that shares root singleton cache.
    *
@@ -229,14 +240,12 @@ export class Container {
    * @throws {ContainerResolutionError} When called after the container was disposed.
    */
   createRequestScope() {
-    if (this.disposed) {
+    if (this.isDisposedInHierarchy()) {
       throw new ContainerResolutionError('Container has been disposed and can no longer create request scopes.', {
         hint: 'Create request scopes before calling container.dispose().'
       });
     }
-    const child = new Container(this, true, this.root().singletonCache);
-    this.root().childScopes.add(child);
-    return child;
+    return new Container(this, true, this.root().singletonCache);
   }
 
   /**
@@ -250,7 +259,7 @@ export class Container {
    * @throws {CircularDependencyError} When provider dependency resolution detects a cycle.
    */
   async resolve(token) {
-    if (this.disposed) {
+    if (this.isDisposedInHierarchy()) {
       throw new ContainerResolutionError('Container has been disposed and can no longer resolve providers.', {
         token,
         hint: 'Ensure all resolves complete before calling container.dispose().'
@@ -282,17 +291,24 @@ export class Container {
   async disposeAll() {
     try {
       // Dispose all live request-scope children first (root only)
-      if (!this.parent && this.childScopes.size > 0) {
+      if (!this.parent && this.childScopes && this.childScopes.size > 0) {
         await Promise.all(Array.from(this.childScopes).map(child => child.dispose()));
         this.childScopes.clear();
       }
       await this.disposeCache(this.disposalCacheEntries());
     } finally {
-      if (this.parent) {
-        this.root().childScopes.delete(this);
+      if (this.parent && this.trackedByRoot) {
+        this.root().childScopes?.delete(this);
+        this.trackedByRoot = false;
       }
     }
   }
+  isDisposedInHierarchy() {
+    if (this.disposed) {
+      return true;
+    }
+    return this.parent?.isDisposedInHierarchy() ?? false;
+  }
   hasMulti(token) {
     if (this.multiRegistrations.has(token)) return true;
     return this.parent?.hasMulti(token) ?? false;
@@ -339,6 +355,51 @@ export class Container {
     }
     return fromParent;
   }
+  providerGraphRequiresRequestScope(token, visited) {
+    if (visited.has(token)) {
+      return true;
+    }
+    visited.add(token);
+    try {
+      const provider = this.lookupProvider(token);
+      const multiProviders = this.collectMultiProviders(token);
+      if (!provider && multiProviders.length === 0) {
+        return this.unregisteredClassRequiresRequestScope(token, visited);
+      }
+      if (provider && this.normalizedProviderRequiresRequestScope(provider, visited)) {
+        return true;
+      }
+      return multiProviders.some(multiProvider => this.normalizedProviderRequiresRequestScope(multiProvider, visited));
+    } finally {
+      visited.delete(token);
+    }
+  }
+  unregisteredClassRequiresRequestScope(token, visited) {
+    if (typeof token !== 'function') {
+      return false;
+    }
+    const metadata = getClassDiMetadata(token);
+    if (metadata?.scope === Scope.REQUEST) {
+      return true;
+    }
+    return (metadata?.inject ?? []).some(depEntry => this.dependencyEntryRequiresRequestScope(depEntry, visited));
+  }
+  normalizedProviderRequiresRequestScope(provider, visited) {
+    if (provider.scope === Scope.REQUEST) {
+      return true;
+    }
+    if (provider.type === 'existing' && provider.useExisting !== undefined) {
+      return this.providerGraphRequiresRequestScope(provider.useExisting, visited);
+    }
+    return provider.inject.some(depEntry => this.dependencyEntryRequiresRequestScope(depEntry, visited));
+  }
+  dependencyEntryRequiresRequestScope(depEntry, visited) {
+    const depToken = this.resolveProviderDependencyToken(depEntry);
+    if (isOptionalToken(depEntry) && !this.has(depToken)) {
+      return false;
+    }
+    return this.providerGraphRequiresRequestScope(depToken, visited);
+  }
   async resolveWithChain(token, chain, activeTokens, allowForwardRef = false) {
     const cachedForwardRef = this.resolveForwardRefCircularDependency(token, chain, activeTokens, allowForwardRef);
     if (cachedForwardRef !== undefined) {
@@ -478,6 +539,25 @@ export class Container {
   root() {
     return this.parent ? this.parent.root() : this;
   }
+  ensureTrackedRequestScope() {
+    if (!this.requestScopeEnabled || !this.parent || this.trackedByRoot) {
+      return;
+    }
+    const root = this.root();
+    root.childScopes ??= new Set();
+    root.childScopes.add(this);
+    this.trackedByRoot = true;
+  }
+  requestCacheForWrite() {
+    this.ensureTrackedRequestScope();
+    this.requestCache ??= new Map();
+    return this.requestCache;
+  }
+  multiRequestCacheForWrite() {
+    this.ensureTrackedRequestScope();
+    this.multiRequestCache ??= new Map();
+    return this.multiRequestCache;
+  }
   lookupProvider(token) {
     const local = this.registrations.get(token);
     if (local) {
@@ -500,7 +580,7 @@ export class Container {
   cacheFor(provider) {
     if (provider.scope === Scope.DEFAULT) {
       if (this.requestScopeEnabled && this.registrations.has(provider.provide)) {
-        return this.requestCache;
+        return this.requestCacheForWrite();
       }
       return this.root().singletonCache;
     }
@@ -511,12 +591,12 @@ export class Container {
         hint: 'Wrap the resolve call inside a request-scoped child container created via container.createRequestScope().'
       });
     }
-    return this.requestCache;
+    return this.requestCacheForWrite();
   }
   multiCacheFor(provider) {
     if (provider.scope === Scope.DEFAULT) {
       if (this.requestScopeEnabled && this.hasLocalMultiProvider(provider)) {
-        return this.multiRequestCache;
+        return this.multiRequestCacheForWrite();
       }
       return this.root().multiSingletonCache;
     }
@@ -527,15 +607,15 @@ export class Container {
         hint: 'Wrap the resolve call inside a request-scoped child container created via container.createRequestScope().'
       });
     }
-    return this.multiRequestCache;
+    return this.multiRequestCacheForWrite();
   }
   hasLocalMultiProvider(provider) {
     return this.multiRegistrations.get(provider.provide)?.includes(provider) ?? false;
   }
   disposalCacheEntries() {
     if (this.parent) {
-      const entries = Array.from(this.requestCache.entries());
-      for (const [provider, promise] of this.multiRequestCache.entries()) {
+      const entries = Array.from(this.requestCache?.entries() ?? []);
+      for (const [provider, promise] of this.multiRequestCache?.entries() ?? []) {
         entries.push([provider, promise]);
       }
       return entries;
@@ -591,8 +671,8 @@ export class Container {
   }
   clearDisposalCaches() {
     if (this.parent) {
-      this.requestCache.clear();
-      this.multiRequestCache.clear();
+      this.requestCache?.clear();
+      this.multiRequestCache?.clear();
       return;
     }
     this.singletonCache.clear();
@@ -716,7 +796,7 @@ export class Container {
     return deps;
   }
   invalidateCachedEntry(token, scope) {
-    if (this.requestCache.has(token)) {
+    if (this.requestCache?.has(token)) {
       const cached = this.requestCache.get(token);
       if (cached) {
         this.scheduleStaleDisposal(cached);
@@ -742,12 +822,16 @@ export class Container {
         this.multiSingletonCache.delete(provider);
       }
     }
-    for (const [provider, cached] of this.multiRequestCache.entries()) {
+    const multiRequestCache = this.multiRequestCache;
+    if (!multiRequestCache) {
+      return;
+    }
+    for (const [provider, cached] of multiRequestCache.entries()) {
       if (provider.provide !== token) {
         continue;
       }
       this.scheduleStaleDisposal(cached);
-      this.multiRequestCache.delete(provider);
+      multiRequestCache.delete(provider);
     }
   }
 }

package/package.json

@@ -9,7 +9,7 @@
     "container",
     "provider"
   ],
-  "version": "1.0.0-beta.3",
+  "version": "1.0.0-beta.5",
   "private": false,
   "license": "MIT",
   "repository": {