ci: add rector

Signed-off-by: Emilien Escalle <emilien.escalle@escemi.com>

Author: neilime

.github/workflows/__shared-ci.yml

@@ -24,25 +24,25 @@ jobs:
           php-version: ${{ matrix.php-versions }}
           extensions: none,iconv,dom,curl,mbstring,tokenizer,xml,xmlwriter,simplexml,ctype
           coverage: pcov
-  
+
       - name: ♻️ Get composer cache directory
         id: composer-cache
         shell: bash
         run: echo "dir=$(composer config cache-files-dir)" >> "$GITHUB_OUTPUT"
-  
+
       - name: ♻️ Cache composer dependencies
         uses: actions/cache@v4
         with:
           path: ${{ steps.composer-cache.outputs.dir }}
           key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.json') }}
           restore-keys: ${{ runner.os }}-composer-
-  
+
       - name: ⚙️ Install dependencies
         shell: bash
         run: |
           composer install --no-progress --prefer-dist --optimize-autoloader
           composer --working-dir=tools install --no-progress --prefer-dist --optimize-autoloader
-  
+
       - name: ♻️ Tools cache
         uses: actions/cache@v4
         with:
@@ -55,9 +55,25 @@ jobs:
         if: matrix.stable
         run: composer php-cs-fixer -- --format=checkstyle | tools/vendor/bin/cs2pr
 
+      - name: 🔬 Rector
+        id: rector
+        if: matrix.stable
+        run: |
+          REPORT=$(composer rector -- --output-format json || true)
+          echo "report<<EOF" >> "$GITHUB_OUTPUT" && echo "$REPORT" >> "$GITHUB_OUTPUT" && echo "EOF" >> "$GITHUB_OUTPUT"
+
+      - uses: actions/github-script@v6
+        if: matrix.stable
+        with:
+          script: |
+            console.log(`::group::Rector report`);
+            console.log(`::error file=src/CssLint/CliArgs.php,line=41::StrStartsWithRector / NullToStrictStringFuncCallArgRector%0A%0A--- Original%0A+++ New%0A@@ -42,13 +42,13 @@%0A %0A         foreach ($aArguments as $sArgument) {%0A             // --foo --bar=baz%0A-            if (substr($sArgument, 0, 2) == '--') {%0A-                $sEqualPosition = strpos($sArgument, '=');%0A+            if (str_starts_with((string) $sArgument, '--')) {%0A+                $sEqualPosition = strpos((string) $sArgument, '=');%0A %0A                 // --bar=baz%0A                 if ($sEqualPosition !== false) {%0A-                    $sKey = substr($sArgument, 2, $sEqualPosition - 2);%0A-                    $sValue = substr($sArgument, $sEqualPosition + 1);%0A+                    $sKey = substr((string) $sArgument, 2, $sEqualPosition - 2);%0A+                    $sValue = substr((string) $sArgument, $sEqualPosition + 1);%0A                     $aParsedArguments[$sKey] = $sValue;%0A                 }%0A             }%0A`);
+            console.log(`::error file=src/CssLint/Linter.php,line=388::StrStartsWithRector%0A%0A--- Original%0A+++ New%0A@@ -389,7 +389,7 @@%0A             $sPropertyName = trim($this->getContextContent());%0A %0A             // Ignore CSS variables (names starting with --)%0A-            if (substr($sPropertyName, 0, 2) === '--') {%0A+            if (str_starts_with($sPropertyName, '--')) {%0A                 $this->setContext(self::CONTEXT_PROPERTY_CONTENT);%0A                 return true;%0A             }%0A`);
+            console.log(`::endgroup::`);
+
       - name: 🔬 Static analysis
         if: matrix.stable
-        run: composer stan -- --error-format=checkstyle | tools/vendor/bin/cs2pr
+        run: composer stan -- --error-format=github
 
       - name: ♻️ Tests cache
         uses: actions/cache@v4
@@ -66,7 +82,7 @@ jobs:
           key: ${{ runner.os }}-tests-${{ github.sha }}
           restore-keys: |
             ${{ runner.os }}-tests-
-      
+
       - name: 🧪 Test
         run: composer test:ci
 

Makefile

