@open-mercato/core 0.6.6-develop.6460.1.f94c74174c → 0.6.6-develop.6471.1.9ab5bdd79a

package/src/modules/business_rules/lib/expression-evaluator.ts

@@ -1,6 +1,9 @@
 import type { ComparisonOperator, LogicalOperator } from '../data/validators'
 import { testLinearRegex } from '@open-mercato/shared/lib/regex/linear'
 import { getNestedValue, resolveSpecialValue } from './value-resolver'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('business_rules').child({ component: 'expression-evaluator' })
 
 /**
  * Type definitions for condition expressions
@@ -83,14 +86,13 @@ function evaluateSimpleCondition(
 
   const result = applyOperator(leftValue, condition.operator, rightValue)
 
-  // Detailed logging for debugging
-  console.log('[RULE EVAL] Simple condition:', {
+  logger.debug('Simple condition evaluated', {
     field: condition.field,
     operator: condition.operator,
     expectedValue: rightValue,
     actualValue: leftValue,
     actualValueType: typeof leftValue,
-    result: result ? '✓ PASS' : '✗ FAIL',
+    passed: result,
   })
 
   return result
@@ -110,7 +112,7 @@ function evaluateGroupCondition(
     return true
   }
 
-  console.log(`[RULE EVAL] Group condition: ${operator} with ${rules.length} rules`)
+  logger.debug('Group condition evaluating', { operator, ruleCount: rules.length })
 
   let result: boolean
 
@@ -137,7 +139,7 @@ function evaluateGroupCondition(
       throw new Error(`Unknown logical operator: ${operator}`)
   }
 
-  console.log(`[RULE EVAL] Group ${operator} result: ${result ? '✓ PASS' : '✗ FAIL'}`)
+  logger.debug('Group condition evaluated', { operator, passed: result })
 
   return result
 }
@@ -357,7 +359,7 @@ function matches(str: any, pattern: any): boolean {
     // Log the error for debugging but don't expose to user
     const message = error instanceof Error ? error.message : 'Unknown error'
     if (process.env.NODE_ENV !== 'test') {
-      console.error('Regex matching failed:', message)
+      logger.error('Regex matching failed', { message })
     }
     return false
   }

package/src/modules/business_rules/lib/rule-engine.ts

@@ -7,6 +7,9 @@ import * as actionExecutor from './action-executor'
 import type { RuleEvaluationContext } from './rule-evaluator'
 import type { ActionContext, ActionExecutionOutcome } from './action-executor'
 import { ruleEngineContextSchema, ruleDiscoveryOptionsSchema, directRuleExecutionContextSchema, ruleIdExecutionContextSchema } from '../data/validators'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('business_rules').child({ component: 'rule-engine' })
 
 /**
  * Constants
@@ -156,7 +159,7 @@ async function getCachedRuleDiscovery(
       cache.get(getRuleDiscoveryCacheKey(options))
     )
   } catch (error) {
-    console.warn('[business_rules] Failed to read rule discovery cache:', error)
+    logger.warn('Failed to read rule discovery cache', { err: error })
     return null
   }
 
@@ -188,7 +191,7 @@ async function cacheRuleDiscovery(
       )
     )
   } catch (error) {
-    console.warn('[business_rules] Failed to write rule discovery cache:', error)
+    logger.warn('Failed to write rule discovery cache', { err: error })
   }
 }
 
@@ -213,7 +216,7 @@ export async function invalidateBusinessRuleDiscoveryCache(
   try {
     await runWithCacheTenant(normalizedTenantId, () => cache.deleteByTags(tags))
   } catch (error) {
-    console.warn('[business_rules] Failed to invalidate rule discovery cache:', error)
+    logger.warn('Failed to invalidate rule discovery cache', { err: error })
   }
 }
 

package/src/modules/business_rules/lib/rule-evaluator.ts

@@ -1,5 +1,8 @@
 import type { BusinessRule } from '../data/entities'
 import { evaluateExpression, type EvaluationContext, type ConditionExpression } from './expression-evaluator'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('business_rules').child({ component: 'rule-evaluator' })
 
 /**
  * Rule evaluation context
@@ -102,20 +105,19 @@ export async function evaluateSingleRule(
 ): Promise<SingleRuleResult> {
   const startTime = Date.now()
 
-  console.log('[RULE EVAL] ========================================')
-  console.log('[RULE EVAL] Evaluating rule:', {
+  logger.debug('Evaluating rule', {
     ruleId: rule.ruleId,
     ruleName: rule.ruleName,
     ruleType: rule.ruleType,
     entityType: rule.entityType,
     eventType: rule.eventType,
+    conditions: rule.conditionExpression,
   })
-  console.log('[RULE EVAL] Conditions:', JSON.stringify(rule.conditionExpression, null, 2))
 
   try {
     // Check if rule is enabled
     if (!rule.enabled) {
-      console.log('[RULE EVAL] ✗ Rule is disabled')
+      logger.debug('Rule is disabled', { ruleId: rule.ruleId })
       return {
         rule,
         conditionsPassed: false,
@@ -127,7 +129,7 @@ export async function evaluateSingleRule(
 
     // Check effective date range
     if (!isRuleEffective(rule)) {
-      console.log('[RULE EVAL] ✗ Rule is not effective (outside date range)')
+      logger.debug('Rule is not effective (outside date range)', { ruleId: rule.ruleId })
       return {
         rule,
         conditionsPassed: false,
@@ -142,12 +144,11 @@ export async function evaluateSingleRule(
 
     const evaluationTime = Math.max(0, Date.now() - startTime)
 
-    console.log('[RULE EVAL] Final result:', {
+    logger.debug('Rule evaluated', {
       ruleId: rule.ruleId,
-      conditionsPassed: conditionsPassed ? '✓ PASS' : '✗ FAIL',
-      evaluationTime: `${evaluationTime}ms`,
+      conditionsPassed,
+      evaluationTimeMs: evaluationTime,
     })
-    console.log('[RULE EVAL] ========================================')
 
     return {
       rule,
@@ -159,8 +160,7 @@ export async function evaluateSingleRule(
     const evaluationTime = Math.max(0, Date.now() - startTime)
     const errorMessage = error instanceof Error ? error.message : String(error)
 
-    console.log('[RULE EVAL] ✗ ERROR:', errorMessage)
-    console.log('[RULE EVAL] ========================================')
+    logger.debug('Rule evaluation error', { ruleId: rule.ruleId, message: errorMessage })
 
     return {
       rule,

package/src/modules/business_rules/subscribers/crud-rule-trigger.ts

@@ -12,6 +12,9 @@
 
 import type { EntityManager } from '@mikro-orm/core'
 import { executeRules, resolveBusinessRuleDiscoveryCache } from '../lib/rule-engine'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('business_rules').child({ component: 'crud-rule-trigger' })
 
 export const metadata = {
   event: '*',
@@ -75,6 +78,6 @@ export default async function handle(
       organizationId,
     }, { cache })
   } catch (error) {
-    console.error(`[business_rules] Rule execution failed for event ${eventName}:`, error)
+    logger.error('Rule execution failed for event', { event: eventName, err: error })
   }
 }

package/src/modules/business_rules/subscribers/rule-execution-failed-notification.ts

@@ -2,6 +2,9 @@ import type { EntityManager } from '@mikro-orm/postgresql'
 import { resolveNotificationService } from '../../notifications/lib/notificationService'
 import { buildFeatureNotificationFromType } from '../../notifications/lib/notificationBuilder'
 import { notificationTypes } from '../notifications'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('business_rules').child({ component: 'rule-execution-failed-notification' })
 
 export const metadata = {
   event: 'business_rules.rule.execution_failed',
@@ -45,6 +48,6 @@ export default async function handle(payload: RuleExecutionFailedPayload, ctx: R
       organizationId: payload.organizationId ?? null,
     })
   } catch (err) {
-    console.error('[business_rules:rule-execution-failed-notification] Failed to create notification:', err)
+    logger.error('Failed to create notification', { err })
   }
 }

package/src/modules/catalog/ai-agents-context.ts

@@ -34,6 +34,9 @@ import type {
   CatalogAiToolDefinition,
   CatalogToolContext,
 } from './ai-tools/types'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 const UUID_REGEX = /^[0-9a-f]{8}-[0-9a-f]{4}-[1-5][0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$/i
 
@@ -110,17 +113,14 @@ async function invokeTool(
 ): Promise<unknown | null> {
   const tool = findTool(toolName)
   if (!tool) {
-    console.warn(`[${reasonPrefix}] resolvePageContext: tool "${toolName}" not registered`)
+    logger.warn('resolvePageContext tool not registered', { reasonPrefix, toolName })
     return null
   }
   try {
     const result = await tool.handler(args as never, toolContext)
     return result ?? null
   } catch (error) {
-    console.warn(
-      `[${reasonPrefix}] resolvePageContext: tool "${toolName}" failed (reason="hydration_error"); skipping`,
-      error instanceof Error ? error.message : error,
-    )
+    logger.warn('resolvePageContext tool failed; skipping', { reasonPrefix, toolName, reason: 'hydration_error', err: error })
     return null
   }
 }

package/src/modules/catalog/ai-tools/_shared.ts

@@ -43,6 +43,9 @@ import {
 import type { CatalogPricingService } from '../services/catalogPricingService'
 import type { PriceRow, PricingContext } from '../lib/pricing'
 import type { CatalogToolContext } from './types'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 /* -------------------------------------------------------------------------- */
 /*  Price-kind enumeration shared core                                         */
