refactor(class): Rename UpdateRectorRefactorParamDocblockFromNodeTypesRector

guanguans · guanguans · commit 7d2003e67243 · 2026-01-19T14:43:30.000+08:00
- Renamed the class to UpdateRectorMethodNodeParamDocblockFromNodeTypesRector for clarity.
- Updated references in configuration and test files to reflect the new class name.
- Ensured consistency across the codebase by replacing old references with the new class name.
diff --git a/config/set/rector.php b/config/set/rector.php
@@ -14,13 +14,13 @@
  */
 
 use Guanguans\RectorRules\Rector\Array_\UpdateRectorCodeSamplesFromFixturesRector;
-use Guanguans\RectorRules\Rector\Class_\UpdateRectorRefactorParamDocblockFromNodeTypesRector;
+use Guanguans\RectorRules\Rector\Class_\UpdateRectorMethodNodeParamDocblockFromNodeTypesRector;
 use Rector\Config\RectorConfig;
 
 return static function (RectorConfig $rectorConfig): void {
     $rectorConfig->import(__DIR__.'/../config.php');
     $rectorConfig->rules([
         UpdateRectorCodeSamplesFromFixturesRector::class,
-        UpdateRectorRefactorParamDocblockFromNodeTypesRector::class,
+        UpdateRectorMethodNodeParamDocblockFromNodeTypesRector::class,
     ]);
 };
diff --git a/docs/rules-overview.md b/docs/rules-overview.md
@@ -140,11 +140,11 @@ Update p h p stan method node param docblock from node types
 
 <br>
 
-### UpdateRectorRefactorParamDocblockFromNodeTypesRector
+### UpdateRectorMethodNodeParamDocblockFromNodeTypesRector
 
-Update rector refactor param docblock from node types
+Update rector method node param docblock from node types
 
-- class: [`Guanguans\RectorRules\Rector\Class_\UpdateRectorRefactorParamDocblockFromNodeTypesRector`](../src/Rector/Class_/UpdateRectorRefactorParamDocblockFromNodeTypesRector.php)
+- class: [`Guanguans\RectorRules\Rector\Class_\UpdateRectorMethodNodeParamDocblockFromNodeTypesRector`](../src/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector.php)
 
 ```diff
  /** @noinspection ALL */
@@ -154,7 +154,7 @@ Update rector refactor param docblock from node types
  use PhpParser\Node;
  use PhpParser\Node\Stmt\Class_;
 
- final class UpdateRectorRefactorParamDocblockFromNodeTypesRector extends AbstractRector
+ final class UpdateRectorMethodNodeParamDocblockFromNodeTypesRector extends AbstractRector
  {
      public function getNodeTypes(): array
      {
diff --git a/src/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector.php b/src/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector.php
@@ -22,9 +22,9 @@
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
 
 /**
- * @see \Guanguans\RectorRulesTests\Rector\Class_\UpdateRectorRefactorParamDocblockFromNodeTypesRector\UpdateRectorRefactorParamDocblockFromNodeTypesRectorTest
+ * @see \Guanguans\RectorRulesTests\Rector\Class_\UpdateRectorMethodNodeParamDocblockFromNodeTypesRector\UpdateRectorMethodNodeParamDocblockFromNodeTypesRectorTest
  */
-final class UpdateRectorRefactorParamDocblockFromNodeTypesRector extends AbstractUpdateClassMethodNodeParamDocblockFromNodeTypesRector
+final class UpdateRectorMethodNodeParamDocblockFromNodeTypesRector extends AbstractUpdateClassMethodNodeParamDocblockFromNodeTypesRector
 {
     /**
      * @return list<\Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample>
@@ -41,7 +41,7 @@ protected function codeSamples(): array
                     use PhpParser\Node;
                     use PhpParser\Node\Stmt\Class_;
 
-                    final class UpdateRectorRefactorParamDocblockFromNodeTypesRector extends AbstractRector
+                    final class UpdateRectorMethodNodeParamDocblockFromNodeTypesRector extends AbstractRector
                     {
                         public function getNodeTypes(): array
                         {
@@ -64,7 +64,7 @@ public function refactor(Node $node): ?Node
                     use PhpParser\Node;
                     use PhpParser\Node\Stmt\Class_;
 
-                    final class UpdateRectorRefactorParamDocblockFromNodeTypesRector extends AbstractRector
+                    final class UpdateRectorMethodNodeParamDocblockFromNodeTypesRector extends AbstractRector
                     {
                         public function getNodeTypes(): array
                         {
diff --git a/tests/Rector/AbstractRectorTestCase.php b/tests/Rector/AbstractRectorTestCase.php
@@ -59,8 +59,10 @@ final public function provideConfigFilePath(): string
     final public function testRectorTestCaseClassName(): void
     {
         self::assertSame(
-            Str::replaceLast('Test', '', (new \ReflectionClass(static::class))->getShortName()),
-            static::rectorReflectionClass()->getShortName()
+            static::class,
+            (string) Str::of(static::rectorClass())
+                ->replace('RectorRules', 'RectorRulesTests')
+                ->append('\\', static::rectorReflectionClass()->getShortName(), 'Test')
         );
     }
 
diff --git a/tests/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector/Fixture/fixture.php.inc b/tests/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector/Fixture/fixture.php.inc
@@ -7,7 +7,7 @@ use Guanguans\RectorRules\Rector\AbstractRector;
 use PhpParser\Node;
 use PhpParser\Node\Stmt\Class_;
 
-final class UpdateRectorRefactorParamDocblockFromNodeTypesRector extends AbstractRector
+final class UpdateRectorMethodNodeParamDocblockFromNodeTypesRector extends AbstractRector
 {
     public function getNodeTypes(): array
     {
@@ -33,7 +33,7 @@ use Guanguans\RectorRules\Rector\AbstractRector;
 use PhpParser\Node;
 use PhpParser\Node\Stmt\Class_;
 
-final class UpdateRectorRefactorParamDocblockFromNodeTypesRector extends AbstractRector
+final class UpdateRectorMethodNodeParamDocblockFromNodeTypesRector extends AbstractRector
 {
     public function getNodeTypes(): array
     {
diff --git a/tests/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector/Fixture/fixture_union_type.php.inc b/tests/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector/Fixture/fixture_union_type.php.inc
diff --git a/tests/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector/Fixture/skip_non_instantiable_rectorr.php.inc b/tests/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector/Fixture/skip_non_instantiable_rectorr.php.inc
diff --git a/tests/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector/Fixture/skip_non_method_node_rector.php.inc b/tests/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector/Fixture/skip_non_method_node_rector.php.inc
@@ -7,7 +7,7 @@ use PhpParser\Node\Stmt\Class_;
 use PhpParser\Node\Stmt\ClassMethod;
 use Rector\Rector\AbstractRector;
 
-final class UpdateRectorRefactorParamDocblockFromNodeTypesRector extends AbstractUpdateClassMethodNodeParamDocblockFromNodeTypesRector
+final class UpdateRectorMethodNodeParamDocblockFromNodeTypesRector extends AbstractUpdateClassMethodNodeParamDocblockFromNodeTypesRector
 {
     protected function classType(): string
     {
diff --git a/tests/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector/Fixture/skip_non_rector.php.inc b/tests/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector/Fixture/skip_non_rector.php.inc
diff --git a/tests/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector/UpdateRectorMethodNodeParamDocblockFromNodeTypesRectorTest.php b/tests/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector/UpdateRectorMethodNodeParamDocblockFromNodeTypesRectorTest.php
@@ -18,15 +18,15 @@
  * @see https://github.com/guanguans/rector-rules
  */
 
-namespace Guanguans\RectorRulesTests\Rector\Class_\UpdateRectorRefactorParamDocblockFromNodeTypesRector;
+namespace Guanguans\RectorRulesTests\Rector\Class_\UpdateRectorMethodNodeParamDocblockFromNodeTypesRector;
 
 use Guanguans\RectorRulesTests\Rector\AbstractRectorTestCase;
 
 /**
  * @covers \Guanguans\RectorRules\Rector\Class_\AbstractUpdateClassMethodNodeParamDocblockFromNodeTypesRector
- * @covers \Guanguans\RectorRules\Rector\Class_\UpdateRectorRefactorParamDocblockFromNodeTypesRector
+ * @covers \Guanguans\RectorRules\Rector\Class_\UpdateRectorMethodNodeParamDocblockFromNodeTypesRector
  */
-final class UpdateRectorRefactorParamDocblockFromNodeTypesRectorTest extends AbstractRectorTestCase
+final class UpdateRectorMethodNodeParamDocblockFromNodeTypesRectorTest extends AbstractRectorTestCase
 {
     protected static function directory(): string
     {
diff --git a/tests/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector/config/configured_rule.php b/tests/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector/config/configured_rule.php
@@ -18,9 +18,9 @@
  * @see https://github.com/guanguans/rector-rules
  */
 
-use Guanguans\RectorRules\Rector\Class_\UpdateRectorRefactorParamDocblockFromNodeTypesRector;
+use Guanguans\RectorRules\Rector\Class_\UpdateRectorMethodNodeParamDocblockFromNodeTypesRector;
 use Rector\Config\RectorConfig;
 
 return static function (RectorConfig $rectorConfig): void {
-    $rectorConfig->rule(UpdateRectorRefactorParamDocblockFromNodeTypesRector::class);
+    $rectorConfig->rule(UpdateRectorMethodNodeParamDocblockFromNodeTypesRector::class);
 };

Original file line number	Diff line number	Diff line change
`@@ -22,9 +22,9 @@`
`22`	`22`	`use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;`
`23`	`23`
`24`	`24`	`/**`
`25`		`- * @see \Guanguans\RectorRulesTests\Rector\Class_\UpdateRectorRefactorParamDocblockFromNodeTypesRector\UpdateRectorRefactorParamDocblockFromNodeTypesRectorTest`
	`25`	`+ * @see \Guanguans\RectorRulesTests\Rector\Class_\UpdateRectorMethodNodeParamDocblockFromNodeTypesRector\UpdateRectorMethodNodeParamDocblockFromNodeTypesRectorTest`
`26`	`26`	`*/`
`27`		`-final class UpdateRectorRefactorParamDocblockFromNodeTypesRector extends AbstractUpdateClassMethodNodeParamDocblockFromNodeTypesRector`
	`27`	`+final class UpdateRectorMethodNodeParamDocblockFromNodeTypesRector extends AbstractUpdateClassMethodNodeParamDocblockFromNodeTypesRector`
`28`	`28`	`{`
`29`	`29`	`/**`
`30`	`30`	`* @return list<\Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample>`
`@@ -41,7 +41,7 @@ protected function codeSamples(): array`
`41`	`41`	`use PhpParser\Node;`
`42`	`42`	`use PhpParser\Node\Stmt\Class_;`
`43`	`43`
`44`		`- final class UpdateRectorRefactorParamDocblockFromNodeTypesRector extends AbstractRector`
	`44`	`+ final class UpdateRectorMethodNodeParamDocblockFromNodeTypesRector extends AbstractRector`
`45`	`45`	`{`
`46`	`46`	`public function getNodeTypes(): array`
`47`	`47`	`{`
`@@ -64,7 +64,7 @@ public function refactor(Node $node): ?Node`
`64`	`64`	`use PhpParser\Node;`
`65`	`65`	`use PhpParser\Node\Stmt\Class_;`
`66`	`66`
`67`		`- final class UpdateRectorRefactorParamDocblockFromNodeTypesRector extends AbstractRector`
	`67`	`+ final class UpdateRectorMethodNodeParamDocblockFromNodeTypesRector extends AbstractRector`
`68`	`68`	`{`
`69`	`69`	`public function getNodeTypes(): array`
`70`	`70`	`{`
Original file line number	Diff line number	Diff line change
`@@ -59,8 +59,10 @@ final public function provideConfigFilePath(): string`
`59`	`59`	`final public function testRectorTestCaseClassName(): void`
`60`	`60`	`{`
`61`	`61`	`self::assertSame(`
`62`		`- Str::replaceLast('Test', '', (new \ReflectionClass(static::class))->getShortName()),`
`63`		`- static::rectorReflectionClass()->getShortName()`
	`62`	`+ static::class,`
	`63`	`+ (string) Str::of(static::rectorClass())`
	`64`	`+ ->replace('RectorRules', 'RectorRulesTests')`
	`65`	`+ ->append('\\', static::rectorReflectionClass()->getShortName(), 'Test')`
`64`	`66`	`);`
`65`	`67`	`}`
`66`	`68`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ use Guanguans\RectorRules\Rector\AbstractRector;`
`7`	`7`	`use PhpParser\Node;`
`8`	`8`	`use PhpParser\Node\Stmt\Class_;`
`9`	`9`
`10`		`-final class UpdateRectorRefactorParamDocblockFromNodeTypesRector extends AbstractRector`
	`10`	`+final class UpdateRectorMethodNodeParamDocblockFromNodeTypesRector extends AbstractRector`
`11`	`11`	`{`
`12`	`12`	`public function getNodeTypes(): array`
`13`	`13`	`{`
`@@ -33,7 +33,7 @@ use Guanguans\RectorRules\Rector\AbstractRector;`
`33`	`33`	`use PhpParser\Node;`
`34`	`34`	`use PhpParser\Node\Stmt\Class_;`
`35`	`35`
`36`		`-final class UpdateRectorRefactorParamDocblockFromNodeTypesRector extends AbstractRector`
	`36`	`+final class UpdateRectorMethodNodeParamDocblockFromNodeTypesRector extends AbstractRector`
`37`	`37`	`{`
`38`	`38`	`public function getNodeTypes(): array`
`39`	`39`	`{`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ use PhpParser\Node\Stmt\Class_;`
`7`	`7`	`use PhpParser\Node\Stmt\ClassMethod;`
`8`	`8`	`use Rector\Rector\AbstractRector;`
`9`	`9`
`10`		`-final class UpdateRectorRefactorParamDocblockFromNodeTypesRector extends AbstractUpdateClassMethodNodeParamDocblockFromNodeTypesRector`
	`10`	`+final class UpdateRectorMethodNodeParamDocblockFromNodeTypesRector extends AbstractUpdateClassMethodNodeParamDocblockFromNodeTypesRector`
`11`	`11`	`{`
`12`	`12`	`protected function classType(): string`
`13`	`13`	`{`