feat: android - no bundle compress (#15190)

[Cal-L]

Description

Reference to original PR - #15190

Related issues

Fixes: #15560

Manual testing steps

1. Go to this page... 2.

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I’ve followed MetaMask Contributor Docs and MetaMask Mobile Coding
  Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

Description

Related issues

Fixes:

Manual testing steps

1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I’ve followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

[github-actions]

CLA Signature Action:

Thank you for your submission, we really appreciate it. We ask that you read and sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just by adding a comment to this pull request with this exact sentence:

I have read the CLA Document and I hereby sign the CLA

By commenting with the above message you are agreeing to the terms of the CLA. Your account will be recorded as agreeing to our CLA so you don't need to sign it again for future contributions to this repository.

0 out of 1 committers have signed the CLA.
❌ @stas-kravchuk

[github-actions]

Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 80c5ab7
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/07c986f3-326f-417b-ac7d-25bf004a3ad5

Note

• You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

Tip

• Check the documentation if you have any doubts on how to understand the failure on bitrise

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[Cal-L]

App start up performance test #1 - https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/f1d83f03-2150-4927-9cdc-caffdac7b3d0

App start up performance test #2 - https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/1f53aebb-e93c-486f-839b-681ebc042bde

Compared 2 runs on this branch as well as main and here are the results that we're seeing

Before (before optimization)

• Measure Fresh Install Cold Start
• Not exceed 500ms
• Actual = 283ms, 287ms
  After (w/ no compress)
• Measure Fresh Install Cold Start
• Not exceed 500ms
• Actual = 292ms, 347ms

For context, the numbers that these are derived from comes from https://github.com/oblador/react-native-performance. More specifically, we're taking the max value between the nativeLaunch and runJsBundle time here - https://github.com/MetaMask/metamask-mobile/blob/main/app/core/Performance/Performance.ts#L21