Fix null-safety bugs, drop mixed types, drop Laravel 10, add test coverage

- readFile() could return false from a failed file_get_contents() where
  string was promised; now guarded. update() silently wrote null to the
  .env file if preg_replace() failed; now falls back to the original
  content instead.
- set(), update(), setOrUpdate(), get(), and formatValue() now type
  their value parameters as string|int|float|bool|null instead of mixed.
- Drop Laravel 10 support (minimum is now Laravel 11); bump PHPStan to
  level 8 and rector.php to target the Laravel 11 floor.
- Add tests for getEnvFile() and the service provider's container
  binding; expand ArchTest.php; rename ExampleTest.php to EnvEditorTest.php.

Author: amdad121

CHANGELOG.md

@@ -2,7 +2,19 @@
 
 All notable changes to `env-editor-laravel` will be documented in this file.
 
-## Unreleased
+## v3.0.0 - 2026-07-21
+
+### Fixed
+- `readFile()` could return `false` from a failed `file_get_contents()` where `string` was promised; now guarded.
+- `update()` silently wrote `null` to the `.env` file if `preg_replace()` failed (e.g. a PCRE backtrack-limit error); it now falls back to the original content instead.
+
+### Changed
+- `set()`, `update()`, `setOrUpdate()`, `get()`, and `formatValue()` now type their value parameters as `string|int|float|bool|null` instead of `mixed`, matching the actual set of values a `.env` value can hold.
+
+### Removed
+- Dropped Laravel 10 support (minimum is now Laravel 11)
+
+## v2.0.0 - 2026-01-03
 
 ### Added
 - `get()` method to read individual .env values with default support

README.md

@@ -7,6 +7,11 @@
 
 Simple and powerful ENV file editor for your Laravel application.
 
+## Requirements
+
+- PHP 8.2, 8.3, 8.4, or 8.5
+- Laravel 11, 12, or 13
+
 ## Installation
 
 You can install the package via composer:

composer.json

@@ -17,13 +17,13 @@
     ],
     "require": {
         "php": "^8.2|^8.3|^8.4|^8.5",
-        "illuminate/support": "^10.0|^11.0|^12.0|^13.0"
+        "illuminate/support": "^11.0|^12.0|^13.0"
     },
     "require-dev": {
         "driftingly/rector-laravel": "^1.0|^2.0",
         "larastan/larastan": "^2.0|^3.0",
         "laravel/pint": "^1.26",
-        "orchestra/testbench": "^8.0|^9.0|^10.0|^11.0",
+        "orchestra/testbench": "^9.0|^10.0|^11.0",
         "pestphp/pest": "^2.0|^3.0|^4.0",
         "pestphp/pest-plugin-laravel": "^2.0|^3.0|^4.0"
     },

phpstan.neon

@@ -3,7 +3,7 @@ includes:
     - vendor/nesbot/carbon/extension.neon
 
 parameters:
-    level: 4
+    level: 8
     paths:
         - src
     tmpDir: build/phpstan

rector.php

