gitlab-auto-reviewers 2.0.1 → 2.1.0

package/dist/cli/parameter-source-tracker.js

@@ -0,0 +1,252 @@
+/**
+ * Parameter Source Tracker
+ *
+ * Tracks and logs the source of each configuration parameter for debugging.
+ * Implements Requirement 3.5: THE System SHALL log the source of each configuration value for debugging
+ *
+ * @module cli/parameter-source-tracker
+ */
+/**
+ * Parameter Source Tracker
+ *
+ * Tracks the source of each configuration parameter and provides
+ * comprehensive logging for debugging configuration issues.
+ */
+export class ParameterSourceTracker {
+    parameters = new Map();
+    logger;
+    /**
+     * Creates a new ParameterSourceTracker instance
+     *
+     * @param logger - Logger instance for outputting parameter source information
+     */
+    constructor(logger) {
+        this.logger = logger;
+    }
+    /**
+     * Track a parameter with its value and source
+     */
+    track(name, value, source, options) {
+        this.parameters.set(name, {
+            name,
+            value,
+            source,
+            sourceDetail: options?.sourceDetail,
+            masked: options?.masked,
+        });
+    }
+    /**
+     * Resolve a parameter value with source tracking
+     * Implements the precedence: CLI argument > environment variable > auto-detected > config file > default
+     */
+    resolve(name, cliValue, envVarName, autoDetectFn, configValue, defaultValue, options) {
+        // Priority 1: CLI argument
+        if (cliValue !== undefined) {
+            this.track(name, cliValue, 'cli-argument', {
+                sourceDetail: `--${this.toKebabCase(name)}`,
+                masked: options?.masked,
+            });
+            return cliValue;
+        }
+        // Priority 2: Environment variable
+        if (envVarName) {
+            const envValue = process.env[envVarName];
+            if (envValue !== undefined && envValue !== '') {
+                const parsedValue = this.parseEnvValue(envValue);
+                this.track(name, parsedValue, 'environment', {
+                    sourceDetail: envVarName,
+                    masked: options?.masked,
+                });
+                return parsedValue;
+            }
+        }
+        // Priority 3: Auto-detection
+        if (autoDetectFn) {
+            const autoValue = autoDetectFn();
+            if (autoValue !== undefined) {
+                this.track(name, autoValue, 'auto-detected', {
+                    masked: options?.masked,
+                });
+                return autoValue;
+            }
+        }
+        // Priority 4: Config file
+        if (configValue !== undefined) {
+            this.track(name, configValue, 'config-file', {
+                masked: options?.masked,
+            });
+            return configValue;
+        }
+        // Priority 5: Default value
+        if (defaultValue !== undefined) {
+            this.track(name, defaultValue, 'default', {
+                masked: options?.masked,
+            });
+            return defaultValue;
+        }
+        // Not set
+        this.track(name, undefined, 'not-set', {
+            masked: options?.masked,
+        });
+        return undefined;
+    }
+    /**
+     * Get all tracked parameters
+     */
+    getAll() {
+        return new Map(this.parameters);
+    }
+    /**
+     * Get a specific tracked parameter
+     *
+     * @param name - The name of the parameter to retrieve
+     * @returns The tracked parameter or undefined if not found
+     */
+    get(name) {
+        return this.parameters.get(name);
+    }
+    /**
+     * Log all parameter sources
+     * This implements Requirement 3.5
+     */
+    logParameterSources() {
+        this.logger.info('Configuration resolved - parameter sources:');
+        const sortedParams = Array.from(this.parameters.entries()).sort(([a], [b]) => a.localeCompare(b));
+        for (const [name, param] of sortedParams) {
+            const displayValue = this.formatValueForLog(param);
+            const sourceInfo = this.formatSourceForLog(param);
+            this.logger.info(`  ${name}: ${displayValue} (${sourceInfo})`);
+        }
+    }
+    /**
+     * Log parameter sources in a structured format (for JSON output)
+     */
+    toStructuredLog() {
+        const result = {};
+        for (const [name, param] of this.parameters) {
+            result[name] = {
+                value: param.masked ? '***' : param.value,
+                source: param.source,
+                sourceDetail: param.sourceDetail,
+            };
+        }
+        return result;
+    }
+    /**
+     * Format value for logging (handles masking and undefined)
+     */
+    formatValueForLog(param) {
+        if (param.value === undefined) {
+            return '<not set>';
+        }
+        if (param.masked) {
+            return '***';
+        }
+        if (typeof param.value === 'string') {
+            // Truncate long strings
+            if (param.value.length > 50) {
+                return `"${param.value.substring(0, 47)}..."`;
+            }
+            return `"${param.value}"`;
+        }
+        return String(param.value);
+    }
+    /**
+     * Format source information for logging
+     */
+    formatSourceForLog(param) {
+        switch (param.source) {
+            case 'cli-argument':
+                return `CLI argument ${param.sourceDetail || ''}`.trim();
+            case 'environment':
+                return `environment variable ${param.sourceDetail || ''}`.trim();
+            case 'auto-detected':
+                return param.sourceDetail ? `auto-detected from ${param.sourceDetail}` : 'auto-detected';
+            case 'config-file':
+                return param.sourceDetail ? `config file ${param.sourceDetail}` : 'config file';
+            case 'default':
+                return 'default';
+            case 'not-set':
+                return 'not set';
+            default:
+                return param.source;
+        }
+    }
+    /**
+     * Convert camelCase to kebab-case
+     */
+    toKebabCase(str) {
+        return str.replace(/([a-z])([A-Z])/g, '$1-$2').toLowerCase();
+    }
+    /**
+     * Parse environment variable value to appropriate type
+     */
+    parseEnvValue(value) {
+        // Try to parse as number
+        const num = Number(value);
+        if (!isNaN(num) && value.trim() !== '') {
+            return num;
+        }
+        // Try to parse as boolean
+        if (value.toLowerCase() === 'true') {
+            return true;
+        }
+        if (value.toLowerCase() === 'false') {
+            return false;
+        }
+        // Return as string
+        return value;
+    }
+}
+/**
+ * Create a parameter source tracker and resolve all CLI parameters
+ *
+ * @param options - CLI options from argument parsing
+ * @param cicdEnv - CI/CD environment detection result
+ * @param logger - Logger instance
+ * @returns Resolved parameters with source tracking
+ */
+export function resolveParametersWithTracking(options, cicdEnv, logger) {
+    const tracker = new ParameterSourceTracker(logger);
+    // Resolve each parameter with source tracking
+    // NOTE: We need to resolve project FIRST to determine if we should auto-detect repoPath
+    // When both project ID and project directory are available in CI, prefer API mode (project ID)
+    // because it provides more reliable data access in CI environments
+    const project = tracker.resolve('project', options.project, 'CI_PROJECT_ID', // Also check CI_PROJECT_ID environment variable
+    () => cicdEnv.isCI ? cicdEnv.projectId : undefined, undefined, undefined);
+    // Only auto-detect repoPath from CI_PROJECT_DIR if project is NOT already set
+    // This prevents the "Cannot specify both --repo-path and --project" conflict
+    const repoPath = tracker.resolve('repoPath', options.repoPath, project ? undefined : 'CI_PROJECT_DIR', // Only use CI_PROJECT_DIR if project is not set
+    undefined, undefined, undefined);
+    // Resolve mergeRequestIid FIRST to determine if we should auto-detect branch
+    // When both MR IID and branch are available in CI, prefer MR IID because
+    // it provides more specific context for reviewer suggestions
+    const mergeRequestIid = tracker.resolve('mergeRequestIid', options.mergeRequestIid, 'CI_MERGE_REQUEST_IID', () => cicdEnv.isCI ? cicdEnv.mergeRequestIid : undefined, undefined, undefined);
+    // Only auto-detect branch from CI_COMMIT_REF_NAME if mergeRequestIid is NOT already set
+    // This prevents the "Cannot specify both merge request IID and branch name" conflict
+    const branch = tracker.resolve('branch', options.branch, mergeRequestIid ? undefined : 'CI_COMMIT_REF_NAME', // Only use CI_COMMIT_REF_NAME if MR IID is not set
+    () => (cicdEnv.isCI && !mergeRequestIid) ? cicdEnv.branch : undefined, undefined, undefined);
+    const gitlabUrl = tracker.resolve('gitlabUrl', options.gitlabUrl, 'GITLAB_URL', () => process.env.CI_SERVER_URL, // Auto-detect from GitLab CI environment
+    undefined, 'https://gitlab.com') || 'https://gitlab.com';
+    const gitlabToken = tracker.resolve('gitlabToken', options.gitlabToken, 'GITLAB_TOKEN', () => process.env.CI_JOB_TOKEN, undefined, undefined, { masked: true });
+    const format = tracker.resolve('format', options.format, undefined, () => cicdEnv.isCI ? 'json' : undefined, undefined, 'text') || 'text';
+    const verbose = tracker.resolve('verbose', options.verbose, undefined, undefined, undefined, false) || false;
+    const dryRun = tracker.resolve('dryRun', options.dryRun, undefined, undefined, undefined, false) || false;
+    const quiet = tracker.resolve('quiet', options.quiet, undefined, () => cicdEnv.isCI ? true : undefined, undefined, false) || false;
+    return {
+        tracker,
+        resolved: {
+            repoPath,
+            project,
+            mergeRequestIid,
+            branch,
+            gitlabUrl,
+            gitlabToken,
+            format,
+            verbose,
+            dryRun,
+            quiet,
+        },
+    };
+}
+//# sourceMappingURL=parameter-source-tracker.js.map

