@remit/web-client 0.0.321 → 0.0.323

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

@@ -13,7 +13,6 @@ import {
 	SEARCH_SCOPE_CHIP_ID,
 	scopeLabelForMailboxName,
 	searchScopeForRoute,
-	semanticMailboxScope,
 } from "./search-scope";
 
 const listFor: Record<string, BrowsedList["list"]> = {
@@ -117,97 +116,7 @@ describe("scopeLabelForMailboxName", () => {
 });
 
 // The invariant: no chip means global, a chip means every engine on that route
-// respects it. The semantic engine used to run unscoped inside a mailbox under
-// an "Everywhere" heading, contradicting the `in:` chip the same bar showed.
-describe("semanticMailboxScope — a chip binds every engine on the route", () => {
-	const mailbox = (mailboxId: string): BrowsedList => ({
-		list: "mailbox",
-		mailboxId,
-	});
-
-	it("pins a mailbox route to its own mailbox", () => {
-		assert.equal(
-			semanticMailboxScope({ browsed: mailbox("mbx-spam") }),
-			"mbx-spam",
-		);
-	});
-
-	it("a mailbox route ignores a caller asking for somewhere else", () => {
-		assert.equal(
-			semanticMailboxScope({
-				browsed: mailbox("mbx-spam"),
-				callerMailboxId: "mbx-archive",
-			}),
-			"mbx-spam",
-		);
-	});
-
-	it("a mailbox route ignores a typed in: term", () => {
-		assert.equal(
-			semanticMailboxScope({
-				browsed: mailbox("mbx-spam"),
-				inTokenMailboxId: "mbx-archive",
-			}),
-			"mbx-spam",
-		);
-	});
-
-	it("a mailbox route never widens to global", () => {
-		const scope = semanticMailboxScope({ browsed: mailbox("mbx-spam") });
-		assert.notEqual(scope, undefined, "a scoped route must carry a scope");
-	});
-
-	// Flagged and outbox are scoped to a collection, which the mailbox-scoped
-	// semantic API cannot express — so neither runs a semantic section, and
-	// neither may fall back to a typed `in:` under its own chip.
-	for (const routeId of [MAIL_FLAGGED_ROUTE_ID, MAIL_OUTBOX_ROUTE_ID]) {
-		it(`${routeId} never honours a typed in: term`, () => {
-			assert.equal(
-				semanticMailboxScope({
-					browsed: browsing(routeId),
-					inTokenMailboxId: "mbx-archive",
-				}),
-				undefined,
-			);
-		});
-	}
-
-	it("the daily brief is global — no route scope, no chip", () => {
-		assert.equal(
-			semanticMailboxScope({ browsed: browsing(MAIL_BRIEF_ROUTE_ID) }),
-			undefined,
-		);
-	});
-
-	it("the daily brief is the one route a typed in: narrows", () => {
-		assert.equal(
-			semanticMailboxScope({
-				browsed: browsing(MAIL_BRIEF_ROUTE_ID),
-				inTokenMailboxId: "mbx-archive",
-			}),
-			"mbx-archive",
-		);
-	});
-
-	it("every scoped route resolves a scope or refuses to widen", () => {
-		// Pins the invariant across the route table rather than one case at a
-		// time: on a scoped route the resolved scope is either the route's own
-		// mailbox, or nothing that came from the typed query.
-		const scoped = [
-			{ browsed: mailbox("mbx-spam"), expected: "mbx-spam" },
-			{ browsed: browsing(MAIL_FLAGGED_ROUTE_ID), expected: undefined },
-			{ browsed: browsing(MAIL_OUTBOX_ROUTE_ID), expected: undefined },
-		];
-		for (const { browsed: m, expected } of scoped) {
-			assert.equal(isScopedRoute(m), true);
-			assert.equal(
-				semanticMailboxScope({ browsed: m, inTokenMailboxId: "mbx-elsewhere" }),
-				expected,
-			);
-		}
-	});
-});
-
+// respects it.
 describe("resultsScopeForRoute", () => {
 	it("passes the global search through as global", () => {
 		assert.deepEqual(

package/src/lib/search-scope.ts

@@ -75,35 +75,6 @@ export function routeMailboxId(browsed: BrowsedList): string | undefined {
 	return browsed.list === "mailbox" ? browsed.mailboxId : undefined;
 }
 
-/**
- * The mailbox every engine on the active route must be pinned to.
- *
- * No chip means global; a chip means nothing on the route reaches past it. The
- * semantic engine used to run unscoped inside a mailbox under an "Everywhere"
- * heading, which contradicted the `in:` chip the same bar was showing, so the
- * scope is now resolved from the route here rather than left to each caller:
- *
- *  - A mailbox route pins to its own mailbox. It beats both the caller's
- *    argument and a typed `in:`, so no call site can widen or redirect it.
- *  - Any other scoped route (flagged, outbox) never falls back to a typed
- *    `in:`. Its scope is a collection rather than a folder, which the semantic
- *    API cannot express — which is why neither view runs a semantic section
- *    today. Honouring `in:` there would be a search reaching past a chip that
- *    promises otherwise.
- *  - An unscoped route (the daily brief) is global unless a typed `in:` narrows
- *    it, which is the one place `useSearchTokenContext` resolves that term.
- */
-export function semanticMailboxScope(args: {
-	browsed: BrowsedList;
-	callerMailboxId?: string;
-	inTokenMailboxId?: string;
-}): string | undefined {
-	const fromRoute = routeMailboxId(args.browsed);
-	if (fromRoute) return fromRoute;
-	if (isScopedRoute(args.browsed)) return args.callerMailboxId;
-	return args.callerMailboxId ?? args.inTokenMailboxId;
-}
-
 /**
  * The scope of the active route.
  *

package/src/lib/search-surface.ts

@@ -3,8 +3,8 @@ import type { LayoutTier } from "@/hooks/useLayoutTier";
 /**
  * Which surface a list view shows its search on. Two surfaces exist:
  *
- * - the read-only two-engine `SearchResults` panel (literal "Top matches" +
- *   semantic "Related", cross-folder, spam offer), swapped into the list-pane
+ * - the read-only `SearchResults` panel ("Top matches", cross-folder, spam
+ *   offer), swapped into the list-pane
  *   body on tablet/desktop while searching; and
  * - the view's own body — for the mailbox route a selectable `MessageList` whose
  *   threads filter to the committed search — which hosts multi-select and the

package/src/lib/search-token-index.test.ts

@@ -69,10 +69,6 @@ describe("buildAccountNameIndex", () => {
 		]);
 		assert.equal(index.get("work"), "acct-1");
 	});
-
-	it("empty accounts yields an empty index", () => {
-		assert.deepEqual(buildAccountNameIndex([]), new Map());
-	});
 });
 
 describe("buildMailboxNameIndex", () => {
@@ -107,10 +103,6 @@ describe("buildMailboxNameIndex", () => {
 		]);
 		assert.equal(index.size, 0);
 	});
-
-	it("empty input yields an empty index", () => {
-		assert.deepEqual(buildMailboxNameIndex([]), new Map());
-	});
 });
 
 describe("buildMailboxSuggestionValues", () => {

package/src/lib/search-tokens.ts

@@ -4,20 +4,20 @@
  * into removable chips. Only tokens the search APIs can actually honor are
  * recognized — everything else is left untouched as ordinary free-text words.
  *
- * The vocabulary is the search parameters `threadOperationsSearchThreads` and
- * `semanticSearchOperationsSemanticSearch` take, said in words:
+ * The vocabulary is the search parameters `threadOperationsSearchThreads`
+ * takes, said in words:
  *
- * | token                  | thread search      | semantic search        |
- * | ---------------------- | ------------------ | ---------------------- |
- * | `from:`                | `from`             | —                      |
- * | `subject:`             | `subject`          | —                      |
- * | `category:`            | `category[]`       | `category`             |
- * | `is:unread` / `is:read`| `unread`           | `isRead`               |
- * | `is:starred`           | `starred`          | `hasStars`             |
- * | `has:attachment`       | `attachments`      | `hasAttachment`        |
- * | `before:` / `after:`   | —                  | `sentDateTo` / `From`  |
- * | `in:`                  | route scope        | `mailboxId`            |
- * | `account:`             | —                  | —                      |
+ * | token                  | thread search      |
+ * | ---------------------- | ------------------ |
+ * | `from:`                | `from`             |
+ * | `subject:`             | `subject`          |
+ * | `category:`            | `category[]`       |
+ * | `is:unread` / `is:read`| `unread`           |
+ * | `is:starred`           | `starred`          |
+ * | `has:attachment`       | `attachments`      |
+ * | `before:` / `after:`   | —                  |
+ * | `in:`                  | route scope        |
+ * | `account:`             | —                  |
  *
  * A token an engine has no parameter for is applied client-side over the rows
  * that engine returned (`matchesSearchTokens` in `lib/brief.ts`), never dropped.
@@ -35,9 +35,8 @@
  *
  * Pure functions only: no React, no fetch — the name index is injected, not
  * fetched here. `MailListHeader` renders the chips and the callers that issue
- * search requests (`MailboxPane`, `DailyBrief`, `FlaggedList`,
- * `useSemanticSearch`) apply the parsed tokens to the params each engine's API
- * actually supports.
+ * search requests (`MailboxPane`, `DailyBrief`, `FlaggedList`) apply the
+ * parsed tokens to the params each engine's API actually supports.
  */
 
 import type { RemitImapMessageCategory } from "@remit/api-http-client/types.gen.ts";

package/src/lib/starred-rows.test.ts

@@ -64,8 +64,4 @@ describe("dedupeByThread", () => {
 			["t1", "t2", "t3"],
 		);
 	});
-
-	test("an empty list stays empty", () => {
-		assert.deepEqual(dedupeByThread([]), []);
-	});
 });

package/src/lib/sync-warning-entries.ts

@@ -0,0 +1,33 @@
+import type { RemitImapSyncWarningResponse } from "@remit/api-http-client/types.gen.ts";
+import { syncWarningState } from "@remit/data-ports/sync-warning-state";
+import {
+	canonicalRoleLabel,
+	folderLeaf,
+	type ResultFolder,
+	type SyncWarningEntry,
+} from "@remit/ui";
+
+export const UNLISTED_FOLDER_LABEL = "A folder no longer listed";
+
+function folderLabel(folder: ResultFolder | undefined): string {
+	if (folder?.role) return canonicalRoleLabel(folder.role);
+	if (folder?.providerPath) {
+		return folderLeaf(folder.providerPath, folder.hierarchyDelimiter);
+	}
+	return UNLISTED_FOLDER_LABEL;
+}
+
+export function toSyncWarningEntry(
+	wire: RemitImapSyncWarningResponse,
+	folder: ResultFolder | undefined,
+): SyncWarningEntry {
+	return {
+		syncWarningId: wire.syncWarningId,
+		uid: wire.uid,
+		stage: wire.stage,
+		state: syncWarningState(wire.stage, wire.attempts),
+		attempts: wire.attempts,
+		createdAt: wire.createdAt,
+		folderLabel: folderLabel(folder),
+	};
+}

package/src/lib/wizard-selection.ts

@@ -2,7 +2,7 @@ import type { ThreadRowData, WizardMessage } from "@remit/ui";
 
 /** A ticked row, as the wizard's samples and its clause prefill read it. */
 export interface WizardSelectionMessage extends WizardMessage {
-	/** Sender address — the widen's literal fallback and the prefill match on it. */
+	/** Sender address — the properties prefill matches on it. */
 	email: string;
 	/**
 	 * Owning account, which a bulk run splits its batches by (#872) — the bulk

package/src/mocks/story-frame/mail-handlers.ts

@@ -27,7 +27,6 @@ import type {
 	RemitImapThreadMessageResponse,
 	RemitImapThreadSearchResponse,
 	RemitImapUpdateOutboxMessageInput,
-	SemanticSearchOperationsSemanticSearchResponse,
 	SyncOperationsGetSyncStatusResponse,
 	SyncOperationsTriggerSyncResponse,
 	SystemOperationsGetSystemUpdateResponse,
@@ -46,6 +45,7 @@ export interface MailHandlerOptions {
 	pageSize?: number;
 	holdLaterPages?: boolean;
 	draftSave?: "saves" | "refused";
+	draftDiscard?: "discards" | "refused";
 	send?: "sends" | "holds";
 	search?: "answers" | "holds";
 	config?: "answers" | "holds";
@@ -64,6 +64,7 @@ const bodyPartsOf = (
 		? [
 				{
 					bodyPartId: `part-${row.messageId}`,
+					partPath: "1",
 					mediaType: "APPLICATION",
 					mediaSubtype: "pdf",
 					sizeOctets: 48_000,
@@ -76,6 +77,7 @@ const bodyPartsOf = (
 		: [
 				{
 					bodyPartId: `part-${row.messageId}`,
+					partPath: "1",
 					mediaType: "TEXT",
 					mediaSubtype: "plain",
 					sizeOctets: (row.snippet ?? "").length,
@@ -215,6 +217,12 @@ export const mailHandlers = (
 			{ status: 409 },
 		);
 
+	const refuseDiscard = () =>
+		HttpResponse.json(
+			{ status: 409, message: "The draft is already gone." },
+			{ status: 409 },
+		);
+
 	return [
 		http.get(`${API}/config`, () =>
 			options.config === "holds"
@@ -485,6 +493,7 @@ export const mailHandlers = (
 			return HttpResponse.json<RemitImapOutboxMessageResponse>(queued);
 		}),
 		http.delete(`${API}/outbox/:outboxMessageId`, ({ params }) => {
+			if (options.draftDiscard === "refused") return refuseDiscard();
 			outbox.delete(String(params.outboxMessageId));
 			return new HttpResponse(null, { status: 204 });
 		}),
@@ -517,16 +526,11 @@ export const mailHandlers = (
 				{ items: [] },
 			),
 		),
-		http.get(`${API}/search/semantic`, () =>
-			options.search === "holds"
-				? pending()
-				: HttpResponse.json<SemanticSearchOperationsSemanticSearchResponse>({
-						items: [],
-					}),
-		),
 		http.get(`${API}/me/quarantine`, () =>
 			HttpResponse.json<MeOperationsListQuarantineResponse>({
 				entries: world.quarantine,
+				warnings: world.syncWarnings,
+				warningsTotal: world.syncWarningsTotal,
 			}),
 		),
 		http.get(`${API}/me/vip-suggestions`, () =>

package/src/mocks/story-frame/mail-world.ts

@@ -10,6 +10,7 @@ import type {
 	RemitImapOutboxMessageResponse,
 	RemitImapOutboxMessageStatus,
 	RemitImapQuarantineResponse,
+	RemitImapSyncWarningResponse,
 	RemitImapSystemUpdateResponse,
 	RemitImapThreadMessageResponse,
 	RemitImapVipSuggestionEntry,
@@ -26,6 +27,8 @@ export interface MailWorld {
 	threads: RemitImapThreadMessageResponse[];
 	outbox: RemitImapOutboxMessageResponse[];
 	quarantine: RemitImapQuarantineResponse[];
+	syncWarnings: RemitImapSyncWarningResponse[];
+	syncWarningsTotal: number;
 	vipSuggestions: RemitImapVipSuggestionEntry[];
 	addresses: RemitImapAddressResponse[];
 	systemUpdate: RemitImapSystemUpdateResponse;
@@ -427,6 +430,8 @@ export const mailWorld = (overrides: Partial<MailWorld> = {}): MailWorld => ({
 	threads: [...inboxThreads, ...serverDrafts],
 	outbox: [...remitDrafts, ...outboxInEveryStatus],
 	quarantine: [],
+	syncWarnings: [],
+	syncWarningsTotal: 0,
 	vipSuggestions: [],
 	addresses: [],
 	systemUpdate: {

package/src/routeTree.gen.ts

@@ -31,6 +31,7 @@ import { Route as SettingsFoldersRouteImport } from './routes/settings/folders'
 import { Route as SettingsLabelsRouteImport } from './routes/settings/labels'
 import { Route as SettingsSendersRouteImport } from './routes/settings/senders'
 import { Route as SettingsSuggestedVipsRouteImport } from './routes/settings/suggested-vips'
+import { Route as SettingsWarningsRouteImport } from './routes/settings/warnings'
 import { Route as CalendarViewDateRouteImport } from './routes/calendar/$view.$date'
 import { Route as MailMailboxIdIndexRouteImport } from './routes/mail/$mailboxId/index'
 import { Route as MailMailboxIdThreadIdRouteImport } from './routes/mail/$mailboxId/$threadId'
@@ -170,6 +171,11 @@ const SettingsSuggestedVipsRoute = SettingsSuggestedVipsRouteImport.update({
   path: '/suggested-vips',
   getParentRoute: () => SettingsRoute,
 } as any)
+const SettingsWarningsRoute = SettingsWarningsRouteImport.update({
+  id: '/warnings',
+  path: '/warnings',
+  getParentRoute: () => SettingsRoute,
+} as any)
 const CalendarViewDateRoute = CalendarViewDateRouteImport.update({
   id: '/$view/$date',
   path: '/$view/$date',
@@ -354,6 +360,7 @@ export interface FileRoutesByFullPath {
   '/settings/labels': typeof SettingsLabelsRoute
   '/settings/senders': typeof SettingsSendersRoute
   '/settings/suggested-vips': typeof SettingsSuggestedVipsRoute
+  '/settings/warnings': typeof SettingsWarningsRoute
   '/calendar/': typeof CalendarIndexRoute
   '/mail/': typeof MailIndexRoute
   '/settings/': typeof SettingsIndexRoute
@@ -399,6 +406,7 @@ export interface FileRoutesByTo {
   '/settings/labels': typeof SettingsLabelsRoute
   '/settings/senders': typeof SettingsSendersRoute
   '/settings/suggested-vips': typeof SettingsSuggestedVipsRoute
+  '/settings/warnings': typeof SettingsWarningsRoute
   '/calendar': typeof CalendarIndexRoute
   '/mail': typeof MailIndexRoute
   '/settings': typeof SettingsIndexRoute
@@ -447,6 +455,7 @@ export interface FileRoutesById {
   '/settings/labels': typeof SettingsLabelsRoute
   '/settings/senders': typeof SettingsSendersRoute
   '/settings/suggested-vips': typeof SettingsSuggestedVipsRoute
+  '/settings/warnings': typeof SettingsWarningsRoute
   '/calendar/': typeof CalendarIndexRoute
   '/mail/': typeof MailIndexRoute
   '/settings/': typeof SettingsIndexRoute
@@ -501,6 +510,7 @@ export interface FileRouteTypes {
     | '/settings/labels'
     | '/settings/senders'
     | '/settings/suggested-vips'
+    | '/settings/warnings'
     | '/calendar/'
     | '/mail/'
     | '/settings/'
@@ -546,6 +556,7 @@ export interface FileRouteTypes {
     | '/settings/labels'
     | '/settings/senders'
     | '/settings/suggested-vips'
+    | '/settings/warnings'
     | '/calendar'
     | '/mail'
     | '/settings'
@@ -593,6 +604,7 @@ export interface FileRouteTypes {
     | '/settings/labels'
     | '/settings/senders'
     | '/settings/suggested-vips'
+    | '/settings/warnings'
     | '/calendar/'
     | '/mail/'
     | '/settings/'
@@ -791,6 +803,13 @@ declare module '@tanstack/react-router' {
       preLoaderRoute: typeof SettingsSuggestedVipsRouteImport
       parentRoute: typeof SettingsRoute
     }
+    '/settings/warnings': {
+      id: '/settings/warnings'
+      path: '/warnings'
+      fullPath: '/settings/warnings'
+      preLoaderRoute: typeof SettingsWarningsRouteImport
+      parentRoute: typeof SettingsRoute
+    }
     '/calendar/$view/$date': {
       id: '/calendar/$view/$date'
       path: '/$view/$date'
@@ -1225,6 +1244,7 @@ interface SettingsRouteChildren {
   SettingsLabelsRoute: typeof SettingsLabelsRoute
   SettingsSendersRoute: typeof SettingsSendersRoute
   SettingsSuggestedVipsRoute: typeof SettingsSuggestedVipsRoute
+  SettingsWarningsRoute: typeof SettingsWarningsRoute
   SettingsIndexRoute: typeof SettingsIndexRoute
 }
 
@@ -1238,6 +1258,7 @@ const SettingsRouteChildren: SettingsRouteChildren = {
   SettingsLabelsRoute: SettingsLabelsRoute,
   SettingsSendersRoute: SettingsSendersRoute,
   SettingsSuggestedVipsRoute: SettingsSuggestedVipsRoute,
+  SettingsWarningsRoute: SettingsWarningsRoute,
   SettingsIndexRoute: SettingsIndexRoute,
 }
 

package/src/routes/calendar/$view.$date/index.tsx

@@ -5,9 +5,9 @@
  * as segments beside it.
  */
 
+import { EmptyState } from "@remit/ui";
 import { createFileRoute } from "@tanstack/react-router";
 import { CalendarDays } from "lucide-react";
-import { EmptyState } from "@/components/ui/EmptyState";
 
 function CalendarReadingPane() {
 	return (

package/src/routes/calendar.tsx

@@ -10,13 +10,14 @@
  * through and takes it back to their mail — nothing calendar-specific joins the
  * union, because none of the calendar's own chrome is a panel.
  */
+
 import { configOperationsGetConfigOptions } from "@remit/api-http-client/@tanstack/react-query.gen.ts";
+import { ErrorState } from "@remit/ui";
 import { useQuery } from "@tanstack/react-query";
 import { createFileRoute, Outlet, useNavigate } from "@tanstack/react-router";
 import { useCallback, useEffect } from "react";
 import { CalendarShellProvider } from "@/components/calendar/CalendarShell";
 import { MailNav } from "@/components/mail/MailNav";
-import { ErrorState } from "@/components/ui/ErrorState";
 import { isSinglePaneTier, useLayoutTier } from "@/hooks/useLayoutTier";
 import { panelsWithOverlay, useOpenPanels, useSetOpenPanels } from "@/routing";
 import "@/lib/client";

package/src/routes/mail/brief.tsx

@@ -7,6 +7,7 @@
  * message inside it are the segments below, which is why nothing here reads a
  * selection out of the query.
  */
+import { ErrorState } from "@remit/ui";
 import {
 	createFileRoute,
 	type ErrorComponentProps,
@@ -14,7 +15,6 @@ import {
 } from "@tanstack/react-router";
 import { MailShell } from "@/components/layout/MailShell";
 import { BriefPane } from "@/components/mail/BriefPane";
-import { ErrorState } from "@/components/ui/ErrorState";
 import { briefSearchSchema } from "@/lib/mail-search";
 import { useOpenThreadPath, useSearchMirror } from "@/routing";
 

package/src/routes/mail/flagged.tsx

@@ -6,6 +6,7 @@
  * route matched below this one, and the open thread and the message inside it
  * are the segments there rather than a selection read out of the query.
  */
+import { ErrorState } from "@remit/ui";
 import {
 	createFileRoute,
 	type ErrorComponentProps,
@@ -13,7 +14,6 @@ import {
 } from "@tanstack/react-router";
 import { MailShell } from "@/components/layout/MailShell";
 import { FlaggedPane } from "@/components/mail/FlaggedPane";
-import { ErrorState } from "@/components/ui/ErrorState";
 import { flaggedSearchSchema } from "@/lib/mail-search";
 import { useOpenThreadPath, useSearchMirror } from "@/routing";
 

package/src/routes/mail.tsx

@@ -2,7 +2,11 @@ import {
 	configOperationsGetConfigOptions,
 	unifiedThreadOperationsListAllThreadsOptions,
 } from "@remit/api-http-client/@tanstack/react-query.gen.ts";
-import { useTriageKeyboard } from "@remit/ui";
+import {
+	ErrorState,
+	KeyboardShortcutsModal,
+	useTriageKeyboard,
+} from "@remit/ui";
 import { useQuery } from "@tanstack/react-query";
 import {
 	createFileRoute,
@@ -16,8 +20,6 @@ import { ComposeFab } from "@/components/layout/ComposeFab";
 import { MailShellProvider } from "@/components/layout/MailShell";
 import { MailTopBar } from "@/components/layout/MailTopBar";
 import { MailNav } from "@/components/mail/MailNav";
-import { ErrorState } from "@/components/ui/ErrorState";
-import { KeyboardShortcutsModal } from "@/components/ui/KeyboardShortcutsModal";
 import { useKeyboardNavigation } from "@/hooks/useKeyboardNavigation";
 import { isSinglePaneTier, useLayoutTier } from "@/hooks/useLayoutTier";
 import { useMailboxNameIndex } from "@/hooks/useMailboxNameIndex";

package/src/routes/settings/-accounts.test.ts

@@ -80,11 +80,6 @@ describe("mapOauthError", () => {
 			`Expected code in message, got: ${result}`,
 		);
 	});
-
-	test("empty string returns generic fallback", () => {
-		const result = mapOauthError("");
-		assert.ok(typeof result === "string" && result.length > 0);
-	});
 });
 
 /**

package/src/routes/settings/-filters.test.ts

@@ -65,7 +65,6 @@ const filter: RemitImapFilterResponse = {
 	scope: "Standing",
 	state: "Active",
 	disabledReason: "None",
-	hasAnchor: false,
 	ruleChangedAt: 0,
 	actionChangedAt: 0,
 	matchOperator: "And",

package/src/routes/settings/-shared/help-copy.tsx

@@ -66,11 +66,26 @@ export const advancedHelp = (
 			<strong className="text-fg">Notification rules</strong> and data export
 			are coming in a future release.
 		</p>
+	</div>
+);
+
+export const warningsHelp = (
+	<div className="space-y-3">
+		<p>
+			A message that fails to sync three times shows here. A{" "}
+			<strong className="text-fg">retrying</strong> one is tried again every
+			sync; a body that <strong className="text-fg">gave up</strong> is not.
+			Either leaves the list once it syncs.
+		</p>
 		<p>
 			A message Remit cannot read is{" "}
 			<strong className="text-fg">set aside</strong> rather than skipped, and
 			the folder keeps syncing. Setting one aside is a defect in Remit, so every
 			entry can be reported with the diagnostics already attached.
 		</p>
+		<p>
+			A report carries no folder name, subject, address, message content or
+			error text.
+		</p>
 	</div>
 );

package/src/routes/settings/accounts.tsx

@@ -12,6 +12,7 @@ import {
 	Banner,
 	Button,
 	Dialog,
+	ErrorState,
 	type RowAction,
 	RowActions,
 	SettingsShell,
@@ -28,7 +29,6 @@ import {
 import { AccountFormPanel } from "@/components/settings/AccountFormPanel";
 import { AccountServices } from "@/components/settings/AccountServices";
 import { DangerZone } from "@/components/settings/DangerZone";
-import { ErrorState } from "@/components/ui/ErrorState";
 import {
 	REDIRECT_STALL_MESSAGE,
 	useRedirectEnded,

package/src/routes/settings/advanced.tsx

@@ -1,13 +1,11 @@
 /**
  * Advanced settings. Updates live here — and only here — with a full pane in
- * Settings › Advanced, alongside the messages sync could not read. Notification
- * rules and export are future scope.
+ * Settings › Advanced. Notification rules and export are future scope.
  */
 import { SettingsShell } from "@remit/ui";
 import { createFileRoute, useNavigate } from "@tanstack/react-router";
 import { useState } from "react";
 import { ConfigTransferPanel } from "@/components/settings/ConfigTransferPanel";
-import { QuarantinePanel } from "@/components/settings/QuarantinePanel";
 import { SelfUpdatePanel } from "@/components/settings/SelfUpdatePanel";
 import { TlsRootCaDownload } from "@/components/settings/TlsRootCaDownload";
 import { AppVersion } from "@/components/ui/AppVersion";
@@ -40,7 +38,6 @@ function AdvancedSettings() {
 			onBackToMail={() => void navigate({ to: "/mail" })}
 		>
 			<SelfUpdatePanel />
-			<QuarantinePanel />
 			<div className="mt-6 border-t border-line pt-4">
 				<ConfigTransferPanel />
 			</div>

package/src/routes/settings/filters.tsx

@@ -3,14 +3,13 @@ import {
 	mailboxOperationsListMailboxesOptions,
 } from "@remit/api-http-client/@tanstack/react-query.gen.ts";
 import type { RemitImapAccountResponse } from "@remit/api-http-client/types.gen.ts";
-import { type LabelOption, SettingsShell } from "@remit/ui";
+import { ErrorState, type LabelOption, SettingsShell } from "@remit/ui";
 import { useQuery } from "@tanstack/react-query";
 import { createFileRoute, useNavigate } from "@tanstack/react-router";
 import { useCallback, useMemo, useState } from "react";
 import { FilterActionError } from "@/components/settings/FilterActionError";
 import { FilterEditorSurface } from "@/components/settings/FilterEditorSurface";
 import { FiltersList } from "@/components/settings/FiltersList";
-import { ErrorState } from "@/components/ui/ErrorState";
 import {
 	useDeleteFilter,
 	useFilterList,