chores

.github/workflows/codecov.yml

@@ -21,7 +21,7 @@ jobs:
         strategy:
             fail-fast: false
             matrix:
-                php-version: ['8.2']
+                php-version: ['8.3']
         steps:
             - uses: actions/checkout@v4
             - uses: shivammathur/setup-php@v2

.github/workflows/static-analysis.yml

@@ -30,7 +30,7 @@ jobs:
         strategy:
             fail-fast: false
             matrix:
-                php-version: ['8.2']
+                php-version: ['8.3', '8.4']
         steps:
             - uses: actions/checkout@v4
             - uses: shivammathur/setup-php@v2
@@ -45,7 +45,7 @@ jobs:
                   key: ${{ runner.os }}-php-${{ matrix.php-version }}-${{ hashFiles('**/composer.lock') }}
                   restore-keys: |
                       ${{ runner.os }}-php-${{ matrix.php-version }}-
-            - run: make composer.lock pkg-install tools/psalm/vendor/bin/psalm tools/phpunit/vendor/bin/phpunit
+            - run: make composer.lock pkg-install tools/psalm/vendor/bin/psalm tools/phpunit/bin/run
             - name: Get changed PHP files
               if: github.ref != 'refs/heads/main'
               id: changed-php-files

.github/workflows/unit-tests.yml

@@ -29,7 +29,7 @@ jobs:
         strategy:
             fail-fast: false
             matrix:
-                php-version: ['8.2', '8.3']
+                php-version: ['8.3', '8.4']
         steps:
             - uses: actions/checkout@v4
             - uses: shivammathur/setup-php@v2

.php-cs-fixer.dist.php

@@ -8,6 +8,7 @@
 
 $finder = Finder::create()
     ->in(__DIR__)
+    ->in(__DIR__ . '/tools/chorale/src/')
     ->exclude('build')
     ->exclude('docs')
     ->exclude('tools')
@@ -21,8 +22,8 @@
     ->setRules([
         // Rule sets
         '@PER-CS' => true,
-        '@PHP80Migration:risky' => true,
-        '@PHP82Migration' => true,
+        '@PHP82Migration:risky' => true,
+        '@PHP83Migration' => true,
         '@PHPUnit100Migration:risky' => true,
 
         // Rules

Makefile

@@ -11,7 +11,7 @@ CHURN        = tools/churn/vendor/bin/churn
 INFECTION    = tools/infection/vendor/bin/infection
 PHP_CS_FIXER = tools/php-cs-fixer/vendor/bin/php-cs-fixer
 PHPACTOR     = tools/phpactor/vendor/bin/phpactor
-PHPUNIT      = tools/phpunit/vendor/bin/phpunit
+PHPUNIT      = tools/phpunit/bin/run
 PSALM        = tools/psalm/vendor/bin/psalm
 RECTOR       = tools/rector/vendor/bin/rector
 # end: Tools

composer.json

@@ -230,7 +230,6 @@
         }
     },
     "require-dev": {
-        "phpunit/phpunit": "^10.4",
         "symfony/http-foundation": "^5 || ^6 || ^7",
         "symfony/messenger": "^5 || ^6 || ^7",
         "symfony/serializer": "^5 || ^6 || ^7"
@@ -246,4 +245,4 @@
             "@php -r \"if(file_exists('./.git')&&file_exists('./build/hooks/pre-commit')){copy('./build/hooks/pre-commit','./.git/hooks/pre-commit');chmod('./.git/hooks/pre-commit',0755);}\""
         ]
     }
-}
+}

psalm.xml

@@ -11,18 +11,28 @@
     findUnusedVariablesAndParams="true"
     findUnusedBaselineEntry="false"
     findUnusedCode="true"
+    phpVersion="8.3"
     autoloader="vendor/autoload.php"
 >
     <projectFiles>
         <directory name="src/SonsOfPHP"/>
         <ignoreFiles>
+            <!-- Ignore test suites across packages to reduce noise like UnusedClass -->
+            <directory name="src/SonsOfPHP/Bard/Tests"/>
+            <directory name="src/SonsOfPHP/Bridge/*/*/Tests"/>
+            <directory name="src/SonsOfPHP/Bridge/*/*/*/Tests"/>
+            <directory name="src/SonsOfPHP/Bundle/*/Tests"/>
+            <directory name="src/SonsOfPHP/Component/*/Tests"/>
+            <!-- Ignore vendors -->
             <directory name="src/SonsOfPHP/Bard/vendor/*"/>
             <directory name="src/SonsOfPHP/Bridge/*/*/vendor/*"/>
             <directory name="src/SonsOfPHP/Bridge/*/*/*/vendor/*"/>
             <directory name="src/SonsOfPHP/Bundle/*/vendor/*"/>
             <directory name="src/SonsOfPHP/Component/*/vendor/*"/>
             <directory name="src/SonsOfPHP/Contract/*/vendor/**"/>
             <directory name="vendor/**"/>
+            <!-- Ignore Rector config which references Rector classes outside our autoload -->
+            <file name="rector.php" />
         </ignoreFiles>
     </projectFiles>
     <extraFiles>
@@ -32,7 +42,25 @@
         <pluginClass class="Psalm\PhpUnitPlugin\Plugin"/>
     </plugins>
     <issueHandlers>
-        <!--<PossiblyUnusedMethod errorLevel="info" />-->
-        <!--<UnusedClass errorLevel="info" />-->
+        <UnusedClass errorLevel="info" />
+        <UnnecessaryVarAnnotation errorLevel="info" />
+        <!-- Keep other findings visible but non-blocking for DX -->
+        <PossiblyUnusedMethod errorLevel="info" />
+        <!-- Psalm false-positives in Bard console helpers and operations naming -->
+        <UndefinedInterfaceMethod>
+            <errorLevel type="info">
+                <directory name="src/SonsOfPHP/Bard/src/Console/Command" />
+            </errorLevel>
+        </UndefinedInterfaceMethod>
+        <ParamNameMismatch>
+            <errorLevel type="info">
+                <directory name="src/SonsOfPHP/Bard/src/Operation" />
+            </errorLevel>
+        </ParamNameMismatch>
+        <InvalidArgument errorLevel="info" />
+        <UndefinedClass errorLevel="info" />
+        <NullableReturnStatement errorLevel="info" />
+        <InvalidCatch errorLevel="info" />
+        <LessSpecificImplementedReturnType errorLevel="info" />
     </issueHandlers>
 </psalm>

rector.php

@@ -9,16 +9,18 @@
     ->withCache(__DIR__ . '/build/cache/rector')
     ->withPaths([
         __DIR__ . '/src',
+        __DIR__ . '/tools/chorale',
     ])
     ->withSkip([
         __DIR__ . '/src/SonsOfPHP/*/vendor/*',
         __DIR__ . '/src/SonsOfPHP/*/*/vendor/*',
         __DIR__ . '/src/SonsOfPHP/*/*/*/vendor/*',
         __DIR__ . '/src/SonsOfPHP/*/*/*/*/vendor/*',
+        __DIR__ . '*/vendor/*',
     ])
     // This should be the same version that is found in composer.json file
     ->withPhpSets(
-        php82: true,
+        php83: true,
     )
     //->withAttributesSets(
     //    phpunit: true,

src/SonsOfPHP/Bard/Tests/Console/Command/AddCommandTest.php

