@open-mercato/core 0.6.8-develop.7099.1.7ba2771d06 → 0.7.0

package/src/modules/query_index/lib/search-tokens.ts

@@ -223,49 +223,6 @@ export async function deleteSearchTokensForRecord(
     .execute()
 }
 
-// NUL, not a printable separator: a field name may itself contain a space, so `a b` + hash `c`
-// would otherwise sign identically to field `a` + hash `b c`.
-const SIGNATURE_SEPARATOR = String.fromCharCode(0)
-
-// Identifies one token row for comparison. `token` is NULL unless `storeRawTokens` is on, and a
-// stored NULL has to sign the same as the `null` a freshly built row carries — otherwise every
-// record compares as changed and the skip never fires.
-function tokenSignature(row: { field?: unknown; token_hash?: unknown; token?: unknown }): string {
-  return [
-    String(row.field ?? ''),
-    String(row.token_hash ?? ''),
-    row.token == null ? '' : String(row.token),
-  ].join(SIGNATURE_SEPARATOR)
-}
-
-// Multiplicities, not sets: #4681 reports token rows duplicated by the concurrent-replacement
-// defect, and a set comparison reads such a record as already correct and preserves the duplicates
-// forever. Counting sends it through a full rewrite, which collapses them.
-function tallyEquals(a: Map<string, number> | undefined, b: Map<string, number> | undefined): boolean {
-  const left = a ?? new Map<string, number>()
-  const right = b ?? new Map<string, number>()
-  if (left.size !== right.size) return false
-  for (const [key, count] of left.entries()) {
-    if (right.get(key) !== count) return false
-  }
-  return true
-}
-
-function tallyTokenRows<TRow extends { field?: unknown; token_hash?: unknown; token?: unknown }>(
-  rows: Iterable<TRow>,
-  keyOf: (row: TRow) => string
-): Map<string, Map<string, number>> {
-  const tallies = new Map<string, Map<string, number>>()
-  for (const row of rows) {
-    const key = keyOf(row)
-    const tally = tallies.get(key) ?? new Map<string, number>()
-    const signature = tokenSignature(row)
-    tally.set(signature, (tally.get(signature) ?? 0) + 1)
-    tallies.set(key, tally)
-  }
-  return tallies
-}
-
 export async function replaceSearchTokensForBatch(
   db: Kysely<any>,
   payloads: Array<BuildTokenOptions & { doc: Record<string, unknown> }>
@@ -299,95 +256,8 @@ export async function replaceSearchTokensForBatch(
     scopeBuckets.set(key, bucket)
   }
 
-  const recordKeyOf = (row: SearchTokenRow) =>
-    `${scopeKey(row.organization_id ?? null, row.tenant_id ?? null)}|${String(row.entity_id)}`
-  const builtTally = tallyTokenRows(rows, recordKeyOf)
-
-  // Read outside the transaction, deliberately. The comparison decides only whether to skip a
-  // rewrite, so a concurrent writer costs us at most a rewrite we declined — declined because the
-  // table already held exactly the rows this call wanted to write. One ordering is worth naming
-  // though: if the read matches and a concurrent writer then commits tokens built from a *staler*
-  // doc, the unconditional rewrite this call used to perform would have overwritten them by
-  // accident. It no longer does, so those stale rows survive until the record's next write. That
-  // is a repair we lose, not a guarantee we break.
-  const changedIdsByBucket = new Map<string, Set<string>>()
-  for (const [key, bucket] of scopeBuckets.entries()) {
-    const ids = Array.from(bucket.ids)
-    const builtCountById = new Map<string, number>()
-    for (const id of ids) {
-      let total = 0
-      const tally = builtTally.get(`${key}|${id}`)
-      if (tally) for (const count of tally.values()) total += count
-      builtCountById.set(id, total)
-    }
-
-    // Count probe first. Its result is one row per record in the batch, so it is bounded by the
-    // batch size — unlike a bare row read, which would be bounded only by how many token rows the
-    // table already holds for these ids, a quantity this function does not control and (per #4681)
-    // has no reason to trust.
-    const storedCounts = await db
-      .selectFrom('search_tokens' as any)
-      .select(['entity_id' as any, sql<number>`count(*)`.as('token_count') as any])
-      .where('entity_type' as any, '=', payloads[0].entityType)
-      .where(sql<boolean>`organization_id is not distinct from ${bucket.organizationId}`)
-      .where(sql<boolean>`tenant_id is not distinct from ${bucket.tenantId}`)
-      .where('entity_id' as any, 'in', ids)
-      .groupBy('entity_id' as any)
-      .execute()
-    const storedCountById = new Map<string, number>()
-    for (const row of storedCounts as any[]) {
-      storedCountById.set(String(row.entity_id), Number(row.token_count))
-    }
-
-    const changed = new Set<string>()
-    // A record whose stored row count already differs is changed, whatever the rows say — the
-    // duplicate case from #4681 resolves here without ever materializing the duplicated rows.
-    const contentCandidates = ids.filter((id) => {
-      const builtCount = builtCountById.get(id) ?? 0
-      if ((storedCountById.get(id) ?? 0) !== builtCount) {
-        changed.add(id)
-        return false
-      }
-      return builtCount > 0
-    })
-
-    if (contentCandidates.length) {
-      const rowBudget = contentCandidates.reduce((sum, id) => sum + (builtCountById.get(id) ?? 0), 0)
-      const stored = await db
-        .selectFrom('search_tokens' as any)
-        .select(['entity_id' as any, 'field' as any, 'token_hash' as any, 'token' as any])
-        .where('entity_type' as any, '=', payloads[0].entityType)
-        .where(sql<boolean>`organization_id is not distinct from ${bucket.organizationId}`)
-        .where(sql<boolean>`tenant_id is not distinct from ${bucket.tenantId}`)
-        .where('entity_id' as any, 'in', contentCandidates)
-        // Counts already match, so this cannot truncate — it bounds the damage if a concurrent
-        // writer inserts between the probe and this read. A truncated read compares as changed,
-        // which costs a rewrite rather than a wrong skip.
-        .limit(rowBudget)
-        .execute()
-      const storedTally = tallyTokenRows(stored as any[], (row) => String(row.entity_id))
-      for (const id of contentCandidates) {
-        if (!tallyEquals(builtTally.get(`${key}|${id}`), storedTally.get(id))) changed.add(id)
-      }
-    }
-    changedIdsByBucket.set(key, changed)
-  }
-
-  const changedRecordKeys = new Set<string>()
-  for (const [key, changed] of changedIdsByBucket.entries()) {
-    for (const id of changed) changedRecordKeys.add(`${key}|${id}`)
-  }
-  debug('batch.skip', {
-    entityType: payloads[0].entityType,
-    recordCount: payloads.length,
-    changedCount: changedRecordKeys.size,
-  })
-  if (!changedRecordKeys.size) return
-
   await db.transaction().execute(async (trx) => {
-    for (const [key, bucket] of scopeBuckets.entries()) {
-      const changed = changedIdsByBucket.get(key)
-      if (!changed?.size) continue
+    for (const [, bucket] of scopeBuckets.entries()) {
       // Delete by entity_id: a batch replaces all of a record's tokens, and a per-field OR over the
       // whole batch overflows the query compiler's call stack on large batches.
       const deleteQuery = trx
@@ -395,12 +265,10 @@ export async function replaceSearchTokensForBatch(
         .where('entity_type' as any, '=', payloads[0].entityType)
         .where(sql<boolean>`organization_id is not distinct from ${bucket.organizationId}`)
         .where(sql<boolean>`tenant_id is not distinct from ${bucket.tenantId}`)
-        .where('entity_id' as any, 'in', Array.from(changed))
+        .where('entity_id' as any, 'in', Array.from(bucket.ids))
       await deleteQuery.execute()
     }
-    const payloadWithTimestamps = rows
-      .filter((row) => changedRecordKeys.has(recordKeyOf(row)))
-      .map((row) => ({ ...row, created_at: sql`now()` }))
+    const payloadWithTimestamps = rows.map((row) => ({ ...row, created_at: sql`now()` }))
     for (const batch of chunk(payloadWithTimestamps, INSERT_BATCH_SIZE)) {
       await trx.insertInto('search_tokens' as any).values(batch as any).execute()
     }

package/src/modules/resources/backend/resources/resource-types/page.tsx

@@ -45,7 +45,6 @@ type ResourceTypesResponse = {
   items?: Array<Record<string, unknown>>
   total?: number
   totalPages?: number
-  totalIsCapped?: boolean
 }
 
 type ResourceTypesMutationContext = {
@@ -64,7 +63,6 @@ export default function ResourcesResourceTypesPage() {
   const [page, setPage] = React.useState(1)
   const [total, setTotal] = React.useState(0)
   const [totalPages, setTotalPages] = React.useState(1)
-  const [totalIsCapped, setTotalIsCapped] = React.useState(false)
   const [sorting, setSorting] = React.useState<SortingState>([{ id: 'name', desc: false }])
   const [search, setSearch] = React.useState('')
   const [isLoading, setIsLoading] = React.useState(true)
@@ -234,7 +232,6 @@ export default function ResourcesResourceTypesPage() {
       setRows(items.map(mapApiResourceType))
       setTotal(typeof payload.total === 'number' ? payload.total : items.length)
       setTotalPages(typeof payload.totalPages === 'number' ? payload.totalPages : Math.max(1, Math.ceil(items.length / PAGE_SIZE)))
-      setTotalIsCapped(payload?.totalIsCapped === true)
     } catch (error) {
       logger.error('Failed to list resource types', { err: error })
       flash(translations.errors.load, 'error')
@@ -311,7 +308,7 @@ export default function ResourcesResourceTypesPage() {
           sortable
           sorting={sorting}
           onSortingChange={setSorting}
-          pagination={{ page, pageSize: PAGE_SIZE, total, totalPages, totalIsCapped, onPageChange: setPage }}
+          pagination={{ page, pageSize: PAGE_SIZE, total, totalPages, onPageChange: setPage }}
           rowActions={(row) => (
             <RowActions
               items={[

package/src/modules/resources/backend/resources/resources/page.tsx

@@ -63,7 +63,6 @@ type ResourcesResponse = {
   total: number
   page: number
   totalPages: number
-  totalIsCapped?: boolean
 }
 
 type ResourceTypesResponse = {
@@ -82,7 +81,6 @@ export default function ResourcesResourcesPage() {
   const [page, setPage] = React.useState(1)
   const [total, setTotal] = React.useState(0)
   const [totalPages, setTotalPages] = React.useState(1)
-  const [totalIsCapped, setTotalIsCapped] = React.useState(false)
   const [search, setSearch] = React.useState('')
   const [filterValues, setFilterValues] = React.useState<FilterValues>({})
   const [isLoading, setIsLoading] = React.useState(true)
@@ -362,7 +360,6 @@ export default function ResourcesResourcesPage() {
           setRows(mapped)
           setTotal(payload.total || 0)
           setTotalPages(payload.totalPages || 1)
-          setTotalIsCapped(payload?.totalIsCapped === true)
         }
       } catch (error) {
         if (!cancelled) {
@@ -549,7 +546,7 @@ export default function ResourcesResourcesPage() {
               createLabel={t('resources.resources.list.actions.create', 'New resource')}
             />
           )}
-          pagination={{ page, pageSize: PAGE_SIZE, total, totalPages, totalIsCapped, onPageChange: setPage }}
+          pagination={{ page, pageSize: PAGE_SIZE, total, totalPages, onPageChange: setPage }}
           isLoading={isLoading}
         />
       </PageBody>

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

@@ -29,7 +29,6 @@ type OffersResponse = {
   items?: Array<Record<string, unknown>>
   total?: number
   totalPages?: number
-  totalIsCapped?: boolean
 }
 
 const PAGE_SIZE = 25
@@ -62,7 +61,6 @@ export default function SalesChannelOffersListPage() {
   const [page, setPage] = React.useState(1)
   const [total, setTotal] = React.useState(0)
   const [totalPages, setTotalPages] = React.useState(1)
-  const [totalIsCapped, setTotalIsCapped] = React.useState(false)
   const [sorting, setSorting] = React.useState<SortingState>([{ id: 'updatedAt', desc: true }])
   const [search, setSearch] = React.useState('')
   const [filterValues, setFilterValues] = React.useState<FilterValues>({})
@@ -273,7 +271,6 @@ export default function SalesChannelOffersListPage() {
       setRows(mapped)
       setTotal(typeof payload.total === 'number' ? payload.total : items.length)
       setTotalPages(typeof payload.totalPages === 'number' ? payload.totalPages : Math.max(1, Math.ceil(items.length / PAGE_SIZE)))
-      setTotalIsCapped(payload?.totalIsCapped === true)
       const ids = mapped
         .map((row) => row.channelId)
         .filter((value): value is string => typeof value === 'string' && value.length > 0)
@@ -365,7 +362,6 @@ export default function SalesChannelOffersListPage() {
             pageSize: PAGE_SIZE,
             total,
             totalPages,
-            totalIsCapped,
             onPageChange: setPage,
           }}
           refreshButton={{

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

@@ -37,7 +37,6 @@ type ChannelsResponse = {
   items?: Array<Record<string, unknown>>
   total?: number
   totalPages?: number
-  totalIsCapped?: boolean
 }
 
 const PAGE_SIZE = 25
@@ -69,7 +68,6 @@ export default function SalesChannelsPage() {
   const [page, setPage] = React.useState(1)
   const [total, setTotal] = React.useState(0)
   const [totalPages, setTotalPages] = React.useState(1)
-  const [totalIsCapped, setTotalIsCapped] = React.useState(false)
   const [sorting, setSorting] = React.useState<SortingState>([])
   const [search, setSearch] = React.useState('')
   const [isLoading, setLoading] = React.useState(true)
@@ -144,7 +142,6 @@ export default function SalesChannelsPage() {
       setRows(items.map(mapApiChannel))
       setTotal(typeof payload.total === 'number' ? payload.total : items.length)
       setTotalPages(typeof payload.totalPages === 'number' ? payload.totalPages : Math.max(1, Math.ceil(items.length / PAGE_SIZE)))
-      setTotalIsCapped(payload?.totalIsCapped === true)
     } catch (err) {
       logger.error('sales.channels.list', { err })
       flash(t('sales.channels.table.errors.load', 'Failed to load channels.'), 'error')
@@ -233,7 +230,6 @@ export default function SalesChannelsPage() {
             pageSize: PAGE_SIZE,
             total,
             totalPages,
-            totalIsCapped,
             onPageChange: setPage,
           }}
           refreshButton={{

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

@@ -38,7 +38,7 @@ import { buildOptimisticLockHeader } from '@open-mercato/ui/backend/utils/optimi
 import { surfaceRecordConflict } from '@open-mercato/ui/backend/conflicts'
 import { collectCustomFieldValues } from '@open-mercato/ui/backend/utils/customFieldValues'
 import { mapCrudServerErrorToFormErrors } from '@open-mercato/ui/backend/utils/serverErrors'
-import { useT, useLocale } from '@open-mercato/shared/lib/i18n/context'
+import { useT } from '@open-mercato/shared/lib/i18n/context'
 import { useConfirmDialog } from '@open-mercato/ui/backend/confirm-dialog'
 import { cn } from '@open-mercato/shared/lib/utils'
 import { ContactEmailDisplay } from '@open-mercato/core/modules/sales/components/ContactEmailDisplay'
@@ -79,17 +79,13 @@ import { createLogger } from '@open-mercato/shared/lib/logger'
 
 const logger = createLogger('sales')
 
-function formatMessageAmount(
-  amount: number | null | undefined,
-  currency: string | null | undefined,
-  locale?: string
-): string | null {
+function formatMessageAmount(amount: number | null | undefined, currency: string | null | undefined): string | null {
   if (typeof amount !== 'number' || !Number.isFinite(amount)) return null
-  if (!currency) return amount.toLocaleString(locale)
+  if (!currency) return amount.toLocaleString()
   try {
-    return new Intl.NumberFormat(locale, { style: 'currency', currency }).format(amount)
+    return new Intl.NumberFormat(undefined, { style: 'currency', currency }).format(amount)
   } catch {
-    return `${amount.toLocaleString(locale)} ${currency}`
+    return `${amount.toLocaleString()} ${currency}`
   }
 }
 
@@ -1904,7 +1900,6 @@ export default function SalesDocumentDetailPage({
   includeAmountInMessageMetadata?: boolean
 }) {
   const t = useT()
-  const locale = useLocale()
   const { enabled: channelsEnabled } = useSalesChannelsEnabled()
   const router = useRouter()
   const searchParams = useSearchParams()
@@ -2812,7 +2807,7 @@ export default function SalesDocumentDetailPage({
       : null
   const contactEmail = resolveCustomerEmail(customerSnapshot) ?? metadataEmail ?? record?.contactEmail ?? null
   const statusDisplay = record?.status ? statusDictionaryMap[record.status] ?? null : null
-  const previewAmount = formatMessageAmount(record?.grandTotalGrossAmount ?? null, record?.currencyCode ?? null, locale)
+  const previewAmount = formatMessageAmount(record?.grandTotalGrossAmount ?? null, record?.currencyCode ?? null)
   const messagePreviewMetadata: Record<string, string> = {}
   if (includeAmountInMessageMetadata && previewAmount) {
     messagePreviewMetadata[t('sales.documents.detail.totals.grandTotalGross')] = previewAmount
@@ -3872,7 +3867,7 @@ export default function SalesDocumentDetailPage({
         renderDisplay: (params) => {
           const { value, emptyLabel } = params
           if (value && value.length) {
-            return <span className="text-sm text-muted-foreground">{new Date(value).toLocaleDateString(locale)}</span>
+            return <span className="text-sm text-muted-foreground">{new Date(value).toLocaleDateString()}</span>
           }
           return <span className="text-sm text-muted-foreground">{emptyLabel}</span>
         },
@@ -3949,7 +3944,6 @@ export default function SalesDocumentDetailPage({
     shippingMethodLoading,
     shippingMethodOptions,
     t,
-    locale,
     kind,
     saveShortcutLabel,
   ])
@@ -4723,7 +4717,7 @@ export default function SalesDocumentDetailPage({
             renderDisplay={({ value, emptyLabel }) =>
               value && value.length ? (
                 <span className="text-sm text-muted-foreground">
-                  {new Date(value).toLocaleDateString(locale)}
+                  {new Date(value).toLocaleDateString()}
                 </span>
               ) : (
                 <span className="text-sm text-muted-foreground">{emptyLabel}</span>

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

@@ -584,11 +584,6 @@ const addressSnapshotSchema = z
   .nullable()
   .optional();
 
-// Mirrors the create schema's `decimal({ min: 0 })` (data/validators.ts). `null`
-// comes first in the union below because the coercion accepts it — `Number(null)`
-// is 0 — and would turn a clear into a written zero.
-const exchangeRateSchema = z.coerce.number().min(0);
-
 export const documentUpdateSchema = z
   .object({
     id: z.string().uuid(),
@@ -607,13 +602,6 @@ export const documentUpdateSchema = z
     orderNumber: z.string().trim().min(1).max(191).optional(),
     quoteNumber: z.string().trim().min(1).max(191).optional(),
     currencyCode: currencyCodeSchema.optional(),
-    // Order-only columns — SalesQuote declares none of them. Declared here
-    // because the create schema declares them and a caller reusing its create
-    // payload on an update otherwise lost them silently; `applyDocumentUpdate`
-    // rejects each with a 400 on a quote rather than dropping it.
-    exchangeRate: z.union([z.null(), exchangeRateSchema]).optional(),
-    paymentStatusEntryId: z.string().uuid().nullable().optional(),
-    fulfillmentStatusEntryId: z.string().uuid().nullable().optional(),
     channelId: z.string().uuid().nullable().optional(),
     statusEntryId: z.string().uuid().nullable().optional(),
     placedAt: z.union([dateOnlySchema, z.null()]).optional(),
@@ -645,9 +633,6 @@ export const documentUpdateSchema = z
       input.expectedDeliveryAt !== undefined ||
       input.channelId !== undefined ||
       input.statusEntryId !== undefined ||
-      input.exchangeRate !== undefined ||
-      input.paymentStatusEntryId !== undefined ||
-      input.fulfillmentStatusEntryId !== undefined ||
       input.shippingAddressId !== undefined ||
       input.billingAddressId !== undefined ||
       input.customerEntityId !== undefined ||
@@ -1213,19 +1198,6 @@ async function applyDocumentUpdate({
   if (typeof input.currencyCode === "string") {
     entity.currencyCode = input.currencyCode;
   }
-  // Orders only — SalesQuote has no exchange_rate column, so the field is
-  // rejected rather than ignored for the same reason as internalNotes above.
-  if (kind === "quote" && input.exchangeRate !== undefined) {
-    throw new CrudHttpError(400, {
-      error: translate(
-        "sales.quotes.exchange_rate_unsupported",
-        "Exchange rate is not available on quotes.",
-      ),
-    });
-  }
-  if (kind === "order" && input.exchangeRate !== undefined) {
-    (entity as SalesOrder).exchangeRate = toNumericString(input.exchangeRate);
-  }
   if (input.channelId !== undefined) {
     if (input.channelId === null) {
       entity.channelId = null;
@@ -1270,62 +1242,6 @@ async function applyDocumentUpdate({
     (entity as any).statusEntryId = input.statusEntryId ?? null;
     (entity as any).status = statusValue;
   }
-  // Orders only — SalesQuote has neither the entry-id nor the derived text
-  // column for payment and fulfillment status, so both are rejected on a quote.
-  if (kind === "quote" && input.paymentStatusEntryId !== undefined) {
-    throw new CrudHttpError(400, {
-      error: translate(
-        "sales.quotes.payment_status_unsupported",
-        "Payment status is not available on quotes.",
-      ),
-    });
-  }
-  if (kind === "order" && input.paymentStatusEntryId !== undefined) {
-    const paymentStatusValue = await resolveDictionaryEntryValue(
-      em,
-      input.paymentStatusEntryId,
-      { tenantId },
-    );
-    if (input.paymentStatusEntryId && !paymentStatusValue) {
-      throw new CrudHttpError(400, {
-        error: translate(
-          "sales.documents.detail.statusInvalid",
-          "Selected status could not be found.",
-        ),
-      });
-    }
-    // Both columns, like the statusEntryId branch above and like order create:
-    // moving the id without its derived text leaves a self-contradicting row.
-    (entity as SalesOrder).paymentStatusEntryId =
-      input.paymentStatusEntryId ?? null;
-    (entity as SalesOrder).paymentStatus = paymentStatusValue;
-  }
-  if (kind === "quote" && input.fulfillmentStatusEntryId !== undefined) {
-    throw new CrudHttpError(400, {
-      error: translate(
-        "sales.quotes.fulfillment_status_unsupported",
-        "Fulfillment status is not available on quotes.",
-      ),
-    });
-  }
-  if (kind === "order" && input.fulfillmentStatusEntryId !== undefined) {
-    const fulfillmentStatusValue = await resolveDictionaryEntryValue(
-      em,
-      input.fulfillmentStatusEntryId,
-      { tenantId },
-    );
-    if (input.fulfillmentStatusEntryId && !fulfillmentStatusValue) {
-      throw new CrudHttpError(400, {
-        error: translate(
-          "sales.documents.detail.statusInvalid",
-          "Selected status could not be found.",
-        ),
-      });
-    }
-    (entity as SalesOrder).fulfillmentStatusEntryId =
-      input.fulfillmentStatusEntryId ?? null;
-    (entity as SalesOrder).fulfillmentStatus = fulfillmentStatusValue;
-  }
   if (input.placedAt !== undefined) {
     if (input.placedAt === null) {
       entity.placedAt = null;
@@ -3816,15 +3732,6 @@ function buildDocumentUpdateChangeKeys(kind: SalesDocumentKind, input: DocumentU
     if (input.internalNotes !== undefined) keys.add("internalNotes");
     if (input.placedAt !== undefined) keys.add("placedAt");
     if (input.expectedDeliveryAt !== undefined) keys.add("expectedDeliveryAt");
-    if (input.exchangeRate !== undefined) keys.add("exchangeRate");
-    if (input.paymentStatusEntryId !== undefined) {
-      keys.add("paymentStatusEntryId");
-      keys.add("paymentStatus");
-    }
-    if (input.fulfillmentStatusEntryId !== undefined) {
-      keys.add("fulfillmentStatusEntryId");
-      keys.add("fulfillmentStatus");
-    }
   }
   if (
     input.shippingAddressId !== undefined ||

package/src/modules/sales/components/PriceWithCurrency.tsx

@@ -1,26 +1,19 @@
 "use client"
 
 import * as React from 'react'
-import { useOptionalLocale } from '@open-mercato/shared/lib/i18n/context'
 import { cn } from '@open-mercato/shared/lib/utils'
 
-/**
- * Pass `locale` to format in the application locale (`useLocale()` in client components).
- * Omitting it keeps the runtime default, which varies per machine and is therefore not
- * assertable in tests.
- */
 export function formatPriceWithCurrency(
   amount: number | string | null | undefined,
   currency: string | null | undefined,
-  fallback = '—',
-  locale?: string
+  fallback = '—'
 ): string {
   if (amount === null || amount === undefined) return fallback
   const parsed = typeof amount === 'string' ? Number(amount) : amount
   if (Number.isNaN(parsed)) return fallback
   if (currency) {
     try {
-      return new Intl.NumberFormat(locale, {
+      return new Intl.NumberFormat(undefined, {
         style: 'currency',
         currency,
         minimumFractionDigits: 2,
@@ -30,7 +23,7 @@ export function formatPriceWithCurrency(
       // fall through to plain number formatting
     }
   }
-  return parsed.toLocaleString(locale, { minimumFractionDigits: 2, maximumFractionDigits: 2 })
+  return parsed.toLocaleString(undefined, { minimumFractionDigits: 2, maximumFractionDigits: 2 })
 }
 
 type PriceWithCurrencyProps = {
@@ -41,14 +34,9 @@ type PriceWithCurrencyProps = {
 }
 
 export function PriceWithCurrency({ amount, currency, fallback = '—', className }: PriceWithCurrencyProps) {
-  // `useOptionalLocale` rather than `useLocale`: this component is deep-importable and had no i18n
-  // dependency at all before it started formatting in the app locale, so making it throw outside
-  // `I18nProvider` would narrow its mounting contract. Without a provider it falls back to the
-  // runtime default, which is exactly the behaviour it had before.
-  const locale = useOptionalLocale()
   const label = React.useMemo(
-    () => formatPriceWithCurrency(amount, currency, fallback, locale),
-    [amount, currency, fallback, locale]
+    () => formatPriceWithCurrency(amount, currency, fallback),
+    [amount, currency, fallback]
   )
   return <span className={cn('font-mono text-sm text-foreground', className)}>{label}</span>
 }

package/src/modules/sales/components/channels/SalesChannelOffersPanel.tsx

@@ -25,7 +25,6 @@ type OffersResponse = {
   items?: Array<Record<string, unknown>>
   total?: number
   totalPages?: number
-  totalIsCapped?: boolean
 }
 
 const PAGE_SIZE = 25
@@ -55,7 +54,6 @@ export function SalesChannelOffersPanel({ channelId, channelName }: { channelId:
   const [page, setPage] = React.useState(1)
   const [total, setTotal] = React.useState(0)
   const [totalPages, setTotalPages] = React.useState(1)
-  const [totalIsCapped, setTotalIsCapped] = React.useState(false)
   const [search, setSearch] = React.useState('')
   const [isLoading, setLoading] = React.useState(true)
   const [sorting, setSorting] = React.useState<SortingState>([])
@@ -136,7 +134,6 @@ export function SalesChannelOffersPanel({ channelId, channelName }: { channelId:
       setRows(items.map(mapOfferRow))
       setTotal(typeof payload.total === 'number' ? payload.total : items.length)
       setTotalPages(typeof payload.totalPages === 'number' ? payload.totalPages : Math.max(1, Math.ceil(items.length / PAGE_SIZE)))
-      setTotalIsCapped(payload?.totalIsCapped === true)
     } catch (err) {
       logger.error('sales.channels.offers', { err })
       flash(t('sales.channels.offers.errors.load', 'Failed to load offers.'), 'error')
@@ -211,7 +208,6 @@ export function SalesChannelOffersPanel({ channelId, channelName }: { channelId:
         pageSize: PAGE_SIZE,
         total,
         totalPages,
-        totalIsCapped,
         onPageChange: setPage,
       }}
       refreshButton={{

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

@@ -23,7 +23,7 @@ import type { SectionAction } from '@open-mercato/core/modules/customers/compone
 import type { SalesAdjustmentKind } from '../../data/entities'
 import { PriceWithCurrency } from '../PriceWithCurrency'
 import { AdjustmentDialog, type AdjustmentRowData, type AdjustmentSubmitPayload } from './AdjustmentDialog'
-import { useT, useLocale } from '@open-mercato/shared/lib/i18n/context'
+import { useT } from '@open-mercato/shared/lib/i18n/context'
 import { useOrganizationScopeDetail } from '@open-mercato/shared/lib/frontend/useOrganizationScope'
 import { extractCustomFieldValues } from './customFieldHelpers'
 import { createLogger } from '@open-mercato/shared/lib/logger'
@@ -60,9 +60,9 @@ function normalizeNumber(value: unknown, fallback = 0): number {
   return fallback
 }
 
-export function formatPercent(value: number | null | undefined, locale?: string): string {
+function formatPercent(value: number | null | undefined): string {
   if (value === null || value === undefined) return '—'
-  const formatter = new Intl.NumberFormat(locale, { style: 'percent', maximumFractionDigits: 2 })
+  const formatter = new Intl.NumberFormat(undefined, { style: 'percent', maximumFractionDigits: 2 })
   return formatter.format(value / 100)
 }
 
@@ -77,7 +77,6 @@ export function SalesDocumentAdjustmentsSection({
   onRowsChange,
 }: SalesDocumentAdjustmentsSectionProps) {
   const t = useT()
-  const locale = useLocale()
   const { organizationId, tenantId } = useOrganizationScopeDetail()
   const resolvedOrganizationId = orgFromProps ?? organizationId ?? null
   const resolvedTenantId = tenantFromProps ?? tenantId ?? null
@@ -434,7 +433,7 @@ export function SalesDocumentAdjustmentsSection({
       {
         id: 'rate',
         header: t('sales.documents.adjustments.rate', 'Rate'),
-        cell: ({ row }) => <span className="font-mono text-sm text-muted-foreground">{formatPercent(row.original.rate, locale)}</span>,
+        cell: ({ row }) => <span className="font-mono text-sm text-muted-foreground">{formatPercent(row.original.rate)}</span>,
       },
       {
         id: 'position',
@@ -442,7 +441,7 @@ export function SalesDocumentAdjustmentsSection({
         cell: ({ row }) => <span className="text-sm text-muted-foreground">{row.original.position}</span>,
       },
     ],
-    [currencyCode, locale, resolveKindLabel, t]
+    [currencyCode, resolveKindLabel, t]
   )
 
   const showLoadingState = loading && rows.length === 0