@smithers-orchestrator/gateway-react 0.18.0 → 0.19.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smithers-orchestrator/gateway-react",
-  "version": "0.18.0",
+  "version": "0.19.0",
   "description": "React hooks and root helpers for Smithers Gateway UIs",
   "type": "module",
   "sideEffects": false,
@@ -20,8 +20,8 @@
     "src/"
   ],
   "dependencies": {
-    "@smithers-orchestrator/gateway": "0.18.0",
-    "@smithers-orchestrator/gateway-client": "0.18.0"
+    "@smithers-orchestrator/gateway": "0.19.0",
+    "@smithers-orchestrator/gateway-client": "0.19.0"
   },
   "peerDependencies": {
     "react": "^19.0.0",

package/src/useGatewayActions.ts

@@ -8,8 +8,13 @@ export function useGatewayActions() {
       launchRun: client.launchRun.bind(client),
       resumeRun: client.resumeRun.bind(client),
       cancelRun: client.cancelRun.bind(client),
+      hijackRun: client.hijackRun.bind(client),
+      rewindRun: client.rewindRun.bind(client),
       submitApproval: client.submitApproval.bind(client),
       submitSignal: client.submitSignal.bind(client),
+      cronCreate: client.cronCreate.bind(client),
+      cronDelete: client.cronDelete.bind(client),
+      cronRun: client.cronRun.bind(client),
     }),
     [client],
   );

package/src/useGatewayRpc.ts

@@ -1,4 +1,4 @@
-import { useCallback, useEffect, useMemo, useState } from "react";
+import { useCallback, useEffect, useMemo, useRef, useState } from "react";
 import type { GatewayRpcMethod } from "@smithers-orchestrator/gateway/rpc";
 import type { GatewayRpcParams, GatewayRpcPayload } from "@smithers-orchestrator/gateway-client";
 import { useSmithersGateway } from "./useSmithersGateway.ts";
@@ -13,6 +13,14 @@ export function useGatewayRpc<Method extends GatewayRpcMethod>(
   const enabled = options.enabled ?? true;
   const paramsKey = useMemo(() => JSON.stringify(params ?? {}), [params]);
   const deps = options.deps ?? [paramsKey];
+  const paramsRef = useRef(params);
+  const previousEffectRef = useRef<{
+    client: typeof client;
+    enabled: boolean;
+    method: Method;
+    deps: readonly unknown[];
+  } | undefined>(undefined);
+  paramsRef.current = params;
   const [data, setData] = useState<GatewayRpcPayload<Method>>();
   const [error, setError] = useState<Error>();
   const [loading, setLoading] = useState(enabled);
@@ -24,17 +32,27 @@ export function useGatewayRpc<Method extends GatewayRpcMethod>(
     setLoading(true);
     setError(undefined);
     try {
-      setData(await client.rpc(method, params));
+      setData(await client.rpc(method, paramsRef.current));
     } catch (cause) {
       setError(cause instanceof Error ? cause : new Error(String(cause)));
     } finally {
       setLoading(false);
     }
-  }, [client, enabled, method, paramsKey]);
+  }, [client, enabled, method]);
 
   useEffect(() => {
-    void refetch();
-  }, [refetch, ...deps]);
+    const previous = previousEffectRef.current;
+    const changed = !previous ||
+      previous.client !== client ||
+      previous.enabled !== enabled ||
+      previous.method !== method ||
+      previous.deps.length !== deps.length ||
+      deps.some((dep, index) => !Object.is(dep, previous.deps[index]));
+    previousEffectRef.current = { client, enabled, method, deps: [...deps] };
+    if (changed) {
+      void refetch();
+    }
+  });
 
   return { data, error, loading, refetch };
 }