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

package/src/modules/sales/migrations/Migration20260821120000_sales_notes_addresses_context_idx.ts

@@ -1,87 +0,0 @@
-import { Migration } from '@mikro-orm/migrations';
-
-// sales_notes and sales_document_addresses each carried a single index —
-// (organization_id, tenant_id) — and are never read by that predicate alone.
-// loadOrderSnapshot / loadQuoteSnapshot (commands/documents.ts) read them by
-// context in the same Promise.all:
-//
-//   findWithDecryption(em, SalesNote, { contextType, contextId, ...scope })
-//   findWithDecryption(em, SalesDocumentAddress, { documentId, documentKind, ...scope })
-//
-// On a single-organization deployment the scope index matches every row, so the
-// planner discards it and scans. The cost is CPU filtering, so a warm cache does
-// not reduce it. Measured on a table of ~331k rows / 110 MB: a parallel
-// sequential scan removing 110,457 rows per worker, 82 ms per read to return 0
-// rows. The command bus runs prepare() and captureAfter() around every command
-// and both load a document snapshot, so one written document — itself several
-// commands (the document, one per changed line, the payment reconcile) —
-// performs roughly 20 of these reads. Counters over one bulk-import batch of 100
-// records showed 1,971 sequential scans reading 217,670,528 rows, 2.18M rows per
-// imported record; repeatable across consecutive batches (1,971 / 1,974 / 1,911).
-//
-// context_type and document_kind are deliberately omitted, as they were for
-// sales_document_tag_assignments_document_idx (Migration20260806120000), and the
-// scope columns with them. sales_orders.id and sales_quotes.id are both
-// gen_random_uuid() from their own tables, so a context_id already determines its
-// context_type and its tenant; the seek returns the handful of rows belonging to
-// one document and the remaining terms are rechecked over those. Folding three
-// more columns in — one text and two uuids — roughly triples the index entry for
-// no selectivity, on tables that take a write per document save and a note per
-// order status change. The sibling scope indexes that do fold scope in
-// (sales_order_lines_scope_idx, sales_payments_scope_idx) predate that reasoning.
-//
-// order_id and quote_id are separate indexes rather than a widening of the above:
-// both tables are the child side of `on delete set null` constraints, which
-// PostgreSQL does not index automatically, and the constraint check probes
-// order_id / quote_id — columns the context index cannot serve even though they
-// hold the same uuid. Without them every delete of a parent order or quote scans
-// the child table in full.
-//
-// Both tables are high-churn, so the indexes are built CONCURRENTLY to avoid
-// blocking writes during the build. CREATE INDEX CONCURRENTLY cannot run inside a
-// transaction, hence isTransactional() => false; the migration runner applies
-// migrations one-by-one, so this opt-out is safe. Drop first so retrying a failed
-// concurrent build removes PostgreSQL's invalid index stub instead of letting IF
-// NOT EXISTS silently accept it: a half-built concurrent index is left INVALID,
-// the planner ignores it, and IF NOT EXISTS would report success while the table
-// stays unindexed. Same shape as Migration20260806120000 and
-// Migration20260731105052_query_index.
-//
-// On a database that does not have these indexes yet — every fresh install — the
-// drops are no-ops and there is no window without them. An operator who already
-// created one out of band should expect it to be rebuilt here, and the reads that
-// depend on it to fall back to their pre-index cost until the concurrent build
-// finishes; on a large table, run this while the writers that depend on it are
-// paused.
-export class Migration20260821120000_sales_notes_addresses_context_idx extends Migration {
-
-  override isTransactional(): boolean {
-    return false;
-  }
-
-  override up(): void | Promise<void> {
-    this.addSql(`drop index concurrently if exists "sales_notes_context_idx";`);
-    this.addSql(`create index concurrently "sales_notes_context_idx" on "sales_notes" ("context_id");`);
-    this.addSql(`drop index concurrently if exists "sales_notes_order_idx";`);
-    this.addSql(`create index concurrently "sales_notes_order_idx" on "sales_notes" ("order_id");`);
-    this.addSql(`drop index concurrently if exists "sales_notes_quote_idx";`);
-    this.addSql(`create index concurrently "sales_notes_quote_idx" on "sales_notes" ("quote_id");`);
-
-    this.addSql(`drop index concurrently if exists "sales_document_addresses_document_idx";`);
-    this.addSql(`create index concurrently "sales_document_addresses_document_idx" on "sales_document_addresses" ("document_id");`);
-    this.addSql(`drop index concurrently if exists "sales_document_addresses_order_idx";`);
-    this.addSql(`create index concurrently "sales_document_addresses_order_idx" on "sales_document_addresses" ("order_id");`);
-    this.addSql(`drop index concurrently if exists "sales_document_addresses_quote_idx";`);
-    this.addSql(`create index concurrently "sales_document_addresses_quote_idx" on "sales_document_addresses" ("quote_id");`);
-  }
-
-  override down(): void | Promise<void> {
-    this.addSql(`drop index concurrently if exists "sales_document_addresses_quote_idx";`);
-    this.addSql(`drop index concurrently if exists "sales_document_addresses_order_idx";`);
-    this.addSql(`drop index concurrently if exists "sales_document_addresses_document_idx";`);
-    this.addSql(`drop index concurrently if exists "sales_notes_quote_idx";`);
-    this.addSql(`drop index concurrently if exists "sales_notes_order_idx";`);
-    this.addSql(`drop index concurrently if exists "sales_notes_context_idx";`);
-  }
-
-}

