Merge branch 'mrschmidt/standalonescript' into mrschmidt/apiextract

schmidt-sebastian · schmidt-sebastian · commit e1091672a396 · 2020-11-19T16:52:04.000-07:00
diff --git a/.changeset/blue-geese-approve.md b/.changeset/blue-geese-approve.md
diff --git a/.changeset/curvy-planets-sneeze.md b/.changeset/curvy-planets-sneeze.md
diff --git a/.changeset/many-snails-kneel.md b/.changeset/many-snails-kneel.md
diff --git a/.changeset/proud-pigs-work.md b/.changeset/proud-pigs-work.md
@@ -0,0 +1,6 @@
+---
+"@firebase/firestore": patch
+"@firebase/webchannel-wrapper": patch
+---
+
+Fix an issue that prevented `experimentalAutoDetectLongPolling` from working correctly.
diff --git a/integration/firebase/package.json b/integration/firebase/package.json
@@ -7,7 +7,7 @@
     "test:ci": "node ../../scripts/run_tests_in_ci.js -s test"
   },
   "devDependencies": {
-    "firebase": "8.0.2",
+    "firebase": "8.1.0",
     "@types/chai": "4.2.14",
     "@types/mocha": "7.0.2",
     "chai": "4.2.0",
diff --git a/integration/firestore/package.json b/integration/firestore/package.json
@@ -15,7 +15,7 @@
   },
   "devDependencies": {
     "@firebase/app": "0.6.13",
-    "@firebase/firestore": "2.0.2",
+    "@firebase/firestore": "2.0.3",
     "@types/mocha": "7.0.2",
     "gulp": "4.0.2",
     "gulp-filter": "6.0.0",
diff --git a/integration/messaging/package.json b/integration/messaging/package.json
@@ -9,7 +9,7 @@
     "test:manual": "mocha --exit"
   },
   "devDependencies": {
-    "firebase": "8.0.2",
+    "firebase": "8.1.0",
     "chai": "4.2.0",
     "chromedriver": "86.0.0",
     "express": "4.17.1",
diff --git a/packages-exp/firebase-exp/package.json b/packages-exp/firebase-exp/package.json
@@ -41,7 +41,7 @@
     "@firebase/app-compat": "0.0.900",
     "@firebase/auth-exp": "0.0.900",
     "@firebase/functions-exp": "0.0.900",
-    "@firebase/firestore": "2.0.2",
+    "@firebase/firestore": "2.0.3",
     "@firebase/performance-exp": "0.0.900",
     "@firebase/remote-config-exp": "0.0.900"
   },
diff --git a/packages/database/CHANGELOG.md b/packages/database/CHANGELOG.md
@@ -1,5 +1,11 @@
 # Unreleased
 
