threadlines 0.1.10 → 0.1.12

package/dist/commands/check.js

@@ -89,6 +89,9 @@ async function checkCommand(options) {
         // 2. Determine review target and get git diff
         let gitDiff;
         let reviewContext = { type: 'local' };
+        let commitSha = undefined;
+        let commitMessage = undefined;
+        let prTitle = undefined;
         // Validate mutually exclusive flags
         const explicitFlags = [options.branch, options.commit, options.file, options.folder, options.files].filter(Boolean);
         if (explicitFlags.length > 1) {
@@ -121,6 +124,12 @@ async function checkCommand(options) {
             console.log(chalk_1.default.gray(`📝 Collecting git changes for commit: ${options.commit}...`));
             gitDiff = await (0, diff_1.getCommitDiff)(repoRoot, options.commit);
             reviewContext = { type: 'commit', value: options.commit };
+            commitSha = options.commit;
+            // Fetch commit message (reliable when we have SHA)
+            const message = await (0, diff_1.getCommitMessage)(repoRoot, options.commit);
+            if (message) {
+                commitMessage = message;
+            }
         }
         else {
             // Auto-detect CI environment or use local changes
@@ -131,6 +140,10 @@ async function checkCommand(options) {
                     console.log(chalk_1.default.gray(`📝 Collecting git changes for ${autoTarget.type.toUpperCase()}: ${autoTarget.value}...`));
                     gitDiff = await (0, diff_1.getPRMRDiff)(repoRoot, autoTarget.sourceBranch, autoTarget.targetBranch);
                     reviewContext = { type: autoTarget.type, value: autoTarget.value };
+                    // Use PR title from GitLab CI (reliable env var)
+                    if (autoTarget.prTitle) {
+                        prTitle = autoTarget.prTitle;
+                    }
                 }
                 else if (autoTarget.type === 'branch') {
                     // Branch: use branch vs base
@@ -143,6 +156,12 @@ async function checkCommand(options) {
                     console.log(chalk_1.default.gray(`📝 Collecting git changes for commit: ${autoTarget.value}...`));
                     gitDiff = await (0, diff_1.getCommitDiff)(repoRoot, autoTarget.value);
                     reviewContext = { type: 'commit', value: autoTarget.value };
+                    commitSha = autoTarget.value;
+                    // Fetch commit message (reliable when we have SHA)
+                    const message = await (0, diff_1.getCommitMessage)(repoRoot, autoTarget.value);
+                    if (message) {
+                        commitMessage = message;
+                    }
                 }
                 else {
                     // Fallback: local development
@@ -210,7 +229,10 @@ async function checkCommand(options) {
             apiKey,
             account,
             repoName: repoName || undefined,
-            branchName: branchName || undefined
+            branchName: branchName || undefined,
+            commitSha: commitSha,
+            commitMessage: commitMessage,
+            prTitle: prTitle
         });
         // 7. Display results (with filtering if --full not specified)
         displayResults(response, options.full || false);

package/dist/git/diff.js

@@ -5,6 +5,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getGitDiff = getGitDiff;
 exports.getBranchDiff = getBranchDiff;
+exports.getCommitMessage = getCommitMessage;
 exports.getCommitDiff = getCommitDiff;
 exports.getPRMRDiff = getPRMRDiff;
 const simple_git_1 = __importDefault(require("simple-git"));
@@ -23,11 +24,11 @@ async function getGitDiff(repoRoot) {
     let diff;
     if (status.staged.length > 0) {
         // Use staged changes
-        diff = await git.diff(['--cached']);
+        diff = await git.diff(['--cached', '-U200']);
     }
     else if (status.files.length > 0) {
         // Use unstaged changes
-        diff = await git.diff();
+        diff = await git.diff(['-U200']);
     }
     else {
         // No changes
@@ -73,7 +74,7 @@ async function getBranchDiff(repoRoot, branchName, baseBranch) {
                 const previousCommit = await git.revparse(['HEAD~1']).catch(() => null);
                 if (previousCommit) {
                     // Use commit-based diff instead
-                    const diff = await git.diff([`${previousCommit}..HEAD`]);
+                    const diff = await git.diff([`${previousCommit}..HEAD`, '-U200']);
                     const diffSummary = await git.diffSummary([`${previousCommit}..HEAD`]);
                     const changedFiles = diffSummary.files.map(f => f.file);
                     return {
@@ -135,7 +136,7 @@ async function getBranchDiff(repoRoot, branchName, baseBranch) {
     }
     // Get diff between base and branch (cumulative diff of all commits)
     // Format: git diff base...branch (three-dot notation finds common ancestor)
-    const diff = await git.diff([`${base}...${branchName}`]);
+    const diff = await git.diff([`${base}...${branchName}`, '-U200']);
     // Get list of changed files
     const diffSummary = await git.diffSummary([`${base}...${branchName}`]);
     const changedFiles = diffSummary.files.map(f => f.file);
@@ -144,6 +145,22 @@ async function getBranchDiff(repoRoot, branchName, baseBranch) {
         changedFiles
     };
 }
+/**
+ * Get commit message for a specific commit SHA
+ * Returns full commit message (subject + body) or null if commit not found
+ */
+async function getCommitMessage(repoRoot, sha) {
+    const git = (0, simple_git_1.default)(repoRoot);
+    try {
+        // Get full commit message (subject + body)
+        const message = await git.show([sha, '--format=%B', '--no-patch']);
+        return message.trim() || null;
+    }
+    catch (error) {
+        // Commit not found or invalid
+        return null;
+    }
+}
 /**
  * Get diff for a specific commit
  */
@@ -160,7 +177,7 @@ async function getCommitDiff(repoRoot, sha) {
     let changedFiles;
     try {
         // Get diff using git show
-        diff = await git.show([sha, '--format=', '--no-color']);
+        diff = await git.show([sha, '--format=', '--no-color', '-U200']);
         // Get changed files using git show --name-only
         const commitFiles = await git.show([sha, '--name-only', '--format=', '--pretty=format:']);
         changedFiles = commitFiles
@@ -171,7 +188,7 @@ async function getCommitDiff(repoRoot, sha) {
     catch (error) {
         // Fallback: try git diff format
         try {
-            diff = await git.diff([`${sha}^..${sha}`]);
+            diff = await git.diff([`${sha}^..${sha}`, '-U200']);
             // Get files from diff summary
             const diffSummary = await git.diffSummary([`${sha}^..${sha}`]);
             changedFiles = diffSummary.files.map(f => f.file);
@@ -197,7 +214,7 @@ async function getPRMRDiff(repoRoot, sourceBranch, targetBranch) {
     }
     // Get diff between target and source (cumulative diff)
     // Format: git diff target...source (three-dot notation finds common ancestor)
-    const diff = await git.diff([`${targetBranch}...${sourceBranch}`]);
+    const diff = await git.diff([`${targetBranch}...${sourceBranch}`, '-U200']);
     // Get list of changed files
     const diffSummary = await git.diffSummary([`${targetBranch}...${sourceBranch}`]);
     const changedFiles = diffSummary.files.map(f => f.file);

package/dist/git/repo.js

@@ -7,46 +7,22 @@ exports.getRepoName = getRepoName;
 exports.getBranchName = getBranchName;
 const simple_git_1 = __importDefault(require("simple-git"));
 /**
- * Gets repository name from git remote URL.
- * Parses common formats: github.com/user/repo, gitlab.com/user/repo, etc.
- * Returns null if no remote or parsing fails.
+ * 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.
  */
 async function getRepoName(repoRoot) {
     const git = (0, simple_git_1.default)(repoRoot);
     try {
         const remotes = await git.getRemotes(true);
         const origin = remotes.find(r => r.name === 'origin');
-        if (!origin) {
-            console.log('   [DEBUG] No origin remote found');
+        if (!origin || !origin.refs?.fetch) {
             return null;
         }
-        if (!origin.refs?.fetch) {
-            console.log('   [DEBUG] Origin remote found but no fetch URL');
-            return null;
-        }
-        const url = origin.refs.fetch;
-        console.log(`   [DEBUG] Git remote URL: ${url}`);
-        // Parse repo name from common URL formats:
-        // - https://github.com/user/repo.git
-        // - git@github.com:user/repo.git
-        // - https://gitlab.com/user/repo.git
-        // - git@gitlab.com:user/repo.git
-        const patterns = [
-            /(?:github\.com|gitlab\.com)[\/:]([^\/]+\/[^\/]+?)(?:\.git)?$/,
-            /([^\/]+\/[^\/]+?)(?:\.git)?$/
-        ];
-        for (const pattern of patterns) {
-            const match = url.match(pattern);
-            if (match && match[1]) {
-                console.log(`   [DEBUG] Matched repo name: ${match[1]}`);
-                return match[1];
-            }
-        }
-        console.log('   [DEBUG] No pattern matched the remote URL');
-        return null;
+        // Return raw URL - let server handle normalization if needed
+        return origin.refs.fetch;
     }
     catch (error) {
-        console.log(`   [DEBUG] Error getting repo name: ${error.message}`);
         return null;
     }
 }

package/dist/utils/ci-detection.js

@@ -22,12 +22,14 @@ function getAutoReviewTarget() {
         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
         if (targetBranch && sourceBranch && mrNumber) {
             return {
                 type: 'mr',
                 value: mrNumber,
                 sourceBranch,
-                targetBranch
+                targetBranch,
+                prTitle: mrTitle || undefined // Only include if present
             };
         }
     }

package/package.json

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