@@ -443,7 +446,7 @@ export async function buildProductBundle(
         }
       }
     } catch (error) {
-      console.warn('[catalog.get_product_bundle] resolvePrice failed, omitting best price', error)
+      logger.warn('catalog.get_product_bundle resolvePrice failed, omitting best price', { err: error })
     }
   }
 

package/src/modules/catalog/ai-tools/authoring-pack.ts

@@ -58,6 +58,9 @@ import {
 } from './_shared'
 import type { CatalogPricingService } from '../services/catalogPricingService'
 import type { PriceRow, PricingContext } from '../lib/pricing'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 /* -------------------------------------------------------------------------- */
 /*  Shared helpers                                                             */
@@ -295,7 +298,7 @@ async function loadUserMedia(
   const tenantScoped = rows.filter((row) => (row.tenantId ?? null) === tenantId)
   for (const dropped of unique) {
     if (!tenantScoped.find((row) => row.id === dropped)) {
-      console.warn(`[catalog.draft_description_from_media] dropping attachment not in scope: ${dropped}`)
+      logger.warn('catalog.draft_description_from_media dropping attachment not in scope', { attachmentId: dropped })
     }
   }
   return tenantScoped.map((row) => ({

package/src/modules/catalog/ai-tools/merchandising-pack.ts

@@ -60,6 +60,9 @@ import {
   type ProductBundle,
   type ProductBundleResult,
 } from './_shared'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 type SearchServiceLike = {
   search: (query: string, options: {
@@ -414,7 +417,7 @@ const listSelectedProductsTool: CatalogAiToolDefinition = {
         items.push(entry.result)
       } else {
         missingIds.push(entry.productId)
-        console.warn(`[catalog.list_selected_products] product not in scope: ${entry.productId}`)
+        logger.warn('catalog.list_selected_products product not in scope', { productId: entry.productId })
       }
     }
     return { items, missingIds }

package/src/modules/catalog/api/dictionaries/[key]/route.ts

@@ -4,6 +4,9 @@ import { Dictionary, DictionaryEntry } from '@open-mercato/core/modules/dictiona
 import { resolveDictionariesRouteContext } from '@open-mercato/core/modules/dictionaries/api/context'
 import { CrudHttpError, isCrudHttpError } from '@open-mercato/shared/lib/crud/errors'
 import type { OpenApiRouteDoc } from '@open-mercato/shared/lib/openapi'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 const KEY_ALIASES: Record<string, string[]> = {
   currency: ['currency', 'currencies'],
@@ -65,7 +68,7 @@ export async function GET(
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[catalog.dictionaries.GET] Unexpected error', err)
+    logger.error('catalog.dictionaries.GET Unexpected error', { err })
     return NextResponse.json({ error: 'Failed to load dictionary.' }, { status: 500 })
   }
 }

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

@@ -27,6 +27,9 @@ import {
   findWithDecryption,
   findOneWithDecryption,
 } from "@open-mercato/shared/lib/encryption/find";
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 const rawBodySchema = z.object({}).passthrough();
 
@@ -235,7 +238,7 @@ const crud = makeCrudRoute({
         Object.assign(filters, cfFilters);
       } catch (err) {
         // Custom field filter parsing may fail for non-existent or misconfigured fields.
-        if (process.env.NODE_ENV === 'development') console.warn('[catalog:prices] custom field filter error', err);
+        logger.debug('catalog.prices custom field filter error', { err });
       }
       if (
         typeof query.quantity === "number" &&

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

@@ -49,6 +49,9 @@ import {
 } from "../openapi";
 import { findWithDecryption } from "@open-mercato/shared/lib/encryption/find";
 import { canonicalizeUnitCode, toUnitLookupKey } from "../../lib/unitCodes";
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 const rawBodySchema = z.object({}).passthrough();
 
 const UUID_REGEX =
@@ -296,7 +299,7 @@ export async function buildProductFilters(
     } catch (err) {
       // Custom field filter parsing may fail for non-existent or misconfigured fields.
       // Fall back to base filters to avoid blocking the product listing.
-      if (process.env.NODE_ENV === 'development') console.warn('[catalog:products] custom field filter error', err);
+      logger.debug('catalog.products custom field filter error', { err });
       return {};
     }
   };
@@ -696,7 +699,7 @@ async function decorateProductsAfterList(
         const productConversions = conversionsByProduct.get(id);
         const factor = productConversions?.get(requestQuantityUnitKey) ?? null;
         if (!factor || !Number.isFinite(factor) || factor <= 0) {
-          if (process.env.NODE_ENV === 'development') console.warn(`[catalog.products] Invalid conversion factor for product=${id} unit=${requestQuantityUnitKey} factor=${factor}`);
+          logger.debug('catalog.products invalid conversion factor', { productId: id, unit: requestQuantityUnitKey, factor });
           return pricingContext.quantity;
         }
         const normalized = pricingContext.quantity * factor;
@@ -757,7 +760,7 @@ async function decorateProductsAfterList(
       }
     }
   } catch (error) {
-    console.error("[decorateProductsAfterList] Failed to load unit conversions", error);
+    logger.error('decorateProductsAfterList Failed to load unit conversions', { err: error });
   }
 
   const searchTerm = ctx.query.search ? sanitizeSearchTerm(ctx.query.search) : null;

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

@@ -14,6 +14,9 @@ import {
   UNIT_PRICE_DISPLAY_ENABLED_DEFAULT,
   UNIT_PRICE_DISPLAY_ENABLED_KEY,
 } from '../../lib/settings'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 export const metadata = {
   GET: { requireAuth: true, requireFeatures: ['catalog.products.view'] },
@@ -75,7 +78,7 @@ async function GET(req: Request) {
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[catalog/settings.GET] Unexpected error', err)
+    logger.error('catalog.settings.GET Unexpected error', { err })
     return NextResponse.json({ error: 'Internal server error' }, { status: 500 })
   }
 }
@@ -130,7 +133,7 @@ async function PUT(req: Request) {
     if (err instanceof z.ZodError) {
       return NextResponse.json({ error: 'Invalid request', details: err.issues }, { status: 400 })
     }
-    console.error('[catalog/settings.PUT] Unexpected error', err)
+    logger.error('catalog.settings.PUT Unexpected error', { err })
     return NextResponse.json({ error: 'Internal server error' }, { status: 500 })
   }
 }

package/src/modules/catalog/backend/catalog/products/[id]/page.tsx

@@ -138,6 +138,9 @@ import {
   DialogTitle,
 } from "@open-mercato/ui/primitives/dialog";
 import { SendObjectMessageDialog } from "@open-mercato/ui/backend/messages/SendObjectMessageDialog.tsx";
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 type ProductResponse = {
   items?: Array<Record<string, unknown>>;
@@ -509,12 +512,12 @@ export default function EditCatalogProductPage({
           }
           setVariantMediaGroups(groups);
         } catch (err) {
-          console.error("catalog.variants.media.fetch failed", err);
+          logger.error('catalog.variants.media.fetch failed', { err });
           setVariantMediaGroups([]);
         }
       })();
     } catch (err) {
-      console.error("catalog.variants.fetch failed", err);
+      logger.error('catalog.variants.fetch failed', { err });
       setVariants([]);
       setVariantMediaGroups([]);
     }
@@ -540,7 +543,7 @@ export default function EditCatalogProductPage({
           thumbnailUrl: item.thumbnailUrl ?? undefined,
         }));
       } catch (err) {
-        console.error("attachments.fetch failed", err);
+        logger.error('attachments.fetch failed', { err });
         return [];
       }
     },
