Merge #1737

1737: Fix package exports r=curquiza a=flevi29

# Pull Request

## Related issue
Fixes #1736

## PR checklist
Please check if your PR fulfills the following requirements:
- [x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
- [x] Have you read the contributing guidelines?
- [x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!


Co-authored-by: F. Levi <55688616+flevi29@users.noreply.github.com>

package.json

@@ -16,9 +16,8 @@
     "qdequele <quentin@meilisearch.com>"
   ],
   "license": "MIT",
-  "main": "./dist/bundles/meilisearch.cjs.js",
-  "module": "./dist/bundles/meilisearch.esm.js",
-  "browser": "./dist/bundles/meilisearch.umd.js",
+  "main": "./dist/bundles/meilisearch.cjs",
+  "module": "./dist/bundles/meilisearch.mjs",
   "typings": "./dist/types/index.d.ts",
   "types": "./dist/types/index.d.ts",
   "jsdelivr": "./dist/bundles/meilisearch.umd.js",
@@ -27,8 +26,8 @@
     ".": {
       "types": "./dist/types/index.d.ts",
       "browser": "./dist/bundles/meilisearch.umd.js",
-      "import": "./dist/bundles/meilisearch.esm.js",
-      "require": "./dist/bundles/meilisearch.cjs.js",
+      "import": "./dist/bundles/meilisearch.mjs",
+      "require": "./dist/bundles/meilisearch.cjs",
       "default": "./dist/bundles/meilisearch.umd.js"
     }
   },
@@ -56,7 +55,7 @@
     "test:env:nodejs": "yarn build && node tests/env/node/index.js && node tests/env/node/getting_started.js",
     "test:env:esm": "yarn --cwd tests/env/esm && yarn --cwd tests/env/esm start",
     "test:env:nitro-app": "yarn build && yarn --cwd tests/env/nitro-app test",
-    "size": "node scripts/file-size ./dist/bundles/meilisearch.esm.min.js ./dist/bundles/meilisearch.umd.min.js",
+    "size": "node scripts/file-size ./dist/bundles/meilisearch.mjs ./dist/bundles/meilisearch.umd.min.js",
     "style": "yarn fmt && yarn lint",
     "style:fix": "yarn fmt:fix && yarn lint:fix",
     "fmt": "prettier -c ./**/*.{js,ts}",

rollup.config.js

@@ -85,10 +85,7 @@ module.exports = [
         sourcemap: env === "production", // create sourcemap for error reporting in production mode
       },
     ],
-    plugins: [
-      env === "production" ? terser() : {}, // will minify the file in production mode
-      ...PLUGINS,
-    ],
+    plugins: PLUGINS,
   },
   // Common JS build (Node).
   // Compatible only in a nodeJS environment.
@@ -102,7 +99,8 @@ module.exports = [
       ),
       exports: "named",
       format: "cjs",
+      sourcemap: env === "production", // create sourcemap for error reporting in production mode
     },
-    plugins: [...PLUGINS],
+    plugins: PLUGINS,
   },
 ];