[graphql-modules-preset] fix: skip sources without a location or document under strict mode (#4683)

.changeset/brave-modules-skip.md

@@ -0,0 +1,5 @@
+---
+'@graphql-codegen/graphql-modules-preset': patch
+---
+
+Skip sources without a `location` when grouping sources by module, instead of throwing from `path.normalize`, and skip sources without a `document` AST when building each module's document. Generated output from the CLI is unchanged.

packages/presets/graphql-modules/src/builder.ts

@@ -439,7 +439,7 @@ export function buildModule(
         defined.objects.push(name);
         collectFields(node, picks.objects);
 
-        if (node.interfaces?.length > 0) {
+        if ((node.interfaces?.length ?? 0) > 0) {
           withIsTypeOf.objects.push(name);
         }
 
@@ -495,7 +495,7 @@ export function buildModule(
 
         pushUnique(extended.objects, name);
 
-        if (node.interfaces?.length > 0) {
+        if ((node.interfaces?.length ?? 0) > 0) {
           pushUnique(withIsTypeOf.objects, name);
         }
 

packages/presets/graphql-modules/src/index.ts

@@ -10,7 +10,7 @@ export const preset: Types.OutputPreset<ModulesConfig> = {
   buildGeneratesSection: options => {
     const { baseOutputDir } = options;
     const { baseTypesPath, encapsulateModuleTypes } = options.presetConfig;
-    const useGraphQLModules = getConfigValue(options?.presetConfig.useGraphQLModules, true);
+    const useGraphQLModules = options?.presetConfig.useGraphQLModules ?? true;
     const requireRootResolvers = getConfigValue(options?.presetConfig.requireRootResolvers, false);
     const useTypeImports = getConfigValue(options?.config.useTypeImports, false) || false;
 
@@ -94,7 +94,9 @@ export const preset: Types.OutputPreset<ModulesConfig> = {
         normalize(join(relativePath, baseTypesFilename));
       const sources = sourcesByModuleMap[moduleName];
 
-      const moduleDocument = concatAST(sources.map(source => source.document));
+      const moduleDocument = concatAST(
+        sources.flatMap(source => (source.document ? [source.document] : [])),
+      );
 
       const shouldDeclare = filename.endsWith('.d.ts');
 
@@ -125,7 +127,7 @@ export const preset: Types.OutputPreset<ModulesConfig> = {
                   schema.getQueryType()?.name,
                   schema.getMutationType()?.name,
                   schema.getSubscriptionType()?.name,
-                ].filter(Boolean),
+                ].filter((name): name is string => !!name),
                 useTypeImports,
               }),
           },

packages/presets/graphql-modules/src/utils.ts

@@ -182,6 +182,10 @@ export function groupSourcesByModule(
   const grouped: Record<string, Source[]> = {};
 
   for (const source of sources) {
+    if (!source.location) {
+      continue;
+    }
+
     const relativePath = getRelativePath(source.location, basePath);
 
     if (relativePath) {