@open-mercato/core 0.6.6-develop.6256.1.9fc16aedc4 → 0.6.6-develop.6257.1.6d0af84d26

package/src/modules/messages/api/[id]/conversation/read/route.ts

@@ -9,10 +9,15 @@ import {
 } from '../../../openapi'
 
 export const metadata = {
+  PUT: { requireAuth: true, requireFeatures: ['messages.view'] },
   DELETE: { requireAuth: true, requireFeatures: ['messages.view'] },
 }
 
-export async function DELETE(req: Request, { params }: { params: { id: string } }) {
+async function runConversationReadMutation(
+  req: Request,
+  id: string,
+  commandId: 'messages.conversation.mark_read_for_actor' | 'messages.conversation.mark_unread_for_actor',
+) {
   const { ctx, scope } = await resolveMessageContext(req)
   const commandBus = ctx.container.resolve('commandBus') as CommandBus
 
@@ -23,7 +28,7 @@ export async function DELETE(req: Request, { params }: { params: { id: string }
       organizationId: scope.organizationId,
       userId: scope.userId,
       resourceKind: 'messages.conversation',
-      resourceId: params.id,
+      resourceId: id,
       operation: 'update',
       requestMethod: req.method,
       requestHeaders: req.headers,
@@ -39,9 +44,9 @@ export async function DELETE(req: Request, { params }: { params: { id: string }
   }
 
   try {
-    const { result, logEntry } = await commandBus.execute('messages.conversation.mark_unread_for_actor', {
+    const { result, logEntry } = await commandBus.execute(commandId, {
       input: {
-        anchorMessageId: params.id,
+        anchorMessageId: id,
         tenantId: scope.tenantId,
         organizationId: scope.organizationId,
         userId: scope.userId,
@@ -59,14 +64,14 @@ export async function DELETE(req: Request, { params }: { params: { id: string }
     const response = Response.json(result)
     attachOperationMetadataHeader(response, logEntry, {
       resourceKind: 'messages.conversation',
-      resourceId: params.id,
+      resourceId: id,
     })
     await runMessageMutationGuardAfterSuccess(guardResult.afterSuccessCallbacks, {
       tenantId: scope.tenantId,
       organizationId: scope.organizationId,
       userId: scope.userId,
       resourceKind: 'messages.conversation',
-      resourceId: params.id,
+      resourceId: id,
       operation: 'update',
       requestMethod: req.method,
       requestHeaders: req.headers,
@@ -85,9 +90,27 @@ export async function DELETE(req: Request, { params }: { params: { id: string }
   }
 }
 
+export async function PUT(req: Request, { params }: { params: { id: string } }) {
+  return runConversationReadMutation(req, params.id, 'messages.conversation.mark_read_for_actor')
+}
+
+export async function DELETE(req: Request, { params }: { params: { id: string } }) {
+  return runConversationReadMutation(req, params.id, 'messages.conversation.mark_unread_for_actor')
+}
+
 export const openApi: OpenApiRouteDoc = {
   tag: 'Messages',
   methods: {
+    PUT: {
+      summary: 'Mark entire conversation as read for current actor',
+      responses: [
+        { status: 200, description: 'Conversation marked read', schema: conversationMutationResponseSchema },
+      ],
+      errors: [
+        { status: 403, description: 'Access denied', schema: errorResponseSchema },
+        { status: 404, description: 'Message not found', schema: errorResponseSchema },
+      ],
+    },
     DELETE: {
       summary: 'Mark entire conversation as unread for current actor',
       responses: [

package/src/modules/messages/api/[id]/route.ts

@@ -122,6 +122,19 @@ export async function GET(req: Request, { params }: { params: { id: string } })
     threadMessage.senderUserId === scope.userId || visibleRecipientMessageIds.has(threadMessage.id)
   ))
 
+  const actorRecipientStatusByMessageId = new Map<string, string>()
+  for (const row of visibleRecipientRows) {
+    actorRecipientStatusByMessageId.set(row.messageId, row.status)
+  }
+  if (recipient) {
+    actorRecipientStatusByMessageId.set(params.id, autoMarkRead ? 'read' : recipient.status)
+  }
+  const actorRecipientStatuses = Array.from(actorRecipientStatusByMessageId.values())
+  const conversationArchived = actorRecipientStatuses.length > 0
+    && actorRecipientStatuses.every((status) => status === 'archived')
+  const conversationAllUnread = actorRecipientStatuses.length > 0
+    && actorRecipientStatuses.every((status) => status === 'unread')
+
   const threadSenderIds = actorVisibleThreadMessages
     .map((threadMessage) => threadMessage.senderUserId)
     .filter((value): value is string => typeof value === 'string' && value.length > 0)
@@ -244,6 +257,8 @@ export async function GET(req: Request, { params }: { params: { id: string } })
       }
     }),
     isRead: recipient ? (autoMarkRead || recipient.status !== 'unread') : true,
+    conversationArchived,
+    conversationAllUnread,
   })
 }
 

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

@@ -121,6 +121,8 @@ export const messageDetailResponseSchema = z.object({
   })),
   thread: z.array(messageThreadItemSchema),
   isRead: z.boolean(),
+  conversationArchived: z.boolean(),
+  conversationAllUnread: z.boolean(),
 })
 
 export const messageTokenDetailResponseSchema = z.object({

package/src/modules/messages/commands/conversation.ts

@@ -228,6 +228,91 @@ const markConversationUnreadForActorCommand: CommandHandler<unknown, { ok: true;
   },
 }
 
+const unarchiveConversationForActorCommand: CommandHandler<unknown, { ok: true; affectedCount: number }> = {
+  id: 'messages.conversation.unarchive_for_actor',
+  async execute(rawInput, ctx) {
+    const input = conversationCommandSchema.parse(rawInput)
+    const em = (ctx.container.resolve('em') as EntityManager).fork()
+    const scope = await resolveConversationScope(em, input)
+    const messageIdsToUnarchive = scope.messageIds.filter((messageId) => scope.recipientMessageIds.has(messageId))
+
+    if (messageIdsToUnarchive.length === 0) {
+      return { ok: true, affectedCount: 0 }
+    }
+
+    const unarchivedIds: string[] = []
+    await em.transactional(async (trx) => {
+      const recipients = await trx.find(MessageRecipient, {
+        messageId: { $in: messageIdsToUnarchive },
+        recipientUserId: input.userId,
+        deletedAt: null,
+      })
+      for (const recipient of recipients) {
+        if (recipient.status === 'archived' || recipient.archivedAt !== null) {
+          recipient.archivedAt = null
+          recipient.status = recipient.readAt ? 'read' : 'unread'
+          unarchivedIds.push(recipient.messageId)
+        }
+      }
+    })
+
+    for (const messageId of unarchivedIds) {
+      await emitMessagesEvent('messages.message.unarchived', {
+        messageId,
+        recipientUserId: input.userId,
+        userId: input.userId,
+        tenantId: input.tenantId,
+        organizationId: input.organizationId,
+      }, { persistent: true })
+    }
+
+    return { ok: true, affectedCount: unarchivedIds.length }
+  },
+}
+
+const markConversationReadForActorCommand: CommandHandler<unknown, { ok: true; affectedCount: number }> = {
+  id: 'messages.conversation.mark_read_for_actor',
+  async execute(rawInput, ctx) {
+    const input = conversationCommandSchema.parse(rawInput)
+    const em = (ctx.container.resolve('em') as EntityManager).fork()
+    const scope = await resolveConversationScope(em, input)
+    const messageIdsToMarkRead = scope.messageIds.filter((messageId) => scope.recipientMessageIds.has(messageId))
+
+    if (messageIdsToMarkRead.length === 0) {
+      return { ok: true, affectedCount: 0 }
+    }
+
+    const readAt = new Date()
+    const markedIds: string[] = []
+    await em.transactional(async (trx) => {
+      const recipients = await trx.find(MessageRecipient, {
+        messageId: { $in: messageIdsToMarkRead },
+        recipientUserId: input.userId,
+        deletedAt: null,
+      })
+      for (const recipient of recipients) {
+        if (recipient.status === 'unread') {
+          recipient.status = 'read'
+          recipient.readAt = readAt
+          markedIds.push(recipient.messageId)
+        }
+      }
+    })
+
+    for (const messageId of markedIds) {
+      await emitMessagesEvent('messages.message.read', {
+        messageId,
+        recipientUserId: input.userId,
+        userId: input.userId,
+        tenantId: input.tenantId,
+        organizationId: input.organizationId,
+      }, { persistent: true })
+    }
+
+    return { ok: true, affectedCount: markedIds.length }
+  },
+}
+
 type DeletedTarget = { messageId: string; target: 'sender' | 'recipient' }
 
 const deleteConversationForActorCommand: CommandHandler<unknown, { ok: true; affectedCount: number }> = {
@@ -289,5 +374,7 @@ const deleteConversationForActorCommand: CommandHandler<unknown, { ok: true; aff
 }
 
 registerCommand(archiveConversationForActorCommand)
+registerCommand(unarchiveConversationForActorCommand)
 registerCommand(markConversationUnreadForActorCommand)
+registerCommand(markConversationReadForActorCommand)
 registerCommand(deleteConversationForActorCommand)

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

@@ -220,6 +220,8 @@ export function MessageDetailPageClient({ id }: { id: string }) {
   const firstConversationMessageId = state.conversationItems[0]?.id ?? null
   const latestConversationMessageId = state.conversationItems[state.conversationItems.length - 1]?.id ?? null
   const canRunConversationActions = Boolean(firstConversationMessageId)
+  const conversationArchived = Boolean(detail.conversationArchived)
+  const conversationAllUnread = Boolean(detail.conversationAllUnread)
 
   return (
     <div className="space-y-3">
@@ -228,6 +230,8 @@ export function MessageDetailPageClient({ id }: { id: string }) {
         priority={(detail.priority as 'low' | 'normal' | 'high' | 'urgent') ?? 'normal'}
         canReply={!detail.isDraft && detail.typeDefinition.allowReply && Boolean(firstConversationMessageId)}
         canForwardAll={!detail.isDraft && detail.typeDefinition.allowForward && Boolean(latestConversationMessageId)}
+        conversationArchived={conversationArchived}
+        conversationAllUnread={conversationAllUnread}
         actionsDisabled={Boolean(state.activeConversationAction)}
         activeActionId={state.activeConversationAction}
         onReply={() => {
@@ -244,13 +248,21 @@ export function MessageDetailPageClient({ id }: { id: string }) {
             messageId: latestConversationMessageId,
           })
         }}
-        onArchiveConversation={() => {
+        onToggleArchiveConversation={() => {
           if (!canRunConversationActions) return
-          void state.archiveConversation(firstConversationMessageId ?? undefined)
+          if (conversationArchived) {
+            void state.unarchiveConversation(firstConversationMessageId ?? undefined)
+          } else {
+            void state.archiveConversation(firstConversationMessageId ?? undefined)
+          }
         }}
-        onMarkAllUnread={() => {
+        onToggleReadConversation={() => {
           if (!canRunConversationActions) return
-          void state.markConversationUnread(firstConversationMessageId ?? undefined)
+          if (conversationAllUnread) {
+            void state.markConversationRead(firstConversationMessageId ?? undefined)
+          } else {
+            void state.markConversationUnread(firstConversationMessageId ?? undefined)
+          }
         }}
         onDeleteConversation={() => {
           if (!canRunConversationActions || state.activeConversationAction) return

package/src/modules/messages/components/message-detail/hooks/useMessageDetailsActions.ts

@@ -20,7 +20,12 @@ type RequestAndRefreshOptions = {
   skipDetailAutoMarkRead?: boolean
 }
 
-type ConversationActionKind = 'archiveConversation' | 'deleteConversation' | 'markAllUnread'
+type ConversationActionKind =
+  | 'archiveConversation'
+  | 'unarchiveConversation'
+  | 'deleteConversation'
+  | 'markAllUnread'
+  | 'markAllRead'
 
 type MessageDetailMutationContext = {
   resourceKind: 'message' | 'conversation'
@@ -188,6 +193,15 @@ export function useMessageDetailsActions({
     })
   }, [id, runConversationAction, t])
 
+  const unarchiveConversation = React.useCallback(async (messageId?: string) => {
+    const targetMessageId = messageId ?? id
+    await runConversationAction('unarchiveConversation', {
+      url: `/api/messages/${encodeURIComponent(targetMessageId)}/conversation/archive`,
+      method: 'DELETE',
+      successMessage: t('messages.flash.conversationUnarchived', 'Conversation restored from archive.'),
+    })
+  }, [id, runConversationAction, t])
+
   const markConversationUnread = React.useCallback(async (messageId?: string) => {
     const targetMessageId = messageId ?? id
     await runConversationAction('markAllUnread', {
@@ -198,6 +212,15 @@ export function useMessageDetailsActions({
     })
   }, [id, runConversationAction, t])
 
+  const markConversationRead = React.useCallback(async (messageId?: string) => {
+    const targetMessageId = messageId ?? id
+    await runConversationAction('markAllRead', {
+      url: `/api/messages/${encodeURIComponent(targetMessageId)}/conversation/read`,
+      method: 'PUT',
+      successMessage: t('messages.flash.conversationMarkedRead', 'Conversation marked read.'),
+    })
+  }, [id, runConversationAction, t])
+
   const deleteConversation = React.useCallback(async (messageId?: string) => {
     const targetMessageId = messageId ?? id
     await runConversationAction('deleteConversation', {
@@ -393,8 +416,10 @@ export function useMessageDetailsActions({
     toggleRead,
     toggleArchive,
     archiveConversation,
+    unarchiveConversation,
     deleteConversation,
     markConversationUnread,
+    markConversationRead,
     requestAndRefresh,
     handleDelete,
     handleDeleteDialogKeyDown,

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

@@ -4,7 +4,7 @@ import * as React from 'react'
 import { useT } from '@open-mercato/shared/lib/i18n/context'
 import { FormHeader, type ActionItem } from '@open-mercato/ui/backend/forms'
 import { IconButton } from '@open-mercato/ui/primitives/icon-button'
-import { Archive, Forward, MailMinus, Reply, Trash2 } from 'lucide-react'
+import { Archive, ArchiveRestore, Forward, MailMinus, MailOpen, Reply, Trash2 } from 'lucide-react'
 import { PriorityBadge } from '../../utils/PriorityBadge'
 
 type MainMessageHeaderProps = {
@@ -12,12 +12,21 @@ type MainMessageHeaderProps = {
   priority: 'low' | 'normal' | 'high' | 'urgent'
   canReply: boolean
   canForwardAll: boolean
+  conversationArchived: boolean
+  conversationAllUnread: boolean
   actionsDisabled?: boolean
-  activeActionId?: 'forwardAll' | 'archiveConversation' | 'markAllUnread' | 'deleteConversation' | null
+  activeActionId?:
+    | 'forwardAll'
+    | 'archiveConversation'
+    | 'unarchiveConversation'
+    | 'markAllUnread'
+    | 'markAllRead'
+    | 'deleteConversation'
+    | null
   onReply: () => void
   onForwardAll: () => void
-  onArchiveConversation: () => void
-  onMarkAllUnread: () => void
+  onToggleArchiveConversation: () => void
+  onToggleReadConversation: () => void
   onDeleteConversation: () => void
 }
 
@@ -42,19 +51,23 @@ export function MainMessageHeader(props: MainMessageHeaderProps) {
     },
     {
       id: 'archive-conversation',
-      label: t('messages.actions.archiveConversation', 'Archive conversation'),
-      icon: Archive,
-      onSelect: props.onArchiveConversation,
+      label: props.conversationArchived
+        ? t('messages.actions.unarchiveConversation', 'Unarchive conversation')
+        : t('messages.actions.archiveConversation', 'Archive conversation'),
+      icon: props.conversationArchived ? ArchiveRestore : Archive,
+      onSelect: props.onToggleArchiveConversation,
       disabled: props.actionsDisabled,
-      loading: props.activeActionId === 'archiveConversation',
+      loading: props.activeActionId === 'archiveConversation' || props.activeActionId === 'unarchiveConversation',
     },
     {
       id: 'mark-all-unread',
-      label: t('messages.actions.markAllUnread', 'Mark all unread'),
-      icon: MailMinus,
-      onSelect: props.onMarkAllUnread,
+      label: props.conversationAllUnread
+        ? t('messages.actions.markAllRead', 'Mark all read')
+        : t('messages.actions.markAllUnread', 'Mark all unread'),
+      icon: props.conversationAllUnread ? MailOpen : MailMinus,
+      onSelect: props.onToggleReadConversation,
       disabled: props.actionsDisabled,
-      loading: props.activeActionId === 'markAllUnread',
+      loading: props.activeActionId === 'markAllUnread' || props.activeActionId === 'markAllRead',
     },
     {
       id: 'delete-conversation',
@@ -68,10 +81,12 @@ export function MainMessageHeader(props: MainMessageHeaderProps) {
     props.actionsDisabled,
     props.canForwardAll,
     props.canReply,
-    props.onArchiveConversation,
+    props.conversationArchived,
+    props.conversationAllUnread,
+    props.onToggleArchiveConversation,
     props.onDeleteConversation,
     props.onForwardAll,
-    props.onMarkAllUnread,
+    props.onToggleReadConversation,
     props.onReply,
     props.activeActionId,
     t,

package/src/modules/messages/components/message-detail/types.ts

@@ -97,6 +97,8 @@ export type MessageDetail = {
     objects?: MessageDetailObject[]
   }>
   isRead: boolean
+  conversationArchived?: boolean
+  conversationAllUnread?: boolean
 }
 
 export type ActionResult = {

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

@@ -9,6 +9,7 @@
   "messages.actions.edit": "Bearbeiten",
   "messages.actions.executing": "Wird ausgeführt...",
   "messages.actions.forwardAll": "Alle weiterleiten",
+  "messages.actions.markAllRead": "Alle als gelesen markieren",
   "messages.actions.markAllUnread": "Alle als ungelesen markieren",
   "messages.actions.markRead": "Als gelesen markieren",
   "messages.actions.markUnread": "Als ungelesen markieren",
@@ -16,6 +17,7 @@
   "messages.actions.taken": "Aktion ausgeführt",
   "messages.actions.title": "Aktionen",
   "messages.actions.unarchive": "Wiederherstellen",
+  "messages.actions.unarchiveConversation": "Konversation wiederherstellen",
   "messages.attachFiles": "Dateien anhängen",
   "messages.attachObject": "Objekt anhängen",
   "messages.attachedFiles": "Anhänge",
@@ -146,7 +148,9 @@
   "messages.flash.actionSuccess": "Aktion abgeschlossen.",
   "messages.flash.conversationArchived": "Konversation archiviert.",
   "messages.flash.conversationDeleted": "Konversation gelöscht.",
+  "messages.flash.conversationMarkedRead": "Konversation als gelesen markiert.",
   "messages.flash.conversationMarkedUnread": "Konversation als ungelesen markiert.",
+  "messages.flash.conversationUnarchived": "Konversation wiederhergestellt.",
   "messages.flash.deleted": "Nachricht gelöscht.",
   "messages.flash.draftSaved": "Entwurf gespeichert.",
   "messages.flash.forwardSuccess": "Nachricht weitergeleitet.",

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

@@ -9,6 +9,7 @@
   "messages.actions.edit": "Edit",
   "messages.actions.executing": "Executing...",
   "messages.actions.forwardAll": "Forward all",
+  "messages.actions.markAllRead": "Mark all read",
   "messages.actions.markAllUnread": "Mark all unread",
   "messages.actions.markRead": "Mark read",
   "messages.actions.markUnread": "Mark unread",
@@ -16,6 +17,7 @@
   "messages.actions.taken": "Action taken",
   "messages.actions.title": "Actions",
   "messages.actions.unarchive": "Unarchive",
+  "messages.actions.unarchiveConversation": "Unarchive conversation",
   "messages.attachFiles": "Attach files",
   "messages.attachObject": "Attach object",
   "messages.attachedFiles": "Attachments",
@@ -146,7 +148,9 @@
   "messages.flash.actionSuccess": "Action completed.",
   "messages.flash.conversationArchived": "Conversation archived.",
   "messages.flash.conversationDeleted": "Conversation deleted.",
+  "messages.flash.conversationMarkedRead": "Conversation marked read.",
   "messages.flash.conversationMarkedUnread": "Conversation marked unread.",
+  "messages.flash.conversationUnarchived": "Conversation restored from archive.",
   "messages.flash.deleted": "Message deleted.",
   "messages.flash.draftSaved": "Draft saved.",
   "messages.flash.forwardSuccess": "Message forwarded.",

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

@@ -9,6 +9,7 @@
   "messages.actions.edit": "Editar",
   "messages.actions.executing": "Ejecutando...",
   "messages.actions.forwardAll": "Reenviar todo",
+  "messages.actions.markAllRead": "Marcar todo como leído",
   "messages.actions.markAllUnread": "Marcar todo como no leído",
   "messages.actions.markRead": "Marcar como leído",
   "messages.actions.markUnread": "Marcar como no leído",
@@ -16,6 +17,7 @@
   "messages.actions.taken": "Acción realizada",
   "messages.actions.title": "Acciones",
   "messages.actions.unarchive": "Desarchivar",
+  "messages.actions.unarchiveConversation": "Desarchivar conversación",
   "messages.attachFiles": "Adjuntar archivos",
   "messages.attachObject": "Adjuntar objeto",
   "messages.attachedFiles": "Adjuntos",
@@ -146,7 +148,9 @@
   "messages.flash.actionSuccess": "Acción completada.",
   "messages.flash.conversationArchived": "Conversación archivada.",
   "messages.flash.conversationDeleted": "Conversación eliminada.",
+  "messages.flash.conversationMarkedRead": "Conversación marcada como leída.",
   "messages.flash.conversationMarkedUnread": "Conversación marcada como no leída.",
+  "messages.flash.conversationUnarchived": "Conversación desarchivada.",
   "messages.flash.deleted": "Mensaje eliminado.",
   "messages.flash.draftSaved": "Borrador guardado.",
   "messages.flash.forwardSuccess": "Mensaje reenviado.",

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

@@ -9,6 +9,7 @@
   "messages.actions.edit": "Edytuj",
   "messages.actions.executing": "Wykonywanie...",
   "messages.actions.forwardAll": "Przekaż cały wątek",
+  "messages.actions.markAllRead": "Oznacz wszystko jako przeczytane",
   "messages.actions.markAllUnread": "Oznacz wszystko jako nieprzeczytane",
   "messages.actions.markRead": "Oznacz jako przeczytane",
   "messages.actions.markUnread": "Oznacz jako nieprzeczytane",
@@ -16,6 +17,7 @@
   "messages.actions.taken": "Wykonana akcja",
   "messages.actions.title": "Akcje",
   "messages.actions.unarchive": "Przywróć z archiwum",
+  "messages.actions.unarchiveConversation": "Przywróć konwersację z archiwum",
   "messages.attachFiles": "Dołącz pliki",
   "messages.attachObject": "Dołącz obiekt",
   "messages.attachedFiles": "Załączniki",
@@ -146,7 +148,9 @@
   "messages.flash.actionSuccess": "Akcja wykonana.",
   "messages.flash.conversationArchived": "Konwersacja zarchiwizowana.",
   "messages.flash.conversationDeleted": "Konwersacja usunięta.",
+  "messages.flash.conversationMarkedRead": "Konwersacja oznaczona jako przeczytana.",
   "messages.flash.conversationMarkedUnread": "Konwersacja oznaczona jako nieprzeczytana.",
+  "messages.flash.conversationUnarchived": "Konwersacja przywrócona z archiwum.",
   "messages.flash.deleted": "Wiadomość usunięta.",
   "messages.flash.draftSaved": "Szkic zapisany.",
   "messages.flash.forwardSuccess": "Wiadomość przekazana.",