fix: SWC plugin #9489

YassinEldeeb · 2023-06-06T05:02:29Z

Fix reproductions and close issues

changeset-bot · 2023-06-06T05:02:32Z

⚠️ No Changeset found

Latest commit: cba43e0

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

YassinEldeeb · 2023-06-06T05:03:07Z

oops, based off of the wrong branch

CaptainN · 2023-06-21T15:34:33Z

Possible to get this merged? The swc plugin doesn't work at all right now.

focux · 2023-07-03T19:20:11Z

Is there any progress on this?

saihaj · 2023-07-07T09:20:34Z

@YassinEldeeb let's rebase this one now?

CaptainN · 2023-07-25T18:17:38Z

I recreated this on top of master, and updated to swc_core 0.79.x but was unable to produce a binary that didn't panic.

info Creating an optimized production build .thread 'thread '<unnamed><unnamed>' panicked at '' panicked at 'failed to invoke plugin: failed to invoke plugin on Some(... every file)

I don't know enough about swc or rust to go much further.

MatteoGiacomo · 2023-08-04T07:26:16Z

Ehy guys, do you have any updates on this? @YassinEldeeb @saihaj, thanks

YassinEldeeb · 2023-08-10T04:32:54Z

SWC Plugin Concerns & Clarifications:

A while ago, I discussed with the team the SWC plugin and the challenges with its integration. Here's an overview of our findings:

1. SWC Core Versioning and Compatibility:

On mid-may, I discovered that SWC provided documentation on selecting the correct SWC Core version. Essentially, each minor version supports a range of nextjs versions, specifically @swc/core - the Next.js compiler.
An important note: Wasm plugins aren't backward compatible. Hence, selecting the correct swc_core version for your plugin is crucial.

From SWC Documentation: "Currently, the Wasm plugins are not backwards compatible. So you need to select an appropriate version of swc_core for your plugin."

LINK to the documentation

2. My Initial Approach:

I explored creating an installation CLI. It would read the nextjs version in a project’s package.json and determine the right SWC plugin version to use.
When publishing from our end, the SWC plugin would provide versions like:
- @graphql-codegen/swc-plugin@v0.78.x
- @graphql-codegen/swc-plugin@v0.76.x
- @graphql-codegen/swc-plugin@v0.75.x

And the CLI would decide on the installation version based on the read nextjs version from the project. And, of course, upgrading your nextjs version would likely break the SWC plugin silently.

3. Renovate Configuration Issues:

Our existing renovate config, which issues PRs to regularly update the swc_core, seems obsolete. It tends to favor the latest or even canary versions of Next.js, and that's what introduced the breaking changes from the beginning until now with the SWC plugin not working unless with a specific range of Next.js versions which is specified in their documentation.
A potential solution could be to have it auto-publish the plugin with the upcoming SWC core version. However, users must be aware of the need to pin the swc_core version. If they update their Next.js version, the SWC plugin also needs an update. The error messages, as you've all seen, might not be straightforward.

4. Future Challenges:

Keeping up with the frequent releases of SWC Core and Next.js versions is challenging. One idea might be to automate some build and publishing steps based on scraping their documentation, but there's no guarantee it'll always be updated promptly.

5. Development Concerns for SWC Plugin:

Currently, swc_core is developed on the Rust nightly version, and not the stable toolchain. We've had to adapt to that environment.

Hoping this sheds some light on the SWC plugin challenges. Feedback and suggestions are welcome!
And I apologize for the long wait on this.

We've been busy preparing for the first GraphQL Conf 2023 held in San Francisco, and we plan to connect with SWC maintainers and figure out a solution together once we get back. I appreciate your patience! 🙏

6. Okay, but what for now?

As for now, it appears to be that SWC plugins are meant to be built and used at an individual level, so you can:

Clone the SWC plugin https://github.com/dotansimha/graphql-code-generator/tree/master/packages/presets/swc-plugin
Change swc_core version in Cargo.toml matching your next.js version, please look up how to find your suitable swc_core version here https://swc.rs/docs/plugin/selecting-swc-core
Follow the toolchain installation guide from here https://swc.rs/docs/plugin/ecmascript/getting-started#install-required-toolchain
Run npm run build-wasm to compile it
This would result in a swc_plugin.wasm file which is linked to the SWC plugin package.json via the "main": "swc_plugin.wasm"
Use your local package within your project via npm link or workspaces implementation of the various package managers (npm, yarn, or pnpm)
And, it works! 🎉

