@link-assistant/hive-mind 1.23.3 → 1.23.4

package/CHANGELOG.md

@@ -1,5 +1,30 @@
 # @link-assistant/hive-mind
 
+## 1.23.4
+
+### Patch Changes
+
+- 22a1940: fix: display skip/fail reasons in merge queue Telegram messages (#1294)
+
+  Previously, when PRs were skipped or failed during merge queue processing, the Telegram message only showed the PR number without explaining why it was skipped. This left users unable to understand what action was required to resolve the issue.
+
+  Now the merge queue displays the reason for each skipped or failed PR in both:
+  - Progress messages (during processing)
+  - Final report messages (after completion)
+
+  Example output:
+
+  ```
+  Results:
+  ⏭️ #1241 (Issue #1240): PR has merge conflicts
+  ⏭️ #1257 (Issue #1256): PR has merge conflicts
+  ```
+
+  This change follows UX best practices for error messages by:
+  - Showing the specific reason for each failure
+  - Using clear, human-readable language
+  - Helping users understand what action is needed
+
 ## 1.23.3
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@link-assistant/hive-mind",
-  "version": "1.23.3",
+  "version": "1.23.4",
   "description": "AI-powered issue solver and hive mind for collaborative problem solving",
   "main": "src/hive.mjs",
   "type": "module",

package/src/telegram-merge-queue.lib.mjs

@@ -462,15 +462,17 @@ export class MergeQueueProcessor {
       message += `${statusEmoji} ${update.current}\n\n`;
     }
 
-    // Show errors/failures inline so user gets immediate feedback (Issue #1269)
-    const failedItems = update.items.filter(item => item.status === MergeItemStatus.FAILED && item.error);
-    if (failedItems.length > 0) {
-      message += `⚠️ *Errors:*\n`;
-      for (const item of failedItems.slice(0, 3)) {
-        message += `  \\#${item.prNumber}: ${this.escapeMarkdown(item.error.substring(0, 60))}${item.error.length > 60 ? '...' : ''}\n`;
+    // Show errors/failures/skips inline so user gets immediate feedback (Issue #1269, #1294)
+    // Include both FAILED and SKIPPED items with their reasons
+    const problemItems = update.items.filter(item => (item.status === MergeItemStatus.FAILED || item.status === MergeItemStatus.SKIPPED) && item.error);
+    if (problemItems.length > 0) {
+      message += `⚠️ *Issues:*\n`;
+      for (const item of problemItems.slice(0, 5)) {
+        const statusEmoji = item.status === MergeItemStatus.FAILED ? '❌' : '⏭️';
+        message += `  ${statusEmoji} \\#${item.prNumber}: ${this.escapeMarkdown(item.error.substring(0, 50))}${item.error.length > 50 ? '...' : ''}\n`;
       }
-      if (failedItems.length > 3) {
-        message += `  _...and ${failedItems.length - 3} more errors_\n`;
+      if (problemItems.length > 5) {
+        message += `  _...and ${problemItems.length - 5} more issues_\n`;
       }
       message += '\n';
     }
@@ -538,7 +540,14 @@ export class MergeQueueProcessor {
       message += `*Results:*\n`;
       for (const item of report.items) {
         const issueRef = item.issueNumber ? ` \\(Issue \\#${item.issueNumber}\\)` : '';
-        message += `${item.emoji} \\#${item.prNumber}${issueRef}\n`;
+        // Issue #1294: Show skip/fail reason so users understand what action is required
+        let reasonText = '';
+        if (item.error && (item.status === MergeItemStatus.SKIPPED || item.status === MergeItemStatus.FAILED)) {
+          // Truncate long reasons and escape for MarkdownV2
+          const truncatedReason = item.error.length > 50 ? item.error.substring(0, 47) + '...' : item.error;
+          reasonText = `: ${this.escapeMarkdown(truncatedReason)}`;
+        }
+        message += `${item.emoji} \\#${item.prNumber}${issueRef}${reasonText}\n`;
       }
     }