@remit/web-client 0.0.325 → 0.0.327

package/src/components/mail/message-body-error-banner-content.ts

@@ -1,87 +0,0 @@
-import {
-	BodyFetchError,
-	type BodyFetchReason,
-} from "@/hooks/useMessageBodyContent";
-
-export interface BannerContent {
-	title: string;
-	detail: string;
-}
-
-/**
- * Pure helpers for `MessageBodyErrorBanner`. Extracted so they can be
- * unit-tested with the Node test runner without importing React /
- * @aws-amplify/ui-react / lucide-react. The banner component composes these
- * with the React surface.
- *
- * The variant copy lives here so a glance at one file shows every banner
- * variant the SPA can render — issue #401 / postmortem #394.
- */
-
-/**
- * Map a `BodyFetchReason` to user-facing banner copy.
- *
- * - `auth` — Lambda@Edge denied the JWT (missing/invalid/expired) or the
- *            tenant didn't match. The user needs to sign in again.
- * - `body-missing` — CloudFront/S3 returned 403/404 without the edge reason
- *            header. The OAC bucket policy only grants `s3:GetObject` so a
- *            missing object surfaces as 403. Either an in-flight sync failure
- *            left the message stuck, or the body part was never written.
- * - `content-type-mismatch` / `spa-shell-leak` — defensive guards; the user
- *            sees a safer "couldn't load" message and is asked to report it.
- * - `generic` — anything else; surfaces the underlying error message so the
- *            user has something to attach to a support ticket.
- */
-export const contentForReason = (
-	reason: BodyFetchReason,
-	fallback: string,
-): BannerContent => {
-	switch (reason) {
-		case "auth":
-			return {
-				title: "Your session expired",
-				detail:
-					"Sign in again to reload the message body. Other parts of the app may also stop responding until you do.",
-			};
-		case "body-missing":
-			return {
-				title: "Message body is missing in storage",
-				detail:
-					"A sync failure may have left this message without its body bytes. Contact support, or run the reconcile script to refetch from the upstream IMAP server.",
-			};
-		case "content-type-mismatch":
-			return {
-				title: "Couldn't load message body",
-				detail:
-					"The server returned an unexpected content type for this part. The message body has not been rendered to keep you safe.",
-			};
-		case "spa-shell-leak":
-			return {
-				title: "Couldn't load message body",
-				detail:
-					"The server returned the app shell instead of the message body. This is an infrastructure bug — please report it.",
-			};
-		case "not-ready":
-			return {
-				title: "Message body is still syncing",
-				detail:
-					"This message's body hasn't finished downloading from the mail server yet. It should appear shortly — reopen the message to try again.",
-			};
-		default:
-			return {
-				title: "Couldn't load message body",
-				detail: fallback,
-			};
-	}
-};
-
-export const extractReason = (error: unknown): BodyFetchReason => {
-	if (error instanceof BodyFetchError) return error.reason;
-	return "generic";
-};
-
-export const extractFallbackDetail = (error: unknown): string => {
-	if (error instanceof Error) return error.message;
-	if (typeof error === "string") return error;
-	return "An unexpected error occurred while loading the message body.";
-};

package/src/components/mail/message-gone.stories.tsx

@@ -1,44 +0,0 @@
-import type { Meta, StoryObj } from "@storybook/react-vite";
-import { HttpResponse, http } from "msw";
-import { expect, within } from "storybook/test";
-import { AppStory } from "@/mocks/story-frame/AppStory";
-import { composeWorld, messageUrl } from "@/mocks/story-frame/compose-world";
-import { mailHandlers } from "@/mocks/story-frame/mail-handlers";
-
-const world = composeWorld();
-
-const goneFromServer = http.get("/api/messages/msg-q3-planning", () =>
-	HttpResponse.json(
-		{
-			code: "message_gone_from_server",
-			message: "This message is no longer on the mail server.",
-			details: { messageId: "msg-q3-planning" },
-		},
-		{ status: 404 },
-	),
-);
-
-const meta = {
-	title: "Playground/Shipped/Mail/Message gone from server",
-	component: AppStory,
-	args: { url: messageUrl("q3-planning") },
-	parameters: {
-		layout: "fullscreen",
-		msw: { handlers: [goneFromServer, ...mailHandlers(world)] },
-	},
-} satisfies Meta<typeof AppStory>;
-export default meta;
-
-type Story = StoryObj<typeof meta>;
-
-export const MovedOrDeletedElsewhere: Story = {
-	play: async () => {
-		const page = within(document.body);
-		await page.findAllByText("Q3 planning");
-		await expect(
-			await page.findByText(
-				"This message is no longer on the mail server. It was moved or deleted in another mail app.",
-			),
-		).toBeVisible();
-	},
-};

