@remit/backend 0.0.45 → 0.0.47

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remit/backend",
-  "version": "0.0.45",
+  "version": "0.0.47",
   "description": "Remit Mail Inspector API backend",
   "license": "MIT",
   "author": "",

package/src/derive/deriveMuted.test.ts

@@ -0,0 +1,34 @@
+import assert from "node:assert/strict";
+import { describe, it } from "node:test";
+import type { AddressFlags } from "@remit/api-openapi-types";
+import { deriveMuted } from "./deriveMuted.js";
+
+const SET_AT = 1_700_000_000_000;
+
+describe("deriveMuted", () => {
+	it("returns false when flags is undefined", () => {
+		assert.equal(deriveMuted(undefined), false);
+	});
+
+	it("returns false for an empty flags object", () => {
+		assert.equal(deriveMuted({}), false);
+	});
+
+	it("returns false when muted.value is false", () => {
+		const flags: AddressFlags = { muted: { value: false, setAt: SET_AT } };
+		assert.equal(deriveMuted(flags), false);
+	});
+
+	it("returns true when muted.value is true", () => {
+		const flags: AddressFlags = { muted: { value: true, setAt: SET_AT } };
+		assert.equal(deriveMuted(flags), true);
+	});
+
+	it("ignores other flags (orthogonal axis)", () => {
+		const flags: AddressFlags = {
+			vip: { value: true, setAt: SET_AT },
+			wellknown: { value: true, setAt: SET_AT },
+		};
+		assert.equal(deriveMuted(flags), false);
+	});
+});

package/src/derive/deriveMuted.ts

@@ -0,0 +1,10 @@
+import type { AddressFlags } from "@remit/api-openapi-types";
+
+/**
+ * Derive whether the From address is muted from an Address's flags map.
+ *
+ * Pure function, no I/O. Frontend never derives this — single source of
+ * truth for filtering the daily brief.
+ */
+export const deriveMuted = (flags: AddressFlags | undefined): boolean =>
+	flags?.muted?.value === true;

package/src/derive/enrichThreadRows.test.ts

@@ -7,11 +7,13 @@ import type {
 	MessageLabelItem,
 	ThreadMessageItem,
 } from "@remit/data-ports";
