fix(percy): float @percy/webdriverio to ^3.3.4 (SDK-7711) by rounak610 · Pull Request #228 · browserstack/wdio-browserstack-service

rounak610 · 2026-09-24T12:28:48Z

What is this about?

Floats @percy/webdriverio from the temporary >=3.3.0 <3.3.3 pin to ^3.3.4.

The pin was introduced in the previous release (#210) as a workaround: @percy/webdriverio@3.3.3 shipped without its _iframe_shim.js file (it was never listed in package.json files), so the package failed to load with MODULE_NOT_FOUND and Percy web snapshots were silently disabled on WebdriverIO. Percy has fixed the packaging in 3.3.4 (percy/percy-webdriverio#1494, released 2026-09-24, now latest) and added a CI npm pack + clean-install smoke test to prevent recurrence, so the cap is no longer needed.

Only the version range changes; the lockfile resolves @percy/webdriverio@3.3.4 (with its @percy/sdk-utils@^1.32.0 transitive). No source changes.

Related Jira task/s

SDK-7711
Percy tracking: PER-10878 (closed) / SDK-7756 / PPLT-6128

Release (mandatory for every PR — required for the `ready-for-review` label)

Version bump: (required — tick exactly one)

minor (backwards-compatible feature)
patch (bug fix or other small change)

Release notes type: (optional)

New Feature
Bug Fix
Other Improvement

Release notes (customer-facing): (optional but encouraged)

Restored Percy visual snapshots on WebdriverIO by picking up the fixed @percy/webdriverio@3.3.4.

Release notes (internal): (required — engineer-facing; what actually changed / why)

Removed the temporary @percy/webdriverio >=3.3.0 <3.3.3 cap and floated to ^3.3.4; 3.3.3 was unloadable (missing _iframe_shim.js in the published tarball), fixed by Percy in 3.3.4. Lockfile updated to resolve 3.3.4 + @percy/sdk-utils@1.32.10.

Checklist

Ready to review
Has it been tested locally?

PR Validations

Run Tests: Comment RUN_TESTS to trigger sanity tests.

Removes the temporary >=3.3.0 <3.3.3 pin now that Percy has fixed the broken 3.3.3 packaging (missing _iframe_shim.js) in 3.3.4. Restores Percy web snapshot capture on WebdriverIO without the version cap. SDK-7711 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

coderabbitai · 2026-09-24T12:28:55Z

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Central YAML (base), Organization UI (inherited), Workspace UI (inherited)

Review profile: ASSERTIVE

Plan: Enterprise

Run ID: 5b3184b2-18db-4aca-9bbb-70f0625f966f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

_{Comment @coderabbitai help to get the list of available commands.}

rounak610 · 2026-09-24T14:05:00Z

RUN_TESTS

rounak610 · 2026-09-25T08:02:26Z

RUN_TESTS

github-actions · 2026-09-25T09:37:36Z

🔴 SDK PR Review gate is red. Pending:

The SDK PR Review Agent has not been run on the current head commit yet — run the SDK PR Review Agent (its verdict is advisory; this gate only requires that it ran on the latest commit).

It turns green once the SDK PR Review Agent has run on the current head commit (any verdict — the gate only requires that the review ran). A native reviewer approval is separately required by branch protection before merge.

rounak610 · 2026-09-25T09:41:11Z

⚠️ Needs human review

Summary: Floats @percy/webdriverio from >=3.3.0 <3.3.3 to ^3.3.4 (Percy's 3.3.3 packaging fix). package-lock.json regenerated to match. No source/logic changes. 0 blocking findings.

Per-File Confidence

File	Status	Reason
`packages/browserstack-service/package.json`	✅ All Clear	Reviewed against `config`, `default`, `external`, `integration`, `observability` packs + the wdio-service card — dependency bump matches its own changeset and PR description (CF-01/CF-06 pinned-bump exemption); no findings

Not covered this run (coverage gap):

Region	Why unjudged
`.changeset/pr-228.md#h1`	Skipped as unchanged since the prior review (content fingerprint identical to the run on commit `1a103aaa`)
`package-lock.json#h1..h4`	Lockfile regions are excluded from unit review by design (generated file, context-only)

The ⚠️ verdict is driven entirely by this coverage gap, not by any suspected defect — the reviewed region (package.json) came back clean.

Process integrity

The review gate could NOT certify this run — the verdict is held at ⚠️ pending regardless of findings:

G3 — u001: no full read receipt for pack:docs
G3 — u001: no full read receipt for slice

Change map (generated deterministically from the diff)

graph LR
  subgraph nwdio_service["wdio-service"]
    n_changeset_pr_228_md["pr-228.md<br/>~5 lines"]
    npackages_browserstack_service_package_json["package.json<br/>~2 lines"]
  end

↻ This verdict comment is the review anchor — it's updated in place on each run (the gate posts its status separately).

— SDK PR Review Agent

github-actions · 2026-09-25T09:41:19Z

🟢 SDK PR Review gate is green — the SDK PR Review Agent has run on the current head commit (verdict: pending).

This gate confirms a review ran on the latest commit. The verdict itself is advisory — read the findings and use your judgement; it does not block merge. A native GitHub reviewer approval is still separately required by branch protection before this PR can merge.

github-actions · 2026-09-25T09:45:25Z

🟢 SDK PR Review gate is green — the SDK PR Review Agent has run on the current head commit (verdict: pending).

This gate confirms a review ran on the latest commit. The verdict itself is advisory — read the findings and use your judgement; it does not block merge. A native GitHub reviewer approval is still separately required by branch protection before this PR can merge.

rounak610 · 2026-09-28T07:01:56Z

RUN_TESTS

The base branch was changed.

rounak610 · 2026-09-28T07:09:19Z

RUN_TESTS

github-actions · 2026-09-28T07:09:22Z

🔴 SDK PR Review gate is red. Pending:

The SDK PR Review Agent has not been run on the current head commit yet — run the SDK PR Review Agent (its verdict is advisory; this gate only requires that it ran on the latest commit).

It turns green once the SDK PR Review Agent has run on the current head commit (any verdict — the gate only requires that the review ran). A native reviewer approval is separately required by branch protection before merge.

github-actions · 2026-09-28T07:33:05Z

🟢 SDK PR Review gate is green — the SDK PR Review Agent has run on the current head commit (verdict: pending).

This gate confirms a review ran on the latest commit. The verdict itself is advisory — read the findings and use your judgement; it does not block merge. A native GitHub reviewer approval is still separately required by branch protection before this PR can merge.

github-actions · 2026-09-28T07:35:03Z

🟢 SDK PR Review gate is green — the SDK PR Review Agent has run on the current head commit (verdict: pending).

This gate confirms a review ran on the latest commit. The verdict itself is advisory — read the findings and use your judgement; it does not block merge. A native GitHub reviewer approval is still separately required by branch protection before this PR can merge.

github-actions · 2026-09-28T07:36:14Z

🟢 SDK PR Review gate is green — the SDK PR Review Agent has run on the current head commit (verdict: pending).

This gate confirms a review ran on the latest commit. The verdict itself is advisory — read the findings and use your judgement; it does not block merge. A native GitHub reviewer approval is still separately required by branch protection before this PR can merge.

rahulpsq · 2026-09-28T07:39:10Z

RUN_TESTS

rounak610 · 2026-09-28T11:21:42Z

Sanity build: https://automation.browserstack.com/projects/WDIO+v9+SDK-QA+Project/builds/WDIO+SDK-QA+Automation/312?bls_state=active%7CActive&bls_projects=2612588

Dalwin-Barnard · 2026-09-29T06:03:48Z

RUN_TESTS

rounak610 requested a review from a team as a code owner September 24, 2026 12:28

rounak610 requested review from 07souravkunda and pranay-v29 and removed request for a team September 24, 2026 12:28

chore(changeset): auto-generate from PR template (patch)

1a103aa

rounak610 requested review from harshit-browserstack and removed request for 07souravkunda September 24, 2026 12:43

rounak610 mentioned this pull request Sep 24, 2026

fix(percy): restore Percy capture on WebdriverIO (v8) (SDK-7756) #231

Merged

3 of 7 tasks

pranay-v29 previously approved these changes Sep 24, 2026

View reviewed changes

harshit-browserstack previously approved these changes Sep 25, 2026

View reviewed changes

rounak610 added the ready-for-review label Sep 25, 2026

rounak610 changed the base branch from release_9.37.0 to main September 28, 2026 07:08

Merge branch 'main' into fix/sdk-7711-percy-webdriverio-unpin

0c1ad22

shivam5643 approved these changes Sep 28, 2026

View reviewed changes

anish353 approved these changes Sep 28, 2026

View reviewed changes

Dalwin-Barnard merged commit 59f32f6 into main Sep 29, 2026
18 of 20 checks passed

Conversation

rounak610 commented Sep 24, 2026 • edited by atlassian Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What is this about?

Related Jira task/s

Release (mandatory for every PR — required for the ready-for-review label)

Checklist

PR Validations

Uh oh!

coderabbitai Bot commented Sep 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Uh oh!

rounak610 commented Sep 24, 2026

Uh oh!

rounak610 commented Sep 25, 2026

Uh oh!

github-actions Bot commented Sep 25, 2026

Uh oh!

rounak610 commented Sep 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Per-File Confidence

Process integrity

Change map (generated deterministically from the diff)

Uh oh!

github-actions Bot commented Sep 25, 2026

Uh oh!

github-actions Bot commented Sep 25, 2026

Uh oh!

rounak610 commented Sep 28, 2026

Uh oh!

rounak610 commented Sep 28, 2026

Uh oh!

github-actions Bot commented Sep 28, 2026

Uh oh!

github-actions Bot commented Sep 28, 2026

Uh oh!

github-actions Bot commented Sep 28, 2026

Uh oh!

github-actions Bot commented Sep 28, 2026

Uh oh!

rahulpsq commented Sep 28, 2026

Uh oh!

rounak610 commented Sep 28, 2026

Uh oh!

Dalwin-Barnard commented Sep 29, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

rounak610 commented Sep 24, 2026 •

edited by atlassian Bot

Loading

Release (mandatory for every PR — required for the `ready-for-review` label)

coderabbitai Bot commented Sep 24, 2026 •

edited

Loading

rounak610 commented Sep 25, 2026 •

edited

Loading