From c1d42b5a0a51f50c9b37f10d33dbd135fb0c7e8c Mon Sep 17 00:00:00 2001
From: Tomas Votruba <tomas.vot@gmail.com>
Date: Mon, 3 Aug 2026 23:19:35 +0200
Subject: [PATCH 1/2] Skip constraint without own constructor in
 LoadValidatorMetadataToAttributeRector

---
 ...constraint_without_own_constructor.php.inc | 19 ++++++++++
 .../Source/UniqueUserAlias.php                | 29 ++++++++++++++
 ...LoadValidatorMetadataToAttributeRector.php |  7 ++++
 .../ConstraintConstructorAnalyzer.php         | 38 +++++++++++++++++++
 4 files changed, 93 insertions(+)
 create mode 100644 rules-tests/CodeQuality/Rector/Class_/LoadValidatorMetadataToAttributeRector/Fixture/skip_constraint_without_own_constructor.php.inc
 create mode 100644 rules-tests/CodeQuality/Rector/Class_/LoadValidatorMetadataToAttributeRector/Source/UniqueUserAlias.php
 create mode 100644 src/NodeAnalyzer/ValidatorAssert/ConstraintConstructorAnalyzer.php

diff --git a/rules-tests/CodeQuality/Rector/Class_/LoadValidatorMetadataToAttributeRector/Fixture/skip_constraint_without_own_constructor.php.inc b/rules-tests/CodeQuality/Rector/Class_/LoadValidatorMetadataToAttributeRector/Fixture/skip_constraint_without_own_constructor.php.inc
new file mode 100644
index 00000000..e80c145c
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Class_/LoadValidatorMetadataToAttributeRector/Fixture/skip_constraint_without_own_constructor.php.inc
@@ -0,0 +1,19 @@
+<?php
+
+namespace Rector\Symfony\Tests\CodeQuality\Rector\Class_\LoadValidatorMetadataToAttributeRector\Fixture;
+
+use Rector\Symfony\Tests\CodeQuality\Rector\Class_\LoadValidatorMetadataToAttributeRector\Source\UniqueUserAlias;
+use Symfony\Component\Validator\Mapping\ClassMetadata;
+
+final class SkipConstraintWithoutOwnConstructor
+{
+    private $alias;
+
+    public static function loadValidatorMetadata(ClassMetadata $metadata): void
+    {
+        $metadata->addConstraint(new UniqueUserAlias([
+            'field'   => 'alias',
+            'message' => 'mautic.lead.list.alias.unique',
+        ]));
+    }
+}
diff --git a/rules-tests/CodeQuality/Rector/Class_/LoadValidatorMetadataToAttributeRector/Source/UniqueUserAlias.php b/rules-tests/CodeQuality/Rector/Class_/LoadValidatorMetadataToAttributeRector/Source/UniqueUserAlias.php
new file mode 100644
index 00000000..74296c82
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Class_/LoadValidatorMetadataToAttributeRector/Source/UniqueUserAlias.php
@@ -0,0 +1,29 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Symfony\Tests\CodeQuality\Rector\Class_\LoadValidatorMetadataToAttributeRector\Source;
+
+use Attribute;
+use Symfony\Component\Validator\Constraint;
+
+/**
+ * Custom constraint that configures itself through properties, without a constructor of its own
+ */
+#[Attribute]
+final class UniqueUserAlias extends Constraint
+{
+    public $message = 'This alias is already in use.';
+
+    public $field = '';
+
+    public function getTargets(): string
+    {
+        return self::CLASS_CONSTRAINT;
+    }
+
+    public function getDefaultOption(): string
+    {
+        return 'field';
+    }
+}
diff --git a/rules/CodeQuality/Rector/Class_/LoadValidatorMetadataToAttributeRector.php b/rules/CodeQuality/Rector/Class_/LoadValidatorMetadataToAttributeRector.php
index d68f7379..c10ca349 100644
--- a/rules/CodeQuality/Rector/Class_/LoadValidatorMetadataToAttributeRector.php
+++ b/rules/CodeQuality/Rector/Class_/LoadValidatorMetadataToAttributeRector.php
@@ -12,6 +12,7 @@
 use Rector\Rector\AbstractRector;
 use Rector\Symfony\NodeAnalyzer\ValidatorAssert\ConstantExpressionAnalyzer;
 use Rector\Symfony\NodeAnalyzer\ValidatorAssert\ConstraintAttributeTargetAnalyzer;
+use Rector\Symfony\NodeAnalyzer\ValidatorAssert\ConstraintConstructorAnalyzer;
 use Rector\Symfony\NodeAnalyzer\ValidatorAssert\MetadataConstraintResolver;
 use Rector\Symfony\NodeFactory\ValidatorAssert\ConstraintAttributeGroupFactory;
 use Rector\Symfony\ValueObject\ValidatorAssert\ClassMethodAndConstraint;
@@ -37,6 +38,7 @@ public function __construct(
         private readonly ConstraintAttributeTargetAnalyzer $constraintAttributeTargetAnalyzer,
         private readonly ConstraintAttributeGroupFactory $constraintAttributeGroupFactory,
         private readonly ConstantExpressionAnalyzer $constantExpressionAnalyzer,
+        private readonly ConstraintConstructorAnalyzer $constraintConstructorAnalyzer,
     ) {
     }
 
@@ -247,6 +249,11 @@ private function resolveConstraintClass(New_ $new): ?string
             return null;
         }
 
+        // e.g. new UniqueUserAlias(['field' => 'alias']) - the options have no matching constructor arguments
+        if ($new->args !== [] && ! $this->constraintConstructorAnalyzer->hasOwnConstructor($constraintClass)) {
+            return null;
+        }
+
         return $constraintClass;
     }
 }
diff --git a/src/NodeAnalyzer/ValidatorAssert/ConstraintConstructorAnalyzer.php b/src/NodeAnalyzer/ValidatorAssert/ConstraintConstructorAnalyzer.php
new file mode 100644
index 00000000..bfa3badb
--- /dev/null
+++ b/src/NodeAnalyzer/ValidatorAssert/ConstraintConstructorAnalyzer.php
@@ -0,0 +1,38 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Symfony\NodeAnalyzer\ValidatorAssert;
+
+use PHPStan\Reflection\ReflectionProvider;
+use Symfony\Component\Validator\Constraint;
+
+/**
+ * A constraint that adds no constructor of its own inherits the one from Symfony\Component\Validator\Constraint,
+ * which takes the options array as a whole. Its options are plain properties, so there are no named arguments to
+ * move them to.
+ */
+final readonly class ConstraintConstructorAnalyzer
+{
+    public function __construct(
+        private ReflectionProvider $reflectionProvider,
+    ) {
+    }
+
+    public function hasOwnConstructor(string $constraintClass): bool
+    {
+        if (! $this->reflectionProvider->hasClass($constraintClass)) {
+            return false;
+        }
+
+        $classReflection = $this->reflectionProvider->getClass($constraintClass);
+        if (! $classReflection->hasConstructor()) {
+            return false;
+        }
+
+        $extendedMethodReflection = $classReflection->getConstructor();
+
+        return $extendedMethodReflection->getDeclaringClass()
+            ->getName() !== Constraint::class;
+    }
+}

From 876953a83ca257f86ce546049c11baabe9bd4c56 Mon Sep 17 00:00:00 2001
From: Tomas Votruba <tomas.vot@gmail.com>
Date: Mon, 3 Aug 2026 23:21:01 +0200
Subject: [PATCH 2/2] Fix method chaining indentation in
 ConstantExpressionAnalyzerTest

---
 .../ValidatorAssert/ConstantExpressionAnalyzerTest.php         | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tests/NodeAnalyzer/ValidatorAssert/ConstantExpressionAnalyzerTest.php b/tests/NodeAnalyzer/ValidatorAssert/ConstantExpressionAnalyzerTest.php
index a9af9caa..4453e4c5 100644
--- a/tests/NodeAnalyzer/ValidatorAssert/ConstantExpressionAnalyzerTest.php
+++ b/tests/NodeAnalyzer/ValidatorAssert/ConstantExpressionAnalyzerTest.php
@@ -53,7 +53,8 @@ public static function provideData(): iterable
 
     private function parseNew(string $newExpression): New_
     {
-        $parser = new ParserFactory()->createForNewestSupportedVersion();
+        $parser = new ParserFactory()
+            ->createForNewestSupportedVersion();
 
         $stmts = $parser->parse('<?php ' . $newExpression . ';');
         $this->assertIsArray($stmts);