@remit/web-client 0.0.321 → 0.0.323

package/src/hooks/useIntelligenceData.test.ts

@@ -97,21 +97,6 @@ describe("buildSenderIntel", () => {
 			const result = buildSenderIntel(thread, address);
 			assert.equal(result.inboundCount, 34);
 		});
-
-		test("surfaces replyCount from address", () => {
-			const thread = makeThread();
-			const address = makeAddress({ inboundCount: 34, replyCount: 12 });
-			const result = buildSenderIntel(thread, address);
-			assert.equal(result.replyCount, 12);
-		});
-
-		test("surfaces both counters together", () => {
-			const thread = makeThread();
-			const address = makeAddress({ inboundCount: 34, replyCount: 12 });
-			const result = buildSenderIntel(thread, address);
-			assert.equal(result.inboundCount, 34);
-			assert.equal(result.replyCount, 12);
-		});
 	});
 
 	describe("suppression — counters absent", () => {
@@ -123,13 +108,6 @@ describe("buildSenderIntel", () => {
 			assert.equal(result.inboundCount, undefined);
 		});
 
-		test("replyCount is undefined when address has no replyCount", () => {
-			const thread = makeThread();
-			const address = makeAddress({ inboundCount: 5 }); // no replyCount
-			const result = buildSenderIntel(thread, address);
-			assert.equal(result.replyCount, undefined);
-		});
-
 		test("both counters are undefined when address is undefined", () => {
 			const thread = makeThread();
 			const result = buildSenderIntel(thread, undefined);
@@ -173,12 +151,6 @@ describe("buildSenderIntel", () => {
 			assert.equal(result.name, "Unknown");
 		});
 
-		test("passes senderTrust through", () => {
-			const thread = makeThread({ senderTrust: "vip" });
-			const result = buildSenderIntel(thread, undefined);
-			assert.equal(result.trust, "vip");
-		});
-
 		test("firstSeenLabel is 'unknown' when no address", () => {
 			const thread = makeThread();
 			const result = buildSenderIntel(thread, undefined);
@@ -225,7 +197,7 @@ describe("buildSenderIntel", () => {
 describe("buildAuthenticityIntel", () => {
 	test("caution when there is no authenticity signal", () => {
 		const thread = makeThread({ fromEmail: "alice@example.com" });
-		const result = buildAuthenticityIntel(thread, 0);
+		const result = buildAuthenticityIntel(thread);
 		assert.equal(result.verdict, "caution");
 		assert.match(result.summary, /can't verify the sender/);
 	});
@@ -239,7 +211,7 @@ describe("buildAuthenticityIntel", () => {
 				dkimMismatch: false,
 			},
 		} as Partial<RemitImapThreadMessageResponse>);
-		const result = buildAuthenticityIntel(thread, 0);
+		const result = buildAuthenticityIntel(thread);
 		assert.equal(result.verdict, "aligned");
 	});
 
@@ -253,19 +225,15 @@ describe("buildAuthenticityIntel", () => {
 				dkimMismatch: true,
 			},
 		} as Partial<RemitImapThreadMessageResponse>);
-		const result = buildAuthenticityIntel(thread, 3);
+		const result = buildAuthenticityIntel(thread);
 		assert.equal(result.verdict, "mismatch");
 		assert.equal(result.addressUnreadable, undefined);
-		assert.equal(result.similarCount, 3);
 	});
 
 	test("no user-facing copy names the verification mechanism", () => {
 		const verdicts = [
-			buildAuthenticityIntel(makeThread(), 0),
-			buildAuthenticityIntel(
-				makeThread({ fromEmail: "x@no-signal.example" }),
-				0,
-			),
+			buildAuthenticityIntel(makeThread()),
+			buildAuthenticityIntel(makeThread({ fromEmail: "x@no-signal.example" })),
 			buildAuthenticityIntel(
 				makeThread({
 					fromEmail: "a@bank.example",
@@ -275,7 +243,6 @@ describe("buildAuthenticityIntel", () => {
 						dkimMismatch: true,
 					},
 				} as Partial<RemitImapThreadMessageResponse>),
-				0,
 			),
 		];
 		for (const v of verdicts) {
@@ -305,7 +272,7 @@ describe("buildAuthenticityIntel", () => {
 		} as Partial<RemitImapThreadMessageResponse>);
 
 		test("is never presented as verified", () => {
-			const result = buildAuthenticityIntel(infoMedics, 0);
+			const result = buildAuthenticityIntel(infoMedics);
 			assert.notEqual(result.verdict, "aligned");
 			assert.doesNotMatch(result.summary, /We verified/i);
 		});
@@ -315,7 +282,7 @@ describe("buildAuthenticityIntel", () => {
 		// signed by a relay or ESP infrastructure domain must not read as
 		// "the name looks nothing like <that other party>".
 		test("leads with the concern, naming the domain the name was actually compared to, and the link destination", () => {
-			const result = buildAuthenticityIntel(infoMedics, 0);
+			const result = buildAuthenticityIntel(infoMedics);
 			assert.equal(result.verdict, "caution");
 			assert.match(result.summary, /^The name it shows/);
 			assert.match(result.summary, /"InfoMedics"/);
@@ -338,7 +305,6 @@ describe("buildAuthenticityIntel", () => {
 						offDomainLinkDomains: [],
 					},
 				} as Partial<RemitImapThreadMessageResponse>),
-				0,
 			);
 			assert.equal(result.verdict, "caution");
 			assert.match(result.summary, /only looks like/);
@@ -357,7 +323,6 @@ describe("buildAuthenticityIntel", () => {
 						offDomainLinkDomains: [],
 					},
 				} as Partial<RemitImapThreadMessageResponse>),
-				0,
 			);
 			assert.equal(result.verdict, "caution");
 			assert.match(result.summary, /"matthijs@ischen\.nl"/);
@@ -377,7 +342,6 @@ describe("buildAuthenticityIntel", () => {
 						offDomainLinkDomains: [],
 					},
 				} as Partial<RemitImapThreadMessageResponse>),
-				0,
 			);
 			assert.equal(result.verdict, "aligned");
 		});
@@ -392,7 +356,6 @@ describe("buildAuthenticityIntel", () => {
 						dkimMismatch: false,
 					},
 				} as Partial<RemitImapThreadMessageResponse>),
-				0,
 			);
 			assert.equal(result.verdict, "aligned");
 		});
@@ -409,7 +372,6 @@ describe("buildAuthenticityIntel", () => {
 						offDomainLinkDomains: "elsewhere.example",
 					},
 				} as unknown as Partial<RemitImapThreadMessageResponse>),
