@open-mercato/core 0.4.2-canary-c71ef83148 → 0.4.2-canary-f821f89ef6

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

@@ -35,14 +35,14 @@
   "configs.systemStatus.actions.purgeCacheUnavailable": "Usługa pamięci podręcznej jest niedostępna.",
   "configs.config.nav.cache": "Pamięć podręczna",
   "configs.cache.title": "Podgląd pamięci podręcznej",
-  "configs.cache.description": "Przeglądaj zapisane odpowiedzi i czyść segmenty w razie potrzeby.",
+  "configs.cache.description": "Przeglądaj zapisane odpowiedzi CRUD i czyść segmenty w razie potrzeby.",
   "configs.cache.loading": "Ładowanie statystyk pamięci podręcznej…",
   "configs.cache.loadError": "Nie udało się wczytać statystyk pamięci podręcznej.",
   "configs.cache.retry": "Spróbuj ponownie",
   "configs.cache.refresh": "Odśwież",
   "configs.cache.generatedAt": "Statystyki z {{timestamp}}",
   "configs.cache.totalEntries": "{{count}} wpisów w pamięci podręcznej",
-  "configs.cache.empty": "Brak zapisanych odpowiedzi dla tego tenanta.",
+  "configs.cache.empty": "Brak zapisanych odpowiedzi CRUD dla tego tenanta.",
   "configs.cache.purgeAll": "Wyczyść całą pamięć",
   "configs.cache.purgeAllLoading": "Czyszczenie…",
   "configs.cache.purgeAllConfirm": "Wyczyścić wszystkie wpisy pamięci podręcznej dla tego tenanta?",
@@ -64,8 +64,6 @@
   "configs.systemStatus.categories.profilingDescription": "Flagi sterujące generowaniem danych z profilowania zapytań i żądań.",
   "configs.systemStatus.categories.logging": "Logowanie",
   "configs.systemStatus.categories.loggingDescription": "Dostosuj poziom logowania i zapisywanie zapytań SQL na potrzeby diagnostyki.",
-  "configs.systemStatus.categories.security": "Bezpieczeństwo",
-  "configs.systemStatus.categories.securityDescription": "Wymagania polityki haseł stosowane przy logowaniu i tworzeniu użytkowników.",
   "configs.systemStatus.categories.caching": "Buforowanie",
   "configs.systemStatus.categories.cachingDescription": "Mechanizmy cache oraz kontrola czasu życia odpowiedzi backendu.",
   "configs.systemStatus.categories.queryIndex": "Indeks zapytań",
@@ -86,14 +84,6 @@
   "configs.systemStatus.variables.logVerbosity.description": "Nadpisuje poziom szczegółowości logów, np. debug lub trace.",
   "configs.systemStatus.variables.logLevel.label": "Poziom logowania",
   "configs.systemStatus.variables.logLevel.description": "Domyślny poziom logowania używany, gdy nie ustawiono szczegółowości.",
-  "configs.systemStatus.variables.passwordMinLength.label": "Minimalna długość hasła",
-  "configs.systemStatus.variables.passwordMinLength.description": "Minimalna liczba znaków wymagana w haśle.",
-  "configs.systemStatus.variables.passwordRequireDigit.label": "Hasło wymaga cyfry",
-  "configs.systemStatus.variables.passwordRequireDigit.description": "Wymagaj co najmniej jednej cyfry.",
-  "configs.systemStatus.variables.passwordRequireUppercase.label": "Hasło wymaga wielkiej litery",
-  "configs.systemStatus.variables.passwordRequireUppercase.description": "Wymagaj co najmniej jednej wielkiej litery.",
-  "configs.systemStatus.variables.passwordRequireSpecial.label": "Hasło wymaga znaku specjalnego",
-  "configs.systemStatus.variables.passwordRequireSpecial.description": "Wymagaj co najmniej jednego znaku specjalnego.",
   "configs.systemStatus.variables.enableCrudApiCache.label": "Cache API CRUD",
   "configs.systemStatus.variables.enableCrudApiCache.description": "Włącza warstwę buforowania odpowiedzi API CRUD.",
   "configs.systemStatus.variables.cacheStrategy.label": "Strategia cache",

