Skip to content

Commit

Permalink
fix: export CJS version for browser
Browse files Browse the repository at this point in the history
  • Loading branch information
MichaelDeBoey committed Aug 23, 2023
1 parent 5a3703c commit 18ed5c6
Show file tree
Hide file tree
Showing 7 changed files with 60 additions and 19 deletions.
9 changes: 9 additions & 0 deletions .changeset/afraid-coats-rescue.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
---
"@remix-run/web-blob": patch
"@remix-run/web-fetch": patch
"@remix-run/web-file": patch
"@remix-run/web-form-data": patch
"@remix-run/web-stream": patch
---

Export CJS version for browser
5 changes: 4 additions & 1 deletion packages/blob/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,10 @@
"exports": {
".": {
"types": "./dist/src/lib.d.ts",
"browser": "./src/lib.js",
"browser": {
"require": "./dist/src/lib.cjs",
"import": "./src/lib.js"
},
"require": "./dist/src/lib.node.cjs",
"import": "./src/lib.node.js"
}
Expand Down
5 changes: 4 additions & 1 deletion packages/fetch/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,10 @@
"exports": {
".": {
"types": "./dist/src/lib.node.d.ts",
"browser": "./src/lib.js",
"browser": {
"require": "./dist/lib.cjs",
"import": "./src/lib.js"
},
"require": "./dist/lib.node.cjs",
"import": "./src/lib.node.js"
},
Expand Down
45 changes: 31 additions & 14 deletions packages/fetch/rollup.config.js
Original file line number Diff line number Diff line change
@@ -1,18 +1,35 @@
import {builtinModules} from 'module';
import {dependencies} from './package.json';

export default {
input: 'src/lib.node.js',
output: {
file: 'dist/lib.node.cjs',
format: 'cjs',
esModule: false,
interop: false,
sourcemap: true,
preferConst: true,
exports: 'named',
// https://github.com/rollup/rollup/issues/1961#issuecomment-534977678
outro: 'exports = module.exports = Object.assign(fetch, exports);'
export default [
{
input: 'src/lib.js',
output: {
file: 'dist/lib.cjs',
format: 'cjs',
esModule: false,
interop: false,
sourcemap: true,
preferConst: true,
exports: 'named',
// https://github.com/rollup/rollup/issues/1961#issuecomment-534977678
outro: 'exports = module.exports = Object.assign(fetch, exports);'
},
external: [...builtinModules, ...Object.keys(dependencies)]
},
external: [...builtinModules, ...Object.keys(dependencies)]
};
{
input: 'src/lib.node.js',
output: {
file: 'dist/lib.node.cjs',
format: 'cjs',
esModule: false,
interop: false,
sourcemap: true,
preferConst: true,
exports: 'named',
// https://github.com/rollup/rollup/issues/1961#issuecomment-534977678
outro: 'exports = module.exports = Object.assign(fetch, exports);'
},
external: [...builtinModules, ...Object.keys(dependencies)]
},
];
5 changes: 4 additions & 1 deletion packages/file/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,10 @@
"exports": {
".": {
"types": "./dist/src/lib.d.ts",
"browser": "./src/lib.js",
"browser": {
"require": "./dist/src/lib.cjs",
"import": "./src/lib.js"
},
"require": "./dist/src/lib.node.cjs",
"node": "./src/lib.node.js"
}
Expand Down
5 changes: 4 additions & 1 deletion packages/form-data/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,10 @@
"exports": {
".": {
"types": "./dist/src/lib.d.ts",
"browser": "./src/lib.js",
"browser": {
"require": "./dist/src/lib.cjs",
"import": "./src/lib.js"
},
"require": "./dist/src/lib.node.cjs",
"import": "./src/lib.node.js"
}
Expand Down
5 changes: 4 additions & 1 deletion packages/stream/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,10 @@
"exports": {
".": {
"types": "./src/lib.d.ts",
"browser": "./src/lib.js",
"browser": {
"require": "./src/stream.cjs",
"import": "./src/lib.js"
},
"require": "./src/stream.cjs",
"import": "./src/lib.node.js"
}
Expand Down

0 comments on commit 18ed5c6

Please sign in to comment.