Move CLI-only function to CLI file

webpack-contrib · Jan 13, 2018 · c8f025d · c8f025d
1 parent 001bb86
commit c8f025d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 10 deletions.
diff --git a/packages/plugin/src/analyzer.js b/packages/plugin/src/analyzer.js
@@ -1,4 +1,3 @@
-const fs = require('fs');
 const path = require('path');
 
 const _ = require('lodash');
@@ -11,8 +10,7 @@ const FILENAME_QUERY_REGEXP = /\?.*$/;
 const MULTI_MODULE_REGEXP = /^multi /;
 
 module.exports = {
-  getChartData,
-  readStatsFromFile
+  getChartData
 };
 
 function getViewerData(bundleStats, bundleDir, opts) {
@@ -105,12 +103,6 @@ function getViewerData(bundleStats, bundleDir, opts) {
   }, []);
 }
 
-function readStatsFromFile(filename) {
-  return JSON.parse(
-    fs.readFileSync(filename, 'utf8')
-  );
-}
-
 function assetHasModule(statAsset, statModule) {
   return _.some(statModule.chunks, moduleChunk =>
     _.includes(statAsset.chunks, moduleChunk)

diff --git a/packages/plugin/src/bin/analyzer.js b/packages/plugin/src/bin/analyzer.js
@@ -1,6 +1,7 @@
 #! /usr/bin/env node
 
 const { resolve, dirname } = require('path');
+const fs = require('fs');
 
 const _ = require('lodash');
 const commander = require('commander');
@@ -74,7 +75,7 @@ if (!bundleDir) bundleDir = dirname(bundleStatsFile);
 
 let bundleStats;
 try {
-  bundleStats = analyzer.readStatsFromFile(bundleStatsFile);
+  bundleStats = readStatsFromFile(bundleStatsFile);
 } catch (err) {
   console.error(`Could't read webpack bundle stats from "${bundleStatsFile}":\n${err}`);
   process.exit(1);
@@ -106,3 +107,9 @@ function showHelp(error) {
 function br(str) {
   return `\n${_.repeat(' ', 28)}${str}`;
 }
+
+function readStatsFromFile(filename) {
+  return JSON.parse(
+    fs.readFileSync(filename, 'utf8')
+  );
+}