Fix code style issues: proper namespace imports and class element ordering

Co-authored-by: Thavarshan <10804999+Thavarshan@users.noreply.github.com>

Author: Copilot

src/Fetch/Concerns/ManagesConnectionPool.php

@@ -35,6 +35,20 @@ trait ManagesConnectionPool
      */
     protected bool $poolingEnabled = false;
 
+    /**
+     * Initialize connection pooling with default configuration.
+     *
+     * @param  PoolConfiguration|null  $config  Optional pool configuration
+     */
+    protected static function initializePool(?PoolConfiguration $config = null): void
+    {
+        if (self::$connectionPool === null) {
+            $poolConfig = $config ?? new PoolConfiguration;
+            self::$connectionPool = new ConnectionPool($poolConfig);
+            self::$dnsCache = new DnsCache($poolConfig->getDnsCacheTtl());
+        }
+    }
+
     /**
      * Configure connection pooling for this handler.
      *
@@ -213,20 +227,6 @@ public function resetPool(): ClientHandler
         return $this;
     }
 
-    /**
-     * Initialize connection pooling with default configuration.
-     *
-     * @param  PoolConfiguration|null  $config  Optional pool configuration
-     */
-    protected static function initializePool(?PoolConfiguration $config = null): void
-    {
-        if (self::$connectionPool === null) {
-            $poolConfig = $config ?? new PoolConfiguration;
-            self::$connectionPool = new ConnectionPool($poolConfig);
-            self::$dnsCache = new DnsCache($poolConfig->getDnsCacheTtl());
-        }
-    }
-
     /**
      * Get cURL options for HTTP/2 support.
      *

src/Fetch/Pool/DnsCache.php

@@ -5,6 +5,7 @@
 namespace Fetch\Pool;
 
 use Fetch\Exceptions\NetworkException;
+use GuzzleHttp\Psr7\Request;
 
 /**
  * DNS caching for improved performance.
@@ -124,6 +125,25 @@ public function setTtl(int $ttl): self
         return $this;
     }
 
+    /**
+     * Prune expired entries from the cache.
+     *
+     * @return int Number of entries removed
+     */
+    public function prune(): int
+    {
+        $removed = 0;
+
+        foreach ($this->cache as $key => $entry) {
+            if ($this->isExpired($entry)) {
+                unset($this->cache[$key]);
+                $removed++;
+            }
+        }
+
+        return $removed;
+    }
+
     /**
      * Check if a cache entry is expired.
      *
@@ -179,29 +199,10 @@ protected function performDnsLookup(string $hostname): array
         if (empty($addresses)) {
             throw new NetworkException(
                 "Failed to resolve hostname: {$hostname}",
-                new \GuzzleHttp\Psr7\Request('GET', "https://{$hostname}/")
+                new Request('GET', "https://{$hostname}/")
             );
         }
 
         return $addresses;
     }
-
-    /**
-     * Prune expired entries from the cache.
-     *
-     * @return int Number of entries removed
-     */
-    public function prune(): int
-    {
-        $removed = 0;
-
-        foreach ($this->cache as $key => $entry) {
-            if ($this->isExpired($entry)) {
-                unset($this->cache[$key]);
-                $removed++;
-            }
-        }
-
-        return $removed;
-    }
 }

src/Fetch/Pool/HostConnectionPool.php

@@ -106,6 +106,46 @@ public function returnConnection(Connection $connection): void
         }
     }
 
+    /**
+     * Get the number of available connections.
+     */
+    public function getAvailableCount(): int
+    {
+        return $this->availableConnections->count();
+    }
+
+    /**
+     * Get pool statistics.
+     *
+     * @return array<string, mixed>
+     */
+    public function getStats(): array
+    {
+        return [
+            'host' => $this->host,
+            'port' => $this->port,
+            'ssl' => $this->ssl,
+            'available' => $this->availableConnections->count(),
+            'total_created' => $this->totalCreated,
+            'total_borrowed' => $this->totalBorrowed,
+            'total_returned' => $this->totalReturned,
+            'success_rate' => $this->totalBorrowed > 0
+                ? $this->totalReturned / $this->totalBorrowed
+                : 1.0,
+        ];
+    }
+
+    /**
+     * Close all connections in the pool.
+     */
+    public function closeAll(): void
+    {
+        while (! $this->availableConnections->isEmpty()) {
+            $connection = $this->availableConnections->dequeue();
+            $connection->close();
+        }
+    }
+
     /**
      * Create a new connection.
      *
@@ -175,44 +215,4 @@ protected function warmupConnections(): void
             }
         }
     }
-
-    /**
-     * Get the number of available connections.
-     */
-    public function getAvailableCount(): int
-    {
-        return $this->availableConnections->count();
-    }
-
-    /**
-     * Get pool statistics.
-     *
-     * @return array<string, mixed>
-     */
-    public function getStats(): array
-    {
-        return [
-            'host' => $this->host,
-            'port' => $this->port,
-            'ssl' => $this->ssl,
-            'available' => $this->availableConnections->count(),
-            'total_created' => $this->totalCreated,
-            'total_borrowed' => $this->totalBorrowed,
-            'total_returned' => $this->totalReturned,
-            'success_rate' => $this->totalBorrowed > 0
-                ? $this->totalReturned / $this->totalBorrowed
-                : 1.0,
-        ];
-    }
-
-    /**
-     * Close all connections in the pool.
-     */
-    public function closeAll(): void
-    {
-        while (! $this->availableConnections->isEmpty()) {
-            $connection = $this->availableConnections->dequeue();
-            $connection->close();
-        }
-    }
 }