fix: address Copilot review feedback

- Default bypass_product_check to true so OpenSearch works out-of-the-box
- Remove unused \$apiVersion assignment in Bulk::_processResponse()
- Remove duplicate native_constant_invocation rule from CS config
- Fix Request context in Pipeline::deletePipeline() catch (DELETE method + full path)
- Fix Request context in Reindex::run() catch (POST method + body)
- Add array|string type to Client::__construct \$config parameter

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: sumitkhopade1986

.php-cs-fixer.dist.php

@@ -15,7 +15,6 @@
         '@Symfony' => true,
         'is_null' => true,
         'method_argument_space' => ['on_multiline' => 'ensure_fully_multiline'],
-        'native_constant_invocation' => true,
         'native_function_invocation' => false,
         'no_alias_functions' => true,
         'no_useless_else' => true,

src/Bulk.php

@@ -311,7 +311,6 @@ protected function _processResponse(Response $response): ResponseSet
             case 413: throw new RequestEntityTooLargeException();
         }
 
-        $apiVersion = $this->_client->getApiVersion();
         $responseData = $response->getData();
 
         $actions = $this->getActions();

src/Client.php

@@ -88,7 +88,7 @@ class Client
      * @throws InvalidException
      */
     public function __construct(
-        $config = [],
+        array|string $config = [],
         $callback = null,
         ?LoggerInterface $logger = null,
         ?RequestCounterInterface $requestCounter = null,

src/Connection.php

@@ -307,9 +307,11 @@ public function getClient(): ElasticsearchClient
 
         $stack = HandlerStack::create();
 
-        // When connecting to OpenSearch (which does not send the X-Elastic-Product header
-        // required by elasticsearch-php v9), set bypass_product_check=true in connection params.
-        if ($this->hasParam('bypass_product_check') && $this->getParam('bypass_product_check')) {
+        // Inject X-Elastic-Product header by default so OpenSearch clusters (which do not send
+        // this header) pass the elasticsearch-php v9 product check. Set bypass_product_check=false
+        // to disable this behaviour when strict product verification is required.
+        $bypassProductCheck = !$this->hasParam('bypass_product_check') || $this->getParam('bypass_product_check');
+        if ($bypassProductCheck) {
             $stack->push(Middleware::mapResponse(
                 static function (ResponseInterface $response): ResponseInterface {
                     return $response->withHeader('X-Elastic-Product', 'Elasticsearch');

src/Pipeline.php

@@ -101,7 +101,7 @@ public function deletePipeline(string $id): Response
                 $bodyStream->rewind();
             }
             $elasticaResponse = new Response((string) $bodyStream, $psrResponse->getStatusCode());
-            throw new ResponseException(new Request($id), $elasticaResponse);
+            throw new ResponseException(new Request('_ingest/pipeline/'.$id, Request::DELETE), $elasticaResponse);
         }
 
         return new Response($esResponse->asArray(), $esResponse->getStatusCode());

src/Reindex.php

@@ -92,7 +92,7 @@ public function run(): Response
                 $bodyStream->rewind();
             }
             $elasticaResponse = new Response((string) $bodyStream, $psrResponse->getStatusCode());
-            throw new ResponseException(new Request('_reindex'), $elasticaResponse);
+            throw new ResponseException(new Request('_reindex', Request::POST, $body), $elasticaResponse);
         }
         $this->_lastResponse = new Response($esResponse->asArray(), $esResponse->getStatusCode());