Adjust to new size analysis implementation.

[yifanyang]

The metrics backend for size analysis is revamped. We should be able to see some UI improvements (a demo report) in the size analysis report.

The new implementation returns 200 (OK) instead of 202 (ACCEPTED). Therefor the script should also treat 200 as success.

[changeset-bot]

⚠️ No Changeset found

Latest commit: 86f0fff

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[google-oss-bot]

Size Analysis Report

Affected Products

No changes between base commit (02dcd92) and head commit (2f2a311).

Test Logs

• Base (02dcd92): https://github.com/firebase/firebase-js-sdk/actions/runs/486701192
• Head (2f2a311): https://github.com/firebase/firebase-js-sdk/actions/runs/488579276

[yifanyang]

@Feiyang1 The original size report are missing. It looks like the that job (Health Metrics / Binary Size (pull_request)) actually failed with:

(node:4844) UnhandledPromiseRejectionWarning: Error: Could not resolve entry module (packages/storage/exp/dist/index.browser.cjs.js).
    at error (/home/runner/work/firebase-js-sdk/firebase-js-sdk/node_modules/rollup/dist/shared/rollup.js:5265:30)
    at ModuleLoader.loadEntryModule (/home/runner/work/firebase-js-sdk/firebase-js-sdk/node_modules/rollup/dist/shared/rollup.js:18454:20)
    at process._tickCallback (internal/process/next_tick.js:68:7)
    at Function.Module.runMain (internal/modules/cjs/loader.js:834:11)
    at Object.main (/home/runner/work/firebase-js-sdk/firebase-js-sdk/node_modules/ts-node/src/bin.ts:198:14)
    at Object.<anonymous> (/home/runner/work/firebase-js-sdk/firebase-js-sdk/node_modules/ts-node/src/bin-script.ts:5:1)
    at Module._compile (internal/modules/cjs/loader.js:778:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
    at Module.load (internal/modules/cjs/loader.js:653:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:593:12)

But it is still marked as success on CI. I recalled that you once had some fix for a similar issue. Is that in?

[Feiyang1]

Yes, #4010. Is it a case this PR doesn't cover? @yifanyang

[Feiyang1]

Should we still accept 202?

[yifanyang]

Yes, for now. The api for the old size report still returns 202. I'm planning to migrate it to 200 as well. I'll remove the 202 here (and a few other places in android repo) when that happens.

[yifanyang]

Yes, #4010. Is it a case this PR doesn't cover? @yifanyang

Got it. We can probably investigate on that unhandled reject later. I'm merging this one now to unblock other developers.