-				0,
 			);
 			assert.equal(result.verdict, "aligned");
 		});
@@ -420,7 +382,7 @@ describe("buildAuthenticityIntel", () => {
 			const thread = makeThread({
 				fromEmail: "missing_mailbox@missing_domain",
 			});
-			const result = buildAuthenticityIntel(thread, 0);
+			const result = buildAuthenticityIntel(thread);
 			assert.equal(result.verdict, "mismatch");
 			assert.equal(result.addressUnreadable, true);
 		});
@@ -433,7 +395,7 @@ describe("buildAuthenticityIntel", () => {
 					dkimMismatch: false,
 				},
 			} as Partial<RemitImapThreadMessageResponse>);
-			const result = buildAuthenticityIntel(thread, 0);
+			const result = buildAuthenticityIntel(thread);
 			assert.equal(result.verdict, "mismatch");
 			assert.equal(result.addressUnreadable, true);
 		});

package/src/hooks/useIntelligenceData.ts

@@ -1,7 +1,4 @@
-import {
-	addressOperationsSearchAddressesOptions,
-	semanticSearchOperationsSemanticSearchOptions,
-} from "@remit/api-http-client/@tanstack/react-query.gen.ts";
+import { addressOperationsSearchAddressesOptions } from "@remit/api-http-client/@tanstack/react-query.gen.ts";
 import type {
 	RemitImapAddressResponse,
 	RemitImapThreadMessageResponse,
@@ -10,17 +7,14 @@ import { DisplayNameCorrespondence } from "@remit/domain-enums";
 import type {
 	AuthenticityIntel,
 	IntelligenceData,
-	MatchedChunk,
 	SenderFlagsIntel,
 	SenderIntel,
-	SimilarMessageIntel,
 } from "@remit/ui";
 import { isThreadCategory } from "@remit/ui";
 import { useQuery } from "@tanstack/react-query";
 import { useMemo } from "react";
-import { useSemanticSearchEnabled } from "@/hooks/useSemanticSearchEnabled";
 import { isServerError } from "@/lib/error-classifier";
-import { formatDate, formatEmailDate } from "@/lib/format";
+import { formatDate } from "@/lib/format";
 import {
 	pickSenderAddress,
 	senderAddressSearchQuery,
@@ -194,27 +188,16 @@ function readDomainList(value: unknown): string[] {
 	);
 }
 
-/**
- * Build authenticity intel from the thread message's authenticity field.
- *
- * `similarCount` is the number of semantic-search lookalikes; the panel uses
- * it to render the "N similar messages" campaign-reveal CTA inside the
- * danger panel. Passed in because it is derived from the semantic-search
- * result, not the thread row.
- */
+/** Build authenticity intel from the thread message's authenticity field. */
 export function buildAuthenticityIntel(
 	thread: RemitImapThreadMessageResponse,
-	similarCount: number,
 ): AuthenticityIntel {
-	const surfacedSimilar = similarCount > 0 ? similarCount : undefined;
-
 	if (isSenderAddressUnverifiable(thread.fromEmail)) {
 		return {
 			verdict: "mismatch",
 			fromDomain: "",
 			addressUnreadable: true,
 			summary: UNREADABLE_SENDER_SUMMARY,
-			similarCount: surfacedSimilar,
 		};
 	}
 
@@ -268,35 +251,11 @@ export function buildAuthenticityIntel(
 		dkimDomain,
 		claimedBrand,
 		summary,
-		// Surface the lookalike count so the panel renders the "N similar
-		// messages" campaign-reveal button.
-		similarCount: surfacedSimilar,
 	};
 }
 
 export interface UseIntelligenceDataResult {
 	data: IntelligenceData | null;
-	/**
-	 * True while similar-messages fetch is in flight. The rest of the data
-	 * (sender, authenticity, category, flags) arrives synchronously from the
-	 * already-loaded thread row.
-	 */
-	isSimilarLoading: boolean;
-	/** Non-null when the similar-messages fetch failed. */
-	similarError: unknown;
-	/**
-	 * True when the similar-messages failure is a fatal first-party 5xx. The
-	 * global escalation overlay handles it; the panel must NOT degrade it to the
-	 * benign grey "Similarity search unavailable" label. A soft (non-fatal)
-	 * error or an empty result keeps the panel quiet.
-	 */
-	similarErrorIsFatal: boolean;
-	/**
-	 * False on an instance with semantic search off (#1068): nothing is embedded,
-	 * so the query is never sent and the section states the setting instead of
-	 * rendering an empty list. `undefined` until the config read answers.
-	 */
-	semanticEnabled: boolean | undefined;
 	/**
 	 * True when the address lookup failed with a fatal first-party 5xx. The panel
 	 * must not silently render the sender's flags (VIP/muted/blocked) as absent —
@@ -318,20 +277,16 @@ export interface UseIntelligenceDataResult {
 
 /**
  * Maps a `RemitImapThreadMessageResponse` into `IntelligenceData` for the
- * `IntelligencePanel` component. Three data sources:
+ * `IntelligencePanel` component. Two data sources:
  *
  * 1. The thread row itself (already loaded) → sender basics, trust, authenticity, category.
  * 2. `GET /addresses/search?q=<email>` → address flags (VIP/muted/blocked/etc.)
  *    and first-seen timestamp. One round-trip per unique sender.
- * 3. `GET /search/semantic` → similar messages. Last to arrive; sidebar still
- *    renders if this fails.
  */
 export function useIntelligenceData(
 	thread: RemitImapThreadMessageResponse | null | undefined,
-	currentMailboxId?: string,
 ): UseIntelligenceDataResult {
 	const senderEmail = thread?.fromEmail ?? null;
-	const subject = thread?.subject ?? "";
 
 	// --- Address lookup: flags + first-seen timestamp ---
 	const { data: addressSearchResult, error: addressError } = useQuery({
@@ -347,70 +302,22 @@ export function useIntelligenceData(
 		senderEmail ?? undefined,
 	);
 
-	// --- Semantic search: similar messages ---
-	const semanticEnabled = useSemanticSearchEnabled();
-	const semanticQuery = [subject, senderEmail].filter(Boolean).join(" ");
-	const {
-		data: semanticResult,
-		isLoading: isSimilarLoading,
-		error: similarError,
-	} = useQuery({
-		...semanticSearchOperationsSemanticSearchOptions({
-			query: { query: semanticQuery, limit: 5 },
-		}),
-		// Waits for the config read rather than racing it. An off instance answers
-		// this query with an empty list, which the panel cannot tell from a sender
-		// nothing resembles, so the request is not worth sending on the chance the
-		// instance turns out to be on. `undefined` is that read still in flight;
-		// it is cached process-wide and every other surface already made it.
-		enabled:
-			Boolean(thread && semanticQuery.length > 3) && semanticEnabled === true,
-		staleTime: 60_000,
-		// A non-fatal failure (e.g. 404, no index yet) is acceptable — the sidebar
-		// degrades gracefully. A 5xx is NOT acceptable: it escalates globally and
-		// the panel must not present it as a benign "unavailable" state.
-		retry: 1,
-	});
-
 	const data = useMemo((): IntelligenceData | null => {
 		if (!thread) return null;
 
-		const similar: SimilarMessageIntel[] = (semanticResult?.items ?? [])
-			// Exclude the current message itself
-			.filter((r) => r.messageId !== thread.messageId)
-			.map((r) => ({
-				id: r.messageId,
-				threadId: r.threadId,
-				// Prefer the mailbox the user is already viewing when the message also
-				// lives there, so opening a similar message stays in the same folder;
-				// otherwise fall back to the first mailbox it lives in.
-				mailboxId:
-					currentMailboxId != null && r.mailboxIds.includes(currentMailboxId)
-						? currentMailboxId
-						: (r.mailboxIds[0] ?? ""),
-				fromName: r.fromName ?? "",
-				subject: r.subject ?? "(No subject)",
-				timeLabel: r.sentDate != null ? formatEmailDate(r.sentDate * 1000) : "",
-				matched: r.matchedChunkType as MatchedChunk,
-			}));
-
 		const sender = buildSenderIntel(thread, address);
 
-		const authenticity = buildAuthenticityIntel(thread, similar.length);
+		const authenticity = buildAuthenticityIntel(thread);
 
 		const category = buildCategoryIntel(thread, address);
 
 		const flags = buildSenderFlags(address);
 
-		return { sender, authenticity, category, flags, similar };
-	}, [thread, address, semanticResult, currentMailboxId]);
+		return { sender, authenticity, category, flags };
+	}, [thread, address]);
 
 	return {
 		data,
-		isSimilarLoading,
-		similarError,
-		similarErrorIsFatal: isServerError(similarError),
-		semanticEnabled,
 		addressErrorIsFatal: isServerError(addressError),
 		addressId: address?.addressId,
 		address,

package/src/hooks/useLayoutTier.test.ts

@@ -93,52 +93,4 @@ describe("isSinglePaneTier — compose surface must mount below desktop", () =>
 	it("is NOT single-pane at desktop (reading pane hosts compose)", () => {
 		assert.equal(isSinglePaneTier("desktop"), false);
 	});
-
-	it("treats every below-desktop width as single-pane via resolveLayoutTier", () => {
-		assert.equal(isSinglePaneTier(resolveLayoutTier(mouse(390))), true);
-		assert.equal(isSinglePaneTier(resolveLayoutTier(mouse(768))), true);
-		assert.equal(isSinglePaneTier(resolveLayoutTier(mouse(1023))), true);
-		assert.equal(isSinglePaneTier(resolveLayoutTier(mouse(1024))), false);
-		assert.equal(
-			isSinglePaneTier(resolveLayoutTier(tabletHeldUpright(1024))),
-			true,
-		);
-	});
-});
-
-describe("exactly one search field is mounted at every width", () => {
-	// Two call sites decide this from the same tier: `mail.tsx` mounts the top
-	// bar's field when the layout is not single-pane, and `MailListHeader`
-	// passes `showSearch` to keep the header's field otherwise. Both read
-	// `isSinglePaneTier`, so they are each other's complement by construction —
-	// if either is rewritten to its own expression this fails. Two mounted
-	// fields compete for "/" and for focus (#59); zero leaves no way to search.
-	const mountsTopBarField = (viewport: ViewportProfile): boolean =>
-		!isSinglePaneTier(resolveLayoutTier(viewport));
-	const mountsHeaderField = (viewport: ViewportProfile): boolean =>
-		isSinglePaneTier(resolveLayoutTier(viewport));
-
-	for (const viewport of [
-		mouse(0),
-		mouse(390),
-		mouse(767),
-		mouse(768),
-		mouse(1023),
-		mouse(1024),
-		mouse(1440),
-		tabletHeldUpright(1024),
-	]) {
-		it(`mounts one field at ${viewport.width}px ${viewport.orientation}/${viewport.pointer}`, () => {
-			const fields =
-				Number(mountsTopBarField(viewport)) +
-				Number(mountsHeaderField(viewport));
-			assert.equal(fields, 1, `${viewport.width}px mounts ${fields} fields`);
-		});
-	}
-
-	it("puts the field in the top bar only from desktop up", () => {
-		assert.equal(mountsTopBarField(mouse(1023)), false);
-		assert.equal(mountsTopBarField(mouse(1024)), true);
-		assert.equal(mountsTopBarField(tabletHeldUpright(1024)), false);
-	});
 });

package/src/hooks/useReportSpam.ts

@@ -54,7 +54,7 @@ export const GENERIC_SPAM_ACTION_FAILURE =
 	"This message could not be processed. Please try again. If it keeps failing, report it.";
 
 /**
- * The server's designed failure text names the message by embedding its raw
+ * The server's designed failure text names the message by putting its raw
  * id as a possessive subject — e.g. "Message 4kv0xxyfhg4dhzqvxd105v840's move
  * to Junk has not settled yet; try again in a moment." (message ids here are
  * 25-char base36, `translator.generate()` in `packages/data-ports/src/id.ts`

package/src/hooks/useRescueCandidates.test.ts

@@ -139,16 +139,6 @@ describe("useRescueCandidates: candidate mapping from real hook output", () => {
 		senderTrust: "wellknown",
 	});
 
-	test("maps messageId to candidate id", () => {
-		const { candidates } = renderHookWithResponse({ items: [vipThread] });
-		assert.equal(candidates[0].id, "m1");
-	});
-
-	test("maps fromName as senderName", () => {
-		const { candidates } = renderHookWithResponse({ items: [vipThread] });
-		assert.equal(candidates[0].senderName, "Anna de Vries");
-	});
-
 	test("falls back to fromEmail when fromName is absent", () => {
 		const thread = makeThread({
 			messageId: "m3",
@@ -193,19 +183,6 @@ describe("useRescueCandidates: candidate mapping from real hook output", () => {
 		assert.equal(candidates[0].trustSubReason, "You've emailed them before");
 	});
 
-	test("sets the constant trustReason headline", () => {
-		const { candidates } = renderHookWithResponse({ items: [vipThread] });
-		assert.equal(candidates[0].trustReason, "We can verify this sender");
-	});
-
-	test("preserves senderTrust on the candidate", () => {
-		const { candidates } = renderHookWithResponse({
-			items: [vipThread, wellknownThread],
-		});
-		assert.equal(candidates[0].senderTrust, "vip");
-		assert.equal(candidates[1].senderTrust, "wellknown");
-	});
-
 	test("maps all response items to candidates in order", () => {
 		const { candidates } = renderHookWithResponse({
 			items: [vipThread, wellknownThread],

package/src/hooks/useRulePreview.ts

@@ -2,7 +2,10 @@ import { organizeOperationsPreviewOrganizeMutation } from "@remit/api-http-clien
 import { type PreviewCount, UNCOUNTABLE_PREDICATE_REASON } from "@remit/ui";
 import { useMutation } from "@tanstack/react-query";
 import { useEffect, useRef, useState } from "react";
-import { buildOrganizeInput } from "@/lib/organize/organize-model";
+import {
+	buildOrganizeInput,
+	type OrganizeMatchPredicate,
+} from "@/lib/organize/organize-model";
 import {
 	derivePreview,
 	isEvaluablePredicate,
@@ -10,13 +13,12 @@ import {
 	type PreviewState,
 	predicateSignature,
 } from "@/lib/organize/rule-model";
-import type { OrganizeMatchPredicate } from "@/lib/organize/sender-fallback";
 
 /**
  * The live match count for the edited rule (RFC 038 D1). Every predicate change
  * schedules a debounced `POST /organize/preview` and, until it lands, reports
  * the last count as stale so the count never blanks and the commit gate can
- * hold apply. Seeded with the widen probe's count, so opening the editor shows a
+ * hold apply. Seeded with a count already known, so opening the editor shows a
  * settled number without an immediate re-fetch.
  *
  * Out-of-order responses are dropped: only the reply to the most recent request
@@ -24,9 +26,9 @@ import type { OrganizeMatchPredicate } from "@/lib/organize/sender-fallback";
  * server bounds the match, so a broad predicate over a large mailbox stays a
  * single cheap request; the debounce is what keeps rapid edits from spamming it.
  *
- * `seedCount` is the opening count a widen probe already knows; omit it (as the
- * settings editor does, having no probe) to open on `loading` and preview the
- * loaded predicate immediately.
+ * `seedCount` is an opening count already known; omit it (as the settings
+ * editor does) to open on `loading` and preview the loaded predicate
+ * immediately.
  *
  * `matchedIds` are the ids that count was counted over — the server's own
  * answer, bounded to the same cap the apply pass uses. A caller applying an
@@ -78,7 +80,6 @@ export const useRulePreview = (
 					setState({
 						count: response.matchedCount,
 						matchedIds: response.messageIds,
-						indexEmpty: response.semanticIndexEmpty === true,
 						previewedSignature: signature,
 					});
 				})

package/src/hooks/useSearchTokenContext.ts

@@ -8,9 +8,8 @@ import { useBrowsedList } from "@/routing";
  *
  * `in:` is recognized only where the route carries no scope. On a scoped view
  * the top bar's chip already answers "which mailbox", and a typed `in:` is a
- * second, competing answer that no engine there acts on — the literal search is
- * pinned to the route's mailbox and the semantic request takes the caller's
- * scope. Leaving the term unresolved means it stays ordinary words: it is
+ * second, competing answer that no engine there acts on — the search is pinned
+ * to the route's mailbox. Leaving the term unresolved means it stays ordinary words: it is
  * matched as text and never chipped, so the field can't advertise a filter that
  * does nothing. To search another folder, drop the scope chip or go to it.
  *

package/src/hooks/useStaleAccountSync.test.ts

@@ -25,12 +25,6 @@ import {
 
 const NOW = 1_700_000_000_000;
 
-describe("STALENESS_THRESHOLD_MS", () => {
-	test("equals 15 minutes in milliseconds", () => {
-		assert.equal(STALENESS_THRESHOLD_MS, 15 * 60 * 1000);
-	});
-});
-
 describe("selectStaleAccountIds", () => {
 	test("treats accounts with undefined lastSyncAt as stale", () => {
 		const ids = selectStaleAccountIds(
@@ -92,10 +86,6 @@ describe("selectStaleAccountIds", () => {
 		assert.deepEqual(ids, ["a-1"]);
 	});
 
-	test("returns an empty list for an empty input", () => {
-		assert.deepEqual(selectStaleAccountIds([], NOW), []);
-	});
-
 	test("equality on the threshold is treated as fresh", () => {
 		// `now - lastSyncAt > thresholdMs` — strict greater-than means an
 		// account synced exactly threshold ms ago is still fresh. Pinning
@@ -108,13 +98,6 @@ describe("selectStaleAccountIds", () => {
 	});
 });
 
-describe("POLL_INTERVAL_MS", () => {
-	// How stale an open tab's mail may be with nobody touching anything.
-	test("defaults to 30 seconds when mailboxPollIntervalSeconds is unset in this test run", () => {
-		assert.equal(POLL_INTERVAL_MS, 30 * 1000);
-	});
-});
-
 describe("resolvePollIntervalMs", () => {
 	// This poll shares POST /sync with the refresh control, and that endpoint's
 	// triggers skip the server's per-mailbox freshness gate

package/src/hooks/useStarredThreads.test.ts

@@ -96,16 +96,6 @@ function renderHook(
 	return captured;
 }
 
-describe("starredThreadsQueryKey", () => {
-	test("asks for the starred listing, newest first", () => {
-		const [params] = starredThreadsQueryKey() as [
-			{ query?: { starred?: boolean; order?: string } },
-		];
-		assert.equal(params.query?.starred, true);
-		assert.equal(params.query?.order, "desc");
-	});
-});
-
 describe("useStarredThreads", () => {
 	test("returns no threads when the listing is empty", () => {
 		assert.deepEqual(renderHook([{ items: [] }]), []);

package/src/hooks/useTriggerSync.test.ts

@@ -15,10 +15,4 @@ describe("buildMailboxListKey", () => {
 		});
 		assert.deepEqual(got, want);
 	});
-
-	test("returns a different key for a different accountId", () => {
-		const a = buildMailboxListKey("acc-1");
-		const b = buildMailboxListKey("acc-2");
-		assert.notDeepEqual(a, b);
-	});
 });

package/src/hooks/useVisualViewport.render.test.ts

@@ -0,0 +1,63 @@
+import assert from "node:assert/strict";
+import { afterEach, describe, it } from "node:test";
+import { createElement } from "react";
+import { createDomHarness, type DomHarness } from "@/test-support/dom";
+import { useVisualViewport } from "./useVisualViewport";
+
+let harness: DomHarness | undefined;
+
+function KeyboardProbe() {
+	const { isKeyboardOpen } = useVisualViewport();
+	return createElement("p", null, isKeyboardOpen ? "open" : "closed");
+}
+
+const mount = (): {
+	dom: DomHarness;
+	fullHeight: number;
+	resize: (height: number) => void;
+} => {
+	const fullHeight = window.innerHeight;
+	const viewport = Object.assign(new window.EventTarget(), {
+		height: fullHeight,
+	});
+	Object.defineProperty(window, "visualViewport", {
+		configurable: true,
+		value: viewport,
+	});
+	const dom = createDomHarness();
+	harness = dom;
+	dom.render(createElement(KeyboardProbe));
+	return {
+		dom,
+		fullHeight,
+		resize: (height) => {
+			viewport.height = height;
+			dom.dispatch(viewport, new window.Event("resize"));
+		},
+	};
+};
+
+afterEach(() => {
+	harness?.close();
+	harness = undefined;
+	Reflect.deleteProperty(window, "visualViewport");
+});
+
+describe("the software keyboard", () => {
+	it("reads as open once the visual viewport shrinks past the threshold, and closed again when it restores", () => {
+		const { dom, fullHeight, resize } = mount();
+		assert.equal(dom.text(), "closed");
+
+		resize(fullHeight - 400);
+		assert.equal(dom.text(), "open");
+
+		resize(fullHeight);
+		assert.equal(dom.text(), "closed");
+	});
+
+	it("stays closed when only the address bar collapses", () => {
+		const { dom, fullHeight, resize } = mount();
+		resize(fullHeight - 50);
+		assert.equal(dom.text(), "closed");
+	});
+});