@opengeni/react 0.4.0 → 0.5.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@opengeni/react",
-  "version": "0.4.0",
+  "version": "0.5.0",
   "description": "React hooks and styled components for OpenGeni: live session streaming, chat composer, message timeline, session status, and fleet views — token-themed (CSS variables), dark-first, built on Tailwind v4 + Radix + Motion.",
   "license": "Apache-2.0",
   "repository": {
@@ -40,7 +40,7 @@
     "demo:build": "vite build demo"
   },
   "dependencies": {
-    "@opengeni/sdk": "^0.4.0",
+    "@opengeni/sdk": "^0.5.0",
     "clsx": "^2.1.1",
     "lucide-react": "^1.8.0",
     "motion": "^12.0.0",

package/src/client.ts

@@ -11,6 +11,7 @@ export type SessionClientLike = Pick<
   | "getClientConfig"
   // Sessions, events, composer
   | "getSession"
+  | "updateSession"
   | "listSessions"
   | "sendMessage"
   | "steerMessage"

package/src/components/fleet-tile.tsx

@@ -15,6 +15,11 @@ export type FleetTileProps = {
 
 /** Best-effort display title for a session. */
 export function sessionDisplayTitle(session: Session): string {
+  // A set title (agent-generated or user-renamed) wins over metadata + the
+  // initial-message fallback.
+  if (typeof session.title === "string" && session.title.trim().length > 0) {
+    return session.title;
+  }
   for (const key of ["title", "name"] as const) {
     const value = session.metadata[key];
     if (typeof value === "string" && value.trim().length > 0) {

package/src/hooks/use-session.ts

@@ -1,33 +1,101 @@
-import type { Session } from "@opengeni/sdk";
-import { useCallback } from "react";
+import type { Session, SessionEvent } from "@opengeni/sdk";
+import { useCallback, useState } from "react";
 import { useOpenGeni, type ClientOverride } from "../provider";
-import { usePolledValue } from "./internal";
+import { useMutationRunner, usePolledValue, useSessionEventTrigger, type SessionEventFeedOptions } from "./internal";
 
-export type UseSessionOptions = ClientOverride & {
-  /** Re-fetch on an interval (ms). Off by default — pair with `useSessionEvents` for live status. */
-  pollIntervalMs?: number | undefined;
-  enabled?: boolean | undefined;
-};
+export type UseSessionOptions = ClientOverride &
+  SessionEventFeedOptions & {
+    /** Re-fetch on an interval (ms). Off by default — pair with `useSessionEvents` for live status. */
+    pollIntervalMs?: number | undefined;
+  };
 
 export type UseSessionResult = {
   session: Session | null;
   loading: boolean;
   error: Error | null;
   refresh: () => Promise<void>;
+  /** Manually rename the session (PATCH, source='user'). Returns the updated session, or null on failure. */
+  updateTitle: (title: string) => Promise<Session | null>;
+  /** True while a rename is in flight. */
+  updating: boolean;
+  mutationError: Error | null;
+  clearMutationError: () => void;
 };
 
-/** Fetch one session (with optional polling). */
+/** Event types that change the session title (auto + cross-client renames). */
+export function isTitleEvent(event: Pick<SessionEvent, "type">): boolean {
+  return event.type === "session.title_set";
+}
+
+/** Fetch one session (with optional polling), live-patching its title on `session.title_set`. */
 export function useSession(sessionId: string | null | undefined, options: UseSessionOptions = {}): UseSessionResult {
   const { client, workspaceId } = useOpenGeni(options);
+  const enabled = (options.enabled ?? true) && Boolean(sessionId);
+  const [override, setOverride] = useState<Session | null>(null);
+  const mutation = useMutationRunner();
   const load = useCallback(async () => {
     if (!sessionId) {
       return null;
     }
-    return await client.getSession(workspaceId, sessionId);
+    const fetched = await client.getSession(workspaceId, sessionId);
+    // A fresh server read supersedes any optimistic/event-driven override.
+    setOverride(null);
+    return fetched;
   }, [client, workspaceId, sessionId]);
   const state = usePolledValue(load, {
     pollIntervalMs: options.pollIntervalMs,
-    enabled: (options.enabled ?? true) && Boolean(sessionId),
+    enabled,
   });
-  return { session: state.data ?? null, loading: state.loading, error: state.error, refresh: state.refresh };
+
+  const base = state.data ?? null;
+  // The override only ever carries title/titleSource patches; it is reset on
+  // every fresh load so it can never go stale against the server snapshot.
+  const session = base && override && override.id === base.id ? { ...base, title: override.title, titleSource: override.titleSource } : base;
+
+  // Live-patch the title on auto (agent) + cross-client (user/agent) renames so
+  // the UI reflects the new title without polling or a full re-fetch.
+  const onTitleEvent = useCallback((event: SessionEvent) => {
+    const payload = (event.payload ?? {}) as { title?: unknown; source?: unknown };
+    const title = payload.title;
+    if (typeof title !== "string") {
+      return;
+    }
+    const source: "user" | "agent" | null = payload.source === "user" || payload.source === "agent" ? payload.source : null;
+    setOverride((current): Session | null => {
+      const next = current ?? base;
+      if (!next) {
+        return current;
+      }
+      return { ...next, title, titleSource: source };
+    });
+  }, [base]);
+  useSessionEventTrigger(client, workspaceId, sessionId, isTitleEvent, onTitleEvent, {
+    enabled,
+    ...(options.events !== undefined ? { events: options.events } : {}),
+  });
+
+  const updateTitle = useCallback(
+    async (title: string): Promise<Session | null> => {
+      if (!sessionId) {
+        return null;
+      }
+      const result = await mutation.run(() => client.updateSession(workspaceId, sessionId, { title }));
+      if (result) {
+        setOverride(result);
+      }
+      return result;
+    },
+    [client, workspaceId, sessionId, mutation.run],
+  );
+
+  return {
+    session,
+    loading: state.loading,
+    error: state.error,
+    refresh: state.refresh,
+    updateTitle,
+    updating: mutation.mutating,
+    mutationError: mutation.mutationError,
+    clearMutationError: mutation.clearMutationError,
+  };
 }

package/src/index.ts

@@ -9,7 +9,7 @@ export { OpenGeniProvider, useOpenGeni, useOpenGeniClient } from "./provider";
 export type { ClientOverride, OpenGeniContextValue, OpenGeniProviderProps } from "./provider";
 
 // Hooks
-export { useSession } from "./hooks/use-session";
+export { useSession, isTitleEvent } from "./hooks/use-session";
 export type { UseSessionOptions, UseSessionResult } from "./hooks/use-session";
 export { useSessionEvents } from "./hooks/use-session-events";
 export type { SessionEventsConnectionState, UseSessionEventsOptions, UseSessionEventsResult } from "./hooks/use-session-events";

package/src/lib/git-patch.ts

@@ -22,10 +22,16 @@ export function gitFileDiffToPatch(file: GitFileDiff): string {
     lines.push(`+++ b/${newPath}`);
   }
   for (const hunk of file.hunks) {
-    const header =
-      hunk.header && hunk.header.startsWith("@@")
-        ? hunk.header
-        : `@@ -${hunk.oldStart},${hunk.oldLines} +${hunk.newStart},${hunk.newLines} @@`;
+    // Only trust a pre-parsed header if it carries the full unified range form
+    // `@@ -<o>[,<n>] +<o>[,<n>] @@`. A synthesized create_file hunk (parsers.ts)
+    // can carry a degenerate `@@ +1 @@` with no `-`/`+` ranges; a generic patch
+    // parser (Pierre) renders zero lines from it, so the expanded diff comes up
+    // empty while the collapsed chip still shows the (correct) addition count.
+    // In that case regenerate a valid header from the hunk's range fields.
+    const headerIsValid = /^@@ -\d+(?:,\d+)? \+\d+(?:,\d+)? @@/.test(hunk.header ?? "");
+    const header = headerIsValid
+      ? hunk.header
+      : `@@ -${hunk.oldStart},${hunk.oldLines} +${hunk.newStart},${hunk.newLines} @@`;
     lines.push(header);
     for (const line of hunk.lines) {
       if (line.type === "meta") continue;

package/src/timeline/parsers.ts

@@ -131,7 +131,12 @@ export function v4aToGitFileDiff(op: ApplyPatchOperation): GitFileDiff {
         hunks.push(cur);
       } else if (cur || op.type === "create_file") {
         if (!cur) {
-          cur = { oldStart: 0, oldLines: 0, newStart: 1, newLines: 0, header: "@@ +1 @@", lines: [] };
+          // No `@@` anchor on a create_file body: synthesize an add-only hunk.
+          // Leave `header` empty so `gitFileDiffToPatch` regenerates a valid
+          // `@@ -0,0 +1,N @@` from the range fields once newLines is counted — a
+          // pre-baked partial header (e.g. `@@ +1 @@`) renders zero lines in a
+          // generic unified-diff parser.
+          cur = { oldStart: 0, oldLines: 0, newStart: 1, newLines: 0, header: "", lines: [] };
           hunks.push(cur);
           oldNo = 0;
           newNo = 1;