[dx] load composer-based sets from laravel and drupal

Author: TomasVotruba

composer.json

@@ -41,8 +41,8 @@
         "webmozart/assert": "^2.4"
     },
     "require-dev": {
+        "driftingly/rector-laravel": "^2.6",
         "nette/robot-loader": "^4.1",
-        "symfony/process": "^8.1",
         "php-parallel-lint/php-parallel-lint": "^1.4",
         "phpstan/extension-installer": "^1.4",
         "phpstan/phpstan-deprecation-rules": "^2.0",
@@ -52,6 +52,7 @@
         "rector/jack": "^1.1",
         "rector/swiss-knife": "^2.4.1",
         "shipmonk/composer-dependency-analyser": "^1.8",
+        "symfony/process": "^8.1",
         "symplify/easy-coding-standard": "^13.2.13",
         "symplify/phpstan-rules": "^14.13",
         "tomasvotruba/class-leak": "^2.1",

src/Configuration/RectorConfigBuilder.php

@@ -32,6 +32,7 @@
 use Rector\Symfony\Set\TwigSetList;
 use Rector\ValueObject\Configuration\LevelOverflow;
 use Rector\ValueObject\PhpVersion;
+use RectorLaravel\Set\LaravelSetList;
 use Symfony\Component\Finder\Finder;
 use Webmozart\Assert\Assert;
 
@@ -56,17 +57,6 @@ final class RectorConfigBuilder
         'withCodingStyleLevel' => [SetList::CODING_STYLE, 'coding style'],
     ];
 
-    /**
-     * The composer-based set of the extensions that rector-src does not require, so their set list class cannot be
-     * imported here. Resolved at run-time; an extension that ships no such set falls back to its set group.
-     *
-     * @var array<SetGroup::*, string>
-     */
-    private const array EXTENSION_COMPOSER_BASED_SET_LISTS = [
-        SetGroup::LARAVEL => 'RectorLaravel\\Set\\LaravelSetList::COMPOSER_BASED',
-        SetGroup::DRUPAL => 'DrupalRector\\Set\\DrupalSetList::COMPOSER_BASED',
-    ];
-
     /**
      * @var string[]
      */
@@ -693,24 +683,13 @@ public function withComposerBased(
         bool $laravel = false,
         bool $drupal = false,
     ): self {
-        $setMap = [
-            SetGroup::LARAVEL => $laravel,
-            SetGroup::DRUPAL => $drupal,
-        ];
-
-        foreach ($setMap as $setGroup => $isEnabled) {
-            if (! $isEnabled) {
-                continue;
-            }
-
-            $setListConstant = self::EXTENSION_COMPOSER_BASED_SET_LISTS[$setGroup];
-            if (defined($setListConstant)) {
-                $setFilePath = constant($setListConstant);
-                Assert::string($setFilePath);
+        if ($laravel) {
+            $this->sets[] = LaravelSetList::COMPOSER_BASED;
+        }
 
-                // single set, as every rule inside is bound to the installed package version on its own
-                $this->sets[] = $setFilePath;
-            }
+        if ($drupal && class_exists('DrupalRector\Set\DrupalSetList')) {
+            // waits on https://github.com/palantirnet/drupal-rector/pull/419/files#diff-c6bd4ee854830efc1363a7d99c1b6a2e7e64f2499a51e503174ab777de7e64e5
+            $this->sets[] = DrupalSetList::COMPOSER_BASED;
         }
 
         if ($phpunit) {

src/Set/Enum/SetGroup.php

@@ -14,11 +14,13 @@ final class SetGroup
     public const string PHP = 'php';
 
     /**
+     * @deprecated Use composer-based.php set instead
      * Version-based set provider
      */
     public const string LARAVEL = 'laravel';
 
     /**
+     * @deprecated Use composer-based.php set instead
      * Version-based set provider
      */
     public const string DRUPAL = 'drupal';