Skip to content

🤖 fix: change Electron appId to com.coder.mux#2586

Closed
ThomasK33 wants to merge 1 commit intomainfrom
electron-app-g3q1
Closed

🤖 fix: change Electron appId to com.coder.mux#2586
ThomasK33 wants to merge 1 commit intomainfrom
electron-app-g3q1

Conversation

@ThomasK33
Copy link
Member

Summary

Change the Electron Builder application identifier from com.mux.app to com.coder.mux.

Background

The current app ID uses the legacy com.mux.app identifier. This PR updates branding alignment to com.coder.mux.

Implementation

  • Updated build.appId in package.json:
    • com.mux.appcom.coder.mux

Validation

  • make static-check

Risks

Potentially breaking for existing installs. Changing Electron Builder appId can affect OS-level app identity and update continuity:

  • macOS: bundle identifier (CFBundleIdentifier)
  • Windows: AUMID / NSIS identity

Existing installations that were distributed with com.mux.app may require reinstall/migration handling instead of seamless in-place updates.


Generated with mux • Model: openai:gpt-5.3-codex • Thinking: xhigh • Cost: $0.28

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b6c2168926

ℹ️ 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".

],
"build": {
"appId": "com.mux.app",
"appId": "com.coder.mux",

Choose a reason for hiding this comment

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

P1 Badge Keep Electron appId stable across released builds

Changing build.appId changes the OS-level application identity (bundle ID/AUMID), so users already installed under com.mux.app can lose seamless in-place upgrade behavior and may end up with parallel installs or broken update continuity on macOS/Windows. This commit updates the identifier but does not include any migration strategy, so existing production installs are at risk during the next upgrade.

Useful? React with 👍 / 👎.

@ThomasK33
Copy link
Member Author

Thanks for the callout.

This appId change is intentional in this PR, and I’ve marked it as potentially breaking in the PR body. I agree this can break in-place update continuity for users already installed under com.mux.app.

Given that migration across OS-level app identity is a release/product decision (not a simple code-only fix), I’m pausing for maintainer direction on whether we should:

  1. keep this change and ship with a one-time manual reinstall path, or
  2. revert and keep com.mux.app for update continuity.

@ThomasK33 ThomasK33 closed this Mar 2, 2026
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