@open-mercato/core 0.6.7-develop.6862.1.c11a64ce0a → 0.6.7

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

@@ -1,28 +1,18 @@
-import { createHash } from 'node:crypto'
 import { z } from 'zod'
 import type { EntityManager } from '@mikro-orm/postgresql'
 import { type Kysely, sql } from 'kysely'
-import { runWithCacheTenant } from '@open-mercato/cache'
 import type { CommandBus } from '@open-mercato/shared/lib/commands/command-bus'
-import {
-  buildCollectionTags,
-  debugCrudCache,
-  isCrudCacheEnabled,
-  normalizeTagSegment,
-  resolveCrudCache,
-} from '@open-mercato/shared/lib/crud/cache'
 import type { OpenApiRouteDoc } from '@open-mercato/shared/lib/openapi/types'
 import { findWithDecryption } from '@open-mercato/shared/lib/encryption/find'
 import { lookupHashCandidates } from '@open-mercato/shared/lib/encryption/aes'
 import { User } from '../../auth/data/entities'
 import { Message, MessageObject } from '../data/entities'
-import { composeMessageSchema, listMessagesSchema, type ListMessagesInput } from '../data/validators'
+import { composeMessageSchema, listMessagesSchema } from '../data/validators'
 import { MESSAGE_ATTACHMENT_ENTITY_ID } from '../lib/constants'
 import { getMessageType } from '../lib/message-types-registry'
 import { validateMessageObjectsForType } from '../lib/object-validation'
 import { attachOperationMetadataHeader } from '../lib/operationMetadata'
 import { canUseMessageEmailFeature, resolveMessageContext } from '../lib/routeHelpers'
-import { applyMessageParticipantScope } from '../lib/participantScope'
 import { resolveUserFeatures, runMessageMutationGuardAfterSuccess, runMessageMutationGuards } from './guards'
 import { findMessageIdsBySearchTokens } from '../lib/searchLookup'
 import { MessageCommandExecuteResult } from '../commands/shared'
@@ -38,8 +28,6 @@ type MessageCommandExecuteResultWithThreadId = MessageCommandExecuteResult & {
 }
 
 const NO_MATCH_ID = '00000000-0000-0000-0000-000000000000'
