@link-assistant/hive-mind 1.1.0 → 1.2.0

package/CHANGELOG.md

@@ -1,5 +1,63 @@
 # @link-assistant/hive-mind
 
+## 1.2.0
+
+### Minor Changes
+
+- Add experimental --execute-tool-with-bun option to improve speed and memory usage
+
+  This feature adds the `--execute-tool-with-bun` option that allows users to execute the AI tool using `bunx claude` instead of `claude`, which may provide performance benefits in terms of speed and memory usage.
+
+  **Supported commands:**
+  - `solve` - Uses `bunx claude` when option is enabled
+  - `task` - Uses `bunx claude` when option is enabled
+  - `review` - Uses `bunx claude` when option is enabled
+  - `hive` - Passes the option through to the `solve` subprocess
+
+  **How It Works:**
+  When `--execute-tool-with-bun` is enabled, the `claudePath` variable is set to `'bunx claude'` instead of `'claude'` (or `CLAUDE_PATH` environment variable).
+
+  **Usage Examples:**
+
+  ```bash
+  # Use with solve command
+  solve https://github.com/owner/repo/issues/123 --execute-tool-with-bun
+
+  # Use with task command
+  task "implement feature X" --execute-tool-with-bun
+
+  # Use with review command
+  review https://github.com/owner/repo/pull/456 --execute-tool-with-bun
+
+  # Use with hive command (passes through to solve)
+  hive https://github.com/owner/repo --execute-tool-with-bun
+  ```
+
+  The option defaults to `false` to maintain backward compatibility.
+
+  Fixes #812
+
+  feat(hive): recheck issue conditions before processing queue items
+
+  Added `recheckIssueConditions()` function to validate issue state right before processing,
+  preventing wasted resources on issues that should be skipped due to changed conditions since queuing.
+
+  **Checks performed:**
+  - **Issue state**: Verifies the issue is still open
+  - **Open PRs**: Checks if issue has PRs (when `--skip-issues-with-prs` is enabled)
+  - **Repository status**: Confirms repository is not archived
+
+  **Benefits:**
+  - Prevents processing closed issues
+  - Avoids duplicate work when PRs already exist
+  - Stops work on newly archived repositories
+  - Saves AI model tokens and compute resources
+
+  **Performance impact:**
+  Minimal overhead per issue (~300-500ms for API calls), negligible compared to 5-15 minute solve time.
+
+  Fixes #810
+
 ## 1.1.0
 
 ### Minor Changes

package/package.json

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

package/src/hive.config.lib.mjs

@@ -286,6 +286,11 @@ export const createYargsConfig = yargsInstance => {
       description: '[EXPERIMENTAL] Include guidance for managing REQUIREMENTS.md and ARCHITECTURE.md files. When enabled, agents will update these documentation files when changes affect requirements or architecture.',
       default: false,
     })
