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

test: deflake test-worker-stack-overflow-stack-size in the Node.js suite by trop[bot] · Pull Request #53076 · electron/electron · GitHub

test: deflake test-worker-stack-overflow-stack-size in the Node.js suite - #53076

Merged
MarshallOfSound merged 1 commit into
42-x-yfrom
trop/42-x-y-bp-test-deflake-test-worker-stack-overflow-stack-size-in-the-node-js-suite-1787306223862
Aug 21, 2026
Merged

test: deflake test-worker-stack-overflow-stack-size in the Node.js suite#53076
MarshallOfSound merged 1 commit into
42-x-yfrom
trop/42-x-y-bp-test-deflake-test-worker-stack-overflow-stack-size-in-the-node-js-suite-1787306223862

Conversation

trop Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Backport of #53062

See that PR for details.

Notes: none

Backports nodejs/node#64271. The test compares the recursion depth two
Workers reach before overflowing, but depth also depends on frame size,
which changes when the recursive function tiers up mid-recursion. On a
busy CI runner that lands non-deterministically and the depths end up
>10% apart. Pin the function to the interpreter so the depth is stable.

The fix is only in Node 26.6.0+, so it is carried as a patch until the
Node roll picks it up.

Co-authored-by: Sam Attard <sattard@anthropic.com>
trop Bot requested a review from a team as a code owner August 21, 2026 09:57
trop Bot requested a review from MarshallOfSound August 21, 2026 09:57
trop Bot added 42-x-y backport This is a backport PR semver/none labels Aug 21, 2026
MarshallOfSound merged commit 01494a8 into 42-x-y Aug 21, 2026
90 of 91 checks passed
MarshallOfSound deleted the trop/42-x-y-bp-test-deflake-test-worker-stack-overflow-stack-size-in-the-node-js-suite-1787306223862 branch August 21, 2026 11:01

release-clerk Bot commented Aug 21, 2026

Copy link
Copy Markdown

No Release Notes

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL