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

Author: cod62627

src/Reporter/BreakingChangeTableReporter.php

@@ -106,6 +106,11 @@ protected function outputTable(OutputInterface $output, Report $report, $context
             $reportForLevel = $report[$context][$level];
             /** @var \PHPSemVerChecker\Operation\Operation $operation */
             foreach ($reportForLevel as $operation) {
+                // Skip private method/property changes as they shouldn't be in breaking change reports
+                if ($this->isPrivateChange($operation)) {
+                    continue;
+                }
+                
                 $levelLabel = $this->getLevelLabel($level);
                 $target = $operation->getTarget();
                 $reason = $operation->getReason();
@@ -136,6 +141,128 @@ private function getLevelLabel(int $level): string
         }
     }
 
+    /**
+     * Check if the operation represents a private method or property change
+     * 
+     * Private changes are filtered out as they don't affect the public API contract.
+     *
+     * @param \PHPSemVerChecker\Operation\Operation $operation
+     * @return bool
+     */
+    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)) {
+            return true;
+        }
+        
+        return false;
+    }
+
     /**
      * Generate the HTML header line for a report section
      *