Skip to content

fix: repository metadata and stale doc links across driver packages - #15

Merged
dprevost-LMI merged 3 commits into
webdriverio:mainfrom
dprevost-LMI:fix/repository-links-provenance
Sep 22, 2026
Merged

dprevost-LMI merged 3 commits into
webdriverio:mainfrom
dprevost-LMI:fix/repository-links-provenance

Conversation

@dprevost-LMI

Copy link
Copy Markdown
Collaborator

npm publish was failing sigstore provenance verification because package.json repository.url still pointed at the old pre-monorepo repos instead of webdriverio/driver. Also updates homepage/bugs URLs, README badges/links, and a couple of stale/dead doc links (Azure CDN domain, Edge driver directory listing) found while auditing the READMEs.

npm publish was failing sigstore provenance verification because
package.json repository.url still pointed at the old pre-monorepo
repos instead of webdriverio/driver. Also updates homepage/bugs
URLs, README badges/links, and a couple of stale/dead doc links
(Azure CDN domain, Edge driver directory listing) found while
auditing the READMEs.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@dprevost-LMI
dprevost-LMI enabled auto-merge (squash) September 22, 2026 01:14
@greptile-apps

greptile-apps Bot commented Sep 22, 2026

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

The PR appears safe to merge; no outstanding correctness, security, or repository-rule issues remain.

Findings

  1. P2 WebSocket behavior is undocumented

Summary

Updates repository metadata for the driver packages so npm provenance points to the current monorepo, refreshes badges and stale documentation links, and corrects EdgeDriver option documentation. The follow-up change also fully documents Geckodriver’s websocketPort default and incompatibility with connectExisting.

Reviews (3) · Last reviewed commit: "docs: clarify websocketPort default and ..."

Comment thread packages/node-geckodriver/README.md
dprevost-LMI and others added 2 commits September 21, 2026 21:18
Keeps all three driver sections consistent so future safaridriver
options stay discoverable without needing to remember to update
the root README's table.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Addresses PR review feedback: the wrapper forces --websocket-port=0
(a random port) when websocketPort is omitted, rather than using
Geckodriver's native default, and throws if websocketPort is set
alongside connectExisting.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@dprevost-LMI
dprevost-LMI enabled auto-merge (squash) September 22, 2026 01:24
@dprevost-LMI
dprevost-LMI merged commit 8c88982 into webdriverio:main Sep 22, 2026
9 of 10 checks passed
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.

1 participant