Revert "feat: attach .js extension to all relative/absolute file imports/exports" (#8068)

* Revert "feat: attach .js extension to all relative/absolute file imports/exports (#8034)"

This reverts commit 68bb30e.

* chore: add changeset

Author: n1ru4l

.changeset/eleven-buckets-explode.md

@@ -0,0 +1,11 @@
+---
+'@graphql-codegen/visitor-plugin-common': patch
+'@graphql-codegen/gql-tag-operations': patch
+'@graphql-codegen/typescript-oclif': patch
+'@graphql-codegen/typescript-resolvers': patch
+'@graphql-codegen/gql-tag-operations-preset': patch
+'@graphql-codegen/graphql-modules-preset': patch
+'@graphql-codegen/near-operation-file-preset': patch
+---
+
+Revert breaking change for Next.js applications that are incapable of resolving an import with a `.js` extension.

dev-test/githunt/__generated__/comment.query.stencil-component.tsx

@@ -1,2 +1,2 @@
-export * from './gql.js';
-export * from './fragment-masking.js';
+export * from './gql';
+export * from './fragment-masking';

dev-test/githunt/__generated__/feed-entry.fragment.stencil-component.tsx

@@ -1,2 +1,2 @@
-export * from './gql.js';
-export * from './fragment-masking.js';
+export * from './gql';
+export * from './fragment-masking';

dev-test/githunt/__generated__/feed.query.stencil-component.tsx

@@ -4,13 +4,13 @@ import { TypedDocumentNode as DocumentNode } from '@graphql-typed-document-node/
 declare module '@urql/core' {
   export function gql(
     source: '\n  query Foo {\n    Tweets {\n      id\n    }\n  }\n'
-  ): typeof import('./graphql.js').FooDocument;
+  ): typeof import('./graphql').FooDocument;
   export function gql(
     source: '\n  fragment Lel on Tweet {\n    id\n    body\n  }\n'
-  ): typeof import('./graphql.js').LelFragmentDoc;
+  ): typeof import('./graphql').LelFragmentDoc;
   export function gql(
     source: '\n  query Bar {\n    Tweets {\n      ...Lel\n    }\n  }\n'
-  ): typeof import('./graphql.js').BarDocument;
+  ): typeof import('./graphql').BarDocument;
   export function gql(source: string): unknown;
 
   export type DocumentType<TDocumentNode extends DocumentNode<any, any>> = TDocumentNode extends DocumentNode<

dev-test/githunt/__generated__/submit-comment.mutation.stencil-component.tsx

@@ -1 +1 @@
-export * from './gql.js';
+export * from './gql';

dev-test/gql-tag-operations-masking-star-wars/gql/index.ts

