@link-assistant/hive-mind 1.23.0 → 1.23.1

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # @link-assistant/hive-mind
 
+## 1.23.1
+
+### Patch Changes
+
+- 5c635fc: Fix agent tool error handling: upload failure logs to PR even when sessionId is not available
+  - Remove overly strict sessionId requirement for failure log upload in solve.mjs
+  - Add FreeUsageLimitError pattern detection for Agent/OpenCode Zen rate limits
+  - Improve rate limit detection by checking multiple sources (lastMessage, errorMatch, fullOutput)
+  - Add comprehensive case study documentation for issue #1287
+  - Add tests for FreeUsageLimitError detection
+
 ## 1.23.0
 
 ### Minor Changes

package/package.json

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

package/src/agent.lib.mjs

@@ -848,7 +848,18 @@ export const executeAgentCommand = async params => {
         };
 
         // Check for usage limit errors first (more specific)
-        const limitInfo = detectUsageLimit(lastMessage);
+        // Issue #1287: Check multiple sources for usage limit detection:
+        // 1. lastMessage (the last chunk of output)
+        // 2. errorMatch (the extracted error message from JSON output)
+        // 3. fullOutput (complete output - fallback)
+        let limitInfo = detectUsageLimit(lastMessage);
+        if (!limitInfo.isUsageLimit && outputError.match) {
+          limitInfo = detectUsageLimit(outputError.match);
+        }
+        if (!limitInfo.isUsageLimit) {
+          // Fallback: scan fullOutput for usage limit patterns
+          limitInfo = detectUsageLimit(fullOutput);
+        }
         if (limitInfo.isUsageLimit) {
           limitReached = true;
           limitResetTime = limitInfo.resetTime;

package/src/solve.mjs

@@ -1112,7 +1112,9 @@ try {
     }
 
     // If --attach-logs is enabled and we have a PR, attach failure logs before exiting
-    if (shouldAttachLogs && sessionId && global.createdPR && global.createdPR.number) {
+    // Note: sessionId is not required - logs should be uploaded even if agent failed before establishing a session
+    // This aligns with the pattern in handleFailure() in solve.error-handlers.lib.mjs
+    if (shouldAttachLogs && global.createdPR && global.createdPR.number) {
       await log('\n📄 Attaching failure logs to Pull Request...');
       try {
         // Build Claude CLI resume command

package/src/usage-limit.lib.mjs

@@ -44,7 +44,7 @@ export function isUsageLimitError(message) {
     'rate limit exceeded',
     'limit reached',
     'limit has been reached',
-    // Provider-specific phrasings we’ve seen in the wild
+    // Provider-specific phrasings we've seen in the wild
     'session limit reached', // Claude
     'weekly limit reached', // Claude
     'daily limit reached',
@@ -52,7 +52,9 @@ export function isUsageLimitError(message) {
     'billing hard limit',
     'please try again at', // Codex/OpenCode style
     'available again at',
-    'resets', // Claude shows: “∙ resets 5am”
+    'resets', // Claude shows: "∙ resets 5am"
+    // Agent/OpenCode Zen specific - issue #1287
+    'freeusagelimiterror', // Agent JSON error type
   ];
 
   return patterns.some(pattern => lowerMessage.includes(pattern));