@zohodesk/codestandard-validator 1.3.1-platform-14 → 1.4.0

package/.babelrc

@@ -4,8 +4,9 @@
       "@babel/preset-env",
       {
         "targets": {
-          "node": "14"
-        }
+          "node": "18"
+        },
+        "exclude": ["@babel/plugin-transform-dynamic-import"]
       }
     ]
   ],

package/build/ai/ollama-service.js

@@ -349,13 +349,12 @@ async function reviewFiles(filePaths, options = {}) {
           } : r.value;
           results.push(out);
         } else {
-          var _r$reason;
           const fp = r.filePath;
           out = {
             success: false,
             fileName: path.basename(fp),
             filePath: fp,
-            error: ((_r$reason = r.reason) === null || _r$reason === void 0 ? void 0 : _r$reason.message) || r.reason || 'Unknown error'
+            error: r.reason?.message || r.reason || 'Unknown error'
           };
           results.push(out);
         }

package/build/chunk/chunk_Restriction.js

@@ -1,6 +1,5 @@
 "use strict";
 
-var _process;
 const {
   execSync
 } = require('child_process');
@@ -22,7 +21,7 @@ const {
   gitEndPoint: _gitEndPoint
 } = getConfigPath(getConfigPathExecute());
 const gitEndPoint = _gitEndPoint || 'https://zgit.csez.zohocorpin.com';
-const compareBranch = ((_process = process) === null || _process === void 0 || (_process = _process.env) === null || _process === void 0 ? void 0 : _process.CHUNK_BRANCH) || 'release';
+const compareBranch = process?.env?.CHUNK_BRANCH || 'release';
 const configPath = path.resolve(process.cwd(), 'lint.config.js');
 function getGitProjectId() {
   try {
@@ -75,7 +74,7 @@ function createBranchDiffChunk(branchName) {
 }
 function parserReleaseDiffRemoteAPI(rawresponse) {
   var formatted = [];
-  if (rawresponse !== null && rawresponse !== void 0 && rawresponse.diffs) {
+  if (rawresponse?.diffs) {
     formatted = rawresponse.diffs.map(changeset => {
       if (isTestOrUATFile(changeset.new_path)) {
         return null;
@@ -178,7 +177,7 @@ if (shouldEnable) {
   const RESET = '\x1b[0m';
   (async function () {
     Logger.log(`****************************** Calculation Chunk Size *******************************`);
-    if (!(globalThis !== null && globalThis !== void 0 && globalThis.chunkSize)) {
+    if (!globalThis?.chunkSize) {
       globalThis.chunkSize = {
         $modifieldLines: 0,
         $removedLines: 0

package/build/hooks/Precommit/commit.js

@@ -35,10 +35,11 @@ async function preCommitHook() {
     return;
   }
   const {
-    JsFiles
+    JsFiles,
+    featureFiles
   } = filterFiles(stagedFiles, [".eslintrc.js"], true);
   const branch = await safeGetBranch();
-  const shouldAbort = await runLintWorkflow(JsFiles, branch);
+  const shouldAbort = await runLintWorkflow([...JsFiles, ...featureFiles], branch);
   if (shouldAbort) {
     Logger.log(Logger.FAILURE_TYPE, "There are linter errors/warnings. Commit aborted 🙁.");
     process.exit(1);

package/build/mutation/mutationCli.js

@@ -1,7 +1,7 @@
 "use strict";
 
 var MutationRunner = require('./mutationRunner');
-var StrykerConfigBuilder = require('./strykerConfigBuilder');
+var path = require('path');
 
 /**
  * MutationCli — CLI argument parser and executor for mutation testing.
@@ -25,6 +25,7 @@ var StrykerConfigBuilder = require('./strykerConfigBuilder');
  *   --concurrency=<n>           (default: cpu count - 1)
  *   --timeoutMS=<ms>            (default: 60000)
  *   --strykerConfig=<path>      (external stryker.conf.js)
+ *   --jestConfig=<path>         (external jest.config.js)
  *
  * Compatible with Node 16+.
  */
@@ -46,7 +47,8 @@ MutationCli.prototype.parseArgs = function (args) {
     logLevel: 'info',
     concurrency: null,
     timeoutMS: null,
-    strykerConfig: null
+    strykerConfig: null,
+    jestConfig: null
   };
   for (var i = 0; i < args.length; i++) {
     var arg = args[i];
@@ -72,8 +74,10 @@ MutationCli.prototype.parseArgs = function (args) {
       parsed.concurrency = arg.split('=')[1];
     } else if (arg.indexOf('--timeoutMS=') === 0) {
       parsed.timeoutMS = arg.split('=')[1];
-    } else if (arg.indexOf('--strykerConfig=') === 0) {
+    } else if (arg.indexOf('--mutationConfigPath=') === 0) {
       parsed.strykerConfig = arg.split('=')[1];
+    } else if (arg.indexOf('--jestConfigPath=') === 0) {
+      parsed.jestConfig = arg.split('=')[1];
     }
   }
   if (parsed.release == null || parsed.release === '') {
@@ -109,26 +113,26 @@ MutationCli.prototype.validate = function (parsed) {
 /* ------------------------------------------------------------------ */
 
 MutationCli.prototype._buildConfig = function (parsed) {
-  var builder = new StrykerConfigBuilder();
-  if (parsed.strykerConfig || this._options.strykerConfigpath) {
-    builder.fromConfigFile(parsed.strykerConfig || this._options.strykerConfigpath);
+  var config = {};
+
+  // Load config from file if specified
+  var configPath = parsed.strykerConfig || this._options.mutationConfigPath;
+  if (configPath) {
+    var resolved = path.resolve(configPath);
+    config = Object.assign({}, require(resolved));
   }
-  builder.setLogLevel(parsed.logLevel || this._options.logLevel || 'info');
+
+  // Apply CLI overrides
+  config.logLevel = parsed.logLevel || this._options.logLevel || 'info';
   var concurrency = parsed.concurrency || this._options.concurrency;
   if (concurrency != null) {
-    builder.setConcurrency(concurrency);
+    config.concurrency = Number(concurrency);
   }
   var timeoutMS = parsed.timeoutMS || this._options.timeoutMS;
   if (timeoutMS != null) {
-    builder.setTimeoutMS(timeoutMS);
-  }
-  if (this._options.jestConfig) {
-    builder.setJest(Object.assign({
-      projectType: 'custom',
-      enableFindRelatedTests: true
-    }, this._options.jestConfig));
+    config.timeoutMS = Number(timeoutMS);
   }
-  return builder;
+  return config;
 };
 
 /* ------------------------------------------------------------------ */
@@ -144,12 +148,12 @@ MutationCli.prototype.execute = function (args) {
       error: validation.error
     });
   }
-  var configBuilder = this._buildConfig(parsed);
+  var mutationConfig = this._buildConfig(parsed);
   var runner = new MutationRunner({
     ...this._options,
     cwd: this._options.cwd,
-    configBuilder: configBuilder,
-    jestConfig: this._options.jestConfig,
+    mutationConfig: mutationConfig,
+    jestConfigPath: parsed.jestConfig || this._options.jestConfig || null,
     outputDir: parsed.outputDir || this._options.outputDir,
     outputFileName: parsed.outputFileName || this._options.outputFileName
   });

package/build/mutation/mutationRunner.js

@@ -1,7 +1,6 @@
 "use strict";
 
 var StrykerWrapper = require('./strykerWrapper');
-var StrykerConfigBuilder = require('./strykerConfigBuilder');
 var BranchDiff = require('./branchDiff');
 var FileResolver = require('./fileResolver');
 var ReportGenerator = require('./reportGenerator');
@@ -30,24 +29,8 @@ var path = require('path');
 function MutationRunner(options) {
   options = options || {};
   this._cwd = options.cwd || process.cwd();
-  var configBuilder = options.configBuilder;
-  if (!configBuilder) {
-    configBuilder = new StrykerConfigBuilder();
-    if (options.strykerConfigpath) {
-      configBuilder.fromConfigFile(options.strykerConfigpath);
-    }
-    if (options.concurrency != null) configBuilder.setConcurrency(options.concurrency);
-    if (options.timeoutMS != null) configBuilder.setTimeoutMS(options.timeoutMS);
-    if (options.logLevel) configBuilder.setLogLevel(options.logLevel);
-    if (options.reportPath) {
-      configBuilder.setHtmlReporter({
-        fileName: options.reportPath
-      }).setJsonReporter({
-        fileName: options.reportPath
-      });
-    }
-  }
-  this._stryker = new StrykerWrapper(configBuilder);
+  var mutationConfig = options.mutationConfig || {};
+  this._stryker = new StrykerWrapper(mutationConfig);
   this._branchDiff = new BranchDiff({
     cwd: this._cwd,
     token: options.token,
@@ -64,7 +47,7 @@ function MutationRunner(options) {
     outputDir: options.outputDir || 'reports/mutation',
     outputFileName: options.outputFileName || 'mutation-report.json'
   });
-  this._jestConfig = options.jestConfig || {};
+  this._jestConfigPath = options.jestConfigPath || null;
 }
 
 /* ------------------------------------------------------------------ */
@@ -187,13 +170,7 @@ MutationRunner.prototype._runMutation = function (sourceFiles, testFiles, ctx) {
   var strykerOpts = {
     rootDirectory: this._branchDiff.getRootDirectory(),
     testFiles: testsToRun,
-    jest: Object.assign({}, self._jestConfig, {
-      config: {
-        testMatch: testsToRun,
-        reporters: ['json']
-      },
-      enableFindRelatedTests: true
-    })
+    jestConfigPath: this._jestConfigPath
   };
   return self._stryker.run(sourcesToMutate, strykerOpts).then(function (mutationResult) {
     var out3 = self._reportGenerator.generateAndWrite(mutationResult, {

package/build/mutation/strykerWrapper.js

@@ -1,49 +1,37 @@
 "use strict";
 
-function _interopRequireWildcard(e, t) { if ("function" == typeof WeakMap) var r = new WeakMap(), n = new WeakMap(); return (_interopRequireWildcard = function (e, t) { if (!t && e && e.__esModule) return e; var o, i, f = { __proto__: null, default: e }; if (null === e || "object" != typeof e && "function" != typeof e) return f; if (o = t ? n : r) { if (o.has(e)) return o.get(e); o.set(e, f); } for (const t in e) "default" !== t && {}.hasOwnProperty.call(e, t) && ((i = (o = Object.defineProperty) && Object.getOwnPropertyDescriptor(e, t)) && (i.get || i.set) ? o(f, t, i) : f[t] = e[t]); return f; })(e, t); }
 var path = require('path');
+var os = require('os');
 var patternConfig = require('./mutatePattern.json');
-var StrykerConfigBuilder = require('./strykerConfigBuilder');
 var DEFAULT_MUTATE_PATTERN = patternConfig.defaultPattern;
+var DEFAULTS = {
+  packageManager: 'npm',
+  testRunner: 'jest',
+  coverageAnalysis: 'perTest',
+  reporters: ['html', 'json'],
+  logLevel: 'info',
+  timeoutMS: 60000,
+  timeoutFactor: 1.5,
+  cleanTempDir: true,
+  allowEmpty: true,
+  thresholds: {
+    high: 80,
+    low: 60,
+    break: null
+  }
+};
 
 /**
- * StrykerWrapper — thin wrapper around @stryker-mutator/core v5.
+ * StrykerWrapper — thin wrapper around @stryker-mutator/core.
  *
- * Accepts a StrykerConfigBuilder (or a raw options hash for backwards
- * compatibility) to produce the Stryker config, runs mutation testing,
- * and normalizes results.
+ * Accepts a plain config object (loaded from a config file or defaults),
+ * merges with DEFAULTS, runs mutation testing, and normalizes results.
  *
  * Compatible with Node 16+.
  */
-function StrykerWrapper(options) {
-  options = options || {};
+function StrykerWrapper(config) {
   this._strykerModule = null;
-  if (options instanceof StrykerConfigBuilder) {
-    this._configBuilder = options;
-  } else if (options.strykerConfigpath) {
-    this._configBuilder = new StrykerConfigBuilder().fromConfigFile(options.strykerConfigpath);
-  } else {
-    this._configBuilder = new StrykerConfigBuilder().setTestRunner('jest').setCoverageAnalysis('perTest').setAllowEmpty(true).setReporters(['html', 'json']).setCleanTempDir(true);
-    if (options.reportPath) {
-      this._configBuilder.setHtmlReporter({
-        fileName: options.reportPath
-      }).setJsonReporter({
-        fileName: options.reportPath
-      });
-    }
-    if (options.concurrency != null) {
-      this._configBuilder.setConcurrency(options.concurrency);
-    }
-    if (options.timeoutMS != null) {
-      this._configBuilder.setTimeoutMS(options.timeoutMS);
-    }
-    if (options.timeoutFactor != null) {
-      this._configBuilder.setTimeoutFactor(options.timeoutFactor);
-    }
-    if (options.logLevel) {
-      this._configBuilder.setLogLevel(options.logLevel);
-    }
-  }
+  this._baseConfig = config && typeof config === 'object' ? Object.assign({}, config) : {};
 }
 
 /* ------------------------------------------------------------------ */
@@ -56,13 +44,13 @@ StrykerWrapper.prototype._loadStryker = async function () {
     return Promise.resolve(self._strykerModule);
   }
   try {
-    self._strykerModule = await Promise.resolve().then(() => _interopRequireWildcard(require('@stryker-mutator/core')));
+    self._strykerModule = await import('@stryker-mutator/core');
     return Promise.resolve(self._strykerModule);
   } catch (err) {
     return Promise.reject(new Error('Failed to load @stryker-mutator/core. Install it:\n' + '  npm i --save-dev @stryker-mutator/core@5 @stryker-mutator/jest-runner@5\n' + err.message));
   }
 };
-StrykerWrapper.prototype._buildMutatePatterns = function (sourceFiles, options) {
+StrykerWrapper.prototype._buildSourceToRun = function (sourceFiles, options) {
   if (!sourceFiles || sourceFiles.length === 0) {
     return [DEFAULT_MUTATE_PATTERN];
   }
@@ -75,45 +63,88 @@ StrykerWrapper.prototype._buildMutatePatterns = function (sourceFiles, options)
 };
 
 /**
- * Build the final Stryker config by cloning the base builder and
- * layering per-run overrides (mutate patterns, test files, jest, etc.).
+ * Load a jest config file (JS or JSON).  Handles both object and
+ * function exports (jest supports `module.exports = async (cfg) => ({...})`).
+ *
+ * Returns null if filePath is falsy or the file cannot be loaded.
+ */
+StrykerWrapper.prototype._loadJestConfigFile = function (filePath) {
+  if (!filePath) return null;
+  try {
+    var resolved = path.resolve(filePath);
+    var exported = require(resolved);
+    if (typeof exported === 'function') {
+      exported = exported();
+    }
+    return exported && typeof exported === 'object' ? exported : null;
+  } catch (err) {
+    throw new Error('Failed to load jest config from "' + filePath + '": ' + err.message);
+  }
+};
+
+/**
+ * Build the final Stryker config by merging DEFAULTS, base config,
+ * and per-run overrides.
+ *
+ * Jest config resolution (highest → lowest precedence inside Stryker):
+ *   configFromFile  (loaded via Jest's native pipeline — handles preset, <rootDir>, transform)
+ *     ↓ merged with
+ *   jest.config     (inline override object — safe for testMatch, reporters, etc.)
+ *     ↓ merged with
+ *   JEST_OVERRIDE_OPTIONS (Stryker's own forced keys — always wins)
  */
 StrykerWrapper.prototype._buildStrykerConfig = function (sourceFiles, options) {
   options = options || {};
-  var mutatePatterns = this._buildMutatePatterns(sourceFiles, options);
-  var builder = this._configBuilder.clone().setMutate(mutatePatterns);
-  if (options.testFiles) {
-    builder.setTestRunner('command');
-    var testPattern = options.testFiles.map(function (f) {
-      var base = path.basename(f);
-      return base.replace(/\.(test|spec)\.(js|ts|jsx|tsx|mjs|cjs)$/, '');
-    }).join('|');
-    var cdCmd = options.rootDirectory ? 'cd ' + options.rootDirectory + ' && ' : '';
-    builder.setCommandRunner({
-      command: 'npx ZDTestingFramework unit-test'
-    });
+
+  // 1. Start with defaults, overlay base config
+  var config = Object.assign({}, DEFAULTS, this._baseConfig);
+
+  // 2. Compute concurrency if not set
+  if (config.concurrency == null) {
+    config.concurrency = Math.max(1, os.cpus().length - 1);
   }
-  if (options.jest) {
-    builder.setJest(Object.assign({
-      projectType: 'custom',
-      enableFindRelatedTests: true
-    }, options.jest));
+
+  // 3. Set mutate = resolved source files (sourceToRun)
+  config.mutate = this._buildSourceToRun(sourceFiles, options);
+
+  // 4. Build jest section
+  var jestSection = {
+    projectType: 'custom',
+    enableFindRelatedTests: false
+  };
+  if (options.jestConfigPath) {
+    jestSection.configFile = options.jestConfigPath;
+  }
+  if (options.testFiles && options.testFiles.length > 0) {
+    // Convert absolute test paths to <rootDir>-relative paths
+    var rootDir = options.rootDirectory || process.cwd();
+    var rootDirRelativeTests = options.testFiles.map(function (f) {
+      var absPath = path.isAbsolute(f) ? f : path.resolve(rootDir, f);
+      var relativePath = path.relative(rootDir, absPath);
+      return '<rootDir>/' + relativePath.replace(/\\/g, '/');
+    });
+    jestSection.config = Object.assign({}, this._loadJestConfigFile(options.jestConfigPath), {
+      testMatch: rootDirRelativeTests
+    });
   }
+  config.jest = jestSection;
+
+  // 5. Per-run overrides
   if (options.configFile) {
-    builder.setConfigFile(options.configFile);
+    config.configFile = options.configFile;
   }
   if (options.reportPath) {
-    builder.setJsonReporter({
+    config.jsonReporter = {
       fileName: options.reportPath
-    });
+    };
   }
   if (options.tempDirName) {
-    builder.setTempDirName(options.tempDirName);
+    config.tempDirName = options.tempDirName;
   }
   if (options.thresholds) {
-    builder.setThresholds(options.thresholds);
+    config.thresholds = options.thresholds;
   }
-  return builder.build();
+  return config;
 };
 
 /* ------------------------------------------------------------------ */
@@ -230,7 +261,11 @@ StrykerWrapper.prototype._emptyNormalizedResult = function () {
   };
 };
 StrykerWrapper.prototype.getDefaultConfig = function () {
-  return this._configBuilder.build();
+  var config = Object.assign({}, DEFAULTS, this._baseConfig);
+  if (config.concurrency == null) {
+    config.concurrency = Math.max(1, os.cpus().length - 1);
+  }
+  return config;
 };
 module.exports = StrykerWrapper;
 module.exports.DEFAULT_MUTATE_PATTERN = DEFAULT_MUTATE_PATTERN;

package/build/utils/General/getGeneralInfo.js

@@ -70,6 +70,7 @@ function getSupportedLanguage() {
   _language.push('.tsx');
   _language.push('.css');
   _language.push('.scss');
+  _language.push('.feature');
   return [...new Set(_language)];
 }
 function getRunningEnv() {
@@ -101,9 +102,8 @@ function getRunningEnv() {
 
 function getLastCommitHash() {
   try {
-    var _JSON$parse$;
     const cmd = `curl --header "PRIVATE-TOKEN: ${process.env.TOOL_TOKEN}" --url ${commitHashEndPoint}`;
-    return (_JSON$parse$ = JSON.parse(execSync(cmd))[0]) === null || _JSON$parse$ === void 0 ? void 0 : _JSON$parse$.id;
+    return JSON.parse(execSync(cmd))[0]?.id;
   } catch (err) {
     Logger.log(Logger.FAILURE_TYPE, err);
     return null;

package/changeLog.md

@@ -37,29 +37,38 @@
 
 # 1.1.0 - SonarQube Enable
 
-1. Enable sonarqube with flag with backward compactibility
+ 1. Enable sonarqube with flag with backward compactibility
 
 # 1.1.3 - SonarQube Enable
 
-1. Remote Installtion flow moved to before and handled process.exit removed
+ 1. Remote Installtion flow moved to before and handled process.exit removed
 
 # 1.1.4 - Minor Bug Fixes
 
-1. ProjectId api revamp
-2. code standard analytics bin bind to code standard validator bin
+ 1. ProjectId api revamp
+ 2. code standard analytics bin bind to code standard validator bin
 
 # 1.2.4
 
-1. Refactor configuration loading and linting process, update mandatory rules handling, and improve pre-commit hook functionality
+ 1. Refactor configuration loading and linting process, update mandatory rules handling, and improve pre-commit hook functionality
 
 # 1.3.4
 
-1. Fix typo in environment variable name: `MANDARORY_RULES` corrected to `MANDATORY_RULES`
-2. Fix typo in Config.js: `intialize` corrected to `initialize` for post-install execution
-3. Remove unused `jest.setup.js` placeholder file
-4. Version bump from `1.2.4-exp-7` to `1.2.4-exp-9`
+ 1. Fix typo in environment variable name: `MANDARORY_RULES` corrected to `MANDATORY_RULES`
+ 2. Fix typo in Config.js: `intialize` corrected to `initialize` for post-install execution
+ 3. Remove unused `jest.setup.js` placeholder file
+ 4. Version bump from `1.2.4-exp-7` to `1.2.4-exp-9`
 
 # 1.3.1 - Bug Fixes
 
-1. Replace asynchronous `fs.rm` with synchronous `fs.rmSync` in folder cleanup utility to align with synchronized command execution.
-2. Update package version from `1.3.0` to `1.3.1`.
+ 1. Replace asynchronous `fs.rm` with synchronous `fs.rmSync` in folder cleanup utility to align with synchronized command execution.
+ 2. Update package version from `1.3.0` to `1.3.1`.
+
+# 1.4.0 - Mutation Refactor & Environment Detection
+
+ 1. Remove `StrykerConfigBuilder` class and simplify Stryker configuration to use a plain defaults object in `strykerWrapper.js`
+ 2. Refactor `mutationCli.js` and `mutationRunner.js` to work without the config builder abstraction
+ 3. Improve `getRunningEnv()` to read environment from `npm_config_lint_env`, `LINT_ENV`, and `CI` env vars before falling back to `npm config get`
+ 4. Add `.feature` file support in `getSupportedLanguage()` and include feature files in pre-commit hook linting
+ 5. Update `@zohodesk/codestandard-analytics` dependency to `1.1.6-node-18`
+ 6. Update unit tests for mutation modules to match refactored API

package/jest.config.js

@@ -1,5 +1,4 @@
 module.exports = {
   testEnvironment: 'node',
-  clearMocks: true,
-  setupFilesAfterEnv: ['<rootDir>/jest.setup.js']
+  clearMocks: true
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zohodesk/codestandard-validator",
-  "version": "1.3.1-platform-14",
+  "version": "1.4.0",
   "description": "library to enforce code standard using eslint",
   "main": "index.js",
   "scripts": {
@@ -21,7 +21,7 @@
     "@stryker-mutator/jest-runner": "8.7.1",
     "@stryker-mutator/typescript-checker": "8.7.1",
     "@zohodesk-private/client_deployment_tool": "0.0.5",
-    "@zohodesk/codestandard-analytics": "1.1.5-node-14",
+    "@zohodesk/codestandard-analytics": "1.2.0",
     "eslint": "8.26.0",
     "glob": "8.1.0",
     "jest": "29.7.0",