x-fidelity 3.15.0 → 3.17.0

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+# [3.17.0](https://github.com/zotoio/x-fidelity/compare/v3.16.0...v3.17.0) (2025-03-20)
+
+
+### Features
+
+* **canary:** example with proxy support for canary installs in ci ([6b5aff3](https://github.com/zotoio/x-fidelity/commit/6b5aff317d0d4fedfccd6cf79304722e8e7a7556))
+
+# [3.16.0](https://github.com/zotoio/x-fidelity/compare/v3.15.0...v3.16.0) (2025-03-19)
+
+
+### Features
+
+* add support for disabling colored logging via XFI_LOG_COLOR env var ([cafcf5b](https://github.com/zotoio/x-fidelity/commit/cafcf5b6fde9b29dbe31a62cba491ab42c24b759))
+
 # [3.15.0](https://github.com/zotoio/x-fidelity/compare/v3.14.0...v3.15.0) (2025-03-12)
 
 

package/README.md

@@ -50,6 +50,7 @@ x-fidelity is an advanced CLI tool and paired config server designed to perform
    - [Basic Usage](#basic-usage)
    - [Advanced Usage](#advanced-usage)
    - [Environment Variables](#environment-variables)
+   - [Logging Options](#logging-options)
    - [Local Configuration](#local-configuration)
    - [Remote Configuration](#remote-configuration)
 8. [Hosting Config Servers](#hosting-config-servers)
@@ -475,6 +476,7 @@ x-fidelity supports the following environment variables:
 - `CERT_PATH`: The path to SSL certificates for HTTPS config server.
 - `NODE_TLS_REJECT_UNAUTHORIZED`: Set to '0' to allow self-signed certificates (use with caution).
 - `XFI_SHARED_SECRET`: Shared secret for securing telemetry and certain server routes.
+- `XFI_LOG_COLOR`: Set to 'false' to disable colored output in logs.
 
 Example usage:
 
@@ -483,6 +485,7 @@ export OPENAI_API_KEY=your_api_key_here
 export OPENAI_MODEL=gpt-4
 export XFI_LISTEN_PORT=9999
 export XFI_SHARED_SECRET=your_shared_secret_here
+export XFI_LOG_COLOR=false
 xfidelity -o true
 ```
 
@@ -500,6 +503,23 @@ The local config directory should contain:
 
 You can override default archetypes or add new ones by placing the corresponding JSON files in the local config directory.
 
+### Logging Options
+
+x-fidelity provides options to customize the logging output:
+
+1. **Log Level**: Set the logging level using the `XFI_LOG_LEVEL` environment variable:
+   ```sh
+   export XFI_LOG_LEVEL=debug
+   ```
+   Available levels: trace, debug, info, warn, error, fatal
+
+2. **Colored Output**: You can disable colored logging output using an environment variable:
+   ```sh
+   export XFI_LOG_COLOR=false
+   ```
+
+This is particularly useful in CI/CD environments or when redirecting logs to files.
+
 ### Remote Configuration
 
 To use a remote configuration server, use the `-c` or `--configServer` option:

package/{canary-install-example.sh → canary-install/canary-install-example.sh}

@@ -3,10 +3,8 @@
 # This script uses the LaunchDarkly Node SDK to evaluate a feature flag
 # and conditionally run a command based on the flag’s value.
 #
-# Requirements:
-#   - Node.js installed.
-#   - LaunchDarkly Node SDK installed:
-#       yarn global add launchdarkly-node-client-sdk
+# global-agent is used to make the launchdarkly connection via proxy set in environment
+# to run a local test proxy: DEBUG=* npx proxy --port 3128
 #
 # Set your LaunchDarkly clientid and feature flag key here.
 # note that the clientid is not a secret, as it is public in the client-side SDKs
@@ -23,7 +21,9 @@ echo "XFI_LD_CLIENT_ID: $XFI_LD_CLIENT_ID"
 echo "XFI_VERSION_FLAG_KEY: $XFI_VERSION_FLAG_KEY"
 
 # Run a Node.js script to evaluate the flag.
-FLAG_VALUE=$(node ./flagCheck.js $XFI_CONTEXT_KEY)
+export GLOBAL_AGENT_ENVIRONMENT_VARIABLE_NAMESPACE=
+FLAG_VALUE=$(yarn add -s global-agent launchdarkly-node-client-sdk --ignore-engines \
+  && node ./flagCheck.js $XFI_CONTEXT_KEY)
 echo "FLAG_VALUE: $FLAG_VALUE"
 
 # Check if the Node.js command succeeded.
@@ -34,11 +34,11 @@ fi
 
 # Decide which command to run based on the flag value.
 if [ "$FLAG_VALUE" == "true" ]; then
-  echo "Installing the NEW version of xfi"
-  # add your canary CI install command here
-  # eg. yarn global add x-fidelity@3.9.1 --ignore-engines
+  export XFI_VERSION=$(cat ./xfi-version-new.txt)
+  echo "Installing the NEW version of xfi $XFI_VERSION"
+  yarn global add x-fidelity@$XFI_VERSION --ignore-engines
 else
-  echo "Installing the current version of xfi"
-  # add your current CI install command here
-  # eg. yarn global add x-fidelity@2.17.2 --ignore-engines
+  export XFI_VERSION=$(cat ./xfi-version-current.txt)
+  echo "Installing the current version of xfi $XFI_VERSION"
+  yarn global add x-fidelity@$XFI_VERSION --ignore-engines
 fi

package/{flagCheck.js → canary-install/flagCheck.js}

@@ -1,3 +1,5 @@
+
+const bootstrap = require('global-agent/bootstrap');
 const ldclient = require('launchdarkly-node-client-sdk');
 
 (async function main(contextKey) {

package/canary-install/xfi-version-current.txt

@@ -0,0 +1 @@
+2.17.2

package/canary-install/xfi-version-new.txt

@@ -0,0 +1 @@
+3.16.0

package/dist/demoConfig/rules/functionComplexity-iterative-rule.json

@@ -17,7 +17,7 @@
                         "cognitiveComplexity": 30,
                         "nestingDepth": 10,
                         "parameterCount": 5,
-                        "returnCount": 5
+                        "returnCount": 10
                     }
                 },
                 "operator": "astComplexity",
@@ -26,7 +26,7 @@
                     "cognitiveComplexity": 30,
                     "nestingDepth": 10,
                     "parameterCount": 5,
-                    "returnCount": 5
+                    "returnCount": 10
                 }
             }
         ]

package/dist/utils/logger.js

@@ -38,6 +38,8 @@ function getLogPrefix() {
 }
 // Initialize logger function that will create the singleton if it doesn't exist
 function getLogger(force) {
+    // Determine if color should be enabled based on environment variable only
+    const useColor = process.env.XFI_LOG_COLOR !== 'false';
     if (!loggerInstance || force) {
         const fileTransport = pino_1.default.destination({
             dest: 'x-fidelity.log',
@@ -49,7 +51,7 @@ function getLogger(force) {
             options: {
                 loglevel: loglevel,
                 sync: false,
-                colorize: true,
+                colorize: useColor,
                 translateTime: 'SYS:yyyy-mm-dd HH:MM:ss.l o',
                 ignore: 'pid,hostname',
                 singleLine: true,

package/dist/utils/logger.test.js

@@ -2,6 +2,16 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const logger_1 = require("./logger");
 describe('Logger utilities', () => {
+    // Save original environment
+    const originalEnv = process.env;
+    beforeEach(() => {
+        // Reset environment before each test
+        process.env = Object.assign({}, originalEnv);
+    });
+    afterAll(() => {
+        // Restore original environment
+        process.env = originalEnv;
+    });
     describe('generateLogPrefix', () => {
         it('should generate a string of length 8', () => {
             const prefix = (0, logger_1.generateLogPrefix)();
@@ -29,4 +39,13 @@ describe('Logger utilities', () => {
             expect(newPrefix.length).toBe(8);
         });
     });
+    describe('logger color configuration', () => {
+        it('should respect XFI_LOG_COLOR environment variable', () => {
+            process.env.XFI_LOG_COLOR = 'false';
+            (0, logger_1.resetLogger)();
+            // This test doesn't directly assert the colorize option
+            // as it's internal to pino, but ensures the code path is executed
+            expect(process.env.XFI_LOG_COLOR).toBe('false');
+        });
+    });
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "x-fidelity",
-  "version": "3.15.0",
+  "version": "3.17.0",
   "description": "cli for opinionated framework adherence checks",
   "main": "dist/index",
   "types": "dist/index.d.ts",
@@ -88,6 +88,7 @@
     "esprima": "^4.0.1",
     "express": "^4.21.2",
     "express-rate-limit": "^7.5.0",
+    "global-agent": "^3.0.0",
     "helmet": "^8.0.0",
     "json-rules-engine": "^7.3.1",
     "jsonpath-plus": "^10.3.0",

package/src/demoConfig/rules/functionComplexity-iterative-rule.json

@@ -17,7 +17,7 @@
                         "cognitiveComplexity": 30,
                         "nestingDepth": 10,
                         "parameterCount": 5,
-                        "returnCount": 5
+                        "returnCount": 10
                     }
                 },
                 "operator": "astComplexity",
@@ -26,7 +26,7 @@
                     "cognitiveComplexity": 30,
                     "nestingDepth": 10,
                     "parameterCount": 5,
-                    "returnCount": 5
+                    "returnCount": 10
                 }
             }
         ]

package/src/plugins/xfiPluginAst/sampleRules/functionComplexity-iterative-rule.json

@@ -17,7 +17,7 @@
                         "cognitiveComplexity": 30,
                         "nestingDepth": 10,
                         "parameterCount": 5,
-                        "returnCount": 5
+                        "returnCount": 10
                     }
                 },
                 "operator": "astComplexity",
@@ -26,7 +26,7 @@
                     "cognitiveComplexity": 30,
                     "nestingDepth": 10,
                     "parameterCount": 5,
-                    "returnCount": 5
+                    "returnCount": 10
                 }
             }
         ]

package/src/utils/logger.test.ts

@@ -1,6 +1,18 @@
-import { generateLogPrefix, setLogPrefix, getLogPrefix, resetLogPrefix } from './logger';
+import { generateLogPrefix, setLogPrefix, getLogPrefix, resetLogPrefix, resetLogger } from './logger';
 
 describe('Logger utilities', () => {
+    // Save original environment
+    const originalEnv = process.env;
+    
+    beforeEach(() => {
+        // Reset environment before each test
+        process.env = { ...originalEnv };
+    });
+    
+    afterAll(() => {
+        // Restore original environment
+        process.env = originalEnv;
+    });
     describe('generateLogPrefix', () => {
         it('should generate a string of length 8', () => {
             const prefix = generateLogPrefix();
@@ -31,4 +43,14 @@ describe('Logger utilities', () => {
             expect(newPrefix.length).toBe(8);
         });
     });
+    
+    describe('logger color configuration', () => {
+        it('should respect XFI_LOG_COLOR environment variable', () => {
+            process.env.XFI_LOG_COLOR = 'false';
+            resetLogger();
+            // This test doesn't directly assert the colorize option
+            // as it's internal to pino, but ensures the code path is executed
+            expect(process.env.XFI_LOG_COLOR).toBe('false');
+        });
+    });
 });

package/src/utils/logger.ts

@@ -1,6 +1,7 @@
 import { randomUUID } from 'crypto';
 import pino from 'pino';
 import { maskSensitiveData } from './maskSensitiveData';
+import { options } from '../core/cli';
 
 // Initialize variables
 let loggerInstance: pino.Logger | undefined;
@@ -32,6 +33,9 @@ export function getLogPrefix(): string {
 
 // Initialize logger function that will create the singleton if it doesn't exist
 function getLogger(force?: boolean): pino.Logger {
+    // Determine if color should be enabled based on environment variable only
+    const useColor = process.env.XFI_LOG_COLOR !== 'false';
+    
     if (!loggerInstance || force) {
         const fileTransport = pino.destination({
             dest: 'x-fidelity.log',
@@ -44,7 +48,7 @@ function getLogger(force?: boolean): pino.Logger {
             options: {
                 loglevel: loglevel,
                 sync: false,
-                colorize: true,
+                colorize: useColor,
                 translateTime: 'SYS:yyyy-mm-dd HH:MM:ss.l o',
                 ignore: 'pid,hostname',
                 singleLine: true,

package/website/docs/environment-variables.md

@@ -42,8 +42,15 @@ GITHUB_WEBHOOK_SECRET=your_github_webhook_secret
 ```bash
 # Set logging level (default: 'info')
 XFI_LOG_LEVEL=debug
+
+# Disable colored output in logs (default: enabled)
+XFI_LOG_COLOR=false
 ```
 
+The `XFI_LOG_COLOR` environment variable controls whether log output includes ANSI color codes. This is particularly useful in CI/CD environments, when redirecting logs to files, or when your terminal doesn't support colors.
+
+Setting `XFI_LOG_COLOR=false` will disable all colored output in logs, making them plain text. This can improve readability in environments where color codes might appear as strange characters.
+
 ## Usage Examples
 
 ### Basic Setup

package/website/docs/getting-started.md

@@ -62,6 +62,7 @@ Options:
 - `CERT_PATH`: SSL certificate path
 - `NODE_TLS_REJECT_UNAUTHORIZED`: Allow self-signed certs
 - `XFI_SHARED_SECRET`: Shared secret for security
+- `XFI_LOG_COLOR`: Set to 'false' to disable colored output in logs
 - `NOTIFICATIONS_ENABLED`: Enable notification system
 - `NOTIFICATION_PROVIDERS`: Comma-separated list of notification providers to use
 - `CODEOWNERS_PATH`: Path to CODEOWNERS file (default: .github/CODEOWNERS)