Fix parseLiteral not called on per-schema scalar overrides for inline arguments

[ruudk]

AST::valueFromAST() called parseLiteral() on the built-in scalar singleton from field definitions, not the custom override registered via the type loader. This meant inline literal arguments like node(id: 123) silently skipped custom parsing logic.

Pass the Schema into valueFromAST() and resolve built-in scalars from it before calling parseLiteral(), mirroring the parseValue() fix in Values::coerceInputValue().

Also thread the schema through getDirectiveValues() so directive arguments (e.g. @include(if: true)) use schema-resolved scalars too.

Extract Type::isBuiltInScalarName() to centralize the name check without requiring an instanceof guard.

Related to #1869

[Copilot]

Pull request overview

This PR fixes an execution-time coercion bug where inline literal arguments (e.g. node(id: 123)) could bypass per-schema overrides of built-in scalars by calling parseLiteral() on the built-in singleton instead of the schema-resolved scalar instance.

Changes:

• Add optional Schema plumbing to AST::valueFromAST() and Values::getArgumentValues() so coercion can resolve built-in scalars from the active schema before invoking parseLiteral().
• Centralize the built-in scalar name check via Type::isBuiltInScalarName().
• Add regression tests for inline-argument parseLiteral() invocation and unit tests for the new helper; update generated class-reference docs.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
src/Utils/AST.php	Adds schema-aware scalar resolution before parseLiteral() for built-in scalar names.
src/Executor/Values.php	Threads optional schema into argument coercion so inline literals can use schema overrides.
src/Executor/ReferenceExecutor.php	Passes execution schema into argument coercion during field resolution.
src/Type/Definition/Type.php	Extracts isBuiltInScalarName() helper and uses it for built-in scalar checks.
tests/Type/ScalarOverridesTest.php	Regression test ensuring overridden ID scalar’s parseLiteral() is called for inline args.
tests/Type/Definition/TypeTest.php	Unit tests for Type::isBuiltInScalarName().
docs/class-reference.md	Updates AST::valueFromAST() signature documentation to include the schema parameter.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

[ruudk]

@spawnia Don't have time today anymore to apply the feedback. If you could have a look that would be great. Otherwise I'll continue tomorrow.

[ruudk]

Thanks!