From c759d8f4f9ded343eff9034ef31acd68c8b17aa6 Mon Sep 17 00:00:00 2001
From: Bert De Block <bert.deblock@bagaar.be>
Date: Tue, 1 Aug 2023 19:24:11 +0200
Subject: [PATCH] Speed up requiring `index.js` file (#249)

---
 packages/ember-svg-jar/lib/index.js | 38 ++++++++++++++++++++---------
 packages/ember-svg-jar/lib/utils.js |  3 ++-
 2 files changed, 29 insertions(+), 12 deletions(-)

diff --git a/packages/ember-svg-jar/lib/index.js b/packages/ember-svg-jar/lib/index.js
index 257c4641..cc9f6155 100644
--- a/packages/ember-svg-jar/lib/index.js
+++ b/packages/ember-svg-jar/lib/index.js
@@ -2,21 +2,18 @@
 
 const fs = require('fs');
 const _ = require('lodash');
-const Funnel = require('broccoli-funnel');
-const MergeTrees = require('broccoli-merge-trees');
-const SVGOptimizer = require('broccoli-svg-optimizer');
-const broccoliReplace = require('broccoli-string-replace');
-const Symbolizer = require('./symbolizer/symbolizer');
-const InlinePacker = require('./inline-packer');
-const ViewerAssetsBuilder = require('./viewer-assets-builder');
-const ViewerBuilder = require('./viewer-builder');
-const buildOptions = require('./build-options');
 const { makeIDForPath } = require('./utils');
-const prepareSymbolLoaderScript = require('./prepare-symbol-loader-script');
 
 function mergeTreesIfNeeded(trees, options) {
+  if (trees.length === 1) {
+    return trees[0];
+  }
+
   let mergedOptions = _.assign({ overwrite: true }, options);
-  return trees.length === 1 ? trees[0] : new MergeTrees(trees, mergedOptions);
+
+  const MergeTrees = require('broccoli-merge-trees');
+
+  return new MergeTrees(trees, mergedOptions);
 }
 
 module.exports = {
@@ -28,6 +25,9 @@ module.exports = {
 
   included(app) {
     this._super.included.apply(this, arguments);
+
+    const buildOptions = require('./build-options');
+
     this.svgJarOptions = buildOptions(app);
   },
 
@@ -41,6 +41,8 @@ module.exports = {
       // Add viewer assets to public dir
       let svgJarPublicTree = this._super.treeForPublic.apply(this, arguments);
 
+      const broccoliReplace = require('broccoli-string-replace');
+
       svgJarPublicTree = broccoliReplace(svgJarPublicTree, {
         files: ['**/index.html'],
         pattern: {
@@ -78,6 +80,8 @@ module.exports = {
       const outputFile = this.optionFor('symbol', 'outputFile');
       const isTestEnv = app.environment === 'test';
 
+      const prepareSymbolLoaderScript = require('./prepare-symbol-loader-script');
+
       return prepareSymbolLoaderScript(rootURL, outputFile, isTestEnv);
     }
 
@@ -103,6 +107,8 @@ module.exports = {
   originalSvgsFor(strategy) {
     let sourceDirs = this.sourceDirsFor(strategy);
 
+    const Funnel = require('broccoli-funnel');
+
     return new Funnel(mergeTreesIfNeeded(sourceDirs), {
       include: ['**/*.svg'],
     });
@@ -111,6 +117,8 @@ module.exports = {
   optimizedSvgsFor(strategy, originalSvgs) {
     let optimizerConfig = this.optionFor(strategy, 'optimizer');
 
+    const SVGOptimizer = require('broccoli-svg-optimizer');
+
     return new SVGOptimizer(originalSvgs, {
       svgoConfig: _.omit(optimizerConfig, 'svgoModule'),
       svgoModule: optimizerConfig.svgoModule,
@@ -132,6 +140,8 @@ module.exports = {
       let stripPath = this.optionFor(strategy, 'stripPath');
       let prefix = this.optionFor(strategy, 'prefix');
 
+      const ViewerAssetsBuilder = require('./viewer-assets-builder');
+
       return new ViewerAssetsBuilder(this.svgsFor(strategy), {
         strategy,
         validationConfig: this.svgJarOptions.validations,
@@ -143,6 +153,8 @@ module.exports = {
       });
     });
 
+    const ViewerBuilder = require('./viewer-builder');
+
     return new ViewerBuilder(mergeTreesIfNeeded(viewerBuilderTrees), {
       outputFile: 'svg-jar-viewer.json',
     });
@@ -152,6 +164,8 @@ module.exports = {
     let idGen = this.optionFor('inline', 'idGen');
     let stripPath = this.optionFor('inline', 'stripPath');
 
+    const InlinePacker = require('./inline-packer');
+
     return new InlinePacker(this.svgsFor('inline'), {
       makeAssetID(relativePath) {
         return makeIDForPath(relativePath, { idGen, stripPath });
@@ -164,6 +178,8 @@ module.exports = {
     let stripPath = this.optionFor('symbol', 'stripPath');
     let prefix = this.optionFor('symbol', 'prefix');
 
+    const Symbolizer = require('./symbolizer/symbolizer');
+
     return new Symbolizer(this.svgsFor('symbol'), {
       outputFile: this.optionFor('symbol', 'outputFile'),
       svgAttrs: this.optionFor('symbol', 'containerAttrs'),
diff --git a/packages/ember-svg-jar/lib/utils.js b/packages/ember-svg-jar/lib/utils.js
index 14959716..f0268708 100644
--- a/packages/ember-svg-jar/lib/utils.js
+++ b/packages/ember-svg-jar/lib/utils.js
@@ -1,7 +1,6 @@
 'use strict';
 
 const fs = require('fs');
-const cheerio = require('cheerio').default;
 const path = require('path').posix;
 const osPathSep = require('path').sep;
 const _ = require('lodash');
@@ -30,6 +29,8 @@ function makeIDForPath(relativePath, { idGen, stripPath, prefix }) {
 }
 
 function svgDataFor(svgContent) {
+  const cheerio = require('cheerio').default;
+
   let $svg = cheerio.load(svgContent, { xmlMode: true })('svg');
 
   return {