+import { deriveAddressId } from "@remit/data-ports/id";
 import { type EnrichClient, enrichThreadRows } from "./enrichThreadRows.js";
 
 const threadRow = (
 	threadMessageId: string,
 	messageId: string,
+	fromEmail?: string,
 ): ThreadMessageItem =>
 	({
 		threadMessageId,
@@ -19,6 +21,7 @@ const threadRow = (
 		messageId,
 		accountConfigId: "acc-1",
 		mailboxId: "mbx-1",
+		fromEmail,
 		sentDate: 1,
 		isRead: true,
 		hasAttachment: false,
@@ -31,9 +34,10 @@ const threadRow = (
 const buildClient = (
 	messageLabels: MessageLabelItem[],
 	labels: LabelItem[],
+	addresses: AddressItem[] = [],
 ): EnrichClient => ({
 	message: { get: async () => [] as MessageItem[] },
-	address: { getAddress: async () => [] as AddressItem[] },
+	address: { getAddress: async () => addresses },
 	messageLabel: {
 		listByMessageIds: async (messageIds: string[]) =>
 			messageLabels.filter((row) => messageIds.includes(row.messageId)),
@@ -161,3 +165,59 @@ describe("enrichThreadRows — labels", () => {
 		assert.equal(second?.labels, undefined);
 	});
 });
+
+describe("enrichThreadRows — muted", () => {
+	const SET_AT = 1_700_000_000_000;
+
+	test("sets muted true from the batch-fetched Address's flags, no extra query", async () => {
+		const fromEmail = "muted@example.com";
+		const addressId = deriveAddressId("acc-1", fromEmail);
+		const rows = [threadRow("tm-1", "msg-1", fromEmail)];
+		const addresses = [
+			{
+				addressId,
+				accountConfigId: "acc-1",
+				flags: { muted: { value: true, setAt: SET_AT } },
+			},
+		] as unknown as AddressItem[];
+
+		let addressCalls = 0;
+		const client: EnrichClient = {
+			message: { get: async () => [] as MessageItem[] },
+			address: {
+				getAddress: async () => {
+					addressCalls += 1;
+					return addresses;
+				},
+			},
+			messageLabel: { listByMessageIds: async () => [] },
+			label: { listByAccountConfig: async () => [] },
+		};
+
+		const [result] = await enrichThreadRows(rows, client, "acc-1");
+		assert.equal(result?.muted, true);
+		assert.equal(addressCalls, 1);
+	});
+
+	test("defaults muted to false when the Address has no muted flag", async () => {
+		const fromEmail = "not-muted@example.com";
+		const addressId = deriveAddressId("acc-1", fromEmail);
+		const rows = [threadRow("tm-1", "msg-1", fromEmail)];
+		const addresses = [
+			{ addressId, accountConfigId: "acc-1", flags: {} },
+		] as unknown as AddressItem[];
+
+		const [result] = await enrichThreadRows(
+			rows,
+			buildClient([], [], addresses),
+			"acc-1",
+		);
+		assert.equal(result?.muted, false);
+	});
+
+	test("defaults muted to false when no Address row resolves", async () => {
+		const rows = [threadRow("tm-1", "msg-1")];
+		const [result] = await enrichThreadRows(rows, buildClient([], []), "acc-1");
+		assert.equal(result?.muted, false);
+	});
+});

package/src/derive/enrichThreadRows.ts

@@ -9,6 +9,7 @@ import type {
 import { deriveAddressId } from "@remit/data-ports/id";
 import { SenderTrust, StarColor } from "@remit/domain-enums";
 import { deriveAutoMoved } from "./autoMoved.js";
+import { deriveMuted } from "./deriveMuted.js";
 import { deriveSenderTrust } from "./senderTrust.js";
 
 /**
@@ -54,6 +55,7 @@ const toResponse = (item: ThreadMessageItem): ThreadMessageResponse => ({
 	createdAt: item.createdAt,
 	updatedAt: item.updatedAt,
 	senderTrust: SenderTrust.Unknown,
+	muted: false,
 });
 
 /**
@@ -98,20 +100,22 @@ export const planBatchFetch = (rows: ThreadMessageItem[]): BatchPlan => {
 };
 
 /**
- * Enrich a page of ThreadMessage rows with `senderTrust` (derived from the From
- * Address's flags map), `authenticity` and `autoMoved` (both projected from the
- * Message row, see `deriveAutoMoved`).
+ * Enrich a page of ThreadMessage rows with `senderTrust` and `muted` (both
+ * derived from the From Address's flags map), `authenticity` and `autoMoved`
+ * (both projected from the Message row, see `deriveAutoMoved`).
  *
- * `category` is not enriched: it is denormalized onto the ThreadMessage row and
- * carried straight through by `toResponse`, so the value a client renders is the
- * value the category filter matched.
+ * `category` is not enriched: it is denormalized onto the ThreadMessage row
+ * (shared with `Message.category`'s write-once value, see body-sync.ts) and
+ * carried straight through by `toResponse`, so the value a client renders is
+ * the value the category filter matched.
  *
  * Two BatchGetItem calls per page, regardless of page size — see
  * `planBatchFetch` for the dedup contract.
  *
- * Missing rows fall back gracefully: `senderTrust` defaults to `"unknown"`, and
- * `authenticity` / `autoMoved` are omitted whenever the Message row is absent or
- * the move isn't a real, in-effect auto-move.
+ * Missing rows fall back gracefully: `senderTrust` defaults to `"unknown"`,
+ * `muted` defaults to `false`, and `authenticity` / `autoMoved` are omitted
+ * whenever the Message row is absent or the move isn't a real, in-effect
+ * auto-move.
  *
  * Not annotated `Promise<ThreadMessageResponse[]>`: `labels` is a new field on
  * it in this same PR, and that package publishes separately from this repo —
@@ -169,6 +173,9 @@ export const enrichThreadRows = async (
 	const trustByAddressId = new Map(
 		addresses.map((a) => [a.addressId, deriveSenderTrust(a.flags)]),
 	);
+	const mutedByAddressId = new Map(
+		addresses.map((a) => [a.addressId, deriveMuted(a.flags)]),
+	);
 
 	return rows.map((row) => {
 		const base = toResponse(row);
@@ -178,6 +185,9 @@ export const enrichThreadRows = async (
 		const senderTrust = addressId
 			? (trustByAddressId.get(addressId) ?? SenderTrust.Unknown)
 			: SenderTrust.Unknown;
+		const muted = addressId
+			? (mutedByAddressId.get(addressId) ?? false)
+			: false;
 		const labels = labelsByMessageId.get(row.messageId);
 		return {
 			...base,
@@ -185,6 +195,7 @@ export const enrichThreadRows = async (
 			...(autoMoved !== undefined ? { autoMoved } : {}),
 			...(labels !== undefined ? { labels } : {}),
 			senderTrust,
+			muted,
 		};
 	});
 };

package/src/derive/filterThreadCriteria.test.ts

@@ -29,6 +29,7 @@ const row = (
 	createdAt: 0,
 	updatedAt: 0,
 	senderTrust: SenderTrust.Unknown,
+	muted: false,
 	...overrides,
 });
 

package/src/handlers/filter.test.ts

@@ -199,6 +199,7 @@ describe("createFilterWithAnchor (#351)", () => {
 		state: FilterState.Active,
 		hasAnchor: false,
 		ruleChangedAt: 1_700_000_000,
+		actionChangedAt: 1_700_000_000,
 		matchOperator: FilterMatchOperator.And,
 		literalClauses: [],
 		actionLabelId: "None",

package/src/handlers/filter.ts

@@ -187,6 +187,7 @@ const toFilterResponse = (item: FilterItem): FilterResponse => ({
 	state: item.state,
 	hasAnchor: item.hasAnchor,
 	ruleChangedAt: item.ruleChangedAt,
+	actionChangedAt: item.actionChangedAt,
 	matchOperator: item.matchOperator,
 	literalClauses: item.literalClauses,
 	actionLabelId: item.actionLabelId,

package/src/service/organize.test.ts

@@ -79,6 +79,7 @@ const filterItem = (over: Partial<FilterItem> = {}): FilterItem => ({
 	state: FilterState.Active,
 	hasAnchor: false,
 	ruleChangedAt: 0,
+	actionChangedAt: 0,
 	matchOperator: FilterMatchOperator.And,
 	literalClauses: [],
 	actionLabelId: "None",
@@ -780,6 +781,7 @@ describe("applyOrganize resolves move precedence against current Active filters
 		const newerFilter = filterItem({
 			filterId: "filter-newer",
 			ruleChangedAt: 1_000,
+			actionChangedAt: 1_000,
 			actionMailboxId: "mbox-new",
 			literalClauses: [{ field: "Subject", value: "reservation" }],
 		});
@@ -828,6 +830,7 @@ describe("applyOrganize resolves move precedence against current Active filters
 		const agreeingFilter = filterItem({
 			filterId: "filter-agrees",
 			ruleChangedAt: 1_000,
+			actionChangedAt: 1_000,
 			actionMailboxId: "mbox-target",
 			literalClauses: [{ field: "Subject", value: "reservation" }],
 		});
@@ -869,6 +872,7 @@ describe("applyOrganize resolves move precedence against current Active filters
 		const newerSemanticFilter = filterItem({
 			filterId: "filter-newer-semantic",
 			ruleChangedAt: 1_000,
+			actionChangedAt: 1_000,
 			actionMailboxId: "mbox-new",
 			hasAnchor: true,
 		});