test(browser): Make browser profiling test less flaky#20664
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 29282fd. Configure here.
| page, | ||
| 1, | ||
| { url, envelopeType: 'profile_chunk', timeout: 5000 }, | ||
| { url, envelopeType: 'profile_chunk', timeout: 15_000 }, |
There was a problem hiding this comment.
Unreliable envelope helper used in flakiness fix
Low Severity
The PR aims to reduce flakiness but still relies on getMultipleSentryEnvelopeRequests, which the project rules explicitly mark as unreliable. The rules recommend using helpers like waitForTransaction, waitForError, waitForSpans, etc. instead. While the timeout increase from 5s to 15s may help, using a more reliable waiting mechanism would better address the root cause of the flakiness. Flagged because the project rules file requires flagging usage of getMultipleEnvelope* helpers.
Triggered by project rule: PR Review Guidelines for Cursor Bot
Reviewed by Cursor Bugbot for commit 29282fd. Configure here.
size-limit report 📦
|


Fixes #20638
Hopefully this reduces flakiness, by increasing timeouts etc. and making sure things run longer.