@@ -34,7 +34,13 @@ lint: ## Execute lint for given PHP version
 	@$(call run-php,composer php-cs-fixer $(filter-out $@,$(MAKECMDGOALS)))
 
 lint-fix: ## Execute lint fixing for given PHP version
-	@$(call run-php,composer php-cs-fixer:fix $(filter-out $@,$(MAKECMDGOALS)))
+	@$(call run-php,composer php-cs-fixer:fix $(filter-out $@,$(MAKECMDGOALS)))	
+
+rector: ## Execute rector for given PHP version
+	@$(call run-php,composer rector $(filter-out $@,$(MAKECMDGOALS)))
+
+rector-fix: ## Execute rector fixing for given PHP version
+	@$(call run-php,composer rector:fix $(filter-out $@,$(MAKECMDGOALS)))
 
 stan: ## Execute PHPStan for given PHP version
 	@$(call run-php,composer stan $(filter-out $@,$(MAKECMDGOALS)))

composer.json

@@ -49,9 +49,12 @@
     "test:ci": "@test -d pcov.enabled=1 -d max_execution_time=0 --coverage-text --coverage-clover ./build/logs/clover.xml --coverage-html ./build/coverage/",
     "php-cs-fixer": "@php-cs-fixer:fix --dry-run",
     "php-cs-fixer:fix": "tools/vendor/bin/php-cs-fixer fix --show-progress=dots --diff --config=.php-cs-fixer.dist.php",
-    "stan": "tools/vendor/bin/phpstan analyse --level 5 src",
+    "rector": "@rector:fix --dry-run",
+    "rector:fix": "tools/vendor/bin/rector process src",
+    "stan": "tools/vendor/bin/phpstan analyse --level max src",
     "ci": [
       "@php-cs-fixer",
+      "@rector",
       "@stan",
       "@test:ci"
     ]

rector.php

@@ -0,0 +1,17 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\Caching\ValueObject\Storage\FileCacheStorage;
+use Rector\Config\RectorConfig;
+
+return RectorConfig::configure()
+    ->withPaths([
+        __DIR__ . '/src',
+        __DIR__ . '/tests',
+    ])
+    ->withPhpSets()
+    ->withCache(
+        cacheClass: FileCacheStorage::class,
+        cacheDirectory: __DIR__ . '/tools/cache/rector'
+    );;

src/CssLint/CliArgs.php

@@ -9,22 +9,22 @@ class CliArgs
 
     /**
      * Constructor
-     * @param array $aArguments arguments to be parsed (@see $_SERVER['argv'])
+     * @param array $arguments arguments to be parsed (@see $_SERVER['argv'])
      *              Accepts "-o", "--options" '{}'
      *              Accepts a string as last argument, a file path or a string containing CSS
      */
-    public function __construct(array $aArguments)
+    public function __construct(array $arguments)
     {
-        if (empty($aArguments) || count($aArguments) === 1) {
+        if (empty($arguments) || count($arguments) === 1) {
             return;
         }
 
-        array_shift($aArguments);
+        array_shift($arguments);
 
-        $this->filePathOrCssString = array_pop($aArguments);
+        $this->filePathOrCssString = array_pop($arguments);
 
-        if ($aArguments) {
-            $aParsedArguments = $this->extractArguments($aArguments);
+        if ($arguments) {
+            $aParsedArguments = $this->extractArguments($arguments);
 
             if (!empty($aParsedArguments['options'])) {
                 $this->options = $aParsedArguments['options'];
@@ -33,14 +33,14 @@ public function __construct(array $aArguments)
     }
 
     /**
-     * @param array $aArguments array of arguments to be parsed (@see $_SERVER['argv'])
+     * @param array $arguments array of arguments to be parsed (@see $_SERVER['argv'])
      * @return array an associative array of key=>value arguments
      */
-    private function extractArguments(array $aArguments): array
+    private function extractArguments(array $arguments): array
     {
         $aParsedArguments = [];
 
-        foreach ($aArguments as $sArgument) {
+        foreach ($arguments as $sArgument) {
             // --foo --bar=baz
             if (substr($sArgument, 0, 2) == '--') {
                 $sEqualPosition = strpos($sArgument, '=');