fix(api): throw on first case of 400s

- remove duplicate `$statusCode` variable assignment in `ApiCall.php`
- clean up redundant conditional check for status code 408
- simplify exception handling flow for better code readability

Author: tharropoulos

src/ApiCall.php

@@ -266,15 +266,20 @@ private function makeRequest(string $method, string $endPoint, bool $asJson, arr
                     ->getContents(), true, 512, JSON_THROW_ON_ERROR) : $response->getBody()
                     ->getContents();
             } catch (HttpException $exception) {
-                if (
-                    $exception->getResponse()
-                    ->getStatusCode() === 408
-                ) {
+                $statusCode = $exception->getResponse()->getStatusCode();
+                
+                if ($statusCode === 408) {
                     continue;
                 }
+                
+                if (400 <= $statusCode && $statusCode < 500) {
+                    $this->setNodeHealthCheck($node, false);
+                    throw $this->getException($statusCode)
+                        ->setMessage($exception->getMessage());
+                }
+                
                 $this->setNodeHealthCheck($node, false);
-                $lastException = $this->getException($exception->getResponse()
-                    ->getStatusCode())
+                $lastException = $this->getException($statusCode)
                     ->setMessage($exception->getMessage());
             } catch (TypesenseClientError | HttpClientException $exception) {
                 $this->setNodeHealthCheck($node, false);