Replace all Function references with AnyFunction

Author: aryaemami59

packages/toolkit/src/actionCreatorInvariantMiddleware.ts

@@ -1,20 +1,23 @@
 import type { Middleware } from 'redux'
 import { isActionCreator as isRTKAction } from './createAction'
+import type { AnyFunction } from './tsHelpers'
 
 export interface ActionCreatorInvariantMiddlewareOptions {
   /**
    * The function to identify whether a value is an action creator.
    * The default checks for a function with a static type property and match method.
    */
-  isActionCreator?: (action: unknown) => action is Function & { type?: unknown }
+  isActionCreator?: (
+    action: unknown,
+  ) => action is AnyFunction & { type?: unknown }
 }
 
 export function getMessage(type?: unknown) {
   const splitType = type ? `${type}`.split('/') : []
   const actionName = splitType[splitType.length - 1] || 'actionCreator'
   return `Detected an action creator with type "${
     type || 'unknown'
-  }" being dispatched. 
+  }" being dispatched.
 Make sure you're calling the action creator before dispatching, i.e. \`dispatch(${actionName}())\` instead of \`dispatch(${actionName})\`. This is necessary even if the action has no payload.`
 }
 

packages/toolkit/src/createAction.ts

@@ -1,9 +1,10 @@
 import { isAction } from 'redux'
 import type {
-  IsUnknownOrNonInferrable,
+  AnyFunction,
   IfMaybeUndefined,
   IfVoid,
   IsAny,
+  IsUnknownOrNonInferrable,
 } from './tsHelpers'
 import { hasMatchFunction } from './tsHelpers'
 
@@ -257,7 +258,7 @@ export function createAction<
   prepareAction: PA,
 ): PayloadActionCreator<ReturnType<PA>['payload'], T, PA>
 
