Gate cached replay fallback logs behind debug flag

Co-authored-by: Shri Sukhani <shrisukhani@users.noreply.github.com>

Author: cursoragent

src/agent/shared/run-cached-action.test.ts

@@ -200,6 +200,51 @@ describe("runCachedStep", () => {
           fallbackElementId: "0-1",
         })
       );
+      expect(logSpy).not.toHaveBeenCalled();
+    } finally {
+      logSpy.mockRestore();
+    }
+  });
+
+  it("logs cached fallback diagnostics only in debug mode", async () => {
+    executeReplaySpecialAction.mockResolvedValue(null);
+    resolveXPathWithCDP.mockRejectedValue(new Error("xpath resolution failed"));
+    const performFallback = jest.fn().mockResolvedValue({
+      taskId: "fallback-task",
+      status: TaskStatus.COMPLETED,
+      steps: [],
+      output: "fallback completed",
+      replayStepMeta: {
+        usedCachedAction: false,
+        fallbackUsed: true,
+        retries: 1,
+        fallbackXPath: "/html/body/button[1]",
+        fallbackElementId: "0-1",
+      },
+    });
+    const logSpy = jest.spyOn(console, "log").mockImplementation(() => {});
+
+    try {
+      await runCachedStep({
+        page: createMockPage(),
+        instruction: "click login",
+        cachedAction: {
+          actionType: "actElement",
+          xpath: "//button[1]",
+          method: "click",
+          frameIndex: 0,
+          arguments: [],
+        },
+        maxSteps: 1,
+        debug: true,
+        tokenLimit: 8000,
+        llm: createMockLLM(),
+        mcpClient: undefined,
+        variables: [],
+        performFallback,
+      });
+
+      expect(logSpy).toHaveBeenCalledTimes(1);
     } finally {
       logSpy.mockRestore();
     }

src/agent/shared/run-cached-action.ts

@@ -139,17 +139,19 @@ export async function runCachedStep(
   // All cached attempts failed; optionally fall back to LLM perform
   if (params.performFallback) {
     const fb = await params.performFallback(instruction);
-    const cachedXPath = cachedAction.xpath || "N/A";
-    const resolvedXPath = fb.replayStepMeta?.fallbackXPath || "N/A";
-    // eslint-disable-next-line no-console
-    console.log(
-      `
+    if (debug) {
+      const cachedXPath = cachedAction.xpath || "N/A";
+      const resolvedXPath = fb.replayStepMeta?.fallbackXPath || "N/A";
+      // eslint-disable-next-line no-console
+      console.log(
+        `
 ⚠️ [runCachedStep] Cached action failed. Falling back to LLM...
    Instruction: "${instruction}"
    ❌ Cached XPath Failed: "${cachedXPath}"
    ✅ LLM Resolved New XPath: "${resolvedXPath}"
 `
-    );
+      );
+    }
     return {
       ...fb,
       replayStepMeta: {