refactor(name): Rename PsrNameRector to ConventionalCaseNameRector

guanguans · guanguans · commit c751fd7d42ab · 2026-01-20T15:54:56.000+08:00
- Updated class name and references from RenameToPsrNameRector to RenameToConventionalCaseNameRector for clarity.
- Adjusted related configuration and documentation files to reflect the new naming convention.
- Ensured consistency across all instances where the old name was used.
diff --git a/README.md b/README.md
@@ -37,7 +37,7 @@ composer require guanguans/rector-rules --dev --ansi -v
 
 ```php
 use Guanguans\RectorRules\Rector\File\SortFileFunctionStmtRector;
-use Guanguans\RectorRules\Rector\Name\RenameToPsrNameRector;
+use Guanguans\RectorRules\Rector\Name\RenameToConventionalCaseNameRector;
 use PhpParser\NodeVisitor\ParentConnectingVisitor;
 use Rector\Config\RectorConfig;
 
@@ -48,7 +48,7 @@ return RectorConfig::configure()
     ])
     // ...
     ->registerDecoratingNodeVisitor(ParentConnectingVisitor::class)
-    ->withConfiguredRule(RenameToPsrNameRector::class, [
+    ->withConfiguredRule(RenameToConventionalCaseNameRector::class, [
         'assertMatches*Snapshot',
         'beforeEach',
         'PDO',
diff --git a/baselines/method.nonObject.neon b/baselines/method.nonObject.neon
@@ -10,4 +10,4 @@ parameters:
         -
             message: '#^Cannot call method getParameterOption\(\) on mixed\.$#'
             count: 1
-            path: ../src/Rector/Name/RenameToPsrNameRector.php
+            path: ../src/Rector/Name/RenameToConventionalCaseNameRector.php
diff --git a/baselines/symplify.preferredClass.neon b/baselines/symplify.preferredClass.neon
@@ -5,4 +5,4 @@ parameters:
         -
             message: '#^Instead of "Closure" class/interface use "PhpParser\\Node\\Expr\\Closure"$#'
             count: 1
-            path: ../src/Rector/Name/RenameToPsrNameRector.php
+            path: ../src/Rector/Name/RenameToConventionalCaseNameRector.php
diff --git a/composer.json b/composer.json
@@ -388,7 +388,7 @@
         "rule-doc-generator:validate": "@rule-doc-generator validate src/Rector/ src/Rector/Array_/UpdateRectorCodeSamplesFromFixturesRector.php",
         "sk": "@php vendor/bin/swiss-knife --ansi -vv",
         "sk:alice-yaml-fixtures-to-php": "@sk alice-yaml-fixtures-to-php --help",
-        "sk:check-commented-code": "@sk check-commented-code src/ --line-limit=5 --skip-file=src/Support/helpers.php --skip-file=src/Rector/Name/RenameToPsrNameRector.php --skip-file=src/Rector/Array_/UpdateRectorCodeSamplesFromFixturesRector.php --skip-file=src/Rector/Class_/AbstractUpdateClassMethodNodeParamDocblockFromNodeTypesRector.php",
+        "sk:check-commented-code": "@sk check-commented-code src/ --line-limit=5 --skip-file=src/Support/helpers.php --skip-file=src/Rector/Name/RenameToConventionalCaseNameRector.php --skip-file=src/Rector/Array_/UpdateRectorCodeSamplesFromFixturesRector.php --skip-file=src/Rector/Class_/AbstractUpdateClassMethodNodeParamDocblockFromNodeTypesRector.php",
         "sk:check-conflicts": "@sk check-conflicts src/",
         "sk:dump-editorconfig": "@sk dump-editorconfig",
         "sk:finalize-classes": "@sk finalize-classes src/",
diff --git a/docs/rules-overview.md b/docs/rules-overview.md
@@ -248,13 +248,13 @@ Sort file function stmt
 
 ## Name
 
-### RenameToPsrNameRector
+### RenameToConventionalCaseNameRector
 
-Rename to psr name
+Rename to conventional case name
 
 :wrench: **configure it!**
 
-- class: [`Guanguans\RectorRules\Rector\Name\RenameToPsrNameRector`](../src/Rector/Name/RenameToPsrNameRector.php)
+- class: [`Guanguans\RectorRules\Rector\Name\RenameToConventionalCaseNameRector`](../src/Rector/Name/RenameToConventionalCaseNameRector.php)
 
 ```diff
  /** @noinspection ALL */
diff --git a/phpstan.neon.dist b/phpstan.neon.dist
@@ -142,10 +142,10 @@ parameters:
 
         -
             identifier: complexity.classLike
-            path: src/Rector/Name/RenameToPsrNameRector.php
+            path: src/Rector/Name/RenameToConventionalCaseNameRector.php
         -
             identifier: complexity.functionLike
-            path: src/Rector/Name/RenameToPsrNameRector.php
+            path: src/Rector/Name/RenameToConventionalCaseNameRector.php
 
         -
             identifier: rector.noClassReflectionStaticReflection
