x-fidelity 3.0.2 → 3.0.4

package/CHANGELOG.md

@@ -1,3 +1,20 @@
+## [3.0.4](https://github.com/zotoio/x-fidelity/compare/v3.0.3...v3.0.4) (2025-02-19)
+
+
+### Bug Fixes
+
+* **plugins:** ensure logger instance reused and fix entrypoint logic ([c21ad6e](https://github.com/zotoio/x-fidelity/commit/c21ad6e764e18f7910170ed970492208fe4197e8))
+
+## [3.0.3](https://github.com/zotoio/x-fidelity/compare/v3.0.2...v3.0.3) (2025-02-18)
+
+
+### Bug Fixes
+
+* **error handling:** deal with unexpected errors globally ensuring async ops finish ([2292d4d](https://github.com/zotoio/x-fidelity/commit/2292d4d09ef75a419778d3df77381a1353217361))
+* improve error handling and telemetry logging ([f7fa9ca](https://github.com/zotoio/x-fidelity/commit/f7fa9ca6b63bcf75ebab8dbc710eb78fc718cefe))
+* normalize event type casing in telemetry test ([9b791f9](https://github.com/zotoio/x-fidelity/commit/9b791f969816f29efa6138b9f95235c6614a3ac7))
+* standardize event type casing to lowercase in error handling ([0416cca](https://github.com/zotoio/x-fidelity/commit/0416ccae4f796ee1a3404f65545a00d67e26bbc7))
+
 ## [3.0.2](https://github.com/zotoio/x-fidelity/compare/v3.0.1...v3.0.2) (2025-02-15)
 
 

package/dist/core/cli.d.ts

@@ -1,3 +1,3 @@
 export declare const DEMO_CONFIG_PATH: string;
-declare const options: import("commander").OptionValues;
-export { options };
+export declare const options: import("commander").OptionValues;
+export declare function initCLI(): void;

package/dist/core/cli.js

@@ -4,6 +4,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.options = exports.DEMO_CONFIG_PATH = void 0;
+exports.initCLI = initCLI;
 const logger_1 = require("../utils/logger");
 const commander_1 = require("commander");
 const path_1 = __importDefault(require("path"));
@@ -11,85 +12,85 @@ const fs_1 = __importDefault(require("fs"));
 const package_json_1 = require("../../package.json");
 const inputValidation_1 = require("../utils/inputValidation");
 const prettyjson_1 = __importDefault(require("prettyjson"));
-// Ensure logger is initialized
-if (!logger_1.logger || typeof logger_1.logger.info !== 'function') {
-    console.error({ msg: 'Logger is not properly initialized' });
-    // Instead of exiting, we'll create a fallback logger
-    const fallbackLogger = {
-        info: (obj) => console.log(JSON.stringify(obj)),
-        error: (obj) => console.error(JSON.stringify(obj)),
-        warn: (obj) => console.warn(JSON.stringify(obj)),
-        debug: (obj) => console.debug(JSON.stringify(obj))
-    };
-    global.logger = fallbackLogger;
-}
-logger_1.logger.info('CLI initialized');
 exports.DEMO_CONFIG_PATH = path_1.default.resolve(__dirname, '../demoConfig');
-// we are overiding the default behavior of commander to exit 
-// the process due to https://github.com/pinojs/pino/issues/871
-commander_1.program.exitOverride(() => {
+exports.options = commander_1.program.opts();
+function initCLI() {
+    // Ensure logger is initialized
+    if (!logger_1.logger || typeof logger_1.logger.info !== 'function') {
+        console.error({ msg: 'Logger is not properly initialized' });
+        // Instead of exiting, we'll create a fallback logger
+        const fallbackLogger = {
+            info: (obj) => console.log(JSON.stringify(obj)),
+            error: (obj) => console.error(JSON.stringify(obj)),
+            warn: (obj) => console.warn(JSON.stringify(obj)),
+            debug: (obj) => console.debug(JSON.stringify(obj))
+        };
+        global.logger = fallbackLogger;
+    }
+    logger_1.logger.info('CLI initialized');
+    // we are overiding the default behavior of commander to exit 
+    // the process due to https://github.com/pinojs/pino/issues/871
+    commander_1.program.exitOverride(() => {
+        try {
+            if (process.env.NODE_ENV !== 'test')
+                process.exit(0);
+        }
+        catch (error) {
+            //swallow
+        }
+    });
+    commander_1.program
+        .option("-d, --dir <directory>", "code directory to analyze. equivalent of directory argument")
+        .option("-a, --archetype <archetype>", "The archetype to use for analysis", "node-fullstack")
+        .option("-c, --configServer <configServer>", "The config server URL for fetching remote archetype configurations and rules. This takes precedence over localConfigPath.")
+        .option("-o, --openaiEnabled <boolean>", "Enable OpenAI analysis", false)
+        .option("-t, --telemetryCollector <telemetryCollector>", "The URL telemetry data will be sent to for usage analysis")
+        .option("-m, --mode <mode>", "Run mode: 'client' or 'server'", "client")
+        .option("-p, --port <port>", "The port to run the server on", "8888")
+        .option("-l, --localConfigPath <path>", "Path to local archetype config and rules", exports.DEMO_CONFIG_PATH)
+        .option("-j, --jsonTTL <minutes>", "Set the server json cache TTL in minutes", "10")
+        .option("-e, --extensions <modules...>", "Space-separated list of npm module names to load as extensions")
+        .option("-x, --examine <archetype>", "Examine the archetype configuration and rules")
+        .version(package_json_1.version, "-v, --version", "Output the version number of xfidelity")
+        .helpOption("-h, --help", "Display help for command")
+        .argument('[directory]', 'code directory to analyze');
+    commander_1.program.parse(process.argv);
+    // Resolve paths
+    const resolvePath = (inputPath) => {
+        const resolvedPath = path_1.default === null || path_1.default === void 0 ? void 0 : path_1.default.resolve(process.cwd(), inputPath);
+        if (!fs_1.default.existsSync(resolvedPath)) {
+            throw new Error(`Path does not exist: ${resolvedPath}`);
+        }
+        if (!(0, inputValidation_1.validateInput)(resolvedPath)) {
+            throw new Error(`Potential malicious input detected: ${inputPath}`);
+        }
+        return resolvedPath;
+    };
+    if (commander_1.program.args.length === 0) {
+        if (process.env.NODE_ENV === 'test' || exports.options.mode === 'server')
+            exports.options.dir = '.';
+        if (!exports.options.dir && process.env.NODE_ENV !== 'test')
+            commander_1.program.help({ error: false });
+    }
+    if (process.env.NODE_ENV === 'test' || exports.options.mode === 'server')
+        exports.options.dir = '.';
     try {
-        if (process.env.NODE_ENV !== 'test')
-            process.exit(0);
+        exports.options.dir = commander_1.program.args.length == 1 && commander_1.program.args[0] !== undefined ? resolvePath(commander_1.program.args[0]) : exports.options.dir && resolvePath(exports.options.dir);
     }
     catch (error) {
-        //swallow
-    }
-});
-commander_1.program
-    .option("-d, --dir <directory>", "code directory to analyze. equivalent of directory argument")
-    .option("-a, --archetype <archetype>", "The archetype to use for analysis", "node-fullstack")
-    .option("-c, --configServer <configServer>", "The config server URL for fetching remote archetype configurations and rules. This takes precedence over localConfigPath.")
-    .option("-o, --openaiEnabled <boolean>", "Enable OpenAI analysis", false)
-    .option("-t, --telemetryCollector <telemetryCollector>", "The URL telemetry data will be sent to for usage analysis")
-    .option("-m, --mode <mode>", "Run mode: 'client' or 'server'", "client")
-    .option("-p, --port <port>", "The port to run the server on", "8888")
-    .option("-l, --localConfigPath <path>", "Path to local archetype config and rules", exports.DEMO_CONFIG_PATH)
-    .option("-j, --jsonTTL <minutes>", "Set the server json cache TTL in minutes", "10")
-    .option("-e, --extensions <modules...>", "Space-separated list of npm module names to load as extensions")
-    .option("-x, --examine <archetype>", "Examine the archetype configuration and rules")
-    .version(package_json_1.version, "-v, --version", "Output the version number of xfidelity")
-    .helpOption("-h, --help", "Display help for command")
-    .argument('[directory]', 'code directory to analyze');
-const options = commander_1.program.opts();
-exports.options = options;
-commander_1.program.parse(process.argv);
-// Resolve paths
-const resolvePath = (inputPath) => {
-    const resolvedPath = path_1.default === null || path_1.default === void 0 ? void 0 : path_1.default.resolve(process.cwd(), inputPath);
-    if (!fs_1.default.existsSync(resolvedPath)) {
-        throw new Error(`Path does not exist: ${resolvedPath}`);
+        if (process.env.NODE_ENV !== 'test')
+            commander_1.program.error(`Error resolving repo path to analyse: ${error}`);
     }
-    if (!(0, inputValidation_1.validateInput)(resolvedPath)) {
-        throw new Error(`Potential malicious input detected: ${inputPath}`);
+    try {
+        if (exports.options.localConfigPath) {
+            exports.options.localConfigPath = resolvePath(exports.options.localConfigPath);
+        }
     }
-    return resolvedPath;
-};
-if (commander_1.program.args.length === 0) {
-    if (process.env.NODE_ENV === 'test' || options.mode === 'server')
-        options.dir = '.';
-    if (!options.dir && process.env.NODE_ENV !== 'test')
-        commander_1.program.help({ error: false });
-}
-if (process.env.NODE_ENV === 'test' || options.mode === 'server')
-    options.dir = '.';
-try {
-    options.dir = commander_1.program.args.length == 1 && commander_1.program.args[0] !== undefined ? resolvePath(commander_1.program.args[0]) : options.dir && resolvePath(options.dir);
-}
-catch (error) {
-    if (process.env.NODE_ENV !== 'test')
-        commander_1.program.error(`Error resolving repo path to analyse: ${error}`);
-}
-try {
-    if (options.localConfigPath) {
-        options.localConfigPath = resolvePath(options.localConfigPath);
+    catch (error) {
+        if (process.env.NODE_ENV !== 'test')
+            commander_1.program.error(`LocalConfigPath does not exist or is invalid: ${error}`);
     }
-}
-catch (error) {
-    if (process.env.NODE_ENV !== 'test')
-        commander_1.program.error(`LocalConfigPath does not exist or is invalid: ${error}`);
-}
-const bannerArt = `\n
+    const bannerArt = `\n
 =====================================
  __    __          ________  ______ 
 | ##  | ##        | ######## \\######
@@ -101,22 +102,23 @@ const bannerArt = `\n
  \\##   \\##         \\##       \\######
 ------------------------------------- 
 `;
-logger_1.logger.info(bannerArt);
-logger_1.logger.info(`\n${prettyjson_1.default.render({
-    version: package_json_1.version,
-    startTime: new Date().toString().slice(0, 24),
-    archetype: options.archetype,
-    directory: options.dir,
-    configServer: options.configServer ? options.configServer : 'none',
-    mode: options.mode,
-    port: options.mode === 'server' ? options.port : 'n/a',
-    localConfigPath: options.localConfigPath ? options.localConfigPath : 'none',
-    jsonTTL: `${options.jsonTTL} minutes`,
-    openaiEnabled: options.openaiEnabled,
-    extensions: options.extensions ? options.extensions : 'none'
-})}
+    logger_1.logger.info(bannerArt);
+    logger_1.logger.info(`\n${prettyjson_1.default.render({
+        version: package_json_1.version,
+        startTime: new Date().toString().slice(0, 24),
+        archetype: exports.options.archetype,
+        directory: exports.options.dir,
+        configServer: exports.options.configServer ? exports.options.configServer : 'none',
+        mode: exports.options.mode,
+        port: exports.options.mode === 'server' ? exports.options.port : 'n/a',
+        localConfigPath: exports.options.localConfigPath ? exports.options.localConfigPath : 'none',
+        jsonTTL: `${exports.options.jsonTTL} minutes`,
+        openaiEnabled: exports.options.openaiEnabled,
+        extensions: exports.options.extensions ? exports.options.extensions : 'none'
+    })}
 -------------------------------------
-`);
-// print help if no arguments are passed
-if (commander_1.program.options.length === 0)
-    commander_1.program.help();
+    `);
+    // print help if no arguments are passed
+    if (commander_1.program.options.length === 0)
+        commander_1.program.help();
+}

package/dist/core/configManager.js

@@ -70,7 +70,10 @@ class ConfigManager {
         return __awaiter(this, void 0, void 0, function* () {
             const { archetype = cli_1.options.archetype, logPrefix } = params;
             if (!ConfigManager.configs[archetype]) {
-                ConfigManager.configs[archetype] = yield ConfigManager.initialize({ archetype, logPrefix });
+                ConfigManager.configs[archetype] = yield ConfigManager.initialize({ archetype, logPrefix }).catch(error => {
+                    logger_1.logger.error(error, `Error initializing config for archetype: ${archetype}`);
+                    throw error;
+                });
             }
             return ConfigManager.configs[archetype];
         });
@@ -118,8 +121,7 @@ class ConfigManager {
                     }
                     catch (error) {
                         logger_1.logger.error(`Failed to load extension ${moduleName} from all locations: ${error}`);
-                        if (process.env.NODE_ENV !== 'test')
-                            process.exit(1);
+                        throw new Error(`Failed to load extension ${moduleName} from all locations: ${error}`);
                     }
                 }
             }
@@ -134,7 +136,10 @@ class ConfigManager {
             if (logPrefix)
                 (0, logger_1.setLogPrefix)(logPrefix);
             // Load plugins during initialization
-            yield this.loadPlugins(cli_1.options.extensions);
+            yield this.loadPlugins(cli_1.options.extensions).catch(error => {
+                logger_1.logger.error(error, `Error loading plugins`);
+                throw error;
+            });
             logger_1.logger.info(`Initializing config manager for archetype: ${archetype}`);
             logger_1.logger.debug(`Initialize params: ${JSON.stringify(params)}`);
             const config = {

package/dist/index.js

@@ -51,11 +51,14 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.main = main;
 const logger_1 = require("./utils/logger");
-(0, logger_1.initializeLogger)();
+const cli_1 = require("./core/cli");
+if (require.main === module) {
+    (0, logger_1.initializeLogger)();
+    (0, logger_1.setLogLevel)(process.env.XFI_LOG_LEVEL || 'info');
+    (0, cli_1.initCLI)();
+}
 const executionLogPrefix = (0, logger_1.getLogPrefix)();
-(0, logger_1.setLogLevel)(process.env.XFI_LOG_LEVEL || 'info');
 const prettyjson_1 = __importDefault(require("prettyjson"));
-const cli_1 = require("./core/cli");
 const analyzer_1 = require("./core/engine/analyzer");
 const configServer_1 = require("./server/configServer");
 const telemetry_1 = require("./utils/telemetry");
@@ -71,7 +74,7 @@ const handleError = (error) => __awaiter(void 0, void 0, void 0, function* () {
         },
         timestamp: new Date().toISOString()
     }, executionLogPrefix);
-    logger_1.logger.error(JSON.stringify(error));
+    logger_1.logger.error(error, 'Execution failure');
 });
 const outcomeMessage = (message) => `\n
 ==========================================================================
@@ -129,7 +132,14 @@ function main() {
             }
         }
         catch (e) {
-            yield handleError(e);
+            yield handleError(e).then(() => {
+                // give some time async ops to finish if not handled directly
+                if (process.env.NODE_ENV !== 'test') {
+                    setTimeout(() => {
+                        process.exit(1);
+                    }, 3000);
+                }
+            });
         }
     });
 }

package/dist/xfidelity

@@ -51,11 +51,14 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.main = main;
 const logger_1 = require("./utils/logger");
-(0, logger_1.initializeLogger)();
+const cli_1 = require("./core/cli");
+if (require.main === module) {
+    (0, logger_1.initializeLogger)();
+    (0, logger_1.setLogLevel)(process.env.XFI_LOG_LEVEL || 'info');
+    (0, cli_1.initCLI)();
+}
 const executionLogPrefix = (0, logger_1.getLogPrefix)();
-(0, logger_1.setLogLevel)(process.env.XFI_LOG_LEVEL || 'info');
 const prettyjson_1 = __importDefault(require("prettyjson"));
-const cli_1 = require("./core/cli");
 const analyzer_1 = require("./core/engine/analyzer");
 const configServer_1 = require("./server/configServer");
 const telemetry_1 = require("./utils/telemetry");
@@ -71,7 +74,7 @@ const handleError = (error) => __awaiter(void 0, void 0, void 0, function* () {
         },
         timestamp: new Date().toISOString()
     }, executionLogPrefix);
-    logger_1.logger.error(JSON.stringify(error));
+    logger_1.logger.error(error, 'Execution failure');
 });
 const outcomeMessage = (message) => `\n
 ==========================================================================
@@ -129,7 +132,14 @@ function main() {
             }
         }
         catch (e) {
-            yield handleError(e);
+            yield handleError(e).then(() => {
+                // give some time async ops to finish if not handled directly
+                if (process.env.NODE_ENV !== 'test') {
+                    setTimeout(() => {
+                        process.exit(1);
+                    }, 3000);
+                }
+            });
         }
     });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "x-fidelity",
-  "version": "3.0.2",
+  "version": "3.0.4",
   "description": "cli for opinionated framework adherence checks",
   "main": "dist/index",
   "types": "dist/index.d.ts",
@@ -17,7 +17,7 @@
   },
   "scripts": {
     "build": "rimraf dist/ && tsc && yarn copy-files",
-    "copy-files": "cp -rf src/demoConfig dist/ && cp dist/index.js dist/xfidelity",
+    "copy-files": "cp -rf src/demoConfig dist/ && cp dist/index.js dist/xfidelity && chmod 755 dist/xfidelity",
     "start": "NODE_ENV=test OPENAI_API_KEY= jest --watch",
     "test": "NODE_ENV=test yarn lint && NODE_ENV=test OPENAI_API_KEY= jest",
     "test:coverage": "NODE_ENV=test OPENAI_API_KEY= jest --coverage",

package/src/core/cli.ts

@@ -6,84 +6,87 @@ import { version } from "../../package.json";
 import { validateInput } from '../utils/inputValidation';
 import json from 'prettyjson';
 
-// Ensure logger is initialized
-if (!logger || typeof logger.info !== 'function') {
-    console.error({ msg: 'Logger is not properly initialized' });
-    // Instead of exiting, we'll create a fallback logger
-    const fallbackLogger = {
-        info: (obj: unknown) => console.log(JSON.stringify(obj)),
-        error: (obj: unknown) => console.error(JSON.stringify(obj)),
-        warn: (obj: unknown) => console.warn(JSON.stringify(obj)), 
-        debug: (obj: unknown) => console.debug(JSON.stringify(obj))
-    };
-    (global as any).logger = fallbackLogger;
-}
-
-logger.info('CLI initialized');
-
 export const DEMO_CONFIG_PATH = path.resolve(__dirname, '../demoConfig');
-
-// we are overiding the default behavior of commander to exit 
-// the process due to https://github.com/pinojs/pino/issues/871
-program.exitOverride(() => {
-    try {
-        if (process.env.NODE_ENV !== 'test') process.exit(0);
-    } catch (error) {
-        //swallow
-    }
-});
-
-program
-    .option("-d, --dir <directory>", "code directory to analyze. equivalent of directory argument")
-    .option("-a, --archetype <archetype>", "The archetype to use for analysis", "node-fullstack")
-    .option("-c, --configServer <configServer>", "The config server URL for fetching remote archetype configurations and rules. This takes precedence over localConfigPath.")
-    .option("-o, --openaiEnabled <boolean>", "Enable OpenAI analysis", false)
-    .option("-t, --telemetryCollector <telemetryCollector>", "The URL telemetry data will be sent to for usage analysis")
-    .option("-m, --mode <mode>", "Run mode: 'client' or 'server'", "client")
-    .option("-p, --port <port>", "The port to run the server on", "8888")
-    .option("-l, --localConfigPath <path>", "Path to local archetype config and rules", DEMO_CONFIG_PATH)
-    .option("-j, --jsonTTL <minutes>", "Set the server json cache TTL in minutes", "10")
-    .option("-e, --extensions <modules...>", "Space-separated list of npm module names to load as extensions")
-    .option("-x, --examine <archetype>", "Examine the archetype configuration and rules")
-    .version(version, "-v, --version", "Output the version number of xfidelity")
-    .helpOption("-h, --help", "Display help for command")
-    .argument('[directory]', 'code directory to analyze');
-
-const options = program.opts();
-
-program.parse(process.argv);
-
-// Resolve paths
-const resolvePath = (inputPath: string) => {
-    const resolvedPath = path?.resolve(process.cwd(), inputPath);
-    if (!fs.existsSync(resolvedPath)) {
-        throw new Error(`Path does not exist: ${resolvedPath}`);
+export const options = program.opts();
+
+export function initCLI() {
+
+    // Ensure logger is initialized
+    if (!logger || typeof logger.info !== 'function') {
+        console.error({ msg: 'Logger is not properly initialized' });
+        // Instead of exiting, we'll create a fallback logger
+        const fallbackLogger = {
+            info: (obj: unknown) => console.log(JSON.stringify(obj)),
+            error: (obj: unknown) => console.error(JSON.stringify(obj)),
+            warn: (obj: unknown) => console.warn(JSON.stringify(obj)), 
+            debug: (obj: unknown) => console.debug(JSON.stringify(obj))
+        };
+        (global as any).logger = fallbackLogger;
     }
-    if (!validateInput(resolvedPath)) {
-        throw new Error(`Potential malicious input detected: ${inputPath}`);
+
+    logger.info('CLI initialized');
+
+    // we are overiding the default behavior of commander to exit 
+    // the process due to https://github.com/pinojs/pino/issues/871
+    program.exitOverride(() => {
+        try {
+            if (process.env.NODE_ENV !== 'test') process.exit(0);
+        } catch (error) {
+            //swallow
+        }
+    });
+
+    program
+        .option("-d, --dir <directory>", "code directory to analyze. equivalent of directory argument")
+        .option("-a, --archetype <archetype>", "The archetype to use for analysis", "node-fullstack")
+        .option("-c, --configServer <configServer>", "The config server URL for fetching remote archetype configurations and rules. This takes precedence over localConfigPath.")
+        .option("-o, --openaiEnabled <boolean>", "Enable OpenAI analysis", false)
+        .option("-t, --telemetryCollector <telemetryCollector>", "The URL telemetry data will be sent to for usage analysis")
+        .option("-m, --mode <mode>", "Run mode: 'client' or 'server'", "client")
+        .option("-p, --port <port>", "The port to run the server on", "8888")
+        .option("-l, --localConfigPath <path>", "Path to local archetype config and rules", DEMO_CONFIG_PATH)
+        .option("-j, --jsonTTL <minutes>", "Set the server json cache TTL in minutes", "10")
+        .option("-e, --extensions <modules...>", "Space-separated list of npm module names to load as extensions")
+        .option("-x, --examine <archetype>", "Examine the archetype configuration and rules")
+        .version(version, "-v, --version", "Output the version number of xfidelity")
+        .helpOption("-h, --help", "Display help for command")
+        .argument('[directory]', 'code directory to analyze');
+
+    
+
+    program.parse(process.argv);
+
+    // Resolve paths
+    const resolvePath = (inputPath: string) => {
+        const resolvedPath = path?.resolve(process.cwd(), inputPath);
+        if (!fs.existsSync(resolvedPath)) {
+            throw new Error(`Path does not exist: ${resolvedPath}`);
+        }
+        if (!validateInput(resolvedPath)) {
+            throw new Error(`Potential malicious input detected: ${inputPath}`);
+        }
+        return resolvedPath;
+    };
+
+    if (program.args.length === 0) {
+        if (process.env.NODE_ENV === 'test' || options.mode === 'server') options.dir = '.';
+        if (!options.dir && process.env.NODE_ENV !== 'test') program.help({ error: false });
     }
-    return resolvedPath;
-};
 
-if (program.args.length === 0) {
     if (process.env.NODE_ENV === 'test' || options.mode === 'server') options.dir = '.';
-    if (!options.dir && process.env.NODE_ENV !== 'test') program.help({ error: false });
-}
-
-if (process.env.NODE_ENV === 'test' || options.mode === 'server') options.dir = '.';
-try {    
-    options.dir = program.args.length == 1 && program.args[0] !== undefined ? resolvePath(program.args[0]) : options.dir && resolvePath(options.dir);
-} catch (error) {
-    if (process.env.NODE_ENV !== 'test') program.error(`Error resolving repo path to analyse: ${error}`)
-}
+    try {    
+        options.dir = program.args.length == 1 && program.args[0] !== undefined ? resolvePath(program.args[0]) : options.dir && resolvePath(options.dir);
+    } catch (error) {
+        if (process.env.NODE_ENV !== 'test') program.error(`Error resolving repo path to analyse: ${error}`)
+    }
 
-try {
-    if (options.localConfigPath) {
-        options.localConfigPath = resolvePath(options.localConfigPath);
+    try {
+        if (options.localConfigPath) {
+            options.localConfigPath = resolvePath(options.localConfigPath);
+        }
+    } catch (error) {
+        if (process.env.NODE_ENV !== 'test') program.error(`LocalConfigPath does not exist or is invalid: ${error}`);
     }
-} catch (error) {
-    if (process.env.NODE_ENV !== 'test') program.error(`LocalConfigPath does not exist or is invalid: ${error}`);
-}
 
 const bannerArt = `\n
 =====================================
@@ -98,25 +101,25 @@ const bannerArt = `\n
 ------------------------------------- 
 `;
 
-logger.info(bannerArt);
-
-logger.info(`\n${json.render({
-    version,
-    startTime: new Date().toString().slice(0, 24),
-    archetype: options.archetype,
-    directory: options.dir,
-    configServer: options.configServer ? options.configServer : 'none',
-    mode: options.mode,
-    port: options.mode === 'server' ? options.port : 'n/a', 
-    localConfigPath: options.localConfigPath ? options.localConfigPath : 'none',
-    jsonTTL: `${options.jsonTTL} minutes`,
-    openaiEnabled: options.openaiEnabled,
-    extensions: options.extensions ? options.extensions : 'none'
-})}
+    logger.info(bannerArt);
+
+    logger.info(`\n${json.render({
+        version,
+        startTime: new Date().toString().slice(0, 24),
+        archetype: options.archetype,
+        directory: options.dir,
+        configServer: options.configServer ? options.configServer : 'none',
+        mode: options.mode,
+        port: options.mode === 'server' ? options.port : 'n/a', 
+        localConfigPath: options.localConfigPath ? options.localConfigPath : 'none',
+        jsonTTL: `${options.jsonTTL} minutes`,
+        openaiEnabled: options.openaiEnabled,
+        extensions: options.extensions ? options.extensions : 'none'
+    })}
 -------------------------------------
-`);
+    `);
 
-// print help if no arguments are passed
-if (program.options.length === 0) program.help();
+    // print help if no arguments are passed
+    if (program.options.length === 0) program.help();
+}
 
-export { options };

package/src/core/configManager.ts

@@ -30,7 +30,10 @@ export class ConfigManager {
     public static async getConfig(params: GetConfigParams): Promise<ExecutionConfig> {
         const { archetype = options.archetype, logPrefix } = params;
         if (!ConfigManager.configs[archetype]) {
-            ConfigManager.configs[archetype] = await ConfigManager.initialize({ archetype, logPrefix });
+            ConfigManager.configs[archetype] = await ConfigManager.initialize({ archetype, logPrefix }).catch(error => {
+                logger.error(error, `Error initializing config for archetype: ${archetype}`);
+                throw error;
+            });
         }
         return ConfigManager.configs[archetype];
     }
@@ -76,7 +79,7 @@ export class ConfigManager {
                     logger.info(`Successfully loaded extension: ${moduleName}`);
                 } catch (error) {
                     logger.error(`Failed to load extension ${moduleName} from all locations: ${error}`);
-                    if (process.env.NODE_ENV !== 'test') process.exit(1);
+                    throw new Error(`Failed to load extension ${moduleName} from all locations: ${error}`);
                 }
             }
         }
@@ -90,7 +93,10 @@ export class ConfigManager {
         if (logPrefix) setLogPrefix(logPrefix);
         
         // Load plugins during initialization
-        await this.loadPlugins(options.extensions);
+        await this.loadPlugins(options.extensions).catch(error => {
+            logger.error(error, `Error loading plugins`);
+            throw error;
+        });
         logger.info(`Initializing config manager for archetype: ${archetype}`);
         logger.debug(`Initialize params: ${JSON.stringify(params)}`);
 

package/src/index.ts

@@ -1,11 +1,15 @@
 #!/usr/bin/env node
 import { logger, initializeLogger, getLogPrefix, setLogLevel } from './utils/logger';
-initializeLogger()
+import { options, initCLI } from "./core/cli";
+if (require.main === module) {
+    initializeLogger()
+    setLogLevel(process.env.XFI_LOG_LEVEL || 'info');
+    initCLI();
+}
+
 const executionLogPrefix = getLogPrefix();
-setLogLevel(process.env.XFI_LOG_LEVEL || 'info');
 
 import json from 'prettyjson';
-import { options } from "./core/cli";
 import { analyzeCodebase } from "./core/engine/analyzer";
 import { startServer } from './server/configServer';
 import { sendTelemetry } from './utils/telemetry';
@@ -23,7 +27,7 @@ const handleError = async (error: Error) => {
         },
         timestamp: new Date().toISOString()
     }, executionLogPrefix);
-    logger.error(JSON.stringify(error));
+    logger.error(error, 'Execution failure');
 };
 
 const outcomeMessage = (message: string) => `\n
@@ -81,8 +85,14 @@ export async function main() {
             }
         }    
     } catch (e: any) {
-        await handleError(e);
-        
+        await handleError(e).then(() => {
+            // give some time async ops to finish if not handled directly
+            if (process.env.NODE_ENV !== 'test') {
+                setTimeout(() => {
+                    process.exit(1);
+                }, 3000);
+            }
+        });    
     }
 }