chore(deps): Update dependency rector/rector to v2.5.0

[renovate]

This PR contains the following updates:

Package	Change	Age	Confidence
rector/rector (source)	2.4.5 → 2.5.0

---

Warning

Some dependencies could not be looked up. Check the Dependency Dashboard for more information.

---

Release Notes

rectorphp/rector (rector/rector)

v2.5.0: Released Rector 2.5

Compare Source

New Features 🥳 🎉 🎉 🎉

This release has 3 interesting new features. Let's look at them:

[dx] Report skips that never matched (#​8058)

• What? - like PHPStan's reportUnusedIgnores, but for Rector ->withSkip(). Flags skip entries that never matched anything during the run, so you can delete stale skips.

• Why? - skips rot. You skip a path/rule to dodge a problem, later the file moves or the rule stops firing there — the skip lingers forever,
  silently masking nothing. This surfaces dead skips so config stays honest.

// rector.php
return RectorConfig::configure()
    ->withSkip([
        SimplifyUselessVariableRector::class => [
            '*/src/Legacy/*',          // still matches — fine
            '*/NonexistentUnused/*',   // matches nothing — stale
        ],
    ])
    ->reportUnusedSkips();

Run output:

 [OK] Rector is done!
  
 [WARNING] This skip is unused, it never matched any element.
           You can remove it from "->withSkip()"

 * Rector\CodeQuality\Rector\FunctionLike\SimplifyUselessVariableRector => */NonexistentUnused/*

---

[dx] Removing unused imports by default (#​8047)

You can update your rector.php config:

 return RectorConfig::configure()
-    ->withImportNames(removeUnusedImports: true);
+    ->withImportNames();

In case it's not for you, turn it off:

 return RectorConfig::configure()
-    ->withImportNames();
+    ->withImportNames(removeUnusedImports: false);

---

[dx] Introducing Drupal composer-based sets (#​8041), Thanks @​bbrala!

If you're using Drupal Rector, you can now enable it's per-version sets via:

 return RectorConfig::configure()
+    ->withComposerBased(drupal: true);

To learn more about composer-based-sets, checkout the documentation.

New Rules 🎉

• [php 8.4] [type-declaration] Add AddArrayAnyAllClosureParamTypeRector and NarrowArrayAnyAllNullableParamTypeRector (#​8049)
• [code-quality] Add MoveInnerFunctionToTopLevelRector (#​8042)
• [code-quality] Add NewArrayItemConcatAssignToAssignRector (#​8045)
• [code-quality] Add FixClassCaseSensitivityVarDocblockRector (#​8046)
• [polyfills] add missing polyfill to PHP 8.4 array functions (#​8052)
• [rector] Add AddParamTypeToRefactorMethodRector if missing (#​8061)

Bugfixes 🐛

• [internal] Streamline use imports management to FileNode (#​8040)
• [PostRector] Do not keep an unused import matched only by a partial docblock name's tail (#​8043), Thanks @​ruudk!
• [CodeQuality] Skip native function on MoveInnerFunctionToTopLevelRector (#​8044)
• Fix --only runs caching files as unchanged, hiding pending changes from full runs (#​8029), Thanks @​SanderMuller!
• [fix] Fix RemoveUnusedPrivateMethodRector for NeverType (#​8050)
• [dead-code] Fix RemoveUnusedVariableAssignRector, allow for SplFileInfo as cleanup on purpose for gc (#​8054)
• [fixes] Couple ClassPropertyAssignToConstructorPromotionRector, RemoveAlwaysTrueIfConditionRector and RemoveUnusedVariableAssignRector fixes (#​8055)
• [TypeDeclaration][DeadCode] Skip class with Doctrine static function mapping (loadMetadata) in TypedPropertyFromAssignsRector and RemoveUnusedPrivatePropertyRector (#​8059)
• [TypeDeclaration] Skip class with Doctrine static function mapping (loadMetadata) in TypedPropertyFromStrictConstructorRector (#​8060)

rectorphp/rector-doctrine 🟠

• [NodeAnalyzer] Detect Doctrine static function mapping (loadMetadata) entity in DoctrineEntityDetector (736bf61)

v2.4.6: Released Rector 2.4.6

Compare Source

New Features 🥳

• [deprecation] Add RenameDeprecatedMethodCallRector inferring rename from @​deprecated docblock (#​8015)
• [sets] kick of named args set (#​8013)

# rector.php
return (RectorConfig::configure())
    ->withPreparedSets(namedArgs: true);

• Add Symfony 8 to the set provider collector (#​8019), Thanks @​mbabker!

Bugfixes 🐛

• [BetterPhpDocParser] Keep import referenced by @​see/@​uses tag with a trailing description (#​8039), Thanks @​ruudk!
• [internals] skip and finalize beforeTraverse() and afterTraverse() as never used, use refactor() instead (#​7765)
• [DeadCode] Skip (void) cast with #[NoDiscard] on target method call on RemoveDeadStmtRector (#​8038)
• Bump composer/pcre to ^3.4.0 and phpstan/phpstan to ^2.2.2 (#​8037)
• [CodeQuality] Handle crash on custom exception not autoloaded on ThrowWithPreviousExceptionRector (#​8036)
• refactor: extract AutoloadFileParameterResolver from bin, add tests (#​8035), Thanks @​SanderMuller!
• Fix cached results surviving a change of --autoload-file (#​8034), Thanks @​SanderMuller!
• [CodingStyle] Remove AstResolver usage on ArrowFunctionAndClosureFirstClassCallableGuard (#​8031)
• [DeadCode] Remove AstResolver on RemoveParentCallWithoutParentRector (#​8032)
• [DeadCode] Skip with use of func_num_args() on RemoveNullArgOnNullDefaultParamRector (#​8030)
• [Php70] Skip rewriting to $this on static method or static closure on StaticCallOnNonStaticToInstanceCallRector (#​8026)
• [ci] add compat test (#​8025)
• Fix ReplaceArgumentDefaultValueRector generating invalid self:: constant in unrelated classes (#​8023)
• Fix RemoveDeadIfBlockRector dropping else block when merging empty if with elseif (#​8022)
• Fix RemoveParentCallWithoutParentRector removing valid call when ancestor hierarchy is unresolvable (#​8018)
• split of RemoveNullNamedArgOnNullDefaultParamRector to handle only named args (#​8014)
• Add failing tests (#​8012), Thanks @​u01jmg3!
• Update rector/swiss-knife version to ^2.4.1 (#​8011)

rectorphp/rector-symfony 🎶

• [Symfony81] Add new rule for deprecated validator test usages (#​948), Thanks @​florianhofsaessC24!
• fix: only convert Twig extensions fully reducible to AsTwig attributes (#​947)
• [Symfony73] Sort optional parameters last in InvokableCommandInputAttributeRector (#​945)
• Add PHPStan rule requiring nested set configs to be imported in parent set config (#​943)
• [Symfony81] Add new rule for Security component (#​942), Thanks @​MrYamous!
• Remove unused imports (#​941), Thanks @​MrYamous!
• [Symfony81] Add new rule for Filesystem (#​940), Thanks @​MrYamous!

rectorphp/rector-phpunit 🟢

• [PHPUnit12] Handle crash on property not exists on PropertyCreateMockToCreateStubRector (#​681)
• [AnnotationsToAttributes] Convert external @​depends ClassName::method to #[DependsExternal] (#​679)
• [PHPUnit12] Drop ConstraintValidatorTestCase from AllowMockObjectsWhereParentClassRector triggers (#​678)

rectorphp/rector-downgrade-php ⬇️

• Add DowngradeDomNodeChildNodesForeachRector for null $childNodes before PHP 8.0 (#​379)
• Update boundwize/structarmed version to ^0.9 (#​378)
• Fix use of existing PhpAttributeAnalyzer service on AddReturnTypeWillChangeAttributeRector (#​377)
• Remove no longer exists '--ansi' flag from swiss-knife (#​376)
• [DowngradePhp81] Add AddReturnTypeWillChangeAttributeRector (#​372), Thanks @​jquiaios!

---

Configuration

📅 Schedule: (UTC)

• Branch creation
  • At any time (no schedule defined)
• Automerge
  • At any time (no schedule defined)

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

---

• If you want to rebase/retry this PR, check this box

---

This PR was generated by Mend Renovate. View the repository job log.