fix(di): removed initialization of dependencies of type providers aka…

… classes before initialization all services are lazy loaded
r-html · Mar 7, 2024 · 196435d · 196435d
1 parent 1627089
commit 196435d
Show file tree

Hide file tree

Showing 12 changed files with 23 additions and 17 deletions.
diff --git a/packages/component/package.json b/packages/component/package.json
@@ -7,6 +7,7 @@
     "patch": "npm run build && npm version patch && npm publish --update-readme --access public && npm run delete-dist",
     "delete-dist": "rm -rf dist",
     "clean": "git clean -dxf",
+    "test": "echo 'missing tests'",
     "lint": "npx eslint . --ext .ts",
     "lint-fix": "npx eslint . --fix --ext .ts",
     "build": "rm -rf dist && tsc"
@@ -32,4 +33,4 @@
   "types": "./dist/index.d.ts",
   "module": "./dist/index.js",
   "typings": "./dist/index.d.ts"
-}
+}
diff --git a/packages/components/package.json b/packages/components/package.json
@@ -6,6 +6,7 @@
     "start": "npx parcel ./examples/index.html --out-dir build/examples",
     "patch": "npm run build && npm version patch && npm publish --update-readme --access public && npm run delete-dist",
     "delete-dist": "rm -rf dist",
+    "test": "echo 'missing tests'",
     "clean": "git clean -dxf",
     "lint": "npx eslint . --ext .ts",
     "lint-fix": "npx eslint . --fix --ext .ts",
@@ -30,4 +31,4 @@
   "types": "./dist/index.d.ts",
   "module": "./dist/index.js",
   "typings": "./dist/index.d.ts"
-}
+}
diff --git a/packages/custom-attributes/package.json b/packages/custom-attributes/package.json
@@ -6,6 +6,7 @@
     "start": "npx parcel ./examples/index.html --out-dir build/examples",
     "patch": "npm run build && npm version patch && npm publish --update-readme --access public && npm run delete-dist",
     "delete-dist": "rm -rf dist",
+    "test": "echo 'missing tests'",
     "clean": "git clean -dxf",
     "lint": "npx eslint . --ext .ts",
     "lint-fix": "npx eslint . --fix --ext .ts",
@@ -26,4 +27,4 @@
   "types": "./dist/index.d.ts",
   "module": "./dist/index.js",
   "typings": "./dist/index.d.ts"
-}
+}
diff --git a/packages/decorators/package.json b/packages/decorators/package.json
@@ -7,6 +7,7 @@
     "patch": "npm run build && npm version patch && npm publish --update-readme --access public && npm run delete-dist",
     "delete-dist": "rm -rf dist",
     "clean": "git clean -dxf",
+    "test": "echo 'missing tests'",
     "lint": "npx eslint . --ext .ts",
     "lint-fix": "npx eslint . --fix --ext .ts",
     "build": "rm -rf dist && npx npx tsc"
@@ -26,4 +27,4 @@
   "types": "./dist/index.d.ts",
   "module": "./dist/index.js",
   "typings": "./dist/index.d.ts"
-}
+}
diff --git a/packages/di/src/module/module.ts b/packages/di/src/module/module.ts
@@ -42,15 +42,10 @@ export const Module = <T>(
 
       const providers = (entries.providers || []) as WithProviders[];
       const factories = filterNonNull(providers).filter(v => !!v.useFactory);
-      const services = filterNonNull(providers).filter(v => !v.useFactory);
-
+
       for (const entry of factories) {
         ProvidersMetadata.set(entry, entry);
       }
-
-      for (const entry of services) {
-        set(entry);
-      }
 
       for (const entry of filterNonNull(
         (entries.bootstrap || []) as ObjectUnion[]

diff --git a/packages/experiments/package.json b/packages/experiments/package.json
@@ -7,6 +7,7 @@
     "patch": "npm run build && npm version patch && npm publish --update-readme --access public && npm run delete-dist",
     "delete-dist": "rm -rf dist",
     "clean": "git clean -dxf",
+    "test": "echo 'missing tests'",
     "lint": "npx eslint . --ext .ts",
     "lint-fix": "npx eslint . --fix --ext .ts",
     "build": "rm -rf dist && tsc"
@@ -29,4 +30,4 @@
   "types": "./dist/index.d.ts",
   "module": "./dist/index.js",
   "typings": "./dist/index.d.ts"
-}
+}
diff --git a/packages/graphql/package.json b/packages/graphql/package.json
@@ -8,6 +8,7 @@
     "delete-dist": "rm -rf dist",
     "clean": "git clean -dxf",
     "lint": "npx eslint . --ext .ts",
+    "test": "echo 'missing tests'",
     "lint-fix": "npx eslint . --fix --ext .ts",
     "build": "cp ./libs/types ../../node_modules/@apollo/client/link/core/types.d.ts && rm -rf dist && tsc"
   },
