@opengeni/react 7.6.1-canary.37073905159001 → 7.6.1

package/src/components/session-commands-panel.tsx

@@ -2,7 +2,6 @@ import { ChevronRightIcon, Loader2Icon, SquareIcon } from "lucide-react";
 import { useState } from "react";
 import type { UseSessionBackgroundCommandsResult } from "../hooks/use-session-background-commands";
 import { formatClockTime } from "../lib/format";
-import { useErrorMessage } from "../lib/error-message";
 
 /** The current session's live commands. Settled commands belong to the timeline. */
 export function SessionCommandsPanel({
@@ -12,7 +11,6 @@ export function SessionCommandsPanel({
   commands: UseSessionBackgroundCommandsResult;
   readOnly?: boolean;
 }) {
-  const formatError = useErrorMessage();
   const [pending, setPending] = useState<string | null>(null);
   const [error, setError] = useState<string | null>(null);
   const active = state.commands.filter(
@@ -24,7 +22,7 @@ export function SessionCommandsPanel({
     try {
       await state.cancel(id);
     } catch (cause) {
-      setError(formatError(cause));
+      setError(cause instanceof Error ? cause.message : "Stop was not confirmed. Try again.");
     } finally {
       setPending(null);
     }

package/src/components/session-conversation.tsx

@@ -37,7 +37,6 @@ import {
 import type { UserMessageDisclosureLabels } from "./user-message-body";
 import { conversationTimeline } from "../conversation-timeline";
 import { cn } from "../lib/cn";
-import { useErrorMessage } from "../lib/error-message";
 
 export type SessionConversationProps = ClientOverride & {
   sessionId: string;
@@ -121,7 +120,6 @@ function Conversation({
 }: SessionConversationProps) {
   const scope = { client, workspaceId };
   const context = useOpenGeni(scope);
-  const formatError = useErrorMessage();
   const config = useClientConfigFlags(context.client);
   const showModelPicker = modelPicker ?? config.modelSelection;
   const catalog = useWorkspaceModelCatalog({
@@ -205,7 +203,7 @@ function Conversation({
       style={{ height }}
       data-og-conversation=""
     >
-      {error && <p role="alert">{formatError(error)}</p>}
+      {error && <p role="alert">{error.message}</p>}
       <MessageTimeline
         renderMessageText={renderMessageText}
         resolveLink={links}
@@ -267,7 +265,7 @@ function Conversation({
                 await human.respond(id, response);
               }}
               respondingRequestId={human.respondingRequestId}
-              error={human.mutationError ? formatError(human.mutationError) : null}
+              error={human.mutationError?.message}
               autoFocus={false}
             />
             {terminal ? (

package/src/components/session-list.tsx

@@ -4,7 +4,6 @@ import { useEffect, useRef, useState, type FormEvent } from "react";
 import { useWorkspaceSessions } from "../hooks/use-workspace-sessions";
 import { cn } from "../lib/cn";
 import { formatRelativeTime } from "../lib/format";
-import { useErrorMessage } from "../lib/error-message";
 import { useOpenGeni, type ClientOverride } from "../session-context";
 
 export type SessionListLabels = {
@@ -91,8 +90,7 @@ export function SessionList({
   }, [refresh, refreshKey]);
   const [editing, setEditing] = useState<{ id: string; title: string } | null>(null);
   const [busy, setBusy] = useState<string | null>(null);
-  const [error, setError] = useState<{ cause: unknown } | null>(null);
-  const formatError = useErrorMessage();
+  const [error, setError] = useState<string | null>(null);
   const archiveClient = context.client as unknown as ArchiveClient;
   const canArchive = archive && typeof archiveClient.updateSessionArchive === "function";
 
@@ -103,7 +101,7 @@ export function SessionList({
       await action();
       await list.refresh();
     } catch (cause) {
-      setError({ cause });
+      setError(cause instanceof Error ? cause.message : String(cause));
     } finally {
       setBusy(null);
     }
@@ -150,16 +148,16 @@ export function SessionList({
           </button>
         ) : null}
       </div>
-      {error || list.error ? (
+      {error ? (
         <p role="alert" className="px-2 text-og-xs text-og-status-failed">
-          {formatError(error ? error.cause : list.error)}
+          {error}
         </p>
       ) : null}
       <ul className="flex min-h-0 flex-1 flex-col gap-0.5 overflow-y-auto px-1 pb-2">
         {list.loading && sessions.length === 0 ? (
           <li className="px-2 py-2 text-og-xs text-og-fg-subtle">{labels.loading}</li>
         ) : null}
-        {!list.loading && !list.error && sessions.length === 0 ? (
+        {!list.loading && sessions.length === 0 ? (
           <li className="px-2 py-2 text-og-xs text-og-fg-subtle">{labels.empty}</li>
         ) : null}
         {sessions.map((session) => {

package/src/components/session-mcp-capability-card.tsx

@@ -127,17 +127,7 @@ function ScopedCard({
     ]);
     if (!invocation || invocation.signal.aborted || lifetime.current !== invocation || !active())
       return null;
-    let resolved = catalog.items.find((entry) => entry.id === capabilityId);
-    // Custom catalog entries may not declare authentication. Discover it from
-    // the live catalog's endpoint, never from recommendation/host copy.
-    if (resolved?.kind === "mcp" && !resolved.enabled && !resolved.authKind) {
-      const endpoint = resolved.mcpUrl ?? resolved.endpointUrl;
-      if (endpoint) {
-        const inspected = await client.inspectMcpAuthentication(workspaceId, endpoint);
-        if (invocation.signal.aborted || lifetime.current !== invocation || !active()) return null;
-        if (inspected.kind === "oauth2") resolved = { ...resolved, authKind: "oauth2" };
-      }
-    }
+    const resolved = catalog.items.find((entry) => entry.id === capabilityId);
     if (!resolved || resolved.kind !== "mcp" || resolved.authKind !== "oauth2")
       throw new Error(
         "This recommendation is not an available OAuth MCP integration. Review the current connection catalog.",

package/src/hooks/internal.ts

@@ -1,7 +1,6 @@
 import type { SessionEvent } from "@opengeni/sdk";
 import { useCallback, useEffect, useLayoutEffect, useRef, useState } from "react";
 import type { EmbeddedSessionEventClientLike } from "../client";
-import { normalizeError } from "../lib/error-message";
 
 const HIDDEN_STREAM_GRACE_MS = 2_000;
 
@@ -117,7 +116,7 @@ export function usePolledValue<T>(
           activeLoadRef.current === load &&
           !requestAbort.signal.aborted
         ) {
-          setError(normalizeError(cause));
+          setError(cause instanceof Error ? cause : new Error(String(cause)));
           setLoading(false);
         }
       } finally {
@@ -259,7 +258,7 @@ export function useMutationRunner(identity: unknown = undefined): MutationState
           generation.current === ownedGeneration &&
           Object.is(identityRef.current, ownedIdentity)
         ) {
-          setMutationError(normalizeError(cause));
+          setMutationError(cause instanceof Error ? cause : new Error(String(cause)));
         }
         return null;
       } finally {

package/src/hooks/use-browser-session.ts

@@ -30,10 +30,6 @@ export type BrowserFrameInputFence = Pick<
   | "frameId"
 >;
 
-export type BrowserInputFailure = Pick<BrowserActionReceipt, "operationId" | "error"> & {
-  state: Exclude<BrowserActionReceipt["state"], "completed">;
-};
-
 export type UseBrowserSessionOptions = EmbeddedBrowserInteractionClientOverride & {
   browserSessionId: string | null;
   enabled?: boolean | undefined;
@@ -51,7 +47,6 @@ export type UseBrowserSessionResult = {
   loading: boolean;
   mutating: boolean;
   error: Error | null;
-  inputFailure: BrowserInputFailure | null;
   refresh: () => Promise<void>;
   observeForInput: () => Promise<BrowserObservation>;
   actFromObservation: (
@@ -101,13 +96,10 @@ export function useBrowserSession(options: UseBrowserSessionOptions): UseBrowser
     loading: boolean;
     mutating: boolean;
     error: Error | null;
-    inputFailure: BrowserInputFailure | null;
   }>(() => emptyState(browserSessionId, enabled));
   const visible =
     state.browserSessionId === browserSessionId ? state : emptyState(browserSessionId, enabled);
   const refreshBlocked = isNonRetryableInteractionError(visible.error);
-  const inputFailureRef = useRef(visible.inputFailure);
-  inputFailureRef.current = visible.inputFailure;
   const selectedTargetIdRef = useRef<string | null>(visible.selectedTargetId);
   selectedTargetIdRef.current = visible.selectedTargetId;
   const selectedTargetRef = useRef<BrowserTarget | null>(
@@ -147,104 +139,88 @@ export function useBrowserSession(options: UseBrowserSessionOptions): UseBrowser
     requestRef.current = { id, controller: null };
   }, []);
 
-  const refreshInventory = useCallback(
-    async (checkInput = false) => {
-      if (!enabled || !browserSessionId) return;
-      const checkedInputFailure = inputFailureRef.current;
-      const id = requestRef.current.id + 1;
-      requestRef.current.controller?.abort();
-      const controller = new AbortController();
-      requestRef.current = { id, controller };
+  const refresh = useCallback(async (): Promise<void> => {
+    if (!enabled || !browserSessionId) return;
+    const id = requestRef.current.id + 1;
+    requestRef.current.controller?.abort();
+    const controller = new AbortController();
+    requestRef.current = { id, controller };
+    try {
+      const [session, targetResponse] = await Promise.all([
+        client.getBrowserSession(workspaceId, browserSessionId, {
+          signal: controller.signal,
+        }),
+        client.listBrowserTargets(workspaceId, browserSessionId, {
+          signal: controller.signal,
+        }),
+      ]);
+      if (!mountedRef.current || requestRef.current.id !== id) return;
+      const selected = chooseTarget(targetResponse.targets, selectedTargetIdRef.current);
+      selectedTargetIdRef.current = selected?.id ?? null;
+      const previousObservation =
+        observationRef.current.browserSessionId === browserSessionId
+          ? observationRef.current.observation
+          : null;
+      const retainedObservation =
+        selected && previousObservation && sameObservationTarget(previousObservation, selected)
+          ? previousObservation
+          : null;
+      observationRef.current = { browserSessionId, observation: retainedObservation };
+      setState((current) =>
+        current.browserSessionId === browserSessionId
+          ? {
+              ...current,
+              session,
+              targets: targetResponse.targets,
+              selectedTargetId: selected?.id ?? null,
+              observation: retainedObservation,
+              loading: false,
+              error: null,
+            }
+          : current,
+      );
+      if (!selected || !semanticObservationEnabledRef.current) return;
       try {
-        const [session, targetResponse] = await Promise.all([
-          client.getBrowserSession(workspaceId, browserSessionId, {
-            signal: controller.signal,
-          }),
-          client.listBrowserTargets(workspaceId, browserSessionId, {
-            signal: controller.signal,
-          }),
-        ]);
+        const observation = await client.observeBrowserTarget(
+          workspaceId,
+          browserSessionId,
+          selected.id,
+          { signal: controller.signal },
+        );
         if (!mountedRef.current || requestRef.current.id !== id) return;
-        const selected = chooseTarget(targetResponse.targets, selectedTargetIdRef.current);
-        selectedTargetIdRef.current = selected?.id ?? null;
-        const previousObservation =
-          observationRef.current.browserSessionId === browserSessionId
-            ? observationRef.current.observation
-            : null;
-        const retainedObservation =
-          selected && previousObservation && sameObservationTarget(previousObservation, selected)
-            ? previousObservation
-            : null;
-        observationRef.current = { browserSessionId, observation: retainedObservation };
+        observationRef.current = { browserSessionId, observation };
+        lastObservationAtRef.current = Date.now();
         setState((current) =>
-          current.browserSessionId === browserSessionId
-            ? {
-                ...current,
-                session,
-                targets: targetResponse.targets,
-                selectedTargetId: selected?.id ?? null,
-                observation: retainedObservation,
-                loading: false,
-                error: null,
-                // Media and background polling cannot confirm an input outcome.
-                // Only an explicit fresh check clears the notice it began with.
-                inputFailure:
-                  checkInput &&
-                  session.id === browserSessionId &&
-                  targetResponse.browserSessionId === browserSessionId &&
-                  current.inputFailure === checkedInputFailure
-                    ? null
-                    : current.inputFailure,
-              }
+          current.browserSessionId === browserSessionId &&
+          current.selectedTargetId === observation.target.id
+            ? { ...current, observation }
             : current,
         );
-        if (!selected || !semanticObservationEnabledRef.current) return;
-        try {
-          const observation = await client.observeBrowserTarget(
-            workspaceId,
-            browserSessionId,
-            selected.id,
-            { signal: controller.signal },
-          );
-          if (!mountedRef.current || requestRef.current.id !== id) return;
-          observationRef.current = { browserSessionId, observation };
-          lastObservationAtRef.current = Date.now();
-          setState((current) =>
-            current.browserSessionId === browserSessionId &&
-            current.selectedTargetId === observation.target.id
-              ? { ...current, observation }
-              : current,
-          );
-        } catch (cause) {
-          if (controller.signal.aborted || !mountedRef.current || requestRef.current.id !== id)
-            return;
-          // Target inventory and the media plane remain authoritative for the
-          // human-facing browser. A failed semantic snapshot must not blank or
-          // disable an otherwise live visual browser.
-          void cause;
-        }
       } catch (cause) {
         if (controller.signal.aborted || !mountedRef.current || requestRef.current.id !== id)
           return;
-        setState((current) =>
-          current.browserSessionId === browserSessionId
-            ? {
-                ...current,
-                loading: false,
-                error: cause instanceof Error ? cause : new Error(String(cause)),
-              }
-            : current,
-        );
-      } finally {
-        if (requestRef.current.id === id) {
-          requestRef.current = { id, controller: null };
-        }
+        // Target inventory and the media plane remain authoritative for the
+        // human-facing browser. A failed semantic snapshot must not blank or
+        // disable an otherwise live visual browser.
+        void cause;
       }
-    },
-    [browserSessionId, client, enabled, workspaceId],
-  );
-
-  const refresh = useCallback(async () => await refreshInventory(true), [refreshInventory]);
+    } catch (cause) {
+      if (controller.signal.aborted || !mountedRef.current || requestRef.current.id !== id) return;
+      setState((current) =>
+        current.browserSessionId === browserSessionId
+          ? {
+              ...current,
+              loading: false,
+              error: cause instanceof Error ? cause : new Error(String(cause)),
+            }
+          : current,
+      );
+    } finally {
+      if (requestRef.current.id === id) {
+        requestRef.current = { id, controller: null };
+      }
+    }
+  }, [browserSessionId, client, enabled, workspaceId]);
 
   const refreshSemantic = useCallback(async (): Promise<void> => {
     if (!enabled || !browserSessionId) return;
@@ -289,12 +265,12 @@ export function useBrowserSession(options: UseBrowserSessionOptions): UseBrowser
         ? { ...current, loading: true }
         : emptyState(browserSessionId, true),
     );
-    void refreshInventory();
+    void refresh();
     return () => {
       mountedRef.current = false;
       invalidateRefresh();
     };
-  }, [browserSessionId, enabled, invalidateRefresh, refreshInventory]);
+  }, [browserSessionId, enabled, invalidateRefresh, refresh]);
 
   useEffect(() => {
     const previous = previousSemanticObservationRef.current;
@@ -330,17 +306,17 @@ export function useBrowserSession(options: UseBrowserSessionOptions): UseBrowser
   useEffect(() => {
     if (!enabled || !pageLive || visible.mutating || refreshBlocked) return;
     const timer = setInterval(() => {
-      if (!requestRef.current.controller) void refreshInventory();
+      if (!requestRef.current.controller) void refresh();
     }, pollIntervalMs);
     return () => clearInterval(timer);
-  }, [enabled, pageLive, pollIntervalMs, refreshInventory, refreshBlocked, visible.mutating]);
+  }, [enabled, pageLive, pollIntervalMs, refresh, refreshBlocked, visible.mutating]);
 
   useEffect(() => {
     if (!enabled || !browserSessionId || !pageLive || refreshBlocked) return;
     let disposed = false;
     const heartbeat = () => {
       void client.heartbeatBrowserSession(workspaceId, browserSessionId).catch(() => {
-        if (!disposed) void refreshInventory();
+        if (!disposed) void refresh();
       });
     };
     const timer = setInterval(heartbeat, 30_000);
@@ -348,7 +324,7 @@ export function useBrowserSession(options: UseBrowserSessionOptions): UseBrowser
       disposed = true;
       clearInterval(timer);
     };
-  }, [browserSessionId, client, enabled, pageLive, refreshInventory, refreshBlocked, workspaceId]);
+  }, [browserSessionId, client, enabled, pageLive, refresh, refreshBlocked, workspaceId]);
 
   const runMutation = useCallback(
     async <T>(scopeBrowserSessionId: string, operation: () => Promise<T>): Promise<T> => {
@@ -556,16 +532,6 @@ export function useBrowserSession(options: UseBrowserSessionOptions): UseBrowser
           observationMode,
           action,
         });
-        if (receipt.state !== "completed") {
-          const inputFailure: BrowserInputFailure = {
-            operationId: receipt.operationId,
-            state: receipt.state,
-            error: receipt.error,
-          };
-          setState((current) =>
-            current.browserSessionId === browserSessionId ? { ...current, inputFailure } : current,
-          );
-        }
         if (receipt.observation) {
           observationRef.current = {
             browserSessionId,
@@ -677,7 +643,6 @@ export function useBrowserSession(options: UseBrowserSessionOptions): UseBrowser
     loading: visible.loading,
     mutating: visible.mutating,
     error: visible.error,
-    inputFailure: visible.inputFailure,
     refresh,
     observeForInput,
     actFromObservation,
@@ -701,7 +666,6 @@ function emptyState(browserSessionId: string | null, loading: boolean) {
     loading,
     mutating: false,
     error: null as Error | null,
-    inputFailure: null as BrowserInputFailure | null,
   };
 }