Skip to content

fix(android): re-patch signing passwords before bubblewrap build - #148

Merged
izzywdev merged 1 commit into
masterfrom
claude/fuzefront-pwa-android-apk-0cj8ql
Jul 3, 2026
Merged

fix(android): re-patch signing passwords before bubblewrap build#148
izzywdev merged 1 commit into
masterfrom
claude/fuzefront-pwa-android-apk-0cj8ql

Conversation

@izzywdev

@izzywdev izzywdev commented Jul 3, 2026

Copy link
Copy Markdown
Owner

Summary

Run #5 revealed the next layer: bubblewrap update clears signingKey.storePassword and keyPassword from twa-manifest.json when it regenerates the Android project. By the time bubblewrap build runs, the password fields are empty so it falls back to an interactive TTY prompt — which fails with exit 130 (no TTY in CI).

Fix: re-apply both passwords from the GitHub Secrets immediately before calling bubblewrap build, using the same node patch already used in the scaffold step.

Progress so far

Run Step that failed Fix
#1 bubblewrap init hung on JDK prompt Pre-configure ~/.bubblewrap/config.json + Android SDK action
#2 Invalid URL from local path passed as manifest URL Switch to bubblewrap update
#3 Icon 404 from app.fuzefront.com Serve icons from local HTTP server
#4 Web manifest 404 / JSON parse error Add frontend/public/manifest.webmanifest, patch webManifestUrl
#5 bubblewrap build prompts for keystore password (exit 130) This PR — re-patch passwords before build

Test plan


Generated by Claude Code

bubblewrap update clears signingKey.storePassword and keyPassword
from twa-manifest.json when it regenerates the Android project, so
bubblewrap build subsequently prompts interactively (exit 130, no
TTY). Re-apply the passwords from secrets immediately before build.

Co-Authored-By: Claude <noreply@anthropic.com>
@izzywdev
izzywdev marked this pull request as ready for review July 3, 2026 07:49
@izzywdev
izzywdev merged commit 3dc2b34 into master Jul 3, 2026
26 of 32 checks passed
@izzywdev
izzywdev deleted the claude/fuzefront-pwa-android-apk-0cj8ql branch July 3, 2026 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants