fix: refactor and memory improvements

Author: pi0

README.md

@@ -96,11 +96,6 @@ Display color (can be HEX like `#xxyyzz` or a web color like `green`).
 
 Enable profiler.
 
-### `stream`
-  - Default: `process.stderr`
-
-Output stream.
-
 ### `bars`
 
   -  Default: `true` when not in CI or testing mode.

src/plugin.js

@@ -15,7 +15,6 @@ const DEFAULTS = {
   name: 'webpack',
   color: 'green',
   profile: false,
-  stream: process.stdout,
   reporters: [],
   reporter: null,
   simple: isMinimal,
@@ -29,7 +28,6 @@ const DEFAULT_STATE = {
   message: '',
   details: [],
   request: null,
-  stats: null,
   hasErrors: false,
 };
 
@@ -129,14 +127,13 @@ export default class WebpackBarPlugin extends webpack.ProgressPlugin {
 
       Object.assign(this.state, {
         ...DEFAULT_STATE,
-        stats,
         progress: 100,
         message: `Compiled ${status} in ${time}`,
         hasErrors,
       });
 
       this.callReporters('progress');
-      this.callReporters('done');
+      this.callReporters('done', { stats });
 
       if (!this.hasRunning) {
         this.callReporters('beforeAllDone');

src/reporters/fancy.js

@@ -7,10 +7,10 @@ import { formatRequest } from '../utils/request';
 import { BULLET, TICK, CROSS } from '../utils/consts';
 import LogUpdate from '../utils/log-update';
 
-let lastRender = Date.now();
-
 const logUpdate = new LogUpdate();
 
+let lastRender = Date.now();
+
 export default class FancyReporter {
   beforeRun() {
     consola.pause();
@@ -37,7 +37,6 @@ export default class FancyReporter {
     const renderedStates = Object.keys(states)
       .sort((n1, n2) => n1.localeCompare(n2))
       .map((name) => ({ name, state: states[name] }))
-      .filter((c) => c.state.progress >= 0)
       .map((c) => this._renderState(c))
       .join('\n\n');
 
@@ -68,13 +67,19 @@ export default class FancyReporter {
             ellipsisLeft(formatRequest(state.request), logUpdate.columns)
           )
         : '';
-    } else if (state.progress === 100) {
-      // Finished
-      line1 = color(`${state.hasErrors ? CROSS : TICK} ${name}`);
-      line2 = chalk.grey('  ' + state.message);
     } else {
-      // Invalid state
-      return '';
+      let icon = ' ';
+
+      if (state.hasErrors) {
+        icon = CROSS;
+      } else if (state.progress === 100) {
+        icon = TICK;
+      } else if (this.progress === -1) {
+        icon = BULLET;
+      }
+
+      line1 = color(`${icon} ${name}`);
+      line2 = chalk.grey('  ' + state.message);
     }
 
     return line1 + '\n' + line2;

src/reporters/profile.js

@@ -2,7 +2,7 @@ export default class ProfileReporter {
   done(context) {
     if (context.options.profile) {
       const formattedStats = context.state.profile.getFormattedStats();
-      context.options.stream.write(`\n${formattedStats}\n`);
+      process.stderr.write(`\n${formattedStats}\n`);
     }
   }
 }