fix: env overrides under bun, the stdio test under bun and fast png in debug builds - #54
Open
mateo-m wants to merge 110 commits into
Open
fix: env overrides under bun, the stdio test under bun and fast png in debug builds#54mateo-m wants to merge 110 commits into
mateo-m wants to merge 110 commits into
Conversation
… onto effect layers
# Conflicts: # packages/native/src/renderer/frame.rs
# Conflicts: # .github/workflows/ci.yml # examples/compile.ts # packages/native/src/custom_elements/anchored.rs # packages/native/src/custom_elements/code.rs # packages/native/src/custom_elements/diff.rs # packages/native/src/custom_elements/img.rs # packages/native/src/custom_elements/input.rs # packages/native/src/custom_elements/markdown.rs # packages/native/src/renderer.rs # packages/react/src/types/host.ts # zed
# Conflicts: # zed
# Conflicts: # packages/native/src/custom_elements/code.rs # packages/native/src/renderer/frame.rs # packages/native/src/style/resolve.rs # packages/react/src/testing.ts # packages/react/src/types/host.ts
# Conflicts: # packages/native/src/renderer/frame.rs # packages/react/src/reconciler/event-registry.ts
mateo-m
force-pushed
the
fix/selection-remount-automation
branch
from
September 4, 2026 14:59
bfcd1ef to
9757935
Compare
mateo-m
changed the title
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
1 participant
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.
Contains #53.
What this adds
range_rects_with_positionsafter this branch did, so the branch keeps the test and drops its own fix.process.envoverrides reach Rust under Bun through asyncEnvVarexport. Bun only updates its JS snapshot, so the Rust side never saw a late assignment. The values land in an override map, not insetenv, becausesetenvracesgetenvon the dedicated UI thread of Windows and Linux.capture_screenshotencodes a png per visual test, and the encoder at opt-level 0 was slow enough that vitest killed the worker.The live
scrollWheelautomation method this PR first added now comes from upstream main with modifier support, so this branch dropped its own copy. The same happened to thebun --hotremount fix: upstream main keeps the renderer state onglobalThisand readsonEventfrom the container, so the fork's version is gone too.What I ran and saw
cargo testat this branch: 327 passed, 0 failed.What I did not verify
Moved from the fork
This PR was mateo-m#5.
The branch merged its parent branch after that one merged upstream main at 028f70a.
The history was rewritten once after the first push. It dropped the commits that reworded AGENTS.md, and the zed pin moved to the rewritten mateo-m/zed branches. No code changed in that rewrite.
On the merged head:
cargo test327 passed, 0 failed, the react suite 477 passed, 0 failed.Stacked on #53. The diff here includes #50, #51, #52, #53.
The zed submodule points at the mateo-m/zed fork until remorses/zed#5, #6 and #7 merge. Then a follow-up commit restores
.gitmodulesto remorses/zed on thegpuixbranch and pins the merged commit. The same GPUI stack is also open upstream as zed-industries/zed#63771, #63772 and #63773.