refactor: Remove fileExists() wrapper method

Following code review feedback from @pushpak1300, removed the
unnecessary fileExists() wrapper method and use file_exists()
directly in isSailProject().

Also removed 3 unit tests that were specifically testing the
fileExists() wrapper - the Sail detection is already covered
by the integration tests in SailDetectionTest.php.

Changes:
- Remove fileExists() wrapper method
- Use file_exists() directly in isSailProject()
- Remove 3 redundant unit tests for fileExists() mocking
- Integration tests still passing (6 tests, 10 assertions)

Author: NIAN97

PR_DESCRIPTION.md

@@ -0,0 +1,117 @@
+# Add Automatic Laravel Sail Detection for MCP Server Configuration
+
+## Summary
+
+This PR adds automatic detection of Laravel Sail environments and configures the MCP server to use Sail when detected. This ensures the MCP server runs inside the Docker container with the correct PHP version and environment, rather than using the host machine's PHP installation.
+
+## Motivation
+
+Currently, users running Laravel Boost in Sail projects must manually configure the MCP server to use `./vendor/bin/sail` instead of `php`. This PR makes the detection automatic, improving the developer experience for Sail users.
+
+### Problem
+
+When Sail users run `php artisan boost:install` or `php artisan boost:update`, the generated MCP configuration uses:
+
+```json
+{
+    "mcpServers": {
+        "laravel-boost": {
+            "command": "php",
+            "args": ["artisan", "boost:mcp"]
+        }
+    }
+}
+```
+
+This runs the MCP server on the host machine, which may have:
+- Different PHP version than the container
+- Missing PHP extensions
+- Different environment configuration
+
+### Solution
+
+With this PR, Sail projects automatically get:
+
+```json
+{
+    "mcpServers": {
+        "laravel-boost": {
+            "command": "./vendor/bin/sail",
+            "args": ["artisan", "boost:mcp"]
+        }
+    }
+}
+```
+
+## Changes
+
+### Core Implementation
+
+**File:** `src/Install/CodeEnvironment/CodeEnvironment.php`
+
+- Added `isSailProject()` method to detect Sail by checking for:
+  - `vendor/bin/sail` executable
+  - `docker-compose.yml` file
+- Modified `getPhpPath()` to return `./vendor/bin/sail` when Sail is detected
+- Modified `getArtisanPath()` to return `artisan` when Sail is detected
+- Added `fileExists()` helper method for improved testability
+
+### Tests
+
+**Unit Tests:** `tests/Unit/Install/CodeEnvironment/CodeEnvironmentTest.php`
+- ✅ `getPhpPath returns sail path when project uses Laravel Sail`
+- ✅ `getArtisanPath returns relative artisan when project uses Laravel Sail`
+- ✅ `isSailProject returns true when both sail and docker-compose exist`
+- ✅ `isSailProject returns false when sail does not exist`
+- ✅ `isSailProject returns false when docker-compose does not exist`
+
+**Integration Tests:** `tests/Feature/Install/SailDetectionTest.php`
+- ✅ `ClaudeCode detects Sail when both sail and docker-compose exist`
+- ✅ `ClaudeCode uses php when Sail is not detected`
+- ✅ `ClaudeCode uses php when only sail exists but no docker-compose`
+- ✅ `ClaudeCode uses php when only docker-compose exists but no sail`
+
+### Documentation
+
+**File:** `README.md`
+
+- Added installation note for Sail users in the Installation section
+- Added dedicated "Laravel Sail Support" section with:
+  - Explanation of automatic detection
+  - Detection criteria
+  - Configuration examples
+  - Benefits of running inside Docker container
+
+## Backward Compatibility
+
+✅ **100% backward compatible** - Projects without Sail continue to work exactly as before using `php artisan boost:mcp`.
+
+## Commands Affected
+
+- ✅ `php artisan boost:install` - Automatically detects and configures Sail
+- ✅ `php artisan boost:update` - Automatically detects and configures Sail (calls `InstallCommand` internally)
+
+## Test Results
+
+```
+Tests:    9 passed (13 assertions) - Sail-specific tests
+Tests:  307 passed (1354 assertions) - Full test suite
+Code Style: PASS (108 files) - Laravel Pint
+Static Analysis: PASS (67 files) - PHPStan Level 5
+```
+
+## Checklist
+
+- [x] Tests added for new functionality
+- [x] All tests passing
+- [x] Code style verified with Pint
+- [x] Static analysis passing (PHPStan)
+- [x] Documentation updated
+- [x] Backward compatible
+- [x] No breaking changes
+
+## Additional Notes
+
+The detection is automatic and transparent to users. When both `vendor/bin/sail` and `docker-compose.yml` exist in the project root, Boost automatically uses Sail. No manual configuration required.
+
+This enhancement benefits all Sail users across all supported IDEs (Claude Code, Cursor, VS Code, PhpStorm, GitHub Copilot, etc.).

src/Install/CodeEnvironment/CodeEnvironment.php

@@ -64,17 +64,8 @@ public function getArtisanPath(bool $forceAbsolutePath = false): string
      */
     protected function isSailProject(): bool
     {
-        return $this->fileExists(base_path('vendor/bin/sail')) &&
-               $this->fileExists(base_path('docker-compose.yml'));
-    }
-
-    /**
-     * Check if a file exists at the given path.
-     * This method can be mocked in tests.
-     */
-    protected function fileExists(string $path): bool
-    {
-        return file_exists($path);
+        return file_exists(base_path('vendor/bin/sail')) &&
+               file_exists(base_path('docker-compose.yml'));
     }
 
     /**

tests/Unit/Install/CodeEnvironment/CodeEnvironmentTest.php

@@ -436,63 +436,3 @@ public function mcpConfigPath(): string
 
     expect($environment->getArtisanPath())->toBe('artisan');
 });
-
-test('isSailProject returns true when both sail and docker-compose exist', function (): void {
-    $environment = Mockery::mock(TestCodeEnvironment::class)->makePartial();
-    $environment->shouldAllowMockingProtectedMethods();
-
-    $environment->shouldReceive('fileExists')
-        ->with(Mockery::on(fn ($path): bool => str_contains((string) $path, 'vendor/bin/sail')))
-        ->andReturn(true);
-
-    $environment->shouldReceive('fileExists')
-        ->with(Mockery::on(fn ($path): bool => str_contains((string) $path, 'docker-compose.yml')))
-        ->andReturn(true);
-
-    // Use reflection to call the protected method
-    $reflection = new ReflectionClass($environment);
-    $method = $reflection->getMethod('isSailProject');
-    $method->setAccessible(true);
-
-    expect($method->invoke($environment))->toBe(true);
-});
-
-test('isSailProject returns false when sail does not exist', function (): void {
-    $environment = Mockery::mock(TestCodeEnvironment::class)->makePartial();
-    $environment->shouldAllowMockingProtectedMethods();
-
-    $environment->shouldReceive('fileExists')
-        ->with(Mockery::on(fn ($path): bool => str_contains((string) $path, 'vendor/bin/sail')))
-        ->andReturn(false);
-
-    $environment->shouldReceive('fileExists')
-        ->with(Mockery::on(fn ($path): bool => str_contains((string) $path, 'docker-compose.yml')))
-        ->andReturn(true);
-
-    // Use reflection to call the protected method
-    $reflection = new ReflectionClass($environment);
-    $method = $reflection->getMethod('isSailProject');
-    $method->setAccessible(true);
-
-    expect($method->invoke($environment))->toBe(false);
-});
-
-test('isSailProject returns false when docker-compose does not exist', function (): void {
-    $environment = Mockery::mock(TestCodeEnvironment::class)->makePartial();
-    $environment->shouldAllowMockingProtectedMethods();
-
-    $environment->shouldReceive('fileExists')
-        ->with(Mockery::on(fn ($path): bool => str_contains((string) $path, 'vendor/bin/sail')))
-        ->andReturn(true);
-
-    $environment->shouldReceive('fileExists')
-        ->with(Mockery::on(fn ($path): bool => str_contains((string) $path, 'docker-compose.yml')))
-        ->andReturn(false);
-
-    // Use reflection to call the protected method
-    $reflection = new ReflectionClass($environment);
-    $method = $reflection->getMethod('isSailProject');
-    $method->setAccessible(true);
-
-    expect($method->invoke($environment))->toBe(false);
-});