-const MESSAGE_LIST_CACHE_TTL_MS = 30_000
-const MESSAGE_LIST_RESOURCE = 'messages.message'
 
 function getDb(em: EntityManager): Kysely<any> {
   return em.getKysely<any>()
@@ -63,59 +51,6 @@ type RecipientCountRow = {
   count: string | number
 }
 
-type MessageListPayload = {
-  items: Array<Record<string, unknown>>
-  page: number
-  pageSize: number
-  total: number
-  totalPages: number
-}
-
-type MessageListScope = Awaited<ReturnType<typeof resolveMessageContext>>['scope']
-
-function normalizeCacheFilterValue(value: unknown): unknown {
-  if (value === undefined) return null
-  if (value instanceof Date) return value.toISOString()
-  if (Array.isArray(value)) return value.map(normalizeCacheFilterValue)
-  if (value && typeof value === 'object') {
-    return Object.entries(value as Record<string, unknown>)
-      .sort(([leftKey], [rightKey]) => leftKey.localeCompare(rightKey))
-      .map(([key, entryValue]) => [key, normalizeCacheFilterValue(entryValue)])
-  }
-  return value
-}
-
-function buildMessageListFilterSignature(input: ListMessagesInput): string {
-  const canonicalInput = Object.entries(input)
-    .sort(([leftKey], [rightKey]) => leftKey.localeCompare(rightKey))
-    .map(([key, value]) => [key, normalizeCacheFilterValue(value)])
-
-  return createHash('sha256')
-    .update(JSON.stringify(canonicalInput))
-    .digest('hex')
-    .slice(0, 32)
-}
-
-function buildMessageListCacheKey(scope: MessageListScope, input: ListMessagesInput): string {
-  return [
-    'messages:list:v1',
-    `tenant:${normalizeTagSegment(scope.tenantId)}`,
-    `org:${normalizeTagSegment(scope.organizationId)}`,
-    `user:${normalizeTagSegment(scope.userId)}`,
-    `filters:${buildMessageListFilterSignature(input)}`,
-  ].join('|')
-}
-
-function isMessageListPayload(value: unknown): value is MessageListPayload {
-  if (!value || typeof value !== 'object') return false
-  const payload = value as Partial<MessageListPayload>
-  return Array.isArray(payload.items)
-    && typeof payload.page === 'number'
-    && typeof payload.pageSize === 'number'
-    && typeof payload.total === 'number'
-    && typeof payload.totalPages === 'number'
-}
-
 export const metadata = {
   GET: { requireAuth: true },
   POST: { requireAuth: true, requireFeatures: ['messages.compose'] },
@@ -123,26 +58,10 @@ export const metadata = {
 
 export async function GET(req: Request) {
   const { ctx, scope } = await resolveMessageContext(req)
+  const em = ctx.container.resolve('em') as EntityManager
   const url = new URL(req.url)
   const params = Object.fromEntries(url.searchParams)
   const input = listMessagesSchema.parse(params)
-
-  const cache = isCrudCacheEnabled() ? resolveCrudCache(ctx.container) : null
-  const cacheKey = cache ? buildMessageListCacheKey(scope, input) : null
-  if (cache && cacheKey) {
-    try {
-      const cached = await runWithCacheTenant(scope.tenantId, () => cache.get(cacheKey))
-      if (isMessageListPayload(cached)) {
-        return Response.json(cached)
-      }
-    } catch (error) {
-      debugCrudCache('messages-list-cache-read-failed', {
-        error: error instanceof Error ? error.message : String(error),
-      })
-    }
-  }
-
-  const em = ctx.container.resolve('em') as EntityManager
   const db = getDb(em) as any
 
   const searchIds = input.search
@@ -201,9 +120,11 @@ export async function GET(req: Request) {
         joinRecipient()
         break
       case 'all':
-        // Sender-OR-recipient participant scope shared with the
-        // communication_channels message enricher — see participantScope.ts (#4133).
-        q = applyMessageParticipantScope(q, scope.userId)
+        joinRecipient()
+        q = q.where((eb: any) => eb.or([
+          eb('m.sender_user_id', '=', scope.userId),
+          eb('r.message_id', 'is not', null),
+        ]))
         break
       default: {
         const unsupportedFolder: never = input.folder
@@ -370,7 +291,7 @@ export async function GET(req: Request) {
     senderMetaById.set(user.id, { name, email: user.email ?? null })
   })
 
-  const payload: MessageListPayload = {
+  return Response.json({
     items: typedRows
       .map((row) => {
         const message = messagesById.get(row.id)
@@ -417,28 +338,7 @@ export async function GET(req: Request) {
     pageSize: input.pageSize,
     total,
     totalPages: Math.ceil(total / input.pageSize),
-  }
-
-  if (cache && cacheKey) {
-    try {
-      await runWithCacheTenant(scope.tenantId, () =>
-        cache.set(cacheKey, payload, {
-          ttl: MESSAGE_LIST_CACHE_TTL_MS,
-          tags: buildCollectionTags(
-            MESSAGE_LIST_RESOURCE,
-            scope.tenantId,
-            [scope.organizationId],
-          ),
-        }),
-      )
-    } catch (error) {
-      debugCrudCache('messages-list-cache-write-failed', {
-        error: error instanceof Error ? error.message : String(error),
-      })
-    }
-  }
-
-  return Response.json(payload)
+  })
 }
 
 export async function POST(req: Request) {

package/src/modules/messages/components/ComposeMessagePageClient.tsx

@@ -1,7 +1,6 @@
 "use client"
 
 import { useRouter } from 'next/navigation'
-import { extensionPoints } from '@open-mercato/core/modules/messages/extension-points'
 import { MessageComposer } from '@open-mercato/ui/backend/messages'
 // UMES extension surface — compose page injection spot (SPEC-045d §9.3a).
 // Channel provider packages inject "composer capabilities" widgets here
@@ -22,7 +21,7 @@ export function ComposeMessagePageClient() {
         scoping) into this spot purely as additional UI siblings.
       */}
       <InjectionSpot
-        spotId={extensionPoints.hosts.composeFields.spotId}
+        spotId="crud-form:messages:message:fields"
         context={{ form: 'compose' }}
         data={{}}
       />

package/src/modules/messages/components/MessageDetailPageClient.tsx

@@ -1,7 +1,6 @@
 "use client"
 
 import * as React from 'react'
-import { extensionPoints } from '@open-mercato/core/modules/messages/extension-points'
 import { MessageComposer } from '@open-mercato/ui/backend/messages'
 import { apiCall } from '@open-mercato/ui/backend/utils/apiCall'
 import { useConfirmDialog } from '@open-mercato/ui/backend/confirm-dialog'
@@ -95,7 +94,7 @@ function MessageConversationDetailItem({
 
           {/* UMES — channel payload renderer mounts here (channel-linked emails, Slack blocks, etc.). */}
           <InjectionSpot
-            spotId={extensionPoints.hosts.detailBodyAfter.spotId}
+            spotId="detail:messages:message:body:after"
             context={{ messageId }}
             data={state.detail}
           />
@@ -106,7 +105,7 @@ function MessageConversationDetailItem({
               `userFeatures` lets feature-gated widgets (e.g. the channel reassignment editor,
               gated on `communication_channels.assign`) render their controls for permitted users. */}
           <InjectionSpot
-            spotId={extensionPoints.hosts.detailSidebar.spotId}
+            spotId="detail:messages:message:sidebar"
             context={{ messageId, userFeatures }}
             data={state.detail}
           />

package/src/modules/messages/components/MessagesInboxPageClient.tsx

@@ -1,7 +1,6 @@
 "use client"
 
 import * as React from 'react'
-import { extensionPoints } from '@open-mercato/core/modules/messages/extension-points'
 import Link from 'next/link'
 import { useRouter } from 'next/navigation'
 import { useQuery, useQueryClient } from '@tanstack/react-query'
@@ -370,7 +369,7 @@ export function MessagesInboxPageClient() {
         // UMES extension surface — opt into widget injection at:
         //   data-table:messages:columns / :row-actions / :bulk-actions / :filters / :toolbar / :search-trailing
         // (SPEC-045d §9.3a — communication_channels hub renders channel badge + delivery status here)
-        extensionTableId={extensionPoints.hosts.inboxTable.tableId}
+        extensionTableId="messages"
         columns={columns}
         data={rows}
         bulkActions={bulkActions}

package/src/modules/messages/components/message-detail/panels/dialogs.tsx

@@ -93,7 +93,7 @@ export function MessageDetailDialogs(props: DialogsProps) {
             </Button>
             <Button
               type="button"
-              variant="destructive-solid"
+              variant="destructive"
               onClick={() => void props.handleDelete()}
               disabled={props.updatingState}
             >

package/src/modules/messages/i18n/de.json

@@ -213,8 +213,6 @@
   "messages.replyBody": "Antwort",
   "messages.saveDraft": "Entwurf speichern",
   "messages.savingDraft": "Entwurf wird gespeichert...",
-  "messages.search.badge.message": "Nachricht",
-  "messages.search.fallback.title": "Nachricht",
   "messages.searchPlaceholder": "Nachrichten suchen",
   "messages.send": "Senden",
   "messages.sendViaEmail": "Auch per E-Mail senden",

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

@@ -213,8 +213,6 @@
   "messages.replyBody": "Reply",
   "messages.saveDraft": "Save draft",
   "messages.savingDraft": "Saving draft...",
-  "messages.search.badge.message": "Message",
-  "messages.search.fallback.title": "Message",
   "messages.searchPlaceholder": "Search messages",
   "messages.send": "Send",
   "messages.sendViaEmail": "Also send via email",

package/src/modules/messages/i18n/es.json

@@ -213,8 +213,6 @@
   "messages.replyBody": "Respuesta",
   "messages.saveDraft": "Guardar borrador",
   "messages.savingDraft": "Guardando borrador...",
-  "messages.search.badge.message": "Mensaje",
-  "messages.search.fallback.title": "Mensaje",
   "messages.searchPlaceholder": "Buscar mensajes",
   "messages.send": "Enviar",
   "messages.sendViaEmail": "Enviar también por correo",

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

@@ -213,8 +213,6 @@
   "messages.replyBody": "Odpowiedź",
   "messages.saveDraft": "Zapisz szkic",
   "messages.savingDraft": "Zapisywanie szkicu...",
-  "messages.search.badge.message": "Wiadomość",
-  "messages.search.fallback.title": "Wiadomość",
   "messages.searchPlaceholder": "Szukaj wiadomości",
   "messages.send": "Wyślij",
   "messages.sendViaEmail": "Wyślij także e-mailem",

package/src/modules/messages/lib/routeHelpers.ts

@@ -1,4 +1,5 @@
 import { resolveRequestContext } from '@open-mercato/shared/lib/api/context'
+import { hasFeature } from '@open-mercato/shared/security/features'
 
 export function hasOrganizationAccess(
   scopeOrganizationId: string | null,
@@ -32,11 +33,10 @@ export async function resolveMessageContext(req: Request): Promise<{
 }
 
 type RbacService = {
-  userHasAllFeatures: (
+  loadAcl: (
     userId: string,
-    required: string[],
     scope: { tenantId: string | null; organizationId: string | null }
-  ) => Promise<boolean>
+  ) => Promise<{ features?: string[]; isSuperAdmin?: boolean }>
 }
 
 export async function parseRequestBodySafe(req: Request): Promise<unknown> {
@@ -56,8 +56,10 @@ export async function canUseMessageEmailFeature(
   if (!scope.userId || !scope.tenantId) return false
 
   const rbac = ctx.container.resolve('rbacService') as RbacService
-  return rbac.userHasAllFeatures(scope.userId, ['messages.email'], {
+  const acl = await rbac.loadAcl(scope.userId, {
     tenantId: scope.tenantId,
     organizationId: scope.organizationId,
   })
+
+  return Boolean(acl.isSuperAdmin) || hasFeature(acl.features, 'messages.email')
 }

package/src/modules/messages/lib/searchLookup.ts

@@ -1,9 +1,10 @@
 import type { EntityManager } from '@mikro-orm/postgresql'
-import type { Kysely } from 'kysely'
-import { findEntityIdsBySearchTokensCompat, type SearchTokenDatabase } from '@open-mercato/shared/lib/search/tokenLookup'
+import { type Kysely, sql } from 'kysely'
+import { resolveSearchConfig } from '@open-mercato/shared/lib/search/config'
+import { tokenizeText } from '@open-mercato/shared/lib/search/tokenize'
 
-function getDb(em: EntityManager): Kysely<SearchTokenDatabase> {
-  return em.getKysely<SearchTokenDatabase>()
+function getDb(em: EntityManager): Kysely<any> {
+  return em.getKysely<any>()
 }
 
 export async function findMessageIdsBySearchTokens({
@@ -19,11 +20,33 @@ export async function findMessageIdsBySearchTokens({
   organizationId: string | null
   fields?: string[]
 }): Promise<string[] | null> {
-  return findEntityIdsBySearchTokensCompat({
-    db: getDb(em),
-    entityType: 'messages:message',
-    query,
-    fields,
-    scope: { tenantId, organizationId },
-  })
+  const trimmed = query.trim()
+  if (!trimmed) return null
+
+  const tokens = tokenizeText(trimmed, resolveSearchConfig())
+  if (!tokens.hashes.length) return []
+
+  const db = getDb(em) as any
+  let searchQuery = db
+    .selectFrom('search_tokens')
+    .select('entity_id')
+    .where('entity_type', '=', 'messages:message')
+    .where('field', 'in', fields)
+    .where('token_hash', 'in', tokens.hashes)
+    .where(sql<boolean>`tenant_id is not distinct from ${tenantId}`)
+
+  if (organizationId) {
+    searchQuery = searchQuery.where('organization_id', '=', organizationId)
+  } else {
+    searchQuery = searchQuery.where(sql<boolean>`organization_id is not distinct from ${null}`)
+  }
+
+  const rows = await searchQuery
+    .groupBy('entity_id')
+    .having(sql<boolean>`count(distinct token_hash) >= ${tokens.hashes.length}`)
+    .execute()
+
+  return rows
+    .map((row: { entity_id?: unknown }) => (typeof row.entity_id === 'string' ? row.entity_id : null))
+    .filter((id: string | null): id is string => typeof id === 'string' && id.length > 0)
 }

package/src/modules/messages/search.ts

@@ -4,8 +4,6 @@ import type {
   SearchIndexSource,
   SearchResultPresenter,
 } from '@open-mercato/shared/modules/search'
-import type { TranslateFn } from '@open-mercato/shared/lib/i18n/context'
-import { resolveTranslations } from '@open-mercato/shared/lib/i18n/server'
 
 function appendLine(lines: string[], label: string, value: unknown) {
   if (value === null || value === undefined) return
@@ -33,8 +31,8 @@ function snippet(value: unknown, max = 140): string | undefined {
   return `${trimmed.slice(0, max - 3)}...`
 }
 
-function buildMessagePresenter(translate: TranslateFn, record: Record<string, unknown>): SearchResultPresenter {
-  const title = pickString(record.subject) ?? translate('messages.search.fallback.title', 'Message')
+function buildMessagePresenter(record: Record<string, unknown>): SearchResultPresenter {
+  const title = pickString(record.subject) ?? 'Message'
   const body = snippet(record.body)
   const externalName = pickString(record.external_name, record.externalName)
   const subtitle = [externalName, body].filter(Boolean).join(' · ') || undefined
@@ -42,7 +40,7 @@ function buildMessagePresenter(translate: TranslateFn, record: Record<string, un
     title: String(title),
     subtitle,
     icon: 'mail',
-    badge: translate('messages.search.badge.message', 'Message'),
+    badge: 'Message',
   }
 }
 
@@ -53,7 +51,6 @@ export const searchConfig: SearchModuleConfig = {
       enabled: true,
       priority: 5,
       buildSource: async (ctx: SearchBuildContext): Promise<SearchIndexSource | null> => {
-        const { t } = await resolveTranslations()
         const record = ctx.record
         const lines: string[] = []
         appendLine(lines, 'Subject', record.subject)
@@ -62,7 +59,7 @@ export const searchConfig: SearchModuleConfig = {
         if (!lines.length) return null
         return {
           text: lines,
-          presenter: buildMessagePresenter(t, record),
+          presenter: buildMessagePresenter(record),
           checksumSource: {
             record: {
               subject: record.subject,
@@ -73,10 +70,7 @@ export const searchConfig: SearchModuleConfig = {
           },
         }
       },
-      formatResult: async (ctx) => {
-        const { t } = await resolveTranslations()
-        return buildMessagePresenter(t, ctx.record)
-      },
+      formatResult: async (ctx) => buildMessagePresenter(ctx.record),
       resolveUrl: async (ctx) => {
         const id = pickString(ctx.record.id)
         return id ? `/backend/messages/${encodeURIComponent(id)}` : null

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

@@ -1,20 +1,8 @@
-import { createHash } from 'node:crypto'
 import { z } from 'zod'
 import type { EntityManager } from '@mikro-orm/core'
-import { runWithCacheTenant } from '@open-mercato/cache'
-import {
-  buildCollectionTags,
-  debugCrudCache,
-  isCrudCacheEnabled,
-  resolveCrudCache,
-} from '@open-mercato/shared/lib/crud/cache'
 import { Notification } from '../data/entities'
 import { listNotificationsSchema, createNotificationSchema } from '../data/validators'
 import { toNotificationDto } from '../lib/notificationMapper'
-import {
-  buildNotificationReadScopeWhere,
-  getNotificationReadScopeTagOrganizationIds,
-} from '../lib/notificationScope'
 import {
   NOTIFICATION_RESOURCE_KIND,
   notificationCrudErrorResponse,
@@ -33,61 +21,6 @@ export const metadata = {
   POST: { requireAuth: true, requireFeatures: ['notifications.create'] },
 }
 
-const NOTIFICATIONS_LIST_TTL_MS = 10_000
-// v2: the key gained the selected-organization scope (#4840). Bumping the version
-// retires v1 entries, which were shared across organizations for the same user.
-const NOTIFICATIONS_LIST_CACHE_VERSION = 2
-const NOTIFICATIONS_LIST_RESOURCE = 'notifications.notification'
-
-type NotificationsListPayload = {
-  items: ReturnType<typeof toNotificationDto>[]
-  total: number
-  page: number
-  pageSize: number
-  totalPages: number
-}
-
-export function buildNotificationsListCacheKey(
-  userId: string,
-  organizationScope: { organizationId: string | null; organizationIds: string[] },
-  input: z.infer<typeof listNotificationsSchema>,
-): string {
-  const normalizedIds = Array.from(new Set(
-    organizationScope.organizationIds.filter((value) => value.trim().length > 0),
-  )).sort((left, right) => left.localeCompare(right))
-  const scopeKey = normalizedIds.length === 0
-    ? 'no-access'
-    : `${organizationScope.organizationId ?? 'none'}:scope=${createHash('sha256')
-        .update(normalizedIds.join('\0'))
-        .digest('hex')
-        .slice(0, 16)}`
-  const filterSignature = JSON.stringify({
-    status: Array.isArray(input.status)
-      ? [...input.status].sort((left, right) => left.localeCompare(right))
-      : input.status ?? null,
-    type: input.type ?? null,
-    severity: input.severity ?? null,
-    sourceEntityType: input.sourceEntityType ?? null,
-    sourceEntityId: input.sourceEntityId ?? null,
-    since: input.since ?? null,
-    page: input.page,
-    pageSize: input.pageSize,
-  })
-  return `notifications:list:v${NOTIFICATIONS_LIST_CACHE_VERSION}:u=${userId}:org=${scopeKey}:filters=${filterSignature}`
-}
-
-function isNotificationsListPayload(value: unknown): value is NotificationsListPayload {
-  if (!value || typeof value !== 'object') return false
-  const payload = value as Partial<NotificationsListPayload>
-  return (
-    Array.isArray(payload.items)
-    && typeof payload.total === 'number'
-    && typeof payload.page === 'number'
-    && typeof payload.pageSize === 'number'
-    && typeof payload.totalPages === 'number'
-  )
-}
-
 export async function GET(req: Request) {
   const { ctx, scope } = await resolveNotificationContext(req)
   const em = ctx.container.resolve('em') as EntityManager
@@ -95,42 +28,10 @@ export async function GET(req: Request) {
   const url = new URL(req.url)
   const queryParams = Object.fromEntries(url.searchParams.entries())
   const input = listNotificationsSchema.parse(queryParams)
-  const userId = scope.userId
-  // Mirrors api/unread-count/route.ts: unrestricted and omitted legacy organization
-  // scopes cannot be safely tagged for invalidation, because organization-specific
-  // writes only invalidate their own collection tag. Leave those scopes uncached
-  // rather than serving a stale tenant-wide list until the TTL expires.
-  const cacheableOrganizationIds = Array.isArray(scope.organizationIds)
-    ? scope.organizationIds
-    : null
-  const cache = userId && cacheableOrganizationIds && isCrudCacheEnabled()
-    ? resolveCrudCache(ctx.container)
-    : null
-  const cacheKey = cache && userId && cacheableOrganizationIds
-    ? buildNotificationsListCacheKey(
-        userId,
-        { organizationId: scope.organizationId, organizationIds: cacheableOrganizationIds },
-        input,
-      )
-    : null
-
-  if (cache && cacheKey) {
-    try {
-      const cached = await runWithCacheTenant(scope.tenantId, () => cache.get(cacheKey))
-      if (isNotificationsListPayload(cached)) {
-        return Response.json(cached)
-      }
-    } catch (error) {
-      debugCrudCache('notifications-list-cache-read-failed', {
-        error: error instanceof Error ? error.message : String(error),
-      })
-    }
-  }
 
   const filters: Record<string, unknown> = {
-    recipientUserId: userId,
+    recipientUserId: scope.userId,
     tenantId: scope.tenantId,
-    ...buildNotificationReadScopeWhere(scope),
   }
 
   if (input.status) {
@@ -165,34 +66,13 @@ export async function GET(req: Request) {
 
   const items = notifications.map(toNotificationDto)
 
-  const payload: NotificationsListPayload = {
+  return Response.json({
     items,
     total,
     page: input.page,
     pageSize: input.pageSize,
     totalPages: Math.ceil(total / input.pageSize),
-  }
-
-  if (cache && cacheKey) {
-    try {
-      await runWithCacheTenant(scope.tenantId, () =>
-        cache.set(cacheKey, payload, {
-          ttl: NOTIFICATIONS_LIST_TTL_MS,
-          tags: buildCollectionTags(
-            NOTIFICATIONS_LIST_RESOURCE,
-            scope.tenantId,
-            getNotificationReadScopeTagOrganizationIds(scope),
-          ),
-        }),
-      )
-    } catch (error) {
-      debugCrudCache('notifications-list-cache-write-failed', {
-        error: error instanceof Error ? error.message : String(error),
-      })
-    }
-  }
-
-  return Response.json(payload)
+  })
 }
 
 export async function POST(req: Request) {