refactor(debug): simplify debug info output in Client and Response classes

guanguans · guanguans · commit 4d4a9da93e38 · 2025-11-17T21:26:14.000+08:00
- Comment out detailed debug info fields in __debugInfo methods to reduce verbosity
- Ensure getHttpClient is called in Client::__debugInfo for side effects or initialization
- Keep summary information in Response::__debugInfo while hiding handlerStats and decodedBody details
- Improve clarity and maintainability by limiting debug data exposed during debugging

Signed-off-by: guanguans &lt;ityaozm@gmail.com&gt;
diff --git a/src/Foundation/Client.php b/src/Foundation/Client.php
@@ -36,12 +36,14 @@ public function __construct(?Authenticator $authenticator = null)
 
     public function __debugInfo(): array
     {
+        $this->getHttpClient();
+
         return $this->mergeDebugInfo([
-            'httpClient' => $this->getHttpClient(),
-            'httpClientResolver' => $this->getHttpClientResolver(),
-            'handlerStack' => $this->getHandlerStack(),
-            'handlerStackResolver' => $this->getHandlerStackResolver(),
-            'httpOptions' => $this->getHttpOptions(),
+            // 'httpClient' => $this->getHttpClient(),
+            // 'httpClientResolver' => $this->getHttpClientResolver(),
+            // 'handlerStack' => $this->getHandlerStack(),
+            // 'handlerStackResolver' => $this->getHandlerStackResolver(),
+            // 'httpOptions' => $this->getHttpOptions(),
         ]);
     }
 
diff --git a/src/Foundation/Response.php b/src/Foundation/Response.php
@@ -63,11 +63,13 @@ class Response extends \GuzzleHttp\Psr7\Response implements \ArrayAccess, \Strin
      */
     public function __debugInfo(): array
     {
+        $this->json();
+
         return $this->mergeDebugInfo([
-            'handlerStats' => $this->handlerStats(),
+            // 'handlerStats' => $this->handlerStats(),
             'requestSummary' => $this->request instanceof RequestInterface ? Message::toString($this->request) : null,
             'responseSummary' => Message::toString($this),
-            'decodedBody' => $this->json(),
+            // 'decodedBody' => $this->json(),
         ]);
     }
 
diff --git a/src/ZohoCliq/Authenticator.php b/src/ZohoCliq/Authenticator.php
@@ -134,13 +134,15 @@ private function authMiddleware(): callable
      */
     private function baseUriMiddleware(string $baseUri): callable
     {
-        $parsedBaseUri = parse_url($baseUri);
-
         return Middleware::mapRequest(
-            static fn (RequestInterface $request): RequestInterface => $request->withUri(
-                $request->getUri()->withScheme($parsedBaseUri['scheme'])->withHost($parsedBaseUri['host']),
-                $request->hasHeader('Host')
-            ),
+            static function (RequestInterface $request) use ($baseUri): RequestInterface {
+                $parsedBaseUri = parse_url($baseUri);
+
+                return $request->withUri(
+                    $request->getUri()->withScheme($parsedBaseUri['scheme'])->withHost($parsedBaseUri['host']),
+                    $request->hasHeader('Host')
+                );
+            },
         );
     }
 