+    .option('execute-tool-with-bun', {
+      type: 'boolean',
+      description: 'Execute the AI tool using bunx (experimental, may improve speed and memory usage) - passed to solve command',
+      default: false,
+    })
     .parserConfiguration({
       'boolean-negation': true,
       'strip-dashed': false,

package/src/hive.mjs

@@ -110,6 +110,8 @@ if (isDirectExecution) {
     const { tryFetchIssuesWithGraphQL } = graphqlLib;
     const solutionDraftsLib = await import('./list-solution-drafts.lib.mjs');
     const { listSolutionDrafts } = solutionDraftsLib;
+    const recheckLib = await import('./hive.recheck.lib.mjs');
+    const { recheckIssueConditions } = recheckLib;
     const commandName = process.argv[1] ? process.argv[1].split('/').pop() : '';
     const isLocalScript = commandName.endsWith('.mjs');
     const solveCommand = isLocalScript ? './solve.mjs' : 'solve';
@@ -713,6 +715,16 @@ if (isDirectExecution) {
 
         await log(`\n👷 Worker ${workerId} processing: ${issueUrl}`);
 
+        // Recheck conditions before processing to avoid wasted work
+        const recheckResult = await recheckIssueConditions(issueUrl, argv);
+        if (!recheckResult.shouldProcess) {
+          await log(`   ⏭️  Skipping issue: ${recheckResult.reason}`);
+          issueQueue.markCompleted(issueUrl);
+          const stats = issueQueue.getStats();
+          await log(`   📊 Queue: ${stats.queued} waiting, ${stats.processing} processing, ${stats.completed} completed, ${stats.failed} failed`);
+          continue;
+        }
+
         // Track if this issue failed
         let issueFailed = false;
 
@@ -756,6 +768,7 @@ if (isDirectExecution) {
             if (argv.promptIssueReporting) args.push('--prompt-issue-reporting');
             if (argv.promptCaseStudies) args.push('--prompt-case-studies');
             if (argv.promptPlaywrightMcp !== undefined) args.push(argv.promptPlaywrightMcp ? '--prompt-playwright-mcp' : '--no-prompt-playwright-mcp');
+            if (argv.executeToolWithBun) args.push('--execute-tool-with-bun');
             // Log the actual command being executed so users can investigate/reproduce
             await log(`   📋 Command: ${solveCommand} ${args.join(' ')}`);
 

package/src/hive.recheck.lib.mjs

@@ -0,0 +1,86 @@
+#!/usr/bin/env node
+// Library for rechecking issue conditions in hive queue processing
+
+import { log, cleanErrorMessage } from './lib.mjs';
+import { batchCheckPullRequestsForIssues, batchCheckArchivedRepositories } from './github.lib.mjs';
+import { reportError } from './sentry.lib.mjs';
+
+/**
+ * Recheck conditions for an issue right before processing
+ * This ensures the issue should still be processed even if conditions changed since queuing
+ * @param {string} issueUrl - The URL of the issue to check
+ * @param {Object} argv - Command line arguments with configuration
+ * @returns {Promise<{shouldProcess: boolean, reason?: string}>}
+ */
+export async function recheckIssueConditions(issueUrl, argv) {
+  try {
+    // Extract owner, repo, and issue number from URL
+    const urlMatch = issueUrl.match(/github\.com\/([^/]+)\/([^/]+)\/issues\/(\d+)/);
+    if (!urlMatch) {
+      await log(`      ⚠️  Could not parse issue URL: ${issueUrl}`, { verbose: true });
+      return { shouldProcess: true }; // Process anyway if we can't parse
+    }
+
+    const [, owner, repo, issueNumber] = urlMatch;
+    const issueNum = parseInt(issueNumber);
+
+    await log(`      🔍 Rechecking conditions for issue #${issueNum}...`, { verbose: true });
+
+    // Check 1: Verify issue is still open
+    try {
+      const { execSync } = await import('child_process');
+      const issueState = execSync(`gh api repos/${owner}/${repo}/issues/${issueNum} --jq .state`, {
+        encoding: 'utf8',
+      }).trim();
+
+      if (issueState === 'closed') {
+        return {
+          shouldProcess: false,
+          reason: 'Issue is now closed',
+        };
+      }
+      await log(`      ✅ Issue is still open`, { verbose: true });
+    } catch (error) {
+      await log(`      ⚠️  Could not check issue state: ${cleanErrorMessage(error)}`, { verbose: true });
+      // Continue checking other conditions
+    }
+
+    // Check 2: If skipIssuesWithPrs is enabled, verify issue still has no open PRs
+    if (argv.skipIssuesWithPrs) {
+      const prResults = await batchCheckPullRequestsForIssues(owner, repo, [issueNum]);
+      const prInfo = prResults[issueNum];
+
+      if (prInfo && prInfo.openPRCount > 0) {
+        return {
+          shouldProcess: false,
+          reason: `Issue now has ${prInfo.openPRCount} open PR${prInfo.openPRCount > 1 ? 's' : ''}`,
+        };
+      }
+      await log(`      ✅ Issue still has no open PRs`, { verbose: true });
+    }
+
+    // Check 3: Verify repository is not archived
+    const archivedStatusMap = await batchCheckArchivedRepositories([{ owner, name: repo }]);
+    const repoKey = `${owner}/${repo}`;
+
+    if (archivedStatusMap[repoKey] === true) {
+      return {
+        shouldProcess: false,
+        reason: 'Repository is now archived',
+      };
+    }
+    await log(`      ✅ Repository is not archived`, { verbose: true });
+
+    await log(`      ✅ All conditions passed, proceeding with processing`, { verbose: true });
+    return { shouldProcess: true };
+  } catch (error) {
+    reportError(error, {
+      context: 'recheck_issue_conditions',
+      issueUrl,
+      operation: 'recheck_conditions',
+    });
+    await log(`      ⚠️  Error rechecking conditions: ${cleanErrorMessage(error)}`, { level: 'warning' });
+    // On error, allow processing to continue (fail open)
+    return { shouldProcess: true };
+  }
+}

package/src/review.mjs

@@ -19,14 +19,15 @@ if (earlyArgs.includes('--help') || earlyArgs.includes('-h')) {
   // Show help and exit
   console.log('Usage: review.mjs <pr-url> [options]');
   console.log('\nOptions:');
-  console.log('  --version          Show version number');
-  console.log('  --help, -h         Show help');
-  console.log('  --resume, -r       Resume from a previous session ID');
-  console.log('  --dry-run, -n      Prepare everything but do not execute Claude');
-  console.log('  --model, -m        Model to use (opus, sonnet, or full model ID) [default: opus]');
-  console.log('  --focus, -f        Focus areas for review [default: all]');
-  console.log('  --approve          If review passes, approve the PR');
-  console.log('  --verbose, -v      Enable verbose logging');
+  console.log('  --version               Show version number');
+  console.log('  --help, -h              Show help');
+  console.log('  --resume, -r            Resume from a previous session ID');
+  console.log('  --dry-run, -n           Prepare everything but do not execute Claude');
+  console.log('  --model, -m             Model to use (opus, sonnet, or full model ID) [default: opus]');
+  console.log('  --focus, -f             Focus areas for review [default: all]');
+  console.log('  --approve               If review passes, approve the PR');
+  console.log('  --verbose, -v           Enable verbose logging');
+  console.log('  --execute-tool-with-bun Execute the AI tool using bunx (experimental) [default: false]');
   process.exit(0);
 }
 
@@ -91,6 +92,11 @@ const argv = yargs()
     alias: 'v',
     default: false,
   })
+  .option('execute-tool-with-bun', {
+    type: 'boolean',
+    description: 'Execute the AI tool using bunx (experimental, may improve speed and memory usage)',
+    default: false,
+  })
   .demandCommand(1, 'The GitHub pull request URL is required')
   .parserConfiguration({
     'boolean-negation': true,
@@ -126,7 +132,9 @@ if (!prUrl.match(/^https:\/\/github\.com\/[^/]+\/[^/]+\/pull\/\d+$/)) {
   process.exit(1);
 }
 
-const claudePath = process.env.CLAUDE_PATH || 'claude';
+// Determine claude command path based on --execute-tool-with-bun option
+// When enabled, uses 'bunx claude' which may improve speed and memory usage
+const claudePath = argv.executeToolWithBun ? 'bunx claude' : process.env.CLAUDE_PATH || 'claude';
 
 // Extract repository and PR number from URL
 const urlParts = prUrl.split('/');

package/src/solve.config.lib.mjs

@@ -253,6 +253,11 @@ export const createYargsConfig = yargsInstance => {
         choices: ['claude', 'opencode', 'codex', 'agent'],
         default: 'claude',
       })
+      .option('execute-tool-with-bun', {
+        type: 'boolean',
+        description: 'Execute the AI tool using bunx (experimental, may improve speed and memory usage)',
+        default: false,
+      })
       .option('enable-workspaces', {
         type: 'boolean',
         description: 'Use separate workspace directory structure with repository/ and tmp/ folders. Works with all tools (claude, opencode, codex, agent). Experimental feature.',

package/src/solve.mjs

@@ -236,7 +236,7 @@ if (argv.verbose) {
   await log(`   Is Issue URL: ${!!isIssueUrl}`, { verbose: true });
   await log(`   Is PR URL: ${!!isPrUrl}`, { verbose: true });
 }
-const claudePath = process.env.CLAUDE_PATH || 'claude';
+const claudePath = argv.executeToolWithBun ? 'bunx claude' : process.env.CLAUDE_PATH || 'claude';
 // Note: owner, repo, and urlNumber are already extracted from validateGitHubUrl() above
 // The parseUrlComponents() call was removed as it had a bug with hash fragments (#issuecomment-xyz)
 // and the validation result already provides these values correctly parsed

package/src/task.mjs

@@ -124,6 +124,11 @@ const argv = yargs()
     default: 'text',
     choices: ['text', 'json'],
   })
+  .option('execute-tool-with-bun', {
+    type: 'boolean',
+    description: 'Execute the AI tool using bunx (experimental, may improve speed and memory usage)',
+    default: false,
+  })
   .check(argv => {
     if (!argv['task-description'] && !argv._[0]) {
       throw new Error('Please provide a task description');
@@ -186,7 +191,7 @@ await log(formatAligned('💡', 'Clarify mode:', argv.clarify ? 'enabled' : 'dis
 await log(formatAligned('🔍', 'Decompose mode:', argv.decompose ? 'enabled' : 'disabled'));
 await log(formatAligned('📄', 'Output format:', argv.outputFormat));
 
-const claudePath = process.env.CLAUDE_PATH || 'claude';
+const claudePath = argv.executeToolWithBun ? 'bunx claude' : process.env.CLAUDE_PATH || 'claude';
 
 // Helper function to execute Claude command
 const executeClaude = (prompt, model) => {