fix(Parser): implement variable interpolation in DefaultParser

- Add interpolateValue method to DefaultParser
- Update parse method to perform interpolation after initial parsing
- Modify Dotenv class to utilize updated parsing logic

This commit addresses the issue with variable interpolation, specifically
resolving the problem with KARIRI_MAIL_FROM_NAME. The changes include:

- Implementing a new interpolateValue method in DefaultParser to handle
  variable substitution using ${VAR_NAME} syntax
- Updating the parse method to perform interpolation as a separate step
  after all variables are initially parsed
- Adjusting the Dotenv class to work with the new parsing approach

Author: walmir-silva

src/Parser/DefaultParser.php

@@ -13,6 +13,8 @@ class DefaultParser implements Parser
     private const SETTER_CHAR = '=';
     private const INVALID_NAME_CHARS = '{}()[]';
 
+    private array $parsedValues = [];
+
     public function __construct(
         private readonly bool $strictMode = false
     ) {
@@ -21,15 +23,19 @@ public function __construct(
     public function parse(string $content): array
     {
         $lines = $this->splitLines($content);
-
-        return array_reduce($lines, function (array $output, string $line) {
+        foreach ($lines as $line) {
             if ($this->isValidSetter($line)) {
                 [$key, $value] = $this->parseEnvironmentVariable($line);
-                $output[$key] = $value;
+                $this->parsedValues[$key] = $value;
             }
+        }
+
+        // Perform interpolation after all variables are parsed
+        foreach ($this->parsedValues as $key => $value) {
+            $this->parsedValues[$key] = $this->interpolateValue($value);
+        }
 
-            return $output;
-        }, []);
+        return $this->parsedValues;
     }
 
     private function splitLines(string $content): array
@@ -58,7 +64,7 @@ private function parseEnvironmentVariable(string $line): array
     {
         [$name, $value] = explode(self::SETTER_CHAR, $line, 2);
         $name = trim($name);
-        $value = $this->interpolateValue(trim($value));
+        $value = trim($value);
 
         $this->validateVariableName($name);
 
@@ -67,11 +73,13 @@ private function parseEnvironmentVariable(string $line): array
 
     private function validateVariableName(string $name): void
     {
-        match (true) {
-            '' === $name => throw new InvalidValueException('Empty variable name'),
-            $this->strictMode && $this->containsInvalidCharacters($name) => throw new InvalidValueException('Invalid character in variable name'),
-            default => null,
-        };
+        if ('' === $name) {
+            throw new InvalidValueException('Empty variable name');
+        }
+
+        if ($this->strictMode && $this->containsInvalidCharacters($name)) {
+            throw new InvalidValueException('Invalid character in variable name');
+        }
     }
 
     private function containsInvalidCharacters(string $name): bool
@@ -81,10 +89,10 @@ private function containsInvalidCharacters(string $name): bool
 
     private function interpolateValue(string $value): string
     {
-        return preg_replace_callback(
-            '/\$\{([A-Z0-9_]+)\}/',
-            fn ($matches) => $_ENV[$matches[1]] ?? $matches[0],
-            $value
-        );
+        return preg_replace_callback('/\${([A-Z0-9_]+)}/', function ($matches) {
+            $varName = $matches[1];
+
+            return $this->parsedValues[$varName] ?? $matches[0];
+        }, $value);
     }
 }

tests/application.php

@@ -15,7 +15,7 @@
     $variables = [
         'KARIRI_APP_ENV' => 'string',
         'KARIRI_APP_NAME' => 'string',
-        'KARIRI_PHP_VERSION' => 'string',
+        'KARIRI_PHP_VERSION' => 'double',
         'KARIRI_PHP_PORT' => 'integer',
         'KARIRI_APP_DEBUG' => 'boolean',
         'KARIRI_APP_URL' => 'string',