Introduce symfony & phpunit entrypoint providers; bump deps

Author: janedbal

.github/workflows/checks.yml

@@ -18,7 +18,7 @@ jobs:
                 name: Setup PHP
                 uses: shivammathur/setup-php@v2
                 with:
-                    php-version: 8.2
+                    php-version: 8.3
             -
                 name: Install dependencies
                 run: composer install --no-progress --prefer-dist --no-interaction
@@ -32,7 +32,7 @@ jobs:
         strategy:
             fail-fast: false
             matrix:
-                php-version: [ '7.4', '8.0', '8.1', '8.2', '8.3' ]
+                php-version: [ '8.0', '8.1', '8.2', '8.3' ]
                 dependency-version: [ prefer-lowest, prefer-stable ]
         steps:
             -

README.md

@@ -17,50 +17,42 @@ includes:
 
 
 ## Configuration:
-- You need to mark all entrypoints of your code to get proper results.
-- This is typically long whitelist of all code that is called by your framework and libraries.
-- You can easily start with DefaultEntrypointProvider which marks all methods declared in vendor packages as entrypoints.
+- All entrypoints of your code (controllers, consumers, commands, ...) need to be known to the detector to get proper results
+- By default, all overridden methods which declaration originates inside vendor are considered entrypoints
+- Also, there are some basic entrypoint providers for `symfony` and `phpunit`
+- For everything else, you can implement your own entrypoint provider
 
 ```neon
-services:
-    -
-        class: ShipMonk\PHPStan\DeadCode\Provider\DefaultEntrypointProvider
-        tags:
-            - shipmonk.deadCode.entrypointProvider
+parameters:
+    deadCode:
+        entrypoints:
+            symfony:
+                enabled: true
+            phpunit:
+                enabled: true
 
+services:
     -
-        class: App\SymfonyEntrypointProvider
+        class: App\MyEntrypointProvider
         tags:
             - shipmonk.deadCode.entrypointProvider
 ```
 ```php
 
 use ReflectionMethod;
-use PHPStan\Reflection\ReflectionProvider;
 use ShipMonk\PHPStan\DeadCode\Provider\EntrypointProvider;
 
-class SymfonyEntrypointProvider implements EntrypointProvider
+class MyEntrypointProvider implements EntrypointProvider
 {
 
-    public function __construct(
-        private ReflectionProvider $reflectionProvider
-    ) {}
-
     public function isEntrypoint(ReflectionMethod $method): bool
     {
-        $methodName = $method->getName();
-        $reflection = $this->reflectionProvider->getClass($method->getDeclaringClass()->getName());
-
-        return $reflection->is(\Symfony\Bundle\FrameworkBundle\Controller\AbstractController::class)
-            || $reflection->is(\Symfony\Component\EventDispatcher\EventSubscriberInterface::class)
-            || $method->getAttributes(\Symfony\Contracts\Service\Attribute\Required::class) !== []
-            // and many more
+        return $method->getDeclaringClass()->implementsInterface(ApiOutput::class));
     }
 }
 ```
 
-## Limitations
-This project is currently a working prototype (we are using it since 2022) with limited functionality:
+## Limitations:
 
 - Only method calls are detected
   - Including static methods, trait methods, interface methods, first class callables, etc.
@@ -79,4 +71,4 @@ This project is currently a working prototype (we are using it since 2022) with
 - All functionality must be tested
 
 ## Supported PHP versions
-- PHP 7.4 - 8.3
+- PHP 8.0 - 8.3

composer.json

@@ -12,18 +12,21 @@
         "dead code"
     ],
     "require": {
-        "php": "^7.4 || ^8.0",
-        "phpstan/phpstan": "^1.10.30"
+        "php": "^8.0",
+        "phpstan/phpstan": "^1.11.0"
     },
     "require-dev": {
         "editorconfig-checker/editorconfig-checker": "^10.3.0",
         "ergebnis/composer-normalize": "^2.28",
         "phpstan/phpstan-phpunit": "^1.1.1",
         "phpstan/phpstan-strict-rules": "^1.2.3",
-        "phpunit/phpunit": "^9.5.20",
+        "phpunit/phpunit": "^10.5.27",
         "shipmonk/name-collision-detector": "^2.0.0",
-        "shipmonk/phpstan-rules": "^2.11",
-        "slevomat/coding-standard": "^8.0.1"
+        "shipmonk/phpstan-rules": "^3.1",
+        "slevomat/coding-standard": "^8.0.1",
+        "symfony/contracts": "^3.5",
+        "symfony/event-dispatcher": "^6.0",
+        "symfony/routing": "^6.0"
     },
     "autoload": {
         "psr-4": {
@@ -65,7 +68,7 @@
         "check:composer": "composer normalize --dry-run --no-check-lock --no-update-lock",
         "check:cs": "phpcs",
         "check:ec": "ec src tests",
-        "check:tests": "phpunit -vvv tests",
+        "check:tests": "phpunit tests",
         "check:types": "phpstan analyse -vvv --ansi",
         "fix:cs": "phpcbf"
     }