fix(admin): address review feedback on admin tab

- Fix superUserModeEnabled default to false (matches sidebar behavior)
- Reset banReason when switching ban target to prevent state bleed
- Guard admin section render with session role check for direct URL access

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/settings/[section]/settings.tsx

@@ -3,6 +3,7 @@
 import dynamic from 'next/dynamic'
 import { useSearchParams } from 'next/navigation'
 import { Skeleton } from '@/components/emcn'
+import { useSession } from '@/lib/auth/auth-client'
 import { AdminSkeleton } from '@/app/workspace/[workspaceId]/settings/components/admin/admin-skeleton'
 import { ApiKeysSkeleton } from '@/app/workspace/[workspaceId]/settings/components/api-keys/api-key-skeleton'
 import { BYOKSkeleton } from '@/app/workspace/[workspaceId]/settings/components/byok/byok-skeleton'
@@ -157,9 +158,15 @@ interface SettingsPageProps {
 export function SettingsPage({ section }: SettingsPageProps) {
   const searchParams = useSearchParams()
   const mcpServerId = searchParams.get('mcpServerId')
+  const { data: session } = useSession()
 
+  const isAdminRole = session?.user?.role === 'admin'
   const effectiveSection =
-    !isBillingEnabled && (section === 'subscription' || section === 'team') ? 'general' : section
+    !isBillingEnabled && (section === 'subscription' || section === 'team')
+      ? 'general'
+      : section === 'admin' && !isAdminRole
+        ? 'general'
+        : section
 
   const label =
     allNavigationItems.find((item) => item.id === effectiveSection)?.label ?? effectiveSection

apps/sim/app/workspace/[workspaceId]/settings/components/admin/admin.tsx

@@ -83,7 +83,7 @@ export function Admin() {
         <Label htmlFor='super-user-mode'>Super admin mode</Label>
         <Switch
           id='super-user-mode'
-          checked={settings?.superUserModeEnabled ?? true}
+          checked={settings?.superUserModeEnabled ?? false}
           onCheckedChange={handleSuperUserModeToggle}
         />
       </div>
@@ -255,7 +255,10 @@ export function Admin() {
                           <Button
                             variant='active'
                             className='h-[28px] px-[8px] text-[12px] text-[var(--text-error)]'
-                            onClick={() => setBanUserId(u.id)}
+                            onClick={() => {
+                              setBanUserId(u.id)
+                              setBanReason('')
+                            }}
                             disabled={actionPending && pendingUserId === u.id}
                           >
                             Ban