@@ -58,10 +58,7 @@ export function generateImportStatement(statement: ImportDeclaration): string {
       : '*';
   const importAlias = importSource.namespace ? ` as ${importSource.namespace}` : '';
   const importStatement = typesImport ? 'import type' : 'import';
-  const importExtension = importPath.startsWith('/') || importPath.startsWith('.') ? '.js' : '';
-  return `${importStatement} ${importNames}${importAlias} from '${importPath}${importExtension}';${
-    importAlias ? '\n' : ''
-  }`;
+  return `${importStatement} ${importNames}${importAlias} from '${importPath}';${importAlias ? '\n' : ''}`;
 }
 
 function resolveImportPath(baseDir: string, outputPath: string, sourcePath: string) {

dev-test/gql-tag-operations-masking/gql/index.ts

@@ -94,7 +94,7 @@ function getGqlOverloadChunk(sourcesWithOperations: Array<SourceWithOperations>,
     const returnType =
       mode === 'lookup'
         ? `(typeof documents)[${JSON.stringify(originalString)}]`
-        : `typeof import('./graphql.js').${operations[0].initialName}`;
+        : `typeof import('./graphql').${operations[0].initialName}`;
     lines.add(`export function gql(source: ${JSON.stringify(originalString)}): ${returnType};\n`);
   }
 

dev-test/gql-tag-operations-urql/gql/gql.d.ts

@@ -31,7 +31,7 @@ export class GraphQLRequestVisitor extends ClientSideBaseVisitor<Config, ClientS
     super(schema, fragments, rawConfig, {});
     this._info = info;
 
-    const { handlerPath = '../../handler.js' } = rawConfig;
+    const { handlerPath = '../../handler' } = rawConfig;
 
     // FIXME: This is taken in part from
     //  presets/near-operation-file/src/index.ts:139. How do I build a path relative to the outputFile in the same way?

dev-test/gql-tag-operations/gql/index.ts

@@ -706,22 +706,6 @@ describe('ResolversTypes', () => {
     spy.mockRestore();
   });
 
-  it('Should generate basic type resolvers with external mappers using a .js extension', async () => {
-    const result = (await plugin(
-      schema,
-      [],
-      {
-        noSchemaStitching: true,
-        mappers: {
-          MyOtherType: './my-file.js#MyCustomOtherType',
-        },
-      },
-      { outputFile: '' }
-    )) as Types.ComplexPluginOutput;
-
-    expect(result.prepend).toContain(`import { MyCustomOtherType } from './my-file.js';`);
-  });
-
   it('Should generate basic type resolvers with external mappers', async () => {
     const result = (await plugin(
       schema,

dev-test/star-wars/__generated__/CreateReviewForEpisode.tsx

@@ -160,7 +160,7 @@ export const preset: Types.OutputPreset<GqlTagConfig> = {
         },
         plugins: [
           {
-            [`add`]: { content: reexports.map(moduleName => `export * from "./${moduleName}.js"`).join('\n') },
+            [`add`]: { content: reexports.map(moduleName => `export * from "./${moduleName}"`).join('\n') },
           },
         ],
         schema: options.schema,

dev-test/star-wars/__generated__/HeroAndFriendsNames.tsx

@@ -30,7 +30,7 @@ describe('gql-tag-operations-preset', () => {
     expect(result).toHaveLength(3);
     // index.ts (re-exports)
     const indexFile = result.find(file => file.filename === 'out1/index.ts');
-    expect(indexFile.content).toEqual('export * from "./gql.js"');
+    expect(indexFile.content).toEqual('export * from "./gql"');
 
     // gql.ts
     const gqlFile = result.find(file => file.filename === 'out1/gql.ts');
@@ -85,7 +85,7 @@ describe('gql-tag-operations-preset', () => {
     expect(result).toHaveLength(3);
     // index.ts (re-exports)
     const indexFile = result.find(file => file.filename === 'out1/index.ts');
-    expect(indexFile.content).toEqual('export * from "./gql.js"');
+    expect(indexFile.content).toEqual('export * from "./gql"');
 
     // gql.ts
     const gqlFile = result.find(file => file.filename === 'out1/gql.ts');
@@ -367,9 +367,9 @@ describe('gql-tag-operations-preset', () => {
 
       const gqlFile = result.find(file => file.filename === 'out1/index.ts');
       expect(gqlFile.content).toMatchInlineSnapshot(`
-        "export * from \\"./gql.js\\"
-        export * from \\"./fragment-masking.js\\""
-      `);
+              "export * from \\"./gql\\"
+              export * from \\"./fragment-masking\\""
+            `);
       const fragmentMaskingFile = result.find(file => file.filename === 'out1/fragment-masking.ts');
       expect(fragmentMaskingFile.content).toMatchInlineSnapshot(`
         "import { TypedDocumentNode as DocumentNode } from '@graphql-typed-document-node/core';

dev-test/star-wars/__generated__/HeroAppearsIn.tsx

@@ -159,22 +159,6 @@ test('should include import statement', () => {
   `);
 });
 
-test('should include esm import statement', () => {
-  const output = buildModule('test', testDoc, {
-    importPath: '../types.js',
-    importNamespace: 'core',
-    encapsulate: 'none',
-    shouldDeclare: false,
-    rootTypes: ROOT_TYPES,
-    baseVisitor,
-    useGraphQLModules: true,
-  });
-
-  expect(output).toBeSimilarStringTo(`
-    import * as core from "../types.js";
-  `);
-});
-
 test('should work with naming conventions', () => {
   const output = buildModule('test', parse(`type query_root { test: ID! } schema { query: query_root }`), {
     importPath: '../types',