Remove unused function

Author: gonzaloriestra

packages/cli-kit/src/private/node/session/store.test.ts

@@ -1,5 +1,5 @@
 import {Sessions} from './schema.js'
-import {store, fetch, remove, getSessionAlias, updateSessionAlias, findSessionByAlias} from './store.js'
+import {store, fetch, remove, getSessionAlias, findSessionByAlias} from './store.js'
 import {getSessions, removeSessions, setSessions, removeCurrentSessionId} from '../conf-store.js'
 import {identityFqdn} from '../../../public/node/context/fqdn.js'
 import {describe, expect, vi, test, beforeEach} from 'vitest'
@@ -131,55 +131,6 @@ describe('session store', () => {
     })
   })
 
-  describe('updateSessionAlias', () => {
-    test('updates alias for existing user', async () => {
-      // Given
-      vi.mocked(getSessions).mockReturnValue(JSON.stringify(mockSessions))
-      const expectedSessions = structuredClone(mockSessions)
-      expectedSessions['identity.fqdn.com']!.user1!.identity.alias = 'New Alias'
-
-      // When
-      await updateSessionAlias('user1', 'New Alias')
-
-      // Then
-      expect(setSessions).toHaveBeenCalledWith(JSON.stringify(expectedSessions))
-    })
-
-    test('does nothing for non-existent user', async () => {
-      // Given
-      vi.mocked(getSessions).mockReturnValue(JSON.stringify(mockSessions))
-
-      // When
-      await updateSessionAlias('nonexistent', 'New Alias')
-
-      // Then
-      expect(setSessions).not.toHaveBeenCalled()
-    })
-
-    test('does nothing when no sessions exist', async () => {
-      // Given
-      vi.mocked(getSessions).mockReturnValue(undefined)
-
-      // When
-      await updateSessionAlias('user1', 'New Alias')
-
-      // Then
-      expect(setSessions).not.toHaveBeenCalled()
-    })
-
-    test('does nothing when fqdn does not exist', async () => {
-      // Given
-      vi.mocked(getSessions).mockReturnValue(JSON.stringify(mockSessions))
-      vi.mocked(identityFqdn).mockResolvedValue('different.fqdn.com')
-
-      // When
-      await updateSessionAlias('user1', 'New Alias')
-
-      // Then
-      expect(setSessions).not.toHaveBeenCalled()
-    })
-  })
-
   describe('findSessionByAlias', () => {
     test('returns userId for existing alias', async () => {
       // Given

packages/cli-kit/src/private/node/session/store.ts

@@ -56,23 +56,6 @@ export async function getSessionAlias(userId: string): Promise<string | undefine
   return sessions[fqdn][userId].identity.alias
 }
 
-/**
- * Updates the session alias with a more human-readable display name if available.
- *
- * @param userId - The user ID of the session to update
- * @param alias - The alias to update the session with
- */
-export async function updateSessionAlias(userId: string, alias: string): Promise<void> {
-  const sessions = await fetch()
-  if (!sessions) return
-
-  const fqdn = await identityFqdn()
-  if (!sessions[fqdn] || !sessions[fqdn][userId]) return
-
-  sessions[fqdn][userId].identity.alias = alias
-  await store(sessions)
-}
-
 /**
  * Finds a session by its alias.
  *

packages/cli-kit/src/public/node/session-prompt.test.ts

@@ -175,7 +175,6 @@ describe('promptSessionSelect', () => {
 
     // Then
     expect(setCurrentSessionId).toHaveBeenCalledWith('user1')
-    expect(sessionStore.updateSessionAlias).not.toHaveBeenCalled()
     expect(result).toEqual('Work Account')
   })