Skip to content

Commit 466e3ce

Browse files
authored
fix: move package exports from publishConfig to top-level fields (toss#318)
1 parent 91fc741 commit 466e3ce

3 files changed

Lines changed: 42 additions & 36 deletions

File tree

.changeset/fix-publishconfig.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
---
2+
'react-simplikit': patch
3+
'@react-simplikit/mobile': patch
4+
---
5+
6+
Fix broken package exports by moving main/types/module/exports from publishConfig to top-level package.json fields
7+
8+
npm does not support publishConfig field overrides for manifest fields like main, types, and exports. The previous versions (react-simplikit@0.0.47, @react-simplikit/mobile@0.0.1) were published with incorrect entry points because publishConfig overrides were not applied during `npm publish`.

packages/core/package.json

Lines changed: 17 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,22 @@
11
{
22
"name": "react-simplikit",
33
"version": "0.0.47",
4-
"main": "./src/index.ts",
4+
"main": "./dist/index.cjs",
5+
"module": "./esm/index.js",
6+
"types": "./dist/index.d.cts",
7+
"exports": {
8+
".": {
9+
"import": {
10+
"types": "./esm/index.d.ts",
11+
"default": "./esm/index.js"
12+
},
13+
"require": {
14+
"types": "./dist/index.d.cts",
15+
"default": "./dist/index.cjs"
16+
}
17+
},
18+
"./package.json": "./package.json"
19+
},
520
"type": "module",
621
"sideEffects": false,
722
"keywords": [
@@ -29,23 +44,7 @@
2944
"test:coverage": "vitest run --coverage"
3045
},
3146
"publishConfig": {
32-
"access": "public",
33-
"main": "./dist/index.cjs",
34-
"types": "./dist/index.d.cts",
35-
"module": "./esm/index.js",
36-
"exports": {
37-
".": {
38-
"import": {
39-
"types": "./esm/index.d.ts",
40-
"default": "./esm/index.js"
41-
},
42-
"require": {
43-
"types": "./dist/index.d.cts",
44-
"default": "./dist/index.cjs"
45-
}
46-
},
47-
"./package.json": "./package.json"
48-
}
47+
"access": "public"
4948
},
5049
"devDependencies": {
5150
"@changesets/changelog-github": "^0.5.1",

packages/mobile/package.json

Lines changed: 17 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,22 @@
33
"version": "0.0.1",
44
"description": "Mobile web utilities for React - fixing viewport, keyboard, and layout issues",
55
"type": "module",
6-
"main": "./src/index.ts",
6+
"main": "./dist/index.cjs",
7+
"module": "./dist/index.js",
8+
"types": "./dist/index.d.ts",
9+
"exports": {
10+
".": {
11+
"import": {
12+
"types": "./dist/index.d.ts",
13+
"default": "./dist/index.js"
14+
},
15+
"require": {
16+
"types": "./dist/index.d.cts",
17+
"default": "./dist/index.cjs"
18+
}
19+
},
20+
"./package.json": "./package.json"
21+
},
722
"files": [
823
"dist/**/*"
924
],
@@ -55,22 +70,6 @@
5570
"vitest": "^2.1.8"
5671
},
5772
"publishConfig": {
58-
"access": "public",
59-
"main": "./dist/index.cjs",
60-
"module": "./dist/index.js",
61-
"types": "./dist/index.d.ts",
62-
"exports": {
63-
".": {
64-
"import": {
65-
"types": "./dist/index.d.ts",
66-
"default": "./dist/index.js"
67-
},
68-
"require": {
69-
"types": "./dist/index.d.cts",
70-
"default": "./dist/index.cjs"
71-
}
72-
},
73-
"./package.json": "./package.json"
74-
}
73+
"access": "public"
7574
}
7675
}

0 commit comments

Comments
 (0)