AC-15344:: Add change level to breaking change table report in SVC

Author: cod62627

src/Reporter/BreakingChangeTableReporter.php

@@ -153,110 +153,9 @@ private function isPrivateChange($operation): bool
     {
         $target = $operation->getTarget();
         $reason = $operation->getReason();
-        $operationClass = get_class($operation);
-        
-        // For visibility operations, check if they involve private visibility
-        if ($operation instanceof \Magento\SemanticVersionChecker\Operation\VisibilityOperation) {
-            try {
-                // Use reflection to access protected properties
-                $reflection = new \ReflectionClass($operation);
-                
-                if ($reflection->hasProperty('memberBefore')) {
-                    $memberBeforeProperty = $reflection->getProperty('memberBefore');
-                    $memberBeforeProperty->setAccessible(true);
-                    $memberBefore = $memberBeforeProperty->getValue($operation);
-                    
-                    if ($memberBefore && method_exists('\PHPSemVerChecker\Operation\Visibility', 'getForContext')) {
-                        $visibilityBefore = \PHPSemVerChecker\Operation\Visibility::getForContext($memberBefore);
-                        // 1 = public, 2 = protected, 3 = private
-                        if ($visibilityBefore === 3) {
-                            return true;
-                        }
-                    }
-                }
-                
-                if ($reflection->hasProperty('memberAfter')) {
-                    $memberAfterProperty = $reflection->getProperty('memberAfter');
-                    $memberAfterProperty->setAccessible(true);
-                    $memberAfter = $memberAfterProperty->getValue($operation);
-                    
-                    if ($memberAfter && method_exists('\PHPSemVerChecker\Operation\Visibility', 'getForContext')) {
-                        $visibilityAfter = \PHPSemVerChecker\Operation\Visibility::getForContext($memberAfter);
-                        // 1 = public, 2 = protected, 3 = private
-                        if ($visibilityAfter === 3) {
-                            return true;
-                        }
-                    }
-                }
-            } catch (\Exception $e) {
-                // Fall back to string matching if reflection fails
-            }
-        }
-        
-        // Check if the reason explicitly mentions private visibility
-        if (preg_match('/\[private\]/', $reason)) {
-            return true;
-        }
-        
-        // Check if the target or reason indicates a private method/property
-        $privateIndicators = [
-            'private method',
-            'private property', 
-            'Private method',
-            'Private property',
-            '::private',
-            ' private ',
-            'private function',
-            'private static',
-            'visibility has been changed to lower lever from private',
-            'visibility has been changed to higher lever from private', 
-            'visibility has been changed from private',
-            'visibility has been changed to private',
-            'Method visibility has been changed from public to private',
-            'Method visibility has been changed from protected to private',
-            'Property visibility has been changed from public to private',
-            'Property visibility has been changed from protected to private',
-        ];
-        
-        foreach ($privateIndicators as $indicator) {
-            if (stripos($target, $indicator) !== false || stripos($reason, $indicator) !== false) {
-                return true;
-            }
-        }
-        
-        // Check for visibility operations that involve private members
-        if (strpos($operationClass, 'Visibility') !== false) {
-            // For visibility operations, check if it involves changing from/to private
-            if (stripos($reason, 'private') !== false) {
-                return true;
-            }
-        }
-        
-        // Check for specific operation classes that handle private changes
-        $privateOperationClasses = [
-            'PrivateMethod',
-            'PrivateProperty', 
-            'Private',
-        ];
-        
-        foreach ($privateOperationClasses as $privateClass) {
-            if (stripos($operationClass, $privateClass) !== false) {
-                return true;
-            }
-        }
-        
-        // Check if the target contains patterns that suggest private members
-        // Pattern: ClassName::privateMethodName or ClassName::$privateProperty
-        if (preg_match('/::([a-z_][a-zA-Z0-9_]*|\$[a-z_][a-zA-Z0-9_]*)/', $target, $matches)) {
-            $memberName = $matches[1];
-            // If member name starts with underscore (common private naming convention)
-            if (strpos($memberName, '_') === 0) {
-                return true;
-            }
-        }
-        
-        // Check for common private method patterns in the target
-        if (preg_match('/::(_[a-zA-Z0-9_]+|[a-z][a-zA-Z0-9]*Private[a-zA-Z0-9]*)\(/', $target)) {
+
+        // Simple string check for 'private' keyword (covers all cases)
+        if (stripos($target, 'private') !== false || stripos($reason, 'private') !== false) {
             return true;
         }