@git.zone/cli 1.16.7 → 1.16.8

package/ts/mod_format/classes.formatplanner.ts

@@ -10,7 +10,7 @@ export class FormatPlanner {
   private plannedChanges: Map<string, IPlannedChange[]> = new Map();
   private dependencyAnalyzer = new DependencyAnalyzer();
   private diffReporter = new DiffReporter();
-  
+
   async planFormat(modules: BaseFormatter[]): Promise<IFormatPlan> {
     const plan: IFormatPlan = {
       summary: {
@@ -18,20 +18,20 @@ export class FormatPlanner {
         filesAdded: 0,
         filesModified: 0,
         filesRemoved: 0,
-        estimatedTime: 0
+        estimatedTime: 0,
       },
       changes: [],
-      warnings: []
+      warnings: [],
     };
-    
+
     for (const module of modules) {
       try {
         const changes = await module.analyze();
         this.plannedChanges.set(module.name, changes);
-        
+
         for (const change of changes) {
           plan.changes.push(change);
-          
+
           // Update summary
           switch (change.type) {
             case 'create':
@@ -49,67 +49,51 @@ export class FormatPlanner {
         plan.warnings.push({
           level: 'error',
           message: `Failed to analyze module ${module.name}: ${error.message}`,
-          module: module.name
+          module: module.name,
         });
       }
     }
-    
-    plan.summary.totalFiles = plan.summary.filesAdded + plan.summary.filesModified + plan.summary.filesRemoved;
+
+    plan.summary.totalFiles =
+      plan.summary.filesAdded +
+      plan.summary.filesModified +
+      plan.summary.filesRemoved;
     plan.summary.estimatedTime = plan.summary.totalFiles * 100; // 100ms per file estimate
-    
+
     return plan;
   }
-  
-  async executePlan(plan: IFormatPlan, modules: BaseFormatter[], context: FormatContext, parallel: boolean = true): Promise<void> {
-    await context.beginOperation();
+
+  async executePlan(
+    plan: IFormatPlan,
+    modules: BaseFormatter[],
+    context: FormatContext,
+    parallel: boolean = false,
+  ): Promise<void> {
     const startTime = Date.now();
-    
+
     try {
-      if (parallel) {
-        // Get execution groups based on dependencies
-        const executionGroups = this.dependencyAnalyzer.getExecutionGroups(modules);
-        
-        logger.log('info', `Executing formatters in ${executionGroups.length} groups...`);
-        
-        for (let i = 0; i < executionGroups.length; i++) {
-          const group = executionGroups[i];
-          logger.log('info', `Executing group ${i + 1}: ${group.map(m => m.name).join(', ')}`);
-          
-          // Execute modules in this group in parallel
-          const promises = group.map(async (module) => {
-            const changes = this.plannedChanges.get(module.name) || [];
-            if (changes.length > 0) {
-              logger.log('info', `Executing ${module.name} formatter...`);
-              await module.execute(changes);
-            }
-          });
-          
-          await Promise.all(promises);
-        }
-      } else {
-        // Sequential execution (original implementation)
-        for (const module of modules) {
-          const changes = this.plannedChanges.get(module.name) || [];
-          
-          if (changes.length > 0) {
-            logger.log('info', `Executing ${module.name} formatter...`);
-            await module.execute(changes);
-          }
+      // Always use sequential execution to avoid race conditions
+      for (const module of modules) {
+        const changes = this.plannedChanges.get(module.name) || [];
+
+        if (changes.length > 0) {
+          logger.log('info', `Executing ${module.name} formatter...`);
+          await module.execute(changes);
         }
       }
-      
+
       const endTime = Date.now();
       const duration = endTime - startTime;
       logger.log('info', `Format operations completed in ${duration}ms`);
-      
-      await context.commitOperation();
     } catch (error) {
-      await context.rollbackOperation();
       throw error;
     }
   }
-  
-  async displayPlan(plan: IFormatPlan, detailed: boolean = false): Promise<void> {
+
+  async displayPlan(
+    plan: IFormatPlan,
+    detailed: boolean = false,
+  ): Promise<void> {
     console.log('\nFormat Plan:');
     console.log('━'.repeat(50));
     console.log(`Summary: ${plan.summary.totalFiles} files will be changed`);
@@ -118,7 +102,7 @@ export class FormatPlanner {
     console.log(`  • ${plan.summary.filesRemoved} deleted files`);
     console.log('');
     console.log('Changes by module:');
-    
+
     // Group changes by module
     const changesByModule = new Map<string, IPlannedChange[]>();
     for (const change of plan.changes) {
@@ -126,14 +110,16 @@ export class FormatPlanner {
       moduleChanges.push(change);
       changesByModule.set(change.module, moduleChanges);
     }
-    
+
     for (const [module, changes] of changesByModule) {
-      console.log(`\n${this.getModuleIcon(module)} ${module} (${changes.length} ${changes.length === 1 ? 'file' : 'files'})`);
-      
+      console.log(
+        `\n${this.getModuleIcon(module)} ${module} (${changes.length} ${changes.length === 1 ? 'file' : 'files'})`,
+      );
+
       for (const change of changes) {
         const icon = this.getChangeIcon(change.type);
         console.log(`  ${icon} ${change.path} - ${change.description}`);
-        
+
         // Show diff for modified files if detailed view is requested
         if (detailed && change.type === 'modify') {
           const diff = await this.diffReporter.generateDiffForChange(change);
@@ -143,7 +129,7 @@ export class FormatPlanner {
         }
       }
     }
-    
+
     if (plan.warnings.length > 0) {
       console.log('\nWarnings:');
       for (const warning of plan.warnings) {
@@ -151,26 +137,26 @@ export class FormatPlanner {
         console.log(`  ${icon} ${warning.message}`);
       }
     }
-    
+
     console.log('\n' + '━'.repeat(50));
   }
-  
+
   private getModuleIcon(module: string): string {
     const icons: Record<string, string> = {
-      'packagejson': '📦',
-      'license': '📝',
-      'tsconfig': '🔧',
-      'cleanup': '🚮',
-      'gitignore': '🔒',
-      'prettier': '✨',
-      'readme': '📖',
-      'templates': '📄',
-      'npmextra': '⚙️',
-      'copy': '📋'
+      packagejson: '📦',
+      license: '📝',
+      tsconfig: '🔧',
+      cleanup: '🚮',
+      gitignore: '🔒',
+      prettier: '✨',
+      readme: '📖',
+      templates: '📄',
+      npmextra: '⚙️',
+      copy: '📋',
     };
     return icons[module] || '📁';
   }
-  
+
   private getChangeIcon(type: 'create' | 'modify' | 'delete'): string {
     switch (type) {
       case 'create':
@@ -181,4 +167,4 @@ export class FormatPlanner {
         return '❌';
     }
   }
-}
+}

package/ts/mod_format/classes.formatstats.ts

@@ -30,7 +30,7 @@ export interface IFormatStats {
 
 export class FormatStats {
   private stats: IFormatStats;
-  
+
   constructor() {
     this.stats = {
       totalExecutionTime: 0,
@@ -44,11 +44,11 @@ export class FormatStats {
         totalDeleted: 0,
         totalErrors: 0,
         cacheHits: 0,
-        cacheMisses: 0
-      }
+        cacheMisses: 0,
+      },
     };
   }
-  
+
   startModule(moduleName: string): void {
     this.stats.moduleStats.set(moduleName, {
       name: moduleName,
@@ -58,31 +58,35 @@ export class FormatStats {
       successes: 0,
       filesCreated: 0,
       filesModified: 0,
-      filesDeleted: 0
+      filesDeleted: 0,
     });
   }
-  
+
   moduleStartTime(moduleName: string): number {
     return Date.now();
   }
-  
+
   endModule(moduleName: string, startTime: number): void {
     const moduleStats = this.stats.moduleStats.get(moduleName);
     if (moduleStats) {
       moduleStats.executionTime = Date.now() - startTime;
     }
   }
-  
-  recordFileOperation(moduleName: string, operation: 'create' | 'modify' | 'delete', success: boolean = true): void {
+
+  recordFileOperation(
+    moduleName: string,
+    operation: 'create' | 'modify' | 'delete',
+    success: boolean = true,
+  ): void {
     const moduleStats = this.stats.moduleStats.get(moduleName);
     if (!moduleStats) return;
-    
+
     moduleStats.filesProcessed++;
-    
+
     if (success) {
       moduleStats.successes++;
       this.stats.overallStats.totalFiles++;
-      
+
       switch (operation) {
         case 'create':
           moduleStats.filesCreated++;
@@ -102,53 +106,66 @@ export class FormatStats {
       this.stats.overallStats.totalErrors++;
     }
   }
-  
+
   recordCacheHit(): void {
     this.stats.overallStats.cacheHits++;
   }
-  
+
   recordCacheMiss(): void {
     this.stats.overallStats.cacheMisses++;
   }
-  
+
   finish(): void {
     this.stats.endTime = Date.now();
     this.stats.totalExecutionTime = this.stats.endTime - this.stats.startTime;
   }
-  
+
   displayStats(): void {
     console.log('\n📊 Format Operation Statistics:');
     console.log('═'.repeat(50));
-    
+
     // Overall stats
     console.log('\nOverall Summary:');
-    console.log(`  Total Execution Time: ${this.formatDuration(this.stats.totalExecutionTime)}`);
+    console.log(
+      `  Total Execution Time: ${this.formatDuration(this.stats.totalExecutionTime)}`,
+    );
     console.log(`  Files Processed: ${this.stats.overallStats.totalFiles}`);
     console.log(`    • Created: ${this.stats.overallStats.totalCreated}`);
     console.log(`    • Modified: ${this.stats.overallStats.totalModified}`);
     console.log(`    • Deleted: ${this.stats.overallStats.totalDeleted}`);
     console.log(`  Errors: ${this.stats.overallStats.totalErrors}`);
-    
-    if (this.stats.overallStats.cacheHits > 0 || this.stats.overallStats.cacheMisses > 0) {
-      const cacheHitRate = this.stats.overallStats.cacheHits / 
-        (this.stats.overallStats.cacheHits + this.stats.overallStats.cacheMisses) * 100;
+
+    if (
+      this.stats.overallStats.cacheHits > 0 ||
+      this.stats.overallStats.cacheMisses > 0
+    ) {
+      const cacheHitRate =
+        (this.stats.overallStats.cacheHits /
+          (this.stats.overallStats.cacheHits +
+            this.stats.overallStats.cacheMisses)) *
+        100;
       console.log(`  Cache Hit Rate: ${cacheHitRate.toFixed(1)}%`);
       console.log(`    • Hits: ${this.stats.overallStats.cacheHits}`);
       console.log(`    • Misses: ${this.stats.overallStats.cacheMisses}`);
     }
-    
+
     // Module stats
     console.log('\nModule Breakdown:');
     console.log('─'.repeat(50));
-    
-    const sortedModules = Array.from(this.stats.moduleStats.values())
-      .sort((a, b) => b.filesProcessed - a.filesProcessed);
-    
+
+    const sortedModules = Array.from(this.stats.moduleStats.values()).sort(
+      (a, b) => b.filesProcessed - a.filesProcessed,
+    );
+
     for (const moduleStats of sortedModules) {
-      console.log(`\n${this.getModuleIcon(moduleStats.name)} ${moduleStats.name}:`);
-      console.log(`  Execution Time: ${this.formatDuration(moduleStats.executionTime)}`);
+      console.log(
+        `\n${this.getModuleIcon(moduleStats.name)} ${moduleStats.name}:`,
+      );
+      console.log(
+        `  Execution Time: ${this.formatDuration(moduleStats.executionTime)}`,
+      );
       console.log(`  Files Processed: ${moduleStats.filesProcessed}`);
-      
+
       if (moduleStats.filesCreated > 0) {
         console.log(`    • Created: ${moduleStats.filesCreated}`);
       }
@@ -158,27 +175,30 @@ export class FormatStats {
       if (moduleStats.filesDeleted > 0) {
         console.log(`    • Deleted: ${moduleStats.filesDeleted}`);
       }
-      
+
       if (moduleStats.errors > 0) {
         console.log(`  ❌ Errors: ${moduleStats.errors}`);
       }
     }
-    
+
     console.log('\n' + '═'.repeat(50));
   }
-  
+
   async saveReport(outputPath: string): Promise<void> {
     const report = {
       timestamp: new Date().toISOString(),
       executionTime: this.stats.totalExecutionTime,
       overallStats: this.stats.overallStats,
-      moduleStats: Array.from(this.stats.moduleStats.values())
+      moduleStats: Array.from(this.stats.moduleStats.values()),
     };
-    
-    await plugins.smartfile.memory.toFs(JSON.stringify(report, null, 2), outputPath);
+
+    await plugins.smartfile.memory.toFs(
+      JSON.stringify(report, null, 2),
+      outputPath,
+    );
     logger.log('info', `Statistics report saved to ${outputPath}`);
   }
-  
+
   private formatDuration(ms: number): string {
     if (ms < 1000) {
       return `${ms}ms`;
@@ -190,20 +210,20 @@ export class FormatStats {
       return `${minutes}m ${seconds}s`;
     }
   }
-  
+
   private getModuleIcon(module: string): string {
     const icons: Record<string, string> = {
-      'packagejson': '📦',
-      'license': '📝',
-      'tsconfig': '🔧',
-      'cleanup': '🚮',
-      'gitignore': '🔒',
-      'prettier': '✨',
-      'readme': '📖',
-      'templates': '📄',
-      'npmextra': '⚙️',
-      'copy': '📋'
+      packagejson: '📦',
+      license: '📝',
+      tsconfig: '🔧',
+      cleanup: '🚮',
+      gitignore: '🔒',
+      prettier: '✨',
+      readme: '📖',
+      templates: '📄',
+      npmextra: '⚙️',
+      copy: '📋',
     };
     return icons[module] || '📁';
   }
-}
+}