-export function createAction(type: string, prepareAction?: Function): any {
+export function createAction(type: string, prepareAction?: AnyFunction): any {
   function actionCreator(...args: any[]) {
     if (prepareAction) {
       const prepared = prepareAction(...args)
@@ -290,7 +291,7 @@ export function createAction(type: string, prepareAction?: Function): any {
  */
 export function isActionCreator(
   action: unknown,
-): action is BaseActionCreator<unknown, string> & Function {
+): action is BaseActionCreator<unknown, string> & AnyFunction {
   return (
     typeof action === 'function' &&
     'type' in action &&

packages/toolkit/src/createReducer.ts

@@ -3,7 +3,7 @@ import { produce as createNextState, isDraft, isDraftable } from 'immer'
 import type { Action, Reducer, UnknownAction } from 'redux'
 import type { ActionReducerMapBuilder } from './mapBuilders'
 import { executeReducerBuilderCallback } from './mapBuilders'
-import type { NoInfer, TypeGuard } from './tsHelpers'
+import type { AnyFunction, NoInfer, TypeGuard } from './tsHelpers'
 import { freezeDraftable } from './utils'
 
 /**
@@ -63,7 +63,7 @@ export type CaseReducers<S, AS extends Actions> = {
   [T in keyof AS]: AS[T] extends Action ? CaseReducer<S, AS[T]> : void
 }
 
-export type NotFunction<T> = T extends Function ? never : T
+export type NotFunction<T> = T extends AnyFunction ? never : T
 
 function isStateFunction<S>(x: unknown): x is () => S {
   return typeof x === 'function'

packages/toolkit/src/createSlice.ts

@@ -25,7 +25,12 @@ import type {
 import { createReducer } from './createReducer'
 import type { ActionReducerMapBuilder, TypedActionCreator } from './mapBuilders'
 import { executeReducerBuilderCallback } from './mapBuilders'
-import type { Id, TypeGuard } from './tsHelpers'
+import type {
+  AnyFunction,
+  AnyNonNullishValue,
+  Id,
+  TypeGuard,
+} from './tsHelpers'
 import { emplace } from './utils'
 
 const asyncThunkSymbol = /* @__PURE__ */ Symbol.for(
@@ -864,7 +869,7 @@ interface ReducerHandlingContext<State> {
   >
   sliceCaseReducersByType: Record<string, CaseReducer<State, any>>
   sliceMatchers: ActionMatcherDescriptionCollection<State>
-  actionCreators: Record<string, Function>
+  actionCreators: Record<string, AnyFunction>
 }
 
 interface ReducerHandlingContextMethods<State> {
@@ -915,7 +920,7 @@ interface ReducerHandlingContextMethods<State> {
    */
   exposeAction(
     name: string,
-    actionCreator: Function,
+    actionCreator: AnyFunction,
   ): ReducerHandlingContextMethods<State>
   /**
    * Add a case reducer to be exposed under the final `slice.caseReducers` key.

packages/toolkit/src/devtoolsExtension.ts

@@ -1,5 +1,6 @@
 import type { Action, ActionCreator, StoreEnhancer } from 'redux'
 import { compose } from 'redux'
+import type { AnyFunction, AnyNonNullishValue } from './tsHelpers'
 
 /**
  * @public
@@ -223,7 +224,7 @@ export const composeWithDevTools: ComposeWithDevTools =
     : function () {
         if (arguments.length === 0) return undefined
         if (typeof arguments[0] === 'object') return compose
-        return compose.apply(null, arguments as any as Function[])
+        return compose.apply(null, arguments as any as AnyFunction[])
       }
 
 /**

packages/toolkit/src/query/endpointDefinitions.ts

@@ -1,4 +1,5 @@
 import type { Api } from '@reduxjs/toolkit/query'
+import type { AnyFunction, AnyNonNullishValue } from '../tsHelpers'
 import type {
   BaseQueryApi,
   BaseQueryArg,
@@ -783,7 +784,7 @@ export function calculateProvidedBy<ResultType, QueryArg, ErrorType, MetaType>(
   return []
 }
 
-function isFunction<T>(t: T): t is Extract<T, Function> {
+function isFunction<T>(t: T): t is Extract<T, AnyFunction> {
   return typeof t === 'function'
 }
 

packages/toolkit/src/tests/actionCreatorInvariantMiddleware.test.ts

@@ -1,8 +1,11 @@
 import type { ActionCreatorInvariantMiddlewareOptions } from '@internal/actionCreatorInvariantMiddleware'
-import { getMessage } from '@internal/actionCreatorInvariantMiddleware'
-import { createActionCreatorInvariantMiddleware } from '@internal/actionCreatorInvariantMiddleware'
+import {
+  createActionCreatorInvariantMiddleware,
+  getMessage,
+} from '@internal/actionCreatorInvariantMiddleware'
 import type { MiddlewareAPI } from '@reduxjs/toolkit'
 import { createAction } from '@reduxjs/toolkit'
+import type { AnyFunction } from '../tsHelpers'
 
 describe('createActionCreatorInvariantMiddleware', () => {
   const consoleSpy = vi.spyOn(console, 'warn').mockImplementation(() => {})
@@ -50,7 +53,7 @@ describe('createActionCreatorInvariantMiddleware', () => {
   it('allows passing a custom predicate', () => {
     let predicateCalled = false
     const testAction = makeActionTester({
-      isActionCreator(action): action is Function {
+      isActionCreator(action): action is AnyFunction {
         predicateCalled = true
         return false
       },

packages/toolkit/src/tests/createAsyncThunk.test-d.ts

@@ -14,6 +14,7 @@ import {
 } from '@reduxjs/toolkit'
 import type { AxiosError } from 'axios'
 import apiRequest from 'axios'
+import type { AnyFunction, AnyNonNullishValue } from '../tsHelpers'
 
 const defaultDispatch = (() => {}) as ThunkDispatch<{}, any, UnknownAction>
 const unknownAction = { type: 'foo' } as UnknownAction
@@ -97,7 +98,7 @@ describe('type tests', () => {
 
     const correctDispatch = (() => {}) as ThunkDispatch<
       BookModel[],
-      { userAPI: Function },
+      { userAPI: AnyFunction },
       UnknownAction
     >
 
@@ -107,7 +108,7 @@ describe('type tests', () => {
       number,
       {
         state: BooksState
-        extra: { userAPI: Function }
+        extra: { userAPI: AnyFunction }
       }
     >(
       'books/fetch',
@@ -118,7 +119,7 @@ describe('type tests', () => {
 
         expectTypeOf(state).toEqualTypeOf<BookModel[]>()
 
-        expectTypeOf(extra).toEqualTypeOf<{ userAPI: Function }>()
+        expectTypeOf(extra).toEqualTypeOf<{ userAPI: AnyFunction }>()
 
         return fakeBooks
       },

packages/toolkit/src/tests/getDefaultMiddleware.test.ts

@@ -10,6 +10,7 @@ import { thunk } from 'redux-thunk'
 import { vi } from 'vitest'
 
 import { buildGetDefaultMiddleware } from '@internal/getDefaultMiddleware'
+import type { AnyFunction, AnyNonNullishValue } from '../tsHelpers'
 
 const getDefaultMiddleware = buildGetDefaultMiddleware()
 
@@ -181,7 +182,7 @@ it('allows passing options to actionCreatorCheck', () => {
       immutableCheck: false,
       serializableCheck: false,
       actionCreatorCheck: {
-        isActionCreator: (action: unknown): action is Function => {
+        isActionCreator: (action: unknown): action is AnyFunction => {
           actionCreatorCheckWasCalled = true
           return false
         },