@remit/backend 0.0.136 → 0.0.137

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remit/backend",
-  "version": "0.0.136",
+  "version": "0.0.137",
   "description": "Remit Mail Inspector API backend",
   "license": "MIT",
   "author": "",

package/src/handlers/calendar-suggestion.test.ts

@@ -29,6 +29,7 @@ import {
 	assertSettleable,
 	CalendarSuggestionActionOperations,
 	CalendarSuggestionOperations,
+	MessageCalendarSuggestionOperations,
 	type MuteSenderDeps,
 	muteSender,
 	settleSuggestion,
@@ -58,6 +59,7 @@ const suggestion = (
 	zoneCertainty: "Explicit",
 	icalData: "BEGIN:VCALENDAR\r\nEND:VCALENDAR\r\n",
 	acceptedCalendarObjectId: "",
+	supersededByMessageId: "",
 	createdAt: 1,
 	updatedAt: 1,
 	...overrides,
@@ -308,12 +310,38 @@ const acceptSuggestion =
 	CalendarSuggestionActionOperations.CalendarSuggestionActionOperations_acceptCalendarSuggestion as Handler;
 const dismissSuggestion =
 	CalendarSuggestionActionOperations.CalendarSuggestionActionOperations_dismissCalendarSuggestion as Handler;
+const listMessageSuggestions =
+	MessageCalendarSuggestionOperations.MessageCalendarSuggestionOperations_listMessageCalendarSuggestions as Handler;
 
 interface Card {
 	suggestionId: string;
 	state: string;
+	supersededByMessageId: string;
+	supersededByThreadId: string;
 }
 
+const fileInThread = async (
+	accountConfigId: string,
+	messageId: string,
+	threadId: string,
+): Promise<void> => {
+	await client.threadMessage.create({
+		accountConfigId,
+		threadId,
+		messageId,
+		mailboxId: "mbx-inbox",
+		uid: 1,
+		referenceOrder: 0,
+		internalDate: 1,
+		sentDate: 1,
+		subject: "Invitation: Quarterly review",
+		isRead: false,
+		isDeleted: false,
+		hasAttachment: false,
+		hasStars: false,
+	});
+};
+
 let client: RemitClient;
 let cleanup: () => void;
 let mintedSubs = 0;
@@ -473,6 +501,74 @@ describe("GET /calendar-suggestions", () => {
 	});
 });
 
+describe("GET /messages/{messageId}/calendar-suggestions", () => {
+	test("names the conversation of the revision that retired a card", async () => {
+		const { accountConfigId, event } = anAccount();
+		const older = await putSuggestion(accountConfigId, "msg-revision-0");
+		await client.calendarSuggestion.supersedeIfPending(
+			accountConfigId,
+			older.suggestionId,
+			"msg-revision-1",
+		);
+		await fileInThread(accountConfigId, "msg-revision-1", "thread-revision-1");
+
+		const listed = (await listMessageSuggestions(
+			contextOf({ params: { messageId: "msg-revision-0" } }),
+			event,
+		)) as unknown as { items: Card[] };
+
+		assert.deepEqual(
+			listed.items.map((card) => [
+				card.state,
+				card.supersededByMessageId,
+				card.supersededByThreadId,
+			]),
+			[
+				[
+					CalendarSuggestionState.Superseded,
+					"msg-revision-1",
+					"thread-revision-1",
+				],
+			],
+		);
+	});
+
+	test("names no conversation when the newer message sits in none", async () => {
+		const { accountConfigId, event } = anAccount();
+		const older = await putSuggestion(accountConfigId, "msg-orphan-0");
+		await client.calendarSuggestion.supersedeIfPending(
+			accountConfigId,
+			older.suggestionId,
+			"msg-orphan-1",
+		);
+
+		const listed = (await listMessageSuggestions(
+			contextOf({ params: { messageId: "msg-orphan-0" } }),
+			event,
+		)) as unknown as { items: Card[] };
+
+		assert.equal(listed.items[0]?.supersededByThreadId, "");
+	});
+
+	test("names no conversation for a card nothing retired", async () => {
+		const { accountConfigId, event } = anAccount();
+		await putSuggestion(accountConfigId, "msg-current");
+
+		const listed = (await listMessageSuggestions(
+			contextOf({ params: { messageId: "msg-current" } }),
+			event,
+		)) as unknown as { items: Card[] };
+
+		assert.deepEqual(
+			listed.items.map((card) => [
+				card.supersededByMessageId,
+				card.supersededByThreadId,
+			]),
+			[["", ""]],
+		);
+	});
+});
+
 describe("POST /calendar-suggestions/{suggestionId}/accept", () => {
 	test("answers not-found for a calendar on another account, before writing anything", async () => {
 		const stranger = anAccount();

package/src/handlers/calendar-suggestion.ts

@@ -12,6 +12,7 @@ import {
 	type ICalendarSuggestionRepository,
 	type IEnvelopeRepository,
 	type IFilterRepository,
+	type IThreadMessageRepository,
 	isSenderMuted,
 } from "@remit/data-ports";
 import { BadRequestError } from "@remit/data-ports/errors";
@@ -45,9 +46,26 @@ export const toCalendarSuggestionResponse = (
 	item: CalendarSuggestionItem,
 ): CalendarSuggestionResponse => {
 	const { icalData: _icalData, ...response } = item;
-	return response;
+	return { ...response, supersededByThreadId: "" };
 };
 
+export const toCalendarSuggestionResponses = (
+	threads: Pick<IThreadMessageRepository, "findByMessageId">,
+	accountConfigId: string,
+	items: CalendarSuggestionItem[],
+): Promise<CalendarSuggestionResponse[]> =>
+	Promise.all(
+		items.map(async (item) => {
+			const response = toCalendarSuggestionResponse(item);
+			if (item.supersededByMessageId === "") return response;
+			const newer = await threads.findByMessageId(
+				accountConfigId,
+				item.supersededByMessageId,
+			);
+			return { ...response, supersededByThreadId: newer?.threadId ?? "" };
+		}),
+	);
+
 /**
  * The states a person can move a pending card into. `Superseded` is the
  * producer's alone — a revision retires a card, a person never does — and the
@@ -177,7 +195,11 @@ export const CalendarSuggestionOperations: Record<
 		);
 
 		return {
-			items: page.items.map(toCalendarSuggestionResponse),
+			items: await toCalendarSuggestionResponses(
+				client.threadMessage,
+				accountConfigId,
+				page.items,
+			),
 			continuationToken: page.continuationToken,
 		};
 	},
@@ -202,7 +224,11 @@ export const MessageCalendarSuggestionOperations: Record<
 		);
 
 		return {
-			items: items.map(toCalendarSuggestionResponse),
+			items: await toCalendarSuggestionResponses(
+				client.threadMessage,
+				accountConfigId,
+				items,
+			),
 			continuationToken: undefined,
 		};
 	},