Skip to content

chore: migrate Android app to Capacitor 8 - #25

Merged
Inmerson merged 1 commit into
mainfrom
security/capacitor-8-upgrade
Sep 12, 2026
Merged

Inmerson merged 1 commit into
mainfrom
security/capacitor-8-upgrade

Conversation

@Inmerson

@Inmerson Inmerson commented Sep 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Migrates the Android app from Capacitor 6 to Capacitor 8.
  • Uses @capacitor/android / @capacitor/core 8.5.2.
  • Pins @capacitor/cli to exact 8.4.3 because the current 8.5.x CLI line carries an xcode -> uuid advisory that is irrelevant to this Android-only project but still affects npm audit.
  • Updates Android build configuration to the Capacitor 8 requirements: AGP 8.13.0, SDK 36, Java 21, current AndroidX/Cordova versions. Existing Gradle wrapper 8.14.3 was already correct.

Verification

Fresh verification on this exact commit:

  • Node 22 TypeScript typecheck: passed.
  • Vitest: 5/5 files, 12/12 tests passed.
  • Vite production build: passed.
  • cap sync android: passed with CLI 8.4.3.
  • Android assembleDebug --no-daemon: BUILD SUCCESSFUL, 93 tasks.
  • npm audit --json: 0 vulnerabilities.
  • npm ci --dry-run --ignore-scripts: passed.
  • git diff --check: passed.
  • Independent review: APPROVED, no Critical/Important/Minor findings.
  • GitHub CI: passed.

Dependency context

Dependency refresh PR #23 is merged into main. This PR is now based directly on main and contains only the Capacitor/Android migration on top of that merged refresh.

Copilot AI lite review requested due to automatic review settings September 12, 2026 23:28
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Inmerson
Inmerson changed the base branch from security/dependency-refresh-2026-09-12 to main September 12, 2026 23:31
@Inmerson
Inmerson merged commit 0809cb3 into main Sep 12, 2026
1 check passed
@Inmerson
Inmerson deleted the security/capacitor-8-upgrade branch September 12, 2026 23:54
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