feat: reduce code output of client preset

[n1ru4l]

We don't need all these types for the GraphQL schema for client preset. I added an option to omit them within @graphql-codegen/visitor-plugin-common and @graphql-codegen/typescript and enabled this option by default for the client preset.

For GraphQL Hive this reduces the generated code for the client preset by over 50%. Also, see the diff and removed code here from the fixtures/examples. :D

We could even reduce generated enums and scalars to only those used within the provided graphql documents. However, let's keep that for another time.

[changeset-bot]

🦋 Changeset detected

Latest commit: bdd83b3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@graphql-codegen/client-preset	Minor

Not sure what this means? Click here to learn what changesets are.

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

[github-actions]

🚀 Snapshot Release (alpha)

The latest changes of this PR are available as alpha on npm (based on the declared changesets):

Package	Version	Info
@graphql-codegen/client-preset	4.4.0-alpha-20240729201742-bdd83b330efab9495a17e7cf0a8226c750fc1772	npm ↗︎ unpkg ↗︎

[github-actions]

💻 Website Preview

The latest changes are available as preview in: https://6b03ddc3.graphql-code-generator.pages.dev

[eddeee888]

Nice one!

[saihaj]

Released @graphql-codegen/client-preset@4.4.0

[jacquesg]

This is a massive breaking change, upgrading from 4.3.X to 4.4 the default should not have changed.

[jacquesg]

Even undoing the patch locally results in the generated file being broken:

diff --git a/codegen.ts b/codegen.ts
index 28d74543..5fcad7cf 100644
--- a/codegen.ts
+++ b/codegen.ts
@@ -6,7 +6,11 @@ const config: CodegenConfig = {
   generates: {
     './src/api/deianira/generated/': {
       preset: 'client',
-      plugins: [],
+      plugins: [
+        {
+          [`typescript`]: {},
+        },
+      ],

src/api/deianira/generated/graphql.ts:5950:13 - error TS2300: Duplicate identifier 'UpdateZoneInput'.

5950 export type UpdateZoneInput = {
                 ~~~~~~~~~~~~~~~


Found 1148 errors in the same file, starting at: src/api/deianira/generated/graphql.ts:13

[asgeo1]

Breaking my app also because I was actually using those types! @n1ru4l do you know how we can get the original behaviour back?

I can see onlyEnumTypes: true, onlyOperationTypes:true were added in the PR, but I don't get how we can revert them back to their original values in a graphql-codgen.ts config file.

[piotr-czerwik]

Same for the app I am working on - we relied on these types.

I've tried adding

onlyEnumTypes: false,
onlyOperationTypes: false,

to the configuration hoping it will 'revert' the current defaults to the previous ones, but it seems not having any effect...

[bmulholland]

Filed as #10167

[nebbles]

PR #10155 adds onlyEnumTypes and onlyOperationTypes to the overridable config.