@open-mercato/core 0.6.4-develop.4217.1.c9aa050183 → 0.6.4-develop.4239.1.4a264a5828

package/src/modules/auth/services/sidebarPreferencesService.ts

@@ -1,5 +1,6 @@
 import { EntityManager, type FilterQuery } from '@mikro-orm/postgresql'
 import { findOneWithDecryption, findWithDecryption } from '@open-mercato/shared/lib/encryption/find'
+import { withAtomicFlush } from '@open-mercato/shared/lib/commands/flush'
 import { Role, RoleSidebarPreference, SidebarVariant, User, UserSidebarPreference } from '../data/entities'
 import {
   SIDEBAR_PREFERENCES_VERSION,
@@ -346,21 +347,24 @@ export async function createSidebarVariant(
     version: input.settings?.version ?? SIDEBAR_PREFERENCES_VERSION,
   })
 
-  if (input.isActive === true) {
-    await deactivateAllVariants(em, scope)
-  }
-
-  const variant = em.create(SidebarVariant, {
-    user: em.getReference(User, userId),
-    tenantId,
-    organizationId,
-    locale,
-    name: finalName,
-    settingsJson: settings,
-    isActive: input.isActive === true,
-    createdAt: new Date(),
-  })
-  await em.flush()
+  let variant!: SidebarVariant
+  await withAtomicFlush(em, [
+    async () => {
+      if (input.isActive === true) {
+        await deactivateAllVariants(em, scope)
+      }
+      variant = em.create(SidebarVariant, {
+        user: em.getReference(User, userId),
+        tenantId,
+        organizationId,
+        locale,
+        name: finalName,
+        settingsJson: settings,
+        isActive: input.isActive === true,
+        createdAt: new Date(),
+      })
+    },
+  ], { transaction: true })
   return toVariantRecord(variant)
 }
 
@@ -383,23 +387,27 @@ export async function updateSidebarVariant(
     { tenantId, organizationId },
   )
   if (!variant) return null
