refactor: update McpInstallationStrategy casing to align with consistent enum naming convention

- Renamed enum cases in `McpInstallationStrategy` to uppercase for standardization.
- Updated references across codebase and adjusted related test cases.

Author: pushpak1300

src/Install/CodeEnvironment/ClaudeCode.php

@@ -43,7 +43,7 @@ public function projectDetectionConfig(): array
 
     public function mcpInstallationStrategy(): McpInstallationStrategy
     {
-        return McpInstallationStrategy::Shell;
+        return McpInstallationStrategy::SHELL;
     }
 
     public function shellMcpCommand(): string

src/Install/CodeEnvironment/CodeEnvironment.php

@@ -76,7 +76,7 @@ public function isMcpClient(): bool
 
     public function mcpInstallationStrategy(): McpInstallationStrategy
     {
-        return McpInstallationStrategy::File;
+        return McpInstallationStrategy::FILE;
     }
 
     public function shellMcpCommand(): ?string
@@ -113,9 +113,9 @@ public function mcpConfigKey(): string
     public function installMcp(string $key, string $command, array $args = [], array $env = []): bool
     {
         return match($this->mcpInstallationStrategy()) {
-            McpInstallationStrategy::Shell => $this->installShellMcp($key, $command, $args, $env),
-            McpInstallationStrategy::File => $this->installFileMcp($key, $command, $args, $env),
-            McpInstallationStrategy::None => false
+            McpInstallationStrategy::SHELL => $this->installShellMcp($key, $command, $args, $env),
+            McpInstallationStrategy::FILE => $this->installFileMcp($key, $command, $args, $env),
+            McpInstallationStrategy::NONE => false
         };
     }
 

src/Install/Enums/McpInstallationStrategy.php

@@ -6,7 +6,7 @@
 
 enum McpInstallationStrategy: string
 {
-    case Shell = 'shell';
-    case File = 'file';
-    case None = 'none';
+    case SHELL = 'shell';
+    case FILE = 'file';
+    case NONE = 'none';
 }

tests/Unit/Install/CodeEnvironment/CodeEnvironmentTest.php

@@ -161,7 +161,7 @@ public function mcpConfigPath(): string
 test('mcpInstallationStrategy returns File by default', function () {
     $environment = new TestCodeEnvironment($this->strategyFactory);
 
-    expect($environment->mcpInstallationStrategy())->toBe(McpInstallationStrategy::File);
+    expect($environment->mcpInstallationStrategy())->toBe(McpInstallationStrategy::FILE);
 });
 
 test('shellMcpCommand returns null by default', function () {
@@ -193,7 +193,7 @@ public function mcpConfigPath(): string
     $environment->shouldAllowMockingProtectedMethods();
 
     $environment->shouldReceive('mcpInstallationStrategy')
-        ->andReturn(McpInstallationStrategy::Shell);
+        ->andReturn(McpInstallationStrategy::SHELL);
 
     $environment->shouldReceive('installShellMcp')
         ->once()
@@ -210,7 +210,7 @@ public function mcpConfigPath(): string
     $environment->shouldAllowMockingProtectedMethods();
 
     $environment->shouldReceive('mcpInstallationStrategy')
-        ->andReturn(McpInstallationStrategy::File);
+        ->andReturn(McpInstallationStrategy::FILE);
 
     $environment->shouldReceive('installFileMcp')
         ->once()
@@ -226,7 +226,7 @@ public function mcpConfigPath(): string
     $environment = Mockery::mock(TestCodeEnvironment::class)->makePartial();
 
     $environment->shouldReceive('mcpInstallationStrategy')
-        ->andReturn(McpInstallationStrategy::None);
+        ->andReturn(McpInstallationStrategy::NONE);
 
     $result = $environment->installMcp('test-key', 'test-command');
 
@@ -249,7 +249,7 @@ public function mcpConfigPath(): string
         ->andReturn('install {key} {command} {args} {env}');
 
     $environment->shouldReceive('mcpInstallationStrategy')
-        ->andReturn(McpInstallationStrategy::Shell);
+        ->andReturn(McpInstallationStrategy::SHELL);
 
     $mockResult = Mockery::mock();
     $mockResult->shouldReceive('successful')->andReturn(true);
@@ -277,7 +277,7 @@ public function mcpConfigPath(): string
         ->andReturn('install {key}');
 
     $environment->shouldReceive('mcpInstallationStrategy')
-        ->andReturn(McpInstallationStrategy::Shell);
+        ->andReturn(McpInstallationStrategy::SHELL);
 
     $mockResult = Mockery::mock();
     $mockResult->shouldReceive('successful')->andReturn(false);
@@ -305,7 +305,7 @@ public function mcpConfigPath(): string
     $environment->shouldAllowMockingProtectedMethods();
 
     $environment->shouldReceive('mcpInstallationStrategy')
-        ->andReturn(McpInstallationStrategy::File);
+        ->andReturn(McpInstallationStrategy::FILE);
 
     File::shouldReceive('ensureDirectoryExists')
         ->once()
@@ -331,7 +331,7 @@ public function mcpConfigPath(): string
     $environment->shouldAllowMockingProtectedMethods();
 
     $environment->shouldReceive('mcpInstallationStrategy')
-        ->andReturn(McpInstallationStrategy::File);
+        ->andReturn(McpInstallationStrategy::FILE);
 
     $existingConfig = json_encode(['mcpServers' => ['existing' => ['command' => 'existing-cmd']]]);