@open-mercato/core 0.6.7-develop.6862.1.c11a64ce0a → 0.6.7

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

@@ -1,7 +1,6 @@
 "use client"
 
 import * as React from 'react'
-import { extensionPoints } from '@open-mercato/core/modules/customers/extension-points'
 import Link from 'next/link'
 import { usePathname, useRouter, useSearchParams } from 'next/navigation'
 import { Page, PageBody } from '@open-mercato/ui/backend/Page'
@@ -915,7 +914,7 @@ export default function CustomersPeoplePage() {
           onSearchChange={(value) => { setSearch(value); setPage(1) }}
           searchPlaceholder={t('customers.people.list.searchPlaceholder')}
           entityIds={[E.customers.customer_entity, E.customers.customer_person_profile]}
-          perspective={{ tableId: extensionPoints.hosts.peopleTable.tableId }}
+          perspective={{ tableId: 'customers.people.list' }}
           onRowClick={(row) => router.push(`/backend/customers/people-v2/${row.id}`)}
           sortable
           manualSorting

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

@@ -1,7 +1,6 @@
 "use client"
 
 import * as React from 'react'
-import { extensionPoints } from '@open-mercato/core/modules/customers/extension-points'
 import Link from 'next/link'
 import { useRouter, useSearchParams, usePathname } from 'next/navigation'
 import { User, Hash, Users, Building2 } from 'lucide-react'
@@ -92,6 +91,10 @@ export default function PersonDetailV2Page({ params }: { params?: { id?: string
   const [isSaving, setIsSaving] = React.useState(false)
   const formWrapperRef = React.useRef<HTMLDivElement>(null)
 
+  const initialTab = React.useMemo(() => {
+    return resolveLegacyTab(searchParams?.get('tab'))
+  }, [searchParams])
+  const [activeTab, setActiveTab] = React.useState<PersonTabId>(initialTab)
   const [sectionAction, setSectionAction] = React.useState<SectionAction | null>(null)
   const [scheduleDialogOpen, setScheduleDialogOpen] = React.useState(false)
   const [scheduleEditData, setScheduleEditData] = React.useState<ScheduleActivityEditData | null>(null)
@@ -325,28 +328,6 @@ export default function PersonDetailV2Page({ params }: { params?: { id?: string
 
   const injectedTabMap = React.useMemo(() => new Map(injectedTabs.map((tab) => [tab.id, tab.render])), [injectedTabs])
 
-  const injectedTabIds = React.useMemo(() => injectedTabs.map((tab) => tab.id), [injectedTabs])
-  const initialTab = React.useMemo(
-    () => resolveLegacyTab(searchParams?.get('tab'), injectedTabIds),
-    [injectedTabIds, searchParams],
-  )
-  const [activeTab, setActiveTab] = React.useState<PersonTabId>(initialTab)
-
-  React.useEffect(() => {
-    setActiveTab(initialTab)
-  }, [initialTab])
-
-  const handleTabChange = React.useCallback(
-    (tab: PersonTabId) => {
-      setActiveTab(tab)
-      if (!pathname) return
-      const nextParams = new URLSearchParams(searchParams?.toString() ?? '')
-      nextParams.set('tab', tab)
-      router.replace(`${pathname}?${nextParams.toString()}`, { scroll: false })
-    },
-    [pathname, router, searchParams],
-  )
-
   // Tags
   const handleTagsChange = React.useCallback((nextTags: TagSummary[]) => {
     setData((prev) => (prev ? { ...prev, tags: nextTags } : prev))
@@ -521,8 +502,8 @@ export default function PersonDetailV2Page({ params }: { params?: { id?: string
       <PageBody>
         <div className="space-y-4">
           {/* UMES header injection (third-party extensions) */}
-          <InjectionSpot spotId={extensionPoints.hosts.personHeader.spotId} context={injectionContext} data={data} />
-          <InjectionSpot spotId={extensionPoints.hosts.personStatusBadges.spotId} context={injectionContext} data={data} />
+          <InjectionSpot spotId="detail:customers.person:header" context={injectionContext} data={data} />
+          <InjectionSpot spotId="detail:customers.person:status-badges" context={injectionContext} data={data} />
 
           {/* Persistent person header */}
           <PersonDetailHeader
@@ -533,7 +514,7 @@ export default function PersonDetailV2Page({ params }: { params?: { id?: string
             onDelete={handleFormDelete}
             isDirty={isDirty}
             isSaving={isSaving}
-            onOpenCompaniesTab={() => handleTabChange('companies')}
+            onOpenCompaniesTab={() => setActiveTab('companies')}
             onDataReload={() => { loadData().catch((err) => logger.warn('onDataReload failed', { component: 'people-v2', err })) }}
             onFocusField={(fieldName) => {
               const selectorMap: Record<string, string> = {
@@ -556,7 +537,7 @@ export default function PersonDetailV2Page({ params }: { params?: { id?: string
                 <CrudForm<PersonEditFormValues>
                   embedded
                   trackDirtyWhenEmbedded
-                  injectionSpotId={extensionPoints.hosts.personForm.spotId}
+                  injectionSpotId="crud-form:customers.person"
                   entityIds={[E.customers.customer_entity, E.customers.customer_person_profile]}
                   schema={formSchema}
                   fields={fields}
@@ -575,7 +556,7 @@ export default function PersonDetailV2Page({ params }: { params?: { id?: string
             const zone2Content = (
               <PersonDetailTabs
                 activeTab={activeTab}
-                onTabChange={handleTabChange}
+                onTabChange={setActiveTab}
                 injectedTabs={injectedTabs.map((tab) => ({ id: tab.id, label: tab.label }))}
                 activitiesCount={interactionCount}
                 dealsCount={dealCount}
@@ -738,7 +719,7 @@ export default function PersonDetailV2Page({ params }: { params?: { id?: string
           })()}
 
           {/* UMES footer injection */}
-          <InjectionSpot spotId={extensionPoints.hosts.personFooter.spotId} context={injectionContext} data={data} />
+          <InjectionSpot spotId="detail:customers.person:footer" context={injectionContext} data={data} />
 
           {/* Schedule Activity Dialog — opened from PlannedActivities "+ Schedule" or other triggers */}
           <ScheduleActivityDialog

package/src/modules/customers/cli.ts

@@ -12,8 +12,6 @@ import { E as CoreEntities } from '#generated/entities.ids.generated'
 import { createProgressBar } from '@open-mercato/shared/lib/cli/progress'
 import { buildIndexDocument, type IndexCustomFieldValue } from '@open-mercato/core/modules/query_index/lib/document'
 import { parseBooleanToken } from '@open-mercato/shared/lib/boolean'
-import { findWithDecryption } from '@open-mercato/shared/lib/encryption/find'
-import { resolveSearchConfig } from '@open-mercato/shared/lib/search/config'
 import type { QueryEngine } from '@open-mercato/shared/lib/query/types'
 import type { EntityId } from '@open-mercato/shared/modules/entities'
 import {
@@ -1397,22 +1395,13 @@ async function seedCustomerExamples(
     )
   )
   if (exampleDealTitles.length > 0) {
-    const exampleTitles = new Set(exampleDealTitles)
-    const batchSize = 100
-    let offset = 0
-    while (true) {
-      const existingDeals = await findWithDecryption(
-        em,
-        CustomerDeal,
-        { tenantId, organizationId },
-        { fields: ['title'], limit: batchSize, offset, orderBy: { id: 'asc' } },
-        { tenantId, organizationId },
-      )
-      if (existingDeals.some((deal) => typeof deal.title === 'string' && exampleTitles.has(deal.title))) {
-        return false
-      }
-      if (existingDeals.length < batchSize) break
-      offset += existingDeals.length
+    const already = await em.count(CustomerDeal, {
+      tenantId,
+      organizationId,
+      title: { $in: exampleDealTitles as any },
+    })
+    if (already > 0) {
+      return false
     }
   }
 
@@ -2005,7 +1994,6 @@ async function seedCustomerStressTest(
       updated_at: Date
       deleted_at: null
     }> = []
-    const searchConfig = resolveSearchConfig()
     for (const [entityType, bucket] of pendingIndexDocs.entries()) {
       for (const entry of bucket.values()) {
         if (!entry.baseRow || Object.keys(entry.baseRow).length === 0) continue
@@ -2014,15 +2002,10 @@ async function seedCustomerStressTest(
           entity_id: entry.recordId,
           organization_id: entry.organizationId,
           tenant_id: entry.tenantId,
-          doc: buildIndexDocument(
-            entry.baseRow,
-            entry.customFields,
-            {
-              organizationId: entry.organizationId,
-              tenantId: entry.tenantId,
-            },
-            { entityType, config: searchConfig },
-          ),
+          doc: buildIndexDocument(entry.baseRow, entry.customFields, {
+            organizationId: entry.organizationId,
+            tenantId: entry.tenantId,
+          }),
           index_version: 1,
           created_at: entry.createdAt,
           updated_at: entry.updatedAt,

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

@@ -273,7 +273,6 @@ type DealSnapshot = {
     lossNotes: string | null
   }
   people: string[]
-  primaryPersonEntityId?: string | null
   companies: string[]
   transitions: DealStageTransitionSnapshot[]
   custom?: Record<string, unknown>
@@ -325,7 +324,6 @@ async function loadDealSnapshot(em: EntityManager, id: string): Promise<DealSnap
     tenantId: deal.tenantId,
     organizationId: deal.organizationId,
   })
-  const primaryPerson = peopleLinks.find((link) => link.isPrimary)?.person
   return {
     deal: {
       id: deal.id,
@@ -350,8 +348,6 @@ async function loadDealSnapshot(em: EntityManager, id: string): Promise<DealSnap
     people: peopleLinks.map((link) =>
       typeof link.person === 'string' ? link.person : link.person.id
     ),
-    primaryPersonEntityId:
-      typeof primaryPerson === 'string' ? primaryPerson : primaryPerson?.id ?? null,
     companies: companyLinks.map((link) =>
       typeof link.company === 'string' ? link.company : link.company.id
     ),
@@ -376,55 +372,18 @@ function toNumericString(value: number | null | undefined): string | null {
 async function syncDealPeople(
   em: EntityManager,
   deal: CustomerDeal,
-  personIds: string[] | undefined | null,
-  primaryPersonEntityId?: string | null
+  personIds: string[] | undefined | null
 ): Promise<void> {
-  if (personIds === undefined) {
-    if (primaryPersonEntityId === undefined) return
-    const links = await em.find(CustomerDealPersonLink, { deal })
-    if (primaryPersonEntityId !== null && !links.some((link) => link.person.id === primaryPersonEntityId)) {
-      const { translate } = await resolveTranslations()
-      throw new CrudHttpError(400, {
-        error: translate(
-          'customers.errors.primaryPersonMustBeLinked',
-          'Primary person must be linked to the deal',
-        ),
-      })
-    }
-    for (const link of links) {
-      link.isPrimary = false
-    }
-    await em.flush()
-    if (primaryPersonEntityId !== null) {
-      const primaryLink = links.find((link) => link.person.id === primaryPersonEntityId)
-      if (primaryLink) primaryLink.isPrimary = true
-    }
-    return
-  }
-  const unique = Array.from(new Set(personIds ?? []))
-  if (primaryPersonEntityId !== undefined && primaryPersonEntityId !== null && !unique.includes(primaryPersonEntityId)) {
-    const { translate } = await resolveTranslations()
-    throw new CrudHttpError(400, {
-      error: translate(
-        'customers.errors.primaryPersonMustBeLinked',
-        'Primary person must be linked to the deal',
-      ),
-    })
-  }
-  let effectivePrimaryId = primaryPersonEntityId
-  if (effectivePrimaryId === undefined) {
-    const existingPrimary = await em.findOne(CustomerDealPersonLink, { deal, isPrimary: true })
-    effectivePrimaryId = existingPrimary?.person?.id ?? null
-  }
+  if (personIds === undefined) return
   await em.nativeDelete(CustomerDealPersonLink, { deal })
-  if (!unique.length) return
+  if (!personIds || !personIds.length) return
+  const unique = Array.from(new Set(personIds))
   for (const personId of unique) {
     const person = await requireCustomerEntity(em, personId, { tenantId: deal.tenantId, organizationId: deal.organizationId }, 'person', 'Person not found')
     ensureSameScope(person, deal.organizationId, deal.tenantId)
     const link = em.create(CustomerDealPersonLink, {
       deal,
       person,
-      isPrimary: personId === effectivePrimaryId,
     })
     em.persist(link)
   }
@@ -520,7 +479,7 @@ const createDealCommand: CommandHandler<DealCreateInput, { dealId: string }> = {
           transitionedByUserId: normalizedTransitionAuthorUserId,
         })
       },
-      () => syncDealPeople(em, deal, parsed.personIds ?? [], parsed.primaryPersonEntityId),
+      () => syncDealPeople(em, deal, parsed.personIds ?? []),
       () => syncDealCompanies(em, deal, parsed.companyIds ?? []),
     ], { transaction: true })
 
@@ -616,7 +575,7 @@ const createDealCommand: CommandHandler<DealCreateInput, { dealId: string }> = {
     }
     const restoredDeal = deal
     await withAtomicFlush(em, [
-      () => syncDealPeople(em, restoredDeal, after.people, after.primaryPersonEntityId),
+      () => syncDealPeople(em, restoredDeal, after.people),
       () => syncDealCompanies(em, restoredDeal, after.companies),
       () => deleteDealStageTransitions(em, restoredDeal),
       () => restoreDealStageTransitions(em, restoredDeal, after.transitions),
@@ -787,7 +746,7 @@ const updateDealCommand: CommandHandler<DealUpdateInput, { dealId: string }> = {
             transitionedByUserId: normalizedTransitionAuthorUserId,
           })
         },
-        () => syncDealPeople(em, record, parsed.personIds, parsed.primaryPersonEntityId),
+        () => syncDealPeople(em, record, parsed.personIds),
         () => syncDealCompanies(em, record, parsed.companyIds),
       ],
     })
@@ -930,7 +889,7 @@ const updateDealCommand: CommandHandler<DealUpdateInput, { dealId: string }> = {
           transitionedByUserId: normalizedTransitionAuthorUserId,
         })
       },
-      () => syncDealPeople(em, deal, before.people, before.primaryPersonEntityId),
+      () => syncDealPeople(em, deal, before.people),
       () => syncDealCompanies(em, deal, before.companies),
     ], { transaction: true })
 
@@ -1049,7 +1008,7 @@ const deleteDealCommand: CommandHandler<{ body?: Record<string, unknown>; query?
         em.persist(deal)
       }
       await withAtomicFlush(em, [
-        () => syncDealPeople(em, deal, before.people, before.primaryPersonEntityId),
+        () => syncDealPeople(em, deal, before.people),
         () => syncDealCompanies(em, deal, before.companies),
         () => deleteDealStageTransitions(em, deal),
         () => restoreDealStageTransitions(em, deal, before.transitions),

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

@@ -181,7 +181,6 @@ type PersonSnapshot = {
     id: string
     dealId: string
     participantRole: string | null
-    isPrimary?: boolean
     createdAt: Date
   }>
   activities: PersonActivitySnapshot[]
@@ -346,7 +345,6 @@ function serializePersonSnapshot(
         id: link.id,
         dealId: link.deal.id,
         participantRole: link.participantRole ?? null,
-        isPrimary: link.isPrimary === true,
         createdAt: link.createdAt,
       })),
     activities: activities.map((activity) => ({
@@ -1524,7 +1522,6 @@ const deletePersonCommand: CommandHandler<{ body?: Record<string, unknown>; quer
           deal,
           person: entity,
           participantRole: link.participantRole,
-          isPrimary: link.isPrimary === true,
           createdAt: link.createdAt,
         })
         em.persist(restoredLink)

package/src/modules/customers/components/CustomerTodosTable.tsx

@@ -1,7 +1,6 @@
 "use client"
 
 import * as React from 'react'
-import { extensionPoints } from '@open-mercato/core/modules/customers/extension-points'
 import Link from 'next/link'
 import { useRouter } from 'next/navigation'
 import type { ColumnDef } from '@tanstack/react-table'
@@ -254,7 +253,7 @@ export function CustomerTodosTable(): React.JSX.Element {
         setSearch(value)
         setPage(1)
       }}
-      perspective={{ tableId: extensionPoints.hosts.todosTable.tableId }}
+      perspective={{ tableId: 'customers.todos.list' }}
       rowActions={(row) => {
         const customerLink = buildCustomerHref(row)
         const todoHref = row.externalHref ?? resolveTodoHref(row.todoSource, row.todoId)

package/src/modules/customers/components/detail/ActiveDealCard.tsx

@@ -7,7 +7,6 @@ import { useT } from '@open-mercato/shared/lib/i18n/context'
 import { Badge } from '@open-mercato/ui/primitives/badge'
 import { IconButton } from '@open-mercato/ui/primitives/icon-button'
 import { readApiResultOrThrow } from '@open-mercato/ui/backend/utils/apiCall'
-import { isOpenDealStatus } from '../../lib/dealStatus'
 import type { DealSummary } from '../formConfig'
 import { formatCurrency } from './utils'
 
@@ -26,7 +25,7 @@ export function ActiveDealCard({ deals, onHide }: ActiveDealCardProps) {
   const [pipelineStages, setPipelineStages] = React.useState<PipelineStageOption[]>([])
 
   const activeDeals = React.useMemo(
-    () => deals.filter((d) => isOpenDealStatus(d.status)),
+    () => deals.filter((d) => d.status !== 'won' && d.status !== 'lost' && d.status !== 'closed'),
     [deals],
   )
 

package/src/modules/customers/components/detail/ActivitiesDayStrip.tsx

@@ -245,8 +245,9 @@ export function ActivitiesDayStrip({ entityId, selectedDate, onSelectDate, refre
         )
         setFetchedEvents(Array.isArray(payload?.items) ? payload.items : [])
       } catch (err) {
-        if ((err as { name?: string } | null)?.name === 'AbortError') return
-        logger.warn('failed to load interactions', { component: 'ActivitiesDayStrip', err })
+        if ((err as { name?: string } | null)?.name !== 'AbortError') {
+          logger.warn('failed to load interactions', { component: 'ActivitiesDayStrip', err })
+        }
         setFetchedEvents([])
       }
     })()

package/src/modules/customers/components/detail/CompanyDetailTabs.tsx

@@ -13,8 +13,6 @@ import {
   Plus,
 } from 'lucide-react'
 import type { SectionAction } from '@open-mercato/ui/backend/detail'
-import { registerComponent } from '@open-mercato/shared/modules/widgets/component-registry'
-import { useRegisteredComponent } from '@open-mercato/ui/backend/injection/useRegisteredComponent'
 import { useDealsAccess } from './useDealsAccess'
 
 export type CompanyTabId =
@@ -32,13 +30,10 @@ type TabDef = {
   count?: React.ReactNode
 }
 
-export const COMPANY_DETAIL_TABS_COMPONENT_ID = 'section:customers.companies.detailTabs'
-
-export type CompanyDetailTabsProps = {
+type CompanyDetailTabsProps = {
   activeTab: CompanyTabId
   onTabChange: (tab: CompanyTabId) => void
   injectedTabs?: Array<{ id: string; label: string; priority?: number }>
-  hiddenTabIds?: string[]
   peopleCount?: number
   dealsCount?: number
   activitiesCount?: number
@@ -68,11 +63,10 @@ function formatTabCount(count: number): string | number | undefined {
   return count > 999 ? '999+' : count
 }
 
-function DefaultCompanyDetailTabs({
+export function CompanyDetailTabs({
   activeTab,
   onTabChange,
   injectedTabs = [],
-  hiddenTabIds = [],
   peopleCount = 0,
   dealsCount = 0,
   activitiesCount = 0,
@@ -123,16 +117,16 @@ function DefaultCompanyDetailTabs({
     [t, canViewDeals, peopleCount, dealsCount, activitiesCount, filesCount],
   )
 
-  const allTabs: TabDef[] = React.useMemo(() => {
-    const hidden = new Set(hiddenTabIds)
-    return [
+  const allTabs: TabDef[] = React.useMemo(
+    () => [
       ...builtInTabs,
       ...injectedTabs.map((tab) => ({
         id: tab.id as CompanyTabId,
         label: tab.label,
       })),
-    ].filter((tab) => !hidden.has(tab.id))
-  }, [builtInTabs, hiddenTabIds, injectedTabs])
+    ],
+    [builtInTabs, injectedTabs],
+  )
 
   return (
     <div>
@@ -176,20 +170,3 @@ function DefaultCompanyDetailTabs({
     </div>
   )
 }
-
-registerComponent<CompanyDetailTabsProps>({
-  id: COMPANY_DETAIL_TABS_COMPONENT_ID,
-  component: DefaultCompanyDetailTabs,
-  metadata: {
-    module: 'customers',
-    description: 'Company detail tab navigation and content.',
-  },
-})
-
-export function CompanyDetailTabs(props: CompanyDetailTabsProps) {
-  const ResolvedTabs = useRegisteredComponent<CompanyDetailTabsProps>(
-    COMPANY_DETAIL_TABS_COMPONENT_ID,
-    DefaultCompanyDetailTabs,
-  )
-  return <ResolvedTabs {...props} />
-}

package/src/modules/customers/components/detail/CompanyKpiBar.tsx

@@ -11,7 +11,6 @@ import {
   writeVersionedIdSet,
   clearVersionedPreference,
 } from '@open-mercato/shared/lib/browser/versionedPreference'
-import { isOpenDealStatus, isWonDealStatus } from '../../lib/dealStatus'
 import type { CompanyOverview, DealSummary, InteractionSummary } from '../formConfig'
 import { formatCurrency } from './utils'
 
@@ -20,7 +19,7 @@ const STORAGE_VERSION = 1
 
 function sumActiveDeals(deals: DealSummary[]): number {
   return deals
-    .filter((d) => isOpenDealStatus(d.status))
+    .filter((d) => d.status !== 'won' && d.status !== 'lost' && d.status !== 'closed')
     .reduce((sum, d) => {
       const amount = typeof d.valueAmount === 'number' ? d.valueAmount : parseFloat(String(d.valueAmount ?? '0'))
       return sum + (Number.isFinite(amount) ? amount : 0)
@@ -28,7 +27,7 @@ function sumActiveDeals(deals: DealSummary[]): number {
 }
 
 function getActiveDeals(deals: DealSummary[]): DealSummary[] {
-  return deals.filter((d) => isOpenDealStatus(d.status))
+  return deals.filter((d) => d.status !== 'won' && d.status !== 'lost' && d.status !== 'closed')
 }
 
 function computeActivityTrend(interactions: InteractionSummary[]): KpiTrend | undefined {
@@ -52,7 +51,7 @@ function computeActivityTrend(interactions: InteractionSummary[]): KpiTrend | un
 }
 
 function computeDealTrend(deals: DealSummary[]): KpiTrend | undefined {
-  const active = deals.filter((d) => isOpenDealStatus(d.status))
+  const active = deals.filter((d) => d.status !== 'won' && d.status !== 'lost' && d.status !== 'closed')
   if (active.length === 0) return undefined
   const now = Date.now()
   const monthMs = 30 * 86_400_000
@@ -82,7 +81,7 @@ export function CompanyKpiBar({ data }: CompanyKpiBarProps) {
 
   const ltvValue = React.useMemo(() => {
     if (data.kpis?.ltvValue !== undefined) return data.kpis.ltvValue
-    const wonDeals = data.deals.filter((d) => isWonDealStatus(d.status))
+    const wonDeals = data.deals.filter((d) => d.status === 'won')
     if (wonDeals.length === 0) return null
     return wonDeals.reduce((sum, d) => {
       const amt = typeof d.valueAmount === 'number' ? d.valueAmount : parseFloat(String(d.valueAmount ?? '0'))
@@ -155,7 +154,7 @@ export function CompanyKpiBar({ data }: CompanyKpiBarProps) {
           : `${v} ${v === 1 ? t('customers.companies.dashboard.kpi.year', 'year') : t('customers.companies.dashboard.kpi.years', 'years')}`
         : undefined,
       comparisonLabel: clientTenureYears !== null
-        ? `${data.kpis?.completedDealsCount ?? data.deals.filter((d) => isWonDealStatus(d.status)).length} ${t('customers.companies.dashboard.kpi.completedDeals', 'completed deals')}`
+        ? `${data.kpis?.completedDealsCount ?? data.deals.filter((d) => d.status === 'won').length} ${t('customers.companies.dashboard.kpi.completedDeals', 'completed deals')}`
         : t('customers.companies.dashboard.kpi.noInteractions', 'No interactions yet'),
     },
   ], [t, activeDealsValue, dealTrend, dealCurrency, activeDeals.length, activityTrend, ltvValue, clientTenureYears, data.deals, data.interactions.length, data.kpis])

package/src/modules/customers/components/detail/ConfirmDealLostDialog.tsx

@@ -253,7 +253,7 @@ export function ConfirmDealLostDialog({
             </Button>
             <Button
               type="button"
-              variant="destructive-solid"
+              variant="destructive"
               disabled={confirmDisabled}
               onClick={() => { void handleConfirm() }}
             >

package/src/modules/customers/components/detail/DealDetailTabs.tsx

@@ -3,8 +3,6 @@
 import * as React from 'react'
 import { Activity, Building2, History, NotebookPen, Paperclip, Users } from 'lucide-react'
 import { useT } from '@open-mercato/shared/lib/i18n/context'
-import { registerComponent } from '@open-mercato/shared/modules/widgets/component-registry'
-import { useRegisteredComponent } from '@open-mercato/ui/backend/injection/useRegisteredComponent'
 import { Tabs, TabsList, TabsTrigger } from '@open-mercato/ui/primitives/tabs'
 
 export type DealTabId =
@@ -23,13 +21,10 @@ type TabDef = {
   count?: React.ReactNode
 }
 
-export const DEAL_DETAIL_TABS_COMPONENT_ID = 'section:customers.deals.detailTabs'
-
-export type DealDetailTabsProps = {
+type DealDetailTabsProps = {
   activeTab: DealTabId
   onTabChange: (tab: DealTabId) => void
   injectedTabs?: Array<{ id: string; label: string }>
-  hiddenTabIds?: string[]
   peopleCount?: number
   companiesCount?: number
   children: React.ReactNode
@@ -37,11 +32,9 @@ export type DealDetailTabsProps = {
 
 const SUPPORTED_TAB_IDS = new Set<DealTabId>(['activities', 'people', 'companies', 'notes', 'files', 'changelog'])
 
-export function resolveLegacyTab(tab: string | null | undefined, knownTabIds?: Iterable<string>): DealTabId {
+export function resolveLegacyTab(tab: string | null | undefined): DealTabId {
   if (!tab) return 'activities'
-  if (SUPPORTED_TAB_IDS.has(tab as DealTabId)) return tab as DealTabId
-  if (knownTabIds && new Set(knownTabIds).has(tab)) return tab
-  return 'activities'
+  return SUPPORTED_TAB_IDS.has(tab as DealTabId) ? (tab as DealTabId) : 'activities'
 }
 
 function formatTabCount(count: number): string | number | undefined {
@@ -49,11 +42,10 @@ function formatTabCount(count: number): string | number | undefined {
   return count > 999 ? '999+' : count
 }
 
-function DefaultDealDetailTabs({
+export function DealDetailTabs({
   activeTab,
   onTabChange,
   injectedTabs = [],
-  hiddenTabIds = [],
   peopleCount = 0,
   companiesCount = 0,
   children,
@@ -99,16 +91,16 @@ function DefaultDealDetailTabs({
     [companiesCount, peopleCount, t],
   )
 
-  const allTabs = React.useMemo<TabDef[]>(() => {
-    const hidden = new Set(hiddenTabIds)
-    return [
+  const allTabs = React.useMemo<TabDef[]>(
+    () => [
       ...builtInTabs,
       ...injectedTabs.map((tab) => ({
         id: tab.id as DealTabId,
         label: tab.label,
       })),
-    ].filter((tab) => !hidden.has(tab.id))
-  }, [builtInTabs, hiddenTabIds, injectedTabs])
+    ],
+    [builtInTabs, injectedTabs],
+  )
 
   return (
     <div>
@@ -135,20 +127,3 @@ function DefaultDealDetailTabs({
     </div>
   )
 }
-
-registerComponent<DealDetailTabsProps>({
-  id: DEAL_DETAIL_TABS_COMPONENT_ID,
-  component: DefaultDealDetailTabs,
-  metadata: {
-    module: 'customers',
-    description: 'Deal detail tab navigation and content.',
-  },
-})
-
-export function DealDetailTabs(props: DealDetailTabsProps) {
-  const ResolvedTabs = useRegisteredComponent<DealDetailTabsProps>(
-    DEAL_DETAIL_TABS_COMPONENT_ID,
-    DefaultDealDetailTabs,
-  )
-  return <ResolvedTabs {...props} />
-}

package/src/modules/customers/components/detail/DealForm.tsx

@@ -15,7 +15,6 @@ import {
 } from '@open-mercato/ui/primitives/select'
 import { apiCall } from '@open-mercato/ui/backend/utils/apiCall'
 import { createCrudFormError } from '@open-mercato/ui/backend/utils/serverErrors'
-import { DEAL_DESCRIPTION_MAX_LENGTH } from '../../data/validators'
 import { DictionarySelectField } from '../formConfig'
 import { createDictionarySelectLabels } from './utils'
 import { E } from '#generated/entities.ids.generated'
@@ -264,10 +263,7 @@ const schema = z.object({
       'customers.people.detail.deals.expectedCloseInvalid',
     )
     .optional(),
-  description: z
-    .string()
-    .max(DEAL_DESCRIPTION_MAX_LENGTH, 'customers.people.detail.deals.descriptionTooLong')
-    .optional(),
+  description: z.string().max(4000, 'customers.people.detail.deals.descriptionTooLong').optional(),
   personIds: z.array(z.string().trim().min(1)).optional(),
   companyIds: z.array(z.string().trim().min(1)).optional(),
 }).passthrough()

package/src/modules/customers/components/detail/DealsSection.tsx

@@ -988,7 +988,7 @@ export function DealsSection({
                     {deal.title || emptyLabel}
                   </Link>
                   {deal.description ? (
-                    <p className="mt-1 line-clamp-3 text-sm text-muted-foreground whitespace-pre-wrap">{deal.description}</p>
+                    <p className="mt-1 text-sm text-muted-foreground whitespace-pre-wrap">{deal.description}</p>
                   ) : null}
                 </div>
                 <div className="flex items-center gap-2">