package/src/modules/wms/api/warehouseSearch.ts

@@ -1,23 +0,0 @@
-import { escapeLikePattern } from '@open-mercato/shared/lib/db/escapeLikePattern'
-import { locales } from '@open-mercato/shared/lib/i18n/config'
-import { matchCountryCodes } from '@open-mercato/shared/lib/location/countries'
-
-/**
- * Warehouse list search matches name/code/city plus both stored country values:
- * ISO codes (`PL`) and legacy free-text (`Poland`). Localized labels shown in
- * the table (`Poland` / `Polska`) must resolve back to the stored ISO code.
- */
-export function buildWarehouseListSearchOr(term: string): Array<Record<string, unknown>> {
-  const like = `%${escapeLikePattern(term)}%`
-  const orFilters: Array<Record<string, unknown>> = [
-    { name: { $ilike: like } },
-    { code: { $ilike: like } },
-    { city: { $ilike: like } },
-    { country: { $ilike: like } },
-  ]
-  const matchedCountryCodes = matchCountryCodes(term, { locales })
-  if (matchedCountryCodes.length > 0) {
-    orFilters.push({ country: { $in: matchedCountryCodes } })
-  }
-  return orFilters
-}

package/src/modules/wms/components/backend/WarehouseEditDialog.tsx

