threadlines 0.2.13 → 0.2.15

package/dist/commands/check.js

@@ -47,7 +47,6 @@ const gitlab_1 = require("../git/gitlab");
 const bitbucket_1 = require("../git/bitbucket");
 const vercel_1 = require("../git/vercel");
 const local_1 = require("../git/local");
-const diff_1 = require("../git/diff");
 const config_file_1 = require("../utils/config-file");
 const logger_1 = require("../utils/logger");
 const fs = __importStar(require("fs"));
@@ -68,8 +67,11 @@ async function getContextForEnvironment(environment, repoRoot, commitSha) {
             return (0, bitbucket_1.getBitbucketContext)(repoRoot);
         case 'vercel':
             return (0, vercel_1.getVercelContext)(repoRoot);
-        default:
+        case 'local':
             return (0, local_1.getLocalContext)(repoRoot, commitSha);
+        default:
+            // TypeScript exhaustiveness check - should never happen
+            throw new Error(`Unrecognized environment: ${environment}`);
     }
 }
 // Get CLI version from package.json
@@ -93,18 +95,20 @@ async function checkCommand(options) {
         }
         gitRoot = (await git.revparse(['--show-toplevel'])).trim();
     }
-    catch {
-        logger_1.logger.error('Failed to get git root. Make sure you are in a git repository.');
+    catch (error) {
+        const message = error instanceof Error ? error.message : String(error);
+        logger_1.logger.error(`Failed to get git root: ${message}`);
         process.exit(1);
     }
     // Pre-flight check: Validate ALL required environment variables at once
     const apiKey = (0, config_1.getThreadlineApiKey)();
     const account = (0, config_1.getThreadlineAccount)();
     const missingVars = [];
-    // Check for undefined, empty string, or literal unexpanded variable (GitLab keeps "$VAR" literal)
-    if (!apiKey || apiKey.startsWith('$'))
+    // Check for undefined, empty string, or literal unexpanded variable
+    // GitLab CI keeps variables as literal "$VAR" if not defined in CI/CD settings
+    if (!apiKey || apiKey === '$THREADLINE_API_KEY')
         missingVars.push('THREADLINE_API_KEY');
-    if (!account || account.startsWith('$'))
+    if (!account || account === '$THREADLINE_ACCOUNT')
         missingVars.push('THREADLINE_ACCOUNT');
     if (missingVars.length > 0) {
         logger_1.logger.error('Missing required environment variables:');
@@ -134,189 +138,171 @@ async function checkCommand(options) {
         console.log(chalk_1.default.gray('Get your credentials at: https://devthreadline.com/settings'));
         process.exit(1);
     }
-    try {
-        // 1. Find and validate threadlines
-        logger_1.logger.info('Finding threadlines...');
-        const threadlines = await (0, experts_1.findThreadlines)(cwd, gitRoot);
-        console.log(chalk_1.default.green(`✓ Found ${threadlines.length} threadline(s)\n`));
-        if (threadlines.length === 0) {
-            console.log(chalk_1.default.yellow('⚠️  No valid threadlines found.'));
-            console.log(chalk_1.default.gray('   Run `npx threadlines init` to create your first threadline.'));
-            process.exit(0);
-        }
-        // 2. Detect environment and context
-        const environment = (0, environment_1.detectEnvironment)();
-        let gitDiff;
-        let repoName;
-        let branchName;
-        let reviewContext;
-        let metadata = {};
-        // Check for explicit flags
-        const explicitFlags = [options.commit, options.file, options.folder, options.files].filter(Boolean);
-        // Validate mutually exclusive flags
-        if (explicitFlags.length > 1) {
-            logger_1.logger.error('Only one review option can be specified at a time');
-            console.log(chalk_1.default.gray('   Options: --commit, --file, --folder, --files'));
-            process.exit(1);
+    // 1. Find and validate threadlines
+    logger_1.logger.info('Finding threadlines...');
+    const threadlines = await (0, experts_1.findThreadlines)(cwd, gitRoot);
+    console.log(chalk_1.default.green(`✓ Found ${threadlines.length} threadline(s)\n`));
+    if (threadlines.length === 0) {
+        console.log(chalk_1.default.yellow('⚠️  No valid threadlines found.'));
+        console.log(chalk_1.default.gray('   Run `npx threadlines init` to create your first threadline.'));
+        process.exit(0);
+    }
+    // 2. Detect environment and context
+    const environment = (0, environment_1.detectEnvironment)();
+    let gitDiff;
+    let repoName;
+    let branchName;
+    let reviewContext;
+    let metadata = {};
+    // Check for explicit flags
+    const explicitFlags = [options.commit, options.file, options.folder, options.files].filter(Boolean);
+    // Validate mutually exclusive flags
+    if (explicitFlags.length > 1) {
+        logger_1.logger.error('Only one review option can be specified at a time');
+        console.log(chalk_1.default.gray('   Options: --commit, --file, --folder, --files'));
+        process.exit(1);
+    }
+    // CI environments: auto-detect only, flags are ignored with warning
+    // Local: full flag support for developer flexibility
+    if ((0, environment_1.isCIEnvironment)(environment)) {
+        // Warn if flags are passed in CI - they're meant for local development
+        if (explicitFlags.length > 0) {
+            const flagName = options.commit ? '--commit' :
+                options.file ? '--file' :
+                    options.folder ? '--folder' : '--files';
+            logger_1.logger.warn(`${flagName} flag ignored in CI environment. Using auto-detection.`);
         }
-        // CI environments: auto-detect only, flags are ignored with warning
-        // Local: full flag support for developer flexibility
-        if ((0, environment_1.isCIEnvironment)(environment)) {
-            // Warn if flags are passed in CI - they're meant for local development
-            if (explicitFlags.length > 0) {
-                const flagName = options.commit ? '--commit' :
-                    options.file ? '--file' :
-                        options.folder ? '--folder' : '--files';
-                logger_1.logger.warn(`${flagName} flag ignored in CI environment. Using auto-detection.`);
-            }
-            // CI auto-detect: use environment-specific context
-            const envNames = {
-                vercel: 'Vercel',
-                github: 'GitHub Actions',
-                gitlab: 'GitLab CI',
-                bitbucket: 'Bitbucket Pipelines'
-            };
-            logger_1.logger.info(`Collecting git context for ${envNames[environment]}...`);
-            const envContext = await getContextForEnvironment(environment, repoRoot);
-            gitDiff = envContext.diff;
-            repoName = envContext.repoName;
-            branchName = envContext.branchName;
-            reviewContext = envContext.reviewContext; // Get from CI context
-            metadata = {
-                commitSha: envContext.commitSha,
-                commitMessage: envContext.commitMessage,
-                commitAuthorName: envContext.commitAuthor.name,
-                commitAuthorEmail: envContext.commitAuthor.email,
-                prTitle: envContext.prTitle
-            };
+        // CI auto-detect: use environment-specific context
+        const envNames = {
+            vercel: 'Vercel',
+            github: 'GitHub Actions',
+            gitlab: 'GitLab CI',
+            bitbucket: 'Bitbucket Pipelines'
+        };
+        logger_1.logger.info(`Collecting git context for ${envNames[environment]}...`);
+        const envContext = await getContextForEnvironment(environment, repoRoot);
+        gitDiff = envContext.diff;
+        repoName = envContext.repoName;
+        branchName = envContext.branchName;
+        reviewContext = envContext.reviewContext; // Get from CI context
+        metadata = {
+            commitSha: envContext.commitSha,
+            commitMessage: envContext.commitMessage,
+            commitAuthorName: envContext.commitAuthor.name,
+            commitAuthorEmail: envContext.commitAuthor.email,
+            prTitle: envContext.prTitle
+        };
+    }
+    else {
+        // Local environment: all flags share the same metadata
+        // 1. Get context and metadata (pass commit SHA if provided)
+        logger_1.logger.info('Collecting local context...');
+        const localContext = await (0, local_1.getLocalContext)(repoRoot, options.commit);
+        repoName = localContext.repoName;
+        branchName = localContext.branchName;
+        metadata = {
+            commitSha: localContext.commitSha,
+            commitMessage: localContext.commitMessage,
+            commitAuthorName: localContext.commitAuthor.name,
+            commitAuthorEmail: localContext.commitAuthor.email
+        };
+        // 2. Get diff (override with specific content if flag provided)
+        if (options.file) {
+            reviewContext = 'file';
+            logger_1.logger.info(`Reading file: ${options.file}...`);
+            gitDiff = await (0, file_1.getFileContent)(repoRoot, options.file);
         }
-        else {
-            // Local environment: support all flags
-            // Detect review context from flags (priority order: file > folder > files > commit > local)
-            if (options.file) {
-                reviewContext = 'file';
-            }
-            else if (options.folder) {
-                reviewContext = 'folder';
-            }
-            else if (options.files && options.files.length > 0) {
-                reviewContext = 'files';
-            }
-            else if (options.commit) {
-                reviewContext = 'commit';
-            }
-            else {
-                reviewContext = 'local';
-            }
-            if (options.file) {
-                logger_1.logger.info(`Reading file: ${options.file}...`);
-                gitDiff = await (0, file_1.getFileContent)(repoRoot, options.file);
-            }
-            else if (options.folder) {
-                logger_1.logger.info(`Reading folder: ${options.folder}...`);
-                gitDiff = await (0, file_1.getFolderContent)(repoRoot, options.folder);
-            }
-            else if (options.files && options.files.length > 0) {
-                logger_1.logger.info(`Reading ${options.files.length} file(s)...`);
-                gitDiff = await (0, file_1.getMultipleFilesContent)(repoRoot, options.files);
-            }
-            else if (options.commit) {
-                logger_1.logger.info(`Collecting git changes for commit: ${options.commit}...`);
-                gitDiff = await (0, diff_1.getCommitDiff)(repoRoot, options.commit);
-                // Use local context for metadata, passing commit SHA for author lookup
-                const localContext = await (0, local_1.getLocalContext)(repoRoot, options.commit);
-                repoName = localContext.repoName;
-                branchName = localContext.branchName;
-                metadata = {
-                    commitSha: localContext.commitSha,
-                    commitMessage: localContext.commitMessage,
-                    commitAuthorName: localContext.commitAuthor.name,
-                    commitAuthorEmail: localContext.commitAuthor.email
-                };
-            }
-            else {
-                // Local auto-detect: staged/unstaged changes
-                logger_1.logger.info('Collecting git context for Local...');
-                const localContext = await (0, local_1.getLocalContext)(repoRoot);
-                gitDiff = localContext.diff;
-                repoName = localContext.repoName;
-                branchName = localContext.branchName;
-                metadata = {
-                    commitSha: localContext.commitSha,
-                    commitMessage: localContext.commitMessage,
-                    commitAuthorName: localContext.commitAuthor.name,
-                    commitAuthorEmail: localContext.commitAuthor.email
-                };
-            }
+        else if (options.folder) {
+            reviewContext = 'folder';
+            logger_1.logger.info(`Reading folder: ${options.folder}...`);
+            gitDiff = await (0, file_1.getFolderContent)(repoRoot, options.folder);
         }
-        if (gitDiff.changedFiles.length === 0) {
-            console.error(chalk_1.default.bold('ℹ️ No changes detected.'));
-            process.exit(0);
+        else if (options.files && options.files.length > 0) {
+            reviewContext = 'files';
+            logger_1.logger.info(`Reading ${options.files.length} file(s)...`);
+            gitDiff = await (0, file_1.getMultipleFilesContent)(repoRoot, options.files);
         }
-        // Check for zero diff (files changed but no actual code changes)
-        if (!gitDiff.diff || gitDiff.diff.trim() === '') {
-            console.log(chalk_1.default.blue('ℹ️  No code changes detected. Diff contains zero lines added or removed.'));
-            console.log(chalk_1.default.gray(`   ${gitDiff.changedFiles.length} file(s) changed but no content modifications detected.`));
-            console.log('');
-            console.log(chalk_1.default.bold('Results:\n'));
-            console.log(chalk_1.default.gray(`${threadlines.length} threadlines checked`));
-            console.log(chalk_1.default.gray(`  ${threadlines.length} not relevant`));
-            console.log('');
-            process.exit(0);
+        else {
+            // Default: use diff from localContext (handles commit and staged/unstaged)
+            reviewContext = options.commit ? 'commit' : 'local';
+            gitDiff = localContext.diff;
         }
-        console.log(chalk_1.default.green(`✓ Found ${gitDiff.changedFiles.length} changed file(s)\n`));
-        // 4. Read context files for each threadline
-        const threadlinesWithContext = threadlines.map(threadline => {
-            const contextContent = {};
-            if (threadline.contextFiles) {
-                for (const contextFile of threadline.contextFiles) {
-                    const fullPath = path.join(repoRoot, contextFile);
-                    if (fs.existsSync(fullPath)) {
+    }
+    if (gitDiff.changedFiles.length === 0) {
+        console.error(chalk_1.default.bold('ℹ️ No changes detected.'));
+        process.exit(0);
+    }
+    // Check for zero diff (files changed but no actual code changes)
+    if (!gitDiff.diff || gitDiff.diff.trim() === '') {
+        console.log(chalk_1.default.blue('ℹ️  No code changes detected. Diff contains zero lines added or removed.'));
+        console.log(chalk_1.default.gray(`   ${gitDiff.changedFiles.length} file(s) changed but no content modifications detected.`));
+        console.log('');
+        console.log(chalk_1.default.bold('Results:\n'));
+        console.log(chalk_1.default.gray(`${threadlines.length} threadlines checked`));
+        console.log(chalk_1.default.gray(`  ${threadlines.length} not relevant`));
+        console.log('');
+        process.exit(0);
+    }
+    console.log(chalk_1.default.green(`✓ Found ${gitDiff.changedFiles.length} changed file(s)\n`));
+    // Log the files being sent
+    for (const file of gitDiff.changedFiles) {
+        logger_1.logger.info(`  → ${file}`);
+    }
+    // 4. Read context files for each threadline
+    const threadlinesWithContext = threadlines.map(threadline => {
+        const contextContent = {};
+        if (threadline.contextFiles) {
+            for (const contextFile of threadline.contextFiles) {
+                const fullPath = path.join(repoRoot, contextFile);
+                if (fs.existsSync(fullPath)) {
+                    try {
                         contextContent[contextFile] = fs.readFileSync(fullPath, 'utf-8');
                     }
+                    catch (error) {
+                        const message = error instanceof Error ? error.message : String(error);
+                        throw new Error(`Failed to read context file '${contextFile}' for threadline '${threadline.id}': ${message}`);
+                    }
+                }
+                else {
+                    throw new Error(`Context file not found for threadline '${threadline.id}': ${contextFile}`);
                 }
             }
-            return {
-                id: threadline.id,
-                version: threadline.version,
-                patterns: threadline.patterns,
-                content: threadline.content,
-                filePath: threadline.filePath,
-                contextFiles: threadline.contextFiles,
-                contextContent
-            };
-        });
-        // 5. Call review API
-        logger_1.logger.info('Running threadline checks...');
-        const client = new client_1.ReviewAPIClient(config.api_url);
-        const response = await client.review({
-            threadlines: threadlinesWithContext,
-            diff: gitDiff.diff,
-            files: gitDiff.changedFiles,
-            apiKey: apiKey,
-            account: account,
-            repoName: repoName,
-            branchName: branchName,
-            commitSha: metadata.commitSha,
-            commitMessage: metadata.commitMessage,
-            commitAuthorName: metadata.commitAuthorName,
-            commitAuthorEmail: metadata.commitAuthorEmail,
-            prTitle: metadata.prTitle,
-            environment: environment,
-            cliVersion: CLI_VERSION,
-            reviewContext: reviewContext
-        });
-        // 7. Display results (with filtering if --full not specified)
-        displayResults(response, options.full || false);
-        // Exit with appropriate code (attention or errors = failure)
-        const hasIssues = response.results.some(r => r.status === 'attention' || r.status === 'error');
-        process.exit(hasIssues ? 1 : 0);
-    }
-    catch (error) {
-        const errorMessage = error instanceof Error ? error.message : 'Unknown error';
-        logger_1.logger.error(errorMessage);
-        process.exit(1);
-    }
+        }
+        return {
+            id: threadline.id,
+            version: threadline.version,
+            patterns: threadline.patterns,
+            content: threadline.content,
+            filePath: threadline.filePath,
+            contextFiles: threadline.contextFiles,
+            contextContent
+        };
+    });
+    // 5. Call review API
+    logger_1.logger.info('Running threadline checks...');
+    const client = new client_1.ReviewAPIClient(config.api_url);
+    const response = await client.review({
+        threadlines: threadlinesWithContext,
+        diff: gitDiff.diff,
+        files: gitDiff.changedFiles,
+        apiKey: apiKey,
+        account: account,
+        repoName: repoName,
+        branchName: branchName,
+        commitSha: metadata.commitSha,
+        commitMessage: metadata.commitMessage,
+        commitAuthorName: metadata.commitAuthorName,
+        commitAuthorEmail: metadata.commitAuthorEmail,
+        prTitle: metadata.prTitle,
+        environment: environment,
+        cliVersion: CLI_VERSION,
+        reviewContext: reviewContext
+    });
+    // 7. Display results (with filtering if --full not specified)
+    displayResults(response, options.full || false);
+    // Exit with appropriate code (attention or errors = failure)
+    const hasIssues = response.results.some(r => r.status === 'attention' || r.status === 'error');
+    process.exit(hasIssues ? 1 : 0);
 }
 function displayResults(response, showFull) {
     const { results, metadata, message } = response;
@@ -429,10 +415,6 @@ function displayResults(response, showFull) {
                     console.log(chalk_1.default.gray(JSON.stringify(item.error.rawResponse, null, 2).split('\n').map(line => '    ' + line).join('\n')));
                 }
             }
-            else if (item.reasoning) {
-                // Fallback to reasoning if no error object
-                console.log(chalk_1.default.red(`  ${item.reasoning}`));
-            }
             console.log(''); // Empty line between errors
         }
     }

package/dist/git/bitbucket.js

@@ -23,7 +23,6 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getBitbucketContext = getBitbucketContext;
 const simple_git_1 = __importDefault(require("simple-git"));
-const child_process_1 = require("child_process");
 const diff_1 = require("./diff");
 const logger_1 = require("../utils/logger");
 /**
@@ -43,8 +42,9 @@ async function getBitbucketContext(repoRoot) {
     const context = detectContext();
     const reviewContext = detectReviewContext();
     const commitSha = getCommitSha();
-    // Get commit author (from git log - Bitbucket doesn't provide this as env var)
-    const commitAuthor = await getCommitAuthor(repoRoot);
+    // Get commit author using shared function (git log)
+    // getCommitAuthor throws on failure with descriptive error
+    const commitAuthor = await (0, diff_1.getCommitAuthor)(repoRoot);
     // Get commit message if we have a SHA
     let commitMessage;
     if (commitSha) {
@@ -172,32 +172,3 @@ function detectReviewContext() {
 function getCommitSha() {
     return process.env.BITBUCKET_COMMIT;
 }
-/**
- * Gets commit author for Bitbucket Pipelines
- *
- * Bitbucket doesn't provide commit author as an environment variable,
- * so we use git log to get it.
- *
- * This approach is verified by our test script (test-bitbucket-context.ts)
- * which successfully retrieves commit author in all scenarios:
- * - Direct commit to main
- * - Feature branch push
- * - PR pipeline
- * - Merge commit
- */
-async function getCommitAuthor(repoRoot) {
-    // Use raw git commands - this is exactly what the test script uses and we know it works
-    try {
-        const name = (0, child_process_1.execSync)('git log -1 --format=%an', { encoding: 'utf-8', cwd: repoRoot }).trim();
-        const email = (0, child_process_1.execSync)('git log -1 --format=%ae', { encoding: 'utf-8', cwd: repoRoot }).trim();
-        if (!name || !email) {
-            throw new Error('git log returned empty name or email');
-        }
-        return { name, email };
-    }
-    catch (error) {
-        const errorMessage = error instanceof Error ? error.message : 'Unknown error';
-        throw new Error(`Bitbucket Pipelines: Failed to get commit author from git log. ` +
-            `Error: ${errorMessage}`);
-    }
-}

package/dist/git/diff.js

@@ -5,6 +5,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getCommitMessage = getCommitMessage;
 exports.getCommitAuthor = getCommitAuthor;
+exports.getPRDiff = getPRDiff;
 exports.getCommitDiff = getCommitDiff;
 const simple_git_1 = __importDefault(require("simple-git"));
 const child_process_1 = require("child_process");
@@ -29,31 +30,92 @@ async function getCommitMessage(repoRoot, sha) {
  *
  * Uses raw git log command to extract author information.
  * Works in all environments where git is available.
+ *
+ * Throws on error - git commits always have authors, so failure indicates
+ * an invalid SHA or repository issue that should surface immediately.
+ *
+ * Used by: GitHub, GitLab, Bitbucket, Vercel, Local (all CI environments)
  */
 async function getCommitAuthor(repoRoot, sha) {
+    const commitRef = sha || 'HEAD';
+    let output;
     try {
         // Use raw git command (same as test scripts) - more reliable than simple-git API
-        const commitRef = sha || 'HEAD';
         const command = `git log -1 --format="%an <%ae>" ${commitRef}`;
-        const output = (0, child_process_1.execSync)(command, {
+        output = (0, child_process_1.execSync)(command, {
             encoding: 'utf-8',
             cwd: repoRoot
         }).trim();
-        // Parse output: "Name <email>"
-        const match = output.match(/^(.+?)\s*<(.+?)>$/);
-        if (!match) {
-            return null;
-        }
-        const name = match[1].trim();
-        const email = match[2].trim();
-        if (!name || !email) {
-            return null;
-        }
-        return { name, email };
     }
-    catch {
-        return null;
+    catch (error) {
+        const message = error instanceof Error ? error.message : String(error);
+        throw new Error(`Failed to get commit author for ${commitRef}: ${message}`);
+    }
+    // Parse output: "Name <email>"
+    const match = output.match(/^(.+?)\s*<(.+?)>$/);
+    if (!match) {
+        throw new Error(`Failed to parse commit author for ${commitRef}. ` +
+            `Expected format "Name <email>", got: "${output}"`);
+    }
+    const name = match[1].trim();
+    const email = match[2].trim();
+    if (!name || !email) {
+        throw new Error(`Commit author for ${commitRef} has empty name or email. ` +
+            `Got name="${name}", email="${email}"`);
+    }
+    return { name, email };
+}
+/**
+ * Get diff for a PR/MR context in CI environments.
+ *
+ * This is a shared implementation for CI environments that do shallow clones.
+ * It fetches the target branch on-demand and compares it against HEAD.
+ *
+ * Strategy:
+ * 1. Fetch target branch: origin/${targetBranch}:refs/remotes/origin/${targetBranch}
+ * 2. Diff: origin/${targetBranch}..HEAD (two dots = direct comparison)
+ *
+ * Why HEAD instead of origin/${sourceBranch}?
+ * - CI shallow clones only have HEAD available by default
+ * - origin/${sourceBranch} doesn't exist until explicitly fetched
+ * - HEAD IS the source branch in PR/MR pipelines
+ *
+ * Currently used by:
+ * - GitLab CI (gitlab.ts)
+ *
+ * Future plan:
+ * - Azure DevOps will use this when added
+ * - Once proven stable in multiple environments, consider migrating
+ *   GitHub (github.ts) and Bitbucket (bitbucket.ts) to use this shared
+ *   implementation instead of their inline versions.
+ *
+ * @param repoRoot - Path to the repository root
+ * @param targetBranch - The branch being merged INTO (e.g., "main", "develop")
+ * @param logger - Optional logger for debug output
+ */
+async function getPRDiff(repoRoot, targetBranch, logger) {
+    const git = (0, simple_git_1.default)(repoRoot);
+    // Fetch target branch on-demand (works with shallow clones)
+    logger?.debug(`Fetching target branch: origin/${targetBranch}`);
+    try {
+        await git.fetch(['origin', `${targetBranch}:refs/remotes/origin/${targetBranch}`, '--depth=1']);
+    }
+    catch (fetchError) {
+        throw new Error(`Failed to fetch target branch origin/${targetBranch}. ` +
+            `This is required for PR/MR diff comparison. ` +
+            `Error: ${fetchError instanceof Error ? fetchError.message : 'Unknown error'}`);
     }
+    // Use two dots (..) for direct comparison (same as GitHub)
+    // Two dots: shows all changes in HEAD that aren't in origin/${targetBranch}
+    // Three dots: requires finding merge base which can fail with shallow clones
+    logger?.debug(`Comparing origin/${targetBranch}..HEAD`);
+    const diff = await git.diff([`origin/${targetBranch}..HEAD`, '-U200']);
+    const diffSummary = await git.diffSummary([`origin/${targetBranch}..HEAD`]);
+    const changedFiles = diffSummary.files.map(f => f.file);
+    return {
+        diff: diff || '',
+        changedFiles
+    };
 }
 /**
  * Get diff for a specific commit

package/dist/git/file.js

@@ -56,15 +56,17 @@ async function getFileContent(repoRoot, filePath) {
     }
     // Read file content
     const content = fs.readFileSync(fullPath, 'utf-8');
+    // Normalize path to forward slashes for cross-platform consistency (git uses forward slashes)
+    const normalizedPath = filePath.replace(/\\/g, '/');
     // Create artificial diff (all lines as additions)
     const lines = content.split('\n');
     const diff = lines.map((line) => `+${line}`).join('\n');
-    // Add diff header
-    const diffHeader = `--- /dev/null\n+++ ${filePath}\n@@ -0,0 +1,${lines.length} @@\n`;
+    // Add git diff header (matches format expected by server's filterDiffByFiles)
+    const diffHeader = `diff --git a/${normalizedPath} b/${normalizedPath}\n--- /dev/null\n+++ b/${normalizedPath}\n@@ -0,0 +1,${lines.length} @@\n`;
     const fullDiff = diffHeader + diff;
     return {
         diff: fullDiff,
-        changedFiles: [filePath]
+        changedFiles: [normalizedPath]
     };
 }
 /**
@@ -81,8 +83,8 @@ async function getFolderContent(repoRoot, folderPath) {
     if (!stats.isDirectory()) {
         throw new Error(`Path '${folderPath}' is not a folder`);
     }
-    // Find all files recursively
-    const pattern = path.join(fullPath, '**', '*');
+    // Find all files recursively (normalize to forward slashes for glob on Windows)
+    const pattern = path.join(fullPath, '**', '*').replace(/\\/g, '/');
     const files = await (0, glob_1.glob)(pattern, {
         cwd: repoRoot,
         absolute: false,
@@ -108,11 +110,13 @@ async function getFolderContent(repoRoot, folderPath) {
         try {
             const content = fs.readFileSync(path.resolve(repoRoot, filePath), 'utf-8');
             const lines = content.split('\n');
-            // Create artificial diff for this file
+            // Normalize path to forward slashes for cross-platform consistency
+            const normalizedPath = filePath.replace(/\\/g, '/');
+            // Create artificial diff for this file (git diff format)
             const fileDiff = lines.map((line) => `+${line}`).join('\n');
-            const diffHeader = `--- /dev/null\n+++ ${filePath}\n@@ -0,0 +1,${lines.length} @@\n`;
+            const diffHeader = `diff --git a/${normalizedPath} b/${normalizedPath}\n--- /dev/null\n+++ b/${normalizedPath}\n@@ -0,0 +1,${lines.length} @@\n`;
             diffs.push(diffHeader + fileDiff);
-            changedFiles.push(filePath);
+            changedFiles.push(normalizedPath);
         }
         catch (error) {
             // Skip files that can't be read (permissions, etc.)
@@ -148,11 +152,13 @@ async function getMultipleFilesContent(repoRoot, filePaths) {
         // Read file content
         const content = fs.readFileSync(fullPath, 'utf-8');
         const lines = content.split('\n');
-        // Create artificial diff for this file
+        // Normalize path to forward slashes for cross-platform consistency
+        const normalizedPath = filePath.replace(/\\/g, '/');
+        // Create artificial diff for this file (git diff format)
         const fileDiff = lines.map((line) => `+${line}`).join('\n');
-        const diffHeader = `--- /dev/null\n+++ ${filePath}\n@@ -0,0 +1,${lines.length} @@\n`;
+        const diffHeader = `diff --git a/${normalizedPath} b/${normalizedPath}\n--- /dev/null\n+++ b/${normalizedPath}\n@@ -0,0 +1,${lines.length} @@\n`;
         diffs.push(diffHeader + fileDiff);
-        changedFiles.push(filePath);
+        changedFiles.push(normalizedPath);
     }
     return {
         diff: diffs.join('\n'),

package/dist/git/github.js

@@ -43,6 +43,7 @@ async function getGitHubContext(repoRoot) {
             'This should be automatically provided by GitHub Actions.');
     }
     // Get commit author using git commands (same approach as Bitbucket/Local)
+    // getCommitAuthor throws on failure with descriptive error
     const commitAuthor = await (0, diff_1.getCommitAuthor)(repoRoot, commitSha);
     // Get commit message if we have a SHA
     let commitMessage;
@@ -54,11 +55,6 @@ async function getGitHubContext(repoRoot) {
     }
     // Get PR title if in PR context
     const prTitle = getPRTitle(context);
-    // Validate commit author was found
-    if (!commitAuthor) {
-        throw new Error(`GitHub Actions: Failed to get commit author from git log for commit ${commitSha || 'HEAD'}. ` +
-            'This should be automatically available in the git repository.');
-    }
     return {
         diff,
         repoName,

package/dist/git/gitlab.js

@@ -37,8 +37,9 @@ async function getGitLabContext(repoRoot) {
     const context = detectContext();
     const reviewContext = detectReviewContext();
     const commitSha = getCommitSha(context);
-    // Get commit author (fails loudly if unavailable)
-    const commitAuthor = await getCommitAuthor();
+    // Get commit author using shared function (git log)
+    // getCommitAuthor throws on failure with descriptive error
+    const commitAuthor = await (0, diff_1.getCommitAuthor)(repoRoot);
     // Get commit message if we have a SHA
     let commitMessage;
     if (commitSha) {
@@ -65,7 +66,7 @@ async function getGitLabContext(repoRoot) {
  * Get diff for GitLab CI environment
  *
  * Strategy:
- * - MR context: Fetch target branch, compare source vs target (full MR diff)
+ * - MR context: Uses shared getPRDiff() - fetches target branch, compares against HEAD
  * - Any push (main or feature branch): Compare last commit only (HEAD~1...HEAD)
  *
  * Note: GitLab CI does a shallow clone, so we fetch the target branch for MR context.
@@ -75,20 +76,13 @@ async function getDiff(repoRoot) {
     const git = (0, simple_git_1.default)(repoRoot);
     const mrIid = process.env.CI_MERGE_REQUEST_IID;
     const targetBranch = process.env.CI_MERGE_REQUEST_TARGET_BRANCH_NAME;
-    const sourceBranch = process.env.CI_MERGE_REQUEST_SOURCE_BRANCH_NAME;
-    // MR Context: Fetch target branch and compare
+    // MR Context: Use shared getPRDiff() implementation
     if (mrIid) {
-        if (!targetBranch || !sourceBranch) {
-            throw new Error('GitLab MR context detected but CI_MERGE_REQUEST_TARGET_BRANCH_NAME or ' +
-                'CI_MERGE_REQUEST_SOURCE_BRANCH_NAME is missing. ' +
+        if (!targetBranch) {
+            throw new Error('GitLab MR context detected but CI_MERGE_REQUEST_TARGET_BRANCH_NAME is missing. ' +
                 'This should be automatically provided by GitLab CI.');
         }
-        logger_1.logger.debug(`Fetching target branch: origin/${targetBranch}`);
-        await git.fetch(['origin', `${targetBranch}:refs/remotes/origin/${targetBranch}`, '--depth=1']);
-        const diff = await git.diff([`origin/${targetBranch}...origin/${sourceBranch}`, '-U200']);
-        const diffSummary = await git.diffSummary([`origin/${targetBranch}...origin/${sourceBranch}`]);
-        const changedFiles = diffSummary.files.map(f => f.file);
-        return { diff: diff || '', changedFiles };
+        return (0, diff_1.getPRDiff)(repoRoot, targetBranch, logger_1.logger);
     }
     // Any push (main or feature branch): Review last commit only
     const diff = await git.diff(['HEAD~1...HEAD', '-U200']);
@@ -173,28 +167,6 @@ function getCommitSha(context) {
     }
     return undefined;
 }
-/**
- * Gets commit author for GitLab CI
- * Uses CI_COMMIT_AUTHOR environment variable (most reliable)
- */
-async function getCommitAuthor() {
-    const commitAuthor = process.env.CI_COMMIT_AUTHOR;
-    if (!commitAuthor) {
-        throw new Error('GitLab CI: CI_COMMIT_AUTHOR environment variable is not set. ' +
-            'This should be automatically provided by GitLab CI.');
-    }
-    // Parse "name <email>" format
-    const match = commitAuthor.match(/^(.+?)\s*<(.+?)>$/);
-    if (!match) {
-        throw new Error(`GitLab CI: CI_COMMIT_AUTHOR format is invalid. ` +
-            `Expected format: "name <email>", got: "${commitAuthor}". ` +
-            `This should be automatically provided by GitLab CI in the correct format.`);
-    }
-    return {
-        name: match[1].trim(),
-        email: match[2].trim()
-    };
-}
 /**
  * Gets MR title for GitLab CI
  */

package/dist/git/local.js

@@ -177,15 +177,8 @@ async function getCommitAuthorFromConfig(repoRoot) {
 }
 /**
  * Gets commit author from git log (for specific commits)
+ * getCommitAuthor throws on failure with descriptive error
  */
 async function getCommitAuthorFromGit(repoRoot, commitSha) {
-    const gitAuthor = await (0, diff_1.getCommitAuthor)(repoRoot, commitSha);
-    if (!gitAuthor || !gitAuthor.email) {
-        throw new Error(`Local: Failed to get commit author from git log for commit ${commitSha}. ` +
-            'This should be available in your local git repository.');
-    }
-    return {
-        name: gitAuthor.name,
-        email: gitAuthor.email
-    };
+    return (0, diff_1.getCommitAuthor)(repoRoot, commitSha);
 }

package/dist/git/vercel.js

@@ -17,7 +17,6 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getVercelContext = getVercelContext;
 const simple_git_1 = __importDefault(require("simple-git"));
-const child_process_1 = require("child_process");
 const diff_1 = require("./diff");
 /**
  * Gets all Vercel context
@@ -36,8 +35,9 @@ async function getVercelContext(repoRoot) {
     const commitSha = getCommitSha();
     const context = { type: 'commit', commitSha };
     const reviewContext = 'commit';
-    // Get commit author (fails loudly if unavailable)
-    const commitAuthor = await getCommitAuthorForVercel(repoRoot, commitSha);
+    // Get commit author using shared function (git log)
+    // getCommitAuthor throws on failure with descriptive error
+    const commitAuthor = await (0, diff_1.getCommitAuthor)(repoRoot, commitSha);
     // Get commit message
     let commitMessage;
     const message = await (0, diff_1.getCommitMessage)(repoRoot, commitSha);
@@ -116,33 +116,3 @@ function getCommitSha() {
     }
     return commitSha;
 }
-/**
- * Gets commit author for Vercel
- * Uses VERCEL_GIT_COMMIT_AUTHOR_NAME for name, raw git log command for email
- *
- * Uses raw `git log` command (same as test script) instead of simple-git library
- * because simple-git's log method may not work correctly in Vercel's shallow clone.
- */
-async function getCommitAuthorForVercel(repoRoot, commitSha) {
-    const authorName = process.env.VERCEL_GIT_COMMIT_AUTHOR_NAME;
-    if (!authorName) {
-        throw new Error('Vercel: VERCEL_GIT_COMMIT_AUTHOR_NAME environment variable is not set. ' +
-            'This should be automatically provided by Vercel.');
-    }
-    // Use raw git log command (same approach as test script) - more reliable than simple-git
-    try {
-        const email = (0, child_process_1.execSync)(`git log ${commitSha} -1 --format=%ae`, { encoding: 'utf-8', cwd: repoRoot }).trim();
-        if (!email) {
-            throw new Error('Email is empty');
-        }
-        return {
-            name: authorName.trim(),
-            email: email.trim()
-        };
-    }
-    catch (error) {
-        throw new Error(`Vercel: Failed to get commit author email from git log for commit ${commitSha}. ` +
-            `This should be available in Vercel's build environment. ` +
-            `Error: ${error instanceof Error ? error.message : 'Unknown error'}`);
-    }
-}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "threadlines",
-  "version": "0.2.13",
+  "version": "0.2.15",
   "description": "Threadlines CLI - AI-powered linter based on your natural language documentation",
   "main": "dist/index.js",
   "bin": {