@link-assistant/hive-mind 1.31.2 → 1.31.4

package/CHANGELOG.md

@@ -1,5 +1,42 @@
 # @link-assistant/hive-mind
 
+## 1.31.4
+
+### Patch Changes
+
+- Extract large inline script blocks from release.yml into ./scripts/ to fix CI line-limit violation (issue #1428)
+
+  fix: configure release pipeline to react to docker=true so Dockerfile changes trigger Docker image rebuild (Issue #1423)
+
+  Previously, commits that changed only `Dockerfile` or `coolify/Dockerfile` produced `docker=true` but `code=false`. The `release` job required all test jobs to `succeed` — but those tests were correctly skipped (no JavaScript code changed). Since `skipped != 'success'`, the release job was also skipped, and no Docker image was rebuilt.
+
+  This was observed when PR #1420 (fixing `/home/hive/.config` ownership) was merged: both Dockerfiles changed, but CI run `23040959919` showed all Docker publish jobs as skipped.
+
+  The `release` job condition is now updated to:
+  - Also trigger when `docker-changed == 'true'` (not only `code=true`)
+  - Accept `skipped` as well as `success` for test/lint jobs (skipped = intentionally not run, not a failure)
+  - Block on any actual job `failure`
+
+  This directly configures CI/CD to react to `docker=true` — without misclassifying Dockerfiles as "code" files.
+
+  Full root cause analysis and timeline in `docs/case-studies/issue-1423/`.
+
+  Migrate GitHub Actions to Node.js 24 compatible versions to eliminate deprecation warnings before the June 2026 deadline
+
+## 1.31.3
+
+### Patch Changes
+
+- b77704d: fix: set Docker image version labels to actual release version (Issue #1419)
+
+  The `docker/metadata-action@v5` defaulted the `org.opencontainers.image.version`
+  OCI label to the Git ref name `"main"` instead of the actual release version.
+  Added explicit `labels` override to all four Docker metadata steps in both regular
+  and instant release pipelines.
+
+  Also added `.config` directory ownership and write-access verification to the Docker
+  image verification script to prevent the permission regression from recurring.
+
 ## 1.31.2
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@link-assistant/hive-mind",
-  "version": "1.31.2",
+  "version": "1.31.4",
   "description": "AI-powered issue solver and hive mind for collaborative problem solving",
   "main": "src/hive.mjs",
   "type": "module",
@@ -47,7 +47,7 @@
   },
   "homepage": "https://github.com/link-assistant/hive-mind#readme",
   "engines": {
-    "node": ">=18.0.0"
+    "node": ">=24.0.0"
   },
   "files": [
     "src",

package/src/config.lib.mjs

@@ -18,7 +18,9 @@ if (typeof globalThis.use === 'undefined') {
   }
 }
 
-const getenv = await use('getenv');
+const getenvModule = await use('getenv');
+// Node 24 CJS/ESM interop may return the whole module object instead of the function directly
+const getenv = typeof getenvModule === 'function' ? getenvModule : getenvModule.default || getenvModule;
 
 // Use semver package for version comparison (see issue #1146)
 import semver from 'semver';

package/src/hive.mjs

@@ -20,8 +20,10 @@ if (earlyArgs.includes('--help') || earlyArgs.includes('-h')) {
     globalThis.use = use;
     const yargsModule = await use('yargs@17.7.2');
     const yargs = yargsModule.default || yargsModule;
-    const { hideBin } = await use('yargs@17.7.2/helpers');
-    const rawArgs = hideBin(process.argv);
+    const helpersModuleHelp = await use('yargs@17.7.2/helpers');
+    const _helpersHelp = helpersModuleHelp.default || helpersModuleHelp;
+    const hideBinHelp = _helpersHelp.hideBin || (argv => argv.slice(2));
+    const rawArgs = hideBinHelp(process.argv);
     // Reuse createYargsConfig from shared module to avoid duplication
     const { createYargsConfig } = await import('./hive.config.lib.mjs');
     const helpYargs = createYargsConfig(yargs(rawArgs)).version(false);
@@ -86,7 +88,9 @@ if (isDirectExecution) {
     );
     const yargsModule = await withTimeout(use('yargs@17.7.2'), 30000, 'loading yargs');
     const yargs = yargsModule.default || yargsModule;
-    const { hideBin } = await withTimeout(use('yargs@17.7.2/helpers'), 30000, 'loading yargs helpers');
+    const helpersModuleMain = await withTimeout(use('yargs@17.7.2/helpers'), 30000, 'loading yargs helpers');
+    const _helpersMain = helpersModuleMain.default || helpersModuleMain;
+    const hideBin = _helpersMain.hideBin || (argv => argv.slice(2));
     const path = (await withTimeout(use('path'), 30000, 'loading path')).default;
     const fs = (await withTimeout(use('fs'), 30000, 'loading fs')).promises;
     // Import shared library functions

package/src/memory-check.mjs

@@ -19,7 +19,10 @@ const $silent = $({ mirror: false, capture: true });
 
 const yargsModule = await use('yargs@17.7.2');
 const yargs = yargsModule.default || yargsModule;
-const { hideBin } = await use('yargs@17.7.2/helpers');
+const helpersModule = await use('yargs@17.7.2/helpers');
+// Node 24 CJS/ESM interop may return the whole module object instead of named exports directly
+const _helpers = helpersModule.default || helpersModule;
+const hideBin = _helpers.hideBin || (argv => argv.slice(2));
 const fs = (await use('fs')).promises;
 
 // Import log function from lib.mjs

package/src/queue-config.lib.mjs

@@ -36,7 +36,9 @@ if (typeof globalThis.use === 'undefined') {
   }
 }
 
-const getenv = await use('getenv');
+const getenvModule = await use('getenv');
+// Node 24 CJS/ESM interop may return the whole module object instead of the function directly
+const getenv = typeof getenvModule === 'function' ? getenvModule : getenvModule.default || getenvModule;
 const linoModule = await use('links-notation');
 const LinoParser = linoModule.Parser || linoModule.default?.Parser;
 

package/src/solve.config.lib.mjs

@@ -17,7 +17,10 @@ export const initializeConfig = async use => {
   // Import yargs with specific version for hideBin support
   const yargsModule = await use('yargs@17.7.2');
   const yargs = yargsModule.default || yargsModule;
-  const { hideBin } = await use('yargs@17.7.2/helpers');
+  const helpersModule = await use('yargs@17.7.2/helpers');
+  // Node 24 CJS/ESM interop may return the whole module object instead of named exports directly
+  const helpers = helpersModule.default || helpersModule;
+  const hideBin = helpers.hideBin || (argv => argv.slice(2));
 
   return { yargs, hideBin };
 };

package/src/solve.mjs

@@ -77,8 +77,7 @@ const { startAutoRestartUntilMergeable } = await import('./solve.auto-merge.lib.
 const { runAutoEnsureRequirements } = await import('./solve.auto-ensure.lib.mjs');
 const exitHandler = await import('./exit-handler.lib.mjs');
 const { initializeExitHandler, installGlobalExitHandlers, safeExit } = exitHandler;
-const interruptLib = await import('./solve.interrupt.lib.mjs');
-const { createInterruptWrapper } = interruptLib;
+const { createInterruptWrapper } = await import('./solve.interrupt.lib.mjs');
 const getResourceSnapshot = memoryCheck.getResourceSnapshot;
 
 // Import new modular components
@@ -125,9 +124,7 @@ try {
 }
 global.verboseMode = argv.verbose;
 
-// If user specified a custom log directory, we would need to move the log file
-// However, this adds complexity, so we accept that early logs go to cwd
-// The trade-off is: early logs in cwd vs missing version/command in error cases
+// Early logs go to cwd; custom log dir takes effect after argv is parsed
 
 // Conditionally import tool-specific functions after argv is parsed
 let checkForUncommittedChanges;
@@ -190,7 +187,6 @@ if (!urlValidation.isValid) {
 }
 const { isIssueUrl, isPrUrl, normalizedUrl, owner, repo, number: urlNumber } = urlValidation;
 issueUrl = normalizedUrl || issueUrl;
-// Store owner and repo globally for error handlers and interrupt context
 global.owner = owner;
 global.repo = repo;
 cleanupContext.owner = owner;
@@ -517,17 +513,13 @@ if (isPrUrl) {
   issueNumber = urlNumber;
   await log(`📝 Issue mode: Working with issue #${issueNumber}`);
 }
-// Create or find temporary directory for cloning the repository
-// Pass workspace info for --enable-workspaces mode (works with all tools)
 const workspaceInfo = argv.enableWorkspaces ? { owner, repo, issueNumber } : null;
 const { tempDir, workspaceTmpDir, needsClone } = await setupTempDirectory(argv, workspaceInfo);
-// Populate cleanup context for signal handlers (owner/repo updated again here for redundancy)
 cleanupContext.tempDir = tempDir;
 cleanupContext.argv = argv;
 cleanupContext.owner = owner;
 cleanupContext.repo = repo;
 if (prNumber) cleanupContext.prNumber = prNumber;
-// Initialize limitReached variable outside try block for finally clause
 let limitReached = false;
 try {
   // Set up repository and clone using the new module

package/src/telegram-bot.mjs

@@ -24,7 +24,9 @@ const { loadLenvConfig } = await import('./lenv-reader.lib.mjs');
 const dotenvxModule = await use('@dotenvx/dotenvx');
 const dotenvx = dotenvxModule.default || dotenvxModule;
 
-const getenv = await use('getenv');
+const getenvModule = await use('getenv');
+// Node 24 CJS/ESM interop may return the whole module object instead of the function directly
+const getenv = typeof getenvModule === 'function' ? getenvModule : getenvModule.default || getenvModule;
 
 // Load .env configuration as base
 // quiet: true suppresses info messages, ignore: ['MISSING_ENV_FILE'] suppresses error when .env doesn't exist
@@ -37,7 +39,10 @@ loadLenvConfig({ override: true, quiet: true });
 
 const yargsModule = await use('yargs@17.7.2');
 const yargs = yargsModule.default || yargsModule;
-const { hideBin } = await use('yargs@17.7.2/helpers');
+const helpersModuleBot = await use('yargs@17.7.2/helpers');
+// Node 24 CJS/ESM interop may return the whole module object instead of named exports directly
+const _helpersBot = helpersModuleBot.default || helpersModuleBot;
+const hideBin = _helpersBot.hideBin || (argv => argv.slice(2));
 // Import yargs configurations, GitHub utilities, and telegram helpers
 const { createYargsConfig: createSolveYargsConfig, detectMalformedFlags } = await import('./solve.config.lib.mjs');
 const { createYargsConfig: createHiveYargsConfig } = await import('./hive.config.lib.mjs');