@remit/imap-worker 0.0.29 → 0.0.31

package/README.md

@@ -47,7 +47,8 @@ npm run cli -- -t SYNC_MESSAGE_BODY -a <accountId> -m <mailboxId> --messageIds i
 
 | Variable                      | Required | Description                                              |
 | ----------------------------- | -------- | -------------------------------------------------------- |
-| `DYNAMODB_TABLE_NAME`         | Yes      | Name of the DynamoDB table for Remit data                |
+| `DATA_BACKEND`                | Yes      | Relational backend to run against (`sqlite`)             |
+| `SQLITE_DB_PATH`              | Yes      | Path to the shared SQLite database file                  |
 | `SQS_QUEUE_URL_MAILBOXES`     | Yes      | FIFO queue URL for `SYNC_MAILBOXES` events               |
 | `SQS_QUEUE_URL_MESSAGES`      | Yes      | FIFO queue URL for `SYNC_MESSAGES` events                |
 | `SQS_QUEUE_URL_BODY`          | Yes      | FIFO queue URL for `SYNC_MESSAGE_BODY` events            |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remit/imap-worker",
-  "version": "0.0.29",
+  "version": "0.0.31",
   "type": "module",
   "main": "dist/index.js",
   "exports": {

package/src/e2e-processor-shim.ts

@@ -109,8 +109,8 @@ if (cluster.isPrimary) {
 	const queueName = new URL(queueUrl).pathname.split("/").pop();
 	const log = createLogger().child({ queue: queueName });
 
-	// The search-index queue carries DynamoDB-stream-shaped events and is owned
-	// by the search-index-worker handler; every other queue is an imap operation.
+	// The search-index queue is owned by the search-index-worker handler; every
+	// other queue is an imap operation.
 	const activeHandler: SQSHandler =
 		queueUrl === searchIndexQueueUrl ? searchIndexHandler : handler;
 

package/src/handlers/flag-push.test.ts

@@ -1,6 +1,6 @@
 import assert from "node:assert/strict";
-import { afterEach, describe, it, mock } from "node:test";
-import { getClient } from "@remit/backend/client";
+import { afterEach, before, describe, it, mock } from "node:test";
+import { getClient, type RemitClient, setClient } from "@remit/backend/client";
 import type { Logger } from "@remit/logger-lambda";
 import type { FlagPushEvent } from "../events.js";
 import {
@@ -58,6 +58,21 @@ describe("handleFlagPush — deleted mailbox is terminal (#287/#289)", () => {
 		flagName,
 	} as FlagPushEvent;
 
+	// The client is supplied by injection (`setClient`), so these tests register
+	// the repositories they then mock rather than reaching a composition.
+	before(() => {
+		setClient({
+			account: { get: async () => undefined },
+			message: { get: async () => undefined },
+			mailbox: { get: async () => undefined },
+			flagPush: {
+				find: async () => undefined,
+				updateState: async () => undefined,
+				delete: async () => undefined,
+			},
+		} as unknown as RemitClient);
+	});
+
 	afterEach(() => mock.restoreAll());
 
 	it("drops the orphaned marker and acks without pushing when the mailbox is gone", async () => {

package/src/handlers/message-move.test.ts

@@ -1,6 +1,6 @@
 import assert from "node:assert";
-import { afterEach, describe, it, mock } from "node:test";
-import { getClient } from "@remit/backend/client";
+import { afterEach, before, describe, it, mock } from "node:test";
+import { getClient, type RemitClient, setClient } from "@remit/backend/client";
 import type { AccountItem, ThreadMessageItem } from "@remit/data-ports";
 import type { Logger } from "@remit/logger-lambda";
 import type { MessageMoveEvent } from "../events.js";
@@ -219,6 +219,17 @@ describe("handleMessageMove — deleted mailbox is terminal (#287/#289)", () =>
 		timestamp: 1700000000000,
 	} as MessageMoveEvent;
 
+	// The client is supplied by injection (`setClient`), so these tests register
+	// the repositories they then mock rather than reaching a composition.
+	before(() => {
+		setClient({
+			account: { get: async () => undefined },
+			mailbox: { get: async () => undefined },
+			message: { updateUid: async () => undefined },
+			secrets: { decrypt: async () => undefined },
+		} as unknown as RemitClient);
+	});
+
 	afterEach(() => mock.restoreAll());
 
 	it("acks without connecting when the source mailbox was deleted", async () => {

package/src/handlers/placement-move-push.test.ts

@@ -1,6 +1,6 @@
 import assert from "node:assert/strict";
-import { afterEach, describe, it, mock } from "node:test";
-import { getClient } from "@remit/backend/client";
+import { afterEach, before, describe, it, mock } from "node:test";
+import { getClient, type RemitClient, setClient } from "@remit/backend/client";
 import type { Logger } from "@remit/logger-lambda";
 import type { IImapConnection } from "@remit/mailbox-service";
 import type { PlacementMovePushEvent } from "../events.js";
@@ -295,6 +295,21 @@ describe("handlePlacementMovePush — deleted mailbox is terminal (#287/#289)",
 		timestamp: 1700000000000,
 	};
 
+	// The client is supplied by injection (`setClient`), so these tests register
+	// the repositories they then mock rather than reaching a composition.
+	before(() => {
+		setClient({
+			account: { get: async () => undefined },
+			message: { get: async () => undefined },
+			mailbox: { get: async () => undefined },
+			placementMove: {
+				find: async () => undefined,
+				updateState: async () => undefined,
+				delete: async () => undefined,
+			},
+		} as unknown as RemitClient);
+	});
+
 	afterEach(() => mock.restoreAll());
 
 	it("drops the marker and acks without pushing when a mailbox is gone", async () => {

package/src/handlers/sync-message-body.test.ts

@@ -221,10 +221,8 @@ describe("syncMessageBody — DLQ propagation (integrated, #1270)", () => {
 	const cappedMailbox = (): MailboxItem =>
 		({ fullPath: "INBOX" }) as unknown as MailboxItem;
 
-	// On the DynamoDB path (main repo, DATA_BACKEND unset) the client is injected
-	// by the composition root; register a client whose repositories these tests
-	// then mock. On the relational path (reader, DATA_BACKEND=postgres) getClient
-	// builds its own client and this injection is inert.
+	// The client is supplied by injection (`setClient`), so these tests register
+	// the repositories they then mock rather than reaching a composition.
 	before(() => {
 		setClient({
 			account: { get: async () => cappedAccount() },