cd(node): ban package.json main entry

fabricjs · Nov 19, 2023 · 682acc1 · 682acc1
1 parent 44a75a3
commit 682acc1
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/package.json b/package.json
@@ -131,7 +131,7 @@
     "node": ">=16.20.0"
   },
   "module": "./dist/index.mjs",
-  "main": "./dist/index.node.cjs",
+  "main": null,
   "types": "./dist/index.d.ts",
   "typesVersions": {
     ">=4.2": {

diff --git a/test/unit/env.js b/test/unit/env.js
@@ -5,12 +5,17 @@ QUnit.module('env', (hooks) => {
             delete global.document;
         })
 
+        QUnit.test('import/require of `main` field of package.json throws', assert => {
+            assert.rejects(import('../..'), 'should not resolve main');
+            assert.throws(() => require('../..'), 'should not resolve main');
+        });
+
         QUnit.test('import/require sets env', async assert => {
             const done = assert.async();
             global.window = { devicePixelRatio: 1.25 };
             global.document = { foo: 'bar' };
             const imported = await import('../../dist/index.node.cjs');
-            const required = require('../..');
+            const required = require('../../dist/index.node.cjs');
             assert.equal(imported.getEnv().document.foo, undefined, 'should be node env');
             assert.equal(required.getEnv().document.foo, undefined, 'should be node env');
             done();