@link-assistant/hive-mind 1.23.2 → 1.23.4

package/CHANGELOG.md

@@ -1,5 +1,38 @@
 # @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
+
+- a797e56: fix: escape owner/repo names for Telegram MarkdownV2 in /merge command
+
+  Fixed the `/merge` command silently failing when updating Telegram messages for repositories with hyphens in their names (e.g., `link-assistant/hive-mind`). The issue was caused by unescaped special characters in MarkdownV2 format.
+
 ## 1.23.2
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@link-assistant/hive-mind",
-  "version": "1.23.2",
+  "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

@@ -440,7 +440,8 @@ export class MergeQueueProcessor {
     const update = this.getProgressUpdate();
 
     let message = `*Merge Queue*\n`;
-    message += `${this.owner}/${this.repo}\n\n`;
+    // Issue #1292: Escape owner/repo for MarkdownV2 (may contain hyphens, underscores, etc.)
+    message += `${this.escapeMarkdown(this.owner)}/${this.escapeMarkdown(this.repo)}\n\n`;
 
     // Progress bar in code block for better style
     const progressBar = getProgressBar(update.progress.percentage);
@@ -461,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';
     }
@@ -515,7 +518,8 @@ export class MergeQueueProcessor {
     }
 
     let message = `${statusEmoji} *Merge Queue ${statusText}*\n`;
-    message += `${this.owner}/${this.repo}\n\n`;
+    // Issue #1292: Escape owner/repo for MarkdownV2 (may contain hyphens, underscores, etc.)
+    message += `${this.escapeMarkdown(this.owner)}/${this.escapeMarkdown(this.repo)}\n\n`;
 
     // Final progress bar in code block
     const percentage = report.stats.total > 0 ? Math.round((report.stats.merged / report.stats.total) * 100) : 0;
@@ -536,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`;
       }
     }