@remit/backend 0.0.14 → 0.0.15

package/package.json

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

package/src/handlers/me.test.ts

@@ -0,0 +1,81 @@
+import assert from "node:assert/strict";
+import { afterEach, describe, it } from "node:test";
+import type { QuarantineItem } from "@remit/data-ports";
+import type { APIGatewayProxyEvent } from "aws-lambda";
+import type { Context } from "openapi-backend";
+import { deriveAccountConfigId } from "../auth.js";
+import {
+	_resetForTest,
+	type RemitClient,
+	setClient,
+} from "../service/dynamodb.js";
+import { MeOperations } from "./me.js";
+
+const listQuarantine = MeOperations.MeOperations_listQuarantine as unknown as (
+	context: Context,
+	event: APIGatewayProxyEvent,
+) => Promise<{ entries: QuarantineItem[] }>;
+
+const SUB = "cognito-sub-1";
+
+const eventFor = (sub: string): APIGatewayProxyEvent =>
+	({
+		requestContext: { authorizer: { claims: { sub } } },
+	}) as unknown as APIGatewayProxyEvent;
+
+const entry = (over: Partial<QuarantineItem> = {}): QuarantineItem =>
+	({
+		quarantineId: "q-1",
+		accountConfigId: deriveAccountConfigId(SUB),
+		accountId: "acct-1",
+		mailboxId: "mbx-1",
+		uidValidity: 1_712_000_000,
+		uid: 40217,
+		mailboxPath: "INBOX",
+		quarantinedAt: 1_000,
+		attempts: 3,
+		failureStage: "BodyParse",
+		failureCode: "UnterminatedMultipartBoundary",
+		failureMessage: "multipart boundary was never closed",
+		workerVersion: "worker 1.0.0",
+		structure: [{ depth: 0, contentType: "multipart/mixed" }],
+		createdAt: 1_000,
+		updatedAt: 1_000,
+		...over,
+	}) as QuarantineItem;
+
+const clientListing = (items: QuarantineItem[], seen: string[]): RemitClient =>
+	({
+		quarantine: {
+			listByAccountConfigId: async (
+				accountConfigId: string,
+			): Promise<QuarantineItem[]> => {
+				seen.push(accountConfigId);
+				return items;
+			},
+		},
+	}) as unknown as RemitClient;
+
+afterEach(() => {
+	_resetForTest();
+});
+
+describe("MeOperations_listQuarantine", () => {
+	it("lists only what belongs to the caller", async () => {
+		const seen: string[] = [];
+		setClient(clientListing([entry()], seen));
+
+		const response = await listQuarantine({} as Context, eventFor(SUB));
+
+		assert.deepEqual(seen, [deriveAccountConfigId(SUB)]);
+		assert.deepEqual(response, { entries: [entry()] });
+	});
+
+	it("returns an empty list rather than failing when nothing is quarantined", async () => {
+		setClient(clientListing([], []));
+
+		const response = await listQuarantine({} as Context, eventFor(SUB));
+
+		assert.deepEqual(response, { entries: [] });
+	});
+});

package/src/handlers/me.ts

@@ -4,6 +4,7 @@ import type {
 	CreateExportResponse,
 	DeleteAccountConfigResponse,
 	DeleteMeInput,
+	QuarantineListResponse,
 	VipSuggestionsResponse,
 } from "@remit/api-openapi-types";
 import { BadRequestError, NotFoundError } from "@remit/data-ports/errors";
@@ -35,6 +36,18 @@ export const MeOperations: Record<
 
 		return { suggestions: items.map(toVipSuggestionEntry) };
 	},
