@remit/web-client 0.0.325 → 0.0.326

package/src/lib/rum-adapter.ts

@@ -1,107 +0,0 @@
-import { AwsRum } from "aws-rum-web";
-import { getRuntimeConfig } from "../runtime-config";
-import type { Telemetry } from "./telemetry";
-import { noopTelemetry } from "./telemetry";
-
-const ID_TOKEN_PATTERN =
-	/([?&](token|access_token|id_token|authorization)=[^&]*)|\/messages\/[a-z0-9_-]+/gi;
-
-function scrubIds(value: string): string {
-	return value.replace(ID_TOKEN_PATTERN, "[redacted]");
-}
-
-function sanitizePath(path: string): string {
-	const qIndex = path.indexOf("?");
-	const hIndex = path.indexOf("#");
-	const cutAt =
-		qIndex === -1 && hIndex === -1
-			? path.length
-			: qIndex === -1
-				? hIndex
-				: hIndex === -1
-					? qIndex
-					: Math.min(qIndex, hIndex);
-	return scrubIds(path.slice(0, cutAt));
-}
-
-function scrubErrorMessage(error: Error): Error {
-	const scrubbed = scrubIds(error.message);
-	if (scrubbed === error.message) {
-		return error;
-	}
-	const next = new Error(scrubbed);
-	next.name = error.name;
-	next.stack = error.stack;
-	return next;
-}
-
-export function createRumTelemetry(rum: AwsRum): Telemetry {
-	return {
-		recordPageView(path: string): void {
-			rum.recordPageView(sanitizePath(path));
-		},
-
-		recordError(error: Error, context?: Record<string, string>): void {
-			rum.recordError(scrubErrorMessage(error));
-			if (context) {
-				rum.recordEvent("telemetry.error_context", context);
-			}
-		},
-
-		recordEvent(name: string, attributes?: Record<string, string>): void {
-			rum.recordEvent(name, attributes ?? {});
-		},
-
-		recordTiming(
-			name: string,
-			durationMs: number,
-			attributes?: Record<string, string>,
-		): void {
-			rum.recordEvent("telemetry.timing", {
-				...attributes,
-				name,
-				durationMs: String(durationMs),
-			});
-		},
-	};
-}
-
-function toError(reason: unknown): Error {
-	if (reason instanceof Error) {
-		return reason;
-	}
-	return new Error(String(reason));
-}
-
-function installGlobalErrorHandlers(telemetry: Telemetry): void {
-	window.addEventListener("error", (event: ErrorEvent): void => {
-		telemetry.recordError(toError(event.error ?? event.message));
-	});
-	window.addEventListener(
-		"unhandledrejection",
-		(event: PromiseRejectionEvent): void => {
-			telemetry.recordError(toError(event.reason));
-		},
-	);
-}
-
-export function initRum(): Telemetry {
-	const { appMonitorId, identityPoolId, region } = getRuntimeConfig().rum;
-	if (!appMonitorId) {
-		return noopTelemetry;
-	}
-
-	const rum = new AwsRum(appMonitorId, "1.0.0", region, {
-		sessionSampleRate: 1,
-		identityPoolId,
-		endpoint: `https://dataplane.rum.${region}.amazonaws.com`,
-		telemetries: ["performance"],
-		disableAutoPageView: true,
-		allowCookies: false,
-		enableXRay: false,
-	});
-
-	const telemetry = createRumTelemetry(rum);
-	installGlobalErrorHandlers(telemetry);
-	return telemetry;
-}

package/src/lib/search-conversion.ts

@@ -1,23 +0,0 @@
-import type { SearchConversion } from "@remit/ui";
-import { createContext, useContext } from "react";
-
-/**
- * The active query as clauses, which is what the wizard's search entry opens on
- * (#484). One conversion, computed once in `MailListHeader`, so the reason the
- * make-this-a-filter affordance gives and the clauses the wizard seeds from
- * cannot disagree.
- *
- * Its own context rather than a field on the list header chrome (#506): the
- * conversion is derived from the query text, so it is a new object on every
- * keystroke, and carrying it on the chrome meant the chrome was too — which
- * re-rendered every row of the virtualized list for a character the list does
- * not show. The wizard is the only reader, and it is mounted once.
- *
- * Absent when no query is up.
- */
-export const SearchConversionContext = createContext<
-	SearchConversion | undefined
->(undefined);
-
-export const useSearchConversion = (): SearchConversion | undefined =>
-	useContext(SearchConversionContext);

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

