FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix(ci): exclude macOS AppIntents system leak by UriahNegbi · Pull Request #642 · EngineSquared/EngineSquared · GitHub

fix(ci): exclude macOS AppIntents system leak - #642

Merged
Miou-zora merged 2 commits into
EngineSquared:mainfrom
UriahNegbi:fix/macos-appintents-leak-check
Aug 29, 2026
Merged

fix(ci): exclude macOS AppIntents system leak#642
Miou-zora merged 2 commits into
EngineSquared:mainfrom
UriahNegbi:fix/macos-appintents-leak-check

Conversation

Copy link
Copy Markdown
Contributor

Summary

The macOS 26 runner reports an Apple AppIntents XPC retain cycle in every RmlUi test process. This adds a narrow leaks exclusion for the full Apple allocation symbol while leaving all project allocations checked.

Verification

A dedicated macOS 26.5 ARM64 run reproduced 287 to 288 leaked nodes, all rooted at LNProcessInstanceRegistryClient and com.apple.linkd.autoShortcut. The same run passed after applying this exact symbol exclusion:

https://github.com/UriahNegbi/EngineSquared/actions/runs/33219987079

This clears the shared macOS failure currently affecting #640 and #641.

coderabbitai Bot commented Aug 28, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

Next included review available in 34 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: c2182c17-6dbe-4f14-b965-412330b2c587

📥 Commits

Reviewing files that changed from the base of the PR and between 1118673 and 627a606.

📒 Files selected for processing (1)
  • tools/xmake/check_leaks.lua

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.

Miou-zora merged commit 23f80a8 into EngineSquared:main Aug 29, 2026
6 checks passed

Copy link
Copy Markdown

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL