@open-mercato/core 0.6.6-develop.6465.1.019f0fb26f → 0.6.6-develop.6472.1.1673e7e66b

package/src/modules/catalog/backend/catalog/products/[id]/page.tsx

@@ -138,6 +138,9 @@ import {
   DialogTitle,
 } from "@open-mercato/ui/primitives/dialog";
 import { SendObjectMessageDialog } from "@open-mercato/ui/backend/messages/SendObjectMessageDialog.tsx";
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 type ProductResponse = {
   items?: Array<Record<string, unknown>>;
@@ -509,12 +512,12 @@ export default function EditCatalogProductPage({
           }
           setVariantMediaGroups(groups);
         } catch (err) {
-          console.error("catalog.variants.media.fetch failed", err);
+          logger.error('catalog.variants.media.fetch failed', { err });
           setVariantMediaGroups([]);
         }
       })();
     } catch (err) {
-      console.error("catalog.variants.fetch failed", err);
+      logger.error('catalog.variants.fetch failed', { err });
       setVariants([]);
       setVariantMediaGroups([]);
     }
@@ -540,7 +543,7 @@ export default function EditCatalogProductPage({
           thumbnailUrl: item.thumbnailUrl ?? undefined,
         }));
       } catch (err) {
-        console.error("attachments.fetch failed", err);
+        logger.error('attachments.fetch failed', { err });
         return [];
       }
     },
@@ -566,7 +569,7 @@ export default function EditCatalogProductPage({
             .filter((item): item is TaxRateSummary => item !== null),
         );
       } catch (err) {
-        console.error("sales.tax-rates.fetch failed", err);
+        logger.error('sales.tax-rates.fetch failed', { err });
         setTaxRates([]);
       }
     };
