#240 address review feedback cycle 4

Restore name gate in auth layout so every authenticated-nameless user is
redirected to /login regardless of entry point. Add redirectTo prop to
NameField so post-name navigation honors the original destination.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: b-at-neu

app/(main)/(auth)/layout.tsx

@@ -13,6 +13,9 @@ export default async function AuthGateLayout({
 }) {
   const user = await getCurrentUser();
 
+  // Name gate — nameless users must set their name before accessing any app route.
+  if (!user.name?.trim()) redirect('/login');
+
   if (user.isAdmin) return <>{children}</>;
 
   if (await isManager(user.id)) return <>{children}</>;

app/login/page.tsx

@@ -40,7 +40,7 @@ export default async function SignInPage({
   return (
     <div className="flex w-full flex-col items-center gap-4">
       {user ? (
-        <NameField defaultName={user.name ?? ''} />
+        <NameField defaultName={user.name ?? ''} redirectTo={safeTo} />
       ) : (
         <AuthView
           path="SIGN_IN"

components/features/name-field.tsx

@@ -41,9 +41,10 @@ type NameFormValues = z.infer<typeof nameSchema>;
 
 interface NameFieldProps {
   defaultName: string;
+  redirectTo: string;
 }
 
-export function NameField({ defaultName }: NameFieldProps) {
+export function NameField({ defaultName, redirectTo }: NameFieldProps) {
   const router = useRouter();
   const [isPending, startTransition] = useTransition();
 
@@ -69,7 +70,7 @@ export function NameField({ defaultName }: NameFieldProps) {
         );
       else toast.success('Name saved');
 
-      router.replace('/');
+      router.replace(redirectTo);
     });
   }