Skip to content

Conversation

selenium-ci
Copy link
Member

@selenium-ci selenium-ci commented Sep 20, 2025

This is an automated pull request to update pinned browsers and drivers

Merge after verify the new browser versions properly passing the tests and no bugs need to be filed

@selenium-ci selenium-ci added the B-build Includes scripting, bazel and CI integrations label Sep 20, 2025
Copy link
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

URL/Checksum Consistency

Verify that each updated download URL matches its corresponding sha256 and that archives extract to expected paths; a mismatch will break Bazel fetch or file moves.

    http_archive(
        name = "linux_firefox",
        url = "https://ftp.mozilla.org/pub/firefox/releases/143.0.1/linux-x86_64/en-US/firefox-143.0.1.tar.xz",
        sha256 = "0d515c65248d4485cc8f5abf4e3b09a9a65363f2b4e6b8435309a4c87e500720",
        build_file_content = """
load("@aspect_rules_js//js:defs.bzl", "js_library")
package(default_visibility = ["//visibility:public"])

filegroup(
    name = "files",
    srcs = glob(["**/*"]),
)

exports_files(["firefox/firefox"])

js_library(
    name = "firefox-js",
    data = [":files"],
)
""",
    )

    dmg_archive(
        name = "mac_firefox",
        url = "https://ftp.mozilla.org/pub/firefox/releases/143.0.1/mac/en-US/Firefox%20143.0.1.dmg",
        sha256 = "c37afd2e0ec1ea5e6d508779ffefdf2c404752392288a6a7f0f1670fe8747e3c",
        build_file_content = """
Edge pkg move path

The 'move' mapping for macOS Edge references the new pkg path; ensure the internal pkg structure still contains 'Payload/Microsoft Edge.app' under the new filename/version to avoid extraction failures.

    pkg_archive(
        name = "mac_edge",
        url = "https://msedge.sf.dl.delivery.mp.microsoft.com/filestreamingservice/files/58ef038e-b996-434e-9c2f-d8f71502826d/MicrosoftEdge-140.0.3485.81.pkg",
        sha256 = "b11cb1210e1732932741d06694044e6cee43b1b5f3599d30d9c62d449ce6ff73",
        move = {
            "MicrosoftEdge-140.0.3485.81.pkg/Payload/Microsoft Edge.app": "Edge.app",
        },
        build_file_content = """
load("@aspect_rules_js//js:defs.bzl", "js_library")
package(default_visibility = ["//visibility:public"])
Driver-Browser Version Sync

Updated msedgedriver versions should match the pinned Edge browser; confirm runtime compatibility across Linux and macOS to prevent WebDriver protocol mismatches in CI.

    http_archive(
        name = "linux_edgedriver",
        url = "https://msedgedriver.microsoft.com/140.0.3485.81/edgedriver_linux64.zip",
        sha256 = "6b10c828769672720b90879f7980817d491eb475eaf28dde3739e47cbe9a6537",
        build_file_content = """
load("@aspect_rules_js//js:defs.bzl", "js_library")
package(default_visibility = ["//visibility:public"])

exports_files(["msedgedriver"])

js_library(
    name = "msedgedriver-js",
    data = ["msedgedriver"],
)
""",
    )

    http_archive(
        name = "mac_edgedriver",
        url = "https://msedgedriver.microsoft.com/140.0.3485.81/edgedriver_mac64.zip",
        sha256 = "aed876db66a8c5c74ab59fe2405f88e0a378e08bf12c447079cebd07f035c771",
        build_file_content = """
load("@aspect_rules_js//js:defs.bzl", "js_library")
package(default_visibility = ["//visibility:public"])

Copy link
Contributor

qodo-merge-pro bot commented Sep 20, 2025

PR Code Suggestions ✨

No code suggestions found for the PR.

@selenium-ci selenium-ci force-pushed the pinned-browser-updates branch from a1e92c2 to 2272389 Compare September 22, 2025 00:46
@navin772 navin772 merged commit 0417a8e into trunk Sep 22, 2025
32 checks passed
@navin772 navin772 deleted the pinned-browser-updates branch September 22, 2025 05:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

B-build Includes scripting, bazel and CI integrations Review effort 2/5

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants