Skip to content

Conversation

@Lemonexe
Copy link
Contributor

@Lemonexe Lemonexe commented Jan 4, 2026

Explicitly delete nullish values from macOS appPlist, because it seems that the plist library handles nullish values differently than the previous implementation (before #8890)

Resolves #9480

I confirmed this fix with local patch on a real app built on macOS, and added test coverage.
The test passes as is, and fails if I comment out the code change ✔️

@changeset-bot
Copy link

changeset-bot bot commented Jan 4, 2026

🦋 Changeset detected

Latest commit: 703beee

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 8 packages
Name Type
app-builder-lib Patch
dmg-builder Patch
electron-builder-squirrel-windows Patch
electron-builder Patch
electron-forge-maker-appimage Patch
electron-forge-maker-nsis-web Patch
electron-forge-maker-nsis Patch
electron-forge-maker-snap Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@mmaietta
Copy link
Collaborator

mmaietta commented Jan 4, 2026

Nice work! Thank you for your contribution!

@mmaietta mmaietta merged commit cb5b9c6 into electron-userland:master Jan 6, 2026
55 of 56 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cannot unset default macOS appPlist entries via extendInfo

2 participants