feat: allow ampersand in file names to load (#4603)

.changeset/tender-forks-yawn.md

@@ -0,0 +1,8 @@
+---
+'@graphql-tools/graphql-file-loader': minor
+'@graphql-tools/utils': minor
+---
+
+Allow `&` in filenames.
+
+Related to https://github.com/dotansimha/graphql-code-generator/issues/6174

packages/loaders/graphql-file/tests/loader.spec.ts

@@ -19,6 +19,7 @@ describe('GraphQLFileLoader', () => {
     })(canLoad => {
       it('should return true for a valid pointer', async () => {
         await expect(canLoad(getPointer('type-defs.graphql'), {})).resolves.toBe(true);
+        await expect(canLoad(getPointer('typedefs-&-copy.graphql'), {})).resolves.toBe(true);
       });
 
       it('should return true for a valid absolute path', async () => {

packages/loaders/graphql-file/tests/test-files/typedefs-&-copy.graphql

@@ -0,0 +1,7 @@
+#import AFragment from "./fragment.graphql"
+
+query MyQuery {
+  a {
+    ...AFragment
+  }
+}

packages/utils/src/helpers.ts

@@ -25,7 +25,7 @@ export function isDocumentString(str: any): boolean {
   return false;
 }
 
-const invalidPathRegex = /[‘“!%&^<=>`]/;
+const invalidPathRegex = /[‘“!%^<=>`]/;
 export function isValidPath(str: any): boolean {
   return typeof str === 'string' && !invalidPathRegex.test(str);
 }