@link-assistant/hive-mind 0.42.1 → 0.42.2

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # @link-assistant/hive-mind
 
+## 0.42.2
+
+### Patch Changes
+
+- dca5bed: Make --auto-continue enabled by default
+
+  - Changed default value from false to true for --auto-continue in both hive and solve commands
+  - Smart handling of -s (--skip-issues-with-prs) flag interaction:
+    - When -s is used, auto-continue is automatically disabled to avoid conflicts
+    - Explicit --auto-continue with -s shows proper error message
+    - Users can still use --no-auto-continue to explicitly disable
+  - This improves user experience as users typically want to continue working on existing PRs
+
+  Fixes #454
+
 ## 0.42.1
 
 ### Patch Changes

package/package.json

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

package/src/hive.mjs

@@ -22,11 +22,9 @@ if (earlyArgs.includes('--help') || earlyArgs.includes('-h')) {
     const yargs = yargsModule.default || yargsModule;
     const { hideBin } = await use('yargs@17.7.2/helpers');
     const rawArgs = hideBin(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);
-
     // Show help and exit
     helpYargs.showHelp();
     process.exit(0);
@@ -48,7 +46,6 @@ export { createYargsConfig } from './hive.config.lib.mjs';
 import { fileURLToPath } from 'url';
 const isDirectExecution = process.argv[1] === fileURLToPath(import.meta.url) ||
                           (process.argv[1] && (process.argv[1].includes('/hive') || process.argv[1].endsWith('hive')));
-
 if (isDirectExecution) {
 console.log('🐝 Hive Mind - AI-powered issue solver');
 console.log('   Initializing...');
@@ -63,7 +60,6 @@ const withTimeout = (promise, timeoutMs, operation) => {
     )
   ]);
 };
-
 // Use use-m to dynamically import modules for cross-runtime compatibility
 if (typeof use === 'undefined') {
   try {
@@ -157,7 +153,6 @@ async function fetchIssuesFromRepositories(owner, scope, monitorTag, fetchAllIss
         return graphqlResult.issues;
       }
     }
-
     // Strategy 2: Fallback to gh api --paginate approach (comprehensive but slower)
     await log('   📋 Using gh api --paginate approach for comprehensive coverage...', { verbose: true });
 
@@ -170,18 +165,15 @@ async function fetchIssuesFromRepositories(owner, scope, monitorTag, fetchAllIss
     } else {
       repoListCmd = `gh api users/${owner}/repos --paginate --jq '.[] | {name: .name, owner: .owner.login, isArchived: .archived}'`;
     }
-
     await log('   📋 Fetching repository list (using --paginate for unlimited pagination)...', { verbose: true });
     await log(`   🔎 Command: ${repoListCmd}`, { verbose: true });
 
     // Add delay for rate limiting
     await new Promise(resolve => setTimeout(resolve, 2000));
-
     const { stdout: repoOutput } = await execAsync(repoListCmd, { encoding: 'utf8', env: process.env });
     // Parse the output line by line, as gh api with --jq outputs one JSON object per line
     const repoLines = repoOutput.trim().split('\n').filter(line => line.trim());
     const allRepositories = repoLines.map(line => JSON.parse(line));
-
     await log(`   📊 Found ${allRepositories.length} repositories`);
 
     // Filter repositories to only include those owned by the target user/org
@@ -190,30 +182,24 @@ async function fetchIssuesFromRepositories(owner, scope, monitorTag, fetchAllIss
       return repoOwner === owner;
     });
     const unownedCount = allRepositories.length - ownedRepositories.length;
-
     if (unownedCount > 0) {
       await log(`   ⏭️  Skipping ${unownedCount} repository(ies) not owned by ${owner}`);
     }
-
     // Filter out archived repositories from owned repositories
     const repositories = ownedRepositories.filter(repo => !repo.isArchived);
     const archivedCount = ownedRepositories.length - repositories.length;
-
     if (archivedCount > 0) {
       await log(`   ⏭️  Skipping ${archivedCount} archived repository(ies)`);
     }
