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

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

@@ -17,6 +17,9 @@ import { Spinner } from '@open-mercato/ui/primitives/spinner'
 import { E } from '#generated/entities.ids.generated'
 import { useT } from '@open-mercato/shared/lib/i18n/context'
 import { normalizeCustomFieldSubmitValue, extractCustomFieldValues } from './customFieldHelpers'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 export type PaymentTotals = {
   paidTotalAmount?: number | null
@@ -153,7 +156,7 @@ export function PaymentDialog({
         if (!query) setPaymentMethods([])
         return []
       } catch (err) {
-        console.error('sales.payments.methods.load', err)
+        logger.error('sales.payments.methods.load', { err })
         return []
       } finally {
         setMethodsLoading(false)
@@ -224,7 +227,7 @@ export function PaymentDialog({
       setDocumentStatuses(mapped)
       return mapped
     } catch (err) {
-      console.error('sales.payments.statuses.load', err)
+      logger.error('sales.payments.statuses.load', { err })
       setDocumentStatuses([])
       return []
     } finally {
@@ -259,7 +262,7 @@ export function PaymentDialog({
       setPaymentStatuses(mapped)
       return mapped
     } catch (err) {
-      console.error('sales.payments.statuses.load', err)
+      logger.error('sales.payments.statuses.load', { err })
       setPaymentStatuses([])
       return []
     } finally {

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

@@ -18,6 +18,9 @@ import { emitSalesDocumentTotalsRefresh } from '@open-mercato/core/modules/sales
 import { PaymentDialog, type PaymentFormData, type PaymentTotals } from './PaymentDialog'
 import { extractCustomFieldValues } from './customFieldHelpers'
 import { Plus } from 'lucide-react'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type PaymentRow = {
   id: string
@@ -161,7 +164,7 @@ export function SalesDocumentPaymentsSection({
         onPaymentsChangeRef.current?.([])
       }
     } catch (err) {
-      console.error('sales.payments.list', err)
+      logger.error('sales.payments.list', { err })
       setError(t('sales.documents.payments.errorLoad', 'Failed to load payments.'))
       onPaymentsChangeRef.current?.([])
     } finally {
@@ -242,7 +245,7 @@ export function SalesDocumentPaymentsSection({
         if (handleSectionMutationError(err, t, () => void loadPayments())) {
           return
         }
-        console.error('sales.payments.delete', err)
+        logger.error('sales.payments.delete', { err })
         flash(t('sales.documents.payments.errorDelete', 'Failed to delete payment.'), 'error')
       }
     },

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

@@ -57,6 +57,9 @@ import {
   type AddressValue,
 } from '@open-mercato/core/modules/customers/utils/addressFormat'
 import { AddressEditor, type AddressEditorDraft } from '@open-mercato/core/modules/customers/components/AddressEditor'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type DocumentKind = 'quote' | 'order'
 
@@ -188,7 +191,7 @@ function CustomerQuickCreate({ t, onCreated }: CustomerQuickCreateProps) {
         })
         closeDialog()
       } catch (err) {
-        console.error('sales.documents.quickCreate.person', err)
+        logger.error('sales.documents.quickCreate.person', { err })
         const message =
           err instanceof Error && err.message
             ? err.message
@@ -232,7 +235,7 @@ function CustomerQuickCreate({ t, onCreated }: CustomerQuickCreateProps) {
         })
         closeDialog()
       } catch (err) {
-        console.error('sales.documents.quickCreate.company', err)
+        logger.error('sales.documents.quickCreate.company', { err })
         const message =
           err instanceof Error && err.message
             ? err.message
@@ -452,7 +455,7 @@ function DocumentNumberField({ value, setValue, values, t }: DocumentNumberField
         setError(call.result?.error || t('sales.documents.form.errors.numberGenerate', 'Could not generate a document number.'))
       }
     } catch (err) {
-      console.error('sales.documents.generateNumber', err)
+      logger.error('sales.documents.generateNumber', { err })
       setError(t('sales.documents.form.errors.numberGenerate', 'Could not generate a document number.'))
     } finally {
       setLoading(false)
@@ -641,7 +644,7 @@ function CustomerGroupComponent({ values, setValue, t, customers, setCustomers,
               if (primary) {
                 setValue('shippingAddressId', primary.id)
               }
-            }).catch((err) => { console.error('sales.documents.autoSelectAddress', err) })
+            }).catch((err) => { logger.error('sales.documents.autoSelectAddress', { err }) })
             if (next) {
               const match = customers.find((entry) => entry.id === next)
               const possibleEmail =
@@ -698,7 +701,7 @@ function CustomerGroupComponent({ values, setValue, t, customers, setCustomers,
                   if (primary) {
                     setValue('shippingAddressId', primary.id)
                   }
-                }).catch((err) => { console.error('sales.documents.autoSelectAddress', err) })
+                }).catch((err) => { logger.error('sales.documents.autoSelectAddress', { err }) })
                 if (email && !values.customerEmail) {
                   setValue('customerEmail', email)
                 }
@@ -802,7 +805,7 @@ export function SalesDocumentForm({ onCreated, isSubmitting = false, initialKind
       }
       return []
     } catch (err) {
-      console.error('sales.documents.currency', err)
+      logger.error('sales.documents.currency', { err })
       return []
     }
   }, [currencyDictionary, refetchCurrencyDictionary])
@@ -822,7 +825,7 @@ export function SalesDocumentForm({ onCreated, isSubmitting = false, initialKind
       setCustomers(merged)
       return merged
     } catch (err) {
-      console.error('sales.documents.loadCustomers', err)
+      logger.error('sales.documents.loadCustomers', { err })
       flash(t('sales.documents.form.errors.customers', 'Failed to load customers.'), 'error')
       return []
     } finally {
@@ -850,7 +853,7 @@ export function SalesDocumentForm({ onCreated, isSubmitting = false, initialKind
         }
         return email ?? null
       } catch (err) {
-        console.error('sales.documents.fetchCustomerEmail', err)
+        logger.error('sales.documents.fetchCustomerEmail', { err })
         return null
       }
     },
@@ -882,7 +885,7 @@ export function SalesDocumentForm({ onCreated, isSubmitting = false, initialKind
         return []
       }
     } catch (err) {
-      console.error('sales.documents.loadChannels', err)
+      logger.error('sales.documents.loadChannels', { err })
       setChannels([])
       return []
     } finally {
@@ -902,7 +905,7 @@ export function SalesDocumentForm({ onCreated, isSubmitting = false, initialKind
         }
       }
     } catch (err) {
-      console.error('sales.documents.loadDefaultCurrency', err)
+      logger.error('sales.documents.loadDefaultCurrency', { err })
     }
   }, [])
 
@@ -974,7 +977,7 @@ export function SalesDocumentForm({ onCreated, isSubmitting = false, initialKind
       }
     } catch (err) {
       if ((err as DOMException)?.name !== 'AbortError') {
-        console.error('sales.documents.loadAddresses', err)
+        logger.error('sales.documents.loadAddresses', { err })
       }
       if (addressRequestRef.current === requestId) {
         setAddressOptions([])
@@ -1039,7 +1042,7 @@ export function SalesDocumentForm({ onCreated, isSubmitting = false, initialKind
           setAddressFormat(format)
         }
       } catch (err) {
-        console.error('sales.documents.addressFormat', err)
+        logger.error('sales.documents.addressFormat', { err })
       } finally {
       }
     }

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

@@ -24,6 +24,9 @@ import {
   normalizeDictionaryEntries,
 } from '@open-mercato/core/modules/dictionaries/components/dictionaryAppearance'
 import { SALES_DOCUMENT_NUMBER_COLUMN_META } from './salesDocumentsColumns'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type SalesDocumentKind = 'order' | 'quote'
 
@@ -257,7 +260,7 @@ export function SalesDocumentsTable({ kind }: { kind: SalesDocumentKind }) {
       const entries = normalizeDictionaryEntries(response.result?.items ?? [], { sort: false })
       setStatusMap(createDictionaryMap(entries))
     } catch (err) {
-      console.error('sales.documents.statuses.load', err)
+      logger.error('sales.documents.statuses.load', { err })
       setStatusMap({})
     }
   }, [])