package/src/modules/configs/lib/system-status.ts

@@ -19,14 +19,7 @@ type SystemStatusVariableDefinition = {
   defaultValue: string | null
 }
 
-const CATEGORY_ORDER: SystemStatusCategoryKey[] = [
-  'profiling',
-  'logging',
-  'security',
-  'caching',
-  'query_index',
-  'entities',
-]
+const CATEGORY_ORDER: SystemStatusCategoryKey[] = ['profiling', 'logging', 'caching', 'query_index', 'entities']
 
 const CATEGORY_METADATA: Record<
   SystemStatusCategoryKey,
@@ -40,10 +33,6 @@ const CATEGORY_METADATA: Record<
     labelKey: 'configs.systemStatus.categories.logging',
     descriptionKey: 'configs.systemStatus.categories.loggingDescription',
   },
-  security: {
-    labelKey: 'configs.systemStatus.categories.security',
-    descriptionKey: 'configs.systemStatus.categories.securityDescription',
-  },
   caching: {
     labelKey: 'configs.systemStatus.categories.caching',
     descriptionKey: 'configs.systemStatus.categories.cachingDescription',
@@ -124,42 +113,6 @@ export const SYSTEM_STATUS_VARIABLES: SystemStatusVariableDefinition[] = [
     docUrl: `${SYSTEM_STATUS_DOC_BASE}#log_level`,
     defaultValue: '',
   },
-  {
-    key: 'OM_PASSWORD_MIN_LENGTH',
-    category: 'security',
-    kind: 'string',
-    labelKey: 'configs.systemStatus.variables.passwordMinLength.label',
-    descriptionKey: 'configs.systemStatus.variables.passwordMinLength.description',
-    docUrl: `${SYSTEM_STATUS_DOC_BASE}#om_password_min_length`,
-    defaultValue: '6',
-  },
-  {
-    key: 'OM_PASSWORD_REQUIRE_DIGIT',
-    category: 'security',
-    kind: 'boolean',
-    labelKey: 'configs.systemStatus.variables.passwordRequireDigit.label',
-    descriptionKey: 'configs.systemStatus.variables.passwordRequireDigit.description',
-    docUrl: `${SYSTEM_STATUS_DOC_BASE}#om_password_require_digit`,
-    defaultValue: 'true',
-  },
-  {
-    key: 'OM_PASSWORD_REQUIRE_UPPERCASE',
-    category: 'security',
-    kind: 'boolean',
-    labelKey: 'configs.systemStatus.variables.passwordRequireUppercase.label',
-    descriptionKey: 'configs.systemStatus.variables.passwordRequireUppercase.description',
-    docUrl: `${SYSTEM_STATUS_DOC_BASE}#om_password_require_uppercase`,
-    defaultValue: 'true',
-  },
-  {
-    key: 'OM_PASSWORD_REQUIRE_SPECIAL',
-    category: 'security',
-    kind: 'boolean',
-    labelKey: 'configs.systemStatus.variables.passwordRequireSpecial.label',
-    descriptionKey: 'configs.systemStatus.variables.passwordRequireSpecial.description',
-    docUrl: `${SYSTEM_STATUS_DOC_BASE}#om_password_require_special`,
-    defaultValue: 'true',
-  },
   {
     key: 'ENABLE_CRUD_API_CACHE',
     category: 'caching',

package/src/modules/configs/lib/system-status.types.ts

@@ -1,7 +1,6 @@
 export type SystemStatusCategoryKey =
   | 'profiling'
   | 'logging'
-  | 'security'
   | 'caching'
   | 'query_index'
   | 'entities'

package/src/modules/configs/lib/upgrade-actions.ts

@@ -11,7 +11,6 @@ import { seedExampleWorkflows } from '@open-mercato/core/modules/workflows/lib/s
 import { seedPlannerAvailabilityRuleSetDefaults, seedPlannerUnavailabilityReasons } from '@open-mercato/core/modules/planner/lib/seeds'
 import { seedResourcesAddressTypes, seedResourcesCapacityUnits, seedResourcesResourceExamples } from '@open-mercato/core/modules/resources/lib/seeds'
 import { seedStaffTeamExamples } from '@open-mercato/core/modules/staff/lib/seeds'
-import { appendWidgetsToRoles, resolveAnalyticsWidgetIds } from '@open-mercato/core/modules/dashboards/lib/role-widgets'
 import { collectCrudCacheStats, purgeCrudCacheSegment } from '@open-mercato/shared/lib/crud/cache-stats'
 import { isCrudCacheEnabled, resolveCrudCache } from '@open-mercato/shared/lib/crud/cache'
 import * as semver from 'semver'
@@ -371,23 +370,6 @@ export const upgradeActions: UpgradeActionDefinition[] = [
       await reindexModules(em, ['planner', 'staff', 'resources'], { tenantId, organizationId, vectorService })
     },
   },
-  {
-    id: 'configs.upgrades.dashboards.analytics_widgets',
-    version: '0.4.2',
-    messageKey: 'upgrades.v042.message',
-    ctaKey: 'upgrades.v042.cta',
-    successKey: 'upgrades.v042.success',
-    loadingKey: 'upgrades.v042.loading',
-    async run({ em, tenantId, organizationId }) {
-      const widgetIds = await resolveAnalyticsWidgetIds()
-      await appendWidgetsToRoles(em, {
-        tenantId,
-        organizationId,
-        roleNames: ['admin', 'employee'],
-        widgetIds,
-      })
-    },
-  },
 ]
 
 export function actionsUpToVersion(version: string): UpgradeActionDefinition[] {

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

@@ -256,14 +256,12 @@ export default function CurrenciesPage() {
             <RowActions
               items={[
                 {
-                  id: 'edit',
                   label: t('common.edit'),
                   href: `/backend/currencies/${row.id}`,
                 },
                 ...(!row.isBase
                   ? [
                       {
-                        id: 'set-base',
                         label: t('currencies.list.actions.setBase'),
                         onSelect: () => handleSetBase(row),
                       },
@@ -272,7 +270,6 @@ export default function CurrenciesPage() {
                 ...(!row.isBase
                   ? [
                       {
-                        id: 'delete',
                         label: t('common.delete'),
                         destructive: true,
                         onSelect: () => handleDelete(row),

package/src/modules/currencies/backend/exchange-rates/page.tsx

@@ -279,12 +279,10 @@ export default function ExchangeRatesPage() {
             <RowActions
               items={[
                 {
-                  id: 'edit',
                   label: t('common.edit'),
                   href: `/backend/exchange-rates/${row.id}`,
                 },
                 {
-                  id: 'delete',
                   label: t('common.delete'),
                   destructive: true,
                   onSelect: () => handleDelete(row),

package/src/modules/customers/backend/customers/companies/page.tsx

@@ -597,17 +597,14 @@ export default function CustomersCompaniesPage() {
             <RowActions
               items={[
                 {
-                  id: 'view',
                   label: t('customers.companies.list.actions.view'),
                   onSelect: () => { router.push(`/backend/customers/companies/${row.id}`) },
                 },
                 {
-                  id: 'open-new-tab',
                   label: t('customers.companies.list.actions.openInNewTab'),
                   onSelect: () => window.open(`/backend/customers/companies/${row.id}`, '_blank', 'noopener'),
                 },
                 {
-                  id: 'delete',
                   label: t('customers.companies.list.actions.delete'),
                   destructive: true,
                   onSelect: () => handleDelete(row),

package/src/modules/customers/backend/customers/deals/page.tsx

@@ -889,12 +889,10 @@ export default function CustomersDealsPage() {
               <RowActions
                 items={[
                   {
-                    id: 'edit',
                     label: t('customers.deals.list.actions.edit', 'Edit'),
                     onSelect: () => { router.push(`/backend/customers/deals/${row.id}`) },
                   },
                   {
-                    id: 'open-new-tab',
                     label: t('customers.deals.list.actions.openInNewTab', 'Open in new tab'),
                     onSelect: () => {
                       if (typeof window !== 'undefined') {
@@ -903,7 +901,6 @@ export default function CustomersDealsPage() {
                     },
                   },
                   {
-                    id: 'delete',
                     label: isDeleting
                       ? t('customers.deals.list.actions.deleting', 'Deleting…')
                       : t('customers.deals.list.actions.delete', 'Delete'),

package/src/modules/customers/backend/customers/people/page.tsx

@@ -614,17 +614,14 @@ export default function CustomersPeoplePage() {
             <RowActions
               items={[
                 {
-                  id: 'view',
                   label: t('customers.people.list.actions.view'),
                   onSelect: () => { router.push(`/backend/customers/people/${row.id}`) },
                 },
                 {
-                  id: 'open-new-tab',
                   label: t('customers.people.list.actions.openInNewTab'),
                   onSelect: () => window.open(`/backend/customers/people/${row.id}`, '_blank', 'noopener'),
                 },
                 {
-                  id: 'delete',
                   label: t('customers.people.list.actions.delete'),
                   destructive: true,
                   onSelect: () => handleDelete(row),

package/src/modules/customers/commands/deals.ts

@@ -35,9 +35,6 @@ import { CrudHttpError } from '@open-mercato/shared/lib/crud/errors'
 import type { CrudIndexerConfig } from '@open-mercato/shared/lib/crud/types'
 import { E } from '#generated/entities.ids.generated'
 import { findWithDecryption } from '@open-mercato/shared/lib/encryption/find'
-import { resolveNotificationService } from '../../notifications/lib/notificationService'
-import { buildNotificationFromType } from '../../notifications/lib/notificationBuilder'
-import { notificationTypes } from '../notifications'
 
 const DEAL_ENTITY_ID = 'customers:customer_deal'
 const dealCrudIndexer: CrudIndexerConfig<CustomerDeal> = {
@@ -284,8 +281,6 @@ const updateDealCommand: CommandHandler<DealUpdateInput, { dealId: string }> = {
     ensureTenantScope(ctx, record.tenantId)
     ensureOrganizationScope(ctx, record.organizationId)
 
-    const previousStatus = record.status
-
     if (parsed.title !== undefined) record.title = parsed.title
     if (parsed.description !== undefined) record.description = parsed.description ?? null
     if (parsed.status !== undefined) record.status = parsed.status ?? record.status
@@ -324,40 +319,6 @@ const updateDealCommand: CommandHandler<DealUpdateInput, { dealId: string }> = {
       indexer: dealCrudIndexer,
     })
 
-    // Send notifications for deal won/lost status changes
-    const newStatus = record.status
-    const normalizedStatus = newStatus === 'win' ? 'won' : newStatus === 'loose' ? 'lost' : newStatus
-    if (previousStatus !== newStatus && (normalizedStatus === 'won' || normalizedStatus === 'lost') && record.ownerUserId) {
-      try {
-        const notificationService = resolveNotificationService(ctx.container)
-        const notificationType = normalizedStatus === 'won' ? 'customers.deal.won' : 'customers.deal.lost'
-        const typeDef = notificationTypes.find((type) => type.type === notificationType)
-        if (typeDef) {
-          const valueDisplay = record.valueAmount && record.valueCurrency
-            ? `${record.valueCurrency} ${record.valueAmount}`
-            : ''
-
-          const notificationInput = buildNotificationFromType(typeDef, {
-            recipientUserId: record.ownerUserId,
-            bodyVariables: {
-              dealTitle: record.title,
-              dealValue: valueDisplay,
-            },
-            sourceEntityType: 'customers:customer_deal',
-            sourceEntityId: record.id,
-            linkHref: `/backend/customers/deals/${record.id}`,
-          })
-
-          await notificationService.create(notificationInput, {
-            tenantId: record.tenantId,
-            organizationId: record.organizationId,
-          })
-        }
-      } catch {
-        // Notification creation is non-critical, don't fail the command
-      }
-    }
-
     return { dealId: record.id }
   },
   buildLog: async ({ snapshots, ctx }) => {

package/src/modules/customers/components/CustomerTodosTable.tsx

@@ -253,7 +253,6 @@ export function CustomerTodosTable(): React.JSX.Element {
           <RowActions
             items={[
               {
-                id: 'open-customer',
                 label: t('customers.workPlan.customerTodos.table.actions.openCustomer'),
                 href: customerLink,
               },

package/src/modules/customers/events.ts

@@ -0,0 +1,63 @@
+import { createModuleEvents } from '@open-mercato/shared/modules/events'
+
+/**
+ * Customers Module Events
+ *
+ * Declares all events that can be emitted by the customers module.
+ */
+const events = [
+  // People
+  { id: 'customers.people.created', label: 'Customer (Person) Created', entity: 'people', category: 'crud' },
+  { id: 'customers.people.updated', label: 'Customer (Person) Updated', entity: 'people', category: 'crud' },
+  { id: 'customers.people.deleted', label: 'Customer (Person) Deleted', entity: 'people', category: 'crud' },
+
+  // Companies
+  { id: 'customers.companies.created', label: 'Customer (Company) Created', entity: 'companies', category: 'crud' },
+  { id: 'customers.companies.updated', label: 'Customer (Company) Updated', entity: 'companies', category: 'crud' },
+  { id: 'customers.companies.deleted', label: 'Customer (Company) Deleted', entity: 'companies', category: 'crud' },
+
+  // Deals
+  { id: 'customers.deals.created', label: 'Deal Created', entity: 'deals', category: 'crud' },
+  { id: 'customers.deals.updated', label: 'Deal Updated', entity: 'deals', category: 'crud' },
+  { id: 'customers.deals.deleted', label: 'Deal Deleted', entity: 'deals', category: 'crud' },
+
+  // Comments
+  { id: 'customers.comments.created', label: 'Comment Created', entity: 'comments', category: 'crud' },
+  { id: 'customers.comments.updated', label: 'Comment Updated', entity: 'comments', category: 'crud' },
+  { id: 'customers.comments.deleted', label: 'Comment Deleted', entity: 'comments', category: 'crud' },
+
+  // Addresses
+  { id: 'customers.addresses.created', label: 'Address Created', entity: 'addresses', category: 'crud' },
+  { id: 'customers.addresses.updated', label: 'Address Updated', entity: 'addresses', category: 'crud' },
+  { id: 'customers.addresses.deleted', label: 'Address Deleted', entity: 'addresses', category: 'crud' },
+
+  // Activities
+  { id: 'customers.activities.created', label: 'Activity Created', entity: 'activities', category: 'crud' },
+  { id: 'customers.activities.updated', label: 'Activity Updated', entity: 'activities', category: 'crud' },
+  { id: 'customers.activities.deleted', label: 'Activity Deleted', entity: 'activities', category: 'crud' },
+
+  // Tags
+  { id: 'customers.tags.created', label: 'Tag Created', entity: 'tags', category: 'crud' },
+  { id: 'customers.tags.updated', label: 'Tag Updated', entity: 'tags', category: 'crud' },
+  { id: 'customers.tags.deleted', label: 'Tag Deleted', entity: 'tags', category: 'crud' },
+  { id: 'customers.tags.assigned', label: 'Tag Assigned', entity: 'tags', category: 'crud' },
+  { id: 'customers.tags.removed', label: 'Tag Removed', entity: 'tags', category: 'crud' },
+
+  // Todos
+  { id: 'customers.todos.created', label: 'Todo Created', entity: 'todos', category: 'crud' },
+  { id: 'customers.todos.updated', label: 'Todo Updated', entity: 'todos', category: 'crud' },
+  { id: 'customers.todos.deleted', label: 'Todo Deleted', entity: 'todos', category: 'crud' },
+] as const
+
+export const eventsConfig = createModuleEvents({
+  moduleId: 'customers',
+  events,
+})
+
+/** Type-safe event emitter for customers module */
+export const emitCustomersEvent = eventsConfig.emit
+
+/** Event IDs that can be emitted by the customers module */
+export type CustomersEventId = typeof events[number]['id']
+
+export default eventsConfig

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

@@ -943,9 +943,5 @@
   "customers.workPlan.customerTodos.table.state.empty": "No customer tasks yet.",
   "customers.workPlan.customerTodos.table.error.load": "Failed to load customer tasks.",
   "customers.workPlan.customerTodos.table.export.view": "Exports the current list with filters and visible columns.",
-  "customers.workPlan.customerTodos.table.export.full": "Exports every linked task field, including hidden attributes.",
-  "customers.notifications.deal.won.title": "Deal Won",
-  "customers.notifications.deal.won.body": "{dealTitle} has been marked as won{dealValue, select, other { ({dealValue})}}",
-  "customers.notifications.deal.lost.title": "Deal Lost",
-  "customers.notifications.deal.lost.body": "{dealTitle} has been marked as lost"
+  "customers.workPlan.customerTodos.table.export.full": "Exports every linked task field, including hidden attributes."
 }

package/src/modules/customers/widgets/dashboard/customer-todos/widget.ts

@@ -1,6 +1,6 @@
-import { lazyDashboardWidget, type DashboardWidgetModule } from '@open-mercato/shared/modules/dashboard/widgets'
+import type { DashboardWidgetModule } from '@open-mercato/shared/modules/dashboard/widgets'
+import CustomerTodosWidget from './widget.client'
 import { DEFAULT_SETTINGS, hydrateCustomerTodoSettings, type CustomerTodoWidgetSettings } from './config'
-const CustomerTodosWidget = lazyDashboardWidget(() => import('./widget.client'))
 
 const widget: DashboardWidgetModule<CustomerTodoWidgetSettings> = {
   metadata: {

package/src/modules/customers/widgets/dashboard/new-customers/widget.ts

@@ -1,6 +1,6 @@
-import { lazyDashboardWidget, type DashboardWidgetModule } from '@open-mercato/shared/modules/dashboard/widgets'
+import type { DashboardWidgetModule } from '@open-mercato/shared/modules/dashboard/widgets'
+import CustomerNewCustomersWidget from './widget.client'
 import { DEFAULT_SETTINGS, hydrateNewCustomersSettings, type CustomerNewCustomersSettings } from './config'
-const CustomerNewCustomersWidget = lazyDashboardWidget(() => import('./widget.client'))
 
 const widget: DashboardWidgetModule<CustomerNewCustomersSettings> = {
   metadata: {

package/src/modules/customers/widgets/dashboard/new-deals/widget.ts

@@ -1,6 +1,6 @@
-import { lazyDashboardWidget, type DashboardWidgetModule } from '@open-mercato/shared/modules/dashboard/widgets'
+import type { DashboardWidgetModule } from '@open-mercato/shared/modules/dashboard/widgets'
+import CustomerNewDealsWidget from './widget.client'
 import { DEFAULT_SETTINGS, hydrateNewDealsSettings, type CustomerNewDealsSettings } from './config'
-const CustomerNewDealsWidget = lazyDashboardWidget(() => import('./widget.client'))
 
 const widget: DashboardWidgetModule<CustomerNewDealsSettings> = {
   metadata: {

package/src/modules/customers/widgets/dashboard/next-interactions/widget.ts

@@ -1,10 +1,10 @@
-import { lazyDashboardWidget, type DashboardWidgetModule } from '@open-mercato/shared/modules/dashboard/widgets'
+import type { DashboardWidgetModule } from '@open-mercato/shared/modules/dashboard/widgets'
+import CustomerNextInteractionsWidget from './widget.client'
 import {
   DEFAULT_SETTINGS,
   hydrateNextInteractionsSettings,
   type CustomerNextInteractionsSettings,
 } from './config'
-const CustomerNextInteractionsWidget = lazyDashboardWidget(() => import('./widget.client'))
 
 const widget: DashboardWidgetModule<CustomerNextInteractionsSettings> = {
   metadata: {

package/src/modules/dashboards/cli.ts

@@ -4,7 +4,6 @@ import type { EntityManager } from '@mikro-orm/postgresql'
 import { DashboardRoleWidgets } from '@open-mercato/core/modules/dashboards/data/entities'
 import { Role } from '@open-mercato/core/modules/auth/data/entities'
 import { loadAllWidgets } from '@open-mercato/core/modules/dashboards/lib/widgets'
-import { appendWidgetsToRoles, resolveAnalyticsWidgetIds } from '@open-mercato/core/modules/dashboards/lib/role-widgets'
 import { seedAnalyticsData } from './seed/analytics'
 
 type Args = Record<string, string>
@@ -42,25 +41,15 @@ export async function seedDashboardDefaultsForTenant(
   const widgetMap = new Map(widgets.map((widget) => [widget.metadata.id, widget]))
   const resolvedWidgetIds = widgetIds && widgetIds.length
     ? widgetIds.filter((id) => widgetMap.has(id))
-    : null
-  const defaultWidgetIds = widgets
-    .filter((widget) => widget.metadata.defaultEnabled)
-    .map((widget) => widget.metadata.id)
-  const allWidgetIds = widgets.map((widget) => widget.metadata.id)
+    : widgets.filter((widget) => widget.metadata.defaultEnabled).map((widget) => widget.metadata.id)
 
-  if (resolvedWidgetIds && resolvedWidgetIds.length === 0) {
+  if (!resolvedWidgetIds.length) {
     log('No widgets resolved for dashboard seeding.')
     return false
   }
 
   await em.transactional(async (tem) => {
     for (const roleName of roleNames) {
-      const isAdminRole = roleName === 'admin' || roleName === 'superadmin'
-      const roleWidgetIds = resolvedWidgetIds ?? (isAdminRole ? allWidgetIds : defaultWidgetIds)
-      if (!roleWidgetIds.length) {
-        log(`No widgets resolved for role "${roleName}".`)
-        continue
-      }
       const role = await tem.findOne(Role, { name: roleName })
       if (!role) {
         log(`Skipping role "${roleName}" (not found)`)
@@ -73,7 +62,7 @@ export async function seedDashboardDefaultsForTenant(
         deletedAt: null,
       })
       if (existing) {
-        existing.widgetIdsJson = roleWidgetIds
+        existing.widgetIdsJson = resolvedWidgetIds
         tem.persist(existing)
         log(`Updated dashboard widgets for role "${roleName}"`)
       } else {
@@ -81,7 +70,7 @@ export async function seedDashboardDefaultsForTenant(
           roleId: String(role.id),
           tenantId,
           organizationId,
-          widgetIdsJson: roleWidgetIds,
+          widgetIdsJson: resolvedWidgetIds,
           createdAt: new Date(),
           updatedAt: null,
           deletedAt: null,
@@ -131,45 +120,6 @@ const seedDefaults: ModuleCli = {
   },
 }
 
-const enableAnalyticsWidgets: ModuleCli = {
-  command: 'enable-analytics-widgets',
-  async run(rest) {
-    const args = parseArgs(rest)
-    const tenantId = args.tenant || args.tenantId || null
-    const organizationId = args.organization || args.organizationId || args.org || null
-    const roleCsv = args.roles || 'admin,employee'
-    if (!tenantId) {
-      console.error('Usage: mercato dashboards enable-analytics-widgets --tenant <tenantId> [--org <orgId>] [--roles admin,employee]')
-      return
-    }
-
-    const roleNames = roleCsv
-      .split(',')
-      .map((name) => name.trim())
-      .filter(Boolean)
-
-    if (!roleNames.length) {
-      console.log('No roles provided, nothing to update.')
-      return
-    }
-
-    const { resolve } = await createRequestContainer()
-    const em = resolve('em') as EntityManager
-    const widgetIds = await resolveAnalyticsWidgetIds()
-
-    const updated = await appendWidgetsToRoles(em, {
-      tenantId,
-      organizationId,
-      roleNames,
-      widgetIds,
-    })
-
-    if (!updated) {
-      console.log('No dashboard role widgets updated.')
-    }
-  },
-}
-
 const seedAnalytics: ModuleCli = {
   command: 'seed-analytics',
   async run(rest) {
@@ -332,4 +282,4 @@ const debugAnalytics: ModuleCli = {
   },
 }
 
-export default [seedDefaults, enableAnalyticsWidgets, seedAnalytics, debugAnalytics]
+export default [seedDefaults, seedAnalytics, debugAnalytics]

package/src/modules/dashboards/components/WidgetVisibilityEditor.tsx

@@ -44,13 +44,9 @@ type UserProps = BaseProps & {
 
 type WidgetVisibilityEditorProps = RoleProps | UserProps
 
-export type WidgetVisibilityEditorHandle = {
-  save: () => Promise<void>
-}
-
 const EMPTY: string[] = []
 
-export const WidgetVisibilityEditor = React.forwardRef<WidgetVisibilityEditorHandle, WidgetVisibilityEditorProps>(function WidgetVisibilityEditor(props, ref) {
+export function WidgetVisibilityEditor(props: WidgetVisibilityEditorProps) {
   const t = useT()
   const { kind, targetId, tenantId, organizationId } = props
   const [catalog, setCatalog] = React.useState<WidgetCatalogItem[]>([])
@@ -64,15 +60,6 @@ export const WidgetVisibilityEditor = React.forwardRef<WidgetVisibilityEditorHan
   const [originalMode, setOriginalMode] = React.useState<'inherit' | 'override'>('inherit')
   const [effective, setEffective] = React.useState<string[]>(EMPTY)
 
-  const dirty = React.useMemo(() => {
-    if (kind === 'user') {
-      if (mode !== originalMode) return true
-      if (mode === 'override') return selected.join('|') !== original.join('|')
-      return false
-    }
-    return selected.join('|') !== original.join('|')
-  }, [kind, mode, original, originalMode, selected])
-
   const loadCatalog = React.useCallback(async () => {
     const data = await readApiResultOrThrow<{ items?: unknown[] }>(
       '/api/dashboards/widgets/catalog',
@@ -162,9 +149,6 @@ export const WidgetVisibilityEditor = React.forwardRef<WidgetVisibilityEditorHan
   }, [original, originalMode])
 
   const save = React.useCallback(async () => {
-    if (loading) return
-    if (error && catalog.length === 0) return
-    if (!dirty) return
     setSaving(true)
     setError(null)
     try {
@@ -218,9 +202,16 @@ export const WidgetVisibilityEditor = React.forwardRef<WidgetVisibilityEditorHan
     } finally {
       setSaving(false)
     }
-  }, [catalog.length, dirty, error, kind, loading, mode, organizationId, selected, t, targetId, tenantId])
+  }, [kind, mode, organizationId, selected, t, targetId, tenantId])
 
-  React.useImperativeHandle(ref, () => ({ save }), [save])
+  const dirty = React.useMemo(() => {
+    if (kind === 'user') {
+      if (mode !== originalMode) return true
+      if (mode === 'override') return selected.join('|') !== original.join('|')
+      return false
+    }
+    return selected.join('|') !== original.join('|')
+  }, [kind, mode, original, originalMode, selected])
 
   if (loading) {
     return (
@@ -306,6 +297,4 @@ export const WidgetVisibilityEditor = React.forwardRef<WidgetVisibilityEditorHan
       </div>
     </div>
   )
-})
-
-WidgetVisibilityEditor.displayName = 'WidgetVisibilityEditor'
+}