@link-assistant/hive-mind 1.54.0 → 1.54.2

package/CHANGELOG.md

@@ -1,5 +1,33 @@
 # @link-assistant/hive-mind
 
+## 1.54.2
+
+### Patch Changes
+
+- 9d4e473: Cap `/limits` CPU cores display at available CPU count when load average demand exceeds capacity.
+- ea0b9f5: Use Anthropic's native Claude Code installer in Docker images so the CLI binary is installed even when Bun blocks dependency postinstall scripts.
+
+## 1.54.1
+
+### Patch Changes
+
+- 5f70953: fix(solve): post tool-generated PR comments again after v1.53.1 regression
+
+  `postTrackedComment()` in `src/tool-comments.lib.mjs` (added in #1626) was
+  passing the comment body to `gh api --input -` via `$({ input: payload })`,
+  but command-stream's option is `stdin`, not `input`. The misnamed key was
+  silently ignored, so `gh` read from the parent's stdin, sent an empty POST
+  body, and GitHub's edge returned `HTTP 400 "Whoa there!"`. Every tool-posted
+  comment — `AI Work Session Started`, log-upload link, `Ready to merge`,
+  `Auto-merged`, billing-limit notice, usage-limit notice — failed from this
+  one call path starting with v1.53.1.
+
+  Fix: use the documented `stdin` option so the JSON payload actually reaches
+  the child's stdin. The regression test pins the option name so a future
+  rename can't silently recur.
+
+  Fixes #1631.
+
 ## 1.54.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@link-assistant/hive-mind",
-  "version": "1.54.0",
+  "version": "1.54.2",
   "description": "AI-powered issue solver and hive mind for collaborative problem solving",
   "main": "src/hive.mjs",
   "type": "module",
@@ -13,7 +13,7 @@
     "hive-telegram-bot": "./src/telegram-bot.mjs"
   },
   "scripts": {
-    "test": "node tests/solve-queue.test.mjs && node tests/limits-display.test.mjs && node tests/test-usage-limit.mjs && node tests/test-issue-1616-pr-issue-link-preservation.mjs && node tests/test-telegram-message-filters.mjs && node tests/test-telegram-bot-command-aliases.mjs && node tests/test-solve-queue-command.mjs && node tests/test-queue-display-1267.mjs && node tests/test-telegram-bot-launcher.mjs",
+    "test": "node tests/solve-queue.test.mjs && node tests/limits-display.test.mjs && node tests/test-usage-limit.mjs && node tests/test-claude-code-install-method.mjs && node tests/test-issue-1616-pr-issue-link-preservation.mjs && node tests/test-telegram-message-filters.mjs && node tests/test-telegram-bot-command-aliases.mjs && node tests/test-solve-queue-command.mjs && node tests/test-queue-display-1267.mjs && node tests/test-telegram-bot-launcher.mjs",
     "test:queue": "node tests/solve-queue.test.mjs",
     "test:limits-display": "node tests/limits-display.test.mjs",
     "test:usage-limit": "node tests/test-usage-limit.mjs",

package/src/limits.lib.mjs

@@ -297,6 +297,15 @@ function formatBytesRange(usedBytes, totalBytes) {
   return `${usedValue.toFixed(decimals)}/${totalValue.toFixed(decimals)} ${sizes[i]} used`;
 }
 
+function formatRoundedNumber(value, decimals = 2) {
+  return parseFloat(value.toFixed(decimals));
+}
+
+function getDisplayCpuCoresUsed(loadAvg5, cpuCount) {
+  const boundedLoad = Math.min(Math.max(loadAvg5, 0), cpuCount);
+  return formatRoundedNumber(boundedLoad);
+}
+
 /**
  * Get GitHub API rate limits by calling gh api rate_limit
  * Returns rate limit info for core, search, graphql, and other resources
@@ -428,6 +437,7 @@ export async function getCpuLoadInfo(verbose = false) {
     // Load average of 1.0 per CPU = 100% utilization
     // Using 5m average for consistency with solve queue (see issue #1137)
     const usagePercentage = Math.min(100, Math.round((loadAvg5 / cpuCount) * 100));
+    const usedCpuCores = getDisplayCpuCoresUsed(loadAvg5, cpuCount);
 
     if (verbose) {
       console.log(`[VERBOSE] /limits CPU load: ${loadAvg1.toFixed(2)} (1m), ${loadAvg5.toFixed(2)} (5m), ${loadAvg15.toFixed(2)} (15m), ${cpuCount} CPUs, ${usagePercentage}% used`);
@@ -441,6 +451,7 @@ export async function getCpuLoadInfo(verbose = false) {
         loadAvg15,
         cpuCount,
         usagePercentage,
+        usedCpuCores,
       },
     };
   } catch (error) {
@@ -1027,9 +1038,14 @@ export function formatUsageMessage(usage, diskSpace = null, githubRateLimit = nu
     // See: https://github.com/link-assistant/hive-mind/issues/1267
     const suffix = cpuLoad.usagePercentage >= DISPLAY_THRESHOLDS.CPU ? ' ⚠️' : ' used';
     section += `${usedBar} ${cpuLoad.usagePercentage}%${suffix}\n`;
-    // Show cores used based on 5m load average (e.g., "0.04/6 CPU cores used" or "3/6 CPU cores used")
-    // Use parseFloat to strip unnecessary trailing zeros (3.00 -> 3, 0.10 -> 0.1, 0.04 -> 0.04)
-    section += `${parseFloat(cpuLoad.loadAvg5.toFixed(2))}/${cpuLoad.cpuCount} CPU cores\n`;
+    // Linux load average is demand, not bounded CPU time. Keep the cores-used
+    // display within CPU capacity and show raw load average only when saturated.
+    const usedCpuCores = cpuLoad.usedCpuCores ?? getDisplayCpuCoresUsed(cpuLoad.loadAvg5, cpuLoad.cpuCount);
+    let cpuCoresLine = `${formatRoundedNumber(usedCpuCores)}/${cpuLoad.cpuCount} CPU cores used`;
+    if (cpuLoad.loadAvg5 > cpuLoad.cpuCount) {
+      cpuCoresLine += ` (5m load avg ${formatRoundedNumber(cpuLoad.loadAvg5)})`;
+    }
+    section += `${cpuCoresLine}\n`;
     sections.push(section);
   }
 

package/src/tool-comments.lib.mjs

@@ -199,13 +199,14 @@ export const postTrackedComment = async ({ $, owner, repo, targetNumber, body })
   const apiPath = `repos/${owner}/${repo}/issues/${targetNumber}/comments`;
   const payload = JSON.stringify({ body });
 
-  // `gh api --input -` reads from stdin. command-stream supports .stdin(...)
-  // on the returned process handle (same API used in interactive-mode.lib.mjs
-  // via execFileAsync). We build the invocation through $ so callers can
-  // inject a mock.
+  // command-stream's options key is `stdin`, not `input` — unknown keys are
+  // silently ignored, which previously left stdin inherited from the parent
+  // and caused `gh api --input -` to POST an empty body. GitHub's edge
+  // replied with HTTP 400 "Whoa there!" *before* the API layer ran. See
+  // issue #1631.
   let result;
   try {
-    result = await $({ input: payload })`gh api ${apiPath} -X POST --input -`;
+    result = await $({ stdin: payload })`gh api ${apiPath} -X POST --input -`;
   } catch (err) {
     return { ok: false, commentId: null, stderr: err && err.message ? err.message : String(err) };
   }