Merge pull request #728 from forehalo/main

fix dual module types exporting
abraham · Feb 6, 2023 · bed14d1 · bed14d1
2 parents c53a3b9 + 0ab5dec
commit bed14d1
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/.github/workflows/index.yaml b/.github/workflows/index.yaml
@@ -20,6 +20,10 @@ jobs:
       - run: npm run lint
       - run: npm run build
       - run: npm run test
+      - name: module test
+        run: |
+          node --input-type=module --eval "import * as m from '@abraham/reflection'; console.log(m)"
+          node --input-type=commonjs --eval "const m = require('@abraham/reflection'); console.log(m)"
       - run: npm run pkg-ok
       - uses: coverallsapp/github-action@master
         with:

diff --git a/package.json b/package.json
@@ -5,9 +5,9 @@
   "type": "module",
   "exports": {
     "import": "./dist/index.js",
-    "require": "./dist/index.umd.js"
+    "require": "./dist/index.umd.cjs"
   },
-  "main": "./dist/index.umd.js",
+  "main": "./dist/index.umd.cjs",
   "module": "./dist/index.js",
   "types": "./dist/index.d.ts",
   "scripts": {
@@ -80,7 +80,7 @@
   "rollup": {
     "input": "dist/index.js",
     "output": {
-      "file": "dist/index.umd.js",
+      "file": "dist/index.umd.cjs",
       "format": "umd",
       "name": "@abraham/reflection"
     }