x-fidelity 3.13.0 → 3.13.1

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [3.13.1](https://github.com/zotoio/x-fidelity/compare/v3.13.0...v3.13.1) (2025-03-11)
+
+
+### Bug Fixes
+
+* **exemptions:** trace logging ([343744c](https://github.com/zotoio/x-fidelity/commit/343744c161d2756d9fa5e419426d0b4f1a37d1d4))
+
 # [3.13.0](https://github.com/zotoio/x-fidelity/compare/v3.12.1...v3.13.0) (2025-03-10)
 
 

package/dist/core/configManager.test.js

@@ -118,7 +118,8 @@ jest.mock('../utils/logger', () => ({
         debug: jest.fn(),
         error: jest.fn(),
         info: jest.fn(),
-        warn: jest.fn()
+        warn: jest.fn(),
+        trace: jest.fn()
     },
     setLogPrefix: jest.fn()
 }));

package/dist/core/engine/engineRunner.test.js

@@ -19,6 +19,7 @@ jest.mock('../../utils/logger', () => ({
         debug: jest.fn(),
         error: jest.fn(),
         warn: jest.fn(),
+        trace: jest.fn()
     },
 }));
 describe('runEngineOnFiles', () => {

package/dist/core/pluginRegistry.test.js

@@ -8,6 +8,7 @@ jest.mock('../utils/logger', () => ({
         warn: jest.fn(),
         error: jest.fn(),
         debug: jest.fn(),
+        trace: jest.fn()
     },
 }));
 describe('XFiPluginRegistry', () => {

package/dist/demoConfig/node-fullstack-exemptions/project1-node-fullstack-exemptions.json

@@ -1,8 +1,8 @@
 [
     {
       "repoUrl": "git@github.com:zotoio/x-fidelity.git",
-      "rule": "outdatedFramework-global",
-      "expirationDate": "2023-12-31",
+      "rule": "noDatabases-iterative",
+      "expirationDate": "2026-01-04",
       "reason": "Upgrading dependencies is scheduled for Q4 2024"
     },
     {

package/dist/facts/globalFileAnalysisFacts.test.js

@@ -17,6 +17,7 @@ jest.mock('../utils/logger', () => ({
         error: jest.fn(),
         info: jest.fn(),
         warn: jest.fn(),
+        trace: jest.fn()
     },
 }));
 describe('globalFileAnalysis', () => {

package/dist/facts/index.test.js

@@ -27,7 +27,9 @@ jest.mock('../utils/logger', () => ({
     logger: {
         info: jest.fn(),
         debug: jest.fn(),
-        warn: jest.fn()
+        warn: jest.fn(),
+        trace: jest.fn(),
+        error: jest.fn()
     }
 }));
 describe('loadFacts', () => {

package/dist/facts/openaiAnalysisFacts.test.js

@@ -22,6 +22,7 @@ jest.mock('../utils/logger', () => ({
         error: jest.fn(),
         info: jest.fn(),
         warn: jest.fn(),
+        trace: jest.fn()
     },
 }));
 jest.mock('openai', () => {

package/dist/operators/fileContains.test.js

@@ -5,7 +5,10 @@ const fileContains_1 = require("./fileContains");
 jest.mock('../utils/logger', () => ({
     logger: {
         debug: jest.fn(),
-        error: jest.fn()
+        error: jest.fn(),
+        trace: jest.fn(),
+        info: jest.fn(),
+        warn: jest.fn()
     },
 }));
 describe('fileContains', () => {

package/dist/operators/globalPatternCount.test.js

@@ -6,6 +6,8 @@ jest.mock('../utils/logger', () => ({
         debug: jest.fn(),
         error: jest.fn(),
         info: jest.fn(),
+        trace: jest.fn(),
+        warn: jest.fn()
     },
 }));
 describe('globalPatternCount', () => {

package/dist/operators/globalPatternRatio.test.js

@@ -6,6 +6,8 @@ jest.mock('../utils/logger', () => ({
         debug: jest.fn(),
         error: jest.fn(),
         info: jest.fn(),
+        trace: jest.fn(),
+        warn: jest.fn()
     },
 }));
 describe('globalPatternRatio', () => {

package/dist/operators/openaiAnalysisHighSeverity.test.js

@@ -7,6 +7,8 @@ jest.mock('../utils/logger', () => ({
         error: jest.fn(),
         debug: jest.fn(),
         info: jest.fn(),
+        trace: jest.fn(),
+        warn: jest.fn()
     },
 }));
 describe('openaiAnalysisHighSeverity', () => {

package/dist/operators/regexMatch.test.js

@@ -5,7 +5,10 @@ const logger_1 = require("../utils/logger");
 jest.mock('../utils/logger', () => ({
     logger: {
         debug: jest.fn(),
-        error: jest.fn()
+        error: jest.fn(),
+        trace: jest.fn(),
+        info: jest.fn(),
+        warn: jest.fn()
     },
 }));
 describe('regexMatch', () => {

package/dist/plugins/xfiPluginRemoteStringValidator/facts/remoteSubstringValidation.test.js

@@ -16,6 +16,8 @@ jest.mock('../../../utils/logger', () => ({
         debug: jest.fn(),
         error: jest.fn(),
         info: jest.fn(),
+        trace: jest.fn(),
+        warn: jest.fn()
     },
 }));
 describe('remoteSubstringValidation', () => {

package/dist/plugins/xfiPluginRemoteStringValidator/operators/invalidRemoteValidation.test.js

@@ -6,6 +6,8 @@ jest.mock('../../../utils/logger', () => ({
         debug: jest.fn(),
         error: jest.fn(),
         info: jest.fn(),
+        trace: jest.fn(),
+        warn: jest.fn()
     },
 }));
 describe('invalidRemoteValidation', () => {

package/dist/plugins/xfiPluginRequiredFiles/operators/missingRequiredFiles.test.js

@@ -5,7 +5,10 @@ const logger_1 = require("../../../utils/logger");
 jest.mock('../../../utils/logger', () => ({
     logger: {
         debug: jest.fn(),
-        error: jest.fn()
+        error: jest.fn(),
+        trace: jest.fn(),
+        info: jest.fn(),
+        warn: jest.fn()
     },
 }));
 describe('missingRequiredFiles', () => {

package/dist/server/cacheManager.test.js

@@ -9,7 +9,8 @@ jest.mock('../utils/logger', () => ({
         debug: jest.fn(),
         info: jest.fn(),
         error: jest.fn(),
-        warn: jest.fn()
+        warn: jest.fn(),
+        trace: jest.fn()
     }
 }));
 describe('Cache Manager', () => {

package/dist/utils/exemptionUtils.js

@@ -166,11 +166,22 @@ function isExempt(params) {
         }
         const now = new Date();
         const normalizedRepoUrl = normalizeGitHubUrl(repoUrl);
-        const exemption = exemptions.find(exemption => normalizeGitHubUrl(exemption.repoUrl) === normalizedRepoUrl &&
-            exemption.rule === ruleName &&
-            new Date(exemption.expirationDate) > now);
+        logger_1.logger.trace(`Checking exemption for rule ${ruleName} in repo ${repoUrl}`);
+        const exemption = exemptions.find(exemption => {
+            logger_1.logger.trace(JSON.stringify(exemption));
+            logger_1.logger.trace(`repoUrl: ${repoUrl}`);
+            logger_1.logger.trace(`ruleName: ${ruleName}`);
+            logger_1.logger.trace(`normalizedRepoUrl: ${normalizedRepoUrl}`);
+            logger_1.logger.trace(`now: ${now}`);
+            const result = normalizeGitHubUrl(exemption.repoUrl) === normalizedRepoUrl &&
+                exemption.rule === ruleName &&
+                new Date(exemption.expirationDate) > now;
+            logger_1.logger.trace(`Exemption check result: ${result}`);
+            return result;
+        });
+        logger_1.logger.trace(`Exemption: ${exemption}`);
         if (exemption) {
-            logger_1.logger.error(`Exempting rule ${ruleName} for repo ${repoUrl} until ${exemption.expirationDate}`);
+            logger_1.logger.error(`Exempting rule ${ruleName} for repo ${normalizedRepoUrl}: ${exemption.expirationDate}`);
             // Send telemetry event for the allowed exemption
             (0, telemetry_1.sendTelemetry)({
                 eventType: 'exemptionAllowed',

package/dist/utils/exemptionUtils.test.js

@@ -29,6 +29,7 @@ jest.mock('./logger', () => ({
         warn: jest.fn(),
         error: jest.fn(),
         debug: jest.fn(),
+        trace: jest.fn(),
     },
 }));
 describe('normalizeGitHubUrl', () => {

package/dist/utils/ruleUtils.test.js

@@ -39,6 +39,7 @@ jest.mock('./logger', () => ({
         error: jest.fn(),
         info: jest.fn(),
         warn: jest.fn(),
+        trace: jest.fn(),
     },
 }));
 describe('loadRules', () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "x-fidelity",
-  "version": "3.13.0",
+  "version": "3.13.1",
   "description": "cli for opinionated framework adherence checks",
   "main": "dist/index",
   "types": "dist/index.d.ts",

package/src/core/configManager.test.ts

@@ -81,7 +81,8 @@ jest.mock('../utils/logger', () => ({
         debug: jest.fn(),
         error: jest.fn(),
         info: jest.fn(),
-        warn: jest.fn()
+        warn: jest.fn(),
+        trace: jest.fn()
     },
     setLogPrefix: jest.fn()
 }));

package/src/core/engine/engineRunner.test.ts

@@ -10,6 +10,7 @@ jest.mock('../../utils/logger', () => ({
         debug: jest.fn(),
         error: jest.fn(),
         warn: jest.fn(),
+        trace: jest.fn()
     },
 }));
 

package/src/core/pluginRegistry.test.ts

@@ -8,6 +8,7 @@ jest.mock('../utils/logger', () => ({
     warn: jest.fn(),
     error: jest.fn(),
     debug: jest.fn(),
+    trace: jest.fn()
   },
 }));
 

package/src/demoConfig/node-fullstack-exemptions/project1-node-fullstack-exemptions.json

@@ -1,8 +1,8 @@
 [
     {
       "repoUrl": "git@github.com:zotoio/x-fidelity.git",
-      "rule": "outdatedFramework-global",
-      "expirationDate": "2023-12-31",
+      "rule": "noDatabases-iterative",
+      "expirationDate": "2026-01-04",
       "reason": "Upgrading dependencies is scheduled for Q4 2024"
     },
     {

package/src/exampleTriggerFiles/mixedUIComponents.tsx

@@ -1,11 +1,11 @@
-import { Button } from 'antd';
-import * as React from 'react';
-import Checkbox from '@mui/material/Checkbox';
+// import { Button } from 'antd';
+// import * as React from 'react';
+// import Checkbox from '@mui/material/Checkbox';
 
-export default function ButtonUsage() {
-  return <Button variant="contained">Hello world</Button>;
-}
+// export default function ButtonUsage() {
+//   return <Button variant="contained">Hello world</Button>;
+// }
 
-export default function CheckboxUsage() {
-  return <Checkbox type="primary">Click Me</Checkbox>;
-}
+// export default function CheckboxUsage() {
+//   return <Checkbox type="primary">Click Me</Checkbox>;
+// }

package/src/facts/globalFileAnalysisFacts.test.ts

@@ -7,6 +7,7 @@ jest.mock('../utils/logger', () => ({
         error: jest.fn(),
         info: jest.fn(),
         warn: jest.fn(),
+        trace: jest.fn()
     },
 }));
 

package/src/facts/index.test.ts

@@ -21,7 +21,9 @@ jest.mock('../utils/logger', () => ({
   logger: {
     info: jest.fn(),
     debug: jest.fn(),
-    warn: jest.fn()
+    warn: jest.fn(),
+    trace: jest.fn(),
+    error: jest.fn()
   }
 }));
 

package/src/facts/openaiAnalysisFacts.test.ts

@@ -14,6 +14,7 @@ jest.mock('../utils/logger', () => ({
         error: jest.fn(),
         info: jest.fn(),
         warn: jest.fn(),
+        trace: jest.fn()
     },
 }));
 

package/src/operators/fileContains.test.ts

@@ -4,7 +4,10 @@ import { fileContains } from './fileContains';
 jest.mock('../utils/logger', () => ({
     logger: {
         debug: jest.fn(),
-        error: jest.fn()
+        error: jest.fn(),
+        trace: jest.fn(),
+        info: jest.fn(),
+        warn: jest.fn()
     },
 }));
 

package/src/operators/globalPatternCount.test.ts

@@ -6,6 +6,8 @@ jest.mock('../utils/logger', () => ({
         debug: jest.fn(),
         error: jest.fn(),
         info: jest.fn(),
+        trace: jest.fn(),
+        warn: jest.fn()
     },
 }));
 

package/src/operators/globalPatternRatio.test.ts

@@ -6,6 +6,8 @@ jest.mock('../utils/logger', () => ({
         debug: jest.fn(),
         error: jest.fn(),
         info: jest.fn(),
+        trace: jest.fn(),
+        warn: jest.fn()
     },
 }));
 

package/src/operators/openaiAnalysisHighSeverity.test.ts

@@ -6,6 +6,8 @@ jest.mock('../utils/logger', () => ({
         error: jest.fn(),
         debug: jest.fn(),
         info: jest.fn(),
+        trace: jest.fn(),
+        warn: jest.fn()
     },
 }));
 

package/src/operators/regexMatch.test.ts

@@ -4,7 +4,10 @@ import { logger } from '../utils/logger';
 jest.mock('../utils/logger', () => ({
     logger: {
         debug: jest.fn(),
-        error: jest.fn()
+        error: jest.fn(),
+        trace: jest.fn(),
+        info: jest.fn(),
+        warn: jest.fn()
     },
 }));
 

package/src/plugins/xfiPluginRemoteStringValidator/facts/remoteSubstringValidation.test.ts

@@ -6,6 +6,8 @@ jest.mock('../../../utils/logger', () => ({
         debug: jest.fn(),
         error: jest.fn(),
         info: jest.fn(),
+        trace: jest.fn(),
+        warn: jest.fn()
     },
 }));
 

package/src/plugins/xfiPluginRemoteStringValidator/operators/invalidRemoteValidation.test.ts

@@ -6,6 +6,8 @@ jest.mock('../../../utils/logger', () => ({
         debug: jest.fn(),
         error: jest.fn(),
         info: jest.fn(),
+        trace: jest.fn(),
+        warn: jest.fn()
     },
 }));
 

package/src/plugins/xfiPluginRequiredFiles/operators/missingRequiredFiles.test.ts

@@ -4,7 +4,10 @@ import { logger } from '../../../utils/logger';
 jest.mock('../../../utils/logger', () => ({
     logger: {
         debug: jest.fn(),
-        error: jest.fn()
+        error: jest.fn(),
+        trace: jest.fn(),
+        info: jest.fn(),
+        warn: jest.fn()
     },
 }));
 

package/src/server/cacheManager.test.ts

@@ -8,7 +8,8 @@ jest.mock('../utils/logger', () => ({
         debug: jest.fn(),
         info: jest.fn(),
         error: jest.fn(),
-        warn: jest.fn()
+        warn: jest.fn(),
+        trace: jest.fn()
     }
 }));
 

package/src/utils/exemptionUtils.test.ts

@@ -18,6 +18,7 @@ jest.mock('./logger', () => ({
         warn: jest.fn(),
         error: jest.fn(),
         debug: jest.fn(),
+        trace: jest.fn(),
     },
 }));
 

package/src/utils/exemptionUtils.ts

@@ -155,13 +155,31 @@ export function isExempt(params: IsExemptParams): boolean {
         }
         const now = new Date();
         const normalizedRepoUrl = normalizeGitHubUrl(repoUrl);
-        const exemption = exemptions.find(exemption =>
-            normalizeGitHubUrl(exemption.repoUrl) === normalizedRepoUrl &&
+    
+        logger.trace(`Checking exemption for rule ${ruleName} in repo ${repoUrl}`);
+
+        const exemption = exemptions.find(exemption => {
+
+            logger.trace(JSON.stringify(exemption));
+            logger.trace(`repoUrl: ${repoUrl}`);
+            logger.trace(`ruleName: ${ruleName}`);
+            logger.trace(`normalizedRepoUrl: ${normalizedRepoUrl}`);
+            logger.trace(`now: ${now}`);
+            
+            const result = normalizeGitHubUrl(exemption.repoUrl) === normalizedRepoUrl &&
             exemption.rule === ruleName &&
-            new Date(exemption.expirationDate) > now
-        );
+            new Date(exemption.expirationDate) > now;
+
+            logger.trace(`Exemption check result: ${result}`);
+
+            return result;
+
+        });
+
+        logger.trace(`Exemption: ${exemption}`);
+
         if (exemption) {
-            logger.error(`Exempting rule ${ruleName} for repo ${repoUrl} until ${exemption.expirationDate}`);
+            logger.error(`Exempting rule ${ruleName} for repo ${normalizedRepoUrl}: ${exemption.expirationDate}`);
 
             // Send telemetry event for the allowed exemption
             sendTelemetry({

package/src/utils/ruleUtils.test.ts

@@ -27,6 +27,7 @@ jest.mock('./logger', () => ({
         error: jest.fn(),
         info: jest.fn(),
         warn: jest.fn(),
+        trace: jest.fn(),
     },
 }));