feat: use absolute path for phpstorm with test

Author: ashleyhindle

src/Console/InstallCommand.php

@@ -462,17 +462,18 @@ private function installMcpServerConfig(): void
             )->toArray()
         );
 
-        /** @var CodeEnvironment $mcpClient */
+        /** @var McpClient $mcpClient */
         foreach ($this->selectedTargetMcpClient as $mcpClient) {
             $ideName = $mcpClient->mcpClientName();
             $ideDisplay = str_pad($ideName, $longestIdeName);
             $this->output->write("  {$ideDisplay}... ");
             $results = [];
 
+            $php = $this->getPhpPathForMcpClient($mcpClient);
             if ($this->shouldInstallMcp()) {
                 try {
-                    $artisan = $mcpClient->useAbsolutePathForMcp ? base_path('artisan') : './artisan';
-                    $result = $mcpClient->installMcp('laravel-boost', 'php', [$artisan, 'boost:mcp']);
+                    $artisan = $this->getArtisanPathForMcpClient($mcpClient);
+                    $result = $mcpClient->installMcp('laravel-boost', $php, [$artisan, 'boost:mcp']);
 
                     if ($result) {
                         $results[] = $this->greenTick.' Boost';
@@ -491,7 +492,7 @@ private function installMcpServerConfig(): void
                 try {
                     $result = $mcpClient->installMcp(
                         key: 'herd',
-                        command: 'php',
+                        command: $this->getPhpPathForMcpClient($mcpClient),
                         args: [$this->herd->mcpPath()],
                         env: ['SITE_PATH' => base_path()]
                     );
@@ -523,6 +524,16 @@ private function installMcpServerConfig(): void
         }
     }
 
+    private function getPhpPathForMcpClient(McpClient $mcpClient): string
+    {
+        return $mcpClient->useAbsolutePathForMcp() ? PHP_BINARY : 'php';
+    }
+
+    private function getArtisanPathForMcpClient(McpClient $mcpClient): string
+    {
+        return $mcpClient->useAbsolutePathForMcp() ? base_path('artisan') : './artisan';
+    }
+
     /**
      * Is the project actually using localization for their new features?
      */

src/Contracts/McpClient.php

@@ -16,6 +16,11 @@ interface McpClient
      */
     public function mcpClientName(): ?string;
 
+    /**
+     * Whether to use absolute paths for MCP commands.
+     */
+    public function useAbsolutePathForMcp(): bool;
+
     /**
      * Install an MCP server configuration in this IDE.
      *

src/Install/CodeEnvironment/CodeEnvironment.php

@@ -35,6 +35,11 @@ public function mcpClientName(): ?string
         return $this->displayName();
     }
 
+    public function useAbsolutePathForMcp(): bool
+    {
+        return $this->useAbsolutePathForMcp;
+    }
+
     /**
      * Get the detection configuration for system-wide installation detection.
      *

tests/Unit/Console/InstallCommandPhpPathTest.php

@@ -0,0 +1,37 @@
+<?php
+
+declare(strict_types=1);
+
+use Laravel\Boost\Console\InstallCommand;
+use Laravel\Boost\Install\CodeEnvironment\Cursor;
+use Laravel\Boost\Install\CodeEnvironment\PhpStorm;
+use Laravel\Boost\Install\Detection\DetectionStrategyFactory;
+
+test('getPhpPathForMcpClient returns absolute PHP_BINARY for PhpStorm', function () {
+    $installCommand = new InstallCommand;
+    $strategyFactory = Mockery::mock(DetectionStrategyFactory::class);
+    $phpStorm = new PhpStorm($strategyFactory);
+
+    $phpPath = invokePrivateMethod($installCommand, 'getPhpPathForMcpClient', [$phpStorm]);
+
+    expect($phpPath)->toBe(PHP_BINARY);
+});
+
+test('getPhpPathForMcpClient returns php string for Cursor', function () {
+    $installCommand = new InstallCommand;
+    $strategyFactory = Mockery::mock(DetectionStrategyFactory::class);
+    $cursor = new Cursor($strategyFactory);
+
+    $phpPath = invokePrivateMethod($installCommand, 'getPhpPathForMcpClient', [$cursor]);
+
+    expect($phpPath)->toBe('php');
+});
+
+function invokePrivateMethod(object $object, string $methodName, array $parameters = []): mixed
+{
+    $reflection = new ReflectionClass($object);
+    $method = $reflection->getMethod($methodName);
+    $method->setAccessible(true);
+
+    return $method->invokeArgs($object, $parameters);
+}