Reduce the usage of lodash

client/components/ModuleItem.jsx

@@ -73,7 +73,8 @@ export default class ModuleItem extends PureComponent {
   }
 
   get invisibleHint() {
-    return `${_.upperFirst(this.itemType)} is not rendered in the treemap because it's too small.`;
+    const itemType = this.itemType.charAt(0).toUpperCase() + this.itemType.slice(1);
+    return `${itemType} is not rendered in the treemap because it's too small.`;
   }
 
   get isVisible() {

src/analyzer.js

@@ -81,7 +81,10 @@ function getViewerData(bundleStats, bundleDir, opts) {
         continue;
       }
 
-      bundlesSources[statAsset.name] = _.pick(bundleInfo, 'src', 'runtimeSrc');
+      bundlesSources[statAsset.name] = {
+        src: bundleInfo.src,
+        runtimeSrc: bundleInfo.runtimeSrc
+      };
       Object.assign(parsedModules, bundleInfo.modules);
     }
 
@@ -96,8 +99,10 @@ function getViewerData(bundleStats, bundleDir, opts) {
     // If asset is a childAsset, then calculate appropriate bundle modules by looking through stats.children
     const assetBundles = statAsset.isChild ? getChildAssetBundles(bundleStats, statAsset.name) : bundleStats;
     const modules = assetBundles ? getBundleModules(assetBundles) : [];
-    const asset = result[statAsset.name] = _.pick(statAsset, 'size');
-    const assetSources = bundlesSources && _.has(bundlesSources, statAsset.name) ?
+    const asset = result[statAsset.name] = {
+      size: statAsset.size
+    };
+    const assetSources = bundlesSources && Object.prototype.hasOwnProperty.call(bundlesSources, statAsset.name) ?
       bundlesSources[statAsset.name] : null;
 
     if (assetSources) {
@@ -157,7 +162,7 @@ function getViewerData(bundleStats, bundleDir, opts) {
     statSize: asset.tree.size || asset.size,
     parsedSize: asset.parsedSize,
     gzipSize: asset.gzipSize,
-    groups: _.invokeMap(asset.tree.children, 'toChartData')
+    groups: asset.tree.children.map(i => i.toChartData())
   }));
 }
 

src/parseUtils.js

@@ -1,5 +1,4 @@
 const fs = require('fs');
-const _ = require('lodash');
 const acorn = require('acorn');
 const walk = require('acorn-walk');
 
