From de74b32c3499267826ab36c512493c288ab266e9 Mon Sep 17 00:00:00 2001
From: "Tommy D. Rossi" <beats.by.morse@gmail.com>
Date: Thu, 26 Feb 2026 13:33:42 +0100
Subject: [PATCH] fix: skip diff mode when search string is provided in
 snapshot/markdown/html

When both showDiffSinceLastCall and search were set, the diff block
returned early and the search filter was silently ignored. Now all
three functions (snapshot, getPageMarkdown, getCleanHTML) skip diffing
when a search string is present, ensuring the agent always gets
filtered results.
---
 playwriter/src/clean-html.ts    | 4 ++--
 playwriter/src/executor.ts      | 4 ++--
 playwriter/src/page-markdown.ts | 4 ++--
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/playwriter/src/clean-html.ts b/playwriter/src/clean-html.ts
index 036fd86..6905e28 100644
--- a/playwriter/src/clean-html.ts
+++ b/playwriter/src/clean-html.ts
@@ -76,8 +76,8 @@ export async function getCleanHTML(options: GetCleanHTMLOptions): Promise<string
   const previousSnapshot = pageSnapshots.get(snapshotKey)
   pageSnapshots.set(snapshotKey, htmlStr)
 
-  // Return diff if we have a previous snapshot and diff mode is enabled
-  if (showDiffSinceLastCall && previousSnapshot) {
+  // Never diff when agent is searching — search should always filter the full content
+  if (showDiffSinceLastCall && previousSnapshot && !search) {
     const diffResult = createSmartDiff({
       oldContent: previousSnapshot,
       newContent: htmlStr,
diff --git a/playwriter/src/executor.ts b/playwriter/src/executor.ts
index f950188..48de56d 100644
--- a/playwriter/src/executor.ts
+++ b/playwriter/src/executor.ts
@@ -772,8 +772,8 @@ export class PlaywrightExecutor {
           this.lastSnapshots.set(resolvedPage, snapshotStr)
         }
 
-        // Return diff if we have a previous snapshot and diff mode is enabled
-        if (showDiffSinceLastCall && previousSnapshot && shouldCacheSnapshot) {
+        // Never diff when agent is searching — search should always filter the full snapshot
+        if (showDiffSinceLastCall && previousSnapshot && shouldCacheSnapshot && !search) {
           const diffResult = createSmartDiff({
             oldContent: previousSnapshot,
             newContent: snapshotStr,
diff --git a/playwriter/src/page-markdown.ts b/playwriter/src/page-markdown.ts
index 97097b4..3020cd6 100644
--- a/playwriter/src/page-markdown.ts
+++ b/playwriter/src/page-markdown.ts
@@ -172,8 +172,8 @@ export async function getPageMarkdown(options: GetPageMarkdownOptions): Promise<
   const previousSnapshot = lastMarkdownSnapshots.get(page)
   lastMarkdownSnapshots.set(page, markdown)
 
-  // Return diff if we have a previous snapshot and diff mode is enabled
-  if (showDiffSinceLastCall && previousSnapshot) {
+  // Never diff when agent is searching — search should always filter the full content
+  if (showDiffSinceLastCall && previousSnapshot && !search) {
     const diffResult = createSmartDiff({
       oldContent: previousSnapshot,
       newContent: markdown,