Add PHP-CS-Fixer for code formatting

Set up PHP code formatting with PHP-CS-Fixer:
- Add friendsofphp/php-cs-fixer as dev dependency
- Create .php-cs-fixer.php configuration (PSR-12 based)
- Add composer scripts: cs-fix, cs-check
- Update .gitignore with .php-cs-fixer.cache
- Add Code Formatting section to README
- Apply formatting fixes to all PHP files

Changes:
- composer.json: Add php-cs-fixer dependency and scripts
- .php-cs-fixer.php: Configuration file
- .gitignore: Add cache file
- README.md: Add Code Formatting section
- src/*.php: Apply PSR-12 formatting
- tests/*.php: Apply formatting, remove unused imports
- examples/*.php: Apply formatting

Author: RahulHere

.gitignore

@@ -8,6 +8,9 @@ composer.lock
 phpunit.xml
 coverage/
 
+# PHP-CS-Fixer
+.php-cs-fixer.cache
+
 # IDE - PhpStorm
 .idea/
 *.iws

.php-cs-fixer.php

@@ -0,0 +1,24 @@
+<?php
+
+$finder = PhpCsFixer\Finder::create()
+    ->in(__DIR__ . '/src')
+    ->in(__DIR__ . '/tests')
+    ->in(__DIR__ . '/examples')
+    ->name('*.php')
+    ->ignoreDotFiles(true)
+    ->ignoreVCS(true);
+
+return (new PhpCsFixer\Config())
+    ->setRules([
+        '@PSR12' => true,
+        'array_syntax' => ['syntax' => 'short'],
+        'ordered_imports' => ['sort_algorithm' => 'alpha'],
+        'no_unused_imports' => true,
+        'single_quote' => true,
+        'trailing_comma_in_multiline' => true,
+        'blank_line_before_statement' => [
+            'statements' => ['return', 'throw', 'try'],
+        ],
+    ])
+    ->setFinder($finder)
+    ->setRiskyAllowed(false);

README.md

@@ -491,6 +491,23 @@ ANTHROPIC_API_KEY=your-key php examples/client_example_json.php
 
 ## Development
 
+### Code Formatting
+
+```bash
+# Format code
+composer cs-fix
+
+# Check formatting without making changes
+composer cs-check
+
+# Or run directly
+./vendor/bin/php-cs-fixer fix
+./vendor/bin/php-cs-fixer fix --dry-run --diff
+
+# Run tests
+./vendor/bin/phpunit --testdox
+```
+
 ### Project Structure
 
 ```

composer.json

@@ -16,6 +16,7 @@
         "ext-ffi": "*"
     },
     "require-dev": {
+        "friendsofphp/php-cs-fixer": "^3.93",
         "phpunit/phpunit": "^9.5"
     },
     "autoload": {
@@ -30,7 +31,9 @@
     },
     "scripts": {
         "test": "phpunit --testdox",
-        "test-coverage": "phpunit --coverage-html coverage"
+        "test-coverage": "phpunit --coverage-html coverage",
+        "cs-fix": "php-cs-fixer fix",
+        "cs-check": "php-cs-fixer fix --dry-run --diff"
     },
     "config": {
         "sort-packages": true

examples/client_example_json.php

@@ -71,6 +71,6 @@
     $agent->dispose();
 
 } catch (Exception $e) {
-    fwrite(STDERR, "Error: " . $e->getMessage() . "\n");
+    fwrite(STDERR, 'Error: ' . $e->getMessage() . "\n");
     exit(1);
 }

src/ConfigBuilder.php

@@ -31,6 +31,7 @@ public static function create(): self
     public function withProvider(string $provider): self
     {
         $this->provider = $provider;
+
         return $this;
     }
 
@@ -43,6 +44,7 @@ public function withProvider(string $provider): self
     public function withModel(string $model): self
     {
         $this->model = $model;
+
         return $this;
     }
 
@@ -56,6 +58,7 @@ public function withModel(string $model): self
     public function withApiKey(string $apiKey): self
     {
         $this->apiKey = $apiKey;
+
         return $this;
     }
 
@@ -69,6 +72,7 @@ public function withApiKey(string $apiKey): self
     public function withServerConfig(string $serverConfig): self
     {
         $this->serverConfig = $serverConfig;
+
         return $this;
     }
 

src/GopherAgent.php

@@ -64,6 +64,7 @@ public static function create(Config $config): self
             $errorMsg = GopherOrch::getLastError();
             GopherOrch::clearError();
             $message = $errorMsg !== '' ? $errorMsg : 'Failed to create agent';
+
             throw new AgentException($message);
         }
 
@@ -165,6 +166,7 @@ public function runDetailedWithTimeout(string $query, int $timeoutMs): AgentResu
     {
         try {
             $response = $this->runWithTimeout($query, $timeoutMs);
+
             return AgentResult::success($response);
         } catch (DisposedException $e) {
             return AgentResult::error($e->getMessage());
@@ -173,6 +175,7 @@ public function runDetailedWithTimeout(string $query, int $timeoutMs): AgentResu
             if (stripos($e->getMessage(), 'timeout') !== false) {
                 return AgentResult::timeout($e->getMessage());
             }
+
             return AgentResult::error($e->getMessage());
         }
     }

src/GopherOrch.php

@@ -85,6 +85,7 @@ public static function isAvailable(): bool
     {
         try {
             self::init();
+
             return true;
         } catch (LibraryException $e) {
             return false;
@@ -149,6 +150,7 @@ public static function getFFI(): FFI
     public static function agentCreateByJson(string $provider, string $model, string $serverJson)
     {
         $ffi = self::getFFI();
+
         return $ffi->gopher_orch_agent_create_by_json($provider, $model, $serverJson);
     }
 
@@ -163,6 +165,7 @@ public static function agentCreateByJson(string $provider, string $model, string
     public static function agentCreateByApiKey(string $provider, string $model, string $apiKey)
     {
         $ffi = self::getFFI();
+
         return $ffi->gopher_orch_agent_create_by_api_key($provider, $model, $apiKey);
     }
 

tests/ConfigBuilderTest.php

@@ -4,7 +4,6 @@
 
 namespace GopherSecurity\Orch\Tests;
 
-use GopherSecurity\Orch\Config;
 use GopherSecurity\Orch\ConfigBuilder;
 use PHPUnit\Framework\TestCase;
 

tests/GopherAgentIntegrationTest.php

@@ -5,10 +5,10 @@
 namespace GopherSecurity\Orch\Tests;
 
 use GopherSecurity\Orch\ConfigBuilder;
-use GopherSecurity\Orch\GopherAgent;
-use GopherSecurity\Orch\GopherOrch;
 use GopherSecurity\Orch\Exception\AgentException;
 use GopherSecurity\Orch\Exception\DisposedException;
+use GopherSecurity\Orch\GopherAgent;
+use GopherSecurity\Orch\GopherOrch;
 use PHPUnit\Framework\TestCase;
 
 /**
@@ -95,6 +95,7 @@ public function testCreateAgentWithServerConfig(): void
             ->build();
 
         $agent = null;
+
         try {
             $agent = GopherAgent::create($config);
             $this->assertNotNull($agent, 'Agent should be created');
@@ -115,6 +116,7 @@ public function testCreateAgentWithHelperMethod(): void
         $this->skipIfNativeLibraryNotAvailable();
 
         $agent = $this->tryCreateAgent();
+
         try {
             $this->assertNotNull($agent);
         } finally {
@@ -153,6 +155,7 @@ public function testRunDetailedReturnsResult(): void
         $this->skipIfNativeLibraryNotAvailable();
 
         $agent = $this->tryCreateAgent();
+
         try {
             // Run with very short timeout to get quick response (likely timeout or error)
             $result = $agent->runDetailedWithTimeout('test query', 100);