refactor: drop common js support and become pure esm

logto-io · Oct 15, 2024 · 0034fc3 · 0034fc3
1 parent 41a3312
commit 0034fc3
Show file tree

Hide file tree

Showing 15 changed files with 22 additions and 53 deletions.
diff --git a/.changeset/many-peaches-kneel.md b/.changeset/many-peaches-kneel.md
@@ -0,0 +1,14 @@
+---
+"@logto/capacitor": major
+"@logto/browser": major
+"@logto/express": major
+"@logto/client": major
+"@logto/react": major
+"@logto/remix": major
+"@logto/next": major
+"@logto/node": major
+"@logto/vue": major
+"@logto/js": major
+---
+
+drop common JS support and become pure ESM
diff --git a/packages/browser/package.json b/packages/browser/package.json
@@ -2,12 +2,10 @@
   "name": "@logto/browser",
   "version": "2.2.19",
   "type": "module",
-  "main": "./lib/index.cjs",
   "module": "./lib/index.js",
   "types": "./lib/index.d.ts",
   "exports": {
     "types": "./lib/index.d.ts",
-    "require": "./lib/index.cjs",
     "import": "./lib/index.js",
     "default": "./lib/index.js"
   },

diff --git a/packages/capacitor/package.json b/packages/capacitor/package.json
@@ -2,12 +2,10 @@
   "name": "@logto/capacitor",
   "version": "2.0.5",
   "type": "module",
-  "main": "./lib/index.cjs",
   "module": "./lib/index.js",
   "types": "./lib/index.d.ts",
   "exports": {
     "types": "./lib/index.d.ts",
-    "require": "./lib/index.cjs",
     "import": "./lib/index.js",
     "default": "./lib/index.js"
   },

diff --git a/packages/client/package.json b/packages/client/package.json
@@ -2,13 +2,11 @@
   "name": "@logto/client",
   "version": "2.8.2",
   "type": "module",
-  "main": "./lib/index.cjs",
   "module": "./lib/index.js",
   "types": "./lib/index.d.ts",
   "react-native": "./lib/shim.js",
   "exports": {
     "types": "./lib/index.d.ts",
-    "require": "./lib/index.cjs",
     "import": "./lib/index.js",
     "default": "./lib/index.js"
   },

diff --git a/packages/express/package.json b/packages/express/package.json
@@ -2,12 +2,10 @@
   "name": "@logto/express",
   "version": "2.4.0",
   "type": "module",
-  "main": "./lib/index.cjs",
   "module": "./lib/index.js",
   "types": "./lib/index.d.ts",
   "exports": {
     "types": "./lib/index.d.ts",
-    "require": "./lib/index.cjs",
     "import": "./lib/index.js",
     "default": "./lib/index.js"
   },
@@ -27,9 +25,8 @@
     "build": "rm -rf lib/ && tsc -p tsconfig.build.json --noEmit && rollup -c",
     "lint": "eslint --ext .ts src",
     "test": "vitest",