@@ -22,7 +22,7 @@
     )
     ->withPhpSets(php82: true)
     ->withImportNames(importDocBlockNames: false, removeUnusedImports: true)->withSets([
-        LaravelLevelSetList::UP_TO_LARAVEL_100,
+        LaravelLevelSetList::UP_TO_LARAVEL_110,
         LaravelSetList::LARAVEL_ARRAYACCESS_TO_METHOD_CALL,
         LaravelSetList::LARAVEL_ARRAY_STR_FUNCTION_TO_STATIC_CALL,
         LaravelSetList::LARAVEL_CODE_QUALITY,

src/EnvEditor.php

@@ -18,7 +18,7 @@ public function __construct(?string $envFile = null)
         $this->backupDir = dirname($this->envFile).'/.env.backup';
     }
 
-    public function set(string $key, $value): bool
+    public function set(string $key, string|int|float|bool|null $value): bool
     {
         $formattedValue = $this->formatValue($value);
 
@@ -29,19 +29,19 @@ public function set(string $key, $value): bool
         return $this->append($key, $formattedValue);
     }
 
-    public function update(string $key, $value): bool
+    public function update(string $key, string|int|float|bool|null $value): bool
     {
         throw_unless($this->keyExists($key), Exception::class, sprintf("Key '%s' does not exist in .env file", $key));
 
         $formattedValue = $this->formatValue($value);
         $envContent = $this->readFile();
 
-        $envContent = preg_replace(sprintf('/^%s=.*$/m', $this->escapeKey($key)), sprintf('%s=%s', $key, $formattedValue), $envContent);
+        $replaced = preg_replace(sprintf('/^%s=.*$/m', $this->escapeKey($key)), sprintf('%s=%s', $key, $formattedValue), $envContent);
 
-        return $this->writeFile($envContent);
+        return $this->writeFile($replaced ?? $envContent);
     }
 
-    public function setOrUpdate(string $key, $value): bool
+    public function setOrUpdate(string $key, string|int|float|bool|null $value): bool
     {
         if ($this->keyExists($key)) {
             return $this->update($key, $value);
@@ -65,7 +65,7 @@ public function remove(string $key): bool
         return $this->writeFile(trim((string) $envContent).PHP_EOL);
     }
 
-    public function get(string $key, $default = null)
+    public function get(string $key, string|int|float|bool|null $default = null): string|int|float|bool|null
     {
         $envContent = $this->readFile();
         $pattern = sprintf('/^%s=(.*)$/m', $this->escapeKey($key));
@@ -77,6 +77,9 @@ public function get(string $key, $default = null)
         return $default;
     }
 
+    /**
+     * @return array<string, string>
+     */
     public function getAll(): array
     {
         $envContent = $this->readFile();
@@ -132,6 +135,9 @@ public function restore(string $backupFile): bool
         return true;
     }
 
+    /**
+     * @return array<int, string>
+     */
     public function listBackups(): array
     {
         if (! is_dir($this->backupDir)) {
@@ -170,15 +176,17 @@ protected function readFile(): string
             return '';
         }
 
-        return file_get_contents($this->envFile);
+        $content = file_get_contents($this->envFile);
+
+        return $content === false ? '' : $content;
     }
 
     protected function writeFile(string $content): bool
     {
         return file_put_contents($this->envFile, $content) !== false;
     }
 
-    protected function formatValue($value): string
+    protected function formatValue(string|int|float|bool|null $value): string
     {
         $value = strval($value);
 

tests/ArchTest.php

@@ -2,6 +2,9 @@
 
 declare(strict_types=1);
 
+arch()->preset()->php();
+arch()->preset()->security();
+
 it('will not use debugging functions')
     ->expect(['dd', 'dump', 'ray'])
     ->each->not->toBeUsed();

tests/ExampleTest.php tests/EnvEditorTest.phptests/ExampleTest.php renamed to tests/EnvEditorTest.php

@@ -23,6 +23,18 @@
     }
 });
 
+it('exposes the path to the underlying env file', function (): void {
+    expect($this->editor->getEnvFile())->toBe($this->envFile);
+});
+
+it('resolves to a singleton bound to the application base path .env file', function (): void {
+    $editor = $this->app->make(EnvEditor::class);
+
+    expect($editor)->toBeInstanceOf(EnvEditor::class)
+        ->and($editor->getEnvFile())->toBe($this->app->basePath('.env'))
+        ->and($this->app->make(EnvEditor::class))->toBe($editor);
+});
+
 it('can get a value from env file', function (): void {
     expect($this->editor->get('APP_NAME'))->toBe('Laravel');
     expect($this->editor->get('APP_ENV'))->toBe('local');
@@ -175,3 +187,33 @@
     expect($this->editor->get('APP_ENV'))->toBe('local')
         ->and($this->editor->get('APP_DEBUG'))->toBe('true');
 });
+
+it('can set boolean values', function (): void {
+    $this->editor->set('FEATURE_ON', true);
+    $this->editor->set('FEATURE_OFF', false);
+
+    expect($this->editor->get('FEATURE_ON'))->toBe('1')
+        ->and($this->editor->get('FEATURE_OFF'))->toBe('');
+});
+
+it('returns default and an empty list when the env file does not exist at all', function (): void {
+    $missingPath = sys_get_temp_dir().'/env_editor_missing_'.uniqid().'.env';
+    $editor = new EnvEditor($missingPath);
+
+    expect($editor->get('ANYTHING', 'fallback'))->toBe('fallback')
+        ->and($editor->getAll())->toBe([])
+        ->and($editor->has('ANYTHING'))->toBeFalse();
+});
+
+it('can create a new env file by setting a key when none exists yet', function (): void {
+    $missingPath = sys_get_temp_dir().'/env_editor_new_'.uniqid().'.env';
+    $editor = new EnvEditor($missingPath);
+
+    $result = $editor->set('FRESH_KEY', 'fresh_value');
+
+    expect($result)->toBeTrue()
+        ->and(file_exists($missingPath))->toBeTrue()
+        ->and($editor->get('FRESH_KEY'))->toBe('fresh_value');
+
+    unlink($missingPath);
+});