@modern-js/app-tools 2.0.0-beta.0 → 2.0.0-beta.1

package/CHANGELOG.md

@@ -1,5 +1,186 @@
 # @modern-js/app-tools
 
+## 2.0.0-beta.1
+
+### Major Changes
+
+- dda38c9: chore: v2
+
+### Minor Changes
+
+- edd1cfb1af: feat: modernjs Access builder compiler
+  feat: modernjs 接入 builder 构建
+- 543be9558e: feat: compile server loader and support handle loader request
+  feat: 编译 server loader 并支持处理 loader 的请求
+
+### Patch Changes
+
+- c9f912ca4d: feat(app-tools): improve build logs of dev and build command
+
+  feat(app-tools): 优化 dev 和 build 过程中的日志展示
+
+- 103973cde9: fix: builder tools.webpackChain config args not match the Modernjs tools.webpackChain
+  fix: builder tools.webpackChain 配置传参无法匹配 Modernjs tools.webpackChain
+- 0b2d1ef02b: fix: repeat register `babel-plugin-lodash`
+  fix: 重复注册 `babel-plugin-lodash`
+- 82cef85ed7: fix: specify builder compiler framework
+  fix: 指明 builder 构建时框架
+- 3e57f2b: feat: add document feature with plugin
+
+  feat: 增加 document 功能插件
+
+- 85edee888c: feat(app-tools): support tools.htmlPlugin config
+
+  feat(app-tools): 支持 tools.htmlPlugin 配置项
+
+- 5402fdb: feat(Builder): add output.disableTsChecker config
+
+  feat(Builder): 新增 output.disableTsChecker 配置项
+
+- dc8eeb9cbb: fix: clear distDirectory in prepare hook & inject data loader plugin to server
+  fix: 在 prepare hook 中清理 dist 目录，并且向 server 中注入 data loader plugin
+- cc971eabfc: refactor: move server plugin load logic in `@modern-js/core`
+  refactor：移除在 `@modern-js/core` 中的 server 插件加载逻辑
+- 5b9049f: feat: inject async js chunk when streaming ssr
+  feat: streaming ssr 时, 注入 async 类型的 js chunk
+- d4a4566: chore: rename plugin-jarvis to plugin-lint
+
+  chore: 重命名 plugin-jarvis 为 plugin-lint
+
+- 6bda14ed71: feat: refactor router with react-router@6.4
+
+  feat: 使用 react-router@6.4 重构路由模块
+
+- d36c6ee126: fix(app-tools): failed to run inspect command
+
+  fix(app-tools): 修复运行 inspect 命令失败的问题
+
+- 92004d1: feat: support load chunks parallelly
+  feat: 支持并行加载 chunks
+- b8bbe036c7: feat: change type logic
+  feat: 修改类型相关的逻辑
+- 40ed587: feat: inject css chunk into html for streaming ssr
+  feat: streaming ssr 返回的 html 注入 css chunk
+- af4422d: feat(builder): complete utils of tools.webpack
+
+  feat(builder): 补全 tools.webpack 提供的 utils 方法
+
+- 87c1ff8: feat(app-tools): attach builder instance to appContext
+
+  feat(app-tools): 将 builder 实例挂载到 appContext 上
+
+- c258e34202: fix: add builder hooks `beforeBuild` params
+  fix: 新增 builder hooks `beforeBuild` 的参数
+- 8b8e1bb571: feat: support nested routes
+  feat: 支持嵌套路由
+- 7de97ae: fix: `deploy` command has't load `builder` instance
+  fix: `deploy` 命令没有加载 builder 实例
+- c3b7de4: fix(app-tools): dev.assetPrefix not work
+
+  fix(app-tools): 修复 dev.assetPrefix 配置项不生效的问题
+
+- 92004d1: fix: use loadable lazy instead of loadable
+  fix: 使用 loadable lazy 组件替代 loadable
+- c677befc22: fix(app-tools): compat legacy resolve behavior
+
+  fix(app-tools): 兼容旧版本 node_modules 解析逻辑
+
+- 3f7cde5caa: fix: builder plugin setup can't get config
+  fix: builder 插件在 setup 阶段无法拿到 config
+- 99213e4bae: fix: process does't exit when exec command
+  fix: 修复执行命令时进程未退出的问题
+- b16fd96: fix: `modern-js/app-tools` pass error config to builder.
+  fix: `modern-js/app-tools` 传递错误的 config 给 builder.
+- 7eefedd7ca: fix: add html-webpack-plugin `__internal__` options, for bottom template
+  fix: 为了 bottom template, 增加 `html-webpack-plugin` `__internal__` 配置项，
+- 14b712d: fix: use consistent alias type and default value across packages
+
+  fix: 在各个包中使用一致的 alias 类型定义和默认值
+
+- Updated dependencies [c9f912ca4d]
+- Updated dependencies [95be7cc49c]
+- Updated dependencies [e439457a51]
+- Updated dependencies [4d1545f8c0]
+- Updated dependencies [2bc090c089]
+- Updated dependencies [f96a725211]
+- Updated dependencies [828f42f9ce]
+- Updated dependencies [c745686]
+- Updated dependencies [c9e800d39a]
+- Updated dependencies [0ff846f]
+- Updated dependencies [57077b2]
+- Updated dependencies [d032d49]
+- Updated dependencies [2ff6167]
+- Updated dependencies [287f298990]
+- Updated dependencies [15bf09d9c8]
+- Updated dependencies [423188db70]
+- Updated dependencies [fd2d652]
+- Updated dependencies [0c2d8dae31]
+- Updated dependencies [2edad29]
+- Updated dependencies [85edee888c]
+- Updated dependencies [a2509bfbdb]
+- Updated dependencies [3998875791]
+- Updated dependencies [3998875791]
+- Updated dependencies [ba86b8b711]
+- Updated dependencies [61f21d1e77]
+- Updated dependencies [5402fdb]
+- Updated dependencies [2ae58176fe]
+- Updated dependencies [92f0ead]
+- Updated dependencies [edd1cfb1af]
+- Updated dependencies [5d67c26]
+- Updated dependencies [cc971eabfc]
+- Updated dependencies [5b9049f]
+- Updated dependencies [d4a4566]
+- Updated dependencies [18360a38d7]
+- Updated dependencies [6bda14ed71]
+- Updated dependencies [0b314e6946]
+- Updated dependencies [92004d1]
+- Updated dependencies [b8bbe036c7]
+- Updated dependencies [40ed587]
+- Updated dependencies [af4422d]
+- Updated dependencies [87c1ff8]
+- Updated dependencies [d5a31df781]
+- Updated dependencies [dda38c9]
+- Updated dependencies [102d32e4ba]
+- Updated dependencies [c258e34202]
+- Updated dependencies [7248342e4d]
+- Updated dependencies [568eab1e42]
+- Updated dependencies [8b8e1bb571]
+- Updated dependencies [3bbea92b2a]
+- Updated dependencies [ae71096d45]
+- Updated dependencies [73cd29dd9f]
+- Updated dependencies [a23010138d]
+- Updated dependencies [75d1b2657c]
+- Updated dependencies [18aaf42]
+- Updated dependencies [f179749]
+- Updated dependencies [3fae2d0]
+- Updated dependencies [8a6d45f]
+- Updated dependencies [ebbeed1]
+- Updated dependencies [90e2879520]
+- Updated dependencies [df41d71]
+- Updated dependencies [f727e5c6cc]
+- Updated dependencies [5e3cecd523]
+- Updated dependencies [abf3421]
+- Updated dependencies [da2d1fc3c2]
+- Updated dependencies [543be9558e]
+- Updated dependencies [14b712d]
+  - @modern-js/builder-webpack-provider@2.0.0-beta.1
+  - @modern-js/builder-shared@2.0.0-beta.1
+  - @modern-js/core@2.0.0-beta.1
+  - @modern-js/prod-server@2.0.0-beta.1
+  - @modern-js/builder-plugin-esbuild@2.0.0-beta.1
+  - @modern-js/server@2.0.0-beta.1
+  - @modern-js/node-bundle-require@2.0.0-beta.1
+  - @modern-js/utils@2.0.0-beta.1
+  - @modern-js/builder-plugin-node-polyfill@2.0.0-beta.1
+  - @modern-js/types@2.0.0-beta.1
+  - @modern-js/plugin-lint@2.0.0-beta.1
+  - @modern-js/plugin@2.0.0-beta.1
+  - @modern-js/builder@2.0.0-beta.1
+  - @modern-js/plugin-data-loader@2.0.0-beta.1
+  - @modern-js/plugin-i18n@2.0.0-beta.1
+  - @modern-js/new-action@2.0.0-beta.1
+  - @modern-js/upgrade@2.0.0-beta.1
+
 ## 2.0.0-beta.0
 
 ### Major Changes

