@remit/web-client 0.0.80 → 0.0.82

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remit/web-client",
-  "version": "0.0.80",
+  "version": "0.0.82",
   "type": "module",
   "description": "Remit web client, published as composable primitives — the app shell, auth shells, and runtime config. A distributor imports what it composes and bundles it.",
   "exports": {

package/src/components/mail/DailyBrief.tsx

@@ -41,6 +41,7 @@ import { useSemanticSearch } from "@/hooks/useSemanticSearch";
 import type { TriageContextUpdate } from "@/hooks/useTriageLayer";
 import { sortAccountsByCreatedAt } from "@/lib/account-order";
 import {
+	excludeMutedSenders,
 	groupBriefSections,
 	matchesBriefSearch,
 	matchesSearchTokens,
@@ -275,12 +276,14 @@ export function DailyBrief({
 	// `BriefSections` filter row's job, so the full per-category sections are
 	// handed to it; it groups, narrows, and flattens.
 	const filteredRows = useMemo<ThreadRowData[]>(() => {
-		const briefRows = (threadsData?.items ?? []).map(toThreadRowData);
+		const briefRows = excludeMutedSenders(threadsData?.items ?? []).map(
+			toThreadRowData,
+		);
 		// No free text: the brief list as it comes, order untouched.
 		const rows = sq
 			? mergeSearchRows(
 					briefRows.filter((t) => matchesBriefSearch(t, sq)),
-					(searchData?.items ?? []).map(toThreadRowData),
+					excludeMutedSenders(searchData?.items ?? []).map(toThreadRowData),
 				)
 			: briefRows;
 		return rows.filter(

package/src/components/settings/FilterEditor.render.test.ts

@@ -49,6 +49,7 @@ const filterFixture = (
 	state: "Active",
 	hasAnchor: false,
 	ruleChangedAt: 100,
+	actionChangedAt: 100,
 	matchOperator: "And",
 	literalClauses: [{ field: "From", value: "receipts@stripe.com" }],
 	actionLabelId: "None",

package/src/components/settings/FiltersList.render.test.ts

@@ -23,6 +23,7 @@ const filter = (
 	state: "Active",
 	hasAnchor: true,
 	ruleChangedAt: 0,
+	actionChangedAt: 0,
 	matchOperator: "And",
 	literalClauses: [],
 	actionLabelId: "None",

package/src/components/settings/settings-filter.stories.tsx

@@ -39,6 +39,7 @@ const makeFilter = (
 	state: "Active",
 	hasAnchor: false,
 	ruleChangedAt: 0,
+	actionChangedAt: 0,
 	matchOperator: "And",
 	literalClauses: [
 		{ field: "From", value: "receipts@stripe.com" },

package/src/hooks/useFilters.test.ts

@@ -34,6 +34,7 @@ const filter = (
 		state: "Active",
 		hasAnchor: false,
 		ruleChangedAt: 0,
+		actionChangedAt: 0,
 		matchOperator: "And",
 		literalClauses: [],
 		actionLabelId: "None",

package/src/hooks/useToggleStar.test.ts

@@ -28,6 +28,7 @@ const make = (
 	hasAttachment: false,
 	star: "none",
 	hasStars,
+	muted: false,
 	createdAt: 0,
 	updatedAt: 0,
 });

package/src/lib/brief.test.ts

@@ -3,6 +3,7 @@ import { describe, test } from "node:test";
 import type { RemitImapThreadMessageResponse } from "@remit/api-http-client/types.gen.ts";
 import type { ThreadRowData } from "@remit/ui";
 import {
+	excludeMutedSenders,
 	groupBriefSections,
 	matchesBriefSearch,
 	matchesSearchTokens,
@@ -32,6 +33,7 @@ function threadResponse(
 		hasStars: false,
 		star: "none",
 		senderTrust: "unknown",
+		muted: false,
 		createdAt: 0,
 		updatedAt: 0,
 		...overrides,
@@ -265,6 +267,81 @@ describe("groupBriefSections", () => {
 	});
 });
 
+// issue #301: `Address.flags.muted` is denormalized onto each row as
+// `muted`; the brief excludes those rows before grouping into sections.
+describe("excludeMutedSenders", () => {
+	test("drops a row whose sender is muted", () => {
+		const kept = threadResponse({ threadMessageId: "keep" });
+		const muted = threadResponse({ threadMessageId: "mute-me", muted: true });
+		const result = excludeMutedSenders([kept, muted]);
+		assert.deepStrictEqual(
+			result.map((t) => t.threadMessageId),
+			["keep"],
+		);
+	});
+
+	test("keeps rows whose sender is not muted", () => {
+		const rows = [
+			threadResponse({ threadMessageId: "a", muted: false }),
+			threadResponse({ threadMessageId: "b" }),
+		];
+		assert.strictEqual(excludeMutedSenders(rows).length, 2);
+	});
+
+	test("returns an empty array when every sender is muted", () => {
+		const rows = [
+			threadResponse({ threadMessageId: "a", muted: true }),
+			threadResponse({ threadMessageId: "b", muted: true }),
+		];
+		assert.deepStrictEqual(excludeMutedSenders(rows), []);
+	});
+
+	test("a muted sender's message is excluded from every section, not folded into uncategorized", () => {
+		const rows = [
+			threadResponse({
+				threadMessageId: "muted-personal",
+				messageId: "muted-personal",
+				category: "personal",
+				muted: true,
+			}),
+			threadResponse({
+				threadMessageId: "kept-personal",
+				messageId: "kept-personal",
+				category: "personal",
+			}),
+		];
+		const sections = groupBriefSections(
+			excludeMutedSenders(rows).map(toThreadRowData),
+		);
+		const allIds = sections.flatMap((s) => s.threads.map((t) => t.id));
+		assert.deepStrictEqual(allIds, ["kept-personal"]);
+		for (const section of sections) {
+			assert.ok(!section.threads.some((t) => t.id === "muted-personal"));
+		}
+	});
+
+	test("muting every candidate row leaves no sections (brief empty state)", () => {
+		const rows = [
+			threadResponse({
+				threadMessageId: "a",
+				messageId: "a",
+				category: "personal",
+				muted: true,
+			}),
+			threadResponse({
+				threadMessageId: "b",
+				messageId: "b",
+				category: "newsletter",
+				muted: true,
+			}),
+		];
+		const sections = groupBriefSections(
+			excludeMutedSenders(rows).map(toThreadRowData),
+		);
+		assert.deepStrictEqual(sections, []);
+	});
+});
+
 describe("matchesBriefSearch", () => {
 	const r = row({
 		id: "1",

package/src/lib/brief.ts

@@ -24,7 +24,11 @@
  * a high-volume mailbox read≠handled and unread≠important; unread is a
  * user-selectable filter chip instead.
  *
- * Muted senders (filtered by the caller) and empty sections are excluded.
+ * Muted senders and empty sections are excluded. Mute filtering happens in
+ * `excludeMutedSenders`, applied by the caller to the raw thread rows before
+ * `toThreadRowData`/grouping — the server denormalizes `muted` onto each row
+ * from the From address's flags (RFC 039 Decision 3, issue #301), so no
+ * client-side Address lookup is needed.
  */
 
 import type { RemitImapThreadMessageResponse } from "@remit/api-http-client/types.gen.ts";
@@ -67,6 +71,19 @@ export function toThreadRowData(
 	};
 }
 
+/**
+ * Excludes rows whose From address is muted (`thread.muted === true`,
+ * denormalized server-side from `Address.flags.muted`). Muting hides a
+ * sender from the brief only — it never deletes, marks read, or moves their
+ * mail, so callers outside the brief (mailbox listings, search) must not
+ * apply this filter.
+ */
+export function excludeMutedSenders(
+	threads: RemitImapThreadMessageResponse[],
+): RemitImapThreadMessageResponse[] {
+	return threads.filter((t) => t.muted !== true);
+}
+
 /**
  * Union of the brief's own rows with the rows the server's cross-folder search
  * returned, newest first.

package/src/lib/dedupe-thread-messages.test.ts

@@ -24,6 +24,7 @@ function threadMessage(
 		hasStars: false,
 		star: "none",
 		senderTrust: "unknown",
+		muted: false,
 		createdAt: 0,
 		updatedAt: 0,
 		...overrides,

package/src/lib/organize/filter-edit-model.test.ts

@@ -20,6 +20,7 @@ const filter = (
 	state: "Active",
 	hasAnchor: false,
 	ruleChangedAt: 0,
+	actionChangedAt: 0,
 	matchOperator: "And",
 	literalClauses: [{ field: "From", value: "receipts@stripe.com" }],
 	actionLabelId: "None",

package/src/test-support/fixtures.ts

@@ -78,6 +78,7 @@ export const makeThreadMessage = (
 	hasStars: false,
 	isDeleted: false,
 	senderTrust: "unknown",
+	muted: false,
 	createdAt: 0,
 	updatedAt: 0,
 	...overrides,