Normalize waitForLoadState targets in replay scripts

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

Author: cursoragent

src/agent/shared/action-cache-script.ts

@@ -38,6 +38,14 @@ const normalizeWaitMs = (value: unknown): number => {
   return parsed >= 0 ? parsed : 1000;
 };
 
+const normalizeWaitUntil = (value: unknown): "domcontentloaded" | "load" | "networkidle" => {
+  const parsed = asNonEmptyTrimmedString(value);
+  if (parsed === "load" || parsed === "networkidle") {
+    return parsed;
+  }
+  return "domcontentloaded";
+};
+
 export function createScriptFromActionCache(
   params: CreateScriptFromActionCacheParams
 ): string {
@@ -111,8 +119,7 @@ ${indent}await page.waitForTimeout(${waitMs});`;
       const actionParams = isRecord(step.actionParams)
         ? step.actionParams
         : undefined;
-      const waitUntil =
-        asNonEmptyTrimmedString(step.arguments?.[0]) ?? "domcontentloaded";
+      const waitUntil = normalizeWaitUntil(step.arguments?.[0]);
       const timeoutMs = asNumber(step.arguments?.[1] ?? actionParams?.timeout);
       if (typeof timeoutMs === "number" && Number.isFinite(timeoutMs)) {
         return `${indent}// Step ${step.stepIndex}

src/agent/shared/action-cache.test.ts

@@ -213,6 +213,28 @@ describe("action cache helpers", () => {
     expect(script).toContain('await page.waitForLoadState("domcontentloaded");');
   });
 
+  it("normalizes unsupported waitForLoadState targets to domcontentloaded", () => {
+    const waitEntry: ActionCacheEntry = {
+      stepIndex: 14,
+      instruction: "wait unsupported",
+      elementId: null,
+      method: null,
+      arguments: ["interactive"],
+      actionType: "waitForLoadState",
+      success: true,
+      message: "ok",
+      frameIndex: null,
+      xpath: null,
+    };
+
+    const script = createScriptFromActionCache({
+      steps: [waitEntry],
+    });
+
+    expect(script).toContain('await page.waitForLoadState("domcontentloaded");');
+    expect(script).not.toContain('await page.waitForLoadState("interactive");');
+  });
+
   it("skips helper generation when xpath is missing", () => {
     const actElementEntry: ActionCacheEntry = {
       stepIndex: 3,