fix: clean for of

alinsjs · Aug 14, 2023 · 0452182 · 0452182
1 parent a9c9b7d
commit 0452182
Show file tree

Hide file tree

Showing 17 changed files with 26 additions and 20 deletions.
diff --git a/packages/client-core/package.json b/packages/client-core/package.json
@@ -25,7 +25,7 @@
     "publishConfig": {
         "registry": "https://registry.npmjs.org/"
     },
-    "module": "dist/alins.esm.min.js",
+    "module": "src/index.ts",
     "unpkg": "dist/alins.iife.min.js",
     "jsdelivr": "dist/alins.iife.min.js"
 }
diff --git a/packages/client-reactive/package.json b/packages/client-reactive/package.json
@@ -26,5 +26,5 @@
     },
     "unpkg": "dist/alins-reactive.iife.min.js",
     "jsdelivr": "dist/alins-reactive.iife.min.js",
-    "module": "dist/alins-reactive.esm.min.js"
+    "module": "src/index.ts"
 }
diff --git a/packages/client-reactive/src/cleaner.ts b/packages/client-reactive/src/cleaner.ts
@@ -32,11 +32,15 @@ export function createCleaner () {
 
     const cleaner: ICleaner = {
         clean () {
-            const keys = cleanMap.keys();
-            for (const key of keys) {
-                // @ts-ignore
-                cleanMap.get(key)();
-            }
+            cleanMap.forEach(clean => {
+                clean();
+            });
+            // ! 使用for of 部分打包工具会转译成 普通for语句 导致元素没有被遍历到，没有释放内存
+            // const keys = cleanMap.keys();
+            // for (const key of keys) {
+            //     // @ts-ignore
+            //     cleanMap.get(key)();
+            // }
             cleanMap.clear();
             // @ts-ignore
             cleanMap = null;

diff --git a/packages/client-standalone/package.json b/packages/client-standalone/package.json
@@ -26,5 +26,5 @@
     "dependencies": {
         "alins": "^0.0.22"
     },
-    "module": "dist/alins-standalone.esm.min.js"
+    "module": "src/index.ts"
 }
diff --git a/packages/client-utils/package.json b/packages/client-utils/package.json
@@ -23,5 +23,5 @@
     },
     "unpkg": "dist/alins-utils.iife.min.js",
     "jsdelivr": "dist/alins-utils.iife.min.js",
-    "module": "dist/alins-utils.esm.min.js"
+    "module": "src/index.ts"
 }
diff --git a/packages/compiler-core/package.json b/packages/compiler-core/package.json
@@ -25,5 +25,5 @@
         "@babel/types": "^7.22.5",
         "@types/babel__traverse": "^7.20.1"
     },
-    "module": "dist/alins-compiler-core.esm.min.js"
+    "module": "src/index.ts"
 }
diff --git a/packages/compiler-node/package.json b/packages/compiler-node/package.json
@@ -27,5 +27,5 @@
         "@babel/preset-typescript": "^7.22.5",
         "alins-compiler-core": "^0.0.22"
     },
-    "module": "dist/alins-compiler-node.esm.min.js"
+    "module": "src/index.ts"
 }
diff --git a/packages/compiler-web/package.json b/packages/compiler-web/package.json
@@ -28,5 +28,5 @@
     },
     "unpkg": "dist/alins-compiler-web.iife.min.js",
     "jsdelivr": "dist/alins-compiler-web.iife.min.js",
-    "module": "dist/alins-compiler-web.esm.min.js"
+    "module": "src/index.ts"
 }
diff --git a/packages/plugin-babel-preset/package.json b/packages/plugin-babel-preset/package.json
@@ -26,5 +26,5 @@
         "@babel/preset-typescript": "^7.22.5",
         "babel-plugin-alins": "^0.0.22"
     },
-    "module": "dist/babel-preset-alins.esm.min.js"
+    "module": "src/index.ts"
 }
diff --git a/packages/plugin-babel/package.json b/packages/plugin-babel/package.json
@@ -24,5 +24,5 @@
     "dependencies": {
         "alins-compiler-core": "^0.0.22"
     },
-    "module": "dist/babel-plugin-alins.esm.min.js"
+    "module": "src/index.ts"
 }
diff --git a/packages/plugin-esbuild/package.json b/packages/plugin-esbuild/package.json
@@ -24,5 +24,5 @@
     "dependencies": {
         "alins-compiler-node": "^0.0.22"
     },
-    "module": "dist/esbuild-plugin-alins.esm.min.js"
+    "module": "src/index.ts"
 }
diff --git a/packages/plugin-eslint/package.json b/packages/plugin-eslint/package.json
@@ -24,5 +24,6 @@
         "@typescript-eslint/parser": "^6.3.0",
         "eslint-plugin-react": "^7.33.1"
     },
-    "typings": "src/index.ts"
+    "typings": "src/index.ts",
+    "module": "src/index.ts"
 }
diff --git a/packages/plugin-rollup/package.json b/packages/plugin-rollup/package.json
@@ -24,5 +24,5 @@
     "dependencies": {
         "alins-compiler-node": "^0.0.22"
     },
-    "module": "dist/rollup-plugin-alins.esm.min.js"
+    "module": "src/index.ts"
 }
diff --git a/packages/plugin-vite/package.json b/packages/plugin-vite/package.json
@@ -24,5 +24,5 @@
     "dependencies": {
         "alins-compiler-node": "^0.0.22"
     },
-    "module": "dist/vite-plugin-alins.esm.min.js"
+    "module": "src/index.ts"
 }
diff --git a/packages/plugin-webpack/package.json b/packages/plugin-webpack/package.json
@@ -24,5 +24,5 @@
     "dependencies": {
         "alins-compiler-node": "^0.0.22"
     },
-    "module": "dist/alins-loader.esm.min.js"
+    "module": "src/index.ts"
 }
diff --git a/scripts/build/utils.js b/scripts/build/utils.js
@@ -64,6 +64,7 @@ function initSinglePackageInfo (dir, isDev = false) {
     if (isDev) {
         package.main = 'src/index.ts';
         package.typings = 'src/index.ts';
+        package.module = 'src/index.ts';
     } else {
         // package.main = `dist/${packageName}.esm.js`;
         // package.typings = `dist/${packageName}.d.ts`;

diff --git a/scripts/dev/samples-list.ts b/scripts/dev/samples-list.ts
@@ -1 +1 @@
-export default ["bench","data","if-sample","increment","list","style-reactive","todo-list"];
+export default ["bench","data","if-sample","increment","list","rfc","style-reactive","todo-list","vjsx"];