[Bug]: RegExp values are not serialized by evaluate(), and returned RegExps leak internal protocol JSON
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 72/100
Hướng nghiên cứu
Bắt đầu trong playwright/_impl/_js_handle.py bằng cách kiểm tra serialize_value() và parse_value(), sau đó xem lại escape_regex_flags() trong _str_utils.py để kiểm tra cách xử lý flag hiện có. Tái hiện ba trường hợp evaluate() từ issue và bổ sung coverage cho các pattern đã biên dịch, các RegExps JavaScript được trả về và các giá trị lồng nhau. Công việc được xem là hoàn tất khi các giá trị regex của Python được serialize chính xác và các RegExps JavaScript được trả về trở thành các đối tượng re.Pattern thay vì các dictionary của protocol.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Version
1.62.0
Steps to reproduce
import re
from playwright.sync_api import sync_playwright
with sync_playwright() as p:
browser = p.chromium.launch()
page = browser.new_page()
# 1. Passing a compiled pattern in
print(page.evaluate("(v) => [typeof v, String(v)]", re.compile(r"a\d+", re.I)))
# 2. Getting a RegExp back out
print(page.evaluate("() => /a\\d+/gi"))
# 3. Nested in a structure
print(page.evaluate("() => ({x: [/foo/m]})"))
browser.close()
Expected behavior
['object', '/a\\d+/i'], and the value is a realRegExpinside the page.re.compile('a\\d+', re.IGNORECASE){'x': [re.compile('foo', re.MULTILINE)]}
This is what playwright-dotnet does today. EvaluateArgumentValueConverter.cs serializes Regex (line 138) and parses "r" back into a Regex (line 351).
Actual behavior
['undefined', 'undefined'], the argument is silently dropped{'r': {'p': 'a\\d+', 'f': 'gi'}}{'x': [{'r': {'p': 'foo', 'f': 'm'}}]}
Two separate problems, both silent:
serialize_value()inplaywright/_impl/_js_handle.pyhas no branch forre.Pattern, so a compiled pattern falls through to{"v": "undefined"}.parse_value()has no branch for"r", so it hits the barereturn valueat the end and hands back the raw wire format. There is no way to tell from the outside that this dict is not the actual result.
The protocol supports this in both directions and the driver already implements both sides ({r: {p, f}} on serialize, new RegExp(v.r.p, v.r.f) on parse), so this is only missing on the Python side.
Additional context
Happy to send a PR, I have one ready. The serialize direction can reuse the existing escape_regex_flags() in _str_utils.py, which already maps re.IGNORECASE|DOTALL|MULTILINE to i/s/m for locators and route matching. The parse direction needs the inverse.
Two things worth an opinion before I do:
- JavaScript flags with no
reequivalent (g,y,d,u,v). I would drop them rather than raise, since/foo/gis very common and raising would be worse than today's behavior. Note that playwright-dotnet'sFromInlineFlagsthrows on these, which looks like a separate bug over there. - A JS pattern that is not valid Python
resyntax, for example/(?<name>x)/(Python spells it(?P<name>x)). With this change that would raise fromre.compileinstead of returning the dict. I think raising is acceptable since the dict was never usable anyway, but let me know if you would rather it degrade some other way.
Environment
- Operating System: macOS 26.3.1
- CPU: arm64
- Browser: Chromium
- Python Version: 3.14.0
- Ngôn ngữ chính
- Python
- Star
- 15k
- Fork
- 1.2k
- Merge trung bình
- 55 phút
- Pull request đã merge (30 ngày)
- 4
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của microsoft/playwright-python
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
microsoft/playwright-python#3186 ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 1/100
microsoft/playwright-python#3195 ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 1/100
microsoft/playwright-python#3172 ·
Tất cả issue của microsoft/playwright-python
Issue tương tự
-
bug ci good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
-
documentation
Độ khó 2/5 Nửa ngày Mức phù hợp với người mới 62/100
inmanta/inmanta-core#10835 ·
-
sponsored
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
-
opensubtitlescom: moviehash never sent when opensubtitles (.org) is not in the provider list Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
Diaoul/subliminal#1382 ·