@open-mercato/ui 0.6.6-develop.6465.1.019f0fb26f → 0.6.6-develop.6471.1.9ab5bdd79a

package/src/backend/notifications/NotificationDispatcher.ts

@@ -10,6 +10,9 @@ import type {
   NotificationHandlerToastOptions,
 } from '@open-mercato/shared/modules/notifications/handler'
 import type { NotificationDto } from '@open-mercato/shared/modules/notifications/types'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('ui').child({ component: 'notifications' })
 
 type RuntimeContext = {
   userId?: string
@@ -116,7 +119,7 @@ class NotificationDispatcher {
         try {
           void Promise.resolve(handler.handle(notification, context))
         } catch (error) {
-          console.error(`[notifications] handler failed: ${handler.id}`, error)
+          logger.error('Notification handler failed', { handlerId: handler.id, err: error })
         }
       }
 
@@ -126,7 +129,7 @@ class NotificationDispatcher {
         try {
           listener.effect(notification)
         } catch (error) {
-          console.error('[notifications] effect listener failed', error)
+          logger.error('Notification effect listener failed', { err: error })
         }
       }
 

package/src/backend/sidebar/SidebarCustomizationEditor.tsx

@@ -42,6 +42,9 @@ import {
   type SidebarGroup,
   type SidebarItem,
 } from './customization-helpers'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('ui').child({ component: 'SidebarCustomizationEditor' })
 
 export type SidebarCustomizationEditorProps = {
   onSaved?: () => void
@@ -358,7 +361,7 @@ export function SidebarCustomizationEditor({
         selectVariantInternal(initial, list)
         setSelectedRoleIds([])
       } catch (err) {
-        console.error('Failed to load sidebar variants', err)
+        logger.error('Failed to load sidebar variants', { err })
         setError(t('appShell.sidebarCustomizationLoadError'))
       } finally {
         setLoading(false)
@@ -437,7 +440,7 @@ export function SidebarCustomizationEditor({
       flash(t('appShell.sidebarCustomizationVariantCreated', 'Variant created.'), 'success')
       return { ok: true }
     } catch (err) {
-      console.error('Failed to create sidebar variant', err)
+      logger.error('Failed to create sidebar variant', { err })
       const message = t('appShell.sidebarCustomizationSaveError')
       if (!options.suppressPageError) setError(message)
       return { ok: false, error: message }
@@ -763,7 +766,7 @@ export function SidebarCustomizationEditor({
       )
       onSaved?.()
     } catch (err) {
-      console.error('Failed to save sidebar variant', err)
+      logger.error('Failed to save sidebar variant', { err })
       setError(t('appShell.sidebarCustomizationSaveError'))
     } finally {
       setSaving(false)
@@ -794,7 +797,7 @@ export function SidebarCustomizationEditor({
       const fresh = list.find((v) => v.id === selectedVariant.id) ?? selectedVariant
       selectVariantInternal(fresh, list)
     } catch (err) {
-      console.error('Failed to toggle variant active state', err)
+      logger.error('Failed to toggle variant active state', { err })
       setError(t('appShell.sidebarCustomizationSaveError'))
     }
   }, [selectedVariant, saving, deleting, variantsApiPath, t, loadVariantsList, selectVariantInternal, runMutation, buildMutationContext])
@@ -831,7 +834,7 @@ export function SidebarCustomizationEditor({
       const fallback = list[0] ?? null
       selectVariantInternal(fallback, list)
     } catch (err) {
-      console.error('Failed to delete variant', err)
+      logger.error('Failed to delete variant', { err })
       setError(t('appShell.sidebarCustomizationSaveError'))
     } finally {
       setDeleting(false)

package/src/backend/utils/__tests__/bulkDelete.test.ts

@@ -1,13 +1,29 @@
 import { groupBulkDeleteFailures, runBulkDelete, type BulkDeleteFailure } from '../bulkDelete'
 import { emitProgressUpdate } from '@open-mercato/shared/lib/frontend/progressEvents'
+import { createLogger } from '@open-mercato/shared/lib/logger'
 
 jest.mock('@open-mercato/shared/lib/frontend/progressEvents', () => ({
   emitProgressUpdate: jest.fn(),
 }))
 
+jest.mock('@open-mercato/shared/lib/logger', () => {
+  const mocked = {
+    debug: jest.fn(),
+    info: jest.fn(),
+    warn: jest.fn(),
+    error: jest.fn(),
+    child: jest.fn(),
+  }
+  mocked.child.mockImplementation(() => mocked)
+  return { createLogger: jest.fn(() => mocked) }
+})
+
+const loggerWarn = createLogger('ui').warn as jest.Mock
+
 describe('runBulkDelete', () => {
   beforeEach(() => {
     jest.mocked(emitProgressUpdate).mockClear()
+    loggerWarn.mockClear()
   })
 
   it('captures error message and code from thrown error', async () => {
@@ -48,32 +64,22 @@ describe('runBulkDelete', () => {
     expect(failures[0].code).toBeNull()
   })
 
-  it('logs failures via console.warn when logTag is provided', async () => {
-    const warnSpy = jest.spyOn(console, 'warn').mockImplementation(() => undefined)
-    try {
-      const err = Object.assign(new Error('linked deals (2).'), { code: 'COMPANY_HAS_DEPENDENTS' })
-      const deleteOne = jest.fn().mockRejectedValue(err)
-
-      await runBulkDelete([{ id: 'x' }], deleteOne, { logTag: 'customers.companies.list' })
-
-      expect(warnSpy).toHaveBeenCalledTimes(1)
-      const [tag, payload] = warnSpy.mock.calls[0]
-      expect(tag).toBe('[customers.companies.list] bulk delete failed')
-      expect(payload).toMatchObject({ id: 'x', code: 'COMPANY_HAS_DEPENDENTS' })
-    } finally {
-      warnSpy.mockRestore()
-    }
+  it('logs failures when logTag is provided', async () => {
+    const err = Object.assign(new Error('linked deals (2).'), { code: 'COMPANY_HAS_DEPENDENTS' })
+    const deleteOne = jest.fn().mockRejectedValue(err)
+
+    await runBulkDelete([{ id: 'x' }], deleteOne, { logTag: 'customers.companies.list' })
+
+    expect(loggerWarn).toHaveBeenCalledTimes(1)
+    const [message, fields] = loggerWarn.mock.calls[0]
+    expect(message).toBe('Bulk delete failed')
+    expect(fields).toMatchObject({ logTag: 'customers.companies.list', id: 'x', code: 'COMPANY_HAS_DEPENDENTS' })
   })
 
   it('does not log when logTag is omitted', async () => {
-    const warnSpy = jest.spyOn(console, 'warn').mockImplementation(() => undefined)
-    try {
-      const deleteOne = jest.fn().mockRejectedValue(new Error('boom'))
-      await runBulkDelete([{ id: 'x' }], deleteOne)
-      expect(warnSpy).not.toHaveBeenCalled()
-    } finally {
-      warnSpy.mockRestore()
-    }
+    const deleteOne = jest.fn().mockRejectedValue(new Error('boom'))
+    await runBulkDelete([{ id: 'x' }], deleteOne)
+    expect(loggerWarn).not.toHaveBeenCalled()
   })
 
   it('emits client progress events when progress metadata is provided', async () => {

package/src/backend/utils/api.ts

@@ -7,6 +7,9 @@ import { readJsonSafe } from '@open-mercato/shared/lib/http/readJsonSafe'
 import { pushOperation } from '../operations/store'
 import { pushPartialIndexWarning } from '../indexes/store'
 import { createScopedHeaderStack } from './scopedHeaderStack'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('ui').child({ component: 'apiFetch' })
 
 const scopedHeaders = createScopedHeaderStack()
 
@@ -195,8 +198,7 @@ export async function apiFetch(input: RequestInfo | URL, init?: RequestInit): Pr
               ? input.url
               : 'unknown'
       try {
-        // eslint-disable-next-line no-console
-        console.warn('[apiFetch] Forbidden response', {
+        logger.warn('Forbidden response', {
           url: target,
           status: res.status,
           requiredRoles: roles,

package/src/backend/utils/bulkDelete.ts

@@ -1,6 +1,9 @@
 "use client"
 
 import { emitProgressUpdate } from '@open-mercato/shared/lib/frontend/progressEvents'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('ui')
 
 export type BulkDeleteFailure = {
   id: string
@@ -87,8 +90,8 @@ export async function runBulkDelete<T extends { id: string }>(
         err && typeof err === 'object' ? (err as { code?: unknown }).code : undefined
       const code = typeof rawCode === 'string' && rawCode.length > 0 ? rawCode : null
       failures.push({ id: row.id, code, message })
-      if (logTag && typeof console !== 'undefined' && typeof console.warn === 'function') {
-        console.warn(`[${logTag}] bulk delete failed`, { id: row.id, code, message, err })
+      if (logTag) {
+        logger.warn('Bulk delete failed', { logTag, id: row.id, code, message, err })
       }
     }
     if (progress && progressJobId) {

package/src/backend/version-history/VersionHistoryPanel.tsx

@@ -13,6 +13,9 @@ import { getVersionHistoryActionLabel, getVersionHistoryStatusLabel } from './la
 import { useAuditPermissions, canUndoEntry, canRedoEntry } from './useAuditPermissions'
 import { Alert, AlertDescription } from '@open-mercato/ui/primitives/alert'
 import { humanizeResourceKind } from './labels'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('ui')
 
 export type VersionHistoryPanelProps = {
   open: boolean
@@ -107,7 +110,7 @@ export function VersionHistoryPanel({
         window.location.reload()
       }
     } catch (err) {
-      console.error(t('audit_logs.actions.undo'), err)
+      logger.error('Undo failed', { err })
     } finally {
       setUndoingToken(null)
     }
@@ -127,7 +130,7 @@ export function VersionHistoryPanel({
         window.location.reload()
       }
     } catch (err) {
-      console.error(t('audit_logs.actions.redo'), err)
+      logger.error('Redo failed', { err })
     } finally {
       setRedoingId(null)
     }

package/src/portal/hooks/usePortalDashboardWidgets.ts

@@ -5,6 +5,9 @@ import type { InjectionSpotId } from '@open-mercato/shared/modules/widgets/injec
 import { loadInjectionWidgetsForSpot, type LoadedInjectionWidget } from '@open-mercato/shared/modules/widgets/injection-loader'
 import { hasAllFeatures } from '@open-mercato/shared/security/features'
 import { apiCall } from '../../backend/utils/apiCall'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('ui').child({ component: 'usePortalDashboardWidgets' })
 
 type PortalFeatureCheckResponse = {
   ok: boolean
@@ -76,7 +79,7 @@ export function usePortalDashboardWidgets(spotId: InjectionSpotId): {
         setGrantedFeatures(granted)
       } catch (loadError) {
         if (!mounted) return
-        console.error(`[usePortalDashboardWidgets] Failed to load widgets for spot ${spotId}:`, loadError)
+        logger.error('Failed to load widgets for spot', { spotId, err: loadError })
         setError(loadError instanceof Error ? loadError.message : String(loadError))
         setWidgets([])
       } finally {

package/src/portal/hooks/usePortalEventBridge.ts

@@ -2,6 +2,9 @@
 import { useEffect, useRef } from 'react'
 import type { AppEventPayload } from '@open-mercato/shared/modules/widgets/injection'
 import { PORTAL_EVENT_DOM_NAME } from './usePortalAppEvent'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('ui').child({ component: 'PortalEventBridge' })
 
 const PORTAL_SSE_ENDPOINT = '/api/customer_accounts/portal/events/stream'
 const HEARTBEAT_TIMEOUT = 45_000
@@ -58,7 +61,7 @@ export function usePortalEventBridge(): void {
     function resetHeartbeatTimer() {
       if (heartbeatTimer.current) clearTimeout(heartbeatTimer.current)
       heartbeatTimer.current = setTimeout(() => {
-        console.warn('[PortalEventBridge] Heartbeat timeout — reconnecting')
+        logger.warn('Heartbeat timeout — reconnecting')
         disconnect()
         scheduleReconnect()
       }, HEARTBEAT_TIMEOUT)

package/src/primitives/Notice.tsx

@@ -1,6 +1,9 @@
 "use client"
 import * as React from 'react'
 import { cn } from '@open-mercato/shared/lib/utils'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('ui')
 
 /**
  * @deprecated Use <Alert variant="destructive|warning|info"> instead.
@@ -48,10 +51,7 @@ export function Notice({
   compact = false,
 }: NoticeProps) {
   if (process.env.NODE_ENV === 'development') {
-    console.warn(
-      '[DS] <Notice> is deprecated. Use <Alert variant="destructive|warning|info"> instead. ' +
-      'See: docs/design-system/migration-tables.md'
-    )
+    logger.warn('<Notice> is deprecated. Use <Alert variant="destructive|warning|info"> instead. See: docs/design-system/migration-tables.md')
   }
 
   const styles = variantStyles[variant]

package/src/theme/ThemeProvider.tsx

@@ -2,6 +2,9 @@
 
 import * as React from 'react'
 import { createContext, useContext } from 'react'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('ui').child({ component: 'ThemeProvider' })
 
 export type Theme = 'light' | 'dark' | 'system'
 
@@ -31,7 +34,7 @@ function getStoredTheme(): Theme {
     // localStorage may be unavailable in private browsing, iframes, or restricted contexts
     // Theme will default to system preference - this is expected graceful degradation
     if (process.env.NODE_ENV === 'development') {
-      console.warn('[ThemeProvider] localStorage read failed:', error)
+      logger.warn('localStorage read failed', { err: error })
     }
   }
   return 'system'
@@ -85,7 +88,7 @@ export function ThemeProvider({ children }: { children: React.ReactNode }) {
     } catch (error) {
       // localStorage may be unavailable - theme still works for this session, just won't persist
       if (process.env.NODE_ENV === 'development') {
-        console.warn('[ThemeProvider] localStorage write failed:', error)
+        logger.warn('localStorage write failed', { err: error })
       }
     }
     const resolved = newTheme === 'system' ? getSystemTheme() : newTheme