@@ -1,77 +0,0 @@
-/**
- * The spelling the address bar is written in.
- *
- * The router reads a repeated param as a list already; this is the other half,
- * and the assertions are the round trip — anything written here has to come
- * back the value it went in as.
- */
-import assert from "node:assert/strict";
-import { describe, it } from "node:test";
-import { parseSearchWith } from "@tanstack/react-router";
-import { stringifySearch } from "./search-params.js";
-
-/** The router's own reader, which this module has to stay symmetrical with. */
-const parseSearch = parseSearchWith(JSON.parse);
-
-/** Spread, because the router's reader answers with a null-prototype object. */
-const roundTrip = (search: Record<string, unknown>): unknown => ({
-	...parseSearch(stringifySearch(search)),
-});
-
-describe("stringifySearch", () => {
-	it("writes a list as the repeated params it is", () => {
-		assert.equal(
-			stringifySearch({ calendarId: ["cal_a", "cal_b"] }),
-			"?calendarId=cal_a&calendarId=cal_b",
-		);
-	});
-
-	it("writes a list of one as one param", () => {
-		assert.equal(
-			stringifySearch({ calendarId: ["cal_a"] }),
-			"?calendarId=cal_a",
-		);
-	});
-
-	it("writes nothing for an empty list, an absent value, or no params", () => {
-		assert.equal(stringifySearch({ calendarId: [] }), "");
-		assert.equal(stringifySearch({ q: undefined }), "");
-		assert.equal(stringifySearch({}), "");
-	});
-
-	it("leaves an ordinary value the way the router already wrote it", () => {
-		assert.equal(stringifySearch({ q: "invoice" }), "?q=invoice");
-		assert.equal(stringifySearch({ wizard: "review" }), "?wizard=review");
-	});
-
-	it("percent-encodes what a query string cannot carry raw", () => {
-		assert.equal(stringifySearch({ q: "from:ada &c" }), "?q=from%3Aada+%26c");
-	});
-});
-
-describe("what is written comes back", () => {
-	it("gives a list back as a list", () => {
-		assert.deepEqual(roundTrip({ calendarId: ["cal_a", "cal_b"] }), {
-			calendarId: ["cal_a", "cal_b"],
-		});
-	});
-
-	it("gives text that reads as JSON back as text", () => {
-		// Unquoted, the reader would hand back a boolean and a number.
-		assert.deepEqual(roundTrip({ q: "true" }), { q: "true" });
-		assert.deepEqual(roundTrip({ q: "2026" }), { q: "2026" });
-		assert.deepEqual(roundTrip({ q: "[draft]" }), { q: "[draft]" });
-	});
-
-	it("gives ordinary text back unchanged", () => {
-		assert.deepEqual(roundTrip({ q: "from:ada invoice" }), {
-			q: "from:ada invoice",
-		});
-	});
-
-	it("gives a structure back as a structure", () => {
-		assert.deepEqual(roundTrip({ filter: { unread: true } }), {
-			filter: { unread: true },
-		});
-	});
-});

package/src/lib/search-params.ts

