@vellumai/assistant 0.4.21 → 0.4.23

package/src/tools/browser/browser-screencast.ts

@@ -1,8 +1,7 @@
-import type { ServerMessage } from '../../daemon/ipc-contract.js';
-import { browserManager, SCREENCAST_HEIGHT, SCREENCAST_WIDTH } from './browser-manager.js';
+import type { ServerMessage } from "../../daemon/ipc-contract.js";
+import { browserManager } from "./browser-manager.js";
 
-// Track which sessions have an active browser page (no PiP surface — the user
-// watches the actual browser window directly).
+// Track which sessions have an active browser page.
 const activeBrowserSessions = new Set<string>();
 
 // Registry of sendToClient callbacks per session
@@ -12,9 +11,15 @@ const sessionSenders = new Map<string, (msg: ServerMessage) => void>();
  * Register a sendToClient callback for a session.
  * Called from session-tool-setup when the session is created.
  */
-export function registerSessionSender(sessionId: string, sendToClient: (msg: ServerMessage) => void): void {
+export function registerSessionSender(
+  sessionId: string,
+  sendToClient: (msg: ServerMessage) => void,
+): void {
   sessionSenders.set(sessionId, sendToClient);
-  browserManager.registerSender(sessionId, sendToClient as (msg: { type: string; sessionId: string }) => void);
+  browserManager.registerSender(
+    sessionId,
+    sendToClient as (msg: { type: string; sessionId: string }) => void,
+  );
 }
 
 /**
@@ -25,14 +30,13 @@ export function unregisterSessionSender(sessionId: string): void {
   browserManager.unregisterSender(sessionId);
 }
 
-function getSender(sessionId: string): ((msg: ServerMessage) => void) | undefined {
+function getSender(
+  sessionId: string,
+): ((msg: ServerMessage) => void) | undefined {
   return sessionSenders.get(sessionId);
 }
 
-export async function ensureScreencast(
-  sessionId: string,
-  _sendToClient: (msg: ServerMessage) => void,
-): Promise<void> {
+export async function ensureScreencast(sessionId: string): Promise<void> {
   if (activeBrowserSessions.has(sessionId)) return;
 
   activeBrowserSessions.add(sessionId);
@@ -40,9 +44,6 @@ export async function ensureScreencast(
   try {
     // Ensure the page exists (may trigger browser launch/connect)
     await browserManager.getOrCreateSessionPage(sessionId);
-
-    // No PiP surface or CDP screencast — the user watches the actual
-    // browser window directly (positioned in top-right via positionWindowSidebar).
   } catch (err) {
     // Roll back so future calls can retry
     activeBrowserSessions.delete(sessionId);
@@ -50,30 +51,7 @@ export async function ensureScreencast(
   }
 }
 
-export function updateBrowserStatus(
-  sessionId: string,
-  _sendToClient: (msg: ServerMessage) => void,
-  _status: 'navigating' | 'idle' | 'interacting',
-  _actionText?: string,
-  _currentUrl?: string,
-): void {
-  // No-op: PiP surface was removed so there is no ui_surface to update.
-  // The function signature is preserved to avoid churn at callsites.
-  if (!activeBrowserSessions.has(sessionId)) return;
-}
-
-export async function updatePagesList(
-  sessionId: string,
-  _sendToClient: (msg: ServerMessage) => void,
-): Promise<void> {
-  // No-op: PiP surface was removed so there is no ui_surface to update.
-  if (!activeBrowserSessions.has(sessionId)) return;
-}
-
-export async function stopBrowserScreencast(
-  sessionId: string,
-  _sendToClient: (msg: ServerMessage) => void,
-): Promise<void> {
+export async function stopBrowserScreencast(sessionId: string): Promise<void> {
   if (!activeBrowserSessions.has(sessionId)) return;
 
   // Safe no-op if CDP screencast was never started
@@ -82,47 +60,13 @@ export async function stopBrowserScreencast(
   activeBrowserSessions.delete(sessionId);
 }
 
-export async function getElementBounds(
-  sessionId: string,
-  selector: string,
-): Promise<{ x: number; y: number; w: number; h: number } | null> {
-  try {
-    const page = await browserManager.getOrCreateSessionPage(sessionId);
-    const result = await page.evaluate(`
-      (() => {
-        const el = document.querySelector(${JSON.stringify(selector)});
-        if (!el) return null;
-        const rect = el.getBoundingClientRect();
-        return { x: rect.x, y: rect.y, w: rect.width, h: rect.height, vw: window.innerWidth, vh: window.innerHeight };
-      })()
-    `) as { x: number; y: number; w: number; h: number; vw: number; vh: number } | null;
-    if (!result) return null;
-    const scale = Math.min(SCREENCAST_WIDTH / result.vw, SCREENCAST_HEIGHT / result.vh);
-    return {
-      x: result.x * scale,
-      y: result.y * scale,
-      w: result.w * scale,
-      h: result.h * scale,
-    };
-  } catch {
-    return null;
-  }
-}
-
-export function updateHighlights(
-  sessionId: string,
-  _sendToClient: (msg: ServerMessage) => void,
-  _highlights: Array<{ x: number; y: number; w: number; h: number; label: string }>,
-): void {
-  // No-op: PiP surface was removed so there is no ui_surface to update.
-  if (!activeBrowserSessions.has(sessionId)) return;
-}
-
 export async function stopAllScreencasts(): Promise<void> {
   for (const sessionId of activeBrowserSessions) {
     try {
       await browserManager.stopScreencast(sessionId);
-    } catch { /* best-effort */ }
+    } catch {
+      /* best-effort */
+    }
   }
   activeBrowserSessions.clear();
 }