@open-mercato/core 0.6.7-develop.6553.1.2ec7246e04 → 0.6.7-develop.6566.1.15385fa179

package/src/modules/customers/commands/interactions.ts

@@ -38,7 +38,7 @@ import {
   loadCustomFieldSnapshot,
   buildCustomFieldResetMap,
 } from '@open-mercato/shared/lib/commands/customFieldSnapshots'
-import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
+import { CrudHttpError, notFound } from '@open-mercato/shared/lib/crud/errors'
 import { enforceRecordGoneIsConflict, enforceCommandOptimisticLockWithGuards } from '@open-mercato/shared/lib/crud/optimistic-lock-command'
 import { findOneWithDecryption } from '@open-mercato/shared/lib/encryption/find'
 import type { CrudIndexerConfig, CrudEventsConfig } from '@open-mercato/shared/lib/crud/types'
@@ -642,7 +642,7 @@ const updateInteractionCommand: CommandHandler<InteractionUpdateInput, { interac
       const interaction = await findOneWithDecryption(trx, CustomerInteraction, { id: parsed.id, deletedAt: null })
       if (!interaction) {
         enforceRecordGoneIsConflict({ resourceKind: 'customers.interaction', resourceId: parsed.id, request: ctx.request ?? null })
-        throw new CrudHttpError(404, { error: 'Interaction not found' })
+        throw notFound('Interaction not found')
       }
       ensureTenantScope(ctx, interaction.tenantId)
       ensureOrganizationScope(ctx, interaction.organizationId)
@@ -684,7 +684,7 @@ const updateInteractionCommand: CommandHandler<InteractionUpdateInput, { interac
             userFeatures: undefined,
           })
         ) {
-          throw new CrudHttpError(404, { error: 'Email not found' })
+          throw notFound('Email not found')
         }
       }
 
