ci: add spiral/code-style with php-cs-fixer

.github/workflows/cs-fix.yml

@@ -0,0 +1,17 @@
+---
+
+on:  # yamllint disable-line rule:truthy
+  push:
+    branches:
+      - '**'
+
+name: Fix Code Style
+
+jobs:
+  cs-fix:
+    permissions:
+      contents: write
+    uses: spiral/gh-actions/.github/workflows/cs-fix.yml@master
+    with:
+      php: >-
+        ['8.4']

.gitignore

@@ -19,3 +19,6 @@ Thumbs.db
 clover.xml
 .env
 builds
+
+# Runtime
+runtime/

.php-cs-fixer.dist.php

@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types=1);
+
+require_once 'vendor/autoload.php';
+
+return \Spiral\CodeStyle\Builder::create()
+    ->include(__DIR__ . '/src')
+    ->include(__DIR__ . '/tests')
+    ->include(__FILE__)
+    ->cache('./runtime/php-cs-fixer.cache')
+    ->allowRisky(false)
+    ->build();

composer.json

@@ -49,6 +49,7 @@
     "require-dev": {
         "mockery/mockery": "^1.5",
         "phpunit/phpunit": "^10.0",
+        "spiral/code-style": "^2.3.1",
         "vimeo/psalm": ">=5.8"
     },
     "autoload-dev": {
@@ -57,6 +58,8 @@
         }
     },
     "scripts": {
+        "cs:diff": "php-cs-fixer fix --dry-run -v --diff",
+        "cs:fix": "php-cs-fixer fix -v",
         "tests": "phpunit",
         "psalm": "psalm --no-cache"
     },

src/Exception/ServiceException.php

@@ -4,7 +4,4 @@
 
 namespace Spiral\RoadRunner\Services\Exception;
 
-class ServiceException extends \Exception
-{
-
-}
+class ServiceException extends \Exception {}

src/Manager.php

@@ -75,7 +75,7 @@ public function create(
         array $env = [],
         int $restartSec = 30,
         bool $serviceNameInLogs = false,
-        int $stopTimeout = 5
+        int $stopTimeout = 5,
     ): bool {
         \assert($processNum > 0, 'Process number must be greater than 0.');
         \assert($execTimeout >= 0, 'Execution timeout must be greater or equal to 0.');
@@ -181,7 +181,7 @@ public function statuses(string $name): array
                     $error = [
                         'code' => $statusError->getCode(),
                         'message' => $statusError->getMessage(),
-                        'details' => \array_map(static fn (Any $any) => [
+                        'details' => \array_map(static fn(Any $any) => [
                             'message' => $any->getValue(),
                             'type_url' => $any->getTypeUrl(),
                         ], \iterator_to_array($statusError->getDetails()->getIterator())),

tests/ManagerTest.php

@@ -22,21 +22,6 @@ final class ManagerTest extends TestCase
     private Manager $manager;
     private m\LegacyMockInterface|m\MockInterface|RPCInterface $rpc;
 
-    protected function setUp(): void
-    {
-        parent::setUp();
-
-        $this->rpc = m::mock(RPCInterface::class);
-
-        $this->rpc
-            ->shouldReceive('withCodec')
-            ->once()
-            ->withArgs(static fn($codec): bool => $codec instanceof ProtobufCodec)
-            ->andReturnSelf();
-
-        $this->manager = new Manager($this->rpc);
-    }
-
     public function testListServices(): void
     {
         $this->rpc
@@ -96,8 +81,8 @@ public function testServiceShouldBeCreated(): void
                 ['FOO' => 'bar', 'BAZ' => 'foo'],
                 50,
                 true,
-                10
-            )
+                10,
+            ),
         );
     }
 
@@ -197,7 +182,7 @@ public function testServiceStatusesShouldBeReturned(): void
                             ]),
                         ]),
                     ],
-                ])
+                ]),
             );
 
         $status = $this->manager->statuses('foo');
@@ -218,4 +203,19 @@ public function testServiceStatusesShouldBeReturned(): void
             ],
         ], $status);
     }
+
+    protected function setUp(): void
+    {
+        parent::setUp();
+
+        $this->rpc = m::mock(RPCInterface::class);
+
+        $this->rpc
+            ->shouldReceive('withCodec')
+            ->once()
+            ->withArgs(static fn($codec): bool => $codec instanceof ProtobufCodec)
+            ->andReturnSelf();
+
+        $this->manager = new Manager($this->rpc);
+    }
 }

tests/TestCase.php

@@ -6,7 +6,4 @@
 
 use PHPUnit\Framework\TestCase as BaseTestCase;
 
-abstract class TestCase extends BaseTestCase
-{
-
-}
+abstract class TestCase extends BaseTestCase {}