Skip to content

chore: configure firebase app distribution for dev debug builds on demand (WPB-27685) - #5196

Draft
yamilmedina wants to merge 8 commits into
developfrom
chore/gradle-appdistribution-config
Draft

chore: configure firebase app distribution for dev debug builds on demand (WPB-27685)#5196
yamilmedina wants to merge 8 commits into
developfrom
chore/gradle-appdistribution-config

Conversation

@yamilmedina

@yamilmedina yamilmedina commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

https://wearezeta.atlassian.net/browse/WPB-27685


PR Submission Checklist for internal contributors

  • The PR Title

    • conforms to the style of semantic commits messages¹ supported in Wire's Github Workflow²
    • contains a reference JIRA issue number like SQPIT-764
    • answers the question: If merged, this PR will: ... ³
  • The PR Description

    • is free of optional paragraphs and you have filled the relevant parts to the best of your ability

What's new in this PR?

  • add firebase app distribution configs
    • this is only for CI, so local development workflows are unaffected (no need to config credentials for example)
  • add manual gh workflow to build
    • from a PR latest successful build apk
    • from a branch to build and deploy
  • align package id to correct existing firebase config
Screenshot 2026-08-20 at 15 11 34

PR Post Submission Checklist for internal contributors (Optional)

  • Wire's Github Workflow has automatically linked the PR to a JIRA issue

PR Post Merge Checklist for internal contributors

  • If any soft of configuration variable was introduced by this PR, it has been added to the relevant documents and the CI jobs have been updated.

References
  1. https://sparkbox.com/foundry/semantic_commit_messages
  2. https://github.com/wireapp/.github#usage
  3. E.g. feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764.

@yamilmedina
yamilmedina requested a review from a team as a code owner August 20, 2026 13:09
@yamilmedina
yamilmedina requested review from MohamadJaara and valerio-bettini and removed request for a team August 20, 2026 13:09
@AndroidBob

AndroidBob commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

New ADR(s) in this PR 📚:

14. Firebase App Distribution for Dev builds

Date: 2026-08-20

Status

Accepted

Context

We need to distribute DevDebug APKs to testers without adding App Distribution uploads to the
shared build workflow. Firebase App Distribution requires the APK package name to match the
selected Firebase Android app.

Decision

Use the Firebase CLI from the distribution workflow. CI writes
FIREBASE_APP_DISTRIBUTION_SERVICE_ACCOUNT_JSON to a temporary credentials file and passes it as
Application Default Credentials. Local builds do not include App Distribution tooling or tasks.

Use a manual Distribute Dev Build workflow for uploads. The workflow accepts a PR number to reuse
the existing DevDebug artifact from its successful Develop PR build. Without a PR number, it builds
DevDebug from the branch selected when the workflow is dispatched.

Dev's base application ID is com.wire.android.dev, so DevDebug uses
com.wire.android.dev.debug, matching the Firebase Android app used for distribution.

Consequences

  • App Distribution is opt-in and does not alter the shared build workflow.
  • The Dev package-ID change does not change firebase_app_id, firebase_push_sender_id, or the
    Firebase project used for push notifications.
  • Existing com.waz.zclient.dev.debug installations do not update in place and should be treated
    as ephemeral Dev installations.

@yamilmedina
yamilmedina requested review from Garzas and removed request for valerio-bettini August 20, 2026 13:15
@yamilmedina

Copy link
Copy Markdown
Collaborator Author

@claude review

@yamilmedina
yamilmedina marked this pull request as draft August 20, 2026 13:25
@codecov

codecov Bot commented Aug 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 52.21%. Comparing base (10f530d) to head (b883754).
⚠️ Report is 4 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #5196      +/-   ##
===========================================
+ Coverage    52.19%   52.21%   +0.02%     
===========================================
  Files          668      668              
  Lines        23972    23987      +15     
  Branches      3950     3952       +2     
===========================================
+ Hits         12512    12526      +14     
- Misses       10300    10301       +1     
  Partials      1160     1160              

see 1 file with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 10f530d...b883754. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread .github/workflows/distribute-dev-firebase.yml Fixed
Comment thread .github/workflows/distribute-dev-firebase.yml Fixed
@github-actions

github-actions Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Test Results

2 271 tests  +4   2 271 ✅ +4   2m 25s ⏱️ -41s
  250 suites ±0       0 💤 ±0 
  250 files   ±0       0 ❌ ±0 

Results for commit b883754. ± Comparison against base commit 04a1761.

♻️ This comment has been updated with latest results.

@wireapp wireapp deleted a comment from github-actions Bot Aug 20, 2026
@github-actions

Copy link
Copy Markdown
Contributor

APKs built during tests are available here. Scroll down to Artifacts!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants