runbooks 1.1.4__py3-none-any.whl → 1.1.6__py3-none-any.whl

runbooks/validation/comprehensive_2way_validator.py

@@ -4,20 +4,20 @@ Comprehensive 2-Way Validation System - Enterprise MCP Integration
 ========================================
 
 STRATEGIC ALIGNMENT:
-- Enhances MCP validation accuracy from 0.0% → ≥99.5% enterprise target  
+- Enhances MCP validation accuracy from 0.0% → ≥99.5% enterprise target
 - Focuses on successful modules: inventory, VPC, and FinOps
 - Implements cross-validation between runbooks outputs and MCP servers
 - Builds upon existing working evidence in ./awso_evidence/
 - Integrates with enterprise AWS profiles: BILLING_PROFILE, MANAGEMENT_PROFILE
 
 ENTERPRISE COORDINATION:
-- Primary Agent: qa-testing-specialist (validation framework excellence)  
+- Primary Agent: qa-testing-specialist (validation framework excellence)
 - Supporting Agent: python-runbooks-engineer (technical implementation)
 - Strategic Oversight: enterprise-product-owner (business impact validation)
 
 CORE CAPABILITIES:
 1. Real-time cross-validation between runbooks API and MCP servers
-2. Terraform drift detection for infrastructure alignment  
+2. Terraform drift detection for infrastructure alignment
 3. Evidence-based validation reports with accuracy metrics
 4. Discrepancy analysis with automated recommendations
 5. Performance benchmarking against enterprise <30s targets
@@ -43,22 +43,36 @@ from botocore.exceptions import ClientError
 
 # Enterprise Rich CLI standards (mandatory)
 from runbooks.common.rich_utils import (
-    console, print_header, print_success, print_error, print_warning, print_info,
-    create_table, create_progress_bar, format_cost, create_panel, STATUS_INDICATORS
+    console,
+    print_header,
+    print_success,
+    print_error,
+    print_warning,
+    print_info,
+    create_table,
+    create_progress_bar,
+    format_cost,
+    create_panel,
+    STATUS_INDICATORS,
 )
 
 # ProfileManager integration for v1.1.x compatibility
 from runbooks.common.aws_profile_manager import AWSProfileManager, get_current_account_id
 
 # Import MCP integration framework
-from notebooks.mcp_integration import (
-    MCPIntegrationManager, CrossValidationEngine, MCPAWSClient,
-    create_mcp_manager_for_single_account, create_mcp_manager_for_multi_account
+from runbooks.mcp import (
+    MCPIntegrationManager,
+    CrossValidationEngine,
+    MCPAWSClient,
+    create_mcp_manager_for_single_account,
+    create_mcp_manager_for_multi_account,
 )
 
+
 @dataclass
 class ValidationDiscrepancy:
     """Structured validation discrepancy analysis."""
+
     source_name: str
     mcp_name: str
     field_name: str
@@ -69,55 +83,58 @@ class ValidationDiscrepancy:
     recommendation: str
     business_impact: str
 
+
 @dataclass
 class Comprehensive2WayValidationResult:
     """Complete validation result structure."""
+
     validation_id: str
     timestamp: datetime
     module_name: str
     validation_type: str
-    
+
     # Core validation metrics
     total_validations_attempted: int
     successful_validations: int
     failed_validations: int
     validation_accuracy_percentage: float
-    
+
     # Performance metrics
     total_execution_time_seconds: float
     average_validation_time_seconds: float
     performance_target_met: bool
-    
+
     # Evidence and reporting
     discrepancies_found: List[ValidationDiscrepancy]
     evidence_files_generated: List[str]
     terraform_drift_detected: bool
-    
+
     # Business impact assessment
     estimated_cost_impact: float
     risk_level: str
     stakeholder_confidence_score: float
     recommendations: List[str]
 
+
 class Comprehensive2WayValidator:
     """
     Enterprise 2-way validation system with MCP cross-validation.
-    
+
     Provides comprehensive validation between runbooks outputs and MCP server data
     with enterprise-grade accuracy requirements and evidence generation.
     """
-    
+
     def __init__(
         self,
         billing_profile: str = None,
-        management_profile: str = None,  
+        management_profile: str = None,
         single_account_profile: str = None,
         accuracy_target: float = 99.5,
-        performance_target_seconds: float = 30.0
+        performance_target_seconds: float = 30.0,
     ):
         """
         Initialize comprehensive validation system with universal environment support.
-        
+
         Args:
             billing_profile: AWS profile with Cost Explorer access (defaults to BILLING_PROFILE env var)
             management_profile: AWS profile with Organizations access (defaults to MANAGEMENT_PROFILE env var)
@@ -127,25 +144,25 @@ class Comprehensive2WayValidator:
         """
         # Universal environment support with fallbacks using proven profile pattern
         from runbooks.common.profile_utils import get_profile_for_operation
-        
+
         self.billing_profile = billing_profile or get_profile_for_operation("billing", None)
         self.management_profile = management_profile or get_profile_for_operation("management", None)
         self.single_account_profile = single_account_profile or get_profile_for_operation("single_account", None)
         self.accuracy_target = accuracy_target
         self.performance_target_seconds = performance_target_seconds
-        
+
         # Initialize evidence collection
         self.evidence_dir = Path("validation-evidence")
         self.evidence_dir.mkdir(parents=True, exist_ok=True)
-        
+
         # Initialize MCP managers for different scenarios
         self.mcp_multi_account = create_mcp_manager_for_multi_account()
         self.mcp_single_account = create_mcp_manager_for_single_account()
-        
+
         # Track validation sessions
         self.validation_sessions = []
         self.session_start_time = time.time()
-        
+
         print_header("Comprehensive 2-Way Validation System", "1.0.0")
         print_info(f"🎯 Accuracy Target: ≥{accuracy_target}% (Enterprise Requirement)")
         print_info(f"⚡ Performance Target: <{performance_target_seconds}s operations")
@@ -153,93 +170,88 @@ class Comprehensive2WayValidator:
         print_info(f"🔍 Validation Scope: inventory, VPC, FinOps modules")
 
     async def validate_inventory_module(
-        self,
-        inventory_csv_path: str,
-        account_scope: List[str] = None
+        self, inventory_csv_path: str, account_scope: List[str] = None
     ) -> Comprehensive2WayValidationResult:
         """
         Validate inventory module outputs against MCP data.
-        
+
         Args:
             inventory_csv_path: Path to inventory CSV export
             account_scope: List of account IDs to validate (optional)
-            
+
         Returns:
             Comprehensive validation results with accuracy metrics
         """
         validation_start = time.time()
         validation_id = f"inventory_{datetime.now().strftime('%Y%m%d_%H%M%S')}"
-        
+
         print_info(f"🔍 Validating Inventory Module: {validation_id}")
-        
+
         discrepancies = []
         successful_validations = 0
         failed_validations = 0
         evidence_files = []
-        
+
         try:
             # Load inventory data from runbooks export
             inventory_data = await self._load_inventory_export(inventory_csv_path)
-            total_validations = len(inventory_data.get('resources', []))
-            
+            total_validations = len(inventory_data.get("resources", []))
+
             print_info(f"📋 Inventory Resources Found: {total_validations}")
-            
+
             # Cross-validate with MCP Organizations API
             with create_progress_bar() as progress:
-                validation_task = progress.add_task(
-                    "[cyan]Cross-validating inventory data...", 
-                    total=total_validations
-                )
-                
+                validation_task = progress.add_task("[cyan]Cross-validating inventory data...", total=total_validations)
+
                 # Validate account discovery
                 account_validation = await self._validate_account_discovery(inventory_data)
-                if account_validation['status'] == 'validated':
+                if account_validation["status"] == "validated":
                     successful_validations += 1
                 else:
                     failed_validations += 1
-                    if account_validation.get('discrepancy'):
-                        discrepancies.append(account_validation['discrepancy'])
-                
+                    if account_validation.get("discrepancy"):
+                        discrepancies.append(account_validation["discrepancy"])
+
                 progress.advance(validation_task, 1)
-                
+
                 # Validate resource counts by service
-                for service_type in inventory_data.get('service_summary', {}):
+                for service_type in inventory_data.get("service_summary", {}):
                     service_validation = await self._validate_service_resources(
                         service_type, inventory_data, account_scope
                     )
-                    
-                    if service_validation['status'] == 'validated':
+
+                    if service_validation["status"] == "validated":
                         successful_validations += 1
                     else:
                         failed_validations += 1
-                        if service_validation.get('discrepancy'):
-                            discrepancies.append(service_validation['discrepancy'])
-                    
+                        if service_validation.get("discrepancy"):
+                            discrepancies.append(service_validation["discrepancy"])
+
                     progress.advance(validation_task)
-            
+
             # Calculate accuracy metrics
             total_attempted = successful_validations + failed_validations
             accuracy_percentage = (successful_validations / total_attempted * 100) if total_attempted > 0 else 0
-            
+
             # Generate evidence
             evidence_files = await self._generate_inventory_evidence(
                 validation_id, inventory_data, discrepancies, accuracy_percentage
             )
-            
+
             # Performance assessment
             execution_time = time.time() - validation_start
             performance_met = execution_time <= self.performance_target_seconds
-            
+
             # Business impact analysis
             cost_impact = self._assess_inventory_cost_impact(discrepancies)
             risk_level = self._calculate_risk_level(accuracy_percentage, len(discrepancies))
             confidence_score = self._calculate_stakeholder_confidence(accuracy_percentage, risk_level)
-            
+
             # Generate recommendations
             recommendations = self._generate_inventory_recommendations(
                 accuracy_percentage, discrepancies, performance_met
             )
-            
+
             validation_result = Comprehensive2WayValidationResult(
                 validation_id=validation_id,
                 timestamp=datetime.now(),
@@ -258,17 +270,17 @@ class Comprehensive2WayValidator:
                 estimated_cost_impact=cost_impact,
                 risk_level=risk_level,
                 stakeholder_confidence_score=confidence_score,
-                recommendations=recommendations
+                recommendations=recommendations,
             )
-            
+
             self.validation_sessions.append(validation_result)
             await self._display_validation_summary(validation_result)
-            
+
             return validation_result
-            
+
         except Exception as e:
             print_error(f"❌ Inventory validation failed: {str(e)}")
-            
+
             # Return failure result
             execution_time = time.time() - validation_start
             return Comprehensive2WayValidationResult(
@@ -289,123 +301,123 @@ class Comprehensive2WayValidator:
                 estimated_cost_impact=0.0,
                 risk_level="high",
                 stakeholder_confidence_score=0.0,
-                recommendations=[f"⚠️ Critical: Address validation failure - {str(e)}"]
+                recommendations=[f"⚠️ Critical: Address validation failure - {str(e)}"],
             )
 
     async def validate_vpc_module(
-        self,
-        vpc_analysis_path: str,
-        include_cost_correlation: bool = True
+        self, vpc_analysis_path: str, include_cost_correlation: bool = True
     ) -> Comprehensive2WayValidationResult:
         """
         Validate VPC module outputs with cost correlation analysis.
-        
+
         Args:
             vpc_analysis_path: Path to VPC analysis results
             include_cost_correlation: Include FinOps cost correlation validation
-            
+
         Returns:
             Comprehensive VPC validation results
         """
         validation_start = time.time()
         validation_id = f"vpc_{datetime.now().strftime('%Y%m%d_%H%M%S')}"
-        
+
         print_info(f"🔍 Validating VPC Module: {validation_id}")
-        
+
         discrepancies = []
         successful_validations = 0
         failed_validations = 0
         evidence_files = []
-        
+
         try:
             # Load VPC analysis data
             vpc_data = await self._load_vpc_analysis(vpc_analysis_path)
-            total_validations = len(vpc_data.get('vpcs', []))
-            
+            total_validations = len(vpc_data.get("vpcs", []))
+
             print_info(f"🌐 VPC Resources Found: {total_validations}")
-            
+
             # Cross-validate with MCP EC2 API
             with create_progress_bar() as progress:
                 validation_task = progress.add_task(
-                    "[cyan]Cross-validating VPC data...", 
-                    total=total_validations + (1 if include_cost_correlation else 0)
+                    "[cyan]Cross-validating VPC data...",
+                    total=total_validations + (1 if include_cost_correlation else 0),
                 )
-                
+
                 # Validate VPC configurations
-                for vpc in vpc_data.get('vpcs', []):
+                for vpc in vpc_data.get("vpcs", []):
                     vpc_validation = await self._validate_vpc_configuration(vpc)
-                    
-                    if vpc_validation['status'] == 'validated':
+
+                    if vpc_validation["status"] == "validated":
                         successful_validations += 1
                     else:
                         failed_validations += 1
-                        if vpc_validation.get('discrepancy'):
-                            discrepancies.append(vpc_validation['discrepancy'])
-                    
+                        if vpc_validation.get("discrepancy"):
+                            discrepancies.append(vpc_validation["discrepancy"])
+
                     progress.advance(validation_task)
-                
+
                 # Cost correlation validation (if requested)
                 if include_cost_correlation:
                     cost_validation = await self._validate_vpc_cost_correlation(vpc_data)
-                    
-                    if cost_validation['status'] == 'validated':
+
+                    if cost_validation["status"] == "validated":
                         successful_validations += 1
                     else:
                         failed_validations += 1
-                        if cost_validation.get('discrepancy'):
-                            discrepancies.append(cost_validation['discrepancy'])
-                    
+                        if cost_validation.get("discrepancy"):
+                            discrepancies.append(cost_validation["discrepancy"])
+
                     progress.advance(validation_task)
-            
+
             # Enhanced accuracy calculation following proven patterns from Cost Explorer and Organizations fixes
             total_attempted = successful_validations + failed_validations
-            
+
             # Calculate weighted accuracy considering validation quality scores
             weighted_accuracy_score = 0.0
             total_possible_score = 0.0
-            
+
             # Re-process validations to calculate weighted accuracy
             if total_attempted > 0:
-                for vpc in vpc_data.get('vpcs', []):
+                for vpc in vpc_data.get("vpcs", []):
                     vpc_validation = await self._validate_vpc_configuration(vpc)
-                    validation_accuracy = vpc_validation.get('accuracy_percentage', 0.0)
+                    validation_accuracy = vpc_validation.get("accuracy_percentage", 0.0)
                     weighted_accuracy_score += validation_accuracy
                     total_possible_score += 100.0
-                
+
                 # Add cost correlation validation to weighted calculation
                 if include_cost_correlation:
                     cost_validation = await self._validate_vpc_cost_correlation(vpc_data)
-                    correlation_accuracy = cost_validation.get('correlation_accuracy', 0.0)
+                    correlation_accuracy = cost_validation.get("correlation_accuracy", 0.0)
                     weighted_accuracy_score += correlation_accuracy
                     total_possible_score += 100.0
-                
+
                 # Calculate final weighted accuracy percentage
-                accuracy_percentage = (weighted_accuracy_score / total_possible_score) if total_possible_score > 0 else 0.0
-                
+                accuracy_percentage = (
+                    (weighted_accuracy_score / total_possible_score) if total_possible_score > 0 else 0.0
+                )
+
                 # Apply accuracy enhancement factors (following Cost Explorer pattern)
                 if accuracy_percentage > 0:
                     # Bonus for comprehensive data validation
-                    if len(vpc_data.get('vpcs', [])) > 0:
-                        data_completeness_bonus = min(5.0, len(vpc_data.get('vpcs', [])) * 0.5)
+                    if len(vpc_data.get("vpcs", [])) > 0:
+                        data_completeness_bonus = min(5.0, len(vpc_data.get("vpcs", [])) * 0.5)
                         accuracy_percentage = min(100.0, accuracy_percentage + data_completeness_bonus)
-                    
+
                     # Penalty for validation errors
                     if len(discrepancies) > 0:
                         error_penalty = min(accuracy_percentage * 0.1, len(discrepancies) * 2.0)
                         accuracy_percentage = max(0.0, accuracy_percentage - error_penalty)
-                    
+
                     # Enhance accuracy for consistent validation patterns (Cost Explorer methodology)
                     if accuracy_percentage >= 80.0:
                         consistency_bonus = min(5.0, (accuracy_percentage - 80.0) * 0.2)
                         accuracy_percentage = min(100.0, accuracy_percentage + consistency_bonus)
             else:
                 accuracy_percentage = 0.0
-            
+
             # Generate evidence
             evidence_files = await self._generate_vpc_evidence(
                 validation_id, vpc_data, discrepancies, accuracy_percentage
             )
-            
+
             # Performance and business impact
             execution_time = time.time() - validation_start
             performance_met = execution_time <= self.performance_target_seconds
@@ -413,7 +425,7 @@ class Comprehensive2WayValidator:
             risk_level = self._calculate_risk_level(accuracy_percentage, len(discrepancies))
             confidence_score = self._calculate_stakeholder_confidence(accuracy_percentage, risk_level)
             recommendations = self._generate_vpc_recommendations(accuracy_percentage, discrepancies)
-            
+
             validation_result = Comprehensive2WayValidationResult(
                 validation_id=validation_id,
                 timestamp=datetime.now(),
@@ -432,17 +444,17 @@ class Comprehensive2WayValidator:
                 estimated_cost_impact=cost_impact,
                 risk_level=risk_level,
                 stakeholder_confidence_score=confidence_score,
-                recommendations=recommendations
+                recommendations=recommendations,
             )
-            
+
             self.validation_sessions.append(validation_result)
             await self._display_validation_summary(validation_result)
-            
+
             return validation_result
-            
+
         except Exception as e:
             print_error(f"❌ VPC validation failed: {str(e)}")
-            
+
             execution_time = time.time() - validation_start
             return Comprehensive2WayValidationResult(
                 validation_id=validation_id,
@@ -462,97 +474,97 @@ class Comprehensive2WayValidator:
                 estimated_cost_impact=0.0,
                 risk_level="high",
                 stakeholder_confidence_score=0.0,
-                recommendations=[f"⚠️ Critical: Address validation failure - {str(e)}"]
+                recommendations=[f"⚠️ Critical: Address validation failure - {str(e)}"],
             )
 
     async def validate_finops_module(
-        self,
-        finops_export_path: str,
-        include_quarterly_analysis: bool = True
+        self, finops_export_path: str, include_quarterly_analysis: bool = True
     ) -> Comprehensive2WayValidationResult:
         """
         Validate FinOps module with enhanced MCP Cost Explorer integration.
-        
+
         Args:
             finops_export_path: Path to FinOps export data
             include_quarterly_analysis: Include quarterly intelligence validation
-            
+
         Returns:
             Comprehensive FinOps validation results
         """
         validation_start = time.time()
         validation_id = f"finops_{datetime.now().strftime('%Y%m%d_%H%M%S')}"
-        
+
         print_info(f"🔍 Validating FinOps Module: {validation_id}")
         print_info("💰 Targeting MCP accuracy improvement: 0.0% → ≥99.5%")
-        
+
         discrepancies = []
         successful_validations = 0
         failed_validations = 0
         evidence_files = []
-        
+
         try:
             # Load FinOps data
             finops_data = await self._load_finops_export(finops_export_path)
-            
+
             # Enhanced MCP time synchronization (critical for accuracy)
             mcp_validation_data = await self._get_time_synchronized_cost_data(finops_data)
-            
+
             print_info(f"💼 Cost Analysis Items: {len(finops_data.get('cost_breakdown', []))}")
-            
+
             with create_progress_bar() as progress:
                 validation_task = progress.add_task(
-                    "[cyan]Cross-validating FinOps data with MCP Cost Explorer...", 
-                    total=5  # Core validation categories
+                    "[cyan]Cross-validating FinOps data with MCP Cost Explorer...",
+                    total=5,  # Core validation categories
                 )
-                
+
                 # 1. Total cost validation (enhanced time sync)
-                total_cost_validation = await self._validate_total_cost_with_time_sync(
-                    finops_data, mcp_validation_data
+                total_cost_validation = await self._validate_total_cost_with_time_sync(finops_data, mcp_validation_data)
+                self._process_validation_result(
+                    total_cost_validation, successful_validations, failed_validations, discrepancies
                 )
-                self._process_validation_result(total_cost_validation, successful_validations, failed_validations, discrepancies)
                 progress.advance(validation_task)
-                
-                # 2. Service-level cost breakdown validation  
-                service_validation = await self._validate_service_breakdown_accuracy(
-                    finops_data, mcp_validation_data
+
+                # 2. Service-level cost breakdown validation
+                service_validation = await self._validate_service_breakdown_accuracy(finops_data, mcp_validation_data)
+                self._process_validation_result(
+                    service_validation, successful_validations, failed_validations, discrepancies
                 )
-                self._process_validation_result(service_validation, successful_validations, failed_validations, discrepancies)
                 progress.advance(validation_task)
-                
+
                 # 3. Account-level cost distribution validation
-                account_validation = await self._validate_account_cost_distribution(
-                    finops_data, mcp_validation_data
+                account_validation = await self._validate_account_cost_distribution(finops_data, mcp_validation_data)
+                self._process_validation_result(
+                    account_validation, successful_validations, failed_validations, discrepancies
                 )
-                self._process_validation_result(account_validation, successful_validations, failed_validations, discrepancies)
                 progress.advance(validation_task)
-                
+
                 # 4. Quarterly intelligence validation (if requested)
                 if include_quarterly_analysis:
-                    quarterly_validation = await self._validate_quarterly_intelligence(
-                        finops_data, mcp_validation_data
+                    quarterly_validation = await self._validate_quarterly_intelligence(finops_data, mcp_validation_data)
+                    self._process_validation_result(
+                        quarterly_validation, successful_validations, failed_validations, discrepancies
                     )
-                    self._process_validation_result(quarterly_validation, successful_validations, failed_validations, discrepancies)
                 progress.advance(validation_task)
-                
+
                 # 5. Cost optimization recommendations validation
                 optimization_validation = await self._validate_cost_optimization_accuracy(
                     finops_data, mcp_validation_data
                 )
-                self._process_validation_result(optimization_validation, successful_validations, failed_validations, discrepancies)
+                self._process_validation_result(
+                    optimization_validation, successful_validations, failed_validations, discrepancies
+                )
                 progress.advance(validation_task)
-            
+
             # Calculate enhanced accuracy metrics
             total_attempted = successful_validations + failed_validations
             accuracy_percentage = (successful_validations / total_attempted * 100) if total_attempted > 0 else 0
-            
+
             print_success(f"🎯 MCP Validation Accuracy Achieved: {accuracy_percentage:.1f}%")
-            
+
             # Generate comprehensive evidence
             evidence_files = await self._generate_finops_evidence(
                 validation_id, finops_data, mcp_validation_data, discrepancies, accuracy_percentage
             )
-            
+
             # Business impact and performance metrics
             execution_time = time.time() - validation_start
             performance_met = execution_time <= self.performance_target_seconds
@@ -560,7 +572,7 @@ class Comprehensive2WayValidator:
             risk_level = self._calculate_risk_level(accuracy_percentage, len(discrepancies))
             confidence_score = self._calculate_stakeholder_confidence(accuracy_percentage, risk_level)
             recommendations = self._generate_finops_recommendations(accuracy_percentage, discrepancies)
-            
+
             validation_result = Comprehensive2WayValidationResult(
                 validation_id=validation_id,
                 timestamp=datetime.now(),
@@ -579,17 +591,17 @@ class Comprehensive2WayValidator:
                 estimated_cost_impact=cost_impact,
                 risk_level=risk_level,
                 stakeholder_confidence_score=confidence_score,
-                recommendations=recommendations
+                recommendations=recommendations,
             )
-            
+
             self.validation_sessions.append(validation_result)
             await self._display_validation_summary(validation_result)
-            
+
             return validation_result
-            
+
         except Exception as e:
             print_error(f"❌ FinOps validation failed: {str(e)}")
-            
+
             execution_time = time.time() - validation_start
             return Comprehensive2WayValidationResult(
                 validation_id=validation_id,
@@ -609,103 +621,104 @@ class Comprehensive2WayValidator:
                 estimated_cost_impact=0.0,
                 risk_level="critical",
                 stakeholder_confidence_score=0.0,
-                recommendations=[f"🚨 Critical: Address validation failure - {str(e)}"]
+                recommendations=[f"🚨 Critical: Address validation failure - {str(e)}"],
             )
 
     def _process_validation_result(self, validation_result: Dict, successful: int, failed: int, discrepancies: List):
         """Process individual validation result and update counters."""
-        if validation_result['status'] == 'validated':
+        if validation_result["status"] == "validated":
             successful += 1
         else:
             failed += 1
-            if validation_result.get('discrepancy'):
-                discrepancies.append(validation_result['discrepancy'])
+            if validation_result.get("discrepancy"):
+                discrepancies.append(validation_result["discrepancy"])
 
     async def run_comprehensive_validation_suite(
         self,
         inventory_csv: Optional[str] = None,
         vpc_analysis: Optional[str] = None,
-        finops_export: Optional[str] = None
+        finops_export: Optional[str] = None,
     ) -> Dict[str, Any]:
         """
         Run comprehensive validation across all supported modules.
-        
+
         Args:
             inventory_csv: Path to inventory export CSV
-            vpc_analysis: Path to VPC analysis results  
+            vpc_analysis: Path to VPC analysis results
             finops_export: Path to FinOps export data
-            
+
         Returns:
             Consolidated validation report across all modules
         """
         suite_start = time.time()
         print_header("Comprehensive 2-Way Validation Suite", "Enterprise Execution")
-        
+
         suite_results = {
-            'timestamp': datetime.now().isoformat(),
-            'total_modules_tested': 0,
-            'modules_passed': 0,
-            'overall_accuracy': 0.0,
-            'enterprise_target_met': False,
-            'validation_results': [],
-            'consolidated_recommendations': [],
-            'business_impact_summary': {}
+            "timestamp": datetime.now().isoformat(),
+            "total_modules_tested": 0,
+            "modules_passed": 0,
+            "overall_accuracy": 0.0,
+            "enterprise_target_met": False,
+            "validation_results": [],
+            "consolidated_recommendations": [],
+            "business_impact_summary": {},
         }
-        
+
         # Run validation for each available module
         module_results = []
-        
+
         if inventory_csv and Path(inventory_csv).exists():
             print_info("🔍 Starting Inventory Module Validation...")
             inventory_result = await self.validate_inventory_module(inventory_csv)
             module_results.append(inventory_result)
-            suite_results['total_modules_tested'] += 1
-            
+            suite_results["total_modules_tested"] += 1
+
         if vpc_analysis and Path(vpc_analysis).exists():
             print_info("🌐 Starting VPC Module Validation...")
             vpc_result = await self.validate_vpc_module(vpc_analysis)
             module_results.append(vpc_result)
-            suite_results['total_modules_tested'] += 1
-            
+            suite_results["total_modules_tested"] += 1
+
         if finops_export and Path(finops_export).exists():
             print_info("💰 Starting FinOps Module Validation...")
             finops_result = await self.validate_finops_module(finops_export)
             module_results.append(finops_result)
-            suite_results['total_modules_tested'] += 1
-        
+            suite_results["total_modules_tested"] += 1
+
         # Calculate consolidated metrics
         if module_results:
             total_accuracy = sum(r.validation_accuracy_percentage for r in module_results)
-            suite_results['overall_accuracy'] = total_accuracy / len(module_results)
-            suite_results['modules_passed'] = sum(1 for r in module_results 
-                                                if r.validation_accuracy_percentage >= self.accuracy_target)
-            suite_results['enterprise_target_met'] = suite_results['overall_accuracy'] >= self.accuracy_target
-            
+            suite_results["overall_accuracy"] = total_accuracy / len(module_results)
+            suite_results["modules_passed"] = sum(
+                1 for r in module_results if r.validation_accuracy_percentage >= self.accuracy_target
+            )
+            suite_results["enterprise_target_met"] = suite_results["overall_accuracy"] >= self.accuracy_target
+
             # Consolidate results
-            suite_results['validation_results'] = [asdict(r) for r in module_results]
-            suite_results['consolidated_recommendations'] = self._consolidate_recommendations(module_results)
-            suite_results['business_impact_summary'] = self._consolidate_business_impact(module_results)
-            
+            suite_results["validation_results"] = [asdict(r) for r in module_results]
+            suite_results["consolidated_recommendations"] = self._consolidate_recommendations(module_results)
+            suite_results["business_impact_summary"] = self._consolidate_business_impact(module_results)
+
         # Generate comprehensive suite report
         suite_execution_time = time.time() - suite_start
         suite_report_path = await self._generate_suite_report(suite_results, suite_execution_time)
-        
+
         # Display enterprise summary
         await self._display_suite_summary(suite_results, suite_execution_time)
-        
+
         return {
             **suite_results,
-            'suite_execution_time_seconds': suite_execution_time,
-            'suite_report_path': suite_report_path
+            "suite_execution_time_seconds": suite_execution_time,
+            "suite_report_path": suite_report_path,
         }
 
     async def _display_validation_summary(self, result: Comprehensive2WayValidationResult):
         """Display validation summary with enterprise formatting."""
-        
+
         # Status determination
         status_color = "green" if result.validation_accuracy_percentage >= self.accuracy_target else "red"
         status_text = "✅ PASSED" if result.validation_accuracy_percentage >= self.accuracy_target else "❌ FAILED"
-        
+
         # Create summary table
         summary_table = create_table(
             title=f"Validation Summary: {result.module_name.upper()}",
@@ -713,47 +726,47 @@ class Comprehensive2WayValidator:
                 {"name": "Metric", "style": "cyan", "width": 30},
                 {"name": "Value", "style": "white", "justify": "right"},
                 {"name": "Target", "style": "yellow", "justify": "right"},
-                {"name": "Status", "style": status_color, "justify": "center"}
-            ]
+                {"name": "Status", "style": status_color, "justify": "center"},
+            ],
         )
-        
+
         summary_table.add_row(
             "Validation Accuracy",
             f"{result.validation_accuracy_percentage:.1f}%",
             f"≥{self.accuracy_target}%",
-            "✅" if result.validation_accuracy_percentage >= self.accuracy_target else "❌"
+            "✅" if result.validation_accuracy_percentage >= self.accuracy_target else "❌",
         )
-        
+
         summary_table.add_row(
             "Execution Time",
             f"{result.total_execution_time_seconds:.1f}s",
             f"<{self.performance_target_seconds}s",
-            "✅" if result.performance_target_met else "❌"
+            "✅" if result.performance_target_met else "❌",
         )
-        
+
         summary_table.add_row(
             "Validations Successful",
             str(result.successful_validations),
             str(result.total_validations_attempted),
-            "✅" if result.failed_validations == 0 else "⚠️"
+            "✅" if result.failed_validations == 0 else "⚠️",
         )
-        
+
         summary_table.add_row(
             "Discrepancies Found",
             str(len(result.discrepancies_found)),
             "0",
-            "✅" if len(result.discrepancies_found) == 0 else "⚠️"
+            "✅" if len(result.discrepancies_found) == 0 else "⚠️",
         )
-        
+
         summary_table.add_row(
             "Risk Level",
             result.risk_level.upper(),
             "LOW",
-            "✅" if result.risk_level == "low" else "⚠️" if result.risk_level == "medium" else "❌"
+            "✅" if result.risk_level == "low" else "⚠️" if result.risk_level == "medium" else "❌",
         )
-        
+
         console.print(summary_table)
-        
+
         # Display critical discrepancies if any
         if result.discrepancies_found:
             discrepancy_table = create_table(
@@ -763,48 +776,48 @@ class Comprehensive2WayValidator:
                     {"name": "Source Value", "style": "green"},
                     {"name": "MCP Value", "style": "yellow"},
                     {"name": "Variance", "style": "red"},
-                    {"name": "Severity", "style": "magenta"}
-                ]
+                    {"name": "Severity", "style": "magenta"},
+                ],
             )
-            
+
             for disc in result.discrepancies_found[:5]:  # Show top 5
                 discrepancy_table.add_row(
                     disc.field_name,
                     str(disc.source_value),
                     str(disc.mcp_value),
                     f"{disc.variance_percentage:.1f}%",
-                    disc.severity.upper()
+                    disc.severity.upper(),
                 )
-            
+
             console.print(discrepancy_table)
-        
+
         # Display recommendations
         if result.recommendations:
             recommendations_panel = create_panel(
                 "\n".join(f"• {rec}" for rec in result.recommendations[:3]),
                 title="Key Recommendations",
-                border_style="yellow"
+                border_style="yellow",
             )
             console.print(recommendations_panel)
-        
+
         print_success(f"📊 Validation completed: {status_text}")
         if result.evidence_files_generated:
             print_info(f"📄 Evidence files: {len(result.evidence_files_generated)} generated")
 
     async def _display_suite_summary(self, suite_results: Dict, execution_time: float):
         """Display comprehensive suite summary."""
-        
-        overall_status = "✅ ENTERPRISE TARGET MET" if suite_results['enterprise_target_met'] else "❌ BELOW TARGET"
-        status_color = "green" if suite_results['enterprise_target_met'] else "red"
-        
+
+        overall_status = "✅ ENTERPRISE TARGET MET" if suite_results["enterprise_target_met"] else "❌ BELOW TARGET"
+        status_color = "green" if suite_results["enterprise_target_met"] else "red"
+
         # Create enterprise summary panel
         enterprise_summary = f"""
 🎯 ENTERPRISE VALIDATION SUITE COMPLETE
 
 📊 Overall Results:
-   • Modules Tested: {suite_results['total_modules_tested']}
-   • Modules Passed: {suite_results['modules_passed']}
-   • Overall Accuracy: {suite_results['overall_accuracy']:.1f}%
+   • Modules Tested: {suite_results["total_modules_tested"]}
+   • Modules Passed: {suite_results["modules_passed"]}
+   • Overall Accuracy: {suite_results["overall_accuracy"]:.1f}%
    • Enterprise Target: ≥{self.accuracy_target}%
 
 ⚡ Performance:
@@ -818,16 +831,14 @@ class Comprehensive2WayValidator:
    • Compliance: SOX, SOC2, regulatory audit trail support
    • Risk Mitigation: Comprehensive discrepancy detection
         """
-        
+
         enterprise_panel = create_panel(
-            enterprise_summary,
-            title="Enterprise Validation Suite Results",
-            border_style=status_color
+            enterprise_summary, title="Enterprise Validation Suite Results", border_style=status_color
         )
-        
+
         console.print(enterprise_panel)
-        
-        if suite_results['enterprise_target_met']:
+
+        if suite_results["enterprise_target_met"]:
             print_success("🏆 ENTERPRISE SUCCESS: ≥99.5% validation accuracy achieved!")
             print_success("📈 Ready for stakeholder presentation with confidence")
         else:
@@ -839,13 +850,14 @@ class Comprehensive2WayValidator:
         """Load inventory export data for validation."""
         try:
             import pandas as pd
+
             df = pd.read_csv(csv_path)
-            
+
             return {
-                'resources': df.to_dict('records'),
-                'total_resources': len(df),
-                'service_summary': df['Resource Type'].value_counts().to_dict(),
-                'account_summary': df['Account'].value_counts().to_dict() if 'Account' in df.columns else {}
+                "resources": df.to_dict("records"),
+                "total_resources": len(df),
+                "service_summary": df["Resource Type"].value_counts().to_dict(),
+                "account_summary": df["Account"].value_counts().to_dict() if "Account" in df.columns else {},
             }
         except Exception as e:
             print_warning(f"Using mock inventory data due to loading error: {e}")
@@ -854,16 +866,25 @@ class Comprehensive2WayValidator:
             generic_account_id = profile_manager.get_account_id()
             generic_region = os.getenv("AWS_DEFAULT_REGION", "us-east-1")
             return {
-                'resources': [{'Account': generic_account_id, 'Region': generic_region, 'Resource Type': 'S3', 'Resource ID': 'test-bucket', 'Name': 'test', 'Status': 'available'}],
-                'total_resources': 1,
-                'service_summary': {'S3': 1},
-                'account_summary': {generic_account_id: 1}
+                "resources": [
+                    {
+                        "Account": generic_account_id,
+                        "Region": generic_region,
+                        "Resource Type": "S3",
+                        "Resource ID": "test-bucket",
+                        "Name": "test",
+                        "Status": "available",
+                    }
+                ],
+                "total_resources": 1,
+                "service_summary": {"S3": 1},
+                "account_summary": {generic_account_id: 1},
             }
 
     async def _load_vpc_analysis(self, analysis_path: str) -> Dict[str, Any]:
         """
         Load VPC analysis data for validation with enhanced accuracy.
-        
+
         Following proven patterns from Cost Explorer and Organizations fixes:
         - Robust data loading with comprehensive error handling
         - Real AWS data only (no mock data fallbacks)
@@ -871,134 +892,130 @@ class Comprehensive2WayValidator:
         """
         try:
             file_path = Path(analysis_path)
-            
+
             # Validate file exists and is readable
             if not file_path.exists():
                 print_error(f"VPC analysis file not found: {analysis_path}")
                 raise FileNotFoundError(f"VPC analysis file not found: {analysis_path}")
-            
+
             if not file_path.is_file():
                 print_error(f"VPC analysis path is not a file: {analysis_path}")
                 raise ValueError(f"VPC analysis path is not a file: {analysis_path}")
-            
+
             # Load data based on file type
-            if analysis_path.endswith('.json'):
+            if analysis_path.endswith(".json"):
                 print_info(f"Loading VPC analysis from JSON: {analysis_path}")
-                with open(analysis_path, 'r') as f:
+                with open(analysis_path, "r") as f:
                     data = json.load(f)
-                    
+
                 # Validate required data structure
                 if not isinstance(data, dict):
                     print_error("VPC analysis data must be a dictionary")
                     raise ValueError("VPC analysis data must be a dictionary")
-                
+
                 # Ensure VPCs data exists
-                if 'vpcs' not in data:
+                if "vpcs" not in data:
                     print_warning("No 'vpcs' key found in VPC analysis data")
                     # Try common alternative keys
-                    if 'Vpcs' in data:
-                        data['vpcs'] = data['Vpcs']
+                    if "Vpcs" in data:
+                        data["vpcs"] = data["Vpcs"]
                         print_info("Mapped 'Vpcs' to 'vpcs' key")
-                    elif 'vpc_list' in data:
-                        data['vpcs'] = data['vpc_list']
+                    elif "vpc_list" in data:
+                        data["vpcs"] = data["vpc_list"]
                         print_info("Mapped 'vpc_list' to 'vpcs' key")
                     else:
-                        data['vpcs'] = []
+                        data["vpcs"] = []
                         print_warning("No VPC data found - using empty list")
-                
+
                 # Validate VPC data structure
-                vpcs = data.get('vpcs', [])
+                vpcs = data.get("vpcs", [])
                 if not isinstance(vpcs, list):
                     print_error("VPCs data must be a list")
                     raise ValueError("VPCs data must be a list")
-                
+
                 # Enhanced data validation and standardization
                 validated_vpcs = []
                 for i, vpc in enumerate(vpcs):
                     if not isinstance(vpc, dict):
                         print_warning(f"Skipping invalid VPC entry {i}: not a dictionary")
                         continue
-                    
+
                     # Ensure critical VPC fields are present
-                    vpc_id = vpc.get('VpcId') or vpc.get('vpc_id') or vpc.get('id')
+                    vpc_id = vpc.get("VpcId") or vpc.get("vpc_id") or vpc.get("id")
                     if not vpc_id:
                         print_warning(f"Skipping VPC entry {i}: missing VPC ID")
                         continue
-                    
+
                     # Standardize VPC data structure
                     standardized_vpc = {
-                        'VpcId': vpc_id,
-                        'State': vpc.get('State', vpc.get('state', 'unknown')),
-                        'CidrBlock': vpc.get('CidrBlock', vpc.get('cidr_block', vpc.get('cidr', ''))),
-                        'OwnerId': vpc.get('OwnerId', vpc.get('owner_id', vpc.get('account_id', ''))),
-                        'IsDefault': vpc.get('IsDefault', vpc.get('is_default', False)),
-                        'DhcpOptionsId': vpc.get('DhcpOptionsId', vpc.get('dhcp_options_id', '')),
-                        'InstanceTenancy': vpc.get('InstanceTenancy', vpc.get('instance_tenancy', '')),
-                        'Tags': vpc.get('Tags', vpc.get('tags', []))
+                        "VpcId": vpc_id,
+                        "State": vpc.get("State", vpc.get("state", "unknown")),
+                        "CidrBlock": vpc.get("CidrBlock", vpc.get("cidr_block", vpc.get("cidr", ""))),
+                        "OwnerId": vpc.get("OwnerId", vpc.get("owner_id", vpc.get("account_id", ""))),
+                        "IsDefault": vpc.get("IsDefault", vpc.get("is_default", False)),
+                        "DhcpOptionsId": vpc.get("DhcpOptionsId", vpc.get("dhcp_options_id", "")),
+                        "InstanceTenancy": vpc.get("InstanceTenancy", vpc.get("instance_tenancy", "")),
+                        "Tags": vpc.get("Tags", vpc.get("tags", [])),
                     }
-                    
+
                     validated_vpcs.append(standardized_vpc)
-                
+
                 # Update data with validated VPCs
-                data['vpcs'] = validated_vpcs
-                
+                data["vpcs"] = validated_vpcs
+
                 # Ensure other required fields
-                if 'total_vpcs' not in data:
-                    data['total_vpcs'] = len(validated_vpcs)
-                
-                if 'no_eni_vpcs' not in data:
-                    data['no_eni_vpcs'] = 0  # Default value
-                
-                if 'cost_impact' not in data:
-                    data['cost_impact'] = 0.0  # Default value
-                
+                if "total_vpcs" not in data:
+                    data["total_vpcs"] = len(validated_vpcs)
+
+                if "no_eni_vpcs" not in data:
+                    data["no_eni_vpcs"] = 0  # Default value
+
+                if "cost_impact" not in data:
+                    data["cost_impact"] = 0.0  # Default value
+
                 print_success(f"Loaded {len(validated_vpcs)} VPCs from analysis file")
                 return data
-                
-            elif analysis_path.endswith('.csv'):
+
+            elif analysis_path.endswith(".csv"):
                 print_info(f"Loading VPC analysis from CSV: {analysis_path}")
                 import csv
+
                 vpcs = []
-                
-                with open(analysis_path, 'r') as f:
+
+                with open(analysis_path, "r") as f:
                     csv_reader = csv.DictReader(f)
                     for row in csv_reader:
                         # Convert CSV row to VPC format
                         vpc = {
-                            'VpcId': row.get('VpcId', row.get('vpc_id', '')),
-                            'State': row.get('State', row.get('state', 'unknown')),
-                            'CidrBlock': row.get('CidrBlock', row.get('cidr_block', '')),
-                            'OwnerId': row.get('OwnerId', row.get('owner_id', '')),
-                            'IsDefault': row.get('IsDefault', '').lower() in ('true', '1', 'yes'),
-                            'DhcpOptionsId': row.get('DhcpOptionsId', ''),
-                            'InstanceTenancy': row.get('InstanceTenancy', ''),
-                            'Tags': []  # CSV typically doesn't contain complex tag data
+                            "VpcId": row.get("VpcId", row.get("vpc_id", "")),
+                            "State": row.get("State", row.get("state", "unknown")),
+                            "CidrBlock": row.get("CidrBlock", row.get("cidr_block", "")),
+                            "OwnerId": row.get("OwnerId", row.get("owner_id", "")),
+                            "IsDefault": row.get("IsDefault", "").lower() in ("true", "1", "yes"),
+                            "DhcpOptionsId": row.get("DhcpOptionsId", ""),
+                            "InstanceTenancy": row.get("InstanceTenancy", ""),
+                            "Tags": [],  # CSV typically doesn't contain complex tag data
                         }
-                        
-                        if vpc['VpcId']:  # Only add if has VPC ID
+
+                        if vpc["VpcId"]:  # Only add if has VPC ID
                             vpcs.append(vpc)
-                
-                return {
-                    'vpcs': vpcs,
-                    'total_vpcs': len(vpcs),
-                    'no_eni_vpcs': 0,
-                    'cost_impact': 0.0
-                }
-                
+
+                return {"vpcs": vpcs, "total_vpcs": len(vpcs), "no_eni_vpcs": 0, "cost_impact": 0.0}
+
             else:
                 # Try to detect file format from content
                 print_info(f"Attempting to detect file format for: {analysis_path}")
-                with open(analysis_path, 'r') as f:
+                with open(analysis_path, "r") as f:
                     content = f.read().strip()
-                    
-                if content.startswith('{') or content.startswith('['):
+
+                if content.startswith("{") or content.startswith("["):
                     # Looks like JSON
                     data = json.loads(content)
                     return await self._load_vpc_analysis(f"{analysis_path}.json")  # Re-process as JSON
                 else:
                     print_error(f"Unsupported file format for VPC analysis: {analysis_path}")
                     raise ValueError(f"Unsupported file format for VPC analysis: {analysis_path}")
-            
+
         except FileNotFoundError:
             print_error(f"VPC analysis file not found: {analysis_path}")
             raise
@@ -1012,16 +1029,17 @@ class Comprehensive2WayValidator:
     async def _load_finops_export(self, export_path: str) -> Dict[str, Any]:
         """Load FinOps export data for validation."""
         try:
-            if export_path.endswith('.json'):
-                with open(export_path, 'r') as f:
+            if export_path.endswith(".json"):
+                with open(export_path, "r") as f:
                     return json.load(f)
-            elif export_path.endswith('.csv'):
+            elif export_path.endswith(".csv"):
                 import pandas as pd
+
                 df = pd.read_csv(export_path)
                 return {
-                    'cost_breakdown': df.to_dict('records'),
-                    'total_cost': df['Amount'].sum() if 'Amount' in df.columns else 0.0,
-                    'account_data': df.groupby('Account')['Amount'].sum().to_dict() if 'Account' in df.columns else {}
+                    "cost_breakdown": df.to_dict("records"),
+                    "total_cost": df["Amount"].sum() if "Amount" in df.columns else 0.0,
+                    "account_data": df.groupby("Account")["Amount"].sum().to_dict() if "Account" in df.columns else {},
                 }
         except Exception as e:
             print_warning(f"Using mock FinOps data due to loading error: {e}")
@@ -1031,31 +1049,33 @@ class Comprehensive2WayValidator:
             mock_cost = float(os.getenv("MOCK_TOTAL_COST", "100.00"))
             current_period = datetime.now().strftime("%Y-%m")
             return {
-                'cost_breakdown': [{'Service': 'S3', 'Account': generic_account_id, 'Amount': mock_cost, 'Period': current_period}],
-                'total_cost': mock_cost,
-                'account_data': {generic_account_id: mock_cost}
+                "cost_breakdown": [
+                    {"Service": "S3", "Account": generic_account_id, "Amount": mock_cost, "Period": current_period}
+                ],
+                "total_cost": mock_cost,
+                "account_data": {generic_account_id: mock_cost},
             }
 
     async def _get_time_synchronized_cost_data(self, finops_data: Dict) -> Dict[str, Any]:
         """Get time-synchronized MCP cost data for enhanced accuracy."""
         print_info("🕐 Implementing enhanced time synchronization for MCP validation...")
-        
+
         # Time period synchronization (critical for 99.5% accuracy)
-        end_date = datetime.now().strftime('%Y-%m-%d')
-        start_date = (datetime.now() - timedelta(days=90)).strftime('%Y-%m-%d')
-        
+        end_date = datetime.now().strftime("%Y-%m-%d")
+        start_date = (datetime.now() - timedelta(days=90)).strftime("%Y-%m-%d")
+
         # Get MCP cost data with time alignment
         mcp_billing_client = MCPAWSClient(self.billing_profile)
-        
+
         # Enhanced time sync: align periods exactly
         cost_data = mcp_billing_client.get_cost_data_raw(start_date, end_date)
-        
+
         return {
-            'status': cost_data.get('status', 'unknown'),
-            'data': cost_data.get('data', {}),
-            'time_period': {'start': start_date, 'end': end_date},
-            'sync_timestamp': datetime.now().isoformat(),
-            'accuracy_enhancement': 'time_synchronized_periods'
+            "status": cost_data.get("status", "unknown"),
+            "data": cost_data.get("data", {}),
+            "time_period": {"start": start_date, "end": end_date},
+            "sync_timestamp": datetime.now().isoformat(),
+            "accuracy_enhancement": "time_synchronized_periods",
         }
 
     # Validation implementation methods
@@ -1063,17 +1083,17 @@ class Comprehensive2WayValidator:
         """Validate account discovery against MCP Organizations."""
         try:
             org_data = self.mcp_multi_account.management_client.get_organizations_data()
-            
-            inventory_accounts = len(inventory_data.get('account_summary', {}))
-            mcp_accounts = org_data.get('total_accounts', 0)
-            
+
+            inventory_accounts = len(inventory_data.get("account_summary", {}))
+            mcp_accounts = org_data.get("total_accounts", 0)
+
             if inventory_accounts == mcp_accounts:
-                return {'status': 'validated', 'message': 'Account discovery validated'}
+                return {"status": "validated", "message": "Account discovery validated"}
             else:
                 variance_pct = abs(inventory_accounts - mcp_accounts) / max(mcp_accounts, 1) * 100
                 return {
-                    'status': 'variance_detected',
-                    'discrepancy': ValidationDiscrepancy(
+                    "status": "variance_detected",
+                    "discrepancy": ValidationDiscrepancy(
                         source_name="inventory_module",
                         mcp_name="organizations_api",
                         field_name="account_count",
@@ -1082,114 +1102,116 @@ class Comprehensive2WayValidator:
                         variance_percentage=variance_pct,
                         severity="medium" if variance_pct < 20 else "high",
                         recommendation=f"Investigate account discovery logic - {variance_pct:.1f}% variance",
-                        business_impact="May affect multi-account reporting accuracy"
-                    )
+                        business_impact="May affect multi-account reporting accuracy",
+                    ),
                 }
         except Exception as e:
-            return {'status': 'validation_error', 'error': str(e)}
+            return {"status": "validation_error", "error": str(e)}
 
-    async def _validate_service_resources(self, service_type: str, inventory_data: Dict, account_scope: List[str]) -> Dict[str, Any]:
+    async def _validate_service_resources(
+        self, service_type: str, inventory_data: Dict, account_scope: List[str]
+    ) -> Dict[str, Any]:
         """Validate service resource counts."""
         # Real AWS service validation implementation required
         # Remove random simulation - use actual AWS API validation
         try:
             # TODO: Implement actual AWS service resource validation
             # This should validate against real AWS API responses
-            return {'status': 'validated', 'message': f'{service_type} resources validated'}
+            return {"status": "validated", "message": f"{service_type} resources validated"}
         except Exception as e:
             return {
-                'status': 'variance_detected',
-                'discrepancy': ValidationDiscrepancy(
+                "status": "variance_detected",
+                "discrepancy": ValidationDiscrepancy(
                     source_name="inventory_module",
                     mcp_name="aws_api_direct",
                     field_name=f"{service_type}_count",
-                    source_value=inventory_data['service_summary'].get(service_type, 0),
-                    mcp_value=inventory_data['service_summary'].get(service_type, 0) + 1,
+                    source_value=inventory_data["service_summary"].get(service_type, 0),
+                    mcp_value=inventory_data["service_summary"].get(service_type, 0) + 1,
                     variance_percentage=5.0,
                     severity="low",
                     recommendation=f"Minor {service_type} count variance detected",
-                    business_impact="Minimal impact on resource management"
-                )
+                    business_impact="Minimal impact on resource management",
+                ),
             }
 
     async def _validate_vpc_configuration(self, vpc: Dict) -> Dict[str, Any]:
         """
         Validate individual VPC configuration with enhanced accuracy.
-        
+
         Following proven patterns from Cost Explorer and Organizations fixes:
-        - Enhanced data structure validation 
+        - Enhanced data structure validation
         - Comprehensive accuracy scoring
         - Real validation logic instead of hardcoded responses
         """
-        vpc_id = vpc.get('VpcId', 'unknown')
-        
+        vpc_id = vpc.get("VpcId", "unknown")
+
         try:
             # Enhanced validation using multiple data points (following Cost Explorer pattern)
             validation_score = 0.0
             validation_checks = 0
             validation_details = {}
-            
+
             # Check 1: VPC ID format validation (critical for accuracy)
-            if vpc_id.startswith('vpc-') and len(vpc_id) >= 8:
+            if vpc_id.startswith("vpc-") and len(vpc_id) >= 8:
                 validation_score += 1.0
-                validation_details['vpc_id_valid'] = True
+                validation_details["vpc_id_valid"] = True
             else:
-                validation_details['vpc_id_valid'] = False
+                validation_details["vpc_id_valid"] = False
             validation_checks += 1
-            
+
             # Check 2: VPC state validation (enterprise requirement)
-            vpc_state = vpc.get('State', 'unknown')
-            if vpc_state in ['available', 'pending']:
+            vpc_state = vpc.get("State", "unknown")
+            if vpc_state in ["available", "pending"]:
                 validation_score += 1.0
-                validation_details['state_valid'] = True
-            elif vpc_state in ['unknown']:
+                validation_details["state_valid"] = True
+            elif vpc_state in ["unknown"]:
                 validation_score += 0.5  # Partial credit for missing data
-                validation_details['state_valid'] = 'partial'
+                validation_details["state_valid"] = "partial"
             else:
-                validation_details['state_valid'] = False
+                validation_details["state_valid"] = False
             validation_checks += 1
-            
+
             # Check 3: CIDR block validation (network configuration accuracy)
-            cidr_block = vpc.get('CidrBlock', '')
-            if cidr_block and '/' in cidr_block:
+            cidr_block = vpc.get("CidrBlock", "")
+            if cidr_block and "/" in cidr_block:
                 try:
                     # Basic CIDR format validation
-                    parts = cidr_block.split('/')
+                    parts = cidr_block.split("/")
                     if len(parts) == 2 and parts[1].isdigit():
                         subnet_bits = int(parts[1])
                         if 8 <= subnet_bits <= 32:  # Valid CIDR range
                             validation_score += 1.0
-                            validation_details['cidr_valid'] = True
+                            validation_details["cidr_valid"] = True
                         else:
                             validation_score += 0.7  # Partial credit for format
-                            validation_details['cidr_valid'] = 'partial'
+                            validation_details["cidr_valid"] = "partial"
                     else:
                         validation_score += 0.3  # Minimal credit for having CIDR
-                        validation_details['cidr_valid'] = 'format_error'
+                        validation_details["cidr_valid"] = "format_error"
                 except:
                     validation_score += 0.3  # Minimal credit for having CIDR
-                    validation_details['cidr_valid'] = 'parse_error'
+                    validation_details["cidr_valid"] = "parse_error"
             else:
-                validation_details['cidr_valid'] = False
+                validation_details["cidr_valid"] = False
             validation_checks += 1
-            
+
             # Check 4: Account ownership validation (security validation)
-            owner_id = vpc.get('OwnerId', '')
+            owner_id = vpc.get("OwnerId", "")
             if owner_id and owner_id.isdigit() and len(owner_id) == 12:
                 validation_score += 1.0
-                validation_details['owner_valid'] = True
+                validation_details["owner_valid"] = True
             elif owner_id:
                 validation_score += 0.5  # Partial credit for having owner
-                validation_details['owner_valid'] = 'partial'
+                validation_details["owner_valid"] = "partial"
             else:
-                validation_details['owner_valid'] = False
+                validation_details["owner_valid"] = False
             validation_checks += 1
-            
+
             # Check 5: VPC attributes validation (configuration completeness)
-            is_default = vpc.get('IsDefault', None)
-            dhcp_options_id = vpc.get('DhcpOptionsId', '')
-            instance_tenancy = vpc.get('InstanceTenancy', '')
-            
+            is_default = vpc.get("IsDefault", None)
+            dhcp_options_id = vpc.get("DhcpOptionsId", "")
+            instance_tenancy = vpc.get("InstanceTenancy", "")
+
             attributes_score = 0.0
             if is_default is not None:  # Boolean field present
                 attributes_score += 0.4
@@ -1197,46 +1219,46 @@ class Comprehensive2WayValidator:
                 attributes_score += 0.3
             if instance_tenancy:
                 attributes_score += 0.3
-            
+
             validation_score += attributes_score
-            validation_details['attributes_complete'] = attributes_score >= 0.8
+            validation_details["attributes_complete"] = attributes_score >= 0.8
             validation_checks += 1
-            
+
             # Check 6: Tags validation (governance and compliance)
-            tags = vpc.get('Tags', [])
+            tags = vpc.get("Tags", [])
             tags_score = 0.0
             if isinstance(tags, list):
                 if tags:  # Has tags
                     tags_score = 1.0
-                    validation_details['has_tags'] = True
+                    validation_details["has_tags"] = True
                     # Bonus for Name tag
-                    name_tag = any(tag.get('Key') == 'Name' for tag in tags)
+                    name_tag = any(tag.get("Key") == "Name" for tag in tags)
                     if name_tag:
                         tags_score = 1.0  # Full score for proper tagging
-                        validation_details['has_name_tag'] = True
+                        validation_details["has_name_tag"] = True
                     else:
-                        validation_details['has_name_tag'] = False
+                        validation_details["has_name_tag"] = False
                 else:
                     tags_score = 0.7  # Partial credit for empty but valid tags structure
-                    validation_details['has_tags'] = False
+                    validation_details["has_tags"] = False
             else:
-                validation_details['has_tags'] = False
-            
+                validation_details["has_tags"] = False
+
             validation_score += tags_score
             validation_checks += 1
-            
+
             # Calculate accuracy percentage (following proven accuracy pattern)
             accuracy_percentage = (validation_score / validation_checks) * 100
-            
+
             # Determine validation status based on accuracy (enterprise thresholds)
             if accuracy_percentage >= 95.0:
-                status = 'validated'
+                status = "validated"
                 message = f"VPC {vpc_id} validation passed with {accuracy_percentage:.1f}% accuracy"
             elif accuracy_percentage >= 80.0:
-                status = 'validated_with_warnings'
+                status = "validated_with_warnings"
                 message = f"VPC {vpc_id} validation passed with {accuracy_percentage:.1f}% accuracy (minor issues)"
             else:
-                status = 'validation_issues'
+                status = "validation_issues"
                 message = f"VPC {vpc_id} validation accuracy {accuracy_percentage:.1f}% below enterprise threshold"
                 # Create discrepancy for tracking
                 discrepancy = ValidationDiscrepancy(
@@ -1248,30 +1270,30 @@ class Comprehensive2WayValidator:
                     variance_percentage=100.0 - accuracy_percentage,
                     severity="medium" if accuracy_percentage >= 70.0 else "high",
                     recommendation=f"Improve VPC {vpc_id} configuration validation",
-                    business_impact="May affect network cost correlation accuracy"
+                    business_impact="May affect network cost correlation accuracy",
                 )
                 return {
-                    'status': status,
-                    'message': message,
-                    'accuracy_percentage': accuracy_percentage,
-                    'discrepancy': discrepancy,
-                    'validation_details': validation_details
+                    "status": status,
+                    "message": message,
+                    "accuracy_percentage": accuracy_percentage,
+                    "discrepancy": discrepancy,
+                    "validation_details": validation_details,
                 }
-            
+
             return {
-                'status': status,
-                'message': message,
-                'accuracy_percentage': accuracy_percentage,
-                'validation_details': validation_details
+                "status": status,
+                "message": message,
+                "accuracy_percentage": accuracy_percentage,
+                "validation_details": validation_details,
             }
-            
+
         except Exception as e:
             print_warning(f"VPC {vpc_id} validation error: {e}")
             return {
-                'status': 'validation_error',
-                'message': f"VPC {vpc_id} validation failed: {str(e)}",
-                'accuracy_percentage': 0.0,
-                'discrepancy': ValidationDiscrepancy(
+                "status": "validation_error",
+                "message": f"VPC {vpc_id} validation failed: {str(e)}",
+                "accuracy_percentage": 0.0,
+                "discrepancy": ValidationDiscrepancy(
                     source_name="vpc_module",
                     mcp_name="aws_ec2_api",
                     field_name=f"vpc_validation_{vpc_id}",
@@ -1280,15 +1302,15 @@ class Comprehensive2WayValidator:
                     variance_percentage=100.0,
                     severity="critical",
                     recommendation=f"Fix VPC {vpc_id} validation error: {str(e)}",
-                    business_impact="Critical validation failure affects accuracy"
+                    business_impact="Critical validation failure affects accuracy",
                 ),
-                'validation_details': {'error': str(e)}
+                "validation_details": {"error": str(e)},
             }
 
     async def _validate_vpc_cost_correlation(self, vpc_data: Dict) -> Dict[str, Any]:
         """
         Validate VPC cost correlation with FinOps data using enhanced accuracy patterns.
-        
+
         Following proven patterns from Cost Explorer and Organizations fixes:
         - Real correlation analysis instead of hardcoded responses
         - Enhanced accuracy calculation with multiple validation points
@@ -1299,88 +1321,88 @@ class Comprehensive2WayValidator:
             correlation_score = 0.0
             correlation_checks = 0
             validation_details = {}
-            
+
             # Check 1: VPC data structure validation
             if isinstance(vpc_data, dict) and vpc_data:
                 correlation_score += 1.0
-                validation_details['data_structure_valid'] = True
+                validation_details["data_structure_valid"] = True
             else:
-                validation_details['data_structure_valid'] = False
+                validation_details["data_structure_valid"] = False
             correlation_checks += 1
-            
+
             # Check 2: Cost-relevant VPC attributes presence
-            cost_relevant_attrs = ['VpcId', 'OwnerId', 'CidrBlock', 'State']
+            cost_relevant_attrs = ["VpcId", "OwnerId", "CidrBlock", "State"]
             present_attrs = sum(1 for attr in cost_relevant_attrs if vpc_data.get(attr))
-            
+
             if present_attrs == len(cost_relevant_attrs):
                 correlation_score += 1.0
-                validation_details['cost_attributes_complete'] = True
+                validation_details["cost_attributes_complete"] = True
             elif present_attrs >= len(cost_relevant_attrs) * 0.8:
                 correlation_score += 0.8
-                validation_details['cost_attributes_complete'] = 'partial'
+                validation_details["cost_attributes_complete"] = "partial"
             else:
-                validation_details['cost_attributes_complete'] = False
+                validation_details["cost_attributes_complete"] = False
             correlation_checks += 1
-            
+
             # Check 3: VPC resources for cost correlation (enhanced detection)
-            vpcs_list = vpc_data.get('vpcs', [])
+            vpcs_list = vpc_data.get("vpcs", [])
             if vpcs_list:
                 # Enhanced cost correlation analysis across all VPCs
                 total_cost_indicators = 0
                 vpcs_with_indicators = 0
-                
+
                 for vpc in vpcs_list:
-                    vpc_id = vpc.get('VpcId', '')
+                    vpc_id = vpc.get("VpcId", "")
                     potential_cost_indicators = []
-                    
+
                     # Check VPC ID pattern (cost-related services often have specific patterns)
                     if vpc_id:
-                        potential_cost_indicators.append('vpc_identity')
-                    
+                        potential_cost_indicators.append("vpc_identity")
+
                     # Check VPC state (active VPCs have cost implications)
-                    vpc_state = vpc.get('State', '')
-                    if vpc_state == 'available':
-                        potential_cost_indicators.append('active_vpc')
-                    
+                    vpc_state = vpc.get("State", "")
+                    if vpc_state == "available":
+                        potential_cost_indicators.append("active_vpc")
+
                     # Check CIDR block (larger networks may have more resources)
-                    cidr_block = vpc.get('CidrBlock', '')
+                    cidr_block = vpc.get("CidrBlock", "")
                     if cidr_block:
                         try:
-                            parts = cidr_block.split('/')
+                            parts = cidr_block.split("/")
                             if len(parts) == 2 and parts[1].isdigit():
                                 subnet_bits = int(parts[1])
                                 if subnet_bits <= 20:  # Larger networks
-                                    potential_cost_indicators.append('large_network')
+                                    potential_cost_indicators.append("large_network")
                                 else:
-                                    potential_cost_indicators.append('standard_network')
+                                    potential_cost_indicators.append("standard_network")
                         except:
-                            potential_cost_indicators.append('network_config')
-                    
+                            potential_cost_indicators.append("network_config")
+
                     # Check tenancy (dedicated instances have higher costs)
-                    tenancy = vpc.get('InstanceTenancy', '')
-                    if tenancy == 'dedicated':
-                        potential_cost_indicators.append('dedicated_tenancy')
-                    elif tenancy == 'default':
-                        potential_cost_indicators.append('shared_tenancy')
-                    
+                    tenancy = vpc.get("InstanceTenancy", "")
+                    if tenancy == "dedicated":
+                        potential_cost_indicators.append("dedicated_tenancy")
+                    elif tenancy == "default":
+                        potential_cost_indicators.append("shared_tenancy")
+
                     # Check tags (well-tagged resources often correlate with cost tracking)
-                    tags = vpc.get('Tags', [])
+                    tags = vpc.get("Tags", [])
                     if isinstance(tags, list) and tags:
-                        potential_cost_indicators.append('tagged_resource')
+                        potential_cost_indicators.append("tagged_resource")
                         # Look for cost-related tag keys
-                        tag_keys = [tag.get('Key', '').lower() for tag in tags]
-                        if any(key in tag_keys for key in ['cost', 'billing', 'project', 'environment']):
-                            potential_cost_indicators.append('cost_tracking_tags')
-                    
+                        tag_keys = [tag.get("Key", "").lower() for tag in tags]
+                        if any(key in tag_keys for key in ["cost", "billing", "project", "environment"]):
+                            potential_cost_indicators.append("cost_tracking_tags")
+
                     if potential_cost_indicators:
                         vpcs_with_indicators += 1
                         total_cost_indicators += len(potential_cost_indicators)
-                
+
                 # Calculate correlation score based on comprehensive analysis
                 if vpcs_with_indicators > 0:
                     vpc_coverage = vpcs_with_indicators / len(vpcs_list)
                     indicator_density = total_cost_indicators / len(vpcs_list)
-                    
+
                     # Score based on coverage and indicator density
                     if vpc_coverage >= 0.8 and indicator_density >= 3.0:
                         correlation_score += 1.0  # Excellent correlation
@@ -1390,62 +1412,62 @@ class Comprehensive2WayValidator:
                         correlation_score += 0.8  # Acceptable correlation
                     else:
                         correlation_score += 0.7  # Basic correlation
-                    
-                    validation_details['cost_indicators_present'] = {
-                        'vpcs_with_indicators': vpcs_with_indicators,
-                        'total_vpcs': len(vpcs_list),
-                        'coverage_percentage': vpc_coverage * 100,
-                        'average_indicators_per_vpc': indicator_density
+
+                    validation_details["cost_indicators_present"] = {
+                        "vpcs_with_indicators": vpcs_with_indicators,
+                        "total_vpcs": len(vpcs_list),
+                        "coverage_percentage": vpc_coverage * 100,
+                        "average_indicators_per_vpc": indicator_density,
                     }
                 else:
                     correlation_score += 0.5  # Minimal correlation
-                    validation_details['cost_indicators_present'] = {
-                        'vpcs_with_indicators': 0,
-                        'total_vpcs': len(vpcs_list),
-                        'coverage_percentage': 0.0,
-                        'average_indicators_per_vpc': 0.0
+                    validation_details["cost_indicators_present"] = {
+                        "vpcs_with_indicators": 0,
+                        "total_vpcs": len(vpcs_list),
+                        "coverage_percentage": 0.0,
+                        "average_indicators_per_vpc": 0.0,
                     }
             else:
                 # Check if VPC data structure itself indicates cost correlation potential
-                cost_impact = vpc_data.get('cost_impact', 0)
+                cost_impact = vpc_data.get("cost_impact", 0)
                 if cost_impact > 0:
                     correlation_score += 0.8  # Has cost impact data
-                    validation_details['cost_indicators_present'] = {'cost_impact_available': True}
+                    validation_details["cost_indicators_present"] = {"cost_impact_available": True}
                 else:
                     correlation_score += 0.3  # Minimal correlation without VPC data
-                    validation_details['cost_indicators_present'] = {'cost_impact_available': False}
-            
+                    validation_details["cost_indicators_present"] = {"cost_impact_available": False}
+
             correlation_checks += 1
-            
+
             # Check 4: Enhanced network topology and infrastructure indicators
             # Analyze overall infrastructure complexity for cost correlation
             infrastructure_score = 0.0
             infrastructure_indicators = []
-            
+
             # Check VPC-level cost factors
             if vpcs_list:
                 # Multi-VPC environment indicates higher complexity and costs
                 if len(vpcs_list) > 1:
                     infrastructure_score += 0.2
-                    infrastructure_indicators.append('multi_vpc_environment')
-                
+                    infrastructure_indicators.append("multi_vpc_environment")
+
                 # Analyze network topology complexity
                 total_network_capacity = 0
                 dedicated_tenancy_count = 0
                 well_tagged_count = 0
-                
+
                 for vpc in vpcs_list:
                     # Network size analysis
-                    cidr_block = vpc.get('CidrBlock', '')
+                    cidr_block = vpc.get("CidrBlock", "")
                     if cidr_block:
                         try:
-                            parts = cidr_block.split('/')
+                            parts = cidr_block.split("/")
                             if len(parts) == 2 and parts[1].isdigit():
                                 subnet_bits = int(parts[1])
                                 # Calculate potential IP capacity as cost indicator
                                 capacity = 2 ** (32 - subnet_bits)
                                 total_network_capacity += capacity
-                                
+
                                 if subnet_bits <= 16:  # Large networks
                                     infrastructure_score += 0.15
                                 elif subnet_bits <= 20:  # Medium-large networks
@@ -1454,86 +1476,90 @@ class Comprehensive2WayValidator:
                                     infrastructure_score += 0.05
                         except:
                             infrastructure_score += 0.02  # Minimal credit for having CIDR
-                    
+
                     # Tenancy model analysis
-                    tenancy = vpc.get('InstanceTenancy', '')
-                    if tenancy == 'dedicated':
+                    tenancy = vpc.get("InstanceTenancy", "")
+                    if tenancy == "dedicated":
                         dedicated_tenancy_count += 1
                         infrastructure_score += 0.1
-                    
+
                     # Governance and tracking analysis
-                    tags = vpc.get('Tags', [])
+                    tags = vpc.get("Tags", [])
                     if isinstance(tags, list) and len(tags) >= 2:
                         well_tagged_count += 1
                         infrastructure_score += 0.05
-                
+
                 # Infrastructure complexity bonuses
                 if total_network_capacity > 65536:  # > /16 network equivalent
                     infrastructure_score += 0.1
-                    infrastructure_indicators.append('large_network_capacity')
-                
+                    infrastructure_indicators.append("large_network_capacity")
+
                 if dedicated_tenancy_count > 0:
                     infrastructure_score += 0.1
-                    infrastructure_indicators.append('dedicated_tenancy_present')
-                
+                    infrastructure_indicators.append("dedicated_tenancy_present")
+
                 if well_tagged_count / len(vpcs_list) >= 0.8:  # 80%+ well-tagged
                     infrastructure_score += 0.1
-                    infrastructure_indicators.append('strong_governance')
-                
+                    infrastructure_indicators.append("strong_governance")
+
                 # Cost impact metadata bonus
-                cost_impact = vpc_data.get('cost_impact', 0)
+                cost_impact = vpc_data.get("cost_impact", 0)
                 if cost_impact > 0:
                     infrastructure_score += 0.15
-                    infrastructure_indicators.append('documented_cost_impact')
-                
+                    infrastructure_indicators.append("documented_cost_impact")
+
                 # Analysis metadata bonus (indicates professional assessment)
-                metadata = vpc_data.get('analysis_metadata', {})
+                metadata = vpc_data.get("analysis_metadata", {})
                 if metadata:
                     infrastructure_score += 0.1
-                    infrastructure_indicators.append('comprehensive_analysis')
-            
+                    infrastructure_indicators.append("comprehensive_analysis")
+
             # Normalize infrastructure score to 0-1 range
             infrastructure_score = min(1.0, infrastructure_score)
             correlation_score += infrastructure_score
-            
-            validation_details['infrastructure_complexity'] = {
-                'score': infrastructure_score,
-                'indicators': infrastructure_indicators,
-                'total_network_capacity': total_network_capacity if 'total_network_capacity' in locals() else 0,
-                'dedicated_tenancy_count': dedicated_tenancy_count if 'dedicated_tenancy_count' in locals() else 0,
-                'governance_coverage': (well_tagged_count / len(vpcs_list) * 100) if vpcs_list and 'well_tagged_count' in locals() else 0
+
+            validation_details["infrastructure_complexity"] = {
+                "score": infrastructure_score,
+                "indicators": infrastructure_indicators,
+                "total_network_capacity": total_network_capacity if "total_network_capacity" in locals() else 0,
+                "dedicated_tenancy_count": dedicated_tenancy_count if "dedicated_tenancy_count" in locals() else 0,
+                "governance_coverage": (well_tagged_count / len(vpcs_list) * 100)
+                if vpcs_list and "well_tagged_count" in locals()
+                else 0,
             }
-            
+
             correlation_checks += 1
-            
+
             # Check 5: VPC state impact on cost correlation
-            vpc_state = vpc_data.get('State', 'unknown')
-            if vpc_state == 'available':
+            vpc_state = vpc_data.get("State", "unknown")
+            if vpc_state == "available":
                 correlation_score += 1.0  # Active VPC, full cost correlation expected
-                validation_details['state_cost_impact'] = 'active'
-            elif vpc_state == 'pending':
+                validation_details["state_cost_impact"] = "active"
+            elif vpc_state == "pending":
                 correlation_score += 0.8  # Transitional state, partial correlation
-                validation_details['state_cost_impact'] = 'transitional'
-            elif vpc_state == 'deleting':
+                validation_details["state_cost_impact"] = "transitional"
+            elif vpc_state == "deleting":
                 correlation_score += 0.3  # Minimal correlation expected
-                validation_details['state_cost_impact'] = 'terminating'
+                validation_details["state_cost_impact"] = "terminating"
             else:
                 correlation_score += 0.1  # Unknown state, minimal correlation
-                validation_details['state_cost_impact'] = 'unknown'
+                validation_details["state_cost_impact"] = "unknown"
             correlation_checks += 1
-            
+
             # Calculate correlation accuracy percentage
             correlation_accuracy = (correlation_score / correlation_checks) * 100
-            
+
             # Determine validation status based on correlation accuracy
             if correlation_accuracy >= 95.0:
-                status = 'validated'
+                status = "validated"
                 message = f"VPC cost correlation validated with {correlation_accuracy:.1f}% accuracy"
             elif correlation_accuracy >= 80.0:
-                status = 'validated_with_warnings'
-                message = f"VPC cost correlation validated with {correlation_accuracy:.1f}% accuracy (minor correlation gaps)"
+                status = "validated_with_warnings"
+                message = (
+                    f"VPC cost correlation validated with {correlation_accuracy:.1f}% accuracy (minor correlation gaps)"
+                )
             else:
-                status = 'correlation_issues'
+                status = "correlation_issues"
                 message = f"VPC cost correlation accuracy {correlation_accuracy:.1f}% below enterprise threshold"
                 # Create discrepancy for tracking
                 discrepancy = ValidationDiscrepancy(
@@ -1545,29 +1571,29 @@ class Comprehensive2WayValidator:
                     variance_percentage=100.0 - correlation_accuracy,
                     severity="medium" if correlation_accuracy >= 70.0 else "high",
                     recommendation=f"Improve VPC cost correlation methodology for {vpc_data.get('VpcId', 'unknown')}",
-                    business_impact="May affect network cost optimization accuracy"
+                    business_impact="May affect network cost optimization accuracy",
                 )
                 return {
-                    'status': status,
-                    'message': message,
-                    'correlation_accuracy': correlation_accuracy,
-                    'discrepancy': discrepancy,
-                    'validation_details': validation_details
+                    "status": status,
+                    "message": message,
+                    "correlation_accuracy": correlation_accuracy,
+                    "discrepancy": discrepancy,
+                    "validation_details": validation_details,
                 }
-            
+
             return {
-                'status': status,
-                'message': message,
-                'correlation_accuracy': correlation_accuracy,
-                'validation_details': validation_details
+                "status": status,
+                "message": message,
+                "correlation_accuracy": correlation_accuracy,
+                "validation_details": validation_details,
             }
-            
+
         except Exception as e:
             return {
-                'status': 'correlation_error',
-                'message': f"VPC cost correlation validation failed: {str(e)}",
-                'correlation_accuracy': 0.0,
-                'discrepancy': ValidationDiscrepancy(
+                "status": "correlation_error",
+                "message": f"VPC cost correlation validation failed: {str(e)}",
+                "correlation_accuracy": 0.0,
+                "discrepancy": ValidationDiscrepancy(
                     source_name="vpc_module",
                     mcp_name="finops_cost_explorer",
                     field_name="vpc_cost_correlation",
@@ -1576,38 +1602,38 @@ class Comprehensive2WayValidator:
                     variance_percentage=100.0,
                     severity="critical",
                     recommendation=f"Fix VPC cost correlation validation error: {str(e)}",
-                    business_impact="Critical correlation failure affects cost optimization"
+                    business_impact="Critical correlation failure affects cost optimization",
                 ),
-                'validation_details': {'error': str(e)}
+                "validation_details": {"error": str(e)},
             }
 
     async def _validate_total_cost_with_time_sync(self, finops_data: Dict, mcp_data: Dict) -> Dict[str, Any]:
         """Validate total cost with enhanced time synchronization."""
-        if mcp_data.get('status') != 'success':
-            return {'status': 'mcp_unavailable', 'message': 'MCP Cost Explorer unavailable'}
-        
-        finops_total = finops_data.get('total_cost', 0.0)
-        
+        if mcp_data.get("status") != "success":
+            return {"status": "mcp_unavailable", "message": "MCP Cost Explorer unavailable"}
+
+        finops_total = finops_data.get("total_cost", 0.0)
+
         # Calculate MCP total with time sync
         mcp_total = 0.0
-        mcp_results = mcp_data.get('data', {}).get('ResultsByTime', [])
-        
+        mcp_results = mcp_data.get("data", {}).get("ResultsByTime", [])
+
         for result in mcp_results:
-            if 'Groups' in result:
-                for group in result['Groups']:
-                    mcp_total += float(group['Metrics']['BlendedCost']['Amount'])
+            if "Groups" in result:
+                for group in result["Groups"]:
+                    mcp_total += float(group["Metrics"]["BlendedCost"]["Amount"])
             else:
-                mcp_total += float(result['Total']['BlendedCost']['Amount'])
-        
+                mcp_total += float(result["Total"]["BlendedCost"]["Amount"])
+
         if finops_total > 0:
             variance_pct = abs(finops_total - mcp_total) / finops_total * 100
-            
+
             if variance_pct <= 5.0:  # Enhanced tolerance for accuracy
-                return {'status': 'validated', 'message': f'Total cost validated: {variance_pct:.1f}% variance'}
+                return {"status": "validated", "message": f"Total cost validated: {variance_pct:.1f}% variance"}
             else:
                 return {
-                    'status': 'variance_detected',
-                    'discrepancy': ValidationDiscrepancy(
+                    "status": "variance_detected",
+                    "discrepancy": ValidationDiscrepancy(
                         source_name="finops_module",
                         mcp_name="cost_explorer_api",
                         field_name="total_monthly_cost",
@@ -1616,160 +1642,170 @@ class Comprehensive2WayValidator:
                         variance_percentage=variance_pct,
                         severity="high" if variance_pct > 20 else "medium",
                         recommendation=f"Investigate cost calculation discrepancy: {variance_pct:.1f}% variance",
-                        business_impact=f"Potential ${abs(finops_total - mcp_total):,.2f} reporting discrepancy"
-                    )
+                        business_impact=f"Potential ${abs(finops_total - mcp_total):,.2f} reporting discrepancy",
+                    ),
                 }
-        
-        return {'status': 'insufficient_data', 'message': 'Insufficient cost data for validation'}
+
+        return {"status": "insufficient_data", "message": "Insufficient cost data for validation"}
 
     async def _validate_service_breakdown_accuracy(self, finops_data: Dict, mcp_data: Dict) -> Dict[str, Any]:
         """Validate service-level cost breakdown accuracy."""
-        return {'status': 'validated', 'message': 'Service breakdown validated'}
+        return {"status": "validated", "message": "Service breakdown validated"}
 
     async def _validate_account_cost_distribution(self, finops_data: Dict, mcp_data: Dict) -> Dict[str, Any]:
         """Validate account-level cost distribution."""
-        return {'status': 'validated', 'message': 'Account cost distribution validated'}
+        return {"status": "validated", "message": "Account cost distribution validated"}
 
     async def _validate_quarterly_intelligence(self, finops_data: Dict, mcp_data: Dict) -> Dict[str, Any]:
         """Validate quarterly intelligence integration."""
-        return {'status': 'validated', 'message': 'Quarterly intelligence validated'}
+        return {"status": "validated", "message": "Quarterly intelligence validated"}
 
     async def _validate_cost_optimization_accuracy(self, finops_data: Dict, mcp_data: Dict) -> Dict[str, Any]:
         """Validate cost optimization recommendation accuracy."""
-        return {'status': 'validated', 'message': 'Cost optimization accuracy validated'}
+        return {"status": "validated", "message": "Cost optimization accuracy validated"}
 
     # Evidence generation methods
-    async def _generate_inventory_evidence(self, validation_id: str, inventory_data: Dict, 
-                                         discrepancies: List, accuracy: float) -> List[str]:
+    async def _generate_inventory_evidence(
+        self, validation_id: str, inventory_data: Dict, discrepancies: List, accuracy: float
+    ) -> List[str]:
         """Generate inventory validation evidence."""
         evidence_files = []
         timestamp = datetime.now().strftime("%Y%m%d_%H%M%S")
-        
+
         # JSON evidence
         json_evidence = {
-            'validation_id': validation_id,
-            'module': 'inventory',
-            'timestamp': timestamp,
-            'accuracy_percentage': accuracy,
-            'inventory_summary': inventory_data,
-            'discrepancies': [asdict(d) for d in discrepancies],
-            'enterprise_compliance': {
-                'accuracy_target_met': accuracy >= self.accuracy_target,
-                'evidence_generated': True,
-                'audit_trail': 'complete'
-            }
+            "validation_id": validation_id,
+            "module": "inventory",
+            "timestamp": timestamp,
+            "accuracy_percentage": accuracy,
+            "inventory_summary": inventory_data,
+            "discrepancies": [asdict(d) for d in discrepancies],
+            "enterprise_compliance": {
+                "accuracy_target_met": accuracy >= self.accuracy_target,
+                "evidence_generated": True,
+                "audit_trail": "complete",
+            },
         }
-        
+
         json_path = self.evidence_dir / f"inventory_validation_{timestamp}.json"
-        with open(json_path, 'w') as f:
+        with open(json_path, "w") as f:
             json.dump(json_evidence, f, indent=2, default=str)
         evidence_files.append(str(json_path))
-        
+
         return evidence_files
 
-    async def _generate_vpc_evidence(self, validation_id: str, vpc_data: Dict, 
-                                   discrepancies: List, accuracy: float) -> List[str]:
+    async def _generate_vpc_evidence(
+        self, validation_id: str, vpc_data: Dict, discrepancies: List, accuracy: float
+    ) -> List[str]:
         """Generate VPC validation evidence."""
         evidence_files = []
         timestamp = datetime.now().strftime("%Y%m%d_%H%M%S")
-        
+
         json_evidence = {
-            'validation_id': validation_id,
-            'module': 'vpc',
-            'timestamp': timestamp,
-            'accuracy_percentage': accuracy,
-            'vpc_summary': vpc_data,
-            'discrepancies': [asdict(d) for d in discrepancies]
+            "validation_id": validation_id,
+            "module": "vpc",
+            "timestamp": timestamp,
+            "accuracy_percentage": accuracy,
+            "vpc_summary": vpc_data,
+            "discrepancies": [asdict(d) for d in discrepancies],
         }
-        
+
         json_path = self.evidence_dir / f"vpc_validation_{timestamp}.json"
-        with open(json_path, 'w') as f:
+        with open(json_path, "w") as f:
             json.dump(json_evidence, f, indent=2, default=str)
         evidence_files.append(str(json_path))
-        
+
         return evidence_files
 
-    async def _generate_finops_evidence(self, validation_id: str, finops_data: Dict, 
-                                      mcp_data: Dict, discrepancies: List, accuracy: float) -> List[str]:
+    async def _generate_finops_evidence(
+        self, validation_id: str, finops_data: Dict, mcp_data: Dict, discrepancies: List, accuracy: float
+    ) -> List[str]:
         """Generate comprehensive FinOps validation evidence."""
         evidence_files = []
         timestamp = datetime.now().strftime("%Y%m%d_%H%M%S")
-        
+
         # Enhanced FinOps evidence with MCP cross-validation
         json_evidence = {
-            'validation_id': validation_id,
-            'module': 'finops',
-            'timestamp': timestamp,
-            'accuracy_percentage': accuracy,
-            'accuracy_improvement': '0.0% → ≥99.5% target implementation',
-            'finops_summary': finops_data,
-            'mcp_validation_data': mcp_data,
-            'discrepancies': [asdict(d) for d in discrepancies],
-            'time_synchronization': {
-                'enabled': True,
-                'method': 'enhanced_period_alignment',
-                'accuracy_impact': 'critical_for_enterprise_target'
+            "validation_id": validation_id,
+            "module": "finops",
+            "timestamp": timestamp,
+            "accuracy_percentage": accuracy,
+            "accuracy_improvement": "0.0% → ≥99.5% target implementation",
+            "finops_summary": finops_data,
+            "mcp_validation_data": mcp_data,
+            "discrepancies": [asdict(d) for d in discrepancies],
+            "time_synchronization": {
+                "enabled": True,
+                "method": "enhanced_period_alignment",
+                "accuracy_impact": "critical_for_enterprise_target",
+            },
+            "enterprise_compliance": {
+                "accuracy_target_met": accuracy >= self.accuracy_target,
+                "mcp_integration": mcp_data.get("status") == "success",
+                "evidence_generated": True,
+                "audit_trail": "comprehensive",
             },
-            'enterprise_compliance': {
-                'accuracy_target_met': accuracy >= self.accuracy_target,
-                'mcp_integration': mcp_data.get('status') == 'success',
-                'evidence_generated': True,
-                'audit_trail': 'comprehensive'
-            }
         }
-        
+
         json_path = self.evidence_dir / f"finops_validation_{timestamp}.json"
-        with open(json_path, 'w') as f:
+        with open(json_path, "w") as f:
             json.dump(json_evidence, f, indent=2, default=str)
         evidence_files.append(str(json_path))
-        
+
         # CSV summary for business stakeholders
         csv_path = self.evidence_dir / f"finops_validation_summary_{timestamp}.csv"
-        with open(csv_path, 'w') as f:
-            f.write("Validation_ID,Module,Accuracy_Percentage,Target_Met,Discrepancies,Cost_Impact,Business_Confidence\n")
-            f.write(f"{validation_id},finops,{accuracy:.1f}%,{'YES' if accuracy >= self.accuracy_target else 'NO'},{len(discrepancies)},${sum(abs(d.source_value - d.mcp_value) for d in discrepancies if isinstance(d.source_value, (int, float)) and isinstance(d.mcp_value, (int, float))):.2f},{'HIGH' if accuracy >= 95 else 'MEDIUM' if accuracy >= 85 else 'LOW'}\n")
+        with open(csv_path, "w") as f:
+            f.write(
+                "Validation_ID,Module,Accuracy_Percentage,Target_Met,Discrepancies,Cost_Impact,Business_Confidence\n"
+            )
+            f.write(
+                f"{validation_id},finops,{accuracy:.1f}%,{'YES' if accuracy >= self.accuracy_target else 'NO'},{len(discrepancies)},${sum(abs(d.source_value - d.mcp_value) for d in discrepancies if isinstance(d.source_value, (int, float)) and isinstance(d.mcp_value, (int, float))):.2f},{'HIGH' if accuracy >= 95 else 'MEDIUM' if accuracy >= 85 else 'LOW'}\n"
+            )
         evidence_files.append(str(csv_path))
-        
+
         return evidence_files
 
     async def _generate_suite_report(self, suite_results: Dict, execution_time: float) -> str:
         """Generate comprehensive suite validation report."""
         timestamp = datetime.now().strftime("%Y%m%d_%H%M%S")
         report_path = self.evidence_dir / f"comprehensive_validation_suite_{timestamp}.json"
-        
+
         comprehensive_report = {
             **suite_results,
-            'execution_metadata': {
-                'total_execution_time_seconds': execution_time,
-                'validation_system_version': '1.0.0',
-                'enterprise_framework': 'FAANG_SDLC_compliant',
-                'accuracy_target': self.accuracy_target,
-                'performance_target': self.performance_target_seconds
+            "execution_metadata": {
+                "total_execution_time_seconds": execution_time,
+                "validation_system_version": "1.0.0",
+                "enterprise_framework": "FAANG_SDLC_compliant",
+                "accuracy_target": self.accuracy_target,
+                "performance_target": self.performance_target_seconds,
+            },
+            "enterprise_assessment": {
+                "stakeholder_ready": suite_results.get("enterprise_target_met", False),
+                "compliance_documentation": "complete",
+                "audit_trail": "comprehensive",
+                "business_confidence": "high" if suite_results.get("overall_accuracy", 0) >= 95 else "medium",
             },
-            'enterprise_assessment': {
-                'stakeholder_ready': suite_results.get('enterprise_target_met', False),
-                'compliance_documentation': 'complete',
-                'audit_trail': 'comprehensive',
-                'business_confidence': 'high' if suite_results.get('overall_accuracy', 0) >= 95 else 'medium'
-            }
         }
-        
-        with open(report_path, 'w') as f:
+
+        with open(report_path, "w") as f:
             json.dump(comprehensive_report, f, indent=2, default=str)
-        
+
         print_success(f"📊 Comprehensive validation report: {report_path}")
         return str(report_path)
 
     # Business analysis methods
     def _assess_inventory_cost_impact(self, discrepancies: List[ValidationDiscrepancy]) -> float:
         """Assess cost impact of inventory discrepancies."""
-        return sum(abs(d.source_value - d.mcp_value) for d in discrepancies 
-                  if isinstance(d.source_value, (int, float)) and isinstance(d.mcp_value, (int, float)))
+        return sum(
+            abs(d.source_value - d.mcp_value)
+            for d in discrepancies
+            if isinstance(d.source_value, (int, float)) and isinstance(d.mcp_value, (int, float))
+        )
 
     def _assess_vpc_cost_impact(self, discrepancies: List[ValidationDiscrepancy]) -> float:
         """Assess cost impact of VPC discrepancies."""
         # VPC discrepancies could have significant network cost implications
-        return sum(100.0 for d in discrepancies if d.severity in ['high', 'critical'])
+        return sum(100.0 for d in discrepancies if d.severity in ["high", "critical"])
 
     def _assess_finops_cost_impact(self, discrepancies: List[ValidationDiscrepancy]) -> float:
         """Assess cost impact of FinOps discrepancies."""
@@ -1797,56 +1833,53 @@ class Comprehensive2WayValidator:
     def _calculate_stakeholder_confidence(self, accuracy: float, risk_level: str) -> float:
         """Calculate stakeholder confidence score."""
         base_score = accuracy / 100.0
-        
-        risk_adjustments = {
-            "low": 0.0,
-            "medium": -0.1,
-            "high": -0.2,
-            "critical": -0.4
-        }
-        
+
+        risk_adjustments = {"low": 0.0, "medium": -0.1, "high": -0.2, "critical": -0.4}
+
         return max(0.0, min(1.0, base_score + risk_adjustments.get(risk_level, -0.2)))
 
     # Recommendation generation methods
-    def _generate_inventory_recommendations(self, accuracy: float, discrepancies: List, performance_met: bool) -> List[str]:
+    def _generate_inventory_recommendations(
+        self, accuracy: float, discrepancies: List, performance_met: bool
+    ) -> List[str]:
         """Generate inventory-specific recommendations."""
         recommendations = []
-        
+
         if accuracy >= self.accuracy_target:
             recommendations.append("✅ Inventory validation passed enterprise standards")
             recommendations.append("📊 Inventory data suitable for stakeholder reporting")
         else:
             recommendations.append("⚠️ Inventory accuracy below enterprise target - investigate discrepancies")
             recommendations.append("🔍 Review account discovery and resource enumeration logic")
-        
+
         if not performance_met:
             recommendations.append("⚡ Consider optimization for enterprise performance targets")
-        
+
         if discrepancies:
             recommendations.append(f"🔧 Address {len(discrepancies)} validation discrepancies for improved accuracy")
-        
+
         return recommendations
 
     def _generate_vpc_recommendations(self, accuracy: float, discrepancies: List) -> List[str]:
         """Generate VPC-specific recommendations."""
         recommendations = []
-        
+
         if accuracy >= self.accuracy_target:
             recommendations.append("✅ VPC validation meets enterprise accuracy standards")
             recommendations.append("🌐 Network cost correlation validated for financial reporting")
         else:
             recommendations.append("⚠️ VPC validation requires attention - network cost implications")
             recommendations.append("💰 Review VPC cost attribution and optimization logic")
-        
+
         if discrepancies:
             recommendations.append("🔧 Address VPC configuration discrepancies for network accuracy")
-        
+
         return recommendations
 
     def _generate_finops_recommendations(self, accuracy: float, discrepancies: List) -> List[str]:
         """Generate FinOps-specific recommendations."""
         recommendations = []
-        
+
         if accuracy >= self.accuracy_target:
             recommendations.append("✅ FinOps MCP validation achieved enterprise target!")
             recommendations.append("📈 Cost analysis ready for executive presentation")
@@ -1855,41 +1888,49 @@ class Comprehensive2WayValidator:
             recommendations.append("⚠️ FinOps accuracy below target - implement time synchronization")
             recommendations.append("🕐 Review MCP Cost Explorer integration for period alignment")
             recommendations.append("💰 Validate cost calculation methodology against AWS APIs")
-        
+
         if discrepancies:
             recommendations.append("🔧 Address cost calculation discrepancies for financial accuracy")
             recommendations.append("📊 Review quarterly intelligence integration for strategic reporting")
-        
+
         return recommendations
 
     def _consolidate_recommendations(self, results: List[Comprehensive2WayValidationResult]) -> List[str]:
         """Consolidate recommendations across all validation results."""
         all_recommendations = []
-        
+
         # Add enterprise-level recommendations
         overall_accuracy = sum(r.validation_accuracy_percentage for r in results) / len(results) if results else 0
-        
+
         if overall_accuracy >= self.accuracy_target:
             all_recommendations.append("🏆 ENTERPRISE SUCCESS: Overall validation accuracy meets enterprise target")
             all_recommendations.append("📊 All modules ready for stakeholder presentation")
         else:
             all_recommendations.append("⚠️ ENTERPRISE ATTENTION: Overall accuracy requires improvement")
             all_recommendations.append("🎯 Focus on modules below enterprise accuracy threshold")
-        
+
         # Add module-specific top recommendations
         for result in results:
             if result.recommendations:
                 all_recommendations.extend(result.recommendations[:2])  # Top 2 per module
-        
+
         return list(set(all_recommendations))  # Remove duplicates
 
     def _consolidate_business_impact(self, results: List[Comprehensive2WayValidationResult]) -> Dict[str, Any]:
         """Consolidate business impact analysis."""
         return {
-            'total_estimated_cost_impact': sum(r.estimated_cost_impact for r in results),
-            'highest_risk_module': max(results, key=lambda r: {'low': 1, 'medium': 2, 'high': 3, 'critical': 4}.get(r.risk_level, 0)).module_name if results else None,
-            'average_stakeholder_confidence': sum(r.stakeholder_confidence_score for r in results) / len(results) if results else 0,
-            'modules_requiring_attention': [r.module_name for r in results if r.validation_accuracy_percentage < self.accuracy_target]
+            "total_estimated_cost_impact": sum(r.estimated_cost_impact for r in results),
+            "highest_risk_module": max(
+                results, key=lambda r: {"low": 1, "medium": 2, "high": 3, "critical": 4}.get(r.risk_level, 0)
+            ).module_name
+            if results
+            else None,
+            "average_stakeholder_confidence": sum(r.stakeholder_confidence_score for r in results) / len(results)
+            if results
+            else 0,
+            "modules_requiring_attention": [
+                r.module_name for r in results if r.validation_accuracy_percentage < self.accuracy_target
+            ],
         }
 
     # Infrastructure drift detection
@@ -1913,40 +1954,43 @@ class Comprehensive2WayValidator:
         if not self.validation_sessions:
             print_warning("No validation sessions available for export")
             return ""
-        
+
         timestamp = datetime.now().strftime("%Y%m%d_%H%M%S")
-        
+
         if output_format.lower() == "json":
             report_path = self.evidence_dir / f"stakeholder_validation_report_{timestamp}.json"
-            
+
             stakeholder_report = {
-                'report_metadata': {
-                    'generated_timestamp': datetime.now().isoformat(),
-                    'validation_system': 'Comprehensive 2-Way Validator',
-                    'version': '1.0.0',
-                    'enterprise_compliance': True
+                "report_metadata": {
+                    "generated_timestamp": datetime.now().isoformat(),
+                    "validation_system": "Comprehensive 2-Way Validator",
+                    "version": "1.0.0",
+                    "enterprise_compliance": True,
                 },
-                'executive_summary': {
-                    'total_validations': len(self.validation_sessions),
-                    'overall_accuracy': sum(s.validation_accuracy_percentage for s in self.validation_sessions) / len(self.validation_sessions),
-                    'enterprise_target_met': all(s.validation_accuracy_percentage >= self.accuracy_target for s in self.validation_sessions),
-                    'modules_validated': [s.module_name for s in self.validation_sessions]
+                "executive_summary": {
+                    "total_validations": len(self.validation_sessions),
+                    "overall_accuracy": sum(s.validation_accuracy_percentage for s in self.validation_sessions)
+                    / len(self.validation_sessions),
+                    "enterprise_target_met": all(
+                        s.validation_accuracy_percentage >= self.accuracy_target for s in self.validation_sessions
+                    ),
+                    "modules_validated": [s.module_name for s in self.validation_sessions],
+                },
+                "detailed_results": [asdict(session) for session in self.validation_sessions],
+                "business_recommendations": self._consolidate_recommendations(self.validation_sessions),
+                "compliance_attestation": {
+                    "sox_compliance": True,
+                    "audit_trail": "comprehensive",
+                    "evidence_collection": "complete",
                 },
-                'detailed_results': [asdict(session) for session in self.validation_sessions],
-                'business_recommendations': self._consolidate_recommendations(self.validation_sessions),
-                'compliance_attestation': {
-                    'sox_compliance': True,
-                    'audit_trail': 'comprehensive',
-                    'evidence_collection': 'complete'
-                }
             }
-            
-            with open(report_path, 'w') as f:
+
+            with open(report_path, "w") as f:
                 json.dump(stakeholder_report, f, indent=2, default=str)
-            
+
             print_success(f"📊 Stakeholder report exported: {report_path}")
             return str(report_path)
-        
+
         else:
             print_error(f"Unsupported export format: {output_format}")
             return ""
@@ -1956,10 +2000,8 @@ class Comprehensive2WayValidator:
 async def main():
     """Main CLI interface for comprehensive validation."""
     import argparse
-    
-    parser = argparse.ArgumentParser(
-        description="Comprehensive 2-Way Validation System - Enterprise MCP Integration"
-    )
+
+    parser = argparse.ArgumentParser(description="Comprehensive 2-Way Validation System - Enterprise MCP Integration")
     parser.add_argument("--inventory-csv", help="Path to inventory CSV export")
     parser.add_argument("--vpc-analysis", help="Path to VPC analysis results")
     parser.add_argument("--finops-export", help="Path to FinOps export data")
@@ -1967,38 +2009,37 @@ async def main():
     parser.add_argument("--performance-target", type=float, default=30.0, help="Performance target in seconds")
     parser.add_argument("--export-report", choices=["json"], default="json", help="Export stakeholder report format")
     parser.add_argument("--run-full-suite", action="store_true", help="Run comprehensive validation suite")
-    
+
     args = parser.parse_args()
-    
+
     # Initialize validator
     validator = Comprehensive2WayValidator(
-        accuracy_target=args.accuracy_target,
-        performance_target_seconds=args.performance_target
+        accuracy_target=args.accuracy_target, performance_target_seconds=args.performance_target
     )
-    
+
     if args.run_full_suite:
         print_header("Enterprise 2-Way Validation Suite", "Full Execution")
-        
+
         # Run comprehensive validation suite
         suite_results = await validator.run_comprehensive_validation_suite(
-            inventory_csv=args.inventory_csv,
-            vpc_analysis=args.vpc_analysis,
-            finops_export=args.finops_export
+            inventory_csv=args.inventory_csv, vpc_analysis=args.vpc_analysis, finops_export=args.finops_export
         )
-        
+
         # Export stakeholder report
         report_path = await validator.export_stakeholder_report(args.export_report)
-        
-        if suite_results['enterprise_target_met']:
+
+        if suite_results["enterprise_target_met"]:
             print_success("🏆 ENTERPRISE VALIDATION COMPLETE: All targets met!")
-            print_success(f"📊 Overall Accuracy: {suite_results['overall_accuracy']:.1f}% (≥{args.accuracy_target}% target)")
+            print_success(
+                f"📊 Overall Accuracy: {suite_results['overall_accuracy']:.1f}% (≥{args.accuracy_target}% target)"
+            )
         else:
             print_warning("⚠️ ENTERPRISE ATTENTION: Review validation results")
             print_info("🔧 Implement recommendations to achieve enterprise targets")
-        
+
         if report_path:
             print_success(f"📄 Stakeholder report ready: {report_path}")
-    
+
     else:
         # Run individual module validations
         print_info("💡 Use --run-full-suite for comprehensive enterprise validation")
@@ -2009,4 +2050,4 @@ async def main():
 
 
 if __name__ == "__main__":
-    asyncio.run(main())
+    asyncio.run(main())