playwriter
Code Issues

improve accessibilitySnapshot diff: preserve baseline, use contextLines for hunks

Browse Source
This commit is contained in:
Tommy D. Rossi
2025-11-27 15:25:29 +01:00
parent 36d0458243
commit 85dd937fe4
2 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
playwriter/src/mcp.ts
+4 -2
View File
@@ -404,13 +404,15 @@ server.tool(
if (showDiffSinceLastCall) { if (showDiffSinceLastCall) {
const previousSnapshot = lastSnapshots.get(targetPage) const previousSnapshot = lastSnapshots.get(targetPage)
lastSnapshots.set(targetPage, snapshotStr)
if (!previousSnapshot) { if (!previousSnapshot) {
lastSnapshots.set(targetPage, snapshotStr)
return 'No previous snapshot available. This is the first call for this page. Full snapshot stored for next diff.' return 'No previous snapshot available. This is the first call for this page. Full snapshot stored for next diff.'
} }
const patch = createPatch('snapshot', previousSnapshot, snapshotStr, 'previous', 'current') const patch = createPatch('snapshot', previousSnapshot, snapshotStr, 'previous', 'current', {
context: contextLines,
})
if (patch.split('\n').length <= 4) { if (patch.split('\n').length <= 4) {
return 'No changes detected since last snapshot' return 'No changes detected since last snapshot'
} }
playwriter/src/prompt.md
+2 -2
View File
@@ -75,8 +75,8 @@ you have access to some functions in addition to playwright methods:
- `async accessibilitySnapshot({ page, search, contextLines, showDiffSinceLastCall })`: gets a human readable snapshot of clickable elements on the page. useful to see the overall structure of the page and what elements you can interact with. - `async accessibilitySnapshot({ page, search, contextLines, showDiffSinceLastCall })`: gets a human readable snapshot of clickable elements on the page. useful to see the overall structure of the page and what elements you can interact with.
- `page`: the page object to snapshot - `page`: the page object to snapshot
- `search`: (optional) a string or regex to filter the snapshot. If provided, returns the first 10 matches with surrounding context - `search`: (optional) a string or regex to filter the snapshot. If provided, returns the first 10 matches with surrounding context
- `contextLines`: (optional) number of lines of context to show around each match (default: 10) - `contextLines`: (optional) number of lines of context to show around each match (default: 10). Also controls context lines in diff output.
- `showDiffSinceLastCall`: (optional) if true, returns a unified diff patch showing only changes since the last snapshot call for this page. Disables search when enabled. Useful to see what changed after an action. - `showDiffSinceLastCall`: (optional) if true, returns a unified diff patch showing only changes since the last non-diff snapshot call for this page. Disables search when enabled. Useful to see what changed after an action. Note: diff calls do not update the stored snapshot, so you can call diff multiple times and always compare against the same baseline.
- `getLatestLogs({ page, count, search })`: retrieves browser console logs. The system automatically captures and stores up to 5000 logs per page. Logs are cleared when a page reloads or navigates. - `getLatestLogs({ page, count, search })`: retrieves browser console logs. The system automatically captures and stores up to 5000 logs per page. Logs are cleared when a page reloads or navigates.
- `page`: (optional) filter logs by a specific page instance. Only returns logs from that page - `page`: (optional) filter logs by a specific page instance. Only returns logs from that page
- `count`: (optional) limit number of logs to return. If not specified, returns all available logs - `count`: (optional) limit number of logs to return. If not specified, returns all available logs