@@ -915,7 +915,7 @@ const completeInteractionCommand: CommandHandler<InteractionCompleteInput, { int
       const interaction = await findOneWithDecryption(trx, CustomerInteraction, { id: parsed.id, deletedAt: null })
       if (!interaction) {
         enforceRecordGoneIsConflict({ resourceKind: 'customers.interaction', resourceId: parsed.id, request: ctx.request ?? null })
-        throw new CrudHttpError(404, { error: 'Interaction not found' })
+        throw notFound('Interaction not found')
       }
       ensureTenantScope(ctx, interaction.tenantId)
       ensureOrganizationScope(ctx, interaction.organizationId)
@@ -1055,7 +1055,7 @@ const cancelInteractionCommand: CommandHandler<InteractionCancelInput, { interac
       const interaction = await findOneWithDecryption(trx, CustomerInteraction, { id: parsed.id, deletedAt: null })
       if (!interaction) {
         enforceRecordGoneIsConflict({ resourceKind: 'customers.interaction', resourceId: parsed.id, request: ctx.request ?? null })
-        throw new CrudHttpError(404, { error: 'Interaction not found' })
+        throw notFound('Interaction not found')
       }
       ensureTenantScope(ctx, interaction.tenantId)
       ensureOrganizationScope(ctx, interaction.organizationId)
@@ -1193,7 +1193,7 @@ const deleteInteractionCommand: CommandHandler<{ body?: Record<string, unknown>;
         const interaction = await findOneWithDecryption(trx, CustomerInteraction, { id, deletedAt: null })
         if (!interaction) {
           enforceRecordGoneIsConflict({ resourceKind: 'customers.interaction', resourceId: id, request: ctx.request ?? null })
-          throw new CrudHttpError(404, { error: 'Interaction not found' })
+          throw notFound('Interaction not found')
         }
         ensureTenantScope(ctx, interaction.tenantId)
         ensureOrganizationScope(ctx, interaction.organizationId)

package/src/modules/customers/commands/labels.ts

@@ -2,7 +2,7 @@ import { registerCommand } from '@open-mercato/shared/lib/commands'
 import type { CommandHandler } from '@open-mercato/shared/lib/commands'
 import type { CrudEventsConfig } from '@open-mercato/shared/lib/crud/types'
 import { emitCrudSideEffects, emitCrudUndoSideEffects } from '@open-mercato/shared/lib/commands/helpers'
-import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
+import { CrudHttpError, notFound } from '@open-mercato/shared/lib/crud/errors'
 import { resolveTranslations } from '@open-mercato/shared/lib/i18n/server'
 import { findOneWithDecryption } from '@open-mercato/shared/lib/encryption/find'
 import type { DataEngine } from '@open-mercato/shared/lib/data/engine'
@@ -287,7 +287,7 @@ const assignLabelCommand: CommandHandler<LabelAssignCommandInput, { assignmentId
       { tenantId: parsed.tenantId, organizationId: parsed.organizationId },
     )
     if (!label) {
-      throw new CrudHttpError(404, { error: 'Label not found' })
+      throw notFound('Label not found')
     }
 
     const entity = await findOneWithDecryption(
@@ -298,7 +298,7 @@ const assignLabelCommand: CommandHandler<LabelAssignCommandInput, { assignmentId
       { tenantId: parsed.tenantId, organizationId: parsed.organizationId },
     )
     if (!entity) {
-      throw new CrudHttpError(404, { error: 'Entity not found' })
+      throw notFound('Entity not found')
     }
 
     const existing = await findOneWithDecryption(
@@ -441,7 +441,7 @@ const assignLabelCommand: CommandHandler<LabelAssignCommandInput, { assignmentId
       { tenantId: after.tenantId, organizationId: after.organizationId },
     )
     if (!label) {
-      throw new CrudHttpError(404, { error: 'Label not found' })
+      throw notFound('Label not found')
     }
     const entity = await findOneWithDecryption(
       em,
@@ -451,7 +451,7 @@ const assignLabelCommand: CommandHandler<LabelAssignCommandInput, { assignmentId
       { tenantId: after.tenantId, organizationId: after.organizationId },
     )
     if (!entity) {
-      throw new CrudHttpError(404, { error: 'Entity not found' })
+      throw notFound('Entity not found')
     }
 
     let assignment = await findOneWithDecryption(
@@ -570,7 +570,7 @@ const unassignLabelCommand: CommandHandler<LabelUnassignCommandInput, { assignme
       { tenantId: parsed.tenantId, organizationId: parsed.organizationId },
     )
     if (!label) {
-      throw new CrudHttpError(404, { error: 'Label not found' })
+      throw notFound('Label not found')
     }
 
     const entity = await findOneWithDecryption(
@@ -581,7 +581,7 @@ const unassignLabelCommand: CommandHandler<LabelUnassignCommandInput, { assignme
       { tenantId: parsed.tenantId, organizationId: parsed.organizationId },
     )
     if (!entity) {
-      throw new CrudHttpError(404, { error: 'Entity not found' })
+      throw notFound('Entity not found')
     }
 
     const existing = await findOneWithDecryption(

package/src/modules/customers/commands/people.ts

@@ -46,7 +46,7 @@ import {
 } from './shared'
 import { withAtomicFlush } from '@open-mercato/shared/lib/commands/flush'
 import { resolveRedoSnapshot } from '@open-mercato/shared/lib/commands/redo'
-import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
+import { CrudHttpError, notFound } from '@open-mercato/shared/lib/crud/errors'
 import { resolveTranslations } from '@open-mercato/shared/lib/i18n/server'
 import {
   loadCustomFieldSnapshot,
@@ -851,7 +851,7 @@ const createPersonCommand: CommandHandler<PersonCreateInput, { entityId: string;
         undefined,
         { tenantId: after.entity.tenantId, organizationId: after.entity.organizationId },
       )
-      if (!existingProfile) throw new CrudHttpError(404, { error: 'Person profile not found' })
+      if (!existingProfile) throw notFound('Person profile not found')
       profile = existingProfile
       const survivingEntity = entity
       await withAtomicFlush(em, [
@@ -927,7 +927,7 @@ const updatePersonCommand: CommandHandler<PersonUpdateInput, { entityId: string
     ensureTenantScope(ctx, record.tenantId)
     ensureOrganizationScope(ctx, record.organizationId)
     const profile = await em.findOne(CustomerPersonProfile, { entity: record })
-    if (!profile) throw new CrudHttpError(404, { error: 'Person profile not found' })
+    if (!profile) throw notFound('Person profile not found')
 
     if (parsed.displayName !== undefined) {
       const nextDisplayName = parsed.displayName.trim()

package/src/modules/customers/commands/personCompanyLinks.ts

@@ -2,7 +2,7 @@ import { registerCommand } from '@open-mercato/shared/lib/commands'
 import type { CommandHandler } from '@open-mercato/shared/lib/commands'
 import type { CrudEventsConfig } from '@open-mercato/shared/lib/crud/types'
 import { emitCrudSideEffects, emitCrudUndoSideEffects } from '@open-mercato/shared/lib/commands/helpers'
-import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
+import { CrudHttpError, notFound } from '@open-mercato/shared/lib/crud/errors'
 import { resolveTranslations } from '@open-mercato/shared/lib/i18n/server'
 import { findOneWithDecryption } from '@open-mercato/shared/lib/encryption/find'
 import type { DataEngine } from '@open-mercato/shared/lib/data/engine'
@@ -126,7 +126,7 @@ async function requirePersonEntity(
     { tenantId, organizationId },
   )
   if (!person) {
-    throw new CrudHttpError(404, { error: 'Person not found' })
+    throw notFound('Person not found')
   }
   return person
 }
@@ -145,7 +145,7 @@ async function requireCompanyEntity(
     { tenantId, organizationId },
   )
   if (!company) {
-    throw new CrudHttpError(404, { error: 'Company not found' })
+    throw notFound('Company not found')
   }
   return company
 }
@@ -162,7 +162,7 @@ async function requirePersonProfile(
     { tenantId: person.tenantId, organizationId: person.organizationId },
   )
   if (!profile) {
-    throw new CrudHttpError(404, { error: 'Person profile not found' })
+    throw notFound('Person profile not found')
   }
   return profile
 }
@@ -433,7 +433,7 @@ const updatePersonCompanyLinkCommand: CommandHandler<PersonCompanyLinkUpdateInpu
       { tenantId: parsed.tenantId, organizationId: parsed.organizationId },
     )
     if (!link) {
-      throw new CrudHttpError(404, { error: 'Company link not found' })
+      throw notFound('Company link not found')
     }
 
     const personId = typeof link.person === 'string' ? link.person : link.person.id
@@ -606,7 +606,7 @@ const deletePersonCompanyLinkCommand: CommandHandler<PersonCompanyLinkDeleteInpu
       { tenantId: parsed.tenantId, organizationId: parsed.organizationId },
     )
     if (!link) {
-      throw new CrudHttpError(404, { error: 'Company link not found' })
+      throw notFound('Company link not found')
     }
 
     const personId = typeof link.person === 'string' ? link.person : link.person.id

package/src/modules/customers/commands/pipeline-stages.ts

@@ -14,7 +14,7 @@ import {
   type PipelineStageReorderInput,
 } from '../data/validators'
 import { ensureOrganizationScope, ensureTenantScope, ensureDictionaryEntry } from './shared'
-import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
+import { CrudHttpError, notFound } from '@open-mercato/shared/lib/crud/errors'
 import { withAtomicFlush } from '@open-mercato/shared/lib/commands/flush'
 import {
   enforceCommandOptimisticLockWithGuards,
@@ -112,7 +112,7 @@ const updatePipelineStageCommand: CommandHandler<PipelineStageUpdateInput, void>
     })
     if (!stage) {
       enforceRecordGoneIsConflict({ resourceKind: 'customers.pipelineStage', resourceId: parsed.id, request: ctx.request ?? null })
-      throw new CrudHttpError(404, { error: 'Pipeline stage not found' })
+      throw notFound('Pipeline stage not found')
     }
 
     ensureTenantScope(ctx, stage.tenantId)
@@ -164,7 +164,7 @@ const deletePipelineStageCommand: CommandHandler<PipelineStageDeleteInput, void>
     })
     if (!stage) {
       enforceRecordGoneIsConflict({ resourceKind: 'customers.pipelineStage', resourceId: parsed.id, request: ctx.request ?? null })
-      throw new CrudHttpError(404, { error: 'Pipeline stage not found' })
+      throw notFound('Pipeline stage not found')
     }
 
     ensureTenantScope(ctx, stage.tenantId)

package/src/modules/customers/commands/pipelines.ts

@@ -11,7 +11,7 @@ import {
   type PipelineDeleteInput,
 } from '../data/validators'
 import { ensureOrganizationScope, ensureTenantScope } from './shared'
-import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
+import { CrudHttpError, notFound } from '@open-mercato/shared/lib/crud/errors'
 import {
   enforceCommandOptimisticLockWithGuards,
   enforceRecordGoneIsConflict,
@@ -62,7 +62,7 @@ const updatePipelineCommand: CommandHandler<PipelineUpdateInput, void> = {
     const pipeline = await em.findOne(CustomerPipeline, { id: parsed.id })
     if (!pipeline) {
       enforceRecordGoneIsConflict({ resourceKind: 'customers.pipeline', resourceId: parsed.id, request: ctx.request ?? null })
-      throw new CrudHttpError(404, { error: 'Pipeline not found' })
+      throw notFound('Pipeline not found')
     }
 
     ensureTenantScope(ctx, pipeline.tenantId)
@@ -102,7 +102,7 @@ const deletePipelineCommand: CommandHandler<PipelineDeleteInput, void> = {
     const pipeline = await em.findOne(CustomerPipeline, { id: parsed.id })
     if (!pipeline) {
       enforceRecordGoneIsConflict({ resourceKind: 'customers.pipeline', resourceId: parsed.id, request: ctx.request ?? null })
-      throw new CrudHttpError(404, { error: 'Pipeline not found' })
+      throw notFound('Pipeline not found')
     }
 
     ensureTenantScope(ctx, pipeline.tenantId)

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

@@ -1,6 +1,6 @@
 import type { EntityManager } from '@mikro-orm/postgresql'
 import { CustomerDeal, CustomerEntity, CustomerTag, CustomerTagAssignment, CustomerDictionaryEntry, type CustomerEntityKind } from '../data/entities'
-import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
+import { CrudHttpError, notFound } from '@open-mercato/shared/lib/crud/errors'
 import type { CommandRuntimeContext } from '@open-mercato/shared/lib/commands'
 import { ensureOrganizationScope, ensureSameScope } from '@open-mercato/shared/lib/commands/scope'
 import { findWithDecryption } from '@open-mercato/shared/lib/encryption/find'
@@ -47,7 +47,7 @@ export async function requireCustomerEntity(
     tenantId: scope.tenantId,
     organizationId: scope.organizationId,
   })
-  if (!entity) throw new CrudHttpError(404, { error: message })
+  if (!entity) throw notFound(message)
   if (kind && entity.kind !== kind) {
     throw new CrudHttpError(400, { error: 'Invalid entity type' })
   }
@@ -80,7 +80,7 @@ export async function requireTimelineParentEntity(
   if (deal) {
     throw new CrudHttpError(422, { error: 'entityId must reference a person or company, not a deal' })
   }
-  throw new CrudHttpError(404, { error: 'Customer not found' })
+  throw notFound('Customer not found')
 }
 
 export async function syncEntityTags(

package/src/modules/customers/commands/tags.ts

@@ -21,7 +21,7 @@ import {
   loadEntityTagIds,
 } from './shared'
 import { resolveTranslations } from '@open-mercato/shared/lib/i18n/server'
-import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
+import { CrudHttpError, notFound } from '@open-mercato/shared/lib/crud/errors'
 import type { CrudEventsConfig } from '@open-mercato/shared/lib/crud/types'
 import { findOneWithDecryption } from '@open-mercato/shared/lib/encryption/find'
 import { emitCustomersEvent } from '../events'
@@ -172,7 +172,7 @@ const updateTagCommand: CommandHandler<TagUpdateInput, { tagId: string }> = {
     const parsed = tagUpdateSchema.parse(rawInput)
     const em = (ctx.container.resolve('em') as EntityManager).fork()
     const tag = await em.findOne(CustomerTag, { id: parsed.id })
-    if (!tag) throw new CrudHttpError(404, { error: 'Tag not found' })
+    if (!tag) throw notFound('Tag not found')
     ensureTenantScope(ctx, tag.tenantId)
     ensureOrganizationScope(ctx, tag.organizationId)
 
@@ -290,7 +290,7 @@ const deleteTagCommand: CommandHandler<{ body?: Record<string, unknown>; query?:
     const id = requireId(input, 'Tag id required')
     const em = (ctx.container.resolve('em') as EntityManager).fork()
     const tag = await em.findOne(CustomerTag, { id })
-    if (!tag) throw new CrudHttpError(404, { error: 'Tag not found' })
+    if (!tag) throw notFound('Tag not found')
     ensureTenantScope(ctx, tag.tenantId)
     ensureOrganizationScope(ctx, tag.organizationId)
     await withAtomicFlush(em, [
@@ -386,7 +386,7 @@ const assignTagCommand: CommandHandler<TagAssignmentInput, { assignmentId: strin
 
     const em = (ctx.container.resolve('em') as EntityManager).fork()
   const tag = await em.findOne(CustomerTag, { id: parsed.tagId, tenantId: parsed.tenantId, organizationId: parsed.organizationId })
-  if (!tag) throw new CrudHttpError(404, { error: 'Tag not found' })
+  if (!tag) throw notFound('Tag not found')
   const entity = await requireCustomerEntity(em, parsed.entityId, { tenantId: parsed.tenantId, organizationId: parsed.organizationId }, undefined, 'Customer not found')
   ensureSameScope(entity, parsed.organizationId, parsed.tenantId)
   const tagIds = await loadEntityTagIds(em, entity)
@@ -477,7 +477,7 @@ const assignTagCommand: CommandHandler<TagAssignmentInput, { assignmentId: strin
     const assignmentId = logEntry?.resourceId ?? null
     const em = (ctx.container.resolve('em') as EntityManager).fork()
     const tag = await em.findOne(CustomerTag, { id: before.tagId })
-    if (!tag) throw new CrudHttpError(404, { error: 'Tag not found' })
+    if (!tag) throw notFound('Tag not found')
     const entity = await requireCustomerEntity(em, before.entityId, { tenantId: before.tenantId, organizationId: before.organizationId }, undefined, 'Customer not found')
     ensureSameScope(entity, before.organizationId, before.tenantId)
 
@@ -537,7 +537,7 @@ const unassignTagCommand: CommandHandler<TagAssignmentInput, { assignmentId: str
       tenantId: parsed.tenantId,
       organizationId: parsed.organizationId,
     })
-    if (!existing) throw new CrudHttpError(404, { error: 'Tag assignment not found' })
+    if (!existing) throw notFound('Tag assignment not found')
     await em.remove(existing).flush()
 
     const de = (ctx.container.resolve('dataEngine') as DataEngine)
@@ -591,7 +591,7 @@ const unassignTagCommand: CommandHandler<TagAssignmentInput, { assignmentId: str
     const tag = await em.findOne(CustomerTag, { id: before.tagId })
     const entity = await requireCustomerEntity(em, before.entityId, { tenantId: before.tenantId, organizationId: before.organizationId }, undefined, 'Customer not found')
     ensureSameScope(entity, before.organizationId, before.tenantId)
-    if (!tag) throw new CrudHttpError(404, { error: 'Tag not found' })
+    if (!tag) throw notFound('Tag not found')
     const existing = await em.findOne(CustomerTagAssignment, {
       tag,
       entity,

package/src/modules/customers/commands/todos.ts

@@ -16,7 +16,7 @@ import {
   resolveParentResourceKind,
 } from './shared'
 import { resolveTranslations } from '@open-mercato/shared/lib/i18n/server'
-import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
+import { notFound } from '@open-mercato/shared/lib/crud/errors'
 import {
   CUSTOMER_INTERACTION_TASK_SOURCE,
   CUSTOMER_INTERACTION_TODO_ADAPTER_SOURCE,
@@ -290,7 +290,7 @@ async function resolveTodoTarget(
   const em = (ctx.container.resolve('em') as EntityManager).fork()
   const legacyLink = await em.findOne(CustomerTodoLink, { id: linkId }, { populate: ['entity'] })
   if (!legacyLink) {
-    throw new CrudHttpError(404, { error: 'Todo link not found' })
+    throw notFound('Todo link not found')
   }
 
   const bridgedSnapshot = await loadInteractionSnapshot(legacyLink.todoId, ctx)
@@ -333,7 +333,7 @@ const unlinkTodoCommand: CommandHandler<
     const target = await resolveTodoTarget(parsed.linkId, ctx)
     if (!target.canonicalExists) {
       if (!target.legacyLink) {
-        throw new CrudHttpError(404, { error: 'Todo link not found' })
+        throw notFound('Todo link not found')
       }
       const canonicalCreate = getRequiredHandler<
         InteractionCreateInput & { customValues?: Record<string, unknown> },

package/src/modules/customers/lib/personCompanies.ts

@@ -1,5 +1,5 @@
 import type { EntityManager } from '@mikro-orm/postgresql'
-import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
+import { CrudHttpError, notFound } from '@open-mercato/shared/lib/crud/errors'
 import { findWithDecryption, findOneWithDecryption } from '@open-mercato/shared/lib/encryption/find'
 import {
   CustomerEntity,
@@ -48,7 +48,7 @@ async function requireCompany(
 ): Promise<CustomerEntity> {
   const company = await findOneWithDecryption(em, CustomerEntity, { id: companyId, kind: 'company', deletedAt: null }, {}, { tenantId, organizationId })
   if (!company) {
-    throw new CrudHttpError(404, { error: 'Company not found' })
+    throw notFound('Company not found')
   }
   if (company.organizationId !== organizationId || company.tenantId !== tenantId) {
     throw new CrudHttpError(403, { error: 'Cannot link company outside current scope' })
@@ -260,7 +260,7 @@ export async function updatePersonCompanyLink(
   }
 
   if (!link) {
-    throw new CrudHttpError(404, { error: 'Company link not found' })
+    throw notFound('Company link not found')
   }
 
   if (patch.isPrimary === true) {
@@ -301,7 +301,7 @@ export async function removePersonCompanyLink(
       profile.company = null
       return
     }
-    throw new CrudHttpError(404, { error: 'Company link not found' })
+    throw notFound('Company link not found')
   }
 
   const removedCompanyId = typeof link.company === 'string' ? link.company : link.company.id

package/src/modules/perspectives/data/validators.ts

@@ -4,6 +4,10 @@ import type { PerspectiveSettings } from '@open-mercato/shared/modules/perspecti
 export const perspectiveSettingsSchema: z.ZodType<PerspectiveSettings> = z.object({
   columnOrder: z.array(z.string().min(1)).max(120).optional(),
   columnVisibility: z.record(z.string(), z.boolean()).optional(),
+  // User-resized column widths in pixels, keyed by column id (#1835). Clamped to
+  // the same [60, 900] bound the client enforces so a saved perspective cannot
+  // carry a column width that collapses or blows out the table.
+  columnSizing: z.record(z.string().min(1), z.number().int().min(60).max(900)).optional(),
   filters: z.record(z.string(), z.unknown()).optional(),
   sorting: z
     .array(z.object({ id: z.string().min(1), desc: z.boolean().optional() }))

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

@@ -1,7 +1,7 @@
 import { z } from 'zod'
 import type { EntityManager } from '@mikro-orm/postgresql'
 import { makeCrudRoute, type CrudCtx } from '@open-mercato/shared/lib/crud/factory'
-import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
+import { CrudHttpError, notFound } from '@open-mercato/shared/lib/crud/errors'
 import { findOneWithDecryption } from '@open-mercato/shared/lib/encryption/find'
 import { resolveTranslations } from '@open-mercato/shared/lib/i18n/server'
 import type { RbacService } from '@open-mercato/core/modules/auth/services/rbacService'
@@ -97,9 +97,7 @@ async function loadNoteForPermission(
   })
 
   if (!note) {
-    throw new CrudHttpError(404, {
-      error: translate('sales.documents.detail.error', 'Document not found or inaccessible.'),
-    })
+    throw notFound(translate('sales.documents.detail.error', 'Document not found or inaccessible.'))
   }
 
   return note

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

@@ -3,7 +3,7 @@ import { z } from 'zod'
 import { createRequestContainer } from '@open-mercato/shared/lib/di/container'
 import type { CommandBus, CommandRuntimeContext } from '@open-mercato/shared/lib/commands'
 import { resolveTranslations } from '@open-mercato/shared/lib/i18n/server'
-import { CrudHttpError, isCrudHttpError } from '@open-mercato/shared/lib/crud/errors'
+import { CrudHttpError, isCrudHttpError, notFound } from '@open-mercato/shared/lib/crud/errors'
 import type { OpenApiRouteDoc } from '@open-mercato/shared/lib/openapi'
 import type { EntityManager } from '@mikro-orm/postgresql'
 import { LockMode } from '@mikro-orm/core'
@@ -93,7 +93,7 @@ export async function POST(req: Request) {
         )
       const quote = (await findQuoteByToken(hashedToken)) ?? (await findQuoteByToken(token))
       if (!quote) {
-        throw new CrudHttpError(404, { error: translate('sales.quotes.accept.notFound', 'Quote not found.') })
+        throw notFound(translate('sales.quotes.accept.notFound', 'Quote not found.'))
       }
 
       const now = new Date()

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

@@ -2,7 +2,7 @@ import { NextResponse } from "next/server";
 import { z } from "zod";
 import { createRequestContainer } from "@open-mercato/shared/lib/di/container";
 import { resolveTranslations } from "@open-mercato/shared/lib/i18n/server";
-import { CrudHttpError, isCrudHttpError } from "@open-mercato/shared/lib/crud/errors";
+import { isCrudHttpError, notFound } from "@open-mercato/shared/lib/crud/errors";
 import type { OpenApiRouteDoc } from "@open-mercato/shared/lib/openapi";
 import type { EntityManager } from "@mikro-orm/postgresql";
 import { findOneWithDecryption, findWithDecryption } from "@open-mercato/shared/lib/encryption/find";
@@ -43,16 +43,12 @@ export async function GET(req: Request, ctx: { params: { token: string } }) {
       }));
     const { translate } = await resolveTranslations();
     if (!quote) {
-      throw new CrudHttpError(404, {
-        error: translate("sales.quotes.public.notFound", "Quote not found."),
-      });
+      throw notFound(translate("sales.quotes.public.notFound", "Quote not found."));
     }
 
     const auth = await getAuthFromRequest(req);
     if (auth?.tenantId && quote.tenantId !== auth.tenantId) {
-      throw new CrudHttpError(404, {
-        error: translate("sales.quotes.public.notFound", "Quote not found."),
-      });
+      throw notFound(translate("sales.quotes.public.notFound", "Quote not found."));
     }
 
     const now = new Date();

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

@@ -5,7 +5,7 @@ import { getAuthFromRequest } from '@open-mercato/shared/lib/auth/server'
 import { resolveOrganizationScopeForRequest } from '@open-mercato/core/modules/directory/utils/organizationScope'
 import type { CommandRuntimeContext } from '@open-mercato/shared/lib/commands'
 import { resolveTranslations, detectLocale } from '@open-mercato/shared/lib/i18n/server'
-import { CrudHttpError, isCrudHttpError } from '@open-mercato/shared/lib/crud/errors'
+import { CrudHttpError, isCrudHttpError, notFound } from '@open-mercato/shared/lib/crud/errors'
 import type { OpenApiRouteDoc } from '@open-mercato/shared/lib/openapi'
 import {
   bridgeLegacyGuard,
@@ -151,7 +151,7 @@ export async function POST(req: Request) {
     const tenantScope = ctx.auth?.tenantId ? { tenantId: ctx.auth.tenantId } : undefined
     const quote = await findOneWithDecryption(em, SalesQuote, { id: input.quoteId, deletedAt: null }, {}, tenantScope)
     if (!quote) {
-      throw new CrudHttpError(404, { error: translate('sales.documents.detail.error', 'Document not found or inaccessible.') })
+      throw notFound(translate('sales.documents.detail.error', 'Document not found or inaccessible.'))
     }
     if (quote.tenantId !== ctx.auth?.tenantId || quote.organizationId !== ctx.selectedOrganizationId) {
       throw new CrudHttpError(403, { error: translate('sales.documents.errors.forbidden', 'Forbidden') })

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

@@ -6,7 +6,7 @@ import { createRequestContainer } from '@open-mercato/shared/lib/di/container'
 import { getAuthFromRequest } from '@open-mercato/shared/lib/auth/server'
 import { resolveOrganizationScopeForRequest } from '@open-mercato/core/modules/directory/utils/organizationScope'
 import { resolveTranslations } from '@open-mercato/shared/lib/i18n/server'
-import { CrudHttpError, isCrudHttpError } from '@open-mercato/shared/lib/crud/errors'
+import { CrudHttpError, isCrudHttpError, notFound } 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'
@@ -56,7 +56,7 @@ export async function GET(req: Request, ctx: { params: { id: string } }) {
       { tenantId: auth.tenantId, organizationId },
     )
     if (!header || !header.order) {
-      throw new CrudHttpError(404, { error: translate('sales.returns.notFound', 'Return not found.') })
+      throw notFound(translate('sales.returns.notFound', 'Return not found.'))
     }
 
     const lines = await findWithDecryption(

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

@@ -9,7 +9,7 @@ import {
   emitCrudUndoSideEffects,
 } from '@open-mercato/shared/lib/commands/helpers'
 import type { EntityManager } from '@mikro-orm/postgresql'
-import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
+import { notFound } from '@open-mercato/shared/lib/crud/errors'
 import { loadCustomFieldSnapshot, buildCustomFieldResetMap } from '@open-mercato/shared/lib/commands/customFieldSnapshots'
 import { makeCreateRedo } from '@open-mercato/shared/lib/commands/redo'
 import { findOneWithDecryption } from '@open-mercato/shared/lib/encryption/find'
@@ -751,7 +751,7 @@ const updateChannelCommand: CommandHandler<ChannelUpdateInput, { channelId: stri
     const { parsed, custom } = parseWithCustomFields(channelUpdateSchema, rawInput)
     const em = (ctx.container.resolve('em') as EntityManager).fork()
     const record = await findOneWithDecryption(em, SalesChannel, { id: parsed.id, deletedAt: null }, {}, { tenantId: parsed.tenantId, organizationId: parsed.organizationId })
-    if (!record) throw new CrudHttpError(404, { error: 'Channel not found' })
+    if (!record) throw notFound('Channel not found')
     const dataEngine = ctx.container.resolve('dataEngine') as DataEngine
     const scope = resolveScopeFromUpdate(record, parsed, ctx)
 
@@ -899,7 +899,7 @@ const deleteChannelCommand: CommandHandler<
     const id = requireId(input, 'Channel id is required')
     const em = (ctx.container.resolve('em') as EntityManager).fork()
     const record = await findOneWithDecryption(em, SalesChannel, { id }, {})
-    if (!record) throw new CrudHttpError(404, { error: 'Channel not found' })
+    if (!record) throw notFound('Channel not found')
     ensureTenantScope(ctx, record.tenantId)
     ensureOrganizationScope(ctx, record.organizationId)
     em.remove(record)
@@ -1097,7 +1097,7 @@ const updateDeliveryWindowCommand: CommandHandler<
     const { parsed, custom } = parseWithCustomFields(deliveryWindowUpdateSchema, rawInput)
     const em = (ctx.container.resolve('em') as EntityManager).fork()
     const record = await em.findOne(SalesDeliveryWindow, { id: parsed.id, deletedAt: null })
-    if (!record) throw new CrudHttpError(404, { error: 'Delivery window not found' })
+    if (!record) throw notFound('Delivery window not found')
     const scope = resolveScopeFromUpdate(record, parsed, ctx)
     record.organizationId = scope.organizationId
     record.tenantId = scope.tenantId
@@ -1201,7 +1201,7 @@ const deleteDeliveryWindowCommand: CommandHandler<
     const id = requireId(input, 'Delivery window id is required')
     const em = (ctx.container.resolve('em') as EntityManager).fork()
     const record = await em.findOne(SalesDeliveryWindow, { id })
-    if (!record) throw new CrudHttpError(404, { error: 'Delivery window not found' })
+    if (!record) throw notFound('Delivery window not found')
     ensureTenantScope(ctx, record.tenantId)
     ensureOrganizationScope(ctx, record.organizationId)
     em.remove(record)
@@ -1377,7 +1377,7 @@ const updateShippingMethodCommand: CommandHandler<
     const { parsed, custom } = parseWithCustomFields(shippingMethodUpdateSchema, rawInput)
     const em = (ctx.container.resolve('em') as EntityManager).fork()
     const record = await em.findOne(SalesShippingMethod, { id: parsed.id, deletedAt: null })
-    if (!record) throw new CrudHttpError(404, { error: 'Shipping method not found' })
+    if (!record) throw notFound('Shipping method not found')
     const scope = resolveScopeFromUpdate(record, parsed, ctx)
     record.organizationId = scope.organizationId
     record.tenantId = scope.tenantId
@@ -1501,7 +1501,7 @@ const deleteShippingMethodCommand: CommandHandler<
     const id = requireId(input, 'Shipping method id is required')
     const em = (ctx.container.resolve('em') as EntityManager).fork()
     const record = await em.findOne(SalesShippingMethod, { id })
-    if (!record) throw new CrudHttpError(404, { error: 'Shipping method not found' })
+    if (!record) throw notFound('Shipping method not found')
     ensureTenantScope(ctx, record.tenantId)
     ensureOrganizationScope(ctx, record.organizationId)
     em.remove(record)
@@ -1672,7 +1672,7 @@ const updatePaymentMethodCommand: CommandHandler<
     const { parsed, custom } = parseWithCustomFields(paymentMethodUpdateSchema, rawInput)
     const em = (ctx.container.resolve('em') as EntityManager).fork()
     const record = await em.findOne(SalesPaymentMethod, { id: parsed.id, deletedAt: null })
-    if (!record) throw new CrudHttpError(404, { error: 'Payment method not found' })
+    if (!record) throw notFound('Payment method not found')
     const scope = resolveScopeFromUpdate(record, parsed, ctx)
     record.organizationId = scope.organizationId
     record.tenantId = scope.tenantId
@@ -1785,7 +1785,7 @@ const deletePaymentMethodCommand: CommandHandler<
     const id = requireId(input, 'Payment method id is required')
     const em = (ctx.container.resolve('em') as EntityManager).fork()
     const record = await em.findOne(SalesPaymentMethod, { id })
-    if (!record) throw new CrudHttpError(404, { error: 'Payment method not found' })
+    if (!record) throw notFound('Payment method not found')
     ensureTenantScope(ctx, record.tenantId)
     ensureOrganizationScope(ctx, record.organizationId)
     em.remove(record)
@@ -1971,7 +1971,7 @@ const updateTaxRateCommand: CommandHandler<TaxRateUpdateInput, { taxRateId: stri
     const { parsed, custom } = parseWithCustomFields(taxRateUpdateSchema, rawInput)
     const em = (ctx.container.resolve('em') as EntityManager).fork()
     const record = await em.findOne(SalesTaxRate, { id: parsed.id, deletedAt: null })
-    if (!record) throw new CrudHttpError(404, { error: 'Tax rate not found' })
+    if (!record) throw notFound('Tax rate not found')
     const scope = resolveScopeFromUpdate(record, parsed, ctx)
     record.organizationId = scope.organizationId
     record.tenantId = scope.tenantId
@@ -2102,7 +2102,7 @@ const deleteTaxRateCommand: CommandHandler<
     const id = requireId(input, 'Tax rate id is required')
     const em = (ctx.container.resolve('em') as EntityManager).fork()
     const record = await em.findOne(SalesTaxRate, { id })
-    if (!record) throw new CrudHttpError(404, { error: 'Tax rate not found' })
+    if (!record) throw notFound('Tax rate not found')
     ensureTenantScope(ctx, record.tenantId)
     ensureOrganizationScope(ctx, record.organizationId)
     const snapshot = await loadTaxRateSnapshot((ctx.container.resolve('em') as EntityManager), id)