@link-assistant/hive-mind 2.10.2 → 2.10.4

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @link-assistant/hive-mind
 
+## 2.10.4
+
+### Patch Changes
+
+- f1d5026: Recover use-m dependency imports when an installed package is missing an internal module or alias cleanup hits a transient filesystem race by removing the incomplete alias with bounded retries and reloading it. The CDN bootstrap fallback is also repinned from use-m 8.13.8 to 8.14.4 so a CDN outage no longer downgrades dependency loading to a use-m without corrupt-alias recovery.
+
+## 2.10.3
+
+### Patch Changes
+
+- 3cfd6c2: Show cost and token/context budget facts in automated working-session summaries.
+
 ## 2.10.2
 
 ### Patch Changes

package/package.json

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

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

@@ -1068,6 +1068,10 @@ No further AI sessions will be started automatically for this run. Please review
               prNumber,
               issueNumber,
               success: true,
+              publicPricingEstimate: toolResult.publicPricingEstimate,
+              anthropicTotalCostUSD: latestAnthropicCost,
+              pricingInfo: toolResult.pricingInfo,
+              budgetStatsData: autoMergeBudgetStatsData,
             });
           } catch (summaryError) {
             reportError(summaryError, {

package/src/solve.mjs

@@ -1228,13 +1228,8 @@ try {
 
   // Issue #2048: commit+push dev log BEFORE any PR readiness signal so its CI gates readiness (was last, breaking CI post-signal; PR #2046, docs/case-studies/issue-2048). Idempotent: trailing call is a no-op. prettier-ignore
   await finalizeDevelopmentLog();
-  // Issue #1263 / #1728: Working session summary attachment.
-  // Routed through the shared maybeAttachWorkingSessionSummary helper so that
-  // top-level solve, auto-restart-until-mergeable, and watch-mode iterations
-  // all use identical attach logic. The helper internally honours
-  // --attach-solution-summary (always attach) and --auto-attach-solution-summary
-  // (attach only if no AI comment was posted during the session).
-  await maybeAttachWorkingSessionSummary({
+  // Issue #1263 / #1728 / #2115: shared summary attachment and usage display.
+  const { budgetStatsData: workingSessionBudgetStatsData = null } = await maybeAttachWorkingSessionSummary({
     argv,
     resultSummary,
     workStartTime,
@@ -1243,10 +1238,14 @@ try {
     prNumber,
     issueNumber,
     success,
+    publicPricingEstimate,
+    anthropicTotalCostUSD,
+    pricingInfo,
+    sessionUsage: { sessionId, tempDir, resultModelUsage, streamTokenUsage, subAgentCalls },
   });
 
   // Search for newly created pull requests and comments
-  const verifyResult = await verifyResults(owner, repo, branchName, issueNumber, prNumber, prUrl, referenceTime, argv, shouldAttachLogs, shouldRestart, sessionId, tempDir, anthropicTotalCostUSD, publicPricingEstimate, pricingInfo, errorDuringExecution, sessionType, resultModelUsage, streamTokenUsage, subAgentCalls);
+  const verifyResult = await verifyResults(owner, repo, branchName, issueNumber, prNumber, prUrl, referenceTime, argv, shouldAttachLogs, shouldRestart, sessionId, tempDir, anthropicTotalCostUSD, publicPricingEstimate, pricingInfo, errorDuringExecution, sessionType, resultModelUsage, streamTokenUsage, subAgentCalls, workingSessionBudgetStatsData);
   const logsAlreadyUploaded = verifyResult?.logUploadSuccess || false;
 
   // Issue #1162: Auto-restart when PR title/description still has placeholder content

package/src/solve.results.lib.mjs

@@ -28,6 +28,8 @@ import { safeExit } from './exit-handler.lib.mjs';
 // Import GitHub-related functions
 const githubLib = await import('./github.lib.mjs');
 const { sanitizeLogContent, attachLogToGitHub } = githubLib;
+const { buildCostInfoString } = await import('./github-cost-info.lib.mjs');
+const { buildBudgetStatsString } = await import('./claude.budget-stats.lib.mjs');
 
 // Issue #1745: process-wide sanitization counters used to print a one-line
 // "we masked N secrets" summary at the end of each run.
@@ -663,11 +665,9 @@ export const showSessionSummary = async (sessionId, limitReached, argv, issueUrl
   }
 };
 
-// Verify results by searching for new PRs and comments
-export const verifyResults = async (owner, repo, branchName, issueNumber, prNumber, prUrl, referenceTime, argv, shouldAttachLogs, shouldRestart = false, sessionId = null, tempDir = null, anthropicTotalCostUSD = null, publicPricingEstimate = null, pricingInfo = null, errorDuringExecution = false, sessionType = 'new', resultModelUsage = null, streamTokenUsage = null, subAgentCalls = null) => {
-  await log('\n🔍 Searching for created pull requests or comments...');
-
-  // Issue #1491, #1526: Build budget stats data for GitHub comment (computed once, used in both PR and issue paths)
+// Build token/context data once so every end-of-session publication can use the
+// same observed facts (working-session summary and attached log alike).
+export const buildSessionBudgetStatsData = async ({ argv, sessionId = null, tempDir = null, resultModelUsage = null, streamTokenUsage = null, subAgentCalls = null, pricingInfo = null }) => {
   let budgetStatsData = null;
   if (argv.tokensBudgetStats && sessionId && tempDir) {
     try {
@@ -692,6 +692,26 @@ export const verifyResults = async (owner, repo, branchName, issueNumber, prNumb
       if (argv.verbose) await log(`  ⚠️  Could not build agent budget stats: ${agentBudgetError.message}`, { verbose: true });
     }
   }
+  return budgetStatsData;
+};
+
+// Verify results by searching for new PRs and comments
+export const verifyResults = async (owner, repo, branchName, issueNumber, prNumber, prUrl, referenceTime, argv, shouldAttachLogs, shouldRestart = false, sessionId = null, tempDir = null, anthropicTotalCostUSD = null, publicPricingEstimate = null, pricingInfo = null, errorDuringExecution = false, sessionType = 'new', resultModelUsage = null, streamTokenUsage = null, subAgentCalls = null, precomputedBudgetStatsData = null) => {
+  await log('\n🔍 Searching for created pull requests or comments...');
+
+  // Issue #1491, #1526, #2115: reuse data already calculated for the working
+  // session summary; retain the fallback for callers that do not precompute it.
+  const budgetStatsData =
+    precomputedBudgetStatsData ??
+    (await buildSessionBudgetStatsData({
+      argv,
+      sessionId,
+      tempDir,
+      resultModelUsage,
+      streamTokenUsage,
+      subAgentCalls,
+      pricingInfo,
+    }));
 
   try {
     // Get the current user's GitHub username
@@ -1241,7 +1261,15 @@ export const checkForAiCreatedComments = async (sessionStartTime, owner, repo, p
  * @param {string} options.repo - Repository name
  * @returns {Promise<boolean>} - True if comment was posted successfully
  */
-export const attachSolutionSummary = async ({ resultSummary, prNumber, issueNumber, owner, repo }) => {
+export const buildWorkingSessionSummaryDetails = ({ publicPricingEstimate = null, anthropicTotalCostUSD = null, pricingInfo = null, budgetStatsData = null } = {}) => {
+  const costInfo = buildCostInfoString(publicPricingEstimate, anthropicTotalCostUSD, pricingInfo, {
+    includeTokenUsage: false,
+  });
+  const budgetStats = budgetStatsData ? buildBudgetStatsString(budgetStatsData.tokenUsage, budgetStatsData.subAgentCalls) : '';
+  return `${costInfo}${budgetStats}`.trim();
+};
+
+export const attachSolutionSummary = async ({ resultSummary, prNumber, issueNumber, owner, repo, publicPricingEstimate = null, anthropicTotalCostUSD = null, pricingInfo = null, budgetStatsData = null }) => {
   if (!resultSummary || typeof resultSummary !== 'string') {
     await log('⚠️  No working session summary available to attach', { verbose: true });
     return false;
@@ -1256,10 +1284,16 @@ export const attachSolutionSummary = async ({ resultSummary, prNumber, issueNumb
   }
 
   try {
+    const usageDetails = buildWorkingSessionSummaryDetails({
+      publicPricingEstimate,
+      anthropicTotalCostUSD,
+      pricingInfo,
+      budgetStatsData,
+    });
     const comment = `${toolComments.WORKING_SESSION_SUMMARY_AUTOMATION_MARKER}
 ## ${toolComments.WORKING_SESSION_SUMMARY_MARKER}
 
-${resultSummary}
+${resultSummary}${usageDetails ? `\n\n${usageDetails}` : ''}
 
 ---
 *${toolComments.WORKING_SESSION_SUMMARY_AUTOMATED_FOOTER}*`;
@@ -1316,7 +1350,7 @@ ${resultSummary}
  * @param {boolean} [options.success=true] - skip attachment for failed iterations
  * @returns {Promise<{attached: boolean, reason: string}>}
  */
-export const maybeAttachWorkingSessionSummary = async ({ argv, resultSummary, workStartTime, owner, repo, prNumber, issueNumber, success = true }) => {
+export const maybeAttachWorkingSessionSummary = async ({ argv, resultSummary, workStartTime, owner, repo, prNumber, issueNumber, success = true, publicPricingEstimate = null, anthropicTotalCostUSD = null, pricingInfo = null, budgetStatsData = null, sessionUsage = null }) => {
   if (!success) {
     return { attached: false, reason: 'iteration_failed' };
   }
@@ -1352,6 +1386,25 @@ export const maybeAttachWorkingSessionSummary = async ({ argv, resultSummary, wo
     return { attached: false, reason: 'no_attach_decision' };
   }
 
-  const ok = await attachSolutionSummary({ resultSummary, prNumber, issueNumber, owner, repo });
-  return { attached: !!ok, reason: ok ? 'attached' : 'post_failed' };
+  const resolvedBudgetStatsData =
+    budgetStatsData ??
+    (sessionUsage
+      ? await buildSessionBudgetStatsData({
+          argv,
+          pricingInfo,
+          ...sessionUsage,
+        })
+      : null);
+  const ok = await attachSolutionSummary({
+    resultSummary,
+    prNumber,
+    issueNumber,
+    owner,
+    repo,
+    publicPricingEstimate,
+    anthropicTotalCostUSD,
+    pricingInfo,
+    budgetStatsData: resolvedBudgetStatsData,
+  });
+  return { attached: !!ok, reason: ok ? 'attached' : 'post_failed', budgetStatsData: resolvedBudgetStatsData };
 };

package/src/solve.watch.lib.mjs

@@ -580,6 +580,10 @@ export const watchForFeedback = async params => {
               prNumber,
               issueNumber,
               success: true,
+              publicPricingEstimate: toolResult.publicPricingEstimate,
+              anthropicTotalCostUSD: latestAnthropicCost,
+              pricingInfo: toolResult.pricingInfo,
+              budgetStatsData: autoRestartBudgetStatsData,
             });
           } catch (summaryError) {
             reportError(summaryError, {

package/src/use-m-bootstrap.lib.mjs

@@ -3,7 +3,14 @@
 import { wrapUseWithRetry } from './use-with-retry.lib.mjs';
 
 export const USE_M_BOOTSTRAP_URL = 'https://unpkg.com/use-m/use.js';
-export const USE_M_BOOTSTRAP_FALLBACK_URL = 'https://unpkg.com/use-m@8.13.8/use.js';
+// Issue #2113: the fallback is only reached when unpkg cannot serve the `latest`
+// bundle, but until now it pinned 8.13.8 — the last release *without* any
+// corrupt-alias self-healing. A CDN hiccup therefore silently downgraded every
+// dependency import to the least resilient loader available. 8.14.4 is the first
+// release that both repairs corrupt aliases (8.14.3, use-m #66/#67) and removes
+// them with a retry budget (8.14.4, use-m #68), so the degraded path now keeps
+// upstream recovery instead of losing it.
+export const USE_M_BOOTSTRAP_FALLBACK_URL = 'https://unpkg.com/use-m@8.14.4/use.js';
 
 const isMissingUseMBundle = code => /^Not found: \/use-m@[^/]+\/use\.js\s*$/.test(code.trim());
 

package/src/use-with-retry.lib.mjs

@@ -16,13 +16,21 @@
  *   3. Node throws `Invalid package config <dir>/package.json.` with
  *      `code: 'ERR_INVALID_PACKAGE_CONFIG'` — the package.json itself
  *      is corrupt/truncated and cannot even be parsed (issue #1712).
+ *   4. Node throws `ERR_MODULE_NOT_FOUND` for a file imported by the package
+ *      entry point — npm left an incomplete package tree (issue #2113).
+ *   5. use-m's self-heal throws `Failed to remove corrupt npm alias ...`
+ *      because a concurrent filesystem mutation made recursive removal fail
+ *      with a retryable code such as `ENOTEMPTY` (issue #2113).
  *
- * The recovery is identical for all three: delete the broken alias install
+ * The recovery is identical for all five: delete the broken alias install
  * directory and ask use-m to re-fetch. A clean reinstall almost always
  * succeeds. This helper centralises that retry so every call site picks
  * it up.
  */
 
+const ALIAS_CLEANUP_ERROR = /^Failed to remove (?:corrupt|incomplete) npm alias '([^']+)'\.$/;
+const RETRYABLE_RM_CODES = new Set(['EBUSY', 'EMFILE', 'ENFILE', 'ENOTEMPTY', 'EPERM']);
+
 /**
  * @param {(specifier: string) => Promise<unknown>} use - the use-m loader.
  * @param {string} specifier - the npm specifier to load (e.g. `'getenv'`).
@@ -94,8 +102,9 @@ export const useWithRetry = async (use, specifier, options = {}) => {
           // Remember the file so the next attempt can bypass Node's poisoned
           // module cache if use-m hands us the same path again.
           cleanedImportPath = /Failed to import module from '/.test(error?.message ?? '') ? corruptedPath : null;
-        } catch {
+        } catch (cleanupError) {
           // Best-effort cleanup; fall through to retry regardless.
+          log(`cleanup of ${resolveAliasDir(corruptedPath)} failed: ${cleanupError?.message}`);
         }
       }
     }
@@ -128,9 +137,23 @@ export const isCorruptInstallError = error => {
   // resolve/import logic gets a chance to run.
   if (error?.code === 'ERR_INVALID_PACKAGE_CONFIG') return true;
   if (cause?.code === 'ERR_INVALID_PACKAGE_CONFIG') return true;
+  // Mode 4 (issue #2113): the installed entry point exists, but one of the
+  // files it imports does not. Restrict this to use-m's import wrapper so an
+  // unrelated application-level ERR_MODULE_NOT_FOUND is not retried.
+  const message = typeof error?.message === 'string' ? error.message : '';
+  if (/^Failed to import module from '/.test(message) && cause?.code === 'ERR_MODULE_NOT_FOUND') return true;
+  // Mode 5 (issue #2113): use-m@8.14.3 added its own alias self-heal, but its
+  // recursive rm used Node's default maxRetries=0, so a concurrent mutation
+  // escaped as ENOTEMPTY (or another rm-retry code). use-m@8.14.4 fixed this
+  // upstream (use-m #68) by giving removePackageAlias its own retry budget.
+  // The downstream guard is deliberately kept: the bootstrap can still land on
+  // an older use-m (a pinned CDN fallback, a preinstalled global, or a cached
+  // bundle), and even 8.14.4 rethrows this exact wrapper once its own budget is
+  // exhausted — in which case a fresh removal plus reinstall is still the right
+  // recovery.
+  if (ALIAS_CLEANUP_ERROR.test(message) && RETRYABLE_RM_CODES.has(cause?.code)) return true;
   // Mode 2 (also seen on hosted CI): npm install completes but the package
   // tree is incomplete, so use-m can't resolve the entry point.
-  const message = typeof error?.message === 'string' ? error.message : '';
   if (/^Failed to resolve the path to /.test(message)) return true;
   // Fallback string match for ERR_INVALID_PACKAGE_CONFIG (in case the error
   // bubbles through use-m without preserving the `code` property).
@@ -150,7 +173,10 @@ export const extractCorruptedFilePath = error => {
   // extract the package.json path so the caller's cleanup() walks up to
   // the alias dir.
   const invalidConfigMatch = message.match(/Invalid package config (\S+?package\.json)/);
-  return invalidConfigMatch ? invalidConfigMatch[1] : null;
+  if (invalidConfigMatch) return invalidConfigMatch[1];
+  // Mode 5 (issue #2113): use-m already reports the whole alias directory.
+  const cleanupMatch = message.match(ALIAS_CLEANUP_ERROR);
+  return cleanupMatch ? cleanupMatch[1] : null;
 };
 
 /**
@@ -176,11 +202,15 @@ export const resolveAliasDir = corruptedPath => {
   return segments.slice(0, -1).join('/') || corruptedPath;
 };
 
-const defaultCleanup = async path => {
-  const { rm } = await import('node:fs/promises');
-  await rm(path, { recursive: true, force: true });
+export const removeAliasWithRetry = async (path, options = {}) => {
+  const remove = options.rm ?? (await import('node:fs/promises')).rm;
+  const maxRetries = options.maxRetries ?? 5;
+  const retryDelay = options.retryDelay ?? 100;
+  await remove(path, { recursive: true, force: true, maxRetries, retryDelay });
 };
 
+const defaultCleanup = removeAliasWithRetry;
+
 // Cache-busting import: a query string makes Node treat the URL as a distinct
 // module, so the freshly reinstalled file is evaluated instead of the cached
 // SyntaxError from the corrupt one.