Skip to content

feat(updater): enable GitHub Releases auto-updates#6

Merged
CodeWithDennis merged 3 commits into
mainfrom
feat/github-auto-updater
Jul 26, 2026
Merged

feat(updater): enable GitHub Releases auto-updates#6
CodeWithDennis merged 3 commits into
mainfrom
feat/github-auto-updater

Conversation

@CodeWithDennis

@CodeWithDennis CodeWithDennis commented Jul 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • Enable NativePHP / electron-updater against public GitHub Releases (GITHUB_PRIVATE=false).
  • Bake updater config into CI native builds and document macOS signing limits.
  • Keep write tokens out of the build job so they cannot end up in updater/publish metadata.

Security

  • No GITHUB_TOKEN / GH_TOKEN in the build job (we use native:build, not --publish).
  • Bundled .env still strips GITHUB_* via cleanup_env_keys.
  • Client updater needs no token while the repo/releases stay public.

Test plan

  • Confirm repo is public before merging/releasing
  • Review workflow: updater env vars present, no write-token secrets in build job
  • Tag a v* release (or workflow_dispatch) and confirm builds attach *.yml / installers
  • Install a Windows build, publish a newer tag, confirm update check/download
  • Note: macOS auto-update still requires Apple Developer ID signing + notarization

Wire NativePHP electron-updater to GitHub Releases, keep write tokens out of the build env, and scan dist artifacts for leaked GitHub token patterns before upload.
A failing grep-based scan can print matches into public Actions logs, which is worse than the protection it adds when no write token is present in the build env.
Separate arm64/x64 jobs each wrote latest-mac.yml; merging artifacts left only one arch in the release feed. NativePHP mac+all runs electron-builder once for both arches.
@CodeWithDennis
CodeWithDennis merged commit 3403882 into main Jul 26, 2026
4 checks passed
@CodeWithDennis
CodeWithDennis deleted the feat/github-auto-updater branch July 26, 2026 10:26
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.

1 participant