@open-mercato/core 0.6.6-develop.6465.1.019f0fb26f → 0.6.6-develop.6471.1.9ab5bdd79a

package/src/modules/dictionaries/api/[dictionaryId]/entries/route.ts

@@ -31,6 +31,9 @@ import {
   resolveDictionaryEntrySortMode,
   sortDictionaryEntries,
 } from '@open-mercato/core/modules/dictionaries/lib/entrySort'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'entries-api' })
 
 const paramsSchema = z.object({ dictionaryId: z.string().uuid() })
 
@@ -157,7 +160,7 @@ export async function GET(req: Request, ctx: { params?: { dictionaryId?: string
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[dictionaries/:id/entries.GET] Unexpected error', err)
+    logger.error('Failed to load dictionary entries', { err })
     return NextResponse.json({ error: 'Failed to load dictionary entries' }, { status: 500 })
   }
 }
@@ -249,7 +252,7 @@ export async function POST(req: Request, ctx: { params?: { dictionaryId?: string
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[dictionaries/:id/entries.POST] Unexpected error', err)
+    logger.error('Failed to create dictionary entry', { err })
     return NextResponse.json({ error: 'Failed to create dictionary entry' }, { status: 500 })
   }
 }

package/src/modules/dictionaries/api/[dictionaryId]/entries/set-default/route.ts

@@ -25,6 +25,9 @@ import {
   setDefaultEntryRequestSchema,
 } from '../../../openapi'
 import { resolveDictionaryActorId } from '../../../context'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'entries-api' })
 
 const paramsSchema = z.object({ dictionaryId: z.string().uuid() })
 
@@ -127,7 +130,7 @@ export async function POST(req: Request, ctx: { params?: { dictionaryId?: string
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[dictionaries/:id/entries/set-default.POST] Unexpected error', err)
+    logger.error('Failed to set default entry', { err })
     return NextResponse.json({ error: 'Failed to set default entry' }, { status: 500 })
   }
 }

package/src/modules/dictionaries/api/[dictionaryId]/route.ts

@@ -22,6 +22,9 @@ import {
   upsertDictionarySchema,
 } from '../openapi'
 import { dictionaryKeySchema } from '@open-mercato/core/modules/dictionaries/data/validators'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'api' })
 
 const paramsSchema = z.object({ dictionaryId: z.string().uuid() })
 // System dictionaries use namespaced keys (e.g. `sales.deal_loss_reason`,
@@ -103,7 +106,7 @@ export async function GET(req: Request, ctx: { params?: { dictionaryId?: string
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[dictionaries/:id.GET] Unexpected error', err)
+    logger.error('Failed to load dictionary', { err })
     return NextResponse.json({ error: 'Failed to load dictionary' }, { status: 500 })
   }
 }
@@ -225,7 +228,7 @@ export async function PATCH(req: Request, ctx: { params?: { dictionaryId?: strin
     if (err instanceof z.ZodError) {
       return NextResponse.json({ error: err.issues[0]?.message ?? 'Validation failed' }, { status: 400 })
     }
-    console.error('[dictionaries/:id.PATCH] Unexpected error', err)
+    logger.error('Failed to update dictionary', { err })
     return NextResponse.json({ error: 'Failed to update dictionary' }, { status: 500 })
   }
 }
@@ -286,7 +289,7 @@ export async function DELETE(req: Request, ctx: { params?: { dictionaryId?: stri
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[dictionaries/:id.DELETE] Unexpected error', err)
+    logger.error('Failed to delete dictionary', { err })
     return NextResponse.json({ error: 'Failed to delete dictionary' }, { status: 500 })
   }
 }

package/src/modules/dictionaries/api/context.ts

@@ -7,6 +7,9 @@ import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
 import type { EntityManager } from '@mikro-orm/postgresql'
 import type { AwilixContainer } from 'awilix'
 import type { CommandRuntimeContext } from '@open-mercato/shared/lib/commands'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'context' })
 
 export type DictionariesRouteContext = {
   container: AwilixContainer
@@ -88,7 +91,7 @@ export async function resolveDictionariesRouteContext(req: Request): Promise<Dic
       for (const id of candidateIds) readableOrganizationIds.add(id)
     }
   } catch (err) {
-    console.warn('[dictionaries.resolveContext] Failed to resolve readable organizations', err)
+    logger.warn('Failed to resolve readable organizations', { err })
     for (const id of candidateIds) readableOrganizationIds.add(id)
   }
 

