Don't use type casting

Automattic · Feb 4, 2025 · 16156da · 16156da
1 parent 88d1c8f
commit 16156da
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/projects/js-packages/react-data-sync-client/src/DataSyncHooks.ts b/projects/js-packages/react-data-sync-client/src/DataSyncHooks.ts
@@ -125,8 +125,13 @@ export function useDataSync<
 	// AbortController is used to track rapid value mutations
 	// and will cancel in-flight requests and prevent
 	// the optimistic value from being reverted.
-	const getAbortController = () =>
-		queryClient.getMutationDefaults( queryKey )?.meta?.abortController as AbortController;
+	const getAbortController = () => {
+		const defaults = queryClient.getMutationDefaults( queryKey );
+		return defaults?.meta?.abortController instanceof AbortController
+			? defaults.meta.abortController
+			: undefined;
+	};
+
 	const setAbortController = ( abortController: AbortController ) => {
 		queryClient.setMutationDefaults( queryKey, {
 			meta: {
@@ -146,7 +151,7 @@ export function useDataSync<
 	 */
 	const mutationConfigDefaults = {
 		meta: {
-			abortController: null as AbortController | null,
+			abortController: null,
 		},
 		// Mutation function that's called when the mutation is triggered
 		mutationFn: value => datasync.SET( value, params, getAbortController()?.signal ),