Skip to content

Improve Electron update CTA flow#1875

Draft
ignaciojimenezr wants to merge 1 commit intoMCPJam:mainfrom
ignaciojimenezr:update-on-electron
Draft

Improve Electron update CTA flow#1875
ignaciojimenezr wants to merge 1 commit intoMCPJam:mainfrom
ignaciojimenezr:update-on-electron

Conversation

@ignaciojimenezr
Copy link
Copy Markdown
Collaborator

@ignaciojimenezr ignaciojimenezr commented Apr 20, 2026

This PR makes the Electron app show the update button earlier.

Before this change, the button only appeared after the update had already been downloaded.
Now, the app can show the button as soon as it knows a newer version exists.

It also keeps the existing restart flow:

  • if the update is already downloaded, clicking the button restarts into the new version
  • if the update is still in progress, the app remembers the user's intent and installs once the download finishes

Why this changed

The old behavior was too late.
Users could be on an old version and see nothing, even though an update was already available.

Extra fixes included

  • expanded the Vite fs allow-list so electron:dev boots correctly

Checks run

  • desktop updater tests
  • client hook tests
  • server test for the sandbox proxy route
  • full npm run build

@chelojimenez
Copy link
Copy Markdown
Contributor

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

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