@@ -776,7 +779,7 @@ export default function EditCatalogProductPage({
         }
         await loadVariants(productId!);
       } catch (err) {
-        console.error("catalog.products.edit.load failed", err);
+        logger.error('catalog.products.edit.load failed', { err });
         if (!cancelled) {
           const message =
             err instanceof Error && err.message
@@ -814,7 +817,7 @@ export default function EditCatalogProductPage({
           setPriceKinds(summaries);
         }
       } catch (err) {
-        console.error("catalog.price-kinds.fetch failed", err);
+        logger.error('catalog.price-kinds.fetch failed', { err });
         if (!cancelled) {
           setPriceKinds([]);
         }
@@ -1347,7 +1350,7 @@ export default function EditCatalogProductPage({
             );
           }
         } catch (err) {
-          console.error("catalog.products.edit.offers.delete", err);
+          logger.error('catalog.products.edit.offers.delete', { err });
           throw createCrudFormError(
             t(
               "catalog.products.edit.offers.deleteError",
@@ -1889,7 +1892,7 @@ function ProductOptionsSection({ values, setValue }: ProductFormGroupProps) {
         setSchemaTemplates([]);
       }
     } catch (err) {
-      console.error("catalog.option-schemas.list failed", err);
+      logger.error('catalog.option-schemas.list failed', { err });
       setSchemaTemplates([]);
     } finally {
       setSchemaLoading(false);
@@ -1917,7 +1920,7 @@ function ProductOptionsSection({ values, setValue }: ProductFormGroupProps) {
         void loadSchemas();
       } catch (err) {
         if (surfaceRecordConflict(err, t)) { void loadSchemas(); return; }
-        console.error("catalog.option-schemas.delete failed", err);
+        logger.error('catalog.option-schemas.delete failed', { err });
       }
     },
     [loadSchemas, schemaTemplates, t],
@@ -2310,7 +2313,7 @@ function ProductVariantsSection({
         onVariantDeleted(variant.id);
       } catch (err) {
         if (surfaceRecordConflict(err, t)) return;
-        console.error("catalog.products.edit.variants.delete", err);
+        logger.error('catalog.products.edit.variants.delete', { err });
         flash(
           t("catalog.variants.form.deleteError", "Failed to delete variant."),
           "error",
@@ -2359,7 +2362,7 @@ function ProductVariantsSection({
       );
       if (onVariantsReload) await onVariantsReload();
     } catch (err) {
-      console.error("catalog.products.edit.variantList.generate", err);
+      logger.error('catalog.products.edit.variantList.generate', { err });
       flash(
         t(
           "catalog.products.edit.variantList.generateError",
@@ -3324,7 +3327,7 @@ function SaveSchemaDialog({
       await onSubmit(name);
       onOpenChange(false);
     } catch (err) {
-      console.error("schema.save.failed", err);
+      logger.error('schema.save.failed', { err });
     } finally {
       setSaving(false);
     }

package/src/modules/catalog/backend/catalog/products/[productId]/variants/[variantId]/page.tsx

@@ -47,6 +47,9 @@ import type { ProductMediaItem } from '@open-mercato/core/modules/catalog/compon
 import { buildAttachmentImageUrl, slugifyAttachmentFileName } from '@open-mercato/core/modules/attachments/lib/imageUrls'
 import { fetchOptionSchemaTemplate } from '../../../optionSchemaClient'
 import CreateVariantPage from '../create/page'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 type VariantResponse = {
   items?: Array<Record<string, unknown>>
@@ -159,7 +162,7 @@ export default function EditVariantPage({ params }: { params?: { productId?: str
         const items = Array.isArray(payload.items) ? payload.items : []
         setPriceKinds(items.map((item) => normalizePriceKindSummary(item)).filter((item): item is PriceKindSummary => !!item))
       } catch (err) {
-        console.error('catalog.price-kinds.fetch failed', err)
+        logger.error('catalog.price-kinds.fetch failed', { err })
         setPriceKinds([])
       }
     }
@@ -181,7 +184,7 @@ export default function EditVariantPage({ params }: { params?: { productId?: str
             .filter((item): item is TaxRateSummary => item !== null),
         )
       } catch (err) {
-        console.error('sales.tax-rates.fetch failed', err)
+        logger.error('sales.tax-rates.fetch failed', { err })
         setTaxRates([])
       }
     }
@@ -385,7 +388,7 @@ export default function EditVariantPage({ params }: { params?: { productId?: str
           })
         }
       } catch (err) {
-        console.error('catalog.variants.load.failed', err)
+        logger.error('catalog.variants.load.failed', { err })
         if (!cancelled) {
           const message = err instanceof Error && err.message ? err.message : t('catalog.variants.form.errors.load', 'Failed to load variant.')
           setError(message)
@@ -740,7 +743,7 @@ async function fetchVariantAttachments(variantId: string): Promise<ProductMediaI
     if (!res.ok) return []
     return Array.isArray(res.result?.items) ? res.result?.items ?? [] : []
   } catch (err) {
-    console.error('catalog.variants.attachments.load', err)
+    logger.error('catalog.variants.attachments.load', { err })
     return []
   }
 }
@@ -765,7 +768,7 @@ async function loadVariantPrices(variantId: string, priceKinds: PriceKindSummary
       page += 1
     }
   } catch (err) {
-    console.error('catalog.variants.prices.load', err)
+    logger.error('catalog.variants.prices.load', { err })
   }
   return drafts
 }
@@ -819,7 +822,7 @@ async function syncVariantPricesUpdate({
             () => deleteCrud('catalog/prices', existingId),
           )
         } catch (err) {
-          console.error('catalog.prices.delete', err)
+          logger.error('catalog.prices.delete', { err })
         }
       }
       continue

package/src/modules/catalog/backend/catalog/products/[productId]/variants/create/page.tsx

@@ -38,6 +38,9 @@ import {
 import type { ProductMediaItem } from '@open-mercato/core/modules/catalog/components/products/ProductMediaManager'
 import { buildAttachmentImageUrl, slugifyAttachmentFileName } from '@open-mercato/core/modules/attachments/lib/imageUrls'
 import { fetchOptionSchemaTemplate } from '../../../optionSchemaClient'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 type ProductResponse = {
   items?: Array<{
@@ -83,7 +86,7 @@ export default function CreateVariantPage({ params }: { params?: { productId?: s
         const items = Array.isArray(payload.items) ? payload.items : []
         setPriceKinds(items.map((item) => normalizePriceKindSummary(item)).filter((item): item is PriceKindSummary => !!item))
       } catch (err) {
-        console.error('catalog.price-kinds.fetch failed', err)
+        logger.error('catalog.price-kinds.fetch failed', { err })
         setPriceKinds([])
       }
     }
@@ -121,7 +124,7 @@ export default function CreateVariantPage({ params }: { params?: { productId?: s
           }),
         )
       } catch (err) {
-        console.error('sales.tax-rates.fetch failed', err)
+        logger.error('sales.tax-rates.fetch failed', { err })
         setTaxRates([])
       }
     }
@@ -191,7 +194,7 @@ export default function CreateVariantPage({ params }: { params?: { productId?: s
           setInitialValues(base)
         }
       } catch (err) {
-        console.error('catalog.variants.loadProduct failed', err)
+        logger.error('catalog.variants.loadProduct failed', { err })
         if (!cancelled) {
           const message = err instanceof Error && err.message ? err.message : t('catalog.variants.form.errors.load', 'Failed to load product context.')
           setError(message)

package/src/modules/catalog/backend/catalog/products/create/page.tsx

@@ -96,6 +96,9 @@ import {
   normalizeProductConversionInputs,
   type ProductUnitConversionInput,
 } from "@open-mercato/core/modules/catalog/components/products/productFormUtils";
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 const productFormTypedSchema =
   productFormSchema as unknown as ZodType<ProductFormValues>;
@@ -268,7 +271,7 @@ export default function CreateCatalogProductPage() {
             .filter((item): item is PriceKindSummary => item !== null),
         );
       } catch (err) {
-        console.error("catalog.price-kinds.fetch failed", err);
+        logger.error('catalog.price-kinds.fetch failed', { err });
         setPriceKinds([]);
       }
     };
@@ -317,7 +320,7 @@ export default function CreateCatalogProductPage() {
           }),
         );
       } catch (err) {
-        console.error("sales.tax-rates.fetch failed", err);
+        logger.error('sales.tax-rates.fetch failed', { err });
         setTaxRates([]);
       }
     };
@@ -775,10 +778,7 @@ export default function CreateCatalogProductPage() {
                   { fallback: null },
                 );
                 if (!transfer.ok) {
-                  console.error(
-                    "attachments.transfer.failed",
-                    transfer.result?.error,
-                  );
+                  logger.error("attachments.transfer.failed", { err: transfer.result?.error });
                 }
               }
 

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