victorandree · 2023-08-10T08:11:32Z

@YassinEldeeb I appreciate the update! I tried to follow your steps on how to update this on my own – but it fails in different ways (basically confirming what @CaptainN commented above).

Leaving this comment so maybe the community can figure this out.

If I specify swc_core@0.79.*, which should work with next@v13.4.10-canary.1 ~ (according to https://swc.rs/docs/plugin/selecting-swc-core#v079x), I can build the plugin. However, if I try to then use this plugin with next@13.4.13 (latest, as of today) or next@13.4.10-canary.1, I get errors about "out of bounds memory access" (full stack trace below).
Note: If I just use the current plugin with next@13.4.13, the error is instead RuntimeError: unreachable.
If I simply try to build the current version of the swc-plugin (which specifies swc_core@0.75.*), I get a build error from proc-macro2 about "unknown feature proc_macro_span_shrink". I'm guessing this is because we have to build this with the Rust nightly toolchain, as mentioned in the comment by @YassinEldeeb, and something has changed here.

Stack trace from swc_core@0.79.* with next@13.4.13. Node version is 18.17.1 (latest LTS).

Error: failed to process failed to invoke plugin: failed to invoke plugin on 'Some("/c/next-app/pages/index.tsx")'

Caused by:
    0: failed to invoke `/c/next-app/node_modules/@graphql-codegen/client-preset-swc-plugin/swc_plugin.wasm` as js transform plugin at /c/next-app/node_modules/@graphql-codegen/client-preset-swc-plugin/swc_plugin.wasm
    1: RuntimeError: out of bounds memory access

Stack backtrace:
   0: _napi_register_module_v1
   1: _napi_register_module_v1
   2: _BrotliDecoderVersion
   3: _BrotliDecoderVersion
   4: _BrotliDecoderVersion
   5: _BrotliDecoderVersion
   6: _BrotliDecoderVersion
   7: <unknown>
   8: <unknown>
   9: <unknown>
  10: <unknown>
  11: <unknown>
  12: <unknown>
  13: <unknown>
  14: <unknown>
  15: <unknown>
  16: <unknown>
  17: <unknown>

Stack backtrace:
   0: _napi_register_module_v1
   1: _napi_register_module_v1
   2: <unknown>
   3: <unknown>

Import trace for requested module:
./pages/index.tsx

victorandree · 2023-08-11T09:52:25Z

Hopefully relevant: The build failure is because of swc-project/swc#7479. Setting lto = false for the release profile allows me to build and use a new version of the plugin 🎉.

I've created a fork and published an npm package that changes lto = false, updates swc_core, and fixes #9445. Looking to get these updates merged upstream - but others might find it useful. Absolutely no support will be provided from me 🙃.

Update Sep 28: I've updated my fork and library to work with Next.js 13.5.3. Approach is basically to update swc_core to the latest version. Note that I rebased my master branch on upstream, which broke previous commits.

JamesCodes · 2023-08-18T07:27:43Z

Hopefully relevant: The build failure is because of swc-project/swc#7479. Setting lto = false for the release profile allows me to build and use a new version of the plugin 🎉.

Thank you! This saved me.

YassinEldeeb · 2023-08-18T12:44:44Z

That's absolutely amazing, @victorandree!! Sorry, I was late getting back to you,
I'm super glad you were able to figure out the issue! ❤️

bvobart · 2023-09-11T12:44:36Z

Hi @YassinEldeeb and @victorandree, thank you both for this MR and your patched NPM package!

This has fixed the original syntax error that I was getting in #9337. Without any further updates to the repo, I was getting an out of bounds memory access error like in #9489 (comment), but that was fixed by upgrading @swc/core to 1.3.80 as prescribed by the SWC docs.

However, the issue isn't entirely fixed yet. As soon as I import code from another TypeScript file, I get a different syntax error, even though I've updated my example repo for #9337 to use the patched SWC plugin and I've updated all dependencies to versions that should be compatible (in particular @swc/core).

Could you look into this again please? 😊

victorandree · 2023-11-16T11:33:19Z

FWIW, I recently noticed that my Next.js builds started ballooning in size, so I wanted to have another look at this issue (suspecting that maybe the plugin broke again). While I haven't had any issues using the plugin with Next.js 14.0.2, I figured it's good to be up-to-date.

TL;DR: New version of my fork released as @victorandree/graphql-codegen-client-preset-swc-plugin@0.3.0, using swc_core@0.86. Updating swc_core required me to change test code; see my fork at https://github.com/victorandree/graphql-code-generator. Unfortunately, did not help with bundle size.

@YassinEldeeb Now that GraphQL Conf 2023 is over, have you had the chance to talk to the SWC maintainers about a way forward?
@bvobart Unfortunately, I'm not familiar with using @swc/jest to transform files for jest. I tried running your reproduction repository, and for me it fails even if I remove the plugin. At a glance, it seems related to your TypeScript code not being compiled - because the syntax error is related to a: number in Foo.ts.

IMHO, the concern about matching swc_core and Next.js versions is valid, but we're also all consenting adults here, who are using the known unstable swcPlugins feature and are seeing "Experiments (use at your own risk): swcPlugins" on every build with a big warning about it in the docs.

For now, I think it'd be worthwhile to release a new version of the plugin with an updated swc_core, considering it's been broken at a patch version of Next.js for some time now.

YassinEldeeb · 2023-11-20T06:09:34Z

Hey, really sorry for the very late reply! We are currently tracking the issue with SWC maintainers to figure out a solution.

Please add a 👍 to the issue and feel free to join the conversation.
swc-project/swc#8315

I'll keep you updated!

YassinEldeeb · 2023-11-20T06:15:46Z

@victorandree @bvobart @MatteoGiacomo @CaptainN @focux as you can see, unfortunately, they replied simply with In short, this is impossible, please refer to the above issue's comment from the maintainers.

victorandree · 2023-11-22T12:38:27Z

@YassinEldeeb I appreciate trying to sync with the SWC maintainers on this, and I also appreciate the challenges of trying to keep things in sync between SWC/Next.js versions (and that Next.js isn't the only game in town).

That said, would you consider releasing new versions of the plugin with documentation to clarify compatibility, or is this effectively unmaintained until SWC stabilizes?

YassinEldeeb · 2023-11-23T16:32:17Z

@victorandree I really appreciate your enthusiasm and engagement in this issue!

would you consider releasing new versions of the plugin with documentation to clarify compatibility

Unfortunately, SWC maintainers weren't able to promise us with up to date documentation on the compatibility between next.js versions and rust's swc_core crate (package), which is the main concern causing all of the issues we're all currently facing.

is this effectively unmaintained until SWC stabilizes?

It's difficult, but I wouldn't say it's completely unmaintained, it's just about chasing up to date compatibility information from SWC, they have effectively claimed plugins to be a "Beta Feature", to avoid people complaining about them releasing breaking changes without notices. So it's tricky.

We are thinking of moving the SWC plugin to the GraphQL Codegen Community repo, where the community can file issues of breaking changes as they detect it, lookup https://swc.rs/docs/plugin/selecting-swc-core for updated documentation, if not, ask SWC maintainer to update it by filing an issue, then finally releasing a new version of the SWC plugin, so there would be:

@graphql-codegen/swc-plugin@v0.78.x
@graphql-codegen/swc-plugin@v0.76.x
@graphql-codegen/swc-plugin@v0.75.x

Where each new version doesn't override the previous one, but adds support for newer next.js versions.

So, what do you think @victorandree? Would it be feasible to let the community (including most people in the current thread) take care of maintaining it given the challenges mentioned?

We think if the community can quickly detect breaking changes, and one person can fix it and release it for everyone else, then it can work.

Curious to hear your thoughts! 👀

victorandree · 2023-11-24T08:52:35Z

@YassinEldeeb

We are thinking of moving the SWC plugin to the GraphQL Codegen Community repo, where the community can file issues of breaking changes as they detect it, lookup https://swc.rs/docs/plugin/selecting-swc-core for updated documentation, if not, ask SWC maintainer to update it by filing an issue, then finally releasing a new version of the SWC plugin, so there would be:

@graphql-codegen/swc-plugin@v0.78.x

@graphql-codegen/swc-plugin@v0.76.x

@graphql-codegen/swc-plugin@v0.75.x

Where each new version doesn't override the previous one, but adds support for newer next.js versions.

So, what do you think @victorandree? Would it be feasible to let the community (including most people in the current thread) take care of maintaining it given the challenges mentioned?

I definitely hope that the community can help with maintaining this, although I'm not sure what difference it makes if this is part of dotansimha/graphql-code-generator-community or dotansimha/graphql-code-generator. The contribution guidelines seem identical for both. Maybe I'm just not up-to-date with your governance structure :)
I'm not super convinced on tracking SWC core versions with version identifiers in this plugin. What if you want to fix a bug or add a feature (for example, @graphql-codegen/client-preset-swc-plugin adds imports before "use client" #9445)? If you do want to maintain support for different SWC core versions, I think it'd be better and just as much of a mess to build and include multiple versions of swc_plugin.wasm in each release of the plugin (so you'd have e.g. swc_plugin-0.78.wasm, swc_plugin-0.76.wasm for however many versions you'd decide to support).
This would move version select to where you configure SWC plugins, instead of package.json. Next.js allows you to specify "an npm module package name or an absolute path to the .wasm binary itself" (source: https://nextjs.org/docs/architecture/nextjs-compiler#swc-plugins-experimental), so I'm admittedly not sure if this is ergonomic at all.
My personal opinion is that you don't need to support anything but the latest version of SWC core. This is an experimental feature, so I don't think anyone can expect backwards compatibility.

WonderPanda · 2023-12-04T15:08:58Z

Just discovered this thread after trying to update a project to use all the latest and greatest with the client preset. I'm very excited about the direction things are going, especially with the focus on composing fragments up the tree!

I'm not using NextJS, just a react app using Vite in an NX monorepo but I'm running into issues with the plugin. Is there a list of known compatible versions of the plugin + swc core version that I could try and pin to?

CaptainN · 2023-12-04T16:04:02Z

This is an experimental feature, so I don't think anyone can expect backwards compatibility.

When you use the generator, it outputs this at the top of your file:


/**
 * Map of all GraphQL operations in the project.
 *
 * This map has several performance disadvantages:
 * 1. It is not tree-shakeable, so it will include all operations in the project.
 * 2. It is not minifiable, so the string of a GraphQL query will be multiple times inside the bundle.
 * 3. It does not support dead code elimination, so it will add unused operations.
 *
 * Therefore it is highly recommended to use the babel or swc plugin for production.
 */

That makes it sound more necessary than experimental.

victorandree · 2023-12-05T12:39:42Z

@CaptainN To be fair, there should also be a (stable) Babel Plugin that achieves the same results, see https://the-guild.dev/graphql/codegen/plugins/presets/preset-client#reducing-bundle-size. This is about the SWC plugin specifically.

CaptainN · 2024-02-06T21:07:24Z

@victorandree That's true, but it's pretty challenging to get that to work with nextjs, which uses SWC by default. (It has a way to change back to babel, but it breaks a bunch of stuff - not easy to pull off.)

CaptainN · 2024-06-05T19:40:54Z

How about if we just don't generate the giant file to start with - it's relatively simple to manually edit it so that you don't get that: #9988

victorandree · 2024-11-06T09:02:05Z

For anyone still following this thread:

I've released a new version of my forked plugin (https://www.npmjs.com/package/@victorandree/graphql-codegen-client-preset-swc-plugin/v/0.5.0) that uses swc_core v3.0.2 which makes it compatible with Next.js 15.
I've tested that without the plugin, the documents map is present in gql.ts, and that activating it removes it. That is, the plugin is still relevant and still works.
swc_core compatibility information is now available at: https://plugins.swc.rs.

Please note that I just make the bare minimum changes to make the plugin "work" for me. I don't know how newer versions work with the issues mentioned at the top of the issue, or with non-Next.js runtimes, or how to resolve the "version selection" problem.

I don't know if swc_core being on a non-0.x version implies anything about compatibility.

karlhorky · 2025-05-14T22:20:06Z

@YassinEldeeb if I understand correctly, there's no actionable way forward here?

Should the @graphql-codegen/client-preset-swc-plugin package be deprecated?

cc @dotansimha @enisdenjo

beerose and others added 19 commits May 16, 2023 18:30

Add gpt docs support

1609120

Add github action

19be617

Move to mendable

5e34374

Finish Mendable integration

8cc1573

remove script

960525e

Merge branch 'master' into gpt

61a95c3

Update website/package.json

bca456c

Add env var

6601862

Merge branch 'gpt' of https://github.com/dotansimha/graphql-code-gene…

30b53f0

…rator into gpt

Upgrade and customize mendable search

3c9cba6

Make bar smaller

dbc2e61

Merge branch 'master' into gpt

57a757f

Update search bar

9577bcd

downgrade mendable

7c9efdb

Upgrade again

f3d57d0

Remove not needed css files

5308324

Update shortcut key

3c5d873

Merge branch 'master' into gpt

52a43e8

fix swc plugin

cba43e0

CaptainN mentioned this pull request Jul 25, 2023

fix: swc plugin #9567

Open

osdiab mentioned this pull request Sep 11, 2023

@graphql-codegen/client-preset-swc-plugin ... #9450

Open

bvobart mentioned this pull request Sep 11, 2023

Enabling @graphql-codegen/client-preset-swc-plugin makes Jest throw SyntaxError when trying to run unit tests #9337

Open

fix: SWC plugin #9489

Are you sure you want to change the base?

fix: SWC plugin #9489

Uh oh!

Conversation

YassinEldeeb commented Jun 6, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Fix reproductions and close issues

Uh oh!

changeset-bot bot commented Jun 6, 2023

⚠️ No Changeset found

Uh oh!

YassinEldeeb commented Jun 6, 2023

Uh oh!

CaptainN commented Jun 21, 2023

Uh oh!

focux commented Jul 3, 2023

Uh oh!

saihaj commented Jul 7, 2023

Uh oh!

CaptainN commented Jul 25, 2023

Uh oh!

MatteoGiacomo commented Aug 4, 2023

Uh oh!

YassinEldeeb commented Aug 10, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

SWC Plugin Concerns & Clarifications:

1. SWC Core Versioning and Compatibility:

2. My Initial Approach:

3. Renovate Configuration Issues:

4. Future Challenges:

5. Development Concerns for SWC Plugin:

6. Okay, but what for now?

Uh oh!

victorandree commented Aug 10, 2023

Uh oh!

victorandree commented Aug 11, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

JamesCodes commented Aug 18, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

YassinEldeeb commented Aug 18, 2023

Uh oh!

bvobart commented Sep 11, 2023

Uh oh!

victorandree commented Nov 16, 2023

Uh oh!

YassinEldeeb commented Nov 20, 2023

Uh oh!

YassinEldeeb commented Nov 20, 2023

Uh oh!

victorandree commented Nov 22, 2023

Uh oh!

YassinEldeeb commented Nov 23, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

victorandree commented Nov 24, 2023

Uh oh!

WonderPanda commented Dec 4, 2023

Uh oh!

CaptainN commented Dec 4, 2023

Uh oh!

victorandree commented Dec 5, 2023

Uh oh!

CaptainN commented Feb 6, 2024

Uh oh!

CaptainN commented Jun 5, 2024

Uh oh!

victorandree commented Nov 6, 2024

Uh oh!

karlhorky commented May 14, 2025

Uh oh!

Uh oh!

YassinEldeeb commented Jun 6, 2023 •

edited

Loading

YassinEldeeb commented Aug 10, 2023 •

edited

Loading

victorandree commented Aug 11, 2023 •

edited

Loading

JamesCodes commented Aug 18, 2023 •

edited

Loading

YassinEldeeb commented Nov 23, 2023 •

edited

Loading