After update, validation error: Invalid number, expected digit but got: "N"

👀

From what I can gather, its something to do with your schema having a field with a hyphen in the field.
EG

type Whatever {
  ti-tle: String
}

GraphQL\Error\SyntaxError: Syntax Error: Invalid number, expected digit but got: "t" in GraphQL\Language\Lexer->readDigits() (line 439 of /app/vendor/webonyx/graphql-php/src/Language/Lexer.php).

Do you have any custom schema or config with a hyphen in the field?

I try to prevent you ever using dashes.
It might be something custom or overridden?

Please attach your graphql compose yml and check for any contrib in graphql that might have a field with `-N` in the name.

Thanks for taking a look so soon.

I don't have any schema, am only using GraphQl Compose.

This happened on production. However, today I've tried to replicate the issue on a blank Lando box, but was unable to.

That left me scratching my head, and I've disabled all Entities on Graphql compose, just to make sure. I have about 60, between Content Types, Media, Paragraphs, Taxonomies, Menus and more.

And found out the culprit: Disabling the Menu removed the error and made everything working again.

As this Menu had some custom fields, I will now investigate further what was causing the issue.

Most likely I've done a mistake somewhere, and I guess the updated GraphQl became stricter on checking data (which is a good thing, IMO).

I'm sorry for troubling you. Thank you so much for you suport.

I'll report back as soon as I have found the exact thing that generated the error, so others can read about it too.

I'm having the exact same issue, this only appeared after the graphql 8.x-4.8 update.

Could either of you attach some config please.

I don't have any configs to attach.

Further to bmateus's tests on menus I have found this:

I did a compete fresh install:

Drupal: 10.3.1
GraphQL: 8.x-4.8
GraphQL compose: 2.1.1

1) If you enable the "GraphQL Compose: Menus" sub module and create a new menu that has non single word machine name. For example:

Create a menu called "Test menu" (the machine name automatically given by Drupal will be "test-menu".

2) Enable the menu in /admin/config/graphql_compose

3) Going to the Exploer will fail: /admin/config/graphql/servers/manage/graphql_compose_server/explorer

GraphQL\Error\SyntaxError: Syntax Error: Invalid number, expected digit but got: "T" in GraphQL\Language\Lexer->readDigits() (line 439 of /opt/drupal/vendor/webonyx/graphql-php/src/Language/Lexer.php).

On our production server we have tons of menus named like "PT - Footer menu" which ends up having a machine name with name dashes (PT---footer-menu). This causes a slightly different error:

GraphQL\Error\SyntaxError: Syntax Error: Invalid number, expected digit but got: "-" in GraphQL\Language\Lexer->readDigits() (line 439 of /opt/drupal/vendor/webonyx/graphql-php/src/Language/Lexer.php).

Got it, thanks that’s really helpful

Cool,
Reproduced, test altered, patch in MR against 2.2.x

Assuming this goes well, i'll release 2.2

MR passing tests
Merging to 2.2.x and creating backport patch to get y'all through til tomorrow and I can release not on a laptop in bed :D

2.1.1 patch at https://git.drupalcode.org/project/graphql_compose/-/merge_requests/89.diff

Lol thank you for working on it so quickly and at such a late hour. It's much appreciated. Have a good night.