@link-assistant/hive-mind 0.54.1 → 0.54.2

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # @link-assistant/hive-mind
 
+## 0.54.2
+
+### Patch Changes
+
+- c5f5194: Fix Telegram message getting stuck at "Starting solve command..."
+  - Add error handling to `executeAndUpdateMessage` function to catch Telegram API errors
+  - Fix critical bug where `messageInfo` was being cleared before the final message update
+  - Add proper error logging for message edit failures in both immediate and queued execution paths
+
 ## 0.54.1
 
 ### Patch Changes

package/package.json

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

package/src/telegram-bot.mjs

@@ -673,20 +673,25 @@ function validateGitHubUrl(args, options = {}) {
  */
 async function executeAndUpdateMessage(ctx, startingMessage, commandName, args, infoBlock) {
   const result = await executeStartScreen(commandName, args);
+  const { chat, message_id } = startingMessage;
 
-  if (result.warning) {
-    await ctx.telegram.editMessageText(startingMessage.chat.id, startingMessage.message_id, undefined, `⚠️  ${result.warning}`, { parse_mode: 'Markdown' });
-    return;
-  }
+  // Safely edit message - catch errors to prevent stuck "Starting..." messages (issue #1062)
+  const safeEdit = async text => {
+    try {
+      await ctx.telegram.editMessageText(chat.id, message_id, undefined, text, { parse_mode: 'Markdown' });
+    } catch (e) {
+      console.error(`[telegram-bot] Failed to update message for ${commandName}: ${e.message}`);
+    }
+  };
+
+  if (result.warning) return safeEdit(`⚠️  ${result.warning}`);
 
   if (result.success) {
-    const sessionNameMatch = result.output.match(/session:\s*(\S+)/i) || result.output.match(/screen -r\s+(\S+)/);
-    const sessionName = sessionNameMatch ? sessionNameMatch[1] : 'unknown';
-    const response = `✅ ${commandName.charAt(0).toUpperCase() + commandName.slice(1)} command started successfully!\n\n📊 Session: \`${sessionName}\`\n\n${infoBlock}`;
-    await ctx.telegram.editMessageText(startingMessage.chat.id, startingMessage.message_id, undefined, response, { parse_mode: 'Markdown' });
+    const match = result.output.match(/session:\s*(\S+)/i) || result.output.match(/screen -r\s+(\S+)/);
+    const session = match ? match[1] : 'unknown';
+    await safeEdit(`✅ ${commandName.charAt(0).toUpperCase() + commandName.slice(1)} command started successfully!\n\n📊 Session: \`${session}\`\n\n${infoBlock}`);
   } else {
-    const response = `❌ Error executing ${commandName} command:\n\n\`\`\`\n${result.error || result.output}\n\`\`\``;
-    await ctx.telegram.editMessageText(startingMessage.chat.id, startingMessage.message_id, undefined, response, { parse_mode: 'Markdown' });
+    await safeEdit(`❌ Error executing ${commandName} command:\n\n\`\`\`\n${result.error || result.output}\n\`\`\``);
   }
 }
 

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

@@ -667,13 +667,18 @@ export class SolveQueue {
           if (sessionMatch) sessionName = sessionMatch[1];
         }
 
+        // IMPORTANT: Save messageInfo BEFORE calling setStarted, because setStarted clears it
+        // This was a bug where the final message update never happened because messageInfo was null
+        // See: https://github.com/link-assistant/hive-mind/issues/1062
+        const savedMessageInfo = item.messageInfo;
+
         // Update to Started status (terminal - forgets message tracking)
         item.setStarted(sessionName);
         this.stats.totalCompleted++;
 
-        // Final message update before forgetting
-        if (item.ctx && result) {
-          const { chatId, messageId } = item.messageInfo || {};
+        // Final message update using saved messageInfo
+        if (item.ctx && result && savedMessageInfo) {
+          const { chatId, messageId } = savedMessageInfo;
           if (chatId && messageId) {
             try {
               if (result.warning) {
@@ -685,8 +690,10 @@ export class SolveQueue {
                 const response = `❌ Error executing solve command:\n\n\`\`\`\n${result.error || result.output}\n\`\`\``;
                 await item.ctx.telegram.editMessageText(chatId, messageId, undefined, response, { parse_mode: 'Markdown' });
               }
-            } catch {
-              // Ignore message edit failures
+            } catch (error) {
+              // Log message edit failures for debugging
+              // See: https://github.com/link-assistant/hive-mind/issues/1062
+              console.error(`[solve-queue] Failed to update message for item ${item.id}: ${error.message}`);
             }
           }
         }
@@ -704,8 +711,10 @@ export class SolveQueue {
       if (chatId && messageId && item.ctx) {
         try {
           await item.ctx.telegram.editMessageText(chatId, messageId, undefined, `❌ Error: ${error.message}`, { parse_mode: 'Markdown' });
-        } catch {
-          // Ignore
+        } catch (editError) {
+          // Log the edit failure for debugging
+          // See: https://github.com/link-assistant/hive-mind/issues/1062
+          console.error(`[solve-queue] Failed to update error message for item ${item.id}: ${editError.message}`);
         }
       }
     } finally {