@open-mercato/core 0.6.7-develop.6863.1.61001cf9a8 → 0.6.7

package/src/modules/sales/workflows.ts

@@ -57,7 +57,7 @@ const orderApproval = defineWorkflow({
           config: {
             commandId: 'sales.orders.update',
             statusDictionary: 'sales.order_status',
-            input: { id: '{{context.orderId}}', statusValue: 'pending_approval' },
+            input: { id: '{{context.id}}', statusValue: 'pending_approval' },
           },
           retryPolicy: { maxAttempts: 3, initialIntervalMs: 1000, backoffCoefficient: 2, maxIntervalMs: 10000 },
         },
@@ -68,7 +68,7 @@ const orderApproval = defineWorkflow({
           async: true,
           config: {
             eventName: 'sales.order.approval.requested',
-            payload: { orderId: '{{context.orderId}}', workflowInstanceId: '{{workflow.instanceId}}' },
+            payload: { orderId: '{{context.id}}', workflowInstanceId: '{{workflow.instanceId}}' },
           },
         },
       ],
@@ -91,7 +91,7 @@ const orderApproval = defineWorkflow({
           config: {
             commandId: 'sales.orders.update',
             statusDictionary: 'sales.order_status',
-            input: { id: '{{context.orderId}}', statusValue: 'approved' },
+            input: { id: '{{context.id}}', statusValue: 'approved' },
           },
           retryPolicy: { maxAttempts: 3, initialIntervalMs: 1000, backoffCoefficient: 2, maxIntervalMs: 10000 },
         },
@@ -103,7 +103,7 @@ const orderApproval = defineWorkflow({
           config: {
             eventName: 'sales.order.approval.approved',
             payload: {
-              orderId: '{{context.orderId}}',
+              orderId: '{{context.id}}',
               workflowInstanceId: '{{workflow.instanceId}}',
               approvedBy: '{{context.completedBy}}',
               comments: '{{context.comments}}',
@@ -130,7 +130,7 @@ const orderApproval = defineWorkflow({
           config: {
             commandId: 'sales.orders.update',
             statusDictionary: 'sales.order_status',
-            input: { id: '{{context.orderId}}', statusValue: 'rejected' },
+            input: { id: '{{context.id}}', statusValue: 'rejected' },
           },
           retryPolicy: { maxAttempts: 3, initialIntervalMs: 1000, backoffCoefficient: 2, maxIntervalMs: 10000 },
         },
@@ -142,7 +142,7 @@ const orderApproval = defineWorkflow({
           config: {
             eventName: 'sales.order.approval.rejected',
             payload: {
-              orderId: '{{context.orderId}}',
+              orderId: '{{context.id}}',
               workflowInstanceId: '{{workflow.instanceId}}',
               rejectedBy: '{{context.completedBy}}',
               comments: '{{context.comments}}',
@@ -175,10 +175,7 @@ const orderApproval = defineWorkflow({
     name: 'Order Approval Trigger',
     description: 'Triggers when a new sales order is created',
     eventPattern: 'sales.order.created',
-    config: {
-      entityType: 'SalesOrder',
-      contextMapping: [{ targetKey: 'orderId', sourceExpression: 'id' }],
-    },
+    config: { entityType: 'SalesOrder' },
     enabled: true,
     priority: 0,
   }],

package/src/modules/shipping_carriers/api/webhook/[provider]/route.ts

@@ -11,7 +11,6 @@ import { getShippingAdapter } from '../../../lib/adapter-registry'
 import { getShippingCarrierQueue } from '../../../lib/queue'
 import { shippingCarriersTag } from '../../openapi'
 import { createLogger } from '@open-mercato/shared/lib/logger'
-import { readBoundedRequestBody, WebhookBodyTooLargeError } from '@open-mercato/shared/lib/webhooks'
 
 const logger = createLogger('shipping_carriers').child({ component: 'webhook' })
 
@@ -52,15 +51,7 @@ export async function POST(req: Request, { params }: { params: Promise<{ provide
   const rateLimitResponse = await checkProviderWebhookRateLimit(container, req, providerKey)
   if (rateLimitResponse) return rateLimitResponse
 
-  let rawBody: string
-  try {
-    rawBody = await readBoundedRequestBody(req)
-  } catch (error) {
-    if (error instanceof WebhookBodyTooLargeError) {
-      return NextResponse.json({ error: 'Webhook payload too large' }, { status: 413 })
-    }
-    throw error
-  }
+  const rawBody = await req.text()
   const headers: Record<string, string> = {}
   req.headers.forEach((value, key) => {
     headers[key] = value
@@ -165,7 +156,6 @@ export const openApi = {
       responses: [
         { status: 202, description: 'Webhook accepted for async processing' },
         { status: 401, description: 'Signature verification failed' },
-        { status: 413, description: 'Webhook payload too large' },
         { status: 404, description: 'Unknown provider' },
       ],
     },

package/src/modules/shipping_carriers/lib/shipment-wizard/components/PackageEditor.tsx

@@ -47,7 +47,7 @@ export const PackageEditor = (props: PackageEditorProps) => {
                 type="button"
                 variant="ghost"
                 size="sm"
-                className="h-auto px-2 py-0.5 text-xs text-destructive hover:text-destructive/80"
+                className="h-auto px-2 py-0.5 text-xs text-red-600 hover:text-red-700"
                 onClick={() => removePackage(index)}
                 disabled={disabled}
               >

package/src/modules/staff/api/interceptors.ts

@@ -1,5 +1,5 @@
 import type { ApiInterceptor, InterceptorContext } from '@open-mercato/shared/lib/crud/api-interceptor'
-import { authorizeFeatures } from '@open-mercato/shared/security/featurePolicy'
+import { hasFeature } from '@open-mercato/shared/security/features'
 import { getStaffMemberByUserId } from '../lib/staffMemberResolver'
 
 type RbacServiceLike = {
@@ -21,6 +21,7 @@ type RbacServiceLike = {
  * from `rbacService.userHasAllFeatures`, which is cached and wildcard-aware.
  */
 async function hasManageAllFeature(context: InterceptorContext): Promise<boolean> {
+  if (hasFeature(context.userFeatures, 'staff.timesheets.manage_all')) return true
   try {
     const rbac = context.container.resolve('rbacService') as RbacServiceLike | undefined
     if (rbac?.userHasAllFeatures) {
@@ -31,12 +32,9 @@ async function hasManageAllFeature(context: InterceptorContext): Promise<boolean
       )
     }
   } catch {
-    // Fall back to the already-loaded snapshot below.
+    // rbacService not available — fall through to deny
   }
-  return authorizeFeatures(
-    ['staff.timesheets.manage_all'],
-    { grantedFeatures: context.userFeatures ?? [] },
-  )
+  return false
 }
 
 export const interceptors: ApiInterceptor[] = [

package/src/modules/staff/api/leave-requests.ts

@@ -1,7 +1,7 @@
 import { z } from 'zod'
 import type { EntityManager } from '@mikro-orm/postgresql'
 import type { RbacService } from '@open-mercato/core/modules/auth/services/rbacService'
-import { makeCrudRoute, type CrudCtx } from '@open-mercato/shared/lib/crud/factory'
+import { makeCrudRoute } from '@open-mercato/shared/lib/crud/factory'
 import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
 import { resolveTranslations } from '@open-mercato/shared/lib/i18n/server'
 import { resolveCrudRecordId, parseScopedCommandInput } from '@open-mercato/shared/lib/api/scoped'
@@ -22,8 +22,6 @@ const SEND_FEATURE = 'staff.leave_requests.send'
 const MY_VIEW_FEATURE = 'staff.my_leave_requests.view'
 const MY_SEND_FEATURE = 'staff.my_leave_requests.send'
 
-const NIL_UUID = '00000000-0000-0000-0000-000000000000'
-
 const routeMetadata = {
   GET: { requireAuth: true },
   POST: { requireAuth: true },
@@ -115,56 +113,6 @@ async function resolveLeaveRequestAccess(ctx: any): Promise<LeaveRequestAccess>
   return { canManage, canSend, canView, memberId: member?.id ?? null }
 }
 
-export async function buildLeaveRequestFilters(
-  query: z.infer<typeof listSchema>,
-  ctx: CrudCtx,
-) {
-  const filters: Record<string, unknown> = {}
-  if (typeof query.ids === 'string' && query.ids.trim().length > 0) {
-    const ids = query.ids
-      .split(',')
-      .map((value) => value.trim())
-      .filter((value) => value.length > 0)
-    if (ids.length > 0) {
-      filters[F.id] = { $in: ids }
-    }
-  }
-  if (query.status) {
-    filters[F.status] = query.status
-  }
-  if (query.memberId) {
-    filters[F.member_id] = query.memberId
-  }
-  const term = sanitizeSearchTerm(query.search)
-  if (term) {
-    const like = `%${escapeLikePattern(term)}%`
-    const em = (ctx.container.resolve('em') as EntityManager).fork()
-    const members = await findWithDecryption(
-      em,
-      StaffTeamMember,
-      { displayName: { $ilike: like }, deletedAt: null },
-      { fields: ['id'] },
-      { tenantId: ctx.auth?.tenantId ?? null, organizationId: ctx.selectedOrganizationId ?? null },
-    )
-    const memberIds = members.map((member) => member.id)
-    if (!memberIds.length) {
-      filters[F.id] = NIL_UUID
-    } else {
-      filters[F.member_id] = { $in: memberIds }
-    }
-  }
-
-  const access = await resolveLeaveRequestAccess(ctx)
-  if (!access.canManage) {
-    if (!access.canView || !access.memberId) {
-      filters[F.id] = NIL_UUID
-    } else {
-      filters[F.member_id] = access.memberId
-    }
-  }
-  return filters
-}
-
 const crud = makeCrudRoute({
   metadata: routeMetadata,
   orm: {
@@ -203,7 +151,52 @@ const crud = makeCrudRoute({
       createdAt: F.created_at,
       updatedAt: F.updated_at,
     },
-    buildFilters: buildLeaveRequestFilters,
+    buildFilters: async (query, ctx) => {
+      const filters: Record<string, unknown> = {}
+      if (typeof query.ids === 'string' && query.ids.trim().length > 0) {
+        const ids = query.ids
+          .split(',')
+          .map((value) => value.trim())
+          .filter((value) => value.length > 0)
+        if (ids.length > 0) {
+          filters[F.id] = { $in: ids }
+        }
+      }
+      if (query.status) {
+        filters[F.status] = query.status
+      }
+      if (query.memberId) {
+        filters[F.member_id] = query.memberId
+      }
+      const term = sanitizeSearchTerm(query.search)
+      if (term) {
+        const like = `%${escapeLikePattern(term)}%`
+        const em = (ctx.container.resolve('em') as EntityManager).fork()
+        const members = await findWithDecryption(
+          em,
+          StaffTeamMember,
+          { displayName: { $ilike: like }, deletedAt: null },
+          { fields: ['id'] },
+          { tenantId: ctx.auth?.tenantId ?? null, organizationId: ctx.selectedOrganizationId ?? null },
+        )
+        const memberIds = members.map((member) => member.id)
+        if (!memberIds.length) {
+          filters[F.id] = '__no_match__'
+        } else {
+          filters[F.member_id] = { $in: memberIds }
+        }
+      }
+
+      const access = await resolveLeaveRequestAccess(ctx)
+      if (!access.canManage) {
+        if (!access.canView || !access.memberId) {
+          filters[F.id] = '__no_access__'
+        } else {
+          filters[F.member_id] = access.memberId
+        }
+      }
+      return filters
+    },
   },
   hooks: {
     afterList: async (payload, ctx) => {

package/src/modules/staff/components/detail/dictionaries.ts

@@ -1,7 +1,6 @@
 "use client"
 
 import { apiCall, apiCallOrThrow } from '@open-mercato/ui/backend/utils/apiCall'
-import { fetchAllDictionaryEntries } from '@open-mercato/core/modules/dictionaries/lib/fetchAllEntries'
 
 export type DictionaryEntryOption = {
   value: string
@@ -52,9 +51,10 @@ export async function loadStaffDictionary(
   const name = kind === 'activityTypes' ? 'Staff activity types' : 'Staff address types'
   const dictionary = await ensureDictionary(key, name)
   if (!dictionary) return { dictionary: null, entries: [] }
-  const entriesCall = await fetchAllDictionaryEntries(dictionary.id)
+  const entriesCall = await apiCall<{ items?: Record<string, unknown>[] }>(`/api/dictionaries/${dictionary.id}/entries`)
   if (!entriesCall.ok) return { dictionary, entries: [] }
-  const entries = entriesCall.items
+  const items = Array.isArray(entriesCall.result?.items) ? entriesCall.result?.items ?? [] : []
+  const entries = items
     .map((entry) => {
       if (!entry || typeof entry !== 'object') return null
       const record = entry as Record<string, unknown>

package/src/modules/sync_excel/api/upload/route.ts

@@ -10,12 +10,7 @@ import { buildSuggestedMapping } from '../../lib/mapping'
 import { parseCsvPreview } from '../../lib/parser'
 import { createSyncExcelUploadAttachment } from '../../lib/upload-storage'
 import { resolveSyncExcelConcreteScope } from '../../lib/scope'
-import {
-  isMultipartRequestWithinUploadLimit,
-  isMultipartUploadLimitError,
-  parseMultipartFormDataWithinUploadLimit,
-  resolveDefaultAttachmentMaxUploadBytes,
-} from '../../../attachments/lib/upload-limits'
+import { isMultipartRequestWithinUploadLimit, resolveDefaultAttachmentMaxUploadBytes } from '../../../attachments/lib/upload-limits'
 
 export const metadata = {
   POST: { requireAuth: true, requireFeatures: ['sync_excel.run'] },
@@ -72,15 +67,7 @@ export async function POST(request: Request) {
     return NextResponse.json({ error: 'CSV upload exceeds the maximum upload size.' }, { status: 413 })
   }
 
-  let formData: FormData
-  try {
-    formData = await parseMultipartFormDataWithinUploadLimit(request)
-  } catch (error) {
-    if (isMultipartUploadLimitError(error)) {
-      return NextResponse.json({ error: 'CSV upload exceeds the maximum upload size.' }, { status: 413 })
-    }
-    throw error
-  }
+  const formData = await request.formData()
   const parsedPayload = multipartSchema.safeParse({
     entityType: typeof formData.get('entityType') === 'string' ? formData.get('entityType') : undefined,
   })

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

@@ -409,7 +409,7 @@ export function ReleaseReservationDialog({
               </Button>
               <Button
                 type="submit"
-                variant="destructive-solid"
+                variant="destructive"
                 disabled={submitDisabled}
                 data-testid="wms-inventory-release-submit"
               >

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

@@ -1,7 +1,6 @@
 "use client"
 
 import * as React from 'react'
-import { extensionPoints } from '@open-mercato/core/modules/wms/extension-points'
 import Link from 'next/link'
 import { useRouter } from 'next/navigation'
 import { z } from 'zod'
@@ -516,7 +515,7 @@ export function WarehouseSection({ viewAllHref }: ConfigSectionOptions = {}) {
             totalPages: query.data?.totalPages ?? 1,
             onPageChange: setPage,
           }}
-          perspective={{ tableId: extensionPoints.hosts.warehousesTable.tableId }}
+          perspective={{ tableId: 'wms.config.warehouses' }}
           emptyState={(
             <EmptyState
               title={t('wms.backend.config.warehouses.empty.title', 'No warehouses')}
@@ -776,7 +775,7 @@ export function ZoneSection({ viewAllHref }: ConfigSectionOptions = {}) {
             totalPages: query.data?.totalPages ?? 1,
             onPageChange: setPage,
           }}
-          perspective={{ tableId: extensionPoints.hosts.zonesTable.tableId }}
+          perspective={{ tableId: 'wms.config.zones' }}
           emptyState={(
             <EmptyState
               title={t('wms.backend.config.zones.empty.title', 'No zones')}
@@ -983,7 +982,7 @@ export function LocationSection({ viewAllHref }: ConfigSectionOptions = {}) {
             totalPages: query.data?.totalPages ?? 1,
             onPageChange: setPage,
           }}
-          perspective={{ tableId: extensionPoints.hosts.locationsTable.tableId }}
+          perspective={{ tableId: 'wms.config.locations' }}
           emptyState={(
             <EmptyState
               title={t('wms.backend.config.locations.empty.title', 'No locations')}
@@ -1277,7 +1276,7 @@ export function InventoryProfilesSection() {
             totalPages: query.data?.totalPages ?? 1,
             onPageChange: setPage,
           }}
-          perspective={{ tableId: extensionPoints.hosts.inventoryProfilesTable.tableId }}
+          perspective={{ tableId: 'wms.config.inventoryProfiles' }}
           emptyState={(
             <EmptyState
               title={t('wms.backend.config.profiles.empty.title', 'No inventory profiles')}

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

@@ -1,7 +1,6 @@
 "use client"
 
 import * as React from 'react'
-import { extensionPoints } from '@open-mercato/core/modules/wms/extension-points'
 import Link from 'next/link'
 import type { ColumnDef, SortingState } from '@tanstack/react-table'
 import { useQuery } from '@tanstack/react-query'
@@ -666,7 +665,7 @@ export function InventoryBalancesSection({
         emptyTitleFallback="No balance buckets"
         emptyDescriptionKey="wms.backend.inventory.balances.empty.description"
         emptyDescriptionFallback="Balances appear after receipts, adjustments, or moves create inventory buckets."
-        tableId={extensionPoints.hosts.balancesTable.tableId}
+        tableId="wms.inventory.balances"
         entityId={E.wms.inventory_balance}
         icon={<Boxes className="size-5" />}
         columns={columns}
@@ -897,7 +896,7 @@ export function InventoryReservationsSection({
         emptyTitleFallback="No reservations"
         emptyDescriptionKey="wms.backend.inventory.reservations.empty.description"
         emptyDescriptionFallback="Reservations show stock committed to orders, transfers, or manual holds."
-        tableId={extensionPoints.hosts.reservationsTable.tableId}
+        tableId="wms.inventory.reservations"
         entityId={E.wms.inventory_reservation}
         icon={<ShieldCheck className="size-5" />}
         columns={columns}
@@ -1059,7 +1058,7 @@ export function InventoryMovementsSection({
       emptyTitleFallback="No inventory movements"
       emptyDescriptionKey="wms.backend.inventory.movements.empty.description"
       emptyDescriptionFallback="Movement rows are created by receipts, reservations, moves, and reconciliation actions."
-      tableId={extensionPoints.hosts.movementsTable.tableId}
+      tableId="wms.inventory.movements"
       entityId={E.wms.inventory_movement}
       icon={<Route className="size-5" />}
       columns={columns}

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

@@ -1,7 +1,6 @@
 "use client"
 
 import * as React from 'react'
-import { extensionPoints } from '@open-mercato/core/modules/wms/extension-points'
 import Link from 'next/link'
 import { useRouter } from 'next/navigation'
 import { z } from 'zod'
@@ -1159,7 +1158,7 @@ export default function WmsLocationDetailPage({ locationId }: WmsLocationDetailP
                 data={pagedBalances}
                 disableRowClick
                 entityId={E.wms.inventory_balance}
-                perspective={{ tableId: extensionPoints.hosts.locationItemsTable.tableId }}
+                perspective={{ tableId: 'wms.location.items' }}
                 pagination={{
                   page: itemsPage,
                   pageSize: itemsPageSize,
@@ -1202,7 +1201,7 @@ export default function WmsLocationDetailPage({ locationId }: WmsLocationDetailP
               data={movementsQuery.data ?? []}
               disableRowClick
               entityId={E.wms.inventory_movement}
-              perspective={{ tableId: extensionPoints.hosts.locationActivityTable.tableId }}
+              perspective={{ tableId: 'wms.location.activity' }}
               emptyState={t('wms.backend.location.activity.empty', 'No recent movements for this location.')}
               actions={(
                 <Button asChild type="button" variant="ghost" size="sm">

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

@@ -1,7 +1,6 @@
 "use client"
 
 import * as React from 'react'
-import { extensionPoints } from '@open-mercato/core/modules/wms/extension-points'
 import Link from 'next/link'
 import { useRouter } from 'next/navigation'
 import { z } from 'zod'
@@ -1210,7 +1209,7 @@ export default function WmsLotDetailPage({ lotId }: WmsLotDetailPageProps) {
                 data={pagedBalances}
                 disableRowClick
                 entityId={E.wms.inventory_balance}
-                perspective={{ tableId: extensionPoints.hosts.lotDistributionTable.tableId }}
+                perspective={{ tableId: 'wms.lot.distribution' }}
                 pagination={{
                   page: distributionPage,
                   pageSize: distributionPageSize,
@@ -1251,7 +1250,7 @@ export default function WmsLotDetailPage({ lotId }: WmsLotDetailPageProps) {
               data={movementsQuery.data ?? []}
               disableRowClick
               entityId={E.wms.inventory_movement}
-              perspective={{ tableId: extensionPoints.hosts.lotActivityTable.tableId }}
+              perspective={{ tableId: 'wms.lot.activity' }}
               emptyState={t('wms.backend.lot.activity.empty', 'No recent movements for this lot.')}
               actions={(
                 <Button asChild type="button" variant="ghost" size="sm">

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

@@ -1,7 +1,6 @@
 "use client"
 
 import * as React from 'react'
-import { extensionPoints } from '@open-mercato/core/modules/wms/extension-points'
 import Link from 'next/link'
 import { useSearchParams } from 'next/navigation'
 import type { ColumnDef, SortingState } from '@tanstack/react-table'
@@ -281,7 +280,7 @@ export default function WmsLotsListPage() {
                 totalPages: lotsQuery.data.totalPages,
                 onPageChange: setPage,
               }}
-              perspective={{ tableId: extensionPoints.hosts.lotsTable.tableId }}
+              perspective={{ tableId: 'wms.lots.list' }}
               emptyState={
                 <EmptyState
                   title={t('wms.backend.lots.empty.title', 'No lots found')}

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

@@ -1,7 +1,6 @@
 "use client"
 
 import * as React from 'react'
-import { extensionPoints } from '@open-mercato/core/modules/wms/extension-points'
 import Link from 'next/link'
 import { useRouter } from 'next/navigation'
 import { z } from 'zod'
@@ -1095,7 +1094,7 @@ export default function WmsSkuDetailPage({ variantId }: WmsSkuDetailPageProps) {
                 data={pagedBalances}
                 disableRowClick
                 entityId={E.wms.inventory_balance}
-                perspective={{ tableId: extensionPoints.hosts.skuDistributionTable.tableId }}
+                perspective={{ tableId: 'wms.sku.distribution' }}
                 pagination={{
                   page: distributionPage,
                   pageSize: distributionPageSize,
@@ -1138,7 +1137,7 @@ export default function WmsSkuDetailPage({ variantId }: WmsSkuDetailPageProps) {
               data={movementsQuery.data ?? []}
               disableRowClick
               entityId={E.wms.inventory_movement}
-              perspective={{ tableId: extensionPoints.hosts.skuActivityTable.tableId }}
+              perspective={{ tableId: 'wms.sku.activity' }}
               emptyState={t('wms.backend.sku.activity.empty', 'No recent movements for this SKU.')}
               actions={(
                 <Button asChild type="button" variant="ghost" size="sm">

package/src/modules/wms/i18n/de.json

@@ -166,7 +166,6 @@
   "wms.backend.dashboard.firstRun.actions.addLocations": "Lagerorte hinzufügen",
   "wms.backend.dashboard.firstRun.actions.createWarehouse": "Lager anlegen",
   "wms.backend.dashboard.firstRun.actions.receiveStock": "Ersten Wareneingang buchen",
-  "wms.backend.dashboard.firstRun.description": "Legen Sie ein Lager an, fügen Sie Lagerorte hinzu und buchen Sie anschließend den ersten Wareneingang, um die Bestandsverfolgung zu starten.",
   "wms.backend.dashboard.firstRun.noLocations.description": "Ihr Lager ist eingerichtet — fügen Sie mindestens einen Lagerort hinzu, bevor Sie Waren empfangen können.",
   "wms.backend.dashboard.firstRun.noLocations.title": "Lagerorte hinzufügen",
   "wms.backend.dashboard.firstRun.title": "Lager einrichten",
@@ -581,7 +580,6 @@
   "wms.backend.inventory.release.dialog.submit": "Reservierung freigeben",
   "wms.backend.inventory.release.dialog.title": "Reservierung freigeben",
   "wms.backend.inventory.release.errors.balanceIntegrityViolation": "Balance-Integritätsfehler – führen Sie „mercato wms verify-balances --repair“ aus, um Abweichungen zu diagnostizieren und zu beheben, bevor Sie es erneut versuchen.",
-  "wms.backend.inventory.release.errors.missingReservation": "Der Reservierungskontext fehlt. Schließen Sie den Dialog und versuchen Sie es erneut über die Reservierungstabelle.",
   "wms.backend.inventory.release.errors.submit": "Reservierung konnte nicht freigegeben werden.",
   "wms.backend.inventory.release.flash.success": "Reservierung freigegeben",
   "wms.backend.inventory.release.form.notes": "Notizen",

package/src/modules/wms/i18n/en.json

@@ -166,7 +166,6 @@
   "wms.backend.dashboard.firstRun.actions.addLocations": "Add locations",
   "wms.backend.dashboard.firstRun.actions.createWarehouse": "Create warehouse",
   "wms.backend.dashboard.firstRun.actions.receiveStock": "Receive first stock",
-  "wms.backend.dashboard.firstRun.description": "Create a warehouse, add storage locations, then receive your first stock to start tracking inventory.",
   "wms.backend.dashboard.firstRun.noLocations.description": "Your warehouse is set up — add at least one storage location before you can receive stock.",
   "wms.backend.dashboard.firstRun.noLocations.title": "Add storage locations",
   "wms.backend.dashboard.firstRun.title": "Set up your warehouse",
@@ -581,7 +580,6 @@
   "wms.backend.inventory.release.dialog.submit": "Release reservation",
   "wms.backend.inventory.release.dialog.title": "Release reservation",
   "wms.backend.inventory.release.errors.balanceIntegrityViolation": "Balance integrity error — run \"mercato wms verify-balances --repair\" to diagnose and fix drift before retrying.",
-  "wms.backend.inventory.release.errors.missingReservation": "Reservation context is missing. Close the dialog and try again from the reservations table.",
   "wms.backend.inventory.release.errors.submit": "Failed to release reservation.",
   "wms.backend.inventory.release.flash.success": "Reservation released",
   "wms.backend.inventory.release.form.notes": "Notes",

package/src/modules/wms/i18n/es.json

@@ -166,7 +166,6 @@
   "wms.backend.dashboard.firstRun.actions.addLocations": "Añadir ubicaciones",
   "wms.backend.dashboard.firstRun.actions.createWarehouse": "Crear almacén",
   "wms.backend.dashboard.firstRun.actions.receiveStock": "Recibir primer stock",
-  "wms.backend.dashboard.firstRun.description": "Cree un almacén, añada ubicaciones de almacenamiento y reciba el primer stock para empezar a controlar el inventario.",
   "wms.backend.dashboard.firstRun.noLocations.description": "Su almacén está configurado — añada al menos una ubicación de almacenamiento antes de poder recibir stock.",
   "wms.backend.dashboard.firstRun.noLocations.title": "Añadir ubicaciones de almacenamiento",
   "wms.backend.dashboard.firstRun.title": "Configure su almacén",
@@ -581,7 +580,6 @@
   "wms.backend.inventory.release.dialog.submit": "Liberar reserva",
   "wms.backend.inventory.release.dialog.title": "Liberar reserva",
   "wms.backend.inventory.release.errors.balanceIntegrityViolation": "Error de integridad de saldo — ejecute \"mercato wms verify-balances --repair\" para diagnosticar y corregir la desviación antes de reintentar.",
-  "wms.backend.inventory.release.errors.missingReservation": "Falta el contexto de la reserva. Cierre el cuadro de diálogo e inténtelo de nuevo desde la tabla de reservas.",
   "wms.backend.inventory.release.errors.submit": "No se pudo liberar la reserva.",
   "wms.backend.inventory.release.flash.success": "Reserva liberada",
   "wms.backend.inventory.release.form.notes": "Notas",

package/src/modules/wms/i18n/pl.json

@@ -166,7 +166,6 @@
   "wms.backend.dashboard.firstRun.actions.addLocations": "Dodaj lokalizacje",
   "wms.backend.dashboard.firstRun.actions.createWarehouse": "Utwórz magazyn",
   "wms.backend.dashboard.firstRun.actions.receiveStock": "Przyjmij pierwszy towar",
-  "wms.backend.dashboard.firstRun.description": "Utwórz magazyn, dodaj lokalizacje składowania, a następnie przyjmij pierwszy towar, aby rozpocząć śledzenie zapasów.",
   "wms.backend.dashboard.firstRun.noLocations.description": "Magazyn jest skonfigurowany — dodaj przynajmniej jedną lokalizację składowania, aby móc przyjmować towar.",
   "wms.backend.dashboard.firstRun.noLocations.title": "Dodaj lokalizacje składowania",
   "wms.backend.dashboard.firstRun.title": "Skonfiguruj swój magazyn",
@@ -581,7 +580,6 @@
   "wms.backend.inventory.release.dialog.submit": "Zwolnij rezerwację",
   "wms.backend.inventory.release.dialog.title": "Zwolnij rezerwację",
   "wms.backend.inventory.release.errors.balanceIntegrityViolation": "Błąd integralności salda — uruchom \"mercato wms verify-balances --repair\" aby zdiagnozować i naprawić dryf przed ponowieniem.",
-  "wms.backend.inventory.release.errors.missingReservation": "Brakuje kontekstu rezerwacji. Zamknij okno dialogowe i spróbuj ponownie z tabeli rezerwacji.",
   "wms.backend.inventory.release.errors.submit": "Nie udało się zwolnić rezerwacji.",
   "wms.backend.inventory.release.flash.success": "Rezerwacja zwolniona",
   "wms.backend.inventory.release.form.notes": "Notatki",

package/src/modules/workflows/backend/definitions/page.tsx

@@ -1,7 +1,6 @@
 "use client"
 
 import * as React from 'react'
-import { extensionPoints } from '@open-mercato/core/modules/workflows/extension-points'
 import Link from 'next/link'
 import { useRouter } from 'next/navigation'
 import { Page, PageBody } from '@open-mercato/ui/backend/Page'
@@ -430,7 +429,7 @@ export default function WorkflowDefinitionsListPage() {
           onFiltersClear={handleFiltersClear}
           onRowClick={(row) => router.push(`/backend/definitions/visual-editor?id=${row.id}`)}
           perspective={{
-            tableId: extensionPoints.hosts.definitionsTable.tableId,
+            tableId: 'workflows.definitions.list',
           }}
           emptyState={(
             <ListEmptyState
@@ -453,7 +452,7 @@ export default function WorkflowDefinitionsListPage() {
               <Button variant="outline" onClick={() => setDeleteTarget(null)}>
                 {t('common.cancel')}
               </Button>
-              <Button variant="destructive-solid" onClick={confirmDelete}>
+              <Button variant="destructive" onClick={confirmDelete}>
                 <Trash2/>
                 {t('common.delete')}
               </Button>

package/src/modules/workflows/backend/definitions/visual-editor/page.tsx

@@ -11,7 +11,6 @@ import { useState, useCallback, useEffect } from 'react'
 import { useRouter, useSearchParams, usePathname } from 'next/navigation'
 import { graphToDefinition, definitionToGraph, validateWorkflowGraph, generateStepId, generateTransitionId, appendWorkflowEdge, ValidationError } from '../../../lib/graph-utils'
 import { performDeleteEdgeFlow, performDeleteNodeFlow } from '../../../lib/visual-editor-delete-flow'
-import { humanizeDefinitionIssuePath } from '../../../lib/format-validation-error'
 import { workflowDefinitionDataSchema } from '../../../data/validators'
 import { Page } from '@open-mercato/ui/backend/Page'
 import { Button } from '@open-mercato/ui/primitives/button'
@@ -358,19 +357,11 @@ export default function VisualEditorPage() {
       triggers: triggers.length > 0 ? triggers : undefined,
     }
 
-    // Run Zod schema validation before saving. Report every issue, not just the
-    // first: a save rejected for one missing activity field used to look like
-    // "nothing happened" once the operator fixed that field and hit a second
-    // one (#4232). Paths are humanized (steps.2.activities.0.config.endpoint →
-    // step 3 › activity 1 › endpoint) so the message points at the node to open.
+    // Run Zod schema validation before saving
     const schemaResult = workflowDefinitionDataSchema.safeParse(definitionData)
     if (!schemaResult.success) {
-      const issues = schemaResult.error.issues
-      const described = issues
-        .slice(0, 3)
-        .map((issue) => `${humanizeDefinitionIssuePath(issue.path)}: ${issue.message}`)
-      const suffix = issues.length > described.length ? ` (+${issues.length - described.length} more)` : ''
-      flash(`Cannot save — ${described.join('; ')}${suffix}`, 'error')
+      const firstIssue = schemaResult.error.issues[0]
+      flash(`Schema error: ${firstIssue.path.join('.')} - ${firstIssue.message}`, 'error')
       return
     }
 
@@ -688,7 +679,7 @@ export default function VisualEditorPage() {
           </DialogHeader>
           <DialogFooter>
             <Button variant="outline" onClick={() => setShowClearConfirm(false)}>{t('common.cancel', 'Cancel')}</Button>
-            <Button variant="destructive-solid" onClick={confirmClear}>{t('common.clear', 'Clear')}</Button>
+            <Button variant="destructive" onClick={confirmClear}>{t('common.clear', 'Clear')}</Button>
           </DialogFooter>
         </DialogContent>
       </Dialog>