Merge pull request #1254 from rebing/mfn/op-params

Fix OperationParams not copying originalInput/readOnly from base

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

@@ -4,6 +4,7 @@
 namespace Rebing\GraphQL\Tests\Unit;
 
 use GraphQL\Error\Error;
+use GraphQL\Server\Helper;
 use GraphQL\Server\OperationParams as BaseOperationParams;
 use PHPUnit\Framework\TestCase;
 use Rebing\GraphQL\Support\OperationParams;
@@ -64,4 +65,27 @@ public function testGetParsedQueryReturnsParsedDocument(): void
         $document2 = $params->getParsedQuery();
         self::assertSame($document, $document2);
     }
+
+    /**
+     * When webonyx's Helper::validateOperationParams() detects invalid variables,
+     * it accesses $params->originalInput['variables'] to build the error message.
+     */
+    public function testValidateOperationParamsWithInvalidVariablesReturnsValidationError(): void
+    {
+        $base = BaseOperationParams::create([
+            'query' => '{ hello }',
+            'variables' => '[1, 2]', // JSON list, not an object — triggers validation error
+        ]);
+        $params = new OperationParams($base);
+
+        $helper = new Helper;
+
+        $errors = $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(),
+        );
+    }
 }