Skip to content

feat: do not attach prefs when debuggerAddress is specified (v8)#14635

Merged
christian-bromann merged 2 commits intowebdriverio:v8from
uladhsi:v8-fix-prefs
Jul 14, 2025
Merged

feat: do not attach prefs when debuggerAddress is specified (v8)#14635
christian-bromann merged 2 commits intowebdriverio:v8from
uladhsi:v8-fix-prefs

Conversation

@uladhsi
Copy link
Contributor

@uladhsi uladhsi commented Jul 14, 2025

Proposed changes

Backported PR for #14634

Disable attaching prefs object to the 'goog:chromeOptions' when the debuggerAddress is specified, because chrome.prefs are supposed to be used for a new session, and debuggerAddress is used when there is an intention to connect to an existing session.

Types of changes

  • Polish (an improvement for an existing feature)
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update (improvements to the project's docs)
  • Specification changes (updates to WebDriver command specifications)
  • Internal updates (everything related to internal scripts, governance documentation and CI files)

Checklist

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have added the necessary documentation (if appropriate)
  • I have added proper type definitions for new commands (if appropriate)

Further comments

Related issue: #14591

Reviewers: @webdriverio/project-committers

@uladhsi uladhsi marked this pull request as draft July 14, 2025 13:49
@uladhsi uladhsi marked this pull request as ready for review July 14, 2025 14:32
@uladhsi uladhsi changed the title feat: do not attach prefs when debuggerAddress is specified feat: do not attach prefs when debuggerAddress is specified (v8) Jul 14, 2025
Copy link
Member

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@christian-bromann christian-bromann added the PR: Polish 💅 PRs that contain improvements on existing features label Jul 14, 2025
@christian-bromann christian-bromann merged commit 3fe1f60 into webdriverio:v8 Jul 14, 2025
3 checks passed
@uladhsi
Copy link
Contributor Author

uladhsi commented Jul 24, 2025

@christian-bromann is there any plan when this is going to be released for v8?

@christian-bromann
Copy link
Member

Thanks for pinging, I just triggered a v8 release.

@uladhsi
Copy link
Contributor Author

uladhsi commented Jul 30, 2025

@christian-bromann thanks for triggering it, but somehow it did not get released. There was a failure during the release, could you take a look if possible?

https://github.com/webdriverio/webdriverio/actions/runs/16529089221/job/46749773768

@christian-bromann
Copy link
Member

@uladhsi mhm 🤔 something broke in the pipeline, not sure why. Mind raising an issue for it?

@uladhsi
Copy link
Contributor Author

uladhsi commented Jul 31, 2025

@christian-bromann Sure, here it is:
#14681

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: Polish 💅 PRs that contain improvements on existing features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants