Build node-fetch-server using esbuild

Author: mjackson

packages/node-fetch-server/.npmignore

@@ -0,0 +1,2 @@
+# Ignore test files within the src directory
+src/**/*.test.ts

packages/node-fetch-server/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 This is the changelog for [`node-fetch-server`](https://github.com/mjackson/remix-the-web/tree/main/packages/node-fetch-server). It follows [semantic versioning](https://semver.org/).
 
+## HEAD
+
+- Add `/src` to npm package, so "go to definition" goes to the actual source
+- Use one set of types for all built files, instead of separate types for ESM and CJS
+- Build using esbuild directly instead of tsup
+
 ## v0.6.1 (2025-02-07)
 
 - Update typings and docs for http2 support

packages/node-fetch-server/package.json

@@ -11,41 +11,35 @@
   "homepage": "https://github.com/mjackson/remix-the-web/tree/main/packages/node-fetch-server#readme",
   "license": "MIT",
   "files": [
-    "dist",
     "LICENSE",
-    "README.md"
+    "README.md",
+    "dist",
+    "src"
   ],
   "type": "module",
   "types": "./dist/node-fetch-server.d.ts",
-  "main": "./dist/node-fetch-server.js",
+  "module": "./dist/node-fetch-server.js",
+  "main": "./dist/node-fetch-server.cjs",
   "exports": {
     ".": {
-      "module-sync": {
-        "types": "./dist/node-fetch-server.d.ts",
-        "default": "./dist/node-fetch-server.js"
-      },
-      "import": {
-        "types": "./dist/node-fetch-server.d.ts",
-        "default": "./dist/node-fetch-server.js"
-      },
-      "require": {
-        "types": "./dist/node-fetch-server.d.cts",
-        "default": "./dist/node-fetch-server.cjs"
-      },
-      "default": {
-        "types": "./dist/node-fetch-server.d.ts",
-        "default": "./dist/node-fetch-server.js"
-      }
+      "types": "./dist/node-fetch-server.d.ts",
+      "import": "./dist/node-fetch-server.js",
+      "require": "./dist/node-fetch-server.cjs",
+      "default": "./dist/node-fetch-server.js"
     },
     "./package.json": "./package.json"
   },
   "devDependencies": {
     "@types/node": "^22.5.0",
-    "tsup": "^8.3.5"
+    "esbuild": "^0.20.2"
   },
   "scripts": {
     "bench": "bash ./bench/runner.sh",
-    "build": "tsup",
+    "build:types": "tsc --project tsconfig.build.json",
+    "build:esm": "esbuild src/node-fetch-server.ts --bundle --outfile=dist/node-fetch-server.js --format=esm --platform=node --sourcemap",
+    "build:cjs": "esbuild src/node-fetch-server.ts --bundle --outfile=dist/node-fetch-server.cjs --format=cjs --platform=node --sourcemap",
+    "build": "pnpm run build:types && pnpm run build:esm && pnpm run build:cjs",
+    "clean": "rm -rf dist",
     "test": "node --experimental-strip-types --disable-warning=ExperimentalWarning --test ./src/**/*.test.ts",
     "prepublishOnly": "pnpm run build"
   },

packages/node-fetch-server/tsconfig.build.json

@@ -0,0 +1,11 @@
+{
+  "extends": "./tsconfig.json",
+  "compilerOptions": {
+    "declaration": true,
+    "emitDeclarationOnly": true,
+    "declarationMap": true,
+    "outDir": "./dist"
+  },
+  "include": ["src"],
+  "exclude": ["src/**/*.test.ts"]
+}