Repository navigation
[WRONG BRANCH] release: sync preview to main 2.86.0 - #7127
Merged
Merged
Conversation
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* fix(test-teardown): settle un-awaited background work before removing a tree The pre-route pool quota prime runs without being awaited and holds the native-main claim database open across ACL hardening and a WHAM read. Teardown retried removal with Bun.sleepSync, which blocks the event loop the prime needs to reach its finally, so on Windows every retry failed EBUSY until the 15 s budget ran out (#7088 and #7091 stubbed the prime per test). Track fire-and-forget work in src/lib/background-work, register the pool prime and its lazy-import callers, settle it in a global afterEach, and add removeTreeAfterBackgroundWork (settle, reap icacls, then retry with waits that yield). The compaction fixture uses it, and the per-test prime stubs from #7088 and #7091 are reverted so the real prime runs again. * fix(responses): serialize snapshot follow-up scheduling with shutdown * fix(config): track the config-dir ACL harden as background work icacls holds the config directory open while it runs; a config write or removal that lands on it before the child exits fails with a sharing violation on Windows. Registering the flight lets the per-case drain settle it, and the V2 refusal test now prints the CLI errors when the handler refuses before the native toggle. * fix(cli): reject occupied auxiliary ports before private startup work * fix(tray): retain timed-out probes until asynchronous reaping Remove WaitForExit from startup-health timeout ticks and retain process ownership until exit and both readers settle. Run lifecycle fixtures with the production Bun engine instead of a nested PowerShell cold start, which failed readiness before maintenance in Windows run 38503986851. * fix(config): preserve Windows ACL memo across repeated reads * test: bound child exit and pipe waits in slow-spawn cases * fix(ci): share one transpiler cache per shard; widen the DPAPI PowerShell bound Run 2 and 3 of the Windows proof failed on first-spawn timeouts (writeJournal child 40s, launcher attest 15s) and a 10s DPAPI PowerShell call. Each CI batch is its own bun test process, and with BUN_RUNTIME_TRANSPILER_CACHE_PATH unset the preload gives every batch an empty private cache, so the first child each batch spawns re-transpiles its whole module graph. The batch runner now creates one private cache per shard and exports it. The DPAPI helper's PowerShell bound moves from 10s to 20s: it is async and off the request path, and a cold PowerShell loading System.Security exceeded 10s on a loaded runner. * test(desktop-egress): give each proxy variable one spelling on Windows Windows environment names are case-insensitive, so the child env that set ALL_PROXY to the fixture proxy and all_proxy to "" defined one variable twice. Env-block ordering decided the winner; adding the shard transpiler cache variable flipped it, and the -all cases reached the real chatgpt.com (403). Clear every spelling and set only the intended route. * fix(test-teardown): never remove or exit under live tracked work; address review findings - removeTreeAfterBackgroundWork keeps the tree and names pending work instead of deleting under a live writer. - Final sandbox cleanup bounds background-work settlement (15 s) and leaves the root for stale recovery with the labels when work never settles. - spawnTestChild rejects when the child never started (ENOENT) instead of resolving an empty result. - Auxiliary preflight rejects loopback and hub management configured on one port. - Tray shutdown keeps a probe tracked until it has actually exited and waits for its pipe readers before disposal. - The free-socket preflight test binds inside try so a lost race still stops the first server. --------- Co-authored-by: jun <jun@juns-Mac-Studio.local>
[WRONG BRANCH] release: promote dev 494c50d to main as 2.86.0
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Contributor
|
✅ Deterministic PR hygiene checks passed. |
github-actions
Bot
changed the title
Contributor
⏳ DRAFT
What to do
Its title has been prefixed with |
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.
Summary
Sync
previewtomainat409026b1bc(stable 2.86.0, #7110; npm latest 2.86.0, Release run 38640052000). Head is a merge ofpreviewancestry whose tree equalsmainexactly (ours-strategy equivalent). No preview release is cut.Owner-delegated release train; merges with owner authority (
--admin).