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

package/src/modules/communication_channels/lib/thread-token.ts

@@ -3,6 +3,9 @@ import type { EntityManager } from '@mikro-orm/postgresql'
 import { findOneWithDecryption } from '@open-mercato/shared/lib/encryption/find'
 import { ChannelThreadToken } from '../data/entities'
 import { isUniqueViolation } from './pg-errors'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'thread-token' })
 
 /**
  * Per-thread crypto token used by the layered thread-matcher to attach
@@ -60,8 +63,8 @@ function getKey(): Buffer {
         ' refusing to sign thread tokens with a static dev key in production.',
     )
   }
-  console.warn(
-    `[communication_channels] No ${HMAC_KEY_ENV} or ${HMAC_FALLBACK_KEY_ENV} configured.` +
+  logger.warn(
+    `No ${HMAC_KEY_ENV} or ${HMAC_FALLBACK_KEY_ENV} configured.` +
       ' Thread tokens will use a dev-only static key — DO NOT USE IN PRODUCTION.',
   )
   cachedKey = createHash('sha256').update('open-mercato-thread-token-dev').digest()

package/src/modules/communication_channels/setup.ts

@@ -1,6 +1,9 @@
 import { createHash } from 'node:crypto'
 import type { ModuleSetupConfig } from '@open-mercato/shared/modules/setup'
 import { COMMUNICATION_CHANNELS_QUEUES } from './lib/queue'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels')
 
 type SchedulerServiceLike = {
   register: (registration: {
@@ -147,10 +150,7 @@ export const setup: ModuleSetupConfig = {
         isEnabled: true,
       })
     } catch (error) {
-      console.warn(
-        '[communication_channels] Failed to register module schedules:',
-        error instanceof Error ? error.message : error,
-      )
+      logger.warn('Failed to register module schedules', { err: error })
     }
   },
 }

package/src/modules/communication_channels/workers/channel-import-history.ts

@@ -18,6 +18,9 @@ import type {
   ProgressService,
   ProgressServiceContext,
 } from '../../progress/lib/progressService'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'channel-import-history' })
 
 /**
  * Spec B § Phase B6 — operator-triggered backlog import worker.
@@ -208,9 +211,7 @@ export default async function handle(
           if (classification.transient) {
             throw err
           }
-          console.warn(
-            `[communication_channels:channel-import-history] permanent ingest failure on channel ${channel.id}; skipping message ${message.externalMessageId}. Reason: ${classification.message}`,
-          )
+          logger.warn('permanent ingest failure; skipping message', { channelId: channel.id, externalMessageId: message.externalMessageId, reason: classification.message })
         }
         processedCount += 1
       }
@@ -241,11 +242,7 @@ export default async function handle(
         progressContext,
       )
     } catch (failErr) {
-      console.error(
-        `[communication_channels:channel-import-history] failed to mark progress job ${progressJobId} as failed: ${
-          failErr instanceof Error ? failErr.message : String(failErr)
-        }`,
-      )
+      logger.error('failed to mark progress job as failed', { progressJobId, err: failErr })
     }
     throw err
   }

package/src/modules/communication_channels/workers/gmail-history-sync.ts

@@ -13,6 +13,9 @@ import { refreshCredentialsIfNeeded } from '../lib/credential-refresh'
 import { classifyOutboundError } from '../lib/error-classification'
 import { writeIngestDeadLetter } from '../lib/dead-letter'
 import type { ChannelAdapterRegistry } from '../lib/registry'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'gmail-history-sync' })
 
 /**
  * Spec C § Phase C2 — Gmail Pub/Sub push delivery worker.
@@ -82,16 +85,14 @@ export default async function handle(
     scope,
   )
   if (!channel) {
-    console.warn(`[gmail-history-sync] channel ${channelId} not found (skip)`)
+    logger.warn('channel not found (skip)', { channelId })
     return
   }
   if (!channel.isActive || channel.status !== 'connected') return
 
   const adapter = adapterRegistry?.get(channel.providerKey)
   if (!adapter || typeof adapter.applyPushNotification !== 'function') {
-    console.warn(
-      `[gmail-history-sync] adapter for '${channel.providerKey}' does not support applyPushNotification`,
-    )
+    logger.warn('adapter does not support applyPushNotification', { providerKey: channel.providerKey })
     return
   }
 
@@ -132,9 +133,7 @@ export default async function handle(
   } catch (err) {
     const classification = classifyOutboundError(err)
     if (classification.transient) throw err
-    console.warn(
-      `[gmail-history-sync] permanent failure applying push for channel ${channel.id}: ${classification.message}`,
-    )
+    logger.warn('permanent failure applying push for channel', { channelId: channel.id, reason: classification.message })
     return
   }
 
@@ -169,9 +168,7 @@ export default async function handle(
           // retry re-runs from a safe point.
           throw err
         }
-        console.warn(
-          `[gmail-history-sync] permanent ingest failure for channel ${channel.id}: ${classification.message}`,
-        )
+        logger.warn('permanent ingest failure for channel', { channelId: channel.id, reason: classification.message })
         await writeIngestDeadLetter({ em, scope, channel, message, err, errorMessage: classification.message })
       }
     }
@@ -191,11 +188,7 @@ export default async function handle(
       channel.lastPolledAt = new Date()
       await em.flush()
     } catch (err) {
-      console.warn(
-        `[gmail-history-sync] failed to persist next cursor for channel ${channel.id}: ${
-          err instanceof Error ? err.message : String(err)
-        }`,
-      )
+      logger.warn('failed to persist next cursor for channel', { channelId: channel.id, err })
     }
   }
 
@@ -215,9 +208,7 @@ export default async function handle(
         { delayMs: 250 },
       )
     } else {
-      console.warn(
-        `[gmail-history-sync] drain page cap (${MAX_DRAIN_PAGES}) reached for channel ${channelId}; stopping re-enqueue`,
-      )
+      logger.warn('drain page cap reached; stopping re-enqueue', { maxDrainPages: MAX_DRAIN_PAGES, channelId })
     }
   }
 }

package/src/modules/communication_channels/workers/gmail-renew-watch.ts

@@ -5,6 +5,9 @@ import { CommunicationChannel } from '../data/entities'
 import { COMMUNICATION_CHANNELS_QUEUES } from '../lib/queue'
 import { pushRenew } from '../commands/push-renew'
 import { emitCommunicationChannelsEvent } from '../events'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'gmail-renew-watch' })
 
 /**
  * Spec C § Phase C4 — Daily cron that re-issues `gmail.users.watch` for
@@ -79,9 +82,7 @@ export default async function handle(
 
     const organizationId = channel.organizationId
     if (!organizationId) {
-      console.warn(
-        `[gmail-renew-watch] skipping channel ${channel.id} — no organizationId on row`,
-      )
+      logger.warn('skipping channel — no organizationId on row', { channelId: channel.id })
       continue
     }
 
@@ -113,15 +114,11 @@ export default async function handle(
       }
     } catch (err) {
       failed += 1
-      console.warn(
-        `[gmail-renew-watch] failed to renew channel ${channel.id}: ${
-          err instanceof Error ? err.message : String(err)
-        }`,
-      )
+      logger.warn('failed to renew channel watch', { channelId: channel.id, err })
     }
   }
   if (renewed > 0 || failed > 0) {
-    console.info(`[gmail-renew-watch] renewed=${renewed} failed=${failed}`)
+    logger.info('gmail watch renewal summary', { renewed, failed })
   }
 }
 

package/src/modules/communication_channels/workers/outbound-delivery.ts

@@ -7,6 +7,9 @@ import {
 } from '../commands/deliver-outbound-message'
 import { computeBackoffMs } from '../lib/error-classification'
 import { COMMUNICATION_CHANNELS_QUEUES, getCommunicationChannelsQueue } from '../lib/queue'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'outbound-delivery' })
 
 /**
  * Worker payload — the subscriber enqueues `{ messageId, scope, attempt? }`.
@@ -92,9 +95,7 @@ export default async function handle(
     // Unexpected error inside the command itself (e.g. DB blip). Re-enqueue
     // up to MAX_ATTEMPTS so we don't lose deliveries due to infrastructure flakes.
     const errorMessage = err instanceof Error ? err.message : String(err)
-    console.warn(
-      `[communication_channels:outbound-delivery] command threw on attempt ${attempt} for message ${messageId}: ${errorMessage}`,
-    )
+    logger.warn('command threw during delivery attempt', { attempt, messageId, reason: errorMessage })
     if (attempt < OUTBOUND_DELIVERY_MAX_ATTEMPTS) {
       await reenqueue(job.payload, attempt)
       return
@@ -121,24 +122,18 @@ export default async function handle(
         !forceCredentialRefresh &&
         attempt < OUTBOUND_DELIVERY_MAX_ATTEMPTS
       ) {
-        console.warn(
-          `[communication_channels:outbound-delivery] reauth failure on attempt ${attempt} for message ${messageId} (${outcome.providerKey}): ${outcome.error}. Retrying once with a forced credential refresh.`,
-        )
+        logger.warn('reauth failure; retrying once with a forced credential refresh', { attempt, messageId, providerKey: outcome.providerKey, reason: outcome.error })
         await reenqueue({ ...job.payload, forceCredentialRefresh: true }, attempt)
         return
       }
       if (outcome.transient && attempt < OUTBOUND_DELIVERY_MAX_ATTEMPTS) {
-        console.warn(
-          `[communication_channels:outbound-delivery] transient failure on attempt ${attempt} for message ${messageId} (${outcome.providerKey}): ${outcome.error}. Re-enqueueing.`,
-        )
+        logger.warn('transient delivery failure; re-enqueueing', { attempt, messageId, providerKey: outcome.providerKey, reason: outcome.error })
         await reenqueue(job.payload, attempt)
         return
       }
       // Permanent or attempts exhausted — `.delivery_failed` was already emitted
       // by the command, so we stop here.
-      console.error(
-        `[communication_channels:outbound-delivery] giving up on message ${messageId} after attempt ${attempt} (${outcome.providerKey}): ${outcome.error}`,
-      )
+      logger.error('giving up on message delivery', { messageId, attempt, providerKey: outcome.providerKey, reason: outcome.error })
       return
     }
   }

package/src/modules/communication_channels/workers/poll-channel.ts

@@ -15,6 +15,9 @@ import { refreshCredentialsIfNeeded } from '../lib/credential-refresh'
 import { emitCommunicationChannelsEvent } from '../events'
 import type { ChannelAdapterRegistry } from '../lib/registry'
 import type { NormalizedInboundMessage } from '../lib/adapter'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'poll-channel' })
 
 /**
  * Job payload for the `communication-channels-poll` queue.
@@ -105,9 +108,7 @@ export default async function handle(
     scope,
   )
   if (!channel) {
-    console.warn(
-      `[communication_channels:poll-channel] channel ${channelId} not found (skipping)`,
-    )
+    logger.warn('channel not found (skipping)', { channelId })
     return
   }
   if (!channel.isActive) return
@@ -119,9 +120,7 @@ export default async function handle(
 
   const adapter = adapterRegistry?.get(channel.providerKey)
   if (!adapter) {
-    console.warn(
-      `[communication_channels:poll-channel] no adapter for provider '${channel.providerKey}' (channel ${channelId})`,
-    )
+    logger.warn('no adapter for provider', { providerKey: channel.providerKey, channelId })
     return
   }
   // Adapter opted out of polling — webhook providers.
@@ -241,9 +240,7 @@ export default async function handle(
     } catch (err) {
       const classification = classifyOutboundError(err)
       if (classification.transient) {
-        console.warn(
-          `[communication_channels:poll-channel] transient ingest failure for channel ${channel.id}; aborting page so cursor is NOT advanced. Reason: ${classification.message}`,
-        )
+        logger.warn('transient ingest failure; aborting page so cursor is not advanced', { channelId: channel.id, reason: classification.message })
         transientIngestAbort = true
         break
       }
@@ -259,9 +256,7 @@ export default async function handle(
         err,
         errorMessage: classification.message,
       })
-      console.warn(
-        `[communication_channels:poll-channel] permanent ingest failure for channel ${channel.id}; recorded in dead-letter and advancing cursor past message ${message.externalMessageId}. Reason: ${classification.message}`,
-      )
+      logger.warn('permanent ingest failure; recorded in dead-letter and advancing cursor', { channelId: channel.id, externalMessageId: message.externalMessageId, reason: classification.message })
     }
   }
 
@@ -324,9 +319,7 @@ export default async function handle(
         { delayMs: 250 },
       )
     } else {
-      console.warn(
-        `[communication_channels:poll-channel] drain page cap (${MAX_DRAIN_PAGES}) reached for channel ${channel.id}; stopping re-enqueue until the next scheduled tick`,
-      )
+      logger.warn('drain page cap reached; stopping re-enqueue until the next scheduled tick', { maxDrainPages: MAX_DRAIN_PAGES, channelId: channel.id })
     }
   }
 }

package/src/modules/communication_channels/workers/poll-tick.ts

@@ -4,6 +4,9 @@ import { findWithDecryption } from '@open-mercato/shared/lib/encryption/find'
 import { CommunicationChannel } from '../data/entities'
 import { COMMUNICATION_CHANNELS_QUEUES, getCommunicationChannelsQueue } from '../lib/queue'
 import type { PollChannelJobPayload } from './poll-channel'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'poll-tick' })
 
 /**
  * Scheduler tick payload. Fired by the `@open-mercato/scheduler` cron entry
@@ -70,9 +73,8 @@ export default async function handle(
   const organizationId =
     raw.scope?.organizationId ?? raw.organizationId ?? null
   if (!tenantId) {
-    console.warn(
-      '[communication_channels:poll-tick] skipping tick — payload has no tenantId',
-      { payload: raw },
+    logger.warn(
+      'skipping tick — payload has no tenantId',
     )
     return
   }
@@ -197,9 +199,7 @@ export default async function handle(
   if (recovered > 0) await em.flush()
 
   if (enqueued > 0 || recovered > 0) {
-    console.log(
-      `[communication_channels:poll-tick] enqueued ${enqueued} normal + ${recovered} auto-recover poll job(s) for tenant ${scope.tenantId}`,
-    )
+    logger.info('enqueued poll jobs for tenant', { enqueued, recovered, tenantId: scope.tenantId })
   }
 }
 

package/src/modules/communication_channels/workers/reaction-processor.ts

@@ -19,6 +19,9 @@ import {
 } from '../lib/reaction-processor-types'
 import { refreshCredentialsIfNeeded } from '../lib/credential-refresh'
 import type { ChannelAdapterRegistry } from '../lib/registry'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'reaction-processor' })
 
 export type { ReactionProcessorPayload }
 
@@ -238,22 +241,16 @@ async function maybeRetry(
   if (result.status === 'ok') return
   if (result.status === 'no_adapter' || result.status === 'channel_inactive') {
     // Permanent — log and stop.
-    console.error(
-      `[communication_channels:reaction-processor] ${result.status}: ${result.message}`,
-    )
+    logger.error('reaction processing failed', { status: result.status, reason: result.message })
     return
   }
   if (!result.transient) {
-    console.error(
-      `[communication_channels:reaction-processor] permanent failure: ${result.message}`,
-    )
+    logger.error('permanent reaction processing failure', { reason: result.message })
     return
   }
   const attempt = payload.attempt ?? 1
   if (attempt >= REACTION_PROCESSOR_MAX_ATTEMPTS) {
-    console.error(
-      `[communication_channels:reaction-processor] giving up after attempt ${attempt}: ${result.message}`,
-    )
+    logger.error('giving up on reaction processing', { attempt, reason: result.message })
     return
   }
   const next: ReactionProcessorPayload = { ...payload, attempt: attempt + 1 }

package/src/modules/configs/api/cache/route.ts

@@ -18,6 +18,9 @@ import {
   cachePurgeSegmentResponseSchema,
   configErrorSchema,
 } from '../openapi'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('configs')
 
 export const metadata = {
   GET: { requireAuth: true, requireFeatures: ['configs.cache.view'] },
@@ -51,7 +54,7 @@ export async function GET(req: Request) {
     const stats = await runWithCacheTenant(auth.tenantId ?? null, () => collectCrudCacheStats(cache))
     return NextResponse.json(stats)
   } catch (error) {
-    console.error('[configs.cache] failed to resolve cache stats', error)
+    logger.error('failed to resolve cache stats', { component: 'cache', err: error })
     return NextResponse.json(
       { error: translate('configs.cache.unavailable', 'Cache service is unavailable.') },
       { status: 500 },
@@ -158,7 +161,7 @@ export async function POST(req: Request) {
 
     return NextResponse.json(responseBody)
   } catch (error) {
-    console.error('[configs.cache] failed to purge cache', error)
+    logger.error('failed to purge cache', { component: 'cache', err: error })
     return NextResponse.json(
       { error: translate('configs.cache.purgeError', 'Failed to purge cache segment.') },
       { status: 500 },

package/src/modules/configs/api/system-status/route.ts

@@ -12,6 +12,9 @@ import {
   purgeCacheResponseSchema,
   configErrorSchema,
 } from '../openapi'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('configs')
 
 export const metadata = {
   GET: { requireAuth: true, requireFeatures: ['configs.system_status.view'] },
@@ -28,7 +31,7 @@ export async function GET(req: Request) {
     const snapshot: SystemStatusSnapshot = buildSystemStatusSnapshot()
     return NextResponse.json(snapshot)
   } catch (error) {
-    console.error('[configs.system-status] failed to build environment snapshot', error)
+    logger.error('failed to build environment snapshot', { component: 'system-status', err: error })
     const { translate } = await resolveTranslations()
     return NextResponse.json(
       { error: translate('configs.systemStatus.error', 'Failed to load system status') },
@@ -65,7 +68,7 @@ export async function POST(req: Request) {
     const cleared = await runWithCacheTenant(tenantScope, () => cache.clear())
     return NextResponse.json({ cleared })
   } catch (error) {
-    console.error('[configs.system-status] failed to purge cache', error)
+    logger.error('failed to purge cache', { component: 'system-status', err: error })
     return NextResponse.json(
       { error: translate('configs.systemStatus.actions.purgeCacheError', 'Failed to purge cache.') },
       { status: 500 }

package/src/modules/configs/api/upgrade-actions/route.ts

@@ -14,6 +14,9 @@ import {
   executeUpgradeActionResponseSchema,
   configErrorSchema,
 } from '../openapi'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('configs')
 
 export const metadata = {
   GET: { requireAuth: true, requireFeatures: ['configs.manage'] },
@@ -91,7 +94,7 @@ export async function GET(req: Request) {
       : []
     return NextResponse.json({ version, actions: payload })
   } catch (error) {
-    console.error('[configs.upgrade-actions] failed to load upgrade actions', error)
+    logger.error('failed to load upgrade actions', { component: 'upgrade-actions', err: error })
     return NextResponse.json({ error: 'Failed to load upgrade actions' }, { status: 500 })
   }
 }
@@ -137,7 +140,7 @@ export async function POST(req: Request) {
       version,
     })
   } catch (error) {
-    console.error('[configs.upgrade-actions] failed to execute upgrade action', error)
+    logger.error('failed to execute upgrade action', { component: 'upgrade-actions', err: error })
     const message = translate('upgrades.runFailed', 'We could not run this upgrade action.')
     const details = error instanceof Error ? error.message : null
     return NextResponse.json({ error: message, details }, { status: 500 })

package/src/modules/configs/components/ModuleTelemetryPanel.tsx

@@ -25,6 +25,9 @@ import type {
   ModuleResourceUsageTimeBucket,
   ModuleResourceUsageTimeBucketModule,
 } from '@open-mercato/shared/lib/modules/resource-usage'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('configs')
 
 const API_PATH = '/api/configs/module-telemetry'
 
@@ -628,7 +631,7 @@ function copyRowStats(module: RangeModuleSummary, translate: (key: string, fallb
       flash(translate('configs.moduleTelemetry.table.copySuccess', 'Row stats copied to clipboard.'), 'success')
     })
     .catch((err) => {
-      console.warn('[ModuleTelemetryPanel] clipboard write failed', err)
+      logger.warn('clipboard write failed', { component: 'ModuleTelemetryPanel', err })
       flash(translate('configs.moduleTelemetry.table.copyError', 'Failed to copy row stats.'), 'error')
     })
 }

package/src/modules/configs/lib/touchGeneratedBarrels.ts

@@ -34,7 +34,7 @@ export function touchGeneratedBarrels(
   options: TouchGeneratedBarrelsOptions = {},
 ): TouchGeneratedBarrelsResult {
   const cwd = options.cwd ?? process.cwd()
-  const log = options.log ?? ((message: string) => console.log(message))
+  const log = options.log ?? ((message: string) => process.stdout.write(`${message}\n`))
   const quiet = options.quiet === true
 
   const generatedDir = findGeneratedDir(cwd)

package/src/modules/configs/services/upgradeActionsService.ts

@@ -5,6 +5,9 @@ import { appVersion } from '@open-mercato/shared/lib/version'
 import { parseBooleanToken } from '@open-mercato/shared/lib/boolean'
 import { UpgradeActionRun } from '../data/entities'
 import { actionsUpToVersion, findUpgradeAction, type UpgradeActionDefinition } from '../lib/upgrade-actions'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('configs')
 
 export type UpgradeActionStatus = 'completed' | 'already_completed'
 
@@ -55,7 +58,8 @@ export async function executeUpgradeAction(
   }
   const em = container.resolve<EntityManager>('em')
   const status = await em.transactional(async (tem) => {
-    console.info('[upgrade-actions] executing', {
+    logger.info('Executing upgrade action', {
+      component: 'upgrade-actions',
       actionId: definition.id,
       version: definition.version,
       requestedVersion: version,
@@ -78,10 +82,10 @@ export async function executeUpgradeAction(
     })
     tem.persist(record)
     await tem.flush()
-    console.info('[upgrade-actions] completed', { actionId: definition.id, version, tenantId, organizationId })
+    logger.info('Upgrade action completed', { component: 'upgrade-actions', actionId: definition.id, version, tenantId, organizationId })
     return 'completed' as const
   }).catch((error) => {
-    console.error('[upgrade-actions] failed', { actionId, tenantId, organizationId, version, error })
+    logger.error('Upgrade action failed', { component: 'upgrade-actions', actionId, tenantId, organizationId, version, err: error })
     if (error instanceof UniqueConstraintViolationException) {
       return 'already_completed' as const
     }

package/src/modules/currencies/components/CurrencyFetchingConfig.tsx

@@ -12,6 +12,9 @@ import { StatusBadge, type StatusBadgeVariant } from '@open-mercato/ui/primitive
 import { Alert } from '@open-mercato/ui/primitives/alert'
 import { TimeInput } from '@open-mercato/ui/backend/inputs/TimeInput'
 import { useT } from '@open-mercato/shared/lib/i18n/context'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('currencies').child({ component: 'CurrencyFetchingConfig' })
 
 interface FetchConfig {
   id: string
@@ -91,7 +94,7 @@ export default function CurrencyFetchingConfig() {
         setConfigs(result.configs as FetchConfig[])
       }
     } catch (err: any) {
-      console.error('Failed to initialize providers:', err)
+      logger.error('Failed to initialize providers', { err })
     } finally {
       setInitializing(false)
     }

package/src/modules/currencies/lib/exchangeRateFormConfig.ts

@@ -1,6 +1,9 @@
 import type { CrudFormGroup, CrudFieldOption } from '@open-mercato/ui/backend/CrudForm'
 import type { ApiCallResult } from '@open-mercato/ui/backend/utils/apiCall'
 import { createCrudFormError } from '@open-mercato/ui/backend/utils/serverErrors'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('currencies').child({ component: 'exchange-rate-form' })
 
 export type CurrencyOption = {
   id: string
@@ -34,7 +37,7 @@ export async function loadCurrencyOptions(
       }))
     }
   } catch (error) {
-    console.error('Failed to load currencies:', error)
+    logger.error('Failed to load currencies', { err: error })
   }
   return []
 }

package/src/modules/currencies/services/providers/nbp.ts

@@ -1,6 +1,9 @@
 import { RateProvider, RateProviderResult } from './base'
 import { fromZonedTime } from 'date-fns-tz'
 import { fetchWithTimeout, resolveTimeoutMs } from '@open-mercato/shared/lib/http/fetchWithTimeout'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('currencies').child({ component: 'nbp' })
 
 const DEFAULT_RATE_FETCH_TIMEOUT_MS = 15_000
 
@@ -41,7 +44,7 @@ export class NBPProvider implements RateProvider {
   ): Promise<RateProviderResult[]> {
     // Check if PLN is available (required as base currency for NBP)
     if (!availableCurrencies.has(this.providerBaseCurrency)) {
-      console.debug('[NBP] Skipping: PLN not found in available currencies')
+      logger.debug('Skipping: PLN not found in available currencies')
       return []
     }
 
@@ -53,7 +56,7 @@ export class NBPProvider implements RateProvider {
 
       if (response.status === 404) {
         // No data for this date (weekend/holiday)
-        console.log(`[NBP] No data available for ${dateStr}`)
+        logger.debug('No data available', { date: dateStr })
         return []
       }
 
@@ -104,10 +107,10 @@ export class NBPProvider implements RateProvider {
         })
       }
 
-      console.log(`[NBP] Fetched ${results.length} rates for ${dateStr}`)
+      logger.info('Fetched rates', { count: results.length, date: dateStr })
       return results
     } catch (err: any) {
-      console.error(`[NBP] Fetch error for ${dateStr}:`, err.message)
+      logger.error('Fetch error', { date: dateStr, err })
       throw new Error(`Failed to fetch NBP rates: ${err.message}`)
     }
   }

package/src/modules/currencies/services/providers/raiffeisen.ts

@@ -1,6 +1,9 @@
 import { RateProvider, RateProviderResult } from './base'
 import { fromZonedTime } from 'date-fns-tz'
 import { fetchWithTimeout, resolveTimeoutMs } from '@open-mercato/shared/lib/http/fetchWithTimeout'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('currencies').child({ component: 'raiffeisen' })
 
 const DEFAULT_RATE_FETCH_TIMEOUT_MS = 15_000
 
@@ -49,7 +52,7 @@ export class RaiffeisenPolandProvider implements RateProvider {
   ): Promise<RateProviderResult[]> {
     // Check if PLN is available (required as base currency for Raiffeisen)
     if (!availableCurrencies.has(this.providerBaseCurrency)) {
-      console.debug('[Raiffeisen] Skipping: PLN not found in available currencies')
+      logger.debug('Skipping: PLN not found in available currencies')
       return []
     }
 
@@ -61,7 +64,7 @@ export class RaiffeisenPolandProvider implements RateProvider {
 
       if (!response.ok) {
         if (response.status === 404) {
-          console.log(`[Raiffeisen] No data available for ${dateStr}`)
+          logger.debug('No data available', { date: dateStr })
           return []
         }
         throw new Error(
@@ -85,7 +88,7 @@ export class RaiffeisenPolandProvider implements RateProvider {
       // Extract time from first rate entry (all rates in same time slot have same time)
       const firstRate = rates[0]
       if (!firstRate) {
-        console.log(`[Raiffeisen] No rates available in time slot ${firstTime}`)
+        logger.debug('No rates available in time slot', { timeSlot: firstTime })
         return []
       }
 
@@ -124,12 +127,10 @@ export class RaiffeisenPolandProvider implements RateProvider {
         })
       }
 
-      console.log(
-        `[Raiffeisen] Fetched ${results.length} rates for ${dateStr} at ${firstRate.time} Warsaw time (${rateDate.toISOString()})`
-      )
+      logger.info('Fetched rates', { count: results.length, date: dateStr, time: firstRate.time, rateDate: rateDate.toISOString() })
       return results
     } catch (err: any) {
-      console.error(`[Raiffeisen] Fetch error for ${dateStr}:`, err)
+      logger.error('Fetch error', { date: dateStr, err })
       throw new Error(`Failed to fetch Raiffeisen rates: ${err.message}`)
     }
   }