threadlines 0.2.20 → 0.2.21

package/dist/commands/check.js

@@ -246,7 +246,7 @@ async function checkCommand(options) {
         console.log('');
         process.exit(0);
     }
-    console.log(chalk_1.default.green(`✓ Found ${gitDiff.changedFiles.length} changed file(s)\n`));
+    console.log(chalk_1.default.green(`✓ Found ${gitDiff.changedFiles.length} changed file(s) (context: ${reviewContext})\n`));
     // Log the files being sent
     for (const file of gitDiff.changedFiles) {
         logger_1.logger.info(`  → ${file}`);

package/dist/git/diff.js

@@ -194,9 +194,13 @@ async function getPRDiff(repoRoot, targetBranch, logger) {
  * This works regardless of CI checkout depth settings (depth=1 or depth=2).
  *
  * Strategy:
- * 1. Get parent SHA from commit metadata (git show --format=%P)
- * 2. Fetch parent commit if available (git fetch origin <parentSHA> --depth=1)
- * 3. Use git show to get diff (now parent is available for comparison)
+ * 1. Get parent SHA using plumbing command (git cat-file -p) to read raw commit object
+ *    - Plumbing commands ignore .git/shallow boundaries and show actual parent SHA
+ *    - Porcelain commands (git show) respect shallow boundaries and hide parents in shallow clones
+ *    - This is critical for CI environments that use shallow clones (depth=1)
+ * 2. Parse first parent line (handles standard commits and merge commits)
+ * 3. Fetch parent commit if available (git fetch origin <parentSHA> --depth=1)
+ * 4. Use git show to get diff (now parent is available for comparison)
  *
  * Used by:
  * - All CI environments for push/commit context (GitHub, GitLab, Bitbucket, Vercel)
@@ -210,13 +214,31 @@ async function getCommitDiff(repoRoot, sha = 'HEAD') {
     // Fetch parent commit on-demand to ensure git show can generate a proper diff
     // This works regardless of CI checkout depth settings (depth=1 or depth=2)
     // If parent is already available, fetch is fast/no-op; if not, we fetch it
-    // Get parent SHA from commit metadata (works even if parent isn't fetched locally)
+    // Get parent SHA using plumbing command (git cat-file) instead of porcelain (git show)
+    // Plumbing commands ignore .git/shallow boundaries and show the actual parent SHA
+    // Porcelain commands (git show) respect shallow boundaries and hide parents in shallow clones
+    // This is critical for CI environments that use shallow clones (depth=1)
     let parentSha;
     try {
-        parentSha = (0, child_process_1.execSync)(`git show ${sha} --format=%P --no-patch`, {
+        // Use git cat-file -p to read raw commit object (plumbing command)
+        // This ignores shallow boundaries and shows the actual parent SHA
+        const commitObject = (0, child_process_1.execSync)(`git cat-file -p ${sha}`, {
             encoding: 'utf-8',
             cwd: repoRoot
-        }).trim();
+        });
+        // Parse commit object to find first parent line
+        // Standard commits have one parent; merge commits have multiple parents
+        // We use the first parent (standard for diffing against previous state of branch)
+        const lines = commitObject.split('\n');
+        const parentLine = lines.find(line => line.startsWith('parent '));
+        if (!parentLine) {
+            throw new Error(`Commit ${sha} has no parent (it might be the root commit of the repository)`);
+        }
+        // Extract SHA from "parent <sha>" line
+        parentSha = parentLine.split(' ')[1].trim();
+        if (!parentSha || parentSha.length !== 40) {
+            throw new Error(`Invalid parent SHA format: "${parentSha}"`);
+        }
     }
     catch (error) {
         const errorMessage = error instanceof Error ? error.message : String(error);
@@ -224,7 +246,7 @@ async function getCommitDiff(repoRoot, sha = 'HEAD') {
             `This is required to generate a proper diff. ` +
             `Error: ${errorMessage}`);
     }
-    // Fetch parent commit if we have a parent (root commits have no parent)
+    // Fetch parent commit (root commits have no parent, so this won't execute for them)
     if (parentSha && parentSha.length === 40) {
         try {
             // Fetch just this one commit (depth=1 is fine, we only need the parent)

package/package.json

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