+	MeOperations_listQuarantine: async (
+		_context: Context,
+		...args: unknown[]
+	): Promise<QuarantineListResponse> => {
+		const event = args[0] as APIGatewayProxyEvent;
+		const accountConfigId = getAccountConfigIdFromEvent(event);
+
+		const { quarantine } = await getClient();
+		const entries = await quarantine.listByAccountConfigId(accountConfigId);
+
+		return { entries };
+	},
 	MeOperations_deleteMe: async (
 		_context: Context,
 		...args: unknown[]

package/src/service/compose-postgres.ts

@@ -21,6 +21,7 @@ import {
 	messageDataSchema,
 	OrganizeJobRequestRepo,
 	OutboxMessageRepo,
+	QuarantineRepo,
 } from "@remit/drizzle-service";
 import { drizzle, type NodePgDatabase } from "drizzle-orm/node-postgres";
 import { env } from "expect-env";
@@ -58,6 +59,7 @@ export const buildPostgresClient = (): RemitClient => {
 		threadMessage: new DrizzleThreadMessageRepository(pgConnectionUrl),
 		envelope: new DrizzleEnvelopeRepository(messageDataDb),
 		accountExportRequest: new AccountExportRequestRepo(genericDb),
+		quarantine: new QuarantineRepo(genericDb),
 		organizeJobRequest: new OrganizeJobRequestRepo(genericDb),
 		placementMove: new MessagePlacementMoveRepo(genericDb),
 		flagPush: new MessageFlagPushRepo(genericDb),

package/src/service/compose-sqlite.ts

@@ -22,6 +22,7 @@ import {
 	messageDataSchema,
 	OrganizeJobRequestRepo,
 	OutboxMessageRepo,
+	QuarantineRepo,
 } from "@remit/drizzle-service";
 import type { NodePgDatabase } from "drizzle-orm/node-postgres";
 import { env } from "expect-env";
@@ -66,6 +67,7 @@ export const buildSqliteClient = async (): Promise<RemitClient> => {
 		threadMessage: new DrizzleThreadMessageRepository(genericDb),
 		envelope: new DrizzleEnvelopeRepository(messageDataDb),
 		accountExportRequest: new AccountExportRequestRepo(genericDb),
+		quarantine: new QuarantineRepo(genericDb),
 		organizeJobRequest: new OrganizeJobRequestRepo(genericDb),
 		placementMove: new MessagePlacementMoveRepo(genericDb),
 		flagPush: new MessageFlagPushRepo(genericDb),

package/src/service/create-remit-client.ts

@@ -18,6 +18,7 @@ import type {
 	IMessageRepository,
 	IOrganizeJobRequestRepository,
 	IOutboxMessageRepository,
+	IQuarantineRepository,
 	IThreadMessageRepository,
 	IUnitOfWork,
 } from "@remit/data-ports";
@@ -71,6 +72,10 @@ export interface RemitClient {
 	envelope: IEnvelopeRepository;
 	accountExportRequest: IAccountExportRequestRepository;
 
+	// Messages the sync path could not read (issue #72). Read-only from the API
+	// process: the sync worker writes the rows, settings lists them.
+	quarantine: IQuarantineRepository;
+
 	// Smart Organize back-apply job (RFC 034, #1278). Present on both backends,
 	// modeled on accountExportRequest — a Pending row the account-fanout worker
 	// picks up and drives to Complete/Failed. Never a Filter.
@@ -151,6 +156,7 @@ export interface RemitClientRepositories {
 	threadMessage: IThreadMessageRepository;
 	envelope: IEnvelopeRepository;
 	accountExportRequest: IAccountExportRequestRepository;
+	quarantine: IQuarantineRepository;
 	organizeJobRequest: IOrganizeJobRequestRepository;
 	placementMove: IMessagePlacementMoveRepository;
 	flagPush: IMessageFlagPushRepository;
@@ -310,6 +316,7 @@ export const createRemitClient = (deps: RemitClientDeps): RemitClient => {
 		threadMessage: repositories.threadMessage,
 		envelope: repositories.envelope,
 		accountExportRequest: repositories.accountExportRequest,
+		quarantine: repositories.quarantine,
 		organizeJobRequest: repositories.organizeJobRequest,
 		filter: repositories.filter,
 		filterAnchor: repositories.filterAnchor,

package/src/service/dynamodb.test.ts

@@ -20,6 +20,7 @@ const REQUIRED_KEYS: ReadonlyArray<keyof RemitClient> = [
 	"threadMessage",
 	"envelope",
 	"accountExportRequest",
+	"quarantine",
 	"storage",
 	"search",
 	"secrets",

package/src/types.ts

@@ -10,6 +10,7 @@ export type OperationIds =
 	| "MeOperations_listVipSuggestions"
 	| "MeOperations_createExport"
 	| "MeOperations_getExport"
+	| "MeOperations_listQuarantine"
 	| "ConfigOperations_getConfig"
 	| "AccountOperations_createAccount"
 	| "AccountOperations_testConnection"