Skip to content

Fix: launch browsers in detached mode on Windows to fix flakiness (#15339) - #3559

Merged
kblok merged 1 commit into
masterfrom
cursor/implement-upstream-pr-15339-4c19
Aug 17, 2026
Merged

Fix: launch browsers in detached mode on Windows to fix flakiness (#15339)#3559
kblok merged 1 commit into
masterfrom
cursor/implement-upstream-pr-15339-4c19

Conversation

@kblok

@kblok kblok commented Aug 17, 2026

Copy link
Copy Markdown
Member

Implements changes from puppeteer/puppeteer#15339 (puppeteer-core v25.8.0).

Upstream

  • Launch browsers with detached: true on all platforms and windowsHide: true
  • Clean a temp user-data dir if the executable is missing
  • Do not rethrow cleanup errors
  • Increase temp-dir removal retries
  • Mark a MacOS Chrome touchscreen test as flaky

PuppeteerSharp

  • Set Process.StartInfo.CreateNoWindow = true (Windows windowsHide equivalent; .NET has no detached flag)
  • Delete the temp user-data directory before throwing when the executable is missing
  • Swallow Firefox prefs-restore errors on exit
  • Temp directory deletion: 10 retries with a 100ms delay, then existing backoff
  • Synced the upstream darwin+chrome flake expectation for touchscreen.spec / three touches

Skipped the Node mocha-runner changes (no PuppeteerSharp equivalent).

Open in Web Open in Cursor 

…5339)

Port upstream Puppeteer PR 15339: hide the browser process window on
Windows, clean temp user data dirs when the executable is missing or
launch fails, retry temp-dir deletion more aggressively, and sync the
MacOS touchscreen flake expectation.

Co-authored-by: Darío Kondratiuk <dariokondratiuk@gmail.com>
@kblok
kblok marked this pull request as ready for review August 17, 2026 12:41
@cursor

cursor Bot commented Aug 17, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@kblok
kblok merged commit 4b34b00 into master Aug 17, 2026
17 checks passed
@kblok
kblok deleted the cursor/implement-upstream-pr-15339-4c19 branch August 17, 2026 17:21
sondresjolyst pushed a commit to sondresjolyst/garge-api that referenced this pull request Aug 30, 2026
Updated [PuppeteerSharp](https://github.com/hardkoded/puppeteer-sharp)
from 25.7.0 to 25.8.0.

<details>
<summary>Release notes</summary>

_Sourced from [PuppeteerSharp's
releases](https://github.com/hardkoded/puppeteer-sharp/releases)._

## 25.8.0

## What's Changed
* chore: bump version to 25.8.0 by @​kblok in
hardkoded/puppeteer-sharp#3561
* Fix: launch browsers in detached mode on Windows to fix flakiness
(#​15339) by @​kblok in
hardkoded/puppeteer-sharp#3559
* Fix: computeSystemExecutablePath support validatePath (#​15340) by
@​kblok in hardkoded/puppeteer-sharp#3557
* Remove redundant overwrite and file access from ScreenRecorder
(#​15352) by @​kblok in
hardkoded/puppeteer-sharp#3560
* New Feature: Support FollowSymlinks option (#​15335) by @​kblok in
hardkoded/puppeteer-sharp#3558


**Full Changelog**:
hardkoded/puppeteer-sharp@v25.7.0...v25.8.0

Commits viewable in [compare
view](hardkoded/puppeteer-sharp@v25.7.0...v25.8.0).
</details>

[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=PuppeteerSharp&package-manager=nuget&previous-version=25.7.0&new-version=25.8.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants