From ae1e4c67de3d6cf52fbe71967da59fbdba036482 Mon Sep 17 00:00:00 2001
From: Admin <admin@localhost>
Date: Wed, 29 Apr 2026 14:02:42 +0330
Subject: [PATCH] improvement: sidebar refactored. COMMING SOON page added for
 all non-developed tabs.

---
 .../src/app/(dashboard)/appointments/page.tsx | 10 +++
 frontend/src/app/(dashboard)/lab/page.tsx     | 10 +++
 frontend/src/app/(dashboard)/layout.tsx       | 55 +++++++++----
 frontend/src/app/(dashboard)/reports/page.tsx | 10 +++
 frontend/src/app/(dashboard)/staff/page.tsx   | 10 +++
 frontend/src/components/ui/Sidebar.tsx        | 18 +++--
 frontend/src/lib/hooks/useAuth.tsx            | 81 +++++++++++--------
 7 files changed, 137 insertions(+), 57 deletions(-)
 create mode 100644 frontend/src/app/(dashboard)/appointments/page.tsx
 create mode 100644 frontend/src/app/(dashboard)/lab/page.tsx
 create mode 100644 frontend/src/app/(dashboard)/reports/page.tsx
 create mode 100644 frontend/src/app/(dashboard)/staff/page.tsx

diff --git a/frontend/src/app/(dashboard)/appointments/page.tsx b/frontend/src/app/(dashboard)/appointments/page.tsx
new file mode 100644
index 0000000..d638c9f
--- /dev/null
+++ b/frontend/src/app/(dashboard)/appointments/page.tsx
@@ -0,0 +1,10 @@
+export default function AppointmentsPage() {
+  return (
+    <div className="space-y-3">
+      <h1 className="text-2xl font-semibold text-text-primary">Appointments</h1>
+      <p className="text-sm text-text-secondary">
+        Appointments module is coming soon.
+      </p>
+    </div>
+  );
+}
diff --git a/frontend/src/app/(dashboard)/lab/page.tsx b/frontend/src/app/(dashboard)/lab/page.tsx
new file mode 100644
index 0000000..42c2cab
--- /dev/null
+++ b/frontend/src/app/(dashboard)/lab/page.tsx
@@ -0,0 +1,10 @@
+export default function LabPage() {
+  return (
+    <div className="space-y-3">
+      <h1 className="text-2xl font-semibold text-text-primary">Lab Management</h1>
+      <p className="text-sm text-text-secondary">
+        Lab management module is coming soon.
+      </p>
+    </div>
+  );
+}
diff --git a/frontend/src/app/(dashboard)/layout.tsx b/frontend/src/app/(dashboard)/layout.tsx
index be84b47..62735b2 100644
--- a/frontend/src/app/(dashboard)/layout.tsx
+++ b/frontend/src/app/(dashboard)/layout.tsx
@@ -1,6 +1,6 @@
 'use client';
 
-import { useEffect } from 'react';
+import { memo, useCallback, useEffect } from 'react';
 import { useRouter } from 'next/navigation';
 import { useAuth } from '@/lib/hooks/useAuth';
 import Sidebar from '@/components/ui/Sidebar';
@@ -10,6 +10,9 @@ import { LogOut } from 'lucide-react';
 export default function DashboardLayout({ children }: { children: React.ReactNode }) {
   const { user, currentOrganization, isAuthReady, logout } = useAuth();
   const router = useRouter();
+  const handleLogout = useCallback(() => {
+    void logout();
+  }, [logout]);
 
   // ✅ AUTH GUARD (runs once per navigation group)
   useEffect(() => {
@@ -48,21 +51,11 @@ export default function DashboardLayout({ children }: { children: React.ReactNod
       <Sidebar />
 
       <div className="flex-1 flex flex-col">
-        <header className="flex justify-between px-6 py-4 border-b border-border/70 backdrop-blur-sm">
-          <h2 className="text-lg font-medium">{currentOrganization.name}</h2>
-
-          <div className="flex items-center gap-4">
-            <ThemeToggle />
-            <span className="text-sm text-text-secondary">{user.name}</span>
-            <button
-              onClick={logout}
-              className="inline-flex items-center gap-2 text-sm text-text-secondary hover:text-text-primary transition-colors"
-            >
-              <LogOut className="w-4 h-4 icon-flat" />
-              Logout
-            </button>
-          </div>
-        </header>
+        <DashboardHeader
+          organizationName={currentOrganization.name}
+          userName={user.name}
+          onLogout={handleLogout}
+        />
 
         <main className="p-6 flex-1 overflow-y-auto">
           <div className="surface-panel p-6 min-h-full">
@@ -72,4 +65,32 @@ export default function DashboardLayout({ children }: { children: React.ReactNod
       </div>
     </div>
   );
-}
\ No newline at end of file
+}
+
+const DashboardHeader = memo(function DashboardHeader({
+  organizationName,
+  userName,
+  onLogout,
+}: {
+  organizationName: string;
+  userName: string;
+  onLogout: () => void;
+}) {
+  return (
+    <header className="flex justify-between px-6 py-4 border-b border-border/70 backdrop-blur-sm">
+      <h2 className="text-lg font-medium">{organizationName}</h2>
+
+      <div className="flex items-center gap-4">
+        <ThemeToggle />
+        <span className="text-sm text-text-secondary">{userName}</span>
+        <button
+          onClick={onLogout}
+          className="inline-flex items-center gap-2 text-sm text-text-secondary hover:text-text-primary transition-colors"
+        >
+          <LogOut className="w-4 h-4 icon-flat" />
+          Logout
+        </button>
+      </div>
+    </header>
+  );
+});
\ No newline at end of file
diff --git a/frontend/src/app/(dashboard)/reports/page.tsx b/frontend/src/app/(dashboard)/reports/page.tsx
new file mode 100644
index 0000000..0d53048
--- /dev/null
+++ b/frontend/src/app/(dashboard)/reports/page.tsx
@@ -0,0 +1,10 @@
+export default function ReportsPage() {
+  return (
+    <div className="space-y-3">
+      <h1 className="text-2xl font-semibold text-text-primary">Reports</h1>
+      <p className="text-sm text-text-secondary">
+        Reports module is coming soon.
+      </p>
+    </div>
+  );
+}
diff --git a/frontend/src/app/(dashboard)/staff/page.tsx b/frontend/src/app/(dashboard)/staff/page.tsx
new file mode 100644
index 0000000..e7167d2
--- /dev/null
+++ b/frontend/src/app/(dashboard)/staff/page.tsx
@@ -0,0 +1,10 @@
+export default function StaffPage() {
+  return (
+    <div className="space-y-3">
+      <h1 className="text-2xl font-semibold text-text-primary">Staff Management</h1>
+      <p className="text-sm text-text-secondary">
+        Staff management module is coming soon.
+      </p>
+    </div>
+  );
+}
diff --git a/frontend/src/components/ui/Sidebar.tsx b/frontend/src/components/ui/Sidebar.tsx
index 427539e..f3c59d3 100644
--- a/frontend/src/components/ui/Sidebar.tsx
+++ b/frontend/src/components/ui/Sidebar.tsx
@@ -1,6 +1,8 @@
 'use client';
 
-import { usePathname, useRouter } from 'next/navigation';
+import Link from 'next/link';
+import { memo } from 'react';
+import { usePathname } from 'next/navigation';
 import {
   LayoutDashboard,
   Users,
@@ -21,9 +23,8 @@ const menu = [
   { name: 'Reports', path: '/reports', icon: FileText },
 ];
 
-export default function Sidebar() {
+function Sidebar() {
   const pathname = usePathname();
-  const router = useRouter();
 
   return (
     <aside className="w-64 bg-background-secondary/90 border-r border-border text-text-primary flex flex-col p-4">
@@ -37,9 +38,10 @@ export default function Sidebar() {
           const isActive = pathname === item.path;
 
           return (
-            <button
+            <Link
               key={item.name}
-              onClick={() => router.push(item.path)}
+              href={item.path}
+              prefetch
               className={`flex items-center gap-3 px-3 py-2.5 rounded-[var(--radius-sm)] border transition-colors ${
                 isActive
                   ? 'bg-primary-soft border-primary/60 text-text-primary'
@@ -48,10 +50,12 @@ export default function Sidebar() {
             >
               <Icon className="w-[18px] h-[18px] icon-flat" />
               <span className="text-sm">{item.name}</span>
-            </button>
+            </Link>
           );
         })}
       </nav>
     </aside>
   );
-}
\ No newline at end of file
+}
+
+export default memo(Sidebar);
\ No newline at end of file
diff --git a/frontend/src/lib/hooks/useAuth.tsx b/frontend/src/lib/hooks/useAuth.tsx
index 1f80eda..a97d5b0 100644
--- a/frontend/src/lib/hooks/useAuth.tsx
+++ b/frontend/src/lib/hooks/useAuth.tsx
@@ -1,6 +1,6 @@
 'use client';
 
-import React, { createContext, useContext, useEffect, useState } from 'react';
+import React, { createContext, useCallback, useContext, useEffect, useMemo, useState } from 'react';
 import { useRouter } from 'next/navigation';
 import { authApi } from '@/lib/api/auth';
 import { User, Organization } from '@/types';
@@ -37,11 +37,7 @@ export function AuthProvider({ children }: { children: React.ReactNode }) {
 
   const router = useRouter();
 
-  useEffect(() => {
-    checkAuth();
-  }, []);
-
-  const normalizeProfilePayload = (payload: any): { user: User | null; organizations: Organization[] } => {
+  const normalizeProfilePayload = useCallback((payload: any): { user: User | null; organizations: Organization[] } => {
     const organizations = payload?.organizations || [];
 
     if (payload?.user) {
@@ -60,9 +56,9 @@ export function AuthProvider({ children }: { children: React.ReactNode }) {
     }
 
     return { user: null, organizations };
-  };
+  }, []);
 
-  const checkAuth = async () => {
+  const checkAuth = useCallback(async () => {
     try {
       setIsLoading(true);
 
@@ -103,10 +99,14 @@ export function AuthProvider({ children }: { children: React.ReactNode }) {
       setIsLoading(false);
       setIsAuthReady(true);
     }
-  };
+  }, [normalizeProfilePayload]);
+
+  useEffect(() => {
+    void checkAuth();
+  }, [checkAuth]);
 
   // ✅ REGISTER
-  const registerTrial = async (
+  const registerTrial = useCallback(async (
     email: string,
     password: string,
     name: string,
@@ -147,10 +147,10 @@ export function AuthProvider({ children }: { children: React.ReactNode }) {
     } finally {
       setIsLoading(false);
     }
-  };
+  }, [router]);
 
   // ✅ LOGIN
-  const login = async (email: string, password: string) => {
+  const login = useCallback(async (email: string, password: string) => {
     try {
       setIsLoading(true);
       setError(null);
@@ -178,17 +178,17 @@ export function AuthProvider({ children }: { children: React.ReactNode }) {
     } finally {
       setIsLoading(false);
     }
-  };
+  }, [router]);
 
-  const logout = async () => {
+  const logout = useCallback(async () => {
     localStorage.clear();
     setUser(null);
     setOrganizations([]);
     setCurrentOrganization(null);
     router.push('/');
-  };
+  }, [router]);
 
-  const selectOrganization = async (orgId: string) => {
+  const selectOrganization = useCallback(async (orgId: string) => {
     try {
       setIsLoading(true);
 
@@ -208,26 +208,41 @@ export function AuthProvider({ children }: { children: React.ReactNode }) {
     } finally {
       setIsLoading(false);
     }
-  };
+  }, [router]);
 
-  const clearError = () => setError(null);
+  const clearError = useCallback(() => setError(null), []);
+
+  const contextValue = useMemo(
+    () => ({
+      user,
+      organizations,
+      currentOrganization,
+      isLoading,
+      isAuthReady,
+      error,
+      registerTrial,
+      login,
+      logout,
+      selectOrganization,
+      clearError,
+    }),
+    [
+      user,
+      organizations,
+      currentOrganization,
+      isLoading,
+      isAuthReady,
+      error,
+      registerTrial,
+      login,
+      logout,
+      selectOrganization,
+      clearError,
+    ],
+  );
 
   return (
-    <AuthContext.Provider
-      value={{
-        user,
-        organizations,
-        currentOrganization,
-        isLoading,
-        isAuthReady, // ✅ expose it
-        error,
-        registerTrial,
-        login,
-        logout,
-        selectOrganization,
-        clearError,
-      }}
-    >
+    <AuthContext.Provider value={contextValue}>
       {children}
     </AuthContext.Provider>
   );