threadlines 0.1.15 → 0.1.16

package/dist/commands/check.js

@@ -48,33 +48,26 @@ const fs = __importStar(require("fs"));
 const path = __importStar(require("path"));
 const chalk_1 = __importDefault(require("chalk"));
 /**
- * Detect the environment where the check is being run
- * Returns: 'vercel', 'github', 'gitlab', or 'local'
+ * Detect the environment where the check is being run.
+ *
+ * CI Environment Detection:
+ * - Vercel: VERCEL=1
+ * - GitHub Actions: GITHUB_ACTIONS=1
+ * - GitLab CI: GITLAB_CI=1 or (CI=1 + CI_COMMIT_SHA)
+ * - Local: None of the above
+ *
+ * Available CI Environment Variables:
+ * - GitHub Actions: GITHUB_REPOSITORY, GITHUB_REF_NAME, GITHUB_SHA, GITHUB_BASE_REF, GITHUB_HEAD_REF
+ * - Vercel: VERCEL_GIT_REPO_OWNER, VERCEL_GIT_REPO_SLUG, VERCEL_GIT_COMMIT_REF, VERCEL_GIT_COMMIT_SHA
+ * - GitLab CI: CI_COMMIT_REF_NAME, CI_COMMIT_SHA, CI_MERGE_REQUEST_IID, CI_MERGE_REQUEST_TITLE
  */
 function detectEnvironment() {
-    console.log(chalk_1.default.gray('   [DEBUG] detectEnvironment: Starting detection...'));
-    console.log(chalk_1.default.gray(`   [DEBUG] detectEnvironment: VERCEL = "${process.env.VERCEL || 'NOT SET'}"`));
-    console.log(chalk_1.default.gray(`   [DEBUG] detectEnvironment: GITHUB_ACTIONS = "${process.env.GITHUB_ACTIONS || 'NOT SET'}"`));
-    console.log(chalk_1.default.gray(`   [DEBUG] detectEnvironment: GITLAB_CI = "${process.env.GITLAB_CI || 'NOT SET'}"`));
-    console.log(chalk_1.default.gray(`   [DEBUG] detectEnvironment: CI = "${process.env.CI || 'NOT SET'}"`));
-    console.log(chalk_1.default.gray(`   [DEBUG] detectEnvironment: CI_COMMIT_SHA = "${process.env.CI_COMMIT_SHA || 'NOT SET'}"`));
-    // Vercel: VERCEL env var is always set in Vercel builds
-    if (process.env.VERCEL) {
-        console.log(chalk_1.default.green('   [DEBUG] detectEnvironment: Detected VERCEL'));
+    if (process.env.VERCEL)
         return 'vercel';
-    }
-    // GitHub Actions: GITHUB_ACTIONS env var is always set
-    if (process.env.GITHUB_ACTIONS) {
-        console.log(chalk_1.default.green('   [DEBUG] detectEnvironment: Detected GITHUB_ACTIONS'));
+    if (process.env.GITHUB_ACTIONS)
         return 'github';
-    }
-    // GitLab CI: GITLAB_CI env var is always set, or CI is set with GitLab-specific vars
-    if (process.env.GITLAB_CI || (process.env.CI && process.env.CI_COMMIT_SHA)) {
-        console.log(chalk_1.default.green('   [DEBUG] detectEnvironment: Detected GITLAB_CI'));
+    if (process.env.GITLAB_CI || (process.env.CI && process.env.CI_COMMIT_SHA))
         return 'gitlab';
-    }
-    // Local development: none of the above
-    console.log(chalk_1.default.yellow('   [DEBUG] detectEnvironment: No CI env vars detected - returning "local"'));
     return 'local';
 }
 async function checkCommand(options) {
@@ -180,6 +173,19 @@ async function checkCommand(options) {
                     console.log(chalk_1.default.gray(`📝 Collecting git changes for branch: ${autoTarget.value}...`));
                     gitDiff = await (0, diff_1.getBranchDiff)(repoRoot, autoTarget.value);
                     reviewContext = { type: 'branch', value: autoTarget.value };
+                    // Capture commit SHA from CI env vars if available
+                    if (process.env.GITHUB_SHA) {
+                        commitSha = process.env.GITHUB_SHA;
+                        const message = await (0, diff_1.getCommitMessage)(repoRoot, process.env.GITHUB_SHA);
+                        if (message)
+                            commitMessage = message;
+                    }
+                    else if (process.env.VERCEL_GIT_COMMIT_SHA) {
+                        commitSha = process.env.VERCEL_GIT_COMMIT_SHA;
+                        const message = await (0, diff_1.getCommitMessage)(repoRoot, process.env.VERCEL_GIT_COMMIT_SHA);
+                        if (message)
+                            commitMessage = message;
+                    }
                 }
                 else if (autoTarget.type === 'commit') {
                     // Commit: use single commit
@@ -242,16 +248,6 @@ async function checkCommand(options) {
             };
         });
         // 4. Get repo name and branch name
