@open-mercato/core 0.6.6-develop.6460.1.f94c74174c → 0.6.6-develop.6471.1.9ab5bdd79a

package/src/modules/dashboards/widgets/dashboard/orders-kpi/widget.client.tsx

@@ -13,6 +13,9 @@ import {
 } from '@open-mercato/ui/backend/date-range'
 import { DEFAULT_SETTINGS, hydrateSettings, type OrdersKpiSettings } from './config'
 import type { WidgetDataResponse } from '../../../services/widgetDataService'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dashboards').child({ component: 'orders-kpi' })
 
 async function fetchOrdersData(settings: OrdersKpiSettings, fetchWidgetData: WidgetDataFetcher): Promise<WidgetDataResponse> {
   const body = {
@@ -62,7 +65,7 @@ const OrdersKpiWidget: React.FC<DashboardWidgetComponentProps<OrdersKpiSettings>
         setTrend(undefined)
       }
     } catch (err) {
-      console.error('Failed to load orders KPI data', err)
+      logger.error('Failed to load orders KPI data', { err })
       setError(t('dashboards.analytics.widgets.ordersKpi.error', 'Failed to load data'))
     } finally {
       setLoading(false)

package/src/modules/dashboards/widgets/dashboard/pipeline-summary/widget.client.tsx

@@ -13,6 +13,9 @@ import {
 import { DEFAULT_SETTINGS, hydrateSettings, type PipelineSummarySettings } from './config'
 import type { WidgetDataResponse } from '../../../services/widgetDataService'
 import { formatCurrencyCompact } from '../../../lib/formatters'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dashboards').child({ component: 'pipeline-summary' })
 
 async function fetchPipelineData(settings: PipelineSummarySettings, fetchWidgetData: WidgetDataFetcher): Promise<WidgetDataResponse> {
   const body = {
@@ -73,7 +76,7 @@ const PipelineSummaryWidget: React.FC<DashboardWidgetComponentProps<PipelineSumm
         }))
       setData(chartData)
     } catch (err) {
-      console.error('Failed to load pipeline data', err)
+      logger.error('Failed to load pipeline data', { err })
       setError(t('dashboards.analytics.widgets.pipelineSummary.error', 'Failed to load data'))
     } finally {
       setLoading(false)

package/src/modules/dashboards/widgets/dashboard/revenue-kpi/widget.client.tsx

@@ -14,6 +14,9 @@ import {
 import { DEFAULT_SETTINGS, hydrateSettings, type RevenueKpiSettings } from './config'
 import type { WidgetDataResponse } from '../../../services/widgetDataService'
 import { formatCurrency } from '../../../lib/formatters'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dashboards').child({ component: 'revenue-kpi' })
 
 async function fetchRevenueData(settings: RevenueKpiSettings, fetchWidgetData: WidgetDataFetcher): Promise<WidgetDataResponse> {
   const body = {
@@ -63,7 +66,7 @@ const RevenueKpiWidget: React.FC<DashboardWidgetComponentProps<RevenueKpiSetting
         setTrend(undefined)
       }
     } catch (err) {
-      console.error('Failed to load revenue KPI data', err)
+      logger.error('Failed to load revenue KPI data', { err })
       setError(t('dashboards.analytics.widgets.revenueKpi.error', 'Failed to load data'))
     } finally {
       setLoading(false)

package/src/modules/dashboards/widgets/dashboard/revenue-trend/widget.client.tsx

@@ -21,6 +21,9 @@ import type { DateGranularity } from '@open-mercato/shared/modules/analytics'
 import { DEFAULT_SETTINGS, hydrateSettings, type RevenueTrendSettings } from './config'
 import type { WidgetDataResponse } from '../../../services/widgetDataService'
 import { formatCurrencyCompact } from '../../../lib/formatters'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dashboards').child({ component: 'revenue-trend' })
 
 async function fetchRevenueTrendData(settings: RevenueTrendSettings, fetchWidgetData: WidgetDataFetcher): Promise<WidgetDataResponse> {
   const body = {
@@ -132,7 +135,7 @@ const RevenueTrendWidget: React.FC<DashboardWidgetComponentProps<RevenueTrendSet
       }))
       setData(chartData)
     } catch (err) {
-      console.error('Failed to load revenue trend data', err)
+      logger.error('Failed to load revenue trend data', { err })
       setError(t('dashboards.analytics.widgets.revenueTrend.error', 'Failed to load data'))
     } finally {
       setLoading(false)

package/src/modules/dashboards/widgets/dashboard/sales-by-region/widget.client.tsx

@@ -10,6 +10,9 @@ import { Input } from '@open-mercato/ui/primitives/input'
 import { DEFAULT_SETTINGS, hydrateSettings, type SalesByRegionSettings } from './config'
 import type { WidgetDataResponse } from '../../../services/widgetDataService'
 import { formatCurrencyCompact } from '../../../lib/formatters'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dashboards').child({ component: 'sales-by-region' })
 
 async function fetchSalesByRegionData(settings: SalesByRegionSettings, fetchWidgetData: WidgetDataFetcher): Promise<WidgetDataResponse> {
   const body = {
@@ -57,7 +60,7 @@ const SalesByRegionWidget: React.FC<DashboardWidgetComponentProps<SalesByRegionS
       }))
       setData(chartData)
     } catch (err) {
-      console.error('Failed to load sales by region data', err)
+      logger.error('Failed to load sales by region data', { err })
       setError(t('dashboards.analytics.widgets.salesByRegion.error', 'Failed to load data'))
     } finally {
       setLoading(false)

package/src/modules/dashboards/widgets/dashboard/top-customers/widget.client.tsx

@@ -10,6 +10,9 @@ import { Input } from '@open-mercato/ui/primitives/input'
 import { DEFAULT_SETTINGS, hydrateSettings, type TopCustomersSettings } from './config'
 import type { WidgetDataResponse } from '../../../services/widgetDataService'
 import { formatCurrencySafe } from '../../../lib/formatters'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dashboards').child({ component: 'top-customers' })
 
 type CustomerRow = {
   rank: number
@@ -93,7 +96,7 @@ const TopCustomersWidget: React.FC<DashboardWidgetComponentProps<TopCustomersSet
       }))
       setData(tableData)
     } catch (err) {
-      console.error('Failed to load top customers data', err)
+      logger.error('Failed to load top customers data', { err })
       setError(t('dashboards.analytics.widgets.topCustomers.error', 'Failed to load data'))
     } finally {
       setLoading(false)

package/src/modules/dashboards/widgets/dashboard/top-products/widget.client.tsx

@@ -17,6 +17,9 @@ import {
 import { DEFAULT_SETTINGS, hydrateSettings, type TopProductsSettings } from './config'
 import type { WidgetDataResponse } from '../../../services/widgetDataService'
 import { formatCurrencyCompact } from '../../../lib/formatters'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dashboards').child({ component: 'top-products' })
 
 async function fetchTopProductsData(settings: TopProductsSettings, fetchWidgetData: WidgetDataFetcher): Promise<WidgetDataResponse> {
   const body = {
@@ -86,7 +89,7 @@ const TopProductsWidget: React.FC<DashboardWidgetComponentProps<TopProductsSetti
       }))
       setData(chartData)
     } catch (err) {
-      console.error('Failed to load top products data', err)
+      logger.error('Failed to load top products data', { err })
       setError(t('dashboards.analytics.widgets.topProducts.error', 'Failed to load data'))
     } finally {
       setLoading(false)

package/src/modules/data_sync/api/run.ts

@@ -13,6 +13,9 @@ import {
   runCrudMutationGuardAfterSuccess,
   validateCrudMutationGuard,
 } from '@open-mercato/shared/lib/crud/mutation-guard'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('data_sync').child({ component: 'run' })
 
 export const metadata = {
   POST: { requireAuth: true, requireFeatures: ['data_sync.run'] },
@@ -138,7 +141,7 @@ export async function POST(req: Request) {
   } catch (error) {
     const message = error instanceof Error ? error.message : String(error)
     const stack = error instanceof Error ? error.stack : undefined
-    console.error('[data_sync.run] unhandled error', { message, stack })
+    logger.error('Unhandled error starting sync run', { message, stack })
     return NextResponse.json(
       { error: 'Failed to start data sync run.' },
       { status: 500 },

package/src/modules/data_sync/backend/data-sync/runs/[id]/page.tsx

@@ -431,7 +431,9 @@ export default function SyncRunDetailPage({ params }: SyncRunDetailPageProps) {
                   message: log.message,
                   body: log.payload ? (
                     <pre className="overflow-x-auto whitespace-pre-wrap rounded-md border bg-card p-3 text-xs">
-                      {JSON.stringify(log.payload, null, 2)}
+                      {log.payload.kind === 'export-item-failure' && typeof log.payload.summary === 'string'
+                        ? log.payload.summary
+                        : JSON.stringify(log.payload, null, 2)}
                     </pre>
                   ) : (
                     <p className="text-sm text-muted-foreground">

package/src/modules/data_sync/lib/adapter-registry.ts

@@ -1,15 +1,27 @@
 import type { DataSyncAdapter } from './adapter'
 
-const adapters = new Map<string, DataSyncAdapter>()
+const DATA_SYNC_ADAPTER_REGISTRY_KEY = Symbol.for('@open-mercato/data-sync/adapter-registry')
+
+type GlobalWithDataSyncRegistry = typeof globalThis & {
+  [DATA_SYNC_ADAPTER_REGISTRY_KEY]?: Map<string, DataSyncAdapter>
+}
+
+function getAdapterRegistry(): Map<string, DataSyncAdapter> {
+  const globalScope = globalThis as GlobalWithDataSyncRegistry
+  if (!globalScope[DATA_SYNC_ADAPTER_REGISTRY_KEY]) {
+    globalScope[DATA_SYNC_ADAPTER_REGISTRY_KEY] = new Map<string, DataSyncAdapter>()
+  }
+  return globalScope[DATA_SYNC_ADAPTER_REGISTRY_KEY]
+}
 
 export function registerDataSyncAdapter(adapter: DataSyncAdapter): void {
-  adapters.set(adapter.providerKey, adapter)
+  getAdapterRegistry().set(adapter.providerKey, adapter)
 }
 
 export function getDataSyncAdapter(providerKey: string): DataSyncAdapter | undefined {
-  return adapters.get(providerKey)
+  return getAdapterRegistry().get(providerKey)
 }
 
 export function getAllDataSyncAdapters(): DataSyncAdapter[] {
-  return Array.from(adapters.values())
+  return Array.from(getAdapterRegistry().values())
 }

package/src/modules/data_sync/lib/sync-engine.ts

@@ -9,6 +9,9 @@ import { emitDataSyncEvent } from '../events'
 import type { DataSyncAdapter, DataMapping, ExportBatch, ImportBatch } from './adapter'
 import { getDataSyncAdapter } from './adapter-registry'
 import type { SyncRunService } from './sync-run-service'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('data_sync').child({ component: 'sync-engine' })
 
 type SyncScope = {
   organizationId: string
@@ -149,6 +152,31 @@ export function createSyncEngine(deps: EngineDeps) {
     }
   }
 
+  async function logExportItemFailures(
+    runId: string,
+    integrationId: string,
+    results: ExportBatch['results'],
+    scope: SyncScope,
+  ): Promise<void> {
+    const failedResults = results.filter((result) => result.status === 'error' && result.error)
+    for (const result of failedResults) {
+      const label = result.externalId ? `${result.externalId} (id: ${result.localId})` : result.localId
+      const errorMessage = result.error!.split('\n')[0]
+      const message = `Failed to export item ${label}: ${errorMessage}`
+
+      await integrationLogService.write(
+        {
+          integrationId,
+          runId,
+          level: 'error',
+          message,
+          payload: { kind: 'export-item-failure', summary: result.error },
+        },
+        scope,
+      )
+    }
+  }
+
   async function writeOperationalLog(params: {
     integrationId: string
     runId: string
@@ -354,7 +382,7 @@ export function createSyncEngine(deps: EngineDeps) {
     async runImport(runId: string, batchSize: number, scope: SyncScope): Promise<void> {
       const run = await syncRunService.getRun(runId, scope)
       if (!run) {
-        console.warn(`[data-sync] Skipping stale import job for missing run ${runId}`)
+        logger.warn('Skipping stale import job for missing run', { runId })
         return
       }
       if (run.status === 'cancelled') {
@@ -499,7 +527,7 @@ export function createSyncEngine(deps: EngineDeps) {
     async runExport(runId: string, batchSize: number, scope: SyncScope): Promise<void> {
       const run = await syncRunService.getRun(runId, scope)
       if (!run) {
-        console.warn(`[data-sync] Skipping stale export job for missing run ${runId}`)
+        logger.warn('Skipping stale export job for missing run', { runId })
         return
       }
       if (run.status === 'cancelled') {
@@ -600,6 +628,7 @@ export function createSyncEngine(deps: EngineDeps) {
             scope,
           )
           await updateProgress(run.progressJobId, processedCount, null, scope)
+          await logExportItemFailures(run.id, run.integrationId, batch.results, scope)
 
           await writeOperationalLog({
             integrationId: run.integrationId,

package/src/modules/data_sync/workers/sync-export.ts

@@ -2,6 +2,9 @@ import type { JobContext, QueuedJob, WorkerMeta } from '@open-mercato/queue'
 import type { ProgressService } from '../../progress/lib/progressService'
 import type { SyncEngine } from '../lib/sync-engine'
 import type { SyncRunService } from '../lib/sync-run-service'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('data_sync').child({ component: 'sync-export' })
 
 type SyncJobPayload = {
   runId: string
@@ -50,7 +53,7 @@ export default async function handle(job: QueuedJob<SyncJobPayload>, ctx: Handle
         }
       }
     } catch (finalizeError) {
-      console.error('[data-sync] Failed to finalize crashed export worker job:', finalizeError)
+      logger.error('Failed to finalize crashed export worker job', { err: finalizeError })
     }
 
     throw error

package/src/modules/data_sync/workers/sync-import.ts

@@ -2,6 +2,9 @@ import type { JobContext, QueuedJob, WorkerMeta } from '@open-mercato/queue'
 import type { ProgressService } from '../../progress/lib/progressService'
 import type { SyncEngine } from '../lib/sync-engine'
 import type { SyncRunService } from '../lib/sync-run-service'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('data_sync').child({ component: 'sync-import' })
 
 type SyncJobPayload = {
   runId: string
@@ -50,7 +53,7 @@ export default async function handle(job: QueuedJob<SyncJobPayload>, ctx: Handle
         }
       }
     } catch (finalizeError) {
-      console.error('[data-sync] Failed to finalize crashed import worker job:', finalizeError)
+      logger.error('Failed to finalize crashed import worker job', { err: finalizeError })
     }
 
     throw error

package/src/modules/dictionaries/api/[dictionaryId]/entries/[entryId]/route.ts

@@ -21,6 +21,9 @@ import {
   dictionariesTag,
   updateDictionaryEntrySchema as updateEntryDocSchema,
 } from '../../../openapi'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'entries-api' })
 const paramsSchema = z.object({
   dictionaryId: z.string().uuid(),
   entryId: z.string().uuid(),
@@ -164,7 +167,7 @@ export async function PATCH(req: Request, ctx: { params?: { dictionaryId?: strin
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[dictionaries/:id/entries/:entryId.PATCH] Unexpected error', err)
+    logger.error('Failed to update dictionary entry', { err })
     return NextResponse.json({ error: 'Failed to update dictionary entry' }, { status: 500 })
   }
 }
@@ -238,7 +241,7 @@ export async function DELETE(req: Request, ctx: { params?: { dictionaryId?: stri
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[dictionaries/:id/entries/:entryId.DELETE] Unexpected error', err)
+    logger.error('Failed to delete dictionary entry', { err })
     return NextResponse.json({ error: 'Failed to delete dictionary entry' }, { status: 500 })
   }
 }

package/src/modules/dictionaries/api/[dictionaryId]/entries/reorder/route.ts

@@ -25,6 +25,9 @@ import {
   reorderEntriesRequestSchema,
 } from '../../../openapi'
 import { resolveDictionaryActorId } from '../../../context'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'entries-api' })
 
 const paramsSchema = z.object({ dictionaryId: z.string().uuid() })
 
@@ -127,7 +130,7 @@ export async function POST(req: Request, ctx: { params?: { dictionaryId?: string
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[dictionaries/:id/entries/reorder.POST] Unexpected error', err)
+    logger.error('Failed to reorder dictionary entries', { err })
     return NextResponse.json({ error: 'Failed to reorder dictionary entries' }, { status: 500 })
   }
 }

package/src/modules/dictionaries/api/[dictionaryId]/entries/route.ts

@@ -31,6 +31,9 @@ import {
   resolveDictionaryEntrySortMode,
   sortDictionaryEntries,
 } from '@open-mercato/core/modules/dictionaries/lib/entrySort'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'entries-api' })
 
 const paramsSchema = z.object({ dictionaryId: z.string().uuid() })
 
@@ -157,7 +160,7 @@ export async function GET(req: Request, ctx: { params?: { dictionaryId?: string
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[dictionaries/:id/entries.GET] Unexpected error', err)
+    logger.error('Failed to load dictionary entries', { err })
     return NextResponse.json({ error: 'Failed to load dictionary entries' }, { status: 500 })
   }
 }
@@ -249,7 +252,7 @@ export async function POST(req: Request, ctx: { params?: { dictionaryId?: string
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[dictionaries/:id/entries.POST] Unexpected error', err)
+    logger.error('Failed to create dictionary entry', { err })
     return NextResponse.json({ error: 'Failed to create dictionary entry' }, { status: 500 })
   }
 }

package/src/modules/dictionaries/api/[dictionaryId]/entries/set-default/route.ts

@@ -25,6 +25,9 @@ import {
   setDefaultEntryRequestSchema,
 } from '../../../openapi'
 import { resolveDictionaryActorId } from '../../../context'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'entries-api' })
 
 const paramsSchema = z.object({ dictionaryId: z.string().uuid() })
 
@@ -127,7 +130,7 @@ export async function POST(req: Request, ctx: { params?: { dictionaryId?: string
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[dictionaries/:id/entries/set-default.POST] Unexpected error', err)
+    logger.error('Failed to set default entry', { err })
     return NextResponse.json({ error: 'Failed to set default entry' }, { status: 500 })
   }
 }

package/src/modules/dictionaries/api/[dictionaryId]/route.ts

@@ -22,6 +22,9 @@ import {
   upsertDictionarySchema,
 } from '../openapi'
 import { dictionaryKeySchema } from '@open-mercato/core/modules/dictionaries/data/validators'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'api' })
 
 const paramsSchema = z.object({ dictionaryId: z.string().uuid() })
 // System dictionaries use namespaced keys (e.g. `sales.deal_loss_reason`,
@@ -103,7 +106,7 @@ export async function GET(req: Request, ctx: { params?: { dictionaryId?: string
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[dictionaries/:id.GET] Unexpected error', err)
+    logger.error('Failed to load dictionary', { err })
     return NextResponse.json({ error: 'Failed to load dictionary' }, { status: 500 })
   }
 }
@@ -225,7 +228,7 @@ export async function PATCH(req: Request, ctx: { params?: { dictionaryId?: strin
     if (err instanceof z.ZodError) {
       return NextResponse.json({ error: err.issues[0]?.message ?? 'Validation failed' }, { status: 400 })
     }
-    console.error('[dictionaries/:id.PATCH] Unexpected error', err)
+    logger.error('Failed to update dictionary', { err })
     return NextResponse.json({ error: 'Failed to update dictionary' }, { status: 500 })
   }
 }
@@ -286,7 +289,7 @@ export async function DELETE(req: Request, ctx: { params?: { dictionaryId?: stri
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[dictionaries/:id.DELETE] Unexpected error', err)
+    logger.error('Failed to delete dictionary', { err })
     return NextResponse.json({ error: 'Failed to delete dictionary' }, { status: 500 })
   }
 }

package/src/modules/dictionaries/api/context.ts

@@ -7,6 +7,9 @@ import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
 import type { EntityManager } from '@mikro-orm/postgresql'
 import type { AwilixContainer } from 'awilix'
 import type { CommandRuntimeContext } from '@open-mercato/shared/lib/commands'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'context' })
 
 export type DictionariesRouteContext = {
   container: AwilixContainer
@@ -88,7 +91,7 @@ export async function resolveDictionariesRouteContext(req: Request): Promise<Dic
       for (const id of candidateIds) readableOrganizationIds.add(id)
     }
   } catch (err) {
-    console.warn('[dictionaries.resolveContext] Failed to resolve readable organizations', err)
+    logger.warn('Failed to resolve readable organizations', { err })
     for (const id of candidateIds) readableOrganizationIds.add(id)
   }
 

package/src/modules/dictionaries/api/route.ts

@@ -20,6 +20,9 @@ import {
   upsertDictionarySchema,
 } from './openapi'
 import { parseBooleanToken } from '@open-mercato/shared/lib/boolean'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'api' })
 
 export const metadata = {
   GET: { requireAuth: true, requireFeatures: ['dictionaries.view'] },
@@ -67,7 +70,7 @@ export async function GET(req: Request) {
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[dictionaries.GET] Unexpected error', err)
+    logger.error('Failed to load dictionaries', { err })
     return NextResponse.json({ error: 'Failed to load dictionaries' }, { status: 500 })
   }
 }
@@ -153,7 +156,7 @@ export async function POST(req: Request) {
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[dictionaries.POST] Unexpected error', err)
+    logger.error('Failed to create dictionary', { err })
     return NextResponse.json({ error: 'Failed to create dictionary' }, { status: 500 })
   }
 }

package/src/modules/dictionaries/components/DictionariesManager.tsx

@@ -28,6 +28,9 @@ import {
   dictionaryEntrySortModes,
   type DictionaryEntrySortMode,
 } from '../lib/entrySort'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'DictionariesManager' })
 
 export type DictionarySummary = {
   id: string
@@ -140,7 +143,7 @@ export function DictionariesManager() {
         return filtered.length ? filtered[0].id : null
       })
     } catch (err) {
-      console.error('Failed to load dictionaries', err)
+      logger.error('Failed to load dictionaries', { err })
       flash(t('dictionaries.config.error.load', 'Failed to load dictionaries.'), 'error')
     } finally {
       setLoading(false)
@@ -300,7 +303,7 @@ export function DictionariesManager() {
       if (surfaceRecordConflict(err, t)) {
         return
       }
-      console.error('Failed to save dictionary', err)
+      logger.error('Failed to save dictionary', { err })
       flash(t('dictionaries.config.error.save', 'Failed to save dictionary.'), 'error')
     } finally {
       setSubmitting(false)
@@ -357,7 +360,7 @@ export function DictionariesManager() {
         if (surfaceRecordConflict(err, t)) {
           return
         }
-        console.error('Failed to delete dictionary', err)
+        logger.error('Failed to delete dictionary', { err })
         flash(t('dictionaries.config.error.delete', 'Failed to delete dictionary.'), 'error')
       } finally {
         setDeleting(null)

package/src/modules/dictionaries/components/DictionaryEntriesEditor.tsx

@@ -33,6 +33,9 @@ import {
   DialogDescription,
 } from '@open-mercato/ui/primitives/dialog'
 import { TranslationManager } from '@open-mercato/core/modules/translations/components/TranslationManager'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'DictionaryEntriesEditor' })
 
 type Entry = DictionaryEntryRecord
 
@@ -224,7 +227,7 @@ export function DictionaryEntriesEditor({ dictionaryId, dictionaryName, readOnly
       if (surfaceRecordConflict(err, t)) {
         return
       }
-      console.error('Failed to save dictionary entry', err)
+      logger.error('Failed to save dictionary entry', { err })
       flash(t('dictionaries.config.entries.error.save', 'Failed to save dictionary entry.'), 'error')
     } finally {
       setIsSaving(false)
@@ -281,7 +284,7 @@ export function DictionaryEntriesEditor({ dictionaryId, dictionaryName, readOnly
         if (surfaceRecordConflict(err, t)) {
           return
         }
-        console.error('Failed to delete dictionary entry', err)
+        logger.error('Failed to delete dictionary entry', { err })
         flash(t('dictionaries.config.entries.error.delete', 'Failed to delete dictionary entry.'), 'error')
       } finally {
         setIsDeleting(false)

package/src/modules/dictionaries/components/DictionaryEntrySelect.tsx

@@ -27,6 +27,9 @@ import { flash } from '@open-mercato/ui/backend/FlashMessages'
 import { buildHrefWithReturnTo } from '@open-mercato/shared/lib/navigation/returnTo'
 import { DictionaryValue, renderDictionaryColor, renderDictionaryIcon } from './dictionaryAppearance'
 import { AppearanceSelector, type AppearanceSelectorLabels, useAppearanceState } from './AppearanceSelector'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'DictionaryEntrySelect' })
 
 const DEFAULT_APPEARANCE_LABELS: AppearanceSelectorLabels = {
   colorLabel: 'Color',
@@ -131,7 +134,7 @@ export function DictionaryEntrySelect({
       const items = await fetchOptions()
       setOptions(sortOptions === 'none' ? items : items.slice().sort((a, b) => a.label.localeCompare(b.label, undefined, { sensitivity: 'base' })))
     } catch (err) {
-      console.error('DictionaryEntrySelect.fetchOptions failed', err)
+      logger.error('Failed to fetch options', { err })
       flash(labels.errorLoad, 'error')
       setOptions([])
     } finally {
@@ -226,7 +229,7 @@ export function DictionaryEntrySelect({
         flash(labels.successCreateLabel, 'success')
       }
     } catch (err) {
-      console.error('DictionaryEntrySelect.createOption failed', err)
+      logger.error('Failed to create option', { err })
       flash(labels.errorSave, 'error')
     } finally {
       setSaving(false)

package/src/modules/dictionaries/components/DictionarySelectControl.tsx

@@ -11,6 +11,9 @@ import {
   ensureDictionaryEntries,
   invalidateDictionaryEntries,
 } from './hooks/useDictionaryEntries'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'DictionarySelectControl' })
 
 type DictionarySelectControlProps = {
   dictionaryId: string
@@ -66,7 +69,7 @@ export function DictionarySelectControl({
         }
         setInlineCreateEnabled(true)
       } catch (err) {
-        console.warn('DictionarySelectControl.inlineCreate check failed', err)
+        logger.warn('Inline create check failed', { err })
         if (!cancelled) {
           setInlineCreateEnabled(allowInlineCreate)
         }

package/src/modules/dictionaries/fields/dictionary.tsx

@@ -15,6 +15,9 @@ import {
 } from '@open-mercato/ui/primitives/select'
 import { DictionarySelectControl } from '../components/DictionarySelectControl'
 import { useDictionaryEntries } from '../components/hooks/useDictionaryEntries'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'dictionary-field' })
 
 type DictionaryFieldDefinition = {
   dictionaryId?: string
@@ -119,7 +122,7 @@ function DictionaryFieldDefEditor({ def, onChange }: { def: { configJson?: Dicti
         }
       } catch (err) {
         if (!cancelled) {
-          console.error('Failed to load dictionaries list', err)
+          logger.error('Failed to load dictionaries list', { err })
           setError(errorLoadLabel)
         }
       } finally {

package/src/modules/directory/api/organization-branding/route.ts

@@ -18,6 +18,9 @@ import { Organization } from '@open-mercato/core/modules/directory/data/entities
 import { organizationUpdateSchema } from '@open-mercato/core/modules/directory/data/validators'
 import { resolveOrganizationScopeForRequest } from '@open-mercato/core/modules/directory/utils/organizationScope'
 import '@open-mercato/core/modules/directory/commands/organizations'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('directory').child({ component: 'organization-branding' })
 
 export const metadata = {
   GET: { requireAuth: true, requireFeatures: ['directory.organizations.view'] },
@@ -254,7 +257,7 @@ export async function PUT(req: Request) {
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('directory.organization-branding.update failed', err)
+    logger.error('Organization branding update failed', { err })
     return NextResponse.json(
       { error: resolved.translate('directory.branding.errors.save', 'Failed to update organization branding.') },
       { status: 400 },