fix loading ESM graphql.config.js configs, pattern too long error and rename flat configs (#2468)

* aa

* aa

* f11x

* fix playground

Author: dimaMachina

.changeset/friendly-singers-rush.md

@@ -0,0 +1,22 @@
+---
+'@graphql-eslint/eslint-plugin': patch
+---
+
+- rename flat configs exports
+
+```diff
+-graphql.flatConfigs['schema-recommended']
++graphql.configs['flat/schema-recommended']
+-graphql.flatConfigs['schema-relay']
++graphql.configs['flat/schema-relay']
+-graphql.flatConfigs['schema-all']
++graphql.configs['flat/schema-all']
+-graphql.flatConfigs['operations-recommended']
++graphql.configs['flat/operations-recommended']
+-graphql.flatConfigs['operations-all']
++graphql.configs['flat/operations-all']
+```
+
+- fix with programmatic usage when passing large schema as string causes `pattern too long` error
+
+- fix loading ESM `graphql.config.js` configs

examples/code-file/eslint.config.js

@@ -12,12 +12,12 @@ export default [
   },
   {
     files: ['**/*.graphql'],
-    plugins: {
-      '@graphql-eslint': { rules: graphql.rules },
-    },
     languageOptions: {
       parser: graphql.parser,
     },
+    plugins: {
+      '@graphql-eslint': { rules: graphql.rules },
+    },
     rules: {
       '@graphql-eslint/no-anonymous-operations': 'error',
       '@graphql-eslint/naming-convention': [

examples/graphql-config/eslint.config.js

@@ -8,12 +8,12 @@ export default [
   },
   {
     files: ['**/*.graphql'],
-    plugins: {
-      '@graphql-eslint': { rules: graphql.rules },
-    },
     languageOptions: {
       parser: graphql.parser,
     },
+    plugins: {
+      '@graphql-eslint': { rules: graphql.rules },
+    },
     rules: {
       '@graphql-eslint/no-anonymous-operations': 'error',
       '@graphql-eslint/no-duplicate-fields': 'error',

examples/monorepo/eslint.config.js

@@ -23,21 +23,21 @@ export default [
   {
     // Setup GraphQL Parser
     files: ['**/*.{graphql,gql}'],
-    plugins: {
-      '@graphql-eslint': { rules: graphql.rules },
-    },
     languageOptions: {
       parser: graphql.parser,
     },
+    plugins: {
+      '@graphql-eslint': { rules: graphql.rules },
+    },
   },
   {
     // Setup recommended config for schema files
     files: [SCHEMA_PATH],
-    ...graphql.flatConfigs['schema-recommended'],
+    rules: graphql.configs['flat/schema-recommended'],
   },
   {
     // Setup recommended config for operations files
     files: ['client/**/*.{graphql,gql}'],
-    ...graphql.flatConfigs['operations-recommended'],
+    rules: graphql.configs['flat/operations-recommended'],
   },
 ];

examples/multiple-projects-graphql-config/eslint.config.js

@@ -10,22 +10,22 @@ export default [
   {
     // Setup GraphQL Parser
     files: ['**/*.graphql'],
-    plugins: {
-      '@graphql-eslint': { rules: graphql.rules },
-    },
     languageOptions: {
       parser: graphql.parser,
     },
+    plugins: {
+      '@graphql-eslint': { rules: graphql.rules },
+    },
   },
   {
     files: ['schema.*.graphql'],
     rules: {
-      ...graphql.flatConfigs['schema-recommended'].rules,
+      ...graphql.configs['flat/schema-recommended'],
       '@graphql-eslint/require-description': 'off',
     },
   },
   {
     files: ['**/*.js/*.graphql'],
-    rules: graphql.flatConfigs['operations-recommended'].rules,
+    rules: graphql.configs['flat/operations-recommended'],
   },
 ];

examples/programmatic/eslint.config.js

@@ -8,9 +8,6 @@ export default [
   },
   {
     files: ['**/*.graphql'],
-    plugins: {
-      '@graphql-eslint': { rules: graphql.rules },
-    },
     languageOptions: {
       parser: graphql.parser,
       parserOptions: {
@@ -20,6 +17,9 @@ export default [
         },
       },
     },
+    plugins: {
+      '@graphql-eslint': { rules: graphql.rules },
+    },
     rules: {
       '@graphql-eslint/require-selections': ['error', { fieldName: '_id' }],
       '@graphql-eslint/unique-fragment-name': 'error',

packages/plugin/package.json

@@ -49,7 +49,7 @@
     "@graphql-tools/utils": "^10.0.0",
     "debug": "^4.3.4",
     "fast-glob": "^3.2.12",
-    "graphql-config": "^5.0.0",
+    "graphql-config": "^5.1.0",
     "graphql-depth-limit": "^1.1.0",
     "lodash.lowercase": "^4.3.0"
   },

packages/plugin/src/cache.ts

@@ -8,11 +8,17 @@ export class ModuleCache<K, T> {
   map = new Map<K, { lastSeen: [number, number]; result: T }>();
 
   set(cacheKey: K, result: T): void {
+    // Remove server-side cache code in browser
+    if (typeof window !== 'undefined') return;
+
     this.map.set(cacheKey, { lastSeen: process.hrtime(), result });
     log('setting entry for', cacheKey);
   }
 
   get(cacheKey: K, settings = { lifetime: 10 /* seconds */ }): T | void {
+    // Remove server-side cache code in browser
+    if (typeof window !== 'undefined') return;
+
     const value = this.map.get(cacheKey);
     if (!value) {
       log('cache miss for', cacheKey);

packages/plugin/src/configs/index.ts

@@ -16,4 +16,15 @@ export const configs = {
   'schema-relay': relayConfig,
   'operations-recommended': operationsRecommendedConfig,
   'operations-all': operationsAllConfig,
-} satisfies Record<ConfigName, unknown>;
+  'flat/schema-recommended': schemaRecommendedConfig.rules,
+  'flat/schema-all': {
+    ...schemaRecommendedConfig.rules,
+    ...schemaAllConfig.rules,
+  },
+  'flat/schema-relay': relayConfig.rules,
+  'flat/operations-recommended': operationsRecommendedConfig.rules,
+  'flat/operations-all': {
+    ...operationsRecommendedConfig.rules,
+    ...operationsAllConfig.rules,
+  },
+} satisfies Record<ConfigName | `flat/${ConfigName}`, unknown>;