@remit/web-client 0.0.321 → 0.0.323

package/src/lib/provider-presets.test.ts

@@ -1,81 +1,9 @@
 import assert from "node:assert/strict";
 import { describe, it } from "node:test";
 import { APP_PASSWORD_URLS, discoverSettings } from "./autodiscovery.js";
-import {
-	getPresetById,
-	PROVIDER_PRESETS,
-	type ProviderPreset,
-	presetIdForEmail,
-} from "./provider-presets.js";
-
-interface ExpectedSettings {
-	imap: { host: string; port: number; security: string };
-	smtp: { host: string; port: number; security: string };
-}
-
-const EXPECTED: Record<string, ExpectedSettings> = {
-	icloud: {
-		imap: { host: "imap.mail.me.com", port: 993, security: "tls" },
-		smtp: { host: "smtp.mail.me.com", port: 587, security: "starttls" },
-	},
-	yahoo: {
-		imap: { host: "imap.mail.yahoo.com", port: 993, security: "tls" },
-		smtp: { host: "smtp.mail.yahoo.com", port: 465, security: "tls" },
-	},
-	aol: {
-		imap: { host: "imap.aol.com", port: 993, security: "tls" },
-		smtp: { host: "smtp.aol.com", port: 465, security: "tls" },
-	},
-	fastmail: {
-		imap: { host: "imap.fastmail.com", port: 993, security: "tls" },
-		smtp: { host: "smtp.fastmail.com", port: 465, security: "tls" },
-	},
-	gmail: {
-		imap: { host: "imap.gmail.com", port: 993, security: "tls" },
-		smtp: { host: "smtp.gmail.com", port: 587, security: "starttls" },
-	},
-};
+import { getPresetById, presetIdForEmail } from "./provider-presets.js";
 
 describe("provider presets", () => {
-	it("exposes exactly the five documented providers", () => {
-		const ids = PROVIDER_PRESETS.map((p) => p.id).sort();
-		assert.deepEqual(ids, ["aol", "fastmail", "gmail", "icloud", "yahoo"]);
-	});
-
-	for (const preset of PROVIDER_PRESETS) {
-		describe(preset.label, () => {
-			const expected = EXPECTED[preset.id];
-
-			it("matches the exact IMAP settings from the table", () => {
-				assert.deepEqual(preset.imap, expected.imap);
-			});
-
-			it("matches the exact SMTP settings from the table", () => {
-				assert.deepEqual(preset.smtp, expected.smtp);
-			});
-
-			it("uses the full email as username", () => {
-				assert.equal(preset.username, "full-email");
-			});
-
-			it("has password help text", () => {
-				assert.ok(preset.passwordHelp.text.length > 0);
-				assert.match(preset.passwordHelp.text, /app[\w-]* password/i);
-			});
-
-			it("has a valid https app-password url", () => {
-				const url = new URL(preset.passwordHelp.url);
-				assert.equal(url.protocol, "https:");
-			});
-		});
-	}
-
-	it("looks up a preset by id", () => {
-		const preset: ProviderPreset | undefined = getPresetById("icloud");
-		assert.ok(preset);
-		assert.equal(preset.label, "iCloud");
-	});
-
 	it("tells a Gmail user to turn on 2-Step Verification before creating an app password", () => {
 		const gmail = getPresetById("gmail");
 		assert.ok(gmail);
@@ -88,10 +16,6 @@ describe("provider presets", () => {
 		assert.equal(url, APP_PASSWORD_URLS["googlemail.com"]);
 	});
 
-	it("returns undefined for an unknown id", () => {
-		assert.equal(getPresetById("custom"), undefined);
-	});
-
 	describe("presetIdForEmail", () => {
 		it("pre-selects a preset for a known provider domain", () => {
 			assert.equal(presetIdForEmail("alice@icloud.com"), "icloud");

package/src/lib/query-error-handler.test.ts

@@ -3,7 +3,6 @@ import { afterEach, describe, it } from "node:test";
 import type { Mutation, Query } from "@tanstack/react-query";
 import { ApiError } from "./api";
 import { __resetFatalError, subscribeFatalError } from "./fatal-error";
-import { NetworkError } from "./network-error";
 import {
 	handleMutationCacheError,
 	handleQueryCacheError,
@@ -41,25 +40,6 @@ afterEach(() => {
 });
 
 describe("handleQueryCacheError (fail-fast contract #1059)", () => {
-	it("escalates a 5xx that breaks the INITIAL load", () => {
-		const seen = escalations();
-		handleQueryCacheError(
-			new ApiError("internal", 500),
-			fakeQuery(undefined, 0),
-		);
-		assert.deepEqual(seen, ["internal"]);
-	});
-
-	it("escalates a 5xx on a background REFETCH (the deleted dataUpdatedAt guard — the headline bug)", () => {
-		const seen = escalations();
-		// dataUpdatedAt !== 0 used to swallow this (PR #758). It must now escalate.
-		handleQueryCacheError(
-			new ApiError("internal", 500),
-			fakeQuery(undefined, 123456),
-		);
-		assert.deepEqual(seen, ["internal"]);
-	});
-
 	it("escalates a 5xx EVEN when the query is marked meta.softError (rule 2 wins)", () => {
 		const seen = escalations();
 		handleQueryCacheError(
@@ -69,24 +49,6 @@ describe("handleQueryCacheError (fail-fast contract #1059)", () => {
 		assert.deepEqual(seen, ["internal"]);
 	});
 
-	it("escalates a 4xx by DEFAULT (no opt-out)", () => {
-		const seen = escalations();
-		handleQueryCacheError(new ApiError("not found", 404), fakeQuery());
-		assert.deepEqual(seen, ["not found"]);
-	});
-
-	it("does NOT escalate a 4xx the query opted out of via meta.softError", () => {
-		let escalated = false;
-		subscribeFatalError(() => {
-			escalated = true;
-		});
-		handleQueryCacheError(
-			new ApiError("not found", 404),
-			fakeQuery({ softError: true }),
-		);
-		assert.equal(escalated, false);
-	});
-
 	it("does NOT escalate an aborted request", () => {
 		let escalated = false;
 		subscribeFatalError(() => {
@@ -95,32 +57,9 @@ describe("handleQueryCacheError (fail-fast contract #1059)", () => {
 		handleQueryCacheError(abortError(), fakeQuery());
 		assert.equal(escalated, false);
 	});
-
-	it("does NOT escalate a network blip tagged at the fetch boundary", () => {
-		let escalated = false;
-		subscribeFatalError(() => {
-			escalated = true;
-		});
-		handleQueryCacheError(
-			new NetworkError(new TypeError("Failed to fetch")),
-			fakeQuery(),
-		);
-		assert.equal(escalated, false);
-	});
 });
 
 describe("handleMutationCacheError (fail-fast contract #1059)", () => {
-	it("escalates a mutation 5xx", () => {
-		const seen = escalations();
-		handleMutationCacheError(
-			new ApiError("mutation blew up", 503),
-			undefined,
-			undefined,
-			fakeMutation(),
-		);
-		assert.deepEqual(seen, ["mutation blew up"]);
-	});
-
 	it("escalates a mutation 4xx by DEFAULT", () => {
 		const seen = escalations();
 		handleMutationCacheError(

package/src/lib/query-escalation.integration.test.ts

@@ -46,16 +46,16 @@ describe("global query/mutation escalation", () => {
 		assert.deepEqual(seen, ["internal server error"]);
 	});
 
-	it("a 5xx on a background REFETCH escalates — the exact #1059 bug (semantic-search-style query that 500s after first rendering data)", async () => {
+	it("a 5xx on a background REFETCH escalates — the exact #1059 bug (search-style query that 500s after first rendering data)", async () => {
 		const seen: string[] = [];
 		subscribeFatalError((fatal) => seen.push(fatal.message));
 		const client = makeClient();
-		const queryKey = ["semanticSearch"];
+		const queryKey = ["search"];
 		let calls = 0;
 		const queryFn = async () => {
 			calls += 1;
 			if (calls === 1) return { items: [] as string[] };
-			throw new ApiError("semantic search exploded", 500);
+			throw new ApiError("search exploded", 500);
 		};
 
 		// First load succeeds → query now has data (dataUpdatedAt !== 0). Under the
@@ -63,7 +63,7 @@ describe("global query/mutation escalation", () => {
 		await client.fetchQuery({ queryKey, queryFn });
 		await client.refetchQueries({ queryKey }).catch(() => {});
 
-		assert.deepEqual(seen, ["semantic search exploded"]);
+		assert.deepEqual(seen, ["search exploded"]);
 	});
 
 	it("a first-party 5xx from a mutation escalates to the fatal overlay seam", async () => {

package/src/lib/result-count.ts

@@ -1,7 +1,7 @@
 import type { ResultCount } from "@remit/ui";
 
 /**
- * Shortest free text a count is asked for. `npm-scripts/sqlite-search-index.sql`
+ * Shortest free text a count is asked for. The search migration set
  * documents that a query under three characters falls out of the trigram index
  * onto a folded `LIKE` scan, so counting one is a mailbox scan per typed
  * character. The list itself still pages — only the count waits.

package/src/lib/result-folder.ts

@@ -5,7 +5,7 @@
  * says where it came from and spam is held out of the list. Both questions are
  * answered by joining the row's mailbox ids against the mailbox lists the
  * sidebar already loads: the id → folder map here is the only thing either
- * needs, so neither the search API nor the semantic index has to carry a folder.
+ * needs, so the search API does not have to carry a folder.
  *
  * The mailbox row carries the hierarchy delimiter its own server reports, and it
  * is kept alongside the path: a chip labelled by path shows the folder's own

package/src/lib/rum-adapter.test.ts

@@ -52,36 +52,12 @@ describe("createRumTelemetry", () => {
 		rum = makeRum();
 	});
 
-	it("recordPageView delegates to the RUM client", () => {
-		const t = createRumTelemetry(rum);
-		t.recordPageView("/inbox");
-		assert.deepEqual(stubCalls(rum).recordPageView, ["/inbox"]);
-	});
-
 	it("recordPageView strips the entire query string", () => {
 		const t = createRumTelemetry(rum);
 		t.recordPageView("/callback?token=secret&next=/inbox");
 		assert.equal(stubCalls(rum).recordPageView[0], "/callback");
 	});
 
-	it("recordPageView strips selectedMessageId query param", () => {
-		const t = createRumTelemetry(rum);
-		t.recordPageView("/inbox?selectedMessageId=msg_abc123def456");
-		assert.equal(stubCalls(rum).recordPageView[0], "/inbox");
-	});
-
-	it("recordPageView strips search query param", () => {
-		const t = createRumTelemetry(rum);
-		t.recordPageView("/inbox?q=sensitive+search+text");
-		assert.equal(stubCalls(rum).recordPageView[0], "/inbox");
-	});
-
-	it("recordPageView strips query string and fragment together", () => {
-		const t = createRumTelemetry(rum);
-		t.recordPageView("/inbox?q=foo#section");
-		assert.equal(stubCalls(rum).recordPageView[0], "/inbox");
-	});
-
 	it("recordPageView strips fragment-only paths", () => {
 		const t = createRumTelemetry(rum);
 		t.recordPageView("/inbox#fragment");
@@ -97,13 +73,6 @@ describe("createRumTelemetry", () => {
 		);
 	});
 
-	it("recordError delegates to the RUM client", () => {
-		const t = createRumTelemetry(rum);
-		const error = new Error("boom");
-		t.recordError(error);
-		assert.deepEqual(stubCalls(rum).recordError, [error]);
-	});
-
 	it("recordError scrubs message id tokens from error messages", () => {
 		const t = createRumTelemetry(rum);
 		const error = new Error("failed to load /messages/msg_abc123def456");
@@ -122,54 +91,6 @@ describe("createRumTelemetry", () => {
 		const recorded = stubCalls(rum).recordError[0] as Error;
 		assert.equal(recorded.name, "TypeError");
 	});
-
-	it("recordError with context emits an additional event", () => {
-		const t = createRumTelemetry(rum);
-		t.recordError(new Error("boom"), { component: "Inbox" });
-		const ctx = stubCalls(rum).recordEvent.find(
-			(e) => e.name === "telemetry.error_context",
-		);
-		assert.ok(ctx, "Expected a telemetry.error_context event");
-		assert.deepEqual(ctx.attributes, { component: "Inbox" });
-	});
-
-	it("recordEvent delegates to the RUM client", () => {
-		const t = createRumTelemetry(rum);
-		t.recordEvent("thread.open", { mailboxId: "inbox" });
-		assert.ok(
-			stubCalls(rum).recordEvent.some((e) => e.name === "thread.open"),
-			"Expected thread.open event",
-		);
-	});
-
-	it("recordEvent with no attributes passes an empty object", () => {
-		const t = createRumTelemetry(rum);
-		t.recordEvent("app.load");
-		const ev = stubCalls(rum).recordEvent.find((e) => e.name === "app.load");
-		assert.ok(ev, "Expected app.load event");
-		assert.deepEqual(ev.attributes, {});
-	});
-
-	it("recordTiming emits a telemetry.timing event with name and duration", () => {
-		const t = createRumTelemetry(rum);
-		t.recordTiming("api.latency", 123);
-		const ev = stubCalls(rum).recordEvent.find(
-			(e) => e.name === "telemetry.timing",
-		);
-		assert.ok(ev, "Expected telemetry.timing event");
-		assert.equal((ev.attributes as Record<string, string>).name, "api.latency");
-		assert.equal((ev.attributes as Record<string, string>).durationMs, "123");
-	});
-
-	it("recordTiming merges extra attributes", () => {
-		const t = createRumTelemetry(rum);
-		t.recordTiming("api.latency", 50, { route: "/inbox" });
-		const ev = stubCalls(rum).recordEvent.find(
-			(e) => e.name === "telemetry.timing",
-		);
-		assert.ok(ev, "Expected telemetry.timing event");
-		assert.equal((ev.attributes as Record<string, string>).route, "/inbox");
-	});
 });
 
 type WindowListener = (event: unknown) => void;
@@ -202,31 +123,6 @@ describe("initRum", () => {
 		delete (globalThis as unknown as { window?: unknown }).window;
 	});
 
-	it("returns a no-op telemetry when VITE_RUM_APP_MONITOR_ID is unset", () => {
-		globalThis.__REMIT_CONFIG__ = {};
-		const t = initRum();
-		assert.doesNotThrow(() => t.recordPageView("/"));
-		assert.doesNotThrow(() => t.recordError(new Error("x")));
-		assert.doesNotThrow(() => t.recordEvent("e"));
-		assert.doesNotThrow(() => t.recordTiming("t", 1));
-	});
-
-	it("creates a RumTelemetry when VITE_RUM_APP_MONITOR_ID is set", () => {
-		resetInstances();
-		installWindowStub();
-		globalThis.__REMIT_CONFIG__ = {
-			rum: {
-				appMonitorId: "test-monitor-id",
-				identityPoolId: "eu-west-1:test-pool",
-				region: "eu-west-1",
-			},
-		};
-		const t = initRum();
-		const instance = getLastInstance();
-		assert.equal(instance.appMonitorId, "test-monitor-id");
-		assert.doesNotThrow(() => t.recordPageView("/inbox"));
-	});
-
 	it("does not enable leaky auto-telemetries", () => {
 		resetInstances();
 		installWindowStub();

package/src/lib/search-query.test.ts

@@ -7,13 +7,6 @@ describe("normalizeSearchQuery", () => {
 		assert.equal(normalizeSearchQuery("ALICE"), "alice");
 	});
 
-	test("lowercases a mixed-case query", () => {
-		assert.equal(
-			normalizeSearchQuery("Alice@Example.COM"),
-			"alice@example.com",
-		);
-	});
-
 	test("trims surrounding whitespace", () => {
 		assert.equal(normalizeSearchQuery("   Bob  "), "bob");
 	});
@@ -22,10 +15,6 @@ describe("normalizeSearchQuery", () => {
 		assert.equal(normalizeSearchQuery("   "), "");
 	});
 
-	test("leaves an already-lowercase query unchanged", () => {
-		assert.equal(normalizeSearchQuery("invoice 2026"), "invoice 2026");
-	});
-
 	test("handles non-ASCII letters via locale-aware lowercasing", () => {
 		// `toLocaleLowerCase` folds these where `String.prototype.toLowerCase`
 		// would also work for Latin-1, but we exercise it to lock the contract.

package/src/lib/search-result.test.ts

@@ -1,102 +1,7 @@
 import assert from "node:assert";
 import { describe, test } from "node:test";
-import type {
-	RemitImapSemanticSearchResult,
-	RemitImapThreadMessageResponse,
-} from "@remit/api-http-client/types.gen.ts";
-import {
-	matchedChunkLabel,
-	relatedSearchResults,
-	semanticToSearchResult,
-	threadToSearchResult,
-} from "./search-result.js";
-
-function hit(
-	overrides: Partial<RemitImapSemanticSearchResult> &
-		Pick<RemitImapSemanticSearchResult, "messageId" | "threadId" | "score">,
-): RemitImapSemanticSearchResult {
-	return {
-		matchedChunkType: "body",
-		mailboxIds: [],
-		...overrides,
-	};
-}
-
-describe("semanticToSearchResult", () => {
-	test("maps display fields and converts epoch seconds to ms", () => {
-		const result = semanticToSearchResult(
-			hit({
-				messageId: "m1",
-				threadId: "t1",
-				score: 0.9,
-				fromName: "Stripe",
-				subject: "Invoice ready",
-				sentDate: 1_700_000_000,
-			}),
-		);
-		assert.strictEqual(result.id, "m1");
-		assert.strictEqual(result.sender, "Stripe");
-		assert.strictEqual(result.subject, "Invoice ready");
-		assert.notStrictEqual(result.date, "");
-	});
-
-	test("falls back when denormalized display fields are absent", () => {
-		const result = semanticToSearchResult(
-			hit({ messageId: "m2", threadId: "t2", score: 0.5 }),
-		);
-		assert.strictEqual(result.sender, "Unknown");
-		assert.strictEqual(result.subject, "(No subject)");
-		assert.strictEqual(result.date, "");
-	});
-
-	// The bug: without thread + mailbox the brief could only look the hit up by
-	// messageId in its loaded list, where a related hit isn't found — so it must
-	// carry both so the conversation opens directly.
-	test("carries the thread and first mailbox so the hit opens directly", () => {
-		const result = semanticToSearchResult(
-			hit({
-				messageId: "m3",
-				threadId: "t3",
-				score: 0.7,
-				mailboxIds: ["mb-inbox", "mb-other"],
-			}),
-		);
-		assert.strictEqual(result.threadId, "t3");
-		assert.strictEqual(result.mailboxId, "mb-inbox");
-	});
-
-	test("omits mailboxId when the hit carries no mailbox", () => {
-		const result = semanticToSearchResult(
-			hit({ messageId: "m4", threadId: "t4", score: 0.3, mailboxIds: [] }),
-		);
-		assert.strictEqual(result.threadId, "t4");
-		assert.strictEqual(result.mailboxId, undefined);
-	});
-
-	test("carries the matched-chunk label and score so the row can show why it matched", () => {
-		const result = semanticToSearchResult(
-			hit({
-				messageId: "m5",
-				threadId: "t5",
-				score: 0.42,
-				matchedChunkType: "attachment",
-			}),
-		);
-		assert.strictEqual(result.matchedChunkLabel, "attachment");
-		assert.strictEqual(result.score, 0.42);
-	});
-});
-
-describe("matchedChunkLabel", () => {
-	test("labels every known chunk type", () => {
-		assert.strictEqual(matchedChunkLabel("body"), "body");
-		assert.strictEqual(matchedChunkLabel("subject"), "subject");
-		assert.strictEqual(matchedChunkLabel("sender"), "sender");
-		assert.strictEqual(matchedChunkLabel("recipient"), "recipient");
-		assert.strictEqual(matchedChunkLabel("attachment"), "attachment");
-		assert.strictEqual(matchedChunkLabel("entities"), "entities");
-	});
-});
+import type { RemitImapThreadMessageResponse } from "@remit/api-http-client/types.gen.ts";
+import { threadToSearchResult } from "./search-result.js";
 
 describe("threadToSearchResult", () => {
 	test("carries thread and mailbox so a literal hit opens directly", () => {
@@ -113,48 +18,3 @@ describe("threadToSearchResult", () => {
 		assert.strictEqual(result.mailboxId, "mb1");
 	});
 });
-
-describe("relatedSearchResults", () => {
-	test("orders by score descending", () => {
-		const results = relatedSearchResults(
-			[
-				hit({ messageId: "m1", threadId: "t1", score: 0.2 }),
-				hit({ messageId: "m2", threadId: "t2", score: 0.9 }),
-				hit({ messageId: "m3", threadId: "t3", score: 0.5 }),
-			],
-			[],
-		);
-		assert.deepStrictEqual(
-			results.map((r) => r.id),
-			["m2", "m3", "m1"],
-		);
-	});
-
-	test("excludes threads already shown under Top matches", () => {
-		const results = relatedSearchResults(
-			[
-				hit({ messageId: "m1", threadId: "shared", score: 0.9 }),
-				hit({ messageId: "m2", threadId: "t2", score: 0.5 }),
-			],
-			["shared"],
-		);
-		assert.deepStrictEqual(
-			results.map((r) => r.id),
-			["m2"],
-		);
-	});
-
-	test("collapses multiple hits in the same thread to one row", () => {
-		const results = relatedSearchResults(
-			[
-				hit({ messageId: "m1", threadId: "t1", score: 0.9 }),
-				hit({ messageId: "m2", threadId: "t1", score: 0.4 }),
-			],
-			[],
-		);
-		assert.deepStrictEqual(
-			results.map((r) => r.id),
-			["m1"],
-		);
-	});
-});

package/src/lib/search-result.ts

@@ -1,14 +1,9 @@
 /**
  * Map the row shapes the app carries onto the kit `SearchResult` the search
- * results render. Two feed the literal "Top matches" section (per-mailbox
- * `RemitImapThreadMessageResponse` and the brief/flagged `ThreadRowData`); the
- * third (`RemitImapSemanticSearchResult`) feeds the semantic "Related" section.
+ * results render: the per-mailbox `RemitImapThreadMessageResponse` and the
+ * brief/flagged `ThreadRowData`.
  */
-import type {
-	RemitImapSemanticSearchChunkType,
-	RemitImapSemanticSearchResult,
-	RemitImapThreadMessageResponse,
-} from "@remit/api-http-client/types.gen.ts";
+import type { RemitImapThreadMessageResponse } from "@remit/api-http-client/types.gen.ts";
 import type { SearchResult, ThreadRowData } from "@remit/ui";
 import { formatEmailDate } from "./format.js";
 import {
@@ -16,26 +11,6 @@ import {
 	resolveResultFolder,
 } from "./result-folder.js";
 
-/**
- * Plain-language label for the `matched: …` chip, so the user understands why
- * a "Related" hit matched (#428). Falls back to the raw chunk type for any
- * value the backend adds later that the client doesn't know about yet.
- */
-const MATCHED_CHUNK_LABELS: Record<RemitImapSemanticSearchChunkType, string> = {
-	sender: "sender",
-	recipient: "recipient",
-	subject: "subject",
-	attachment: "attachment",
-	body: "body",
-	entities: "entities",
-};
-
-export function matchedChunkLabel(
-	chunkType: RemitImapSemanticSearchChunkType,
-): string {
-	return MATCHED_CHUNK_LABELS[chunkType] ?? chunkType;
-}
-
 export function threadToSearchResult(
 	thread: RemitImapThreadMessageResponse,
 	folders?: ResultFolderIndex,
@@ -78,48 +53,3 @@ export function rowToSearchResult(
 		...(folder ? { folder } : {}),
 	};
 }
-
-/**
- * Map a semantic hit onto a `SearchResult`. Display fields are denormalized at
- * index time and absent for older entries, so each falls back. Read/star state
- * isn't carried by the semantic index, so the row renders neutral. `sentDate` is
- * epoch seconds; `formatEmailDate` takes epoch ms.
- */
-export function semanticToSearchResult(
-	hit: RemitImapSemanticSearchResult,
-	folders?: ResultFolderIndex,
-): SearchResult {
-	const { mailboxId, folder } = resolveResultFolder(folders, hit.mailboxIds);
-	return {
-		id: hit.messageId,
-		sender: hit.fromName ?? "Unknown",
-		subject: hit.subject ?? "(No subject)",
-		snippet: "",
-		date: hit.sentDate != null ? formatEmailDate(hit.sentDate * 1000) : "",
-		threadId: hit.threadId,
-		matchedChunkLabel: matchedChunkLabel(hit.matchedChunkType),
-		score: hit.score,
-		...(mailboxId ? { mailboxId } : {}),
-		...(folder ? { folder } : {}),
-	};
-}
-
-/**
- * Build the semantic "Related" section: order by score (most relevant first),
- * collapse to one row per thread, and drop any thread already shown under the
- * literal "Top matches" — literal precedence, so a thread never appears twice.
- */
-export function relatedSearchResults(
-	hits: RemitImapSemanticSearchResult[],
-	literalThreadIds: Iterable<string>,
-	folders?: ResultFolderIndex,
-): SearchResult[] {
-	const seenThreadIds = new Set<string>(literalThreadIds);
-	const results: SearchResult[] = [];
-	for (const hit of [...hits].sort((a, b) => b.score - a.score)) {
-		if (seenThreadIds.has(hit.threadId)) continue;
-		seenThreadIds.add(hit.threadId);
-		results.push(semanticToSearchResult(hit, folders));
-	}
-	return results;
-}