feat(build-ui-android): multi-platform CI, UI refresh, and Android preview scaffolding - #9
Conversation
…anced loading and status bar, and experimental Android Capacitor support This commit introduces a multi-platform CI workflow, refreshes the UI with a new loading spinner and StatusBar, and adds an experimental Capacitor-based Android scaffold. It also updates docs and packaging config to support ARM64 and Universal builds. No breaking changes expected; Android support is preview only.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - name: Package for ${{ matrix.platform }} | ||
| run: npm run package | ||
| env: | ||
| ELECTRON_BUILDER_ARCH: ${{ matrix.arch }} |
There was a problem hiding this comment.
Arch matrix ignored by package step
The matrix arch values never reach electron-builder: lines 53‑56 run npm run package with an ELECTRON_BUILDER_ARCH env var that electron-builder does not consume (its CLI only switches arch via --x64/--arm64/--universal, per npx electron-builder --help). As a result the mac job still produces a host‑arch x64 build even though the matrix advertises arch: universal, so universal/arm64 artifacts will never be emitted.
Useful? React with 👍 / 👎.
| - os: ubuntu-latest | ||
| platform: linux | ||
| arch: x64 |
There was a problem hiding this comment.
Linux arm64 artifacts never built
The new workflow only defines a single Linux matrix entry with arch: x64 (lines 26‑28), but package.json now declares Linux targets for both x64 and arm64 (lines 123‑127). With no arm64 job, the promised arm64 .AppImage/.deb outputs are never produced or uploaded, leaving the new multi‑arch targets uncovered.
Useful? React with 👍 / 👎.
Summary
This PR enables multi-platform packaging for the Electron app, refreshes the UI with a new loading spinner and a streamlined status bar, and introduces an experimental Android Capacitor integration.
Details
Notes: Android support is experimental; core features for SSH/MCP remain desktop-focused.
Warning: Task VM test is not passing, cto.new will perform much better if you fix the setup