Fix further CI errors (code sniffer)

Author: MarcelloDuarte

phpcs.xml

@@ -13,6 +13,16 @@
     <exclude-pattern>*/vendor/*</exclude-pattern>
     <exclude-pattern>*/coverage/*</exclude-pattern>
 
+    <!-- Allow descriptive test method names in test files -->
+    <rule ref="PSR1.Methods.CamelCapsMethodName">
+        <exclude-pattern>*/tests/*</exclude-pattern>
+    </rule>
+
+    <!-- Allow multiple classes in test files -->
+    <rule ref="PSR1.Classes.ClassDeclaration.MultipleClasses">
+        <exclude-pattern>*/tests/*</exclude-pattern>
+    </rule>
+
     <!-- Show progress -->
     <arg value="p"/>
 

src/Integrations/LLMClientInterface.php

@@ -4,38 +4,38 @@
 
 /**
  * LLM Client Interface
- * 
+ *
  * Abstract interface for LLM providers (OpenAI, Anthropic, etc.)
  * Allows AIAgent to work with any LLM provider without tight coupling.
  */
 interface LLMClientInterface
 {
     /**
      * Generate a chat completion response
-     * 
+     *
      * @param array $requestData Request data including messages, model, etc.
      * @return array Response data from the LLM provider
      */
     public function chat(array $requestData): array;
-    
+
     /**
      * Get supported models for this provider
-     * 
+     *
      * @return array List of supported model names
      */
     public function getSupportedModels(): array;
-    
+
     /**
      * Get provider name
-     * 
+     *
      * @return string Provider name (e.g., 'openai', 'anthropic')
      */
     public function getProviderName(): string;
-    
+
     /**
      * Check if the provider supports tool calling
-     * 
+     *
      * @return bool True if tool calling is supported
      */
     public function supportsToolCalling(): bool;
-} 
+}

src/Integrations/MockOpenAIClient.php

@@ -4,33 +4,33 @@
 
 /**
  * Mock OpenAI Client for Testing
- * 
+ *
  * Mock implementation of OpenAIClient interface for testing purposes.
  * Returns predefined responses without making actual API calls.
  */
 class MockOpenAIClient implements OpenAIClient
 {
     private array $responses = [];
     private bool $echoInput = true;
-    
+
     public function __construct(array $responses = [], bool $echoInput = true)
     {
         $this->responses = $responses;
         $this->echoInput = $echoInput;
     }
-    
+
     public function chat(array $context): array
     {
         $userMessage = $this->extractUserMessage($context);
-        
+
         if (isset($this->responses[$userMessage])) {
             $response = $this->responses[$userMessage];
         } elseif ($this->echoInput) {
             $response = $userMessage;
         } else {
             $response = 'Mock response for: ' . $userMessage;
         }
-        
+
         return [
             'choices' => [
                 [
@@ -48,32 +48,32 @@ public function chat(array $context): array
             ]
         ];
     }
-    
+
     private function extractUserMessage(array $context): string
     {
         $messages = $context['messages'] ?? [];
-        
+
         foreach ($messages as $message) {
             if ($message['role'] === 'user') {
                 return $message['content'];
             }
         }
-        
+
         return '';
     }
-    
+
     public function setResponses(array $responses): self
     {
         $this->responses = $responses;
         return $this;
     }
-    
+
     public function setEchoInput(bool $echoInput): self
     {
         $this->echoInput = $echoInput;
         return $this;
     }
-    
+
     public function getSupportedModels(): array
     {
         return [
@@ -82,14 +82,14 @@ public function getSupportedModels(): array
             'mock-model'
         ];
     }
-    
+
     public function getProviderName(): string
     {
         return 'mock-openai';
     }
-    
+
     public function supportsToolCalling(): bool
     {
         return true;
     }
-} 
+}

src/Integrations/OpenAIClient.php

@@ -7,7 +7,7 @@
  *
  * Provides a contract for making calls to OpenAI's API,
  * allowing for both real and mock implementations.
- * 
+ *
  * @deprecated Use LLMClientInterface instead for better abstraction
  */
 interface OpenAIClient extends LLMClientInterface

src/Integrations/RealOpenAIClient.php

@@ -4,37 +4,37 @@
 
 /**
  * Real OpenAI Client Implementation
- * 
+ *
  * Concrete implementation of OpenAIClient interface for making
  * actual calls to OpenAI API.
  */
 class RealOpenAIClient implements OpenAIClient
 {
     private string $apiKey;
     private string $baseUrl = 'https://api.openai.com/v1';
-    
+
     public function __construct(string $apiKey = '')
     {
         $this->apiKey = $apiKey ?: getenv('OPENAI_API_KEY') ?: '';
     }
-    
+
     public function chat(array $context): array
     {
         if (empty($this->apiKey)) {
             throw new \InvalidArgumentException('OpenAI API key is required');
         }
-        
+
         $url = $this->baseUrl . '/chat/completions';
         $headers = [
             'Authorization: Bearer ' . $this->apiKey,
             'Content-Type: application/json',
         ];
-        
+
         $response = $this->makeRequest($url, $context, $headers);
-        
+
         return json_decode($response, true) ?: [];
     }
-    
+
     public function getSupportedModels(): array
     {
         return [
@@ -45,17 +45,17 @@ public function getSupportedModels(): array
             'gpt-4-turbo-preview'
         ];
     }
-    
+
     public function getProviderName(): string
     {
         return 'openai';
     }
-    
+
     public function supportsToolCalling(): bool
     {
         return true;
     }
-    
+
     private function makeRequest(string $url, array $data, array $headers): string
     {
         $ch = curl_init();
@@ -67,22 +67,22 @@ private function makeRequest(string $url, array $data, array $headers): string
             CURLOPT_RETURNTRANSFER => true,
             CURLOPT_TIMEOUT => 30,
         ]);
-        
+
         $response = curl_exec($ch);
         $httpCode = curl_getinfo($ch, CURLINFO_HTTP_CODE);
-        
+
         if ($response === false) {
             $error = curl_error($ch);
             curl_close($ch);
             throw new \RuntimeException("cURL request failed: {$error}");
         }
-        
+
         curl_close($ch);
-        
+
         if ($httpCode !== 200) {
             throw new \RuntimeException("OpenAI API request failed with HTTP code: {$httpCode}");
         }
-        
+
         return (string) $response;
     }
-} 
+}