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

package/src/modules/resources/commands/activities.ts

@@ -7,7 +7,6 @@ import {
   emitCrudUndoSideEffects,
   buildChanges,
   requireId,
-  normalizeAuthorUserId,
 } from '@open-mercato/shared/lib/commands/helpers'
 import type { DataEngine } from '@open-mercato/shared/lib/data/engine'
 import type { EntityManager } from '@mikro-orm/postgresql'
@@ -30,7 +29,7 @@ import {
   type ResourcesResourceActivityUpdateInput,
 } from '../data/validators'
 import { resourcesResourceActivityCrudEvents } from '../lib/crud'
-import { ensureOrganizationScope, ensureTenantScope, extractUndoPayload, requireResource } from './shared'
+import { ensureOrganizationScope, ensureTenantScope, extractUndoPayload, requireResource, resolveResourceAuthorUserId } from './shared'
 import { E } from '#generated/entities.ids.generated'
 
 const ACTIVITY_ENTITY_ID = E.resources.resources_resource_activity
@@ -117,12 +116,15 @@ const createActivityCommand: CommandHandler<ResourcesResourceActivityCreateInput
     const { parsed, custom } = parseWithCustomFields(resourcesResourceActivityCreateSchema, rawInput)
     ensureTenantScope(ctx, parsed.tenantId)
     ensureOrganizationScope(ctx, parsed.organizationId)
-    const normalizedAuthor = normalizeAuthorUserId(parsed.authorUserId, ctx.auth)
 
     const em = (ctx.container.resolve('em') as EntityManager).fork()
     const resource = await requireResource(em, parsed.entityId, 'Resource not found')
     ensureTenantScope(ctx, resource.tenantId)
     ensureOrganizationScope(ctx, resource.organizationId)
