baseguard 1.0.1 → 1.0.3

package/src/commands/fix.ts

@@ -3,7 +3,7 @@ import { UIComponents } from '../ui/index.js';
 import { BaseGuard } from '../core/index.js';
 import { ConfigurationManager } from '../core/configuration.js';
 import { ErrorHandler } from '../core/error-handler.js';
-import { JulesImplementer } from '../ai/jules-implementer.js';
+import { UnifiedCodeFixer } from '../ai/unified-code-fixer.js';
 import { GeminiAnalyzer } from '../ai/gemini-analyzer.js';
 import { glob } from 'glob';
 
@@ -34,30 +34,34 @@ export async function fix(options: {
     
     // Initialize services
     const baseGuard = new BaseGuard(config);
-    const julesImplementer = new JulesImplementer(config.apiKeys.jules);
+    const unifiedCodeFixer = new UnifiedCodeFixer(config);
     const geminiAnalyzer = new GeminiAnalyzer(config.apiKeys.gemini);
     
-    // Check GitHub integration
-    const isGitHubSetup = await julesImplementer.isGitHubIntegrationSetup();
-    if (!isGitHubSetup) {
-      console.log(chalk.yellow('⚠️ Jules GitHub integration not set up'));
-      
-      const { default: inquirer } = await import('inquirer');
-      const { setupNow } = await inquirer.prompt([
-        {
-          type: 'confirm',
-          name: 'setupNow',
-          message: 'Would you like to set up Jules GitHub integration now?',
-          default: true
-        }
-      ]);
-      
-      if (setupNow) {
-        await julesImplementer.setupGitHubIntegration();
-      } else {
-        console.log(chalk.yellow('GitHub integration required for Jules fixing. Exiting.'));
-        process.exit(0);
-      }
+    // Show agent status and recommendations
+    console.log(chalk.cyan('🤖 Coding Agent Status:'));
+    const agentStatus = await unifiedCodeFixer.getAgentStatus();
+    
+    const primaryAvailable = agentStatus.primary === 'jules' ? agentStatus.jules.available : agentStatus.gemini.available;
+    const fallbackAvailable = agentStatus.fallback === 'jules' ? agentStatus.jules.available : agentStatus.gemini.available;
+    
+    console.log(`   Primary: ${agentStatus.primary} ${primaryAvailable ? '✅' : '❌'}`);
+    console.log(`   Fallback: ${agentStatus.fallback} ${fallbackAvailable ? '✅' : '❌'}`);
+    
+    if (!primaryAvailable && !fallbackAvailable) {
+      console.log(chalk.red('\n❌ No coding agents are available'));
+      console.log(chalk.cyan('💡 Configure API keys to enable code fixing:'));
+      console.log(chalk.cyan('   • Run "base config set-keys" to configure API keys'));
+      console.log(chalk.cyan('   • Get Gemini API key: https://aistudio.google.com'));
+      console.log(chalk.cyan('   • Get Jules API key: https://jules.google.com/settings#api'));
+      process.exit(1);
+    }
+    
+    // Show agent-specific information
+    if (agentStatus.primary === 'jules' && agentStatus.jules.repoDetected) {
+      console.log(chalk.cyan('🔗 GitHub repository detected - Jules available for autonomous fixing'));
+    } else if (agentStatus.primary === 'gemini' || !agentStatus.jules.repoDetected) {
+      console.log(chalk.cyan('💎 Using Gemini 2.5 Pro for local file fixing'));
+      console.log(chalk.dim('   This works with any local files, no GitHub required'));
     }
     
     // Step 1: Check for violations
@@ -102,88 +106,88 @@ export async function fix(options: {
       return;
     }
     
-    // Step 3: Generate and apply fixes with Jules
-    console.log(chalk.cyan('\n🤖 Generating fixes with Jules AI...'));
-    const results = await julesImplementer.generateAndApplyFixes(violations, analyses);
+    // Step 3: Generate fixes with unified code fixer
+    console.log(chalk.cyan(`\n🤖 Generating fixes with ${agentStatus.primary}...`));
     
-    // Show results
-    console.log(chalk.cyan('\n📊 Fix Results:\n'));
+    const fixes = [];
+    let successCount = 0;
+    let failedCount = 0;
     
-    if (results.applied.length > 0) {
-      console.log(chalk.green(`✅ Applied ${results.applied.length} fixes:`));
-      results.applied.forEach(fix => {
-        console.log(chalk.green(`   • ${fix.filePath} - ${fix.violation.feature}`));
-      });
-      console.log();
+    for (let i = 0; i < violations.length; i++) {
+      const violation = violations[i];
+      const analysis = analyses[i];
+      
+      if (!violation || !analysis) continue;
+      
+      try {
+        console.log(chalk.cyan(`\n🔧 Fixing ${violation.feature} in ${violation.file}...`));
+        
+        const fix = await unifiedCodeFixer.generateFix(violation, analysis);
+        fixes.push(fix);
+        successCount++;
+        
+        console.log(chalk.green(`✅ Fix generated (confidence: ${Math.round(fix.confidence * 100)}%)`));
+        
+        // Show preview if not in auto mode
+        if (!options.auto) {
+          console.log(chalk.dim('\nPreview:'));
+          console.log(chalk.dim(fix.preview.substring(0, 200) + (fix.preview.length > 200 ? '...' : '')));
+        }
+        
+      } catch (error) {
+        failedCount++;
+        console.log(chalk.red(`❌ Failed to generate fix: ${error instanceof Error ? error.message : 'Unknown error'}`));
+      }
     }
     
-    if (results.skipped.length > 0) {
-      console.log(chalk.yellow(`⏭️ Skipped ${results.skipped.length} fixes:`));
-      results.skipped.forEach(fix => {
-        console.log(chalk.yellow(`   • ${fix.filePath} - ${fix.violation.feature}`));
-      });
-      console.log();
+    if (fixes.length === 0) {
+      console.log(chalk.yellow('\n⚠️ No fixes were generated'));
+      return;
     }
     
-    if (results.failed.length > 0) {
-      console.log(chalk.red(`❌ Failed ${results.failed.length} fixes:`));
-      results.failed.forEach(({ fix, error }) => {
-        console.log(chalk.red(`   • ${fix.filePath} - ${fix.violation.feature}: ${error}`));
-      });
-      console.log();
+    // Show results summary
+    console.log(chalk.cyan('\n📊 Fix Generation Results:\n'));
+    console.log(chalk.green(`✅ Generated ${successCount} fixes`));
+    if (failedCount > 0) {
+      console.log(chalk.red(`❌ Failed to generate ${failedCount} fixes`));
     }
     
-    // Show rollback option if fixes were applied
-    if (results.applied.length > 0) {
-      const { default: inquirer } = await import('inquirer');
-      const { showRollback } = await inquirer.prompt([
-        {
-          type: 'confirm',
-          name: 'showRollback',
-          message: 'Would you like to see rollback options?',
-          default: false
-        }
-      ]);
-      
-      if (showRollback) {
-        const { rollbackAction } = await inquirer.prompt([
-          {
-            type: 'list',
-            name: 'rollbackAction',
-            message: 'Rollback options:',
-            choices: [
-              { name: 'Keep all fixes', value: 'keep' },
-              { name: 'Rollback all fixes', value: 'rollback_all' },
-              { name: 'Rollback specific fixes', value: 'rollback_specific' }
-            ]
-          }
-        ]);
+    // Show fixes with previews
+    if (!options.auto && fixes.length > 0) {
+      console.log(chalk.cyan('\n🔍 Generated Fixes:\n'));
+      fixes.forEach((fix, index) => {
+        console.log(chalk.white(`${index + 1}. ${fix.violation.feature} in ${fix.filePath}`));
+        console.log(chalk.dim(`   Confidence: ${Math.round(fix.confidence * 100)}%`));
+        console.log(chalk.dim(`   Strategy: ${fix.analysis.fixStrategy}`));
         
-        if (rollbackAction === 'rollback_all') {
-          await julesImplementer.rollbackAllFixes();
-          UIComponents.showSuccessBox('All fixes have been rolled back');
-        } else if (rollbackAction === 'rollback_specific') {
-          const appliedFiles = julesImplementer.getAppliedFixes();
-          const { filesToRollback } = await inquirer.prompt([
-            {
-              type: 'checkbox',
-              name: 'filesToRollback',
-              message: 'Select fixes to rollback:',
-              choices: appliedFiles.map(file => ({ name: file, value: file }))
-            }
-          ]);
-          
-          for (const file of filesToRollback) {
-            await julesImplementer.rollbackFix(file);
-          }
-          
-          if (filesToRollback.length > 0) {
-            UIComponents.showSuccessBox(`Rolled back ${filesToRollback.length} fixes`);
-          }
+        // Show a snippet of the explanation
+        const explanation = fix.explanation?.split('\n')[0] || 'No explanation available';
+        if (explanation.length > 80) {
+          console.log(chalk.dim(`   ${explanation.substring(0, 80)}...`));
+        } else {
+          console.log(chalk.dim(`   ${explanation}`));
         }
-      }
+        console.log();
+      });
     }
     
+    // Show summary
+    console.log(chalk.cyan(`\n📈 Summary: ${successCount}/${violations.length} fixes generated successfully`));
+    
+    if (failedCount > 0) {
+      console.log(chalk.yellow(`\n⚠️ ${failedCount} fixes failed to generate. This may be due to:`));
+      console.log(chalk.yellow('   • Complex compatibility issues requiring manual review'));
+      console.log(chalk.yellow('   • API rate limits or temporary service issues'));
+      console.log(chalk.yellow('   • Files that couldn\'t be read or analyzed'));
+    }
+    
+    // Show next steps
+    console.log(chalk.cyan('\n💡 Next Steps:'));
+    console.log(chalk.cyan('   • Review the generated fixes above'));
+    console.log(chalk.cyan('   • Apply fixes manually to your code'));
+    console.log(chalk.cyan('   • Test your application after applying fixes'));
+    console.log(chalk.cyan('   • Run "base check" to verify fixes resolve violations'));
+    
     UIComponents.showSuccessBox('Fix process completed!');
     
   } catch (error) {
@@ -203,7 +207,7 @@ export async function fix(options: {
       UIComponents.showList([
         'Run "base init" to set up BaseGuard configuration',
         'Configure API keys with "base config set-keys"',
-        'Check GitHub integration for Jules fixing'
+        'Ensure you are in a GitHub repository for Jules fixing'
       ]);
     } else if (apiError.type === 'network') {
       UIComponents.showList([

package/src/commands/init.ts

@@ -69,11 +69,10 @@ export async function init(options: {
       }
     }
     
-    // Set up API keys if not skipped
+    // Set up API keys and coding agent if not skipped
     if (!options.skipApiKeys) {
       spinner.stop();
       const apiKeys = await Prompts.setupApiKeys();
-      spinner.start();
       
       if (apiKeys.julesApiKey) {
         config.apiKeys.jules = apiKeys.julesApiKey;
@@ -81,6 +80,21 @@ export async function init(options: {
       if (apiKeys.geminiApiKey) {
         config.apiKeys.gemini = apiKeys.geminiApiKey;
       }
+      
+      // Configure coding agent based on available keys
+      if (apiKeys.julesApiKey && apiKeys.geminiApiKey) {
+        const codingAgentChoice = await Prompts.chooseCodingAgent();
+        config.codingAgent.primary = codingAgentChoice.primary;
+        config.codingAgent.fallback = codingAgentChoice.fallback;
+      } else if (apiKeys.julesApiKey) {
+        config.codingAgent.primary = 'jules';
+        config.codingAgent.fallback = 'jules';
+      } else if (apiKeys.geminiApiKey) {
+        config.codingAgent.primary = 'gemini';
+        config.codingAgent.fallback = 'gemini';
+      }
+      
+      spinner.start();
     }
     
     await ConfigurationManager.save(config);

package/src/core/cache-manager.ts

@@ -28,8 +28,10 @@ export class LRUCache<K, V> {
       this.cache.delete(key);
     } else if (this.cache.size >= this.maxSize) {
       // Remove least recently used (first item)
-      const firstKey = this.cache.keys().next().value;
-      this.cache.delete(firstKey);
+      const firstKey = this.cache.keys().next().value as K;
+      if (firstKey !== undefined) {
+        this.cache.delete(firstKey);
+      }
     }
     this.cache.set(key, value);
   }

package/src/core/configuration.ts

@@ -77,6 +77,10 @@ export class ConfigurationManager {
         jules: null,
         gemini: null
       },
+      codingAgent: {
+        primary: 'gemini', // 'jules' or 'gemini'
+        fallback: 'gemini' // fallback when primary fails
+      },
       automation: {
         enabled: false,
         trigger: 'pre-commit',
@@ -119,6 +123,10 @@ export class ConfigurationManager {
         jules: config.apiKeys?.jules || null,
         gemini: config.apiKeys?.gemini || null
       },
+      codingAgent: {
+        primary: this.validateCodingAgent(config.codingAgent?.primary) || defaultConfig.codingAgent.primary,
+        fallback: this.validateCodingAgent(config.codingAgent?.fallback) || defaultConfig.codingAgent.fallback
+      },
       automation: {
         enabled: config.automation?.enabled ?? defaultConfig.automation.enabled,
         trigger: this.validateTrigger(config.automation?.trigger) || defaultConfig.automation.trigger,
@@ -194,6 +202,16 @@ export class ConfigurationManager {
     return null;
   }
 
+  /**
+   * Validate coding agent selection
+   */
+  private static validateCodingAgent(agent: any): 'jules' | 'gemini' | null {
+    if (agent === 'jules' || agent === 'gemini') {
+      return agent;
+    }
+    return null;
+  }
+
   /**
    * Parse browser target string (e.g., "chrome 100", "safari baseline")
    */
@@ -437,6 +455,18 @@ export class ConfigurationManager {
       }
     }
 
+    // Validate coding agent
+    if (!config.codingAgent || typeof config.codingAgent !== 'object') {
+      errors.push('Coding agent configuration must be an object');
+    } else {
+      if (config.codingAgent.primary !== 'jules' && config.codingAgent.primary !== 'gemini') {
+        errors.push('Primary coding agent must be "jules" or "gemini"');
+      }
+      if (config.codingAgent.fallback !== 'jules' && config.codingAgent.fallback !== 'gemini') {
+        errors.push('Fallback coding agent must be "jules" or "gemini"');
+      }
+    }
+
     // Validate automation
     if (!config.automation || typeof config.automation !== 'object') {
       errors.push('Automation configuration must be an object');
@@ -481,6 +511,13 @@ export class ConfigurationManager {
         migratedConfig.apiKeys.gemini = config.apiKeys.gemini || null;
       }
       
+      if (config.codingAgent) {
+        migratedConfig.codingAgent = {
+          primary: this.validateCodingAgent(config.codingAgent.primary) || 'gemini',
+          fallback: this.validateCodingAgent(config.codingAgent.fallback) || 'gemini'
+        };
+      }
+      
       if (config.automation) {
         migratedConfig.automation = {
           ...migratedConfig.automation,

package/src/core/debug-logger.ts

@@ -130,8 +130,8 @@ export class DebugLogger {
     }
 
     this.currentSession.endTime = new Date();
-    this.currentSession.summary.totalDuration = 
-      this.currentSession.endTime.getTime() - this.currentSession.startTime.getTime();
+    const duration = this.currentSession.endTime.getTime() - this.currentSession.startTime.getTime();
+    this.currentSession.summary.performance.totalDuration = duration;
 
     this.info('session', `Debug session ended: ${this.currentSession.id}`);
     

package/src/core/file-processor.ts

@@ -103,12 +103,15 @@ export class FileProcessor {
       );
 
       batchResults.forEach((result, index) => {
+        const task = batch[index];
+        if (!task) return;
+        
         if (result.status === 'fulfilled') {
           allFeatures.push(...result.value);
           // Cache the result
-          this.cacheManager.setCachedParseResult(batch[index].filePath, result.value);
+          this.cacheManager.setCachedParseResult(task.filePath, result.value);
         } else {
-          console.warn(`Failed to process ${batch[index].filePath}: ${result.reason}`);
+          console.warn(`Failed to process ${task.filePath}: ${result.reason}`);
         }
       });
 
@@ -222,7 +225,11 @@ export class FileProcessor {
   /**
    * Assign task to worker
    */
-  private async assignTaskToWorker(worker: Worker, task: WorkerTask): Promise<void> {
+  private async assignTaskToWorker(worker: Worker | undefined, task: WorkerTask): Promise<void> {
+    if (!worker) {
+      throw new Error('Worker is not available');
+    }
+    
     return new Promise((resolve) => {
       const timeout = setTimeout(() => {
         console.warn(`Worker task ${task.id} timed out`);

package/src/core/index.ts

@@ -10,4 +10,8 @@ export { FileProcessor } from './file-processor.js';
 export { DirectoryFilter } from './directory-filter.js';
 export { LazyLoader } from './lazy-loader.js';
 export { MemoryManager } from './memory-manager.js';
-export { StartupOptimizer } from './startup-optimizer.js';
+export { StartupOptimizer } from './startup-optimizer.js';
+export { SystemErrorHandler } from './system-error-handler.js';
+export { GracefulDegradationManager } from './graceful-degradation-manager.js';
+export { ConfigurationRecovery } from './configuration-recovery.js';
+export { logger, DebugLogger } from './debug-logger.js';

package/src/core/memory-manager.ts

@@ -1,9 +1,9 @@
 import { createReadStream } from 'fs';
 import { createInterface } from 'readline';
 
-// Add global type declaration for gc
+// Use Node.js built-in gc type
 declare global {
-  var gc: (() => void) | undefined;
+  var gc: NodeJS.GCFunction | undefined;
 }
 
 /**
@@ -291,4 +291,5 @@ class ViolationTracker {
     this.violations.clear();
     this.fileIndex.clear();
     this.nextFileId = 0;
-  }
+  }
+}

package/src/core/startup-optimizer.ts

@@ -4,6 +4,8 @@
 export class StartupOptimizer {
   private static startTime = Date.now();
   private static initialized = false;
+  private static gcInterval: NodeJS.Timeout | null = null;
+  private static memoryInterval: NodeJS.Timeout | null = null;
 
   /**
    * Initialize BaseGuard with optimized startup
@@ -25,7 +27,7 @@ export class StartupOptimizer {
       this.initialized = true;
 
       const initTime = Date.now() - startTime;
-      if (initTime > 200) {
+      if (initTime > 1000) {
         console.warn(`Slow startup detected: ${initTime}ms`);
       }
     } catch (error) {
@@ -57,7 +59,7 @@ export class StartupOptimizer {
     // Enable garbage collection hints if available
     if (global.gc) {
       // Set up periodic GC for long-running processes
-      setInterval(() => {
+      this.gcInterval = setInterval(() => {
         const { MemoryManager } = require('./memory-manager.js');
         const memCheck = MemoryManager.checkMemoryUsage();
         
@@ -124,6 +126,16 @@ export class StartupOptimizer {
     const { LazyLoader } = require('./lazy-loader.js');
     const { MemoryManager } = require('./memory-manager.js');
 
+    // Clear intervals
+    if (this.gcInterval) {
+      clearInterval(this.gcInterval);
+      this.gcInterval = null;
+    }
+    if (this.memoryInterval) {
+      clearInterval(this.memoryInterval);
+      this.memoryInterval = null;
+    }
+
     // Clear caches
     LazyLoader.clearCache();
     
@@ -145,7 +157,7 @@ export class StartupOptimizer {
     const metrics = this.getStartupMetrics();
     
     // Check startup time
-    if (metrics.totalStartupTime > 500) {
+    if (metrics.totalStartupTime > 2000) {
       issues.push(`Slow startup: ${metrics.totalStartupTime}ms`);
       recommendations.push('Consider reducing dependencies or using lazy loading');
     }
@@ -170,4 +182,74 @@ export class StartupOptimizer {
       recommendations
     };
   }
+
+  /**
+   * Optimize memory usage by reducing object overhead
+   */
+  static optimizeMemoryUsage(): void {
+    // Enable V8 memory optimizations if available
+    if (process.env.NODE_ENV !== 'development') {
+      // Set V8 flags for better memory management
+      process.env.NODE_OPTIONS = (process.env.NODE_OPTIONS || '') + ' --max-old-space-size=512 --optimize-for-size';
+    }
+
+    // Setup memory monitoring
+    if (global.gc) {
+      this.memoryInterval = setInterval(() => {
+        const usage = process.memoryUsage();
+        const heapUsedMB = usage.heapUsed / 1024 / 1024;
+        
+        // Force GC if memory usage is high
+        if (heapUsedMB > 100 && global.gc) {
+          global.gc();
+        }
+      }, 30000);
+    }
+  }
+
+  /**
+   * Reduce startup time by deferring non-critical operations
+   */
+  static async deferNonCriticalOperations(): Promise<void> {
+    // Defer heavy operations until after startup
+    setTimeout(async () => {
+      try {
+        // Preload remaining dependencies
+        const { LazyLoader } = await import('./lazy-loader.js');
+        LazyLoader.preloadCommon();
+        
+        // Initialize caches
+        const { CacheManager } = await import('./cache-manager.js');
+        // Cache initialization would happen here
+        
+        // Cleanup old logs
+        const { logger } = await import('./debug-logger.js');
+        logger.cleanupOldLogs().catch(() => {});
+      } catch (error) {
+        // Ignore errors in deferred operations
+      }
+    }, 100); // Defer by 100ms
+  }
+
+  /**
+   * Optimize startup by preloading critical dependencies
+   */
+  static async optimizeStartup(): Promise<void> {
+    // Start loading critical dependencies in background
+    const { LazyLoader } = await import('./lazy-loader.js');
+    
+    const criticalLoads = [
+      LazyLoader.getWebFeatures().catch(() => {}),
+      LazyLoader.getBabelParser().catch(() => {})
+    ];
+
+    // Don't wait for all to complete, just start the process
+    Promise.all(criticalLoads);
+    
+    // Setup memory optimizations
+    this.optimizeMemoryUsage();
+    
+    // Defer non-critical operations
+    this.deferNonCriticalOperations();
+  }
 }

package/src/core/system-error-handler.ts

@@ -499,22 +499,28 @@ export class SystemErrorHandler {
 
   static async handleProcessSignals(): Promise<void> {
     const gracefulShutdown = async (signal: string) => {
-      console.log(chalk.yellow(`\n⚠️ Received ${signal}, shutting down gracefully...`));
+      // Only log for non-interactive signals (SIGTERM, not SIGINT from Ctrl+C)
+      if (signal !== 'SIGINT') {
+        console.log(chalk.yellow(`\n⚠️ Received ${signal}, shutting down gracefully...`));
+      }
       
-      // Save error log before exit
+      // Cleanup intervals and timers
       try {
-        await SystemErrorHandler.saveErrorLog();
+        const { StartupOptimizer } = await import('./startup-optimizer.js');
+        StartupOptimizer.cleanup();
       } catch (error) {
-        // Ignore errors during shutdown
+        // Ignore cleanup errors
       }
       
-      // Show error summary if there were issues
-      const summary = SystemErrorHandler.getErrorSummary();
-      if (summary.total > 0) {
-        console.log(chalk.cyan(`\n📊 Session summary: ${summary.total} errors encountered`));
-        if (summary.critical > 0) {
-          console.log(chalk.red(`   ${summary.critical} critical errors require attention`));
-        }
+      // Save error log before exit (but don't wait too long)
+      try {
+        const savePromise = SystemErrorHandler.saveErrorLog();
+        await Promise.race([
+          savePromise,
+          new Promise(resolve => setTimeout(resolve, 100)) // Max 100ms wait
+        ]);
+      } catch (error) {
+        // Ignore errors during shutdown
       }
       
       process.exit(0);