@link-assistant/hive-mind 1.7.0 → 1.7.2

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # @link-assistant/hive-mind
 
+## 1.7.2
+
+### Patch Changes
+
+- e6a656f: Use `screen -R` instead of `screen -S` and `screen -r` in all docs and code for better session management. The `-R` flag ensures we open existing screen if created, and new if not yet created, making it the most safe and universal option.
+
+## 1.7.1
+
+### Patch Changes
+
+- d86ba79: Prevent duplicate URLs from being added to the /solve queue (Issue #1080)
+  - Added `findByUrl()` method to SolveQueue to detect existing items by URL
+  - Updated /solve command handler to check for duplicates before queueing
+  - Uses normalized URLs for consistent comparison
+  - Returns informative error message when duplicate is detected
+
 ## 1.7.0
 
 ### Minor Changes

package/README.md

@@ -209,7 +209,7 @@ See [docs/HELM.md](./docs/HELM.md) for detailed Helm configuration options.
    **Using Links Notation (recommended):**
 
    ```
-   screen -S bot # Enter new screen for bot
+   screen -R bot # Enter new screen for bot
 
    hive-telegram-bot --configuration "
      TELEGRAM_BOT_TOKEN: '849...355:AAG...rgk_YZk...aPU'
@@ -238,7 +238,7 @@ See [docs/HELM.md](./docs/HELM.md) for detailed Helm configuration options.
    **Using individual command-line options:**
 
    ```
-   screen -S bot # Enter new screen for bot
+   screen -R bot # Enter new screen for bot
 
    hive-telegram-bot --token 849...355:AAG...rgk_YZk...aPU --allowed-chats "(
      -1002975819706

package/package.json

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

package/src/start-screen.mjs

@@ -163,7 +163,7 @@ async function createOrEnterScreen(sessionName, command, args, autoTerminate = f
       // The \n at the end simulates pressing Enter
       await execAsync(`screen -S ${sessionName} -X stuff '${escapedCommand}\n'`);
       console.log(`Command sent to session '${sessionName}' successfully.`);
-      console.log(`To attach and view the session, run: screen -r ${sessionName}`);
+      console.log(`To attach and view the session, run: screen -R ${sessionName}`);
     } catch (error) {
       console.error('Failed to send command to existing screen session:', error.message);
       console.error('You may need to terminate the old session and try again.');
@@ -208,7 +208,7 @@ async function createOrEnterScreen(sessionName, command, args, autoTerminate = f
     } else {
       console.log('Session will remain active after command completes');
     }
-    console.log(`To attach to this session, run: screen -r ${sessionName}`);
+    console.log(`To attach to this session, run: screen -R ${sessionName}`);
   } catch (error) {
     console.error('Failed to create screen session:', error.message);
     process.exit(1);

package/src/telegram-bot.mjs

@@ -654,7 +654,7 @@ async function executeAndUpdateMessage(ctx, startingMessage, commandName, args,
   if (result.warning) return safeEdit(`⚠️  ${result.warning}`);
 
   if (result.success) {
-    const match = result.output.match(/session:\s*(\S+)/i) || result.output.match(/screen -r\s+(\S+)/);
+    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 {
@@ -1069,18 +1069,32 @@ bot.command(/^solve$/i, async ctx => {
     return;
   }
 
+  // Use normalized URL from validation to ensure consistent duplicate detection
+  // See: https://github.com/link-assistant/hive-mind/issues/1080
+  const normalizedUrl = validation.parsed.normalized;
+
   const requester = buildUserMention({ user: ctx.from, parseMode: 'Markdown' });
   const optionsText = args.slice(1).join(' ') || 'none';
-  let infoBlock = `Requested by: ${requester}\nURL: ${escapeMarkdown(args[0])}\nOptions: ${optionsText}`;
+  let infoBlock = `Requested by: ${requester}\nURL: ${escapeMarkdown(normalizedUrl)}\nOptions: ${optionsText}`;
   if (solveOverrides.length > 0) infoBlock += `\n🔒 Locked options: ${solveOverrides.join(' ')}`;
   const solveQueue = getSolveQueue({ verbose: VERBOSE });
+
+  // Check for duplicate URL in queue
+  // See: https://github.com/link-assistant/hive-mind/issues/1080
+  const existingItem = solveQueue.findByUrl(normalizedUrl);
+  if (existingItem) {
+    const statusText = existingItem.status === 'starting' || existingItem.status === 'started' ? 'being processed' : 'already in the queue';
+    await ctx.reply(`❌ This URL is ${statusText}.\n\nURL: ${escapeMarkdown(normalizedUrl)}\nStatus: ${existingItem.status}\n\n💡 Use /solve-queue to check the queue status.`, { parse_mode: 'Markdown', reply_to_message_id: ctx.message.message_id });
+    return;
+  }
+
   const check = await solveQueue.canStartCommand();
   const queueStats = solveQueue.getStats();
   if (check.canStart && queueStats.queued === 0) {
     const startingMessage = await ctx.reply(`🚀 Starting solve command...\n\n${infoBlock}`, { parse_mode: 'Markdown', reply_to_message_id: ctx.message.message_id });
     await executeAndUpdateMessage(ctx, startingMessage, 'solve', args, infoBlock);
   } else {
-    const queueItem = solveQueue.enqueue({ url: args[0], args, ctx, requester, infoBlock, tool: solveTool });
+    const queueItem = solveQueue.enqueue({ url: normalizedUrl, args, ctx, requester, infoBlock, tool: solveTool });
     let queueMessage = `📋 Solve command queued (position #${queueStats.queued + 1})\n\n${infoBlock}`;
     if (check.reason) queueMessage += `\n\n⏳ Waiting: ${check.reason}`;
     const queuedMessage = await ctx.reply(queueMessage, { parse_mode: 'Markdown', reply_to_message_id: ctx.message.message_id });

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

@@ -34,22 +34,22 @@ import { getCachedClaudeLimits, getCachedGitHubLimits, getCachedMemoryInfo, getC
 export const QUEUE_CONFIG = {
   // Resource thresholds (usage ratios: 0.0 - 1.0)
   // All thresholds use >= comparison (inclusive)
-  RAM_THRESHOLD: 0.5, // Stop if RAM usage >= 50%
+  RAM_THRESHOLD: 0.65, // Stop if RAM usage >= 65%
   // CPU threshold uses 5-minute load average, not instantaneous CPU usage
-  CPU_THRESHOLD: 0.5, // Stop if 5-minute load average >= 50% of CPU count
+  CPU_THRESHOLD: 0.75, // Stop if 5-minute load average >= 75% of CPU count
   DISK_THRESHOLD: 0.95, // One-at-a-time if disk usage >= 95%
 
   // API limit thresholds (usage ratios: 0.0 - 1.0)
   // All thresholds use >= comparison (inclusive)
-  CLAUDE_5_HOUR_SESSION_THRESHOLD: 0.9, // Stop if 5-hour limit >= 90%
-  CLAUDE_WEEKLY_THRESHOLD: 0.99, // One-at-a-time if weekly limit >= 99%
+  CLAUDE_5_HOUR_SESSION_THRESHOLD: 0.85, // Stop if 5-hour limit >= 85%
+  CLAUDE_WEEKLY_THRESHOLD: 0.98, // One-at-a-time if weekly limit >= 98%
   GITHUB_API_THRESHOLD: 0.8, // Stop if GitHub >= 80% with parallel claude
 
   // Timing
   // MIN_START_INTERVAL_MS: Time to allow solve command to start actual claude process
   // This ensures that when API limits are checked, the running process is counted
-  MIN_START_INTERVAL_MS: 120000, // 2 minutes between starts (was 1 minute)
-  CONSUMER_POLL_INTERVAL_MS: 60000, // 1 minute between queue checks (was 5 seconds)
+  MIN_START_INTERVAL_MS: 60000, // 1 minutes between starts
+  CONSUMER_POLL_INTERVAL_MS: 60000, // 1 minute between queue checks
   MESSAGE_UPDATE_INTERVAL_MS: 60000, // 1 minute between status message updates
 
   // Process detection
@@ -303,6 +303,30 @@ export class SolveQueue {
     return item;
   }
 
+  /**
+   * Find an item by URL in the queue or processing items
+   * Used to prevent duplicate URLs from being added to the queue
+   * @param {string} url - The URL to search for
+   * @returns {SolveQueueItem|null} The found item or null
+   * @see https://github.com/link-assistant/hive-mind/issues/1080
+   */
+  findByUrl(url) {
+    // Check queued items
+    const queuedItem = this.queue.find(item => item.url === url);
+    if (queuedItem) {
+      return queuedItem;
+    }
+
+    // Check processing items
+    for (const item of this.processing.values()) {
+      if (item.url === url) {
+        return item;
+      }
+    }
+
+    return null;
+  }
+
   /**
    * Cancel a queued item by ID
    * @param {string} id - Item ID
@@ -734,7 +758,7 @@ export class SolveQueue {
         // Extract session name from result
         let sessionName = 'unknown';
         if (result && result.output) {
-          const sessionMatch = result.output.match(/session:\s*(\S+)/i) || result.output.match(/screen -r\s+(\S+)/);
+          const sessionMatch = result.output.match(/session:\s*(\S+)/i) || result.output.match(/screen -R\s+(\S+)/);
           if (sessionMatch) sessionName = sessionMatch[1];
         }