@@ -1,216 +0,0 @@
-"use client"
-
-import * as React from 'react'
-import { z } from 'zod'
-import { CrudForm, type CrudField } from '@open-mercato/ui/backend/CrudForm'
-import { flash } from '@open-mercato/ui/backend/FlashMessages'
-import { useGuardedMutation } from '@open-mercato/ui/backend/injection/useGuardedMutation'
-import { apiCall } from '@open-mercato/ui/backend/utils/apiCall'
-import { raiseCrudError } from '@open-mercato/ui/backend/utils/serverErrors'
-import { Dialog, DialogContent, DialogHeader, DialogTitle } from '@open-mercato/ui/primitives/dialog'
-import { extractCustomFieldEntries } from '@open-mercato/shared/lib/crud/custom-fields-client'
-import { useLocale, useT } from '@open-mercato/shared/lib/i18n/context'
-import { E } from '#generated/entities.ids.generated'
-import { flashMutationError } from '../../lib/flashMutationError'
-import {
-  countryOptionFromStored,
-  loadCountryOptions,
-  loadTimezoneOptions,
-  timezoneOptionFromStored,
-} from './warehouseFormOptions'
-
-export type WarehouseDialogRow = {
-  id: string
-  name?: string | null
-  code?: string | null
-  city?: string | null
-  country?: string | null
-  timezone?: string | null
-  is_active?: boolean | null
-  is_primary?: boolean | null
-  updated_at?: string | null
-  updatedAt?: string | null
-  customValues?: Record<string, unknown> | null
-}
-
-export type WarehouseFormValues = {
-  name: string
-  code: string
-  city?: string
-  country?: string
-  timezone?: string
-  isActive: boolean
-  isPrimary: boolean
-}
-
-export const warehouseFormSchema = z.object({
-  name: z.string().trim().min(1),
-  code: z.string().trim().min(1),
-  city: z.string().trim().optional(),
-  country: z.string().trim().optional(),
-  timezone: z.string().trim().optional(),
-  isActive: z.boolean().default(true),
-  isPrimary: z.boolean().default(false),
-}).passthrough()
-
-type WarehouseEditDialogProps = {
-  open: boolean
-  onOpenChange: (open: boolean) => void
-  mode: 'create' | 'edit'
-  row?: WarehouseDialogRow | null
-  onSaved?: (info: {
-    mode: 'create' | 'edit'
-    id?: string
-    values: WarehouseFormValues
-    updatedAt?: string | null
-  }) => void | Promise<void>
-}
-
-export function WarehouseEditDialog({ open, onOpenChange, mode, row, onSaved }: WarehouseEditDialogProps) {
-  const t = useT()
-  const locale = useLocale()
-  const { runMutation } = useGuardedMutation<Record<string, unknown>>({ contextId: 'wms-config-warehouses' })
-  const [submitting, setSubmitting] = React.useState(false)
-
-  const countrySeedOptions = React.useMemo(() => {
-    if (mode !== 'edit' || !row?.country?.trim()) return undefined
-    return [countryOptionFromStored(row.country, locale)]
-  }, [locale, mode, row])
-
-  const timezoneSeedOptions = React.useMemo(() => {
-    if (mode !== 'edit' || !row?.timezone?.trim()) return undefined
-    return [timezoneOptionFromStored(row.timezone)]
-  }, [mode, row])
-
-  const fields = React.useMemo<CrudField[]>(() => [
-    { id: 'name', type: 'text', label: t('wms.backend.config.warehouses.form.name', 'Name'), required: true },
-    { id: 'code', type: 'text', label: t('wms.backend.config.warehouses.form.code', 'Code'), required: true },
-    { id: 'city', type: 'text', label: t('wms.backend.config.warehouses.form.city', 'City') },
-    {
-      id: 'country',
-      type: 'combobox',
-      label: t('wms.backend.config.warehouses.form.country', 'Country'),
-      placeholder: t('wms.backend.config.warehouses.form.countryPlaceholder', 'Search country'),
-      loadOptions: (query) => loadCountryOptions(query, locale),
-      allowCustomValues: false,
-      seedOptions: countrySeedOptions,
-      resolveLabel: (value) => countryOptionFromStored(value, locale).label,
-    },
-    {
-      id: 'timezone',
-      type: 'combobox',
-      label: t('wms.backend.config.warehouses.form.timezone', 'Timezone'),
-      placeholder: t('wms.backend.config.warehouses.form.timezonePlaceholder', 'Search timezone'),
-      loadOptions: loadTimezoneOptions,
-      allowCustomValues: false,
-      seedOptions: timezoneSeedOptions,
-    },
-    { id: 'isPrimary', type: 'checkbox', label: t('wms.backend.config.warehouses.form.primary', 'Primary warehouse') },
-    { id: 'isActive', type: 'checkbox', label: t('wms.backend.config.warehouses.form.active', 'Active') },
-  ], [countrySeedOptions, locale, t, timezoneSeedOptions])
-
-  const initialValues = React.useMemo<WarehouseFormValues>(() => {
-    if (mode === 'edit' && row) {
-      return {
-        name: row.name || '',
-        code: row.code || '',
-        city: row.city || '',
-        country: row.country || '',
-        timezone: row.timezone || '',
-        isActive: row.is_active !== false,
-        isPrimary: row.is_primary === true,
-        ...extractCustomFieldEntries(row as Record<string, unknown>),
-      } as WarehouseFormValues
-    }
-    return {
-      name: '',
-      code: '',
-      city: '',
-      country: '',
-      timezone: '',
-      isActive: true,
-      isPrimary: false,
-    }
-  }, [mode, row])
-
-  const handleSubmit = React.useCallback(async (values: WarehouseFormValues) => {
-    setSubmitting(true)
-    try {
-      const payload = mode === 'edit' && row ? { id: row.id, ...values } : values
-      const call = await runMutation({
-        operation: async () => {
-          const result = await apiCall<{ id?: string | null; updatedAt?: string | null; updated_at?: string | null }>(
-            '/api/wms/warehouses',
-            {
-              method: mode === 'edit' ? 'PUT' : 'POST',
-              headers: { 'x-om-forbidden-redirect': '0' },
-              body: JSON.stringify(payload),
-            },
-          )
-          if (!result.ok) {
-            await raiseCrudError(result.response, t('wms.backend.config.warehouses.errors.save', 'Failed to save warehouse.'))
-          }
-          return result
-        },
-        context: {},
-        mutationPayload: payload,
-      })
-      flash(
-        mode === 'edit'
-          ? t('wms.backend.config.warehouses.flash.updated', 'Warehouse updated')
-          : t('wms.backend.config.warehouses.flash.created', 'Warehouse created'),
-        'success',
-      )
-      const resultBody =
-        call?.result && typeof call.result === 'object'
-          ? (call.result as { id?: unknown; updatedAt?: unknown; updated_at?: unknown })
-          : null
-      const createdId =
-        mode === 'create' && typeof resultBody?.id === 'string'
-          ? resultBody.id.trim()
-          : undefined
-      const updatedAt =
-        (typeof resultBody?.updatedAt === 'string' && resultBody.updatedAt.trim()) ||
-        (typeof resultBody?.updated_at === 'string' && resultBody.updated_at.trim()) ||
-        undefined
-      onOpenChange(false)
-      await onSaved?.({
-        mode,
-        id: createdId || row?.id,
-        values,
-        updatedAt: updatedAt || null,
-      })
-    } catch (error) {
-      flashMutationError(error, t('wms.backend.config.warehouses.errors.save', 'Failed to save warehouse.'), t)
-    } finally {
-      setSubmitting(false)
-    }
-  }, [mode, onOpenChange, onSaved, row, runMutation, t])
-
-  return (
-    <Dialog open={open} onOpenChange={onOpenChange}>
-      <DialogContent className="max-w-3xl">
-        <DialogHeader>
-          <DialogTitle>
-            {mode === 'edit'
-              ? t('wms.backend.config.warehouses.dialog.edit', 'Edit warehouse')
-              : t('wms.backend.config.warehouses.dialog.create', 'Create warehouse')}
-          </DialogTitle>
-        </DialogHeader>
-        <CrudForm<WarehouseFormValues>
-          schema={warehouseFormSchema}
-          fields={fields}
-          entityId={E.wms.warehouse}
-          initialValues={initialValues}
-          submitLabel={t('common.save', 'Save')}
-          onSubmit={handleSubmit}
-          embedded
-          disableInitialFocus
-          isLoading={submitting}
-          twoColumn
-          optimisticLockUpdatedAt={mode === 'edit' ? (row?.updatedAt ?? row?.updated_at ?? null) : undefined}
-        />
-      </DialogContent>
-    </Dialog>
-  )
-}

