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

package/src/modules/communication_channels/commands/deliver-outbound-message.ts

@@ -22,6 +22,9 @@ import {
   ExternalMessage,
   MessageChannelLink,
 } from '../data/entities'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const moduleLogger = createLogger('communication_channels').child({ component: 'deliver-outbound-message' })
 
 /**
  * Sentinel — `Message.threadId` of an internal-only (no channel link) message
@@ -311,7 +314,7 @@ const deliverOutboundMessageCommand: CommandHandler<
       },
       {
         credentialsService,
-        logger: (...args) => console.warn(...args),
+        logger: (...args) => moduleLogger.warn('credential refresh diagnostic', { details: args }),
       },
     )
     credentials = refreshResult.credentials
@@ -334,10 +337,7 @@ const deliverOutboundMessageCommand: CommandHandler<
       // Token creation should never block a send — if it fails, the message
       // still goes out, just without our thread-token attachment point.
       // Threading falls back to the JWZ strategy via Message-Id headers.
-      console.warn(
-        '[communication_channels:deliver-outbound] thread token unavailable, proceeding without it:',
-        tokenErr instanceof Error ? tokenErr.message : tokenErr,
-      )
+      moduleLogger.warn('thread token unavailable, proceeding without it', { err: tokenErr })
     }
 
     // (5) + (6) Convert + send.

package/src/modules/communication_channels/commands/disconnect-channel.ts

@@ -7,6 +7,9 @@ import { findOneWithDecryption } from '@open-mercato/shared/lib/encryption/find'
 import { CommunicationChannel } from '../data/entities'
 import { emitCommunicationChannelsEvent } from '../events'
 import { pushUnregister } from './push-unregister'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'disconnect-channel' })
 
 const disconnectChannelSchema = z.object({
   channelId: z.string().uuid(),
@@ -125,11 +128,7 @@ const disconnectChannelCommand: CommandHandler<
           input: { channelId: channel.id },
         })
       } catch (err) {
-        console.warn(
-          `[disconnect-channel] push unregister failed for ${channel.id}: ${
-            err instanceof Error ? err.message : String(err)
-          }`,
-        )
+        logger.warn('push unregister failed for channel', { channelId: channel.id, err })
       }
     }
 

package/src/modules/communication_channels/commands/ingest-inbound-message.ts

@@ -19,6 +19,9 @@ import {
 import { normalizedInboundMessageSchema } from '../data/validators'
 import { resolveCommunicationChannelsSystemUserId } from '../lib/system-user'
 import { isUniqueViolation } from '../lib/pg-errors'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'ingest-inbound-message' })
 
 const ingestInputSchema = z.object({
   channelId: z.string().uuid(),
@@ -152,10 +155,7 @@ const ingestInboundMessageCommand: CommandHandler<IngestInboundMessageInput, Ing
       } catch (dedupErr) {
         // Sent-folder dedup is best-effort — a failure here must not abort
         // ingest (better a possible duplicate than a lost inbound message).
-        console.warn(
-          '[communication_channels:ingest-inbound] sent-folder dedup query failed, continuing:',
-          dedupErr instanceof Error ? dedupErr.message : dedupErr,
-        )
+        logger.warn('sent-folder dedup query failed, continuing', { err: dedupErr })
       }
     }
 
@@ -285,10 +285,7 @@ const ingestInboundMessageCommand: CommandHandler<IngestInboundMessageInput, Ing
     } catch (matcherErr) {
       // Matcher failure must not block ingest — fall back to the existing
       // conversation-based thread mapping so the message still lands.
-      console.warn(
-        '[communication_channels:ingest-inbound] thread matcher failed, falling back to conversation mapping:',
-        matcherErr instanceof Error ? matcherErr.message : matcherErr,
-      )
+      logger.warn('thread matcher failed, falling back to conversation mapping', { err: matcherErr })
     }
 
     // (4) Contact resolution (best-effort, advisory).
@@ -316,10 +313,7 @@ const ingestInboundMessageCommand: CommandHandler<IngestInboundMessageInput, Ing
       // Best-effort: contact resolution is advisory and must not abort ingest.
       // Log like the sibling dedup/matcher catches so a misbehaving resolver is
       // visible in operator logs instead of failing silently.
-      console.warn(
-        '[communication_channels:ingest-inbound] contact resolution failed, continuing without a CRM match:',
-        contactErr instanceof Error ? contactErr.message : contactErr,
-      )
+      logger.warn('contact resolution failed, continuing without a CRM match', { err: contactErr })
       contactHint = null
     }
     const matchedPersonId = contactHint?.matchedPersonId ?? null

package/src/modules/communication_channels/commands/push-unregister.ts

@@ -7,6 +7,9 @@ import { getChannelAdapterRegistry } from '../lib/adapter-registry-singleton'
 import { refreshCredentialsIfNeeded } from '../lib/credential-refresh'
 import { POLLING_ONLY_DEFAULT_INTERVAL_SECONDS } from '../lib/connect-channel'
 import { emitCommunicationChannelsEvent } from '../events'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'push-unregister' })
 
 /**
  * Spec C § Phase C5 — Tear down a previously-registered push delivery.
@@ -130,9 +133,7 @@ export async function pushUnregister(params: {
     })
   } catch (err) {
     const message = err instanceof Error ? err.message : String(err)
-    console.warn(
-      `[push-unregister] adapter.unregisterPush failed for channel ${channel.id}: ${message}`,
-    )
+    logger.warn('adapter.unregisterPush failed for channel', { channelId: channel.id, reason: message })
     return {
       channelId: channel.id,
       status: 'failed',

package/src/modules/communication_channels/commands/reassign-conversation.ts

@@ -6,6 +6,9 @@ import { extractUndoPayload as extractSharedUndoPayload } from '@open-mercato/sh
 import { findOneWithDecryption } from '@open-mercato/shared/lib/encryption/find'
 import { ChannelThreadMapping, ExternalConversation } from '../data/entities'
 import { emitCommunicationChannelsEvent } from '../events'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'reassign-conversation' })
 
 const reassignConversationSchema = z.object({
   threadId: z.string().uuid(),
@@ -165,9 +168,9 @@ const reassignConversationCommand: CommandHandler<
     } catch (emitErr) {
       // Best-effort lifecycle/workflow-trigger signal — a bus failure must not
       // abort the reassignment (the rows are already committed above).
-      console.warn(
-        '[communication_channels:reassign-conversation] reassigned event emit failed:',
-        emitErr instanceof Error ? emitErr.message : emitErr,
+      logger.warn(
+        'reassigned event emit failed',
+        { err: emitErr },
       )
     }
 

package/src/modules/communication_channels/lib/dead-letter.ts

@@ -2,6 +2,9 @@ import type { EntityManager } from '@mikro-orm/postgresql'
 import { findOneWithDecryption } from '@open-mercato/shared/lib/encryption/find'
 import { ChannelIngestDeadLetter } from '../data/entities'
 import type { NormalizedInboundMessage } from './adapter'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'dead-letter' })
 
 const DEAD_LETTER_RAW_BODY_MAX_BYTES_DEFAULT = 32_768
 
@@ -78,10 +81,6 @@ export async function writeIngestDeadLetter(args: WriteIngestDeadLetterArgs): Pr
     em.persist(deadLetter)
     await em.flush()
   } catch (ddlErr) {
-    console.error(
-      `[communication_channels:dead-letter] failed to record dead-letter for channel ${channel.id}: ${
-        ddlErr instanceof Error ? ddlErr.message : String(ddlErr)
-      }`,
-    )
+    logger.error('failed to record dead-letter for channel', { channelId: channel.id, err: ddlErr })
   }
 }

package/src/modules/communication_channels/lib/email-mime.ts

@@ -1,6 +1,9 @@
 import crypto from 'node:crypto'
 import type { NormalizedInboundMessage, NormalizedAttachment } from './adapter'
 import { EMAIL_MAX_ATTACHMENT_BYTES } from './email-capabilities'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'email-mime' })
 
 /**
  * Aggregate ceiling for all attachments on a single inbound message. Inbound
@@ -337,15 +340,11 @@ export function normalizeAttachments(attachments: ParsedAttachment[]): Normalize
     if (!att.content) continue
     const byteLength = att.content.byteLength
     if (byteLength > EMAIL_MAX_ATTACHMENT_BYTES) {
-      console.warn(
-        `[email-mime] dropping oversized inbound attachment "${att.filename ?? 'attachment'}" (${byteLength} bytes > ${EMAIL_MAX_ATTACHMENT_BYTES} cap)`,
-      )
+      logger.warn('dropping oversized inbound attachment', { fileName: att.filename ?? 'attachment', bytes: byteLength, maxBytes: EMAIL_MAX_ATTACHMENT_BYTES })
       continue
     }
     if (totalBytes + byteLength > TOTAL_INBOUND_ATTACHMENTS_MAX_BYTES) {
-      console.warn(
-        `[email-mime] aggregate inbound attachment size exceeded ${TOTAL_INBOUND_ATTACHMENTS_MAX_BYTES} bytes; dropping remaining attachments`,
-      )
+      logger.warn('aggregate inbound attachment size exceeded; dropping remaining attachments', { maxBytes: TOTAL_INBOUND_ATTACHMENTS_MAX_BYTES })
       break
     }
     totalBytes += byteLength

package/src/modules/communication_channels/lib/oauth-client-config.ts

@@ -1,3 +1,7 @@
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'oauth-client-config' })
+
 /**
  * Minimal shape of the integrations credentials service `resolve` we depend on.
  * Declared locally so this helper compiles even when the integrations module is
@@ -65,9 +69,9 @@ export async function resolveOAuthClientCredentials(
       // A resolve error (e.g. a transient DB issue) for this org scope shouldn't
       // abort the lookup — fall through to the next scope — but surface it so a
       // real misconfiguration isn't silently swallowed.
-      console.warn(
-        '[internal] [communication_channels] resolveOAuthClientCredentials: credential resolve failed for an org scope:',
-        resolveErr instanceof Error ? resolveErr.message : resolveErr,
+      logger.warn(
+        'resolveOAuthClientCredentials credential resolve failed for an org scope',
+        { err: resolveErr },
       )
       row = null
     }

package/src/modules/communication_channels/lib/send-as-user.ts

@@ -12,6 +12,9 @@ import { ChannelMutationBlockedError, guardOutboundCreate } from './mutation-gua
 import { COMMUNICATION_CHANNELS_QUEUES, getCommunicationChannelsQueue } from './queue'
 import type { OutboundDeliveryPayload } from '../workers/outbound-delivery'
 import { htmlToText } from './email-mime'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels')
 
 export type SendAsUserActor = {
   userId: string
@@ -158,7 +161,7 @@ export async function sendAsUser(
       },
     )
   } catch (err) {
-    console.error('[communication_channels] send-as-user compose failed', err)
+    logger.error('communication_channels send-as-user compose failed', { err })
     return { ok: false, status: 500, error: '[internal] compose failed' }
   }
 

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 },