From ae700e74996e223268bbbf1e7fded9510624485f Mon Sep 17 00:00:00 2001
From: Dave Liddament <dave@lampbristol.com>
Date: Mon, 22 Jun 2026 23:30:13 +0100
Subject: [PATCH] [code-quality] Add AttributeNamedArgsRector

Convert positional arguments on configured attributes into named
arguments, taking the names from the attribute constructor signature.
Configurable per attribute class via the AttributeNamedArgs value
object, with an optional firstNamedPosition threshold to leave leading
arguments positional. Skips already-named arguments and bails out when
an argument maps to a variadic or missing parameter to avoid producing
invalid PHP.
---
 ...eNamedArgsRectorFirstNamedPositionTest.php |  28 +++
 .../AttributeNamedArgsRectorTest.php          |  28 +++
 .../Fixture/positional_except.php.inc         |  27 +++
 .../Fixture/positional_middleware.php.inc     |  27 +++
 .../Fixture/positional_only.php.inc           |  27 +++
 .../Fixture/skip_already_named.php.inc        |  11 ++
 .../Fixture/skip_other_attribute.php.inc      |  10 ++
 .../Fixture/skip_variadic.php.inc             |  11 ++
 .../Fixture/variadic_named_prefix.php.inc     |  27 +++
 .../positional_except.php.inc                 |  27 +++
 .../positional_only.php.inc                   |  27 +++
 .../skip_middleware_left_positional.php.inc   |  11 ++
 .../Source/Middleware.php                     |  11 ++
 .../Source/MiddlewareWithVariadic.php         |  11 ++
 .../Source/OtherAttribute.php                 |  11 ++
 .../Source/RequireToken.php                   |   7 +
 .../config/configured_rule.php                |  16 ++
 .../configured_rule_first_named_position.php  |  14 ++
 .../Attribute/AttributeNamedArgsRector.php    | 169 ++++++++++++++++++
 .../ValueObject/AttributeNamedArgs.php        |  31 ++++
 20 files changed, 531 insertions(+)
 create mode 100644 rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/AttributeNamedArgsRectorFirstNamedPositionTest.php
 create mode 100644 rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/AttributeNamedArgsRectorTest.php
 create mode 100644 rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/positional_except.php.inc
 create mode 100644 rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/positional_middleware.php.inc
 create mode 100644 rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/positional_only.php.inc
 create mode 100644 rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/skip_already_named.php.inc
 create mode 100644 rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/skip_other_attribute.php.inc
 create mode 100644 rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/skip_variadic.php.inc
 create mode 100644 rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/variadic_named_prefix.php.inc
 create mode 100644 rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/FixtureFirstNamedPosition/positional_except.php.inc
 create mode 100644 rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/FixtureFirstNamedPosition/positional_only.php.inc
 create mode 100644 rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/FixtureFirstNamedPosition/skip_middleware_left_positional.php.inc
 create mode 100644 rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Source/Middleware.php
 create mode 100644 rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Source/MiddlewareWithVariadic.php
 create mode 100644 rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Source/OtherAttribute.php
 create mode 100644 rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Source/RequireToken.php
 create mode 100644 rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/config/configured_rule.php
 create mode 100644 rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/config/configured_rule_first_named_position.php
 create mode 100644 rules/CodeQuality/Rector/Attribute/AttributeNamedArgsRector.php
 create mode 100644 rules/CodeQuality/ValueObject/AttributeNamedArgs.php

diff --git a/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/AttributeNamedArgsRectorFirstNamedPositionTest.php b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/AttributeNamedArgsRectorFirstNamedPositionTest.php
new file mode 100644
index 00000000000..b6a986a395e
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/AttributeNamedArgsRectorFirstNamedPositionTest.php
@@ -0,0 +1,28 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector;
+
+use Iterator;
+use PHPUnit\Framework\Attributes\DataProvider;
+use Rector\Testing\PHPUnit\AbstractRectorTestCase;
+
+final class AttributeNamedArgsRectorFirstNamedPositionTest extends AbstractRectorTestCase
+{
+    #[DataProvider('provideData')]
+    public function test(string $filePath): void
+    {
+        $this->doTestFile($filePath);
+    }
+
+    public static function provideData(): Iterator
+    {
+        return self::yieldFilesFromDirectory(__DIR__ . '/FixtureFirstNamedPosition');
+    }
+
+    public function provideConfigFilePath(): string
+    {
+        return __DIR__ . '/config/configured_rule_first_named_position.php';
+    }
+}
diff --git a/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/AttributeNamedArgsRectorTest.php b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/AttributeNamedArgsRectorTest.php
new file mode 100644
index 00000000000..0293e82123b
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/AttributeNamedArgsRectorTest.php
@@ -0,0 +1,28 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector;
+
+use Iterator;
+use PHPUnit\Framework\Attributes\DataProvider;
+use Rector\Testing\PHPUnit\AbstractRectorTestCase;
+
+final class AttributeNamedArgsRectorTest extends AbstractRectorTestCase
+{
+    #[DataProvider('provideData')]
+    public function test(string $filePath): void
+    {
+        $this->doTestFile($filePath);
+    }
+
+    public static function provideData(): Iterator
+    {
+        return self::yieldFilesFromDirectory(__DIR__ . '/Fixture');
+    }
+
+    public function provideConfigFilePath(): string
+    {
+        return __DIR__ . '/config/configured_rule.php';
+    }
+}
diff --git a/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/positional_except.php.inc b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/positional_except.php.inc
new file mode 100644
index 00000000000..a1a5519f2be
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/positional_except.php.inc
@@ -0,0 +1,27 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Fixture;
+
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\Middleware;
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\RequireToken;
+
+#[Middleware(RequireToken::class, null, ['home'])]
+class PositionalExcept
+{
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Fixture;
+
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\Middleware;
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\RequireToken;
+
+#[Middleware(middleware: RequireToken::class, only: null, except: ['home'])]
+class PositionalExcept
+{
+}
+
+?>
diff --git a/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/positional_middleware.php.inc b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/positional_middleware.php.inc
new file mode 100644
index 00000000000..d5e1188a94b
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/positional_middleware.php.inc
@@ -0,0 +1,27 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Fixture;
+
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\Middleware;
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\RequireToken;
+
+#[Middleware(RequireToken::class, except: ['home'])]
+class PositionalMiddleware
+{
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Fixture;
+
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\Middleware;
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\RequireToken;
+
+#[Middleware(middleware: RequireToken::class, except: ['home'])]
+class PositionalMiddleware
+{
+}
+
+?>
diff --git a/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/positional_only.php.inc b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/positional_only.php.inc
new file mode 100644
index 00000000000..3c4bf778795
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/positional_only.php.inc
@@ -0,0 +1,27 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Fixture;
+
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\Middleware;
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\RequireToken;
+
+#[Middleware(RequireToken::class, ['export'])]
+class PositionalOnly
+{
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Fixture;
+
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\Middleware;
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\RequireToken;
+
+#[Middleware(middleware: RequireToken::class, only: ['export'])]
+class PositionalOnly
+{
+}
+
+?>
diff --git a/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/skip_already_named.php.inc b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/skip_already_named.php.inc
new file mode 100644
index 00000000000..7076df4b111
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/skip_already_named.php.inc
@@ -0,0 +1,11 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Fixture;
+
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\Middleware;
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\RequireToken;
+
+#[Middleware(middleware: RequireToken::class, only: ['export'])]
+class SkipAlreadyNamed
+{
+}
diff --git a/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/skip_other_attribute.php.inc b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/skip_other_attribute.php.inc
new file mode 100644
index 00000000000..a7c357bd4e6
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/skip_other_attribute.php.inc
@@ -0,0 +1,10 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Fixture;
+
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\OtherAttribute;
+
+#[OtherAttribute('a', 'b')]
+class SkipOtherAttribute
+{
+}
diff --git a/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/skip_variadic.php.inc b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/skip_variadic.php.inc
new file mode 100644
index 00000000000..9ce41ad0ec2
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/skip_variadic.php.inc
@@ -0,0 +1,11 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Fixture;
+
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\MiddlewareWithVariadic;
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\RequireToken;
+
+#[MiddlewareWithVariadic(RequireToken::class, 'a', 'b')]
+class SkipVariadic
+{
+}
diff --git a/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/variadic_named_prefix.php.inc b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/variadic_named_prefix.php.inc
new file mode 100644
index 00000000000..10cc808dcf7
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Fixture/variadic_named_prefix.php.inc
@@ -0,0 +1,27 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Fixture;
+
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\MiddlewareWithVariadic;
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\RequireToken;
+
+#[MiddlewareWithVariadic(RequireToken::class)]
+class VariadicNamedPrefix
+{
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Fixture;
+
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\MiddlewareWithVariadic;
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\RequireToken;
+
+#[MiddlewareWithVariadic(middleware: RequireToken::class)]
+class VariadicNamedPrefix
+{
+}
+
+?>
diff --git a/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/FixtureFirstNamedPosition/positional_except.php.inc b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/FixtureFirstNamedPosition/positional_except.php.inc
new file mode 100644
index 00000000000..5d29d159a01
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/FixtureFirstNamedPosition/positional_except.php.inc
@@ -0,0 +1,27 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\FixtureFirstNamedPosition;
+
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\Middleware;
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\RequireToken;
+
+#[Middleware(RequireToken::class, null, ['home'])]
+class PositionalExcept
+{
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\FixtureFirstNamedPosition;
+
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\Middleware;
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\RequireToken;
+
+#[Middleware(RequireToken::class, only: null, except: ['home'])]
+class PositionalExcept
+{
+}
+
+?>
diff --git a/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/FixtureFirstNamedPosition/positional_only.php.inc b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/FixtureFirstNamedPosition/positional_only.php.inc
new file mode 100644
index 00000000000..1f048f0173b
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/FixtureFirstNamedPosition/positional_only.php.inc
@@ -0,0 +1,27 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\FixtureFirstNamedPosition;
+
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\Middleware;
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\RequireToken;
+
+#[Middleware(RequireToken::class, ['export'])]
+class PositionalOnly
+{
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\FixtureFirstNamedPosition;
+
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\Middleware;
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\RequireToken;
+
+#[Middleware(RequireToken::class, only: ['export'])]
+class PositionalOnly
+{
+}
+
+?>
diff --git a/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/FixtureFirstNamedPosition/skip_middleware_left_positional.php.inc b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/FixtureFirstNamedPosition/skip_middleware_left_positional.php.inc
new file mode 100644
index 00000000000..5f1b08412de
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/FixtureFirstNamedPosition/skip_middleware_left_positional.php.inc
@@ -0,0 +1,11 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\FixtureFirstNamedPosition;
+
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\Middleware;
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\RequireToken;
+
+#[Middleware(RequireToken::class, except: ['home'])]
+class SkipMiddlewareLeftPositional
+{
+}
diff --git a/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Source/Middleware.php b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Source/Middleware.php
new file mode 100644
index 00000000000..cd6c7ddab07
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Source/Middleware.php
@@ -0,0 +1,11 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source;
+
+#[\Attribute]
+class Middleware
+{
+    public function __construct($middleware = null, $only = null, $except = null)
+    {
+    }
+}
diff --git a/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Source/MiddlewareWithVariadic.php b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Source/MiddlewareWithVariadic.php
new file mode 100644
index 00000000000..7e754f6e99b
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Source/MiddlewareWithVariadic.php
@@ -0,0 +1,11 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source;
+
+#[\Attribute]
+class MiddlewareWithVariadic
+{
+    public function __construct($middleware = null, ...$groups)
+    {
+    }
+}
diff --git a/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Source/OtherAttribute.php b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Source/OtherAttribute.php
new file mode 100644
index 00000000000..fb6214bb518
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Source/OtherAttribute.php
@@ -0,0 +1,11 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source;
+
+#[\Attribute]
+class OtherAttribute
+{
+    public function __construct($a = null, $b = null)
+    {
+    }
+}
diff --git a/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Source/RequireToken.php b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Source/RequireToken.php
new file mode 100644
index 00000000000..3f376ee5171
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/Source/RequireToken.php
@@ -0,0 +1,7 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source;
+
+class RequireToken
+{
+}
diff --git a/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/config/configured_rule.php b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/config/configured_rule.php
new file mode 100644
index 00000000000..89b077ff518
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/config/configured_rule.php
@@ -0,0 +1,16 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\CodeQuality\Rector\Attribute\AttributeNamedArgsRector;
+use Rector\CodeQuality\ValueObject\AttributeNamedArgs;
+use Rector\Config\RectorConfig;
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\Middleware;
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\MiddlewareWithVariadic;
+
+return static function (RectorConfig $rectorConfig): void {
+    $rectorConfig->ruleWithConfiguration(AttributeNamedArgsRector::class, [
+        new AttributeNamedArgs(Middleware::class),
+        new AttributeNamedArgs(MiddlewareWithVariadic::class),
+    ]);
+};
diff --git a/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/config/configured_rule_first_named_position.php b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/config/configured_rule_first_named_position.php
new file mode 100644
index 00000000000..f7e976081cd
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/Attribute/AttributeNamedArgsRector/config/configured_rule_first_named_position.php
@@ -0,0 +1,14 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\CodeQuality\Rector\Attribute\AttributeNamedArgsRector;
+use Rector\CodeQuality\ValueObject\AttributeNamedArgs;
+use Rector\Config\RectorConfig;
+use Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\Source\Middleware;
+
+return static function (RectorConfig $rectorConfig): void {
+    $rectorConfig->ruleWithConfiguration(AttributeNamedArgsRector::class, [
+        new AttributeNamedArgs(Middleware::class, 1),
+    ]);
+};
diff --git a/rules/CodeQuality/Rector/Attribute/AttributeNamedArgsRector.php b/rules/CodeQuality/Rector/Attribute/AttributeNamedArgsRector.php
new file mode 100644
index 00000000000..cd3541b28cf
--- /dev/null
+++ b/rules/CodeQuality/Rector/Attribute/AttributeNamedArgsRector.php
@@ -0,0 +1,169 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\CodeQuality\Rector\Attribute;
+
+use PhpParser\Node;
+use PhpParser\Node\Arg;
+use PhpParser\Node\Attribute;
+use PhpParser\Node\Identifier;
+use PHPStan\Reflection\MethodReflection;
+use PHPStan\Reflection\ParameterReflection;
+use PHPStan\Reflection\ParametersAcceptorSelector;
+use Rector\CodeQuality\ValueObject\AttributeNamedArgs;
+use Rector\Contract\Rector\ConfigurableRectorInterface;
+use Rector\Rector\AbstractRector;
+use Rector\Reflection\ReflectionResolver;
+use Rector\ValueObject\PhpVersionFeature;
+use Rector\VersionBonding\Contract\MinPhpVersionInterface;
+use Symplify\RuleDocGenerator\ValueObject\CodeSample\ConfiguredCodeSample;
+use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
+use Webmozart\Assert\Assert;
+
+/**
+ * @see \Rector\Tests\CodeQuality\Rector\Attribute\AttributeNamedArgsRector\AttributeNamedArgsRectorTest
+ */
+final class AttributeNamedArgsRector extends AbstractRector implements ConfigurableRectorInterface, MinPhpVersionInterface
+{
+    /**
+     * @var AttributeNamedArgs[]
+     */
+    private array $attributeNamedArgs = [];
+
+    public function __construct(
+        private readonly ReflectionResolver $reflectionResolver
+    ) {
+    }
+
+    public function getRuleDefinition(): RuleDefinition
+    {
+        return new RuleDefinition(
+            'Convert positional arguments on configured attributes into named arguments, using the attribute constructor parameter names',
+            [
+                new ConfiguredCodeSample(
+                    <<<'CODE_SAMPLE'
+#[SomeAttribute(SomeClass::class, null, ['home'])]
+class SomeClass
+{
+}
+CODE_SAMPLE
+                    ,
+                    <<<'CODE_SAMPLE'
+#[SomeAttribute(value: SomeClass::class, only: null, except: ['home'])]
+class SomeClass
+{
+}
+CODE_SAMPLE
+                    ,
+                    [new AttributeNamedArgs('Some\Attribute\SomeAttribute')]
+                ),
+            ]
+        );
+    }
+
+    /**
+     * @return array<class-string<Node>>
+     */
+    public function getNodeTypes(): array
+    {
+        return [Attribute::class];
+    }
+
+    /**
+     * @param Attribute $node
+     */
+    public function refactor(Node $node): ?Node
+    {
+        foreach ($this->attributeNamedArgs as $attributeNamedArg) {
+            if ($this->isName($node->name, $attributeNamedArg->getAttributeClass())) {
+                return $this->nameArguments($node, $attributeNamedArg);
+            }
+        }
+
+        return null;
+    }
+
+    /**
+     * @param mixed[] $configuration
+     */
+    public function configure(array $configuration): void
+    {
+        Assert::allIsAOf($configuration, AttributeNamedArgs::class);
+
+        $this->attributeNamedArgs = $configuration;
+    }
+
+    public function provideMinPhpVersion(): int
+    {
+        return PhpVersionFeature::NAMED_ARGUMENTS;
+    }
+
+    private function nameArguments(Attribute $attribute, AttributeNamedArgs $attributeNamedArgs): ?Attribute
+    {
+        $methodReflection = $this->reflectionResolver->resolveConstructorReflectionFromAttribute($attribute);
+        if (! $methodReflection instanceof MethodReflection) {
+            return null;
+        }
+
+        $extendedParametersAcceptor = ParametersAcceptorSelector::combineAcceptors($methodReflection->getVariants());
+        $parameters = $extendedParametersAcceptor->getParameters();
+
+        $namesToApply = $this->resolveArgNamesToApply(
+            $attribute->args,
+            $parameters,
+            $attributeNamedArgs->getFirstNamedPosition()
+        );
+        if ($namesToApply === []) {
+            return null;
+        }
+
+        foreach ($namesToApply as $position => $name) {
+            $attribute->args[$position]->name = new Identifier($name);
+        }
+
+        return $attribute;
+    }
+
+    /**
+     * Resolve the positional arguments to name, as a position => parameter-name map, or [] when
+     * nothing should change. Naming an argument forces every later positional argument to be named
+     * too (PHP forbids a positional argument after a named one). So if any argument in the named
+     * range maps to a variadic parameter, or to no parameter at all (overflow past a variadic),
+     * the whole attribute is left untouched rather than producing invalid PHP.
+     *
+     * @param Arg[]                 $args
+     * @param ParameterReflection[] $parameters
+     * @return array<int, string>
+     */
+    private function resolveArgNamesToApply(array $args, array $parameters, int $firstNamedPosition): array
+    {
+        $namesToApply = [];
+
+        $count = count($args);
+        for ($position = $firstNamedPosition; $position < $count; ++$position) {
+            $arg = $args[$position];
+
+            // already named
+            if ($arg->name instanceof Identifier) {
+                continue;
+            }
+
+            $parameter = $parameters[$position] ?? null;
+
+            // no matching parameter, e.g. overflow past a variadic
+            if ($parameter === null) {
+                return [];
+            }
+
+            // naming a variadic would rebind it or strand later positional arguments
+            if ($parameter->isVariadic()) {
+                return [];
+            }
+
+            $namesToApply[$position] = $parameter->getName();
+        }
+
+        return $namesToApply;
+    }
+}
diff --git a/rules/CodeQuality/ValueObject/AttributeNamedArgs.php b/rules/CodeQuality/ValueObject/AttributeNamedArgs.php
new file mode 100644
index 00000000000..226aeaaccec
--- /dev/null
+++ b/rules/CodeQuality/ValueObject/AttributeNamedArgs.php
@@ -0,0 +1,31 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Rector\CodeQuality\ValueObject;
+
+final readonly class AttributeNamedArgs
+{
+    /**
+     * @param class-string $attributeClass     the attribute whose positional arguments should be named
+     * @param int          $firstNamedPosition first positional index to name (0 = name all arguments)
+     */
+    public function __construct(
+        private string $attributeClass,
+        private int $firstNamedPosition = 0
+    ) {
+    }
+
+    /**
+     * @return class-string
+     */
+    public function getAttributeClass(): string
+    {
+        return $this->attributeClass;
+    }
+
+    public function getFirstNamedPosition(): int
+    {
+        return $this->firstNamedPosition;
+    }
+}