Merge remote-tracking branch 'origin/main' into issue-1458-95ae87b62e25

Author: konard

CHANGELOG.md

@@ -1,5 +1,15 @@
 # @link-assistant/hive-mind
 
+## 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

docs/case-studies/issue-1389/CASE-STUDY.md

@@ -24,7 +24,7 @@ All times UTC, from https://github.com/link-assistant/hive-mind/pull/1388
 | Time (UTC)        | Event                                                                                              |
 | ----------------- | -------------------------------------------------------------------------------------------------- |
 | 09:24:34          | **Session 1** starts: initial solution draft log uploaded to PR #1388                              |
-| 09:26:50          | Session 1: auto-restart triggered (attempt 1) due to CI failures                                   |
+| 09:26:50          | Session 1: auto-restart triggered (iteration 1) due to CI failures                                 |
 | 09:36:08          | Session 1 iteration 1: auto-restart-until-mergeable log uploaded (cost: $0.52)                     |
 | **09:37:14**      | **Session 1**: `## ✅ Ready to merge` comment posted — all CI checks passed                        |
 | 09:51:14          | Human feedback: "Will it not break queue display that goes after claude limits section?"           |

package-lock.json

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

package.json

@@ -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) {

src/solve.auto-merge.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('');

src/solve.watch.lib.mjs

@@ -289,15 +289,15 @@ test('wait is at least buffer duration even without a reset time', () => {
   assert(totalWaitMs === 10 * 60 * 1000, 'Wait should be exactly buffer when no reset time');
 });
 
-// ===== Test: Auto-restart comment includes attempt number =====
+// ===== Test: Auto-restart comment includes iteration number =====
 console.log('\n📋 Auto-restart Comment Tracking Tests\n');
 
-test('auto-restart comment includes attempt number', () => {
+test('auto-restart comment includes iteration number', () => {
   const restartCount = 3;
   const restartReason = 'CI failures detected';
-  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.*`;
 
-  assert(commentBody.includes('(attempt 3)'), 'Comment should include attempt number');
+  assert(commentBody.includes('(iteration 3)'), 'Comment should include iteration number');
   assert(commentBody.includes('CI failures detected'), 'Comment should include reason');
 });
 

tests/test-auto-restart-usage-limit-1356.mjs

@@ -110,7 +110,7 @@ test('OLD BUG: Session 1 posts "Ready to merge" → Session 3 sees it in history
   // PR comment history after Session 1
   const allPrCommentBodies = [
     '## 🤖 Solution Draft Log\nCost: $0.729114...',
-    '## 🔄 Auto-restart triggered (attempt 1)\n\nReason: CI failures...',
+    '## 🔄 Auto-restart triggered (iteration 1)\n\nReason: CI failures...',
     '## 🔄 Auto-restart-until-mergeable Log (iteration 1)...',
     '## ✅ Ready to merge\n\nThis pull request is now ready to be merged:\n- All CI checks have passed\n...', // ← Session 1 comment
     'Human: "Will it not break queue display?"',