@link-assistant/hive-mind 1.35.2 → 1.35.4

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # @link-assistant/hive-mind
 
+## 1.35.4
+
+### Patch Changes
+
+- 0df2139: Harden Telegram message formatting: escape special characters in user mentions, options text, and server overrides. Add safeReply with plain text fallback and diagnostic logging when Telegram rejects Markdown. Improve error messages with user identity context for root cause analysis.
+
+## 1.35.3
+
+### Patch Changes
+
+- 22ae6d6: fix: rename "attempt" to "iteration" in auto-restart messages (Issue #1456)
+
+  The auto-restart PR comment title and log message now use "iteration" instead of "attempt" to match the project's terminology. Affected messages:
+  - PR comment: `Auto-restart triggered (iteration N)` (was `attempt N`)
+  - Log: `Exiting auto-restart mode after N iterations` (was `attempts`)
+
 ## 1.35.2
 
 ### Patch Changes

package/package.json

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

package/src/buildUserMention.lib.mjs

@@ -40,9 +40,12 @@ export function buildUserMention({ user, id: idParam, username: usernameParam, f
   const link = username ? `https://t.me/${username}` : `tg://user?id=${id}`;
 
   switch (parseMode) {
-    case 'Markdown':
+    case 'Markdown': {
       // Legacy Markdown: [text](url)
-      return `[${displayName}](${link})`;
+      // Escape _ and * in display name to prevent "can't find end of entity" errors (issue #1460)
+      const escapedMarkdownName = displayName.replace(/_/g, '\\_').replace(/\*/g, '\\*');
+      return `[${escapedMarkdownName}](${link})`;
+    }
     case 'MarkdownV2': {
       // MarkdownV2 requires escaping special characters
       const escapedName = displayName.replace(/([_*[\]()~`>#+\-=|{}.!])/g, '\\$1');

package/src/solve.auto-merge.lib.mjs

@@ -658,7 +658,7 @@ Once the billing issue is resolved, you can re-run the CI checks or push a new c
         // Post a comment to PR about the restart
         // Issue #1356: Include restart count for tracking and add deduplication
         try {
-          const commentBody = `## 🔄 Auto-restart triggered (attempt ${restartCount})\n\n**Reason:** ${restartReason}\n\nStarting new session to address the issues.\n\n---\n*Auto-restart-until-mergeable mode is active. Will continue until PR becomes mergeable.*`;
+          const commentBody = `## 🔄 Auto-restart triggered (iteration ${restartCount})\n\n**Reason:** ${restartReason}\n\nStarting new session to address the issues.\n\n---\n*Auto-restart-until-mergeable mode is active. Will continue until PR becomes mergeable.*`;
           await $`gh pr comment ${prNumber} --repo ${owner}/${repo} --body ${commentBody}`;
           await log(formatAligned('', '💬 Posted auto-restart notification to PR', '', 2));
         } catch (commentError) {

package/src/solve.watch.lib.mjs

@@ -112,7 +112,7 @@ export const watchForFeedback = async params => {
       // Check if we've reached max iterations
       if (autoRestartCount >= maxAutoRestartIterations) {
         await log('');
-        await log(formatAligned('⚠️', 'MAX ITERATIONS REACHED', `Exiting auto-restart mode after ${autoRestartCount} attempts`));
+        await log(formatAligned('⚠️', 'MAX ITERATIONS REACHED', `Exiting auto-restart mode after ${autoRestartCount} iterations`));
         await log(formatAligned('', 'Some uncommitted changes may remain', '', 2));
         await log(formatAligned('', 'Please review and commit manually if needed', '', 2));
         await log('');

package/src/telegram-bot.mjs

@@ -992,9 +992,11 @@ async function handleSolveCommand(ctx) {
 
   const requester = buildUserMention({ user: ctx.from, parseMode: 'Markdown' });
   // Issue #1228: Show only user-provided options (exclude locked overrides to avoid duplication)
-  const userOptionsText = userArgs.slice(1).join(' ') || 'none';
-  let infoBlock = `Requested by: ${requester}\nURL: ${escapeMarkdown(normalizedUrl)}\n\n🛠 Options: ${userOptionsText}`;
-  if (solveOverrides.length > 0) infoBlock += `\n🔒 Locked options: ${solveOverrides.join(' ')}`;
+  // Issue #1460: Escape options text to prevent Markdown parsing errors
+  const userOptionsRaw = userArgs.slice(1).join(' ');
+  let infoBlock = `Requested by: ${requester}\nURL: ${escapeMarkdown(normalizedUrl)}`;
+  if (userOptionsRaw) infoBlock += `\n\n🛠 Options: ${escapeMarkdown(userOptionsRaw)}`;
+  if (solveOverrides.length > 0) infoBlock += `${userOptionsRaw ? '\n' : '\n\n'}🔒 Locked options: ${escapeMarkdown(solveOverrides.join(' '))}`;
   const solveQueue = getSolveQueue({ verbose: VERBOSE });
 
   // Check for duplicate URL in queue
@@ -1162,10 +1164,12 @@ async function handleHiveCommand(ctx) {
   const requester = buildUserMention({ user: ctx.from, parseMode: 'Markdown' });
   const escapedUrl = escapeMarkdown(args[0]);
   // Issue #1228: Show only user-provided options (exclude locked overrides to avoid duplication)
-  const userOptionsText = normalizedArgs.slice(1).join(' ') || 'none';
-  let infoBlock = `Requested by: ${requester}\nURL: ${escapedUrl}\n\n🛠 Options: ${userOptionsText}`;
+  // Issue #1460: Escape options text to prevent Markdown parsing errors
+  const userOptionsRaw = normalizedArgs.slice(1).join(' ');
+  let infoBlock = `Requested by: ${requester}\nURL: ${escapedUrl}`;
+  if (userOptionsRaw) infoBlock += `\n\n🛠 Options: ${escapeMarkdown(userOptionsRaw)}`;
   if (hiveOverrides.length > 0) {
-    infoBlock += `\n🔒 Locked options: ${hiveOverrides.join(' ')}`;
+    infoBlock += `${userOptionsRaw ? '\n' : '\n\n'}🔒 Locked options: ${escapeMarkdown(hiveOverrides.join(' '))}`;
   }
 
   const startingMessage = await ctx.reply(`🚀 Starting hive command...\n\n${infoBlock}`, { parse_mode: 'Markdown', reply_to_message_id: ctx.message.message_id });
@@ -1305,18 +1309,22 @@ bot.catch((error, ctx) => {
     let errorMessage;
 
     if (isTelegramParsingError) {
-      // Special handling for Telegram API parsing errors caused by unescaped special characters
-      errorMessage = `❌ A message formatting error occurred.\n\n💡 This usually means there was a problem with special characters in the response.\nPlease try your command again with a different URL or contact support.`;
-      // Show the user's input with special characters visible (if available)
+      // Issue #1460: Log detailed context for root cause analysis (always logged, not just in verbose mode)
+      const userInfo = ctx.from ? { id: ctx.from.id, username: ctx.from.username, first_name: ctx.from.first_name, last_name: ctx.from.last_name } : 'unknown';
+      console.error(`[telegram-bot] Parsing error: ${error.message}`);
+      console.error(`[telegram-bot] Parsing error context - user: ${JSON.stringify(userInfo)}, command: ${ctx.message?.text?.split(' ')[0] || 'unknown'}`);
+      console.error(`[telegram-bot] User input text: ${ctx.message?.text || 'none'}`);
       if (ctx.message?.text) {
+        const visibleInput = makeSpecialCharsVisible(ctx.message.text, { maxLength: 500 });
+        console.error(`[telegram-bot] User input (special chars visible): ${visibleInput}`);
         const cleanedInput = cleanNonPrintableChars(ctx.message.text);
-        const visibleInput = makeSpecialCharsVisible(cleanedInput, { maxLength: 150 });
-        if (visibleInput !== cleanedInput) errorMessage += `\n\n📝 Your input (with special chars visible):\n\`${escapeMarkdown(visibleInput)}\``;
-      }
-      if (VERBOSE) {
-        const escapedError = escapeMarkdown(error.message || 'Unknown error');
-        errorMessage += `\n\n🔍 Debug info: ${escapedError}\nUpdate ID: ${ctx.update.update_id}`;
+        if (cleanedInput !== ctx.message.text) {
+          console.error(`[telegram-bot] ${ctx.message.text.length - cleanedInput.length} hidden character(s) detected in input`);
+        }
       }
+
+      // Issue #1460: Show user a simple, non-confusing message — all details are in the logs
+      errorMessage = `❌ Failed to send formatted message. Please try your command again.\n\nIf the issue persists, contact support with Update ID: ${ctx.update.update_id}`;
     } else {
       // Build informative error message for other errors
       errorMessage = '❌ An error occurred while processing your request.\n\n';
@@ -1334,14 +1342,21 @@ bot.catch((error, ctx) => {
       if (VERBOSE) errorMessage += `\n\n🔍 Debug info: Update ID: ${ctx.update.update_id}`;
     }
 
-    ctx.reply(errorMessage, { parse_mode: 'Markdown' }).catch(replyError => {
-      console.error('Failed to send error message to user:', replyError);
-      // Try sending a simple text message without Markdown if Markdown parsing failed
-      const plainMessage = `An error occurred while processing your request. Please try again or contact support.\n\nError: ${error.message || 'Unknown error'}`;
-      ctx.reply(plainMessage).catch(fallbackError => {
-        console.error('Failed to send fallback error message:', fallbackError);
+    // Issue #1460: For parsing errors, always send as plain text (we already know Markdown is the problem)
+    // For other errors, try Markdown first, then fall back to plain text
+    if (isTelegramParsingError) {
+      ctx.reply(errorMessage).catch(fallbackError => {
+        console.error('Failed to send plain text error message:', fallbackError);
       });
-    });
+    } else {
+      ctx.reply(errorMessage, { parse_mode: 'Markdown' }).catch(replyError => {
+        console.error('Failed to send error message to user:', replyError);
+        const plainMessage = `An error occurred while processing your request. Please try again or contact support.\n\nError: ${error.message || 'Unknown error'}`;
+        ctx.reply(plainMessage).catch(fallbackError => {
+          console.error('Failed to send fallback error message:', fallbackError);
+        });
+      });
+    }
   }
 });