package/dist/cli/parameter-source-tracker.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"parameter-source-tracker.js","sourceRoot":"","sources":["../../src/cli/parameter-source-tracker.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AA6DH;;;;;GAKG;AACH,MAAM,OAAO,sBAAsB;IACzB,UAAU,GAAkC,IAAI,GAAG,EAAE,CAAC;IACtD,MAAM,CAAS;IAEvB;;;;OAIG;IACH,YAAY,MAAc;QACxB,IAAI,CAAC,MAAM,GAAG,MAAM,CAAC;IACvB,CAAC;IAED;;OAEG;IACH,KAAK,CACH,IAAY,EACZ,KAAoB,EACpB,MAAuB,EACvB,OAGC;QAED,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,EAAE;YACxB,IAAI;YACJ,KAAK;YACL,MAAM;YACN,YAAY,EAAE,OAAO,EAAE,YAAY;YACnC,MAAM,EAAE,OAAO,EAAE,MAAM;SACxB,CAAC,CAAC;IACL,CAAC;IAED;;;OAGG;IACH,OAAO,CACL,IAAY,EACZ,QAAuB,EACvB,UAA8B,EAC9B,YAAkC,EAClC,WAAe,EACf,YAAgB,EAChB,OAA8B;QAE9B,2BAA2B;QAC3B,IAAI,QAAQ,KAAK,SAAS,EAAE,CAAC;YAC3B,IAAI,CAAC,KAAK,CAAC,IAAI,EAAE,QAAQ,EAAE,cAAc,EAAE;gBACzC,YAAY,EAAE,KAAK,IAAI,CAAC,WAAW,CAAC,IAAI,CAAC,EAAE;gBAC3C,MAAM,EAAE,OAAO,EAAE,MAAM;aACxB,CAAC,CAAC;YACH,OAAO,QAAQ,CAAC;QAClB,CAAC;QAED,mCAAmC;QACnC,IAAI,UAAU,EAAE,CAAC;YACf,MAAM,QAAQ,GAAG,OAAO,CAAC,GAAG,CAAC,UAAU,CAAC,CAAC;YACzC,IAAI,QAAQ,KAAK,SAAS,IAAI,QAAQ,KAAK,EAAE,EAAE,CAAC;gBAC9C,MAAM,WAAW,GAAG,IAAI,CAAC,aAAa,CAAI,QAAQ,CAAC,CAAC;gBACpD,IAAI,CAAC,KAAK,CAAC,IAAI,EAAE,WAAW,EAAE,aAAa,EAAE;oBAC3C,YAAY,EAAE,UAAU;oBACxB,MAAM,EAAE,OAAO,EAAE,MAAM;iBACxB,CAAC,CAAC;gBACH,OAAO,WAAW,CAAC;YACrB,CAAC;QACH,CAAC;QAED,6BAA6B;QAC7B,IAAI,YAAY,EAAE,CAAC;YACjB,MAAM,SAAS,GAAG,YAAY,EAAE,CAAC;YACjC,IAAI,SAAS,KAAK,SAAS,EAAE,CAAC;gBAC5B,IAAI,CAAC,KAAK,CAAC,IAAI,EAAE,SAAS,EAAE,eAAe,EAAE;oBAC3C,MAAM,EAAE,OAAO,EAAE,MAAM;iBACxB,CAAC,CAAC;gBACH,OAAO,SAAS,CAAC;YACnB,CAAC;QACH,CAAC;QAED,0BAA0B;QAC1B,IAAI,WAAW,KAAK,SAAS,EAAE,CAAC;YAC9B,IAAI,CAAC,KAAK,CAAC,IAAI,EAAE,WAAW,EAAE,aAAa,EAAE;gBAC3C,MAAM,EAAE,OAAO,EAAE,MAAM;aACxB,CAAC,CAAC;YACH,OAAO,WAAW,CAAC;QACrB,CAAC;QAED,4BAA4B;QAC5B,IAAI,YAAY,KAAK,SAAS,EAAE,CAAC;YAC/B,IAAI,CAAC,KAAK,CAAC,IAAI,EAAE,YAAY,EAAE,SAAS,EAAE;gBACxC,MAAM,EAAE,OAAO,EAAE,MAAM;aACxB,CAAC,CAAC;YACH,OAAO,YAAY,CAAC;QACtB,CAAC;QAED,UAAU;QACV,IAAI,CAAC,KAAK,CAAC,IAAI,EAAE,SAAS,EAAE,SAAS,EAAE;YACrC,MAAM,EAAE,OAAO,EAAE,MAAM;SACxB,CAAC,CAAC;QACH,OAAO,SAAS,CAAC;IACnB,CAAC;IAED;;OAEG;IACH,MAAM;QACJ,OAAO,IAAI,GAAG,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;IAClC,CAAC;IAED;;;;;OAKG;IACH,GAAG,CAAC,IAAY;QACd,OAAO,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;IACnC,CAAC;IAED;;;OAGG;IACH,mBAAmB;QACjB,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,6CAA6C,CAAC,CAAC;QAEhE,MAAM,YAAY,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC,OAAO,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC,CAAC;QAElG,KAAK,MAAM,CAAC,IAAI,EAAE,KAAK,CAAC,IAAI,YAAY,EAAE,CAAC;YACzC,MAAM,YAAY,GAAG,IAAI,CAAC,iBAAiB,CAAC,KAAK,CAAC,CAAC;YACnD,MAAM,UAAU,GAAG,IAAI,CAAC,kBAAkB,CAAC,KAAK,CAAC,CAAC;YAElD,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,KAAK,IAAI,KAAK,YAAY,KAAK,UAAU,GAAG,CAAC,CAAC;QACjE,CAAC;IACH,CAAC;IAED;;OAEG;IACH,eAAe;QACb,MAAM,MAAM,GAAuF,EAAE,CAAC;QAEtG,KAAK,MAAM,CAAC,IAAI,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC,UAAU,EAAE,CAAC;YAC5C,MAAM,CAAC,IAAI,CAAC,GAAG;gBACb,KAAK,EAAE,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,KAAK;gBACzC,MAAM,EAAE,KAAK,CAAC,MAAM;gBACpB,YAAY,EAAE,KAAK,CAAC,YAAY;aACjC,CAAC;QACJ,CAAC;QAED,OAAO,MAAM,CAAC;IAChB,CAAC;IAED;;OAEG;IACK,iBAAiB,CAAC,KAAuB;QAC/C,IAAI,KAAK,CAAC,KAAK,KAAK,SAAS,EAAE,CAAC;YAC9B,OAAO,WAAW,CAAC;QACrB,CAAC;QAED,IAAI,KAAK,CAAC,MAAM,EAAE,CAAC;YACjB,OAAO,KAAK,CAAC;QACf,CAAC;QAED,IAAI,OAAO,KAAK,CAAC,KAAK,KAAK,QAAQ,EAAE,CAAC;YACpC,wBAAwB;YACxB,IAAI,KAAK,CAAC,KAAK,CAAC,MAAM,GAAG,EAAE,EAAE,CAAC;gBAC5B,OAAO,IAAI,KAAK,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,EAAE,EAAE,CAAC,MAAM,CAAC;YAChD,CAAC;YACD,OAAO,IAAI,KAAK,CAAC,KAAK,GAAG,CAAC;QAC5B,CAAC;QAED,OAAO,MAAM,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;IAC7B,CAAC;IAED;;OAEG;IACK,kBAAkB,CAAC,KAAuB;QAChD,QAAQ,KAAK,CAAC,MAAM,EAAE,CAAC;YACrB,KAAK,cAAc;gBACjB,OAAO,gBAAgB,KAAK,CAAC,YAAY,IAAI,EAAE,EAAE,CAAC,IAAI,EAAE,CAAC;YAC3D,KAAK,aAAa;gBAChB,OAAO,wBAAwB,KAAK,CAAC,YAAY,IAAI,EAAE,EAAE,CAAC,IAAI,EAAE,CAAC;YACnE,KAAK,eAAe;gBAClB,OAAO,KAAK,CAAC,YAAY,CAAC,CAAC,CAAC,sBAAsB,KAAK,CAAC,YAAY,EAAE,CAAC,CAAC,CAAC,eAAe,CAAC;YAC3F,KAAK,aAAa;gBAChB,OAAO,KAAK,CAAC,YAAY,CAAC,CAAC,CAAC,eAAe,KAAK,CAAC,YAAY,EAAE,CAAC,CAAC,CAAC,aAAa,CAAC;YAClF,KAAK,SAAS;gBACZ,OAAO,SAAS,CAAC;YACnB,KAAK,SAAS;gBACZ,OAAO,SAAS,CAAC;YACnB;gBACE,OAAO,KAAK,CAAC,MAAM,CAAC;QACxB,CAAC;IACH,CAAC;IAED;;OAEG;IACK,WAAW,CAAC,GAAW;QAC7B,OAAO,GAAG,CAAC,OAAO,CAAC,iBAAiB,EAAE,OAAO,CAAC,CAAC,WAAW,EAAE,CAAC;IAC/D,CAAC;IAED;;OAEG;IACK,aAAa,CAAI,KAAa;QACpC,yBAAyB;QACzB,MAAM,GAAG,GAAG,MAAM,CAAC,KAAK,CAAC,CAAC;QAC1B,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,IAAI,KAAK,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE,CAAC;YACvC,OAAO,GAAQ,CAAC;QAClB,CAAC;QAED,0BAA0B;QAC1B,IAAI,KAAK,CAAC,WAAW,EAAE,KAAK,MAAM,EAAE,CAAC;YACnC,OAAO,IAAS,CAAC;QACnB,CAAC;QACD,IAAI,KAAK,CAAC,WAAW,EAAE,KAAK,OAAO,EAAE,CAAC;YACpC,OAAO,KAAU,CAAC;QACpB,CAAC;QAED,mBAAmB;QACnB,OAAO,KAAU,CAAC;IACpB,CAAC;CACF;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,6BAA6B,CAC3C,OAWC,EACD,OAOC,EACD,MAAc;IAgBd,MAAM,OAAO,GAAG,IAAI,sBAAsB,CAAC,MAAM,CAAC,CAAC;IAEnD,8CAA8C;IAC9C,wFAAwF;IACxF,+FAA+F;IAC/F,mEAAmE;IACnE,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAC7B,SAAS,EACT,OAAO,CAAC,OAAO,EACf,eAAe,EAAG,gDAAgD;IAClE,GAAG,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,CAAC,SAAS,EAClD,SAAS,EACT,SAAS,CACV,CAAC;IAEF,8EAA8E;IAC9E,6EAA6E;IAC7E,MAAM,QAAQ,GAAG,OAAO,CAAC,OAAO,CAC9B,UAAU,EACV,OAAO,CAAC,QAAQ,EAChB,OAAO,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,gBAAgB,EAAG,gDAAgD;IACzF,SAAS,EACT,SAAS,EACT,SAAS,CACV,CAAC;IAEF,6EAA6E;IAC7E,yEAAyE;IACzE,6DAA6D;IAC7D,MAAM,eAAe,GAAG,OAAO,CAAC,OAAO,CACrC,iBAAiB,EACjB,OAAO,CAAC,eAAe,EACvB,sBAAsB,EACtB,GAAG,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,OAAO,CAAC,eAAe,CAAC,CAAC,CAAC,SAAS,EACxD,SAAS,EACT,SAAS,CACV,CAAC;IAEF,wFAAwF;IACxF,qFAAqF;IACrF,MAAM,MAAM,GAAG,OAAO,CAAC,OAAO,CAC5B,QAAQ,EACR,OAAO,CAAC,MAAM,EACd,eAAe,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,oBAAoB,EAAG,mDAAmD;IACxG,GAAG,EAAE,CAAC,CAAC,OAAO,CAAC,IAAI,IAAI,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,SAAS,EACrE,SAAS,EACT,SAAS,CACV,CAAC;IAEF,MAAM,SAAS,GAAG,OAAO,CAAC,OAAO,CAC/B,WAAW,EACX,OAAO,CAAC,SAAS,EACjB,YAAY,EACZ,GAAG,EAAE,CAAC,OAAO,CAAC,GAAG,CAAC,aAAa,EAAG,yCAAyC;IAC3E,SAAS,EACT,oBAAoB,CACrB,IAAI,oBAAoB,CAAC;IAE1B,MAAM,WAAW,GAAG,OAAO,CAAC,OAAO,CACjC,aAAa,EACb,OAAO,CAAC,WAAW,EACnB,cAAc,EACd,GAAG,EAAE,CAAC,OAAO,CAAC,GAAG,CAAC,YAAY,EAC9B,SAAS,EACT,SAAS,EACT,EAAE,MAAM,EAAE,IAAI,EAAE,CACjB,CAAC;IAEF,MAAM,MAAM,GAAG,OAAO,CAAC,OAAO,CAC5B,QAAQ,EACR,OAAO,CAAC,MAAM,EACd,SAAS,EACT,GAAG,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,SAAS,EACvC,SAAS,EACT,MAAM,CACP,IAAI,MAAM,CAAC;IAEZ,MAAM,OAAO,GAAG,OAAO,CAAC,OAAO,CAC7B,SAAS,EACT,OAAO,CAAC,OAAO,EACf,SAAS,EACT,SAAS,EACT,SAAS,EACT,KAAK,CACN,IAAI,KAAK,CAAC;IAEX,MAAM,MAAM,GAAG,OAAO,CAAC,OAAO,CAC5B,QAAQ,EACR,OAAO,CAAC,MAAM,EACd,SAAS,EACT,SAAS,EACT,SAAS,EACT,KAAK,CACN,IAAI,KAAK,CAAC;IAEX,MAAM,KAAK,GAAG,OAAO,CAAC,OAAO,CAC3B,OAAO,EACP,OAAO,CAAC,KAAK,EACb,SAAS,EACT,GAAG,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,SAAS,EACrC,SAAS,EACT,KAAK,CACN,IAAI,KAAK,CAAC;IAEX,OAAO;QACL,OAAO;QACP,QAAQ,EAAE;YACR,QAAQ;YACR,OAAO;YACP,eAAe;YACf,MAAM;YACN,SAAS;YACT,WAAW;YACX,MAAM;YACN,OAAO;YACP,MAAM;YACN,KAAK;SACN;KACF,CAAC;AACJ,CAAC"}

