Fix redirect guard routing behaviour

Author: IzumiSy

.changeset/fix-redirect-guard-routing.md

@@ -0,0 +1,7 @@
+---
+"@tailor-platform/app-shell": patch
+---
+
+Fixed `redirectTo()` guard not working on modules/resources without a component. Index routes are now created for loader-only routes, enabling redirect guards to execute properly.
+
+Fixed sidebar navigation hiding modules/resources that use `redirectTo()` guards. Navigation filtering now only excludes `hidden()` guards, so redirect-guarded items remain visible in the sidebar.

packages/core/src/routing/navigation.tsx

@@ -64,7 +64,7 @@ const buildNavItems = async (props: BuildNavItemsProps) => {
       if (module.path.startsWith(":")) return null;
 
       const guardResult = await runGuards(module.guards);
-      if (guardResult.type !== "pass") return null;
+      if (guardResult.type === "hidden") return null;
 
       const visibleResources = await filterVisibleResources(
         module.resources,
@@ -100,7 +100,7 @@ const filterVisibleResources = async (
       if (resource.path.startsWith(":")) return null;
 
       const guardResult = await runGuards(resource.guards);
-      if (guardResult.type !== "pass") return null;
+      if (guardResult.type === "hidden") return null;
 
       const resourcePath = `${basePath}/${resource.path}`;
       const resourceTitle = resolveTitle(resource.meta.title, resource.path);

packages/core/src/routing/routes.tsx

@@ -39,7 +39,15 @@ const createRoute = (
         Component: source.component,
         ...(source.loader && { loader: source.loader }),
       }
-    : undefined;
+    : source.loader
+      ? {
+          index: true,
+          loader: source.loader,
+          // Component is required to suppress React Router's warning about empty leaf routes,
+          // even though the loader always redirects/throws and this component will never render.
+          Component: () => null,
+        }
+      : undefined;
 
   return {
     path: source.path,