-    "test:coverage": "node test.cjs && vitest --silent --coverage",
-    "prepack": "pnpm build && pnpm test",
-    "postpack": "node test.cjs"
+    "test:coverage": "vitest --silent --coverage",
+    "prepack": "pnpm build && pnpm test"
   },
   "dependencies": {
     "@logto/node": "workspace:^"

diff --git a/packages/express/test.cjs b/packages/express/test.cjs
diff --git a/packages/js/package.json b/packages/js/package.json
@@ -2,12 +2,10 @@
   "name": "@logto/js",
   "version": "4.2.1",
   "type": "module",
-  "main": "./lib/index.cjs",
   "module": "./lib/index.js",
   "types": "./lib/index.d.ts",
   "exports": {
     "types": "./lib/index.d.ts",
-    "require": "./lib/index.cjs",
     "import": "./lib/index.js"
   },
   "files": [

diff --git a/packages/next/package.json b/packages/next/package.json
@@ -2,7 +2,6 @@
   "name": "@logto/next",
   "version": "3.7.2",
   "type": "module",
-  "main": "./lib/src/index.cjs",
   "module": "./lib/src/index.js",
   "types": "./lib/src/index.d.ts",
   "typesVersions": {
@@ -20,17 +19,14 @@
   },
   "exports": {
     ".": {
-      "require": "./lib/src/index.cjs",
       "import": "./lib/src/index.js",
       "types": "./lib/src/index.d.ts"
     },
     "./edge": {
-      "require": "./lib/edge/index.cjs",
       "import": "./lib/edge/index.js",
       "types": "./lib/edge/index.d.ts"
     },
     "./server-actions": {
-      "require": "./lib/server-actions/index.cjs",
       "import": "./lib/server-actions/index.js",
       "types": "./lib/server-actions/index.d.ts"
     }
@@ -51,9 +47,8 @@
     "build": "rm -rf lib/ && tsc -p tsconfig.build.json --noEmit && rollup -c",
     "lint": "eslint --ext .ts src",
     "test": "vitest",
-    "test:coverage": "node test.cjs && vitest --silent --coverage",
-    "prepack": "pnpm build && pnpm test",
-    "postpack": "node test.cjs"
+    "test:coverage": "vitest --silent --coverage",
+    "prepack": "pnpm build && pnpm test"
   },
   "dependencies": {
     "@edge-runtime/cookies": "^5.0.0",

diff --git a/packages/next/test.cjs b/packages/next/test.cjs
diff --git a/packages/node/package.json b/packages/node/package.json
@@ -2,22 +2,18 @@
   "name": "@logto/node",
   "version": "2.5.9",
   "type": "module",
-  "main": "./lib/src/index.cjs",
   "module": "./lib/src/index.js",
   "types": "./lib/src/index.d.ts",
   "exports": {
     ".": {
-      "require": "./lib/src/index.cjs",
       "import": "./lib/src/index.js",
       "types": "./lib/src/index.d.ts"
     },
     "./edge": {
-      "require": "./lib/edge/index.cjs",
       "import": "./lib/edge/index.js",
       "types": "./lib/edge/index.d.ts"
     },
     "./exports": {
-      "require": "./lib/exports/index.cjs",
       "import": "./lib/exports/index.js",
       "types": "./lib/exports/index.d.ts"
     }
@@ -38,9 +34,8 @@
     "build": "rm -rf lib/ && tsc -p tsconfig.build.json --noEmit && rollup -c",
     "lint": "eslint --ext .ts src",
     "test": "vitest",
-    "test:coverage": "node test.cjs && vitest --silent --coverage",
-    "prepack": "pnpm build && pnpm test",
-    "postpack": "node test.cjs"
+    "test:coverage": "vitest --silent --coverage",
+    "prepack": "pnpm build && pnpm test"
   },
   "dependencies": {
     "@logto/client": "workspace:^",

diff --git a/packages/node/test.cjs b/packages/node/test.cjs
diff --git a/packages/react/package.json b/packages/react/package.json
@@ -2,12 +2,10 @@
   "name": "@logto/react",
   "version": "3.0.17",
   "type": "module",
-  "main": "./lib/index.cjs",
   "module": "./lib/index.js",
   "types": "./lib/index.d.ts",
   "exports": {
     "types": "./lib/index.d.ts",
-    "require": "./lib/index.cjs",
     "import": "./lib/index.js",
     "default": "./lib/index.js"
   },

diff --git a/packages/remix/package.json b/packages/remix/package.json
@@ -2,12 +2,10 @@
   "name": "@logto/remix",
   "version": "2.2.8",
   "type": "module",
-  "main": "./lib/index.cjs",
   "module": "./lib/index.js",
   "types": "./lib/index.d.ts",
   "exports": {
     "types": "./lib/index.d.ts",
-    "require": "./lib/index.cjs",
     "import": "./lib/index.js",
     "default": "./lib/index.js"
   },

diff --git a/packages/vue/package.json b/packages/vue/package.json
@@ -2,12 +2,10 @@
   "name": "@logto/vue",
   "version": "2.2.18",
   "type": "module",
-  "main": "./lib/index.cjs",
   "module": "./lib/index.js",
   "types": "./lib/index.d.ts",
   "exports": {
     "types": "./lib/index.d.ts",
-    "require": "./lib/index.cjs",
     "import": "./lib/index.js",
     "default": "./lib/index.js"
   },

diff --git a/rollup.config.js b/rollup.config.js
@@ -9,11 +9,11 @@ const configs = {
   input: ['src/index.ts'],
   output: [
     {
-      format: 'cjs',
+      format: 'esm',
       dir: 'lib',
       preserveModules: true,
       exports: 'named',
-      entryFileNames: '[name].cjs',
+      entryFileNames: '[name].js',
       interop: 'auto',
     },
     { dir: 'lib', preserveModules: true },