Merge pull request #3904 from reduxjs/redux5-rc1

Author: markerikson

docs/api/matching-utilities.mdx

@@ -13,7 +13,6 @@ Redux Toolkit exports several type-safe action matching utilities that you can l
 
 ### General Purpose
 
-- [`isAction`](#isaction) - returns true if a passed value is a standard action object, with a type string
 - [`isAllOf`](#isallof) - returns true when **all** conditions are met
 - [`isAnyOf`](#isanyof) - returns true when **at least one of** the conditions are met
 
@@ -27,16 +26,6 @@ All these matchers can either be called with one or more thunks as arguments, in
 - [`isRejected`](#isrejected) - accepts one or more action creators and returns true when all match
 - [`isRejectedWithValue`](#isrejectedwithvalue) - accepts one or more action creators and returns true when all match
 
-## `isAction`
-
-A type guard that takes an unknown variable, and returns `true` if it's a standard Redux action with a string `type` property.
-
-:::caution
-
-As of Redux 5.0, action types are _required_ to be a string.
-
-:::
-
 ## `isAllOf`
 
 A higher-order function that accepts one or more of:

packages/toolkit/package.json

@@ -112,7 +112,7 @@
   ],
   "dependencies": {
     "immer": "^10.0.3",
-    "redux": "^5.0.0-rc.0",
+    "redux": "^5.0.0-rc.1",
     "redux-thunk": "^3.0.0-rc.0",
     "reselect": "^5.0.0-beta.1"
   },

packages/toolkit/src/configureStore.ts

@@ -7,11 +7,16 @@ import type {
   Store,
   UnknownAction,
 } from 'redux'
-import { applyMiddleware, createStore, compose, combineReducers } from 'redux'
+import {
+  applyMiddleware,
+  createStore,
+  compose,
+  combineReducers,
+  isPlainObject,
+} from 'redux'
 import type { DevToolsEnhancerOptions as DevToolsOptions } from './devtoolsExtension'
 import { composeWithDevTools } from './devtoolsExtension'
 
-import isPlainObject from './isPlainObject'
 import type {
   ThunkMiddlewareFor,
   GetDefaultMiddleware,

packages/toolkit/src/createAction.ts

@@ -1,12 +1,11 @@
-import type { Action, UnknownAction } from 'redux'
+import { isAction } from 'redux'
 import type {
   IsUnknownOrNonInferrable,
   IfMaybeUndefined,
   IfVoid,
   IsAny,
 } from './tsHelpers'
 import { hasMatchFunction } from './tsHelpers'
-import isPlainObject from './isPlainObject'
 
 /**
  * An action with a string type and an associated payload. This is the
@@ -84,7 +83,7 @@ export type _ActionCreatorWithPreparedPayload<
  */
 export interface BaseActionCreator<P, T extends string, M = never, E = never> {
   type: T
-  match: (action: Action<string>) => action is PayloadAction<P, T, M, E>
+  match: (action: unknown) => action is PayloadAction<P, T, M, E>
 }
 
 /**
@@ -279,23 +278,12 @@ export function createAction(type: string, prepareAction?: Function): any {
 
   actionCreator.type = type
 
-  actionCreator.match = (action: Action<string>): action is PayloadAction =>
-    action.type === type
+  actionCreator.match = (action: unknown): action is PayloadAction =>
+    isAction(action) && action.type === type
 
   return actionCreator
 }
 
-/**
- * Returns true if value is a plain object with a `type` property.
- */
-export function isAction(action: unknown): action is Action<string> {
-  return (
-    isPlainObject(action) &&
-    'type' in action &&
-    typeof (action as Record<'type', unknown>).type === 'string'
-  )
-}
-
 /**
  * Returns true if value is an RTK-like action creator, with a static type property and match method.
  */

packages/toolkit/src/dynamicMiddleware/index.ts

@@ -4,7 +4,7 @@ import type {
   UnknownAction,
 } from 'redux'
 import { compose } from 'redux'
-import { createAction, isAction } from '../createAction'
+import { createAction } from '../createAction'
 import { isAllOf } from '../matchers'
 import { nanoid } from '../nanoid'
 import { emplace, find } from '../utils'
@@ -75,11 +75,7 @@ export const createDynamicMiddleware = <
     return compose(...appliedMiddleware)
   }
 
-  const isWithMiddleware = isAllOf(
-    isAction,
-    withMiddleware,
-    matchInstance(instanceId)
-  )
+  const isWithMiddleware = isAllOf(withMiddleware, matchInstance(instanceId))
 
   const middleware: DynamicMiddleware<State, Dispatch> =
     (api) => (next) => (action) => {

packages/toolkit/src/dynamicMiddleware/tests/index.test.ts

@@ -2,7 +2,6 @@ import type { Middleware } from 'redux'
 import { createDynamicMiddleware } from '../index'
 import { configureStore } from '../../configureStore'
 import type { BaseActionCreator, PayloadAction } from '../../createAction'
-import { isAction } from '../../createAction'
 import { createAction } from '../../createAction'
 import { isAllOf } from '../../matchers'
 
@@ -25,7 +24,7 @@ export const makeProbeableMiddleware = <Id extends number>(
 ): Middleware<{
   (action: PayloadAction<Id, typeof probeType>): Id
 }> => {
-  const isMiddlewareAction = isAllOf(isAction, probeMiddleware, matchId(id))
+  const isMiddlewareAction = isAllOf(probeMiddleware, matchId(id))
   return (api) => (next) => (action) => {
     if (isMiddlewareAction(action)) {
       return id

packages/toolkit/src/index.ts

@@ -42,7 +42,6 @@ export type { DevToolsEnhancerOptions } from './devtoolsExtension'
 export {
   // js
   createAction,
-  isAction,
   isActionCreator,
   isFSA as isFluxStandardAction,
 } from './createAction'
@@ -157,8 +156,6 @@ export type {
 
 export { nanoid } from './nanoid'
 
-export { default as isPlainObject } from './isPlainObject'
-
 export type {
   ListenerEffect,
   ListenerMiddleware,

packages/toolkit/src/isPlainObject.ts

@@ -1,6 +1,7 @@
 import type { Action, Dispatch, MiddlewareAPI, UnknownAction } from 'redux'
+import { isAction } from 'redux'
 import type { ThunkDispatch } from 'redux-thunk'
-import { createAction, isAction } from '../createAction'
+import { createAction } from '../createAction'
 import { nanoid } from '../nanoid'
 
 import type {

packages/toolkit/src/listenerMiddleware/index.ts

@@ -1,7 +1,6 @@
-import isPlainObject from './isPlainObject'
 import type { Middleware } from 'redux'
+import { isAction, isPlainObject } from 'redux'
 import { getTimeMeasureUtils } from './utils'
-import { isAction } from './createAction'
 
 /**
  * Returns true if the passed value is "plain", i.e. a value that is either