[CodeQuality] Skip InlineConstructorDefaultToPropertyRector on non-final class

rules-tests/CodeQuality/Rector/Class_/InlineConstructorDefaultToPropertyRector/Fixture/skip_non_final_class.php.inc

@@ -0,0 +1,16 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Class_\InlineConstructorDefaultToPropertyRector\Fixture;
+
+use Rector\Tests\CodeQuality\Rector\Class_\InlineConstructorDefaultToPropertyRector\Source\ValidatedBase;
+
+class SkipsParentConstructor extends ValidatedBase
+{
+    private bool $validated = false;
+
+    public function __construct()
+    {
+        $this->validate();
+        $this->validated = true;
+    }
+}

rules-tests/CodeQuality/Rector/Class_/InlineConstructorDefaultToPropertyRector/Source/ValidatedBase.php

@@ -0,0 +1,12 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Tests\CodeQuality\Rector\Class_\InlineConstructorDefaultToPropertyRector\Source;
+
+abstract class ValidatedBase
+{
+    protected function validate(): void
+    {
+    }
+}

rules/CodeQuality/Rector/Class_/InlineConstructorDefaultToPropertyRector.php

@@ -82,6 +82,11 @@ public function getNodeTypes(): array
      */
     public function refactor(Node $node): ?Node
     {
+        // A child class may bypass the constructor and depend on the declared property default
+        if (! $node->isFinal()) {
+            return null;
+        }
+
         $hasChanged = false;
 
         $constructClassMethod = $node->getMethod(MethodName::CONSTRUCT);

src/Bridge/SetProviderCollector.php

@@ -25,8 +25,9 @@
     /**
      * @param SetProviderInterface[] $setProviders
      */
-    public function __construct(private array $setProviders = [])
-    {
+    public function __construct(
+        private array $setProviders = []
+    ) {
     }
 
     /**