-
     await log(`   ✅ Processing ${repositories.length} non-archived repositories owned by ${owner}`);
 
     let collectedIssues = [];
     let processedRepos = 0;
-
     // Process repositories in batches to avoid overwhelming the API
     for (const repo of repositories) {
       try {
         const repoName = repo.name;
         const ownerName = repo.owner?.login || owner;
-
         await log(`   🔍 Fetching issues from ${ownerName}/${repoName}...`, { verbose: true });
 
         // Build the appropriate issue list command
@@ -223,7 +209,6 @@ async function fetchIssuesFromRepositories(owner, scope, monitorTag, fetchAllIss
         } else {
           issueCmd = `gh issue list --repo ${ownerName}/${repoName} --state open --label "${monitorTag}" --json url,title,number,createdAt`;
         }
-
         // Add delay between repository requests
         await new Promise(resolve => setTimeout(resolve, 1000));
 
@@ -488,13 +473,26 @@ if (argv.projectMode) {
 const tool = argv.tool || 'claude';
 await validateAndExitOnInvalidModel(argv.model, tool, safeExit);
 
-// Validate conflicting options
-if (argv.skipIssuesWithPrs && argv.autoContinue) {
-  await log('❌ Conflicting options: --skip-issues-with-prs and --auto-continue cannot be used together', { level: 'error' });
-  await log('   --skip-issues-with-prs: Skips issues that have any open PRs', { level: 'error' });
-  await log('   --auto-continue: Continues with existing PRs instead of creating new ones', { level: 'error' });
-  await log(`   📁 Full log file: ${absoluteLogPath}`, { level: 'error' });
-  await safeExit(1, 'Error occurred');
+// Handle -s (--skip-issues-with-prs) and --auto-continue interaction
+// Detect if user explicitly passed --auto-continue or --no-auto-continue
+const hasExplicitAutoContinue = rawArgs.includes('--auto-continue');
+const hasExplicitNoAutoContinue = rawArgs.includes('--no-auto-continue');
+
+if (argv.skipIssuesWithPrs) {
+  // If user explicitly passed --auto-continue with -s, that's a conflict
+  if (hasExplicitAutoContinue) {
+    await log('❌ Conflicting options: --skip-issues-with-prs and --auto-continue cannot be used together', { level: 'error' });
+    await log('   --skip-issues-with-prs: Skips issues that have any open PRs', { level: 'error' });
+    await log('   --auto-continue: Continues with existing PRs instead of creating new ones', { level: 'error' });
+    await log(`   📁 Full log file: ${absoluteLogPath}`, { level: 'error' });
+    await safeExit(1, 'Error occurred');
+  }
+
+  // If user didn't explicitly set auto-continue, disable it when -s is used
+  // This is because -s means "skip issues with PRs" which conflicts with auto-continue
+  if (!hasExplicitNoAutoContinue) {
+    argv.autoContinue = false;
+  }
 }
 
 // Helper function to check GitHub permissions - moved to github.lib.mjs
@@ -751,7 +749,7 @@ async function worker(workerId) {
         const dryRunFlag = argv.dryRun ? ' --dry-run' : '';
         const skipToolConnectionCheckFlag = (argv.skipToolConnectionCheck || argv.toolConnectionCheck === false) ? ' --skip-tool-connection-check' : '';
         const toolFlag = argv.tool ? ` --tool ${argv.tool}` : '';
-        const autoContinueFlag = argv.autoContinue ? ' --auto-continue' : '';
+        const autoContinueFlag = argv.autoContinue ? ' --auto-continue' : ' --no-auto-continue';
         const thinkFlag = argv.think ? ` --think ${argv.think}` : '';
         const promptPlanSubAgentFlag = argv.promptPlanSubAgent ? ' --prompt-plan-sub-agent' : '';
         const noSentryFlag = !argv.sentry ? ' --no-sentry' : '';
@@ -794,6 +792,8 @@ async function worker(workerId) {
         }
         if (argv.autoContinue) {
           args.push('--auto-continue');
+        } else {
+          args.push('--no-auto-continue');
         }
         if (argv.think) {
           args.push('--think', argv.think);