@@ -496,7 +499,7 @@ export function SalesDocumentsTable({ kind }: { kind: SalesDocumentKind }) {
       setTotalPages(pages)
       setCacheStatus(call.cacheStatus ?? null)
     } catch (err) {
-      console.error('sales.documents.list', err)
+      logger.error('sales.documents.list', { err })
       flash(t('sales.documents.list.errors.load', 'Failed to load documents.'), 'error')
     } finally {
       setLoading(false)
@@ -561,7 +564,7 @@ export function SalesDocumentsTable({ kind }: { kind: SalesDocumentKind }) {
           handleRefresh()
         }
       } catch (err) {
-        console.error('sales.documents.delete', err)
+        logger.error('sales.documents.delete', { err })
         flash(t('sales.documents.list.table.deleteError', 'Failed to delete document.'), 'error')
       }
     },

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

@@ -25,6 +25,9 @@ import { formatMoney, normalizeNumber } from './lineItemUtils'
 import type { OrderLine, ShipmentRow } from './shipmentTypes'
 import { formatAddressString, type AddressFormatStrategy, type AddressValue } from '@open-mercato/core/modules/customers/utils/addressFormat'
 import { normalizeCustomFieldSubmitValue, extractCustomFieldValues } from './customFieldHelpers'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type ShippingMethodOption = {
   id: string
@@ -498,7 +501,7 @@ export function ShipmentDialog({
       setAddressError(null)
       return mapped
     } catch (err) {
-      console.error('sales.shipments.addresses.load', err)
+      logger.error('sales.shipments.addresses.load', { err })
       setAddressError(
         t('sales.documents.shipments.addressLoadError', 'Failed to load addresses.'),
       )
@@ -583,7 +586,7 @@ export function ShipmentDialog({
         }
         return options
       } catch (err) {
-        console.error('sales.shipments.shipping-methods.load', err)
+        logger.error('sales.shipments.shipping-methods.load', { err })
         return []
       } finally {
         if (applyLoadingState) setShippingMethodLoading(false)
@@ -685,7 +688,7 @@ export function ShipmentDialog({
       setDocumentStatuses(mapped)
       return mapped
     } catch (err) {
-      console.error('sales.shipments.statuses.load', err)
+      logger.error('sales.shipments.statuses.load', { err })
       setDocumentStatuses([])
       return []
     } finally {
@@ -720,7 +723,7 @@ export function ShipmentDialog({
       setLineStatuses(mapped)
       return mapped
     } catch (err) {
-      console.error('sales.shipments.line-statuses.load', err)
+      logger.error('sales.shipments.line-statuses.load', { err })
       setLineStatuses([])
       return []
     } finally {
@@ -749,7 +752,7 @@ export function ShipmentDialog({
       )
       return mapped
     } catch (err) {
-      console.error('sales.shipments.statuses.load', err)
+      logger.error('sales.shipments.statuses.load', { err })
       setShipmentStatuses([])
       return []
     } finally {
@@ -1172,7 +1175,7 @@ export function ShipmentDialog({
               )
               emitSalesDocumentTotalsRefresh({ documentId: orderId, kind: 'order' })
             } catch (err) {
-              console.warn('sales.shipments.adjustment.create', err)
+              logger.warn('sales.shipments.adjustment.create', { err })
               flash(
                 t(
                   'sales.documents.shipments.shippingAdjustmentError',
@@ -1215,7 +1218,7 @@ export function ShipmentDialog({
           try {
             await onAddComment(note)
           } catch (err) {
-            console.warn('sales.shipments.comment', err)
+            logger.warn('sales.shipments.comment', { err })
           }
         }
         await onSaved()

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

@@ -23,6 +23,9 @@ import { ShipmentDialog } from './ShipmentDialog'
 import { handleSectionMutationError, readRowUpdatedAt } from './optimisticLock'
 import { extractCustomFieldValues } from './customFieldHelpers'
 import type { OrderLine, ShipmentRow, ShipmentItem } from './shipmentTypes'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 const ADDRESS_SNAPSHOT_KEY = 'shipmentAddressSnapshot'
 const ADDRESS_FORMAT: 'line_first' = 'line_first'
@@ -353,7 +356,7 @@ export function SalesShipmentsSection({
         .filter((entry): entry is ShipmentRow => Boolean(entry))
       setShipments(mapped)
     } catch (err) {
-      console.error('sales.shipments.load', err)
+      logger.error('sales.shipments.load', { err })
       setError(t('sales.documents.shipments.errorLoad', 'Failed to load shipments.'))
     } finally {
       setLoading(false)
@@ -425,7 +428,7 @@ export function SalesShipmentsSection({
         if (handleSectionMutationError(err, t, () => void loadShipments())) {
           return
         }
-        console.error('sales.shipments.delete', err)
+        logger.error('sales.shipments.delete', { err })
         flash(t('sales.documents.shipments.errorDelete', 'Failed to delete shipment.'), 'error')
       }
     },

package/src/modules/sales/frontend/quote/[token]/page.tsx

@@ -7,6 +7,9 @@ import { Spinner } from '@open-mercato/ui/primitives/spinner'
 import { EmptyState } from '@open-mercato/ui/primitives/empty-state'
 import { SearchX } from 'lucide-react'
 import { useT } from '@open-mercato/shared/lib/i18n/context'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type PublicQuoteResponse = {
   quote: {
@@ -65,7 +68,7 @@ export default function QuotePublicPage({ params }: { params: { token: string }
         if (!mounted) return
         setData(call.result ?? null)
       } catch (err) {
-        console.error('sales.quotes.public.load', err)
+        logger.error('sales.quotes.public.load', { err })
         if (!mounted) return
         setError(t('sales.quotes.public.failed'))
       } finally {
@@ -90,7 +93,7 @@ export default function QuotePublicPage({ params }: { params: { token: string }
       })
       setAcceptedOrder(call.result ?? null)
     } catch (err) {
-      console.error('sales.quotes.accept', err)
+      logger.error('sales.quotes.accept', { err })
       setError(t('sales.quotes.public.acceptFailed'))
     } finally {
       setAccepting(false)

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

@@ -36,6 +36,9 @@ import {
   CustomerPersonProfile,
 } from '@open-mercato/core/modules/customers/data/entities'
 import { findWithDecryption } from '@open-mercato/shared/lib/encryption/find'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type ExampleAddress = {
   role: 'billing' | 'shipping'
@@ -1074,7 +1077,7 @@ async function loadCustomerLookups(
 
   for (const lookup of lookups) {
     if (!map.has(lookup.key)) {
-      console.warn(`[sales.examples] Customer "${lookup.displayName}" not found; skipping seeded linkage.`)
+      logger.warn('sales.examples customer not found; skipping seeded linkage', { displayName: lookup.displayName })
     }
   }
 
@@ -1278,7 +1281,7 @@ export async function seedSalesExamples(
     const quoteId = randomUUID()
     const customer = seed.customerKey ? customerLookups.get(seed.customerKey) ?? null : null
     if (seed.customerKey && !customer) {
-      console.warn(`[sales.examples] Skipping quote ${seed.quoteNumber} because customer "${seed.customerKey}" is missing.`)
+      logger.warn('sales.examples skipping quote because customer is missing', { quoteNumber: seed.quoteNumber, customerKey: seed.customerKey })
       continue
     }
     const shippingMethod = seed.shippingMethodCode
@@ -1292,9 +1295,7 @@ export async function seedSalesExamples(
     const lineSnapshots = seed.lines.map((line) => {
       const catalogRef = resolveCatalogLine(line, catalogLookups)
       if ((line.kind ?? 'product') === 'product' && !catalogRef.productId) {
-        console.warn(
-          `[sales.examples] Missing catalog match for quote line "${line.name ?? line.variantSku ?? line.productHandle ?? 'unknown'}".`
-        )
+        logger.warn('sales.examples missing catalog match for quote line', { line: line.name ?? line.variantSku ?? line.productHandle ?? 'unknown' })
       }
       const unitPriceNet = line.unitPriceNet ?? catalogRef.unitPriceNet ?? 0
       const unitPriceGross = line.unitPriceGross ?? catalogRef.unitPriceGross ?? unitPriceNet
@@ -1498,7 +1499,7 @@ export async function seedSalesExamples(
     const orderId = randomUUID()
     const customer = seed.customerKey ? customerLookups.get(seed.customerKey) ?? null : null
     if (seed.customerKey && !customer) {
-      console.warn(`[sales.examples] Skipping order ${seed.orderNumber} because customer "${seed.customerKey}" is missing.`)
+      logger.warn('sales.examples skipping order because customer is missing', { orderNumber: seed.orderNumber, customerKey: seed.customerKey })
       continue
     }
     const shippingMethod = seed.shippingMethodCode
@@ -1512,9 +1513,7 @@ export async function seedSalesExamples(
     const lineSnapshots = seed.lines.map((line) => {
       const catalogRef = resolveCatalogLine(line, catalogLookups)
       if ((line.kind ?? 'product') === 'product' && !catalogRef.productId) {
-        console.warn(
-          `[sales.examples] Missing catalog match for order line "${line.name ?? line.variantSku ?? line.productHandle ?? 'unknown'}".`
-        )
+        logger.warn('sales.examples missing catalog match for order line', { line: line.name ?? line.variantSku ?? line.productHandle ?? 'unknown' })
       }
       const unitPriceNet = line.unitPriceNet ?? catalogRef.unitPriceNet ?? 0
       const unitPriceGross = line.unitPriceGross ?? catalogRef.unitPriceGross ?? unitPriceNet

package/src/modules/sales/subscribers/quote-expiring-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('sales')
 
 export const metadata = {
   event: 'sales.quote.expiring',
@@ -48,6 +51,6 @@ export default async function handle(payload: QuoteExpiringPayload, ctx: Resolve
       organizationId: payload.organizationId ?? null,
     })
   } catch (err) {
-    console.error('[sales:quote-expiring-notification] Failed to create notification:', err)
+    logger.error('sales.quote-expiring-notification Failed to create notification', { err })
   }
 }

package/src/modules/sales/subscribers/reconcileOnAddressDelete.ts

@@ -1,5 +1,8 @@
 import type { EntityManager } from '@mikro-orm/postgresql'
 import { SalesOrder, SalesQuote } from '@open-mercato/core/modules/sales/data/entities'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 export const metadata = {
   event: 'customers.address.deleted',
@@ -42,6 +45,6 @@ export default async function handle(
       ),
     ])
   } catch (err) {
-    console.error('[sales:reconcile-on-address-delete] Failed to null address references on sales documents:', err)
+    logger.error('sales.reconcile-on-address-delete Failed to null address references on sales documents', { err })
   }
 }

package/src/modules/sales/subscribers/reconcileOnCompanyDelete.ts

@@ -1,5 +1,8 @@
 import type { EntityManager } from '@mikro-orm/postgresql'
 import { SalesOrder, SalesQuote } from '@open-mercato/core/modules/sales/data/entities'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 export const metadata = {
   event: 'customers.company.deleted',
@@ -32,6 +35,6 @@ export default async function handle(
       ),
     ])
   } catch (err) {
-    console.error('[sales:reconcile-on-company-delete] Failed to null customerEntityId on sales documents:', err)
+    logger.error('sales.reconcile-on-company-delete Failed to null customerEntityId on sales documents', { err })
   }
 }

package/src/modules/sales/subscribers/reconcileOnCustomerDelete.ts

@@ -1,5 +1,8 @@
 import type { EntityManager } from '@mikro-orm/postgresql'
 import { SalesOrder, SalesQuote } from '@open-mercato/core/modules/sales/data/entities'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 export const metadata = {
   event: 'customers.person.deleted',
@@ -32,6 +35,6 @@ export default async function handle(
       ),
     ])
   } catch (err) {
-    console.error('[sales:reconcile-on-person-delete] Failed to null customerEntityId on sales documents:', err)
+    logger.error('sales.reconcile-on-person-delete Failed to null customerEntityId on sales documents', { err })
   }
 }

package/src/modules/sales/widgets/dashboard/makeDashboardWidgetRoute.ts

@@ -11,6 +11,9 @@ import { findAndCountWithDecryption } from '@open-mercato/shared/lib/encryption/
 import { resolveDateRange } from '@open-mercato/ui/backend/date-range'
 import type { DatePeriodOption } from '../../api/dashboard/widgets/helpers'
 import { resolveWidgetScope, type WidgetScopeContext } from '@open-mercato/core/modules/dashboards/lib/widgetScope'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 const WIDGET_CACHE_TTL = 120_000
 const WIDGET_CACHE_SEGMENT_TTL = 86_400_000
@@ -155,7 +158,7 @@ export function makeDashboardWidgetRoute<TEntity extends object, TItem extends R
             return NextResponse.json(cached)
           }
         } catch (err) {
-          console.debug('[widget-cache] read failed', err)
+          logger.debug('widget-cache read failed', { err })
         }
       }
 
@@ -198,7 +201,7 @@ export function makeDashboardWidgetRoute<TEntity extends object, TItem extends R
             { ttl: WIDGET_CACHE_SEGMENT_TTL, tags: ['widget-data'] },
           ))
         } catch (err) {
-          console.debug('[widget-cache] write failed', err)
+          logger.debug('widget-cache write failed', { err })
         }
       }
 
@@ -207,7 +210,7 @@ export function makeDashboardWidgetRoute<TEntity extends object, TItem extends R
       if (isCrudHttpError(err)) {
         return NextResponse.json(err.body, { status: err.status })
       }
-      console.error(`${config.errorPrefix} failed`, err)
+      logger.error('dashboard widget route failed', { errorPrefix: config.errorPrefix, err })
       return NextResponse.json(
         { error: translate(`${config.errorPrefix}.error`, config.openApi.errorFallback) },
         { status: 500 },