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

package/src/modules/sales/commands/documentAddresses.ts

@@ -104,6 +104,12 @@ function applyDocumentAddressSnapshot(em: EntityManager, entity: SalesDocumentAd
   entity.quote = snapshot.documentKind === 'quote' ? em.getReference(SalesQuote, snapshot.documentId) : null
 }
 
+function assertDocumentAddressParent(entity: SalesDocumentAddress, input: { documentId: string; documentKind: 'order' | 'quote' }): void {
+  if (entity.documentId !== input.documentId || entity.documentKind !== input.documentKind) {
+    throw new CrudHttpError(404, { error: 'sales.document.address.not_found' })
+  }
+}
+
 async function emitDocumentAddressIndexSideEffects(
   ctx: { container: { resolve: (name: string) => unknown } },
   action: 'created' | 'updated' | 'deleted',
@@ -256,6 +262,7 @@ const updateDocumentAddress: CommandHandler<DocumentAddressUpdateInput, { id: st
       'sales.document.address.not_found'
     )
     ensureSameScope(entity, input.organizationId, input.tenantId)
+    assertDocumentAddressParent(entity, input)
     const document = await requireDocument(em, input.documentKind, input.documentId, input.organizationId, input.tenantId)
     if (input.documentKind === 'order') {
       await assertAddressEditable(em, {
@@ -344,6 +351,7 @@ const deleteDocumentAddress: CommandHandler<
       'sales.document.address.not_found'
     )
     ensureSameScope(entity, input.organizationId, input.tenantId)
+    assertDocumentAddressParent(entity, input)
     const document = await requireDocument(em, input.documentKind, input.documentId, input.organizationId, input.tenantId)
     if (input.documentKind === 'order') {
       await assertAddressEditable(em, {

package/src/modules/sales/commands/documents.ts

@@ -137,6 +137,9 @@ import {
 } from "@open-mercato/shared/lib/units/unitCodes";
 import type { AuthContext } from "@open-mercato/shared/lib/auth/server";
 import type { TranslateWithFallbackFn } from "@open-mercato/shared/lib/i18n/translate";
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 // CRUD events configuration for workflow triggers
 const orderCrudEvents: CrudEventsConfig<SalesOrder> = {
@@ -4742,10 +4745,7 @@ const createQuoteCommand: CommandHandler<
       }
     } catch (err) {
       // Notification creation is non-critical, don't fail the command
-      console.error(
-        "[sales.quotes.create] Failed to create notification:",
-        err,
-      );
+      logger.error("sales.quotes.create failed to create notification", { err });
     }
 
     // Emit CRUD side effects to trigger workflow event listeners
@@ -5748,10 +5748,7 @@ const createOrderCommand: CommandHandler<
       }
     } catch (err) {
       // Notification creation is non-critical, don't fail the command
-      console.error(
-        "[sales.orders.create] Failed to create notification:",
-        err,
-      );
+      logger.error("sales.orders.create failed to create notification", { err });
     }
 
     // Emit CRUD side effects to trigger workflow event listeners

package/src/modules/sales/commands/payments.ts

@@ -43,6 +43,9 @@ import { findOneWithDecryption, findWithDecryption } from '@open-mercato/shared/
 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('sales')
 
 export type PaymentAllocationSnapshot = {
   id: string
@@ -543,7 +546,7 @@ const createPaymentCommand: CommandHandler<
       }
     } catch (err) {
       // Notification creation is non-critical, don't fail the command
-      console.error('[sales.payments.create] Failed to create notification:', err)
+      logger.error('sales.payments.create Failed to create notification', { err })
     }
 
     return { paymentId: payment.id, orderTotals: totals, orderPaymentMethodIdBefore, orderPaymentMethodCodeBefore }

package/src/modules/sales/commands/shipments.ts

@@ -38,6 +38,9 @@ import type { DataEngine } from '@open-mercato/shared/lib/data/engine'
 import { emitCrudSideEffects } from '@open-mercato/shared/lib/commands/helpers'
 import type { CrudEventsConfig } from '@open-mercato/shared/lib/crud/types'
 import { findOneWithDecryption, findWithDecryption } from '@open-mercato/shared/lib/encryption/find'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const fallbackShipmentLogger = createLogger('sales').child({ component: 'shipments' })
 
 const shipmentCrudEvents: CrudEventsConfig = {
   module: 'sales',
@@ -156,8 +159,7 @@ const logShipmentDeleteScopeRejection = (
     logger.warn(payload, reason)
     return
   }
-  // eslint-disable-next-line no-console
-  console.warn(`[sales.shipments.delete] ${reason}`, payload)
+  fallbackShipmentLogger.warn(reason, payload)
 }
 
 export async function loadShipmentSnapshot(em: EntityManager, id: string): Promise<ShipmentSnapshot | null> {

package/src/modules/sales/components/AdjustmentKindSettings.tsx

@@ -25,6 +25,9 @@ import { AppearanceSelector } from '@open-mercato/core/modules/dictionaries/comp
 import { renderDictionaryColor, renderDictionaryIcon } from '@open-mercato/core/modules/dictionaries/components/dictionaryAppearance'
 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('sales')
 
 type AdjustmentKind = {
   id: string
@@ -182,7 +185,7 @@ export function AdjustmentKindSettings() {
         : []
       setItems(mapped)
     } catch (err) {
-      console.error('sales.adjustment-kinds.list failed', err)
+      logger.error('sales.adjustment-kinds.list failed', { err })
       flash(labels.loadError, 'error')
     } finally {
       setLoading(false)
@@ -265,7 +268,7 @@ export function AdjustmentKindSettings() {
       closeDialog()
       await loadItems()
     } catch (err) {
-      console.error('sales.adjustment-kinds.save failed', err)
+      logger.error('sales.adjustment-kinds.save failed', { err })
       const message = err instanceof Error ? err.message : labels.saveError
       setError(message)
     } finally {
@@ -304,7 +307,7 @@ export function AdjustmentKindSettings() {
         flash(labels.deleted, 'success')
         await loadItems()
       } catch (err) {
-        console.error('sales.adjustment-kinds.delete failed', err)
+        logger.error('sales.adjustment-kinds.delete failed', { err })
         const message = err instanceof Error ? err.message : labels.deleteError
         flash(message, 'error')
       }

package/src/modules/sales/components/DocumentNumberSettings.tsx

@@ -15,6 +15,9 @@ import {
   DEFAULT_ORDER_NUMBER_FORMAT,
   DEFAULT_QUOTE_NUMBER_FORMAT,
 } from '../lib/documentNumberTokens'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type SettingsResponse = {
   orderNumberFormat: string
@@ -115,7 +118,7 @@ export function DocumentNumberSettings() {
         flash(translations.errors.load, 'error')
       }
     } catch (err) {
-      console.error('sales.document-number-settings.load failed', err)
+      logger.error('sales.document-number-settings.load failed', { err })
       flash(translations.errors.load, 'error')
     } finally {
       setLoading(false)
@@ -164,7 +167,7 @@ export function DocumentNumberSettings() {
       setTokens(Array.isArray(call.result?.tokens) && call.result.tokens.length ? call.result.tokens : DOCUMENT_NUMBER_TOKENS)
       flash(translations.messages.saved, 'success')
     } catch (err) {
-      console.error('sales.document-number-settings.save failed', err)
+      logger.error('sales.document-number-settings.save failed', { err })
       flash(translations.errors.save, 'error')
     } finally {
       setSaving(false)

package/src/modules/sales/components/OrderEditingSettings.tsx

@@ -11,6 +11,9 @@ import { useOrganizationScopeVersion } from '@open-mercato/shared/lib/frontend/u
 import { useT } from '@open-mercato/shared/lib/i18n/context'
 
 import { Badge } from '@open-mercato/ui/primitives/badge'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type OrderStatusOption = {
   id: string
@@ -96,7 +99,7 @@ export function OrderEditingSettings() {
       setCustomerStatuses(normalizeStatusList(call.result?.orderCustomerEditableStatuses))
       setAddressStatuses(normalizeStatusList(call.result?.orderAddressEditableStatuses))
     } catch (err) {
-      console.error('sales.order-editing-settings.load failed', err)
+      logger.error('sales.order-editing-settings.load failed', { err })
       flash(translations.messages.loadError, 'error')
     } finally {
       setLoading(false)
@@ -158,7 +161,7 @@ export function OrderEditingSettings() {
       setOptions(Array.isArray(call.result?.orderStatuses) ? call.result.orderStatuses : [])
       flash(translations.messages.saved, 'success')
     } catch (err) {
-      console.error('sales.order-editing-settings.save failed', err)
+      logger.error('sales.order-editing-settings.save failed', { err })
       flash(translations.messages.saveError, 'error')
     } finally {
       setSaving(false)

package/src/modules/sales/components/PaymentMethodsSettings.tsx

@@ -28,6 +28,9 @@ import {
 } from '../lib/providers'
 import { isRecord } from '@open-mercato/shared/lib/utils'
 import { renderProviderFieldInput } from './ProviderFieldInput'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type PaymentMethodRow = {
   id: string
@@ -247,7 +250,7 @@ export function PaymentMethodsSettings() {
         })
       )
     } catch (err) {
-      console.error('sales.payment-methods.list failed', err)
+      logger.error('sales.payment-methods.list failed', { err })
       flash(translations.errors.load, 'error')
     } finally {
       setLoading(false)
@@ -306,7 +309,7 @@ export function PaymentMethodsSettings() {
       flash(translations.messages.deleted, 'success')
       await loadEntries()
     } catch (err) {
-      console.error('sales.payment-methods.delete failed', err)
+      logger.error('sales.payment-methods.delete failed', { err })
       const message =
         err instanceof Error ? err.message : translations.errors.delete
       flash(message, 'error')
@@ -344,7 +347,7 @@ export function PaymentMethodsSettings() {
       await loadEntries()
       closeDialog()
     } catch (err) {
-      console.error('sales.payment-methods.save failed', err)
+      logger.error('sales.payment-methods.save failed', { err })
       const message = err instanceof Error ? err.message : translations.errors.save
       flash(message, 'error')
     } finally {

package/src/modules/sales/components/ShippingMethodsSettings.tsx

@@ -37,6 +37,9 @@ import {
 } from '../lib/providers'
 import { isRecord } from '@open-mercato/shared/lib/utils'
 import { renderProviderFieldInput } from './ProviderFieldInput'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type ShippingMethodRow = {
   id: string
@@ -500,7 +503,7 @@ export function ShippingMethodsSettings() {
         })
       )
     } catch (err) {
-      console.error('sales.shipping-methods.list failed', err)
+      logger.error('sales.shipping-methods.list failed', { err })
       flash(translations.errors.load, 'error')
     } finally {
       setLoading(false)
@@ -564,7 +567,7 @@ export function ShippingMethodsSettings() {
       flash(translations.messages.deleted, 'success')
       await loadEntries()
     } catch (err) {
-      console.error('sales.shipping-methods.delete failed', err)
+      logger.error('sales.shipping-methods.delete failed', { err })
       const message = err instanceof Error ? err.message : translations.errors.delete
       flash(message, 'error')
     }
@@ -661,7 +664,7 @@ export function ShippingMethodsSettings() {
       await loadEntries()
       closeDialog()
     } catch (err) {
-      console.error('sales.shipping-methods.save failed', err)
+      logger.error('sales.shipping-methods.save failed', { err })
       const message = err instanceof Error ? err.message : translations.errors.save
       flash(message, 'error')
     } finally {

package/src/modules/sales/components/StatusSettings.tsx

@@ -24,6 +24,9 @@ import {
   type DictionaryTableEntry,
 } from '@open-mercato/core/modules/dictionaries/components/DictionaryTable'
 import { raiseCrudError } from '@open-mercato/ui/backend/utils/serverErrors'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type SalesStatusKind = 'order-statuses' | 'order-line-statuses' | 'shipment-statuses' | 'payment-statuses'
 
@@ -148,7 +151,7 @@ export function StatusSettings() {
       }))
       setEntriesByKind((prev) => ({ ...prev, [kind]: items }))
     } catch (err) {
-      console.error('sales.statuses.list failed', err)
+      logger.error('sales.statuses.list failed', { err })
       flash(translate('sales.config.statuses.error.load', 'Failed to load status entries.'), 'error')
     } finally {
       setLoadingKind((prev) => ({ ...prev, [kind]: false }))
@@ -240,7 +243,7 @@ export function StatusSettings() {
       flash(translate('sales.config.statuses.success.delete', 'Status deleted.'), 'success')
       await loadEntries(kind)
     } catch (err) {
-      console.error('sales.statuses.delete failed', err)
+      logger.error('sales.statuses.delete failed', { err })
       const message = err instanceof Error ? err.message : translate('sales.config.statuses.error.delete', 'Failed to delete status.')
       flash(message, 'error')
     }
@@ -301,7 +304,7 @@ export function StatusSettings() {
       closeDialog()
       await loadEntries(dialog.kind)
     } catch (err) {
-      console.error('sales.statuses.submit failed', err)
+      logger.error('sales.statuses.submit failed', { err })
       const message = err instanceof Error ? err.message : translate('sales.config.statuses.error.save', 'Failed to save status.')
       flash(message, 'error')
       throw err instanceof Error ? err : new Error(message)

package/src/modules/sales/components/TaxRatesSettings.tsx

@@ -18,6 +18,9 @@ import { useOrganizationScopeVersion } from '@open-mercato/shared/lib/frontend/u
 import { useT } from '@open-mercato/shared/lib/i18n/context'
 import { useConfirmDialog } from '@open-mercato/ui/backend/confirm-dialog'
 import { formatDateTime } from '@open-mercato/shared/lib/time'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type TaxRateRow = {
   id: string
@@ -168,7 +171,7 @@ export function TaxRatesSettings() {
         }),
       )
     } catch (err) {
-      console.error('sales.tax-rates.list failed', err)
+      logger.error('sales.tax-rates.list failed', { err })
       flash(translations.errors.load, 'error')
     } finally {
       setLoading(false)
@@ -224,7 +227,7 @@ export function TaxRatesSettings() {
       closeDialog()
       await loadEntries()
     } catch (err) {
-      console.error('sales.tax-rates.save failed', err)
+      logger.error('sales.tax-rates.save failed', { err })
       flash(translations.errors.save, 'error')
     }
   }, [dialog, translations.errors.save, translations.messages.saved, closeDialog, loadEntries])
@@ -251,7 +254,7 @@ export function TaxRatesSettings() {
       flash(translations.messages.deleted, 'success')
       await loadEntries()
     } catch (err) {
-      console.error('sales.tax-rates.delete failed', err)
+      logger.error('sales.tax-rates.delete failed', { err })
       flash(translations.errors.delete, 'error')
     }
   }, [confirm, translations.actions.deleteConfirm, translations.errors.delete, translations.messages.deleted, loadEntries])

package/src/modules/sales/components/channels/ChannelOfferForm.tsx

@@ -27,6 +27,9 @@ import { extractCustomFieldEntries } from '@open-mercato/shared/lib/crud/custom-
 import { E } from '#generated/entities.ids.generated'
 import { buildAttachmentImageUrl, slugifyAttachmentFileName } from '@open-mercato/core/modules/attachments/lib/imageUrls'
 import { cn } from '@open-mercato/shared/lib/utils'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type PriceKindSummary = {
   id: string
@@ -216,7 +219,7 @@ export function ChannelOfferForm({ channelId: lockedChannelId, offerId, mode }:
           setVariantPreviews(variants)
         }
       } catch (err) {
-        console.error('sales.channels.offer.initialHydrate', err)
+        logger.error('sales.channels.offer.initialHydrate', { err })
       }
     }
     void hydrateExistingProduct()
@@ -275,12 +278,12 @@ export function ChannelOfferForm({ channelId: lockedChannelId, offerId, mode }:
             setPriceKinds(mapItems(items))
             return
           } catch (err) {
-            console.error('sales.channels.price-kinds.fetch', { endpoint, err })
+            logger.error('sales.channels.price-kinds.fetch', { endpoint, err })
           }
         }
         setPriceKinds([])
       } catch (err) {
-        console.error('catalog.price-kinds.list', err)
+        logger.error('catalog.price-kinds.list', { err })
       }
     }
     void loadKinds()
@@ -340,7 +343,7 @@ export function ChannelOfferForm({ channelId: lockedChannelId, offerId, mode }:
             })
             attachmentCache.current.set(productId, preloadedMedia)
           } catch (err) {
-            console.error('sales.channels.offer.media.preload', err)
+            logger.error('sales.channels.offer.media.preload', { err })
           }
         }
         if (!cancelled) {
@@ -350,7 +353,7 @@ export function ChannelOfferForm({ channelId: lockedChannelId, offerId, mode }:
           setMediaOptions(preloadedMedia)
         }
       } catch (err) {
-        console.error('sales.channels.offer.load', err)
+        logger.error('sales.channels.offer.load', { err })
         if (!cancelled) setError(t('sales.channels.offers.errors.loadOffer', 'Failed to load offer.'))
       } finally {
         if (!cancelled) setLoading(false)
@@ -461,7 +464,7 @@ export function ChannelOfferForm({ channelId: lockedChannelId, offerId, mode }:
       return true
     } catch (err) {
       if (surfaceRecordConflict(err, t)) return false
-      console.error('sales.channels.pricing.remove', err)
+      logger.error('sales.channels.pricing.remove', { err })
       flash(t('sales.channels.offers.errors.removePrice', 'Failed to remove price override.'), 'error')
       return false
     }
@@ -866,7 +869,7 @@ async function syncPriceOverrides(params: {
         () => deleteCrud('catalog/prices', id),
       )
     } catch (err) {
-      console.error('catalog.prices.delete', err)
+      logger.error('catalog.prices.delete', { err })
     }
   }
 }
