@omercnet/paseo-shared-browser 1.1.0-next.198.1 → 1.1.0-next.210.1

package/README.md

@@ -93,15 +93,19 @@ Before upgrading the plugin on Windows, close active Shared Browser sessions and
 daemon; Windows does not allow the installer to replace runtime executables that are still running.
 
 Open a workspace, search the Command Center for **Open Shared Browser**, or tap the **Shared
-Browser** composer pill while a workspace session is open.
+Browser** composer pill while a workspace session is open. On Paseo 0.11 or newer, a **Shared
+Browser (n)** row in the sidebar footer appears while browser sessions are open; it lists them and
+jumps to the chosen workspace's browser panel.
 
 ## Agent MCP access
 
 The plugin automatically injects its stdio MCP adapter only when a new, non-internal agent is
 created with a provider that accepts external MCP servers. Agents that already exist, resumed
-sessions, imported sessions, and Paseo's internal agents are not modified. Paseo's OMP provider
-uses native host-tool injection and rejects external MCP servers, so OMP agents are left unchanged.
-Pi agents continue to receive the adapter, but they require Pi's optional MCP support to launch it.
+sessions, imported sessions, and Paseo's internal agents are not modified. Paseo's built-in OMP
+provider accepts session MCP servers from Paseo 0.11, so new OMP agents receive the adapter there.
+On Paseo 0.9 and 0.10 the built-in OMP adapter rejects external MCP servers, so OMP agents are left
+unchanged. Pi agents continue to receive the adapter, but they require Pi's optional MCP support to
+launch it.
 
 The injected MCP server exposes exactly these tools: `shared_browser_status`,
 `shared_browser_capture`, `shared_browser_acquire_control`, `shared_browser_release_control`,

package/client/browser.tsx

@@ -29,6 +29,8 @@ import {
   type BrowserInputEvent,
   type BrowserState,
   captureBrowserRpc,
+  containedRect,
+  createImageSizeLoader,
   DEVICE_PRESETS,
   type DevicePresetId,
   detachBrowserRpc,
@@ -41,6 +43,7 @@ import {
   releaseControlRpc,
   resizeBrowserRpc,
   sendBrowserInputRpc,
+  toViewportPoint,
 } from "../shared/browser";
 
 const SPACE = {
@@ -88,12 +91,7 @@ interface Size {
   height: number;
 }
 
-interface DisplayRect extends Size {
-  x: number;
-  y: number;
-}
-
-interface DisplayPoint extends Size {
+interface DisplayPoint {
   x: number;
   y: number;
 }
@@ -115,21 +113,6 @@ const SPECIAL_KEYS: readonly { key: SpecialKey; label: string }[] = [
   { key: "Space", label: "Space" },
 ];
 
-function containedRect(container: Size, image: Size): DisplayRect | null {
-  if (container.width <= 0 || container.height <= 0 || image.width <= 0 || image.height <= 0) {
-    return null;
-  }
-  const scale = Math.min(container.width / image.width, container.height / image.height);
-  const width = image.width * scale;
-  const height = image.height * scale;
-  return {
-    x: (container.width - width) / 2,
-    y: (container.height - height) / 2,
-    width,
-    height,
-  };
-}
-
 function errorMessage(error: unknown): string {
   if (error instanceof Error && error.message.trim()) return error.message;
   return "The shared browser request failed.";
@@ -1253,16 +1236,34 @@ export function SharedBrowserPanel({
   const currentFrame = frame && state && isFrameCurrent(frame, state) ? frame : null;
   const canSendInput = canControl && Boolean(currentFrame) && !inputMutation.isPending;
 
+  const frameUri = useMemo(
+    () => (currentFrame ? `data:${currentFrame.mimeType};base64,${currentFrame.dataBase64}` : null),
+    [currentFrame],
+  );
+  const [imageSize, setImageSize] = useState<{ uri: string; size: Size } | null>(null);
+  const loadImageSize = useMemo(
+    () =>
+      createImageSizeLoader(
+        (uri, success, failure) => Image.getSize(uri, success, failure),
+        (uri, size) => setImageSize(size ? { uri, size } : null),
+      ),
+    [],
+  );
+  useEffect(() => {
+    if (frameUri) loadImageSize(frameUri);
+  }, [frameUri, loadImageSize]);
+  // The bitmap can disagree with the frame metadata; its own aspect ratio decides the rectangle.
   const displayRect = useMemo(
     () =>
       currentFrame
-        ? containedRect(containerSize, { width: currentFrame.width, height: currentFrame.height })
+        ? containedRect(
+            containerSize,
+            imageSize?.uri === frameUri
+              ? imageSize.size
+              : { width: currentFrame.width, height: currentFrame.height },
+          )
         : null,
-    [containerSize, currentFrame],
-  );
-  const frameUri = useMemo(
-    () => (currentFrame ? `data:${currentFrame.mimeType};base64,${currentFrame.dataBase64}` : null),
-    [currentFrame],
+    [containerSize, currentFrame, frameUri, imageSize],
   );
 
   const controlContext = useCallback(() => {
@@ -1329,7 +1330,7 @@ export function SharedBrowserPanel({
       if (!displayRect) return null;
       const x = Math.min(displayRect.width, Math.max(0, event.nativeEvent.locationX));
       const y = Math.min(displayRect.height, Math.max(0, event.nativeEvent.locationY));
-      return { x, y, width: displayRect.width, height: displayRect.height };
+      return { x, y };
     },
     [displayRect],
   );
@@ -1338,17 +1339,18 @@ export function SharedBrowserPanel({
     (event: GestureResponderEvent) => {
       const start = dragStartRef.current;
       const end = pointFromEvent(event);
+      const viewport = stateRef.current?.viewport;
       dragStartRef.current = null;
-      if (!start || !end) return;
+      if (!start || !end || !displayRect || !viewport) return;
+      const toViewport = (point: DisplayPoint) => toViewportPoint(point, displayRect, viewport);
       lastPointRef.current = { x: end.x, y: end.y };
       const distance = Math.hypot(end.x - start.x, end.y - start.y);
       if (distance >= DRAG_THRESHOLD) {
         if (swipeMode === "scroll") {
-          const current = stateRef.current;
-          const scale = current ? current.viewport.width / end.width : 1;
+          const scale = viewport.width / displayRect.width;
           sendEvent({
             kind: "scroll",
-            point: { ...start, width: end.width, height: end.height },
+            point: toViewport(start),
             deltaX: clampScrollDelta((start.x - end.x) * scale),
             deltaY: clampScrollDelta((start.y - end.y) * scale),
           });
@@ -1356,20 +1358,20 @@ export function SharedBrowserPanel({
         }
         sendEvent({
           kind: "drag",
-          start: { ...start, width: end.width, height: end.height },
-          end,
+          start: toViewport(start),
+          end: toViewport(end),
           button: interactionMode === "right" ? "right" : "left",
         });
         return;
       }
       sendEvent({
         kind: "click",
-        point: end,
+        point: toViewport(end),
         button: interactionMode === "right" ? "right" : "left",
         clickCount: interactionMode === "double" ? 2 : 1,
       });
     },
-    [interactionMode, pointFromEvent, sendEvent, swipeMode],
+    [displayRect, interactionMode, pointFromEvent, sendEvent, swipeMode],
   );
 
   const panResponder = useMemo(
@@ -1474,13 +1476,14 @@ export function SharedBrowserPanel({
 
   const scroll = useCallback(
     (deltaX: number, deltaY: number) => {
-      if (!displayRect) return;
+      const viewport = stateRef.current?.viewport;
+      if (!displayRect || !viewport) return;
       const previous = lastPointRef.current;
       const x = previous && previous.x <= displayRect.width ? previous.x : displayRect.width / 2;
       const y = previous && previous.y <= displayRect.height ? previous.y : displayRect.height / 2;
       sendEvent({
         kind: "scroll",
-        point: { x, y, width: displayRect.width, height: displayRect.height },
+        point: toViewportPoint({ x, y }, displayRect, viewport),
         deltaX,
         deltaY,
       });

package/client/sidebar.tsx

@@ -0,0 +1,134 @@
+import {
+  type PluginClientContext,
+  type PluginPopoverProps,
+  type PluginSidebarItemProps,
+  useRpc,
+  useWorkspace,
+} from "@getpaseo/plugin/client";
+import { SidebarRow } from "@getpaseo/plugin/client/ui";
+import { useQuery } from "@tanstack/react-query";
+import { useMemo } from "react";
+import { Pressable, StyleSheet, Text, View } from "react-native";
+import { listOpenBrowserWorkspacesRpc } from "../shared/browser";
+
+const PANEL_ID = "shared-browser";
+const PRESENCE_POLL_MS = 5_000;
+const TOUCH_TARGET = 44;
+const ROW_HEIGHT = 36;
+const RADIUS = 8;
+
+function useOpenBrowserWorkspaceIds(): readonly string[] {
+  const listOpen = useRpc(listOpenBrowserWorkspacesRpc);
+  const query = useQuery({
+    queryKey: ["shared-browser", "presence"],
+    queryFn: async () => (await listOpen({})).workspaceIds,
+    refetchInterval: PRESENCE_POLL_MS,
+    retry: false,
+  });
+  return query.data ?? [];
+}
+
+function SessionRow({
+  workspaceId,
+  theme,
+  compact,
+  onPress,
+}: {
+  workspaceId: string;
+  theme: PluginPopoverProps["theme"];
+  compact: boolean;
+  onPress(): void;
+}) {
+  const name = useWorkspace(workspaceId, (workspace) => workspace.title ?? workspace.name);
+  const styles = useMemo(
+    () =>
+      StyleSheet.create({
+        row: {
+          minHeight: compact ? TOUCH_TARGET : ROW_HEIGHT,
+          justifyContent: "center",
+          paddingHorizontal: 12,
+          borderRadius: RADIUS,
+        },
+        label: { color: theme.colors.foreground, fontSize: 14 },
+      }),
+    [theme, compact],
+  );
+  // A workspace this client has not loaded cannot host the panel; openPanel would throw.
+  if (name === null) return null;
+  return (
+    <Pressable
+      accessibilityRole="button"
+      accessibilityLabel={`Open Shared Browser for ${name}`}
+      onPress={onPress}
+      style={({ pressed }) => [
+        styles.row,
+        pressed ? { backgroundColor: theme.colors.surface1 } : null,
+      ]}
+    >
+      <Text numberOfLines={1} style={styles.label}>
+        {name}
+      </Text>
+    </Pressable>
+  );
+}
+
+function createSessionsPopover(client: PluginClientContext) {
+  return function SessionsPopover({ theme, layout, close }: PluginPopoverProps) {
+    const workspaceIds = useOpenBrowserWorkspaceIds();
+    const styles = useMemo(
+      () =>
+        StyleSheet.create({
+          container: { padding: layout.compact ? 16 : 8, gap: 4, minWidth: 220 },
+          title: {
+            color: theme.colors.foregroundMuted,
+            fontSize: 12,
+            paddingHorizontal: 12,
+            paddingBottom: 4,
+          },
+        }),
+      [theme, layout.compact],
+    );
+    return (
+      <View style={styles.container}>
+        <Text style={styles.title}>Open browser sessions</Text>
+        {workspaceIds.map((workspaceId) => (
+          <SessionRow
+            key={workspaceId}
+            workspaceId={workspaceId}
+            theme={theme}
+            compact={layout.compact}
+            onPress={() => {
+              close();
+              client.openPanel(PANEL_ID, { workspaceId });
+            }}
+          />
+        ))}
+      </View>
+    );
+  };
+}
+
+function createSidebarItem(client: PluginClientContext) {
+  const SessionsPopover = createSessionsPopover(client);
+  return function SharedBrowserSidebarItem({ openPopover }: PluginSidebarItemProps) {
+    const count = useOpenBrowserWorkspaceIds().length;
+    if (count === 0) return null;
+    return (
+      <SidebarRow
+        icon="PanelsTopLeft"
+        label={`Shared Browser (${count})`}
+        onPress={() => openPopover(SessionsPopover)}
+      />
+    );
+  };
+}
+
+/** Sidebar footer row listing open sessions. Hosts before 0.11 lack the API and get no row. */
+export function contributeSharedBrowserSidebar(client: PluginClientContext): () => void {
+  if (typeof client.addSidebarFooterItem !== "function") return () => {};
+  return client.addSidebarFooterItem({
+    id: "shared-browser-sessions",
+    title: "Shared Browser",
+    Component: createSidebarItem(client),
+  });
+}

package/index.client.tsx

@@ -1,5 +1,6 @@
 import type { PluginClientContext } from "@getpaseo/plugin/client";
 import { contributeSharedBrowserClient, SharedBrowserPanel } from "./client/browser";
+import { contributeSharedBrowserSidebar } from "./client/sidebar";
 
 export default function contribute(client: PluginClientContext) {
   client.addWorkspacePanel({
@@ -31,5 +32,10 @@ export default function contribute(client: PluginClientContext) {
     },
   });
 
-  return contributeSharedBrowserClient(client);
+  const removeSidebar = contributeSharedBrowserSidebar(client);
+  const cleanup = contributeSharedBrowserClient(client);
+  return () => {
+    removeSidebar();
+    return cleanup();
+  };
 }

package/index.server.ts

@@ -77,8 +77,12 @@ export default function contribute(server: PluginServerContext) {
   server.handle(resizeBrowserRpc, handleResizeBrowser);
   server.handle(applyDevicePresetRpc, handleApplyDevicePreset);
   server.handle(sendBrowserInputRpc, handleSendBrowserInput);
+  // OMP accepts session MCP servers from Paseo 0.11.0-beta.1, the release that added
+  // registerUsageSource; 0.9/0.10 reject agent creation when an OMP config carries MCP servers.
+  const ompAcceptsMcp = typeof server.registerUsageSource === "function";
   server.before("agent.create", async ({ request }) => {
-    if (request.config.internal || request.config.provider === "omp") return request;
+    if (request.config.internal) return request;
+    if (request.config.provider === "omp" && !ompAcceptsMcp) return request;
     const ticket = randomBytes(32).toString("base64url");
     if (!(await issueTicketWithinDeadline(ticket))) return request;
     return {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@omercnet/paseo-shared-browser",
-  "version": "1.1.0-next.198.1",
+  "version": "1.1.0-next.210.1",
   "description": "One real Chromium session per Paseo workspace, shared live across every connected client.",
   "keywords": [
     "browser",
@@ -43,12 +43,12 @@
     "check:write": "biome check --write .",
     "typecheck": "tsc --noEmit",
     "prepare:runtime": "node scripts/prepare-runtime.mjs",
-    "test": "vitest run tests/contracts.test.ts tests/lease.test.ts tests/runtime-lifecycle.test.ts tests/agent-access.test.ts tests/composer-pills.test.ts",
-    "test:unit": "vitest run tests/contracts.test.ts tests/lease.test.ts tests/runtime-lifecycle.test.ts tests/agent-access.test.ts tests/composer-pills.test.ts",
+    "test": "vitest run tests/contracts.test.ts tests/lease.test.ts tests/runtime-lifecycle.test.ts tests/agent-access.test.ts tests/composer-pills.test.ts tests/sidebar.test.ts tests/pointer-mapping.test.ts",
+    "test:unit": "vitest run tests/contracts.test.ts tests/lease.test.ts tests/runtime-lifecycle.test.ts tests/agent-access.test.ts tests/composer-pills.test.ts tests/sidebar.test.ts tests/pointer-mapping.test.ts",
     "test:smoke": "vitest run tests/browser.smoke.test.ts --testTimeout=30000"
   },
   "dependencies": {
-    "@getpaseo/plugin": "0.10.1",
+    "@getpaseo/plugin": "0.11.0-beta.3",
     "@modelcontextprotocol/sdk": "1.30.0",
     "agent-browser": "0.37.1",
     "esbuild": "0.28.2",

package/shared/browser.ts

@@ -365,3 +365,70 @@ export function mapDisplayedPoint(
     y: Math.min(viewport.height - 1, Math.max(0, (point.y / point.height) * viewport.height)),
   };
 }
+
+export interface FrameSize {
+  width: number;
+  height: number;
+}
+
+export interface FrameRect extends FrameSize {
+  x: number;
+  y: number;
+}
+
+export function containedRect(container: FrameSize, image: FrameSize): FrameRect | null {
+  if (container.width <= 0 || container.height <= 0 || image.width <= 0 || image.height <= 0) {
+    return null;
+  }
+  const scale = Math.min(container.width / image.width, container.height / image.height);
+  const width = image.width * scale;
+  const height = image.height * scale;
+  return {
+    x: (container.width - width) / 2,
+    y: (container.height - height) / 2,
+    width,
+    height,
+  };
+}
+
+/** Converts a point in the displayed image to the canonical viewport, scaling both axes by viewport.width / rect.width. */
+export function toViewportPoint(
+  point: MappedPoint,
+  rect: FrameSize,
+  viewport: Viewport,
+): MappedPoint & FrameSize {
+  const scale = viewport.width / rect.width;
+  return {
+    x: point.x * scale,
+    y: Math.min(viewport.height, point.y * scale),
+    width: viewport.width,
+    height: viewport.height,
+  };
+}
+
+type GetImageSize = (
+  uri: string,
+  success: (width: number, height: number) => void,
+  failure: () => void,
+) => void;
+
+/** Looks up the intrinsic size once per distinct URI and drops results superseded by a newer URI. */
+export function createImageSizeLoader(
+  getSize: GetImageSize,
+  onSize: (uri: string, size: FrameSize | null) => void,
+): (uri: string) => void {
+  let current: string | null = null;
+  return (uri) => {
+    if (uri === current) return;
+    current = uri;
+    getSize(
+      uri,
+      (width, height) => {
+        if (current === uri) onSize(uri, { width, height });
+      },
+      () => {
+        if (current === uri) onSize(uri, null);
+      },
+    );
+  };
+}