Merge pull request #803 from acacode/tsup

Add support for both ESM and CJS

Author: smorimoto

.gitignore

@@ -1,3 +1,4 @@
 /.yarn/*
 !/.yarn/releases
+/dist/
 /node_modules/

index.js

@@ -1,16 +1,13 @@
 #!/usr/bin/env node
 
-import { createRequire } from "node:module";
 import { resolve } from "node:path";
 import { cli } from "./cli/index.js";
+import packageJson from "./package.json";
 import { TemplatesGenConfig } from "./src/commands/generate-templates/configuration.js";
 import { CodeGenConfig } from "./src/configuration.js";
 import { HTTP_CLIENT } from "./src/constants.js";
 import { generateApi, generateTemplates } from "./src/index.js";
 
-const require = createRequire(import.meta.url);
-const packageJson = require("./package.json");
-
 const codeGenBaseConfig = new CodeGenConfig({});
 const templateGenBaseConfig = new TemplatesGenConfig({});
 

package.json

@@ -11,29 +11,39 @@
     "Sora Morimoto <[email protected]>"
   ],
   "type": "module",
-  "main": "./src/index.js",
+  "exports": {
+    ".": {
+      "import": {
+        "types": "./index.d.ts",
+        "default": "./dist/lib.js"
+      },
+      "require": {
+        "types": "./index.d.ts",
+        "default": "./dist/lib.cjs"
+      }
+    }
+  },
+  "main": "./dist/lib.cjs",
+  "module": "./dist/lib.js",
   "types": "./index.d.ts",
   "bin": {
-    "sta": "./index.js",
-    "swagger-typescript-api": "./index.js"
+    "sta": "./dist/cli.js",
+    "swagger-typescript-api": "./dist/cli.js"
   },
   "files": [
-    "cli",
-    "src",
+    "dist",
     "templates",
-    "index.d.ts",
-    "index.js"
+    "index.d.ts"
   ],
   "scripts": {
+    "build": "tsup",
     "ci": "biome ci .",
     "cli:help": "node index.js -h",
     "cli:json": "node index.js -r -d -p ./swagger-test-cli.json -n swagger-test-cli.ts",
     "cli:yaml": "node index.js -r -d -p ./swagger-test-cli.yaml -n swagger-test-cli.ts",
     "format": "biome format --write .",
     "format:check": "biome format .",
-    "test": "vitest run",
-    "validate": "node tests/validate.js",
-    "validate:debug": "node --nolazy tests/validate.js"
+    "test": "vitest run"
   },
   "dependencies": {
     "@types/swagger-schema-official": "^2.0.25",
@@ -60,6 +70,7 @@
     "@types/swagger2openapi": "7.0.4",
     "axios": "1.7.2",
     "shx": "0.3.4",
+    "tsup": "8.1.0",
     "vitest": "1.6.0"
   },
   "packageManager": "[email protected]",

src/constants.js

@@ -1,7 +1,4 @@
-import { createRequire } from "node:module";
-
-const require = createRequire(import.meta.url);
-const packageJson = require("../package.json");
+import packageJson from "../package.json";
 
 const DEFAULT_BODY_ARG_NAME = "data";
 

tsconfig.json

@@ -1,9 +1,13 @@
 {
-  "extends": ["@tsconfig/strictest", "@tsconfig/node18"],
+  "extends": [
+    "@tsconfig/strictest/tsconfig.json",
+    "@tsconfig/node18/tsconfig.json"
+  ],
   "compilerOptions": {
     "checkJs": false,
     "noEmit": true,
     "noPropertyAccessFromIndexSignature": false,
     "resolveJsonModule": true
-  }
+  },
+  "exclude": ["dist", "node_modules"]
 }

tsup.config.ts

@@ -0,0 +1,15 @@
+import { defineConfig } from "tsup";
+
+export default defineConfig({
+  entry: {
+    lib: "src/index.js",
+    cli: "index.js",
+  },
+  clean: true,
+  format: ["esm", "cjs"],
+  minify: true,
+  sourcemap: true,
+  splitting: true,
+  target: "node22",
+  treeshake: true,
+});