@@ -566,7 +569,7 @@ export default function EditCatalogProductPage({
             .filter((item): item is TaxRateSummary => item !== null),
         );
       } catch (err) {
-        console.error("sales.tax-rates.fetch failed", err);
+        logger.error('sales.tax-rates.fetch failed', { err });
         setTaxRates([]);
       }
     };
@@ -776,7 +779,7 @@ export default function EditCatalogProductPage({
         }
         await loadVariants(productId!);
       } catch (err) {
-        console.error("catalog.products.edit.load failed", err);
+        logger.error('catalog.products.edit.load failed', { err });
         if (!cancelled) {
           const message =
             err instanceof Error && err.message
@@ -814,7 +817,7 @@ export default function EditCatalogProductPage({
           setPriceKinds(summaries);
         }
       } catch (err) {
-        console.error("catalog.price-kinds.fetch failed", err);
+        logger.error('catalog.price-kinds.fetch failed', { err });
         if (!cancelled) {
           setPriceKinds([]);
         }
@@ -1347,7 +1350,7 @@ export default function EditCatalogProductPage({
             );
           }
         } catch (err) {
-          console.error("catalog.products.edit.offers.delete", err);
+          logger.error('catalog.products.edit.offers.delete', { err });
           throw createCrudFormError(
             t(
               "catalog.products.edit.offers.deleteError",
@@ -1889,7 +1892,7 @@ function ProductOptionsSection({ values, setValue }: ProductFormGroupProps) {
         setSchemaTemplates([]);
       }
     } catch (err) {
-      console.error("catalog.option-schemas.list failed", err);
+      logger.error('catalog.option-schemas.list failed', { err });
       setSchemaTemplates([]);
     } finally {
       setSchemaLoading(false);
@@ -1917,7 +1920,7 @@ function ProductOptionsSection({ values, setValue }: ProductFormGroupProps) {
         void loadSchemas();
       } catch (err) {
         if (surfaceRecordConflict(err, t)) { void loadSchemas(); return; }
-        console.error("catalog.option-schemas.delete failed", err);
+        logger.error('catalog.option-schemas.delete failed', { err });
       }
     },
     [loadSchemas, schemaTemplates, t],
@@ -2310,7 +2313,7 @@ function ProductVariantsSection({
         onVariantDeleted(variant.id);
       } catch (err) {
         if (surfaceRecordConflict(err, t)) return;
-        console.error("catalog.products.edit.variants.delete", err);
+        logger.error('catalog.products.edit.variants.delete', { err });
         flash(
           t("catalog.variants.form.deleteError", "Failed to delete variant."),
           "error",
@@ -2359,7 +2362,7 @@ function ProductVariantsSection({
       );
       if (onVariantsReload) await onVariantsReload();
     } catch (err) {
-      console.error("catalog.products.edit.variantList.generate", err);
+      logger.error('catalog.products.edit.variantList.generate', { err });
       flash(
         t(
           "catalog.products.edit.variantList.generateError",
@@ -3324,7 +3327,7 @@ function SaveSchemaDialog({
       await onSubmit(name);
       onOpenChange(false);
     } catch (err) {
-      console.error("schema.save.failed", err);
+      logger.error('schema.save.failed', { err });
     } finally {
       setSaving(false);
     }

package/src/modules/catalog/backend/catalog/products/[productId]/variants/[variantId]/page.tsx

@@ -47,6 +47,9 @@ import type { ProductMediaItem } from '@open-mercato/core/modules/catalog/compon
 import { buildAttachmentImageUrl, slugifyAttachmentFileName } from '@open-mercato/core/modules/attachments/lib/imageUrls'
 import { fetchOptionSchemaTemplate } from '../../../optionSchemaClient'
 import CreateVariantPage from '../create/page'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 type VariantResponse = {
   items?: Array<Record<string, unknown>>
@@ -159,7 +162,7 @@ export default function EditVariantPage({ params }: { params?: { productId?: str
         const items = Array.isArray(payload.items) ? payload.items : []
         setPriceKinds(items.map((item) => normalizePriceKindSummary(item)).filter((item): item is PriceKindSummary => !!item))
       } catch (err) {
-        console.error('catalog.price-kinds.fetch failed', err)
+        logger.error('catalog.price-kinds.fetch failed', { err })
         setPriceKinds([])
       }
     }
@@ -181,7 +184,7 @@ export default function EditVariantPage({ params }: { params?: { productId?: str
             .filter((item): item is TaxRateSummary => item !== null),
         )
       } catch (err) {
-        console.error('sales.tax-rates.fetch failed', err)
+        logger.error('sales.tax-rates.fetch failed', { err })
         setTaxRates([])
       }
     }
@@ -385,7 +388,7 @@ export default function EditVariantPage({ params }: { params?: { productId?: str
           })
         }
       } catch (err) {
-        console.error('catalog.variants.load.failed', err)
+        logger.error('catalog.variants.load.failed', { err })
         if (!cancelled) {
           const message = err instanceof Error && err.message ? err.message : t('catalog.variants.form.errors.load', 'Failed to load variant.')
           setError(message)
@@ -740,7 +743,7 @@ async function fetchVariantAttachments(variantId: string): Promise<ProductMediaI
     if (!res.ok) return []
     return Array.isArray(res.result?.items) ? res.result?.items ?? [] : []
   } catch (err) {
-    console.error('catalog.variants.attachments.load', err)
+    logger.error('catalog.variants.attachments.load', { err })
     return []
   }
 }
@@ -765,7 +768,7 @@ async function loadVariantPrices(variantId: string, priceKinds: PriceKindSummary
       page += 1
     }
   } catch (err) {
-    console.error('catalog.variants.prices.load', err)
+    logger.error('catalog.variants.prices.load', { err })
   }
   return drafts
 }
@@ -819,7 +822,7 @@ async function syncVariantPricesUpdate({
             () => deleteCrud('catalog/prices', existingId),
           )
         } catch (err) {
-          console.error('catalog.prices.delete', err)
+          logger.error('catalog.prices.delete', { err })
         }
       }
       continue

package/src/modules/catalog/backend/catalog/products/[productId]/variants/create/page.tsx

@@ -38,6 +38,9 @@ import {
 import type { ProductMediaItem } from '@open-mercato/core/modules/catalog/components/products/ProductMediaManager'
 import { buildAttachmentImageUrl, slugifyAttachmentFileName } from '@open-mercato/core/modules/attachments/lib/imageUrls'
 import { fetchOptionSchemaTemplate } from '../../../optionSchemaClient'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 type ProductResponse = {
   items?: Array<{
@@ -83,7 +86,7 @@ export default function CreateVariantPage({ params }: { params?: { productId?: s
         const items = Array.isArray(payload.items) ? payload.items : []
         setPriceKinds(items.map((item) => normalizePriceKindSummary(item)).filter((item): item is PriceKindSummary => !!item))
       } catch (err) {
-        console.error('catalog.price-kinds.fetch failed', err)
+        logger.error('catalog.price-kinds.fetch failed', { err })
         setPriceKinds([])
       }
     }
@@ -121,7 +124,7 @@ export default function CreateVariantPage({ params }: { params?: { productId?: s
           }),
         )
       } catch (err) {
-        console.error('sales.tax-rates.fetch failed', err)
+        logger.error('sales.tax-rates.fetch failed', { err })
         setTaxRates([])
       }
     }
@@ -191,7 +194,7 @@ export default function CreateVariantPage({ params }: { params?: { productId?: s
           setInitialValues(base)
         }
       } catch (err) {
-        console.error('catalog.variants.loadProduct failed', err)
+        logger.error('catalog.variants.loadProduct failed', { err })
         if (!cancelled) {
           const message = err instanceof Error && err.message ? err.message : t('catalog.variants.form.errors.load', 'Failed to load product context.')
           setError(message)

package/src/modules/catalog/backend/catalog/products/create/page.tsx

@@ -96,6 +96,9 @@ import {
   normalizeProductConversionInputs,
   type ProductUnitConversionInput,
 } from "@open-mercato/core/modules/catalog/components/products/productFormUtils";
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('catalog')
 
 const productFormTypedSchema =
   productFormSchema as unknown as ZodType<ProductFormValues>;
@@ -268,7 +271,7 @@ export default function CreateCatalogProductPage() {
             .filter((item): item is PriceKindSummary => item !== null),
         );
       } catch (err) {
-        console.error("catalog.price-kinds.fetch failed", err);
+        logger.error('catalog.price-kinds.fetch failed', { err });
         setPriceKinds([]);
       }
     };
@@ -317,7 +320,7 @@ export default function CreateCatalogProductPage() {
           }),
         );
       } catch (err) {
-        console.error("sales.tax-rates.fetch failed", err);
+        logger.error('sales.tax-rates.fetch failed', { err });
         setTaxRates([]);
       }
     };
@@ -775,10 +778,7 @@ export default function CreateCatalogProductPage() {
                   { fallback: null },
                 );
                 if (!transfer.ok) {
-                  console.error(
-                    "attachments.transfer.failed",
-                    transfer.result?.error,
-                  );
+                  logger.error("attachments.transfer.failed", { err: transfer.result?.error });
                 }
               }