perf(desktop): harden GPUI and Tauri releases across all platforms - #2146
Merged
richiemcilroy merged 34 commits intoAug 26, 2026
Conversation
Manage your Superagent protectionSuperagent has paused scans for this repository because this unlinked GitHub App installation has used all three included PR scans. You have 0 of 3 included PR scans remaining. Create a free account to continue protection, manage scan settings, review security history, and control which repositories are protected. |
Member
Author
|
hey @greptileai, please re-review the PR |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Member
Author
|
hey @greptileai, please re-review the PR |
Member
Author
|
hey @greptileai, please re-review the PR |
Member
Author
|
hey @greptileai, please re-review the PR |
Member
Author
|
hey @greptileai, please re-review the PR |
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.
Summary
Validation status
Latest source:
a32c5595de37872572fdcc518d94deaca87f489a.The latest Greptile review is 5/5. All A/V jobs have passed, including 27/27 editor-audio tests on each platform. Windows and Linux fully executed all 41 scenario records each; macOS fully executed 40, with one random 450-fps noise-video leg skipped after the runner stalled 1.18 seconds (its audio leg passed). The skipped portion is not counted as a passed test. Each platform also deliberately ignored three pre-existing manual tests; these are excluded from passing totals.
All desktop CI jobs passed: optimized sidecars, GPUI compilation, final Tauri application builds on all three platforms, and both strict workspace/all-features Clippy jobs. GPUI tests passed 577/577 on macOS ARM64, 571/571 on Windows x86-64, and 574/574 on Linux x86-64, with none failed or ignored. The CI application builds are development-profile, not newly production-signed installers. GitHub shows 25 successful checks, no failed or pending checks, and no unresolved review threads. Human review remains required; this is not a claim of perfect coverage or final production-release approval.
Optimized-artifact and real-world evidence
The Windows and Linux installers below were tested before the later audio-only fix. The separate encoded-media check validates that fix, while current CI validates the complete committed source. Do not interpret the installer hashes as containing the later fix.
cap 0.6.0 install ok installed. Actual Radeon-backed Wayland with DISPLAY unset: Tauri recorded screen at 58.95 fps and webcam at 720p30 without drops, with separate 443-Hz microphone and 997-Hz system tracks. GPUI Studio/Instant ran around 59.6 fps without drops, including area pause/resume and camera-only recording. Verified visible playback, rapid scrubbing, exports, screenshot privacy, real D-Bus notifications and a valid native image clipboard.Authenticated Instant tests used a controlled authenticated test endpoint, not the production backend. Linux AMD Tauri produced 33 real requests, 20 PUTs, 36.25 MB of progressive media and exactly one completion.
Windows NSIS SHA-256:
e16de5a49e6c76949e976d583342d4cba944af5e20982395b24a801d84f598c6.Linux Debian package SHA-256:
d88e364c1fff670cbc816be3bc130279804ccdf64dc6cd5b0f4c32ea3901343f.Measured performance
Later 1x clip-mute fix: actual MP4 proof
Exported two real recording-derived projects, each with three contiguous two-second clips and only the middle clip muted, both with and without independent music. Compared the earlier packaged CLI against a newly rebuilt CLI containing the exact audio fix on Linux NVIDIA.
All four outputs are valid six-second, 180-frame, 1280x720 H.264/AAC files using actual Tesla T4/Vulkan and NVENC. Independently decoded AAC measurements, using one-second interior windows in each two-second clip, reproduce the old bug and confirm the fix:
The rebuilt CLI is debug-profile: this proves encoded-output correctness, not a new signed installer or a release-profile speed comparison. Normal Linux Cargo execution and current CI on all three platforms passed all 27 editor-audio tests with no failures or ignored tests.
Canonical installed Linux paths and real switching
A separate final AMD check verified the installed desktop entry, clean package verification, all five executable hashes and native dependency lookup without development-library overrides. Actual Tauri and GPUI processes ran from
/usr/bin, loaded FFmpeg from/usr/lib/cap, selected the Radeon GPU and displayed native Wayland windows with noDISPLAY,CAP_GPUI_BINorCAP_TAURI_BINoverrides.All four actual Experimental flows passed: forward cancel/confirm and reverse cancel/confirm. Cancel preserved the original process and preference; completed countdowns launched the correct installed sibling and shut down the prior app. Screenshots, process executable paths, hashes, preferences and logs were checked.
All apps ran as the existing
ubuntuuser. The switch fixture retained the documented vblank fallback, an inherited WebKit sandbox-disabling flag and a hook that only opened GPUI settings. A separate final ordinary-user Tauri launch passed with the sandbox-disabling flag and settings hook absent, retaining only the vblank fallback. No users, permissions or application source were changed. This is installed-path verification of the original optimized package, not another performance matrix or a rebuilt installer containing the later audio fix.Linux GPUI fixture stability
The intermittent display-listing failure was traced to Xvfb resetting when its last client disconnected, independently reproduced with a root-window property probe. Adding
-noresetchanges only the CI display fixture:Supporting automated checks include 174 desktop Vitest tests, 18 extension tests, packaging-script tests, real NVENC tests, Wayland privacy checks and shared-buffer lifetime regressions.
Release boundaries
Greptile Summary
The PR hardens cross-platform GPUI and Tauri packaging while reducing work throughout recording, playback, rendering, and import paths.
Confidence Score: 5/5
The PR appears safe to merge because no blocking failure remains.
No blocking failure remains.
Important Files Changed
Reviews (5): Last reviewed commit: "test(ci): run editor audio regressions o..." | Re-trigger Greptile
Context used: