@open-mercato/shared 0.6.7-develop.6669.1.40b669666b → 0.6.7-develop.6670.1.4efa7961c6

package/dist/lib/frontend/organizationEvents.js

@@ -4,6 +4,7 @@ let currentScope = {
   tenantId: null
 };
 let currentVersion = 0;
+let hasEmitted = false;
 function getCurrentOrganizationScope() {
   return { ...currentScope };
 }
@@ -13,11 +14,15 @@ function getCurrentOrganizationScopeVersion() {
 function emitOrganizationScopeChanged(detail) {
   if (typeof window === "undefined" || typeof CustomEvent === "undefined") return;
   const hasChanged = currentScope.organizationId !== detail.organizationId || currentScope.tenantId !== detail.tenantId;
+  const isFirstEmit = !hasEmitted;
   currentScope = { ...detail };
+  hasEmitted = true;
   if (hasChanged) {
     currentVersion++;
   }
-  window.dispatchEvent(new CustomEvent(ORGANIZATION_SCOPE_CHANGED_EVENT, { detail }));
+  if (hasChanged || isFirstEmit) {
+    window.dispatchEvent(new CustomEvent(ORGANIZATION_SCOPE_CHANGED_EVENT, { detail }));
+  }
 }
 function subscribeOrganizationScopeChanged(handler) {
   if (typeof window === "undefined") return () => {

package/dist/lib/frontend/organizationEvents.js.map

@@ -1,7 +1,7 @@
 {
   "version": 3,
   "sources": ["../../../src/lib/frontend/organizationEvents.ts"],
-  "sourcesContent": ["export const ORGANIZATION_SCOPE_CHANGED_EVENT = 'om:organization-scope-changed'\n\nexport type OrganizationScopeChangedDetail = {\n  organizationId: string | null\n  tenantId: string | null\n}\n\n// Module-level state to track current scope and version\nlet currentScope: OrganizationScopeChangedDetail = { \n  organizationId: null, \n  tenantId: null \n}\nlet currentVersion = 0\n\nexport function getCurrentOrganizationScope(): OrganizationScopeChangedDetail {\n  return { ...currentScope }\n}\n\nexport function getCurrentOrganizationScopeVersion(): number {\n  return currentVersion\n}\n\nexport function emitOrganizationScopeChanged(detail: OrganizationScopeChangedDetail): void {\n  if (typeof window === 'undefined' || typeof CustomEvent === 'undefined') return\n  \n  // Detect actual changes\n  const hasChanged = \n    currentScope.organizationId !== detail.organizationId ||\n    currentScope.tenantId !== detail.tenantId\n  \n  // Update module-level state\n  currentScope = { ...detail }\n  \n  // Increment version only if actual change detected\n  if (hasChanged) {\n    currentVersion++\n  }\n  \n  // Emit event\n  window.dispatchEvent(new CustomEvent<OrganizationScopeChangedDetail>(ORGANIZATION_SCOPE_CHANGED_EVENT, { detail }))\n}\n\nexport function subscribeOrganizationScopeChanged(\n  handler: (detail: OrganizationScopeChangedDetail) => void\n): () => void {\n  if (typeof window === 'undefined') return () => {}\n  const listener = (event: Event) => {\n    const detail = (event as CustomEvent<OrganizationScopeChangedDetail>).detail ?? { organizationId: null, tenantId: null }\n    handler(detail)\n  }\n  window.addEventListener(ORGANIZATION_SCOPE_CHANGED_EVENT, listener as EventListener)\n  return () => {\n    window.removeEventListener(ORGANIZATION_SCOPE_CHANGED_EVENT, listener as EventListener)\n  }\n}\n"],
-  "mappings": "AAAO,MAAM,mCAAmC;AAQhD,IAAI,eAA+C;AAAA,EACjD,gBAAgB;AAAA,EAChB,UAAU;AACZ;AACA,IAAI,iBAAiB;AAEd,SAAS,8BAA8D;AAC5E,SAAO,EAAE,GAAG,aAAa;AAC3B;AAEO,SAAS,qCAA6C;AAC3D,SAAO;AACT;AAEO,SAAS,6BAA6B,QAA8C;AACzF,MAAI,OAAO,WAAW,eAAe,OAAO,gBAAgB,YAAa;AAGzE,QAAM,aACJ,aAAa,mBAAmB,OAAO,kBACvC,aAAa,aAAa,OAAO;AAGnC,iBAAe,EAAE,GAAG,OAAO;AAG3B,MAAI,YAAY;AACd;AAAA,EACF;AAGA,SAAO,cAAc,IAAI,YAA4C,kCAAkC,EAAE,OAAO,CAAC,CAAC;AACpH;AAEO,SAAS,kCACd,SACY;AACZ,MAAI,OAAO,WAAW,YAAa,QAAO,MAAM;AAAA,EAAC;AACjD,QAAM,WAAW,CAAC,UAAiB;AACjC,UAAM,SAAU,MAAsD,UAAU,EAAE,gBAAgB,MAAM,UAAU,KAAK;AACvH,YAAQ,MAAM;AAAA,EAChB;AACA,SAAO,iBAAiB,kCAAkC,QAAyB;AACnF,SAAO,MAAM;AACX,WAAO,oBAAoB,kCAAkC,QAAyB;AAAA,EACxF;AACF;",
+  "sourcesContent": ["export const ORGANIZATION_SCOPE_CHANGED_EVENT = 'om:organization-scope-changed'\n\nexport type OrganizationScopeChangedDetail = {\n  organizationId: string | null\n  tenantId: string | null\n}\n\n// Module-level state to track current scope and version\nlet currentScope: OrganizationScopeChangedDetail = { \n  organizationId: null, \n  tenantId: null \n}\nlet currentVersion = 0\nlet hasEmitted = false\n\nexport function getCurrentOrganizationScope(): OrganizationScopeChangedDetail {\n  return { ...currentScope }\n}\n\nexport function getCurrentOrganizationScopeVersion(): number {\n  return currentVersion\n}\n\nexport function emitOrganizationScopeChanged(detail: OrganizationScopeChangedDetail): void {\n  if (typeof window === 'undefined' || typeof CustomEvent === 'undefined') return\n  \n  // Detect actual changes\n  const hasChanged = \n    currentScope.organizationId !== detail.organizationId ||\n    currentScope.tenantId !== detail.tenantId\n  // Guard so the very first emission always syncs subscribers, even when the\n  // initial scope matches the default all-orgs `{ null, null }` state.\n  const isFirstEmit = !hasEmitted\n  \n  // Update module-level state\n  currentScope = { ...detail }\n  hasEmitted = true\n  \n  // Increment version only if actual change detected\n  if (hasChanged) {\n    currentVersion++\n  }\n  \n  // Only dispatch on real scope changes (or the first emission). Suppressing\n  // no-op re-emits avoids spurious DataTable refreshes, chrome refetches, and\n  // scope-hook churn when navigating between pages with the same scope.\n  if (hasChanged || isFirstEmit) {\n    window.dispatchEvent(new CustomEvent<OrganizationScopeChangedDetail>(ORGANIZATION_SCOPE_CHANGED_EVENT, { detail }))\n  }\n}\n\nexport function subscribeOrganizationScopeChanged(\n  handler: (detail: OrganizationScopeChangedDetail) => void\n): () => void {\n  if (typeof window === 'undefined') return () => {}\n  const listener = (event: Event) => {\n    const detail = (event as CustomEvent<OrganizationScopeChangedDetail>).detail ?? { organizationId: null, tenantId: null }\n    handler(detail)\n  }\n  window.addEventListener(ORGANIZATION_SCOPE_CHANGED_EVENT, listener as EventListener)\n  return () => {\n    window.removeEventListener(ORGANIZATION_SCOPE_CHANGED_EVENT, listener as EventListener)\n  }\n}\n"],
+  "mappings": "AAAO,MAAM,mCAAmC;AAQhD,IAAI,eAA+C;AAAA,EACjD,gBAAgB;AAAA,EAChB,UAAU;AACZ;AACA,IAAI,iBAAiB;AACrB,IAAI,aAAa;AAEV,SAAS,8BAA8D;AAC5E,SAAO,EAAE,GAAG,aAAa;AAC3B;AAEO,SAAS,qCAA6C;AAC3D,SAAO;AACT;AAEO,SAAS,6BAA6B,QAA8C;AACzF,MAAI,OAAO,WAAW,eAAe,OAAO,gBAAgB,YAAa;AAGzE,QAAM,aACJ,aAAa,mBAAmB,OAAO,kBACvC,aAAa,aAAa,OAAO;AAGnC,QAAM,cAAc,CAAC;AAGrB,iBAAe,EAAE,GAAG,OAAO;AAC3B,eAAa;AAGb,MAAI,YAAY;AACd;AAAA,EACF;AAKA,MAAI,cAAc,aAAa;AAC7B,WAAO,cAAc,IAAI,YAA4C,kCAAkC,EAAE,OAAO,CAAC,CAAC;AAAA,EACpH;AACF;AAEO,SAAS,kCACd,SACY;AACZ,MAAI,OAAO,WAAW,YAAa,QAAO,MAAM;AAAA,EAAC;AACjD,QAAM,WAAW,CAAC,UAAiB;AACjC,UAAM,SAAU,MAAsD,UAAU,EAAE,gBAAgB,MAAM,UAAU,KAAK;AACvH,YAAQ,MAAM;AAAA,EAChB;AACA,SAAO,iBAAiB,kCAAkC,QAAyB;AACnF,SAAO,MAAM;AACX,WAAO,oBAAoB,kCAAkC,QAAyB;AAAA,EACxF;AACF;",
   "names": []
 }

package/dist/lib/version.js

@@ -1,4 +1,4 @@
-const APP_VERSION = "0.6.7-develop.6669.1.40b669666b";
+const APP_VERSION = "0.6.7-develop.6670.1.4efa7961c6";
 const appVersion = APP_VERSION;
 export {
   APP_VERSION,

package/dist/lib/version.js.map

@@ -1,7 +1,7 @@
 {
   "version": 3,
   "sources": ["../../src/lib/version.ts"],
-  "sourcesContent": ["// Build-time generated version\nexport const APP_VERSION = '0.6.7-develop.6669.1.40b669666b'\nexport const appVersion = APP_VERSION\n"],
+  "sourcesContent": ["// Build-time generated version\nexport const APP_VERSION = '0.6.7-develop.6670.1.4efa7961c6'\nexport const appVersion = APP_VERSION\n"],
   "mappings": "AACO,MAAM,cAAc;AACpB,MAAM,aAAa;",
   "names": []
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-mercato/shared",
-  "version": "0.6.7-develop.6669.1.40b669666b",
+  "version": "0.6.7-develop.6670.1.4efa7961c6",
   "license": "MIT",
   "type": "module",
   "main": "./dist/index.js",
@@ -97,7 +97,7 @@
     "@mikro-orm/core": "^7.1.5",
     "@mikro-orm/decorators": "^7.1.5",
     "@mikro-orm/postgresql": "^7.1.5",
-    "@open-mercato/cache": "0.6.7-develop.6669.1.40b669666b",
+    "@open-mercato/cache": "0.6.7-develop.6670.1.4efa7961c6",
     "@types/sanitize-html": "^2.16.1",
     "dotenv": "^17.4.2",
     "pino": "^10.3.1",

package/src/lib/commands/__tests__/command-bus.cache.test.ts

@@ -26,6 +26,7 @@ describe('CommandBus cache invalidation for sales documents', () => {
   afterEach(() => {
     unregisterCommand('sales.orders.update')
     unregisterCommand('sales.orders.noop-update')
+    unregisterCommand('wms.warehouses.create')
     invalidateMock.mockClear()
   })
 
@@ -194,4 +195,49 @@ describe('CommandBus cache invalidation for sales documents', () => {
       expect.any(Array)
     )
   })
+
+  it('resolves record id from warehouseId-style command results for cache tags', async () => {
+    const logMock = jest.fn(async () => ({ id: 'log-entry' }))
+
+    registerCommand({
+      id: 'wms.warehouses.create',
+      execute: jest.fn(async () => ({ warehouseId: 'warehouse-uuid-1' })),
+      buildLog: jest.fn(() => ({
+        actionLabel: 'Create warehouse',
+        resourceKind: 'wms.warehouse',
+        tenantId: 'tenant-1',
+        organizationId: 'org-1',
+      })),
+    })
+
+    const container = createContainer({ injectionMode: InjectionMode.CLASSIC })
+    container.register({
+      actionLogService: asValue({
+        log: logMock,
+        findByUndoToken: jest.fn(async () => null),
+        markUndone: jest.fn(async () => {}),
+      }),
+      dataEngine: asValue({ flushOrmEntityChanges: jest.fn() }),
+    })
+
+    const bus = new CommandBus()
+    const ctx = {
+      container,
+      auth: { sub: 'user-1', tenantId: 'tenant-1', orgId: 'org-1' },
+      organizationScope: null,
+      selectedOrganizationId: 'org-1',
+      organizationIds: null,
+    }
+
+    await bus.execute('wms.warehouses.create', { input: {}, ctx })
+
+    expect(invalidateMock).toHaveBeenCalledWith(
+      container,
+      'wms.warehouse',
+      { id: 'warehouse-uuid-1', organizationId: 'org-1', tenantId: 'tenant-1' },
+      'tenant-1',
+      'command:wms.warehouses.create:execute',
+      expect.any(Array)
+    )
+  })
 })

package/src/lib/commands/command-bus.ts

@@ -48,6 +48,28 @@ function asRecord(input: unknown): Record<string, unknown> | null {
   return input as Record<string, unknown>
 }
 
+/** Command handlers often return domain keys (e.g. warehouseId) without `id`; cache invalidation must still resolve the record. */
+function extractPrimaryIdFromCommandResult(result: unknown): string | null {
+  const r = asRecord(result)
+  if (!r) return null
+  const direct = pickFirstIdentifier(r.id, r.entityId, r.recordId)
+  if (direct) return direct
+  for (const key of [
+    'warehouseId',
+    'zoneId',
+    'locationId',
+    'lotId',
+    'reservationId',
+    'profileId',
+    'movementId',
+    'balanceId',
+  ]) {
+    const v = r[key]
+    if (typeof v === 'string' && v.trim().length > 0) return v.trim()
+  }
+  return null
+}
+
 function toISOString(value: unknown): string | null {
   if (value instanceof Date) {
     const iso = value.toISOString()
@@ -584,6 +606,7 @@ export class CommandBus {
 
       const recordId = pickFirstIdentifier(
         metadata?.resourceId,
+        extractPrimaryIdFromCommandResult(result),
         resultRecord?.entityId,
         resultRecord?.id,
         resultRecord?.recordId,

package/src/lib/crud/factory.ts

@@ -228,6 +228,8 @@ export type ListConfig<TList> = {
    * filters/sorts, `search_tokens` fulltext filtering, and vector-search branches are bypassed.
    */
   omitAutomaticTenantOrgScope?: boolean
+  /** When true, skip server-side CRUD GET cache for this list (avoids stale empty payloads after mutations). */
+  disableListCache?: boolean
 }
 
 export type CrudExportColumnConfig = {
@@ -1461,7 +1463,8 @@ export function makeCrudRoute<TCreate = any, TUpdate = any, TList = any>(opts: C
       const exportFullRequested = exportRequested && (exportScope === 'full' || parseBooleanToken((queryParams as any).full) === true)
       profiler.mark('export_configured', { exportRequested, exportFullRequested })
 
-      const cacheEnabled = isCrudCacheEnabled() && !exportRequested
+      const cacheEnabled =
+        isCrudCacheEnabled() && !exportRequested && !opts.list?.disableListCache
       const cacheTimerStart = cacheEnabled && isCrudCacheDebugEnabled()
         ? process.hrtime.bigint()
         : null

package/src/lib/frontend/organizationEvents.ts

@@ -11,6 +11,7 @@ let currentScope: OrganizationScopeChangedDetail = {
   tenantId: null 
 }
 let currentVersion = 0
+let hasEmitted = false
 
 export function getCurrentOrganizationScope(): OrganizationScopeChangedDetail {
   return { ...currentScope }
@@ -27,17 +28,25 @@ export function emitOrganizationScopeChanged(detail: OrganizationScopeChangedDet
   const hasChanged = 
     currentScope.organizationId !== detail.organizationId ||
     currentScope.tenantId !== detail.tenantId
+  // Guard so the very first emission always syncs subscribers, even when the
+  // initial scope matches the default all-orgs `{ null, null }` state.
+  const isFirstEmit = !hasEmitted
   
   // Update module-level state
   currentScope = { ...detail }
+  hasEmitted = true
   
   // Increment version only if actual change detected
   if (hasChanged) {
     currentVersion++
   }
   
-  // Emit event
-  window.dispatchEvent(new CustomEvent<OrganizationScopeChangedDetail>(ORGANIZATION_SCOPE_CHANGED_EVENT, { detail }))
+  // Only dispatch on real scope changes (or the first emission). Suppressing
+  // no-op re-emits avoids spurious DataTable refreshes, chrome refetches, and
+  // scope-hook churn when navigating between pages with the same scope.
+  if (hasChanged || isFirstEmit) {
+    window.dispatchEvent(new CustomEvent<OrganizationScopeChangedDetail>(ORGANIZATION_SCOPE_CHANGED_EVENT, { detail }))
+  }
 }
 
 export function subscribeOrganizationScopeChanged(

package/src/modules/widgets/injection.ts

@@ -248,6 +248,13 @@ export type InjectionWidgetModule<TContext = unknown, TData = unknown> = {
 
 export type InjectionColumnDefinition = {
   id: string
+  /**
+   * i18n translation key for the column header.
+   * When present, consumers should call `t(headerKey, header)` instead of treating `header` as a key.
+   * Follows the same `label`/`labelKey` convention used by `InjectionFieldDefinition`.
+   */
+  headerKey?: string
+  /** Display text (and i18n fallback) for the column header. */
   header: string
   accessorKey: string
   cell?: (props: { getValue: () => unknown }) => ReactNode