chore(roll): v1.61.0

.claude/skills/playwright-roll/SKILL.md

@@ -27,21 +27,6 @@ The upstream documentation source of truth is `docs/src/api/*.md` in the playwri
 
 > **The mistake the 1.59 roll made twice over:** classifying things as "internal tooling, N/A for Python" based on the *name* of the API (Screencast, Debugger, pickLocator, clearConsoleMessages, artifactsDir, …). Almost all of those had empty `langs: {}` in `api.json` and were real Python APIs. Sounding tooling-y is not a `langs` filter. **The `langs` field on the member in `api.json` is the only authoritative signal.** When in doubt, dump it (see "Verifying classifications" below).
 
-## Pre-flight
-
-You will need two checkouts in the parent directory:
-- `~/code/playwright-python` — this repo.
-- `~/code/playwright` — the upstream playwright monorepo (used read-only for diffing).
-
-Bring upstream up to date and ensure release branches/tags are present:
-
-```sh
-git -C ~/code/playwright fetch --tags
-git -C ~/code/playwright fetch origin 'release-*:release-*'
-```
-
-There is sometimes no `vX.Y.0` tag for the latest release (the bots cut release branches first and tag later). Anchor on commits, not tags — see "Identify the commit range" below.
-
 ## Process
 
 ### 1. Set up the env
@@ -76,18 +61,29 @@ build + per-platform Node downloads).
 
 ### 3. Identify the commit range
 
+The build step (step 2) clones the upstream monorepo into `driver/playwright-src`.
+Bring it up to date and ensure release branches/tags are present before walking
+the range:
+
+```sh
+git -C driver/playwright-src fetch --tags
+git -C driver/playwright-src fetch origin 'release-*:release-*'
+```
+
+There is sometimes no `vX.Y.0` tag for the latest release (the bots cut release branches first and tag later). Anchor on commits, not tags.
+
 The diff range is "every commit on the new release branch since the previous release was cut". Anchor commits:
 
 - **Previous release end**: the `chore: bump version to vX.Y.0-next` commit on `main`. That commit is the first commit *after* the previous release (X.Y-1) was cut. Use its parent (`<sha>~1`) as the lower bound.
   ```sh
-  git -C ~/code/playwright log --all --grep="bump version to v" --oneline | head
+  git -C driver/playwright-src log --all --grep="bump version to v" --oneline | head
   ```
 - **New release end**: the tip of `release-<new>` (or the matching tag if it exists).
 
 Save the commit list, oldest first, scoped to `docs/src/api/`:
 
 ```sh
-git -C ~/code/playwright log <prev-anchor>~1..release-<new> --oneline --reverse -- docs/src/api > /tmp/roll-<new>-commits.md
+git -C driver/playwright-src log <prev-anchor>~1..release-<new> --oneline --reverse -- docs/src/api > /tmp/roll-<new>-commits.md
 ```
 
 A normal roll yields 50–100 commits. If you see 0 or thousands, the range is wrong.