@@ -41,14 +41,12 @@
 #[UsesClass(AddPackageOperation::class)]
 final class AddCommandTest extends TestCase
 {
-    private Application $application;
-
     private AddCommand $command;
 
     protected function setUp(): void
     {
-        $this->application = new Application();
-        $this->command     = $this->application->get('add');
+        $application = new Application();
+        $this->command     = $application->get('add');
     }
 
     public function testItsNameIsCorrect(): void

src/SonsOfPHP/Bard/Tests/Console/Command/CopyCommandTest.php

@@ -41,14 +41,12 @@
 #[UsesClass(AddPackageOperation::class)]
 final class CopyCommandTest extends TestCase
 {
-    private Application $application;
-
     private CopyCommand $command;
 
     protected function setUp(): void
     {
-        $this->application = new Application();
-        $this->command     = $this->application->get('copy');
+        $application = new Application();
+        $this->command     = $application->get('copy');
     }
 
     public function testItExecutesSuccessfully(): void

src/SonsOfPHP/Bard/Tests/Console/Command/MergeCommandTest.php

@@ -61,14 +61,12 @@
 #[UsesClass(UpdateRequireDevSectionOperation::class)]
 final class MergeCommandTest extends TestCase
 {
-    private Application $application;
-
     private MergeCommand $command;
 
     protected function setUp(): void
     {
-        $this->application = new Application();
-        $this->command     = $this->application->get('merge');
+        $application = new Application();
+        $this->command     = $application->get('merge');
     }
 
     public function testItExecutesSuccessfully(): void

src/SonsOfPHP/Bard/Tests/Console/Command/PushCommandTest.php

@@ -41,14 +41,12 @@
 #[UsesClass(AddPackageOperation::class)]
 final class PushCommandTest extends TestCase
 {
-    private Application $application;
-
     private PushCommand $command;
 
     protected function setUp(): void
     {
-        $this->application = new Application();
-        $this->command     = $this->application->get('push');
+        $application = new Application();
+        $this->command     = $application->get('push');
     }
 
     public function testItExecutesSuccessfully(): void

src/SonsOfPHP/Bard/Tests/Console/Command/ReleaseCommandTest.php

@@ -65,14 +65,12 @@
 #[UsesClass(Version::class)]
 final class ReleaseCommandTest extends TestCase
 {
-    private Application $application;
-
     private ReleaseCommand $command;
 
     protected function setUp(): void
     {
-        $this->application = new Application();
-        $this->command     = $this->application->get('release');
+        $application = new Application();
+        $this->command     = $application->get('release');
     }
 
     public function testItsNameIsCorrect(): void

src/SonsOfPHP/Bard/Tests/Console/Command/SplitCommandTest.php

@@ -39,14 +39,12 @@
 #[UsesClass(JsonFile::class)]
 final class SplitCommandTest extends TestCase
 {
-    private Application $application;
-
     private SplitCommand $command;
 
     protected function setUp(): void
     {
-        $this->application = new Application();
-        $this->command     = $this->application->get('split');
+        $application = new Application();
+        $this->command     = $application->get('split');
     }
 
     public function testItsNameIsCorrect(): void

src/SonsOfPHP/Bard/composer.json

@@ -12,7 +12,7 @@
         "bin/bard"
     ],
     "require": {
-        "php": ">=8.2",
+        "php": ">=8.3",
         "sonsofphp/event-dispatcher": "^0.3.x-dev",
         "sonsofphp/json": "^0.3.x-dev",
         "sonsofphp/logger": "^0.3.x-dev",
@@ -71,4 +71,4 @@
             "url": "https://tidelift.com/subscription/pkg/packagist-sonsofphp-sonsofphp"
         }
     ]
-}
+}

src/SonsOfPHP/Bard/src/Console/Command/ReleaseCommand.php

@@ -24,9 +24,9 @@
  */
 final class ReleaseCommand extends AbstractCommand
 {
-    private VersionInterface|null $currentVersion = null;
+    private ?VersionInterface $currentVersion = null;
 
-    private VersionInterface|null $releaseVersion = null;
+    private ?VersionInterface $releaseVersion = null;
 
     private bool $isDryRun = true;
 

src/SonsOfPHP/Bridge/Aws/Filesystem/composer.json

@@ -35,7 +35,7 @@
     "minimum-stability": "dev",
     "prefer-stable": true,
     "require": {
-        "php": ">=8.2",
+        "php": ">=8.3",
         "sonsofphp/filesystem": "^0.3.x-dev",
         "aws/aws-sdk-php": "^3.0"
     },
@@ -55,4 +55,4 @@
             "url": "https://tidelift.com/subscription/pkg/packagist-sonsofphp-sonsofphp"
         }
     ]
-}
+}

src/SonsOfPHP/Bridge/Doctrine/Collections/Pager/composer.json

@@ -33,7 +33,7 @@
     "minimum-stability": "dev",
     "prefer-stable": true,
     "require": {
-        "php": ">=8.2",
+        "php": ">=8.3",
         "doctrine/collections": "^2",
         "sonsofphp/pager": "^0.3.x-dev"
     },
@@ -53,4 +53,4 @@
             "url": "https://tidelift.com/subscription/pkg/packagist-sonsofphp-sonsofphp"
         }
     ]
-}
+}

src/SonsOfPHP/Bridge/Doctrine/DBAL/Pager/composer.json

@@ -33,7 +33,7 @@
     "minimum-stability": "dev",
     "prefer-stable": true,
     "require": {
-        "php": ">=8.2",
+        "php": ">=8.3",
         "doctrine/dbal": "^3",
         "sonsofphp/pager": "^0.3.x-dev"
     },
@@ -53,4 +53,4 @@
             "url": "https://tidelift.com/subscription/pkg/packagist-sonsofphp-sonsofphp"
         }
     ]
-}
+}

src/SonsOfPHP/Bridge/Doctrine/EventSourcing/composer.json

@@ -33,7 +33,7 @@
     "minimum-stability": "dev",
     "prefer-stable": true,
     "require": {
-        "php": ">=8.2",
+        "php": ">=8.3",
         "doctrine/dbal": "^3",
         "sonsofphp/event-sourcing": "^0.3.x-dev"
     },
@@ -53,4 +53,4 @@
             "url": "https://tidelift.com/subscription/pkg/packagist-sonsofphp-sonsofphp"
         }
     ]
-}
+}

src/SonsOfPHP/Bridge/Doctrine/ORM/Pager/composer.json

@@ -33,7 +33,7 @@
     "minimum-stability": "dev",
     "prefer-stable": true,
     "require": {
-        "php": ">=8.2",
+        "php": ">=8.3",
         "doctrine/orm": "^2 || ^3",
         "sonsofphp/pager": "^0.3.x-dev"
     },
@@ -53,4 +53,4 @@
             "url": "https://tidelift.com/subscription/pkg/packagist-sonsofphp-sonsofphp"
         }
     ]
-}
+}

src/SonsOfPHP/Bridge/LiipImagine/Filesystem/composer.json

@@ -38,7 +38,7 @@
     "minimum-stability": "dev",
     "prefer-stable": true,
     "require": {
-        "php": ">=8.2",
+        "php": ">=8.3",
         "sonsofphp/filesystem": "^0.3.x-dev",
         "liip/imagine-bundle": "^2.0"
     },
@@ -58,4 +58,4 @@
             "url": "https://tidelift.com/subscription/pkg/packagist-sonsofphp-sonsofphp"
         }
     ]
-}
+}

src/SonsOfPHP/Bridge/Symfony/Cqrs/composer.json

@@ -41,7 +41,7 @@
     },
     "minimum-stability": "dev",
     "require": {
-        "php": ">=8.2",
+        "php": ">=8.3",
         "sonsofphp/cqrs": "^0.3.x-dev",
         "symfony/options-resolver": "^6 || ^7"
     },
@@ -67,4 +67,4 @@
             "url": "https://tidelift.com/subscription/pkg/packagist-sonsofphp-sonsofphp"
         }
     ]
-}
+}