@@ -1,59 +0,0 @@
-/**
- * How the address bar spells the query string.
- *
- * The router's own serializer writes one param per key, so a list of values
- * comes out as a JSON blob — `?calendarId=%5B%22cal_a%22%2C%22cal_b%22%5D`. A
- * list of values is what a query string has always been able to say, and
- * saying it the ordinary way is what makes a link hand-writable, readable in a
- * log, and usable by anything that is not this router. The router's own reader
- * already collects repeated params into an array, so only the writing side was
- * missing.
- *
- * Every other value keeps the spelling the router already gives it, including
- * the quoting that text reading as JSON needs: the reader JSON-parses whatever
- * it can, so `?q=true` would come back the boolean unless the string went in
- * quoted.
- */
-
-/**
- * Whether reading this text back would produce something other than the text.
- *
- * Decided by shape rather than by parsing it, because a block try/catch is not
- * available and an over-quoted string still reads back as itself — the reader
- * unquotes it — where an under-quoted one comes back the wrong type.
- */
-const readsAsJson = (value: string): boolean =>
-	value === "true" ||
-	value === "false" ||
-	value === "null" ||
-	/^["[{]/.test(value) ||
-	(value !== "" && Number.isFinite(Number(value)));
-
-/** The router's own value spelling. */
-function stringifyValue(value: unknown): string {
-	if (typeof value === "object" && value !== null) return JSON.stringify(value);
-	if (typeof value === "string")
-		return readsAsJson(value) ? JSON.stringify(value) : value;
-	return String(value);
-}
-
-/**
- * A query string, with a list written as the repeated params it is.
- *
- * `undefined` is absent rather than empty: a fact the view has nothing to say
- * about does not belong in the address at all.
- */
-export function stringifySearch(search: Record<string, unknown>): string {
-	const params = new URLSearchParams();
-	for (const key of Object.keys(search)) {
-		const value = search[key];
-		if (value === undefined) continue;
-		if (Array.isArray(value)) {
-			for (const entry of value) params.append(key, stringifyValue(entry));
-			continue;
-		}
-		params.set(key, stringifyValue(value));
-	}
-	const written = params.toString();
-	return written === "" ? "" : `?${written}`;
-}

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

@@ -1,24 +0,0 @@
-import assert from "node:assert";
-import { describe, test } from "node:test";
-import { normalizeSearchQuery } from "./search-query.js";
-
-describe("normalizeSearchQuery", () => {
-	test("lowercases an ASCII query", () => {
-		assert.equal(normalizeSearchQuery("ALICE"), "alice");
-	});
-
-	test("trims surrounding whitespace", () => {
-		assert.equal(normalizeSearchQuery("   Bob  "), "bob");
-	});
-
-	test("returns an empty string for whitespace-only input", () => {
-		assert.equal(normalizeSearchQuery("   "), "");
-	});
-
-	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.
-		assert.equal(normalizeSearchQuery("ÄPPEL"), "äppel");
-		assert.equal(normalizeSearchQuery("STRAẞE"), "straße");
-	});
-});

package/src/lib/search-query.ts

@@ -1,11 +0,0 @@
-// Pre-query normalizer for the mail search bar. We keep the user's typed
-// casing visible in the input and the "results for X" header, but the value
-// sent to the backend (and used as the React Query cache key) is folded to
-// lowercase so equivalent searches collide on the same cache entry and the
-// server-side comparison is case-insensitive.
-//
-// `toLocaleLowerCase` (no locale arg) uses the host locale, which handles
-// Unicode edge cases that ASCII `toLowerCase` mishandles — most notably the
-// Turkish dotless I (`I` → `ı` / `İ` → `i`).
-export const normalizeSearchQuery = (query: string): string =>
-	query.trim().toLocaleLowerCase();

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

@@ -1,20 +0,0 @@
-import assert from "node:assert";
-import { describe, test } from "node:test";
-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", () => {
-		const result = threadToSearchResult({
-			messageId: "m1",
-			threadId: "t1",
-			mailboxId: "mb1",
-			subject: "Hi",
-			isRead: true,
-			sentDate: 1_700_000_000_000,
-		} as RemitImapThreadMessageResponse);
-		assert.strictEqual(result.id, "m1");
-		assert.strictEqual(result.threadId, "t1");
-		assert.strictEqual(result.mailboxId, "mb1");
-	});
-});

package/src/lib/search-result.ts

@@ -1,55 +0,0 @@
-/**
- * Map the row shapes the app carries onto the kit `SearchResult` the search
- * results render: the per-mailbox `RemitImapThreadMessageResponse` and the
- * brief/flagged `ThreadRowData`.
- */
-import type { RemitImapThreadMessageResponse } from "@remit/api-http-client/types.gen.ts";
-import type { SearchResult, ThreadRowData } from "@remit/ui";
-import { formatEmailDate } from "./format.js";
-import {
-	type ResultFolderIndex,
-	resolveResultFolder,
-} from "./result-folder.js";
-
-export function threadToSearchResult(
-	thread: RemitImapThreadMessageResponse,
-	folders?: ResultFolderIndex,
-): SearchResult {
-	const { folder } = resolveResultFolder(folders, [thread.mailboxId]);
-	return {
-		id: thread.messageId,
-		sender: thread.fromName ?? thread.fromEmail ?? "Unknown",
-		...(thread.fromEmail ? { senderEmail: thread.fromEmail } : {}),
-		subject: thread.subject ?? "(No subject)",
-		snippet: thread.snippet ?? "",
-		date: formatEmailDate(thread.sentDate),
-		unread: !thread.isRead,
-		flagged: thread.hasStars === true,
-		threadId: thread.threadId,
-		mailboxId: thread.mailboxId,
-		...(folder ? { folder } : {}),
-	};
-}
-
-export function rowToSearchResult(
-	row: ThreadRowData,
-	folders?: ResultFolderIndex,
-): SearchResult {
-	const { folder } = resolveResultFolder(
-		folders,
-		row.mailboxId ? [row.mailboxId] : [],
-	);
-	return {
-		id: row.id,
-		sender: row.fromName,
-		...(row.fromEmail ? { senderEmail: row.fromEmail } : {}),
-		subject: row.subject,
-		snippet: row.snippet,
-		date: row.timeLabel,
-		unread: !row.isRead,
-		flagged: row.starred === true,
-		...(row.threadId ? { threadId: row.threadId } : {}),
-		...(row.mailboxId ? { mailboxId: row.mailboxId } : {}),
-		...(folder ? { folder } : {}),
-	};
-}

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

@@ -1,174 +0,0 @@
-import assert from "node:assert/strict";
-import { describe, it } from "node:test";
-import {
-	type BrowsedList,
-	MAIL_BRIEF_ROUTE_ID,
-	MAIL_FLAGGED_ROUTE_ID,
-	MAIL_MAILBOX_ROUTE_ID,
-	MAIL_OUTBOX_ROUTE_ID,
-} from "./mail-route";
-import {
-	isScopedRoute,
-	resultsScopeForRoute,
-	SEARCH_SCOPE_CHIP_ID,
-	scopeLabelForMailboxName,
-	searchScopeForRoute,
-} from "./search-scope";
-
-const listFor: Record<string, BrowsedList["list"]> = {
-	[MAIL_BRIEF_ROUTE_ID]: "brief",
-	[MAIL_FLAGGED_ROUTE_ID]: "flagged",
-	[MAIL_OUTBOX_ROUTE_ID]: "outbox",
-	[MAIL_MAILBOX_ROUTE_ID]: "mailbox",
-};
-
-const browsing = (routeId: string): BrowsedList => ({
-	list: listFor[routeId],
-	mailboxId: undefined,
-});
-
-describe("searchScopeForRoute", () => {
-	it("gives the daily brief no scope — it is the global view", () => {
-		assert.deepEqual(searchScopeForRoute(browsing(MAIL_BRIEF_ROUTE_ID)), {
-			kind: "global",
-		});
-	});
-
-	it("scopes a mailbox route to its sidebar label", () => {
-		assert.deepEqual(
-			searchScopeForRoute(browsing(MAIL_MAILBOX_ROUTE_ID), "Spam"),
-			{
-				kind: "scoped",
-				chip: { id: SEARCH_SCOPE_CHIP_ID, label: "in:spam" },
-			},
-		);
-	});
-
-	it("is pending, not global, while the mailbox name resolves", () => {
-		// The list under the bar is already one mailbox. A chip reading a raw
-		// uuid is worse than no chip, but the field must not claim to search
-		// everything either.
-		assert.deepEqual(
-			searchScopeForRoute(browsing(MAIL_MAILBOX_ROUTE_ID), null),
-			{ kind: "pending" },
-		);
-	});
-
-	it("scopes the starred view", () => {
-		const scope = searchScopeForRoute(browsing(MAIL_FLAGGED_ROUTE_ID));
-		assert.equal(scope.kind === "scoped" && scope.chip.label, "is:starred");
-	});
-
-	it("scopes the outbox", () => {
-		const scope = searchScopeForRoute(browsing(MAIL_OUTBOX_ROUTE_ID));
-		assert.equal(scope.kind === "scoped" && scope.chip.label, "in:outbox");
-	});
-
-	it("prefers the virtual views over a lingering mailbox name", () => {
-		const scope = searchScopeForRoute(browsing(MAIL_OUTBOX_ROUTE_ID), "Spam");
-		assert.equal(scope.kind === "scoped" && scope.chip.label, "in:outbox");
-	});
-});
-
-describe("isScopedRoute", () => {
-	// This predicate decides whether a typed `in:` is recognized at all, so
-	// every route has to answer it the same way it answers for its chip: a
-	// route showing a scope chip must not also honour a competing typed term.
-	for (const routeId of [
-		MAIL_MAILBOX_ROUTE_ID,
-		MAIL_FLAGGED_ROUTE_ID,
-		MAIL_OUTBOX_ROUTE_ID,
-	]) {
-		it(`${routeId} carries a scope`, () => {
-			assert.equal(isScopedRoute(browsing(routeId)), true);
-		});
-	}
-
-	it("the daily brief carries none", () => {
-		assert.equal(isScopedRoute(browsing(MAIL_BRIEF_ROUTE_ID)), false);
-	});
-
-	it("agrees with the chip on every route", () => {
-		for (const routeId of [
-			MAIL_BRIEF_ROUTE_ID,
-			MAIL_MAILBOX_ROUTE_ID,
-			MAIL_FLAGGED_ROUTE_ID,
-			MAIL_OUTBOX_ROUTE_ID,
-		]) {
-			const scope = searchScopeForRoute(browsing(routeId), "Spam");
-			assert.equal(
-				isScopedRoute(browsing(routeId)),
-				scope.kind === "scoped",
-				`${routeId} disagrees about whether it has a scope`,
-			);
-		}
-	});
-});
-
-describe("scopeLabelForMailboxName", () => {
-	it("lower-cases so the chip reads like the operator it mimics", () => {
-		assert.equal(scopeLabelForMailboxName("Archive"), "in:archive");
-	});
-
-	it("quotes a multi-word folder so the chip stays one term", () => {
-		assert.equal(scopeLabelForMailboxName("Work  Stuff"), 'in:"work stuff"');
-	});
-});
-
-// The invariant: no chip means global, a chip means every engine on that route
-// respects it.
-describe("resultsScopeForRoute", () => {
-	it("passes the global search through as global", () => {
-		assert.deepEqual(
-			resultsScopeForRoute(browsing(MAIL_BRIEF_ROUTE_ID), { kind: "global" }),
-			{
-				kind: "global",
-			},
-		);
-	});
-
-	it("treats a mailbox whose name has not loaded as a folder, not global", () => {
-		assert.deepEqual(
-			resultsScopeForRoute(browsing(MAIL_MAILBOX_ROUTE_ID), {
-				kind: "pending",
-			}),
-			{
-				kind: "folder",
-			},
-		);
-	});
-
-	it("carries the mailbox's role so a Spam search shows its rows", () => {
-		assert.deepEqual(
-			resultsScopeForRoute(
-				browsing(MAIL_MAILBOX_ROUTE_ID),
-				{
-					kind: "scoped",
-					chip: { id: SEARCH_SCOPE_CHIP_ID, label: "in:spam" },
-				},
-				"junk",
-			),
-			{ kind: "folder", role: "junk" },
-		);
-	});
-
-	it("calls starred a collection, so its spam is not dropped", () => {
-		assert.deepEqual(
-			resultsScopeForRoute(browsing(MAIL_FLAGGED_ROUTE_ID), {
-				kind: "scoped",
-				chip: { id: SEARCH_SCOPE_CHIP_ID, label: "is:starred" },
-			}),
-			{ kind: "collection" },
-		);
-	});
-
-	it("keeps the outbox a folder — it is one queue, and never holds spam", () => {
-		assert.deepEqual(
-			resultsScopeForRoute(browsing(MAIL_OUTBOX_ROUTE_ID), {
-				kind: "scoped",
-				chip: { id: SEARCH_SCOPE_CHIP_ID, label: "in:outbox" },
-			}),
-			{ kind: "folder" },
-		);
-	});
-});

package/src/lib/search-scope.ts

@@ -1,140 +0,0 @@
-/**
- * The search scope is the route.
- *
- * Search is the app's search: one field in the top bar, and by default it
- * covers everything. Navigating the sidebar narrows it, and the narrowing is
- * shown as one chip in the field — `in:spam` while viewing Spam. The chip is
- * derived from the active route, not parsed out of the typed text: the route
- * decides which pane is mounted and which mail that pane reads, and the chip is
- * that decision made visible. Navigating elsewhere replaces the chip because the
- * route changed.
- *
- * Removing the chip means "search everything", which is the daily brief — the
- * cross-account view whose scope is nothing. So removal is a navigation to
- * `/mail/brief` carrying the query, not an edit of the query text.
- *
- * There is exactly one scope, and it is this one. A typed `in:` term is a
- * second, competing answer to "which mailbox", so it is recognized only where
- * there is no route scope to compete with — see `isScopedRoute` and
- * `useSearchTokenContext`. Every engine parses the query through that one
- * context, so a scoped view cannot show a chip for a term it is ignoring.
- *
- * Pure functions only. `useSearchScope` binds these to the router.
- */
-import type { FolderRole, SearchScope as ResultsScope } from "@remit/ui";
-import type { BrowsedList } from "./mail-route";
-
-/**
- * Chip id of the scope chip. The top bar owns exactly one, so a fixed id is
- * enough of a removal handle.
- */
-export const SEARCH_SCOPE_CHIP_ID = "search-scope";
-
-export interface SearchScope {
-	id: string;
-	/** What the chip reads, e.g. `in:spam`. */
-	label: string;
-}
-
-/**
- * What the field can say about its own scope.
- *
- * `pending` is the mailbox route before its name has loaded: the list under the
- * bar is already one mailbox, so the field must not claim to search everything,
- * but there is no name yet to put in a chip. It is a real third state, not a
- * flavour of `global`.
- */
-export type SearchScopeState =
-	| { kind: "global" }
-	| { kind: "pending" }
-	| { kind: "scoped"; chip: SearchScope };
-
-/**
- * Render a mailbox name as an `in:` term. Lower-cased so it reads like the
- * operator it mimics, and quoted when the name has whitespace so the chip
- * stays one legible term.
- */
-export function scopeLabelForMailboxName(name: string): string {
-	const collapsed = name.trim().replace(/\s+/g, " ").toLowerCase();
-	return /\s/.test(collapsed) ? `in:"${collapsed}"` : `in:${collapsed}`;
-}
-
-/**
- * True on the routes whose pane reads one mailbox or one virtual collection.
- * These carry a scope; the daily brief (and any route not recognized here) does
- * not.
- */
-export function isScopedRoute(browsed: BrowsedList): boolean {
-	return browsed.list !== undefined && browsed.list !== "brief";
-}
-
-/**
- * The mailbox id carried by a mailbox route, or `undefined` on any other route.
- */
-export function routeMailboxId(browsed: BrowsedList): string | undefined {
-	return browsed.list === "mailbox" ? browsed.mailboxId : undefined;
-}
-
-/**
- * The scope of the active route.
- *
- * `mailboxName` is the sidebar's own label for the mailbox route (resolved by
- * `useCurrentMailboxName`). Until it resolves the scope is `pending`: no chip,
- * because a chip reading a raw uuid is worse than none, but not `global`
- * either, because the list underneath is already narrowed.
- */
-export function searchScopeForRoute(
-	browsed: BrowsedList,
-	mailboxName?: string | null,
-): SearchScopeState {
-	if (browsed.list === "flagged") {
-		return {
-			kind: "scoped",
-			chip: { id: SEARCH_SCOPE_CHIP_ID, label: "is:starred" },
-		};
-	}
-	if (browsed.list === "outbox") {
-		return {
-			kind: "scoped",
-			chip: { id: SEARCH_SCOPE_CHIP_ID, label: "in:outbox" },
-		};
-	}
-	if (browsed.list === "mailbox") {
-		if (!mailboxName) return { kind: "pending" };
-		return {
-			kind: "scoped",
-			chip: {
-				id: SEARCH_SCOPE_CHIP_ID,
-				label: scopeLabelForMailboxName(mailboxName),
-			},
-		};
-	}
-	return { kind: "global" };
-}
-
-/**
- * The route's scope as the results list understands it.
- *
- * `is:starred` is a collection, not a folder: it spans every folder a star can
- * be set in, so its rows carry provenance, and it is the user's own hand-picked
- * mail, so a starred spam message is shown rather than held back. Only a search
- * confined to a folder drops spam.
- *
- * The bar's third state, `pending`, is a mailbox route whose name has not
- * loaded. It maps to `folder` with no role yet, because the list underneath is
- * already one mailbox; calling it global for that one frame would flash folder
- * labels and a spam offer and then retract them.
- *
- * `role` is the appointed role of the mailbox the route is on, which is what
- * makes a search scoped to Spam show its rows instead of dropping them. A
- * folder nobody appointed carries no role.
- */
-export function resultsScopeForRoute(
-	browsed: BrowsedList,
-	state: SearchScopeState,
-	role?: FolderRole,
-): ResultsScope {
-	if (state.kind === "global") return { kind: "global" };
-	if (browsed.list === "flagged") return { kind: "collection" };
-	return { kind: "folder", ...(role ? { role } : {}) };
-}