Skip to content

Commit 9b99329

Browse files
committed
Add type: module
1 parent 4566904 commit 9b99329

File tree

4 files changed

+39
-46
lines changed

4 files changed

+39
-46
lines changed

packages/react-router-dom/package.json

+6-5
Original file line numberDiff line numberDiff line change
@@ -9,20 +9,21 @@
99
"directory": "packages/react-router-dom"
1010
},
1111
"license": "MIT",
12-
"main": "./main.js",
12+
"main": "./main.cjs",
1313
"module": "./index.js",
1414
"types": "./index.d.ts",
1515
"exports": {
1616
".": {
1717
"node": {
18-
"require": "./main.js",
19-
"import": "./index.mjs"
18+
"require": "./index.cjs",
19+
"import": "./index.js"
2020
},
21+
"types": "./index.d.ts",
2122
"default": "./index.js"
2223
},
2324
"./server": {
24-
"require": "./server.js",
25-
"import": "./server.mjs"
25+
"require": "./server.cjs",
26+
"import": "./server.js"
2627
},
2728
"./package.json": "./package.json"
2829
},

packages/react-router/package.json

+4-3
Original file line numberDiff line numberDiff line change
@@ -9,15 +9,16 @@
99
"directory": "packages/react-router"
1010
},
1111
"license": "MIT",
12-
"main": "./main.js",
12+
"main": "./index.cjs",
1313
"module": "./index.js",
1414
"types": "./index.d.ts",
1515
"exports": {
1616
".": {
1717
"node": {
18-
"require": "./main.js",
19-
"import": "./index.mjs"
18+
"require": "./index.cjs",
19+
"import": "./index.js"
2020
},
21+
"types": "./index.d.ts",
2122
"default": "./index.js"
2223
},
2324
"./package.json": "./package.json"

rollup.config.js

+25-34
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,13 @@ function getVersion(sourceDir) {
2424
return require(`./${sourceDir}/package.json`).version;
2525
}
2626

27+
function addTypeModule(contents) {
28+
return JSON.stringify({
29+
type: "module",
30+
...JSON.parse(contents.toString()),
31+
}, null, 2);
32+
}
33+
2734
function reactRouter() {
2835
const SOURCE_DIR = "packages/react-router";
2936
const OUTPUT_DIR = "build/node_modules/react-router";
@@ -33,20 +40,12 @@ function reactRouter() {
3340
const modules = [
3441
{
3542
input: `${SOURCE_DIR}/index.tsx`,
36-
output: [
37-
{
38-
file: `${OUTPUT_DIR}/index.js`,
39-
format: "esm",
40-
sourcemap: !PRETTY,
41-
banner: createBanner("React Router", version)
42-
},
43-
{
44-
file: `${OUTPUT_DIR}/index.mjs`,
45-
format: "esm",
46-
sourcemap: !PRETTY,
47-
banner: createBanner("React Router", version)
48-
}
49-
],
43+
output: {
44+
file: `${OUTPUT_DIR}/index.js`,
45+
format: "esm",
46+
sourcemap: !PRETTY,
47+
banner: createBanner("React Router", version)
48+
},
5049
external: ["history", "react"],
5150
plugins: [
5251
babel({
@@ -61,7 +60,7 @@ function reactRouter() {
6160
}),
6261
copy({
6362
targets: [
64-
{ src: `${SOURCE_DIR}/package.json`, dest: OUTPUT_DIR },
63+
{ src: `${SOURCE_DIR}/package.json`, dest: OUTPUT_DIR, transform: addTypeModule },
6564
{ src: `${SOURCE_DIR}/README.md`, dest: OUTPUT_DIR },
6665
{ src: "LICENSE.md", dest: OUTPUT_DIR }
6766
],
@@ -208,7 +207,7 @@ function reactRouter() {
208207
{
209208
input: `${SOURCE_DIR}/node-main.js`,
210209
output: {
211-
file: `${OUTPUT_DIR}/main.js`,
210+
file: `${OUTPUT_DIR}/index.cjs`,
212211
format: "cjs",
213212
banner: createBanner("React Router", version)
214213
},
@@ -228,20 +227,12 @@ function reactRouterDom() {
228227
const modules = [
229228
{
230229
input: `${SOURCE_DIR}/index.tsx`,
231-
output: [
232-
{
233-
file: `${OUTPUT_DIR}/index.js`,
234-
format: "esm",
235-
sourcemap: !PRETTY,
236-
banner: createBanner("React Router DOM", version)
237-
},
238-
{
239-
file: `${OUTPUT_DIR}/index.mjs`,
240-
format: "esm",
241-
sourcemap: !PRETTY,
242-
banner: createBanner("React Router DOM", version)
243-
}
244-
],
230+
output: {
231+
file: `${OUTPUT_DIR}/index.js`,
232+
format: "esm",
233+
sourcemap: !PRETTY,
234+
banner: createBanner("React Router DOM", version)
235+
},
245236
external: ["history", "react", "react-dom", "react-router"],
246237
plugins: [
247238
babel({
@@ -256,7 +247,7 @@ function reactRouterDom() {
256247
}),
257248
copy({
258249
targets: [
259-
{ src: `${SOURCE_DIR}/package.json`, dest: OUTPUT_DIR },
250+
{ src: `${SOURCE_DIR}/package.json`, dest: OUTPUT_DIR, transform: addTypeModule },
260251
{ src: `${SOURCE_DIR}/README.md`, dest: OUTPUT_DIR },
261252
{ src: "LICENSE.md", dest: OUTPUT_DIR }
262253
],
@@ -413,7 +404,7 @@ function reactRouterDom() {
413404
{
414405
input: `${SOURCE_DIR}/node-main.js`,
415406
output: {
416-
file: `${OUTPUT_DIR}/main.js`,
407+
file: `${OUTPUT_DIR}/index.cjs`,
417408
format: "cjs",
418409
banner: createBanner("React Router DOM", version)
419410
},
@@ -422,7 +413,7 @@ function reactRouterDom() {
422413
{
423414
input: `${SOURCE_DIR}/server.tsx`,
424415
output: {
425-
file: `${OUTPUT_DIR}/server.js`,
416+
file: `${OUTPUT_DIR}/server.cjs`,
426417
format: "cjs"
427418
},
428419
external: [
@@ -449,7 +440,7 @@ function reactRouterDom() {
449440
{
450441
input: `${SOURCE_DIR}/server.tsx`,
451442
output: {
452-
file: `${OUTPUT_DIR}/server.mjs`,
443+
file: `${OUTPUT_DIR}/server.js`,
453444
format: "esm"
454445
},
455446
external: [

yarn.lock

+4-4
Original file line numberDiff line numberDiff line change
@@ -4476,10 +4476,10 @@ history@^5.0.0:
44764476
dependencies:
44774477
"@babel/runtime" "^7.7.6"
44784478

4479-
history@^5.0.3:
4480-
version "5.0.3"
4481-
resolved "https://registry.npmjs.org/history/-/history-5.0.3.tgz#23d0b3046f695623c95a870506545e2b67e82edb"
4482-
integrity sha512-LoyCVOpCBkNAgrsdpTDZP77fys7lFDg8JdxTr7s6GHueZbTplKf8NAJu3y6/QuJIjk6TFsEGrhtILVP814X8+A==
4479+
history@^5.1.0:
4480+
version "5.1.0"
4481+
resolved "https://registry.yarnpkg.com/history/-/history-5.1.0.tgz#2e93c09c064194d38d52ed62afd0afc9d9b01ece"
4482+
integrity sha512-zPuQgPacm2vH2xdORvGGz1wQMuHSIB56yNAy5FnLuwOwgSYyPKptJtcMm6Ev+hRGeS+GzhbmRacHzvlESbFwDg==
44834483
dependencies:
44844484
"@babel/runtime" "^7.7.6"
44854485

0 commit comments

Comments
 (0)