refactor: use array for dependencies

exuanbo · Oct 11, 2024 · cdb4c2e · cdb4c2e
1 parent 3b3f8df
commit cdb4c2e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/src/decorators.ts b/src/decorators.ts
@@ -44,7 +44,7 @@ export function Inject<Values extends unknown[]>(...injections: Injections<Value
 export function Deferred<Values extends unknown[]>(...injections: Injections<Values>): ClassFieldDecorator<Values[number]> {
   return (_value, context) => {
     const metadata = metadataRegistry.ensure(context.metadata)
-    metadata.dependencies.add({
+    metadata.dependencies.push({
       injections,
       setValue: context.access.set,
     })

diff --git a/src/metadata.ts b/src/metadata.ts
@@ -5,8 +5,7 @@ import type {Constructor, InjectionToken} from './token'
 export interface InjectionMetadata<This extends object = any> {
   scope?: InjectionScope
   tokens: InjectionToken<This>[]
-  // TODO: use array
-  dependencies: Set<InjectionDependency<This>>
+  dependencies: InjectionDependency<This>[]
 }
 
 export interface InjectionDependency<This extends object, Value = any> {
@@ -24,7 +23,7 @@ class InjectionMetadataRegistry {
   ensure<T extends object>(key: DecoratorMetadata): InjectionMetadata<T> {
     let metadata = this.map.get(key)
     if (!metadata) {
-      metadata = {tokens: [], dependencies: new Set()}
+      metadata = {tokens: [], dependencies: []}
       this.map.set(key, metadata)
     }
     return metadata