Hide admin navigation from non-admins

src/client/main.tsx

@@ -33,6 +33,7 @@ import {
   speechChunks,
   type ReaderSpeechPreferences
 } from './readerSpeech'
+import { shouldShowAdminNavigation } from './navigation'
 
 type WalletSubstrate = 'auto' | 'json-api' | 'secure-json-api' | 'react-native' | 'Cicada' | 'XDM' | 'window.CWI'
 
@@ -1197,7 +1198,7 @@ function Shell ({ children, status }: { children: React.ReactNode, status: Statu
           <Link to='/about'><Info size={18} /> About</Link>
           <Link to='/settings'><SlidersHorizontal size={18} /> Reader settings</Link>
           <Link to='/author'><User size={18} /> Author</Link>
-          <Link to='/admin'><Settings size={18} /> Admin</Link>
+          {shouldShowAdminNavigation(status?.isAdmin) && <Link to='/admin'><Settings size={18} /> Admin</Link>}
         </nav>
         <button
           className='nav-feedback'

src/client/navigation.test.ts

@@ -0,0 +1,17 @@
+import { describe, expect, it } from 'vitest'
+import { shouldShowAdminNavigation } from './navigation'
+
+describe('shouldShowAdminNavigation', () => {
+  it('hides admin navigation while status is unknown', () => {
+    expect(shouldShowAdminNavigation(undefined)).toBe(false)
+    expect(shouldShowAdminNavigation(null)).toBe(false)
+  })
+
+  it('hides admin navigation for non-admins', () => {
+    expect(shouldShowAdminNavigation(false)).toBe(false)
+  })
+
+  it('shows admin navigation for admins', () => {
+    expect(shouldShowAdminNavigation(true)).toBe(true)
+  })
+})

src/client/navigation.ts

@@ -0,0 +1,3 @@
+export function shouldShowAdminNavigation (isAdmin: boolean | null | undefined): boolean {
+  return isAdmin === true
+}