Add explicit exports for Node (#645)

Fixes #610 This adds explicit exports for node into the package.json for all published packages in Protobuf-ES. In addition, it modifies the path for the top-level import statement to use ESM instead of the proxy. For additional context, see connectrpc/connect-es#921
bufbuild · Dec 6, 2023 · 093967d · 093967d
1 parent 2062cd7
commit 093967d
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 6 deletions.
diff --git a/packages/protobuf/package.json b/packages/protobuf/package.json
@@ -25,9 +25,13 @@
   "main": "./dist/cjs/index.js",
   "exports": {
     ".": {
+      "node": {
+        "import": "./dist/proxy/index.js",
+        "require": "./dist/cjs/index.js"
+      },
       "module": "./dist/esm/index.js",
-      "require": "./dist/cjs/index.js",
-      "import": "./dist/proxy/index.js"
+      "import": "./dist/esm/index.js",
+      "require": "./dist/cjs/index.js"
     }
   },
   "devDependencies": {

diff --git a/packages/protoplugin/package.json b/packages/protoplugin/package.json
@@ -21,14 +21,22 @@
   "main": "./dist/cjs/index.js",
   "exports": {
     ".": {
+      "node": {
+        "import": "./dist/proxy/index.js",
+        "require": "./dist/cjs/index.js"
+      },
       "module": "./dist/esm/index.js",
-      "require": "./dist/cjs/index.js",
-      "import": "./dist/proxy/index.js"
+      "import": "./dist/esm/index.js",
+      "require": "./dist/cjs/index.js"
     },
     "./ecmascript": {
+      "node": {
+        "import": "./dist/proxy/ecmascript/index.js",
+        "require": "./dist/cjs/ecmascript/index.js"
+      },
       "module": "./dist/esm/ecmascript/index.js",
-      "require": "./dist/cjs/ecmascript/index.js",
-      "import": "./dist/proxy/ecmascript/index.js"
+      "import": "./dist/esm/ecmascript/index.js",
+      "require": "./dist/cjs/ecmascript/index.js"
     }
   },
   "typesVersions": {