Skip to content

fix(mobile): pin Capacitor 8 packages so CI cap add android works - #417

Merged
crs48 merged 1 commit into
mainfrom
claude/upbeat-napier-8b894c
Jul 8, 2026
Merged

fix(mobile): pin Capacitor 8 packages so CI cap add android works#417
crs48 merged 1 commit into
mainfrom
claude/upbeat-napier-8b894c

Conversation

@crs48

@crs48 crs48 commented Jul 8, 2026

Copy link
Copy Markdown
Owner

Summary

The dispatch-only mobile-e2e.yml workflow (exploration 0238, un-broken in #414) failed at Add Android project: apps/mobile had no Capacitor dependencies and every script shelled out to npx --yes @capacitor/cli@latest, but Capacitor 8's cap add android requires @capacitor/android installed in the project.

  • Pin @capacitor/cli, @capacitor/core, @capacitor/android, @capacitor/ios at 8.4.1 as devDependencies of xnet-mobile-shell (private, no changeset needed).
  • Switch all scripts from npx-latest to the pinned CLI (pnpm exec cap …) — reproducible and no surprise majors.
  • Update the README prerequisites section, which documented the npx approach.

Verification

gh workflow run mobile-e2e.yml --ref claude/upbeat-napier-8b894c -f platform=androidrun 28968475239 success: every step green through the Run Maestro flows TODO, including [success] android platform added! and Capacitor sync. Emulator/Maestro wiring remains the documented TODO.

🤖 Generated with Claude Code

@crs48
crs48 temporarily deployed to pr-417 July 8, 2026 19:12 — with GitHub Actions Inactive
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

✓ Changelog fragment found — thanks!

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Preview removed for PR #417.

github-actions Bot added a commit that referenced this pull request Jul 8, 2026
Capacitor 8's `cap add android` requires @capacitor/android installed in
the project, so the npx-latest scripts fail in the mobile-e2e workflow.
Pin @capacitor/{cli,core,android,ios}@8.4.1 as devDependencies of
xnet-mobile-shell and switch the scripts to the pinned CLI (pnpm exec cap).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Signed-off-by: xNet Test <test@xnet.dev>
@crs48
crs48 force-pushed the claude/upbeat-napier-8b894c branch from e04067a to c421534 Compare July 8, 2026 19:48
@crs48
crs48 temporarily deployed to pr-417 July 8, 2026 19:48 — with GitHub Actions Inactive
@crs48 crs48 added the skip-changelog Exclude this PR from the changelog label Jul 8, 2026
github-actions Bot added a commit that referenced this pull request Jul 8, 2026
@crs48
crs48 merged commit 3b3977e into main Jul 8, 2026
14 of 15 checks passed
@crs48
crs48 deleted the claude/upbeat-napier-8b894c branch July 8, 2026 20:09
github-actions Bot added a commit that referenced this pull request Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Exclude this PR from the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant