Fix: hide builtin tool actions (#1257)

* fix: hide tool card actions for builtin tools

* fix: prevent stale data from preventing tool polling

ui/admin/app/components/tools/toolGrid/ToolCardActions.tsx

@@ -28,6 +28,8 @@ export function ToolCardActions({ tool }: { tool: ToolReference }) {
 		}
 	);
 
+	if (tool.builtin) return null;
+
 	return (
 		<div className="flex items-center gap-2">
 			{(forceRefresh.isLoading || isPolling) && <LoadingSpinner />}

ui/admin/app/hooks/usePollSingleTool.ts

@@ -1,10 +1,11 @@
-import { useEffect, useState } from "react";
+import { useEffect, useRef, useState } from "react";
 import useSWR from "swr";
 
 import { ToolReferenceService } from "~/lib/service/api/toolreferenceService";
 
 export function usePollSingleTool(toolId: string) {
 	const [isPolling, setIsPolling] = useState(false);
+	const isInitial = useRef(false);
 
 	const { mutate: updateTools } = useSWR(
 		isPolling ? ToolReferenceService.getToolReferences.key("tool") : null,
@@ -20,6 +21,14 @@ export function usePollSingleTool(toolId: string) {
 	);
 
 	useEffect(() => {
+		// skip initial poll in case data is stale
+		// stale data could give a false positive on the `resolved` property
+		// which would prematurely stop polling
+		if (isInitial.current) {
+			isInitial.current = false;
+			return;
+		}
+
 		if (!getTool.data) return;
 
 		setIsPolling(!getTool.data.resolved);
@@ -43,7 +52,10 @@ export function usePollSingleTool(toolId: string) {
 	}, [getTool.data, updateTools, toolId]);
 
 	return {
-		startPolling: () => setIsPolling(true),
+		startPolling: () => {
+			isInitial.current = true;
+			setIsPolling(true);
+		},
 		isPolling,
 	};
 }