Apply PHP-CS-Fixer changes

Author: ilvalerione

src/Workflow/Exporter/MermaidExporter.php

@@ -26,11 +26,11 @@ public function export(Workflow $graph): string
     private function getShortClassName(string $class): string
     {
         // Check if it's a class name (contains namespace separator) and class exists
-        if (strpos($class, '\\') !== false && class_exists($class)) {
+        if (\strpos($class, '\\') !== false && \class_exists($class)) {
             $reflection = new ReflectionClass($class);
             return $reflection->getShortName();
         }
-        
+
         // Otherwise, it's a custom string key, use it directly
         return $class;
     }

src/Workflow/Workflow.php

@@ -229,8 +229,8 @@ public function addNode(NodeInterface $node, ?string $key = null): self
     public function addNodes(array $nodes): Workflow
     {
         // Check if it's an associative array
-        $isAssociative = count(array_filter(array_keys($nodes), 'is_string')) > 0;
-        
+        $isAssociative = \count(\array_filter(\array_keys($nodes), 'is_string')) > 0;
+
         if ($isAssociative) {
             // If associative, use the keys
             foreach ($nodes as $key => $node) {
@@ -252,11 +252,11 @@ public function getNodes(): array
     {
         if ($this->nodes === []) {
             $nodeDefinitions = $this->nodes();
-            
+
             // Check if it's an associative array (has string keys)
             // An associative array has string keys or non-sequential numeric keys
-            $isAssociative = count(array_filter(array_keys($nodeDefinitions), 'is_string')) > 0;
-            
+            $isAssociative = \count(\array_filter(\array_keys($nodeDefinitions), 'is_string')) > 0;
+
             if ($isAssociative) {
                 // New behavior: use provided keys directly
                 $this->nodes = $nodeDefinitions;

tests/Workflow/WorkflowStringKeysTest.php

@@ -16,7 +16,7 @@ class AddNode extends Node
     public function __construct(private int $value)
     {
     }
-    
+
     public function run(WorkflowState $state): WorkflowState
     {
         $current = $state->get('value', 0);
@@ -33,7 +33,7 @@ class MultiplyNode extends Node
     public function __construct(private int $value)
     {
     }
-    
+
     public function run(WorkflowState $state): WorkflowState
     {
         $current = $state->get('value', 0);
@@ -50,7 +50,7 @@ class SubtractNode extends Node
     public function __construct(private int $value)
     {
     }
-    
+
     public function run(WorkflowState $state): WorkflowState
     {
         $current = $state->get('value', 0);
@@ -94,26 +94,26 @@ public function nodes(): array
             'finish_odd' => new FinishOddNode()
         ];
     }
-    
+
     public function edges(): array
     {
         return [
             // ((startingValue + 1) * 3) * 3) - 1
             new Edge('add1', 'multiply3_first'),
             new Edge('multiply3_first', 'multiply3_second'),
             new Edge('multiply3_second', 'sub1'),
-            
+
             // Branch based on even/odd
-            new Edge('sub1', 'finish_even', fn($state) => $state->get('value') % 2 === 0),
-            new Edge('sub1', 'finish_odd', fn($state) => $state->get('value') % 2 !== 0)
+            new Edge('sub1', 'finish_even', fn ($state) => $state->get('value') % 2 === 0),
+            new Edge('sub1', 'finish_odd', fn ($state) => $state->get('value') % 2 !== 0)
         ];
     }
-    
+
     protected function start(): string
     {
         return 'add1';
     }
-    
+
     protected function end(): array
     {
         return ['finish_even', 'finish_odd'];
@@ -125,30 +125,30 @@ class WorkflowStringKeysTest extends TestCase
     public function test_workflow_with_string_keys(): void
     {
         $workflow = new CalculatorWorkflow();
-        
+
         // Test with initial value 2: ((2 + 1) * 3) * 3) - 1 = 26 (even)
         $initialState = new WorkflowState(['value' => 2]);
         $result = $workflow->run($initialState);
-        
+
         $this->assertEquals(26, $result->get('value'));
         $this->assertEquals('even', $result->get('result_type'));
         $this->assertContains('Added 1', $result->get('history'));
         $this->assertContains('Multiplied by 3', $result->get('history'));
         $this->assertContains('Subtracted 1', $result->get('history'));
     }
-    
+
     public function test_workflow_with_string_keys_odd_result(): void
     {
         $workflow = new CalculatorWorkflow();
-        
+
         // Test with initial value 1: ((1 + 1) * 3) * 3) - 1 = 17 (odd)
         $initialState = new WorkflowState(['value' => 1]);
         $result = $workflow->run($initialState);
-        
+
         $this->assertEquals(17, $result->get('value'));
         $this->assertEquals('odd', $result->get('result_type'));
     }
-    
+
     public function test_programmatic_workflow_with_string_keys(): void
     {
         $workflow = new Workflow();
@@ -160,21 +160,21 @@ public function test_programmatic_workflow_with_string_keys(): void
         ])
         ->addEdges([
             new Edge('add1', 'multiply2'),
-            new Edge('multiply2', 'finish_even', fn($state) => $state->get('value') % 2 === 0),
-            new Edge('multiply2', 'finish_odd', fn($state) => $state->get('value') % 2 !== 0)
+            new Edge('multiply2', 'finish_even', fn ($state) => $state->get('value') % 2 === 0),
+            new Edge('multiply2', 'finish_odd', fn ($state) => $state->get('value') % 2 !== 0)
         ])
         ->setStart('add1')
         ->setEnd('finish_even')
         ->setEnd('finish_odd');
-        
+
         // Test with initial value 3: (3 + 1) * 2 = 8 (even)
         $initialState = new WorkflowState(['value' => 3]);
         $result = $workflow->run($initialState);
-        
+
         $this->assertEquals(8, $result->get('value'));
         $this->assertEquals('even', $result->get('result_type'));
     }
-    
+
     public function test_mermaid_export_with_string_keys(): void
     {
         $workflow = new Workflow();
@@ -189,13 +189,13 @@ public function test_mermaid_export_with_string_keys(): void
         ])
         ->setStart('start')
         ->setEnd('finish');
-        
+
         $export = $workflow->export();
-        
+
         $this->assertStringContainsString('start --> middle', $export);
         $this->assertStringContainsString('middle --> finish', $export);
     }
-    
+
     public function test_backward_compatibility_with_class_names(): void
     {
         // This test ensures the old behavior still works
@@ -205,12 +205,12 @@ public function test_backward_compatibility_with_class_names(): void
             ->addEdge(new Edge(StartNode::class, FinishNode::class))
             ->setStart(StartNode::class)
             ->setEnd(FinishNode::class);
-        
+
         $result = $workflow->run();
-        
+
         $this->assertEquals('end', $result->get('step'));
     }
-    
+
     public function test_mixed_mode_nodes_and_edges(): void
     {
         // Test mixing both approaches - indexed array with class name edges
@@ -226,10 +226,10 @@ public function test_mixed_mode_nodes_and_edges(): void
         ])
         ->setStart(StartNode::class)
         ->setEnd(FinishNode::class);
-        
+
         $result = $workflow->run();
-        
+
         $this->assertEquals('end', $result->get('step'));
         $this->assertEquals(1, $result->get('counter'));
     }
-}
+}