-        console.log(chalk_1.default.gray('\n📡 Detecting repository and branch information...'));
-        // Log Vercel-specific env vars that might help with repo detection
-        if (process.env.VERCEL) {
-            console.log(chalk_1.default.gray('   [DEBUG] Vercel environment variables:'));
-            console.log(chalk_1.default.gray(`   [DEBUG]   VERCEL_GIT_REPO_OWNER = "${process.env.VERCEL_GIT_REPO_OWNER || 'NOT SET'}"`));
-            console.log(chalk_1.default.gray(`   [DEBUG]   VERCEL_GIT_REPO_SLUG = "${process.env.VERCEL_GIT_REPO_SLUG || 'NOT SET'}"`));
-            console.log(chalk_1.default.gray(`   [DEBUG]   VERCEL_GIT_REPO_ID = "${process.env.VERCEL_GIT_REPO_ID || 'NOT SET'}"`));
-            console.log(chalk_1.default.gray(`   [DEBUG]   VERCEL_GIT_COMMIT_REF = "${process.env.VERCEL_GIT_COMMIT_REF || 'NOT SET'}"`));
-            console.log(chalk_1.default.gray(`   [DEBUG]   VERCEL_GIT_COMMIT_SHA = "${process.env.VERCEL_GIT_COMMIT_SHA || 'NOT SET'}"`));
-        }
         const repoName = await (0, repo_1.getRepoName)(repoRoot);
         const branchName = await (0, repo_1.getBranchName)(repoRoot);
         // 5. Get API URL
