@zohodesk/codestandard-validator 0.0.4-exp-16 → 0.0.4-exp-17

package/build/hooks/Precommit/pre-commit.js

@@ -1,8 +1,7 @@
 "use strict";
 
 const {
-  exec,
-  execSync
+  exec
 } = require('child_process');
 const fs = require('fs');
 const path = require('path');
@@ -29,9 +28,6 @@ const {
   getConfigurationPrecommit,
   getSupportedLanguage
 } = require('../../utils/General/getGeneralInfo');
-const {
-  getRootDirectory
-} = require('../../utils/General/RootDirectoryUtils/getRootDirectory');
 const {
   impactBasedPrecommit,
   shouldWarningsAbortCommit
@@ -66,7 +62,7 @@ async function getStagedFiles() {
       if (error) {
         if (error != null) reject("Couldn't fetch staged files");
       } else if (stderr) {
-        resolve(filterDeltedFileFromStagedFiles(stderr.trim().split('\n')));
+        resolve(stderr.trim().split('\n'));
       } else if (stdout.trim() === '') {
         resolve(stdout.trim());
       }
@@ -74,44 +70,13 @@ async function getStagedFiles() {
   });
 }
 
-/**
- * @function {filterDeltedFileFromStagedFiles} - filter deleted staged files
- * @param {Array<string>} files - staged files
- * @returns 
- */
-function filterDeltedFileFromStagedFiles(files) {
-  return files.filter(file => {
-    const absolutePath = path.resolve(getRootDirectory(), file);
-    if (fs.existsSync(absolutePath)) {
-      return true;
-    }
-    return false;
-  });
-}
-async function lintFiles(filePath) {
-  switch (path.extname(filePath)) {
-    case '.js' || '.ts' || '.tsx' || '.jsx':
-      {
-        return await findEslintErrors(filePath);
-      }
-    case '.css' || '.scss':
-      {
-        return await findStyleLintErrors(filePath);
-      }
-    default:
-      {
-        return [];
-      }
-  }
-}
-
 /**
  * @function findEslintErrors - method  Lint given file based on given configuration
  * @param {*} file - path of file to lint
  * @returns {Array<string>} -  array of command line report as a string 
  */
 
