[ci] add autoremoval of unused deps

.github/workflows/code_analysis.yaml

@@ -71,10 +71,6 @@ jobs:
                         name: 'Check no "*.php" files in rules Fixture directory'
                         run: php scripts/no-php-file-in-fixtures.php
 
-                    -
-                        name: 'Detect composer dependency issues'
-                        run: vendor/bin/composer-dependency-analyser
-
                     -
                         name: "PHP Linter"
                         run: |

.github/workflows/remove_unused_deps.yaml

@@ -0,0 +1,39 @@
+name: Remove Unused Dependencies
+
+on:
+    pull_request: null
+
+jobs:
+    remove_unused_deps:
+        strategy:
+            fail-fast: false
+
+        runs-on: ubuntu-latest
+        timeout-minutes: 8
+
+        if: github.event.pull_request.head.repo.full_name == 'rectorphp/rector-src'
+        steps:
+            -
+                uses: actions/checkout@v4
+                with:
+                    # Must be used to trigger workflow after push
+                    token: ${{ secrets.ACCESS_TOKEN }}
+
+            -
+                uses: shivammathur/setup-php@v2
+                with:
+                    php-version: 8.3
+                    coverage: none
+
+            -   run: composer install --no-progress --ansi
+
+            ## detect unused dependencies and remove them from composer.json
+            -   run: php scripts/remove-unused-deps.php --ansi
+
+            -
+                # commit only to core contributors who have repository access
+                uses: stefanzweifel/git-auto-commit-action@v5
+                with:
+                    commit_message: '[ci-review] Remove unused dependencies'
+                    commit_author: 'GitHub Action <actions@github.com>'
+                    commit_user_email: 'action@github.com'

phpstan.neon

@@ -445,3 +445,8 @@ parameters:
         -
             message: '#Access to deprecated property \$file of class Rector\\Rector\\AbstractRector#'
             path: src/Rector/AbstractRector.php
+
+        # allowed
+        -
+            message: '#"@\\simplexml_load_string\(\$junitXml\)" is forbidden to use#'
+            path: scripts/remove-unused-deps.php

scripts/remove-unused-deps.php

@@ -0,0 +1,48 @@
+<?php
+
+declare(strict_types=1);
+
+use Symfony\Component\Console\Input\ArrayInput;
+use Symfony\Component\Console\Output\ConsoleOutput;
+use Symfony\Component\Console\Style\SymfonyStyle;
+
+require __DIR__ . '/../vendor/autoload.php';
+
+$symfonyStyle = new SymfonyStyle(new ArrayInput([]), new ConsoleOutput());
+
+// 1. run the same workflow as CI "Detect composer dependency issues", in machine-readable format
+exec(__DIR__ . '/../vendor/bin/composer-dependency-analyser --format junit 2>/dev/null', $outputLines);
+$junitXml = implode("\n", $outputLines);
+
+$simpleXml = @simplexml_load_string($junitXml);
+if (! $simpleXml instanceof SimpleXMLElement) {
+    $symfonyStyle->error('Failed to parse composer-dependency-analyser output');
+    exit(1);
+}
+
+// 2. collect unused dependencies from the "unused dependencies" testsuite
+$unusedDependencies = [];
+foreach ($simpleXml->testsuite as $testsuite) {
+    if ((string) $testsuite['name'] !== 'unused dependencies') {
+        continue;
+    }
+
+    foreach ($testsuite->testcase as $testcase) {
+        $unusedDependencies[] = (string) $testcase['name'];
+    }
+}
+
+if ($unusedDependencies === []) {
+    $symfonyStyle->success('No unused dependencies found');
+    exit(0);
+}
+
+$symfonyStyle->listing($unusedDependencies);
+
+// 3. remove unused dependencies from composer.json (composer auto-detects require/require-dev)
+//    the autocommit to the branch is handled by the workflow's git-auto-commit-action
+foreach ($unusedDependencies as $unusedDependency) {
+    exec(sprintf('composer remove %s --no-update --no-interaction 2>&1', escapeshellarg($unusedDependency)));
+}
+
+$symfonyStyle->success(sprintf('Removed %d unused dependency(ies) from composer.json', count($unusedDependencies)));