feat(rbac): refactor and multiple accessRoutes feature for RBAC added

BREAKING CHANGE: you must define an accessRoute for each role defined

Author: imjulianeral

src/components/NextShield.tsx

@@ -1,7 +1,7 @@
 import React, { ReactNode, useEffect } from 'react'
 
 import { NextShieldProps } from '../types/props'
-import { verifyPath } from '../libs/routes'
+import { getAccessRoute, verifyPath } from '../libs/routes'
 
 /**
  * 😉 The shield that every Next.js project needs
@@ -66,23 +66,11 @@ export function NextShield<
 
   useEffect(() => {
     if (!isAuth && !isLoading && pathIsPrivate) replace(loginRoute)
-    if (isAuth && !isLoading && pathIsPublic) {
-      if (typeof accessRoute !== 'undefined') {
-        replace(accessRoute)
-      }
-      if (RBAC && userRole) {
-        replace(RBAC[userRole].accessRoute)
-      }
-    }
+    if (isAuth && !isLoading && pathIsPublic)
+      getAccessRoute(RBAC, userRole, accessRoute, loginRoute)
 
-    if (isAuth && userRole && !isLoading && !pathIsHybrid && !pathIsAuthorized) {
-      if (typeof accessRoute !== 'undefined') {
-        replace(accessRoute)
-      }
-      if (RBAC && userRole) {
-        replace(RBAC[userRole].accessRoute)
-      }
-    }
+    if (isAuth && userRole && !isLoading && !pathIsHybrid && !pathIsAuthorized)
+      getAccessRoute(RBAC, userRole, accessRoute, loginRoute)
   }, [
     replace,
     userRole,

src/libs/routes.ts

@@ -1,3 +1,18 @@
+import { RoleAccess } from '../types/props'
+
 export function verifyPath(routes: string[] | undefined, uri: string) {
   return routes?.some(route => route === uri)
 }
+
+export function getAccessRoute(
+  RBAC: RoleAccess<string[]> | undefined,
+  userRole: string | undefined,
+  accessRoute: string | undefined,
+  loginRoute: string
+) {
+  if (typeof accessRoute !== 'undefined') return accessRoute
+
+  if (RBAC && userRole) return RBAC[userRole].accessRoute
+
+  return loginRoute
+}

src/types/props.ts

@@ -1,7 +1,7 @@
 import type { ReactNode } from "react";
 import type { NextRouter } from 'next/router'
 
-type RoleAccess<Routes extends string[]> = {
+export type RoleAccess<Routes extends string[]> = {
   [index: string]: {
     grantedRoutes: Routes,
     accessRoute: Routes[number]