build(deps-dev): updated rector/rector to ^2.1.4

Author: mickverm

composer.json

@@ -18,7 +18,7 @@
     "require-dev": {
         "doctrine/coding-standard": "^12.0",
         "phpunit/phpunit": "^10.5.35",
-        "rector/rector": "^1.2",
+        "rector/rector": "^2.1.4",
         "squizlabs/php_codesniffer": "^3.7",
         "vimeo/psalm": "6.5.*"
     },

rector.php

@@ -5,38 +5,29 @@
 use Rector\Php70\Rector\StmtsAwareInterface\IfIssetToCoalescingRector;
 use Rector\Php71\Rector\FuncCall\RemoveExtraParametersRector;
 use Rector\Php80\Rector\Switch_\ChangeSwitchToMatchRector;
-use Rector\PHPUnit\PHPUnit100\Rector\Class_\StaticDataProviderClassMethodRector;
-use Rector\PHPUnit\Set\PHPUnitSetList;
-use Rector\Set\ValueObject\LevelSetList;
 
-return static function (RectorConfig $rectorConfig): void {
-    $rectorConfig->paths([
+return RectorConfig::configure()
+    ->withPaths([
         __DIR__ . '/examples',
         __DIR__ . '/src',
         __DIR__ . '/tests',
         __DIR__ . '/tools',
-    ]);
-
-    // Modernize code
-    $rectorConfig->sets([
-        LevelSetList::UP_TO_PHP_74,
-        PHPUnitSetList::PHPUNIT_100,
-    ]);
-
-    $rectorConfig->rule(ChangeSwitchToMatchRector::class);
-    $rectorConfig->rule(StaticDataProviderClassMethodRector::class);
-
+    ])
+    ->withPhpSets(php74: true)
+    ->withComposerBased(phpunit: true)
+    ->withRules([
+        ChangeSwitchToMatchRector::class,
+    ])
+    // All classes are public API by default, unless marked with @internal.
+    ->withConfiguredRule(RemoveAnnotationRector::class, ['api'])
     // phpcs:disable Squiz.Arrays.ArrayDeclaration.KeySpecified
-    $rectorConfig->skip([
+    ->withSkip([
         RemoveExtraParametersRector::class,
         // Do not use ternaries extensively
         IfIssetToCoalescingRector::class,
         ChangeSwitchToMatchRector::class => [
             __DIR__ . '/tests/SpecTests/Operation.php',
         ],
-    ]);
+    ])
     // phpcs:enable
-
-    // All classes are public API by default, unless marked with @internal.
-    $rectorConfig->ruleWithConfiguration(RemoveAnnotationRector::class, ['api']);
-};
+    ->withImportNames(importNames: false, removeUnusedImports: true);

src/GridFS/Bucket.php

@@ -35,7 +35,6 @@
 use MongoDB\GridFS\Exception\StreamException;
 use MongoDB\Model\BSONArray;
 use MongoDB\Model\BSONDocument;
-use MongoDB\Operation\Find;
 
 use function array_intersect_key;
 use function array_key_exists;

tools/connect.php

@@ -2,7 +2,7 @@
 
 function getHosts(string $uri): array
 {
-    if (strpos($uri, '://') === false) {
+    if (! str_contains($uri, '://')) {
         return [$uri];
     }
 

tools/detect-extension.php

@@ -5,11 +5,11 @@ function grepIniFile(string $filename, string $extension): int
     $lines = [];
 
     foreach (new SplFileObject($filename) as $i => $line) {
-        if (strpos($line, 'extension') === false) {
+        if (! str_contains($line, 'extension')) {
             continue;
         }
 
-        if (strpos($line, $extension) === false) {
+        if (! str_contains($line, $extension)) {
             continue;
         }