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

package/src/modules/catalog/backend/catalog/products/optionSchemaClient.ts

@@ -1,4 +1,7 @@
 import { apiCall } from '@open-mercato/ui/backend/utils/apiCall'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 export type OptionSchemaRecord = {
   version?: number | null
@@ -38,7 +41,7 @@ export async function fetchOptionSchemaTemplate(id: string): Promise<OptionSchem
     const record = Array.isArray(res.result?.items) ? res.result?.items?.[0] : null
     return record ?? null
   } catch (err) {
-    console.error('catalog.option-schemas.fetch-one failed', err)
+    logger.error('catalog.option-schemas.fetch-one failed', { err })
     return null
   }
 }

package/src/modules/catalog/components/PriceKindSettings.tsx

@@ -28,6 +28,9 @@ import { DictionaryEntrySelect } from '@open-mercato/core/modules/dictionaries/c
 import { useCurrencyDictionary } from '@open-mercato/core/modules/customers/components/detail/hooks/useCurrencyDictionary'
 import type { DictionaryOption } from '@open-mercato/core/modules/dictionaries/components/DictionaryEntrySelect'
 import type { CatalogPriceDisplayMode } from '../data/types'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 type PriceKind = {
   id: string
@@ -153,7 +156,7 @@ export function PriceKindSettings() {
       const normalized = Array.isArray(payload.items) ? payload.items.map((item) => normalizePriceKind(item)) : []
       setItems(normalized)
     } catch (err) {
-      console.error('catalog.price-kinds.list failed', err)
+      logger.error('catalog.price-kinds.list failed', { err })
       flash(loadErrorMessage, 'error')
     } finally {
       setLoading(false)
@@ -236,7 +239,7 @@ export function PriceKindSettings() {
       closeDialog()
       await loadItems()
     } catch (err) {
-      console.error('catalog.price-kinds.save failed', err)
+      logger.error('catalog.price-kinds.save failed', { err })
       // Route a concurrent-edit 409 through the single conflict surface (unified
       // conflict bar, or the enterprise merge dialog when its handler is mounted)
       // and close the editor so that surface owns the resolution. Other errors
@@ -276,7 +279,7 @@ export function PriceKindSettings() {
         flash(t('catalog.priceKinds.messages.deleted', 'Price kind deleted.'), 'success')
         await loadItems()
       } catch (err) {
-        console.error('catalog.price-kinds.delete failed', err)
+        logger.error('catalog.price-kinds.delete failed', { err })
         // Route a concurrent-edit 409 through the single conflict surface; fall
         // back to a flash for any other delete failure.
         if (surfaceRecordConflict(err, t, { onRefresh: () => { void loadItems() } })) return

package/src/modules/catalog/components/UnitPriceDisplaySettings.tsx

@@ -8,6 +8,9 @@ import { apiCall, readApiResultOrThrow } from '@open-mercato/ui/backend/utils/ap
 import { raiseCrudError } from '@open-mercato/ui/backend/utils/serverErrors'
 import { useOrganizationScopeVersion } from '@open-mercato/shared/lib/frontend/useOrganizationScope'
 import { useT } from '@open-mercato/shared/lib/i18n/context'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 type CatalogSettingsResponse = {
   unitPriceDisplayEnabled?: boolean
@@ -30,7 +33,7 @@ export function UnitPriceDisplaySettings() {
       )
       setEnabled(payload.unitPriceDisplayEnabled !== false)
     } catch (err) {
-      console.error('catalog.settings.load failed', err)
+      logger.error('catalog.settings.load failed', { err })
       flash(loadError, 'error')
       setEnabled(true)
     }
@@ -58,7 +61,7 @@ export function UnitPriceDisplaySettings() {
       }
       flash(t('catalog.settings.messages.saved', 'Settings saved.'), 'success')
     } catch (err) {
-      console.error('catalog.settings.save failed', err)
+      logger.error('catalog.settings.save failed', { err })
       setEnabled(previous)
       const message = err instanceof Error
         ? err.message

package/src/modules/catalog/inbox-actions.ts

@@ -9,6 +9,9 @@ import {
   resolveProductDiscrepanciesInProposal,
 } from '../inbox_ops/lib/executionHelpers'
 import { CatalogPriceKind } from './data/entities'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 async function executeCreateProductAction(
   action: { id: string; proposalId: string; payload: unknown },
@@ -60,7 +63,7 @@ async function executeCreateProductAction(
     )
     variantId = variantResult.variantId ?? null
   } catch (variantErr) {
-    console.warn('[catalog:inbox-action] Failed to create default variant (non-fatal):', variantErr instanceof Error ? variantErr.message : variantErr)
+    logger.warn('catalog.inbox-action failed to create default variant (non-fatal)', { err: variantErr })
   }
 
   if (variantId && payload.unitPrice && payload.currencyCode) {
@@ -88,7 +91,7 @@ async function executeCreateProductAction(
         })
       }
     } catch (priceErr) {
-      console.warn('[catalog:inbox-action] Failed to set price on default variant (non-fatal):', priceErr)
+      logger.warn('catalog.inbox-action Failed to set price on default variant (non-fatal)', { err: priceErr })
     }
   }
 

package/src/modules/catalog/seed/examples.ts

@@ -39,6 +39,9 @@ import {
   slugifyAttachmentFileName,
 } from "@open-mercato/core/modules/attachments/lib/imageUrls";
 import { canonicalizeUnitCode } from "../lib/unitCodes";
+import { createLogger } from '@open-mercato/shared/lib/logger';
+
+const logger = createLogger('catalog');
 
 type SeedScope = { tenantId: string; organizationId: string };
 
@@ -84,7 +87,7 @@ async function attachMediaFromExamples(
     try {
       buffer = await fs.readFile(sourcePath);
     } catch (error) {
-      console.warn(`[catalog.seed] Example media missing: ${sourcePath}`);
+      logger.warn('catalog.seed example media missing', { sourcePath });
       continue;
     }
     const stored = await storePartitionFile({
@@ -1043,10 +1046,7 @@ export async function seedCatalogExamples(
     try {
       await assign();
     } catch (err) {
-      console.warn(
-        "[catalog.seed] Failed to set example custom field values",
-        err,
-      );
+      logger.warn("catalog.seed failed to set example custom field values", { err });
     }
   }
 

package/src/modules/catalog/subscribers/low-stock-notification.ts

@@ -2,6 +2,9 @@ import type { EntityManager } from '@mikro-orm/postgresql'
 import { resolveNotificationService } from '../../notifications/lib/notificationService'
 import { buildFeatureNotificationFromType } from '../../notifications/lib/notificationBuilder'
 import { notificationTypes } from '../notifications'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 export const metadata = {
   event: 'catalog.product.stock_low',
@@ -47,6 +50,6 @@ export default async function handle(payload: LowStockPayload, ctx: ResolverCont
       organizationId: payload.organizationId ?? null,
     })
   } catch (err) {
-    console.error('[catalog:low-stock-notification] Failed to create notification:', err)
+    logger.error('catalog.low-stock-notification Failed to create notification', { err })
   }
 }

package/src/modules/catalog/widgets/injection/product-seo/state.ts

@@ -1,4 +1,7 @@
 import type { TranslateFn } from '@open-mercato/shared/lib/i18n/context'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 export type ProductSeoValidationPayload = {
   ok: boolean
@@ -30,7 +33,7 @@ export function publishProductSeoValidation(payload: ProductSeoValidationPayload
     try {
       listener(payload)
     } catch (err) {
-      console.error('[product-seo] Failed to notify listener', err)
+      logger.error('product-seo Failed to notify listener', { err })
     }
   })
 }

package/src/modules/communication_channels/api/get/oauth/[provider]/callback/route.ts

@@ -13,6 +13,9 @@ import {
   OAuthStateError,
   verifyOAuthState,
 } from '../../../../../lib/oauth-state'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'oauth-callback' })
 
 export const metadata = {
   path: '/communication_channels/oauth/[provider]/callback',
@@ -176,9 +179,9 @@ export async function GET(req: Request, context: RouteContext): Promise<Response
       stateExtra: statePayload.extra,
     })
   } catch (err) {
-    console.warn(
-      `[communication_channels:oauth] code exchange failed for provider ${provider}:`,
-      err instanceof Error ? err.message : err,
+    logger.warn(
+      'code exchange failed for provider',
+      { provider, err },
     )
     return redirectWithFlash(req, returnUrl, {
       type: 'error',
@@ -219,9 +222,9 @@ export async function GET(req: Request, context: RouteContext): Promise<Response
       )
       credentialsPersisted = true
     } catch (err) {
-      console.warn(
-        `[communication_channels:oauth] persisting credentials failed for provider ${provider}:`,
-        err instanceof Error ? err.message : err,
+      logger.warn(
+        'persisting credentials failed for provider',
+        { provider, err },
       )
     }
   }
@@ -276,9 +279,7 @@ export async function GET(req: Request, context: RouteContext): Promise<Response
     // Same mailbox already connected via another provider — don't create a second
     // channel that double-ingests every message; send the user back with a flash.
     if (err instanceof MailboxAlreadyConnectedError) {
-      console.warn(
-        `[communication_channels:oauth] mailbox ${err.externalIdentifier} already connected via ${err.existingProviderKey}`,
-      )
+      logger.warn('mailbox already connected via another provider', { existingProviderKey: err.existingProviderKey })
       return redirectWithFlash(req, returnUrl, {
         type: 'error',
         code: 'mailbox_already_connected',
@@ -313,10 +314,9 @@ export async function GET(req: Request, context: RouteContext): Promise<Response
         input: { channelId: channel.id },
       })
     } catch (err) {
-      console.warn(
-        `[oauth-callback] best-effort pushRegister failed for ${channel.id}: ${
-          err instanceof Error ? err.message : String(err)
-        }`,
+      logger.warn(
+        'best-effort pushRegister failed for channel',
+        { channelId: channel.id, err },
       )
     }
   }

package/src/modules/communication_channels/api/post/channels/[id]/import-history/route.ts

@@ -12,6 +12,9 @@ import {
   queueImportHistorySchema,
 } from '../../../../../commands/queue-import-history'
 import { validateRouteMutationGuard } from '../../../../../lib/route-mutation-guard'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'import-history' })
 
 type RbacServiceLike = {
   loadAcl: (
@@ -169,7 +172,7 @@ export async function POST(req: Request, context: RouteContext): Promise<Respons
         { status: candidate.status },
       )
     }
