Skip to content

Commit b3caa51

Browse files
Add NodeJS support to Storage V8 (#5149)
1 parent fb3e359 commit b3caa51

File tree

3 files changed

+30
-9
lines changed

3 files changed

+30
-9
lines changed

.changeset/curly-shrimps-wait.md

+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
@firebase/storage: minor
3+
---
4+
5+
Add NodeJS support to Cloud Storage for Firebase. This release changes the `main` field in package.json to point to a Node specific build. If you are building a bundle for borwser usage, please make sure that your bundler uses the `browser` field (the default).

packages/storage/package.json

+5-5
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,10 @@
33
"version": "0.5.6",
44
"description": "",
55
"author": "Firebase <[email protected]> (https://firebase.google.com/)",
6-
"main": "dist/index.cjs.js",
7-
"module": "dist/index.esm.js",
8-
"browser": "dist/index.esm.js",
9-
"esm2017": "dist/index.esm2017.js",
6+
"main": "dist/index.node.cjs.js",
7+
"module": "dist/index.browser.esm.js",
8+
"browser": "dist/index.browser.esm.js",
9+
"esm2017": "dist/index.browser.esm2017.js",
1010
"files": [
1111
"dist",
1212
"exp/dist"
@@ -66,4 +66,4 @@
6666
"url": "https://github.com/firebase/firebase-js-sdk/issues"
6767
},
6868
"typings": "dist/index.d.ts"
69-
}
69+
}

packages/storage/rollup.config.js

+20-4
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,9 @@ const { generateAliasConfig } = require('./rollup.shared');
2626
const deps = Object.keys(
2727
Object.assign({}, pkg.peerDependencies, pkg.dependencies)
2828
);
29+
30+
const nodeDeps = [...deps, 'util'];
31+
2932
/**
3033
* ES5 Builds
3134
*/
@@ -39,10 +42,7 @@ const es5BuildPlugins = [
3942
const es5Builds = [
4043
{
4144
input: './index.ts',
42-
output: [
43-
{ file: pkg.main, format: 'cjs', sourcemap: true },
44-
{ file: pkg.module, format: 'es', sourcemap: true }
45-
],
45+
output: { file: pkg.module, format: 'es', sourcemap: true },
4646
plugins: [alias(generateAliasConfig('browser')), ...es5BuildPlugins],
4747
external: id => deps.some(dep => id === dep || id.startsWith(`${dep}/`)),
4848
treeshake: {
@@ -67,6 +67,22 @@ const es2017BuildPlugins = [
6767
];
6868

6969
const es2017Builds = [
70+
// Node
71+
{
72+
input: './index.ts',
73+
output: {
74+
file: pkg.main,
75+
format: 'cjs',
76+
sourcemap: true
77+
},
78+
plugins: [alias(generateAliasConfig('node')), ...es2017BuildPlugins],
79+
external: id =>
80+
nodeDeps.some(dep => id === dep || id.startsWith(`${dep}/`)),
81+
treeshake: {
82+
moduleSideEffects: false
83+
}
84+
},
85+
// Browser
7086
{
7187
input: './index.ts',
7288
output: {

0 commit comments

Comments
 (0)