@deepkit/injector - npm Package Compare versions

Comparing version 1.0.1-alpha.119 to 1.0.1-alpha.120

CHANGELOG.md

@@ -6,2 +6,8 @@ # Change Log
 
+## [1.0.1-alpha.120](https://github.com/deepkit/deepkit-framework/compare/v1.0.1-alpha.119...v1.0.1-alpha.120) (2024-01-29)
+
+### Features
+
+- **type:** preserve modifiers in homomorphic mapped types ([f2091d0](https://github.com/deepkit/deepkit-framework/commit/f2091d0beeb7360d0bdcc7475d0c88e53dee5de2)), closes [#515](https://github.com/deepkit/deepkit-framework/issues/515) [#514](https://github.com/deepkit/deepkit-framework/issues/514)
+
 ## [1.0.1-alpha.119](https://github.com/deepkit/deepkit-framework/compare/v1.0.1-alpha.118...v1.0.1-alpha.119) (2024-01-28)
@@ -8,0 +14,0 @@

dist/cjs/src/injector.js

@@ -754,5 +754,7 @@ "use strict";
                         const found = (0, module_js_1.findModuleForConfig)(current.indexAccessOrigin.container.classType, resolveDependenciesFrom);
-                        if (!found)
-                            return () => undefined;
-                        config = (0, core_1.getPathValue)(found.module.getConfig(), found.path);
+                        // Only because it has indexAccessOrigin as class doesn't mean it must be a config reference.
+                        // We can safely ignore it if it's not a config reference.
+                        if (found) {
+                            config = (0, core_1.getPathValue)(found.module.getConfig(), found.path);
+                        }
                     }
@@ -759,0 +761,0 @@ if (config !== undefined && current.indexAccessOrigin.index.kind === type_1.ReflectionKind.literal) {

dist/esm/src/injector.js

@@ -739,5 +739,7 @@ const __ΩReturnType = ['T', 'args', '', 'ReturnType', 'l>e"!R"RPde#!P"@2"h"!/#qk#\'QRb!Pde"!p)w$y'];
                         const found = findModuleForConfig(current.indexAccessOrigin.container.classType, resolveDependenciesFrom);
-                        if (!found)
-                            return () => undefined;
-                        config = getPathValue(found.module.getConfig(), found.path);
+                        // Only because it has indexAccessOrigin as class doesn't mean it must be a config reference.
+                        // We can safely ignore it if it's not a config reference.
+                        if (found) {
+                            config = getPathValue(found.module.getConfig(), found.path);
+                        }
                     }
@@ -744,0 +746,0 @@ if (config !== undefined && current.indexAccessOrigin.index.kind === ReflectionKind.literal) {

package.json

 {
   "name": "@deepkit/injector",
-  "version": "1.0.1-alpha.119",
+  "version": "1.0.1-alpha.120",
   "description": "Deepkit Dependency Injection",
@@ -31,3 +31,3 @@ "type": "commonjs",
     "@deepkit/core": "^1.0.1-alpha.114",
-    "@deepkit/type": "^1.0.1-alpha.119",
+    "@deepkit/type": "^1.0.1-alpha.120",
     "benchmark": "^2.1.4"
@@ -52,3 +52,3 @@ },
   },
-  "gitHead": "4d7d89f1fa95442c550a13ba1002b185a59c8432"
+  "gitHead": "e8def52ffe47517994290654e4311f82eeeb8631"
 }

src/injector.ts

@@ -841,4 +841,7 @@ import {
                         const found = findModuleForConfig(current.indexAccessOrigin.container.classType, resolveDependenciesFrom);
-                        if (!found) return () => undefined;
-                        config = getPathValue(found.module.getConfig(), found.path);
+                        // Only because it has indexAccessOrigin as class doesn't mean it must be a config reference.
+                        // We can safely ignore it if it's not a config reference.
+                        if (found) {
+                            config = getPathValue(found.module.getConfig(), found.path);
+                        }
                     }
@@ -845,0 +848,0 @@ if (config !== undefined && current.indexAccessOrigin.index.kind === ReflectionKind.literal) {

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

507273

increased by0.2%

Lines of code

6686

increased by0.1%

No dependency changes