@@ -34,4 +35,4 @@
   "types": "./dist/index.d.ts",
   "module": "./dist/index.js",
   "typings": "./dist/index.d.ts"
-}
+}
diff --git a/packages/hooks/package.json b/packages/hooks/package.json
@@ -7,6 +7,7 @@
     "patch": "npm run build && npm version patch && npm publish --update-readme --access public && npm run delete-dist",
     "delete-dist": "rm -rf dist",
     "clean": "git clean -dxf",
+    "test": "echo 'missing tests'",
     "lint": "npx eslint . --ext .ts",
     "lint-fix": "npx eslint . --fix --ext .ts",
     "build": "rm -rf dist && tsc"
@@ -28,4 +29,4 @@
   "types": "./dist/index.d.ts",
   "module": "./dist/index.js",
   "typings": "./dist/index.d.ts"
-}
+}
diff --git a/packages/modifiers/package.json b/packages/modifiers/package.json
@@ -7,6 +7,7 @@
     "patch": "npm run build && npm version patch && npm publish --update-readme --access public && npm run delete-dist",
     "delete-dist": "rm -rf dist",
     "clean": "git clean -dxf",
+    "test": "echo 'missing tests'",
     "lint": "npx eslint . --ext .ts",
     "lint-fix": "npx eslint . --fix --ext .ts",
     "build": "rm -rf dist && npx tsc"
@@ -28,4 +29,4 @@
   "types": "./dist/index.d.ts",
   "module": "./dist/index.js",
   "typings": "./dist/index.d.ts"
-}
+}
diff --git a/packages/operators/package.json b/packages/operators/package.json
@@ -9,6 +9,7 @@
     "clean": "git clean -dxf",
     "lint": "npx eslint . --ext .ts",
     "lint-fix": "npx eslint . --fix --ext .ts",
+    "test": "echo 'missing tests'",
     "build": "rm -rf dist && tsc"
   },
   "repository": {
@@ -29,4 +30,4 @@
   "types": "./dist/index.d.ts",
   "module": "./dist/index.js",
   "typings": "./dist/index.d.ts"
-}
+}
diff --git a/packages/renderer/package.json b/packages/renderer/package.json
@@ -6,6 +6,7 @@
     "patch": "npm run build && npm version patch && npm publish --update-readme --access public && npm run delete-dist",
     "delete-dist": "rm -rf dist",
     "clean": "git clean -dxf",
+    "test": "echo 'missing tests'",
     "lint": "npx eslint . --ext .ts",
     "lint-fix": "npx eslint . --fix --ext .ts",
     "build": "rm -rf dist && tsc"
@@ -30,4 +31,4 @@
   "types": "./dist/index.d.ts",
   "module": "./dist/index.js",
   "typings": "./dist/index.d.ts"
-}
+}
diff --git a/packages/schematics/package.json b/packages/schematics/package.json
@@ -8,6 +8,7 @@
     "delete-dist": "rm -rf dist",
     "clean": "git clean -dxf",
     "lint": "echo LINT",
+    "test": "echo 'missing tests'",
     "lint-fix": "echo LINT-FIX",
     "lint-old": "npx eslint . --ext .ts || true",
     "lint-fix-old": "npx eslint . --fix --ext .ts",
@@ -34,4 +35,4 @@
   "module": "./dist/index.js",
   "typings": "./dist/index.d.ts",
   "schematics": "./collection.json"
-}
+}