@@ -259,19 +255,9 @@ async function checkCommand(options) {
             process.env.THREADLINE_API_URL ||
             'https://devthreadline.com';
         // 6. Detect environment
-        console.log(chalk_1.default.gray('\n🌍 Detecting environment...'));
         const environment = detectEnvironment();
-        // 7. Log final values being sent to API
-        console.log(chalk_1.default.gray('\n📤 Final values being sent to API:'));
-        console.log(chalk_1.default.gray(`   repoName: ${repoName ? `"${repoName}"` : 'null/undefined'}`));
-        console.log(chalk_1.default.gray(`   branchName: ${branchName ? `"${branchName}"` : 'null/undefined'}`));
-        console.log(chalk_1.default.gray(`   commitSha: ${commitSha ? `"${commitSha}"` : 'null/undefined'}`));
-        console.log(chalk_1.default.gray(`   commitMessage: ${commitMessage ? `"${commitMessage.substring(0, 50)}..."` : 'null/undefined'}`));
-        console.log(chalk_1.default.gray(`   prTitle: ${prTitle ? `"${prTitle}"` : 'null/undefined'}`));
-        console.log(chalk_1.default.gray(`   environment: "${environment}"`));
-        console.log(chalk_1.default.gray(`   reviewContext: ${JSON.stringify(reviewContext)}`));
-        // 8. Call review API
-        console.log(chalk_1.default.gray('\n🤖 Running threadline checks...'));
+        // 7. Call review API
+        console.log(chalk_1.default.gray('🤖 Running threadline checks...'));
         const client = new client_1.ReviewAPIClient(apiUrl);
         const response = await client.review({
             threadlines: threadlinesWithContext,

package/dist/git/repo.js

@@ -6,72 +6,58 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.getRepoName = getRepoName;
 exports.getBranchName = getBranchName;
 const simple_git_1 = __importDefault(require("simple-git"));
-const chalk_1 = __importDefault(require("chalk"));
 /**
- * Gets the raw git remote URL from origin.
- * Returns the URL exactly as Git provides it - no parsing or normalization.
- * Server-side can normalize variations if needed based on actual data.
+ * Gets repository URL. Prefers CI environment variables over git commands.
+ *
+ * CI Environment Variables:
+ * - GitHub Actions: GITHUB_REPOSITORY (format: "owner/repo"), GITHUB_SERVER_URL
+ * - Vercel: VERCEL_GIT_REPO_OWNER, VERCEL_GIT_REPO_SLUG
+ * - GitLab CI: Not available (falls back to git)
+ * - Local: Falls back to git origin remote
  */
 async function getRepoName(repoRoot) {
+    // GitHub Actions: GITHUB_REPOSITORY = "owner/repo"
+    if (process.env.GITHUB_REPOSITORY) {
+        const serverUrl = process.env.GITHUB_SERVER_URL || 'https://github.com';
+        return `${serverUrl}/${process.env.GITHUB_REPOSITORY}.git`;
+    }
+    // Vercel: Construct from owner + slug
+    if (process.env.VERCEL_GIT_REPO_OWNER && process.env.VERCEL_GIT_REPO_SLUG) {
+        return `https://github.com/${process.env.VERCEL_GIT_REPO_OWNER}/${process.env.VERCEL_GIT_REPO_SLUG}.git`;
+    }
+    // Fallback: git origin remote (local dev, GitLab CI)
     const git = (0, simple_git_1.default)(repoRoot);
-    console.log(chalk_1.default.gray('   [DEBUG] getRepoName: Starting...'));
-    console.log(chalk_1.default.gray(`   [DEBUG] getRepoName: repoRoot = ${repoRoot}`));
     try {
         const remotes = await git.getRemotes(true);
-        console.log(chalk_1.default.gray(`   [DEBUG] getRepoName: Found ${remotes.length} remote(s)`));
-        if (remotes.length > 0) {
-            remotes.forEach((remote, idx) => {
-                console.log(chalk_1.default.gray(`   [DEBUG] getRepoName: Remote[${idx}] name="${remote.name}", fetch="${remote.refs?.fetch || 'N/A'}", push="${remote.refs?.push || 'N/A'}"`));
-            });
-        }
-        else {
-            console.log(chalk_1.default.yellow('   [DEBUG] getRepoName: No remotes found'));
-        }
         const origin = remotes.find(r => r.name === 'origin');
-        if (!origin) {
-            console.log(chalk_1.default.yellow('   [DEBUG] getRepoName: No "origin" remote found'));
-            return null;
-        }
-        console.log(chalk_1.default.gray(`   [DEBUG] getRepoName: Found origin remote: fetch="${origin.refs?.fetch || 'N/A'}"`));
-        if (!origin.refs?.fetch) {
-            console.log(chalk_1.default.yellow('   [DEBUG] getRepoName: Origin remote found but no fetch URL'));
-            return null;
-        }
-        console.log(chalk_1.default.green(`   [DEBUG] getRepoName: Success - returning "${origin.refs.fetch}"`));
-        // Return raw URL - let server handle normalization if needed
-        return origin.refs.fetch;
+        return origin?.refs?.fetch || null;
     }
-    catch (error) {
-        console.log(chalk_1.default.red(`   [DEBUG] getRepoName: Error occurred: ${error.message || error}`));
-        if (error.stack) {
-            console.log(chalk_1.default.gray(`   [DEBUG] getRepoName: Stack: ${error.stack}`));
-        }
+    catch {
         return null;
     }
 }
 /**
- * Gets current branch name from git.
- * Returns null if not in a git repo or error occurs.
+ * Gets branch name. Prefers CI environment variables over git commands.
+ *
+ * CI Environment Variables:
+ * - GitHub Actions: GITHUB_REF_NAME
+ * - Vercel: VERCEL_GIT_COMMIT_REF
+ * - GitLab CI: CI_COMMIT_REF_NAME
+ * - Local: Falls back to git revparse --abbrev-ref HEAD
  */
 async function getBranchName(repoRoot) {
+    if (process.env.GITHUB_REF_NAME)
+        return process.env.GITHUB_REF_NAME;
+    if (process.env.VERCEL_GIT_COMMIT_REF)
+        return process.env.VERCEL_GIT_COMMIT_REF;
+    if (process.env.CI_COMMIT_REF_NAME)
+        return process.env.CI_COMMIT_REF_NAME;
+    // Fallback: git command
     const git = (0, simple_git_1.default)(repoRoot);
-    console.log(chalk_1.default.gray('   [DEBUG] getBranchName: Starting...'));
-    console.log(chalk_1.default.gray(`   [DEBUG] getBranchName: repoRoot = ${repoRoot}`));
     try {
-        const branch = await git.revparse(['--abbrev-ref', 'HEAD']);
-        console.log(chalk_1.default.gray(`   [DEBUG] getBranchName: git revparse --abbrev-ref HEAD returned: "${branch || '(empty)'}"`));
-        if (!branch) {
-            console.log(chalk_1.default.yellow('   [DEBUG] getBranchName: Git command returned empty string'));
-            return null;
-        }
-        console.log(chalk_1.default.green(`   [DEBUG] getBranchName: Success - returning "${branch}"`));
-        return branch;
+        return await git.revparse(['--abbrev-ref', 'HEAD']) || null;
     }
-    catch (error) {
-        console.log(chalk_1.default.red(`   [DEBUG] getBranchName: Error occurred: ${error.message || error}`));
-        if (error.stack) {
-            console.log(chalk_1.default.gray(`   [DEBUG] getBranchName: Stack: ${error.stack}`));
-        }
+    catch {
         return null;
     }
 }

package/dist/utils/ci-detection.js

@@ -1,42 +1,15 @@
 "use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getAutoReviewTarget = getAutoReviewTarget;
-const chalk_1 = __importDefault(require("chalk"));
 function getAutoReviewTarget() {
-    console.log(chalk_1.default.gray('   [DEBUG] getAutoReviewTarget: Starting detection...'));
-    // Log ALL GitHub Actions env vars that start with GITHUB_ to see what's available
-    if (process.env.GITHUB_ACTIONS) {
-        console.log(chalk_1.default.gray('   [DEBUG] getAutoReviewTarget: All GITHUB_* env vars:'));
-        const githubVars = Object.keys(process.env)
-            .filter(key => key.startsWith('GITHUB_'))
-            .sort();
-        githubVars.forEach(key => {
-            const value = process.env[key];
-            // Truncate long values for readability
-            const displayValue = value && value.length > 100 ? value.substring(0, 100) + '...' : value;
-            console.log(chalk_1.default.gray(`     ${key} = "${displayValue || 'NOT SET'}"`));
-        });
-    }
-    // 1. Check for PR/MR context (most authoritative)
-    // GitHub Actions PR
-    console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: Checking GitHub Actions PR...`));
-    console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: GITHUB_EVENT_NAME = "${process.env.GITHUB_EVENT_NAME || 'NOT SET'}"`));
+    // 1. PR/MR context
     if (process.env.GITHUB_EVENT_NAME === 'pull_request') {
         const targetBranch = process.env.GITHUB_BASE_REF;
         const sourceBranch = process.env.GITHUB_HEAD_REF;
-        // Try multiple possible env var names for PR number
-        const prNumber = process.env.GITHUB_EVENT_PULL_REQUEST_NUMBER ||
-            process.env.GITHUB_PR_NUMBER ||
-            process.env.GITHUB_EVENT_NUMBER;
-        console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: GITHUB_BASE_REF = "${targetBranch || 'NOT SET'}"`));
-        console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: GITHUB_HEAD_REF = "${sourceBranch || 'NOT SET'}"`));
-        console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: GITHUB_EVENT_PULL_REQUEST_NUMBER = "${process.env.GITHUB_EVENT_PULL_REQUEST_NUMBER || 'NOT SET'}"`));
-        console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: GITHUB_EVENT_NUMBER = "${process.env.GITHUB_EVENT_NUMBER || 'NOT SET'}"`));
+        const prNumber = process.env.GITHUB_EVENT_PULL_REQUEST_NUMBER || process.env.GITHUB_EVENT_NUMBER;
+        // Log PR detection for verification (remove after confirming it works)
+        console.log(`[PR Detection] GITHUB_BASE_REF=${targetBranch || 'NOT SET'}, GITHUB_HEAD_REF=${sourceBranch || 'NOT SET'}, PR_NUMBER=${prNumber || 'NOT SET'}`);
         if (targetBranch && sourceBranch && prNumber) {
-            console.log(chalk_1.default.green(`   [DEBUG] getAutoReviewTarget: Detected GitHub PR #${prNumber}`));
             return {
                 type: 'pr',
                 value: prNumber,
@@ -44,66 +17,36 @@ function getAutoReviewTarget() {
                 targetBranch
             };
         }
-        else {
-            console.log(chalk_1.default.yellow('   [DEBUG] getAutoReviewTarget: GitHub PR env vars incomplete'));
-        }
     }
-    // GitLab CI MR
-    console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: Checking GitLab CI MR...`));
-    console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: CI_MERGE_REQUEST_IID = "${process.env.CI_MERGE_REQUEST_IID || 'NOT SET'}"`));
     if (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;
         const mrNumber = process.env.CI_MERGE_REQUEST_IID;
-        const mrTitle = process.env.CI_MERGE_REQUEST_TITLE; // Reliable GitLab CI env var
-        console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: CI_MERGE_REQUEST_TARGET_BRANCH_NAME = "${targetBranch || 'NOT SET'}"`));
-        console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: CI_MERGE_REQUEST_SOURCE_BRANCH_NAME = "${sourceBranch || 'NOT SET'}"`));
-        console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: CI_MERGE_REQUEST_TITLE = "${mrTitle || 'NOT SET'}"`));
+        const mrTitle = process.env.CI_MERGE_REQUEST_TITLE;
         if (targetBranch && sourceBranch && mrNumber) {
-            console.log(chalk_1.default.green(`   [DEBUG] getAutoReviewTarget: Detected GitLab MR #${mrNumber}`));
             return {
                 type: 'mr',
                 value: mrNumber,
                 sourceBranch,
                 targetBranch,
-                prTitle: mrTitle || undefined // Only include if present
+                prTitle: mrTitle || undefined
             };
         }
