@sanity/sdk 2.13.0 → 2.14.0

package/src/auth/subscribeToStateAndFetchCurrentUser.test.ts

@@ -1,6 +1,6 @@
 import {type CurrentUser} from '@sanity/types'
 import {delay, of, throwError} from 'rxjs'
-import {beforeEach, describe, it} from 'vitest'
+import {beforeEach, describe, expect, it, vi} from 'vitest'
 
 import {createSanityInstance} from '../store/createSanityInstance'
 import {createStoreState} from '../store/createStoreState'
@@ -8,13 +8,28 @@ import {AuthStateType} from './authStateType'
 import {authStore} from './authStore'
 import {subscribeToStateAndFetchCurrentUser} from './subscribeToStateAndFetchCurrentUser'
 
+// Mock logger to prevent actual logging during tests
+vi.mock('../utils/logger', async (importOriginal) => {
+  const original = await importOriginal<typeof import('../utils/logger')>()
+  return {
+    ...original,
+    createLogger: vi.fn(() => ({
+      info: vi.fn(),
+      debug: vi.fn(),
+      warn: vi.fn(),
+      error: vi.fn(),
+      trace: vi.fn(),
+    })),
+  }
+})
+
 describe('subscribeToStateAndFetchCurrentUser', () => {
   beforeEach(() => {
     vi.clearAllMocks()
   })
 
   it('fetches the current user if the is logged in without a current user present', () => {
-    const mockUser = {id: 'example-user'} as CurrentUser
+    const mockUser = {id: 'example-user', email: 'user@example.com'} as CurrentUser
     const mockRequest = vi.fn().mockReturnValue(of(mockUser))
     const mockClient = {observable: {request: mockRequest}}
     const clientFactory = vi.fn().mockReturnValue(mockClient)

package/src/auth/subscribeToStateAndFetchCurrentUser.ts

@@ -11,6 +11,7 @@ import {
 
 import {type StoreContext} from '../store/defineStore'
 import {DEFAULT_API_VERSION, REQUEST_TAG_PREFIX} from './authConstants'
+import {getAuthLogger} from './authLogger'
 import {isStudioConfig} from './authMode'
 import {AuthStateType} from './authStateType'
 import {type AuthMethodOptions, type AuthState, type AuthStoreState} from './authStore'
@@ -30,6 +31,8 @@ export const subscribeToStateAndFetchCurrentUser = (
   {state, instance}: StoreContext<AuthStoreState>,
   fetchOptions?: {useProjectHostname?: boolean},
 ): Subscription => {
+  const logger = getAuthLogger(instance)
+
   const {clientFactory, apiHost} = state.get().options
   const useProjectHostname = fetchOptions?.useProjectHostname ?? isStudioConfig(instance.config)
   const projectId = instance.config.projectId
@@ -82,6 +85,7 @@ export const subscribeToStateAndFetchCurrentUser = (
              * @see SDK-1409
              */
             catchError((error) => {
+              logger.error('Failed to fetch current user', {error})
               state.set('setError', {authState: {type: AuthStateType.ERROR, error}})
               return EMPTY
             }),
@@ -91,6 +95,11 @@ export const subscribeToStateAndFetchCurrentUser = (
 
   return currentUser$.subscribe({
     next: (currentUser) => {
+      logger.info('Current user fetched successfully', {
+        hasEmail: !!currentUser.email,
+      })
+      // userId is PII, so only surface it at debug level
+      logger.debug('Current user details', {userId: currentUser.id})
       state.set('setCurrentUser', (prev) => ({
         authState:
           prev.authState.type === AuthStateType.LOGGED_IN