@salesforce/templates 66.8.0 → 66.9.0

package/lib/templates/uiBundles/reactbasic/src/api/graphqlClient.ts

@@ -1,28 +1,28 @@
 /**
- * Thin GraphQL client: createDataSDK + data.graphql with centralized error handling.
- * Use with gql-tagged queries and generated operation types for type-safe calls.
+ * Thin GraphQL client: createDataSDK + sdk.graphql.query with centralized
+ * error handling. Use with gql-tagged queries and generated operation types
+ * for type-safe calls.
  */
-import { createDataSDK } from '@salesforce/sdk-data';
+import { createDataSDK } from '@salesforce/platform-sdk';
 
 export async function executeGraphQL<TData, TVariables>(
   query: string,
   variables?: TVariables
 ): Promise<TData> {
   const data = await createDataSDK();
-  // SDK types graphql() first param as string; at runtime it may accept gql DocumentNode too
-  const response = await data.graphql?.<TData, TVariables>({
-    query,
-    variables,
+  const result = await data.graphql!.query<TData, TVariables>({
+    query: query,
+    variables: variables,
   });
 
-  if (!response) {
-    throw new Error('GraphQL response is undefined');
+  if (result.errors?.length) {
+    const msg = result.errors.map(e => e.message).join('; ');
+    throw new Error(`GraphQL Error: ${msg}`);
   }
 
-  if (response?.errors?.length) {
-    const msg = response.errors.map(e => e.message).join('; ');
-    throw new Error(`GraphQL Error: ${msg}`);
+  if (result.data == null) {
+    throw new Error('GraphQL response data is null');
   }
 
-  return response.data;
+  return result.data;
 }

package/lib/templates/uiBundles/reactbasic/src/components/alerts/status-alert.tsx

@@ -1,5 +1,5 @@
 import { cva, type VariantProps } from 'class-variance-authority';
-import { AlertCircleIcon, CheckCircle2Icon } from 'lucide-react';
+import { AlertCircleIcon, CheckCircle2Icon, InfoIcon } from 'lucide-react';
 import { Alert, AlertDescription } from '../../components/ui/alert';
 import { useId } from 'react';
 
@@ -8,6 +8,7 @@ const statusAlertVariants = cva('', {
     variant: {
       error: '',
       success: '',
+      info: 'text-blue-600 *:[svg]:text-current *:data-[slot=alert-description]:text-blue-600/90',
     },
   },
   defaultVariants: {
@@ -18,11 +19,11 @@ const statusAlertVariants = cva('', {
 interface StatusAlertProps extends VariantProps<typeof statusAlertVariants> {
   children?: React.ReactNode;
   /** Alert variant type. @default "error" */
-  variant?: 'error' | 'success';
+  variant?: 'error' | 'success' | 'info';
 }
 
 /**
- * Status alert component for displaying error or success messages.
+ * Status alert component for displaying error, success, or info messages.
  * Returns null if no children are provided.
  */
 export function StatusAlert({ children, variant = 'error' }: StatusAlertProps) {
@@ -31,6 +32,12 @@ export function StatusAlert({ children, variant = 'error' }: StatusAlertProps) {
 
   const isError = variant === 'error';
 
+  const icon = {
+    error: <AlertCircleIcon aria-hidden="true" />,
+    success: <CheckCircle2Icon aria-hidden="true" />,
+    info: <InfoIcon aria-hidden="true" />,
+  }[variant];
+
   return (
     <Alert
       variant={isError ? 'destructive' : 'default'}
@@ -38,11 +45,7 @@ export function StatusAlert({ children, variant = 'error' }: StatusAlertProps) {
       aria-describedby={descriptionId}
       role={isError ? 'alert' : 'status'}
     >
-      {isError ? (
-        <AlertCircleIcon aria-hidden="true" />
-      ) : (
-        <CheckCircle2Icon aria-hidden="true" />
-      )}
+      {icon}
       <AlertDescription id={descriptionId}>{children}</AlertDescription>
     </Alert>
   );

package/lib/templates/uiBundles/reactbasic/src/components/ui/input.tsx

@@ -8,7 +8,7 @@ function Input({ className, type, ...props }: React.ComponentProps<'input'>) {
       type={type}
       data-slot="input"
       className={cn(
-        'dark:bg-input/30 border-input focus-visible:border-ring focus-visible:ring-ring/50 aria-invalid:ring-destructive/20 dark:aria-invalid:ring-destructive/40 aria-invalid:border-destructive dark:aria-invalid:border-destructive/50 disabled:bg-input/50 dark:disabled:bg-input/80 h-8 rounded-lg border bg-transparent px-2.5 py-1 text-base transition-colors file:h-6 file:text-sm file:font-medium focus-visible:ring-3 aria-invalid:ring-3 md:text-sm file:text-foreground placeholder:text-muted-foreground w-full min-w-0 outline-none file:inline-flex file:border-0 file:bg-transparent disabled:pointer-events-none disabled:cursor-not-allowed disabled:opacity-50',
+        'dark:bg-input/30 border-input focus-visible:border-ring focus-visible:ring-ring/50 aria-invalid:ring-destructive/20 dark:aria-invalid:ring-destructive/40 aria-invalid:border-destructive dark:aria-invalid:border-destructive/50 disabled:bg-input/50 dark:disabled:bg-input/80 h-8 rounded-lg border bg-transparent px-2.5 py-1 text-base transition-colors file:h-6 file:text-sm file:font-medium focus-visible:ring-3 aria-invalid:ring-3 md:text-sm file:text-foreground placeholder:text-muted-foreground/70 placeholder:italic w-full min-w-0 outline-none file:inline-flex file:border-0 file:bg-transparent disabled:pointer-events-none disabled:cursor-not-allowed disabled:opacity-50',
         className
       )}
       {...props}

package/lib/templates/uiBundles/reactbasic/src/hooks/useAsyncData.ts

@@ -0,0 +1,67 @@
+import { useEffect, useRef, useState } from 'react';
+
+interface UseAsyncDataResult<T> {
+  data: T | null;
+  loading: boolean;
+  error: string | null;
+}
+
+/**
+ * Runs an async fetcher on mount and whenever `deps` change.
+ * Returns the loading/error/data state. Does not cache — every call
+ * to the fetcher hits the source directly.
+ *
+ * A cleanup flag prevents state updates if the component unmounts
+ * or deps change before the fetch completes (avoids React warnings
+ * and stale updates from out-of-order responses).
+ */
+export function useAsyncData<T>(
+  fetcher: () => Promise<T>,
+  deps: React.DependencyList
+): UseAsyncDataResult<T> {
+  const [data, setData] = useState<T | null>(null);
+  const [loading, setLoading] = useState(true);
+  const [error, setError] = useState<string | null>(null);
+  const [generation, setGeneration] = useState(0);
+
+  const fetcherRef = useRef(fetcher);
+  useEffect(() => {
+    fetcherRef.current = fetcher;
+  });
+
+  // Detect dep changes during render to reset loading state and bump generation
+  const [prevDeps, setPrevDeps] = useState(deps);
+  if (
+    deps.length !== prevDeps.length ||
+    deps.some((d, i) => d !== prevDeps[i])
+  ) {
+    setPrevDeps(deps);
+    setGeneration(g => g + 1);
+    if (!loading) setLoading(true);
+    if (error !== null) setError(null);
+  }
+
+  useEffect(() => {
+    let cancelled = false;
+
+    fetcherRef
+      .current()
+      .then(result => {
+        if (!cancelled) setData(result);
+      })
+      .catch(err => {
+        console.error(err);
+        if (!cancelled)
+          setError(err instanceof Error ? err.message : 'An error occurred');
+      })
+      .finally(() => {
+        if (!cancelled) setLoading(false);
+      });
+
+    return () => {
+      cancelled = true;
+    };
+  }, [generation]);
+
+  return { data, loading, error };
+}

package/lib/templates/uiBundles/reactbasic/tsconfig.json

@@ -28,11 +28,8 @@
       "@utils/*": ["./src/utils/*"],
       "@styles/*": ["./src/styles/*"],
       "@assets/*": ["./src/assets/*"],
-      "@salesforce/sdk-core": [
-        "../../../../../../../../../../packages/sdk/sdk-core/src/index.ts"
-      ],
-      "@salesforce/sdk-data": [
-        "../../../../../../../../../../packages/sdk/sdk-data/src/index.ts"
+      "@salesforce/platform-sdk": [
+        "../../../../../../../../../../packages/sdk/platform-sdk/src/index.ts"
       ]
     }
   },
@@ -41,7 +38,8 @@
     "e2e",
     "vite-env.d.ts",
     "vitest-env.d.ts",
-    "vitest.setup.ts"
+    "vitest.setup.ts",
+    "../../../../../../../../../../packages/sdk/platform-sdk/types"
   ],
   "references": [{ "path": "./tsconfig.node.json" }]
 }

package/lib/templates/uiBundles/reactbasic/vite-env.d.ts

@@ -2,6 +2,3 @@
 
 /** Salesforce API version injected at build time by the Vite define plugin. */
 declare const __SF_API_VERSION__: string;
-
-/** Package version of @salesforce/sdk-data injected at build time by its own Vite build. */
-declare const __SDK_DATA_VERSION__: string;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@salesforce/templates",
-  "version": "66.8.0",
+  "version": "66.9.0",
   "description": "Salesforce JS library for templates",
   "bugs": "https://github.com/forcedotcom/salesforcedx-templates/issues",
   "main": "lib/index.js",
@@ -25,10 +25,10 @@
     "@salesforce/dev-config": "^4.3.3",
     "@salesforce/dev-scripts": "^11.0.2",
     "@salesforce/prettier-config": "^0.0.4",
-    "@salesforce/ui-bundle-template-app-react-template-b2e": "^1.135.0",
-    "@salesforce/ui-bundle-template-app-react-template-b2x": "^1.135.0",
-    "@salesforce/ui-bundle-template-base-react-app": "^1.135.0",
-    "@salesforce/ui-bundle-template-base-web-app": "^1.135.0",
+    "@salesforce/ui-bundle-template-app-react-template-b2e": "^9.16.0",
+    "@salesforce/ui-bundle-template-app-react-template-b2x": "^9.16.0",
+    "@salesforce/ui-bundle-template-base-react-app": "^9.16.0",
+    "@salesforce/ui-bundle-template-base-web-app": "^9.16.0",
     "@types/chai-as-promised": "^7.1.8",
     "@types/ejs": "^3.1.5",
     "@types/mime-types": "^3.0.1",

package/lib/templates/project/reactexternalapp/_p_/_m_/_w_/_a_/src/_f_/_os_/hooks/useCachedAsyncData.ts

@@ -1,188 +0,0 @@
-import { useEffect, useRef, useState } from "react";
-
-interface UseAsyncDataResult<T> {
-	data: T | null;
-	loading: boolean;
-	error: string | null;
-}
-
-interface CacheOptions {
-	/** Unique cache key. Used for lookups and invalidation via `clearCacheEntry`. */
-	key: string;
-	/** Time-to-live in ms. Default: 30_000 (30s) */
-	ttl?: number;
-	/** Max entries in the cache. Default: 50. Evicts oldest entry when exceeded. */
-	maxSize?: number;
-}
-
-interface CacheEntry {
-	data: unknown;
-	timestamp: number;
-}
-
-/**
- * Module-level cache shared across all useCachedAsyncData consumers.
- * Cleared automatically on page reload since it only lives in memory.
- * Keys are caller-provided so different hook call-sites get independent entries.
- */
-const cache = new Map<string, CacheEntry>();
-
-/**
- * Returns a cached entry if it exists and hasn't exceeded its TTL.
- * Expired entries are deleted lazily here rather than on a timer,
- * so there's no background cleanup overhead.
- */
-function getValidEntry(key: string, ttl: number): CacheEntry | undefined {
-	const entry = cache.get(key);
-	if (!entry) return undefined;
-	if (Date.now() - entry.timestamp > ttl) {
-		cache.delete(key);
-		return undefined;
-	}
-	return entry;
-}
-
-/**
- * Stores a result in the cache. If the cache is at capacity and the key
- * is new, the oldest entry (first in Map insertion order — FIFO) is evicted.
- */
-function setEntry(key: string, data: unknown, maxSize: number): void {
-	if (!cache.has(key) && cache.size >= maxSize) {
-		const firstKey = cache.keys().next().value;
-		if (firstKey !== undefined) cache.delete(firstKey);
-	}
-	cache.set(key, { data, timestamp: Date.now() });
-}
-
-/**
- * Removes a single cache entry by key. The key must match the
- * `options.key` passed to `useCachedAsyncData`.
- *
- * @example
- * // Hook:
- * useCachedAsyncData(() => fetchAccountDetail(id), [id], { key: `account:${id}` });
- *
- * // Invalidate after a mutation:
- * await updateAccount(id, fields);
- * clearCacheEntry(`account:${id}`);
- */
-export function clearCacheEntry(key: string): void {
-	cache.delete(key);
-}
-
-/**
- * Removes all cache entries. Useful for global invalidation scenarios
- * like user logout or after a bulk operation.
- *
- * @example
- * async function handleLogout() {
- *   await logout();
- *   clearCache();
- *   navigate("/login");
- * }
- */
-export function clearCache(): void {
-	cache.clear();
-}
-
-/**
- * Async data hook with in-memory caching. Works like `useAsyncData` but
- * avoids redundant network calls when the same data is requested again
- * (e.g. navigating away from a page and back).
- *
- * Cache behaviour:
- * - **Key**: Provided via `options.key`. Must be unique per logical data source.
- *   Use the same key with `clearCacheEntry` to invalidate.
- * - **Hit**: Data is returned synchronously on the initial render —
- *   `loading` starts as `false`, so there's no flash of a loading state.
- * - **Miss**: The fetcher runs, and the result is stored for future hits.
- * - **TTL**: Entries expire after `options.ttl` ms (default 30 s).
- *   Expiry is checked lazily on read, not on a timer.
- * - **Max size**: Oldest entries are evicted FIFO when the cache exceeds
- *   `options.maxSize` (default 50).
- *
- * @example
- * // Cache picklist options for 5 minutes (data rarely changes)
- * const { data: types } = useCachedAsyncData(fetchDistinctTypes, [], {
- *   key: "distinctTypes",
- *   ttl: 300_000,
- * });
- *
- * // Cache search results with default 30 s TTL (back-nav returns instantly)
- * const { data } = useCachedAsyncData(
- *   () => searchAccounts({ where, orderBy, first, after }),
- *   [where, orderBy, first, after],
- *   { key: `accounts:${JSON.stringify({ where, orderBy, first, after })}` },
- * );
- *
- * // Invalidate a specific entry after a mutation
- * await updateAccount(id, fields);
- * clearCacheEntry(`account:${id}`);
- *
- * // Or clear everything (e.g. on logout)
- * clearCache();
- */
-export function useCachedAsyncData<T>(
-	fetcher: () => Promise<T>,
-	deps: React.DependencyList,
-	options: CacheOptions,
-): UseAsyncDataResult<T> {
-	const ttl = options.ttl ?? 30_000;
-	const maxSize = options.maxSize ?? 50;
-	const cacheKey = options.key;
-
-	// Synchronous cache check during state initialization so a cache hit
-	// never triggers a loading → loaded transition (avoids UI flicker).
-	const cached = getValidEntry(cacheKey, ttl);
-
-	const [data, setData] = useState<T | null>((cached?.data as T) ?? null);
-	const [loading, setLoading] = useState(!cached);
-	const [error, setError] = useState<string | null>(null);
-
-	const fetcherRef = useRef(fetcher);
-	useEffect(() => {
-		fetcherRef.current = fetcher;
-	});
-
-	useEffect(() => {
-		// Re-check the cache inside the effect because deps may have changed
-		// since the initial render (e.g. StrictMode double-invoke).
-		const entry = getValidEntry(cacheKey, ttl);
-		if (entry) {
-			setData(entry.data as T);
-			setLoading(false);
-			setError(null);
-			return;
-		}
-
-		// No cache hit — fetch from the network.
-		let cancelled = false;
-		setLoading(true);
-		setError(null);
-
-		fetcherRef
-			.current()
-			.then((result) => {
-				if (!cancelled) {
-					setEntry(cacheKey, result, maxSize);
-					setData(result);
-				}
-			})
-			.catch((err) => {
-				console.error(err);
-				if (!cancelled) setError(err instanceof Error ? err.message : "An error occurred");
-			})
-			.finally(() => {
-				if (!cancelled) setLoading(false);
-			});
-
-		// Cleanup: if deps change or the component unmounts before the fetch
-		// completes, the cancelled flag prevents stale state updates.
-		return () => {
-			cancelled = true;
-		};
-		// eslint-disable-next-line react-hooks/exhaustive-deps --- deps are explicitly managed by the caller
-	}, [...deps, cacheKey, ttl, maxSize]);
-
-	return { data, loading, error };
-}

package/lib/templates/project/reactexternalapp/_p_/_m_/_w_/_a_/src/_f_/authentication/layout/card-skeleton.tsx

@@ -1,38 +0,0 @@
-import { CenteredPageLayout } from "./centered-page-layout";
-import { Card, CardContent, CardHeader } from "../../../components/ui/card";
-import { Skeleton } from "../../../components/ui/skeleton";
-
-interface CardSkeletonProps {
-	/**
-	 * Maximum width of the content container.
-	 * @default "sm"
-	 */
-	contentMaxWidth?: "sm" | "md" | "lg";
-	/**
-	 * Accessible label for screen readers.
-	 * @default "Loading…"
-	 */
-	loadingText?: string;
-}
-
-/**
- * Full-page loading indicator with skeleton card placeholder.
- */
-export function CardSkeleton({ contentMaxWidth, loadingText = "Loading…" }: CardSkeletonProps) {
-	return (
-		<CenteredPageLayout contentMaxWidth={contentMaxWidth}>
-			<div role="status" aria-live="polite">
-				<Card className="w-full">
-					<CardHeader>
-						<Skeleton className="h-4 w-2/3" />
-						<Skeleton className="h-4 w-1/2" />
-					</CardHeader>
-					<CardContent>
-						<Skeleton className="aspect-video w-full" />
-					</CardContent>
-				</Card>
-				<span className="sr-only">{loadingText}</span>
-			</div>
-		</CenteredPageLayout>
-	);
-}

package/lib/templates/project/reactexternalapp/_p_/_m_/_w_/_a_/src/_f_/authentication/layouts/authenticationRouteLayout.tsx

@@ -1,21 +0,0 @@
-import { Navigate, Outlet, useSearchParams } from "react-router";
-import { useAuth } from "../context/AuthContext";
-import { getStartUrl } from "../authHelpers";
-import { CardSkeleton } from "../layout/card-skeleton";
-
-/**
- * [Dev Note] "Public Only" Route Guard:
- * This component protects routes that should NOT be accessible if the user is already logged in
- * (e.g., Login, Register, Forgot Password).
- * If an authenticated user tries to access these pages, they are automatically redirected
- * to the default authenticated view (e.g., Home or Profile) to prevent confusion.
- */
-export default function AuthenticationRoute() {
-	const { isAuthenticated, loading } = useAuth();
-	const [searchParams] = useSearchParams();
-
-	if (loading) return <CardSkeleton contentMaxWidth="md" />;
-	if (isAuthenticated) return <Navigate to={getStartUrl(searchParams)} replace />;
-
-	return <Outlet />;
-}

package/lib/templates/project/reactinternalapp/_p_/_m_/_w_/_a_/src/_f_/_os_/hooks/useCachedAsyncData.ts

@@ -1,188 +0,0 @@
-import { useEffect, useRef, useState } from "react";
-
-interface UseAsyncDataResult<T> {
-	data: T | null;
-	loading: boolean;
-	error: string | null;
-}
-
-interface CacheOptions {
-	/** Unique cache key. Used for lookups and invalidation via `clearCacheEntry`. */
-	key: string;
-	/** Time-to-live in ms. Default: 30_000 (30s) */
-	ttl?: number;
-	/** Max entries in the cache. Default: 50. Evicts oldest entry when exceeded. */
-	maxSize?: number;
-}
-
-interface CacheEntry {
-	data: unknown;
-	timestamp: number;
-}
-
-/**
- * Module-level cache shared across all useCachedAsyncData consumers.
- * Cleared automatically on page reload since it only lives in memory.
- * Keys are caller-provided so different hook call-sites get independent entries.
- */
-const cache = new Map<string, CacheEntry>();
-
-/**
- * Returns a cached entry if it exists and hasn't exceeded its TTL.
- * Expired entries are deleted lazily here rather than on a timer,
- * so there's no background cleanup overhead.
- */
-function getValidEntry(key: string, ttl: number): CacheEntry | undefined {
-	const entry = cache.get(key);
-	if (!entry) return undefined;
-	if (Date.now() - entry.timestamp > ttl) {
-		cache.delete(key);
-		return undefined;
-	}
-	return entry;
-}
-
-/**
- * Stores a result in the cache. If the cache is at capacity and the key
- * is new, the oldest entry (first in Map insertion order — FIFO) is evicted.
- */
-function setEntry(key: string, data: unknown, maxSize: number): void {
-	if (!cache.has(key) && cache.size >= maxSize) {
-		const firstKey = cache.keys().next().value;
-		if (firstKey !== undefined) cache.delete(firstKey);
-	}
-	cache.set(key, { data, timestamp: Date.now() });
-}
-
-/**
- * Removes a single cache entry by key. The key must match the
- * `options.key` passed to `useCachedAsyncData`.
- *
- * @example
- * // Hook:
- * useCachedAsyncData(() => fetchAccountDetail(id), [id], { key: `account:${id}` });
- *
- * // Invalidate after a mutation:
- * await updateAccount(id, fields);
- * clearCacheEntry(`account:${id}`);
- */
-export function clearCacheEntry(key: string): void {
-	cache.delete(key);
-}
-
-/**
- * Removes all cache entries. Useful for global invalidation scenarios
- * like user logout or after a bulk operation.
- *
- * @example
- * async function handleLogout() {
- *   await logout();
- *   clearCache();
- *   navigate("/login");
- * }
- */
-export function clearCache(): void {
-	cache.clear();
-}
-
-/**
- * Async data hook with in-memory caching. Works like `useAsyncData` but
- * avoids redundant network calls when the same data is requested again
- * (e.g. navigating away from a page and back).
- *
- * Cache behaviour:
- * - **Key**: Provided via `options.key`. Must be unique per logical data source.
- *   Use the same key with `clearCacheEntry` to invalidate.
- * - **Hit**: Data is returned synchronously on the initial render —
- *   `loading` starts as `false`, so there's no flash of a loading state.
- * - **Miss**: The fetcher runs, and the result is stored for future hits.
- * - **TTL**: Entries expire after `options.ttl` ms (default 30 s).
- *   Expiry is checked lazily on read, not on a timer.
- * - **Max size**: Oldest entries are evicted FIFO when the cache exceeds
- *   `options.maxSize` (default 50).
- *
- * @example
- * // Cache picklist options for 5 minutes (data rarely changes)
- * const { data: types } = useCachedAsyncData(fetchDistinctTypes, [], {
- *   key: "distinctTypes",
- *   ttl: 300_000,
- * });
- *
- * // Cache search results with default 30 s TTL (back-nav returns instantly)
- * const { data } = useCachedAsyncData(
- *   () => searchAccounts({ where, orderBy, first, after }),
- *   [where, orderBy, first, after],
- *   { key: `accounts:${JSON.stringify({ where, orderBy, first, after })}` },
- * );
- *
- * // Invalidate a specific entry after a mutation
- * await updateAccount(id, fields);
- * clearCacheEntry(`account:${id}`);
- *
- * // Or clear everything (e.g. on logout)
- * clearCache();
- */
-export function useCachedAsyncData<T>(
-	fetcher: () => Promise<T>,
-	deps: React.DependencyList,
-	options: CacheOptions,
-): UseAsyncDataResult<T> {
-	const ttl = options.ttl ?? 30_000;
-	const maxSize = options.maxSize ?? 50;
-	const cacheKey = options.key;
-
-	// Synchronous cache check during state initialization so a cache hit
-	// never triggers a loading → loaded transition (avoids UI flicker).
-	const cached = getValidEntry(cacheKey, ttl);
-
-	const [data, setData] = useState<T | null>((cached?.data as T) ?? null);
-	const [loading, setLoading] = useState(!cached);
-	const [error, setError] = useState<string | null>(null);
-
-	const fetcherRef = useRef(fetcher);
-	useEffect(() => {
-		fetcherRef.current = fetcher;
-	});
-
-	useEffect(() => {
-		// Re-check the cache inside the effect because deps may have changed
-		// since the initial render (e.g. StrictMode double-invoke).
-		const entry = getValidEntry(cacheKey, ttl);
-		if (entry) {
-			setData(entry.data as T);
-			setLoading(false);
-			setError(null);
-			return;
-		}
-
-		// No cache hit — fetch from the network.
-		let cancelled = false;
-		setLoading(true);
-		setError(null);
-
-		fetcherRef
-			.current()
-			.then((result) => {
-				if (!cancelled) {
-					setEntry(cacheKey, result, maxSize);
-					setData(result);
-				}
-			})
-			.catch((err) => {
-				console.error(err);
-				if (!cancelled) setError(err instanceof Error ? err.message : "An error occurred");
-			})
-			.finally(() => {
-				if (!cancelled) setLoading(false);
-			});
-
-		// Cleanup: if deps change or the component unmounts before the fetch
-		// completes, the cancelled flag prevents stale state updates.
-		return () => {
-			cancelled = true;
-		};
-		// eslint-disable-next-line react-hooks/exhaustive-deps --- deps are explicitly managed by the caller
-	}, [...deps, cacheKey, ttl, maxSize]);
-
-	return { data, loading, error };
-}