+    const normalizedAuthor = await resolveResourceAuthorUserId(em, parsed.authorUserId, ctx, {
+      tenantId: resource.tenantId,
+      organizationId: resource.organizationId,
+    })
 
     const activity = em.create(ResourcesResourceActivity, {
       organizationId: parsed.organizationId,
@@ -287,7 +289,6 @@ const updateActivityCommand: CommandHandler<ResourcesResourceActivityUpdateInput
     if (parsed.subject !== undefined) activity.subject = parsed.subject ?? null
     if (parsed.body !== undefined) activity.body = parsed.body ?? null
     if (parsed.occurredAt !== undefined) activity.occurredAt = parsed.occurredAt ?? null
-    if (parsed.authorUserId !== undefined) activity.authorUserId = parsed.authorUserId ?? null
     if (parsed.appearanceIcon !== undefined) activity.appearanceIcon = parsed.appearanceIcon ?? null
     if (parsed.appearanceColor !== undefined) activity.appearanceColor = parsed.appearanceColor ?? null
 

package/src/modules/resources/commands/comments.ts

@@ -1,6 +1,6 @@
 import { registerCommand } from '@open-mercato/shared/lib/commands'
 import type { CommandHandler } from '@open-mercato/shared/lib/commands'
-import { emitCrudSideEffects, emitCrudUndoSideEffects, buildChanges, requireId, normalizeAuthorUserId } from '@open-mercato/shared/lib/commands/helpers'
+import { emitCrudSideEffects, emitCrudUndoSideEffects, buildChanges, requireId } from '@open-mercato/shared/lib/commands/helpers'
 import type { DataEngine } from '@open-mercato/shared/lib/data/engine'
 import type { EntityManager } from '@mikro-orm/postgresql'
 import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
@@ -15,7 +15,7 @@ import {
 } from '../data/validators'
 import { resourcesResourceCommentCrudEvents } from '../lib/crud'
 import { makeCreateRedo } from '@open-mercato/shared/lib/commands/redo'
-import { ensureOrganizationScope, ensureTenantScope, extractUndoPayload, requireResource } from './shared'
+import { ensureOrganizationScope, ensureTenantScope, extractUndoPayload, requireResource, resolveResourceAuthorUserId } from './shared'
 import { E } from '#generated/entities.ids.generated'
 
 const commentCrudIndexer: CrudIndexerConfig<ResourcesResourceComment> = {
@@ -62,12 +62,15 @@ const createCommentCommand: CommandHandler<
     const parsed = resourcesResourceCommentCreateSchema.parse(rawInput)
     ensureTenantScope(ctx, parsed.tenantId)
     ensureOrganizationScope(ctx, parsed.organizationId)
-    const normalizedAuthor = normalizeAuthorUserId(parsed.authorUserId, ctx.auth)
 
     const em = (ctx.container.resolve('em') as EntityManager).fork()
     const resource = await requireResource(em, parsed.entityId, 'Resource not found')
     ensureTenantScope(ctx, resource.tenantId)
     ensureOrganizationScope(ctx, resource.organizationId)
+    const normalizedAuthor = await resolveResourceAuthorUserId(em, parsed.authorUserId, ctx, {
+      tenantId: resource.tenantId,
+      organizationId: resource.organizationId,
+    })
 
     const comment = em.create(ResourcesResourceComment, {
       organizationId: parsed.organizationId,
@@ -177,7 +180,6 @@ const updateCommentCommand: CommandHandler<ResourcesResourceCommentUpdateInput,
       comment.resource = resource
     }
     if (parsed.body !== undefined) comment.body = parsed.body
-    if (parsed.authorUserId !== undefined) comment.authorUserId = parsed.authorUserId ?? null
     if (parsed.appearanceIcon !== undefined) comment.appearanceIcon = parsed.appearanceIcon ?? null
     if (parsed.appearanceColor !== undefined) comment.appearanceColor = parsed.appearanceColor ?? null
 

package/src/modules/resources/commands/shared.ts

@@ -1,7 +1,11 @@
 import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
+import type { CommandRuntimeContext } from '@open-mercato/shared/lib/commands'
 import { ensureOrganizationScope, ensureTenantScope } from '@open-mercato/shared/lib/commands/scope'
 import { extractUndoPayload } from '@open-mercato/shared/lib/commands/undo'
+import { normalizeAuthorUserId } from '@open-mercato/shared/lib/commands/helpers'
+import { findOneWithDecryption } from '@open-mercato/shared/lib/encryption/find'
 import type { EntityManager } from '@mikro-orm/postgresql'
+import { User } from '@open-mercato/core/modules/auth/data/entities'
 import { ResourcesResource } from '../data/entities'
 
 export { ensureOrganizationScope, ensureTenantScope, extractUndoPayload }
@@ -15,3 +19,35 @@ export async function requireResource(
   if (!resource) throw new CrudHttpError(404, { error: message })
   return resource
 }
+
+export async function resolveResourceAuthorUserId(
+  em: EntityManager,
+  explicitAuthorUserId: string | undefined | null,
+  ctx: CommandRuntimeContext,
+  scope: { tenantId: string; organizationId: string },
+): Promise<string | null> {
+  const normalizedAuthor = normalizeAuthorUserId(explicitAuthorUserId, ctx.auth)
+  const fallbackAuthor = normalizeAuthorUserId(null, ctx.auth)
+  const isExplicitSuperAdminAuthor =
+    Boolean(explicitAuthorUserId) &&
+    normalizedAuthor === explicitAuthorUserId &&
+    ctx.auth?.isApiKey !== true &&
+    (ctx.auth as { isSuperAdmin?: boolean } | null)?.isSuperAdmin === true
+
+  if (!isExplicitSuperAdminAuthor || !normalizedAuthor) return normalizedAuthor
+
+  const author = await findOneWithDecryption(
+    em,
+    User,
+    {
+      id: normalizedAuthor,
+      tenantId: scope.tenantId,
+      organizationId: scope.organizationId,
+      deletedAt: null,
+    },
+    undefined,
+    { tenantId: scope.tenantId, organizationId: scope.organizationId },
+  )
+
+  return author ? normalizedAuthor : fallbackAuthor
+}

package/src/modules/sales/api/channels/route.ts

@@ -15,6 +15,9 @@ import {
 } from '../openapi'
 import { CatalogOffer } from '@open-mercato/core/modules/catalog/data/entities'
 import { parseBooleanToken } from '@open-mercato/shared/lib/boolean'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 const rawBodySchema = z.object({}).passthrough()
 
@@ -248,6 +251,6 @@ export async function decorateChannelsWithOfferCounts(
       ;(item as Record<string, unknown>).offerCount = countMap.get(id) ?? 0
     })
   } catch (err) {
-    console.warn('[sales.channels] failed to resolve channel offer counts', err)
+    logger.warn('sales.channels failed to resolve channel offer counts', { err })
   }
 }

package/src/modules/sales/api/document-history/route.ts

@@ -13,6 +13,9 @@ import type { OpenApiRouteDoc } from '@open-mercato/shared/lib/openapi'
 import { buildHistoryEntries } from '../../lib/historyHelpers'
 import { SalesNote } from '../../data/entities'
 import { findWithDecryption } from '@open-mercato/shared/lib/encryption/find'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 // Spec: SPEC-006-2026-01-23-order-status-history
 
@@ -130,7 +133,7 @@ export async function GET(req: Request) {
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('sales.document-history.get failed', err)
+    logger.error('sales.document-history.get failed', { err })
     const { translate } = await resolveTranslations()
     return NextResponse.json(
       { error: translate('sales.documents.history.error', 'Failed to load history.') },

package/src/modules/sales/api/document-numbers/route.ts

@@ -11,6 +11,9 @@ import type { RbacService } from '@open-mercato/core/modules/auth/services/rbacS
 import { documentNumberRequestSchema } from '../../data/validators'
 import { withScopedPayload } from '../utils'
 import { SalesDocumentNumberGenerator } from '../../services/salesDocumentNumberGenerator'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 export const metadata = {
   POST: { requireAuth: true },
@@ -105,7 +108,7 @@ export async function POST(req: Request) {
       return NextResponse.json(err.body, { status: err.status })
     }
     const { translate } = await resolveTranslations()
-    console.error('sales.document-numbers.generate failed', err)
+    logger.error('sales.document-numbers.generate failed', { err })
     return NextResponse.json(
       { error: translate('sales.documents.errors.number_generate_failed', 'Failed to generate document number.') },
       { status: 400 }

package/src/modules/sales/api/quotes/accept/route.ts

@@ -19,6 +19,9 @@ import { quoteAcceptSchema } from '../../../data/validators'
 import { sendEmail } from '@open-mercato/shared/lib/email/send'
 import { resolveStatusEntryIdByValue } from '../../../lib/statusHelpers'
 import { QuoteAcceptedAdminEmail } from '../../../emails/QuoteAcceptedAdminEmail'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type ConvertToOrderResult = {
   result?: { orderId?: string } | null
@@ -160,7 +163,7 @@ export async function POST(req: Request) {
           react: QuoteAcceptedAdminEmail({ orderUrl, copy }),
         })
       } catch (err) {
-        console.error('sales.quotes.accept.adminEmail failed', err)
+        logger.error('sales.quotes.accept.adminEmail failed', { err })
       }
     }
 
@@ -170,7 +173,7 @@ export async function POST(req: Request) {
       return NextResponse.json(err.body, { status: err.status })
     }
     const { translate } = await resolveTranslations()
-    console.error('sales.quotes.accept failed', err)
+    logger.error('sales.quotes.accept failed', { err })
     return NextResponse.json({ error: translate('sales.quotes.accept.failed', 'Failed to accept quote.') }, { status: 400 })
   }
 }

package/src/modules/sales/api/quotes/convert/route.ts

@@ -15,6 +15,9 @@ import {
   type MutationGuardInput,
 } from '@open-mercato/shared/lib/crud/mutation-guard-registry'
 import { withScopedPayload } from '../../utils'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 const convertSchema = z.object({
   quoteId: z.string().uuid(),
@@ -173,7 +176,7 @@ export async function POST(req: Request) {
       return NextResponse.json(err.body, { status: err.status })
     }
     const { translate } = await resolveTranslations()
-    console.error('sales.quotes.convert failed', err)
+    logger.error('sales.quotes.convert failed', { err })
     return NextResponse.json(
       { error: translate('sales.documents.detail.convertError', 'Failed to convert quote.') },
       { status: 400 }

package/src/modules/sales/api/quotes/public/[token]/route.ts

@@ -14,6 +14,9 @@ import {
 } from "../../../../data/entities";
 import { canonicalizeUnitCode } from "@open-mercato/shared/lib/units/unitCodes";
 import { getAuthFromRequest } from "@open-mercato/shared/lib/auth/server";
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 const paramsSchema = z.object({
   token: z.string().uuid(),
@@ -142,7 +145,7 @@ export async function GET(req: Request, ctx: { params: { token: string } }) {
       return NextResponse.json(err.body, { status: err.status });
     }
     const { translate } = await resolveTranslations();
-    console.error("sales.quotes.public failed", err);
+    logger.error('sales.quotes.public failed', { err });
     return NextResponse.json(
       {
         error: translate("sales.quotes.public.failed", "Failed to load quote."),

package/src/modules/sales/api/quotes/send/route.ts

@@ -23,6 +23,9 @@ import { quoteSendSchema } from '../../../data/validators'
 import { sendEmail } from '@open-mercato/shared/lib/email/send'
 import { resolveStatusEntryIdByValue } from '../../../lib/statusHelpers'
 import { QuoteSentEmail } from '../../../emails/QuoteSentEmail'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 export const metadata = {
   POST: { requireAuth: true, requireFeatures: ['sales.quotes.manage'] },
@@ -234,7 +237,7 @@ export async function POST(req: Request) {
       return NextResponse.json(err.body, { status: err.status })
     }
     const { translate } = await resolveTranslations()
-    console.error('sales.quotes.send failed', err)
+    logger.error('sales.quotes.send failed', { err })
     return NextResponse.json(
       { error: translate('sales.quotes.send.failed', 'Failed to send quote.') },
       { status: 400 }

package/src/modules/sales/api/returns/[id]/route.ts

@@ -9,6 +9,9 @@ import { resolveTranslations } from '@open-mercato/shared/lib/i18n/server'
 import { CrudHttpError, isCrudHttpError } from '@open-mercato/shared/lib/crud/errors'
 import { findOneWithDecryption, findWithDecryption } from '@open-mercato/shared/lib/encryption/find'
 import { SalesReturn, SalesReturnLine } from '../../../data/entities'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 export const metadata = {
   GET: { requireAuth: true, requireFeatures: ['sales.returns.view'] },
@@ -102,7 +105,7 @@ export async function GET(req: Request, ctx: { params: { id: string } }) {
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('sales.returns.get failed', err)
+    logger.error('sales.returns.get failed', { err })
     const { translate } = await resolveTranslations()
     return NextResponse.json({ error: translate('sales.returns.error', 'Failed to load return.') }, { status: 400 })
   }

package/src/modules/sales/api/settings/document-numbers/route.ts

@@ -18,6 +18,9 @@ import {
   runCrudMutationGuardAfterSuccess,
   validateCrudMutationGuard,
 } from '@open-mercato/shared/lib/crud/mutation-guard'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 export const metadata = {
   GET: { requireAuth: true, requireFeatures: ['sales.settings.manage'] },
@@ -88,7 +91,7 @@ export async function GET(req: Request) {
       return NextResponse.json(err.body, { status: err.status })
     }
     const { translate } = await resolveTranslations()
-    console.error('sales.settings.document-numbers.get failed', err)
+    logger.error('sales.settings.document-numbers.get failed', { err })
     return NextResponse.json(
       { error: translate('sales.settings.errors.load_failed', 'Failed to load sales settings') },
       { status: 400 }
@@ -158,7 +161,7 @@ export async function PUT(req: Request) {
       return NextResponse.json(err.body, { status: err.status })
     }
     const { translate } = await resolveTranslations()
-    console.error('sales.settings.document-numbers.put failed', err)
+    logger.error('sales.settings.document-numbers.put failed', { err })
     return NextResponse.json(
       { error: translate('sales.settings.errors.save_failed', 'Failed to save sales settings') },
       { status: 400 }

package/src/modules/sales/api/settings/order-editing/route.ts

@@ -19,6 +19,9 @@ import {
 } from '@open-mercato/shared/lib/crud/mutation-guard'
 import { ensureSalesDictionary } from '../../../lib/dictionaries'
 import { DictionaryEntry } from '@open-mercato/core/modules/dictionaries/data/entities'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 export const metadata = {
   GET: { requireAuth: true, requireFeatures: ['sales.settings.manage'] },
@@ -108,7 +111,7 @@ export async function GET(req: Request) {
       return NextResponse.json(err.body, { status: err.status })
     }
     const { translate } = await resolveTranslations()
-    console.error('sales.settings.order-editing.get failed', err)
+    logger.error('sales.settings.order-editing.get failed', { err })
     return NextResponse.json(
       { error: translate('sales.settings.errors.load_failed', 'Failed to load sales settings') },
       { status: 400 }
@@ -177,7 +180,7 @@ export async function PUT(req: Request) {
       return NextResponse.json(err.body, { status: err.status })
     }
     const { translate } = await resolveTranslations()
-    console.error('sales.settings.order-editing.put failed', err)
+    logger.error('sales.settings.order-editing.put failed', { err })
     return NextResponse.json(
       { error: translate('sales.settings.errors.save_failed', 'Failed to save sales settings') },
       { status: 400 }

package/src/modules/sales/backend/sales/channels/[channelId]/edit/page.tsx

@@ -15,6 +15,9 @@ import { useChannelFields, buildChannelPayload, type ChannelFormValues } from '@
 import { E } from '#generated/entities.ids.generated'
 import { SalesChannelOffersPanel } from '@open-mercato/core/modules/sales/components/channels/SalesChannelOffersPanel'
 import { SendObjectMessageDialog } from '@open-mercato/ui/backend/messages'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type ChannelApiResponse = {
   items?: Array<Record<string, unknown>>
@@ -63,7 +66,7 @@ export default function EditChannelPage({ params }: { params?: { channelId?: str
           setInitialValues(mapChannelToFormValues(item))
         }
       } catch (err) {
-        console.error('sales.channels.load', err)
+        logger.error('sales.channels.load', { err })
         if (!cancelled) {
           if ((err as { status?: number }).status === 404) {
             setIsNotFound(true)

package/src/modules/sales/backend/sales/channels/offers/page.tsx

@@ -18,6 +18,9 @@ import { useGuardedMutation } from '@open-mercato/ui/backend/injection/useGuarde
 import { useOrganizationScopeVersion } from '@open-mercato/shared/lib/frontend/useOrganizationScope'
 import { useT } from '@open-mercato/shared/lib/i18n/context'
 import { mapOfferRow, renderOfferPriceSummary, type OfferRow } from '@open-mercato/core/modules/sales/components/channels/offerTableUtils'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 type OffersResponse = {
   items?: Array<Record<string, unknown>>
@@ -113,7 +116,7 @@ export default function SalesChannelOffersListPage() {
       upsertChannelOptions(options)
       return options
     } catch (err) {
-      console.warn('[sales.channels.offers] failed to load channel options', err)
+      logger.warn('sales.channels.offers failed to load channel options', { err })
       return []
     }
   }, [t, upsertChannelOptions])
@@ -153,7 +156,7 @@ export default function SalesChannelOffersListPage() {
         .filter((option) => !!option) as FilterOption[]
       upsertChannelOptions(options)
     } catch (err) {
-      console.warn('[sales.channels.offers] failed to hydrate channel metadata', err)
+      logger.warn('sales.channels.offers failed to hydrate channel metadata', { err })
     }
   }, [t, upsertChannelOptions])
 
@@ -269,7 +272,7 @@ export default function SalesChannelOffersListPage() {
         .filter((value): value is string => typeof value === 'string' && value.length > 0)
       if (ids.length) void ensureChannelMetadata(Array.from(new Set(ids)))
     } catch (err) {
-      console.error('sales.channels.offers.list', err)
+      logger.error('sales.channels.offers.list', { err })
       flash(t('sales.channels.offers.errors.load', 'Failed to load offers.'), 'error')
     } finally {
       setLoading(false)
@@ -316,7 +319,7 @@ export default function SalesChannelOffersListPage() {
       handleRefresh()
     } catch (err) {
       if (surfaceRecordConflict(err, t)) { handleRefresh(); return }
-      console.error('sales.channels.offers.delete', err)
+      logger.error('sales.channels.offers.delete', { err })
     }
   }, [handleRefresh, mutationContext, runMutation, t])
 

package/src/modules/sales/backend/sales/channels/page.tsx

@@ -16,6 +16,9 @@ import { buildOptimisticLockHeader, extractOptimisticLockConflict } from '@open-
 import { useGuardedMutation } from '@open-mercato/ui/backend/injection/useGuardedMutation'
 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 ChannelRow = {
   id: string
@@ -136,7 +139,7 @@ export default function SalesChannelsPage() {
       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.list', err)
+      logger.error('sales.channels.list', { err })
       flash(t('sales.channels.table.errors.load', 'Failed to load channels.'), 'error')
     } finally {
       setLoading(false)
@@ -183,7 +186,7 @@ export default function SalesChannelsPage() {
         handleRefresh()
         return
       }
-      console.error('sales.channels.delete', err)
+      logger.error('sales.channels.delete', { err })
     }
   }, [handleRefresh, mutationContext, runMutation, t])
 

package/src/modules/sales/backend/sales/documents/[id]/page.tsx

@@ -69,6 +69,9 @@ import { readMarkdownPreferenceCookie, writeMarkdownPreferenceCookie } from '@op
 import { InjectionSpot, useInjectionWidgets } from '@open-mercato/ui/backend/injection/InjectionSpot'
 import { useGuardedMutation } from '@open-mercato/ui/backend/injection/useGuardedMutation'
 import { buildRecordInjectionContext, useSetCurrentRecordInjectionContext } from '@open-mercato/ui/backend/injection/recordContext'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('sales')
 
 function formatMessageAmount(amount: number | null | undefined, currency: string | null | undefined): string | null {
   if (typeof amount !== 'number' || !Number.isFinite(amount)) return null
@@ -155,7 +158,7 @@ function CurrencyInlineEditor({
       await onSave(draft ?? null)
       setEditing(false)
     } catch (err) {
-      console.error('sales.documents.currency.save', err)
+      logger.error('sales.documents.currency.save', { err })
     } finally {
       setSaving(false)
     }
@@ -467,7 +470,7 @@ function CustomerInlineEditor({
       await onSave(draftId, draftEmail)
       setMode(null)
     } catch (err) {
-      console.error('sales.documents.customer.save', err)
+      logger.error('sales.documents.customer.save', { err })
     }
   }, [draftEmail, draftId, onClearError, onSave])
 
@@ -488,7 +491,7 @@ function CustomerInlineEditor({
       await onSaveSnapshot(payload)
       setMode(null)
     } catch (err) {
-      console.error('sales.documents.customer.snapshot.save', err)
+      logger.error('sales.documents.customer.snapshot.save', { err })
     }
   }, [buildSnapshotPayload, onClearError, onSaveSnapshot, phoneIsValid])
 
@@ -2131,7 +2134,7 @@ export default function SalesDocumentDetailPage({
         })
         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 {
@@ -2166,7 +2169,7 @@ export default function SalesDocumentDetailPage({
         if (!query) upsertChannelOptions([])
         return []
       } catch (err) {
-        console.error('sales.documents.loadChannels', err)
+        logger.error('sales.documents.loadChannels', { err })
         if (!query) {
           flash(t('sales.channels.offers.filters.channelsLoadError', 'Failed to load channels'), 'error')
         }
@@ -2238,7 +2241,7 @@ export default function SalesDocumentDetailPage({
         if (!query) upsertShippingMethodOptions([])
         return []
       } catch (err) {
-        console.error('sales.documents.loadShippingMethods', err)
+        logger.error('sales.documents.loadShippingMethods', { err })
         if (!query) {
           flash(
             t('sales.documents.detail.shippingMethodLoadError', 'Failed to load shipping methods.'),
@@ -2295,7 +2298,7 @@ export default function SalesDocumentDetailPage({
         if (!query) upsertPaymentMethodOptions([])
         return []
       } catch (err) {
-        console.error('sales.documents.loadPaymentMethods', err)
+        logger.error('sales.documents.loadPaymentMethods', { err })
         if (!query) {
           flash(
             t('sales.documents.detail.paymentMethodLoadError', 'Failed to load payment methods.'),
@@ -2325,7 +2328,7 @@ export default function SalesDocumentDetailPage({
       const items = Array.isArray(response.result?.items) ? response.result.items : []
       setHasPayments(items.some((item) => item && typeof (item as any).id === 'string'))
     } catch (err) {
-      console.error('sales.documents.currency.paymentsGuard', err)
+      logger.error('sales.documents.currency.paymentsGuard', { err })
     }
   }, [kind, record?.id])
 
@@ -2349,7 +2352,7 @@ export default function SalesDocumentDetailPage({
         }
         return email ?? null
       } catch (err) {
-        console.error('sales.documents.fetchCustomerEmail', err)
+        logger.error('sales.documents.fetchCustomerEmail', { err })
         return null
       }
     },
@@ -2375,7 +2378,7 @@ export default function SalesDocumentDetailPage({
           return option
         }
       } catch (err) {
-        console.error('sales.documents.channel.ensure', err)
+        logger.error('sales.documents.channel.ensure', { err })
       }
       return null
     },
@@ -2479,7 +2482,7 @@ export default function SalesDocumentDetailPage({
       upsertStatusOptions([])
       return []
     } catch (err) {
-      console.error('sales.documents.loadStatuses', err)
+      logger.error('sales.documents.loadStatuses', { err })
       flash(t('sales.documents.detail.status.errorLoad', 'Failed to load statuses.'), 'error')
       return []
     } finally {
@@ -2613,7 +2616,7 @@ export default function SalesDocumentDetailPage({
           addresses: normalizeGuardList(call.result?.orderAddressEditableStatuses ?? null),
         })
       } catch (err) {
-        console.error('sales.documents.loadGuards', err)
+        logger.error('sales.documents.loadGuards', { err })
       }
     }
     void loadGuards()
@@ -2719,7 +2722,7 @@ export default function SalesDocumentDetailPage({
       const ordered = [...mapped].sort((a, b) => (a.position ?? 0) - (b.position ?? 0))
       setAdjustmentRows(ordered)
     } catch (err) {
-      console.error('sales.documents.adjustments.totals.load', err)
+      logger.error('sales.documents.adjustments.totals.load', { err })
     }
   }, [kind, parseNumber, record?.currencyCode, record?.id])
 
@@ -2736,7 +2739,7 @@ export default function SalesDocumentDetailPage({
       }
       await loadAdjustmentsForTotals()
     } catch (err) {
-      console.error('sales.documents.totals.refresh', err)
+      logger.error('sales.documents.totals.refresh', { err })
     }
   }, [fetchDocumentByKind, kind, loadAdjustmentsForTotals, record?.id])
 
@@ -3719,7 +3722,7 @@ export default function SalesDocumentDetailPage({
         router.replace(`/backend/sales/orders/${orderId}`)
       }, { quoteId: record.id })
     } catch (err) {
-      console.error('sales.documents.convert', err)
+      logger.error('sales.documents.convert', { err })
       if (!handleDocumentMutationError(err, t, () => setReloadKey((prev) => prev + 1))) {
         flash(t('sales.documents.detail.convertError', 'Failed to convert quote.'), 'error')
       }
@@ -3744,7 +3747,7 @@ export default function SalesDocumentDetailPage({
         if (updated) setRecord(updated)
       }, { quoteId: record.id, validForDays })
     } catch (err) {
-      console.error('sales.quotes.send', err)
+      logger.error('sales.quotes.send', { err })
       flash(t('sales.quotes.send.failed', 'Failed to send quote.'), 'error')
     } finally {
       setSending(false)
@@ -3776,7 +3779,7 @@ export default function SalesDocumentDetailPage({
       const listPath = kind === 'order' ? '/backend/sales/orders' : '/backend/sales/quotes'
       router.push(listPath)
     } catch (err) {
-      console.error('sales.documents.delete', err)
+      logger.error('sales.documents.delete', { err })
       if (!handleDocumentMutationError(err, t, () => setReloadKey((prev) => prev + 1))) {
         flash(t('sales.documents.detail.deleteFailed', 'Could not delete document.'), 'error')
       }
@@ -4127,7 +4130,7 @@ export default function SalesDocumentDetailPage({
           appearanceColor: '#0ea5e9',
         })
       } catch (err) {
-        console.error('sales.shipments.comment', err)
+        logger.error('sales.shipments.comment', { err })
       }
     },
     [record, salesNotesAdapter],

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