-function findEslintErrors(file) {
+async function findEslintErrors(file) {
   let nodeModulesPathOfProject = `${getNodeModulesPath()}`;
   let eslintExecutablePath = getEslintExecutablePath();
   let eslintConfigurationFilePath = `${nodeModulesPathOfProject}/.eslintrc.js`;
@@ -138,35 +103,6 @@ function findEslintErrors(file) {
     Logger.log(Logger.INFO_TYPE, 'node_modules not found');
   }
 }
-
-/**
- * 
- * @param {*} params 
- */
-function findStyleLintErrors(filePath) {
-  const configFilePath = path.resolve(getNodeModulesPath(), '.stylelintrc.json');
-  const absolutePath = path.join(getRootDirectory(), filePath);
-  try {
-    return new Promise((resolve, reject) => {
-      exec(`npx stylelint ${absolutePath} --config ${configFilePath}`, {
-        cwd: getNodeModulesPath()
-      }, (error, stderr, stdout) => {
-        if (stderr) {
-          resolve(stderr.trim().split('\n'));
-        } else if (error) {
-          Logger.log(Logger.FAILURE_TYPE, error);
-          reject("Error executing eslint command");
-        } else {
-          resolve([]);
-        }
-      });
-    });
-  } catch (error) {
-    Logger.log(Logger.FAILURE_TYPE, `Issue is lint css files`);
-    return [];
-  }
-}
-
 /**
  * @function {calculateGitDiffForFile} -  method calculate diff of file
  * @param {*} branch_name - branch name
@@ -203,10 +139,10 @@ function isOnlyWarningsPresentInFile(eslintErrorsPresent) {
   let severityOfEachErrorInFile = [];
   eslintErrorsPresent.map(error => {
     let partsInString = error.split(" ");
-    let severityOfError = partsInString.find(word => word === 'error' || word === 'warning' || word === '✖');
+    let severityOfError = partsInString.find(word => word === 'error' || word === 'warning');
     severityOfEachErrorInFile.push(severityOfError);
   });
-  return !(severityOfEachErrorInFile.includes('✖') || severityOfEachErrorInFile.includes('error'));
+  return !severityOfEachErrorInFile.includes('error');
 }
 
 /**
@@ -239,33 +175,29 @@ async function preCommitHook() {
       try {
         staged_files = await getStagedFiles();
         if (!staged_files.length == 0) {
-          const {
-            JsFiles: staged_filesJS,
-            CssFiles
-          } = filterFiles(staged_files, eslintConfigFiles, true);
+          staged_files = filterFiles(staged_files, eslintConfigFiles, true);
 
-          // staged_filesJS = filterFiles(staged_filesJS,exemptionFiles) //this is the code for giving exemption to a file during pre commit
-          // if(staged_filesJS.length === 0){
+          // staged_files = filterFiles(staged_files,exemptionFiles) //this is the code for giving exemption to a file during pre commit
+          // if(staged_files.length === 0){
           //     Logger.log(Logger.SUCCESS_TYPE,`Commit Successful`)
           //     process.exit(0)
           // }
           areFilesStaged = true;
-          var stagedFiles = [...staged_filesJS, ...CssFiles];
-          for (let file in stagedFiles) {
-            let currentFileName = stagedFiles[file];
+          for (let file in staged_files) {
+            let currentFileName = staged_files[file];
             let changedLinesArray = [];
             let eslintErrorsInChangedLines = [];
             let isOnlyEslintWarningsPresentInFile = false;
-            if (getSupportedLanguage().includes(path.extname(stagedFiles[file]))) {
+            if (getSupportedLanguage().includes(path.extname(staged_files[file]))) {
               try {
-                var errorsInFile = await lintFiles(stagedFiles[file]);
-                // eslintErrorsInFile = impactBasedPrecommit == false ? filterWarningInFile(errorsInFile) : errorsInFile
-                if (stagedFiles[file] && typeof stagedFiles[file] == 'string') {
-                  if (!errorsInFile.length == 0) {
+                var eslintErrorsInFile = await findEslintErrors(staged_files[file]);
+                // eslintErrorsInFile = impactBasedPrecommit == false ? filterWarningInFile(eslintErrorsInFile) : eslintErrorsInFile
+                if (staged_files[file] && typeof staged_files[file] == 'string') {
+                  if (!eslintErrorsInFile.length == 0) {
                     //Calculating changed lines in a file and storing them in respective arrays
                     if (impactBasedPrecommit) {
                       //git diff is computed and stored in an array
-                      let git_diff = await calculateGitDiffForFile(current_branch, stagedFiles[file]);
+                      let git_diff = await calculateGitDiffForFile(current_branch, staged_files[file]);
                       changedLinesArray = git_diff.filter(line => line.startsWith('@@'));
                       let changedLinesStartArray = [];
                       let changedLinesEndArray = [];
@@ -279,15 +211,15 @@ async function preCommitHook() {
                           changedLinesEndArray.push(changesStartLine + parseInt(changesEndLine) - 1);
                         }
                       }
-                      for (let error = 1; error < errorsInFile.length - 2; error++) {
-                        //errorsInFile[error].trim() - 69:26  error => Do not hardcode content. Use I18N key instead  no-hardcoding/no-hardcoding, 
-                        //errorsInFile[error].trim().split(' ')[0] => 69:26
-                        //errorsInFile[error].trim().split(' ')[0].split(':')[0] => 69
+                      for (let error = 1; error < eslintErrorsInFile.length - 2; error++) {
+                        //eslintErrorsInFile[error].trim() - 69:26  error => Do not hardcode content. Use I18N key instead  no-hardcoding/no-hardcoding, 
+                        //eslintErrorsInFile[error].trim().split(' ')[0] => 69:26
+                        //eslintErrorsInFile[error].trim().split(' ')[0].split(':')[0] => 69
 
-                        let eslintErrorLineNumber = errorsInFile[error].trim().split(' ')[0].split(':')[0];
+                        let eslintErrorLineNumber = eslintErrorsInFile[error].trim().split(' ')[0].split(':')[0];
                         for (let lineNumber in changedLinesStartArray) {
                           if (eslintErrorLineNumber >= changedLinesStartArray[lineNumber] && eslintErrorLineNumber <= changedLinesEndArray[lineNumber]) {
-                            eslintErrorsInChangedLines.push(errorsInFile[error]);
+                            eslintErrorsInChangedLines.push(eslintErrorsInFile[error]);
                           }
                         }
                       }
@@ -308,10 +240,10 @@ async function preCommitHook() {
                         }
                       }
                     } else {
-                      if (errorsInFile.length > 0) {
+                      if (eslintErrorsInFile.length > 0) {
                         let startIndex = 1;
-                        let endIndex = errorsInFile.length - 2;
-                        let listOsEslintErrors = errorsInFile.slice(startIndex, endIndex);
+                        let endIndex = eslintErrorsInFile.length - 2;
+                        let listOsEslintErrors = eslintErrorsInFile.slice(startIndex, endIndex);
                         isOnlyEslintWarningsPresentInFile = isOnlyWarningsPresentInFile(listOsEslintErrors);
                         Logger.log(Logger.FAILURE_TYPE, `\x1b[1m${currentFileName}\x1b[0m`);
                         for (let eslintError of listOsEslintErrors) {
@@ -343,9 +275,9 @@ async function preCommitHook() {
         Logger.log(Logger.INFO_TYPE, 'Error executing pre commit hook');
       }
       if (shouldAbortCommit) {
-        Logger.log(Logger.FAILURE_TYPE, `There are eslint errors/warnings present. So commit is aborted.`);
+        Logger.log(Logger.FAILURE_TYPE, `There are eslint errors present. So commit is aborted. If possible try to fix the eslint warnings also.`);
         process.exit(1);
-      } else if (shouldAbortCommit === false && staged_files.length !== 0) {
+      } else {
         Logger.log(Logger.SUCCESS_TYPE, `Commit Successful`);
         process.exit(0);
       }

package/build/lib/cli.js

@@ -15,9 +15,6 @@ const {
 const {
   installPlugins
 } = require("../utils/PluginsInstallation/installPlugins");
-const {
-  checkIfPluginsAreInstalled
-} = require("../utils/PluginsInstallation/checkIfPluginsAreInstalled");
 const [,, action, ...options] = process.argv;
 async function run() {
   switch (action) {
@@ -33,10 +30,7 @@ async function run() {
       }
     case "setupPlugins":
       {
-        const {
-          uninstalledPlugins
-        } = checkIfPluginsAreInstalled();
-        await executeMethodsThatReturnBooleanValue("Some issue occurred in installing plugins", installPlugins, uninstalledPlugins);
+        await executeMethodsThatReturnBooleanValue("Some issue occurred in installing plugins", installPlugins, null);
         break;
       }
     case "setupExtension":

package/build/lib/postinstall.js

@@ -35,7 +35,7 @@ async function postInstall() {
     isGitInitialized() ? await executeMethodsThatReturnBooleanValue("Some issue occurred in setting up husky.", setupHusky, null) : null;
     await executeMethodsThatReturnBooleanValue("Make sure zgit.csez.zohocorpin.com is accessible", cloneViaCdt, null);
     await executeMethodsThatReturnBooleanValue("Some issue occurred in creating eslint config file.", createEslintConfigFile, null);
-    Logger.log(Logger.SUCCESS_TYPE, "Pre commit setup successfull");
+    Logger.log(Logger.SUCCESS_TYPE, "Pre commit setup successful");
     arePluginsInstalled();
   } catch (error) {
     Logger.log(Logger.FAILURE_TYPE, `Kindly retry npm install. & ${error.message}`);

package/build/setup/sample.config.js

@@ -8,7 +8,7 @@ const path = require("path");
  * @property {boolean} impactBased - Indicates if the linting is impact-based.
  * @property {string} lintReportPath - The path to the lint report JSON file.
  * @property {string} metricServerHost - The URL of the SonarQube server.
- * @property {string} meticHandler_api_token -  The token for authentication with the SonarQube server.
+ * @property {string} exemptionInstanceHost -  This is Exemption running host URL
  * @property {string} metric_token - The token for authentication with the SonarQube server.
  * @property {string} gitEndPoint - API EndPoint for Git Actions
  * @property {string} tsConfigurationPath - The path of the ts configuration Path
@@ -24,7 +24,7 @@ module.exports = {
   impactBased: true,
   lintReportPath: path.resolve(process.cwd(), "lint-report", "lintReport.json"),
   metricServerHost: "https://client-linters.zohodesk.csez.zohocorpin.com",
-  meticHandler_api_token: "oyRcKJZxm2RkXytfAzVtmyRlLIftou5anC==",
+  exemptionInstanceHost: "",
   metric_token: "zxh_9737850jh2l53ml17223929ihii73072j54j2260",
   branchDiffPath: path.resolve(process.cwd(), "diffBranch.json"),
   gitEndPoint: "https://zgit.csez.zohocorpin.com",

package/build/utils/CloneCommonLinterRepo/cloneViaCdt.js

@@ -16,42 +16,21 @@ const {
 const {
   executeSynchronizedCommands
 } = require('../General/executeSyncCommands');
-var {
+const {
   type,
   endPoint,
   branch,
   cacheDirectory,
-  commonLinterRepoName,
-  user
+  commonLinterRepoName
 } = require('../../../jsonUtils/commonLinterRepoDetails');
-const {
-  getConfigurationPrecommit,
-  getRunningEnv
-} = require("../General/getGeneralInfo");
-const {
-  decrypt
-} = require("../General/Hash");
-const {
-  Logger
-} = require("../Logger/Logger");
 /**
  * @function cloneViaCdt - Using the "clint development tool" clones the common linter_configuration repo into precommit library
  * @returns {boolean} - indicating the success or failure of the cloning process
  */
 
 function cloneViaCdt() {
-  const {
-    token
-  } = getConfigurationPrecommit();
   removeFolder(getDeleteDirPath());
-  const userName = user;
-  var absoluteEndPoint = `https://${endPoint}`;
-  const runningEnv = getRunningEnv();
-  if (runningEnv === "CI" || runningEnv === "DEVAUTOMATION") {
-    Logger.log(Logger.INFO_TYPE, `Running in ${runningEnv}`);
-    absoluteEndPoint = `https://${userName}:${decrypt(token, 12)}@${endPoint}`;
-  }
-  var commandToCloneCommonConfigRepo = `npx cdt clone --clone:type=${type} --clone:url=${absoluteEndPoint} --clone:branch=${branch} --clone:cacheDir=${cacheDirectory} --clone:proj:name=${commonLinterRepoName}`;
+  const commandToCloneCommonConfigRepo = `npx cdt clone --clone:type=${type} --clone:url=${endPoint} --clone:branch=${branch} --clone:cacheDir=${cacheDirectory} --clone:proj:name=${commonLinterRepoName}`;
   let isCommonConfigurationClonedSuccessfully = executeSynchronizedCommands(execSync, [commandToCloneCommonConfigRepo, {
     cwd: getClonedDirPath()
   }], `Lint Configuration Cloned Successfully - ${getRepoName() || 'common'}`, 'Could not clone the linters common repo', false, true);

package/build/utils/FileAndFolderOperations/filterFiles.js

@@ -3,9 +3,7 @@
 const {
   Logger
 } = require('../Logger/Logger');
-const {
-  existsSync
-} = require("fs");
+
 /**
  * @function filterFiles - removes certain files from set of source files
  * @param {Array} arrayOfFilesToBeFiltered - array of files which must be filtered 
@@ -14,29 +12,6 @@ const {
  * @returns {Array} - containing the resultant set of files after filtering
  */
 function filterFiles(arrayOfFilesToBeFiltered, filesToBeRemoved, isConfigFileNeedToBeRemoved = false) {
-  /**
-   * @function filterFilesByExtension - filter javascript files. omit feature files
-   * @param {Array<String>} lintFiles - linter files as Array
-   * @returns {Array<String>} 
-   * */
-  function filterFilesByExtension(lintFiles) {
-    return lintFiles.reduce((files, currentFile) => {
-      if (currentFile.includes('.feature') && existsSync(currentFile)) {
-        files.featureFiles.push(currentFile);
-      }
-      if (currentFile.includes('.js') || currentFile.includes('.ts') || currentFile.includes('.tsx') || currentFile.includes('.jsx') && existsSync(currentFile)) {
-        files.JsFiles.push(currentFile);
-      }
-      if (currentFile.includes('.css') && existsSync(currentFile)) {
-        files.CssFiles.push(currentFile);
-      }
-      return files;
-    }, {
-      featureFiles: [],
-      JsFiles: [],
-      CssFiles: []
-    });
-  }
   if (filesToBeRemoved.length !== 0) {
     if (isConfigFileNeedToBeRemoved) {
       arrayOfFilesToBeFiltered.filter(file => {
@@ -56,7 +31,7 @@ function filterFiles(arrayOfFilesToBeFiltered, filesToBeRemoved, isConfigFileNee
         return false;
       }
     });
-    return filterFilesByExtension(filteredArrayofFilesWithoutConfigFile);
+    return filteredArrayofFilesWithoutConfigFile;
   } else if (filesToBeRemoved.length === 0) {
     return arrayOfFilesToBeFiltered;
   }

package/build/utils/General/getGeneralInfo.js

@@ -3,9 +3,6 @@
 const os = require('os');
 const path = require('path');
 const fs = require("fs");
-const {
-  execSync
-} = require('child_process');
 
 /**
  * @function getTimeStampInfo - to fetch various timestamp details
@@ -59,20 +56,11 @@ function getSupportedLanguage() {
   _language.push('.jsx');
   _language.push('.ts');
   _language.push('.tsx');
-  _language.push('.css');
-  _language.push('.scss');
   return _language;
 }
-function getRunningEnv() {
-  const command = "npm config get lint_env";
-  return execSync(command, {
-    shell: true
-  }).toString().trim();
-}
 module.exports = {
   getSupportedLanguage,
   getTimeStampInfo,
   getEnv,
-  getConfigurationPrecommit,
-  getRunningEnv
+  getConfigurationPrecommit
 };

package/build/utils/HuskySetup/initializeHusky.js

@@ -23,7 +23,7 @@ function initializeHusky() {
   let isNavigationToRootDirectorySuccessful = navigateToRootDirectory(getRootDirectory());
   if (isNavigationToRootDirectorySuccessful) {
     try {
-      execSync('npx husky@7 install');
+      execSync('npx husky install');
       return true;
     } catch (error) {
       Logger.log(Logger.FAILURE_TYPE, error.toString());

package/build/utils/HuskySetup/setupHusky.js

@@ -1,8 +1,21 @@
 "use strict";
 
 /* eslint-disable no-console */
+const {
+  execSync
+} = require('child_process');
+const {
+  readdirSync
+} = require('fs');
+const path = require('path');
 const initializeHusky = require('./initializeHusky');
 const configurePrecommitHook = require('./configurePrecommitHook');
+const {
+  getNodeModulesPath
+} = require('../General/getNodeModulesPath');
+const {
+  executeSynchronizedCommands
+} = require('../General/executeSyncCommands');
 const {
   Logger
 } = require('../Logger/Logger');
@@ -13,21 +26,51 @@ let isCustomPrecommitConfigurationSuccessful = false;
  * @returns {boolean} - indicating whether entire custom precommit hook setup using husky process is success or failure
  */
 function setupHusky() {
-  let isHuskyInitializedSuccessfully = initializeHusky();
-  if (isHuskyInitializedSuccessfully) {
-    Logger.log(Logger.SUCCESS_TYPE, 'Husky installation successful');
-    isCustomPrecommitConfigurationSuccessful = configurePrecommitHook();
-    if (isCustomPrecommitConfigurationSuccessful) {
-      Logger.log(Logger.SUCCESS_TYPE, 'Custom pre commit hook setup was successful');
-      return true;
+  let isHuskyPackageInstalled = installHuskyPackage();
+  if (isHuskyPackageInstalled) {
+    let isHuskyInitializedSuccessfully = initializeHusky();
+    if (isHuskyInitializedSuccessfully) {
+      Logger.log(Logger.SUCCESS_TYPE, 'Husky installation successful');
+      isCustomPrecommitConfigurationSuccessful = configurePrecommitHook();
+      if (isCustomPrecommitConfigurationSuccessful) {
+        Logger.log(Logger.SUCCESS_TYPE, 'Custom pre commit hook setup was successful');
+        return true;
+      } else {
+        Logger.log(Logger.FAILURE_TYPE, 'Couldn\'t configure custom pre commit hook');
+        return false;
+      }
     } else {
-      Logger.log(Logger.FAILURE_TYPE, 'Couldn\'t configure custom pre commit hook');
       return false;
     }
   } else {
+    Logger.log(Logger.FAILURE_TYPE, 'Some issue in installing husky package.');
     return false;
   }
 }
+
+/**
+ * @function installHuskyPackage - installs husky package if not installed for the project
+ * @returns {boolean} - indicating if husky package installed successfully or not
+ */
+function installHuskyPackage() {
+  const nodeModulesPathOfProject = path.join(getNodeModulesPath(), 'node_modules');
+  let pluginsInNodeModules = executeSynchronizedCommands(readdirSync, [nodeModulesPathOfProject], '', 'Unable to get directories in node_modules when checking if husky is installed', true, false);
+  let husky = {
+    packageName: 'husky',
+    version: '^7.0.4'
+  };
+  let isHuskyInstalled = pluginsInNodeModules.includes(husky.packageName) ? true : false;
+  if (!isHuskyInstalled) {
+    Logger.log(Logger.INFO_TYPE, 'Installing husky package ....');
+    let installCommand = `npm install ${husky.packageName}@${husky.version} --save-dev`;
+    let isHuskyPackageInstalled = executeSynchronizedCommands(execSync, [installCommand, {
+      stdio: 'inherit'
+    }], 'Husky package Installed Successfully', 'Unable to install husky package', false, true);
+    return isHuskyPackageInstalled;
+  }
+  Logger.log(Logger.SUCCESS_TYPE, 'Husky package already installed!');
+  return true;
+}
 module.exports = {
   setupHusky
 };

package/build/utils/PluginsInstallation/arePluginsInstalled.js

@@ -7,11 +7,8 @@ const {
   printUninstalledPlugins
 } = require('./printUninstalledPlugins');
 function arePluginsInstalled() {
-  let {
-    uninstalledPlugins,
-    noPluginMessage
-  } = checkIfPluginsAreInstalled();
-  printUninstalledPlugins(uninstalledPlugins, noPluginMessage);
+  let uninstalledPlugins = checkIfPluginsAreInstalled();
+  printUninstalledPlugins(uninstalledPlugins);
 }
 module.exports = {
   arePluginsInstalled

package/build/utils/PluginsInstallation/checkIfPluginsAreInstalled.js

@@ -74,50 +74,30 @@ function getAllPlugins() {
  */
 function checkIfPluginsAreInstalled() {
   const pluginsForCurrentRepo = getAllPlugins();
-  var uninstalledPlugins = [];
-  var plugin = {
+  let uninstalledPlugins = [];
+  let plugin = {
     noPluginMessage: '',
     uninstalledPlugins: []
   };
-  // let pluginsIndevDependencies = {}
-  // let pluginNamesOfDevDependencyPlugins = []
-  // let arePluginsPresentIndevDependencies = false
-  // var arePluginsPresentForCurrentRepo = false
-
-  // if(existsSync(path.join(getNodeModulesPath(),'package.json'))){
-  //     let packageJsonContent = require(path.join(getNodeModulesPath(),'package.json'))
-  //     pluginsIndevDependencies = packageJsonContent.devDependencies
-
-  //     if(pluginsIndevDependencies !== undefined){
-  //         arePluginsPresentIndevDependencies = true
-  //         pluginNamesOfDevDependencyPlugins = Object.keys(pluginsIndevDependencies)
-  //     }
-  //     else if(pluginsIndevDependencies === undefined){
-  //         Logger.log(Logger.INFO_TYPE,'No devDependencies present!')
-  //     }
-  // }
-  // else{
-  //     arePluginsPresentIndevDependencies = false
-  // }
-
-  // arePluginsPresentForCurrentRepo = pluginsForCurrentRepo.length !== 0 ? true : false
-
-  uninstalledPlugins = pluginsForCurrentRepo.map(plugin => {
-    const {
-      packageName,
-      version
-    } = plugin;
-    const {
-      isPluginPresent,
-      pluginPath
-    } = checkPluginsPresentInNodemodules(packageName);
-    if (isPluginPresent && checkPluginHasProperVersion(packageName, pluginPath, version)) {
-      return false;
+  let pluginsIndevDependencies = {};
+  let pluginNamesOfDevDependencyPlugins = [];
+  let arePluginsPresentIndevDependencies = false;
+  let arePluginsPresentForCurrentRepo = false;
+  if (existsSync(path.join(getNodeModulesPath(), 'package.json'))) {
+    let packageJsonContent = require(path.join(getNodeModulesPath(), 'package.json'));
+    pluginsIndevDependencies = packageJsonContent.devDependencies;
+    if (pluginsIndevDependencies !== undefined) {
+      arePluginsPresentIndevDependencies = true;
+      pluginNamesOfDevDependencyPlugins = Object.keys(pluginsIndevDependencies);
+    } else if (pluginsIndevDependencies === undefined) {
+      Logger.log(Logger.INFO_TYPE, 'No devDependencies present!');
     }
-    return `${packageName}@${version}`;
-  }).filter(Boolean);
-  if (uninstalledPlugins.length > 0) {
-    // uninstalledPlugins = getUnInstalledPlugins(pluginsForCurrentRepo, pluginNamesOfDevDependencyPlugins, pluginsIndevDependencies);
+  } else {
+    arePluginsPresentIndevDependencies = false;
+  }
+  arePluginsPresentForCurrentRepo = pluginsForCurrentRepo.length !== 0 ? true : false;
+  if (arePluginsPresentForCurrentRepo || arePluginsPresentIndevDependencies) {
+    uninstalledPlugins = getUnInstalledPlugins(pluginsForCurrentRepo, pluginNamesOfDevDependencyPlugins, pluginsIndevDependencies);
     plugin.noPluginMessage = '';
     plugin.uninstalledPlugins = uninstalledPlugins;
     return plugin;
@@ -127,40 +107,6 @@ function checkIfPluginsAreInstalled() {
     return plugin;
   }
 }
-
-/**
- * @function checkPluginsPresentInNodemodules - to check if the plugin is present in the node_modules of the project
- * @param {string} rulePluginName 
- * @returns 
- */
-function checkPluginsPresentInNodemodules(rulePluginName) {
-  try {
-    const pluginPath = require.resolve(rulePluginName.toString());
-    return {
-      pluginPath: pluginPath,
-      isPluginPresent: existsSync(pluginPath)
-    };
-  } catch (error) {
-    return {
-      pluginPath: "",
-      isPluginPresent: false
-    };
-  }
-}
-
-/**
- * @function checkPluginHasProperVersion - to check if the plugin has the proper version`
- * @param {string} rulePluginName 
- * @param {string} installationPath 
- * @param {string} remotePluginVersion 
- * @returns 
- */
-
-function checkPluginHasProperVersion(rulePluginName, installationPath, remotePluginVersion) {
-  const pluginPathArray = installationPath.toString().split(rulePluginName);
-  const rulePluginPackageJSONPath = path.join(pluginPathArray[0], rulePluginName, 'package.json');
-  return remotePluginVersion == require(rulePluginPackageJSONPath).version ? true : false;
-}
 module.exports = {
   checkIfPluginsAreInstalled
 };

package/build/utils/PluginsInstallation/installPlugins.js

@@ -12,30 +12,23 @@ const {
 const {
   Logger
 } = require('../Logger/Logger');
-const {
-  createVersionControlFile
-} = require('../FileAndFolderOperations/versionControl');
-const {
-  getNodeModulesPath
-} = require('../General/getNodeModulesPath');
 
 /**
  * @function installPlugins - installs uninstalled plugins for the project
  * @returns {boolean} - indicating if plugins to be installed are installed successfully or not
  */
-function installPlugins(pluginsToBeInstalled) {
-  // let {uninstalledPlugins : pluginsToBeInstalled} = checkIfPluginsAreInstalled()
+function installPlugins() {
+  let uninstalledPlugins = checkIfPluginsAreInstalled();
+  let pluginsToBeInstalled = uninstalledPlugins.uninstalledPlugins;
   if (pluginsToBeInstalled.length > 0) {
-    let installCommand = `npm install --save ${pluginsToBeInstalled.join(' ')}`;
+    let installCommand = `npm install --save-dev ${pluginsToBeInstalled.join(' ')}`;
     Logger.log(Logger.INFO_TYPE, 'Installing plugins ....');
     Logger.log(Logger.INFO_TYPE, `Install command being executed: ${installCommand}`);
     let arePluginsInstalledSuccessfully = executeSynchronizedCommands(execSync, [installCommand, {
-      stdio: 'inherit',
-      cwd: getNodeModulesPath()
+      stdio: 'inherit'
     }], '', 'Some issue occured while installing plugins', false, true);
     if (arePluginsInstalledSuccessfully) {
       Logger.log(Logger.SUCCESS_TYPE, 'Plugins installation successful');
-      createVersionControlFile(pluginsToBeInstalled);
       return true;
     } else {
       Logger.log(Logger.FAILURE_TYPE, "Unable to install plugins.Kindly retry the command");

package/build/utils/PluginsInstallation/printUninstalledPlugins.js

@@ -8,17 +8,17 @@ const {
  * @function printUninstalledPlugins - to notify the user regarding plugins for the repository has been installed or not 
  * @param {Array} uninstalledPlugins - contains list of uninstalled plugins for the project
  */
-function printUninstalledPlugins(uninstalledPlugins, noPluginMessage) {
+function printUninstalledPlugins(uninstalledPlugins) {
   let commandToInstallPlugins = 'npx ZDPrecommit setupPlugins';
-  if (uninstalledPlugins.length !== 0) {
+  if (uninstalledPlugins.uninstalledPlugins.length !== 0) {
     Logger.log(Logger.INFO_TYPE, 'The following plugins are not installed:');
-    uninstalledPlugins.map(plugin => {
+    uninstalledPlugins.uninstalledPlugins.map(plugin => {
       Logger.log(Logger.INFO_TYPE, `"${plugin}"`);
     });
     Logger.log(Logger.FAILURE_TYPE, `Kindly execute the command \x1b[37m"${commandToInstallPlugins}\x1b[0m" \x1b[31mfrom the location where package.json of the repo is present to install the plugins`);
-  } else if (uninstalledPlugins.length === 0 && noPluginMessage.trim() === '') {
+  } else if (uninstalledPlugins.uninstalledPlugins.length === 0 && uninstalledPlugins.noPluginMessage.trim() === '') {
     Logger.log(Logger.INFO_TYPE, 'Plugins are installed already');
-  } else if (uninstalledPlugins.length === 0 && noPluginMessage.trim() !== '') {
+  } else if (uninstalledPlugins.uninstalledPlugins.length === 0 && uninstalledPlugins.noPluginMessage.trim() !== '') {
     Logger.log(Logger.INFO_TYPE, 'No plugins present for the repository');
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zohodesk/codestandard-validator",
-  "version": "0.0.4-exp-16",
+  "version": "0.0.4-exp-17",
   "description": "library to enforce code standard using eslint",
   "main": "index.js",
   "scripts": {

package/build/utils/FileAndFolderOperations/versionControl.js

@@ -1,25 +0,0 @@
-"use strict";
-
-const path = require('path');
-const {
-  getNodeModulesPath
-} = require('../General/getNodeModulesPath');
-const {
-  writeFileSync
-} = require('fs');
-const {
-  Logger
-} = require('../Logger/Logger');
-function createVersionControlFile(uninstalledPlugins) {
-  Logger.log(Logger.INFO_TYPE, `Rule Plugin Versions are Noted in pluginVersionControl.js`);
-  const versionfilePath = path.join(getNodeModulesPath(), 'node_modules', '@zohodesk', 'codestandard-validator', 'pluginVersionControl.js');
-  writeFileSync(versionfilePath, `module.exports.plugins = [${uninstalledPlugins.map(plugin => {
-    return JSON.stringify({
-      plugin: `${plugin}`,
-      time: `${new Date()}`
-    });
-  }).join(',')}]`);
-}
-module.exports = {
-  createVersionControlFile
-};

package/build/utils/General/Hash.js

@@ -1,21 +0,0 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.encrypt = exports.decrypt = void 0;
-const caesarCipher = (str, shift) => {
-  return str.split("").map(char => {
-    let code = char.charCodeAt();
-    if (code >= 65 && code <= 90) {
-      return String.fromCharCode((code - 65 + shift) % 26 + 65);
-    } else if (code >= 97 && code <= 122) {
-      return String.fromCharCode((code - 97 + shift) % 26 + 97);
-    }
-    return char;
-  }).join("");
-};
-const encrypt = (plaintext, shift) => caesarCipher(plaintext, shift);
-exports.encrypt = encrypt;
-const decrypt = (ciphertext, shift) => caesarCipher(ciphertext, 26 - shift);
-exports.decrypt = decrypt;

package/build/utils/PluginsInstallation/Worker/installPluginsByWoker.js

@@ -1,35 +0,0 @@
-"use strict";
-
-const {
-  Worker
-} = require('worker_threads');
-async function installPluginsByWorker(plugins) {
-  try {
-    const results = await Promise.all(plugins.map(pkg => runWorker({
-      packageName: pkg.packageName,
-      version: pkg.version
-    })));
-    console.log('Package are installed By from workers:', results.join('\n'));
-  } catch (error) {
-    console.error('Error:', error);
-  }
-}
-function runWorker(data) {
-  return new Promise((resolve, reject) => {
-    const worker = new Worker(`${__dirname}/worker.js`, {
-      workerData: data
-    });
-    worker.on('message', result => {
-      resolve(result);
-    });
-    worker.on('error', error => {
-      reject(error);
-    });
-    worker.on('exit', code => {
-      if (code !== 0) {
-        reject(new Error(`Worker stopped with exit code ${code}`));
-      }
-    });
-  });
-}
-module.exports = installPluginsByWorker;

package/build/utils/PluginsInstallation/Worker/worker.js

@@ -1,33 +0,0 @@
-"use strict";
-
-const {
-  workerData,
-  parentPort
-} = require('worker_threads');
-const {
-  spawnSync
-} = require('child_process');
-const {
-  getNodeModulesPath
-} = require('../../General/getNodeModulesPath');
-function performanceInstalllation({
-  packageName,
-  version
-}) {
-  const start = performance.now();
-  try {
-    require.resolve(packageName);
-  } catch (err) {
-    spawnSync('npm', ['install', `${packageName}@${version}`, '--no-save'], {
-      stdio: 'inherit',
-      cwd: getNodeModulesPath()
-    });
-  }
-  const end = performance.now();
-  return `Package ${packageName}@${version} installed successfully and installed in ${end - start} ms`;
-}
-const result = performanceInstalllation({
-  packageName: workerData.packageName,
-  version: workerData.version
-});
-parentPort.postMessage(result);