refactor(connection): remove redundant connection rebuild logic

- Eliminated the unused `rebuildConnection` method.
- Removed the redundant `rebuild` property from the Connection class.
- Adjusted `getClient` to build the connection if the client is missing.
- Deleted the call to `rebuildConnection` in the Bridge class.

Because who needs another rebuild when you can just health-check? 🔧✨

Author: UseTheFork

src/Connection.php

@@ -74,8 +74,6 @@ class Connection extends BaseConnection
 
     protected mixed $elasticMetaHeader = null;
 
-    protected bool $rebuild = false;
-
     protected string $connectionName;
 
     /**
@@ -214,11 +212,6 @@ public function getDriverName(): string
         return 'elasticsearch';
     }
 
-    public function rebuildConnection(): void
-    {
-        $this->rebuild = true;
-    }
-
     public function getClient(): ?Client
     {
         return $this->client;
@@ -244,9 +237,10 @@ public function __call($method, $parameters)
         if (! $this->index) {
             $this->index = $this->indexPrefix.'*';
         }
-        if ($this->rebuild) {
+
+        // If we are missing a database connection client we need to reconnect.
+        if (! $this->client) {
             $this->client = $this->buildConnection();
-            $this->rebuild = false;
         }
         $bridge = new Bridge($this);
 

src/DSL/Bridge.php

@@ -1434,7 +1434,6 @@ private function _throwError(Exception $exception, $params, $queryTag): QueryExc
         $errorMsg = $exception->getMessage();
         $errorCode = $exception->getCode();
         $queryTag = str_replace('_', '', $queryTag);
-        $this->connection->rebuildConnection();
         $error = new Results([], [], $params, $queryTag);
         $error->setError($errorMsg, $errorCode);