@open-mercato/core 0.4.2-canary-bdaa640a68 → 0.4.2-canary-3b5064ce72

package/src/modules/notifications/workers/create-notification.worker.ts

@@ -0,0 +1,122 @@
+import type { EntityManager } from '@mikro-orm/core'
+import type { Knex } from 'knex'
+import { Notification } from '../data/entities'
+import type { CreateNotificationInput, CreateRoleNotificationInput, CreateFeatureNotificationInput } from '../data/validators'
+import { buildNotificationEntity, emitNotificationCreated, emitNotificationCreatedBatch } from '../lib/notificationFactory'
+import { getRecipientUserIdsForFeature, getRecipientUserIdsForRole } from '../lib/notificationRecipients'
+
+function getKnex(em: EntityManager): Knex {
+  return (em.getConnection() as unknown as { getKnex: () => Knex }).getKnex()
+}
+
+export const NOTIFICATIONS_QUEUE_NAME = 'notifications'
+
+export type CreateNotificationJob = {
+  type: 'create'
+  input: CreateNotificationInput
+  tenantId: string
+  organizationId?: string | null
+}
+
+export type CreateRoleNotificationJob = {
+  type: 'create-role'
+  input: CreateRoleNotificationInput
+  tenantId: string
+  organizationId?: string | null
+}
+
+export type CreateFeatureNotificationJob = {
+  type: 'create-feature'
+  input: CreateFeatureNotificationInput
+  tenantId: string
+  organizationId?: string | null
+}
+
+export type CleanupExpiredJob = {
+  type: 'cleanup-expired'
+}
+
+export type NotificationJob = CreateNotificationJob | CreateRoleNotificationJob | CreateFeatureNotificationJob | CleanupExpiredJob
+
+export const metadata = {
+  queue: NOTIFICATIONS_QUEUE_NAME,
+  id: 'notifications:create',
+  concurrency: 5,
+}
+
+type HandlerContext = {
+  resolve: <T = unknown>(name: string) => T
+}
+
+export default async function handle(
+  job: { payload: NotificationJob },
+  ctx: HandlerContext
+): Promise<void> {
+  const { payload } = job
+
+  if (payload.type === 'create') {
+    const em = (ctx.resolve('em') as EntityManager).fork()
+    const eventBus = ctx.resolve('eventBus') as { emit: (event: string, payload: unknown) => Promise<void> }
+    const { input, tenantId, organizationId } = payload
+    const { recipientUserId, ...content } = input
+    const notification = buildNotificationEntity(em, content, recipientUserId, { tenantId, organizationId })
+
+    await em.persistAndFlush(notification)
+
+    await emitNotificationCreated(eventBus, notification, { tenantId, organizationId })
+  } else if (payload.type === 'create-role') {
+    const em = (ctx.resolve('em') as EntityManager).fork()
+    const eventBus = ctx.resolve('eventBus') as { emit: (event: string, payload: unknown) => Promise<void> }
+    const { input, tenantId, organizationId } = payload
+
+    const knex = getKnex(em)
+    const recipientUserIds = await getRecipientUserIdsForRole(knex, tenantId, input.roleId)
+    if (recipientUserIds.length === 0) {
+      return
+    }
+
+    const { roleId: _roleId, ...content } = input
+    const notifications: Notification[] = []
+    for (const recipientUserId of recipientUserIds) {
+      const notification = buildNotificationEntity(em, content, recipientUserId, { tenantId, organizationId })
+      notifications.push(notification)
+    }
+
+    await em.persistAndFlush(notifications)
+
+    await emitNotificationCreatedBatch(eventBus, notifications, { tenantId, organizationId })
+  } else if (payload.type === 'create-feature') {
+    const em = (ctx.resolve('em') as EntityManager).fork()
+    const eventBus = ctx.resolve('eventBus') as { emit: (event: string, payload: unknown) => Promise<void> }
+    const { input, tenantId, organizationId } = payload
+
+    const knex = getKnex(em)
+    const recipientUserIds = await getRecipientUserIdsForFeature(knex, tenantId, input.requiredFeature)
+
+    if (recipientUserIds.length === 0) {
+      return
+    }
+
+    const notifications: Notification[] = []
+    const { requiredFeature: _requiredFeature, ...content } = input
+    for (const recipientUserId of recipientUserIds) {
+      const notification = buildNotificationEntity(em, content, recipientUserId, { tenantId, organizationId })
+      notifications.push(notification)
+    }
+
+    await em.persistAndFlush(notifications)
+
+    await emitNotificationCreatedBatch(eventBus, notifications, { tenantId, organizationId })
+  } else if (payload.type === 'cleanup-expired') {
+    const em = (ctx.resolve('em') as EntityManager).fork()
+    const knex = getKnex(em)
+
+    await knex('notifications')
+      .where('expires_at', '<', knex.fn.now())
+      .whereNotIn('status', ['actioned', 'dismissed'])
+      .update({
+        status: 'dismissed',
+        dismissed_at: knex.fn.now(),
+      })
+  }
+}

package/src/modules/planner/backend/planner/availability-rulesets/page.tsx

@@ -202,8 +202,8 @@ export default function PlannerAvailabilityRuleSetsPage() {
           rowActions={(row) => (
             <RowActions
               items={[
-                { label: labels.actions.edit, href: `/backend/planner/availability-rulesets/${row.id}` },
-                { label: labels.actions.delete, destructive: true, onSelect: () => { void handleDelete(row) } },
+                { id: 'edit', label: labels.actions.edit, href: `/backend/planner/availability-rulesets/${row.id}` },
+                { id: 'delete', label: labels.actions.delete, destructive: true, onSelect: () => { void handleDelete(row) } },
               ]}
             />
           )}

package/src/modules/query_index/cli.ts

@@ -22,6 +22,50 @@ import type { VectorIndexService } from '@open-mercato/search/vector'
 
 type ParsedArgs = Record<string, string | boolean>
 
+type PartitionProgressInfo = { processed: number; total: number }
+
+function isIndexerVerbose(): boolean {
+  const parsed = parseBooleanToken(process.env.OM_INDEXER_VERBOSE ?? '')
+  return parsed === true
+}
+
+function createGroupedProgress(label: string, partitionTargets: number[]) {
+  const totals = new Map<number, number>()
+  const processed = new Map<number, number>()
+  let bar: ProgressBarHandle | null = null
+
+  const getTotals = () => {
+    let total = 0
+    let done = 0
+    for (const value of totals.values()) total += value
+    for (const value of processed.values()) done += value
+    return { total, done }
+  }
+
+  const tryInitBar = () => {
+    if (bar) return
+    if (totals.size < partitionTargets.length) return
+    const { total } = getTotals()
+    if (total <= 0) return
+    bar = createProgressBar(label, total) as ProgressBarHandle
+  }
+
+  return {
+    onProgress(partition: number, info: PartitionProgressInfo) {
+      processed.set(partition, info.processed)
+      if (!totals.has(partition)) totals.set(partition, info.total)
+      tryInitBar()
+      if (!bar) return
+      const { done } = getTotals()
+      bar.update(done)
+    },
+    complete() {
+      if (bar) bar.complete()
+    },
+    getTotals,
+  }
+}
+
 function parseArgs(rest: string[]): ParsedArgs {
   const args: ParsedArgs = {}
   for (let i = 0; i < rest.length; i += 1) {
@@ -550,8 +594,14 @@ const reindex: ModuleCli = {
           await purgeIndexScope(baseEm, { entityType: entity, organizationId: orgId, tenantId })
         }
         console.log(`Reindexing ${entity}${force ? ' (forced)' : ''} in ${partitionTargets.length} partition(s)...`)
-        const progressState = new Map<number, { last: number }>()
-        const renderProgress = (part: number, entityId: string, info: { processed: number; total: number }) => {
+        const verbose = isIndexerVerbose()
+        const progressState = verbose ? new Map<number, { last: number }>() : null
+        const groupedProgress =
+          !verbose && partitionTargets.length > 1
+            ? createGroupedProgress(`Reindexing ${entity}`, partitionTargets)
+            : null
+        const renderProgress = (part: number, entityId: string, info: PartitionProgressInfo) => {
+          if (!progressState) return
           const state = progressState.get(part) ?? { last: 0 }
           const now = Date.now()
           if (now - state.last < 1000 && info.processed < info.total) return
@@ -568,7 +618,7 @@ const reindex: ModuleCli = {
             const label = partitionTargets.length > 1 ? ` [partition ${part + 1}/${partitionCount}]` : ''
             if (partitionTargets.length === 1) {
               console.log(`  -> processing${label}`)
-            } else if (idx === 0) {
+            } else if (verbose && idx === 0) {
               console.log(`  -> processing partitions in parallel (count=${partitionTargets.length})`)
             }
             const partitionContainer = await createRequestContainer()
@@ -596,9 +646,14 @@ const reindex: ModuleCli = {
                 onProgress(info) {
                   if (useBar) {
                     if (info.total > 0 && !progressBar) {
-                    progressBar = createProgressBar(`Reindexing ${entity}${label}`, info.total) as ProgressBarHandle
+                      progressBar = createProgressBar(
+                        `Reindexing ${entity}${label}`,
+                        info.total,
+                      ) as ProgressBarHandle
                     }
                     progressBar?.update(info.processed)
+                  } else if (groupedProgress) {
+                    groupedProgress.onProgress(part, info)
                   } else {
                     renderProgress(part, entity, info)
                   }
@@ -607,7 +662,9 @@ const reindex: ModuleCli = {
               if (progressBar) {
                 (progressBar as ProgressBarHandle).complete()
               }
-              if (!useBar) {
+              if (!useBar && groupedProgress) {
+                groupedProgress.onProgress(part, { processed: partitionStats.processed, total: partitionStats.total })
+              } else if (!useBar) {
                 renderProgress(part, entity, { processed: partitionStats.processed, total: partitionStats.total })
               } else {
                 console.log(
@@ -622,6 +679,7 @@ const reindex: ModuleCli = {
             }
           }),
         )
+        groupedProgress?.complete()
         const totalProcessed = stats.reduce((acc, value) => acc + value, 0)
         console.log(`Finished ${entity}: processed ${totalProcessed} row(s) across ${partitionTargets.length} partition(s)`)
         await recordIndexerLog(
@@ -677,8 +735,14 @@ const reindex: ModuleCli = {
         console.log(
           `[${idx + 1}/${entityIds.length}] Reindexing ${id}${force ? ' (forced)' : ''} in ${partitionTargets.length} partition(s)...`,
         )
-        const progressState = new Map<number, { last: number }>()
-        const renderProgress = (part: number, entityId: string, info: { processed: number; total: number }) => {
+        const verbose = isIndexerVerbose()
+        const progressState = verbose ? new Map<number, { last: number }>() : null
+        const groupedProgress =
+          !verbose && partitionTargets.length > 1
+            ? createGroupedProgress(`Reindexing ${id}`, partitionTargets)
+            : null
+        const renderProgress = (part: number, entityId: string, info: PartitionProgressInfo) => {
+          if (!progressState) return
           const state = progressState.get(part) ?? { last: 0 }
           const now = Date.now()
           if (now - state.last < 1000 && info.processed < info.total) return
@@ -695,7 +759,7 @@ const reindex: ModuleCli = {
             const label = partitionTargets.length > 1 ? ` [partition ${part + 1}/${partitionCount}]` : ''
             if (partitionTargets.length === 1) {
               console.log(`  -> processing${label}`)
-            } else if (partitionIdx === 0) {
+            } else if (verbose && partitionIdx === 0) {
               console.log(`  -> processing partitions in parallel (count=${partitionTargets.length})`)
             }
             const partitionContainer = await createRequestContainer()
@@ -723,18 +787,22 @@ const reindex: ModuleCli = {
                 onProgress(info) {
                   if (useBar) {
                     if (info.total > 0 && !progressBar) {
-                    progressBar = createProgressBar(`Reindexing ${id}${label}`, info.total) as ProgressBarHandle
+                      progressBar = createProgressBar(`Reindexing ${id}${label}`, info.total) as ProgressBarHandle
                     }
                     progressBar?.update(info.processed)
+                  } else if (groupedProgress) {
+                    groupedProgress.onProgress(part, info)
                   } else {
                     renderProgress(part, id, info)
                   }
                 },
               })
-            if (progressBar) {
-              (progressBar as ProgressBarHandle).complete()
-            }
-              if (!useBar) {
+              if (progressBar) {
+                (progressBar as ProgressBarHandle).complete()
+              }
+              if (!useBar && groupedProgress) {
+                groupedProgress.onProgress(part, { processed: result.processed, total: result.total })
+              } else if (!useBar) {
                 renderProgress(part, id, { processed: result.processed, total: result.total })
               } else {
                 console.log(
@@ -749,6 +817,7 @@ const reindex: ModuleCli = {
             }
           }),
         )
+        groupedProgress?.complete()
         const totalProcessed = partitionResults.reduce((acc, value) => acc + value, 0)
         console.log(`  -> ${id} complete: processed ${totalProcessed} row(s) across ${partitionTargets.length} partition(s)`)
         await recordIndexerLog(

package/src/modules/query_index/components/QueryIndexesTable.tsx

@@ -366,13 +366,15 @@ export default function QueryIndexesTable() {
       onSortingChange={setSorting}
       perspective={{ tableId: 'query_index.status.list' }}
       rowActions={(row) => {
-        const items: Array<{ label: string; onSelect: () => void; destructive?: boolean }> = [
-          { label: t('query_index.table.actions.reindex'), onSelect: () => trigger('reindex', row.entityId) },
+        const items: Array<{ id: string; label: string; onSelect: () => void; destructive?: boolean }> = [
+          { id: 'reindex', label: t('query_index.table.actions.reindex'), onSelect: () => trigger('reindex', row.entityId) },
           {
+            id: 'reindex-force',
             label: t('query_index.table.actions.reindexForce'),
             onSelect: () => trigger('reindex', row.entityId, { force: true }),
           },
           {
+            id: 'purge',
             label: t('query_index.table.actions.purge'),
             destructive: true,
             onSelect: () => trigger('purge', row.entityId),
@@ -382,10 +384,12 @@ export default function QueryIndexesTable() {
         if (row.vectorEnabled) {
           items.push(
             {
+              id: 'vector-reindex',
               label: t('query_index.table.actions.vectorReindex'),
               onSelect: () => triggerVector('reindex', row.entityId),
             },
             {
+              id: 'vector-purge',
               label: t('query_index.table.actions.vectorPurge'),
               destructive: true,
               onSelect: () => triggerVector('purge', row.entityId),
@@ -396,10 +400,12 @@ export default function QueryIndexesTable() {
         if (row.fulltextEnabled) {
           items.push(
             {
+              id: 'fulltext-reindex',
               label: t('query_index.table.actions.fulltextReindex'),
               onSelect: () => triggerFulltext('reindex', row.entityId),
             },
             {
+              id: 'fulltext-purge',
               label: t('query_index.table.actions.fulltextPurge'),
               destructive: true,
               onSelect: () => triggerFulltext('purge', row.entityId),

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

@@ -266,10 +266,10 @@ export default function ResourcesResourceTypesPage() {
           rowActions={(row) => (
             <RowActions
               items={[
-                { label: translations.actions.edit, href: `/backend/resources/resource-types/${row.id}/edit` },
+                { id: 'edit', label: translations.actions.edit, href: `/backend/resources/resource-types/${row.id}/edit` },
                 ...(row.resourceCount > 0
                   ? []
-                  : [{ label: translations.actions.delete, destructive: true, onSelect: () => handleDelete(row) }]),
+                  : [{ id: 'delete', label: translations.actions.delete, destructive: true, onSelect: () => handleDelete(row) }]),
               ]}
             />
           )}

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

@@ -481,8 +481,8 @@ export default function ResourcesResourcesPage() {
             if (!canManage || row.rowKind !== 'resource') return null
             return (
               <RowActions items={[
-                { label: t('common.edit', 'Edit'), href: `/backend/resources/resources/${encodeURIComponent(row.id)}` },
-                { label: t('common.delete', 'Delete'), destructive: true, onSelect: () => { void handleDelete(row) } },
+                { id: 'edit', label: t('common.edit', 'Edit'), href: `/backend/resources/resources/${encodeURIComponent(row.id)}` },
+                { id: 'delete', label: t('common.delete', 'Delete'), destructive: true, onSelect: () => { void handleDelete(row) } },
               ]} />
             )
           }}

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

@@ -334,10 +334,12 @@ export default function SalesChannelOffersListPage() {
               <RowActions
                 items={[
                   {
+                    id: 'edit',
                     label: t('sales.channels.offers.actions.edit', 'Edit'),
                     href: `/backend/sales/channels/${row.channelId}/offers/${row.id}/edit`,
                   },
                   {
+                    id: 'delete',
                     label: t('sales.channels.offers.actions.delete', 'Delete'),
                     onSelect: () => handleDelete(row),
                     destructive: true,

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

@@ -191,10 +191,12 @@ export default function SalesChannelsPage() {
             <RowActions
               items={[
                 {
+                  id: 'edit',
                   label: t('sales.channels.table.actions.edit', 'Edit'),
                   href: `/backend/sales/channels/${row.id}/edit`,
                 },
                 {
+                  id: 'delete',
                   label: t('sales.channels.table.actions.delete', 'Delete'),
                   onSelect: () => handleDelete(row),
                 },

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

@@ -11,6 +11,8 @@ import type { DataEngine } from '@open-mercato/shared/lib/data/engine'
 import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
 import { deriveResourceFromCommandId, invalidateCrudCache } from '@open-mercato/shared/lib/crud/cache'
 import { resolveTranslations } from '@open-mercato/shared/lib/i18n/server'
+import { resolveNotificationService } from '../../notifications/lib/notificationService'
+import { buildFeatureNotificationFromType } from '../../notifications/lib/notificationBuilder'
 import { setRecordCustomFields } from '@open-mercato/core/modules/entities/lib/helpers'
 import { loadCustomFieldValues } from '@open-mercato/shared/lib/crud/custom-fields'
 import { normalizeCustomFieldValues } from '@open-mercato/shared/lib/custom-fields/normalize'
@@ -88,6 +90,7 @@ import { resolveDictionaryEntryValue } from '../lib/dictionaries'
 import { resolveStatusEntryIdByValue } from '../lib/statusHelpers'
 import { SalesDocumentNumberGenerator } from '../services/salesDocumentNumberGenerator'
 import { loadSalesSettings } from './settings'
+import { notificationTypes } from '../notifications'
 
 type DocumentAddressSnapshot = {
   id: string
@@ -3111,6 +3114,37 @@ const createQuoteCommand: CommandHandler<QuoteCreateInput, { quoteId: string }>
     })
     await em.flush()
 
+    // Create notification for users with sales.quotes.manage feature
+    try {
+      const notificationService = resolveNotificationService(ctx.container)
+      const typeDef = notificationTypes.find((type) => type.type === 'sales.quote.created')
+      if (typeDef) {
+        const totalAmount = quote.grandTotalGrossAmount && quote.currencyCode
+          ? `${quote.grandTotalGrossAmount} ${quote.currencyCode}`
+          : ''
+        const totalDisplay = totalAmount ? ` (${totalAmount})` : ''
+        const notificationInput = buildFeatureNotificationFromType(typeDef, {
+          requiredFeature: 'sales.quotes.manage',
+          bodyVariables: {
+            quoteNumber: quote.quoteNumber,
+            total: totalDisplay,
+            totalAmount,
+          },
+          sourceEntityType: 'sales:quote',
+          sourceEntityId: quote.id,
+          linkHref: `/backend/sales/quotes/${quote.id}`,
+        })
+
+        await notificationService.createForFeature(notificationInput, {
+          tenantId: quote.tenantId,
+          organizationId: quote.organizationId ?? null,
+        })
+      }
+    } catch (err) {
+      // Notification creation is non-critical, don't fail the command
+      console.error('[sales.quotes.create] Failed to create notification:', err)
+    }
+
     return { quoteId: quote.id }
   },
   captureAfter: async (_input, result, ctx) => {
@@ -3782,6 +3816,37 @@ const createOrderCommand: CommandHandler<OrderCreateInput, { orderId: string }>
     })
     await em.flush()
 
+    // Create notification for users with sales.orders.manage feature
+    try {
+      const notificationService = resolveNotificationService(ctx.container)
+      const typeDef = notificationTypes.find((type) => type.type === 'sales.order.created')
+      if (typeDef) {
+        const totalAmount = order.grandTotalGrossAmount && order.currencyCode
+          ? `${order.grandTotalGrossAmount} ${order.currencyCode}`
+          : ''
+        const totalDisplay = totalAmount ? ` (${totalAmount})` : ''
+        const notificationInput = buildFeatureNotificationFromType(typeDef, {
+          requiredFeature: 'sales.orders.manage',
+          bodyVariables: {
+            orderNumber: order.orderNumber,
+            total: totalDisplay,
+            totalAmount,
+          },
+          sourceEntityType: 'sales:order',
+          sourceEntityId: order.id,
+          linkHref: `/backend/sales/orders/${order.id}`,
+        })
+
+        await notificationService.createForFeature(notificationInput, {
+          tenantId: order.tenantId,
+          organizationId: order.organizationId ?? null,
+        })
+      }
+    } catch (err) {
+      // Notification creation is non-critical, don't fail the command
+      console.error('[sales.orders.create] Failed to create notification:', err)
+    }
+
     return { orderId: order.id }
   },
   captureAfter: async (_input, result, ctx) => {

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

@@ -35,6 +35,9 @@ import { invalidateCrudCache } from '@open-mercato/shared/lib/crud/cache'
 import { emitCrudSideEffects } from '@open-mercato/shared/lib/commands/helpers'
 import type { DataEngine } from '@open-mercato/shared/lib/data/engine'
 import { findOneWithDecryption, findWithDecryption } from '@open-mercato/shared/lib/encryption/find'
+import { resolveNotificationService } from '../../notifications/lib/notificationService'
+import { buildFeatureNotificationFromType } from '../../notifications/lib/notificationBuilder'
+import { notificationTypes } from '../notifications'
 
 export type PaymentAllocationSnapshot = {
   id: string
@@ -422,6 +425,36 @@ const createPaymentCommand: CommandHandler<
     const totals = await recomputeOrderPaymentTotals(em, order)
     await em.flush()
     await invalidateOrderCache(ctx.container, order, ctx.auth?.tenantId ?? null)
+
+    // Create notification for payment received
+    try {
+      const notificationService = resolveNotificationService(ctx.container)
+      const typeDef = notificationTypes.find((type) => type.type === 'sales.payment.received')
+      if (typeDef) {
+        const amountDisplay = payment.amount && payment.currencyCode
+          ? `${payment.currencyCode} ${payment.amount}`
+          : ''
+        const notificationInput = buildFeatureNotificationFromType(typeDef, {
+          requiredFeature: 'sales.orders.manage',
+          bodyVariables: {
+            orderNumber: order.orderNumber ?? '',
+            amount: amountDisplay,
+          },
+          sourceEntityType: 'sales:order',
+          sourceEntityId: order.id,
+          linkHref: `/backend/sales/orders/${order.id}`,
+        })
+
+        await notificationService.createForFeature(notificationInput, {
+          tenantId: payment.tenantId,
+          organizationId: payment.organizationId ?? null,
+        })
+      }
+    } catch (err) {
+      // Notification creation is non-critical, don't fail the command
+      console.error('[sales.payments.create] Failed to create notification:', err)
+    }
+
     return { paymentId: payment.id, orderTotals: totals }
   },
   captureAfter: async (_input, result, ctx) => {

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

@@ -333,8 +333,8 @@ export function AdjustmentKindSettings() {
           rowActions={(entry) => (
             <RowActions
               items={[
-                { label: labels.edit, onSelect: () => openDialog({ mode: 'edit', entry }) },
-                { label: labels.delete, destructive: true, onSelect: () => { void handleDelete(entry) } },
+                { id: 'edit', label: labels.edit, onSelect: () => openDialog({ mode: 'edit', entry }) },
+                { id: 'delete', label: labels.delete, destructive: true, onSelect: () => { void handleDelete(entry) } },
               ]}
             />
           )}

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

@@ -526,8 +526,8 @@ export function PaymentMethodsSettings() {
           rowActions={(row) => (
             <RowActions
               items={[
-                { label: translations.actions.edit, onSelect: () => openEdit(row) },
-                { label: translations.actions.delete, destructive: true, onSelect: () => deleteEntry(row) },
+                { id: 'edit', label: translations.actions.edit, onSelect: () => openEdit(row) },
+                { id: 'delete', label: translations.actions.delete, destructive: true, onSelect: () => deleteEntry(row) },
               ]}
             />
           )}

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

@@ -820,8 +820,8 @@ export function ShippingMethodsSettings() {
           rowActions={(row) => (
             <RowActions
               items={[
-                { label: translations.actions.edit, onSelect: () => openEdit(row) },
-                { label: translations.actions.delete, destructive: true, onSelect: () => deleteEntry(row) },
+                { id: 'edit', label: translations.actions.edit, onSelect: () => openEdit(row) },
+                { id: 'delete', label: translations.actions.delete, destructive: true, onSelect: () => deleteEntry(row) },
               ]}
             />
           )}

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

@@ -335,8 +335,8 @@ export function TaxRatesSettings() {
           rowActions={(row) => (
             <RowActions
               items={[
-                { label: translations.actions.edit, onSelect: () => openEdit(row) },
-                { label: translations.actions.delete, destructive: true, onSelect: () => deleteEntry(row) },
+                { id: 'edit', label: translations.actions.edit, onSelect: () => openEdit(row) },
+                { id: 'delete', label: translations.actions.delete, destructive: true, onSelect: () => deleteEntry(row) },
               ]}
             />
           )}

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

@@ -185,10 +185,12 @@ export function SalesChannelOffersPanel({ channelId, channelName }: { channelId:
             <RowActions
               items={[
                 {
+                  id: 'edit',
                   label: t('sales.channels.offers.actions.edit', 'Edit'),
                   href: `/backend/sales/channels/${channelId}/offers/${row.id}/edit`,
                 },
                 {
+                  id: 'delete',
                   label: t('sales.channels.offers.actions.delete', 'Delete'),
                   onSelect: () => handleDelete(row),
                   destructive: true,

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

@@ -427,10 +427,12 @@ export function SalesDocumentAdjustmentsSection({
       <RowActions
         items={[
           {
+            id: 'edit',
             label: t('ui.actions.edit', 'Edit'),
             onSelect: () => handleEdit(row),
           },
           {
+            id: 'delete',
             label: t('ui.actions.delete', 'Delete'),
             destructive: true,
             onSelect: () => handleDelete(row),

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

@@ -295,8 +295,9 @@ export function SalesDocumentPaymentsSection({
           return (
             <RowActions
               items={[
-                { label: editActionLabel, onSelect: () => openEditPayment(row.original) },
+                { id: 'edit', label: editActionLabel, onSelect: () => openEditPayment(row.original) },
                 {
+                  id: 'delete',
                   label: deleteActionLabel,
                   destructive: true,
                   onSelect: () => void handleDelete(row.original),

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

@@ -699,10 +699,12 @@ export function SalesDocumentsTable({ kind }: { kind: SalesDocumentKind }) {
             <RowActions
               items={[
                 {
+                  id: 'open',
                   label: t('sales.documents.list.table.open', 'Open'),
                   href: `/backend/sales/${resource}/${row.id}?kind=${kind}`,
                 },
                 {
+                  id: 'delete',
                   label:
                     kind === 'order'
                       ? t('sales.documents.list.table.deleteOrder', 'Delete order')