-
Notifications
You must be signed in to change notification settings - Fork 57
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make fast-json-stringify optional #83
Comments
That would be ok for me! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Thanks for the great project.
Is your feature request related to a problem? Please describe.
I'm experimenting with using
graphql-jit
in the browser. Currentlygraphql-jit
is bundled withfast-json-stringify
, which makes its size extremely big to be used in browser.https://bundlephobia.com/result?p=graphql-jit
Removing
fast-json-stringify
would cut the size in half.Describe the solution you'd like
Currently, we have the option
compilerOptions.customJSONSerializer
, which accepts a boolean value. However the option does not provide much use because it only switches betweenJSON.stringify
andfastJSON
, not really 'custom'.My proposal is that we replace it with a user-supplied function that accepts CompilerContext and returns the stringify function.
The text was updated successfully, but these errors were encountered: