Remove Enum support in CLI mode

Author: nyamsprod

CHANGELOG.md

@@ -23,6 +23,7 @@ All Notable changes to `bakame/stackwatch` will be documented in this file.
 ### Remove
 
 - `MetricsAssertions` trait replaced by the `PerformanceAssertions` trait
+- **BC BREAK:** Enums can no longer be profiled using the CLI instance as it was making no sense.
 
 ## [0.15.0 - Ouagadougou](https://github.com/bakame-php/stackwatch/compare/0.14.0...0.15.0) - 2025-09-09
 

composer.json

@@ -26,21 +26,21 @@
     "require": {
         "php": "^8.1",
         "psr/log": "^3.0.2",
-        "symfony/console": "^6.4 || ^7.3.2",
-        "symfony/process": "^6.4 || ^7.3"
+        "symfony/console": "^6.4 || ^7.4.4 || ^v8.0",
+        "symfony/process": "^6.4 || ^7.4.5 || ^v8.0"
     },
     "require-dev": {
-        "friendsofphp/php-cs-fixer": "^3.85.1",
-        "monolog/monolog": "^3.9",
+        "friendsofphp/php-cs-fixer": "^3.93.1",
+        "monolog/monolog": "^3.10",
         "nyholm/psr7": "^1.8.2",
-        "open-telemetry/exporter-otlp": "^1.3.2",
-        "phpstan/phpstan": "^2.1.21",
+        "open-telemetry/exporter-otlp": "^1.3.4",
+        "phpstan/phpstan": "^2.1.38",
         "phpstan/phpstan-deprecation-rules": "^2.0.3",
-        "phpstan/phpstan-phpunit": "^2.0.7",
-        "phpstan/phpstan-strict-rules": "^2.0.6",
-        "phpunit/phpunit": "^10.5.15 || ^11.5.25 || ^12.2.9",
-        "symfony/http-client": "^v6.4.19 || ^7.3.2",
-        "symfony/var-dumper": "^6.4.21 || ^7.3.2"
+        "phpstan/phpstan-phpunit": "^2.0.12",
+        "phpstan/phpstan-strict-rules": "^2.0.8",
+        "phpunit/phpunit": "^10.5.15 || ^11.5.25 || ^12.5.9",
+        "symfony/http-client": "^v6.4.19 || ^7.4.5 || ^v8.0",
+        "symfony/var-dumper": "^6.4.21 || ^7.4.4 || ^v8.0"
     },
     "autoload": {
         "psr-4": {

docs/index.md

@@ -53,19 +53,17 @@ require 'vendor/autoload.php';
 
 trait TimerTrait {
     #[Profile]
-    private function test() : int {
+    private function test() : int
+    {
         usleep(100);
 
         return random_int(1, 100);
     }
 }
 
-enum MyEnum
+class MyClass
 {
     use TimerTrait;
-
-    case Case1;
-    case Case2;
 }
 ```
 
@@ -83,7 +81,7 @@ Runtime: PHP 8.3.25 OS: Linux Memory Limit: 64M
 
 ............................
 
-Target: Foobar\Baz\MyEnum::test; Path: /path/to/profiling/code.php; Iterations: 3; Warmup: 0; Type: Full;
+Target: Foobar\Baz\MyClass::test; Path: /path/to/profiling/code.php; Iterations: 3; Warmup: 0; Type: Full;
 
 +-------------------------------+------------+------------+------------+------------+------------+------------+--------------+---------------+-----------+-----------+
 | Metrics                       | iterations | Min Value  | Max Value  | Range      | Sum        | Average    | Median Value | Variance      | Std Dev   | Coef Var  |

src/Console/PathProfilerTest.php

@@ -120,7 +120,7 @@ public function testMethod(): void
         $output = $this->stdout->fetch();
         $errorOutput = $this->stderr->fetch();
 
-        self::assertStringContainsString('Full', $output);
+        self::assertEmpty($output);
         self::assertEmpty($errorOutput, 'No errors expected');
     }
 
@@ -134,10 +134,8 @@ public function it_will_skip_methods_with_arguments(): void
 use Bakame\Stackwatch\AggregationType;
 use Bakame\Stackwatch\Profile;
 
-enum TestMethodWithArguments
+class TestMethodWithArguments
 {
-    case Foo;
-
     #[Profile(iterations: 3, warmup: 1, type: AggregationType::Average)]
     public function testMethod(string $foo): string
     {

src/Console/UnitOfWork.php

@@ -229,14 +229,14 @@ function_exists($this->function) || throw new UnableToProfile('The function '.$t
         0 === $refMethod->getNumberOfRequiredParameters() || throw new UnableToProfile('The method '.$this->class.'::'.$this->method.' cannot be profiled because it has required parameters.');
 
         if ($refMethod->isStatic()) {
-            $this->callback =  fn () => $refMethod->invoke(null);
+            $this->callback = fn () => $refMethod->invoke(null);
 
             return $this->callback;
         }
 
-        $instance = $refClass instanceof ReflectionEnum ? $refClass->getCases()[0]->getValue() : $refClass->newInstance();
+        !$refClass instanceof ReflectionEnum || throw new UnableToProfile('The method '.$this->class.'::'.$this->method.' belongs to an enum and can only be profiled when executed by a case.');
 
-        $this->callback = fn () => $refMethod->invoke($instance);
+        $this->callback = fn () => $refMethod->invoke($refClass->newInstance());
 
         return $this->callback;
     }

src/Console/UnitOfWorkGenerator.php

@@ -20,7 +20,9 @@
 use function array_reduce;
 use function class_exists;
 use function count;
+use function enum_exists;
 use function function_exists;
+use function in_array;
 
 final class UnitOfWorkGenerator
 {
@@ -90,8 +92,12 @@ private function prepareMethodsProcess(string $className, Input $input): array
             return [];
         }
 
-        $refClass = enum_exists($className) ? new ReflectionEnum($className) : new ReflectionClass($className);
-        $targetRequiresConstructorArgs = !$refClass instanceof ReflectionEnum && (($refClass->getConstructor()?->getNumberOfRequiredParameters() ?? 0) !== 0);
+        if (enum_exists($className)) {
+            return [];
+        }
+
+        $refClass = new ReflectionClass($className);
+        $targetRequiresConstructorArgs = 0 !== ($refClass->getConstructor()?->getNumberOfRequiredParameters() ?? 0);
         $parentProfile = $this->findProfile($refClass);
 
         $results = [];