package/src/modules/wms/components/backend/warehouseFormOptions.ts

@@ -1,72 +0,0 @@
-import type { CrudFieldOption } from '@open-mercato/ui/backend/CrudForm'
-import { buildCountryOptions, resolveCountryName } from '@open-mercato/shared/lib/location/countries'
-
-const OPTION_LIMIT = 100
-
-function matchesQuery(haystack: string, query: string): boolean {
-  return haystack.toLowerCase().includes(query.toLowerCase())
-}
-
-export function countryOptionFromStored(value: string, locale?: string): CrudFieldOption {
-  const trimmed = value.trim()
-  const code = trimmed.toUpperCase()
-  const label = resolveCountryName(code, { locale })
-  if (label && label !== code) {
-    return { value: code, label }
-  }
-  return { value: trimmed, label: trimmed }
-}
-
-export function formatWarehouseCountryLabel(value: string | null | undefined, locale?: string): string {
-  const trimmed = value?.trim()
-  if (!trimmed) return '—'
-  return countryOptionFromStored(trimmed, locale).label
-}
-
-export function timezoneOptionFromStored(value: string): CrudFieldOption {
-  const trimmed = value.trim()
-  return { value: trimmed, label: trimmed }
-}
-
-export async function loadCountryOptions(query?: string, locale?: string): Promise<CrudFieldOption[]> {
-  const options = buildCountryOptions({ locale }).map((entry) => ({
-    value: entry.code,
-    label: entry.label,
-  }))
-  const term = query?.trim()
-  const filtered = term
-    ? options.filter((option) => matchesQuery(option.label, term) || matchesQuery(option.value, term))
-    : options
-  return filtered.slice(0, OPTION_LIMIT)
-}
-
-function resolveLocalTimeZone(): string | undefined {
-  try {
-    return Intl.DateTimeFormat().resolvedOptions().timeZone || undefined
-  } catch {
-    return undefined
-  }
-}
-
-function listSupportedTimeZones(): string[] {
-  try {
-    const intlWithSupportedValues = Intl as typeof Intl & { supportedValuesOf?: (input: 'timeZone') => string[] }
-    if (typeof intlWithSupportedValues.supportedValuesOf !== 'function') return []
-    return intlWithSupportedValues.supportedValuesOf('timeZone').filter(Boolean)
-  } catch {
-    return []
-  }
-}
-
-export async function loadTimezoneOptions(query?: string): Promise<CrudFieldOption[]> {
-  const localZone = resolveLocalTimeZone()
-  const preferred = localZone && localZone !== 'UTC' ? ['UTC', localZone] : ['UTC']
-  const zones = new Set<string>([...preferred, ...listSupportedTimeZones()])
-  const rest = Array.from(zones).filter((tz) => !preferred.includes(tz))
-  const allTz = [...preferred, ...rest]
-  const term = query?.trim()
-  const filtered = term
-    ? allTz.filter((tz) => matchesQuery(tz, term))
-    : allTz
-  return filtered.slice(0, OPTION_LIMIT).map((tz) => ({ value: tz, label: tz }))
-}