package/src/components/mail/offer-times-reply.render.test.ts

@@ -1,557 +0,0 @@
-import assert from "node:assert/strict";
-import { afterEach, before, describe, it } from "node:test";
-import type {
-	RemitImapCalendarResponse,
-	RemitImapCalendarSuggestionResponse,
-	RemitImapDescribeMessageResponse,
-	RemitImapThreadMessageResponse,
-} from "@remit/api-http-client/types.gen.ts";
-import { type IntelligenceData, IntelligencePanel } from "@remit/ui";
-import {
-	type AnyRouter,
-	createMemoryHistory,
-	RouterContextProvider,
-} from "@tanstack/react-router";
-import { createElement } from "react";
-import { ComposeProvider } from "@/components/compose/ComposeProvider";
-import { useReplyWithText } from "@/components/compose/reply-with-times";
-import { useIntelligenceCalendar } from "@/hooks/useIntelligenceCalendar";
-import { noopTelemetry } from "@/lib/telemetry";
-import { createAppRouter } from "@/router";
-import { createDomHarness, type DomHarness } from "@/test-support/dom";
-import {
-	makeAccount,
-	makeMailbox,
-	makeThreadMessage,
-} from "@/test-support/fixtures";
-import { type HttpCall, type HttpMock, mockFetch } from "@/test-support/http";
-import { ConversationView } from "./ConversationView";
-
-const ACCOUNT_ID = "acc-1";
-const MAILBOX_ID = "mbx-inbox";
-const THREAD_ID = "thread-1";
-const MESSAGE_ID = "msg-1";
-const OTHER_ID = "msg-2";
-const DRAFT_ID = "ob-times";
-const CALENDAR = "11111111-1111-4111-8111-111111111111";
-const SUGGESTION = "22222222-2222-4222-8222-222222222222";
-const AUTOSAVE_DEBOUNCE_MS = 2000;
-
-const account = makeAccount({
-	accountId: ACCOUNT_ID,
-	email: "me@example.com",
-	username: "me@example.com",
-	smtpUsername: "me@example.com",
-});
-
-const inbox = makeMailbox({
-	mailboxId: MAILBOX_ID,
-	accountId: ACCOUNT_ID,
-	fullPath: "INBOX",
-});
-
-const thread: RemitImapThreadMessageResponse = makeThreadMessage({
-	messageId: MESSAGE_ID,
-	threadId: THREAD_ID,
-	mailboxId: MAILBOX_ID,
-	accountId: ACCOUNT_ID,
-	subject: "Quarterly review",
-	fromName: "Organizer",
-	fromEmail: "organizer@example.test",
-	isRead: true,
-});
-
-const otherThread: RemitImapThreadMessageResponse = {
-	...thread,
-	messageId: OTHER_ID,
-};
-
-const describeMessage: RemitImapDescribeMessageResponse = {
-	message: {
-		messageId: MESSAGE_ID,
-		mailboxId: MAILBOX_ID,
-		uid: 1,
-		rfc822Size: 512,
-		internalDate: 1_767_225_600_000,
-		status: "active",
-		syncStatus: "pending",
-		abandonedMutation: "none",
-	},
-	envelope: {
-		messageId: MESSAGE_ID,
-		date: 1_767_225_600_000,
-		subject: "Quarterly review",
-		messageIdValue: "<review-1@example.test>",
-		from: [
-			{
-				addressId: "addr-organizer",
-				displayName: "Organizer",
-				normalizedEmail: "organizer@example.test",
-				addressRole: "from",
-				addressOrder: 0,
-			},
-		],
-		to: [
-			{
-				addressId: "addr-me",
-				normalizedEmail: "me@example.com",
-				addressRole: "to",
-				addressOrder: 0,
-			},
-		],
-		cc: [],
-		bcc: [],
-		replyTo: [],
-		category: "uncategorized",
-		senderTrust: "unknown",
-	},
-	flags: ["\\Seen"],
-	bodyParts: [],
-	references: [],
-};
-
-const calendars = [
-	{
-		calendarId: CALENDAR,
-		accountConfigId: "cfg-1",
-		urlSegment: "default",
-		displayName: "Personal",
-		color: "Cal2",
-		componentSet: "VeventOnly",
-		source: "Default",
-		timezone: "Europe/Amsterdam",
-		syncSequence: 1,
-		subscriptionUrl: "",
-		subscriptionEnabled: false,
-		subscriptionCheckedAt: 0,
-		subscriptionFetchedAt: 0,
-		subscriptionError: "",
-		createdAt: 0,
-		updatedAt: 0,
-	},
-] satisfies RemitImapCalendarResponse[];
-
-const invitation: RemitImapCalendarSuggestionResponse = {
-	suggestionId: SUGGESTION,
-	accountConfigId: "cfg-1",
-	messageId: MESSAGE_ID,
-	bodyPartId: "part-1",
-	icalUid: "uid-1",
-	sequence: 0,
-	method: "Request",
-	source: "IcalendarPart",
-	state: "Pending",
-	summary: "Quarterly review",
-	dtStart: "2026-09-01T08:00:00+00:00",
-	dtEnd: "2026-09-01T09:00:00+00:00",
-	endsAtUtc: "2026-09-01T09:00:00Z",
-	allDay: false,
-	location: "",
-	organizer: "organizer@example.test",
-	zoneCertainty: "Explicit",
-	acceptedCalendarObjectId: "",
-	supersededByMessageId: "",
-	supersededByThreadId: "",
-	createdAt: 0,
-	updatedAt: 0,
-	answerOvertakenBy: "None",
-};
-
-const sender: IntelligenceData = {
-	sender: {
-		name: "Organizer",
-		email: "organizer@example.test",
-		trust: "wellknown",
-		firstSeenLabel: "Jan 2025",
-	},
-	authenticity: {
-		verdict: "aligned",
-		fromDomain: "example.test",
-		summary: "Signed by example.test.",
-	},
-	category: { value: "personal" },
-};
-
-let harness: DomHarness | undefined;
-let http: HttpMock | undefined;
-
-before(() => {
-	Object.defineProperty(Range.prototype, "getBoundingClientRect", {
-		value: () => ({
-			top: 0,
-			bottom: 0,
-			left: 0,
-			right: 0,
-			width: 0,
-			height: 0,
-			x: 0,
-			y: 0,
-		}),
-		configurable: true,
-	});
-});
-
-afterEach(() => {
-	harness?.close();
-	harness = undefined;
-	http?.restore();
-	http = undefined;
-});
-
-(globalThis as { self?: typeof globalThis }).self ??= globalThis;
-
-const MESSAGE_PATH = `/mail/${MAILBOX_ID}/${THREAD_ID}/${MESSAGE_ID}`;
-
-const describeOther: RemitImapDescribeMessageResponse = {
-	...describeMessage,
-	message: { ...describeMessage.message, messageId: OTHER_ID, uid: 2 },
-	envelope: {
-		...describeMessage.envelope,
-		messageId: OTHER_ID,
-		messageIdValue: "<review-2@example.test>",
-	},
-};
-
-const CalendarRail = ({
-	railThread,
-}: {
-	railThread: RemitImapThreadMessageResponse;
-}) => {
-	const calendar = useIntelligenceCalendar(railThread, useReplyWithText());
-	return createElement(IntelligencePanel, {
-		data: sender,
-		calendar: calendar.surface,
-		tab: calendar.tab,
-		onTabChange: calendar.onTabChange,
-	});
-};
-
-const Surface = ({
-	railThread,
-}: {
-	railThread: RemitImapThreadMessageResponse;
-}) =>
-	createElement(
-		ComposeProvider,
-		null,
-		createElement(ConversationView, {
-			threadId: THREAD_ID,
-			mailboxId: MAILBOX_ID,
-			subject: "Quarterly review",
-			selectedMessageId: MESSAGE_ID,
-		}),
-		createElement(CalendarRail, { railThread }),
-	);
-
-const savedDraft = {
-	outboxMessageId: DRAFT_ID,
-	accountId: ACCOUNT_ID,
-	fromAddress: "me@example.com",
-	toAddresses: ["organizer@example.test"],
-	ccAddresses: [],
-	bccAddresses: [],
-	attachments: [],
-	references: [],
-	subject: "Re: Quarterly review",
-	textBody: "See you then",
-	htmlBody: "<p>See you then</p>",
-	status: "draft",
-};
-
-const creates = (): HttpCall[] =>
-	(http?.calls ?? []).filter(
-		(call) => call.method === "POST" && call.path.endsWith("/outbox"),
-	);
-
-const draftWrites = (): HttpCall[] => [
-	...creates(),
-	...(http?.calls ?? []).filter(
-		(call) =>
-			call.method === "PATCH" && call.path.endsWith(`/outbox/${DRAFT_ID}`),
-	),
-];
-
-// An autosave still waiting out its debounce outlives the test that scheduled
-// it and lands on the next test's server, so a test that writes waits for it.
-const savedHolding = (mounted: DomHarness, text: string): Promise<void> =>
-	mounted.waitFor(
-		() =>
-			draftWrites().some(
-				(call) =>
-					(call.body as { textBody?: string }).textBody?.includes(text) ===
-					true,
-			),
-		`the draft to be saved holding ${JSON.stringify(text)}`,
-		AUTOSAVE_DEBOUNCE_MS + 3000,
-	);
-
-const mount = async (
-	href: string,
-	{
-		railThread = thread,
-		draftArrives = Promise.resolve(),
-	}: {
-		railThread?: RemitImapThreadMessageResponse;
-		draftArrives?: Promise<void>;
-	} = {},
-): Promise<{ mounted: DomHarness; router: AnyRouter; redraw: () => void }> => {
-	let saved: Record<string, unknown> = savedDraft;
-	http = mockFetch(async (call) => {
-		if (call.path.endsWith("/config")) return { accounts: [account] };
-		if (call.path.endsWith(`/accounts/${ACCOUNT_ID}/mailboxes`))
-			return { items: [inbox] };
-		if (call.path.endsWith(`/threads/${THREAD_ID}/messages`))
-			return { items: [thread, otherThread] };
-		if (call.path.endsWith(`/messages/${MESSAGE_ID}`)) return describeMessage;
-		if (call.path.endsWith(`/messages/${OTHER_ID}`)) return describeOther;
-		if (call.path.endsWith(`/messages/${OTHER_ID}/calendar-suggestions`))
-			return { items: [{ ...invitation, messageId: OTHER_ID }] };
-		if (call.path.endsWith("/calendars")) return { items: calendars };
-		if (call.path.endsWith(`/messages/${MESSAGE_ID}/calendar-suggestions`))
-			return { items: [invitation] };
-		if (call.method === "POST" && call.path.endsWith("/outbox")) {
-			saved = {
-				...savedDraft,
-				...(call.body as Record<string, unknown>),
-			};
-			return saved;
-		}
-		if (call.path.endsWith(`/outbox/${DRAFT_ID}`)) {
-			await draftArrives;
-			return saved;
-		}
-		return { items: [] };
-	});
-
-	const mounted = createDomHarness();
-	harness = mounted;
-	const router = createAppRouter(
-		mounted.queryClient,
-		noopTelemetry,
-		createMemoryHistory({ initialEntries: [href] }),
-	) as unknown as AnyRouter;
-	await router.load();
-	const redraw = () =>
-		mounted.renderApp(
-			createElement(RouterContextProvider, {
-				router,
-				// biome-ignore lint/correctness/noChildrenProp: RouterContextProvider types `children` as a required prop, which createElement's rest-argument form does not satisfy
-				children: createElement(Surface, { railThread }),
-			}),
-		);
-	redraw();
-	await mounted.waitFor(
-		() => mounted.text().includes("Offer other times"),
-		"the invitation card to be drawn",
-	);
-	return { mounted, router, redraw };
-};
-
-const drawnUntil = async (
-	mounted: DomHarness,
-	redraw: () => void,
-	predicate: () => boolean,
-	description: string,
-): Promise<void> => {
-	const deadline = Date.now() + 5000;
-	while (!predicate()) {
-		if (Date.now() > deadline)
-			throw new Error(`gave up waiting for ${description}`);
-		redraw();
-		await mounted.wait(50);
-	}
-};
-
-const composerText = (mounted: DomHarness): string =>
-	mounted.query("[data-testid=compose-body]")?.textContent ?? "";
-
-const plainBody = (mounted: DomHarness): HTMLTextAreaElement | null =>
-	mounted.query<HTMLTextAreaElement>("[data-testid=compose-body-plain]");
-
-const freeSlot = (mounted: DomHarness): HTMLElement | undefined =>
-	mounted
-		.queryAll("button[aria-pressed]")
-		.find((button) =>
-			/\d{2}:\d{2} – \d{2}:\d{2}/.test(button.textContent ?? ""),
-		);
-
-const pickAndReply = async (mounted: DomHarness): Promise<string> => {
-	mounted.click(mounted.byText("button", "Offer other times"));
-	await mounted.waitFor(
-		() => freeSlot(mounted) !== undefined,
-		"free half-hours to be offered",
-	);
-	const slot = freeSlot(mounted);
-	assert.ok(slot, "a free half-hour is offered");
-	const picked = (slot.textContent ?? "").trim();
-	mounted.click(slot);
-	await mounted.flush();
-	mounted.click(mounted.byText("button", "Reply with these times"));
-	return picked;
-};
-
-const seedInAddress = (router: AnyRouter): unknown =>
-	(router.state.location.search as Record<string, unknown>).body;
-
-describe("offering other times from an invitation", () => {
-	it("opens the reply with the picked times written into it", async () => {
-		const { mounted, router, redraw } = await mount(MESSAGE_PATH);
-
-		const picked = await pickAndReply(mounted);
-
-		await drawnUntil(
-			mounted,
-			redraw,
-			() => composerText(mounted).includes(picked),
-			"the picked time to be written into the reply",
-		);
-		assert.match(composerText(mounted), /September/);
-		assert.equal(router.state.location.pathname, `${MESSAGE_PATH}/reply`);
-
-		await mounted.wait(AUTOSAVE_DEBOUNCE_MS + 300);
-		await mounted.waitFor(
-			() => creates().length === 1,
-			"the reply to be saved as a draft",
-		);
-		const draft = creates()[0].body as { textBody?: string };
-		assert.ok(
-			draft.textBody?.includes(picked),
-			`the saved draft holds the picked time, got ${JSON.stringify(draft.textBody)}`,
-		);
-
-		await drawnUntil(
-			mounted,
-			redraw,
-			() => router.state.location.pathname.endsWith(`/reply/${DRAFT_ID}`),
-			"the address to adopt the draft",
-		);
-		assert.equal(
-			seedInAddress(router),
-			undefined,
-			"once the draft holds the times, the address no longer carries them",
-		);
-		await drawnUntil(
-			mounted,
-			redraw,
-			() => composerText(mounted).includes(picked),
-			"the composer to keep the times once the draft is adopted",
-		);
-	});
-
-	it("keeps what was typed when the times go into a reply already open", async () => {
-		const { mounted, router } = await mount(`${MESSAGE_PATH}/reply`);
-		await mounted.waitFor(
-			() => mounted.query("[data-testid=compose-mode-toggle]") !== null,
-			"the reply composer to open",
-			5000,
-		);
-		const toggle = mounted.query("[data-testid=compose-mode-toggle]");
-		assert.ok(toggle);
-		mounted.click(toggle);
-		await mounted.waitFor(
-			() => plainBody(mounted) !== null,
-			"the plain writing surface",
-		);
-		const typing = plainBody(mounted);
-		assert.ok(typing);
-		mounted.type(typing, "Hi");
-		await mounted.flush();
-
-		const picked = await pickAndReply(mounted);
-
-		await mounted.waitFor(
-			() => plainBody(mounted)?.value.includes(picked) === true,
-			"the picked time to go into the open reply",
-		);
-		assert.match(plainBody(mounted)?.value ?? "", /^Hi/);
-		assert.equal(router.state.location.pathname, `${MESSAGE_PATH}/reply`);
-		assert.equal(seedInAddress(router), undefined);
-		await savedHolding(mounted, picked);
-	});
-
-	it("keeps an open reply-all a reply-all and adds the times to it", async () => {
-		const href = `${MESSAGE_PATH}/reply-all/${DRAFT_ID}`;
-		const { mounted, router } = await mount(href);
-		await mounted.waitFor(
-			() => composerText(mounted).includes("See you then"),
-			"the saved reply-all to reopen",
-			5000,
-		);
-
-		const picked = await pickAndReply(mounted);
-
-		await mounted.waitFor(
-			() => composerText(mounted).includes(picked),
-			"the picked time to go into the open reply-all",
-		);
-		assert.ok(composerText(mounted).includes("See you then"));
-		assert.equal(router.state.location.pathname, href);
-		assert.equal(seedInAddress(router), undefined);
-		await savedHolding(mounted, picked);
-	});
-
-	it("adds the times to a forward of the same message already open", async () => {
-		const href = `${MESSAGE_PATH}/forward`;
-		const { mounted, router } = await mount(href);
-		await mounted.waitFor(
-			() => mounted.query("[data-testid=compose-body]") !== null,
-			"the forward to open",
-			5000,
-		);
-
-		const picked = await pickAndReply(mounted);
-
-		await mounted.waitFor(
-			() => composerText(mounted).includes(picked),
-			"the picked time to go into the open forward",
-		);
-		assert.equal(router.state.location.pathname, href);
-	});
-
-	it("opens a reply holding the times when the open reply answers another message", async () => {
-		const { mounted, router, redraw } = await mount(`${MESSAGE_PATH}/reply`, {
-			railThread: otherThread,
-		});
-		await mounted.waitFor(
-			() => mounted.query("[data-testid=compose-body]") !== null,
-			"the reply to the first message to open",
-			5000,
-		);
-
-		const picked = await pickAndReply(mounted);
-
-		await drawnUntil(
-			mounted,
-			redraw,
-			() => composerText(mounted).includes(picked),
-			"a reply to the other message holding the times",
-		);
-		assert.equal(
-			router.state.location.pathname,
-			`/mail/${MAILBOX_ID}/${THREAD_ID}/${OTHER_ID}/reply`,
-		);
-		await savedHolding(mounted, picked);
-	});
-
-	it("adds the times on top of a draft still loading when they were picked", async () => {
-		let deliver = () => {};
-		const draftArrives = new Promise<void>((resolve) => {
-			deliver = resolve;
-		});
-		const { mounted } = await mount(`${MESSAGE_PATH}/reply-all/${DRAFT_ID}`, {
-			draftArrives,
-		});
-
-		const picked = await pickAndReply(mounted);
-		await mounted.wait(100);
-		deliver();
-
-		await mounted.waitFor(
-			() =>
-				composerText(mounted).includes("See you then") &&
-				composerText(mounted).includes(picked),
-			"the loaded draft to hold the picked time",
-			5000,
-		);
-	});
-});