@micha.bigler/ui-core-micha 2.16.0 → 2.17.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (38) hide show
  1. package/WORK_ORDERS.md +2 -1
  2. package/dev/entries.jsx +4 -0
  3. package/dist/i18n/messagingTranslations.js +26 -0
  4. package/dist/index.js +1 -0
  5. package/dist/messaging/Composer.js +41 -5
  6. package/dist/messaging/ConversationList.js +10 -3
  7. package/dist/messaging/DirectMessageLauncher.js +48 -0
  8. package/dist/messaging/MessageBubble.js +84 -4
  9. package/dist/messaging/MessagingProvider.js +62 -16
  10. package/dist/messaging/ReactionBar.js +1 -1
  11. package/dist/messaging/ReadTicks.js +7 -4
  12. package/dist/messaging/Thread.js +45 -8
  13. package/dist/messaging/api.js +3 -3
  14. package/dist/messaging/composerImageCompression.js +36 -0
  15. package/docs/messaging-deviations.md +91 -26
  16. package/package.json +1 -1
  17. package/src/i18n/messagingTranslations.ts +26 -0
  18. package/src/index.js +1 -0
  19. package/src/messaging/Composer.jsx +54 -8
  20. package/src/messaging/ConversationList.jsx +10 -4
  21. package/src/messaging/DirectMessageLauncher.jsx +96 -0
  22. package/src/messaging/MessageBubble.jsx +74 -7
  23. package/src/messaging/MessagingProvider.jsx +83 -11
  24. package/src/messaging/ReactionBar.jsx +1 -1
  25. package/src/messaging/ReadTicks.jsx +5 -3
  26. package/src/messaging/Thread.jsx +39 -8
  27. package/src/messaging/api.js +3 -2
  28. package/src/messaging/composerImageCompression.js +35 -0
  29. package/tests/Thread.test.jsx +1 -1
  30. package/tests/messagingChunk6.test.jsx +109 -0
  31. package/tests/messagingComposerChunk5.test.jsx +110 -0
  32. package/tests/messagingContractConformance.test.js +42 -0
  33. package/tests/messagingDirectMessageLauncher.test.jsx +78 -0
  34. package/tests/messagingExports.test.js +2 -1
  35. package/tests/messagingMessageActions.test.jsx +93 -0
  36. package/tests/messagingThreadingQuoting.test.jsx +86 -0
  37. package/tests/messagingUnreadLifecycle.test.jsx +63 -0
  38. package/work-orders/MSG-3b.md +1002 -0
@@ -1,5 +1,5 @@
1
1
  import { Alert, Box, Button, CircularProgress, Divider, Stack, Typography } from '@mui/material';
2
- import { useCallback, useContext, useMemo, useRef, useState } from 'react';
2
+ import { useCallback, useContext, useEffect, useMemo, useRef, useState } from 'react';
3
3
  import { useTranslation } from 'react-i18next';
4
4
 
5
5
  import { AuthContext } from '../auth/AuthContext';
@@ -8,6 +8,7 @@ import { useMessaging } from './MessagingProvider';
8
8
  import { ReadTicks } from './ReadTicks';
9
9
 
10
10
  function chronological(messages) { return [...messages].sort((left, right) => new Date(left.created_at || 0) - new Date(right.created_at || 0)); }
11
+ function replyToId(message) { return message.reply_to_id ?? message.reply_to; }
11
12
  // A still-pending optimistic row (chunk 4) has a fake `local-<requestId>` id
12
13
  // and no durable server row yet — mounting ReadTicks against it would fire a
13
14
  // getMessageReadStatus REST call for an id the server has never heard of.