package/dist/js/modern/analyze/generateCode.js

@@ -8,11 +8,9 @@ import { getClientRoutes, getClientRoutesLegacy } from "./getClientRoutes";
 import { FILE_SYSTEM_ROUTES_FILE_NAME, ENTRY_POINT_FILE_NAME, ENTRY_BOOTSTRAP_FILE_NAME } from "./constants";
 import { getDefaultImports } from "./utils";
 import { walk } from "./nestedRoutes";
-
 const createImportSpecifier = specifiers => {
   let defaults = '';
   const named = [];
-
   for (const {
     local,
     imported
@@ -25,7 +23,6 @@ const createImportSpecifier = specifiers => {
       named.push(imported);
     }
   }
-
   if (defaults && named.length) {
     return `${defaults}, { ${named.join(', ')} }`;
   } else if (defaults) {
@@ -34,12 +31,10 @@ const createImportSpecifier = specifiers => {
     return `{ ${named.join(', ')} }`;
   }
 };
-
 export const createImportStatements = statements => {
   // merge import statements with the same value.
   const deDuplicated = [];
   const seen = new Map();
-
   for (const {
     value,
     specifiers,
@@ -54,22 +49,19 @@ export const createImportStatements = statements => {
       seen.set(value, specifiers);
     } else {
       var _deDuplicated$modifyI, _deDuplicated$modifyI2;
-
-      seen.get(value).push(...specifiers); // make "initialize" param can be connected when multiple plugins were imported from same package
-
+      seen.get(value).push(...specifiers);
+      // make "initialize" param can be connected when multiple plugins were imported from same package
       const modifyIndex = deDuplicated.findIndex(v => v.value === value);
       const originInitialize = (_deDuplicated$modifyI = (_deDuplicated$modifyI2 = deDuplicated[modifyIndex]) === null || _deDuplicated$modifyI2 === void 0 ? void 0 : _deDuplicated$modifyI2.initialize) !== null && _deDuplicated$modifyI !== void 0 ? _deDuplicated$modifyI : '';
       deDuplicated[modifyIndex].initialize = originInitialize.concat(`\n${initialize || ''}`);
     }
   }
-
   return deDuplicated.map(({
     value,
     specifiers,
     initialize
   }) => `import ${createImportSpecifier(specifiers)} from '${value}';\n${initialize || ''}`).join('\n');
 };
-
 const buildLoader = async (entry, outfile) => {
   const loader = {
     '.js': 'jsx',
@@ -89,32 +81,27 @@ const buildLoader = async (entry, outfile) => {
     outfile,
     plugins: [{
       name: 'make-all-packages-external',
-
       setup(build) {
         // https://github.com/evanw/esbuild/issues/619#issuecomment-751995294
         build.onResolve({
           filter: EXTERNAL_REGEXP
         }, args => {
-          let external = true; // FIXME: windows external entrypoint
-
+          let external = true;
+          // FIXME: windows external entrypoint
           if (args.kind === 'entry-point') {
             external = false;
           }
-
           return {
             path: args.path,
             external
           };
         });
       }
-
     }]
   });
 };
-
 export const generateCode = async (appContext, config, entrypoints, api) => {
   var _config$runtime, _config$runtime$route;
-
   const {
     internalDirectory,
     distDirectory,
@@ -129,7 +116,6 @@ export const generateCode = async (appContext, config, entrypoints, api) => {
   } = config.output;
   const getRoutes = islegacy ? getClientRoutesLegacy : getClientRoutes;
   await Promise.all(entrypoints.map(generateEntryCode));
-
   async function generateEntryCode(entrypoint) {
     const {
       entryName,
@@ -137,13 +123,11 @@ export const generateCode = async (appContext, config, entrypoints, api) => {
       customBootstrap,
       fileSystemRoutes
     } = entrypoint;
-
     if (isAutoMount) {
       // generate routes file for file system routes entrypoint.
       if (fileSystemRoutes) {
         let initialRoutes = [];
         let nestedRoute = null;
-
         if (entrypoint.entry) {
           initialRoutes = getRoutes({
             entrypoint,
@@ -153,24 +137,21 @@ export const generateCode = async (appContext, config, entrypoints, api) => {
             internalDirAlias
           });
         }
-
         if (entrypoint.nestedRoutesEntry) {
           if (!islegacy) {
             nestedRoute = await walk(entrypoint.nestedRoutesEntry, entrypoint.nestedRoutesEntry, {
               name: internalSrcAlias,
               basename: srcDirectory
-            });
-
+            }, entrypoint.entryName);
             if (nestedRoute) {
               initialRoutes.unshift(nestedRoute);
             }
           } else {
-            logger.error('Nested routes is not supported in legacy mode.'); // eslint-disable-next-line no-process-exit
-
+            logger.error('Nested routes is not supported in legacy mode.');
+            // eslint-disable-next-line no-process-exit
             process.exit(1);
           }
         }
-
         const {
           routes
         } = await hookRunners.modifyFileSystemRoutes({
@@ -180,13 +161,19 @@ export const generateCode = async (appContext, config, entrypoints, api) => {
         const config = useResolvedConfigContext();
         const ssr = config === null || config === void 0 ? void 0 : config.server.ssr;
         let mode;
-
         if (ssr) {
           mode = typeof ssr === 'object' ? ssr.mode || 'string' : 'string';
         } else {
           mode = false;
         }
-
+        if (mode === 'stream') {
+          const hasPageRoute = routes.some(route => 'type' in route && route.type === 'page');
+          if (hasPageRoute) {
+            logger.error('streaming ssr is not supported when pages dir exists');
+            // eslint-disable-next-line no-process-exit
+            process.exit(1);
+          }
+        }
         const {
           code
         } = await hookRunners.beforeGenerateRoutes({
@@ -194,10 +181,12 @@ export const generateCode = async (appContext, config, entrypoints, api) => {
           code: templates.fileSystemRoutes({
             routes,
             ssrMode: mode,
-            nestedRoutesEntry: entrypoint.nestedRoutesEntry
+            nestedRoutesEntry: entrypoint.nestedRoutesEntry,
+            entryName: entrypoint.entryName
           })
-        }); // extract nested router loaders
+        });
 
+        // extract nested router loaders
         if (entrypoint.nestedRoutesEntry) {
           const routesServerFile = path.join(internalDirectory, entryName, 'routes.server.js');
           const outputRoutesServerFile = path.join(distDirectory, 'loader-routes', entryName, 'index.js');
@@ -212,11 +201,10 @@ export const generateCode = async (appContext, config, entrypoints, api) => {
           await fs.writeFile(routesServerFile, code);
           await buildLoader(routesServerFile, outputRoutesServerFile);
         }
-
         fs.outputFileSync(path.resolve(internalDirectory, `./${entryName}/${FILE_SYSTEM_ROUTES_FILE_NAME}`), code, 'utf8');
-      } // call modifyEntryImports hook
-
+      }
 
+      // call modifyEntryImports hook
       const {
         imports: importStatements
       } = await hookRunners.modifyEntryImports({
@@ -228,15 +216,17 @@ export const generateCode = async (appContext, config, entrypoints, api) => {
           internalDirAlias,
           internalDirectory
         })
-      }); // call modifyEntryRuntimePlugins hook
+      });
 
+      // call modifyEntryRuntimePlugins hook
       const {
         plugins
       } = await hookRunners.modifyEntryRuntimePlugins({
         entrypoint,
         plugins: []
-      }); // call modifyEntryRenderFunction hook
+      });
 
+      // call modifyEntryRenderFunction hook
       const {
         code: renderFunction
       } = await hookRunners.modifyEntryRenderFunction({
@@ -246,8 +236,9 @@ export const generateCode = async (appContext, config, entrypoints, api) => {
           customBootstrap,
           fileSystemRoutes
         })
-      }); // call modifyEntryExport hook
+      });
 
+      // call modifyEntryExport hook
       const {
         exportStatement
       } = await hookRunners.modifyEntryExport({
@@ -261,8 +252,9 @@ export const generateCode = async (appContext, config, entrypoints, api) => {
         exportStatement
       });
       const entryFile = path.resolve(internalDirectory, `./${entryName}/${ENTRY_POINT_FILE_NAME}`);
-      entrypoint.entry = entryFile; // generate entry file.
+      entrypoint.entry = entryFile;
 
+      // generate entry file.
       if (config.source.enableAsyncEntry) {
         const {
           code: asyncEntryCode

package/dist/js/modern/analyze/getBundleEntry.js

@@ -2,30 +2,24 @@ import path from 'path';
 import { ensureAbsolutePath, fs, findExists, MAIN_ENTRY_NAME } from '@modern-js/utils';
 import { getFileSystemEntry } from "./getFileSystemEntry";
 import { JS_EXTENSIONS } from "./constants";
-
 const ensureExtensions = file => {
   if (!path.extname(file)) {
     return findExists(JS_EXTENSIONS.map(ext => `${file}${ext}`)) || file;
   }
-
   return file;
 };
-
 const ifAlreadyExists = (entrypoints, checked) => entrypoints.some(entrypoint => {
   if (ensureExtensions(entrypoint.entry) === ensureExtensions(checked.entry)) {
     // reset entryName
     checked.entryName = entrypoint.entryName;
     return true;
-  } // filesystem routes entrypoint conflict with normal entrypoint.
-
-
+  }
+  // filesystem routes entrypoint conflict with normal entrypoint.
   if (entrypoint.entry.startsWith(checked.entry) || checked.entry.startsWith(entrypoint.entry)) {
     throw new Error(`Entry configuration conflicts\n Your configuration: ${checked.entry}.\n Default entrypoint: ${entrypoint.entry}\n Please reset source.entries or set source.disableDefaultEntries to disable the default entry rules.`);
   }
-
   return false;
 });
-
 export const getBundleEntry = (appContext, config) => {
   const {
     appDirectory,
@@ -38,8 +32,9 @@ export const getBundleEntry = (appContext, config) => {
       entriesDir
     }
   } = config;
-  const defaults = disableDefaultEntries ? [] : getFileSystemEntry(appContext, config); // merge entrypoints from user config with directory convention.
+  const defaults = disableDefaultEntries ? [] : getFileSystemEntry(appContext, config);
 
+  // merge entrypoints from user config with directory convention.
   if (entries) {
     Object.keys(entries).forEach(name => {
       const value = entries[name];
@@ -54,13 +49,11 @@ export const getBundleEntry = (appContext, config) => {
         isAutoMount: !value.disableMount,
         fileSystemRoutes: value.enableFileSystemRoutes ? {} : undefined
       };
-
       if (!ifAlreadyExists(defaults, entrypoint)) {
         defaults.push(entrypoint);
       }
     });
   }
-
   if (!disableDefaultEntries) {
     // find main entry point which server route is '/'.
     const entriesDirAbs = ensureAbsolutePath(appDirectory, entriesDir);
@@ -71,6 +64,5 @@ export const getBundleEntry = (appContext, config) => {
     }) => entryName === packageName || path.dirname(entry) === entriesDirAbs || path.dirname(_nestedRoutesEntry) === entriesDirAbs);
     found && (found.entryName = MAIN_ENTRY_NAME);
   }
-
   return defaults;
 };

package/dist/js/modern/analyze/getClientRoutes/getRoutes.js

@@ -1,30 +1,24 @@
 function ownKeys(object, enumerableOnly) { var keys = Object.keys(object); if (Object.getOwnPropertySymbols) { var symbols = Object.getOwnPropertySymbols(object); enumerableOnly && (symbols = symbols.filter(function (sym) { return Object.getOwnPropertyDescriptor(object, sym).enumerable; })), keys.push.apply(keys, symbols); } return keys; }
-
 function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { var source = null != arguments[i] ? arguments[i] : {}; i % 2 ? ownKeys(Object(source), !0).forEach(function (key) { _defineProperty(target, key, source[key]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(target, Object.getOwnPropertyDescriptors(source)) : ownKeys(Object(source)).forEach(function (key) { Object.defineProperty(target, key, Object.getOwnPropertyDescriptor(source, key)); }); } return target; }
-
 function _defineProperty(obj, key, value) { if (key in obj) { Object.defineProperty(obj, key, { value: value, enumerable: true, configurable: true, writable: true }); } else { obj[key] = value; } return obj; }
-
 import path from 'path';
 import { fs } from '@modern-js/utils';
 import { makeLegalIdentifier } from "../makeLegalIdentifier";
 import { FILE_SYSTEM_ROUTES_COMPONENTS_DIR, FILE_SYSTEM_ROUTES_DYNAMIC_REGEXP, FILE_SYSTEM_ROUTES_INDEX, FILE_SYSTEM_ROUTES_LAYOUT } from "../constants";
 import { replaceWithAlias } from "../utils";
 import { debug, findLayout, shouldSkip, getRouteWeight } from "./utils";
-
 const compName = (srcDirectory, filePath) => {
   const legalCompName = makeLegalIdentifier(path.relative(srcDirectory, filePath));
   return `Comp_${legalCompName}`;
 };
-
 const layoutNameAbbr = filePath => {
   const prefix = 'L_';
   const dirName = path.dirname(filePath).split('/').pop() || '';
   return `${prefix}${makeLegalIdentifier(dirName)}`;
 };
-
 const parents = [];
-/* eslint-disable no-param-reassign */
 
+/* eslint-disable no-param-reassign */
 const recursiveReadDir = ({
   dir,
   routes,
@@ -36,7 +30,6 @@ const recursiveReadDir = ({
   let resetParent = false;
   let parent = parents[parents.length - 1];
   const layout = findLayout(dir);
-
   if (layout) {
     if (_basePath === '/') {
       throw new Error(`should use _app instead of _layout in ${dir}`);
@@ -58,16 +51,13 @@ const recursiveReadDir = ({
       routes = route.children;
     }
   }
-
   for (const relative of fs.readdirSync(dir)) {
     const filePath = path.join(dir, relative);
-
     if (!shouldSkip(filePath)) {
       const filename = path.basename(filePath, path.extname(filePath));
       const alias = replaceWithAlias(srcDirectory, filePath, srcAlias);
       const componentName = compName(srcDirectory, filePath);
       const dynamicRouteMatched = FILE_SYSTEM_ROUTES_DYNAMIC_REGEXP.exec(filename);
-
       if (dynamicRouteMatched) {
         if (hasDynamicRoute) {
           throw new Error(`Can't set two dynamic route in one directory: ${dir}`);
@@ -75,7 +65,6 @@ const recursiveReadDir = ({
           hasDynamicRoute = true;
         }
       }
-
       const route = {
         path: `${_basePath}${dynamicRouteMatched ? `:${dynamicRouteMatched[1]}${dynamicRouteMatched[2]}` : filename}`,
         _component: alias,
@@ -83,7 +72,6 @@ const recursiveReadDir = ({
         parent,
         type: 'page'
       };
-
       if (fs.statSync(filePath).isDirectory()) {
         recursiveReadDir({
           dir: filePath,
@@ -94,38 +82,32 @@ const recursiveReadDir = ({
         });
         continue;
       }
-
       if (filename === FILE_SYSTEM_ROUTES_LAYOUT) {
         continue;
       }
-
       if (filename === FILE_SYSTEM_ROUTES_INDEX) {
         route.path = _basePath === '/' ? _basePath : `${_basePath.substring(0, _basePath.length - 1)}`;
       }
-
       if (filename === '404' && _basePath === '/') {
         route.path = '*';
       }
-
       routes.push(route);
     }
   }
-
   if (resetParent) {
     parents.pop();
   }
 };
 /* eslint-enable  no-param-reassign */
 
-
 const normalizeNestedRoutes = (nested, internalComponentsDir, internalDirectory, internalDirAlias) => {
   const flat = routes => routes.reduce((memo, route) => memo.concat(Array.isArray(route.children) ? flat(route.children) : [route]), []);
-
   const generate = route => {
     const codes = [];
     let lastComponent = route.component;
-    const imports = [`import React from 'react';`, `import ${lastComponent} from '${route._component}'`]; // eslint-disable-next-line no-param-reassign, no-cond-assign
+    const imports = [`import React from 'react';`, `import ${lastComponent} from '${route._component}'`];
 
+    // eslint-disable-next-line no-param-reassign, no-cond-assign
     while (route = route.parent) {
       const layoutComponent = route.component;
       const layoutComponentAbbr = layoutNameAbbr(route._component);
@@ -134,7 +116,6 @@ const normalizeNestedRoutes = (nested, internalComponentsDir, internalDirectory,
       codes.push(`const ${currentComponent} = props => <${layoutComponent} Component={${lastComponent}} {...props} />;`);
       lastComponent = currentComponent;
     }
-
     const file = path.resolve(internalComponentsDir, `${lastComponent}.jsx`);
     fs.outputFileSync(file, `${imports.join('\n')}\n${codes.join('\n')}\nexport default ${lastComponent}`);
     return {
@@ -142,7 +123,6 @@ const normalizeNestedRoutes = (nested, internalComponentsDir, internalDirectory,
       _component: replaceWithAlias(internalDirectory, file, internalDirAlias)
     };
   };
-
   const normalized = flat(nested).map(route => route.parent ? _objectSpread(_objectSpread(_objectSpread({}, route), generate(route)), {}, {
     parent: undefined
   }) : _objectSpread(_objectSpread({}, route), {}, {
@@ -150,7 +130,6 @@ const normalizeNestedRoutes = (nested, internalComponentsDir, internalDirectory,
   }));
   return normalized;
 };
-
 export const getClientRoutes = ({
   entrypoint,
   srcDirectory,
@@ -162,15 +141,12 @@ export const getClientRoutes = ({
     entry,
     entryName
   } = entrypoint;
-
   if (!fs.existsSync(entry)) {
     throw new Error(`generate file system routes error, ${entry} directory not found.`);
   }
-
   if (!(fs.existsSync(entry) && fs.statSync(entry).isDirectory())) {
     throw new Error(`generate file system routes error, ${entry} should be directory.`);
   }
-
   let routes = [];
   recursiveReadDir({
     dir: entry,
@@ -182,15 +158,14 @@ export const getClientRoutes = ({
   const internalComponentsDir = path.resolve(internalDirectory, `${entryName}/${FILE_SYSTEM_ROUTES_COMPONENTS_DIR}`);
   fs.emptyDirSync(internalComponentsDir);
   routes = normalizeNestedRoutes(routes, internalComponentsDir, internalDirectory, internalDirAlias);
-  parents.length = 0; // FIXME: support more situations
+  parents.length = 0;
 
+  // FIXME: support more situations
   routes.sort((a, b) => {
     const delta = getRouteWeight(a.path) - getRouteWeight(b.path);
-
     if (delta === 0) {
       return a.path.length - b.path.length;
     }
-
     return delta;
   });
   debug(`fileSystem routes: %o`, routes);