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

Author: cod62627

src/Analyzer/ClassMethodAnalyzer.php

@@ -259,7 +259,8 @@ protected function reportChanged($report, $contextBefore, $contextAfter, $method
                     $report->add($this->context, $data);
                     $signatureChanged = true;
                 } elseif ($signatureChanges['parameter_typing_changed']) {
-                    if ($signatureChanges['parameter_nullable_type_added']) {
+                    if ($signatureChanges['parameter_nullable_type_added']
+                        || $signatureChanges['parameter_nullable_type_removed']) {
                         $data = new ClassMethodParameterTypingChangedNullable(
                             $this->context,
                             $this->fileAfter,
@@ -465,8 +466,7 @@ private function isReturnsEqualByNullability(ClassMethod $before, ClassMethod $a
      */
     private function getDocReturnDeclaration(ClassMethod $method)
     {
-        if (
-            ($parsedComment = $method->getAttribute('docCommentParsed'))
+        if (($parsedComment = $method->getAttribute('docCommentParsed'))
             && isset($parsedComment['return'])
         ) {
             if ($parsedComment['return'][0] instanceof NullableType) {

src/Comparator/Signature.php

@@ -147,11 +147,14 @@ public static function analyze(array $parametersA, array $parametersB): array
                 if ($parametersA[$i]->type !== null && $parametersB[$i]->type !== null) {
                     $changes['parameter_typing_changed'] = true;
                     // Custom: detect nullable added
-                    if (
-                        $typeBefore instanceof \PhpParser\Node\NullableType
+                    if ($typeBefore instanceof \PhpParser\Node\NullableType
                         && !$typeAfter instanceof \PhpParser\Node\NullableType
                     ) {
                         $changes['parameter_nullable_type_added'] = true;
+                    } elseif (!$typeBefore instanceof \PhpParser\Node\NullableType
+                        && $typeAfter instanceof \PhpParser\Node\NullableType
+                    ) {
+                        $changes['parameter_nullable_type_removed'] = true;
                     }
                 } elseif ($parametersA[$i]->type !== null) {
                     $changes['parameter_typing_removed'] = true;

src/Reporter/BreakingChangeTableReporter.php

@@ -16,6 +16,7 @@ class BreakingChangeTableReporter extends TableReporter
     private $breakChangeLevels = [
         Level::MAJOR,
         Level::MINOR,
+        Level::PATCH,
     ];
 
     /**
@@ -96,7 +97,7 @@ private function outputChangeReport(OutputInterface $output, Report $report, $co
     protected function outputTable(OutputInterface $output, Report $report, $context)
     {
         $table = new HtmlTableRenderer($output);
-        $table->setHeaders(['What changed', 'How it changed']);
+        $table->setHeaders(['<strong>Change Level</strong>', '<strong>What Changed</strong>', '<strong>How It Changed</strong>']);
         $rows = [];
         foreach (Level::asList('desc') as $level) {
             if (!in_array($level, $this->breakChangeLevels)) {
@@ -105,15 +106,36 @@ protected function outputTable(OutputInterface $output, Report $report, $context
             $reportForLevel = $report[$context][$level];
             /** @var \PHPSemVerChecker\Operation\Operation $operation */
             foreach ($reportForLevel as $operation) {
+                $levelLabel = $this->getLevelLabel($level);
                 $target = $operation->getTarget();
                 $reason = $operation->getReason();
-                $rows[] = [$target, $reason];
+                $rows[] = [$levelLabel, $target, $reason];
             }
         }
         $table->setRows($rows);
         $table->render();
     }
 
+    /**
+     * Get a human-readable label for the change level
+     *
+     * @param int $level
+     * @return string
+     */
+    private function getLevelLabel(int $level): string
+    {
+        switch ($level) {
+            case Level::MAJOR:
+                return '<span style="color: #d73a49; font-weight: bold;">MAJOR (Breaking)</span>';
+            case Level::MINOR:
+                return '<span style="color: #f6a434; font-weight: bold;">MINOR (Non-breaking)</span>';
+            case Level::PATCH:
+                return '<span style="color: #28a745; font-weight: bold;">PATCH</span>';
+            default:
+                return 'UNKNOWN';
+        }
+    }
+
     /**
      * Generate the HTML header line for a report section
      *