Fail fast when desktop build resources are missing - #1807
Merged
Conversation
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
executor-cloud | 0d4e1d4 | Aug 28 2026, 07:00 AM |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
executor-marketing | 0d4e1d4 | Commit Preview URL Branch Preview URL |
Aug 28 2026, 06:58 AM |
Contributor
Cloudflare previewTorn down — the PR is closed. |
@executor-js/cli
@executor-js/config
@executor-js/execution
@executor-js/sdk
@executor-js/codemode-core
@executor-js/runtime-quickjs
@executor-js/plugin-file-secrets
@executor-js/plugin-graphql
@executor-js/plugin-keychain
@executor-js/plugin-mcp
@executor-js/plugin-onepassword
@executor-js/plugin-openapi
executor
commit: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
The v1.6.1 desktop publish failed on both mac legs (x64 and arm64), twice, with:
The named
locale.pakis a red herring — it is just the first leaf codesignreached. The real cause is that
apps/desktop/build/entitlements.mac.plistand
build/icon.pngwere deleted by an unrelated PR, and the release tag wascut from a commit that did not have them.
The runner image, Xcode, electron 41.2.1, electron-builder 26.8.1 and the
signing identity are byte-identical between the last good release and the
failing one, so none of those are involved.
build/holds committed static inputs that no module imports. Deleting thembreaks no build, no type, and no test, so nothing caught it. The only signal
before the failure was a single line,
default Electron icon is used, fromthe missing icon — easy to miss, and it does not fail the build at all.
Fix
Give those files a reference so their absence is an error.
apps/desktop/scripts/check-build-resources.tsreads the paths out ofelectron-builder.config.tsand reports any input that is missing, empty,or not a plist. It exits non-zero with the actual cause.
apps/desktop/src/build-resources.test.tsasserts the same check passes.This runs in
bun run teston every PR, which is the gate that would haveblocked the deletion.
publish-desktop.ymlruns the check right after dependency install, so atag missing these files fails in seconds instead of ~8 minutes later inside
codesign with a misleading message.
No signing, packaging, or config behaviour changes.
Testing
bun ./scripts/check-build-resources.tspasses on a good tree, and exits 1naming both files when they are removed.
vitest run src/build-resources.test.tspasses, and fails withmissing: build/entitlements.mac.plistwhen the file is removed.typecheck,lint, andformatare clean.This PR does not by itself make v1.6.1 buildable — the tag still points at a
commit without the files. The assets are already restored on main (#1805), so
the release needs to be re-cut from a commit that includes them. Final proof
is the re-dispatched desktop build going green.