@link-assistant/hive-mind 0.46.1 → 0.47.0

package/src/reviewers-hive.mjs

@@ -16,18 +16,18 @@ let logFile = null;
 // Helper function to log to both console and file
 const log = async (message, options = {}) => {
   const { level = 'info', verbose = false } = options;
-  
+
   // Skip verbose logs unless --verbose is enabled
   if (verbose && !global.verboseMode) {
     return;
   }
-  
+
   // Write to file if log file is set
   if (logFile) {
     const logMessage = `[${new Date().toISOString()}] [${level.toUpperCase()}] ${message}`;
     await fs.appendFile(logFile, logMessage + '\n').catch(() => {});
   }
-  
+
   // Write to console based on level
   switch (level) {
     case 'error':
@@ -49,92 +49,91 @@ const argv = yargs(process.argv.slice(2))
   .usage('Usage: $0 <github-url> [options]')
   .positional('github-url', {
     type: 'string',
-    description: 'GitHub organization, repository, or user URL to monitor for pull requests'
+    description: 'GitHub organization, repository, or user URL to monitor for pull requests',
   })
   .option('review-label', {
     type: 'string',
     description: 'GitHub label to identify PRs needing review',
     default: 'needs-review',
-    alias: 'l'
+    alias: 'l',
   })
   .option('all-prs', {
     type: 'boolean',
     description: 'Review all open pull requests regardless of labels',
     default: false,
-    alias: 'a'
+    alias: 'a',
   })
   .option('skip-draft', {
     type: 'boolean',
     description: 'Skip draft pull requests',
     default: true,
-    alias: 'd'
+    alias: 'd',
   })
   .option('skip-approved', {
     type: 'boolean',
     description: 'Skip pull requests that already have approvals',
-    default: true
+    default: true,
   })
   .option('concurrency', {
     type: 'number',
     description: 'Number of concurrent review.mjs instances',
     default: 2,
-    alias: 'c'
+    alias: 'c',
   })
   .option('reviews-per-pr', {
     type: 'number',
     description: 'Number of reviews to generate per PR (for diverse perspectives)',
     default: 1,
-    alias: 'r'
+    alias: 'r',
   })
   .option('model', {
     type: 'string',
     description: 'Model to use for review.mjs (opus or sonnet)',
     alias: 'm',
     default: 'opus',
-    choices: ['opus', 'sonnet']
+    choices: ['opus', 'sonnet'],
   })
   .option('focus', {
     type: 'string',
     description: 'Focus areas for reviews (security, performance, logic, style, tests, all)',
     default: 'all',
-    alias: 'f'
+    alias: 'f',
   })
   .option('auto-approve', {
     type: 'boolean',
     description: 'Auto-approve PRs that pass review criteria',
-    default: false
+    default: false,
   })
   .option('interval', {
     type: 'number',
     description: 'Polling interval in seconds',
     default: 300, // 5 minutes
-    alias: 'i'
+    alias: 'i',
   })
   .option('max-prs', {
     type: 'number',
     description: 'Maximum number of PRs to process (0 = unlimited)',
-    default: 0
+    default: 0,
   })
   .option('dry-run', {
     type: 'boolean',
     description: 'List PRs that would be reviewed without actually reviewing them',
-    default: false
+    default: false,
   })
   .option('verbose', {
     type: 'boolean',
     description: 'Enable verbose logging',
     alias: 'v',
-    default: false
+    default: false,
   })
   .option('once', {
     type: 'boolean',
     description: 'Run once and exit instead of continuous monitoring',
-    default: false
+    default: false,
   })
   .demandCommand(1, 'GitHub URL is required')
   .help('h')
-  .alias('h', 'help')
-  .argv;
+  .alias('h', 'help').argv;
 
 const githubUrl = argv['github-url'] || argv._[0];
 
@@ -227,9 +226,7 @@ class PRQueue {
 
   // Add PR to queue if not already processed or in queue
   enqueue(prUrl) {
-    if (this.completed.has(prUrl) || 
-        this.processing.has(prUrl) || 
-        this.queue.includes(prUrl)) {
+    if (this.completed.has(prUrl) || this.processing.has(prUrl) || this.queue.includes(prUrl)) {
       return false;
     }
     this.queue.push(prUrl);
@@ -264,7 +261,7 @@ class PRQueue {
       queued: this.queue.length,
       processing: this.processing.size,
       completed: this.completed.size,
-      failed: this.failed.size
+      failed: this.failed.size,
     };
   }
 
@@ -280,10 +277,10 @@ const prQueue = new PRQueue();
 // Worker function to review PRs from queue
 async function reviewer(reviewerId) {
   await log(`🔍 Reviewer ${reviewerId} started`, { verbose: true });
-  
+
   while (prQueue.isRunning) {
     const prUrl = prQueue.dequeue();
-    
+
     if (!prUrl) {
       // No work available, wait a bit
       await new Promise(resolve => setTimeout(resolve, 5000));
@@ -291,13 +288,13 @@ async function reviewer(reviewerId) {
     }
 
     await log(`\n👀 Reviewer ${reviewerId} reviewing: ${prUrl}`);
-    
+
     // Review the PR multiple times if needed (for diverse perspectives)
     for (let reviewNum = 1; reviewNum <= argv.reviewsPerPr; reviewNum++) {
       if (argv.reviewsPerPr > 1) {
         await log(`   📝 Creating review ${reviewNum}/${argv.reviewsPerPr} for PR`);
       }
-      
+
       try {
         if (argv.dryRun) {
           await log(`   🧪 [DRY RUN] Would execute: ./review.mjs "${prUrl}" --model ${argv.model} --focus ${argv.focus}${argv.autoApprove ? ' --approve' : ''}`);
@@ -305,14 +302,14 @@ async function reviewer(reviewerId) {
         } else {
           // Execute review.mjs using command-stream
           await log(`   🚀 Executing review.mjs for ${prUrl}...`);
-          
+
           const startTime = Date.now();
           let reviewCommand = $`./review.mjs "${prUrl}" --model ${argv.model} --focus ${argv.focus}`;
-          
+
           if (argv.autoApprove) {
             reviewCommand = $`./review.mjs "${prUrl}" --model ${argv.model} --focus ${argv.focus} --approve`;
           }
-          
+
           // Stream output and capture result
           let exitCode = 0;
           for await (const chunk of reviewCommand.stream()) {
@@ -330,16 +327,16 @@ async function reviewer(reviewerId) {
               exitCode = chunk.code;
             }
           }
-          
+
           const duration = Math.round((Date.now() - startTime) / 1000);
-          
+
           if (exitCode === 0) {
             await log(`   ✅ Reviewer ${reviewerId} completed ${prUrl} (${duration}s)`);
           } else {
             throw new Error(`review.mjs exited with code ${exitCode}`);
           }
         }
-        
+
         // Small delay between multiple reviews for same PR
         if (reviewNum < argv.reviewsPerPr) {
           await new Promise(resolve => setTimeout(resolve, 10000));
@@ -350,14 +347,14 @@ async function reviewer(reviewerId) {
         break; // Stop trying more reviews for this PR
       }
     }
-    
+
     prQueue.markCompleted(prUrl);
-    
+
     // Show queue stats
     const stats = prQueue.getStats();
     await log(`   📊 Queue: ${stats.queued} waiting, ${stats.processing} reviewing, ${stats.completed} completed, ${stats.failed} failed`);
   }
-  
+
   await log(`🔍 Reviewer ${reviewerId} stopped`, { verbose: true });
 }
 
@@ -379,12 +376,12 @@ async function hasApprovals(prUrl) {
 
     const { stdout } = await execAsync(cmd, { encoding: 'utf8', env: process.env });
     const approvalCount = parseInt(stdout.trim()) || 0;
-    
+
     if (approvalCount > 0) {
       await log(`      ↳ Skipping (has ${approvalCount} approval${approvalCount > 1 ? 's' : ''})`, { verbose: true });
       return true;
     }
-    
+
     return false;
   } catch (error) {
     // If we can't check, assume no approvals
@@ -400,10 +397,10 @@ async function fetchPullRequests() {
   } else {
     await log(`\n🔍 Fetching pull requests with label "${argv.reviewLabel}"...`);
   }
-  
+
   try {
     let prs = [];
-    
+
     if (argv.allPrs) {
       // Fetch all open PRs without label filter
       let searchCmd;
@@ -415,7 +412,7 @@ async function fetchPullRequests() {
         // User scope
         searchCmd = `gh search prs user:${owner} is:open --limit 100 --json url,title,number,repository,isDraft`;
       }
-      
+
       await log(`   🔎 Command: ${searchCmd}`, { verbose: true });
 
       // Use async exec to avoid escaping issues
@@ -424,7 +421,6 @@ async function fetchPullRequests() {
       const execAsync = promisify(exec);
       const { stdout } = await execAsync(searchCmd, { encoding: 'utf8', env: process.env });
       prs = JSON.parse(stdout || '[]');
-      
     } else {
       // Use label filter
       const { exec } = await import('child_process');
@@ -451,11 +447,11 @@ async function fetchPullRequests() {
         } else {
           baseQuery = `user:${owner} is:pr is:open`;
         }
-        
+
         // Handle label with potential spaces
         let searchQuery;
         let searchCmd;
-        
+
         if (argv.reviewLabel.includes(' ')) {
           searchQuery = `${baseQuery} label:"${argv.reviewLabel}"`;
           searchCmd = `gh search prs '${searchQuery}' --limit 100 --json url,title,number,repository,isDraft`;
@@ -463,7 +459,7 @@ async function fetchPullRequests() {
           searchQuery = `${baseQuery} label:${argv.reviewLabel}`;
           searchCmd = `gh search prs '${searchQuery}' --limit 100 --json url,title,number,repository,isDraft`;
         }
-        
+
         await log(`   🔎 Search query: ${searchQuery}`, { verbose: true });
         await log(`   🔎 Command: ${searchCmd}`, { verbose: true });
 
@@ -476,7 +472,7 @@ async function fetchPullRequests() {
         }
       }
     }
-    
+
     if (prs.length === 0) {
       if (argv.allPrs) {
         await log('   ℹ️  No open pull requests found');
@@ -485,13 +481,13 @@ async function fetchPullRequests() {
       }
       return [];
     }
-    
+
     if (argv.allPrs) {
       await log(`   📋 Found ${prs.length} open pull request(s)`);
     } else {
       await log(`   📋 Found ${prs.length} pull request(s) with label "${argv.reviewLabel}"`);
     }
-    
+
     // Filter out draft PRs if option is enabled
     if (argv.skipDraft) {
       const nonDraftPrs = prs.filter(pr => !pr.isDraft);
@@ -501,19 +497,19 @@ async function fetchPullRequests() {
       }
       prs = nonDraftPrs;
     }
-    
+
     // Apply max PRs limit if set
     let prsToProcess = prs;
     if (argv.maxPrs > 0 && prs.length > argv.maxPrs) {
       prsToProcess = prs.slice(0, argv.maxPrs);
       await log(`   🔢 Limiting to first ${argv.maxPrs} PRs`);
     }
-    
+
     // Filter out PRs with approvals if option is enabled
     if (argv.skipApproved) {
       await log('   🔍 Checking for existing approvals...');
       const filteredPrs = [];
-      
+
       for (const pr of prsToProcess) {
         const hasApproval = await hasApprovals(pr.url);
         if (hasApproval) {
@@ -522,14 +518,14 @@ async function fetchPullRequests() {
           filteredPrs.push(pr);
         }
       }
-      
+
       const skippedCount = prsToProcess.length - filteredPrs.length;
       if (skippedCount > 0) {
         await log(`   ⏭️  Skipped ${skippedCount} PR(s) with existing approvals`);
       }
       prsToProcess = filteredPrs;
     }
-    
+
     // In dry-run mode, show the PRs that would be reviewed
     if (argv.dryRun && prsToProcess.length > 0) {
       await log('\n   📝 PRs that would be reviewed:');
@@ -537,9 +533,8 @@ async function fetchPullRequests() {
         await log(`      - ${pr.title || 'Untitled'} (${pr.url})`);
       }
     }
-    
+
     return prsToProcess.map(pr => pr.url);
-    
   } catch (error) {
     await log(`   ❌ Error fetching pull requests: ${error.message}`, { level: 'error' });
     return [];
@@ -549,22 +544,22 @@ async function fetchPullRequests() {
 // Main monitoring loop
 async function monitor() {
   await log('\n🚀 Starting Reviewers Hive Mind monitoring system...');
-  
+
   // Start reviewers
   await log(`\n👀 Starting ${argv.concurrency} reviewers...`);
   for (let i = 1; i <= argv.concurrency; i++) {
     prQueue.workers.push(reviewer(i));
   }
-  
+
   // Main monitoring loop
   let iteration = 0;
   while (true) {
     iteration++;
     await log(`\n🔄 Monitoring iteration ${iteration} at ${new Date().toISOString()}`);
-    
+
     // Fetch PRs
     const prUrls = await fetchPullRequests();
-    
+
     // Add new PRs to queue
     let newPrs = 0;
     for (const url of prUrls) {
@@ -573,13 +568,13 @@ async function monitor() {
         await log(`   ➕ Added to review queue: ${url}`);
       }
     }
-    
+
     if (newPrs > 0) {
       await log(`   📥 Added ${newPrs} new PR(s) to review queue`);
     } else {
       await log('   ℹ️  No new PRs to add (all already reviewed or in queue)');
     }
-    
+
     // Show current stats
     const stats = prQueue.getStats();
     await log('\n📊 Current Status:');
@@ -587,11 +582,11 @@ async function monitor() {
     await log(`   ⚙️  Reviewing: ${stats.processing}`);
     await log(`   ✅ Completed: ${stats.completed}`);
     await log(`   ❌ Failed: ${stats.failed}`);
-    
+
     // If running once, wait for queue to empty then exit
     if (argv.once) {
       await log('\n🏁 Single run mode - waiting for review queue to empty...');
-      
+
       while (stats.queued > 0 || stats.processing > 0) {
         await new Promise(resolve => setTimeout(resolve, 5000));
         const currentStats = prQueue.getStats();
@@ -600,22 +595,22 @@ async function monitor() {
         }
         Object.assign(stats, currentStats);
       }
-      
+
       await log('\n✅ All PRs reviewed!');
       await log(`   Completed: ${stats.completed}`);
       await log(`   Failed: ${stats.failed}`);
       break;
     }
-    
+
     // Wait for next iteration
     await log(`\n⏰ Next check in ${argv.interval} seconds...`);
     await new Promise(resolve => setTimeout(resolve, argv.interval * 1000));
   }
-  
+
   // Stop reviewers
   prQueue.stop();
   await Promise.all(prQueue.workers);
-  
+
   await log('\n👋 Reviewers Hive Mind monitoring stopped');
 }
 
@@ -640,4 +635,4 @@ try {
 } catch (error) {
   await log(`\n❌ Fatal error: ${error.message}`, { level: 'error' });
   process.exit(1);
-}
+}

package/src/sentry.lib.mjs

@@ -117,12 +117,15 @@ export const withSpan = async (name, callback) => {
     return callback();
   }
 
-  return sentry.startSpan({
-    name,
-    op: 'function',
-  }, async () => {
-    return callback();
-  });
+  return sentry.startSpan(
+    {
+      name,
+      op: 'function',
+    },
+    async () => {
+      return callback();
+    }
+  );
 };
 
 /**
@@ -173,7 +176,7 @@ export const reportWarning = (warning, context = {}) => {
  * Add breadcrumb for better error context
  * @param {Object} breadcrumb - Breadcrumb data
  */
-export const addBreadcrumb = async (breadcrumb) => {
+export const addBreadcrumb = async breadcrumb => {
   if (!isSentryEnabled() || sentryDisabled) {
     return;
   }
@@ -188,7 +191,7 @@ export const addBreadcrumb = async (breadcrumb) => {
  * Set user context for Sentry
  * @param {Object} user - User data
  */
-export const setUserContext = async (user) => {
+export const setUserContext = async user => {
   if (!isSentryEnabled() || sentryDisabled) {
     return;
   }
@@ -219,7 +222,7 @@ export const setExtraContext = async (key, value) => {
  * Set tags for Sentry
  * @param {Object} tags - Tags to set
  */
-export const setTags = async (tags) => {
+export const setTags = async tags => {
   if (!isSentryEnabled() || sentryDisabled) {
     return;
   }
@@ -281,4 +284,4 @@ export const closeSentry = async (timeout = 2000) => {
 };
 
 // Export the Sentry check function
-export { isSentryEnabled };
+export { isSentryEnabled };