Commit b3ce99e
authored
Add mode, depth and boxes arguments and a parsed return type to Get Aria Snapshot (#5163)
Get Aria Snapshot now exposes the three options that locator.ariaSnapshot()
already offers in the pinned Playwright version, and gains a return type that
makes a snapshot usable from a test without string parsing.
New named-only arguments:
- mode=ai returns the snapshot optimized for AI consumption. It adds element
references such as [ref=e2], includes snapshots of iframes inside the target,
and does not wait for a matching element: it fails immediately instead of
running into the usual timeout. That last difference is documented on the
keyword and covered by its own test.
- depth limits the snapshot to the given number of tree levels. Values of zero
or less are rejected with a ValueError.
- boxes appends [box=x,y,width,height] to every node.
New return type:
return_type=dict loads Playwright's yaml as it is, which keeps role, name and
every annotation inside the dictionary keys. boxes= and mode=ai made that worse
by adding more annotations. return_type=parsed splits a snapshot into a tree of
nodes with role, name, text, props and children. Annotations without a value,
such as [selected], become True, [level=2] becomes an integer, and box uses the
same keys as Get BoundingBox. The /url entry of a link is an annotation in
Playwright rather than an element, so it becomes the url property of the link.
Playwright's own parseAriaSnapshot is bundled but not exported, so the parsing
happens on the Python side in Browser/utils/aria_snapshot.py.
Everything is additive. The defaults reproduce the previous output byte for
byte, yaml and dict are untouched, and no existing return value changed.
Tests: unit tests for the parser covering escaped quotes, valueless
annotations, text nodes, hoisted /url and unknown future label syntax, plus
acceptance tests for every argument and return type. One acceptance test
compares a complete parsed tree against a full expected structure on a page
that exercises every node key in both of its shapes; it was mutation-tested
against dropped children, swapped name and text, and an unstripped /url.
Closes #5101
1 parent 35fe86a commit b3ce99e
8 files changed
Lines changed: 707 additions & 7 deletions
File tree
- Browser
- keywords
- utils
- atest/test/02_Content_Keywords
- node/playwright-wrapper
- protobuf
- utest
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
69 | 69 | | |
70 | 70 | | |
71 | 71 | | |
72 | | - | |
| 72 | + | |
73 | 73 | | |
74 | 74 | | |
75 | 75 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
37 | 37 | | |
38 | 38 | | |
39 | 39 | | |
| 40 | + | |
40 | 41 | | |
41 | 42 | | |
42 | 43 | | |
| 44 | + | |
43 | 45 | | |
44 | 46 | | |
45 | 47 | | |
| |||
69 | 71 | | |
70 | 72 | | |
71 | 73 | | |
72 | | - | |
73 | | - | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
74 | 80 | | |
75 | 81 | | |
76 | 82 | | |
77 | | - | |
| 83 | + | |
78 | 84 | | |
79 | 85 | | |
80 | 86 | | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
81 | 90 | | |
82 | 91 | | |
83 | 92 | | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
84 | 103 | | |
85 | 104 | | |
86 | 105 | | |
| |||
93 | 112 | | |
94 | 113 | | |
95 | 114 | | |
| 115 | + | |
| 116 | + | |
96 | 117 | | |
97 | 118 | | |
98 | 119 | | |
99 | | - | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
| 126 | + | |
100 | 127 | | |
101 | 128 | | |
102 | 129 | | |
103 | 130 | | |
104 | 131 | | |
105 | 132 | | |
| 133 | + | |
| 134 | + | |
106 | 135 | | |
107 | 136 | | |
108 | 137 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
0 commit comments