@opengeni/react 0.13.0 → 0.20.0

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

@@ -57,7 +57,8 @@ export function useSessionLineage(
   sessionId: string | null | undefined,
   options: UseSessionLineageOptions = {},
 ): UseSessionLineageResult {
-  const { client, workspaceId } = useOpenGeni(options);
+  const { client, workspaceId, workspaceControlEvent, registerSessionReconciler } =
+    useOpenGeni(options);
   const enabled = (options.enabled ?? true) && Boolean(sessionId);
   const load = useCallback(
     async () =>
@@ -67,7 +68,15 @@ export function useSessionLineage(
     [client, workspaceId, sessionId],
   );
   const state = usePolledValue(load, { pollIntervalMs: options.pollIntervalMs, enabled });
-  const refreshSoon = useDebouncedCallback(() => void state.refresh(), 150);
+  const refresh = state.refresh;
+  useEffect(() => {
+    if (enabled && workspaceControlEvent) void refresh();
+  }, [enabled, refresh, workspaceControlEvent]);
+  useEffect(() => {
+    if (!sessionId || !enabled) return;
+    return registerSessionReconciler(sessionId, "lineage", refresh);
+  }, [enabled, refresh, registerSessionReconciler, sessionId]);
+  const refreshSoon = useDebouncedCallback(() => void refresh(), 150);
   const delayedChildRefreshRef = useRef<ReturnType<typeof setTimeout> | null>(null);
   // Clear a pending delayed refresh on session/workspace SWITCH (not just
   // unmount): otherwise a timer scheduled for the previous session can fire
@@ -81,15 +90,15 @@ export function useSessionLineage(
     };
   }, [sessionId, workspaceId]);
   const refreshAfterChildCreate = useCallback(() => {
-    void state.refresh();
+    void refresh();
     if (delayedChildRefreshRef.current !== null) {
       clearTimeout(delayedChildRefreshRef.current);
     }
     delayedChildRefreshRef.current = setTimeout(() => {
       delayedChildRefreshRef.current = null;
-      void state.refresh();
+      void refresh();
     }, 2500);
-  }, [state]);
+  }, [refresh]);
   useSessionEventTrigger(
     client,
     workspaceId,
@@ -114,6 +123,6 @@ export function useSessionLineage(
     lineage: state.data,
     loading: state.loading,
     error: state.error,
-    refresh: state.refresh,
+    refresh,
   };
 }

package/src/hooks/use-session.ts

@@ -1,5 +1,5 @@
 import type { Session, SessionEvent } from "@opengeni/sdk";
-import { useCallback, useState } from "react";
+import { useCallback, useEffect, useState } from "react";
 import { useOpenGeni, type ClientOverride } from "../provider";
 import {
   useMutationRunner,
@@ -37,7 +37,8 @@ export function useSession(
   sessionId: string | null | undefined,
   options: UseSessionOptions = {},
 ): UseSessionResult {
-  const { client, workspaceId } = useOpenGeni(options);
+  const { client, workspaceId, workspaceControlEvent, registerSessionReconciler } =
+    useOpenGeni(options);
   const enabled = (options.enabled ?? true) && Boolean(sessionId);
   const [override, setOverride] = useState<Session | null>(null);
   const { run, mutating, mutationError, clearMutationError } = useMutationRunner();
@@ -54,6 +55,13 @@ export function useSession(
     pollIntervalMs: options.pollIntervalMs,
     enabled,
   });
+  useEffect(() => {
+    if (enabled && workspaceControlEvent) void refresh();
+  }, [enabled, refresh, workspaceControlEvent]);
+  useEffect(() => {
+    if (!sessionId || !enabled) return;
+    return registerSessionReconciler(sessionId, "session", refresh);
+  }, [enabled, refresh, registerSessionReconciler, sessionId]);
 
   const base = data ?? null;
   // The override only ever carries title/titleSource patches; it is reset on

package/src/hooks/use-slash-commands.ts

@@ -194,6 +194,10 @@ export function useSlashCommands(options: UseSlashCommandsOptions): UseSlashComm
     }
   }, [items, clampedHighlight, autocomplete]);
 
+  // Pointer and keyboard activation share this synchronous fence. React state
+  // cannot disable a command row before a second same-tick activation arrives.
+  const runningRef = useRef(false);
+
   // Resolve a SPECIFIC, already-chosen command against the current draft, then
   // either autocomplete (name-only / required arg missing) or execute. This is
   // the shared core for both Enter (which first resolves WHICH command via the
@@ -201,39 +205,44 @@ export function useSlashCommands(options: UseSlashCommandsOptions): UseSlashComm
   // command — the clicked row — and must not re-resolve to a near-match).
   const runResolved = useCallback(
     async (command: SlashCommand, executionOptions?: { explicit?: boolean }): Promise<void> => {
-      if (!parsed) {
-        return;
-      }
-      const explicit = executionOptions?.explicit ?? false;
-      // Token-vs-command equality is case-insensitive (matching the registry's
-      // matchCommand/filterCommands), so a fully-typed "/Clear" counts as having
-      // named `clear` and runs rather than autocompleting.
-      const nameMatchesToken =
-        command.name === parsed.name.toLowerCase() ||
-        (command.aliases?.includes(parsed.name.toLowerCase()) ?? false);
-      // A name-only token whose name doesn't yet equal the resolved command (e.g.
-      // "/cl" -> clear) first autocompletes so the operator sees the full name.
-      // An EXPLICIT pointer click skips this: the operator already chose the row,
-      // so clicking "/clear-view" (while the token is "clear") runs it outright
-      // rather than merely filling the name and waiting for a second Enter.
-      if (!explicit && !activeCommand && !nameMatchesToken && !parsed.hasTrailingSpace) {
-        autocomplete(command);
+      if (!parsed || runningRef.current) {
         return;
       }
-      // When the click resolves a different command than the typed token, the
-      // typed token's tail isn't this command's args — run with no positional
-      // args and let the required-arg guard below prompt for them via autocomplete.
-      const args = nameMatchesToken || parsed.hasTrailingSpace ? parsed.args : [];
-      const missing = firstMissingRequiredArg(command, args);
-      if (missing) {
-        // A required arg is absent: keep the palette open at the arg hint rather
-        // than firing a half-formed command.
-        if (!parsed.hasTrailingSpace) {
+      runningRef.current = true;
+      try {
+        const explicit = executionOptions?.explicit ?? false;
+        // Token-vs-command equality is case-insensitive (matching the registry's
+        // matchCommand/filterCommands), so a fully-typed "/Clear" counts as having
+        // named `clear` and runs rather than autocompleting.
+        const nameMatchesToken =
+          command.name === parsed.name.toLowerCase() ||
+          (command.aliases?.includes(parsed.name.toLowerCase()) ?? false);
+        // A name-only token whose name doesn't yet equal the resolved command (e.g.
+        // "/cl" -> clear) first autocompletes so the operator sees the full name.
+        // An EXPLICIT pointer click skips this: the operator already chose the row,
+        // so clicking "/clear-view" (while the token is "clear") runs it outright
+        // rather than merely filling the name and waiting for a second Enter.
+        if (!explicit && !activeCommand && !nameMatchesToken && !parsed.hasTrailingSpace) {
           autocomplete(command);
+          return;
         }
-        return;
+        // When the click resolves a different command than the typed token, the
+        // typed token's tail isn't this command's args — run with no positional
+        // args and let the required-arg guard below prompt for them via autocomplete.
+        const args = nameMatchesToken || parsed.hasTrailingSpace ? parsed.args : [];
+        const missing = firstMissingRequiredArg(command, args);
+        if (missing) {
+          // A required arg is absent: keep the palette open at the arg hint rather
+          // than firing a half-formed command.
+          if (!parsed.hasTrailingSpace) {
+            autocomplete(command);
+          }
+          return;
+        }
+        await execute(command, args);
+      } finally {
+        runningRef.current = false;
       }
-      await execute(command, args);
     },
     [parsed, activeCommand, autocomplete, execute],
   );
@@ -286,9 +295,6 @@ export function useSlashCommands(options: UseSlashCommandsOptions): UseSlashComm
     [items, runResolved],
   );
 
-  // Track an in-flight run so Enter can't double-fire.
-  const runningRef = useRef(false);
-
   const onKeyDown = useCallback(
     (event: KeyboardEvent<HTMLTextAreaElement>): boolean => {
       if (!open) {
@@ -322,13 +328,7 @@ export function useSlashCommands(options: UseSlashCommandsOptions): UseSlashComm
             return false;
           }
           event.preventDefault();
-          if (runningRef.current) {
-            return true;
-          }
-          runningRef.current = true;
-          void runHighlighted().finally(() => {
-            runningRef.current = false;
-          });
+          void runHighlighted();
           return true;
         }
         case "Escape": {