@@ -143,9 +142,7 @@ function parseBundle(bundlePath) {
   let modules;
 
   if (walkState.locations) {
-    modules = _.mapValues(walkState.locations,
-      loc => content.slice(loc.start, loc.end)
-    );
+    modules = walkState.locations.map(loc => content.slice(loc.start, loc.end));
   } else {
     modules = {};
   }

src/tree/BaseFolder.js

@@ -1,5 +1,3 @@
-import _ from 'lodash';
-
 import Node from './Node';
 
 export default class BaseFolder extends Node {
@@ -10,15 +8,15 @@ export default class BaseFolder extends Node {
   }
 
   get src() {
-    if (!_.has(this, '_src')) {
+    if (!Object.prototype.hasOwnProperty.call(this, '_src')) {
       this._src = this.walk((node, src) => (src += node.src || ''), '', false);
     }
 
     return this._src;
   }
 
   get size() {
-    if (!_.has(this, '_size')) {
+    if (!Object.prototype.hasOwnProperty.call(this, '_size')) {
       this._size = this.walk((node, size) => (size + node.size), 0, false);
     }
 
@@ -111,7 +109,7 @@ export default class BaseFolder extends Node {
       label: this.name,
       path: this.path,
       statSize: this.size,
-      groups: _.invokeMap(this.children, 'toChartData')
+      groups: this.children.map(i => i.toChartData())
     };
   }
 

src/tree/ConcatenatedModule.js

@@ -58,14 +58,14 @@ export default class ConcatenatedModule extends Module {
   }
 
   mergeNestedFolders() {
-    _.invokeMap(this.children, 'mergeNestedFolders');
+    this.children.forEach(child => child.mergeNestedFolders());
   }
 
   toChartData() {
     return {
       ...super.toChartData(),
       concatenated: true,
-      groups: _.invokeMap(this.children, 'toChartData')
+      groups: this.children.map(i => i.toChartData())
     };
   }
 

src/tree/Folder.js

@@ -13,7 +13,7 @@ export default class Folder extends BaseFolder {
   }
 
   get gzipSize() {
-    if (!_.has(this, '_gzipSize')) {
+    if (!Object.prototype.hasOwnProperty.call(this, '_gzipSize')) {
       this._gzipSize = this.src ? gzipSize.sync(this.src) : 0;
     }
 

src/tree/Module.js

@@ -1,4 +1,3 @@
-import _ from 'lodash';
 import gzipSize from 'gzip-size';
 
 import Node from './Node';
@@ -32,7 +31,7 @@ export default class Module extends Node {
   }
 
   get gzipSize() {
-    if (!_.has(this, '_gzipSize')) {
+    if (!Object.prototype.hasOwnProperty.call(this, '_gzipSize')) {
       this._gzipSize = this.src ? gzipSize.sync(this.src) : undefined;
     }
 

test/analyzer.js

@@ -1,7 +1,6 @@
 const chai = require('chai');
 chai.use(require('chai-subset'));
 const {expect} = chai;
-const _ = require('lodash');
 const fs = require('fs');
 const path = require('path');
 const del = require('del');
@@ -107,7 +106,7 @@ describe('Analyzer', function () {
   it('should gracefully parse invalid chunks', async function () {
     generateReportFrom('with-invalid-chunk/stats.json');
     const chartData = await getChartData();
-    const invalidChunk = _.find(chartData, {label: 'invalid-chunk.js'});
+    const invalidChunk = chartData.find(i => i.label === 'invalid-chunk.js');
     expect(invalidChunk.groups).to.containSubset([
       {
         id: 1,
@@ -123,14 +122,14 @@ describe('Analyzer', function () {
   it('should gracefully process missing chunks', async function () {
     generateReportFrom('with-missing-chunk/stats.json');
     const chartData = await getChartData();
-    const invalidChunk = _.find(chartData, {label: 'invalid-chunk.js'});
+    const invalidChunk = chartData.find(i => i.label === 'invalid-chunk.js');
     expect(invalidChunk).to.exist;
     expect(invalidChunk.statSize).to.equal(24);
     forEachChartItem([invalidChunk], item => {
       expect(typeof item.statSize).to.equal('number');
       expect(item.parsedSize).to.be.undefined;
     });
-    const validChunk = _.find(chartData, {label: 'valid-chunk.js'});
+    const validChunk = chartData.find(i => i.label === 'valid-chunk.js');
     forEachChartItem([validChunk], item => {
       expect(typeof item.statSize).to.equal('number');
       expect(typeof item.parsedSize).to.equal('number');
@@ -140,14 +139,14 @@ describe('Analyzer', function () {
   it('should gracefully process missing chunks', async function () {
     generateReportFrom('with-missing-module-chunks/stats.json');
     const chartData = await getChartData();
-    const invalidChunk = _.find(chartData, {label: 'invalid-chunk.js'});
+    const invalidChunk = chartData.find(i => i.label === 'invalid-chunk.js');
     expect(invalidChunk).to.exist;
     expect(invalidChunk.statSize).to.equal(568);
     forEachChartItem([invalidChunk], item => {
       expect(typeof item.statSize).to.equal('number');
       expect(item.parsedSize).to.be.undefined;
     });
-    const validChunk = _.find(chartData, {label: 'valid-chunk.js'});
+    const validChunk = chartData.find(i => i.label === 'valid-chunk.js');
     forEachChartItem([validChunk], item => {
       expect(typeof item.statSize).to.equal('number');
       expect(typeof item.parsedSize).to.equal('number');

test/helpers.js

@@ -17,7 +17,7 @@ const getAvailableWebpackVersions = _.memoize(() =>
 function forEachWebpackVersion(versions, cb) {
   const availableVersions = getAvailableWebpackVersions();
 
-  if (_.isFunction(versions)) {
+  if (typeof versions === 'function') {
     cb = versions;
     versions = availableVersions;
   } else {

test/plugin.js

@@ -3,7 +3,6 @@ chai.use(require('chai-subset'));
 const {expect} = chai;
 const fs = require('fs');
 const del = require('del');
-const _ = require('lodash');
 const path = require('path');
 const puppeteer = require('puppeteer');
 const BundleAnalyzerPlugin = require('../lib/BundleAnalyzerPlugin');
@@ -113,7 +112,7 @@ describe('Plugin', function () {
           await webpackCompile(config);
 
           const chartData = await getChartDataFromReport();
-          expect(_.map(chartData, 'label'))
+          expect(chartData.map(i => i.label))
             .to
             .deep
             .equal(['bundle.js']);