Skip to content

Commit 85ce232

Browse files
authored
Fix usage in Jest, drop *.module.js for just .mjs (files were nearly identical), don't include "browser" export map key, do include "module" export map key (to avoid dual package hazard).
1 parent 34c8752 commit 85ce232

File tree

7 files changed

+14
-13
lines changed

7 files changed

+14
-13
lines changed

package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
"scripts": {
55
"prebuild": "rimraf packages/core/dist/ packages/preact/dist",
66
"build": "pnpm build:core && pnpm build:preact && pnpm build:react-runtime && pnpm build:react-auto && pnpm build:react && pnpm build:react-transform",
7-
"_build": "microbundle --raw --globals @preact/signals-core=preactSignalsCore,preact/hooks=preactHooks,@preact/signals-react/runtime=reactSignalsRuntime",
7+
"_build": "microbundle --format esm,cjs --raw --globals @preact/signals-core=preactSignalsCore,preact/hooks=preactHooks,@preact/signals-react/runtime=reactSignalsRuntime",
88
"build:core": "pnpm _build --cwd packages/core && pnpm postbuild:core",
99
"build:preact": "pnpm _build --cwd packages/preact && pnpm postbuild:preact",
1010
"build:react": "pnpm _build --cwd packages/react --external \"react,@preact/signals-react/runtime,@preact/signals-core\" && pnpm postbuild:react",

packages/core/package.json

+2-2
Original file line numberDiff line numberDiff line change
@@ -19,15 +19,15 @@
1919
},
2020
"amdName": "preactSignalsCore",
2121
"main": "dist/signals-core.js",
22-
"module": "dist/signals-core.module.js",
22+
"module": "dist/signals-core.mjs",
2323
"unpkg": "dist/signals-core.min.js",
2424
"types": "dist/signals-core.d.ts",
2525
"source": "src/index.ts",
2626
"sideEffects": false,
2727
"exports": {
2828
".": {
2929
"types": "./dist/signals-core.d.ts",
30-
"browser": "./dist/signals-core.module.js",
30+
"module": "./dist/signals-core.mjs",
3131
"import": "./dist/signals-core.mjs",
3232
"require": "./dist/signals-core.js"
3333
}

packages/preact/package.json

+2-2
Original file line numberDiff line numberDiff line change
@@ -20,14 +20,14 @@
2020
},
2121
"amdName": "preactSignals",
2222
"main": "dist/signals.js",
23-
"module": "dist/signals.module.js",
23+
"module": "dist/signals.mjs",
2424
"unpkg": "dist/signals.min.js",
2525
"types": "dist/signals.d.ts",
2626
"source": "src/index.ts",
2727
"exports": {
2828
".": {
2929
"types": "./dist/signals.d.ts",
30-
"browser": "./dist/signals.module.js",
30+
"module": "./dist/signals.mjs",
3131
"import": "./dist/signals.mjs",
3232
"require": "./dist/signals.js"
3333
}

packages/react-transform/package.json

+2-1
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,13 @@
2323
"url": "https://opencollective.com/preact"
2424
},
2525
"main": "dist/signals-transform.js",
26-
"module": "dist/signals-transform.module.js",
26+
"module": "dist/signals-transform.mjs",
2727
"types": "dist/signals-transform.d.ts",
2828
"source": "src/index.ts",
2929
"exports": {
3030
".": {
3131
"types": "./dist/signals-transform.d.ts",
32+
"module": "./dist/signals-transform.mjs",
3233
"import": "./dist/signals-transform.mjs",
3334
"require": "./dist/signals-transform.js"
3435
}

packages/react/auto/package.json

+2-2
Original file line numberDiff line numberDiff line change
@@ -4,14 +4,14 @@
44
"private": true,
55
"amdName": "reactSignalsAuto",
66
"main": "dist/auto.js",
7-
"module": "dist/auto.module.js",
7+
"module": "dist/auto.mjs",
88
"unpkg": "dist/auto.min.js",
99
"types": "dist/index.d.ts",
1010
"mangle": "../../../mangle.json",
1111
"exports": {
1212
".": {
1313
"types": "./dist/index.d.ts",
14-
"browser": "./dist/auto.module.js",
14+
"module": "./dist/auto.mjs",
1515
"import": "./dist/auto.mjs",
1616
"require": "./dist/auto.js"
1717
}

packages/react/package.json

+3-3
Original file line numberDiff line numberDiff line change
@@ -27,20 +27,20 @@
2727
"exports": {
2828
".": {
2929
"types": "./dist/signals.d.ts",
30-
"browser": "./dist/signals.module.js",
30+
"module": "./dist/signals.mjs",
3131
"import": "./dist/signals.mjs",
3232
"require": "./dist/signals.js"
3333
},
3434
"./runtime": {
3535
"types": "./runtime/dist/index.d.ts",
36-
"browser": "./runtime/dist/runtime.module.js",
36+
"module": "./runtime/dist/runtime.mjs",
3737
"import": "./runtime/dist/runtime.mjs",
3838
"require": "./runtime/dist/runtime.js"
3939
},
4040
"./runtime/package.json": "./runtime/package.json",
4141
"./auto": {
4242
"types": "./auto/dist/index.d.ts",
43-
"browser": "./auto/dist/auto.module.js",
43+
"module": "./auto/dist/auto.mjs",
4444
"import": "./auto/dist/auto.mjs",
4545
"require": "./auto/dist/auto.js"
4646
},

packages/react/runtime/package.json

+2-2
Original file line numberDiff line numberDiff line change
@@ -4,15 +4,15 @@
44
"private": true,
55
"amdName": "reactSignalsRuntime",
66
"main": "dist/runtime.js",
7-
"module": "dist/runtime.module.js",
7+
"module": "dist/runtime.mjs",
88
"unpkg": "dist/runtime.min.js",
99
"types": "dist/index.d.ts",
1010
"source": "src/index.ts",
1111
"mangle": "../../../mangle.json",
1212
"exports": {
1313
".": {
1414
"types": "./dist/index.d.ts",
15-
"browser": "./dist/runtime.module.js",
15+
"module": "./dist/runtime.mjs",
1616
"import": "./dist/runtime.mjs",
1717
"require": "./dist/runtime.js"
1818
}

0 commit comments

Comments
 (0)