Skip to content

feat: switch to git-based updates and simplify PR checks#529

Merged
idofrizler merged 2 commits intostagingfrom
feature/fix-upgrades
Apr 6, 2026
Merged

feat: switch to git-based updates and simplify PR checks#529
idofrizler merged 2 commits intostagingfrom
feature/fix-upgrades

Conversation

@idofrizler
Copy link
Copy Markdown
Collaborator

Summary

Switches Cooper from GitHub Releases-based update checking to a simpler package.json-based approach, and simplifies PR CI since we no longer distribute packaged binaries.

Changes

Version check (src/main/main.ts)

  • Now fetches package.json from main branch via raw.githubusercontent.com instead of querying the GitHub Releases API
  • Removed GitHubRelease interface and release asset detection logic

Update modal (UpdateAvailableModal.tsx)

  • Replaced "Open Releases" button with a copyable git pull && npm run dist command
  • Users click the command block or "Copy Command" button to copy to clipboard
  • Removed dependency on electronAPI.updates.openDownloadUrl

PR checks (.github/workflows/build-pr.yml)

  • Removed all packaging steps (electron-builder), artifact uploads, and Fedora build job
  • Keeps: tests, electron-vite build, and bundle size guardrail

Cleanup

  • Removed downloadUrl, releaseUrl, releaseNotes from update check response (preload + renderer)
  • Updated CONTRIBUTING.md to reflect simplified PR checks

Testing

  • npm test — 493 tests pass
  • npm run build — clean build
  • window.showUpdateModal() in DevTools to preview the new modal

Ido Frizler and others added 2 commits April 6, 2026 12:07
- Version check now fetches package.json from main branch instead of
  GitHub Releases API (releases will stop being published)
- Update modal shows "git pull && npm run dist" command with copy-to-
  clipboard instead of linking to releases page
- PR workflow simplified: removed packaging, artifact uploads, and
  Fedora build job — keeps tests, build, and bundle guardrail
- Cleaned up preload bridge types and App.tsx state (removed downloadUrl,
  releaseUrl, releaseNotes from update check response)
- Updated tests and CONTRIBUTING.md to match

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@idofrizler idofrizler merged commit eb1fab1 into staging Apr 6, 2026
7 checks passed
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