-  if (typeof input.name === 'string' && input.name.trim().length > 0) {
-    variant.name = input.name.trim()
-  }
-  if (input.settings) {
-    variant.settingsJson = normalizeSidebarSettings({
-      ...input.settings,
-      version: input.settings.version ?? SIDEBAR_PREFERENCES_VERSION,
-    })
-  }
-  if (typeof input.isActive === 'boolean') {
-    if (input.isActive) {
-      await deactivateAllVariants(em, scope, variantId)
-    }
-    variant.isActive = input.isActive
-  }
-  await em.flush()
-  return toVariantRecord(variant)
+  const target = variant
+  await withAtomicFlush(em, [
+    async () => {
+      if (typeof input.name === 'string' && input.name.trim().length > 0) {
+        target.name = input.name.trim()
+      }
+      if (input.settings) {
+        target.settingsJson = normalizeSidebarSettings({
+          ...input.settings,
+          version: input.settings.version ?? SIDEBAR_PREFERENCES_VERSION,
+        })
+      }
+      if (typeof input.isActive === 'boolean') {
+        if (input.isActive) {
+          await deactivateAllVariants(em, scope, variantId)
+        }
+        target.isActive = input.isActive
+      }
+    },
+  ], { transaction: true })
+  return toVariantRecord(target)
 }
 
 export async function deleteSidebarVariant(

package/src/modules/business_rules/backend/logs/[id]/page.tsx

@@ -10,6 +10,7 @@ import { Page, PageBody } from '@open-mercato/ui/backend/Page'
 import { Button } from '@open-mercato/ui/primitives/button'
 import { Spinner } from '@open-mercato/ui/primitives/spinner'
 import { JsonDisplay } from '@open-mercato/ui/backend/JsonDisplay'
+import { RecordNotFoundState, ErrorMessage } from '@open-mercato/ui/backend/detail'
 
 type RuleExecutionLog = {
   id: string
@@ -54,7 +55,13 @@ export default function ExecutionLogDetailPage() {
     queryFn: async () => {
       const response = await apiFetch(`/api/business_rules/logs/${logId}`)
       if (!response.ok) {
-        throw new Error(t('business_rules.logs.errors.fetchFailed'))
+        const httpErr = new Error(
+          response.status === 404
+            ? t('business_rules.logs.errors.notFound', 'Execution log not found.')
+            : t('business_rules.logs.errors.fetchFailed')
+        ) as Error & { status: number }
+        httpErr.status = response.status
+        throw httpErr
       }
       const result = await response.json()
       return result as RuleExecutionLog
@@ -75,16 +82,34 @@ export default function ExecutionLogDetailPage() {
     )
   }
 
+  const isNotFound = !isLoading && (error as (Error & { status?: number }) | null)?.status === 404
+
+  if (isNotFound) {
+    return (
+      <Page>
+        <PageBody>
+          <RecordNotFoundState
+            label={t('business_rules.logs.errors.notFound', 'Execution log not found.')}
+            backHref="/backend/logs"
+            backLabel={t('business_rules.logs.backToList', 'Back to logs')}
+          />
+        </PageBody>
+      </Page>
+    )
+  }
+
   if (error || !log) {
     return (
       <Page>
         <PageBody>
-          <div className="flex h-[50vh] flex-col items-center justify-center gap-2 text-muted-foreground">
-            <p>{error ? t('business_rules.logs.errors.loadFailed') : t('business_rules.logs.errors.notFound')}</p>
-            <Button asChild variant="outline">
-              <Link href="/backend/logs">{t('business_rules.logs.backToList')}</Link>
-            </Button>
-          </div>
+          <ErrorMessage
+            label={(error as Error | null)?.message ?? t('business_rules.logs.errors.loadFailed')}
+            action={
+              <Button asChild variant="outline" size="sm">
+                <Link href="/backend/logs">{t('business_rules.logs.backToList', 'Back to logs')}</Link>
+              </Button>
+            }
+          />
         </PageBody>
       </Page>
     )

package/src/modules/catalog/api/offers/route.ts

@@ -94,11 +94,25 @@ export async function decorateOffersWithDetails(
     .filter((value): value is string => !!value)
   if (!offerIds.length && !productIds.length) return
   const em = ctx.container.resolve('em') as EntityManager
+  const scopeTenantId = ctx.auth?.tenantId ?? null
+  if (!scopeTenantId) {
+    throw new CrudHttpError(403, '[internal] Missing tenant scope for offer decoration')
+  }
+  const scopeOrgIds =
+    Array.isArray(ctx.organizationIds) && ctx.organizationIds.length
+      ? Array.from(new Set(ctx.organizationIds))
+      : (ctx.selectedOrganizationId ?? ctx.auth?.orgId ?? null)
+        ? [(ctx.selectedOrganizationId ?? ctx.auth?.orgId) as string]
+        : []
+  const scopeWhere: Record<string, unknown> = { tenantId: scopeTenantId }
+  if (scopeOrgIds.length === 1) scopeWhere.organizationId = scopeOrgIds[0]
+  else if (scopeOrgIds.length > 1) scopeWhere.organizationId = { $in: scopeOrgIds }
+  const scope = { tenantId: scopeTenantId, organizationId: scopeOrgIds.length === 1 ? scopeOrgIds[0] : null }
   const [products, prices, defaultVariants] = await Promise.all([
     productIds.length
       ? em.find(
           CatalogProduct,
-          { id: { $in: productIds } },
+          { id: { $in: productIds }, ...scopeWhere },
           {
             fields: ['id', 'title', 'description', 'defaultMediaId', 'defaultMediaUrl', 'sku'],
           },
@@ -108,15 +122,15 @@ export async function decorateOffersWithDetails(
       ? findWithDecryption(
           em,
           CatalogProductPrice,
-          { offer: { $in: offerIds } },
+          { offer: { $in: offerIds }, ...scopeWhere },
           { populate: ['priceKind'] },
-          { tenantId: ctx.auth?.tenantId ?? null, organizationId: ctx.auth?.orgId ?? null },
+          scope,
         )
       : [],
     productIds.length
       ? em.find(
           CatalogProductVariant,
-          { product: { $in: productIds }, isDefault: true },
+          { product: { $in: productIds }, isDefault: true, ...scopeWhere },
           { fields: ['id', 'product'] },
         )
       : [],
@@ -227,6 +241,7 @@ export async function decorateOffersWithDetails(
         CatalogProductPrice,
         {
           offer: null,
+          ...scopeWhere,
           $and: [
             { $or: fallbackTargets },
             channelFilterValues.includes(null)
@@ -240,7 +255,7 @@ export async function decorateOffersWithDetails(
           ],
         },
         { populate: ['priceKind'] },
-        { tenantId: ctx.auth?.tenantId ?? null, organizationId: ctx.auth?.orgId ?? null },
+        scope,
       )
     : []
   fallbackEntries.forEach((entry) => {

package/src/modules/catalog/commands/categories.ts

@@ -2,6 +2,7 @@ import { registerCommand } from '@open-mercato/shared/lib/commands'
 import type { CommandHandler } from '@open-mercato/shared/lib/commands'
 import { buildChanges, requireId, parseWithCustomFields, setCustomFieldsIfAny, emitCrudSideEffects } from '@open-mercato/shared/lib/commands/helpers'
 import { loadCustomFieldSnapshot, buildCustomFieldResetMap } from '@open-mercato/shared/lib/commands/customFieldSnapshots'
+import { withAtomicFlush } from '@open-mercato/shared/lib/commands/flush'
 import type { EntityManager } from '@mikro-orm/postgresql'
 import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
 import { resolveTranslations } from '@open-mercato/shared/lib/i18n/server'
@@ -160,8 +161,16 @@ const createCategoryCommand: CommandHandler<CategoryCreateInput, { categoryId: s
       updatedAt: now,
     })
     em.persist(record)
-    await em.flush()
-    await rebuildCategoryHierarchyForOrganization(em, record.organizationId, record.tenantId)
+    await withAtomicFlush(
+      em,
+      [
+        () => em.flush(),
+        async () => {
+          await rebuildCategoryHierarchyForOrganization(em, record.organizationId, record.tenantId)
+        },
+      ],
+      { transaction: true }
+    )
     await setCustomFieldsIfAny({
       dataEngine: ctx.container.resolve('dataEngine'),
       entityId: E.catalog.catalog_product_category,
@@ -215,8 +224,16 @@ const createCategoryCommand: CommandHandler<CategoryCreateInput, { categoryId: s
     ensureOrganizationScope(ctx, record.organizationId)
     record.deletedAt = new Date()
     record.isActive = false
-    await em.flush()
-    await rebuildCategoryHierarchyForOrganization(em, record.organizationId, record.tenantId)
+    await withAtomicFlush(
+      em,
+      [
+        () => em.flush(),
+        async () => {
+          await rebuildCategoryHierarchyForOrganization(em, record.organizationId, record.tenantId)
+        },
+      ],
+      { transaction: true }
+    )
     const resetValues = buildCustomFieldResetMap(undefined, after.custom ?? undefined)
     if (Object.keys(resetValues).length) {
       await setCustomFieldsIfAny({
@@ -288,8 +305,16 @@ const updateCategoryCommand: CommandHandler<CategoryUpdateInput, { categoryId: s
       record.isActive = parsed.isActive
     }
 
-    await em.flush()
-    await rebuildCategoryHierarchyForOrganization(em, record.organizationId, record.tenantId)
+    await withAtomicFlush(
+      em,
+      [
+        () => em.flush(),
+        async () => {
+          await rebuildCategoryHierarchyForOrganization(em, record.organizationId, record.tenantId)
+        },
+      ],
+      { transaction: true }
+    )
     await setCustomFieldsIfAny({
       dataEngine: ctx.container.resolve('dataEngine'),
       entityId: E.catalog.catalog_product_category,
@@ -377,8 +402,16 @@ const updateCategoryCommand: CommandHandler<CategoryUpdateInput, { categoryId: s
       record.isActive = before.isActive
       record.deletedAt = null
     }
-    await em.flush()
-    await rebuildCategoryHierarchyForOrganization(em, before.organizationId, before.tenantId)
+    await withAtomicFlush(
+      em,
+      [
+        () => em.flush(),
+        async () => {
+          await rebuildCategoryHierarchyForOrganization(em, before.organizationId, before.tenantId)
+        },
+      ],
+      { transaction: true }
+    )
     const resetValues = buildCustomFieldResetMap(payload?.after?.custom ?? undefined, before.custom ?? undefined)
     if (Object.keys(resetValues).length) {
       await setCustomFieldsIfAny({
@@ -417,8 +450,16 @@ const deleteCategoryCommand: CommandHandler<{ id?: string }, { categoryId: strin
 
     record.deletedAt = new Date()
     record.isActive = false
-    await em.flush()
-    await rebuildCategoryHierarchyForOrganization(em, record.organizationId, record.tenantId)
+    await withAtomicFlush(
+      em,
+      [
+        () => em.flush(),
+        async () => {
+          await rebuildCategoryHierarchyForOrganization(em, record.organizationId, record.tenantId)
+        },
+      ],
+      { transaction: true }
+    )
     if (snapshot?.custom && Object.keys(snapshot.custom).length) {
       const resetValues = buildCustomFieldResetMap(snapshot.custom, undefined)
       if (Object.keys(resetValues).length) {
@@ -504,8 +545,16 @@ const deleteCategoryCommand: CommandHandler<{ id?: string }, { categoryId: strin
       record.childIds = before.childIds
       record.descendantIds = before.descendantIds
     }
-    await em.flush()
-    await rebuildCategoryHierarchyForOrganization(em, before.organizationId, before.tenantId)
+    await withAtomicFlush(
+      em,
+      [
+        () => em.flush(),
+        async () => {
+          await rebuildCategoryHierarchyForOrganization(em, before.organizationId, before.tenantId)
+        },
+      ],
+      { transaction: true }
+    )
     if (before.custom && Object.keys(before.custom).length) {
       await setCustomFieldsIfAny({
         dataEngine: ctx.container.resolve('dataEngine'),

package/src/modules/catalog/commands/products.ts

@@ -26,6 +26,7 @@ import {
   loadCustomFieldSnapshot,
   buildCustomFieldResetMap,
 } from "@open-mercato/shared/lib/commands/customFieldSnapshots";
+import { withAtomicFlush } from "@open-mercato/shared/lib/commands/flush";
 import { E } from "#generated/entities.ids.generated";
 import { slugifyTagLabel } from "@open-mercato/shared/lib/utils";
 import { parseObjectLike } from "@open-mercato/shared/lib/json/parseObjectLike";
@@ -950,15 +951,15 @@ type VariantCleanupSnapshot = {
   custom: Record<string, unknown> | null;
 };
 
-async function deleteProductVariantsAndRelatedData(opts: {
-  em: EntityManager;
-  product: CatalogProduct;
-  dataEngine: DataEngine;
-  ctx: CommandRuntimeContext;
-}): Promise<void> {
-  const { em, product, dataEngine, ctx } = opts;
+async function collectProductVariantCleanup(
+  em: EntityManager,
+  product: CatalogProduct,
+): Promise<{
+  variants: CatalogProductVariant[];
+  cleanupEntries: VariantCleanupSnapshot[];
+}> {
   const variants = await em.find(CatalogProductVariant, { product });
-  if (!variants.length) return;
+  if (!variants.length) return { variants: [], cleanupEntries: [] };
   const cleanupEntries: VariantCleanupSnapshot[] = await Promise.all(
     variants.map(async (variant) => {
       const custom = await loadCustomFieldSnapshot(em, {
@@ -975,6 +976,14 @@ async function deleteProductVariantsAndRelatedData(opts: {
       };
     }),
   );
+  return { variants, cleanupEntries };
+}
+
+async function removeProductVariants(
+  em: EntityManager,
+  variants: CatalogProductVariant[],
+): Promise<void> {
+  if (!variants.length) return;
   const variantIds = variants.map((variant) => variant.id);
   if (variantIds.length) {
     await em.nativeDelete(CatalogProductPrice, {
@@ -984,7 +993,14 @@ async function deleteProductVariantsAndRelatedData(opts: {
   for (const variant of variants) {
     em.remove(variant);
   }
-  await em.flush();
+}
+
+async function emitProductVariantCleanupSideEffects(opts: {
+  dataEngine: DataEngine;
+  ctx: CommandRuntimeContext;
+  cleanupEntries: VariantCleanupSnapshot[];
+}): Promise<void> {
+  const { dataEngine, ctx, cleanupEntries } = opts;
   for (const cleanup of cleanupEntries) {
     if (!cleanup.custom) continue;
     const resetValues = buildCustomFieldResetMap(cleanup.custom, undefined);
@@ -1307,15 +1323,16 @@ const createProductCommand: CommandHandler<
     }
     em.persist(record);
     try {
-      await em.flush();
-    } catch (error) {
-      await rethrowProductUniqueConstraint(error);
-    }
-    await syncOffers(em, record, parsed.offers);
-    await syncCategoryAssignments(em, record, parsed.categoryIds);
-    await syncProductTags(em, record, parsed.tags);
-    try {
-      await em.flush();
+      await withAtomicFlush(
+        em,
+        [
+          () => em.flush(),
+          () => syncOffers(em, record, parsed.offers),
+          () => syncCategoryAssignments(em, record, parsed.categoryIds),
+          () => syncProductTags(em, record, parsed.tags),
+        ],
+        { transaction: true },
+      );
     } catch (error) {
       await rethrowProductUniqueConstraint(error);
     }
@@ -1614,15 +1631,16 @@ const updateProductCommand: CommandHandler<
       record.isConfigurable = parsed.isConfigurable;
     if (parsed.isActive !== undefined) record.isActive = parsed.isActive;
     try {
-      await em.flush();
-    } catch (error) {
-      await rethrowProductUniqueConstraint(error);
-    }
-    await syncOffers(em, record, parsed.offers);
-    await syncCategoryAssignments(em, record, parsed.categoryIds);
-    await syncProductTags(em, record, parsed.tags);
-    try {
-      await em.flush();
+      await withAtomicFlush(
+        em,
+        [
+          () => em.flush(),
+          () => syncOffers(em, record, parsed.offers),
+          () => syncCategoryAssignments(em, record, parsed.categoryIds),
+          () => syncProductTags(em, record, parsed.tags),
+        ],
+        { transaction: true },
+      );
     } catch (error) {
       await rethrowProductUniqueConstraint(error);
     }
@@ -1732,16 +1750,16 @@ const updateProductCommand: CommandHandler<
     ensureTenantScope(ctx, before.tenantId);
     ensureOrganizationScope(ctx, before.organizationId);
     applyProductSnapshot(em, record, before);
-    await em.flush();
-
-    const relationEm = em.fork();
-    const relationRecord = await findOneWithDecryption(relationEm, CatalogProduct, { id: before.id });
-    if (relationRecord) {
-      await restoreOffersFromSnapshot(relationEm, relationRecord, before.offers);
-      await syncCategoryAssignments(relationEm, relationRecord, before.categoryIds);
-      await syncProductTags(relationEm, relationRecord, before.tags);
-      await relationEm.flush();
-    }
+    await withAtomicFlush(
+      em,
+      [
+        () => em.flush(),
+        () => restoreOffersFromSnapshot(em, record, before.offers),
+        () => syncCategoryAssignments(em, record, before.categoryIds),
+        () => syncProductTags(em, record, before.tags),
+      ],
+      { transaction: true },
+    );
     const dataEngine = ctx.container.resolve("dataEngine") as DataEngine;
     const resetValues = buildCustomFieldResetMap(
       before.custom ?? undefined,
@@ -1803,23 +1821,38 @@ const deleteProductCommand: CommandHandler<
     ensureTenantScope(ctx, record.tenantId);
     ensureOrganizationScope(ctx, record.organizationId);
     const dataEngine = ctx.container.resolve("dataEngine") as DataEngine;
-    await deleteProductVariantsAndRelatedData({
+    const { variants, cleanupEntries } = await collectProductVariantCleanup(
       em,
-      product: record,
+      record,
+    );
+    await withAtomicFlush(
+      em,
+      [
+        () => removeProductVariants(em, variants),
+        async () => {
+          await em.nativeDelete(CatalogProductPrice, { product: record.id });
+        },
+        async () => {
+          const templateToRemove = await resolveOptionSchemaTemplateForRemoval(
+            em,
+            record,
+          );
+          if (templateToRemove) {
+            record.optionSchemaTemplate = null;
+            em.remove(templateToRemove);
+          }
+        },
+        () => {
+          em.remove(record);
+        },
+      ],
+      { transaction: true },
+    );
+    await emitProductVariantCleanupSideEffects({
       dataEngine,
       ctx,
+      cleanupEntries,
     });
-    await em.nativeDelete(CatalogProductPrice, { product: record.id });
-    const templateToRemove = await resolveOptionSchemaTemplateForRemoval(
-      em,
-      record,
-    );
-    if (templateToRemove) {
-      record.optionSchemaTemplate = null;
-      em.remove(templateToRemove);
-    }
-    em.remove(record);
-    await em.flush();
     if (snapshot?.custom && Object.keys(snapshot.custom).length) {
       const resetValues = buildCustomFieldResetMap(snapshot.custom, undefined);
       if (Object.keys(resetValues).length) {
@@ -1908,16 +1941,16 @@ const deleteProductCommand: CommandHandler<
     ensureTenantScope(ctx, before.tenantId);
     ensureOrganizationScope(ctx, before.organizationId);
     applyProductSnapshot(em, record, before);
-    await em.flush();
-
-    const relationEm = em.fork();
-    const relationRecord = await findOneWithDecryption(relationEm, CatalogProduct, { id: before.id });
-    if (relationRecord) {
-      await restoreOffersFromSnapshot(relationEm, relationRecord, before.offers);
-      await syncCategoryAssignments(relationEm, relationRecord, before.categoryIds);
-      await syncProductTags(relationEm, relationRecord, before.tags);
-      await relationEm.flush();
-    }
+    await withAtomicFlush(
+      em,
+      [
+        () => em.flush(),
+        () => restoreOffersFromSnapshot(em, record, before.offers),
+        () => syncCategoryAssignments(em, record, before.categoryIds),
+        () => syncProductTags(em, record, before.tags),
+      ],
+      { transaction: true },
+    );
     const dataEngine = ctx.container.resolve("dataEngine") as DataEngine;
     if (before.custom && Object.keys(before.custom).length) {
       await setCustomFieldsIfAny({

package/src/modules/catalog/commands/variants.ts

@@ -6,6 +6,7 @@ import { UniqueConstraintViolationException } from '@mikro-orm/core'
 import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
 import { resolveTranslations } from '@open-mercato/shared/lib/i18n/server'
 import { loadCustomFieldSnapshot, buildCustomFieldResetMap } from '@open-mercato/shared/lib/commands/customFieldSnapshots'
+import { withAtomicFlush } from '@open-mercato/shared/lib/commands/flush'
 import { E } from '#generated/entities.ids.generated'
 import { findWithDecryption } from '@open-mercato/shared/lib/encryption/find'
 import {
@@ -588,21 +589,25 @@ const createVariantCommand: CommandHandler<VariantCreateInput, { variantId: stri
       updatedAt: now,
     })
     em.persist(record)
+    let previousDefaultVariantId: string | null = null
     try {
-      await em.flush()
+      await withAtomicFlush(
+        em,
+        [
+          () => em.flush(),
+          async () => {
+            if (record.isDefault) {
+              previousDefaultVariantId = await enforceSingleDefaultVariant(em, record)
+              await em.flush()
+            }
+          },
+          () => aggregateVariantMediaToProduct(em, record),
+        ],
+        { transaction: true }
+      )
     } catch (error) {
       await rethrowVariantUniqueConstraint(error)
     }
-    let previousDefaultVariantId: string | null = null
-    if (record.isDefault) {
-      previousDefaultVariantId = await enforceSingleDefaultVariant(em, record)
-      try {
-        await em.flush()
-      } catch (error) {
-        await rethrowVariantUniqueConstraint(error)
-      }
-    }
-    await aggregateVariantMediaToProduct(em, record)
     await setCustomFieldsIfAny({
       dataEngine: ctx.container.resolve('dataEngine'),
       entityId: E.catalog.catalog_product_variant,
@@ -753,15 +758,23 @@ const updateVariantCommand: CommandHandler<VariantUpdateInput, { variantId: stri
     }
 
     let previousDefaultVariantId: string | null = null
-    if (parsed.isDefault === true) {
-      previousDefaultVariantId = await enforceSingleDefaultVariant(em, record)
-    }
     try {
-      await em.flush()
+      await withAtomicFlush(
+        em,
+        [
+          async () => {
+            if (parsed.isDefault === true) {
+              previousDefaultVariantId = await enforceSingleDefaultVariant(em, record)
+            }
+          },
+          () => em.flush(),
+          () => aggregateVariantMediaToProduct(em, record),
+        ],
+        { transaction: true }
+      )
     } catch (error) {
       await rethrowVariantUniqueConstraint(error)
     }
-    await aggregateVariantMediaToProduct(em, record)
     if (custom && Object.keys(custom).length) {
       await setCustomFieldsIfAny({
         dataEngine: ctx.container.resolve('dataEngine'),