@@ -99,7 +95,7 @@ Format the file as a markdown checklist and add the standard preamble (status le
 For each commit, in chronological order:
 
 ```sh
-git -C ~/code/playwright show <sha> -- docs/src/api/
+git -C driver/playwright-src show <sha> -- docs/src/api/
 ```
 
 Look for:
@@ -144,7 +140,7 @@ A few rules of thumb that catch most "actually a PORT" cases:
 
 #### PORT
 
-Implement the change in `playwright/_impl/<module>.py`. Use the upstream JS implementation as a reference: `~/code/playwright/packages/playwright-core/src/client/<module>.ts`. Translate idioms:
+Implement the change in `playwright/_impl/<module>.py`. Use the upstream JS implementation as a reference: `driver/playwright-src/packages/playwright-core/src/client/<module>.ts`. Translate idioms:
 
 | Upstream JS | Python |
 |---|---|
@@ -285,7 +281,7 @@ Class names use the upstream PascalCase (`BrowserContext`, `BrowserType`); metho
 - **A cluster of suppressions on the same class is a smell.** If you're about to add five `Method not implemented: Foo.*` lines, you're almost certainly looking at a class that needs to be implemented. Implement the whole thing once and the suppressions disappear.
 - **Watch for revert pairs in the same range.** 1.59 added and reverted `Browser.isRemote` (#39613 / #39620) inside the same release. Walking chronologically lets you skip the add when you see the revert later.
 - **Watch for rename-revert pairs.** 1.59 had `Locator.normalize` → `Locator.toCode` (#39648) → `Locator.normalize` (#39754). Final state wins; only port the last.
-- **Doc renames almost always include a wire-protocol rename.** Whenever you see `### param: X.y.oldName` → `### param: X.y.newName` in a doc commit, also `git -C ~/code/playwright show <sha> -- packages/protocol/src/protocol.yml` and the corresponding `*Dispatcher.ts` file. If the wire field changed too, the channel-send dict key in `_impl/` must change. Suppressing the doc-side mismatch is hiding a real bug — the previous Python code is silently sending an unknown field that the new server ignores.
+- **Doc renames almost always include a wire-protocol rename.** Whenever you see `### param: X.y.oldName` → `### param: X.y.newName` in a doc commit, also `git -C driver/playwright-src show <sha> -- packages/protocol/src/protocol.yml` and the corresponding `*Dispatcher.ts` file. If the wire field changed too, the channel-send dict key in `_impl/` must change. Suppressing the doc-side mismatch is hiding a real bug — the previous Python code is silently sending an unknown field that the new server ignores.
 - **TypedDicts beat `Dict[str, X]` for any structured return.** When the docs describe a return as `[Object]` with named fields (or even `[Object=Foo]`), define a `TypedDict` in `_api_structures.py`, re-export from both public `__init__.py` files, and use it. Zero runtime cost (it's still a `dict`), and the doc generator's type comparator matches by structure via `get_type_hints`.
 - **Positional renames are free.** A param with no default before any `*` separator is positional-or-keyword in Python, but realistic call sites pass it positionally. Renaming such a param doesn't break callers.
 - **The "Backport changes" GitHub issue can be misleading.** In the 1.59 roll its checkboxes were all marked `[x]` with annotations like "✅ IMPLEMENTED", but several of those features had not actually been merged into the Python port. Trust the `docs/src/api/` walk over the issue.

DRIVER_SHA

@@ -1 +1 @@
-87bb9ddbd78f329df18c2b24847bc9409240cd07
+ac7cdd4bdf15f90fe7229243be6b35a53e0296d1

README.md

@@ -4,9 +4,9 @@ Playwright is a Python library to automate [Chromium](https://www.chromium.org/H
 
 |          | Linux | macOS | Windows |
 |   :---   | :---: | :---: | :---:   |
-| Chromium <!-- GEN:chromium-version -->148.0.7778.96<!-- GEN:stop --> | ✅ | ✅ | ✅ |
+| Chromium <!-- GEN:chromium-version -->149.0.7827.22<!-- GEN:stop --> | ✅ | ✅ | ✅ |
 | WebKit <!-- GEN:webkit-version -->26.4<!-- GEN:stop --> | ✅ | ✅ | ✅ |
-| Firefox <!-- GEN:firefox-version -->150.0.2<!-- GEN:stop --> | ✅ | ✅ | ✅ |
+| Firefox <!-- GEN:firefox-version -->151.0<!-- GEN:stop --> | ✅ | ✅ | ✅ |
 
 ## Documentation
 

playwright/_impl/_api_structures.py

@@ -344,7 +344,21 @@ class DebuggerPausedDetails(TypedDict):
     title: str
 
 
+class ScreencastSize(TypedDict):
+    width: int
+    height: int
+
+
+class VirtualCredential(TypedDict):
+    id: str
+    rpId: str
+    userHandle: str
+    privateKey: str
+    publicKey: str
+
+
 class ScreencastFrame(TypedDict):
     data: bytes
+    timestamp: float
     viewportWidth: int
     viewportHeight: int

playwright/_impl/_assertions.py

@@ -21,13 +21,11 @@
     AriaRole,
     ExpectedTextValue,
     FrameExpectOptions,
-    FrameExpectResult,
 )
 from playwright._impl._connection import format_call_log
 from playwright._impl._errors import Error
 from playwright._impl._fetch import APIResponse
 from playwright._impl._helper import is_textual_mime_type
-from playwright._impl._js_handle import parse_value
 from playwright._impl._locator import Locator
 from playwright._impl._page import Page
 from playwright._impl._str_utils import escape_regex_flags
@@ -79,7 +77,7 @@ def __init__(
 
     async def _call_expect(
         self, expression: str, expect_options: FrameExpectOptions, title: Optional[str]
-    ) -> FrameExpectResult:
+    ) -> None:
         raise NotImplementedError(
             "_call_expect must be implemented in a derived class."
         )
@@ -100,35 +98,16 @@ async def _expect_impl(
             message = message.replace("expected to", "expected not to")
         if "useInnerText" in expect_options and expect_options["useInnerText"] is None:
             del expect_options["useInnerText"]
-        result = await self._call_expect(expression, expect_options, title)
-        if result["matches"] == self._is_not:
-            received = result.get("received") or {}
-            aria_snapshot = None
-            if isinstance(received, dict):
-                aria_snapshot = received.get("ariaSnapshot")
-                value = received.get("value")
-                actual = parse_value(value) if value is not None else None
-            else:
-                actual = received
+        try:
+            await self._call_expect(expression, expect_options, title)
+        except Error as e:
             if self._custom_message:
                 out_message = self._custom_message
                 if expected is not None:
                     out_message += f"\nExpected value: '{expected or '<None>'}'"
             else:
-                out_message = (
-                    f"{message} '{expected}'" if expected is not None else f"{message}"
-                )
-            error_message = result.get("errorMessage")
-            error_message = f"\n{error_message}" if error_message else ""
-            aria_snapshot_message = (
-                f"\nAria snapshot:\n{aria_snapshot}" if aria_snapshot else ""
-            )
-            _record_soft_or_raise(
-                AssertionError(
-                    f"{out_message}\nActual value: {actual}{error_message} {format_call_log(result.get('log'))}{aria_snapshot_message}"
-                ),
-                self._is_soft,
-            )
+                out_message = str(e)
+            _record_soft_or_raise(AssertionError(out_message), self._is_soft)
 
 
 class PageAssertions(AssertionsBase):
@@ -145,9 +124,9 @@ def __init__(
 
     async def _call_expect(
         self, expression: str, expect_options: FrameExpectOptions, title: Optional[str]
-    ) -> FrameExpectResult:
+    ) -> None:
         __tracebackhide__ = True
-        return await self._actual_page.main_frame._expect(
+        await self._actual_page.main_frame._expect(
             None, expression, expect_options, title
         )
 
@@ -243,9 +222,9 @@ def __init__(
 
     async def _call_expect(
         self, expression: str, expect_options: FrameExpectOptions, title: Optional[str]
-    ) -> FrameExpectResult:
+    ) -> None:
         __tracebackhide__ = True
-        return await self._actual_locator._expect(expression, expect_options, title)
+        await self._actual_locator._expect(expression, expect_options, title)
 
     @property
     def _not(self) -> "LocatorAssertions":

playwright/_impl/_browser_context.py

@@ -46,6 +46,7 @@
     from_nullable_channel,
 )
 from playwright._impl._console_message import ConsoleMessage
+from playwright._impl._credentials import Credentials
 from playwright._impl._debugger import Debugger
 from playwright._impl._dialog import Dialog
 from playwright._impl._disposable import Disposable, DisposableStub
@@ -133,6 +134,7 @@ def __init__(
         self._request: APIRequestContext = from_channel(initializer["requestContext"])
         self._request._timeout_settings = self._timeout_settings
         self._clock = Clock(self)
+        self._credentials = Credentials(self)
         self._channel.on(
             "bindingCall",
             lambda params: self._on_binding(from_channel(params["binding"])),
@@ -741,3 +743,7 @@ def request(self) -> "APIRequestContext":
     @property
     def clock(self) -> Clock:
         return self._clock
+
+    @property
+    def credentials(self) -> Credentials:
+        return self._credentials

playwright/_impl/_browser_type.py

@@ -202,6 +202,7 @@ async def connect_over_cdp(
         headers: Dict[str, str] = None,
         isLocal: bool = None,
         noDefaults: bool = None,
+        artifactsDir: Union[str, Path] = None,
     ) -> Browser:
         params = locals_to_params(locals())
         if params.get("headers"):

playwright/_impl/_credentials.py

@@ -0,0 +1,57 @@
+# Copyright (c) Microsoft Corporation.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+from typing import TYPE_CHECKING, List
+
+from playwright._impl._api_structures import VirtualCredential
+from playwright._impl._helper import locals_to_params
+
+if TYPE_CHECKING:
+    from playwright._impl._browser_context import BrowserContext
+
+
+class Credentials:
+    def __init__(self, browser_context: "BrowserContext") -> None:
+        self._browser_context = browser_context
+        self._loop = browser_context._loop
+        self._dispatcher_fiber = browser_context._dispatcher_fiber
+
+    async def install(self) -> None:
+        await self._browser_context._channel.send("credentialsInstall", None)
+
+    async def create(
+        self,
+        rpId: str,
+        id: str = None,
+        userHandle: str = None,
+        privateKey: str = None,
+        publicKey: str = None,
+    ) -> VirtualCredential:
+        result = await self._browser_context._channel.send_return_as_dict(
+            "credentialsCreate", None, locals_to_params(locals())
+        )
+        return (result or {})["credential"]
+
+    async def delete(self, id: str) -> None:
+        await self._browser_context._channel.send("credentialsDelete", None, {"id": id})
+
+    async def get(
+        self,
+        rpId: str = None,
+        id: str = None,
+    ) -> List[VirtualCredential]:
+        result = await self._browser_context._channel.send_return_as_dict(
+            "credentialsGet", None, locals_to_params(locals())
+        )
+        return (result or {}).get("credentials", [])

playwright/_impl/_fetch.py

@@ -28,6 +28,8 @@
     Headers,
     HttpCredentials,
     ProxySettings,
+    RemoteAddr,
+    SecurityDetails,
     ServerFilePayload,
     StorageState,
 )
@@ -557,6 +559,12 @@ async def json(self) -> Any:
         content = await self.text()
         return json.loads(content)
 
+    async def security_details(self) -> Optional[SecurityDetails]:
+        return self._initializer.get("securityDetails") or None
+
+    async def server_addr(self) -> Optional[RemoteAddr]:
+        return self._initializer.get("serverAddr") or None
+
     async def dispose(self) -> None:
         await self._request._channel.send(
             "disposeAPIResponse",

playwright/_impl/_frame.py

@@ -35,7 +35,6 @@
     DropPayload,
     FilePayload,
     FrameExpectOptions,
-    FrameExpectResult,
     Position,
 )
 from playwright._impl._connection import (
@@ -186,7 +185,7 @@ async def _expect(
         expression: str,
         options: FrameExpectOptions,
         title: str = None,
-    ) -> FrameExpectResult:
+    ) -> dict:
         if "expectedValue" in options:
             options["expectedValue"] = serialize_argument(options["expectedValue"])
         result = await self._channel.send_return_as_dict(
@@ -199,6 +198,8 @@ async def _expect(
             },
             title=title,
         )
+        if result is None:
+            return {}
         if result.get("received"):
             result["received"] = parse_value(result["received"])
         return result

playwright/_impl/_locator.py

@@ -37,7 +37,6 @@
     FilePayload,
     FloatRect,
     FrameExpectOptions,
-    FrameExpectResult,
     Position,
 )
 from playwright._impl._element_handle import ElementHandle
@@ -771,7 +770,7 @@ async def _expect(
         expression: str,
         options: FrameExpectOptions,
         title: str = None,
-    ) -> FrameExpectResult:
+    ) -> dict:
         return await self._frame._expect(self._selector, expression, options, title)
 
     async def highlight(self, style: str = None) -> None: