From dbb8bd02e4c39bb806b70fb963d78ccca8170392 Mon Sep 17 00:00:00 2001
From: Diablohu <Diablohu@users.noreply.github.com>
Date: Tue, 5 Nov 2019 14:43:13 +0800
Subject: [PATCH] fix #172 & #185

---
 .eslintignore                                 |  1 +
 CHANGELOG.md                                  |  7 +++-
 package.json                                  |  2 +-
 .../factory-config/module/rules/javascript.js |  8 ++--
 .../factory-config/transform-config-server.js |  2 +-
 packages/koot-webpack/loaders/babel/index.js  | 38 +++++++++++++++----
 packages/koot/package.json                    |  2 +-
 7 files changed, 44 insertions(+), 16 deletions(-)
 create mode 100644 .eslintignore

diff --git a/.eslintignore b/.eslintignore
new file mode 100644
index 00000000..fa9455ef
--- /dev/null
+++ b/.eslintignore
@@ -0,0 +1 @@
+/test/projects/*/dist*/**/*
diff --git a/CHANGELOG.md b/CHANGELOG.md
index cf2d4143..187425a2 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -4,10 +4,13 @@
 
 **koot**
 
+-   **重大改动**
+    -   _Node.js_ 最低版本要求提升到 `8.12.0`
+    -   [SSR & 客户端渲染] 组件 CSS 的 `<style>` 标签上不再有 `id` 属性，以避免和元素冲突
+        -   如果项目中有用到根据 `id` 选择 `<style>` 标签的场景，可改为选择标签属性 `[data-koot-module]`
+    -   [服务器] 服务器端打包代码不再会对 `await` `async` 进行转译 ([#172](https://github.com/cmux/koot/issues/172), [#186](https://github.com/cmux/koot/issues/186))
 -   **新特性**
     -   **新工具函数** `koot/utils/client-get-styles` - 获取当前全局 CSS 和所有组件 CSS。具体用法请参见文档 [全局与工具函数/工具函数/客户端](https://koot.js.org/#/utilities?id=客户端) ([#185](https://github.com/cmux/koot/issues/185))
--   优化
-    -   组件 CSS 的 `<style>` 标签上不再有 `id` 属性，以避免和元素冲突
 
 ## [0.11.15] - 2019-10-15
 
diff --git a/package.json b/package.json
index ac92bd80..e8340760 100644
--- a/package.json
+++ b/package.json
@@ -39,7 +39,7 @@
     },
     "homepage": "https://github.com/cmux/koot",
     "engines": {
-        "node": ">=8.0.0"
+        "node": ">=8.12.0"
     },
     "license": "Apache-2.0",
     "dependencies": {
diff --git a/packages/koot-webpack/factory-config/module/rules/javascript.js b/packages/koot-webpack/factory-config/module/rules/javascript.js
index 4b113c9a..1e06fc86 100644
--- a/packages/koot-webpack/factory-config/module/rules/javascript.js
+++ b/packages/koot-webpack/factory-config/module/rules/javascript.js
@@ -165,7 +165,7 @@ module.exports = (kootBuildConfig = {}) => {
             {
                 test: /\.(js|mjs|cjs|jsx)$/,
                 use: [
-                    ...ruleUseLoaders(),
+                    ...ruleUseLoaders({}),
                     require.resolve('../../../loaders/koot-dev-ssr.js')
                 ]
             },
@@ -184,11 +184,13 @@ module.exports = (kootBuildConfig = {}) => {
     return [
         {
             test: /\.(js|mjs|cjs|jsx)$/,
-            use: ruleUseBabelLoader()
+            use: ruleUseBabelLoader({})
         },
         {
             test: /\.(ts|tsx)$/,
-            use: ruleUseBabelLoader({ __typescript: true })
+            use: ruleUseBabelLoader({
+                __typescript: true
+            })
         }
     ];
 };
diff --git a/packages/koot-webpack/factory-config/transform-config-server.js b/packages/koot-webpack/factory-config/transform-config-server.js
index d3119288..5c083b22 100644
--- a/packages/koot-webpack/factory-config/transform-config-server.js
+++ b/packages/koot-webpack/factory-config/transform-config-server.js
@@ -113,7 +113,7 @@ module.exports = async (kootBuildConfig = {}) => {
     // entry / 入口
     const entryIndex = [
         // '@babel/register',
-        '@babel/polyfill',
+        // '@babel/polyfill',
         // 'core-js/stable',
         // path.resolve(__dirname, '../../../defaults/server-stage-0.js'),
         require('../libs/get-koot-file')(`${appType}/server`)
diff --git a/packages/koot-webpack/loaders/babel/index.js b/packages/koot-webpack/loaders/babel/index.js
index 33fdf8a1..910835d8 100644
--- a/packages/koot-webpack/loaders/babel/index.js
+++ b/packages/koot-webpack/loaders/babel/index.js
@@ -36,8 +36,10 @@ module.exports = require('babel-loader').custom(babel => {
                 __typescript = false
             } = customOptions;
             const { presets, plugins, ...options } = cfg.options;
+            const stage = process.env.WEBPACK_BUILD_STAGE;
             // console.log({ options });
 
+            // presets ========================================================
             const newPresets = [...presets];
             if (__typescript) {
                 newPresets.unshift([
@@ -68,7 +70,25 @@ module.exports = require('babel-loader').custom(babel => {
             //     }
             //     return preset
             // })
+            if (stage === 'server') {
+                newPresets.map(preset => {
+                    if (
+                        typeof preset.file === 'object' &&
+                        /^@babel\/preset-env$/.test(preset.file.request)
+                    ) {
+                        if (!preset.options) preset.options = {};
+                        preset.options.modules = false;
+                        preset.options.exclude = [
+                            '@babel/plugin-transform-regenerator',
+                            '@babel/plugin-transform-async-to-generator'
+                        ];
+                        return preset;
+                    }
+                    return preset;
+                });
+            }
 
+            // plugins ========================================================
             const newPlugins = plugins.filter(plugin => {
                 if (
                     typeof plugin.file === 'object' &&
@@ -78,14 +98,16 @@ module.exports = require('babel-loader').custom(babel => {
                         ))
                 )
                     return false;
-                // if (
-                //     process.env.WEBPACK_BUILD_STAGE === 'server' &&
-                //     typeof plugin.file === 'object' &&
-                //     /@babel(\/|\\)plugin-transform-regenerator/.test(
-                //         plugin.file.request
-                //     )
-                // )
-                //     return false;
+
+                if (
+                    stage === 'server' &&
+                    typeof plugin.file === 'object' &&
+                    /@babel(\/|\\)plugin-transform-regenerator/.test(
+                        plugin.file.request
+                    )
+                )
+                    return false;
+
                 return true;
             });
 
diff --git a/packages/koot/package.json b/packages/koot/package.json
index d318c6bd..fa0b9bba 100644
--- a/packages/koot/package.json
+++ b/packages/koot/package.json
@@ -43,7 +43,7 @@
     },
     "homepage": "https://github.com/cmux/koot",
     "engines": {
-        "node": ">=8.9.0"
+        "node": ">=8.12.0"
     },
     "dependencies": {
         "@babel/core": "^7.6.4",