@open-mercato/core 0.6.4-develop.4382.1.6b4f656b77 → 0.6.4

package/src/modules/customers/api/todos/route.ts

@@ -19,6 +19,7 @@ import type { QueryEngine } from '@open-mercato/shared/lib/query/types'
 import { createCustomersCrudOpenApi, createPagedListResponseSchema, defaultOkResponseSchema } from '../openapi'
 import { CustomerInteraction, CustomerTodoLink } from '../../data/entities'
 import { todoLinkWithTodoCreateSchema } from '../../data/validators'
+import { withOperationMetadata } from '../../lib/operationMetadata'
 import { resolveCustomerInteractionFeatureFlags } from '../../lib/interactionFeatureFlags'
 import { resolveCustomersRequestContext } from '../../lib/interactionRequestContext'
 import {
@@ -344,7 +345,7 @@ export async function POST(request: Request): Promise<Response> {
     }
     const customValues = collectTodoCustomValues(body as Record<string, unknown>)
 
-    const { result } = await commandBus.execute('customers.interactions.create', {
+    const { result, logEntry } = await commandBus.execute('customers.interactions.create', {
       input: {
         tenantId: auth.tenantId,
         organizationId: selectedOrganizationId ?? auth.orgId,
@@ -394,12 +395,16 @@ export async function POST(request: Request): Promise<Response> {
           ? result.id
           : null
     return withAdapterHeaders(
-      NextResponse.json(
-        {
-          linkId: interactionId,
-          todoId: interactionId,
-        },
-        { status: 201 },
+      withOperationMetadata(
+        NextResponse.json(
+          {
+            linkId: interactionId,
+            todoId: interactionId,
+          },
+          { status: 201 },
+        ),
+        logEntry,
+        { resourceKind: 'customers.todoLink', resourceId: interactionId },
       ),
     )
   } catch (err) {
@@ -459,7 +464,7 @@ export async function PUT(request: Request): Promise<Response> {
     const customValues = collectTodoCustomValues(body as Record<string, unknown>)
     const nextDone = normalizeTodoStatusInput(body)
 
-    await commandBus.execute('customers.interactions.update', {
+    const { logEntry } = await commandBus.execute('customers.interactions.update', {
       input: {
         id: interactionId,
         ...(body.title !== undefined ? { title: body.title } : {}),
@@ -492,7 +497,13 @@ export async function PUT(request: Request): Promise<Response> {
       })
     }
 
-    return withAdapterHeaders(NextResponse.json({ ok: true }))
+    return withAdapterHeaders(
+      withOperationMetadata(
+        NextResponse.json({ ok: true }),
+        logEntry,
+        { resourceKind: 'customers.todoLink', resourceId: body.linkId ?? body.id },
+      ),
+    )
   } catch (err) {
     if (isCrudHttpError(err)) {
       return withAdapterHeaders(NextResponse.json(err.body, { status: err.status }))
@@ -548,7 +559,7 @@ export async function DELETE(request: Request): Promise<Response> {
           auth.tenantId,
         )
 
-    await commandBus.execute('customers.interactions.delete', {
+    const { logEntry } = await commandBus.execute('customers.interactions.delete', {
       input: { id: interactionId },
       ctx: commandContext,
     })
@@ -566,7 +577,13 @@ export async function DELETE(request: Request): Promise<Response> {
       })
     }
 
-    return withAdapterHeaders(NextResponse.json({ ok: true }))
+    return withAdapterHeaders(
+      withOperationMetadata(
+        NextResponse.json({ ok: true }),
+        logEntry,
+        { resourceKind: 'customers.todoLink', resourceId: body.id },
+      ),
+    )
   } catch (err) {
     if (isCrudHttpError(err)) {
       return withAdapterHeaders(NextResponse.json(err.body, { status: err.status }))
@@ -595,6 +612,7 @@ const todoItemSchema = z.object({
   todoDueAt: z.string().nullable().optional(),
   todoCustomValues: z.record(z.string(), z.unknown()).nullable().optional(),
   todoOrganizationId: z.string().nullable(),
+  todoUpdatedAt: z.string().nullable().optional(),
   organizationId: z.string(),
   tenantId: z.string(),
   createdAt: z.string(),

package/src/modules/customers/backend/config/customers/deals/page.tsx

@@ -81,6 +81,7 @@ export default function DealsSettingsPage(): React.ReactElement {
     try {
       await runMutation({
         operation: async () => {
+          // optimistic-lock-exempt: tenant-scoped preference setting (single stuckThresholdDays value, no loaded record version)
           await apiCallOrThrow(
             '/api/customers/settings/stuck-threshold',
             {

package/src/modules/customers/backend/config/customers/pipeline-stages/page.tsx

@@ -3,7 +3,8 @@
 import * as React from 'react'
 import { Page, PageBody } from '@open-mercato/ui/backend/Page'
 import { useT } from '@open-mercato/shared/lib/i18n/context'
-import { apiCall } from '@open-mercato/ui/backend/utils/apiCall'
+import { apiCall, withScopedApiRequestHeaders } from '@open-mercato/ui/backend/utils/apiCall'
+import { buildOptimisticLockHeader } from '@open-mercato/ui/backend/utils/optimisticLock'
 import { flash } from '@open-mercato/ui/backend/FlashMessages'
 import { Button } from '@open-mercato/ui/primitives/button'
 import { Input } from '@open-mercato/ui/primitives/input'
@@ -24,6 +25,7 @@ type Pipeline = {
   id: string
   name: string
   isDefault: boolean
+  updatedAt?: string | null
 }
 
 type PipelineStage = {
@@ -33,6 +35,7 @@ type PipelineStage = {
   order: number
   color: string | null
   icon: string | null
+  updatedAt?: string | null
 }
 
 type PipelineDialogState =
@@ -146,11 +149,14 @@ export default function PipelineStagesPage() {
         await loadPipelines()
         if (newId) setSelectedPipelineId(newId)
       } else if (pipelineDialog?.mode === 'edit') {
-        const result = await apiCall('/api/customers/pipelines', {
-          method: 'PUT',
-          body: JSON.stringify({ id: pipelineDialog.pipeline.id, name: pipelineName.trim(), isDefault: pipelineIsDefault }),
-          headers: { 'Content-Type': 'application/json' },
-        })
+        const result = await withScopedApiRequestHeaders(
+          buildOptimisticLockHeader(pipelineDialog.pipeline.updatedAt),
+          () => apiCall('/api/customers/pipelines', {
+            method: 'PUT',
+            body: JSON.stringify({ id: pipelineDialog.pipeline.id, name: pipelineName.trim(), isDefault: pipelineIsDefault }),
+            headers: { 'Content-Type': 'application/json' },
+          }),
+        )
         if (!result.ok) {
           flash(t('customers.config.pipelineStages.errorUpdatePipeline', 'Failed to update pipeline'), 'error')
           return
@@ -175,11 +181,14 @@ export default function PipelineStagesPage() {
       variant: 'destructive',
     })
     if (!confirmed) return
-    const result = await apiCall('/api/customers/pipelines', {
-      method: 'DELETE',
-      body: JSON.stringify({ id: pipeline.id }),
-      headers: { 'Content-Type': 'application/json' },
-    })
+    const result = await withScopedApiRequestHeaders(
+      buildOptimisticLockHeader(pipeline.updatedAt),
+      () => apiCall('/api/customers/pipelines', {
+        method: 'DELETE',
+        body: JSON.stringify({ id: pipeline.id }),
+        headers: { 'Content-Type': 'application/json' },
+      }),
+    )
     if (!result.ok) {
       const error = (result.result as { error?: string })?.error
       flash(error ?? t('customers.config.pipelineStages.errorDeletePipeline', 'Failed to delete pipeline'), 'error')
@@ -224,11 +233,14 @@ export default function PipelineStagesPage() {
         }
         flash(t('customers.config.pipelineStages.createdStage', 'Stage created'), 'success')
       } else if (stageDialog?.mode === 'edit') {
-        const result = await apiCall('/api/customers/pipeline-stages', {
-          method: 'PUT',
-          body: JSON.stringify({ id: stageDialog.stage.id, label: stageName.trim(), color: stageColor, icon: stageIcon }),
-          headers: { 'Content-Type': 'application/json' },
-        })
+        const result = await withScopedApiRequestHeaders(
+          buildOptimisticLockHeader(stageDialog.stage.updatedAt),
+          () => apiCall('/api/customers/pipeline-stages', {
+            method: 'PUT',
+            body: JSON.stringify({ id: stageDialog.stage.id, label: stageName.trim(), color: stageColor, icon: stageIcon }),
+            headers: { 'Content-Type': 'application/json' },
+          }),
+        )
         if (!result.ok) {
           flash(t('customers.config.pipelineStages.errorUpdateStage', 'Failed to update stage'), 'error')
           return
@@ -253,11 +265,14 @@ export default function PipelineStagesPage() {
       variant: 'destructive',
     })
     if (!confirmed) return
-    const result = await apiCall('/api/customers/pipeline-stages', {
-      method: 'DELETE',
-      body: JSON.stringify({ id: stage.id }),
-      headers: { 'Content-Type': 'application/json' },
-    })
+    const result = await withScopedApiRequestHeaders(
+      buildOptimisticLockHeader(stage.updatedAt),
+      () => apiCall('/api/customers/pipeline-stages', {
+        method: 'DELETE',
+        body: JSON.stringify({ id: stage.id }),
+        headers: { 'Content-Type': 'application/json' },
+      }),
+    )
     if (!result.ok) {
       const error = (result.result as { error?: string })?.error
       flash(error ?? t('customers.config.pipelineStages.errorDeleteStage', 'Failed to delete stage'), 'error')

package/src/modules/customers/backend/customers/companies/[id]/page.tsx

@@ -8,7 +8,8 @@ import { Button } from '@open-mercato/ui/primitives/button'
 import { Separator } from '@open-mercato/ui/primitives/separator'
 import { Spinner } from '@open-mercato/ui/primitives/spinner'
 import { flash } from '@open-mercato/ui/backend/FlashMessages'
-import { apiCallOrThrow, readApiResultOrThrow } from '@open-mercato/ui/backend/utils/apiCall'
+import { apiCallOrThrow, readApiResultOrThrow, withScopedApiRequestHeaders } from '@open-mercato/ui/backend/utils/apiCall'
+import { buildOptimisticLockHeader } from '@open-mercato/ui/backend/utils/optimisticLock'
 import { collectCustomFieldValues } from '@open-mercato/ui/backend/utils/customFieldValues'
 import { mapCrudServerErrorToFormErrors } from '@open-mercato/ui/backend/utils/serverErrors'
 import { E } from '#generated/entities.ids.generated'
@@ -70,6 +71,7 @@ type CompanyOverview = {
     nextInteractionIcon?: string | null
     nextInteractionColor?: string | null
     organizationId?: string | null
+    updatedAt?: string | null
   }
   profile: {
     id: string
@@ -324,19 +326,31 @@ export default function CustomerCompanyDetailPage({ params }: { params?: { id?:
     async (patch: Record<string, unknown>, apply: (prev: CompanyOverview) => CompanyOverview) => {
       if (!data) return
       const payload = { id: data.company.id, ...patch }
-      await runMutationWithContext(
-        () => apiCallOrThrow(
-          '/api/customers/companies',
-          {
-            method: 'PUT',
-            headers: { 'content-type': 'application/json' },
-            body: JSON.stringify(payload),
-          },
-          { errorMessage: t('customers.companies.detail.inline.error', 'Unable to update company.') },
+      const result = await runMutationWithContext(
+        () => withScopedApiRequestHeaders(
+          buildOptimisticLockHeader((data?.company as { updatedAt?: string } | undefined)?.updatedAt),
+          () => apiCallOrThrow<{ ok?: boolean; updatedAt?: string | null }>(
+            '/api/customers/companies',
+            {
+              method: 'PUT',
+              headers: { 'content-type': 'application/json' },
+              body: JSON.stringify(payload),
+            },
+            { errorMessage: t('customers.companies.detail.inline.error', 'Unable to update company.') },
+          ),
         ),
         payload,
       )
-      setData((prev) => (prev ? apply(prev) : prev))
+      // Refresh the optimistic-lock token so the next sequential inline edit does
+      // not send a stale updatedAt and falsely 409 (#2055, default-ON locking).
+      const nextUpdatedAt = result?.result?.updatedAt ?? null
+      setData((prev) => {
+        if (!prev) return prev
+        const applied = apply(prev)
+        return nextUpdatedAt
+          ? { ...applied, company: { ...applied.company, updatedAt: nextUpdatedAt } }
+          : applied
+      })
     },
     [data, runMutationWithContext, t],
   )
@@ -417,19 +431,24 @@ export default function CustomerCompanyDetailPage({ params }: { params?: { id?:
         id: data.company.id,
         customFields: customPayload,
       }
+      let customFieldsUpdatedAt: string | null = null
       try {
-        await runMutationWithContext(
-          () => apiCallOrThrow(
-            '/api/customers/companies',
-            {
-              method: 'PUT',
-              headers: { 'content-type': 'application/json' },
-              body: JSON.stringify(payload),
-            },
-            { errorMessage: t('customers.companies.detail.inline.error', 'Unable to update company.') },
+        const result = await runMutationWithContext(
+          () => withScopedApiRequestHeaders(
+            buildOptimisticLockHeader((data?.company as { updatedAt?: string } | undefined)?.updatedAt),
+            () => apiCallOrThrow<{ ok?: boolean; updatedAt?: string | null }>(
+              '/api/customers/companies',
+              {
+                method: 'PUT',
+                headers: { 'content-type': 'application/json' },
+                body: JSON.stringify(payload),
+              },
+              { errorMessage: t('customers.companies.detail.inline.error', 'Unable to update company.') },
+            ),
           ),
           payload,
         )
+        customFieldsUpdatedAt = result?.result?.updatedAt ?? null
       } catch (err) {
         const { message: helperMessage, fieldErrors } = mapCrudServerErrorToFormErrors(err)
         const message = helperMessage ?? t('customers.companies.detail.inline.error', 'Unable to update company.')
@@ -448,6 +467,9 @@ export default function CustomerCompanyDetailPage({ params }: { params?: { id?:
         if (!prev) return prev
         return {
           ...prev,
+          company: customFieldsUpdatedAt
+            ? { ...prev.company, updatedAt: customFieldsUpdatedAt }
+            : prev.company,
           customFields: {
             ...prev.customFields,
             ...normalized,
@@ -493,13 +515,16 @@ export default function CustomerCompanyDetailPage({ params }: { params?: { id?:
     setIsDeleting(true)
     try {
       await runMutationWithContext(
-        () => apiCallOrThrow(
-          `/api/customers/companies?id=${encodeURIComponent(currentCompanyId)}`,
-          {
-            method: 'DELETE',
-            headers: { 'content-type': 'application/json' },
-          },
-          { errorMessage: t('customers.companies.list.deleteError', 'Failed to delete company.') },
+        () => withScopedApiRequestHeaders(
+          buildOptimisticLockHeader((data?.company as { updatedAt?: string } | undefined)?.updatedAt),
+          () => apiCallOrThrow(
+            `/api/customers/companies?id=${encodeURIComponent(currentCompanyId)}`,
+            {
+              method: 'DELETE',
+              headers: { 'content-type': 'application/json' },
+            },
+            { errorMessage: t('customers.companies.list.deleteError', 'Failed to delete company.') },
+          ),
         ),
         { id: currentCompanyId },
       )

package/src/modules/customers/backend/customers/companies/page.tsx

@@ -44,6 +44,7 @@ import { useAutoDiscoveredFields } from '@open-mercato/ui/backend/utils/useAutoD
 import { useAdvancedFilterTree } from '@open-mercato/ui/backend/hooks/useAdvancedFilter'
 import { AdvancedFilterPanel } from '@open-mercato/ui/backend/filters/AdvancedFilterPanel'
 import { ActiveFilterChips } from '@open-mercato/ui/backend/filters/ActiveFilterChips'
+import { ListEmptyState } from '@open-mercato/ui/backend/filters/ListEmptyState'
 import type { FilterPreset } from '@open-mercato/ui/backend/filters/QuickFilters'
 import { useQueryClient } from '@tanstack/react-query'
 import { ensureCustomerDictionary } from '../../../components/detail/hooks/useCustomerDictionary'
@@ -448,6 +449,7 @@ export default function CustomersCompaniesPage() {
     try {
       await runSingleMutation({
         operation: async () => {
+          // optimistic-lock-exempt: delete-only mutation — no field-level lost-update
           await apiCallOrThrow(
             `/api/customers/companies?id=${encodeURIComponent(company.id)}`,
             {
@@ -487,6 +489,7 @@ export default function CustomersCompaniesPage() {
         runBulkDelete(
           selectedRows,
           async (row) => {
+            // optimistic-lock-exempt: bulk delete-only mutation — no field-level lost-update
             await apiCallOrThrow(`/api/customers/companies?id=${encodeURIComponent(row.id)}`, {
               method: 'DELETE',
               headers: { 'content-type': 'application/json' },
@@ -936,7 +939,7 @@ export default function CustomersCompaniesPage() {
           }}
           activeFilterChips={(
             <ActiveFilterChips
-              tree={filterPanel.tree}
+              tree={filterPanel.appliedTree}
               fields={advancedFilterFields}
               popoverOpen={filtersOpen}
               onRemoveNode={(id) => filterPanel.dispatch({ type: 'removeNode', nodeId: id })}
@@ -950,6 +953,13 @@ export default function CustomersCompaniesPage() {
             onClearAll: handleAdvancedFilterClear,
             onRemoveLast: () => filterPanel.dispatch({ type: 'removeLast' }),
           }}
+          emptyState={(
+            <ListEmptyState
+              entityName={t('customers.companies.entityPlural', 'companies')}
+              createHref="/backend/customers/companies/create"
+              createLabel={t('customers.companies.list.actions.new')}
+            />
+          )}
           virtualized
           pagination={{ page, pageSize, total, totalPages, onPageChange: setPage, pageSizeOptions: [10, 25, 50, 100], onPageSizeChange: handlePageSizeChange, cacheStatus }}
           isLoading={isLoading}

package/src/modules/customers/backend/customers/companies-v2/[id]/page.tsx

@@ -6,8 +6,10 @@ import { useRouter, useSearchParams } from 'next/navigation'
 import { Page, PageBody } from '@open-mercato/ui/backend/Page'
 import { Building2, Hash, Users, BarChart3, StickyNote } from 'lucide-react'
 import { updateCrud, deleteCrud } from '@open-mercato/ui/backend/utils/crud'
-import { apiCallOrThrow, readApiResultOrThrow } from '@open-mercato/ui/backend/utils/apiCall'
+import { apiCallOrThrow, readApiResultOrThrow, withScopedApiRequestHeaders } from '@open-mercato/ui/backend/utils/apiCall'
+import { buildOptimisticLockHeader } from '@open-mercato/ui/backend/utils/optimisticLock'
 import { flash } from '@open-mercato/ui/backend/FlashMessages'
+import { surfaceRecordConflict } from '@open-mercato/ui/backend/conflicts'
 import { useT } from '@open-mercato/shared/lib/i18n/context'
 import { Button } from '@open-mercato/ui/primitives/button'
 import { AttachmentsSection, ErrorMessage, LoadingMessage, RecordNotFoundState, type SectionAction } from '@open-mercato/ui/backend/detail'
@@ -204,6 +206,7 @@ export default function CompanyDetailV2Page({ params }: { params?: { id?: string
     // moment instead of "today" (#1807 prefill).
     const editPayload = {
       id: activity.id,
+      updatedAt: typeof raw.updatedAt === 'string' ? raw.updatedAt as string : typeof raw.updated_at === 'string' ? raw.updated_at as string : null,
       interactionType: typeof activity.interactionType === 'string' ? activity.interactionType : undefined,
       title: typeof activity.title === 'string' ? activity.title : null,
       body: typeof activity.body === 'string' ? activity.body : null,
@@ -323,10 +326,28 @@ export default function CompanyDetailV2Page({ params }: { params?: { id?: string
       variant: 'destructive',
     })
     if (!approved) return
-    await runMutationWithContext(
-      () => deleteCrud('customers/companies', { id: companyId }),
-      { id: companyId, operation: 'deleteCompany' },
-    )
+    try {
+      await runMutationWithContext(
+        () => withScopedApiRequestHeaders(
+          buildOptimisticLockHeader((data?.company as { updatedAt?: string } | undefined)?.updatedAt),
+          () => deleteCrud('customers/companies', { id: companyId }),
+        ),
+        { id: companyId, operation: 'deleteCompany' },
+      )
+    } catch (err) {
+      // The guarded mutation already routes a 409 to the unified conflict bar;
+      // surface any other server error (e.g. "Cannot delete company: linked
+      // deals…") as a flash instead of letting it crash the page.
+      if (!surfaceRecordConflict(err, t)) {
+        flash(
+          err instanceof Error && err.message.trim().length > 0
+            ? err.message
+            : t('customers.companies.detail.deleteError', 'Failed to delete company.'),
+          'error',
+        )
+      }
+      return
+    }
     flash(t('customers.companies.list.deleteSuccess', 'Company deleted.'), 'success')
     router.push('/backend/customers/companies')
   }, [confirm, data?.company?.id, router, runMutationWithContext, t])
@@ -457,6 +478,7 @@ export default function CompanyDetailV2Page({ params }: { params?: { id?: string
                   initialValues={initialValues}
                   onSubmit={handleFormSubmit}
                   onDelete={handleDelete}
+                  optimisticLockUpdatedAt={(data?.company as { updatedAt?: string } | undefined)?.updatedAt}
                   hideFooterActions
                   collapsibleGroups={{ pageType: 'company-v2', chevronPosition: 'right' }}
                   sortableGroups={{ pageType: 'company-v2' }}

package/src/modules/customers/backend/customers/deals/[id]/hooks/useDealFormHandlers.ts

@@ -1,7 +1,10 @@
 import * as React from 'react'
 import { useRouter } from 'next/navigation'
 import { flash } from '@open-mercato/ui/backend/FlashMessages'
+import { surfaceRecordConflict } from '@open-mercato/ui/backend/conflicts'
 import { deleteCrud, updateCrud } from '@open-mercato/ui/backend/utils/crud'
+import { withScopedApiRequestHeaders } from '@open-mercato/ui/backend/utils/apiCall'
+import { buildOptimisticLockHeader } from '@open-mercato/ui/backend/utils/optimisticLock'
 import { useT } from '@open-mercato/shared/lib/i18n/context'
 import { useConfirmDialog } from '@open-mercato/ui/backend/confirm-dialog'
 import type { DealFormSubmitPayload } from '../../../../../components/detail/DealForm'
@@ -40,11 +43,23 @@ export function useDealFormHandlers({
       if (!data) return
       setIsSaving(true)
       try {
-        await updateCrud('customers/deals', {
+        const body: Record<string, unknown> = {
           id: data.deal.id,
           ...payload.base,
-          ...payload.custom,
-        })
+        }
+        // Custom-field values MUST travel under `customFields` (bare keys). The
+        // deal update route's `splitCustomFieldPayload` only routes
+        // `customFields`/`customValues`/`cf_`/`cf:` entries to the custom-field
+        // writer; spreading bare keys into the body would land them in `base`
+        // where `dealUpdateSchema.parse` silently drops them, so edits never
+        // persisted (boolean/select/multi all reverted on refresh).
+        if (payload.custom && Object.keys(payload.custom).length) {
+          body.customFields = payload.custom
+        }
+        await withScopedApiRequestHeaders(
+          buildOptimisticLockHeader(data.deal.updatedAt),
+          () => updateCrud('customers/deals', body),
+        )
         flash(t('customers.deals.detail.updateSuccess', 'Deal updated.'), 'success')
         await loadData()
       } finally {
@@ -64,10 +79,27 @@ export function useDealFormHandlers({
       variant: 'destructive',
     })
     if (!approved) return
-    await runMutationWithContext(
-      () => deleteCrud('customers/deals', currentDealId),
-      { id: currentDealId, operation: 'deleteDeal' },
-    )
+    try {
+      await runMutationWithContext(
+        () => withScopedApiRequestHeaders(
+          buildOptimisticLockHeader(data.deal.updatedAt),
+          () => deleteCrud('customers/deals', currentDealId),
+        ),
+        { id: currentDealId, operation: 'deleteDeal' },
+      )
+    } catch (err) {
+      // The guarded mutation routes a 409 to the unified conflict bar; surface
+      // any other server error as a flash instead of letting it crash the page.
+      if (!surfaceRecordConflict(err, t)) {
+        flash(
+          err instanceof Error && err.message.trim().length > 0
+            ? err.message
+            : t('customers.deals.detail.deleteError', 'Failed to delete deal.'),
+          'error',
+        )
+      }
+      return
+    }
     flash(t('customers.deals.detail.deleteSuccess', 'Deal deleted.'), 'success')
     router.push('/backend/customers/deals')
   }, [confirm, currentDealId, data, router, runMutationWithContext, t])

package/src/modules/customers/backend/customers/deals/[id]/page.tsx

@@ -228,6 +228,7 @@ export default function DealDetailPage({ params }: { params?: { id?: string } })
     const rawActivity = activity as unknown as Record<string, unknown>
     openScheduleEdit({
       id: activity.id,
+      updatedAt: typeof rawActivity.updatedAt === 'string' ? rawActivity.updatedAt as string : typeof rawActivity.updated_at === 'string' ? rawActivity.updated_at as string : null,
       interactionType: activity.interactionType,
       title: activity.title ?? null,
       body: activity.body ?? null,

package/src/modules/customers/backend/customers/deals/page.tsx

@@ -11,7 +11,9 @@ import type { AdvancedFilterTree } from '@open-mercato/shared/lib/query/advanced
 import { createEmptyTree, makeRuleTree, makeMultiRuleTree } from '@open-mercato/shared/lib/query/advanced-filter-tree'
 import { deserializeTree, deserializeAdvancedFilter, flatToTree, mapDictionaryColorToTone, serializeTree, type FilterFieldDef, type FilterOption as AdvancedFilterOption } from '@open-mercato/shared/lib/query/advanced-filter'
 import { useCurrentUserId } from '@open-mercato/ui/backend/utils/useCurrentUserId'
-import { apiCall } from '@open-mercato/ui/backend/utils/apiCall'
+import { apiCall, withScopedApiRequestHeaders } from '@open-mercato/ui/backend/utils/apiCall'
+import { buildOptimisticLockHeader } from '@open-mercato/ui/backend/utils/optimisticLock'
+import { surfaceRecordConflict } from '@open-mercato/ui/backend/conflicts'
 import { buildCrudExportUrl, deleteCrud } from '@open-mercato/ui/backend/utils/crud'
 import { groupBulkDeleteFailures, runBulkDelete } from '@open-mercato/ui/backend/utils/bulkDelete'
 import { useGuardedMutation } from '@open-mercato/ui/backend/injection/useGuardedMutation'
@@ -46,6 +48,7 @@ import { useAutoDiscoveredFields } from '@open-mercato/ui/backend/utils/useAutoD
 import { useAdvancedFilterTree } from '@open-mercato/ui/backend/hooks/useAdvancedFilter'
 import { AdvancedFilterPanel } from '@open-mercato/ui/backend/filters/AdvancedFilterPanel'
 import { ActiveFilterChips } from '@open-mercato/ui/backend/filters/ActiveFilterChips'
+import { ListEmptyState } from '@open-mercato/ui/backend/filters/ListEmptyState'
 import type { FilterPreset } from '@open-mercato/ui/backend/filters/QuickFilters'
 import {
   ensureCurrentUserFilterOption,
@@ -447,14 +450,18 @@ export default function CustomersDealsPage() {
         variant: 'destructive',
       })
       if (!confirmed) return
+      const lockVersion = rows.find((row) => row.id === dealId)?.updatedAt ?? null
       setPendingDeleteId(dealId)
       try {
         await runSingleMutation({
           operation: async () => {
-            await deleteCrud('customers/deals', {
-              body: { id: dealId },
-              errorMessage: t('customers.deals.list.deleteError', 'Failed to delete deal.'),
-            })
+            await withScopedApiRequestHeaders(
+              buildOptimisticLockHeader(lockVersion),
+              () => deleteCrud('customers/deals', {
+                body: { id: dealId },
+                errorMessage: t('customers.deals.list.deleteError', 'Failed to delete deal.'),
+              }),
+            )
           },
           context: {
             formId: singleMutationContextId,
@@ -468,6 +475,12 @@ export default function CustomersDealsPage() {
         setTotal((prev) => Math.max(0, prev - 1))
         handleRefresh()
       } catch (err) {
+        // A stale delete surfaces the unified conflict bar (via the guarded
+        // mutation) — skip the generic error flash to avoid a double message (#2332).
+        if (surfaceRecordConflict(err, t)) {
+          handleRefresh()
+          return
+        }
         const message =
           err instanceof Error && err.message
             ? err.message
@@ -477,7 +490,7 @@ export default function CustomersDealsPage() {
         setPendingDeleteId(null)
       }
     },
-    [confirm, handleRefresh, pendingDeleteId, retrySingleMutation, runSingleMutation, singleMutationContextId, t],
+    [confirm, handleRefresh, pendingDeleteId, retrySingleMutation, rows, runSingleMutation, singleMutationContextId, t],
   )
 
   const handlePageSizeChange = React.useCallback((newSize: number) => {
@@ -1011,7 +1024,7 @@ export default function CustomersDealsPage() {
                 onOpen={() => setFiltersOpen(true)}
               />
               <ActiveFilterChips
-                tree={filterPanel.tree}
+                tree={filterPanel.appliedTree}
                 fields={advancedFilterFields}
                 popoverOpen={filtersOpen}
                 onRemoveNode={(id) => filterPanel.dispatch({ type: 'removeNode', nodeId: id })}
@@ -1026,6 +1039,13 @@ export default function CustomersDealsPage() {
             onClearAll: handleAdvancedFilterClear,
             onRemoveLast: () => filterPanel.dispatch({ type: 'removeLast' }),
           }}
+          emptyState={(
+            <ListEmptyState
+              entityName={t('customers.deals.entityPlural', 'deals')}
+              createHref="/backend/customers/deals/create"
+              createLabel={t('customers.deals.list.actions.new', 'New deal')}
+            />
+          )}
           virtualized
         />
         <AdvancedFilterPanel