refactor: rename ideName to mcpClientName and enhance MCP client handling

- Replaced `ideName` references with `mcpClientName` for clarity.
- Updated `InstallCommand` to use specific type hints (`Agent`, `McpClient`) for better consistency and readability.
- Added docblocks and exceptions for improved code documentation and error handling.
- Adjusted tests to reflect the `McpClient` changes and renamed corresponding test cases.
- Minor updates in `VSCode` and contract interfaces to align with the new naming convention.

Author: pushpak1300

src/Console/InstallCommand.php

@@ -10,6 +10,7 @@
 use Illuminate\Support\Collection;
 use Illuminate\Support\Str;
 use Laravel\Boost\Contracts\Agent;
+use Laravel\Boost\Contracts\McpClient;
 use Laravel\Boost\Install\Cli\DisplayHelper;
 use Laravel\Boost\Install\CodeEnvironment\CodeEnvironment;
 use Laravel\Boost\Install\CodeEnvironmentsDetector;
@@ -38,10 +39,10 @@ class InstallCommand extends Command
 
     private Terminal $terminal;
 
-    /** @var Collection<int, CodeEnvironment> */
+    /** @var Collection<int, Agent> */
     private Collection $selectedTargetAgents;
 
-    /** @var Collection<int, CodeEnvironment> */
+    /** @var Collection<int, McpClient> */
     private Collection $selectedTargetMcpClient;
 
     /** @var Collection<int, string> */
@@ -157,11 +158,11 @@ private function outro(): void
     {
         $label = 'https://boost.laravel.com/installed';
 
-        $ideNames = $this->selectedTargetMcpClient->map(fn ($ide) => 'i:'.$ide->ideName())->toArray();
-        $agentNames = $this->selectedTargetAgents->map(fn ($agent) => 'a:'.$agent->agentName())->toArray();
+        $ideNames = $this->selectedTargetMcpClient->map(fn (McpClient $mcpClient) => 'i:'.$mcpClient->mcpClientName())
+            ->toArray();
+        $agentNames = $this->selectedTargetAgents->map(fn (Agent $agent) => 'a:'.$agent->agentName())->toArray();
         $boostFeatures = $this->selectedBoostFeatures->map(fn ($feature) => 'b:'.$feature)->toArray();
 
-        // Guidelines installed (prefix: g)
         $guidelines = [];
         if ($this->shouldInstallAiGuidelines()) {
             $guidelines[] = 'g:ai';
@@ -436,12 +437,12 @@ private function installMcpServerConfig(): void
         $longestIdeName = max(
             1,
             ...$this->selectedTargetMcpClient->map(
-                fn ($mcpClient) => Str::length($mcpClient->ideName())
+                fn (McpClient $mcpClient) => Str::length($mcpClient->mcpClientName())
             )->toArray()
         );
 
         foreach ($this->selectedTargetMcpClient as $mcpClient) {
-            $ideName = $mcpClient->ideName();
+            $ideName = $mcpClient->mcpClientName();
             $ideDisplay = str_pad($ideName, $longestIdeName);
             $this->output->write("  {$ideDisplay}... ");
             $results = [];

src/Contracts/Agent.php

@@ -9,6 +9,13 @@
  */
 interface Agent
 {
+    /**
+     * Get the display name of the Agent.
+     *
+     * @return string|null
+     */
+    public function agentName(): ?string;
+
     /**
      * Get the file path where AI guidelines should be written.
      *

src/Contracts/McpClient.php

@@ -9,6 +9,13 @@
  */
 interface McpClient
 {
+    /**
+     * Get the display name of the MCP (Model Context Protocol) client.
+     *
+     * @return string|null
+     */
+    public function mcpClientName(): ?string;
+
     /**
      * Install an MCP server configuration in this IDE.
      *

src/Install/CodeEnvironment/CodeEnvironment.php

@@ -28,7 +28,7 @@ public function agentName(): ?string
         return $this->name();
     }
 
-    public function ideName(): ?string
+    public function mcpClientName(): ?string
     {
         return $this->name();
     }
@@ -71,7 +71,7 @@ public function IsAgent(): bool
 
     public function isMcpClient(): bool
     {
-        return $this->ideName() && $this instanceof McpClient;
+        return $this->mcpClientName() && $this instanceof McpClient;
     }
 
     public function mcpInstallationStrategy(): McpInstallationStrategy

src/Install/CodeEnvironment/Copilot.php

@@ -39,7 +39,7 @@ public function detectOnSystem(Platform $platform): bool
         return false;
     }
 
-    public function ideName(): ?string
+    public function mcpClientName(): ?string
     {
         return null;
     }

src/Install/CodeEnvironment/VSCode.php

@@ -16,7 +16,7 @@ public function name(): string
 
     public function displayName(): string
     {
-        return 'Visual Studio Code';
+        return 'Vs Code';
     }
 
     public function systemDetectionConfig(Platform $platform): array
@@ -44,11 +44,6 @@ public function projectDetectionConfig(): array
         ];
     }
 
-    public function agentName(): ?string
-    {
-        return null;
-    }
-
     public function mcpConfigPath(): string
     {
         return '.vscode/mcp.json';

tests/Unit/Install/CodeEnvironment/CodeEnvironmentTest.php

@@ -128,10 +128,10 @@ public function mcpConfigPath(): string
     expect($environment->agentName())->toBe('test');
 });
 
-test('ideName returns name by default', function () {
+test('mcpClientName returns name by default', function () {
     $environment = new TestCodeEnvironment($this->strategyFactory);
 
-    expect($environment->ideName())->toBe('test');
+    expect($environment->mcpClientName())->toBe('test');
 });
 
 test('IsAgent returns true when implements Agent interface and has agentName', function () {
@@ -146,7 +146,7 @@ public function mcpConfigPath(): string
     expect($environment->IsAgent())->toBe(false);
 });
 
-test('isMcpClient returns true when implements McpClient interface and has ideName', function () {
+test('isMcpClient returns true when implements McpClient interface and has mcpClientName', function () {
     $mcpClient = new TestMcpClient($this->strategyFactory);
 
     expect($mcpClient->isMcpClient())->toBe(true);