threadlines 0.1.36 → 0.1.38

package/dist/commands/check.js

@@ -422,18 +422,23 @@ function displayResults(response, showFull) {
     // Show attention items
     if (attentionItems.length > 0) {
         for (const item of attentionItems) {
-            console.log(chalk_1.default.yellow(`⚠️  ${item.expertId}`));
+            console.log(chalk_1.default.yellow(`[attention] ${item.expertId}`));
             if (item.fileReferences && item.fileReferences.length > 0) {
+                // List all files as bullet points
                 for (const fileRef of item.fileReferences) {
                     const lineRef = item.lineReferences?.[item.fileReferences.indexOf(fileRef)];
                     const lineStr = lineRef ? `:${lineRef}` : '';
-                    console.log(chalk_1.default.gray(`   ${fileRef}${lineStr} - ${item.reasoning || 'Needs attention'}`));
+                    console.log(chalk_1.default.gray(`* ${fileRef}${lineStr}`));
                 }
             }
-            else if (item.reasoning) {
-                console.log(chalk_1.default.gray(`   ${item.reasoning}`));
+            // Show reasoning once at the end (if available)
+            if (item.reasoning) {
+                console.log(chalk_1.default.gray(item.reasoning));
             }
+            else if (!item.fileReferences || item.fileReferences.length === 0) {
+                console.log(chalk_1.default.gray('Needs attention'));
+            }
+            console.log(''); // Empty line between threadlines
         }
-        console.log('');
     }
 }

package/dist/git/vercel.js

@@ -17,6 +17,7 @@ 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 in one call - completely isolated from other environments.
@@ -75,7 +76,7 @@ async function getDiff(repoRoot) {
         .filter(line => line.trim().length > 0)
         .map(line => line.trim());
     return {
-        diff: diff || '',
+        diff: diff || '', // Empty diff is legitimate (e.g., metadata-only commits, merge commits)
         changedFiles
     };
 }
@@ -115,7 +116,10 @@ function getCommitSha() {
 }
 /**
  * Gets commit author for Vercel
- * Uses VERCEL_GIT_COMMIT_AUTHOR_NAME for name, git log for email
+ * 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;
@@ -123,14 +127,20 @@ async function getCommitAuthorForVercel(repoRoot, commitSha) {
         throw new Error('Vercel: VERCEL_GIT_COMMIT_AUTHOR_NAME environment variable is not set. ' +
             'This should be automatically provided by Vercel.');
     }
-    // Get email from git log - fail loudly if this doesn't work
-    const gitAuthor = await (0, diff_1.getCommitAuthor)(repoRoot, commitSha);
-    if (!gitAuthor || !gitAuthor.email) {
+    // 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.`);
+            `This should be available in Vercel's build environment. ` +
+            `Error: ${error instanceof Error ? error.message : 'Unknown error'}`);
     }
-    return {
-        name: authorName.trim(),
-        email: gitAuthor.email.trim()
-    };
 }

package/package.json

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