Publish as ESM only

Author: JoviDeCroock

eslint.config.mjs

@@ -764,7 +764,7 @@ export default tsConfig(
       'n/no-missing-import': [
         'error',
         {
-          allowModules: ['graphql', 'graphql-esm'],
+          allowModules: ['graphql'],
         },
       ],
       'n/no-missing-require': [

integrationTests/node/index.cjs renamed to integrationTests/node/index.js

@@ -1,15 +1,9 @@
-'use strict';
-
-const assert = require('node:assert');
-const { readFileSync } = require('node:fs');
-
-const {
-  experimentalExecuteIncrementally,
-  graphqlSync,
-  parse,
-} = require('graphql');
-const { buildSchema } = require('graphql/utilities');
-const { version } = require('graphql/version');
+import assert from 'node:assert';
+import { readFileSync } from 'node:fs';
+
+import { experimentalExecuteIncrementally, graphqlSync, parse } from 'graphql';
+import { buildSchema } from 'graphql/utilities';
+import { version } from 'graphql/version';
 
 assert.deepStrictEqual(
   version,

integrationTests/node/index.mjs

@@ -6,7 +6,6 @@
     "test": "node test.js"
   },
   "dependencies": {
-    "graphql": "file:../graphql.tgz",
-    "graphql-esm": "file:../graphql-esm.tgz"
+    "graphql": "file:../graphql.tgz"
   }
 }

integrationTests/node/package.json

@@ -15,12 +15,7 @@ for (const version of nodeVersions) {
   console.log(`Testing on node@${version} ...`);
 
   childProcess.execSync(
-    `docker run --rm --volume "$PWD":/usr/src/app -w /usr/src/app node:${version}-slim node ./index.cjs`,
-    { stdio: 'inherit' },
-  );
-
-  childProcess.execSync(
-    `docker run --rm --volume "$PWD":/usr/src/app -w /usr/src/app node:${version}-slim node ./index.mjs`,
+    `docker run --rm --volume "$PWD":/usr/src/app -w /usr/src/app node:${version}-slim node ./index.js`,
     { stdio: 'inherit' },
   );
 }

integrationTests/node/test.js

@@ -7,7 +7,6 @@
   },
   "dependencies": {
     "graphql": "file:../graphql.tgz",
-    "graphql-esm": "file:../graphql-esm.tgz",
     "typescript-4.9": "npm:[email protected]",
     "typescript-5.0": "npm:[email protected]",
     "typescript-5.1": "npm:[email protected]",

integrationTests/ts/esm.ts

@@ -7,7 +7,6 @@
   },
   "dependencies": {
     "graphql": "file:../graphql.tgz",
-    "graphql-esm": "file:../graphql-esm.tgz",
     "webpack": "5.x.x",
     "webpack-cli": "4.x.x"
   }

integrationTests/ts/package.json

@@ -1,18 +1,10 @@
 import assert from 'node:assert';
 
 /* eslint-disable n/no-missing-import */
-import cjs from './dist/main-cjs.cjs';
-import mjs from './dist/main-mjs.cjs';
+import js from './dist/main-js.js';
 /* eslint-enable n/no-missing-import */
 
-assert.deepStrictEqual(cjs.result, {
-  data: {
-    __proto__: null,
-    hello: 'world',
-  },
-});
-
-assert.deepStrictEqual(mjs.result, {
+assert.deepStrictEqual(js.result, {
   data: {
     __proto__: null,
     hello: 'world',

integrationTests/webpack/entry-esm.mjs

@@ -1,11 +1,10 @@
 {
   "mode": "production",
   "entry": {
-    "cjs": "./entry.js",
-    "mjs": "./entry-esm.mjs"
+    "js": "./entry.js"
   },
   "output": {
-    "filename": "main-[name].cjs",
+    "filename": "main-[name].js",
     "library": {
       "type": "commonjs2"
     }

integrationTests/webpack/package.json

@@ -15,14 +15,10 @@ import {
 } from './utils.js';
 
 console.log('\n./npmDist');
-await buildPackage('./npmDist', false);
+await buildPackage('./npmDist');
 showDirStats('./npmDist');
 
-console.log('\n./npmEsmDist');
-await buildPackage('./npmEsmDist', true);
-showDirStats('./npmEsmDist');
-
-async function buildPackage(outDir: string, isESMOnly: boolean): Promise<void> {
+async function buildPackage(outDir: string): Promise<void> {
   fs.rmSync(outDir, { recursive: true, force: true });
   fs.mkdirSync(outDir);
 
@@ -82,36 +78,25 @@ async function buildPackage(outDir: string, isESMOnly: boolean): Promise<void> {
     );
   }
 
-  if (isESMOnly) {
-    packageJSON.exports = {};
+  packageJSON.exports = {};
 
-    const { emittedTSFiles } = emitTSFiles({
-      outDir,
-      module: 'es2020',
-      extension: '.js',
-    });
+  const { emittedTSFiles } = emitTSFiles({
+    outDir,
+    module: 'es2020',
+    extension: '.js',
+  });
 
-    for (const filepath of emittedTSFiles) {
-      if (path.basename(filepath) === 'index.js') {
-        const relativePath = './' + path.relative('./npmEsmDist', filepath);
-        packageJSON.exports[path.dirname(relativePath)] = relativePath;
-      }
+  for (const filepath of emittedTSFiles) {
+    if (path.basename(filepath) === 'index.js') {
+      const relativePath = './' + path.relative('./npmDist', filepath);
+      packageJSON.exports[path.dirname(relativePath)] = relativePath;
     }
-
-    // Temporary workaround to allow "internal" imports, no grantees provided
-    packageJSON.exports['./*.js'] = './*.js';
-    packageJSON.exports['./*'] = './*.js';
-
-    packageJSON.publishConfig.tag += '-esm';
-    packageJSON.version += '+esm';
-  } else {
-    delete packageJSON.type;
-    packageJSON.main = 'index';
-    packageJSON.module = 'index.mjs';
-    emitTSFiles({ outDir, module: 'commonjs', extension: '.js' });
-    emitTSFiles({ outDir, module: 'es2020', extension: '.mjs' });
   }
 
+  // Temporary workaround to allow "internal" imports, no grantees provided
+  packageJSON.exports['./*.js'] = './*.js';
+  packageJSON.exports['./*'] = './*.js';
+
   const packageJsonPath = `./${outDir}/package.json`;
   const prettified = await prettify(
     packageJsonPath,

integrationTests/webpack/test.js

@@ -16,12 +16,6 @@ describe('Integration Tests', () => {
   const archiveName = npm({ cwd: tmpDirPath(), quiet: true }).pack(distDir);
   fs.renameSync(tmpDirPath(archiveName), tmpDirPath('graphql.tgz'));
 
-  const esmDistDir = localRepoPath('npmEsmDist');
-  const archiveEsmName = npm({ cwd: tmpDirPath(), quiet: true }).pack(
-    esmDistDir,
-  );
-  fs.renameSync(tmpDirPath(archiveEsmName), tmpDirPath('graphql-esm.tgz'));
-
   npm().run('build:deno');
 
   function testOnNodeProject(projectName: string) {