refactor(cli): 优化 watch 时的编译效率

Author: deepfunc

.editorconfig

@@ -0,0 +1,9 @@
+root = true
+
+[*]
+charset = utf-8
+indent_style = space
+indent_size = 2
+end_of_line = lf
+insert_final_newline = true
+trim_trailing_whitespace = true

packages/cli/core/compile.js

@@ -168,64 +168,85 @@ class Compile extends Hook {
     return initCompiler(this, this.options.compilers);
   }
 
-  start () {
+  start (files) {
     if (this.running) {
       return;
     }
+
     this.running = true;
+    this.logger.info('build', 'start...');
+    let parseComps;
+
+    if (files === undefined || files.length === 0) {
+      // if files is empty, compile from entry
+      parseComps = this.hookUnique('wepy-parser-wpy',
+        { path: this.options.entry, type: 'app' }).then(app => {
+        let sfc = app.sfc;
+        let script = sfc.script;
+        let styles = sfc.styles;
+        let config = sfc.config;
+
+        let appConfig = config.parsed.output;
+        if (!appConfig.pages || appConfig.pages.length === 0) {
+          appConfig.pages = [];
+          this.hookUnique('error-handler', {
+            type: 'warn',
+            ctx: app,
+            message: `Missing "pages" in App config`
+          });
+        }
+        let pages = appConfig.pages.map(v => {
+          return path.resolve(app.file, '..', v);
+        });
 
-    this.hookUnique('wepy-parser-wpy', { path: this.options.entry, type: 'app' }).then(app => {
+        if (appConfig.subPackages || appConfig.subpackages) {
+          (appConfig.subpackages || appConfig.subPackages).forEach(sub => {
+            sub.pages.forEach(v => {
+              pages.push(path.resolve(app.file, '../'+sub.root || '', v));
+            });
 
-      let sfc = app.sfc;
-      let script = sfc.script;
-      let styles = sfc.styles;
-      let config = sfc.config;
+          });
+        }
 
-      let appConfig = config.parsed.output;
-      if (!appConfig.pages || appConfig.pages.length === 0) {
-        appConfig.pages = [];
-        this.hookUnique('error-handler', {
-          type: 'warn',
-          ctx: app,
-          message: `Missing "pages" in App config`
-        });
-      }
-      let pages = appConfig.pages.map(v => {
-        return path.resolve(app.file, '..', v);
-      });
+        let tasks = pages.map(v => {
+          let file;
 
-      if (appConfig.subPackages || appConfig.subpackages) {
-        (appConfig.subpackages || appConfig.subPackages).forEach(sub => {
-           sub.pages.forEach(v => {
-            pages.push(path.resolve(app.file, '../'+sub.root || '', v));
+          file = v + this.options.wpyExt;
+          if (fs.existsSync(file)) {
+            return this.hookUnique('wepy-parser-wpy', { path: file, type: 'page' });
+          }
+          file = v + '.js';
+          if (fs.existsSync(file)) {
+            return this.hookUnique('wepy-parser-component', { path: file, type: 'page', npm: false });
+          }
+          this.hookUnique('error-handler', {
+            type: 'error',
+            ctx: app,
+            message: `Can not resolve page: ${v}`
           });
-
         });
-      }
-
-      let tasks = pages.map(v => {
-        let file;
 
-        file = v + this.options.wpyExt;
+        this.hookSeq('build-app', app);
+        this.hookUnique('output-app', app);
+        return Promise.all(tasks);
+      })
+    } else {
+      // else just compile these
+      const tasks = files.map(file => {
         if (fs.existsSync(file)) {
           return this.hookUnique('wepy-parser-wpy', { path: file, type: 'page' });
         }
-        file = v + '.js';
-        if (fs.existsSync(file)) {
-          return this.hookUnique('wepy-parser-component', { path: file, type: 'page', npm: false });
-        }
         this.hookUnique('error-handler', {
           type: 'error',
-          ctx: app,
-          message: `Can not resolve page: ${v}`
+          ctx: {},
+          message: `Can not resolve page: ${file}`
         });
       });
 
-      this.hookSeq('build-app', app);
-      this.hookUnique('output-app', app);
-      return Promise.all(tasks);
-    }).then(comps => {
+      parseComps = Promise.all(tasks);
+    }
 
+    parseComps.then(comps => {
       function buildComponents (comps) {
         if (!comps) {
           return null;
@@ -257,6 +278,7 @@ class Compile extends Hook {
           return null;
         }
       }
+
       return buildComponents.bind(this)(comps);
     }).then(() => {
       let vendorData = this.hookSeq('build-vendor', {});
@@ -269,7 +291,7 @@ class Compile extends Hook {
     }).then(() => {
       this.hookSeq('process-done');
       this.running = false;
-      this.logger.info('process finished');
+      this.logger.info('build', 'finished');
       if (this.options.watch) {
         this.logger.info('watching...');
         this.watch();
@@ -320,7 +342,13 @@ class Compile extends Hook {
         }
         if (involvedFile) {
           this.logger.silly('watch', `Watcher triggered by file changes: ${absolutePath}`);
-          this.start();
+          if (absolutePath !== this.options.entry &&
+            path.extname(absolutePath) === this.options.wpyExt) {
+            // if changed file is wpyExt and is not entry, just compile it
+            this.start([absolutePath]);
+          } else {
+            this.start();
+          }
         }
       }
     })
@@ -432,4 +460,4 @@ exports = module.exports = (program) => {
   let opt = parseOptions.convert(program);
 
   return new Compile(opt);
-}
+};