Skip to content

ci(lite): add signing and notarizing to macOS distribution#13490

Merged
slarse merged 5 commits intomasterfrom
add-macos-signing-to-lite-packaging
Apr 23, 2026
Merged

ci(lite): add signing and notarizing to macOS distribution#13490
slarse merged 5 commits intomasterfrom
add-macos-signing-to-lite-packaging

Conversation

@slarse
Copy link
Copy Markdown
Contributor

@slarse slarse commented Apr 23, 2026

🧢 Changes

Adds in signing and notarizing to the macOS distribution so we can more easily install it.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds macOS code signing and notarization support to the Lite Electron distribution to make macOS installs smoother (Gatekeeper/notarization compliant).

Changes:

  • Add an afterSign hook that runs @electron/notarize on macOS builds.
  • Enable hardened runtime and add macOS entitlements in the electron-builder config.
  • Provide signing/notarization secrets to the Lite GitHub Actions workflow and add the notarization dependency.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
pnpm-lock.yaml Locks @electron/notarize dependency and related resolution changes.
apps/lite/scripts/after-sign.js Implements the macOS notarization hook executed after signing.
apps/lite/resources/entitlements.mac.plist Adds hardened runtime entitlements used when signing the main app.
apps/lite/resources/entitlements.mac.inherit.plist Adds entitlements inherited by child processes/bundled components.
apps/lite/package.json Wires afterSign hook and sets macOS signing/hardened runtime options for electron-builder.
.github/workflows/lite.yml Supplies signing + notarization environment variables/secrets for macOS CI builds.
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Comment thread apps/lite/scripts/after-sign.js Outdated
Comment thread apps/lite/package.json Outdated
Comment thread apps/lite/scripts/after-sign.js Outdated
Comment thread .github/workflows/lite.yml
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Comment thread apps/lite/package.json
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Comment thread .github/workflows/lite.yml
@slarse slarse force-pushed the add-macos-signing-to-lite-packaging branch from 129401b to bbd4148 Compare April 23, 2026 10:22
Copilot AI review requested due to automatic review settings April 23, 2026 10:35
@slarse slarse force-pushed the add-macos-signing-to-lite-packaging branch from bbd4148 to 8c0ae80 Compare April 23, 2026 10:35
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 2 comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Comment thread pnpm-lock.yaml
Comment thread apps/lite/scripts/after-sign.js Outdated
@slarse slarse force-pushed the add-macos-signing-to-lite-packaging branch from 8c0ae80 to f8df5b1 Compare April 23, 2026 10:57
Copilot AI review requested due to automatic review settings April 23, 2026 11:08
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Comment thread .oxlintignore Outdated
Copilot AI review requested due to automatic review settings April 23, 2026 11:24
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

@slarse
Copy link
Copy Markdown
Contributor Author

slarse commented Apr 23, 2026

Appears to be working as intended so this is going in!

@slarse slarse merged commit 570b57b into master Apr 23, 2026
43 checks passed
@slarse slarse deleted the add-macos-signing-to-lite-packaging branch April 23, 2026 14:00
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.

3 participants