Fix OperationParams not copying originalInput/readOnly from base

Causing TypeError on validation errors

Author: mfn

CHANGELOG.md

@@ -12,6 +12,7 @@ CHANGELOG
 ### Fixed
 - Fix `SelectFields` crashing when field types use callable [\#1252 / mfn](https://github.com/rebing/graphql-laravel/pull/1252)
 - Fix APQ middleware race condition (TOCTOU) [\#1253 / mfn](https://github.com/rebing/graphql-laravel/pull/1253)
+- Fix `OperationParams` not copying `originalInput`/`readOnly`, causing TypeError [\#1254 / mfn](https://github.com/rebing/graphql-laravel/pull/1254)
 
 2026-03-21, 10.0.0-RC2
 ----------------------

src/GraphQL.php

@@ -134,6 +134,8 @@ public function queryAndReturnResult(string $query, ?array $variables = null, ar
         $baseParams->query = $query;
         $baseParams->operation = $operationName;
         $baseParams->variables = $variables;
+        $baseParams->originalInput = [];
+        $baseParams->readOnly = false;
         $params = new OperationParams($baseParams);
 
         return $this->executeAndReturnResult($schemaName, $schema, $params, $rootValue, $context);

src/Support/OperationParams.php

@@ -28,6 +28,8 @@ protected function init(BaseOperationParams $baseOperationParams): void
         $this->operation = $baseOperationParams->operation;
         $this->variables = $baseOperationParams->variables;
         $this->extensions = $baseOperationParams->extensions;
+        $this->originalInput = $baseOperationParams->originalInput;
+        $this->readOnly = $baseOperationParams->readOnly;
 
         $this->baseOperationParams = $baseOperationParams;
     }

tests/Unit/OperationParamsTest.php

@@ -69,12 +69,8 @@ public function testGetParsedQueryReturnsParsedDocument(): void
     /**
      * When webonyx's Helper::validateOperationParams() detects invalid variables,
      * it accesses $params->originalInput['variables'] to build the error message.
-     *
-     * Since OperationParams::init() doesn't copy originalInput from the base,
-     * the typed property (array) is uninitialized, causing a TypeError instead
-     * of returning the proper validation error.
      */
-    public function testValidateOperationParamsWithInvalidVariablesCrashesOnUninitializedOriginalInput(): void
+    public function testValidateOperationParamsWithInvalidVariablesReturnsValidationError(): void
     {
         $base = BaseOperationParams::create([
             'query' => '{ hello }',
@@ -84,11 +80,12 @@ public function testValidateOperationParamsWithInvalidVariablesCrashesOnUninitia
 
         $helper = new Helper;
 
-        // Should return a validation error about variables,
-        // but crashes with TypeError because originalInput is uninitialized
-        $this->expectException(\Error::class);
-        $this->expectExceptionMessage('must not be accessed before initialization');
+        $errors = $helper->validateOperationParams($params);
 
-        $helper->validateOperationParams($params);
+        self::assertCount(1, $errors);
+        self::assertSame(
+            'GraphQL Request parameter "variables" must be object or JSON string parsed to object, but got "[1, 2]"',
+            $errors[0]->getMessage(),
+        );
     }
 }