@@ -903,7 +906,7 @@ function ChannelSelectInput({
           code: typeof item.code === 'string' ? item.code : null,
         })))
       } catch (err) {
-        console.error('sales.channels.options', err)
+        logger.error('sales.channels.options', { err })
       }
     }
     void load()
@@ -932,7 +935,7 @@ function ChannelSelectInput({
           return [...prev, entry]
         })
       } catch (err) {
-        console.error('sales.channels.lookup', err)
+        logger.error('sales.channels.lookup', { err })
       }
     }
     void loadSingle()
@@ -1028,7 +1031,7 @@ function ProductSelectInput({
         }
       } catch (err) {
         if ((err as Error).name === 'AbortError') return
-        console.error('catalog.products.lookup', err)
+        logger.error('catalog.products.lookup', { err })
       } finally {
         if (!cancelled) setLoading(false)
       }
@@ -1358,7 +1361,7 @@ function OfferFormWatchers({
           setVariantPreviews(variants)
         }
       } catch (err) {
-        console.error('sales.channels.offer.watchers', err)
+        logger.error('sales.channels.offer.watchers', { err })
       }
     }
     void load()
@@ -1539,7 +1542,7 @@ async function loadProductMedia(productId: string): Promise<MediaOption[]> {
       return normalize(primary.result.items)
     }
   } catch (err) {
-    console.error('catalog.product-media.lookup', err)
+    logger.error('catalog.product-media.lookup', { err })
   }
   try {
     const fallback = await apiCall<AttachmentsResponse>(
@@ -1551,7 +1554,7 @@ async function loadProductMedia(productId: string): Promise<MediaOption[]> {
       return normalize(fallback.result.items)
     }
   } catch (err) {
-    console.error('attachments.lookup', err)
+    logger.error('attachments.lookup', { err })
   }
   return []
 }
@@ -1671,7 +1674,7 @@ async function resolveVariantThumbnail(
       return info
     }
   } catch (err) {
-    console.error('sales.channels.offer.variantMedia', err)
+    logger.error('sales.channels.offer.variantMedia', { err })
   }
   const empty: VariantThumbnailInfo = { attachmentId: null, thumbnailUrl: null, fileName: null }
   cache.current.set(variantId, empty)

package/src/modules/sales/components/channels/SalesChannelOffersPanel.tsx

@@ -16,6 +16,9 @@ import { surfaceRecordConflict } from '@open-mercato/ui/backend/conflicts'
 import { useGuardedMutation } from '@open-mercato/ui/backend/injection/useGuardedMutation'
 import { useT } from '@open-mercato/shared/lib/i18n/context'
 import { mapOfferRow, renderOfferPriceSummary, type OfferRow } from './offerTableUtils'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type OffersResponse = {
   items?: Array<Record<string, unknown>>
@@ -131,7 +134,7 @@ export function SalesChannelOffersPanel({ channelId, channelName }: { channelId:
       setTotal(typeof payload.total === 'number' ? payload.total : items.length)
       setTotalPages(typeof payload.totalPages === 'number' ? payload.totalPages : Math.max(1, Math.ceil(items.length / PAGE_SIZE)))
     } catch (err) {
-      console.error('sales.channels.offers', err)
+      logger.error('sales.channels.offers', { err })
       flash(t('sales.channels.offers.errors.load', 'Failed to load offers.'), 'error')
     } finally {
       setLoading(false)
@@ -158,7 +161,7 @@ export function SalesChannelOffersPanel({ channelId, channelName }: { channelId:
       setReloadToken((token) => token + 1)
     } catch (err) {
       if (surfaceRecordConflict(err, t)) return
-      console.error('sales.channels.offers.delete', err)
+      logger.error('sales.channels.offers.delete', { err })
       flash(t('sales.channels.offers.errors.delete', 'Failed to delete offer.'), 'error')
     }
   }, [mutationContext, runMutation, t])

package/src/modules/sales/components/documents/AddressesSection.tsx

@@ -28,6 +28,9 @@ import {
   type AddressValue,
 } from '@open-mercato/core/modules/customers/utils/addressFormat'
 import { Pencil, Plus, Save, Trash2 } from 'lucide-react'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type Translator = (key: string, fallback?: string, params?: Record<string, string | number>) => string
 
@@ -352,7 +355,7 @@ export function SalesDocumentAddressesSection({
         setDocumentAddresses([])
       }
     } catch (err) {
-      console.error('sales.documents.addresses.document.load', err)
+      logger.error('sales.documents.addresses.document.load', { err })
       const message = t('sales.documents.detail.addresses.loadError', 'Failed to load addresses.')
       flash(message, 'error')
       setDocumentAddressesError(message)
@@ -442,7 +445,7 @@ export function SalesDocumentAddressesSection({
           setAddressOptions([])
         }
       } catch (err) {
-        console.error('sales.documents.addresses.load', err)
+        logger.error('sales.documents.addresses.load', { err })
         const message = t('sales.documents.detail.addresses.loadError', 'Failed to load addresses.')
         setAddressesError(message)
         flash(message, 'error')
@@ -519,7 +522,7 @@ export function SalesDocumentAddressesSection({
           setAddressFormat(format)
         }
       } catch (err) {
-        console.error('sales.documents.addresses.format', err)
+        logger.error('sales.documents.addresses.format', { err })
       }
     }
     fetchAddressFormat().catch(() => {})

package/src/modules/sales/components/documents/AdjustmentDialog.tsx

@@ -26,6 +26,9 @@ import type { SalesAdjustmentKind } from '../../data/entities'
 import { E } from '#generated/entities.ids.generated'
 import { Settings } from 'lucide-react'
 import { extractCustomFieldValues, normalizeCustomFieldSubmitValue } from './customFieldHelpers'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type TaxRateOption = {
   id: string
@@ -248,7 +251,7 @@ export function AdjustmentDialog({
       setTaxRates(parsed)
       return parsed
     } catch (err) {
-      console.error('sales.tax-rates.fetch', err)
+      logger.error('sales.tax-rates.fetch', { err })
       taxRatesRef.current = []
       setTaxRates([])
       taxRatesLoadedRef.current = true

package/src/modules/sales/components/documents/AdjustmentsSection.tsx

@@ -26,6 +26,9 @@ import { AdjustmentDialog, type AdjustmentRowData, type AdjustmentSubmitPayload
 import { useT } from '@open-mercato/shared/lib/i18n/context'
 import { useOrganizationScopeDetail } from '@open-mercato/shared/lib/frontend/useOrganizationScope'
 import { extractCustomFieldValues } from './customFieldHelpers'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type AdjustmentRow = AdjustmentRowData
 
@@ -153,7 +156,7 @@ export function SalesDocumentAdjustmentsSection({
       kindLoadingRef.current = false
       return options
     } catch (err) {
-      console.error('sales.adjustment-kinds.fetch', err)
+      logger.error('sales.adjustment-kinds.fetch', { err })
       kindLoadingRef.current = false
       setKindOptions(fallbackKindOptions)
       return fallbackKindOptions
@@ -240,7 +243,7 @@ export function SalesDocumentAdjustmentsSection({
       setRows(ordered)
       if (onRowsChange) onRowsChange(ordered)
     } catch (err) {
-      console.error('sales.document.adjustments.load', err)
+      logger.error('sales.document.adjustments.load', { err })
       setError(t('sales.documents.adjustments.errorLoad', 'Failed to load adjustments.'))
     } finally {
       setLoading(false)
@@ -389,7 +392,7 @@ export function SalesDocumentAdjustmentsSection({
         }
       } catch (err) {
         if (handleSectionMutationError(err, t, () => void loadAdjustments())) return
-        console.error('sales.document.adjustments.delete', err)
+        logger.error('sales.document.adjustments.delete', { err })
         flash(t('sales.documents.adjustments.errorDelete', 'Failed to delete adjustment.'), 'error')
       }
     },

package/src/modules/sales/components/documents/ItemsSection.tsx

@@ -27,6 +27,9 @@ import type { SectionAction } from "@open-mercato/ui/backend/detail";
 import { extractCustomFieldValues } from "./customFieldHelpers";
 import { canonicalizeUnitCode } from "@open-mercato/shared/lib/units/unitCodes";
 import type { SalesLineUomSnapshot } from "../../lib/types";
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type ResolvedUnitPriceReference = {
   grossPerReference: number;
@@ -164,7 +167,7 @@ export function SalesDocumentItemsSection({
       const entries = normalizeDictionaryEntries(response.result?.items ?? [], { sort: false });
       setLineStatusMap(createDictionaryMap(entries));
     } catch (err) {
-      console.error("sales.document.line-statuses.load", err);
+      logger.error('sales.document.line-statuses.load', { err });
       setLineStatusMap({});
     }
   }, []);
@@ -315,7 +318,7 @@ export function SalesDocumentItemsSection({
         if (onItemsChange) onItemsChange([]);
       }
     } catch (err) {
-      console.error("sales.document.items.load", err);
+      logger.error('sales.document.items.load', { err });
       setError(t("sales.documents.items.errorLoad", "Failed to load items."));
       if (onItemsChange) onItemsChange([]);
     } finally {
@@ -364,7 +367,7 @@ export function SalesDocumentItemsSection({
         setShippedTotals(new Map());
       }
     } catch (err) {
-      console.error("sales.document.shipments.load", err);
+      logger.error('sales.document.shipments.load', { err });
       setShippedTotals(new Map());
     }
   }, [documentId, kind]);
@@ -476,7 +479,7 @@ export function SalesDocumentItemsSection({
         }
       } catch (err) {
         if (handleSectionMutationError(err, t, () => void loadItems())) return;
-        console.error("sales.document.items.delete", err);
+        logger.error('sales.document.items.delete', { err });
         const normalized = normalizeCrudServerError(err);
         const fallback = t(
           "sales.documents.items.errorDelete",