browserclaw 0.3.2 → 0.3.4

package/dist/index.d.cts

@@ -151,6 +151,8 @@ interface SnapshotOptions {
      * - `'role'` — uses Playwright's `ariaSnapshot()` + `getByRole()` resolution
      */
     mode?: 'role' | 'aria';
+    /** Timeout in milliseconds for the snapshot operation (role mode only, default: 5000) */
+    timeoutMs?: number;
     /**
      * How refs are stored for role-mode snapshots.
      * - `'role'` (default) — refs resolved via `getByRole()`

package/dist/index.d.ts

@@ -151,6 +151,8 @@ interface SnapshotOptions {
      * - `'role'` — uses Playwright's `ariaSnapshot()` + `getByRole()` resolution
      */
     mode?: 'role' | 'aria';
+    /** Timeout in milliseconds for the snapshot operation (role mode only, default: 5000) */
+    timeoutMs?: number;
     /**
      * How refs are stored for role-mode snapshots.
      * - `'role'` (default) — refs resolved via `getByRole()`

package/dist/index.js

@@ -742,12 +742,13 @@ async function getPageForTargetId(opts) {
   const found = await findPageByTargetId(browser, opts.targetId, opts.cdpUrl);
   if (!found) {
     if (pages.length === 1) return first;
-    throw new Error(`Tab not found (targetId: ${opts.targetId}). Use browser.tabs() to list open tabs.`);
+    throw new Error(`Tab not found (targetId: ${opts.targetId}). Call browser.tabs() to list open tabs.`);
   }
   return found;
 }
 function refLocator(page, ref) {
   const normalized = ref.startsWith("@") ? ref.slice(1) : ref.startsWith("ref=") ? ref.slice(4) : ref;
+  if (!normalized.trim()) throw new Error("ref is required");
   if (/^e\d+$/.test(normalized)) {
     const state = pageStates.get(page);
     if (state?.roleRefsMode === "aria") {
@@ -1096,7 +1097,7 @@ async function snapshotRole(opts) {
   const frameSelector = opts.frameSelector?.trim() || "";
   const selector = opts.selector?.trim() || "";
   const locator = frameSelector ? selector ? page.frameLocator(frameSelector).locator(selector) : page.frameLocator(frameSelector).locator(":root") : selector ? page.locator(selector) : page.locator(":root");
-  const ariaSnapshot = await locator.ariaSnapshot({ timeout: 1e4 });
+  const ariaSnapshot = await locator.ariaSnapshot({ timeout: normalizeTimeoutMs(opts.timeoutMs, 5e3) });
   const built = buildRoleSnapshotFromAriaSnapshot(String(ariaSnapshot ?? ""), opts.options);
   storeRoleRefsForTarget({
     page,
@@ -1825,7 +1826,7 @@ async function setGeolocationViaPlaywright(opts) {
     return;
   }
   if (opts.latitude === void 0 || opts.longitude === void 0) {
-    throw new Error("latitude and longitude are required when not clearing geolocation.");
+    throw new Error("latitude and longitude are required (or set clear=true)");
   }
   await context.grantPermissions(["geolocation"], opts.origin ? { origin: opts.origin } : void 0);
   await context.setGeolocation({
@@ -1972,8 +1973,9 @@ async function responseBodyViaPlaywright(opts) {
   const response = await page.waitForResponse(opts.url, { timeout });
   let body = await response.text();
   let truncated = false;
-  if (opts.maxChars && body.length > opts.maxChars) {
-    body = body.slice(0, opts.maxChars);
+  const maxChars = typeof opts.maxChars === "number" && Number.isFinite(opts.maxChars) ? Math.max(1, Math.min(5e6, Math.floor(opts.maxChars))) : void 0;
+  if (maxChars !== void 0 && body.length > maxChars) {
+    body = body.slice(0, maxChars);
     truncated = true;
   }
   const headers = {};
@@ -2144,6 +2146,7 @@ var CrawlPage = class {
         selector: opts.selector,
         frameSelector: opts.frameSelector,
         refsMode: opts.refsMode,
+        timeoutMs: opts.timeoutMs,
         options: {
           interactive: opts.interactive,
           compact: opts.compact,