@remit/data-ports 0.0.18 → 0.0.20

package/package.json

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

package/src/conformance/label.ts

@@ -81,6 +81,47 @@ 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();
+
+			// Created concurrently so their `createdAt` millisecond can tie (as it
+			// reliably does against the synchronous sqlite driver): the keyset's
+			// `labelId` tiebreak then decides order, so this asserts pagination is
+			// exhaustive and duplicate-free rather than a specific creation order.
+			const created = await Promise.all([
+				repo.create({ accountConfigId, name: "First" }),
+				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.ok(page1.continuationToken);
+
+			const page2 = await repo.listPageByAccountConfig(accountConfigId, {
+				limit: 1,
+				continuationToken: page1.continuationToken,
+			});
+			assert.equal(page2.items.length, 1);
+			assert.equal(page2.continuationToken, undefined);
+
+			const seen = new Set([
+				...page1.items.map((label) => label.labelId),
+				...page2.items.map((label) => label.labelId),
+			]);
+			assert.equal(
+				seen.size,
+				created.length,
+				"every label is paged exactly once",
+			);
+			for (const label of created) {
+				assert.ok(seen.has(label.labelId));
+			}
+		});
+
 		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>;
 }