@remit/data-ports 0.0.17 → 0.0.19

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remit/data-ports",
-  "version": "0.0.17",
+  "version": "0.0.19",
   "type": "module",
   "main": "src/index.ts",
   "types": "src/index.ts",

package/src/conformance/label.ts

@@ -81,6 +81,30 @@ export function labelRepositoryConformance(
 			assert.equal(missing, null);
 		});
 
+		test("listPageByAccountConfig pages through an account's labels, scoped to the account", async () => {
+			const accountConfigId = harness.makeId();
+			const other = harness.makeId();
+
+			const first = await repo.create({ accountConfigId, name: "First" });
+			const second = await repo.create({ accountConfigId, name: "Second" });
+			await repo.create({ accountConfigId: other, name: "Foreign" });
+
+			const page1 = await repo.listPageByAccountConfig(accountConfigId, {
+				limit: 1,
+			});
+			assert.equal(page1.items.length, 1);
+			assert.equal(page1.items[0]?.labelId, first.labelId);
+			assert.ok(page1.continuationToken);
+
+			const page2 = await repo.listPageByAccountConfig(accountConfigId, {
+				limit: 1,
+				continuationToken: page1.continuationToken,
+			});
+			assert.equal(page2.items.length, 1);
+			assert.equal(page2.items[0]?.labelId, second.labelId);
+			assert.equal(page2.continuationToken, undefined);
+		});
+
 		test("delete removes the row", async () => {
 			const accountConfigId = harness.makeId();
 			const label = await repo.create({ accountConfigId, name: "Gone" });

package/src/interfaces/label.ts

@@ -1,6 +1,7 @@
 import type {
 	CreateLabelInput,
 	LabelItem,
+	ResultList,
 	UpdateLabelInput,
 } from "../types.js";
 
@@ -14,6 +15,14 @@ export interface ILabelRepository {
 	): Promise<LabelItem>;
 	delete(accountConfigId: string, labelId: string): Promise<void>;
 	listByAccountConfig(accountConfigId: string): Promise<LabelItem[]>;
+	/**
+	 * A single signed page of an account config's labels (issue #26), mirroring
+	 * `IFilterRepository.listPageByAccountConfig` — the settings-page/API list.
+	 */
+	listPageByAccountConfig(
+		accountConfigId: string,
+		options?: { limit?: number; continuationToken?: string },
+	): Promise<ResultList<LabelItem>>;
 	findByNormalizedName(
 		accountConfigId: string,
 		normalizedName: string,

package/src/interfaces/message-label.ts

@@ -4,8 +4,22 @@ export interface IMessageLabelRepository {
 	apply(input: CreateMessageLabelInput): Promise<MessageLabelItem>;
 	remove(messageId: string, labelId: string): Promise<void>;
 	listByMessageId(messageId: string): Promise<MessageLabelItem[]>;
+	/**
+	 * Batch-fetch every MessageLabel row across a page of messages in one query
+	 * — the read-time enrichment a thread listing needs (issue #26), mirroring
+	 * the one-BatchGet-per-page contract `enrichThreadRows` already holds for
+	 * Message/Address.
+	 */
+	listByMessageIds(messageIds: string[]): Promise<MessageLabelItem[]>;
 	listByLabelId(
 		accountConfigId: string,
 		labelId: string,
 	): Promise<MessageLabelItem[]>;
+	/**
+	 * Delete every MessageLabel row for a label in one statement — the message
+	 * side of a label delete's cascade (issue #26). A label delete cascades in
+	 * both directions: this clears the applied-to-messages side, and the
+	 * caller separately deletes any filter whose actionLabelId is this label.
+	 */
+	removeAllByLabelId(accountConfigId: string, labelId: string): Promise<void>;
 }

package/src/types.ts

@@ -500,10 +500,14 @@ export type CreateFilterInput = Omit<
 };
 
 // ruleChangedAt is never client-supplied: FilterService derives it, bumping
-// only when a predicate/action field is present in the update (RFC 034
-// Decision 3.2) — a cosmetic `name` rename must not move it.
+// only when the predicate, the action, scope, or expiresAt is present in the
+// update (RFC 034 Decision 3.2, reader #266) — a cosmetic `name` rename must
+// not move it. `scope` is updatable here (reader #266): the handler resolves
+// the merged scope/expiresAt/ttl/state before calling through to the repo, so
+// this type must carry `scope` as a valid key, unlike the API-facing
+// `UpdateFilterInput` for that same reason.
 export type UpdateFilterInput = Partial<
-	Omit<CreateFilterInput, "accountConfigId" | "scope">
+	Omit<CreateFilterInput, "accountConfigId">
 >;
 
 export type CreateFilterAnchorInput = Omit<