Moving react to peerDependencies and removing reducer from options.

Author: ericdowell

package-lock.json

@@ -48,11 +48,13 @@
   "engines": {
     "node": "^10.12.0 || >=12.0.0"
   },
+  "peerDependencies": {
+    "react": "^16.14.0 | ^17.0.1",
+    "react-dom": "^16.14.0 | ^17.0.1"
+  },
   "dependencies": {
     "axios": "^0.21.0",
     "qs": "^6.9.4",
-    "react": "^17.0.1",
-    "react-dom": "^17.0.1",
     "resource-endpoint": "^3.3.0",
     "url-join": "^4.0.1"
   },

package.json

@@ -5,8 +5,7 @@ import { applyReducerState } from './helpers'
 
 export function createStateProvider<S, R extends React.Reducer<any, any>>(options: {
   initialState: S
-  actions?: Record<string, string>
-  reducer?: R
+  actions: Record<string, string>
   actionCases?: StateActionCases<S>
   providerHelpers?: (dispatch: React.Dispatch<React.ReducerAction<R>>) => StateProviderHelpers
 }): [
@@ -30,10 +29,10 @@ export function createStateProvider<S, R extends React.Reducer<any, any>>(option
     helpers: any
     state: S
   }
-  if (!options.reducer && !options.actions) {
+  if (!options.actions) {
     throw new Error("The 'reducer' or 'actions' option must be provided, one of them needs to be passed.")
   }
-  const actionTypes = Object.values(options.actions ?? {})
+  const actionTypes = Object.values(options.actions)
   const knownActions = JSON.stringify(actionTypes)
   const Context = React.createContext<ProviderProps>({
     // eslint-disable-next-line @typescript-eslint/no-unused-vars
@@ -42,18 +41,15 @@ export function createStateProvider<S, R extends React.Reducer<any, any>>(option
     state: options.initialState, // dispatch/state will be maintained by Reducer provided in StateProvider going forward
   })
   function StateProvider(props: StateProviderProps): React.ReactElement<React.ProviderProps<ProviderProps>> {
-    const reducer = options.reducer
-      ? options.reducer
-      : (prevState: S, action: StateAction): any => {
-          if (!actionTypes.includes(action.type)) {
-            throw new Error(`Unknown action: "${action.type}", known actions: ${knownActions}`)
-          }
-          if (typeof options?.actionCases?.[action.type] !== 'function') {
-            return applyReducerState(prevState, action)
-          }
-          return options.actionCases[action.type](prevState, action)
-        }
-    const [state, dispatch] = React.useReducer(reducer, options.initialState)
+    const [state, dispatch] = React.useReducer((prevState: S, action: StateAction): any => {
+      if (!actionTypes.includes(action.type)) {
+        throw new Error(`Unknown action: "${action.type}", known actions: ${knownActions}`)
+      }
+      if (typeof options?.actionCases?.[action.type] !== 'function') {
+        return applyReducerState(prevState, action)
+      }
+      return options.actionCases[action.type](prevState, action)
+    }, options.initialState)
     return (
       <Context.Provider
         value={{

src/context/stateProvider.tsx

@@ -54,6 +54,12 @@ module.exports = {
       amd: 'react',
       root: 'react',
     },
+    'react-dom': {
+      commonjs: 'react-dom',
+      commonjs2: 'react-dom',
+      amd: 'react-dom',
+      root: 'react-dom',
+    },
     'url-join': {
       commonjs: 'url-join',
       commonjs2: 'url-join',