perf: avoid container clone when possible

Author: castarco

lambda-ioc/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@coderspirit/lambda-ioc",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "main": "./dist/cjs/index.js",
   "module": "./dist/esm/index.js",
   "types": "./dist/cjs/index.d.ts",

lambda-ioc/src/container.ts

@@ -287,13 +287,21 @@ function __createContainer<
         Container<TSyncDependencies, {}>
       >,
     ): ContainerWithNewSyncDep<TName, TDependency> {
-      return __createContainer(
-        {
-          ...syncDependencies,
-          [name]: dependency,
-        },
-        asyncDependencies,
-      ) as ContainerWithNewSyncDep<TName, TDependency>
+      if (name in syncDependencies) {
+        return __createContainer(
+          {
+            ...syncDependencies,
+            [name]: dependency,
+          },
+          asyncDependencies,
+        ) as ContainerWithNewSyncDep<TName, TDependency>
+      } else {
+        ;(syncDependencies as Record<TName, unknown>)[name] = dependency
+        return __createContainer(
+          syncDependencies,
+          asyncDependencies,
+        ) as ContainerWithNewSyncDep<TName, TDependency>
+      }
     },
 
     registerAsync<TName extends ContainerKey, TDependency>(
@@ -303,23 +311,39 @@ function __createContainer<
         Container<TSyncDependencies, TAsyncDependencies>
       >,
     ): ContainerWithNewAsyncDep<TName, TDependency> {
-      return __createContainer(syncDependencies, {
-        ...asyncDependencies,
-        [name]: dependency,
-      }) as ContainerWithNewAsyncDep<TName, TDependency>
+      if (name in asyncDependencies) {
+        return __createContainer(syncDependencies, {
+          ...asyncDependencies,
+          [name]: dependency,
+        }) as ContainerWithNewAsyncDep<TName, TDependency>
+      } else {
+        ;(asyncDependencies as Record<TName, unknown>)[name] = dependency
+        return __createContainer(
+          syncDependencies,
+          asyncDependencies,
+        ) as ContainerWithNewAsyncDep<TName, TDependency>
+      }
     },
 
     registerValue<TName extends ContainerKey, TDependency>(
       name: TName,
       dependency: TDependency,
     ): ContainerWithNewSyncDep<TName, TDependency> {
-      return __createContainer(
-        {
-          ...syncDependencies,
-          [name]: () => dependency,
-        },
-        asyncDependencies,
-      ) as ContainerWithNewSyncDep<TName, TDependency>
+      if (name in syncDependencies) {
+        return __createContainer(
+          {
+            ...syncDependencies,
+            [name]: () => dependency,
+          },
+          asyncDependencies,
+        ) as ContainerWithNewSyncDep<TName, TDependency>
+      } else {
+        ;(syncDependencies as Record<TName, unknown>)[name] = () => dependency
+        return __createContainer(
+          syncDependencies,
+          asyncDependencies,
+        ) as ContainerWithNewSyncDep<TName, TDependency>
+      }
     },
 
     resolve<TName extends keyof TSyncDependencies>(