package/src/modules/dictionaries/api/route.ts

@@ -20,6 +20,9 @@ import {
   upsertDictionarySchema,
 } from './openapi'
 import { parseBooleanToken } from '@open-mercato/shared/lib/boolean'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'api' })
 
 export const metadata = {
   GET: { requireAuth: true, requireFeatures: ['dictionaries.view'] },
@@ -67,7 +70,7 @@ export async function GET(req: Request) {
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[dictionaries.GET] Unexpected error', err)
+    logger.error('Failed to load dictionaries', { err })
     return NextResponse.json({ error: 'Failed to load dictionaries' }, { status: 500 })
   }
 }
@@ -153,7 +156,7 @@ export async function POST(req: Request) {
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('[dictionaries.POST] Unexpected error', err)
+    logger.error('Failed to create dictionary', { err })
     return NextResponse.json({ error: 'Failed to create dictionary' }, { status: 500 })
   }
 }

package/src/modules/dictionaries/components/DictionariesManager.tsx

@@ -28,6 +28,9 @@ import {
   dictionaryEntrySortModes,
   type DictionaryEntrySortMode,
 } from '../lib/entrySort'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'DictionariesManager' })
 
 export type DictionarySummary = {
   id: string
@@ -140,7 +143,7 @@ export function DictionariesManager() {
         return filtered.length ? filtered[0].id : null
       })
     } catch (err) {
-      console.error('Failed to load dictionaries', err)
+      logger.error('Failed to load dictionaries', { err })
       flash(t('dictionaries.config.error.load', 'Failed to load dictionaries.'), 'error')
     } finally {
       setLoading(false)
@@ -300,7 +303,7 @@ export function DictionariesManager() {
       if (surfaceRecordConflict(err, t)) {
         return
       }
-      console.error('Failed to save dictionary', err)
+      logger.error('Failed to save dictionary', { err })
       flash(t('dictionaries.config.error.save', 'Failed to save dictionary.'), 'error')
     } finally {
       setSubmitting(false)
@@ -357,7 +360,7 @@ export function DictionariesManager() {
         if (surfaceRecordConflict(err, t)) {
           return
         }
-        console.error('Failed to delete dictionary', err)
+        logger.error('Failed to delete dictionary', { err })
         flash(t('dictionaries.config.error.delete', 'Failed to delete dictionary.'), 'error')
       } finally {
         setDeleting(null)

package/src/modules/dictionaries/components/DictionaryEntriesEditor.tsx

@@ -33,6 +33,9 @@ import {
   DialogDescription,
 } from '@open-mercato/ui/primitives/dialog'
 import { TranslationManager } from '@open-mercato/core/modules/translations/components/TranslationManager'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'DictionaryEntriesEditor' })
 
 type Entry = DictionaryEntryRecord
 
@@ -224,7 +227,7 @@ export function DictionaryEntriesEditor({ dictionaryId, dictionaryName, readOnly
       if (surfaceRecordConflict(err, t)) {
         return
       }
-      console.error('Failed to save dictionary entry', err)
+      logger.error('Failed to save dictionary entry', { err })
       flash(t('dictionaries.config.entries.error.save', 'Failed to save dictionary entry.'), 'error')
     } finally {
       setIsSaving(false)
@@ -281,7 +284,7 @@ export function DictionaryEntriesEditor({ dictionaryId, dictionaryName, readOnly
         if (surfaceRecordConflict(err, t)) {
           return
         }
-        console.error('Failed to delete dictionary entry', err)
+        logger.error('Failed to delete dictionary entry', { err })
         flash(t('dictionaries.config.entries.error.delete', 'Failed to delete dictionary entry.'), 'error')
       } finally {
         setIsDeleting(false)

package/src/modules/dictionaries/components/DictionaryEntrySelect.tsx

@@ -27,6 +27,9 @@ import { flash } from '@open-mercato/ui/backend/FlashMessages'
 import { buildHrefWithReturnTo } from '@open-mercato/shared/lib/navigation/returnTo'
 import { DictionaryValue, renderDictionaryColor, renderDictionaryIcon } from './dictionaryAppearance'
 import { AppearanceSelector, type AppearanceSelectorLabels, useAppearanceState } from './AppearanceSelector'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'DictionaryEntrySelect' })
 
 const DEFAULT_APPEARANCE_LABELS: AppearanceSelectorLabels = {
   colorLabel: 'Color',
@@ -131,7 +134,7 @@ export function DictionaryEntrySelect({
       const items = await fetchOptions()
       setOptions(sortOptions === 'none' ? items : items.slice().sort((a, b) => a.label.localeCompare(b.label, undefined, { sensitivity: 'base' })))
     } catch (err) {
-      console.error('DictionaryEntrySelect.fetchOptions failed', err)
+      logger.error('Failed to fetch options', { err })
       flash(labels.errorLoad, 'error')
       setOptions([])
     } finally {
@@ -226,7 +229,7 @@ export function DictionaryEntrySelect({
         flash(labels.successCreateLabel, 'success')
       }
     } catch (err) {
-      console.error('DictionaryEntrySelect.createOption failed', err)
+      logger.error('Failed to create option', { err })
       flash(labels.errorSave, 'error')
     } finally {
       setSaving(false)

package/src/modules/dictionaries/components/DictionarySelectControl.tsx

@@ -11,6 +11,9 @@ import {
   ensureDictionaryEntries,
   invalidateDictionaryEntries,
 } from './hooks/useDictionaryEntries'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'DictionarySelectControl' })
 
 type DictionarySelectControlProps = {
   dictionaryId: string
@@ -66,7 +69,7 @@ export function DictionarySelectControl({
         }
         setInlineCreateEnabled(true)
       } catch (err) {
-        console.warn('DictionarySelectControl.inlineCreate check failed', err)
+        logger.warn('Inline create check failed', { err })
         if (!cancelled) {
           setInlineCreateEnabled(allowInlineCreate)
         }

package/src/modules/dictionaries/fields/dictionary.tsx

@@ -15,6 +15,9 @@ import {
 } from '@open-mercato/ui/primitives/select'
 import { DictionarySelectControl } from '../components/DictionarySelectControl'
 import { useDictionaryEntries } from '../components/hooks/useDictionaryEntries'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('dictionaries').child({ component: 'dictionary-field' })
 
 type DictionaryFieldDefinition = {
   dictionaryId?: string
@@ -119,7 +122,7 @@ function DictionaryFieldDefEditor({ def, onChange }: { def: { configJson?: Dicti
         }
       } catch (err) {
         if (!cancelled) {
-          console.error('Failed to load dictionaries list', err)
+          logger.error('Failed to load dictionaries list', { err })
           setError(errorLoadLabel)
         }
       } finally {

package/src/modules/directory/api/organization-branding/route.ts

@@ -18,6 +18,9 @@ import { Organization } from '@open-mercato/core/modules/directory/data/entities
 import { organizationUpdateSchema } from '@open-mercato/core/modules/directory/data/validators'
 import { resolveOrganizationScopeForRequest } from '@open-mercato/core/modules/directory/utils/organizationScope'
 import '@open-mercato/core/modules/directory/commands/organizations'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('directory').child({ component: 'organization-branding' })
 
 export const metadata = {
   GET: { requireAuth: true, requireFeatures: ['directory.organizations.view'] },
@@ -254,7 +257,7 @@ export async function PUT(req: Request) {
     if (isCrudHttpError(err)) {
       return NextResponse.json(err.body, { status: err.status })
     }
-    console.error('directory.organization-branding.update failed', err)
+    logger.error('Organization branding update failed', { err })
     return NextResponse.json(
       { error: resolved.translate('directory.branding.errors.save', 'Failed to update organization branding.') },
       { status: 400 },

package/src/modules/directory/api/organization-switcher/route.ts

@@ -16,6 +16,9 @@ import { Tenant } from '@open-mercato/core/modules/directory/data/entities'
 import type { EntityManager } from '@mikro-orm/postgresql'
 import type { RbacService } from '@open-mercato/core/modules/auth/services/rbacService'
 import type { FilterQuery } from '@mikro-orm/core'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('directory').child({ component: 'organization-switcher' })
 
 type OrganizationMenuNode = {
   id: string
@@ -220,7 +223,7 @@ export async function GET(req: NextRequest) {
 
     return NextResponse.json(response)
   } catch (err) {
-    console.error('Failed to build organization switcher payload', err)
+    logger.error('Failed to build organization switcher payload', { err })
     return NextResponse.json({ items: [], selectedId: null, canManage: false, tenantId: null, tenants: [], isSuperAdmin: false }, { status: 500 })
   }
 }

package/src/modules/directory/commands/tenants.ts

@@ -17,6 +17,9 @@ import { resolveTranslations } from '@open-mercato/shared/lib/i18n/server'
 import type { CrudEventsConfig, CrudIndexerConfig } from '@open-mercato/shared/lib/crud/types'
 import { isTenantDataEncryptionEnabled } from '@open-mercato/shared/lib/encryption/toggles'
 import { makeCreateRedo } from '@open-mercato/shared/lib/commands/redo'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('directory').child({ component: 'tenants' })
 
 export const tenantCrudEvents: CrudEventsConfig = {
   module: 'directory',
@@ -70,12 +73,12 @@ const createTenantCommand: CommandHandler<TenantPayload, Tenant> = {
         const kms = ctx.container.resolve('kmsService') as { createTenantDek?: (id: string) => Promise<unknown>; isHealthy?: () => boolean }
         if (kms?.isHealthy?.()) {
           await kms?.createTenantDek?.(String(tenant.id))
-          console.info('🔑 [encryption][tenant] created tenant DEK', { tenantId: String(tenant.id) })
+          logger.info('Created tenant DEK', { tenantId: String(tenant.id) })
         } else {
-          console.warn('⚠️ [encryption][tenant] kms not healthy, skipping tenant DEK provisioning', { tenantId: String(tenant.id) })
+          logger.warn('KMS not healthy, skipping tenant DEK provisioning', { tenantId: String(tenant.id) })
         }
       } catch (err) {
-        console.warn('⚠️ [encryption] Failed to provision tenant key', err)
+        logger.warn('Failed to provision tenant key', { err })
       }
     }
 

package/src/modules/directory/utils/organizationScope.ts

@@ -7,6 +7,9 @@ import type { RbacService } from '@open-mercato/core/modules/auth/services/rbacS
 import type { AuthContext } from '@open-mercato/shared/lib/auth/server'
 import type { CacheStrategy } from '@open-mercato/cache'
 import { parseSelectedOrganizationCookie, parseSelectedTenantCookie } from './scopeCookies'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('directory').child({ component: 'org-scope-cache' })
 
 export { parseSelectedOrganizationCookie, parseSelectedTenantCookie }
 
@@ -99,7 +102,7 @@ export async function invalidateOrganizationScopeCacheForUser(
   try {
     await cache.deleteByTags([buildOrgScopeUserCacheTag(userId)])
   } catch (err) {
-    console.warn('[org-scope:cache] invalidate user failed', err)
+    logger.warn('Cache invalidate user failed', { err })
   }
 }
 
@@ -112,7 +115,7 @@ export async function invalidateOrganizationScopeCacheForTenant(
   try {
     await cache.deleteByTags([buildOrgScopeTenantCacheTag(tenantId)])
   } catch (err) {
-    console.warn('[org-scope:cache] invalidate tenant failed', err)
+    logger.warn('Cache invalidate tenant failed', { err })
   }
 }
 
@@ -453,7 +456,7 @@ export async function resolveOrganizationScopeForRequest({
         const cached = await cache.get(cacheKey)
         if (isValidCachedScope(cached)) return cached
       } catch (err) {
-        console.warn('[org-scope:cache] read failed', err)
+        logger.warn('Cache read failed', { err })
       }
     }
 
@@ -472,7 +475,7 @@ export async function resolveOrganizationScopeForRequest({
           tags: buildOrgScopeCacheTags({ userId, effectiveTenantId }),
         })
       } catch (err) {
-        console.warn('[org-scope:cache] write failed', err)
+        logger.warn('Cache write failed', { err })
       }
     }
 

package/src/modules/entities/api/definitions.cache.ts

@@ -1,4 +1,7 @@
 import type { CacheStrategy } from '@open-mercato/cache'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('entities').child({ component: 'definitions-cache' })
 
 const CACHE_PREFIX = 'entities:definitions'
 export const ENTITY_DEFINITIONS_CACHE_TTL_MS = 5 * 60 * 1000
@@ -54,6 +57,6 @@ export async function invalidateDefinitionsCache(
   try {
     await cache.deleteByTags(targets)
   } catch (err) {
-    console.warn('[entities.definitions.cache] Failed to invalidate cache', err)
+    logger.warn('Failed to invalidate cache', { err })
   }
 }

package/src/modules/entities/api/definitions.ts

@@ -34,6 +34,9 @@ import {
   selectVisibleDefinitionWinner,
 } from '../lib/definition-scope'
 import { resolveEntityDefinitionsVersion } from '../lib/definitions-version'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('entities').child({ component: 'definitions' })
 
 /**
  * Validate defaultValue against the field kind. Returns an error message string
@@ -82,7 +85,7 @@ async function validateDefaultValueByKind(
           if (!entry) return `defaultValue "${value}" does not match any entry in the configured dictionary`
         } catch (err) {
           // If the dictionaries module is not available, skip entry validation
-          console.debug('[entities.definitions] dictionary validation skipped — module not available', err)
+          logger.debug('Dictionary validation skipped — module not available', { err })
         }
       }
       return null
@@ -99,7 +102,7 @@ async function validateDefaultValueByKind(
         if (!currency) return `defaultValue "${value}" does not match any available currency`
       } catch (err) {
         // If the currencies module is not available, skip currency validation
-        console.debug('[entities.definitions] currency validation skipped — module not available', err)
+        logger.debug('Currency validation skipped — module not available', { err })
       }
       return null
     }
@@ -265,7 +268,7 @@ export async function GET(req: Request) {
         createOnly: true,
       })
     } catch (err) {
-      console.warn('[entities.definitions] Failed to synchronize module-backed definitions', {
+      logger.warn('Failed to synchronize module-backed definitions', {
         tenantId,
         entityIds,
         err,
@@ -286,7 +289,7 @@ export async function GET(req: Request) {
         return NextResponse.json(cached)
       }
     } catch (err) {
-      console.warn('[entities.definitions.cache] Failed to read cache', err)
+      logger.warn('Failed to read cache', { err })
     }
   }
 
@@ -471,7 +474,7 @@ export async function GET(req: Request) {
         tags,
       })
     } catch (err) {
-      console.warn('[entities.definitions.cache] Failed to store cache entry', err)
+      logger.warn('Failed to store cache entry', { err })
     }
   }
 

package/src/modules/entities/api/records.ts

@@ -15,6 +15,9 @@ import { enforceCommandOptimisticLockWithGuards } from '@open-mercato/shared/lib
 import { isCrudHttpError } from '@open-mercato/shared/lib/crud/errors'
 import { getModules } from '@open-mercato/shared/lib/i18n/server'
 import { assertEntityAclForRequest } from '../lib/entityAcl'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('entities').child({ component: 'records' })
 
 let declaredCustomEntityIds: Set<string> | null = null
 function isDeclaredCustomEntity(entityId: string): boolean {
@@ -348,7 +351,7 @@ export async function GET(req: Request) {
     return NextResponse.json(payload)
   } catch (e) {
     if (isCrudHttpError(e)) return NextResponse.json(e.body, { status: e.status })
-    try { console.error('[entities.records.GET] Error', e) } catch {}
+    logger.error('Records GET failed', { err: e })
     return NextResponse.json({ error: 'Internal server error' }, { status: 500 })
   }
 }
@@ -434,7 +437,7 @@ export async function POST(req: Request) {
     return NextResponse.json({ ok: true, item: { entityId, recordId: id } })
   } catch (e) {
     if (isCrudHttpError(e)) return NextResponse.json(e.body, { status: e.status })
-    try { console.error('[entities.records.POST] Error', e) } catch {}
+    logger.error('Records POST failed', { err: e })
     return NextResponse.json({ error: 'Internal server error' }, { status: 500 })
   }
 }

package/src/modules/entities/components/EncryptionManager.tsx

@@ -22,6 +22,9 @@ import { Plus, Save, Trash2 } from 'lucide-react'
 import { useOrganizationScopeVersion } from '@open-mercato/shared/lib/frontend/useOrganizationScope'
 import { useT } from '@open-mercato/shared/lib/i18n/context'
 import { getEntityFields } from '#generated/entity-fields-registry'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('entities').child({ component: 'EncryptionManager' })
 
 type EntityOption = { entityId: string; label?: string; source?: string }
 type FieldOption = { value: string; label: string }
@@ -172,7 +175,7 @@ export function EncryptionManager() {
     // Use static registry instead of dynamic import for Turbopack compatibility
     const mod = getEntityFields(entitySlug)
     if (!mod) {
-      console.warn('[encryption] No fields found for entity', entitySlug)
+      logger.warn('No fields found for entity', { entitySlug })
       setBaseFieldOptions((prev) => (prev.length ? [] : prev))
       return
     }

package/src/modules/entities/lib/makeActivityRoute.ts

@@ -11,6 +11,9 @@ import {
   createPagedListResponseSchema as createSharedPagedListResponseSchema,
   defaultOkResponseSchema as sharedDefaultOkResponseSchema,
 } from '@open-mercato/shared/lib/openapi/crud'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('entities').child({ component: 'activity-route' })
 
 // eslint-disable-next-line @typescript-eslint/no-explicit-any -- MikroORM entity class constructor
 type EntityClass = new (...args: any[]) => unknown
@@ -271,7 +274,7 @@ export function makeActivityRoute(config: ActivityRouteConfig) {
             if (!('author_email' in record)) record.author_email = meta.email
           })
         } catch (err) {
-          console.warn(`${logPrefix} failed to enrich author metadata`, err)
+          logger.warn('Failed to enrich author metadata', { logPrefix, err })
         }
       },
     },

package/src/modules/feature_toggles/api/overrides/route.ts

@@ -24,6 +24,9 @@ import {
   changeOverrideStateResponseSchema,
 } from '../openapi'
 import type { OpenApiRouteDoc, OpenApiMethodDoc } from '@open-mercato/shared/lib/openapi'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('feature_toggles').child({ component: 'overrides' })
 
 export async function GET(req: Request) {
   const { auth, ctx, scope } = await buildContext(req)
@@ -188,7 +191,7 @@ export async function PUT(req: Request) {
     if (message === 'INVALID_STATE' || message === 'INVALID_TOGGLE_ID' || message === 'INVALID_TENANT_ID') {
       return NextResponse.json({ error: message }, { status: 400 })
     }
-    console.error('feature_toggles.overrides.changeState failed', error)
+    logger.error('Override state change failed', { err: error })
     return NextResponse.json({ error: 'Failed to update override' }, { status: 500 })
   }
 }

package/src/modules/feature_toggles/lib/feature-flag-check.ts

@@ -1,6 +1,9 @@
 import { FeatureToggle, FeatureToggleOverride } from "../data/entities"
 import { EntityManager } from "@mikro-orm/core"
 import { CacheService, runWithCacheTenant } from "@open-mercato/cache"
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('feature_toggles').child({ component: 'flag-check' })
 
 type ToggleValueType = "boolean" | "string" | "number" | "json"
 
@@ -145,7 +148,7 @@ export class FeatureTogglesService {
     const resolution = await this.resolveToggle(identifier, ctx.tenantId)
 
     if (resolution.source === "missing") {
-      console.warn(`[feature_toggles] Toggle "${identifier}" not found (missing).`)
+      logger.warn('Toggle not found', { identifier })
       return {
         ok: false,
         error: {
@@ -163,10 +166,7 @@ export class FeatureTogglesService {
 
 
     if (resolution.valueType !== ctx.valueType) {
-      console.error(
-        `[feature_toggles] Toggle "${identifier}" has type "${resolution.valueType}" but "${ctx.valueType}" was requested.`,
-        { resolution }
-      )
+      logger.error('Toggle type mismatch', { identifier, expectedType: ctx.valueType, actualType: resolution.valueType, resolution })
       return {
         ok: false,
         error: {
@@ -188,10 +188,7 @@ export class FeatureTogglesService {
       (ctx.valueType === "json")
 
     if (!isValueValid) {
-      console.error(
-        `[feature_toggles] Toggle "${identifier}" has invalid value for type "${resolution.valueType}".`,
-        { resolution }
-      )
+      logger.error('Toggle has invalid value for type', { identifier, valueType: resolution.valueType, resolution })
       return {
         ok: false,
         error: {

package/src/modules/inbox_ops/api/emails/[id]/reprocess/route.ts

@@ -10,6 +10,9 @@ import {
   extractPathSegment,
   UnauthorizedError,
 } from '../../../routeHelpers'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('inbox_ops').child({ component: 'email-reprocess' })
 
 export const metadata = {
   POST: { requireAuth: true, requireFeatures: ['inbox_ops.proposals.manage'] },
@@ -74,7 +77,7 @@ export async function POST(req: Request) {
         subject: email.subject,
       })
     } catch (eventError) {
-      console.error('[inbox_ops:email:reprocess] Failed to emit events:', eventError)
+      logger.error('Failed to emit events', { err: eventError })
     }
 
     return NextResponse.json({ ok: true, ...retiredCounts })
@@ -86,7 +89,7 @@ export async function POST(req: Request) {
       return NextResponse.json({ error: err.message }, { status: 409 })
     }
 
-    console.error('[inbox_ops:email:reprocess] Error:', err)
+    logger.error('Reprocess failed', { err })
     return NextResponse.json({ error: 'Failed to reprocess email' }, { status: 500 })
   }
 }

package/src/modules/inbox_ops/api/emails/[id]/route.ts

@@ -11,6 +11,9 @@ import {
   extractPathSegment,
   UnauthorizedError,
 } from '../../routeHelpers'
+import { createLogger } from '@open-mercato/shared/lib/logger'
+
+const logger = createLogger('inbox_ops').child({ component: 'emails' })
 
 export const metadata = {
   GET: { requireAuth: true, requireFeatures: ['inbox_ops.log.view'] },
@@ -50,7 +53,7 @@ export async function GET(req: Request) {
     if (err instanceof UnauthorizedError) {
       return NextResponse.json({ error: 'Unauthorized' }, { status: 401 })
     }
-    console.error('[inbox_ops:emails:detail] Error:', err)
+    logger.error('Failed to load email detail', { err })
     return NextResponse.json({ error: 'Failed to load email' }, { status: 500 })
   }
 }
@@ -114,7 +117,7 @@ export async function DELETE(req: Request) {
     if (err instanceof UnauthorizedError) {
       return NextResponse.json({ error: 'Unauthorized' }, { status: 401 })
     }
-    console.error('[inbox_ops:emails:delete] Error:', err)
+    logger.error('Failed to delete email', { err })
     return NextResponse.json({ error: 'Failed to delete email' }, { status: 500 })
   }
 }