Merge pull request #13 from smeghead/fix/issue-11

fix: fix empty scope error.

Author: smeghead

CHANGELOG.md

@@ -4,6 +4,7 @@
 
  * Fixed command execution to properly return error codes when failures occur
  * Improved error handling in all command implementations
+ * fix empty scope error. #11
 
 ## v0.0.4 (2025-03-20)
 

src/Analyze/AnalysisResult.php

@@ -17,10 +17,11 @@ public function __construct(
         public readonly array $scopes
     )
     {
-        $this->maxVariableHardUsage = max(array_map(fn(Scope $scope) => $scope->getVariableHardUsage(), $scopes));
         if (count($scopes) === 0) {
+            $this->maxVariableHardUsage = 0;
             $this->avarageVariableHardUsage = 0;
         } else {
+            $this->maxVariableHardUsage = max(array_map(fn(Scope $scope) => $scope->getVariableHardUsage(), $scopes));
             $this->avarageVariableHardUsage = array_sum(array_map(fn(Scope $scope) => $scope->getVariableHardUsage(), $scopes)) / count($scopes);
         }
     }

test/VariableAnalizerTest.php

@@ -9,6 +9,16 @@
 
 class VariableAnalizerTest extends TestCase
 {
+    public function testAnalyzeEmpty(): void
+    {
+        $sut = new VariableAnalyzer('target.php', []);
+        $result = $sut->analyze();
+        $this->assertSame('target.php', $result->filename);
+        $scopes = $result->scopes;
+
+        $this->assertCount(0, $scopes);
+    }
+
     public function testAnalyzeFunctionSimple(): void
     {
         $func = new Func(null, 'testFunction');