@link-assistant/hive-mind 0.46.0 → 0.47.0

package/src/solve.error-handlers.lib.mjs

@@ -14,22 +14,8 @@ import { handleErrorWithIssueCreation } from './github-issue-creator.lib.mjs';
 /**
  * Handles log attachment and PR closing on failure
  */
-export const handleFailure = async (options) => {
-  const {
-    error,
-    errorType,
-    shouldAttachLogs,
-    argv,
-    global,
-    owner,
-    repo,
-    log,
-    getLogFile,
-    attachLogToGitHub,
-    cleanErrorMessage,
-    sanitizeLogContent,
-    $
-  } = options;
+export const handleFailure = async options => {
+  const { error, errorType, shouldAttachLogs, argv, global, owner, repo, log, getLogFile, attachLogToGitHub, cleanErrorMessage, sanitizeLogContent, $ } = options;
 
   // Offer to create GitHub issue for the error
   try {
@@ -41,14 +27,14 @@ export const handleFailure = async (options) => {
         owner: global.owner || owner,
         repo: global.repo || repo,
         prNumber: global.createdPR?.number,
-        errorType
+        errorType,
       },
-      skipPrompt: !process.stdin.isTTY || argv.noIssueCreation
+      skipPrompt: !process.stdin.isTTY || argv.noIssueCreation,
     });
   } catch (issueError) {
     reportError(issueError, {
       context: 'automatic_issue_creation',
-      operation: 'handle_error_with_issue_creation'
+      operation: 'handle_error_with_issue_creation',
     });
     await log(`⚠️  Could not create issue: ${issueError.message}`, { level: 'warning' });
   }
@@ -67,7 +53,7 @@ export const handleFailure = async (options) => {
         log,
         sanitizeLogContent,
         verbose: argv.verbose,
-        errorMessage: cleanErrorMessage(error)
+        errorMessage: cleanErrorMessage(error),
       });
       if (logUploadSuccess) {
         await log('📎 Failure log attached to Pull Request');
@@ -77,7 +63,7 @@ export const handleFailure = async (options) => {
         context: 'attach_failure_log',
         prNumber: global.createdPR?.number,
         errorType,
-        operation: 'attach_log_to_pr'
+        operation: 'attach_log_to_pr',
       });
       await log(`⚠️  Could not attach failure log: ${attachError.message}`, { level: 'warning' });
     }
@@ -87,11 +73,7 @@ export const handleFailure = async (options) => {
   if (argv.autoClosePullRequestOnFail && global.createdPR && global.createdPR.number) {
     await log('\n🔒 Auto-closing pull request due to failure...');
     try {
-      const closeMessage = errorType === 'uncaughtException'
-        ? 'Auto-closed due to uncaught exception. Logs have been attached for debugging.'
-        : errorType === 'unhandledRejection'
-        ? 'Auto-closed due to unhandled rejection. Logs have been attached for debugging.'
-        : 'Auto-closed due to execution failure. Logs have been attached for debugging.';
+      const closeMessage = errorType === 'uncaughtException' ? 'Auto-closed due to uncaught exception. Logs have been attached for debugging.' : errorType === 'unhandledRejection' ? 'Auto-closed due to unhandled rejection. Logs have been attached for debugging.' : 'Auto-closed due to execution failure. Logs have been attached for debugging.';
 
       const result = await $`gh pr close ${global.createdPR.number} --repo ${global.owner || owner}/${global.repo || repo} --comment ${closeMessage}`;
       if (result.exitCode === 0) {
@@ -103,7 +85,7 @@ export const handleFailure = async (options) => {
         prNumber: global.createdPR?.number,
         owner,
         repo,
-        operation: 'close_pull_request'
+        operation: 'close_pull_request',
       });
       await log(`⚠️  Could not close pull request: ${closeError.message}`, { level: 'warning' });
     }
@@ -113,23 +95,10 @@ export const handleFailure = async (options) => {
 /**
  * Creates an uncaught exception handler
  */
-export const createUncaughtExceptionHandler = (options) => {
-  const {
-    log,
-    cleanErrorMessage,
-    absoluteLogPath,
-    shouldAttachLogs,
-    argv,
-    global,
-    owner,
-    repo,
-    getLogFile,
-    attachLogToGitHub,
-    sanitizeLogContent,
-    $
-  } = options;
+export const createUncaughtExceptionHandler = options => {
+  const { log, cleanErrorMessage, absoluteLogPath, shouldAttachLogs, argv, global, owner, repo, getLogFile, attachLogToGitHub, sanitizeLogContent, $ } = options;
 
-  return async (error) => {
+  return async error => {
     await log(`\n❌ Uncaught Exception: ${cleanErrorMessage(error)}`, { level: 'error' });
     await log(`   📁 Full log file: ${absoluteLogPath}`, { level: 'error' });
 
@@ -146,7 +115,7 @@ export const createUncaughtExceptionHandler = (options) => {
       attachLogToGitHub,
       cleanErrorMessage,
       sanitizeLogContent,
-      $
+      $,
     });
 
     await safeExit(1, 'Error occurred');
@@ -156,23 +125,10 @@ export const createUncaughtExceptionHandler = (options) => {
 /**
  * Creates an unhandled rejection handler
  */
-export const createUnhandledRejectionHandler = (options) => {
-  const {
-    log,
-    cleanErrorMessage,
-    absoluteLogPath,
-    shouldAttachLogs,
-    argv,
-    global,
-    owner,
-    repo,
-    getLogFile,
-    attachLogToGitHub,
-    sanitizeLogContent,
-    $
-  } = options;
+export const createUnhandledRejectionHandler = options => {
+  const { log, cleanErrorMessage, absoluteLogPath, shouldAttachLogs, argv, global, owner, repo, getLogFile, attachLogToGitHub, sanitizeLogContent, $ } = options;
 
-  return async (reason) => {
+  return async reason => {
     await log(`\n❌ Unhandled Rejection: ${cleanErrorMessage(reason)}`, { level: 'error' });
     await log(`   📁 Full log file: ${absoluteLogPath}`, { level: 'error' });
 
@@ -189,7 +145,7 @@ export const createUnhandledRejectionHandler = (options) => {
       attachLogToGitHub,
       cleanErrorMessage,
       sanitizeLogContent,
-      $
+      $,
     });
 
     await safeExit(1, 'Error occurred');
@@ -199,22 +155,8 @@ export const createUnhandledRejectionHandler = (options) => {
 /**
  * Handles execution errors in the main catch block
  */
-export const handleMainExecutionError = async (options) => {
-  const {
-    error,
-    log,
-    cleanErrorMessage,
-    absoluteLogPath,
-    shouldAttachLogs,
-    argv,
-    global,
-    owner,
-    repo,
-    getLogFile,
-    attachLogToGitHub,
-    sanitizeLogContent,
-    $
-  } = options;
+export const handleMainExecutionError = async options => {
+  const { error, log, cleanErrorMessage, absoluteLogPath, shouldAttachLogs, argv, global, owner, repo, getLogFile, attachLogToGitHub, sanitizeLogContent, $ } = options;
 
   // Special handling for authentication errors
   if (error.isAuthError) {
@@ -249,8 +191,8 @@ export const handleMainExecutionError = async (options) => {
     attachLogToGitHub,
     cleanErrorMessage,
     sanitizeLogContent,
-    $
+    $,
   });
 
   await safeExit(1, 'Error occurred');
-};
+};

package/src/solve.execution.lib.mjs

@@ -24,12 +24,7 @@ const memoryCheck = await import('./memory-check.mjs');
 
 // Import shared library functions
 const lib = await import('./lib.mjs');
-const {
-  log,
-  getLogFile,
-  cleanErrorMessage,
-  formatAligned
-} = lib;
+const { log, getLogFile, cleanErrorMessage, formatAligned } = lib;
 
 // Import GitHub-related functions
 const githubLib = await import('./github.lib.mjs');
@@ -37,13 +32,10 @@ const githubLib = await import('./github.lib.mjs');
 const sentryLib = await import('./sentry.lib.mjs');
 const { reportError } = sentryLib;
 
-const {
-  sanitizeLogContent,
-  attachLogToGitHub
-} = githubLib;
+const { sanitizeLogContent, attachLogToGitHub } = githubLib;
 
 // Create or find temporary directory for cloning the repository
-export const setupTempDirectory = async (argv) => {
+export const setupTempDirectory = async argv => {
   let tempDir;
   let isResuming = argv.resume;
 
@@ -65,7 +57,7 @@ export const setupTempDirectory = async (argv) => {
       reportError(err, {
         context: 'resume_session_setup',
         sessionId: argv.resume,
-        operation: 'find_session_log'
+        operation: 'find_session_log',
       });
       await log(`Warning: Session log for ${argv.resume} not found, but continuing with resume attempt`);
       tempDir = path.join(os.tmpdir(), `gh-issue-solver-resume-${argv.resume}-${Date.now()}`);
@@ -146,7 +138,7 @@ export const setupRepository = async (argv, owner, repo) => {
 
         for (let attempt = 1; attempt <= maxRetries; attempt++) {
           const delay = baseDelay * Math.pow(2, attempt - 1); // 2s, 4s, 8s, 16s, 32s
-          await log(`${formatAligned('⏳', 'Verifying fork:', `Attempt ${attempt}/${maxRetries} (waiting ${delay/1000}s)...`)}`);
+          await log(`${formatAligned('⏳', 'Verifying fork:', `Attempt ${attempt}/${maxRetries} (waiting ${delay / 1000}s)...`)}`);
           await new Promise(resolve => setTimeout(resolve, delay));
 
           const reCheckResult = await $`gh repo view ${forkFullName} --json name 2>/dev/null`;
@@ -201,7 +193,7 @@ export const handleExecutionError = async (error, shouldAttachLogs, owner, repo,
           log,
           sanitizeLogContent,
           verbose: argv.verbose || false,
-          errorMessage: cleanErrorMessage(error)
+          errorMessage: cleanErrorMessage(error),
         });
 
         if (logUploadSuccess) {
@@ -211,7 +203,7 @@ export const handleExecutionError = async (error, shouldAttachLogs, owner, repo,
         reportError(attachError, {
           context: 'attach_error_log',
           prNumber: global.createdPR?.number,
-          operation: 'attach_log_to_pr'
+          operation: 'attach_log_to_pr',
         });
         await log(`⚠️  Could not attach failure log: ${attachError.message}`, { level: 'warning' });
       }
@@ -232,7 +224,7 @@ export const handleExecutionError = async (error, shouldAttachLogs, owner, repo,
       reportError(closeError, {
         context: 'close_pr_on_error',
         prNumber: global.createdPR?.number,
-        operation: 'close_pull_request'
+        operation: 'close_pull_request',
       });
       await log(`⚠️  Could not close pull request: ${closeError.message}`, { level: 'warning' });
     }
@@ -253,7 +245,7 @@ export const cleanupTempDirectory = async (tempDir, argv, limitReached) => {
       reportError(cleanupError, {
         context: 'cleanup_temp_directory',
         tempDir,
-        operation: 'remove_temp_dir'
+        operation: 'remove_temp_dir',
       });
       await log(' ⚠️  (failed)');
     }
@@ -288,4 +280,4 @@ export const executeMainSolveLogic = async (tempDir, repoToClone) => {
 };
 
 // Use getResourceSnapshot from memory-check module
-export const getResourceSnapshot = memoryCheck.getResourceSnapshot;
+export const getResourceSnapshot = memoryCheck.getResourceSnapshot;

package/src/solve.feedback.lib.mjs

@@ -6,23 +6,8 @@
 // Import Sentry integration
 import { reportError } from './sentry.lib.mjs';
 
-export const detectAndCountFeedback = async (params) => {
-  const {
-    prNumber,
-    branchName,
-    owner,
-    repo,
-    issueNumber,
-    isContinueMode,
-    argv,
-    mergeStateStatus,
-    prState,
-    workStartTime,
-    log,
-    formatAligned,
-    cleanErrorMessage,
-    $
-  } = params;
+export const detectAndCountFeedback = async params => {
+  const { prNumber, branchName, owner, repo, issueNumber, isContinueMode, argv, mergeStateStatus, prState, workStartTime, log, formatAligned, cleanErrorMessage, $ } = params;
 
   let newPrComments = 0;
   let newIssueComments = 0;
@@ -40,7 +25,7 @@ export const detectAndCountFeedback = async (params) => {
   } catch (error) {
     reportError(error, {
       context: 'get_current_user',
-      operation: 'gh_api_user'
+      operation: 'gh_api_user',
     });
     await log('Warning: Could not get current GitHub user', { level: 'warning' });
   }
@@ -91,7 +76,7 @@ export const detectAndCountFeedback = async (params) => {
           reportError(error, {
             context: 'get_last_commit_time',
             prNumber,
-            operation: 'fetch_commit_timestamp'
+            operation: 'fetch_commit_timestamp',
           });
           await log(`Warning: Could not get last commit time: ${cleanErrorMessage(error)}`, { level: 'warning' });
         }
@@ -99,14 +84,8 @@ export const detectAndCountFeedback = async (params) => {
 
       // Only proceed if we have a last commit time
       if (lastCommitTime) {
-
         // Define log patterns to filter out comments containing logs from solve.mjs
-        const logPatterns = [
-          /📊.*Log file|solution\s+draft.*log/i,
-          /🔗.*Link:|💻.*Session:/i,
-          /Generated with.*solve\.mjs/i,
-          /Session ID:|Log file available:/i
-        ];
+        const logPatterns = [/📊.*Log file|solution\s+draft.*log/i, /🔗.*Link:|💻.*Session:/i, /Generated with.*solve\.mjs/i, /Session ID:|Log file available:/i];
 
         // Count new PR comments after last commit (both code review comments and conversation comments)
         let prReviewComments = [];
@@ -271,10 +250,12 @@ export const detectAndCountFeedback = async (params) => {
             reportError(error, {
               context: 'check_description_edits',
               prNumber,
-              operation: 'fetch_pr_timeline'
+              operation: 'fetch_pr_timeline',
             });
             if (argv.verbose) {
-              await log(`Warning: Could not check description edit times: ${cleanErrorMessage(error)}`, { level: 'warning' });
+              await log(`Warning: Could not check description edit times: ${cleanErrorMessage(error)}`, {
+                level: 'warning',
+              });
             }
           }
 
@@ -299,10 +280,12 @@ export const detectAndCountFeedback = async (params) => {
             reportError(error, {
               context: 'check_branch_commits',
               branchName,
-              operation: 'fetch_commit_messages'
+              operation: 'fetch_commit_messages',
             });
             if (argv.verbose) {
-              await log(`Warning: Could not check default branch commits: ${cleanErrorMessage(error)}`, { level: 'warning' });
+              await log(`Warning: Could not check default branch commits: ${cleanErrorMessage(error)}`, {
+                level: 'warning',
+              });
             }
           }
 
@@ -316,12 +299,12 @@ export const detectAndCountFeedback = async (params) => {
           // 5. Check merge status (non-clean indicates issues with merging)
           if (mergeStateStatus && mergeStateStatus !== 'CLEAN') {
             const statusDescriptions = {
-              'DIRTY': 'Merge status is DIRTY (conflicts detected)',
-              'UNSTABLE': 'Merge status is UNSTABLE (non-passing commit status)',
-              'BLOCKED': 'Merge status is BLOCKED',
-              'BEHIND': 'Merge status is BEHIND (head ref is out of date)',
-              'HAS_HOOKS': 'Merge status is HAS_HOOKS (has pre-receive hooks)',
-              'UNKNOWN': 'Merge status is UNKNOWN'
+              DIRTY: 'Merge status is DIRTY (conflicts detected)',
+              UNSTABLE: 'Merge status is UNSTABLE (non-passing commit status)',
+              BLOCKED: 'Merge status is BLOCKED',
+              BEHIND: 'Merge status is BEHIND (head ref is out of date)',
+              HAS_HOOKS: 'Merge status is HAS_HOOKS (has pre-receive hooks)',
+              UNKNOWN: 'Merge status is UNKNOWN',
             };
             const description = statusDescriptions[mergeStateStatus] || `Merge status is ${mergeStateStatus}`;
             feedbackLines.push(description);
@@ -334,9 +317,7 @@ export const detectAndCountFeedback = async (params) => {
             const checksResult = await $`gh api repos/${owner}/${repo}/commits/$(gh api repos/${owner}/${repo}/pulls/${prNumber} --jq '.head.sha')/check-runs`;
             if (checksResult.code === 0) {
               const checksData = JSON.parse(checksResult.stdout.toString());
-              const failedChecks = checksData.check_runs?.filter(check =>
-                check.conclusion === 'failure' && new Date(check.completed_at) > lastCommitTime
-              ) || [];
+              const failedChecks = checksData.check_runs?.filter(check => check.conclusion === 'failure' && new Date(check.completed_at) > lastCommitTime) || [];
 
               if (failedChecks.length > 0) {
                 feedbackLines.push(`Failed pull request checks: ${failedChecks.length}`);
@@ -348,7 +329,7 @@ export const detectAndCountFeedback = async (params) => {
             reportError(error, {
               context: 'check_pr_status_checks',
               prNumber,
-              operation: 'fetch_status_checks'
+              operation: 'fetch_status_checks',
             });
             if (argv.verbose) {
               await log(`Warning: Could not check PR status checks: ${cleanErrorMessage(error)}`, { level: 'warning' });
@@ -360,9 +341,7 @@ export const detectAndCountFeedback = async (params) => {
             const reviewsResult = await $`gh api repos/${owner}/${repo}/pulls/${prNumber}/reviews`;
             if (reviewsResult.code === 0) {
               const reviews = JSON.parse(reviewsResult.stdout.toString());
-              const changesRequestedReviews = reviews.filter(review =>
-                review.state === 'CHANGES_REQUESTED' && new Date(review.submitted_at) > lastCommitTime
-              );
+              const changesRequestedReviews = reviews.filter(review => review.state === 'CHANGES_REQUESTED' && new Date(review.submitted_at) > lastCommitTime);
 
               if (changesRequestedReviews.length > 0) {
                 feedbackLines.push(`Changes requested in reviews: ${changesRequestedReviews.length}`);
@@ -374,7 +353,7 @@ export const detectAndCountFeedback = async (params) => {
             reportError(error, {
               context: 'check_pr_reviews',
               prNumber,
-              operation: 'fetch_pr_reviews'
+              operation: 'fetch_pr_reviews',
             });
             if (argv.verbose) {
               await log(`Warning: Could not check PR reviews: ${cleanErrorMessage(error)}`, { level: 'warning' });
@@ -419,7 +398,7 @@ export const detectAndCountFeedback = async (params) => {
       reportError(error, {
         context: 'count_new_comments',
         prNumber,
-        operation: 'detect_and_count_feedback'
+        operation: 'detect_and_count_feedback',
       });
       await log(`Warning: Could not count new comments: ${cleanErrorMessage(error)}`, { level: 'warning' });
     }
@@ -433,4 +412,4 @@ export const detectAndCountFeedback = async (params) => {
   }
 
   return { newPrComments, newIssueComments, commentInfo, feedbackLines };
-};
+};