@eui/tools 6.14.10 → 6.14.11

package/.version.properties

@@ -1 +1 @@
-6.14.10
+6.14.11

package/CHANGELOG.md

@@ -1,3 +1,12 @@
+## 6.14.11 (2023-12-08)
+
+##### Chores
+
+* **other:**
+  * improve styles audit for non-SoC component style used - EUI-7121 [EUI-7121](https://webgate.ec.europa.eu/CITnet/jira/browse/EUI-7121)  ([41cee1b9](https://webgate.ec.europa.eu/CITnet/stash/scm/csdr/eui-tools.git/commits/41cee1b96c507872f08ebca4f002d646869566c0))
+
+* * *
+* * *
 ## 6.14.10 (2023-12-06)
 
 ##### Chores

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@eui/tools",
-    "version": "6.14.10",
+    "version": "6.14.11",
     "tag": "latest",
     "license": "EUPL-1.1",
     "description": "eUI common tools and scripts",

package/scripts/csdr/audit/styles.js

@@ -73,10 +73,10 @@ const runStylesAudit = (module.exports.runStylesAudit = (pkg, customPath) => {
     srcPath = customPath;
     tools.logTitle(`Auditing : ${customPath}`);
   } else {
-    if (pkg.build && pkg.build.srcRootDefault === false) {
-      srcPath = pkg.paths.root;
-    } else {
+    if (pkg.srcRootDefault) {
       srcPath = pkg.paths.src;
+    } else {
+      srcPath = pkg.paths.root;
     }
     tools.logTitle(`Auditing : ${pkg.name}`);
   }
@@ -98,6 +98,12 @@ const runStylesAudit = (module.exports.runStylesAudit = (pkg, customPath) => {
       count: 0,
       files: [],
     },
+    tsNotSoC: {
+      count: 0,
+      files: [],
+      templateCount: 0,
+      styleCount: 0,
+    },
     thirdPartyDependencies: {
       count: 0,
       dependencies: [],
@@ -152,11 +158,11 @@ const runStylesAudit = (module.exports.runStylesAudit = (pkg, customPath) => {
     },
     uxCmp: {
       count: 0,
-      indices: []
+      indices: [],
     },
     mywpSharedAll: {
       count: 0,
-      indices: []
+      indices: [],
     },
     finalReport: {},
   };
@@ -167,13 +173,113 @@ const runStylesAudit = (module.exports.runStylesAudit = (pkg, customPath) => {
     report.package = pkg.name;
   }
 
+
   let files;
+
+  // CHECKING TS FILES CONTENT
+  // -------------------------
+
+  files = glob.sync('**/*.ts', { cwd: srcPath, nodir: true, follow: true, dot: true });
+
+  files.forEach((file) => {
+    let result, regex;
+
+    const filePath = path.join(srcPath, file);
+    const fileContent = tools.getFileContent(filePath);
+    const linesCount = fileContent.split('\n').length;
+
+    if (fileContent.indexOf('styles-audit-disabled') !== -1) {
+      report.stylesAuditDisabledCount++;
+      return;
+    }
+
+    if (linesCount > 500) {
+      report.tsOver500.count++;
+      report.tsOver500.files.push(file);
+    }
+
+    let tsNotSoCTemplateIndices = [];
+    regex = /template:/gi;
+    while ((result = regex.exec(fileContent))) {
+      tsNotSoCTemplateIndices.push(result.index);
+    }
+    report.tsNotSoC.count += tsNotSoCTemplateIndices.length;
+    report.tsNotSoC.templateCount += tsNotSoCTemplateIndices.length;
+    if (tsNotSoCTemplateIndices.length > 0) {
+      report.tsNotSoC.files.push(file);
+    }
+
+    let tsNotSoCStyleIndices = [];
+    regex = /styles:/gi;
+    while ((result = regex.exec(fileContent))) {
+      tsNotSoCStyleIndices.push(result.index);
+    }
+    report.tsNotSoC.count += tsNotSoCStyleIndices.length;
+    report.tsNotSoC.styleCount += tsNotSoCStyleIndices.length;
+    if (tsNotSoCStyleIndices.length > 0) {
+      report.tsNotSoC.files.push(file);
+    }
+
+    report.tsNotSoC.files = tools.removeArrayDuplicates(report.tsNotSoC.files);
+
+
+    let primengIndices = [];
+    regex = /primeng/gi;
+    while ((result = regex.exec(fileContent))) {
+      primengIndices.push(result.index);
+    }
+    report.primeng.count += primengIndices.length;
+    report.primeng.indices = [...report.primeng.indices, ...generateIndicesContent(primengIndices, file, fileContent)];
+
+    let materialIndices = [];
+    regex = /@angular\/material/gi;
+    while ((result = regex.exec(fileContent))) {
+      materialIndices.push(result.index);
+    }
+    const indicesContent = generateIndicesContent(materialIndices, file, fileContent, [
+      '@angular/material/core',
+      '@angular/material-moment-adapter',
+    ]);
+
+    report.material.count += recalculateIndicesCount(indicesContent);
+    report.material.indices = [...report.material.indices, ...indicesContent];
+
+    let allModulesIndices = [];
+    regex = /EuiAllModule/gi;
+    while ((result = regex.exec(fileContent))) {
+      allModulesIndices.push(result.index);
+    }
+    regex = /UxAllModule/gi;
+    while ((result = regex.exec(fileContent))) {
+      allModulesIndices.push(result.index);
+    }
+    report.allModules.count += allModulesIndices.length;
+    report.allModules.indices = [...report.allModules.indices, ...generateIndicesContent(allModulesIndices, file, fileContent)];
+
+    let mywpSharedAllIndices = [];
+    regex = /\@mywp\/shared'|\@mywp\/shared"/gi;
+    while ((result = regex.exec(fileContent))) {
+      mywpSharedAllIndices.push(result.index);
+    }
+    report.mywpSharedAll.count += mywpSharedAllIndices.length;
+    report.mywpSharedAll.indices = [...report.mywpSharedAll.indices, ...generateIndicesContent(mywpSharedAllIndices, file, fileContent)];
+  });
+
+
+  // CHECKING STYLES FILE CONTENT
+  // ----------------------------
+
   const scssFiles = glob.sync('**/*.scss', { cwd: srcPath, nodir: true, follow: true, dot: true });
   const cssFiles = glob.sync('**/*.css', { cwd: srcPath, nodir: true, follow: true, dot: true });
 
-  files = [...scssFiles, ...cssFiles];
+  let tsFiles = [];
+  if (report.tsNotSoC.styleCount > 0) {
+    tsFiles = glob.sync('**/*.ts', { cwd: srcPath, nodir: true, follow: true, dot: true });
+  }
+
+  files = [...scssFiles, ...cssFiles, ...tsFiles];
 
-  report.scssFilesCount += files.length;
+  report.scssFilesCount += [...scssFiles, ...cssFiles].length;
 
   files.forEach((file) => {
     const filePath = path.join(srcPath, file);
@@ -188,16 +294,21 @@ const runStylesAudit = (module.exports.runStylesAudit = (pkg, customPath) => {
     if (fileContent === '') {
       report.scssEmptyFiles++;
     }
-    report.scssLinescount += linesCount;
+    if (file.indexOf('.scss') > -1 || file.indexOf('.css') > -1) {
+      report.scssLinescount += linesCount;
+    }
 
-    let result,
-      euiIndices = [];
-    let regex = /\.eui|\.ux/gi;
-    while ((result = regex.exec(fileContent))) {
-      euiIndices.push(result.index);
+    let result, regex;
+
+    if (file.indexOf('.scss') > -1 || file.indexOf('.css') > -1) {
+      let euiIndices = [];
+      regex = /\.eui|\.ux/gi;
+      while ((result = regex.exec(fileContent))) {
+        euiIndices.push(result.index);
+      }
+      report.euiOverrides.count += euiIndices.length;
+      report.euiOverrides.indices = [...report.euiOverrides.indices, ...generateIndicesContent(euiIndices, file, fileContent)];
     }
-    report.euiOverrides.count += euiIndices.length;
-    report.euiOverrides.indices = [...report.euiOverrides.indices, ...generateIndicesContent(euiIndices, file, fileContent)];
 
     let sizesIndices = [];
     regex = /px/gi;
@@ -256,9 +367,20 @@ const runStylesAudit = (module.exports.runStylesAudit = (pkg, customPath) => {
     report.zIndex.indices = [...report.zIndex.indices, ...generateIndicesContent(zIndexIndices, file, fileContent)];
   });
 
-  files = glob.sync('**/*.html', { cwd: srcPath, nodir: true, follow: true, dot: true });
 
-  report.htmlFilesCount += files.length;
+  // CHECKING HTML FILES CONTENT
+  // ---------------------------
+
+  const htmlFiles = glob.sync('**/*.html', { cwd: srcPath, nodir: true, follow: true, dot: true });
+
+  tsFiles = [];
+  if (report.tsNotSoC.templateCount.length > 0) {
+    tsFiles = glob.sync('**/*.ts', { cwd: srcPath, nodir: true, follow: true, dot: true });
+  }
+
+  files = [...htmlFiles, ...tsFiles];
+
+  report.htmlFilesCount += htmlFiles.length;
 
   files.forEach((file) => {
     const filePath = path.join(srcPath, file);
@@ -270,11 +392,13 @@ const runStylesAudit = (module.exports.runStylesAudit = (pkg, customPath) => {
       return;
     }
 
-    report.htmlLinesCount += linesCount;
+    if (file.indexOf('.html') > -1) {
+      report.htmlLinesCount += linesCount;
+    }
 
     let result,
       inlineStylesIndices = [];
-    let regex = /style=/gi;
+    let regex = /style=|style =/gi;
     while ((result = regex.exec(fileContent))) {
       inlineStylesIndices.push(result.index);
     }
@@ -282,77 +406,17 @@ const runStylesAudit = (module.exports.runStylesAudit = (pkg, customPath) => {
     report.inlineStyles.indices = [...report.inlineStyles.indices, ...generateIndicesContent(inlineStylesIndices, file, fileContent)];
 
     let uxCmpIndices = [];
-    regex = /ux-/gi;
+    regex = /<ux-/gi;
     while ((result = regex.exec(fileContent))) {
       uxCmpIndices.push(result.index);
     }
-    const indicesContent = generateIndicesContent(uxCmpIndices, file, fileContent, [
-      'ux-dyn-forms',
-    ]);
+    const indicesContent = generateIndicesContent(uxCmpIndices, file, fileContent);
 
     report.uxCmp.count += recalculateIndicesCount(indicesContent);
     report.uxCmp.indices = [...report.uxCmp.indices, ...indicesContent];
   });
 
-  files = glob.sync('**/*.ts', { cwd: srcPath, nodir: true, follow: true, dot: true });
-
-  files.forEach((file) => {
-    const filePath = path.join(srcPath, file);
-    const fileContent = tools.getFileContent(filePath);
-    const linesCount = fileContent.split('\n').length;
-
-    if (fileContent.indexOf('styles-audit-disabled') !== -1) {
-      report.stylesAuditDisabledCount++;
-      return;
-    }
-
-    if (linesCount > 500) {
-      report.tsOver500.count++;
-      report.tsOver500.files.push(file);
-    }
-
-    let result,
-      primengIndices = [];
-    let regex = /primeng/gi;
-    while ((result = regex.exec(fileContent))) {
-      primengIndices.push(result.index);
-    }
-    report.primeng.count += primengIndices.length;
-    report.primeng.indices = [...report.primeng.indices, ...generateIndicesContent(primengIndices, file, fileContent)];
-
-    let materialIndices = [];
-    regex = /@angular\/material/gi;
-    while ((result = regex.exec(fileContent))) {
-      materialIndices.push(result.index);
-    }
-    const indicesContent = generateIndicesContent(materialIndices, file, fileContent, [
-      '@angular/material/core',
-      '@angular/material-moment-adapter',
-    ]);
-
-    report.material.count += recalculateIndicesCount(indicesContent);
-    report.material.indices = [...report.material.indices, ...indicesContent];
-
-    let allModulesIndices = [];
-    regex = /EuiAllModule/gi;
-    while ((result = regex.exec(fileContent))) {
-      allModulesIndices.push(result.index);
-    }
-    regex = /UxAllModule/gi;
-    while ((result = regex.exec(fileContent))) {
-      allModulesIndices.push(result.index);
-    }
-    report.allModules.count += allModulesIndices.length;
-    report.allModules.indices = [...report.allModules.indices, ...generateIndicesContent(allModulesIndices, file, fileContent)];
 
-    let mywpSharedAllIndices = [];
-    regex = /\@mywp\/shared'|\@mywp\/shared"/gi;
-    while ((result = regex.exec(fileContent))) {
-      mywpSharedAllIndices.push(result.index);
-    }
-    report.mywpSharedAll.count += mywpSharedAllIndices.length;
-    report.mywpSharedAll.indices = [...report.mywpSharedAll.indices, ...generateIndicesContent(mywpSharedAllIndices, file, fileContent)];
-  });
 
   // Analyzing 3rd party dependencies usage
 
@@ -371,14 +435,13 @@ const runStylesAudit = (module.exports.runStylesAudit = (pkg, customPath) => {
 
     // checking against allowed CSDR known package scopes
     if (!allowedScopes.includes(scope)) {
-
       // exclusions known
       let exclusionFound = false;
       exclusions.forEach((exc) => {
         if (scope.indexOf(exc) > -1) {
           exclusionFound = true;
         }
-      })
+      });
 
       if (!exclusionFound) {
         thirdPartyDeps.push(d);
@@ -431,18 +494,18 @@ const runStylesAudit = (module.exports.runStylesAudit = (pkg, customPath) => {
     };
   }
 
-  if (report.stylesAuditDisabledCount !==0) {
+  if (report.stylesAuditDisabledCount !== 0) {
     report.finalReport['stylesAuditDisabledCount'] = {
       description: 'audit disabled count',
       value: report.stylesAuditDisabledCount,
-      status: report.stylesAuditDisabledCount !== 0 ? 'IMPROVE': 'OK',
-      comment : report.stylesAuditDisabledCount !==0 ? 'If not done yet, files to be checked by eUI team': null,
-    }
+      status: report.stylesAuditDisabledCount !== 0 ? 'IMPROVE' : 'OK',
+      comment: report.stylesAuditDisabledCount !== 0 ? 'If not done yet, files to be checked by eUI team' : null,
+    };
   }
 
   // BALANCED - styles content
 
-  if (report.scssFilesCount !== 0) {
+  if (report.scssFilesCount !== 0 || report.tsNotSoC.styleCount !== 0) {
     report.finalReport['euiOverrides'] = {
       description: 'eUI overrides found',
       value: report.euiOverrides.count,
@@ -501,7 +564,7 @@ const runStylesAudit = (module.exports.runStylesAudit = (pkg, customPath) => {
 
   // CRITICAL ONLY
 
-  if (report.htmlLinesCount !== 0) {
+  if (report.htmlLinesCount !== 0 || report.tsNotSoC.templateCount !== 0) {
     report.finalReport['inlineStyles'] = {
       description: 'inline styles found in HTML',
       value: report.inlineStyles.count,
@@ -551,16 +614,25 @@ const runStylesAudit = (module.exports.runStylesAudit = (pkg, customPath) => {
       status: report.tsOver500.count !== 0 ? 'IMPROVE' : 'OK',
       comment: report.tsOver500.count !== 0 ? 'Split TS files into multiple ones for readability' : null,
     };
+
+    report.finalReport['tsNotSoC'] = {
+      description: 'TS file not respecting SoC template OR scss split content',
+      value: report.tsNotSoC.count,
+      status: report.tsNotSoC.count !== 0 ? 'IMPROVE' : 'OK',
+      comment: report.tsNotSoC.count !== 0 ? 'Split TS file into types of concerns' : null,
+    };
   }
 
   report.finalReport['thirdPartyDependencies'] = {
     description: 'Third party dependencies usage',
     value: report.thirdPartyDependencies.count,
     status: report.thirdPartyDependencies.count !== 0 ? 'IMPROVE' : 'OK',
-    comment: report.thirdPartyDependencies.count !== 0 ? 'Use 3rd party dependencies only if really necessary (bundle size increase) - contact eUI team for more info' : null,
+    comment:
+      report.thirdPartyDependencies.count !== 0
+        ? 'Use 3rd party dependencies only if really necessary (bundle size increase) - contact eUI team for more info'
+        : null,
   };
 
-
   if (Object.keys(report.finalReport).length === 0) {
     tools.logInfo('No sources to audit...skipping');
     return null;

package/scripts/csdr/config/packages.js

@@ -9,10 +9,9 @@ const tools = require('../../utils/tools');
 // INNER MODULES
 const innerGlobal = require('./global');
 
-
 module.exports.getCsdrPackages = () => {
   return innerGlobal.getCsdrConfig(false).packages;
-}
+};
 
 module.exports.getCsdrPackagesFull = (flattenChildren = true) => {
   const packages = innerGlobal.getCsdrConfig(flattenChildren).packages;
@@ -26,11 +25,11 @@ module.exports.getCsdrPackagesFull = (flattenChildren = true) => {
       // tools.logInfo(`Package: ${p} is in local config but not found in global... should be cleaned up`);
       // console.log(e);
     }
-  })
+  });
   return outputPackages;
-}
+};
 
-const getPackages = module.exports.getPackages = () => {
+const getPackages = (module.exports.getPackages = () => {
   const packages = innerGlobal.getConfig().packages;
   const packagesKeys = Object.keys(packages);
 
@@ -42,10 +41,9 @@ const getPackages = module.exports.getPackages = () => {
       // tools.logInfo(`Package: ${p} is in local config but not found in global... should be cleaned up`);
       // console.log(e);
     }
-  })
+  });
   return outputPackages;
-}
-
+});
 
 module.exports.getPackageByNpmPkg = (npmPkg, fromCsdrConfig) => {
   let packages;
@@ -54,7 +52,7 @@ module.exports.getPackageByNpmPkg = (npmPkg, fromCsdrConfig) => {
   if (fromCsdrConfig) {
     packages = innerGlobal.getCsdrConfig().packages;
 
-  // get only packages installed config based on super .euirc.json file generated
+    // get only packages installed config based on super .euirc.json file generated
   } else {
     packages = innerGlobal.getConfig().packages;
   }
@@ -69,7 +67,7 @@ module.exports.getPackageByNpmPkg = (npmPkg, fromCsdrConfig) => {
   }
 
   return;
-}
+};
 
 module.exports.getPackageByRepository = (repository, fromCsdrConfig) => {
   let packages;
@@ -78,7 +76,7 @@ module.exports.getPackageByRepository = (repository, fromCsdrConfig) => {
   if (fromCsdrConfig) {
     packages = innerGlobal.getCsdrConfig().packages;
 
-  // get only packages installed config based on super .euirc.json file generated
+    // get only packages installed config based on super .euirc.json file generated
   } else {
     packages = innerGlobal.getConfig().packages;
   }
@@ -93,16 +91,15 @@ module.exports.getPackageByRepository = (repository, fromCsdrConfig) => {
   }
 
   return;
-}
+};
 
-module.exports.isLocalPackage = (npmPkg) => {
+const isLocalPackage = module.exports.isLocalPackage = (npmPkg) => {
   tools.logInfo(`Checking if ${npmPkg} is localled installed`);
 
-  const packagesFound = getPackages().filter(p => p.npmPkg === npmPkg);
+  const packagesFound = getPackages().filter((p) => p.npmPkg === npmPkg);
 
   return packagesFound.length > 0;
-}
-
+};
 
 module.exports.getPackage = (pkgName, fromCsdrConfig, optional) => {
   let packages;
@@ -111,12 +108,11 @@ module.exports.getPackage = (pkgName, fromCsdrConfig, optional) => {
   if (fromCsdrConfig) {
     packages = innerGlobal.getCsdrConfig().packages;
 
-  // get only packages installed config based on super .euirc.json file generated
+    // get only packages installed config based on super .euirc.json file generated
   } else {
     packages = innerGlobal.getConfig().packages;
   }
 
-
   // checking if package has been provided as input param, if not we default to command line root argument
   // if that case package name is f.e. : pkg:build <package-name>
   if (!pkgName) {
@@ -126,7 +122,6 @@ module.exports.getPackage = (pkgName, fromCsdrConfig, optional) => {
   // fetch package from config
   let pkg = packages[pkgName];
 
-
   if (!pkg && !optional) {
     throw 'PACKAGE_NOT_FOUND';
   }
@@ -136,7 +131,7 @@ module.exports.getPackage = (pkgName, fromCsdrConfig, optional) => {
 
   // additional pkg infos
   pkg.paths = this.getPackagePaths(pkg);
-  pkg.tsConfigFileName = `${path.parse(pkg.paths.tsConfig).name}.json` ;
+  pkg.tsConfigFileName = `${path.parse(pkg.paths.tsConfig).name}.json`;
 
   // devops file location
   if (pkg.child) {
@@ -166,13 +161,28 @@ module.exports.getPackage = (pkgName, fromCsdrConfig, optional) => {
     }
   }
 
-  return pkg;
-}
+  // set local package flag
+  if (tools.isDirExists(pkg.paths.root)) {
+    pkg.local = true;
+  } else {
+    pkg.local = false;
+  }
 
+  // derive srcRoot structure flag
+  if (pkg.local) {
+    if (tools.isDirExists(pkg.paths.srcLib)) {
+      pkg.srcRootDefault = true;
+    } else {
+      pkg.srcRootDefault = false;
+    }
+  }
+
+  return pkg;
+};
 
 module.exports.getPackagePaths = (pkg) => {
   const rootDirectory = process.cwd();
-  const resolvePath = relativePath => path.resolve(rootDirectory, relativePath);
+  const resolvePath = (relativePath) => path.resolve(rootDirectory, relativePath);
 
   const packagesBasePath = 'packages';
 
@@ -197,7 +207,8 @@ module.exports.getPackagePaths = (pkg) => {
   }
 
   // Remotes specific, to switch install root dir to the remote folder itself (independent dependencies / angular & eUI version, etc...)
-  var repoNodeModules, remoteNodeModules = false;
+  var repoNodeModules,
+    remoteNodeModules = false;
   const depsBaseFile = path.join(resolvePath(packagesPath + '/' + folder), 'dependencies-base.json');
   const depsCompositeFile = path.join(resolvePath(packagesPath + '/' + folder), 'dependencies-composite.json');
   if (pkg.remote && (tools.isFileExists(depsBaseFile) || tools.isFileExists(depsCompositeFile))) {
@@ -207,7 +218,6 @@ module.exports.getPackagePaths = (pkg) => {
     repoNodeModules = resolvePath('node_modules');
   }
 
-
   // Central gathering for pkg paths, always refer to this when fetching pkg paths in code, single sources of truth
   const paths = {
     rootDirectory,
@@ -220,37 +230,38 @@ module.exports.getPackagePaths = (pkg) => {
     publish: resolvePath(packagesPath + '/' + folder + publishFolder),
     src: resolvePath(packagesPath + '/' + folder + '/src'),
     lib: resolvePath(packagesPath + '/' + folder + '/src/lib'),
+    srcLib: resolvePath(packagesPath + '/' + folder + '/src/lib'),
     tsConfig: this.getTSConfigPath(resolvePath, packagesPath + '/' + folder),
   };
 
   return paths;
-}
-
-
+};
 
 module.exports.getBuildablePackages = () => {
-  return this.getPackages().filter(pkg => {
+  return this.getPackages().filter((pkg) => {
     if (!pkg.build) {
       return true;
     }
     return pkg.build.exec !== false;
-  })
-}
+  });
+};
 
 module.exports.getRemotePackages = () => {
-  return this.getPackages().filter(pkg => {
+  return this.getPackages().filter((pkg) => {
     return pkg.remote;
-  })
-}
+  });
+};
 
 module.exports.getBuildablePackagesWithDeps = () => {
-  return this.getDepGraph().overallOrder().filter(pkg => {
-    if (!pkg.build) {
-      return true;
-    }
-    return pkg.build.exec !== false;
-  })
-}
+  return this.getDepGraph()
+    .overallOrder()
+    .filter((pkg) => {
+      if (!pkg.build) {
+        return true;
+      }
+      return pkg.build.exec !== false;
+    });
+};
 
 module.exports.getDepGraph = () => {
   const DepGraph = require('dependency-graph').DepGraph;
@@ -273,8 +284,7 @@ module.exports.getDepGraph = () => {
   });
 
   return graph;
-}
-
+};
 
 const getEuiVersionCore = (pkg) => {
   let version;
@@ -288,7 +298,7 @@ const getEuiVersionCore = (pkg) => {
     if (euiDepsBase) {
       let euiVersion = euiDepsBase.split('.')[0];
 
-      if (euiVersion.substr(0,1) === '^') {
+      if (euiVersion.substr(0, 1) === '^') {
         euiVersion = euiVersion.substr(1);
       }
       version = `${euiVersion}.x`;
@@ -300,10 +310,9 @@ const getEuiVersionCore = (pkg) => {
   }
 
   return version;
-}
-
+};
 
-const getPackageEuiVersion = module.exports.getPackageEuiVersion = (pkg, log = false) => {
+const getPackageEuiVersion = (module.exports.getPackageEuiVersion = (pkg, log = false) => {
   if (pkg.parent !== true && pkg.tsPackage !== false) {
     let version;
 
@@ -331,9 +340,7 @@ const getPackageEuiVersion = module.exports.getPackageEuiVersion = (pkg, log = f
 
     return version;
   }
-}
-
-
+});
 
 module.exports.getLocalPackagesEuiVersion = (log = true) => {
   tools.logTitle('getting local packages installed eUI version...');
@@ -347,29 +354,29 @@ module.exports.getLocalPackagesEuiVersion = (log = true) => {
     if (version) {
       versionsFound.push(version);
     }
-  })
+  });
 
   versionsFound = tools.removeArrayDuplicates(versionsFound);
 
   return versionsFound;
-}
+};
 
 module.exports.getTSConfigPath = (resolvePath, packagePath) => {
   const tsPath = resolvePath(`${packagePath}/tsconfig.json`);
   const oldPath = resolvePath(`${packagePath}/tsconfig.lib.json`);
 
   return tools.isFileExists(tsPath) ? tsPath : oldPath;
-}
+};
 
 module.exports.getTSConfigFileName = (tsConfigPath) => {
   return tsConfigPath.split('/')[tsConfigPath.split('/').length - 1];
-}
+};
 
 module.exports.getPackageScopes = () => {
   const packagesObj = this.getCsdrPackages();
   let packages = Object.keys(packagesObj);
-  packages = packages.map(p => packagesObj[p]).filter(p => !p.backend && !p.remote && p.npmPkg !== undefined);
-  const scopes = packages.map(p => p.npmPkg.split('/')[0]);
+  packages = packages.map((p) => packagesObj[p]).filter((p) => !p.backend && !p.remote && p.npmPkg !== undefined);
+  const scopes = packages.map((p) => p.npmPkg.split('/')[0]);
 
   return tools.removeArrayDuplicates(scopes);
-}
+};