-    console.error(`[import-history] failed to enqueue for channel ${id}:`, err)
+    logger.error('failed to enqueue for channel', { channelId: id, err })
     return NextResponse.json(
       { error: err instanceof Error ? err.message : 'Failed to queue import-history' },
       { status: 500 },

package/src/modules/communication_channels/api/post/channels/[id]/push/register/route.ts

@@ -9,6 +9,9 @@ import { CommunicationChannel } from '../../../../../../data/entities'
 import { ChannelAccessDeniedError, assertCanManageChannel } from '../../../../../../lib/access-control'
 import { pushRegister } from '../../../../../../commands/push-register'
 import { validateRouteMutationGuard } from '../../../../../../lib/route-mutation-guard'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'push-register' })
 
 type RbacServiceLike = {
   loadAcl: (
@@ -127,7 +130,7 @@ export async function POST(req: Request, context: RouteContext): Promise<Respons
         { status: candidate.status },
       )
     }
-    console.error(`[push-register] failed for channel ${id}:`, err)
+    logger.error('push register failed for channel', { channelId: id, err })
     return NextResponse.json(
       { error: err instanceof Error ? err.message : 'Failed to register push' },
       { status: 500 },

package/src/modules/communication_channels/api/post/webhook/[provider]/route.ts

@@ -11,6 +11,9 @@ import {
 import type { InboundMessage } from '../../../../lib/adapter'
 import type { InboundProcessorPayload } from '../../../../workers/inbound-processor'
 import type { ReactionInboundJob } from '../../../../lib/reaction-processor-types'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'inbound-webhook' })
 
 /**
  * Inbound webhook endpoint for the communication_channels hub.
@@ -201,9 +204,9 @@ export async function POST(req: Request, { params }: RouteContext): Promise<Resp
   } catch (error: unknown) {
     // Do not echo adapter/verification internals to an unauthenticated caller;
     // log the detail server-side and return a fixed, minimal message.
-    console.warn(
-      '[communication_channels] inbound webhook verification failed:',
-      error instanceof Error ? error.message : error,
+    logger.warn(
+      'inbound webhook verification failed',
+      { err: error },
     )
     return NextResponse.json({ error: 'verification_failed' }, { status: 401 })
   }

package/src/modules/communication_channels/api/post/webhooks/gmail/route.ts

@@ -12,6 +12,9 @@ import {
   getGmailPubSubVerifier,
   GmailPubSubJwtError,
 } from '../../../../lib/gmail-pubsub-jwt'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'gmail-webhook' })
 
 /**
  * Spec C § Phase C2 — Gmail Pub/Sub push webhook.
@@ -51,9 +54,7 @@ export async function POST(req: Request): Promise<Response> {
   if (!expectedAudience || !expectedEmail) {
     // Misconfiguration is operator-facing; log and 503 so Pub/Sub retries
     // briefly and the operator notices.
-    console.error(
-      '[gmail-webhook] OM_GMAIL_PUBSUB_AUDIENCE / OM_GMAIL_PUBSUB_SERVICE_ACCOUNT_EMAIL not set',
-    )
+    logger.error('OM_GMAIL_PUBSUB_AUDIENCE / OM_GMAIL_PUBSUB_SERVICE_ACCOUNT_EMAIL not set')
     return NextResponse.json({ error: 'webhook not configured' }, { status: 503 })
   }
 
@@ -131,9 +132,9 @@ export async function POST(req: Request): Promise<Response> {
     try {
       await queue.enqueue(job as unknown as Record<string, unknown>)
     } catch (err) {
-      console.error(
-        `[gmail-webhook] failed to enqueue history-sync for channel ${channel.id}:`,
-        err,
+      logger.error(
+        'failed to enqueue history-sync for channel',
+        { channelId: channel.id, err },
       )
     }
   }

package/src/modules/communication_channels/commands/connect-credential-channel.ts

@@ -4,6 +4,9 @@ import type { CommandHandler } from '@open-mercato/shared/lib/commands'
 import { registerCommand } from '@open-mercato/shared/lib/commands'
 import type { ChannelAdapterRegistry } from '../lib/registry'
 import { createConnectedChannelRow, MailboxAlreadyConnectedError } from '../lib/connect-channel'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'connect-credential-channel' })
 
 const connectCredentialChannelSchema = z.object({
   providerKey: z.string().min(1).max(64),
@@ -120,11 +123,7 @@ const connectCredentialChannelCommand: CommandHandler<
         )
         credentialsPersisted = true
       } catch (err) {
-        console.warn(
-          `[communication_channels:connect_credential] credentials persist failed for ${input.providerKey}: ${
-            err instanceof Error ? err.message : String(err)
-          }`,
-        )
+        logger.warn('credentials persist failed for provider', { providerKey: input.providerKey, err })
       }
       // Resolve the saved row id so we can link channel.credentialsRef. Best-effort.
       if (credentialsPersisted) {
@@ -225,11 +224,7 @@ const connectCredentialChannelCommand: CommandHandler<
         } catch (err) {
           // Never fail the connect on push errors — operator can manually
           // re-register via the dedicated route. Log + continue.
-          console.warn(
-            `[connect-credential-channel] best-effort pushRegister failed for ${channel.id}: ${
-              err instanceof Error ? err.message : String(err)
-            }`,
-          )
+          logger.warn('best-effort pushRegister failed for channel', { channelId: channel.id, err })
         }
       }
     }

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

@@ -7,6 +7,9 @@ import { findOneWithDecryption } from '@open-mercato/shared/lib/encryption/find'
 import { CommunicationChannel } from '../data/entities'
 import { pushUnregister } from './push-unregister'
 import { emitCommunicationChannelsEvent } from '../events'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('communication_channels').child({ component: 'delete-channel' })
 
 const deleteChannelSchema = z.object({
   channelId: z.string().uuid(),
@@ -99,11 +102,7 @@ const deleteChannelCommand: CommandHandler<DeleteChannelInput, DeleteChannelResu
           input: { channelId: channel.id },
         })
       } catch (err) {
-        console.warn(
-          `[delete-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/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' }
   }