-        else {
-            console.log(chalk_1.default.yellow('   [DEBUG] getAutoReviewTarget: GitLab MR env vars incomplete'));
-        }
     }
-    // 2. Check for branch name (CI with branch)
-    console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: Checking branch env vars...`));
-    console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: GITHUB_REF_NAME = "${process.env.GITHUB_REF_NAME || 'NOT SET'}"`));
-    console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: CI_COMMIT_REF_NAME = "${process.env.CI_COMMIT_REF_NAME || 'NOT SET'}"`));
-    console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: VERCEL_GIT_COMMIT_REF = "${process.env.VERCEL_GIT_COMMIT_REF || 'NOT SET'}"`));
-    const branch = process.env.GITHUB_REF_NAME || // GitHub Actions
-        process.env.CI_COMMIT_REF_NAME || // GitLab CI
-        process.env.VERCEL_GIT_COMMIT_REF; // Vercel
+    // 2. Branch name
+    const branch = process.env.GITHUB_REF_NAME ||
+        process.env.CI_COMMIT_REF_NAME ||
+        process.env.VERCEL_GIT_COMMIT_REF;
     if (branch) {
-        console.log(chalk_1.default.green(`   [DEBUG] getAutoReviewTarget: Detected branch "${branch}"`));
-        return {
-            type: 'branch',
-            value: branch
-        };
+        return { type: 'branch', value: branch };
     }