+## 0.8.0
+
+### Minor Changes
+
+- [`34973cde2`](https://github.com/firebase/firebase-js-sdk/commit/34973cde218e570baccd235d5bb6c6146559f80b) [#3812](https://github.com/firebase/firebase-js-sdk/pull/3812) - Add a `get` method for database queries that returns server result when connected
+
 ## 0.7.1
 
 ### Patch Changes
diff --git a/packages/database/package.json b/packages/database/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@firebase/database",
-  "version": "0.7.1",
+  "version": "0.8.0",
   "description": "",
   "author": "Firebase <firebase-support@google.com> (https://firebase.google.com/)",
   "main": "dist/index.node.cjs.js",
diff --git a/packages/firebase/CHANGELOG.md b/packages/firebase/CHANGELOG.md
@@ -1,5 +1,17 @@
 # firebase
 
+## 8.1.0
+
+### Minor Changes
+
+- [`34973cde2`](https://github.com/firebase/firebase-js-sdk/commit/34973cde218e570baccd235d5bb6c6146559f80b) [#3812](https://github.com/firebase/firebase-js-sdk/pull/3812) - Add a `get` method for database queries that returns server result when connected
+
+### Patch Changes
+
+- Updated dependencies [[`6c6c49ad6`](https://github.com/firebase/firebase-js-sdk/commit/6c6c49ad6b3c3d66e9ecb8397c4ac39bea256e80), [`e0bf3f70b`](https://github.com/firebase/firebase-js-sdk/commit/e0bf3f70bf82f3587e60ab4484fe37d01cea0051), [`34973cde2`](https://github.com/firebase/firebase-js-sdk/commit/34973cde218e570baccd235d5bb6c6146559f80b)]:
+  - @firebase/firestore@2.0.3
+  - @firebase/database@0.8.0
+
 ## 8.0.2
 
 ### Patch Changes
diff --git a/packages/firebase/package.json b/packages/firebase/package.json
@@ -1,6 +1,6 @@
 {
   "name": "firebase",
-  "version": "8.0.2",
+  "version": "8.1.0",
   "description": "Firebase JavaScript library for web and Node.js",
   "author": "Firebase <firebase-support@google.com> (https://firebase.google.com/)",
   "license": "Apache-2.0",
@@ -48,8 +48,8 @@
     "@firebase/app": "0.6.13",
     "@firebase/app-types": "0.6.1",
     "@firebase/auth": "0.15.2",
-    "@firebase/database": "0.7.1",
-    "@firebase/firestore": "2.0.2",
+    "@firebase/database": "0.8.0",
+    "@firebase/firestore": "2.0.3",
     "@firebase/functions": "0.6.1",
     "@firebase/installations": "0.4.19",
     "@firebase/messaging": "0.7.3",
diff --git a/packages/firestore/CHANGELOG.md b/packages/firestore/CHANGELOG.md
@@ -1,5 +1,13 @@
 # @firebase/firestore
 
+## 2.0.3
+
+### Patch Changes
+
+- [`6c6c49ad6`](https://github.com/firebase/firebase-js-sdk/commit/6c6c49ad6b3c3d66e9ecb8397c4ac39bea256e80) [#4053](https://github.com/firebase/firebase-js-sdk/pull/4053) - Internal changes for the upcoming modular API.
+
+* [`e0bf3f70b`](https://github.com/firebase/firebase-js-sdk/commit/e0bf3f70bf82f3587e60ab4484fe37d01cea0051) [#4080](https://github.com/firebase/firebase-js-sdk/pull/4080) (fixes [#4071](https://github.com/firebase/firebase-js-sdk/issues/4071)) - Fixes a regression introduced in v8.0.2 that returned invalid values for `DocumentChange.newIndex`.
+
 ## 2.0.2
 
 ### Patch Changes
diff --git a/packages/firestore/package.json b/packages/firestore/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@firebase/firestore",
-  "version": "2.0.2",
+  "version": "2.0.3",
   "engines": {
     "node": "^8.13.0 || >=10.10.0"
   },
diff --git a/packages/rules-unit-testing/CHANGELOG.md b/packages/rules-unit-testing/CHANGELOG.md
@@ -1,5 +1,12 @@
 # @firebase/rules-unit-testing
 
+## 1.1.2
+
+### Patch Changes
+
+- Updated dependencies [[`34973cde2`](https://github.com/firebase/firebase-js-sdk/commit/34973cde218e570baccd235d5bb6c6146559f80b)]:
+  - firebase@8.1.0
+
 ## 1.1.1
 
 ### Patch Changes
diff --git a/packages/rules-unit-testing/package.json b/packages/rules-unit-testing/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@firebase/rules-unit-testing",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "description": "",
   "author": "Firebase <firebase-support@google.com> (https://firebase.google.com/)",
   "main": "dist/index.cjs.js",
@@ -19,7 +19,7 @@
   },
   "license": "Apache-2.0",
   "dependencies": {
-    "firebase": "8.0.2",
+    "firebase": "8.1.0",
     "@firebase/logger": "0.2.6",
     "@firebase/util": "0.3.4",
     "request": "2.88.2"
diff --git a/packages/rxfire/package.json b/packages/rxfire/package.json
@@ -40,7 +40,7 @@
     "rxjs": "6.x.x"
   },
   "devDependencies": {
-    "firebase": "8.0.2",
+    "firebase": "8.1.0",
     "rollup": "2.33.2",
     "@rollup/plugin-commonjs": "15.1.0",
     "@rollup/plugin-node-resolve": "9.0.0",
diff --git a/scripts/size_report/report_binary_size.ts b/scripts/size_report/report_binary_size.ts
@@ -15,7 +15,7 @@
  * limitations under the License.
  */
 
-import { resolve } from 'path';
+import { resolve as pathResolve } from 'path';
 import * as fs from 'fs';
 import { execSync } from 'child_process';
 import * as terser from 'terser';
@@ -25,6 +25,8 @@ import {
   RequestBody,
   RequestEndpoint
 } from './size_report_helper';
+import * as rollup from 'rollup';
+import commonjs from '@rollup/plugin-commonjs';
 
 interface Report {
   sdk: string;
@@ -38,7 +40,7 @@ interface BinarySizeRequestBody extends RequestBody {
 // CDN scripts
 function generateReportForCDNScripts(): Report[] {
   const reports = [];
-  const firebaseRoot = resolve(__dirname, '../../packages/firebase');
+  const firebaseRoot = pathResolve(__dirname, '../../packages/firebase');
   const pkgJson = require(`${firebaseRoot}/package.json`);
 
   const special_files = [
@@ -103,8 +105,16 @@ async function generateReportForNPMPackages(): Promise<Report[]> {
 
       for (const field of fields) {
         if (packageJson[field]) {
-          const filePath = `${path}/${packageJson[field]}`;
-          const rawCode = fs.readFileSync(filePath, 'utf-8');
+          const filePath = pathResolve(path, packageJson[field]);
+          // Need to create a bundle and get the size of the bundle instead of reading the size of the file directly.
+          // It is because some packages might be split into multiple files in order to share code between entry points.
+          const bundle = await rollup.rollup({
+            input: filePath,
+            plugins: [commonjs()]
+          });
+
+          const { output } = await bundle.generate({ format: 'es' });
+          const rawCode = output[0].code;
 
           // remove comments and whitespaces, then get size
           const { code } = await terser.minify(rawCode, {
@@ -122,7 +132,6 @@ async function generateReportForNPMPackages(): Promise<Report[]> {
 
       resolve();
     });
-
     taskPromises.push(promise);
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@firebase/database",`
`3`		`- "version": "0.7.1",`
	`3`	`+ "version": "0.8.0",`
`4`	`4`	`"description": "",`
`5`	`5`	`"author": "Firebase <[email protected]> (https://firebase.google.com/)",`
`6`	`6`	`"main": "dist/index.node.cjs.js",`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@firebase/firestore",`
`3`		`- "version": "2.0.2",`
	`3`	`+ "version": "2.0.3",`
`4`	`4`	`"engines": {`
`5`	`5`	`"node": "^8.13.0 \|\| >=10.10.0"`
`6`	`6`	`},`