@@ -27,27 +28,57 @@ function canShowReadTicks(message, user) {
27
28
  * `getMessageReadStatus` REST calls a rendered timeline fires — one per own
28
29
  * message shown, not one per message in the conversation.
29
30
  */
30
- export function Thread({ conversationId, onReplyTargetChange }) {
31
+ export function Thread({ conversationId, onReplyTargetChange, canModerateMessages = false, onAnnouncementLink }) {
31
32
  const { t } = useTranslation();
32
33
  const { user } = useContext(AuthContext) || {};
33
- const { cache, loadMoreMessages, loadThreadReplies } = useMessaging();
34
+ const { cache, loadMoreMessages, loadThreadReplies, markConversationRead, markThreadRead } = useMessaging();
34
35
  const scrollRef = useRef(null);
36
+ const wasNearBottomRef = useRef(true);
37
+ const messageCountRef = useRef(0);
35
38
  const [loadingOlder, setLoadingOlder] = useState(false);
36
39
  const [error, setError] = useState(null);
37
40
  const [replyingTo, setReplyingTo] = useState(null);
38
41
  const [openThreads, setOpenThreads] = useState({});
39
42
  const conversation = cache.conversations[conversationId];
40
- const roots = useMemo(() => chronological(Object.values(cache.messages).filter((message) => message.conversation_id === conversationId && !message.reply_to)), [cache.messages, conversationId]);
43
+ const conversationMessages = useMemo(() => chronological(Object.values(cache.messages).filter((message) => message.conversation_id === conversationId)), [cache.messages, conversationId]);
44
+ const roots = useMemo(() => chronological(Object.values(cache.messages).filter((message) => message.conversation_id === conversationId && !replyToId(message))), [cache.messages, conversationId]);
45
+
46
+ useEffect(() => { if (conversationId != null) markConversationRead(conversationId).catch(() => {}); }, [conversationId, markConversationRead]);
47
+ useEffect(() => { wasNearBottomRef.current = true; messageCountRef.current = 0; }, [conversationId]);
48
+ useEffect(() => {
49
+ if (replyingTo && cache.messages[replyingTo.id]?.deleted_at) {
50
+ setReplyingTo(null);
51
+ onReplyTargetChange?.(null);
52
+ }
53
+ }, [cache.messages, onReplyTargetChange, replyingTo]);
54
+ useEffect(() => {
55
+ const isNewMessage = conversationMessages.length > messageCountRef.current;
56
+ messageCountRef.current = conversationMessages.length;
57
+ if (!isNewMessage || !wasNearBottomRef.current || !scrollRef.current) return;
58
+ const timeline = scrollRef.current;
59
+ if (typeof timeline.scrollTo === 'function') timeline.scrollTo({ top: timeline.scrollHeight, behavior: 'smooth' });
60
+ else timeline.scrollTop = timeline.scrollHeight;
61
+ }, [conversationMessages]);
41
62
 
42
63
  const loadOlder = useCallback(async () => {
43
64
  setLoadingOlder(true); setError(null);
44
65
  try { await loadMoreMessages(conversationId); } catch { setError(t('MessagingThread.LOAD_ERROR')); } finally { setLoadingOlder(false); }
45
66
  }, [conversationId, loadMoreMessages, t]);
46
- const onScroll = () => { if (scrollRef.current?.scrollTop === 0 && cache.cursors.messages[conversationId] && !loadingOlder) loadOlder(); };
67
+ const onScroll = () => {
68
+ const timeline = scrollRef.current;
69
+ if (timeline) wasNearBottomRef.current = timeline.scrollHeight - timeline.scrollTop - timeline.clientHeight < 80;
70
+ if (timeline?.scrollTop === 0 && cache.cursors.messages[conversationId] && !loadingOlder) loadOlder();
71
+ };
47
72
  const reply = (message) => { setReplyingTo(message); onReplyTargetChange?.(message); };
73
+ const jumpToMessage = (messageId) => {
74
+ const target = [...(scrollRef.current?.querySelectorAll('[data-message-id]') || [])]
75
+ .find((element) => String(element.dataset.messageId) === String(messageId));
76
+ target?.scrollIntoView?.({ block: 'center', behavior: 'smooth' });
77
+ };
48
78
  const toggleReplies = async (root) => {
49
79
  if (!openThreads[root.id]) {
50
80
  try { await loadThreadReplies(root.id); } catch { setError(t('MessagingThread.REPLIES_ERROR')); return; }
81
+ markThreadRead(root.id).catch(() => {});
51
82
  }
52
83
  setOpenThreads((current) => ({ ...current, [root.id]: !current[root.id] }));
53
84
  };
@@ -59,10 +90,10 @@ export function Thread({ conversationId, onReplyTargetChange }) {
59
90
  {cache.cursors.messages[conversationId] && <Box textAlign="center" py={1}><Button onClick={loadOlder} disabled={loadingOlder}>{loadingOlder ? <CircularProgress size={18} /> : t('MessagingThread.LOAD_OLDER')}</Button></Box>}
60
91
  {!roots.length && <Typography color="text.secondary" textAlign="center" py={4}>{t('MessagingThread.EMPTY')}</Typography>}
61
92
  <Stack spacing={1}>{roots.map((message) => {
62
- const replies = chronological(Object.values(cache.messages).filter((item) => item.reply_to === message.id));
63
- return <Stack key={message.id} spacing={0.75}><MessageBubble message={message} onReply={reply}>{canShowReadTicks(message, user) && <ReadTicks messageId={message.id} conversation={conversation} />}</MessageBubble>
93
+ const replies = chronological(Object.values(cache.messages).filter((item) => String(replyToId(item)) === String(message.id)));
94
+ return <Stack key={message.id} spacing={0.75}><MessageBubble message={message} conversation={conversation} onReply={reply} onJumpToMessage={jumpToMessage} onAnnouncementLink={onAnnouncementLink} canModerateMessages={canModerateMessages}>{canShowReadTicks(message, user) && <ReadTicks messageId={message.id} conversation={conversation} />}</MessageBubble>
64
95
  {(message.reply_count || replies.length) > 0 && <Button size="small" onClick={() => toggleReplies(message)}>{openThreads[message.id] ? t('MessagingThread.HIDE_REPLIES') : t('MessagingThread.SHOW_REPLIES', { count: message.reply_count || replies.length })}</Button>}
65
- {openThreads[message.id] && <Stack spacing={0.75} sx={{ pl: 3, borderLeft: 2, borderColor: 'divider' }}>{replies.map((item) => <MessageBubble key={item.id} message={item} replyTo={message} onReply={reply}>{canShowReadTicks(item, user) && <ReadTicks messageId={item.id} conversation={conversation} />}</MessageBubble>)}</Stack>}
96
+ {openThreads[message.id] && <Stack spacing={0.75} sx={{ pl: 3, borderLeft: 2, borderColor: 'divider' }}>{replies.map((item) => <MessageBubble key={item.id} message={item} replyTo={message} conversation={conversation} onReply={reply} onJumpToMessage={jumpToMessage} onAnnouncementLink={onAnnouncementLink} canModerateMessages={canModerateMessages}>{canShowReadTicks(item, user) && <ReadTicks messageId={item.id} conversation={conversation} />}</MessageBubble>)}</Stack>}
66
97
  <Divider /></Stack>;
67
98
  })}</Stack>
68
99
  </Box>
@@ -7,11 +7,12 @@ function data(response) {
7
7
  return response.data;
8
8
  }
9
9
 
10
- function requestConfig({ cursor, params, headers } = {}) {
10
+ function requestConfig({ cursor, params, headers, onUploadProgress } = {}) {
11
11
  return {
12
12
  ...(params ? { params } : {}),
13
13
  ...(cursor ? { params: { ...(params || {}), cursor } } : {}),
14
14
  ...(headers ? { headers } : {}),
15
+ ...(onUploadProgress ? { onUploadProgress } : {}),
15
16
  };
16
17
  }
17
18
 
@@ -29,7 +30,7 @@ export function archiveConversation(conversationId, archived) { return apiClient
29
30
  export function patchConversationPreferences(conversationId, patch) { return apiClient.post(`${conversationsUrl}${conversationId}/preferences/`, patch).then(data); }
30
31
  export function listMessages(conversationId, { cursor } = {}) { return apiClient.get(`${conversationsUrl}${conversationId}/messages/`, requestConfig({ cursor })).then(data); }
31
32
  export function createMessage(conversationId, payload, { idempotencyKey } = {}) { return apiClient.post(`${conversationsUrl}${conversationId}/messages/`, payload, requestConfig({ headers: idempotencyKey ? { 'Idempotency-Key': idempotencyKey } : undefined })).then(data); }
32
- export function uploadAttachments(conversationId, formData, { idempotencyKey } = {}) { return apiClient.post(`${conversationsUrl}${conversationId}/attachments/`, formData, requestConfig({ headers: idempotencyKey ? { 'Idempotency-Key': idempotencyKey } : undefined })).then(data); }
33
+ export function uploadAttachments(conversationId, formData, { idempotencyKey, onUploadProgress } = {}) { return apiClient.post(`${conversationsUrl}${conversationId}/attachments/`, formData, requestConfig({ headers: idempotencyKey ? { 'Idempotency-Key': idempotencyKey } : undefined, onUploadProgress })).then(data); }
33
34
  export function createPoll(conversationId, payload, { idempotencyKey } = {}) { return apiClient.post(`${conversationsUrl}${conversationId}/polls/`, payload, requestConfig({ headers: idempotencyKey ? { 'Idempotency-Key': idempotencyKey } : undefined })).then(data); }
34
35
  export function getMessage(messageId) { return apiClient.get(`${BASE_URL}messages/${messageId}/`).then(data); }
35
36
  export function patchMessage(messageId, patch) { return apiClient.patch(`${BASE_URL}messages/${messageId}/`, patch).then(data); }
@@ -0,0 +1,35 @@
1
+ const MAX_IMAGE_BYTES = 2 * 1024 * 1024;
2
+ const MAX_IMAGE_EDGE = 2560;
3
+
4
+ function imageDimensions(file) {
5
+ return new Promise((resolve, reject) => {
6
+ const url = URL.createObjectURL(file);
7
+ const image = new Image();
8
+ image.onload = () => { URL.revokeObjectURL(url); resolve(image); };
9
+ image.onerror = () => { URL.revokeObjectURL(url); reject(new Error('Unable to decode image')); };
10
+ image.src = url;
11
+ });
12
+ }
13
+
14
+ function canvasBlob(canvas) {
15
+ return new Promise((resolve) => canvas.toBlob(resolve, 'image/jpeg', 0.85));
16
+ }
17
+
18
+ /** Compresses large images opportunistically; callers always receive a sendable file. */
19
+ export async function compressImageForUpload(file) {
20
+ if (!file?.type?.startsWith('image/') || file.size <= MAX_IMAGE_BYTES) return file;
21
+ try {
22
+ const image = await imageDimensions(file);
23
+ const largestEdge = Math.max(image.naturalWidth || image.width, image.naturalHeight || image.height);
24
+ if (!largestEdge || largestEdge <= MAX_IMAGE_EDGE) return file;
25
+ const scale = MAX_IMAGE_EDGE / largestEdge;
26
+ const canvas = document.createElement('canvas');
27
+ canvas.width = Math.round((image.naturalWidth || image.width) * scale);
28
+ canvas.height = Math.round((image.naturalHeight || image.height) * scale);
29
+ canvas.getContext('2d')?.drawImage(image, 0, 0, canvas.width, canvas.height);
30
+ const blob = await canvasBlob(canvas);
31
+ return blob ? new File([blob], file.name.replace(/\.[^.]+$/, '') + '.jpg', { type: 'image/jpeg', lastModified: file.lastModified }) : file;
32
+ } catch {
33
+ return file;
34
+ }
35
+ }
@@ -61,7 +61,7 @@ describe('messaging Thread surfaces', () => {
61
61
  api.getReadStatus.mockResolvedValue({ all_read: false, delivered_count: 2, recipient_detail: [{ display_name: 'Visible recipient' }] });
62
62
  render(<MessagingProvider api={api} active={false}><ReadTicks messageId={2} conversation={{ id: 1, kind: 'group' }} /></MessagingProvider>);
63
63
  const tick = await screen.findByLabelText('MessagingReadTicks.DELIVERED:2');
64
- fireEvent.mouseOver(tick);
64
+ fireEvent.click(tick);
65
65
  await screen.findByText('Visible recipient');
66
66
  });
67
67
 
@@ -0,0 +1,109 @@
1
+ // @vitest-environment jsdom
2
+ import React from 'react';
3
+ import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
4
+ import { cleanup, fireEvent, render, screen, waitFor } from '@testing-library/react';
5
+
6
+ let frameHandler;
7
+ vi.mock('react-i18next', () => ({ useTranslation: () => ({ t: (key, options) => options?.count != null ? `${key}:${options.count}` : key, i18n: { language: 'en' } }) }));
8
+ vi.mock('../src/notifications/realtime', () => ({ useRealtime: () => ({ subscribe: (_envelope, handler) => { frameHandler = handler; return () => {}; }, onReconnect: () => () => {} }) }));
9
+
10
+ import { Composer } from '../src/messaging/Composer';
11
+ import { ConversationList } from '../src/messaging/ConversationList';
12
+ import { MessageBubble } from '../src/messaging/MessageBubble';
13
+ import { MessagingProvider } from '../src/messaging/MessagingProvider';
14
+ import { ReadTicks } from '../src/messaging/ReadTicks';
15
+ import { Thread } from '../src/messaging/Thread';
16
+
17
+ function makeApi(overrides = {}) {
18
+ return {
19
+ listConversations: vi.fn().mockResolvedValue({ results: [] }),
20
+ listMessages: vi.fn().mockResolvedValue({ results: [] }),
21
+ listThread: vi.fn().mockResolvedValue({ results: [] }),
22
+ getUnreadCount: vi.fn().mockResolvedValue({ unread_count: 0, by_conversation: {} }),
23
+ getReadStatus: vi.fn().mockResolvedValue({ all_read: false, delivered_count: 2 }),
24
+ markConversationRead: vi.fn().mockResolvedValue({}), markThreadRead: vi.fn().mockResolvedValue({}),
25
+ createPoll: vi.fn().mockResolvedValue({}),
26
+ ...overrides,
27
+ };
28
+ }
29
+
30
+ afterEach(() => { cleanup(); vi.useRealTimers(); vi.restoreAllMocks(); frameHandler = undefined; });
31
+
32
+ describe('MSG-3b chunk 6', () => {
33
+ it('renders localized relative list timestamps for now, minutes, hours, and days', async () => {
34
+ vi.spyOn(Date, 'now').mockReturnValue(new Date('2026-07-31T12:00:00Z').getTime());
35
+ const api = makeApi({ listConversations: vi.fn().mockResolvedValue({ results: [
36
+ { id: 1, title: 'Now', last_message_at: '2026-07-31T11:59:50Z' },
37
+ { id: 2, title: 'Minutes', last_message_at: '2026-07-31T11:55:00Z' },
38
+ { id: 3, title: 'Hours', last_message_at: '2026-07-31T09:00:00Z' },
39
+ { id: 4, title: 'Days', last_message_at: '2026-07-29T12:00:00Z' },
40
+ ] }) });
41
+ render(<MessagingProvider api={api}><ConversationList /></MessagingProvider>);
42
+ await screen.findByText('Now');
43
+ expect(screen.getByText('now')).toBeTruthy();
44
+ expect(screen.getByText(/5m/)).toBeTruthy();
45
+ expect(screen.getByText(/3h/)).toBeTruthy();
46
+ expect(screen.getByText(/2d/)).toBeTruthy();
47
+ });
48
+
49
+ it('auto-scrolls only for active-conversation messages when the reader was near the bottom', async () => {
50
+ const api = makeApi({ listConversations: vi.fn().mockResolvedValue({ results: [{ id: 1, kind: 'group' }] }), listMessages: vi.fn().mockResolvedValue({ results: [{ id: 1, conversation_id: 1, body: 'First' }] }) });
51
+ render(<MessagingProvider api={api} activeConversationId={1}><Thread conversationId={1} /></MessagingProvider>);
52
+ const timeline = await screen.findByLabelText('MessagingThread.TIMELINE');
53
+ Object.defineProperties(timeline, { scrollHeight: { configurable: true, value: 1000 }, clientHeight: { configurable: true, value: 300 }, scrollTop: { configurable: true, writable: true, value: 690 } });
54
+ timeline.scrollTo = vi.fn();
55
+ fireEvent.scroll(timeline);
56
+ frameHandler({ type: 'message', conversation_id: 1, message: { id: 2, body: 'Near bottom' } });
57
+ await waitFor(() => expect(timeline.scrollTo).toHaveBeenCalledTimes(1));
58
+ timeline.scrollTo.mockClear(); timeline.scrollTop = 100;
59
+ fireEvent.scroll(timeline);
60
+ frameHandler({ type: 'message', conversation_id: 1, message: { id: 3, body: 'Reading history' } });
61
+ await screen.findByText('Reading history');
62
+ expect(timeline.scrollTo).not.toHaveBeenCalled();
63
+ });
64
+
65
+ it('renders announcement deep links only with a host handler and supplies its exact target', () => {
66
+ const onAnnouncementLink = vi.fn();
67
+ const message = { id: 3, kind: 'announcement', title: 'Notice', body: 'Read this', link_target: '/events/7' };
68
+ const bubble = (props) => <MessagingProvider api={makeApi()} active={false}><MessageBubble {...props} /></MessagingProvider>;
69
+ const view = render(bubble({ message, onAnnouncementLink }));
70
+ fireEvent.click(screen.getByRole('button', { name: 'MessagingAnnouncement.OPEN_LINK' }));
71
+ expect(onAnnouncementLink).toHaveBeenCalledWith('/events/7');
72
+ view.rerender(bubble({ message }));
73
+ expect(screen.queryByRole('button', { name: 'MessagingAnnouncement.OPEN_LINK' })).toBeNull();
74
+ view.rerender(bubble({ message: { ...message, link_target: null }, onAnnouncementLink }));
75
+ expect(screen.queryByRole('button', { name: 'MessagingAnnouncement.OPEN_LINK' })).toBeNull();
76
+ });
77
+
78
+ it('shows senders for group messages and suppresses them for direct messages', () => {
79
+ const message = { id: 4, body: 'Hi', sender: { display_name: 'Ava' } };
80
+ const bubble = (conversation) => <MessagingProvider api={makeApi()} active={false}><MessageBubble message={message} conversation={conversation} /></MessagingProvider>;
81
+ const view = render(bubble({ kind: 'group' }));
82
+ expect(screen.getByText('Ava')).toBeTruthy();
83
+ view.rerender(bubble({ kind: 'direct' }));
84
+ expect(screen.queryByText('Ava')).toBeNull();
85
+ });
86
+
87
+ it('opens receipt recipients by click for non-direct conversations and never exposes direct details', async () => {
88
+ const api = makeApi({ getReadStatus: vi.fn().mockResolvedValue({ all_read: false, delivered_count: 2, recipient_detail: [{ display_name: 'Ava' }] }) });
89
+ const view = render(<MessagingProvider api={api} active={false}><ReadTicks messageId={7} conversation={{ kind: 'group' }} /></MessagingProvider>);
90
+ const detail = await screen.findByRole('button', { name: 'MessagingReadTicks.DELIVERED:2' });
91
+ fireEvent.click(detail);
92
+ expect(await screen.findByText('Ava')).toBeTruthy();
93
+ view.unmount();
94
+ render(<MessagingProvider api={api} active={false}><ReadTicks messageId={7} conversation={{ kind: 'direct' }} /></MessagingProvider>);
95
+ await screen.findByLabelText('MessagingReadTicks.DELIVERED:2');
96
+ expect(screen.queryByRole('button', { name: 'MessagingReadTicks.DELIVERED:2' })).toBeNull();
97
+ expect(screen.queryByText('Ava')).toBeNull();
98
+ });
99
+
100
+ it('caps poll composer options at ten without changing the existing two-option start', () => {
101
+ render(<MessagingProvider api={makeApi()} active={false}><Composer conversationId={1} /></MessagingProvider>);
102
+ fireEvent.click(screen.getByRole('button', { name: 'MessagingPoll.CREATE' }));
103
+ const addOption = screen.getByRole('button', { name: 'MessagingPoll.ADD_OPTION' });
104
+ expect(screen.getAllByLabelText(/MessagingPoll.OPTION/)).toHaveLength(2);
105
+ for (let count = 0; count < 8; count += 1) fireEvent.click(addOption);
106
+ expect(screen.getAllByLabelText(/MessagingPoll.OPTION/)).toHaveLength(10);
107
+ expect(addOption.disabled).toBe(true);
108
+ });
109
+ });
@@ -0,0 +1,110 @@
1
+ // @vitest-environment jsdom
2
+ import React from 'react';
3
+ import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
4
+ import { cleanup, fireEvent, render, screen, waitFor } from '@testing-library/react';
5
+
6
+ vi.mock('react-i18next', () => ({ useTranslation: () => ({ t: (key, options) => options?.name ? `${key}:${options.name}` : key }) }));
7
+ vi.mock('../src/notifications/realtime', () => ({ useRealtime: () => ({ subscribe: () => () => {}, onReconnect: () => () => {} }) }));
8
+
9
+ import { Composer } from '../src/messaging/Composer';
10
+ import { MessagingProvider } from '../src/messaging/MessagingProvider';
11
+
12
+ function makeApi() {
13
+ return {
14
+ listConversations: vi.fn().mockResolvedValue({ results: [] }), listMessages: vi.fn().mockResolvedValue({ results: [] }), getUnreadCount: vi.fn().mockResolvedValue({ unread_count: 0, by_conversation: {} }),
15
+ createMessage: vi.fn().mockResolvedValue({ id: 1, conversation_id: 7 }), uploadAttachments: vi.fn().mockResolvedValue({ id: 2, conversation_id: 7 }), getAttachment: vi.fn(), getAttachmentThumbnail: vi.fn(),
16
+ };
17
+ }
18
+ function renderComposer(api = makeApi(), conversationId = 7) {
19
+ return render(<MessagingProvider api={api} active={false}><Composer conversationId={conversationId} /></MessagingProvider>);
20
+ }
21
+ function attachmentInput() { return document.querySelector('input[type=file]'); }
22
+ function messageInput() { return screen.getByLabelText('MessagingComposer.MESSAGE'); }
23
+ function imageFile(name = 'photo.png') { return new File(['image'], name, { type: 'image/png' }); }
24
+
25
+ beforeEach(() => {
26
+ URL.createObjectURL = vi.fn((file) => `blob:${file.name}`);
27
+ URL.revokeObjectURL = vi.fn();
28
+ });
29
+ afterEach(() => { cleanup(); vi.restoreAllMocks(); });
30
+
31
+ describe('Composer chunk 5 behavior', () => {
32
+ it('sends on Enter and leaves Shift+Enter available for a newline', async () => {
33
+ const api = makeApi(); renderComposer(api);
34
+ fireEvent.change(messageInput(), { target: { value: 'first' } });
35
+ fireEvent.keyDown(messageInput(), { key: 'Enter', code: 'Enter' });
36
+ await waitFor(() => expect(api.createMessage).toHaveBeenCalledTimes(1));
37
+
38
+ fireEvent.change(messageInput(), { target: { value: 'first' } });
39
+ fireEvent.keyDown(messageInput(), { key: 'Enter', code: 'Enter', shiftKey: true });
40
+ fireEvent.change(messageInput(), { target: { value: 'first\nsecond' } });
41
+ expect(api.createMessage).toHaveBeenCalledTimes(1);
42
+ expect(messageInput().value).toBe('first\nsecond');
43
+ });
44
+
45
+ it('shows staged image previews and removes only the chosen file before upload', async () => {
46
+ const api = makeApi(); renderComposer(api);
47
+ const first = imageFile('first.png'); const second = imageFile('second.png');
48
+ fireEvent.change(attachmentInput(), { target: { files: [first, second] } });
49
+ expect(screen.getByRole('img', { name: 'first.png' })).toBeTruthy();
50
+ expect(screen.getByRole('img', { name: 'second.png' })).toBeTruthy();
51
+ fireEvent.click(screen.getByRole('button', { name: 'MessagingComposer.REMOVE_ATTACHMENT:first.png' }));
52
+ expect(screen.queryByRole('img', { name: 'first.png' })).toBeNull();
53
+ fireEvent.click(screen.getByRole('button', { name: 'MessagingComposer.SEND' }));
54
+ await waitFor(() => expect(api.uploadAttachments).toHaveBeenCalledTimes(1));
55
+ const sentFiles = Array.from(api.uploadAttachments.mock.calls[0][1].getAll('files[]'));
56
+ expect(sentFiles).toHaveLength(1);
57
+ expect(sentFiles[0].name).toBe('second.png');
58
+ });
59
+
60
+ it('shows real byte-percentage upload progress while an attachment upload is in flight', async () => {
61
+ let resolveUpload; const api = makeApi(); api.uploadAttachments.mockReturnValue(new Promise((resolve) => { resolveUpload = resolve; }));
62
+ renderComposer(api); fireEvent.change(attachmentInput(), { target: { files: [imageFile()] } });
63
+ fireEvent.click(screen.getByRole('button', { name: 'MessagingComposer.SEND' }));
64
+ expect(await screen.findByRole('status')).toBeTruthy();
65
+ const { onUploadProgress } = api.uploadAttachments.mock.calls[0][2];
66
+ onUploadProgress({ loaded: 25, total: 100 });
67
+ await waitFor(() => expect(screen.getAllByRole('progressbar')[0].getAttribute('aria-valuenow')).toBe('25'));
68
+ resolveUpload({ id: 2, conversation_id: 7 });
69
+ await waitFor(() => expect(screen.queryByRole('status')).toBeNull());
70
+ });
71
+
72
+ it('compresses oversized images before adding them to FormData and falls back silently if compression fails', async () => {
73
+ const largeImage = new File([new Uint8Array(2 * 1024 * 1024 + 1)], 'large.png', { type: 'image/png' });
74
+ const api = makeApi();
75
+ class SuccessfulImage { naturalWidth = 5120; naturalHeight = 2560; set src(_) { this.onload(); } }
76
+ vi.stubGlobal('Image', SuccessfulImage);
77
+ vi.spyOn(HTMLCanvasElement.prototype, 'getContext').mockReturnValue({ drawImage: vi.fn() });
78
+ vi.spyOn(HTMLCanvasElement.prototype, 'toBlob').mockImplementation((callback) => callback(new Blob(['compressed'], { type: 'image/jpeg' })));
79
+ renderComposer(api); fireEvent.change(attachmentInput(), { target: { files: [largeImage] } }); fireEvent.click(screen.getByRole('button', { name: 'MessagingComposer.SEND' }));
80
+ await waitFor(() => expect(api.uploadAttachments).toHaveBeenCalledTimes(1));
81
+ const compressed = api.uploadAttachments.mock.calls[0][1].get('files[]');
82
+ expect(compressed.name).toBe('large.jpg'); expect(compressed.type).toBe('image/jpeg');
83
+
84
+ cleanup(); const fallbackApi = makeApi(); class FailingImage { set src(_) { this.onerror(); } }
85
+ vi.stubGlobal('Image', FailingImage);
86
+ renderComposer(fallbackApi); fireEvent.change(attachmentInput(), { target: { files: [largeImage] } }); fireEvent.click(screen.getByRole('button', { name: 'MessagingComposer.SEND' }));
87
+ await waitFor(() => expect(fallbackApi.uploadAttachments).toHaveBeenCalledTimes(1));
88
+ const fallback = fallbackApi.uploadAttachments.mock.calls[0][1].get('files[]');
89
+ expect(fallback.name).toBe('large.png'); expect(fallback.type).toBe('image/png');
90
+ });
91
+
92
+ it('inserts the selected emoji at the current cursor position', async () => {
93
+ renderComposer(); fireEvent.change(messageInput(), { target: { value: 'abCD' } });
94
+ messageInput().setSelectionRange(2, 2);
95
+ fireEvent.click(screen.getByRole('button', { name: 'MessagingComposer.ADD_EMOJI' }));
96
+ fireEvent.click(await screen.findByRole('menuitem', { name: '👍' }));
97
+ await waitFor(() => expect(messageInput().value).toBe('ab👍CD'));
98
+ });
99
+
100
+ it('disables poll composing while an image is staged and resets drafts for a new conversation', async () => {
101
+ const api = makeApi(); const view = renderComposer(api, 7);
102
+ fireEvent.change(attachmentInput(), { target: { files: [imageFile()] } });
103
+ expect(screen.getByRole('button', { name: 'MessagingPoll.CREATE' }).disabled).toBe(true);
104
+ fireEvent.click(screen.getByRole('button', { name: 'MessagingComposer.REMOVE_ATTACHMENT:photo.png' }));
105
+ expect(screen.getByRole('button', { name: 'MessagingPoll.CREATE' }).disabled).toBe(false);
106
+ fireEvent.change(messageInput(), { target: { value: 'unsent draft' } });
107
+ view.rerender(<MessagingProvider api={api} active={false}><Composer conversationId={8} /></MessagingProvider>);
108
+ await waitFor(() => expect(messageInput().value).toBe(''));
109
+ });
110
+ });
@@ -0,0 +1,42 @@
1
+ import fs from 'node:fs';
2
+ import path from 'node:path';
3
+ import { describe, expect, it } from 'vitest';
4
+
5
+ const root = path.resolve(import.meta.dirname, '..');
6
+ const messagingRoot = path.join(root, 'src', 'messaging');
7
+
8
+ // These are deliberately named and time-bounded. Chunks 3 and 4 must remove
9
+ // their entries when they wire their existing adapters; the final three are
10
+ // retained only if their stated product rationale still holds at WO close.
11
+ const API_EXEMPTIONS = {
12
+ createManagedConversation: 'No managed-conversation launcher is in the current public surface.',
13
+ createObjectThreadConversation: 'Object-thread creation is host-owned routing/scope work.',
14
+ getMessage: 'Timeline data is loaded through listMessages and listThread.',
15
+ };
16
+ const FRAME_TYPES = ['conversation_upsert', 'conversation_archived', 'message', 'message_edited', 'message_deleted', 'attachment_ready', 'reaction', 'poll_updated', 'delivered', 'read_state', 'thread_read_state', 'participant_changed'];
17
+ const FRAME_EXEMPTIONS = {
18
+ attachment_ready: 'Reserved by the design and deliberately unemitted.',
19
+ delivered: 'Receipt synchronisation is outside MSG-3b.',
20
+ read_state: 'Receipt synchronisation is outside MSG-3b.',
21
+ thread_read_state: 'Receipt synchronisation is outside MSG-3b.',
22
+ };
23
+
24
+ describe('messaging adapter and realtime contract conformance', () => {
25
+ it('has a caller for every API export unless it has an explicit rationale', () => {
26
+ const apiSource = fs.readFileSync(path.join(messagingRoot, 'api.js'), 'utf8');
27
+ const source = fs.readdirSync(messagingRoot).filter((file) => file.endsWith('.jsx')).map((file) => fs.readFileSync(path.join(messagingRoot, file), 'utf8')).join('\n');
28
+ const exports = [...apiSource.matchAll(/export function (\w+)\(/g)].map((match) => match[1]);
29
+ // Attachment readers are intentionally passed through the provider to the
30
+ // standalone AttachmentList, where they are called. All other adapters
31
+ // are invoked by the provider/component that owns the action.
32
+ const unwired = exports.filter((name) => !new RegExp(`api\\.${name}\\(|${name}: api\\.${name}`).test(source) && !API_EXEMPTIONS[name]);
33
+ expect(unwired).toEqual([]);
34
+ });
35
+
36
+ it('handles only design-listed frames and accounts for intentionally unhandled frames', () => {
37
+ const provider = fs.readFileSync(path.join(messagingRoot, 'MessagingProvider.jsx'), 'utf8');
38
+ const handled = [...provider.matchAll(/frame\.type === '([^']+)'/g)].map((match) => match[1]);
39
+ expect(handled.every((type) => FRAME_TYPES.includes(type))).toBe(true);
40
+ expect(FRAME_TYPES.filter((type) => !handled.includes(type) && !FRAME_EXEMPTIONS[type])).toEqual([]);
41
+ });
42
+ });
@@ -0,0 +1,78 @@
1
+ // @vitest-environment jsdom
2
+ import React from 'react';
3
+ import { afterEach, describe, expect, it, vi } from 'vitest';
4
+ import { cleanup, fireEvent, render, screen, waitFor } from '@testing-library/react';
5
+
6
+ vi.mock('react-i18next', () => ({ useTranslation: () => ({ t: (key) => key }) }));
7
+ vi.mock('../src/notifications/realtime', () => ({ useRealtime: () => ({ subscribe: () => () => {}, onReconnect: () => () => {} }) }));
8
+
9
+ import { DirectMessageLauncher } from '../src/index';
10
+ import { MessagingProvider } from '../src/messaging/MessagingProvider';
11
+
12
+ const candidates = [{ id: 2, display_name: 'Alex' }, { id: 3, display_name: 'Sam' }];
13
+
14
+ function renderLauncher({ api = { createDirectConversation: vi.fn().mockResolvedValue({ id: 41, kind: 'direct' }) }, ...props } = {}) {
15
+ const onOpen = props.onOpen || vi.fn();
16
+ render(<MessagingProvider api={api} active={false}><DirectMessageLauncher candidates={candidates} onOpen={onOpen} {...props} /></MessagingProvider>);
17
+ return { api, onOpen };
18
+ }
19
+ function openPicker() { fireEvent.click(screen.getByRole('button', { name: 'MessagingDirect.LAUNCH' })); }
20
+
21
+ afterEach(() => {
22
+ cleanup();
23
+ window.matchMedia = undefined;
24
+ });
25
+
26
+ describe('DirectMessageLauncher', () => {
27
+ it('opens a picker over host-supplied candidates and renders a distinct empty state', () => {
28
+ renderLauncher();
29
+ openPicker();
30
+ expect(screen.getByRole('dialog')).toBeTruthy();
31
+ expect(screen.getByText('Alex')).toBeTruthy();
32
+ expect(screen.getByText('Sam')).toBeTruthy();
33
+
34
+ cleanup();
35
+ renderLauncher({ candidates: [] });
36
+ openPicker();
37
+ expect(screen.getByText('MessagingDirect.EMPTY')).toBeTruthy();
38
+ });
39
+
40
+ it('requires a recipient, disables while starting, then opens the created conversation with scope', async () => {
41
+ let resolveRequest;
42
+ const api = { createDirectConversation: vi.fn(() => new Promise((resolve) => { resolveRequest = resolve; })) };
43
+ const { onOpen } = renderLauncher({ api, scope: { kind: 'event', id: 8 } });
44
+ openPicker();
45
+ const start = screen.getByRole('button', { name: 'MessagingDirect.START' });
46
+ expect(start.disabled).toBe(true);
47
+ fireEvent.click(screen.getByText('Alex'));
48
+ expect(start.disabled).toBe(false);
49
+ fireEvent.click(start);
50
+ expect(api.createDirectConversation).toHaveBeenCalledWith({ target_user_id: 2, scope: { kind: 'event', id: 8 } });
51
+ expect(start.disabled).toBe(true);
52
+ expect(screen.getByRole('status')).toBeTruthy();
53
+ resolveRequest({ id: 41, kind: 'direct' });
54
+ await waitFor(() => expect(onOpen).toHaveBeenCalledWith({ id: 41, kind: 'direct' }));
55
+ await waitFor(() => expect(screen.queryByRole('dialog')).toBeNull());
56
+ });
57
+
58
+ it('surfaces a policy rejection without closing the picker', async () => {
59
+ const api = { createDirectConversation: vi.fn().mockRejectedValue({ response: { data: { detail: 'Direct messages are restricted.' } } }) };
60
+ renderLauncher({ api });
61
+ openPicker();
62
+ fireEvent.click(screen.getByText('Alex'));
63
+ fireEvent.click(screen.getByRole('button', { name: 'MessagingDirect.START' }));
64
+ expect((await screen.findByRole('alert')).textContent).toContain('Direct messages are restricted.');
65
+ expect(screen.getByRole('dialog')).toBeTruthy();
66
+ });
67
+
68
+ it('uses a fullscreen dialog at the small-viewport media query', () => {
69
+ window.matchMedia = vi.fn().mockImplementation(() => ({
70
+ matches: true,
71
+ addEventListener: vi.fn(),
72
+ removeEventListener: vi.fn(),
73
+ }));
74
+ renderLauncher();
75
+ openPicker();
76
+ expect(document.querySelector('.MuiDialog-paperFullScreen')).not.toBeNull();
77
+ });
78
+ });
@@ -1,5 +1,5 @@
1
1
  import { describe, expect, it } from 'vitest';
2
- import { AttachmentList, Composer, ConversationLaunchers, ConversationList, MessageBubble, MessagingProvider, MessagingScopeConfig, messagingTranslations, PollCard, ReactionBar, ReadTicks, Thread, useMessaging } from '../src/index';
2
+ import { AttachmentList, Composer, ConversationLaunchers, ConversationList, DirectMessageLauncher, MessageBubble, MessagingProvider, MessagingScopeConfig, messagingTranslations, PollCard, ReactionBar, ReadTicks, Thread, useMessaging } from '../src/index';
3
3
 
4
4
  describe('messaging barrel exports', () => {
5
5
  it('exports the provider, hook, standalone list and launchers', () => {
@@ -7,6 +7,7 @@ describe('messaging barrel exports', () => {
7
7
  expect(useMessaging).toBeDefined();
8
8
  expect(ConversationList).toBeDefined();
9
9
  expect(ConversationLaunchers).toBeDefined();
10
+ expect(DirectMessageLauncher).toBeDefined();
10
11
  expect(Thread).toBeDefined();
11
12
  expect(MessageBubble).toBeDefined();
12
13
  expect(ReadTicks).toBeDefined();
@@ -0,0 +1,93 @@
1
+ // @vitest-environment jsdom
2
+ import React from 'react';
3
+ import { afterEach, describe, expect, it, vi } from 'vitest';
4
+ import { cleanup, fireEvent, render, screen, waitFor } from '@testing-library/react';
5
+
6
+ vi.mock('react-i18next', () => ({ useTranslation: () => ({ t: (key, options) => options?.message ? `${key}:${options.message}` : key }) }));
7
+ vi.mock('../src/notifications/realtime', () => ({ useRealtime: () => ({ subscribe: () => () => {}, onReconnect: () => () => {} }) }));
8
+
9
+ import { AuthContext } from '../src/auth/AuthContext';
10
+ import { Composer } from '../src/messaging/Composer';
11
+ import { MessageBubble } from '../src/messaging/MessageBubble';
12
+ import { MessagingProvider } from '../src/messaging/MessagingProvider';
13
+
14
+ function makeApi() {
15
+ return {
16
+ patchMessage: vi.fn().mockImplementation(async (id, patch) => ({ id, ...patch, edited_at: '2026-07-31T10:01:00Z' })),
17
+ deleteMessage: vi.fn().mockImplementation(async (id) => ({ id, deleted_at: '2026-07-31T10:01:00Z' })),
18
+ };
19
+ }
20
+ function renderBubble(message, { userId = 1, canModerateMessages = false, api = makeApi() } = {}) {
21
+ const result = render(<AuthContext.Provider value={{ user: { id: userId } }}><MessagingProvider api={api} active={false}><MessageBubble message={message} canModerateMessages={canModerateMessages} /></MessagingProvider></AuthContext.Provider>);
22
+ return { ...result, api };
23
+ }
24
+ function openActions() { fireEvent.click(screen.getByRole('button', { name: 'MessagingActions.MENU' })); }
25
+
26
+ afterEach(cleanup);
27
+
28
+ describe('message actions', () => {
29
+ it('gates the menu to the author or host-supplied moderator capability, never deleted or poll messages', () => {
30
+ const mine = { id: 1, body: 'Mine', sender: { id: 1 } };
31
+ const theirs = { id: 2, body: 'Theirs', sender: { id: 2 } };
32
+ const { rerender } = renderBubble(mine);
33
+ expect(screen.getByRole('button', { name: 'MessagingActions.MENU' })).toBeTruthy();
34
+ rerender(<AuthContext.Provider value={{ user: { id: 1 } }}><MessagingProvider api={makeApi()} active={false}><MessageBubble message={theirs} /></MessagingProvider></AuthContext.Provider>);
35
+ expect(screen.queryByRole('button', { name: 'MessagingActions.MENU' })).toBeNull();
36
+ rerender(<AuthContext.Provider value={{ user: { id: 1 } }}><MessagingProvider api={makeApi()} active={false}><MessageBubble message={theirs} canModerateMessages /></MessagingProvider></AuthContext.Provider>);
37
+ expect(screen.getByRole('button', { name: 'MessagingActions.MENU' })).toBeTruthy();
38
+ rerender(<AuthContext.Provider value={{ user: { id: 1 } }}><MessagingProvider api={makeApi()} active={false}><MessageBubble message={{ ...mine, deleted_at: '2026-07-31T10:00:00Z' }} /></MessagingProvider></AuthContext.Provider>);
39
+ expect(screen.queryByRole('button', { name: 'MessagingActions.MENU' })).toBeNull();
40
+ rerender(<AuthContext.Provider value={{ user: { id: 1 } }}><MessagingProvider api={makeApi()} active={false}><MessageBubble message={{ ...mine, kind: 'poll' }} /></MessagingProvider></AuthContext.Provider>);
41
+ expect(screen.queryByRole('button', { name: 'MessagingActions.MENU' })).toBeNull();
42
+ });
43
+
44
+ it('edits an author message through the API and auto-cancels when deleted elsewhere', async () => {
45
+ const api = makeApi();
46
+ const message = { id: 4, body: 'Before', sender: { id: 1 } };
47
+ const { rerender } = renderBubble(message, { api });
48
+ openActions();
49
+ fireEvent.click(screen.getByRole('menuitem', { name: 'MessagingActions.EDIT' }));
50
+ const field = screen.getByRole('textbox', { name: 'MessagingActions.EDIT' });
51
+ fireEvent.change(field, { target: { value: 'After' } });
52
+ fireEvent.click(screen.getByRole('button', { name: 'MessagingActions.SAVE' }));
53
+ await waitFor(() => expect(api.patchMessage).toHaveBeenCalledWith(4, { body: 'After' }));
54
+ openActions();
55
+ fireEvent.click(screen.getByRole('menuitem', { name: 'MessagingActions.EDIT' }));
56
+ expect(screen.getByRole('textbox', { name: 'MessagingActions.EDIT' })).toBeTruthy();
57
+ rerender(<AuthContext.Provider value={{ user: { id: 1 } }}><MessagingProvider api={api} active={false}><MessageBubble message={{ ...message, deleted_at: '2026-07-31T10:02:00Z' }} /></MessagingProvider></AuthContext.Provider>);
58
+ await waitFor(() => expect(screen.queryByRole('textbox', { name: 'MessagingActions.EDIT' })).toBeNull());
59
+ });
60
+
61
+ it('requires confirmation before author or moderator deletion reaches the API', async () => {
62
+ const api = makeApi();
63
+ const { rerender } = renderBubble({ id: 5, body: 'Delete me', sender: { id: 1 } }, { api });
64
+ openActions();
65
+ fireEvent.click(screen.getByRole('menuitem', { name: 'MessagingActions.DELETE' }));
66
+ expect(api.deleteMessage).not.toHaveBeenCalled();
67
+ fireEvent.click(screen.getByRole('button', { name: 'MessagingActions.DELETE' }));
68
+ await waitFor(() => expect(api.deleteMessage).toHaveBeenCalledWith(5));
69
+ await waitFor(() => expect(screen.queryByRole('dialog')).toBeNull());
70
+ const moderatorApi = makeApi();
71
+ rerender(<AuthContext.Provider value={{ user: { id: 1 } }}><MessagingProvider api={moderatorApi} active={false}><MessageBubble message={{ id: 6, body: 'Moderator delete', sender: { id: 2 } }} canModerateMessages /></MessagingProvider></AuthContext.Provider>);
72
+ openActions();
73
+ fireEvent.click(screen.getByRole('menuitem', { name: 'MessagingActions.DELETE' }));
74
+ fireEvent.click(screen.getByRole('button', { name: 'MessagingActions.DELETE' }));
75
+ await waitFor(() => expect(moderatorApi.deleteMessage).toHaveBeenCalledWith(6));
76
+ });
77
+
78
+ it('clears a deleted reply target and copies the exact message body', async () => {
79
+ const onReplyTargetChange = vi.fn();
80
+ const replyTarget = { id: 7, body: 'Reply source', sender: { display_name: 'Ava' } };
81
+ const { rerender } = render(<MessagingProvider api={makeApi()} active={false}><Composer conversationId={1} replyTarget={replyTarget} onReplyTargetChange={onReplyTargetChange} /></MessagingProvider>);
82
+ onReplyTargetChange.mockClear();
83
+ rerender(<MessagingProvider api={makeApi()} active={false}><Composer conversationId={1} replyTarget={{ ...replyTarget, deleted_at: '2026-07-31T10:00:00Z' }} onReplyTargetChange={onReplyTargetChange} /></MessagingProvider>);
84
+ await waitFor(() => expect(onReplyTargetChange).toHaveBeenCalledWith(null));
85
+
86
+ const writeText = vi.fn().mockResolvedValue(undefined);
87
+ Object.defineProperty(navigator, 'clipboard', { configurable: true, value: { writeText } });
88
+ renderBubble({ id: 8, body: 'Copy exactly this', sender: { id: 1 } });
89
+ openActions();
90
+ fireEvent.click(screen.getByRole('menuitem', { name: 'MessagingActions.COPY' }));
91
+ await waitFor(() => expect(writeText).toHaveBeenCalledWith('Copy exactly this'));
92
+ });
93
+ });