-    // 3. Check for commit SHA (CI without branch)
-    console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: Checking commit SHA env vars...`));
-    console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: GITHUB_SHA = "${process.env.GITHUB_SHA || 'NOT SET'}"`));
-    console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: CI_COMMIT_SHA = "${process.env.CI_COMMIT_SHA || 'NOT SET'}"`));
-    console.log(chalk_1.default.gray(`   [DEBUG] getAutoReviewTarget: VERCEL_GIT_COMMIT_SHA = "${process.env.VERCEL_GIT_COMMIT_SHA || 'NOT SET'}"`));
-    const commit = process.env.GITHUB_SHA || // GitHub Actions
-        process.env.CI_COMMIT_SHA || // GitLab CI
-        process.env.VERCEL_GIT_COMMIT_SHA; // Vercel
+    // 3. Commit SHA
+    const commit = process.env.GITHUB_SHA ||
+        process.env.CI_COMMIT_SHA ||
+        process.env.VERCEL_GIT_COMMIT_SHA;
     if (commit) {
-        console.log(chalk_1.default.green(`   [DEBUG] getAutoReviewTarget: Detected commit "${commit}"`));
-        return {
-            type: 'commit',
-            value: commit
-        };
+        return { type: 'commit', value: commit };
     }
-    // 4. Local development (no CI env vars)
-    console.log(chalk_1.default.yellow('   [DEBUG] getAutoReviewTarget: No CI env vars detected - returning null (local mode)'));
+    // 4. Local development
     return null;
 }

package/package.json

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