Conversation
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>
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Central YAML (base), Organization UI (inherited), Workspace UI (inherited) Review profile: ASSERTIVE Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
|
RUN_TESTS |
|
RUN_TESTS |
|
🔴 SDK PR Review gate is red. Pending:
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. |
Process integrityThe review gate could NOT certify this run — the verdict is held at
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 |
|
🟢 SDK PR Review gate is green — the SDK PR Review Agent has run on the current head commit (verdict: 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. |
1 similar comment
|
🟢 SDK PR Review gate is green — the SDK PR Review Agent has run on the current head commit (verdict: 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. |
What is this about?
Floats
@percy/webdriveriofrom the temporary>=3.3.0 <3.3.3pin to^3.3.4.The pin was introduced in the previous release (#210) as a workaround:
@percy/webdriverio@3.3.3shipped without its_iframe_shim.jsfile (it was never listed inpackage.jsonfiles), so the package failed to load withMODULE_NOT_FOUNDand 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, nowlatest) and added a CInpm 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.0transitive). No source changes.Related Jira task/s
Release (mandatory for every PR — required for the
ready-for-reviewlabel)Version bump: (required — tick exactly one)
Release notes type: (optional)
Release notes (customer-facing): (optional but encouraged)
@percy/webdriverio@3.3.4.Release notes (internal): (required — engineer-facing; what actually changed / why)
@percy/webdriverio>=3.3.0 <3.3.3cap and floated to^3.3.4; 3.3.3 was unloadable (missing_iframe_shim.jsin the published tarball), fixed by Percy in 3.3.4. Lockfile updated to resolve 3.3.4 +@percy/sdk-utils@1.32.10.Checklist
PR Validations
Run Tests: Comment RUN_TESTS to trigger sanity tests.