browserclaw 0.1.0 → 0.1.1

package/dist/index.js

@@ -603,13 +603,13 @@ function storeRoleRefsForTarget(opts) {
 function restoreRoleRefsForTarget(opts) {
   const targetId = opts.targetId?.trim() || "";
   if (!targetId) return;
-  const cached2 = roleRefsByTarget.get(roleRefsKey(opts.cdpUrl, targetId));
-  if (!cached2) return;
+  const entry = roleRefsByTarget.get(roleRefsKey(opts.cdpUrl, targetId));
+  if (!entry) return;
   const state = ensurePageState(opts.page);
   if (state.roleRefs) return;
-  state.roleRefs = cached2.refs;
-  state.roleRefsFrameSelector = cached2.frameSelector;
-  state.roleRefsMode = cached2.mode;
+  state.roleRefs = entry.refs;
+  state.roleRefsFrameSelector = entry.frameSelector;
+  state.roleRefsMode = entry.mode;
 }
 async function connectBrowser(cdpUrl) {
   const normalized = normalizeCdpUrl(cdpUrl);
@@ -726,8 +726,8 @@ function toAIFriendlyError(error, selector) {
   }
   return error instanceof Error ? error : new Error(message);
 }
-function normalizeTimeoutMs(timeoutMs, fallback) {
-  return Math.max(500, Math.min(12e4, timeoutMs ?? fallback));
+function normalizeTimeoutMs(timeoutMs, fallback, maxMs = 12e4) {
+  return Math.max(500, Math.min(maxMs, timeoutMs ?? fallback));
 }
 
 // src/snapshot/ref-map.ts
@@ -987,7 +987,7 @@ async function snapshotAi(opts) {
     throw new Error("Playwright _snapshotForAI is not available. Upgrade playwright-core to >= 1.50.");
   }
   const result = await maybe._snapshotForAI({
-    timeout: Math.max(500, Math.min(6e4, Math.floor(opts.timeoutMs ?? 5e3))),
+    timeout: normalizeTimeoutMs(opts.timeoutMs, 5e3, 6e4),
     track: "response"
   });
   let snapshot = String(result?.full ?? "");
@@ -1101,7 +1101,7 @@ async function clickViaPlaywright(opts) {
   ensurePageState(page);
   restoreRoleRefsForTarget({ cdpUrl: opts.cdpUrl, targetId: opts.targetId, page });
   const locator = refLocator(page, opts.ref);
-  const timeout = Math.max(500, Math.min(6e4, Math.floor(opts.timeoutMs ?? 8e3)));
+  const timeout = normalizeTimeoutMs(opts.timeoutMs, 8e3, 6e4);
   try {
     if (opts.doubleClick) {
       await locator.dblclick({ timeout, button: opts.button, modifiers: opts.modifiers });
@@ -1118,7 +1118,7 @@ async function hoverViaPlaywright(opts) {
   restoreRoleRefsForTarget({ cdpUrl: opts.cdpUrl, targetId: opts.targetId, page });
   try {
     await refLocator(page, opts.ref).hover({
-      timeout: Math.max(500, Math.min(6e4, opts.timeoutMs ?? 8e3))
+      timeout: normalizeTimeoutMs(opts.timeoutMs, 8e3, 6e4)
     });
   } catch (err) {
     throw toAIFriendlyError(err, opts.ref);
@@ -1130,7 +1130,7 @@ async function typeViaPlaywright(opts) {
   ensurePageState(page);
   restoreRoleRefsForTarget({ cdpUrl: opts.cdpUrl, targetId: opts.targetId, page });
   const locator = refLocator(page, opts.ref);
-  const timeout = Math.max(500, Math.min(6e4, opts.timeoutMs ?? 8e3));
+  const timeout = normalizeTimeoutMs(opts.timeoutMs, 8e3, 6e4);
   try {
     if (opts.slowly) {
       await locator.click({ timeout });
@@ -1150,7 +1150,7 @@ async function selectOptionViaPlaywright(opts) {
   restoreRoleRefsForTarget({ cdpUrl: opts.cdpUrl, targetId: opts.targetId, page });
   try {
     await refLocator(page, opts.ref).selectOption(opts.values, {
-      timeout: Math.max(500, Math.min(6e4, opts.timeoutMs ?? 8e3))
+      timeout: normalizeTimeoutMs(opts.timeoutMs, 8e3, 6e4)
     });
   } catch (err) {
     throw toAIFriendlyError(err, opts.ref);
@@ -1162,7 +1162,7 @@ async function dragViaPlaywright(opts) {
   restoreRoleRefsForTarget({ cdpUrl: opts.cdpUrl, targetId: opts.targetId, page });
   try {
     await refLocator(page, opts.startRef).dragTo(refLocator(page, opts.endRef), {
-      timeout: Math.max(500, Math.min(6e4, opts.timeoutMs ?? 8e3))
+      timeout: normalizeTimeoutMs(opts.timeoutMs, 8e3, 6e4)
     });
   } catch (err) {
     throw toAIFriendlyError(err, `${opts.startRef} -> ${opts.endRef}`);
@@ -1172,12 +1172,12 @@ async function fillFormViaPlaywright(opts) {
   const page = await getPageForTargetId({ cdpUrl: opts.cdpUrl, targetId: opts.targetId });
   ensurePageState(page);
   restoreRoleRefsForTarget({ cdpUrl: opts.cdpUrl, targetId: opts.targetId, page });
-  const timeout = Math.max(500, Math.min(6e4, opts.timeoutMs ?? 8e3));
+  const timeout = normalizeTimeoutMs(opts.timeoutMs, 8e3, 6e4);
   for (const field of opts.fields) {
     const ref = field.ref.trim();
     const type = field.type.trim();
     const rawValue = field.value;
-    const value = typeof rawValue === "string" ? rawValue : typeof rawValue === "number" || typeof rawValue === "boolean" ? String(rawValue) : "";
+    const value = rawValue == null ? "" : String(rawValue);
     if (!ref || !type) continue;
     const locator = refLocator(page, ref);
     if (type === "checkbox" || type === "radio") {
@@ -1565,12 +1565,12 @@ var CrawlPage = class {
       return snapshotRole({
         cdpUrl: this.cdpUrl,
         targetId: this.targetId,
-        selector: opts?.selector,
-        frameSelector: opts?.frameSelector,
+        selector: opts.selector,
+        frameSelector: opts.frameSelector,
         options: {
-          interactive: opts?.interactive,
-          compact: opts?.compact,
-          maxDepth: opts?.maxDepth
+          interactive: opts.interactive,
+          compact: opts.compact,
+          maxDepth: opts.maxDepth
         }
       });
     }