diff --git a/rector.php b/rector.php
@@ -20,7 +20,7 @@
 use Guanguans\RectorRules\Contract\ThrowableContract;
 use Guanguans\RectorRules\Rector\Array_\SortListItemOfSameScalarTypeRector;
 use Guanguans\RectorRules\Rector\File\AddNoinspectionDocblockToFileFirstStmtRector;
-use Guanguans\RectorRules\Rector\Name\RenameToPsrNameRector;
+use Guanguans\RectorRules\Rector\Name\RenameToConventionalCaseNameRector;
 use Guanguans\RectorRules\Rector\New_\NewExceptionToNewAnonymousExtendsExceptionImplementsRector;
 use Illuminate\Support\Str;
 use PhpParser\NodeVisitor\ParentConnectingVisitor;
@@ -143,7 +143,7 @@
     ])
     ->withConfiguredRule(NewExceptionToNewAnonymousExtendsExceptionImplementsRector::class, [ThrowableContract::class])
     ->registerDecoratingNodeVisitor(ParentConnectingVisitor::class)
-    ->withConfiguredRule(RenameToPsrNameRector::class, [
+    ->withConfiguredRule(RenameToConventionalCaseNameRector::class, [
         'MIT',
     ])
     // ->withConfiguredRule(SortListItemOfSameScalarTypeRector::class, [
diff --git a/src/Rector/Name/RenameToConventionalCaseNameRector.php b/src/Rector/Name/RenameToConventionalCaseNameRector.php
@@ -61,9 +61,9 @@
 use function Guanguans\RectorRules\Support\is_instance_of_any;
 
 /**
- * @see \Guanguans\RectorRulesTests\Rector\Name\RenameToPsrNameRector\RenameToPsrNameRectorTest
+ * @see \Guanguans\RectorRulesTests\Rector\Name\RenameToConventionalCaseNameRector\RenameToConventionalCaseNameRectorTest
  */
-final class RenameToPsrNameRector extends AbstractRector implements ConfigurableRectorInterface
+final class RenameToConventionalCaseNameRector extends AbstractRector implements ConfigurableRectorInterface
 {
     /** @var list<string> */
     private array $except = [
diff --git a/tests/Rector/AbstractRectorTestCase.php b/tests/Rector/AbstractRectorTestCase.php
@@ -23,7 +23,7 @@
 
 namespace Guanguans\RectorRulesTests\Rector;
 
-use Guanguans\RectorRules\Rector\Name\RenameToPsrNameRector;
+use Guanguans\RectorRules\Rector\Name\RenameToConventionalCaseNameRector;
 use Illuminate\Support\Str;
 use PhpCsFixer\FileRemoval;
 use PhpParser\NodeVisitor\ParentConnectingVisitor;
@@ -37,7 +37,7 @@ abstract class AbstractRectorTestCase extends \Rector\Testing\PHPUnit\AbstractRe
      */
     final public static function setUpBeforeClass(): void
     {
-        // if (RenameToPsrNameRector::class !== static::rectorClass()) {
+        // if (RenameToConventionalCaseNameRector::class !== static::rectorClass()) {
         //     return;
         // }
 
diff --git a/tests/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector/Fixture/fixture_union_type.php.inc b/tests/Rector/Class_/UpdateRectorMethodNodeParamDocblockFromNodeTypesRector/Fixture/fixture_union_type.php.inc
@@ -10,7 +10,7 @@ use PhpParser\Node\Expr\Variable;
 use PhpParser\Node\Identifier;
 use PhpParser\Node\Name;
 
-final class RenameToPsrNameRector extends AbstractRector
+final class RenameToConventionalCaseNameRector extends AbstractRector
 {
     public function getNodeTypes(): array
     {
@@ -42,7 +42,7 @@ use PhpParser\Node\Expr\Variable;
 use PhpParser\Node\Identifier;
 use PhpParser\Node\Name;
 
-final class RenameToPsrNameRector extends AbstractRector
+final class RenameToConventionalCaseNameRector extends AbstractRector
 {
     public function getNodeTypes(): array
     {
diff --git a/tests/Rector/Name/RenameToConventionalCaseNameRector/Fixture/fixture.php.inc b/tests/Rector/Name/RenameToConventionalCaseNameRector/Fixture/fixture.php.inc
diff --git a/tests/Rector/Name/RenameToConventionalCaseNameRector/Fixture/fixture_variable_scope.php.inc b/tests/Rector/Name/RenameToConventionalCaseNameRector/Fixture/fixture_variable_scope.php.inc
diff --git a/tests/Rector/Name/RenameToConventionalCaseNameRector/Fixture/skip_non_renamed.php.inc b/tests/Rector/Name/RenameToConventionalCaseNameRector/Fixture/skip_non_renamed.php.inc
diff --git a/tests/Rector/Name/RenameToConventionalCaseNameRector/RenameToConventionalCaseNameRectorTest.php b/tests/Rector/Name/RenameToConventionalCaseNameRector/RenameToConventionalCaseNameRectorTest.php
@@ -18,15 +18,15 @@
  * @see https://github.com/guanguans/rector-rules
  */
 
-namespace Guanguans\RectorRulesTests\Rector\Name\RenameToPsrNameRector;
+namespace Guanguans\RectorRulesTests\Rector\Name\RenameToConventionalCaseNameRector;
 
 use Guanguans\RectorRulesTests\Rector\AbstractRectorTestCase;
 
 /**
  * @covers \Guanguans\RectorRules\Exception\RectorErrorException
- * @covers \Guanguans\RectorRules\Rector\Name\RenameToPsrNameRector
+ * @covers \Guanguans\RectorRules\Rector\Name\RenameToConventionalCaseNameRector
  */
-final class RenameToPsrNameRectorTest extends AbstractRectorTestCase
+final class RenameToConventionalCaseNameRectorTest extends AbstractRectorTestCase
 {
     protected function setUp(): void
     {
diff --git a/tests/Rector/Name/RenameToConventionalCaseNameRector/config/configured_rule.php b/tests/Rector/Name/RenameToConventionalCaseNameRector/config/configured_rule.php
@@ -19,7 +19,7 @@
  * @see https://github.com/guanguans/rector-rules
  */
 
-use Guanguans\RectorRules\Rector\Name\RenameToPsrNameRector;
+use Guanguans\RectorRules\Rector\Name\RenameToConventionalCaseNameRector;
 use PhpParser\NodeVisitor\ParentConnectingVisitor;
 use Rector\Config\RectorConfig;
 use Rector\Config\RegisteredService;
@@ -32,7 +32,7 @@
 //  */
 // return RectorConfig::configure()
 //     ->registerDecoratingNodeVisitor(ParentConnectingVisitor::class)
-//     ->withConfiguredRule(RenameToPsrNameRector::class, [
+//     ->withConfiguredRule(RenameToConventionalCaseNameRector::class, [
 //     ]);
 
 return static function (RectorConfig $rectorConfig): void {
@@ -45,8 +45,8 @@
     // $rectorConfig->tag($registeredService->getClassName(), $registeredService->getTag());
 
     // Ensure that using the default configuration is valid.
-    $rectorConfig->rule(RenameToPsrNameRector::class);
-    $rectorConfig->ruleWithConfiguration(RenameToPsrNameRector::class, [
+    $rectorConfig->rule(RenameToConventionalCaseNameRector::class);
+    $rectorConfig->ruleWithConfiguration(RenameToConventionalCaseNameRector::class, [
         'afterAll',
         'afterEach',
         'assertMatches*Snapshot',

Original file line number	Diff line number	Diff line change
`@@ -10,4 +10,4 @@ parameters:`
`10`	`10`	`-`
`11`	`11`	`message: '#^Cannot call method getParameterOption\(\) on mixed\.$#'`
`12`	`12`	`count: 1`
`13`		`- path: ../src/Rector/Name/RenameToPsrNameRector.php`
	`13`	`+ path: ../src/Rector/Name/RenameToConventionalCaseNameRector.php`
Original file line number	Diff line number	Diff line change
`@@ -5,4 +5,4 @@ parameters:`
`5`	`5`	`-`
`6`	`6`	`message: '#^Instead of "Closure" class/interface use "PhpParser\\Node\\Expr\\Closure"$#'`
`7`	`7`	`count: 1`
`8`		`- path: ../src/Rector/Name/RenameToPsrNameRector.php`
	`8`	`+ path: ../src/Rector/Name/RenameToConventionalCaseNameRector.php`
Original file line number	Diff line number	Diff line change
`@@ -142,10 +142,10 @@ parameters:`
`142`	`142`
`143`	`143`	`-`
`144`	`144`	`identifier: complexity.classLike`
`145`		`- path: src/Rector/Name/RenameToPsrNameRector.php`
	`145`	`+ path: src/Rector/Name/RenameToConventionalCaseNameRector.php`
`146`	`146`	`-`
`147`	`147`	`identifier: complexity.functionLike`
`148`		`- path: src/Rector/Name/RenameToPsrNameRector.php`
	`148`	`+ path: src/Rector/Name/RenameToConventionalCaseNameRector.php`
`149`	`149`
`150`	`150`	`-`
`151`	`151`	`identifier: rector.noClassReflectionStaticReflection`
Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ use PhpParser\Node\Expr\Variable;`
`10`	`10`	`use PhpParser\Node\Identifier;`
`11`	`11`	`use PhpParser\Node\Name;`
`12`	`12`
`13`		`-final class RenameToPsrNameRector extends AbstractRector`
	`13`	`+final class RenameToConventionalCaseNameRector extends AbstractRector`
`14`	`14`	`{`
`15`	`15`	`public function getNodeTypes(): array`
`16`	`16`	`{`
`@@ -42,7 +42,7 @@ use PhpParser\Node\Expr\Variable;`
`42`	`42`	`use PhpParser\Node\Identifier;`
`43`	`43`	`use PhpParser\Node\Name;`
`44`	`44`
`45`		`-final class RenameToPsrNameRector extends AbstractRector`
	`45`	`+final class RenameToConventionalCaseNameRector extends AbstractRector`
`46`	`46`	`{`
`47`	`47`	`public function getNodeTypes(): array`
`48`	`48`	`{`