Error after upgrading to graphql 16.7.0: "Uncaught SyntaxError: Unexpected string"

[oidualc]

Prerequisites

• I confirm my issue is not in the opened issues
• I confirm the Frequently Asked Questions didn't contain the answer to my issue

Environment check

• I'm using the latest msw version
• I'm using Node.js version 14 or higher

Browsers

Chromium (Chrome, Brave, etc.)

Reproduction repository

https://github.com/oidualc/msw-graphql-issue

Reproduction steps

• Download reproduction repository
• run pnpm install
• run pnpm run dev
• open http://localhost:5173

Current behavior

Blank page and error on console Uncaught SyntaxError: Unexpected string.
This happens since upgrading graphql to 16.7.0, does not happen forcing the version to 16.6.0.

Expected behavior

Page rendered correctly, no errors on console.

[SwagMuffinMcYoloPants]

I get the same issue too. When i roll back to msw 0.40.2, it seems to work again.

[oidualc]

This seems to be caused by graphql/graphql-js#3887, and it's been reported multiple times on the graphql repo as it's causing problems for projects other than msw. For example:

• Regression: Importing the package fails in a brand new Vite project graphql/graphql-js#3918
• Syntax error in Nuxt project due to graphql-js pull request #3887 graphql/graphql-js#3919
• graphql-js v16.7.0 breaks the compilation of SvelteKit graphql/graphql-js#3920
• GraphQL 16.7.0 breaks Vue3/Vite3 graphql/graphql-js#3921

[SwagMuffinMcYoloPants]

@oidualc I'm using REST and it's exhibiting the same problem. I can make a repo if you want.

[thepassle]

Repo wont be needed, but thank you for offering :) The issue has already been identified, waiting for action on the side of the graphql maintainers.

[SwagMuffinMcYoloPants]

ah, but the issue I'm seeing the issue with REST and would it be caused by the same issue? Or is it something completely different?

[oidualc]

Closing the issue as it was resolved upstream with graphql/graphql-js#3923. For future reference for anyone who will encounter this bug, just make sure to upgrade the graphql sub-dependency at least to https://github.com/graphql/graphql-js/releases/tag/v16.7.1.

[EhabSalah]

This could be an error introduced with version 16.7.0 of the graphql package. Please downgrade for 16.6.0 for now, we are looking into this.

[kristojorg]

@oidualc , is a newer version of graphql installed in msw? I just tried on latest and was still seeing this error. I don't believe npm provides a way (like yarn resolutions) to upgrade a sub-dependency. I tried installing latest graphql and this did not resolve the problem

[integratorivan]

@kristojorg hello! I have updated vite to the latest version (3.0.7 -> 4.4.7) and am updating msw - and the problem is solved

check this issue #1655

[kristojorg]

Thank you @integratorivan ! I'll give it a try : )