fix(ui): avoid stale load closure in useViewResource reload

Keep the latest load callback in a ref so reload() always invokes the
current fetcher without changing its own identity on every render.
Reduces full-suite test failures from 5 to 3 (remaining are pre-existing).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

job-tracker-ui/src/hooks/useViewResource.ts

@@ -65,11 +65,16 @@ export function useViewResource<T>(
   const [hasLoaded, setHasLoaded] = useState(false);
   const [error, setError] = useState<ViewResourceError | null>(null);
   const hasLoadedRef = useRef(hasLoaded);
+  const loadRef = useRef(load);
 
   useEffect(() => {
     hasLoadedRef.current = hasLoaded;
   }, [hasLoaded]);
 
+  useEffect(() => {
+    loadRef.current = load;
+  }, [load]);
+
   const reload = useCallback(async () => {
     if (!enabled) return;
 
@@ -77,7 +82,7 @@ export function useViewResource<T>(
     setLoading(!alreadyLoaded);
     setRefreshing(alreadyLoaded);
     try {
-      const next = await load();
+      const next = await loadRef.current();
       setData(next);
       setError(null);
       setHasLoaded(true);
@@ -88,7 +93,7 @@ export function useViewResource<T>(
       setLoading(false);
       setRefreshing(false);
     }
-  }, [enabled, errorMessage, load]);
+  }, [enabled, errorMessage]);
 
   useEffect(() => {
     if (!enabled) {