Backport dual-build (and CI improvements) from express5

.changeset/tame-carpets-beam.md

@@ -0,0 +1,5 @@
+---
+'@as-integrations/express4': minor
+---
+
+The package is now built for both CJS and ESM, instead of only for CJS; this is the same build approach taken by `@apollo/server`. This provides better compatibility with ESM-based build systems.

.github/workflows/ci.yml

@@ -1,7 +1,14 @@
 name: CI
 
 on:
-  pull_request
+  pull_request:
+    types:
+    - opened
+    - reopened
+    - synchronize
+    # So you can add a label to make it run, including no-changeset-needed
+    - labeled
+    - unlabeled
 
 jobs:
   build-and-test:
@@ -61,11 +68,11 @@ jobs:
   changeset:
     runs-on: ubuntu-latest
     name: Changesets
+    if: ${{ (! startsWith(github.head_ref, 'changeset-release/')) && (!contains(github.event.pull_request.labels.*.name, 'no-changeset-needed')) }}
     steps:
     - uses: actions/checkout@v4
       with:
         # This makes Actions fetch all Git history so that Changesets can generate changelogs with the correct commits
         fetch-depth: 0
     - uses: ./.github/actions/node
     - run: npm run changeset-check
-      if: ${{ ! startsWith(github.head_ref, 'changeset-release/') }}

package.json

@@ -12,18 +12,31 @@
   "bugs": {
     "url": "https://github.com/apollo-server-integrations/apollo-server-integration-express4/issues"
   },
-  "main": "dist/index.js",
-  "types": "dist/index.d.ts",
+  "type": "module",
+  "main": "dist/cjs/index.js",
+  "module": "dist/esm/index.js",
+  "types": "dist/esm/index.d.ts",
+  "exports": {
+    ".": {
+      "types": {
+        "require": "./dist/cjs/index.d.ts",
+        "default": "./dist/esm/index.d.ts"
+      },
+      "import": "./dist/esm/index.js",
+      "require": "./dist/cjs/index.js"
+    }
+  },
   "engines": {
     "node": ">=20"
   },
   "scripts": {
-    "build": "tsc --build tsconfig.build.json",
+    "compile": "tsc --build tsconfig.build.json",
+    "postcompile": "echo '{\"type\":\"module\"}' > dist/esm/package.json && echo '{\"type\":\"commonjs\"}' > dist/cjs/package.json",
     "clean": "git clean -dfqX",
-    "prepack": "npm run build",
+    "prepack": "npm run compile",
     "prettier-check": "prettier --check .",
     "prettier-fix": "prettier --write .",
-    "changeset-publish": "npm run build && changeset publish",
+    "changeset-publish": "npm run compile && changeset publish",
     "changeset-check": "changeset status --verbose --since=origin/main",
     "changeset-version": "changeset version && npm i",
     "spell-check": "cspell lint '**' --no-progress || (echo 'Add any real words to cspell-dict.txt.'; exit 1)",

tsconfig.base.json

@@ -1,9 +1,10 @@
 {
   "compilerOptions": {
+    "composite": true,
     "rootDir": "./src",
-    "outDir": "./dist",
+    "outDir": "./dist/esm",
     "target": "es2020",
-    "module": "commonjs",
+    "module": "esnext",
     "moduleResolution": "node",
     "esModuleInterop": true,
     "sourceMap": true,

tsconfig.build.cjs.json

@@ -0,0 +1,7 @@
+{
+  "extends": "./tsconfig.build.esm.json",
+  "compilerOptions": {
+    "module": "commonjs",
+    "outDir": "./dist/cjs"
+  }
+}

tsconfig.build.esm.json

@@ -0,0 +1,5 @@
+{
+  "extends": "./tsconfig.base.json",
+  "include": ["src/**/*"],
+  "exclude": ["**/__tests__"]
+}

tsconfig.build.json

@@ -1,5 +1,11 @@
 {
-  "extends": "./tsconfig.base.json",
-  "include": ["src/**/*"],
-  "exclude": ["**/__tests__"],
+  "compilerOptions": {
+    "composite": true
+  },
+  "files": [],
+  "include": [],
+  "references": [
+    { "path": "./tsconfig.build.esm.json" },
+    { "path": "./tsconfig.build.cjs.json" }
+  ]
 }