Skip to content

fix(unity-builder): retry mac builds on Unity's transient licensing-client signature error - #226

Merged
frostebite merged 1 commit into
mainfrom
fix/mac-licensing-client-retry
Aug 28, 2026
Merged

fix(unity-builder): retry mac builds on Unity's transient licensing-client signature error#226
frostebite merged 1 commit into
mainfrom
fix/mac-licensing-client-retry

Conversation

@frostebite

Copy link
Copy Markdown
Member

Summary

  • Fixes the flaky macOS build failures seen on Thin wrapper: invoke game-ci/cli as a subprocess unity-builder#844's CI (the "Cancelled after 65m" jobs, plus a separate one-off failure): Unity's Licensing Client occasionally fails its own codesign verification (Error: Code 10 while verifying Licensing Client signature) right after a fresh Unity Hub install on a GitHub-hosted runner, before any real build work starts.
  • This is a known, transient Unity/macOS quirk — not something fixable in Unity itself — but it produces a false-negative CI signal unrelated to whether the tool's own build logic is correct.
  • MacBuilder.run now retries the whole invocation up to 3 times (10s backoff) only when the output matches this specific known-transient error pattern. Any other failure (a real compile error, etc.) still fails immediately on the first attempt — this can't mask genuine build breakage.

Test plan

  • New mac-builder.test.ts: succeeds without retry, fails immediately on unrelated errors, retries+recovers on the transient pattern, gives up after 3 attempts and returns the real exit code
  • vitest run — 458/458 pass (4 new)
  • tsc --noEmit — clean

…lient signature error

Root cause of the "Cancelled after 65m" and one-off macOS build
failures seen on game-ci/unity-builder#844's CI: Unity's own Licensing
Client occasionally fails its own codesign verification
("Error: Code 10 while verifying Licensing Client signature") right
after a fresh Unity Hub install on a GitHub-hosted runner, before any
real build work starts. It's a known, transient Unity/macOS flake, not
something this tool can fix in Unity itself - but failing an entire
build (and the PR check with it) on a licensing hiccup unrelated to the
actual build's correctness is exactly the false-negative CI signal we
don't want: it tells us nothing about whether this tool is working.

MacBuilder.run now retries up to 3 times with a 10s backoff, but only
when the failure output matches this specific known-transient pattern -
any other failure (a real compile error, etc.) still fails immediately
on the first attempt, so this can't mask genuine build breakage.
@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 24 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 0a5a97f2-074c-4d83-91e5-f796a3d8cd0c

📥 Commits

Reviewing files that changed from the base of the PR and between e63ebca and 9c81652.

⛔ Files ignored due to path filters (2)
  • plugins/unity/dist/unity-builder/model/mac-builder.d.ts is excluded by !**/dist/**
  • plugins/unity/dist/unity-builder/model/mac-builder.js is excluded by !**/dist/**
📒 Files selected for processing (2)
  • plugins/unity/src/unity-builder/model/mac-builder.test.ts
  • plugins/unity/src/unity-builder/model/mac-builder.ts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@frostebite
frostebite merged commit 65f5019 into main Aug 28, 2026
15 checks passed
@frostebite
frostebite deleted the fix/mac-licensing-client-retry branch August 28, 2026 15:39
frostebite added a commit that referenced this pull request Aug 28, 2026
…ensing-client signature error (#226)" (#228)

This reverts commit 65f5019.
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