package/dist/config/config.service.d.ts

@@ -37,17 +37,36 @@ export interface MCPServerConfig {
  *
  * Loads configuration from environment variables, applies defaults,
  * validates settings, and provides type-safe access to configuration.
+ *
+ * Supports singleton pattern to avoid multiple instantiations and duplicate logging.
  */
 export declare class ConfigService {
+    private static instance;
+    private static hasLoggedDefaults;
     private config;
     private appliedDefaults;
     /**
      * Create a new configuration service
      *
      * Automatically loads and validates configuration from environment variables.
-     * Logs any default values that were applied.
+     * Logs any default values that were applied (only once per process).
+     *
+     * @param skipLogging - If true, skip logging applied defaults (useful for testing)
      */
-    constructor();
+    constructor(skipLogging?: boolean);
+    /**
+     * Get the singleton instance of ConfigService
+     *
+     * This ensures configuration is only loaded and logged once per process,
+     * avoiding duplicate log messages when multiple components need config.
+     *
+     * @returns The shared ConfigService instance
+     */
+    static getInstance(): ConfigService;
+    /**
+     * Reset the singleton instance (primarily for testing)
+     */
+    static resetInstance(): void;
     /**
      * Get the current configuration (read-only)
      */
@@ -58,6 +77,8 @@ export declare class ConfigService {
     getAppliedDefaults(): ReadonlyMap<string, unknown>;
     /**
      * Load configuration from environment variables and validate
+     *
+     * @param skipLogging - If true, skip logging applied defaults
      */
     private loadAndValidate;
     /**
@@ -83,6 +104,7 @@ export declare class ConfigService {
     /**
      * Log which default values were applied
      * Uses console.error since MCP servers log to stderr
+     * Only logs once per process to avoid duplicate messages
      */
     private logAppliedDefaults;
 }

package/dist/config/config.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"config.service.d.ts","sourceRoot":"","sources":["../../src/config/config.service.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH;;;;;;;;;;;;;;GAcG;AACH,MAAM,WAAW,eAAe;IAE9B,SAAS,EAAE,MAAM,CAAC;IAClB,WAAW,CAAC,EAAE,MAAM,CAAC;IAGrB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAGlB,YAAY,EAAE,OAAO,CAAC;IACtB,QAAQ,EAAE,MAAM,CAAC;IACjB,mBAAmB,EAAE,MAAM,CAAC;IAG5B,eAAe,EAAE,MAAM,CAAC;IACxB,UAAU,EAAE,MAAM,CAAC;IACnB,YAAY,EAAE,MAAM,CAAC;IAGrB,QAAQ,EAAE,OAAO,GAAG,MAAM,GAAG,MAAM,GAAG,OAAO,CAAC;IAC9C,KAAK,EAAE,OAAO,CAAC;CAChB;AAED;;;;;GAKG;AACH,qBAAa,aAAa;IACxB,OAAO,CAAC,MAAM,CAAkB;IAChC,OAAO,CAAC,eAAe,CAAmC;IAE1D;;;;;OAKG;;IAKH;;OAEG;IACH,GAAG,IAAI,QAAQ,CAAC,eAAe,CAAC;IAIhC;;OAEG;IACH,kBAAkB,IAAI,WAAW,CAAC,MAAM,EAAE,OAAO,CAAC;IAKlD;;OAEG;IACH,OAAO,CAAC,eAAe;IA6BvB;;OAEG;IACH,OAAO,CAAC,cAAc;IA2CtB;;OAEG;IACH,OAAO,CAAC,eAAe;IASvB;;OAEG;IACH,OAAO,CAAC,aAAa;IASrB;;OAEG;IACH,OAAO,CAAC,YAAY;IAepB;;OAEG;IACH,OAAO,CAAC,WAAW;IAiBnB;;;OAGG;IACH,OAAO,CAAC,kBAAkB;CAY3B"}
+{"version":3,"file":"config.service.d.ts","sourceRoot":"","sources":["../../src/config/config.service.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH;;;;;;;;;;;;;;GAcG;AACH,MAAM,WAAW,eAAe;IAE9B,SAAS,EAAE,MAAM,CAAC;IAClB,WAAW,CAAC,EAAE,MAAM,CAAC;IAGrB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAGlB,YAAY,EAAE,OAAO,CAAC;IACtB,QAAQ,EAAE,MAAM,CAAC;IACjB,mBAAmB,EAAE,MAAM,CAAC;IAG5B,eAAe,EAAE,MAAM,CAAC;IACxB,UAAU,EAAE,MAAM,CAAC;IACnB,YAAY,EAAE,MAAM,CAAC;IAGrB,QAAQ,EAAE,OAAO,GAAG,MAAM,GAAG,MAAM,GAAG,OAAO,CAAC;IAC9C,KAAK,EAAE,OAAO,CAAC;CAChB;AAED;;;;;;;GAOG;AACH,qBAAa,aAAa;IACxB,OAAO,CAAC,MAAM,CAAC,QAAQ,CAA8B;IACrD,OAAO,CAAC,MAAM,CAAC,iBAAiB,CAAS;IAEzC,OAAO,CAAC,MAAM,CAAkB;IAChC,OAAO,CAAC,eAAe,CAAmC;IAE1D;;;;;;;OAOG;gBACS,WAAW,UAAQ;IAI/B;;;;;;;OAOG;IACH,MAAM,CAAC,WAAW,IAAI,aAAa;IAOnC;;OAEG;IACH,MAAM,CAAC,aAAa,IAAI,IAAI;IAK5B;;OAEG;IACH,GAAG,IAAI,QAAQ,CAAC,eAAe,CAAC;IAIhC;;OAEG;IACH,kBAAkB,IAAI,WAAW,CAAC,MAAM,EAAE,OAAO,CAAC;IAKlD;;;;OAIG;IACH,OAAO,CAAC,eAAe;IA+BvB;;OAEG;IACH,OAAO,CAAC,cAAc;IA2CtB;;OAEG;IACH,OAAO,CAAC,eAAe;IASvB;;OAEG;IACH,OAAO,CAAC,aAAa;IASrB;;OAEG;IACH,OAAO,CAAC,YAAY;IAepB;;OAEG;IACH,OAAO,CAAC,WAAW;IAiBnB;;;;OAIG;IACH,OAAO,CAAC,kBAAkB;CAa3B"}

package/dist/config/config.service.js

@@ -9,18 +9,45 @@
  *
  * Loads configuration from environment variables, applies defaults,
  * validates settings, and provides type-safe access to configuration.
+ *
+ * Supports singleton pattern to avoid multiple instantiations and duplicate logging.
  */
 export class ConfigService {
+    static instance = null;
+    static hasLoggedDefaults = false;
     config;
     appliedDefaults = new Map();
     /**
      * Create a new configuration service
      *
      * Automatically loads and validates configuration from environment variables.
-     * Logs any default values that were applied.
+     * Logs any default values that were applied (only once per process).
+     *
+     * @param skipLogging - If true, skip logging applied defaults (useful for testing)
      */
-    constructor() {
-        this.config = this.loadAndValidate();
+    constructor(skipLogging = false) {
+        this.config = this.loadAndValidate(skipLogging);
+    }
+    /**
+     * Get the singleton instance of ConfigService
+     *
+     * This ensures configuration is only loaded and logged once per process,
+     * avoiding duplicate log messages when multiple components need config.
+     *
+     * @returns The shared ConfigService instance
+     */
+    static getInstance() {
+        if (!ConfigService.instance) {
+            ConfigService.instance = new ConfigService();
+        }
+        return ConfigService.instance;
+    }
+    /**
+     * Reset the singleton instance (primarily for testing)
+     */
+    static resetInstance() {
+        ConfigService.instance = null;
+        ConfigService.hasLoggedDefaults = false;
     }
     /**
      * Get the current configuration (read-only)
@@ -37,8 +64,10 @@ export class ConfigService {
     }
     /**
      * Load configuration from environment variables and validate
+     *
+     * @param skipLogging - If true, skip logging applied defaults
      */
-    loadAndValidate() {
+    loadAndValidate(skipLogging = false) {
         const config = {
             // GitLab Configuration
             gitlabUrl: this.getEnvOrDefault('GITLAB_URL', 'https://gitlab.com'),
@@ -58,7 +87,9 @@ export class ConfigService {
             debug: this.getBooleanEnv('DEBUG', false),
         };
         this.validateConfig(config);
-        this.logAppliedDefaults();
+        if (!skipLogging) {
+            this.logAppliedDefaults();
+        }
         return config;
     }
     /**
@@ -154,11 +185,13 @@ export class ConfigService {
     /**
      * Log which default values were applied
      * Uses console.error since MCP servers log to stderr
+     * Only logs once per process to avoid duplicate messages
      */
     logAppliedDefaults() {
-        if (this.appliedDefaults.size === 0) {
+        if (this.appliedDefaults.size === 0 || ConfigService.hasLoggedDefaults) {
             return;
         }
+        ConfigService.hasLoggedDefaults = true;
         const timestamp = new Date().toISOString();
         console.error(`[${timestamp}] [INFO] [ConfigService] Applied default configuration values:`);
         for (const [key, value] of this.appliedDefaults.entries()) {

package/dist/config/config.service.js.map

@@ -1 +1 @@
-{"version":3,"file":"config.service.js","sourceRoot":"","sources":["../../src/config/config.service.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAwCH;;;;;GAKG;AACH,MAAM,OAAO,aAAa;IAChB,MAAM,CAAkB;IACxB,eAAe,GAAyB,IAAI,GAAG,EAAE,CAAC;IAE1D;;;;;OAKG;IACH;QACE,IAAI,CAAC,MAAM,GAAG,IAAI,CAAC,eAAe,EAAE,CAAC;IACvC,CAAC;IAED;;OAEG;IACH,GAAG;QACD,OAAO,MAAM,CAAC,MAAM,CAAC,EAAE,GAAG,IAAI,CAAC,MAAM,EAAE,CAAC,CAAC;IAC3C,CAAC;IAED;;OAEG;IACH,kBAAkB;QAChB,6DAA6D;QAC7D,OAAO,IAAI,GAAG,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC;IACvC,CAAC;IAED;;OAEG;IACK,eAAe;QACrB,MAAM,MAAM,GAAoB;YAC9B,uBAAuB;YACvB,SAAS,EAAE,IAAI,CAAC,eAAe,CAAC,YAAY,EAAE,oBAAoB,CAAC;YACnE,WAAW,EAAE,OAAO,CAAC,GAAG,CAAC,YAAY;YAErC,2BAA2B;YAC3B,QAAQ,EAAE,OAAO,CAAC,GAAG,CAAC,SAAS;YAE/B,4BAA4B;YAC5B,YAAY,EAAE,IAAI,CAAC,aAAa,CAAC,eAAe,EAAE,IAAI,CAAC;YACvD,QAAQ,EAAE,IAAI,CAAC,YAAY,CAAC,WAAW,EAAE,MAAM,CAAC,EAAE,oBAAoB;YACtE,mBAAmB,EAAE,IAAI,CAAC,YAAY,CAAC,uBAAuB,EAAE,CAAC,CAAC;YAElE,oBAAoB;YACpB,eAAe,EAAE,IAAI,CAAC,YAAY,CAAC,kBAAkB,EAAE,GAAG,CAAC;YAC3D,UAAU,EAAE,IAAI,CAAC,YAAY,CAAC,aAAa,EAAE,CAAC,CAAC;YAC/C,YAAY,EAAE,IAAI,CAAC,YAAY,CAAC,gBAAgB,EAAE,IAAI,CAAC;YAEvD,wBAAwB;YACxB,QAAQ,EAAE,IAAI,CAAC,WAAW,EAAE;YAC5B,KAAK,EAAE,IAAI,CAAC,aAAa,CAAC,OAAO,EAAE,KAAK,CAAC;SAC1C,CAAC;QAEF,IAAI,CAAC,cAAc,CAAC,MAAM,CAAC,CAAC;QAC5B,IAAI,CAAC,kBAAkB,EAAE,CAAC;QAC1B,OAAO,MAAM,CAAC;IAChB,CAAC;IAED;;OAEG;IACK,cAAc,CAAC,MAAuB;QAC5C,MAAM,MAAM,GAAa,EAAE,CAAC;QAE5B,sBAAsB;QACtB,IAAI,CAAC,MAAM,CAAC,SAAS,EAAE,CAAC;YACtB,MAAM,CAAC,IAAI,CAAC,wBAAwB,CAAC,CAAC;QACxC,CAAC;aAAM,CAAC;YACN,IAAI,CAAC;gBACH,IAAI,GAAG,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC;YAC5B,CAAC;YAAC,MAAM,CAAC;gBACP,MAAM,CAAC,IAAI,CAAC,kCAAkC,MAAM,CAAC,SAAS,EAAE,CAAC,CAAC;YACpE,CAAC;QACH,CAAC;QAED,0BAA0B;QAC1B,IAAI,MAAM,CAAC,QAAQ,GAAG,CAAC,EAAE,CAAC;YACxB,MAAM,CAAC,IAAI,CAAC,wCAAwC,MAAM,CAAC,QAAQ,EAAE,CAAC,CAAC;QACzE,CAAC;QAED,IAAI,MAAM,CAAC,mBAAmB,GAAG,CAAC,EAAE,CAAC;YACnC,MAAM,CAAC,IAAI,CAAC,kDAAkD,MAAM,CAAC,mBAAmB,EAAE,CAAC,CAAC;QAC9F,CAAC;QAED,IAAI,MAAM,CAAC,eAAe,GAAG,CAAC,EAAE,CAAC;YAC/B,MAAM,CAAC,IAAI,CAAC,6CAA6C,MAAM,CAAC,eAAe,EAAE,CAAC,CAAC;QACrF,CAAC;QAED,IAAI,MAAM,CAAC,UAAU,GAAG,CAAC,EAAE,CAAC;YAC1B,MAAM,CAAC,IAAI,CAAC,0CAA0C,MAAM,CAAC,UAAU,EAAE,CAAC,CAAC;QAC7E,CAAC;QAED,IAAI,MAAM,CAAC,YAAY,GAAG,CAAC,EAAE,CAAC;YAC5B,MAAM,CAAC,IAAI,CAAC,6CAA6C,MAAM,CAAC,YAAY,EAAE,CAAC,CAAC;QAClF,CAAC;QAED,IAAI,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACtB,MAAM,IAAI,KAAK,CACb,qCAAqC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM;gBACjF,wDAAwD,CACzD,CAAC;QACJ,CAAC;IACH,CAAC;IAED;;OAEG;IACK,eAAe,CAAI,GAAW,EAAE,YAAe;QACrD,MAAM,KAAK,GAAG,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;QAC/B,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;YACxB,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,GAAG,EAAE,YAAY,CAAC,CAAC;YAC5C,OAAO,YAAY,CAAC;QACtB,CAAC;QACD,OAAO,KAAU,CAAC;IACpB,CAAC;IAED;;OAEG;IACK,aAAa,CAAC,GAAW,EAAE,YAAqB;QACtD,MAAM,KAAK,GAAG,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;QAC/B,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;YACxB,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,GAAG,EAAE,YAAY,CAAC,CAAC;YAC5C,OAAO,YAAY,CAAC;QACtB,CAAC;QACD,OAAO,KAAK,CAAC,WAAW,EAAE,KAAK,MAAM,IAAI,KAAK,KAAK,GAAG,CAAC;IACzD,CAAC;IAED;;OAEG;IACK,YAAY,CAAC,GAAW,EAAE,YAAoB;QACpD,MAAM,KAAK,GAAG,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;QAC/B,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;YACxB,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,GAAG,EAAE,YAAY,CAAC,CAAC;YAC5C,OAAO,YAAY,CAAC;QACtB,CAAC;QACD,MAAM,MAAM,GAAG,QAAQ,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC;QACnC,IAAI,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC;YAClB,MAAM,IAAI,KAAK,CACb,wBAAwB,GAAG,2BAA2B,KAAK,EAAE,CAC9D,CAAC;QACJ,CAAC;QACD,OAAO,MAAM,CAAC;IAChB,CAAC;IAED;;OAEG;IACK,WAAW;QACjB,MAAM,KAAK,GAAG,OAAO,CAAC,GAAG,CAAC,SAAS,EAAE,WAAW,EAAE,CAAC;QACnD,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;YACxB,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,WAAW,EAAE,MAAM,CAAC,CAAC;YAC9C,OAAO,MAAM,CAAC;QAChB,CAAC;QAED,MAAM,WAAW,GAAG,CAAC,OAAO,EAAE,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,CAAC;QACvD,IAAI,CAAC,WAAW,CAAC,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC;YACjC,MAAM,IAAI,KAAK,CACb,6BAA6B,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,UAAU,KAAK,EAAE,CACrE,CAAC;QACJ,CAAC;QAED,OAAO,KAA4C,CAAC;IACtD,CAAC;IAED;;;OAGG;IACK,kBAAkB;QACxB,IAAI,IAAI,CAAC,eAAe,CAAC,IAAI,KAAK,CAAC,EAAE,CAAC;YACpC,OAAO;QACT,CAAC;QAED,MAAM,SAAS,GAAG,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;QAC3C,OAAO,CAAC,KAAK,CAAC,IAAI,SAAS,gEAAgE,CAAC,CAAC;QAE7F,KAAK,MAAM,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC,eAAe,CAAC,OAAO,EAAE,EAAE,CAAC;YAC1D,OAAO,CAAC,KAAK,CAAC,IAAI,SAAS,8BAA8B,GAAG,MAAM,IAAI,CAAC,SAAS,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;QAC7F,CAAC;IACH,CAAC;CACF"}
+{"version":3,"file":"config.service.js","sourceRoot":"","sources":["../../src/config/config.service.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAwCH;;;;;;;GAOG;AACH,MAAM,OAAO,aAAa;IAChB,MAAM,CAAC,QAAQ,GAAyB,IAAI,CAAC;IAC7C,MAAM,CAAC,iBAAiB,GAAG,KAAK,CAAC;IAEjC,MAAM,CAAkB;IACxB,eAAe,GAAyB,IAAI,GAAG,EAAE,CAAC;IAE1D;;;;;;;OAOG;IACH,YAAY,WAAW,GAAG,KAAK;QAC7B,IAAI,CAAC,MAAM,GAAG,IAAI,CAAC,eAAe,CAAC,WAAW,CAAC,CAAC;IAClD,CAAC;IAED;;;;;;;OAOG;IACH,MAAM,CAAC,WAAW;QAChB,IAAI,CAAC,aAAa,CAAC,QAAQ,EAAE,CAAC;YAC5B,aAAa,CAAC,QAAQ,GAAG,IAAI,aAAa,EAAE,CAAC;QAC/C,CAAC;QACD,OAAO,aAAa,CAAC,QAAQ,CAAC;IAChC,CAAC;IAED;;OAEG;IACH,MAAM,CAAC,aAAa;QAClB,aAAa,CAAC,QAAQ,GAAG,IAAI,CAAC;QAC9B,aAAa,CAAC,iBAAiB,GAAG,KAAK,CAAC;IAC1C,CAAC;IAED;;OAEG;IACH,GAAG;QACD,OAAO,MAAM,CAAC,MAAM,CAAC,EAAE,GAAG,IAAI,CAAC,MAAM,EAAE,CAAC,CAAC;IAC3C,CAAC;IAED;;OAEG;IACH,kBAAkB;QAChB,6DAA6D;QAC7D,OAAO,IAAI,GAAG,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC;IACvC,CAAC;IAED;;;;OAIG;IACK,eAAe,CAAC,WAAW,GAAG,KAAK;QACzC,MAAM,MAAM,GAAoB;YAC9B,uBAAuB;YACvB,SAAS,EAAE,IAAI,CAAC,eAAe,CAAC,YAAY,EAAE,oBAAoB,CAAC;YACnE,WAAW,EAAE,OAAO,CAAC,GAAG,CAAC,YAAY;YAErC,2BAA2B;YAC3B,QAAQ,EAAE,OAAO,CAAC,GAAG,CAAC,SAAS;YAE/B,4BAA4B;YAC5B,YAAY,EAAE,IAAI,CAAC,aAAa,CAAC,eAAe,EAAE,IAAI,CAAC;YACvD,QAAQ,EAAE,IAAI,CAAC,YAAY,CAAC,WAAW,EAAE,MAAM,CAAC,EAAE,oBAAoB;YACtE,mBAAmB,EAAE,IAAI,CAAC,YAAY,CAAC,uBAAuB,EAAE,CAAC,CAAC;YAElE,oBAAoB;YACpB,eAAe,EAAE,IAAI,CAAC,YAAY,CAAC,kBAAkB,EAAE,GAAG,CAAC;YAC3D,UAAU,EAAE,IAAI,CAAC,YAAY,CAAC,aAAa,EAAE,CAAC,CAAC;YAC/C,YAAY,EAAE,IAAI,CAAC,YAAY,CAAC,gBAAgB,EAAE,IAAI,CAAC;YAEvD,wBAAwB;YACxB,QAAQ,EAAE,IAAI,CAAC,WAAW,EAAE;YAC5B,KAAK,EAAE,IAAI,CAAC,aAAa,CAAC,OAAO,EAAE,KAAK,CAAC;SAC1C,CAAC;QAEF,IAAI,CAAC,cAAc,CAAC,MAAM,CAAC,CAAC;QAC5B,IAAI,CAAC,WAAW,EAAE,CAAC;YACjB,IAAI,CAAC,kBAAkB,EAAE,CAAC;QAC5B,CAAC;QACD,OAAO,MAAM,CAAC;IAChB,CAAC;IAED;;OAEG;IACK,cAAc,CAAC,MAAuB;QAC5C,MAAM,MAAM,GAAa,EAAE,CAAC;QAE5B,sBAAsB;QACtB,IAAI,CAAC,MAAM,CAAC,SAAS,EAAE,CAAC;YACtB,MAAM,CAAC,IAAI,CAAC,wBAAwB,CAAC,CAAC;QACxC,CAAC;aAAM,CAAC;YACN,IAAI,CAAC;gBACH,IAAI,GAAG,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC;YAC5B,CAAC;YAAC,MAAM,CAAC;gBACP,MAAM,CAAC,IAAI,CAAC,kCAAkC,MAAM,CAAC,SAAS,EAAE,CAAC,CAAC;YACpE,CAAC;QACH,CAAC;QAED,0BAA0B;QAC1B,IAAI,MAAM,CAAC,QAAQ,GAAG,CAAC,EAAE,CAAC;YACxB,MAAM,CAAC,IAAI,CAAC,wCAAwC,MAAM,CAAC,QAAQ,EAAE,CAAC,CAAC;QACzE,CAAC;QAED,IAAI,MAAM,CAAC,mBAAmB,GAAG,CAAC,EAAE,CAAC;YACnC,MAAM,CAAC,IAAI,CAAC,kDAAkD,MAAM,CAAC,mBAAmB,EAAE,CAAC,CAAC;QAC9F,CAAC;QAED,IAAI,MAAM,CAAC,eAAe,GAAG,CAAC,EAAE,CAAC;YAC/B,MAAM,CAAC,IAAI,CAAC,6CAA6C,MAAM,CAAC,eAAe,EAAE,CAAC,CAAC;QACrF,CAAC;QAED,IAAI,MAAM,CAAC,UAAU,GAAG,CAAC,EAAE,CAAC;YAC1B,MAAM,CAAC,IAAI,CAAC,0CAA0C,MAAM,CAAC,UAAU,EAAE,CAAC,CAAC;QAC7E,CAAC;QAED,IAAI,MAAM,CAAC,YAAY,GAAG,CAAC,EAAE,CAAC;YAC5B,MAAM,CAAC,IAAI,CAAC,6CAA6C,MAAM,CAAC,YAAY,EAAE,CAAC,CAAC;QAClF,CAAC;QAED,IAAI,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACtB,MAAM,IAAI,KAAK,CACb,qCAAqC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM;gBACjF,wDAAwD,CACzD,CAAC;QACJ,CAAC;IACH,CAAC;IAED;;OAEG;IACK,eAAe,CAAI,GAAW,EAAE,YAAe;QACrD,MAAM,KAAK,GAAG,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;QAC/B,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;YACxB,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,GAAG,EAAE,YAAY,CAAC,CAAC;YAC5C,OAAO,YAAY,CAAC;QACtB,CAAC;QACD,OAAO,KAAU,CAAC;IACpB,CAAC;IAED;;OAEG;IACK,aAAa,CAAC,GAAW,EAAE,YAAqB;QACtD,MAAM,KAAK,GAAG,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;QAC/B,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;YACxB,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,GAAG,EAAE,YAAY,CAAC,CAAC;YAC5C,OAAO,YAAY,CAAC;QACtB,CAAC;QACD,OAAO,KAAK,CAAC,WAAW,EAAE,KAAK,MAAM,IAAI,KAAK,KAAK,GAAG,CAAC;IACzD,CAAC;IAED;;OAEG;IACK,YAAY,CAAC,GAAW,EAAE,YAAoB;QACpD,MAAM,KAAK,GAAG,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;QAC/B,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;YACxB,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,GAAG,EAAE,YAAY,CAAC,CAAC;YAC5C,OAAO,YAAY,CAAC;QACtB,CAAC;QACD,MAAM,MAAM,GAAG,QAAQ,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC;QACnC,IAAI,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC;YAClB,MAAM,IAAI,KAAK,CACb,wBAAwB,GAAG,2BAA2B,KAAK,EAAE,CAC9D,CAAC;QACJ,CAAC;QACD,OAAO,MAAM,CAAC;IAChB,CAAC;IAED;;OAEG;IACK,WAAW;QACjB,MAAM,KAAK,GAAG,OAAO,CAAC,GAAG,CAAC,SAAS,EAAE,WAAW,EAAE,CAAC;QACnD,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;YACxB,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,WAAW,EAAE,MAAM,CAAC,CAAC;YAC9C,OAAO,MAAM,CAAC;QAChB,CAAC;QAED,MAAM,WAAW,GAAG,CAAC,OAAO,EAAE,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,CAAC;QACvD,IAAI,CAAC,WAAW,CAAC,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC;YACjC,MAAM,IAAI,KAAK,CACb,6BAA6B,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,UAAU,KAAK,EAAE,CACrE,CAAC;QACJ,CAAC;QAED,OAAO,KAA4C,CAAC;IACtD,CAAC;IAED;;;;OAIG;IACK,kBAAkB;QACxB,IAAI,IAAI,CAAC,eAAe,CAAC,IAAI,KAAK,CAAC,IAAI,aAAa,CAAC,iBAAiB,EAAE,CAAC;YACvE,OAAO;QACT,CAAC;QAED,aAAa,CAAC,iBAAiB,GAAG,IAAI,CAAC;QACvC,MAAM,SAAS,GAAG,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;QAC3C,OAAO,CAAC,KAAK,CAAC,IAAI,SAAS,gEAAgE,CAAC,CAAC;QAE7F,KAAK,MAAM,CAAC,GAAG,EAAE,KAAK,CAAC,IAAI,IAAI,CAAC,eAAe,CAAC,OAAO,EAAE,EAAE,CAAC;YAC1D,OAAO,CAAC,KAAK,CAAC,IAAI,SAAS,8BAA8B,GAAG,MAAM,IAAI,CAAC,SAAS,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;QAC7F,CAAC;IACH,CAAC"}

package/dist/datasources/local-git-data-source.d.ts

@@ -23,6 +23,12 @@ export declare class LocalGitDataSource extends GitLabAPIDataSource implements G
      * @param config - Optional configuration service for settings
      */
     constructor(gitlabApi: GitLabAPI, repoPath: string, logger?: Logger, config?: ConfigService);
+    /**
+     * Get the repository path
+     *
+     * @returns The local repository path
+     */
+    get repositoryPath(): string;
     /**
      * Get the project path from the git remote URL
      *
@@ -62,10 +68,30 @@ export declare class LocalGitDataSource extends GitLabAPIDataSource implements G
      * @returns Promise resolving to array of contributor email addresses (up to 100)
      */
     getProjectContributors(project: string, commitSha: string): Promise<string[]>;
+    /**
+     * Check if a path is a git submodule (gitlink with mode 160000)
+     *
+     * @param path - The file path to check
+     * @param ref - The commit SHA to check at
+     * @returns Promise resolving to true if path is a submodule
+     */
+    private isSubmodule;
+    /**
+     * Get blame-like data for a submodule by using commit history
+     *
+     * Since git blame doesn't work on submodule gitlinks, we get the
+     * commit history for who changed the submodule reference.
+     *
+     * @param path - The submodule path
+     * @param ref - The commit SHA
+     * @returns Promise resolving to array of blame lines from commit history
+     */
+    private getSubmoduleBlame;
     /**
      * Get git blame information from local repository
      *
      * Validates commit existence before attempting blame.
+     * For submodules, uses commit history instead of blame.
      * Falls back to API if local blame fails (graceful degradation).
      *
      * @param project - The project ID or path

package/dist/datasources/local-git-data-source.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"local-git-data-source.d.ts","sourceRoot":"","sources":["../../src/datasources/local-git-data-source.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,SAAS,EAAE,MAAM,sBAAsB,CAAC;AAEjD,OAAO,EAAE,aAAa,EAAE,MAAM,gCAAgC,CAAC;AAC/D,OAAO,EAAE,mBAAmB,EAAE,MAAM,6BAA6B,CAAC;AAClE,OAAO,EAAE,MAAM,EAAE,MAAM,8BAA8B,CAAC;AAEtD,OAAO,EAAE,aAAa,EAAE,MAAM,6BAA6B,CAAC;AAC5D,OAAO,EAAE,SAAS,EAAE,gBAAgB,EAAE,MAAM,mBAAmB,CAAC;AAEhE;;;;;;GAMG;AACH,qBAAa,kBACX,SAAQ,mBACR,YAAW,aAAa;IActB,OAAO,CAAC,QAAQ;IAZlB,OAAO,CAAC,MAAM,CAAgB;IAE9B;;;;;;;OAOG;gBAED,SAAS,EAAE,SAAS,EACZ,QAAQ,EAAE,MAAM,EACxB,MAAM,CAAC,EAAE,MAAM,EACf,MAAM,CAAC,EAAE,aAAa;IAMxB;;;;;;;;OAQG;IACG,oBAAoB,IAAI,OAAO,CAAC,MAAM,CAAC;IAyD7C;;;;;OAKG;IACG,gBAAgB,IAAI,OAAO,CAAC,MAAM,CAAC;IAmCzC;;;;;;;;;OASG;IACG,gBAAgB,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC;IAwExD;;;;;;;;;OASG;IACG,sBAAsB,CAC1B,OAAO,EAAE,MAAM,EACf,SAAS,EAAE,MAAM,GAChB,OAAO,CAAC,MAAM,EAAE,CAAC;IA+EpB;;;;;;;;;;OAUG;IACG,QAAQ,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,SAAS,EAAE,CAAC;IAyFhF;;OAEG;YACW,oBAAoB;IAkBlC;;;;;;;;;;OAUG;IACG,QAAQ,CAAC,OAAO,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,GAAG,OAAO,CAAC,gBAAgB,EAAE,CAAC;IAiHxE;;OAEG;IACH,OAAO,CAAC,kBAAkB;IAc1B;;OAEG;YACW,cAAc;IA2D5B;;;;;;;;;;OAUG;IACG,UAAU,CACd,OAAO,EAAE,MAAM,EACf,MAAM,EAAE,MAAM,EACd,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC;IA8CzB;;;;;;;;OAQG;IACG,WAAW,CAAC,CAAC,GAAG,OAAO,EAC3B,OAAO,EAAE,MAAM,EACf,MAAM,EAAE,MAAM,EACd,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,CAAC,GAAG,IAAI,CAAC;IAsBpB;;;;;;;;;;;OAWG;IACG,UAAU,CAAC,OAAO,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,mBAAmB,EAAE,UAAU,CAAC;CAc7G"}
+{"version":3,"file":"local-git-data-source.d.ts","sourceRoot":"","sources":["../../src/datasources/local-git-data-source.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,SAAS,EAAE,MAAM,sBAAsB,CAAC;AAEjD,OAAO,EAAE,aAAa,EAAE,MAAM,gCAAgC,CAAC;AAC/D,OAAO,EAAE,mBAAmB,EAAE,MAAM,6BAA6B,CAAC;AAClE,OAAO,EAAE,MAAM,EAAE,MAAM,8BAA8B,CAAC;AAEtD,OAAO,EAAE,aAAa,EAAE,MAAM,6BAA6B,CAAC;AAC5D,OAAO,EAAE,SAAS,EAAE,gBAAgB,EAAE,MAAM,mBAAmB,CAAC;AAEhE;;;;;;GAMG;AACH,qBAAa,kBACX,SAAQ,mBACR,YAAW,aAAa;IActB,OAAO,CAAC,QAAQ;IAZlB,OAAO,CAAC,MAAM,CAAgB;IAE9B;;;;;;;OAOG;gBAED,SAAS,EAAE,SAAS,EACZ,QAAQ,EAAE,MAAM,EACxB,MAAM,CAAC,EAAE,MAAM,EACf,MAAM,CAAC,EAAE,aAAa;IAMxB;;;;OAIG;IACH,IAAI,cAAc,IAAI,MAAM,CAE3B;IAED;;;;;;;;OAQG;IACG,oBAAoB,IAAI,OAAO,CAAC,MAAM,CAAC;IAyD7C;;;;;OAKG;IACG,gBAAgB,IAAI,OAAO,CAAC,MAAM,CAAC;IAmCzC;;;;;;;;;OASG;IACG,gBAAgB,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC;IAwExD;;;;;;;;;OASG;IACG,sBAAsB,CAC1B,OAAO,EAAE,MAAM,EACf,SAAS,EAAE,MAAM,GAChB,OAAO,CAAC,MAAM,EAAE,CAAC;IA+EpB;;;;;;OAMG;YACW,WAAW;IAezB;;;;;;;;;OASG;YACW,iBAAiB;IAiD/B;;;;;;;;;;;OAWG;IACG,QAAQ,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,SAAS,EAAE,CAAC;IAoGhF;;OAEG;YACW,oBAAoB;IAkBlC;;;;;;;;;;OAUG;IACG,QAAQ,CAAC,OAAO,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,GAAG,OAAO,CAAC,gBAAgB,EAAE,CAAC;IAiHxE;;OAEG;IACH,OAAO,CAAC,kBAAkB;IAc1B;;OAEG;YACW,cAAc;IA2D5B;;;;;;;;;;OAUG;IACG,UAAU,CACd,OAAO,EAAE,MAAM,EACf,MAAM,EAAE,MAAM,EACd,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC;IA8CzB;;;;;;;;OAQG;IACG,WAAW,CAAC,CAAC,GAAG,OAAO,EAC3B,OAAO,EAAE,MAAM,EACf,MAAM,EAAE,MAAM,EACd,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,CAAC,GAAG,IAAI,CAAC;IAsBpB;;;;;;;;;;;OAWG;IACG,UAAU,CAAC,OAAO,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,mBAAmB,EAAE,UAAU,CAAC;CAc7G"}

package/dist/datasources/local-git-data-source.js

@@ -26,7 +26,15 @@ export class LocalGitDataSource extends GitLabAPIDataSource {
     constructor(gitlabApi, repoPath, logger, config) {
         super(gitlabApi, logger || new Logger('LocalGitDataSource'));
         this.repoPath = repoPath;
-        this.config = config || new ConfigService();
+        this.config = config || ConfigService.getInstance();
+    }
+    /**
+     * Get the repository path
+     *
+     * @returns The local repository path
+     */
+    get repositoryPath() {
+        return this.repoPath;
     }
     /**
      * Get the project path from the git remote URL
@@ -240,10 +248,88 @@ export class LocalGitDataSource extends GitLabAPIDataSource {
             throw ErrorHandler.gitOperationError('Failed to get project contributors', context, error instanceof Error ? error : new Error(String(error)));
         }
     }
+    /**
+     * Check if a path is a git submodule (gitlink with mode 160000)
+     *
+     * @param path - The file path to check
+     * @param ref - The commit SHA to check at
+     * @returns Promise resolving to true if path is a submodule
+     */
+    async isSubmodule(path, ref) {
+        try {
+            const git = simpleGit(this.repoPath);
+            const result = await git.raw(['ls-tree', ref, '--', path]);
+            // Submodules have mode 160000 (gitlink)
+            return result.startsWith('160000');
+        }
+        catch (error) {
+            // Re-throw MCPErrors, otherwise return false (graceful degradation)
+            if (error instanceof Error && error.message.includes('MCP')) {
+                throw error;
+            }
+            return false;
+        }
+    }
+    /**
+     * Get blame-like data for a submodule by using commit history
+     *
+     * Since git blame doesn't work on submodule gitlinks, we get the
+     * commit history for who changed the submodule reference.
+     *
+     * @param path - The submodule path
+     * @param ref - The commit SHA
+     * @returns Promise resolving to array of blame lines from commit history
+     */
+    async getSubmoduleBlame(path, ref) {
+        const git = simpleGit(this.repoPath);
+        try {
+            // Get the last commit that modified this submodule reference
+            const logResult = await git.raw([
+                'log',
+                '-1',
+                '--format=%H%n%ae%n%an%n%aI%n%s',
+                ref,
+                '--',
+                path,
+            ]);
+            const lines = logResult.trim().split('\n');
+            if (lines.length < 5) {
+                return [];
+            }
+            const [commitId, authorEmail, authorName, authoredDate, message] = lines;
+            return [{
+                    commit: {
+                        id: commitId,
+                        message,
+                        parent_ids: [],
+                        authored_date: authoredDate,
+                        author_name: authorName,
+                        author_email: authorEmail,
+                        committed_date: authoredDate,
+                        committer_name: authorName,
+                        committer_email: authorEmail,
+                    },
+                    lines: ['1'],
+                }];
+        }
+        catch (error) {
+            // Re-throw MCPErrors, otherwise log and return empty (graceful degradation)
+            if (error instanceof Error && error.message.includes('MCP')) {
+                throw error;
+            }
+            this.logger.debug('Failed to get submodule commit history', {
+                path,
+                ref: ref.substring(0, 8),
+                error: error instanceof Error ? error.message : String(error),
+            });
+            return [];
+        }
+    }
     /**
      * Get git blame information from local repository
      *
      * Validates commit existence before attempting blame.
+     * For submodules, uses commit history instead of blame.
      * Falls back to API if local blame fails (graceful degradation).
      *
      * @param project - The project ID or path
@@ -257,6 +343,16 @@ export class LocalGitDataSource extends GitLabAPIDataSource {
         try {
             // Validate commit exists before attempting blame (Requirement 6.1)
             await this.validateCommitExists(ref);
+            // Check if path is a submodule - use commit history instead of blame
+            if (await this.isSubmodule(path, ref)) {
+                this.logger.info('Path is a submodule, using commit history instead of blame', {
+                    ...context,
+                    isSubmodule: true,
+                });
+                const submoduleBlame = await this.getSubmoduleBlame(path, ref);
+                endTimer();
+                return submoduleBlame;
+            }
             const git = simpleGit(this.repoPath);
             // Wrap git operation with retry logic (Requirement 3.2)
             const result = await ErrorHandler.withRetry(async () => {