@rtrvr-ai/rover 1.1.0 → 1.1.1

package/dist/worker/rover-worker.js

@@ -795,7 +795,6 @@ function buildRoverRuntimeContext(config2) {
   const compactExternalTabs = normalizeRuntimeExternalTabs(runtimeContext.externalTabs).slice(0, 8);
   return {
     mode: "rover_embed",
-    embeddedDomain: String(runtimeContext.embeddedDomain || "").trim() || void 0,
     agentName: normalizeAgentName(runtimeContext.agentName) || "Rover",
     externalNavigationPolicy: runtimeContext.externalNavigationPolicy || config2.externalNavigationPolicy,
     tabIdContract: runtimeContext.tabIdContract || "tree_index_mapped_by_tab_order",
@@ -803,7 +802,6 @@ function buildRoverRuntimeContext(config2) {
   };
 }
 function buildExternalPlaceholderPageData(params) {
-  const embedded = params.embeddedDomain || "the configured domain";
   const agentName = normalizeAgentName(params.agentName) || "Rover";
   const reason = params.reason || "external_domain_inaccessible";
   const title = params.title || "External Tab (Inaccessible)";
@@ -813,7 +811,7 @@ function buildExternalPlaceholderPageData(params) {
     url,
     title,
     contentType: "text/html",
-    content: `${agentName} is embedded inside ${embedded}. This external tab is tracked, but live DOM control and accessibility-tree access are unavailable here.${reasonLine}`,
+    content: `${agentName} is running in virtual external-tab mode. Live DOM control and accessibility-tree access are unavailable here.${reasonLine}`,
     metadata: {
       inaccessible: true,
       external: true,
@@ -865,8 +863,9 @@ function createAgentContext(config2, bridgeRpc2, tabularStore2) {
   const RETRY_DELAYS = [1e3, 3e3];
   const MAX_RETRIES3 = RETRY_DELAYS.length;
   const ACTIVE_TAB_CACHE_TTL_MS = 250;
+  const EXTERNAL_PAGE_CACHE_TTL_MS = 45e3;
   const callExtensionRouter = async (action, data) => {
-    const token = config2.apiKey || config2.authToken;
+    const token = config2.authToken || config2.apiKey;
     if (!token) {
       throw createRoverError({
         code: "MISSING_API_KEY",
@@ -950,13 +949,19 @@ function createAgentContext(config2, bridgeRpc2, tabularStore2) {
     const normalizedUrl = String(url || "").trim();
     if (!normalizedUrl)
       throw new Error("external page data requires url");
-    const cacheKey = normalizedUrl;
-    if (externalPageDataCache.has(cacheKey)) {
-      return externalPageDataCache.get(cacheKey);
+    const cacheTabId = Number(options?.tabId) || 0;
+    const cacheKey = `${cacheTabId}:${normalizedUrl}`;
+    const nowMs = Date.now();
+    const cachedData = externalPageDataCache.get(cacheKey);
+    if (cachedData && nowMs - cachedData.ts <= EXTERNAL_PAGE_CACHE_TTL_MS) {
+      return cachedData.data;
     }
-    if (externalPageDataErrorCache.has(cacheKey)) {
-      throw new Error(externalPageDataErrorCache.get(cacheKey) || "external page data fetch failed");
+    externalPageDataCache.delete(cacheKey);
+    const cachedError = externalPageDataErrorCache.get(cacheKey);
+    if (cachedError && nowMs - cachedError.ts <= EXTERNAL_PAGE_CACHE_TTL_MS) {
+      throw new Error(cachedError.message || "external page data fetch failed");
     }
+    externalPageDataErrorCache.delete(cacheKey);
     if (externalPageDataDisabledReason) {
       throw new Error(externalPageDataDisabledReason);
     }
@@ -973,12 +978,12 @@ function createAgentContext(config2, bridgeRpc2, tabularStore2) {
     try {
       const response = await callExtensionRouter(SUB_AGENTS.roverExternalPageData, payload);
       const pageData = response?.data || response;
-      externalPageDataCache.set(cacheKey, pageData);
+      externalPageDataCache.set(cacheKey, { data: pageData, ts: nowMs });
       return pageData;
     } catch (error) {
       const envelope = toRoverErrorEnvelope(error, "external page data fetch failed");
       const normalizedMessage = envelope?.message || "external page data fetch failed";
-      externalPageDataErrorCache.set(cacheKey, normalizedMessage);
+      externalPageDataErrorCache.set(cacheKey, { message: normalizedMessage, ts: nowMs });
       if (envelope.code === "PERMISSION_DENIED" || envelope.code === "MISSING_API_KEY" || envelope.code === "INVALID_API_KEY") {
         externalPageDataDisabledReason = `${envelope.code}: ${normalizedMessage}`;
       }
@@ -1023,7 +1028,6 @@ function createAgentContext(config2, bridgeRpc2, tabularStore2) {
         tabId: numericTabId,
         url: pageUrl,
         title: localPageData?.title,
-        embeddedDomain: runtimeContext?.embeddedDomain,
         agentName: runtimeContext?.agentName,
         reason: `cloud_fetch_disabled:${externalPageDataDisabledReason}`
       });
@@ -1037,7 +1041,6 @@ function createAgentContext(config2, bridgeRpc2, tabularStore2) {
         tabId: numericTabId,
         url: pageUrl,
         title: localPageData?.title,
-        embeddedDomain: runtimeContext?.embeddedDomain,
         agentName: runtimeContext?.agentName,
         reason: `policy_blocked:${ruleCheck.reason || "blocked"}`
       });
@@ -1070,7 +1073,6 @@ function createAgentContext(config2, bridgeRpc2, tabularStore2) {
         tabId: numericTabId,
         url: pageUrl,
         title: localPageData?.title,
-        embeddedDomain: runtimeContext?.embeddedDomain,
         agentName: runtimeContext?.agentName,
         reason: `cloud_fetch_failed:${envelope?.code || "unknown"}`
       });
@@ -4158,13 +4160,6 @@ var RPC_TIMEOUT_MS = 3e4;
 var DETACHED_EXTERNAL_TAB_MAX_AGE_MS = 9e4;
 var MAX_CHATLOG_ENTRIES2 = 24;
 var MAX_CHATLOG_MESSAGE_CHARS2 = 1e3;
-function resolveEmbeddedDomain(config2) {
-  const domains = Array.isArray(config2?.allowedDomains) ? config2.allowedDomains : [];
-  const first = String(domains[0] || "").trim();
-  if (!first)
-    return void 0;
-  return first.replace(/^\*?\./, "").replace(/^=/, "");
-}
 function resolveAgentName(config2) {
   const raw = String(config2?.ui?.agent?.name || "").trim();
   if (!raw)
@@ -4186,6 +4181,14 @@ function extractWebToolsConfig(config2) {
     return void 0;
   return config2.tools.web;
 }
+function resolveRuntimeExternalNavigationPolicy(config2) {
+  if (!config2)
+    return void 0;
+  if (config2.externalNavigationPolicy === "open_new_tab_notice" || config2.externalNavigationPolicy === "block" || config2.externalNavigationPolicy === "allow") {
+    return config2.externalNavigationPolicy;
+  }
+  return void 0;
+}
 function buildRoverRuntimeContext2(params) {
   const externalTabs = params.tabs.map((tab, index) => {
     if (!tab.external && tab.accessMode !== "external_placeholder" && tab.accessMode !== "external_scraped") {
@@ -4202,7 +4205,6 @@ function buildRoverRuntimeContext2(params) {
   }).filter((tab) => !!tab).slice(0, 8);
   return {
     mode: "rover_embed",
-    embeddedDomain: params.embeddedDomain,
     agentName: params.agentName,
     externalNavigationPolicy: params.externalNavigationPolicy,
     tabIdContract: "tree_index_mapped_by_tab_order",
@@ -4856,6 +4858,13 @@ function deriveDirectToolRunOutcome(result) {
   if (!result || typeof result !== "object") {
     return { taskComplete: false };
   }
+  const topLevelStatus = String(result.status || "").trim().toLowerCase();
+  if (topLevelStatus === "failure" || topLevelStatus === "failed" || topLevelStatus === "error") {
+    return { taskComplete: false };
+  }
+  if (topLevelStatus === "waiting_input" || topLevelStatus === "needs_input" || topLevelStatus === "pending_user_input") {
+    return { taskComplete: false, needsUserInput: true };
+  }
   if (result.error) {
     return { taskComplete: false };
   }
@@ -4870,6 +4879,9 @@ function deriveDirectToolRunOutcome(result) {
     if (Array.isArray(output.questions) && output.questions.length > 0) {
       return { taskComplete: false, needsUserInput: true };
     }
+    if (output.error) {
+      return { taskComplete: false };
+    }
     if (typeof output.taskComplete === "boolean") {
       return { taskComplete: !!output.taskComplete };
     }
@@ -4884,10 +4896,16 @@ function deriveDirectToolRunOutcome(result) {
       if (taskStatus === "completed" || taskStatus === "complete" || taskStatus === "done" || taskStatus === "success") {
         return { taskComplete: true };
       }
+      if (taskStatus === "failure" || taskStatus === "failed" || taskStatus === "error") {
+        return { taskComplete: false };
+      }
     }
     if (output.success === false) {
       return { taskComplete: false };
     }
+    if (String(output.status || "").trim().toLowerCase() === "failure") {
+      return { taskComplete: false };
+    }
   }
   if (output != null) {
     return { taskComplete: true };
@@ -4960,13 +4978,11 @@ async function handleUserMessage(text, options) {
   if (!tabularStore) {
     tabularStore = new TabularStore(`rover-${trajectoryId}`);
   }
-  const embeddedDomain = resolveEmbeddedDomain(config);
   const agentName = resolveAgentName(config);
   const runtimeContext = buildRoverRuntimeContext2({
     tabs: tabsForRun,
-    embeddedDomain,
     agentName,
-    externalNavigationPolicy: config.externalNavigationPolicy
+    externalNavigationPolicy: resolveRuntimeExternalNavigationPolicy(config)
   });
   const ctx = createAgentContext({
     ...config,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rtrvr-ai/rover",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "private": false,
   "type": "module",
   "main": "dist/rover.js",