@remit/imap-worker 0.0.21 → 0.0.22

package/package.json

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

package/src/handlers/sync-mailboxes.test.ts

@@ -1,6 +1,11 @@
 import assert from "node:assert/strict";
 import { describe, it } from "node:test";
-import { MAILBOX_FRESHNESS_MS, mailboxNeedsSync } from "./sync-mailboxes.js";
+import {
+	DEFAULT_MAILBOX_FRESHNESS_MS,
+	MAILBOX_FRESHNESS_MS,
+	mailboxNeedsSync,
+	resolveMailboxFreshnessMs,
+} from "./sync-mailboxes.js";
 
 const NOW = 1_700_000_000_000;
 
@@ -45,3 +50,37 @@ describe("mailboxNeedsSync", () => {
 		);
 	});
 });
+
+describe("resolveMailboxFreshnessMs", () => {
+	it("defaults to the production window when unset", () => {
+		assert.equal(resolveMailboxFreshnessMs({}), DEFAULT_MAILBOX_FRESHNESS_MS);
+		assert.equal(DEFAULT_MAILBOX_FRESHNESS_MS, 60_000);
+	});
+
+	it("honors a valid override", () => {
+		assert.equal(resolveMailboxFreshnessMs({ MAILBOX_FRESHNESS_MS: "0" }), 0);
+		assert.equal(
+			resolveMailboxFreshnessMs({ MAILBOX_FRESHNESS_MS: "1000" }),
+			1000,
+		);
+	});
+
+	it("falls back to the default for a non-numeric or negative value", () => {
+		assert.equal(
+			resolveMailboxFreshnessMs({ MAILBOX_FRESHNESS_MS: "" }),
+			DEFAULT_MAILBOX_FRESHNESS_MS,
+		);
+		assert.equal(
+			resolveMailboxFreshnessMs({ MAILBOX_FRESHNESS_MS: "abc" }),
+			DEFAULT_MAILBOX_FRESHNESS_MS,
+		);
+		assert.equal(
+			resolveMailboxFreshnessMs({ MAILBOX_FRESHNESS_MS: "-5" }),
+			DEFAULT_MAILBOX_FRESHNESS_MS,
+		);
+		assert.equal(
+			resolveMailboxFreshnessMs({ MAILBOX_FRESHNESS_MS: "1.5" }),
+			DEFAULT_MAILBOX_FRESHNESS_MS,
+		);
+	});
+});

package/src/handlers/sync-mailboxes.ts

@@ -35,7 +35,25 @@ const EVENT_EMIT_CONCURRENCY = 20;
  * (`MIN_POLL_INTERVAL_MS` in useStaleAccountSync), so the one caller that
  * skips this gate on a timer still cannot drive a fan-out faster than it.
  */
-export const MAILBOX_FRESHNESS_MS = 60_000;
+export const DEFAULT_MAILBOX_FRESHNESS_MS = 60_000;
+
+/**
+ * Resolve the freshness window from the environment, defaulting to production's
+ * {@link DEFAULT_MAILBOX_FRESHNESS_MS}. Deployments that drive sync from a test
+ * harness rather than a real client set `MAILBOX_FRESHNESS_MS` low so their
+ * explicit nudges are never no-oped by the gate; nothing else touches it, so
+ * production keeps the default.
+ */
+export const resolveMailboxFreshnessMs = (
+	env: NodeJS.ProcessEnv = process.env,
+): number => {
+	const value = env.MAILBOX_FRESHNESS_MS?.trim();
+	if (!value) return DEFAULT_MAILBOX_FRESHNESS_MS;
+	const raw = Number(value);
+	return Number.isInteger(raw) && raw >= 0 ? raw : DEFAULT_MAILBOX_FRESHNESS_MS;
+};
+
+export const MAILBOX_FRESHNESS_MS = resolveMailboxFreshnessMs();
 
 /**
  * Whether this fan-out should enqueue a mailbox.