@cosmicdrift/kumiko-bundled-features 0.310.0 → 0.311.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-bundled-features",
-  "version": "0.310.0",
+  "version": "0.311.0",
   "description": "Built-in features — tenant, user, auth, delivery. The stuff you'd rewrite anyway, already typed.",
   "license": "BUSL-1.1",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -133,12 +133,12 @@
     "./workflow-runner": "./src/workflow-runner/index.ts"
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-dispatcher-live": "0.310.0",
-    "@cosmicdrift/kumiko-framework": "0.310.0",
-    "@cosmicdrift/kumiko-headless": "0.310.0",
-    "@cosmicdrift/kumiko-renderer": "0.310.0",
-    "@cosmicdrift/kumiko-renderer-web": "0.310.0",
-    "@cosmicdrift/kumiko-types": "0.310.0",
+    "@cosmicdrift/kumiko-dispatcher-live": "0.311.0",
+    "@cosmicdrift/kumiko-framework": "0.311.0",
+    "@cosmicdrift/kumiko-headless": "0.311.0",
+    "@cosmicdrift/kumiko-renderer": "0.311.0",
+    "@cosmicdrift/kumiko-renderer-web": "0.311.0",
+    "@cosmicdrift/kumiko-types": "0.311.0",
     "@mollie/api-client": "^4.5.0",
     "@node-rs/argon2": "^2.0.2",
     "@types/mailparser": "^3.4.6",
@@ -167,8 +167,8 @@
   ],
   "devDependencies": {
     "@testing-library/user-event": "^14.6.1",
-    "@cosmicdrift/kumiko-locale-de": "0.310.0",
-    "@cosmicdrift/kumiko-locale-es": "0.310.0",
+    "@cosmicdrift/kumiko-locale-de": "0.311.0",
+    "@cosmicdrift/kumiko-locale-es": "0.311.0",
     "jsqr": "^1.4.0"
   }
 }

package/src/audit/__tests__/escape-hatch-audit.integration.test.ts

@@ -12,6 +12,7 @@ import {
   type TestStack,
   testTenantId,
 } from "@cosmicdrift/kumiko-framework/stack";
+import { waitFor } from "@cosmicdrift/kumiko-framework/testing";
 import * as z from "zod";
 import { createConfigFeature } from "../../config";
 import { createTenantFeature } from "../../tenant";
@@ -71,18 +72,21 @@ type AuditRow = {
 type AuditResponse = { rows: AuditRow[]; nextBefore: string | null };
 
 async function pollForEscapeHatchRow(): Promise<AuditRow> {
-  const deadline = Date.now() + 2000;
-  while (Date.now() < deadline) {
-    const res = await stack.http.queryOk<AuditResponse>(
-      AuditQueries.list,
-      { eventType: ESCAPE_HATCH_USED_EVENT },
-      adminOfSameTenant,
-    );
-    const row = res.rows[0];
-    if (row) return row;
-    await new Promise((resolve) => setTimeout(resolve, 50));
-  }
-  throw new Error("escape-hatch-used audit row did not appear within 2s");
+  let row: AuditRow | undefined;
+  await waitFor(
+    async () => {
+      const res = await stack.http.queryOk<AuditResponse>(
+        AuditQueries.list,
+        { eventType: ESCAPE_HATCH_USED_EVENT },
+        adminOfSameTenant,
+      );
+      row = res.rows[0];
+      return row !== undefined;
+    },
+    { delays: Array(40).fill(50) },
+  );
+  if (row === undefined) throw new Error("escape-hatch-used audit row did not appear within 2s");
+  return row;
 }
 
 describe("createEscapeHatchAuditSink — persists audit:event:escape-hatch-used", () => {

package/src/delivery/__tests__/delivery.integration.test.ts

@@ -26,6 +26,7 @@ import {
   testTenantId,
   unsafePushTables,
 } from "@cosmicdrift/kumiko-framework/stack";
+import { waitFor } from "@cosmicdrift/kumiko-framework/testing";
 import * as z from "zod";
 import { createChannelEmailFeature } from "../../channel-email/feature";
 import { createInMemoryTransport, type EmailMessage } from "../../channel-email/types";
@@ -1502,21 +1503,6 @@ describe("flow 16: repeated unsubscribe clicks are idempotent", () => {
 // non-string `to` as a broadcast target ("tenant" in to).
 const asyncRecipient = "7";
 
-async function waitFor(check: () => Promise<void>, timeoutMs = 10000): Promise<void> {
-  const start = Date.now();
-  let lastErr: unknown;
-  for (;;) {
-    try {
-      await check();
-      return;
-    } catch (err) {
-      lastErr = err;
-      if (Date.now() - start > timeoutMs) throw lastErr;
-      await new Promise((resolve) => setTimeout(resolve, 50));
-    }
-  }
-}
-
 function makeStubRunner(): {
   runner: JobRunner;
   dispatched: Array<{
@@ -1580,13 +1566,16 @@ describe("flow 17: async render→send pipeline", () => {
 
       // Email: queued → render job → send job → sent. Proves the framework
       // jobRunner-into-job-ctx injection (render dispatches send) end-to-end.
-      await waitFor(async () => {
-        const rows = await selectMany(db, deliveryAttemptsTable, {
-          notificationType: "app:notify:async-e2e",
-          channel: "email",
-        });
-        expect(rows.some((r) => r["status"] === "sent")).toBe(true);
-      });
+      await waitFor(
+        async () => {
+          const rows = await selectMany(db, deliveryAttemptsTable, {
+            notificationType: "app:notify:async-e2e",
+            channel: "email",
+          });
+          expect(rows.some((r) => r["status"] === "sent")).toBe(true);
+        },
+        { delays: Array(40).fill(250) },
+      );
       const email = emailTransport.sent.find((m) => m.to === testEmail(asyncRecipient));
       expect(email).toBeDefined();
       expect(email?.subject).toBe("Async Subject");
@@ -1594,13 +1583,16 @@ describe("flow 17: async render→send pipeline", () => {
       expect(email?.html).toContain("<!DOCTYPE html>");
 
       // Push: no render step → dispatched straight to delivery.send.
-      await waitFor(async () => {
-        const rows = await selectMany(db, deliveryAttemptsTable, {
-          notificationType: "app:notify:async-e2e",
-          channel: "push",
-        });
-        expect(rows.some((r) => r["status"] === "sent")).toBe(true);
-      });
+      await waitFor(
+        async () => {
+          const rows = await selectMany(db, deliveryAttemptsTable, {
+            notificationType: "app:notify:async-e2e",
+            channel: "push",
+          });
+          expect(rows.some((r) => r["status"] === "sent")).toBe(true);
+        },
+        { delays: Array(40).fill(250) },
+      );
       expect(
         pushTransport.sent.find((m) => m.token === testPushToken(asyncRecipient)),
       ).toBeDefined();

package/src/inbound-provider-imap/__tests__/plugin-mocked.test.ts

@@ -5,6 +5,7 @@
 import { afterAll, beforeEach, describe, expect, mock, test } from "bun:test";
 import { EventEmitter } from "node:events";
 import { createSecret } from "@cosmicdrift/kumiko-framework/secrets";
+import { sleep, waitFor } from "@cosmicdrift/kumiko-framework/testing";
 import {
   type InboundMailContext,
   isInboundAuthError,
@@ -152,13 +153,6 @@ const account: MailAccountRecord = {
   watchState: "idle",
 };
 
-async function waitFor(predicate: () => boolean, timeoutMs = 2000): Promise<void> {
-  const t0 = Date.now();
-  while (!predicate() && Date.now() - t0 < timeoutMs) {
-    await Bun.sleep(5);
-  }
-}
-
 function ctxWithDoc(doc: string | null): InboundMailContext {
   return {
     _userId: "imap-plugin-mocked",
@@ -332,7 +326,9 @@ describe("imapInboundMailPlugin — mocked imapflow", () => {
 
     // drainNew() does MIME-parsing before onMessages fires — a fixed sleep
     // flakes on a loaded CI runner; poll instead.
-    await waitFor(() => received.some((batch) => batch.some((m) => m.subject === "pushed")));
+    await waitFor(() => received.some((batch) => batch.some((m) => m.subject === "pushed")), {
+      delays: Array(40).fill(5),
+    });
     expect(received.some((batch) => batch.some((m) => m.subject === "pushed"))).toBe(true);
 
     await stop();
@@ -347,11 +343,13 @@ describe("imapInboundMailPlugin — mocked imapflow", () => {
       },
     });
     lastIdleClient?.emit("error", new Error("socket hang up"));
-    await waitFor(() => errors >= 1);
+    await waitFor(() => errors >= 1, { delays: Array(40).fill(5) });
     lastIdleClient?.emit("error", new Error("second"));
-    // Confirms onError stays unsubscribed after the first error — polls the
-    // same bounded window rather than betting on a fixed sleep outlasting it.
-    await waitFor(() => errors >= 2, 100);
+    // Confirms onError stays unsubscribed after the first error. This is a
+    // grace period for a negative outcome, not a wait-for-true condition, so
+    // it's a single sleep (not a poll loop) rather than waitFor, which would
+    // throw once errors never reaches 2.
+    await sleep(100);
     expect(errors).toBe(1);
     await stop().catch(() => {});
   });

package/src/jobs/__tests__/projection-rebuild-job.integration.test.ts

@@ -40,7 +40,7 @@ import {
   unsafeCreateEntityTable,
   unsafePushTables,
 } from "@cosmicdrift/kumiko-framework/stack";
-import { sleep } from "@cosmicdrift/kumiko-framework/testing";
+import { waitFor } from "@cosmicdrift/kumiko-framework/testing";
 import { createJobsFeature } from "../feature";
 import { createJobRunLogger } from "../job-run-logger";
 import { jobRunLogsTable, jobRunsTable } from "../job-run-table";
@@ -148,7 +148,7 @@ describe("projection-rebuild job (jobs feature composed)", () => {
     }
 
     // Poll until the worker drained the queue and the rebuild refilled.
-    for (let i = 0; i < 40 && (await getCount()) !== 2; i++) await sleep(200);
+    await waitFor(async () => (await getCount()) === 2, { delays: Array(40).fill(200) });
     expect(await getCount()).toBe(2);
 
     // getCount()==2 only proves rebuildProjection's own writes landed — the
@@ -156,13 +156,15 @@ describe("projection-rebuild job (jobs feature composed)", () => {
     // async append that starts only after the handler returns, so it can
     // still be in flight here. Poll status too instead of racing it.
     let runs: readonly { jobName: string; status: string }[] = [];
-    for (let i = 0; i < 40; i++) {
-      runs = await selectMany<{ jobName: string; status: string }>(db, jobRunsTable, {
-        jobName: PROJECTION_REBUILD_JOB,
-      });
-      if (runs.some((r) => r.status === "completed")) break;
-      await sleep(200);
-    }
+    await waitFor(
+      async () => {
+        runs = await selectMany<{ jobName: string; status: string }>(db, jobRunsTable, {
+          jobName: PROJECTION_REBUILD_JOB,
+        });
+        return runs.some((r) => r.status === "completed");
+      },
+      { delays: Array(40).fill(200) },
+    );
     expect(runs.length).toBeGreaterThanOrEqual(1);
     expect(runs.some((r) => r.status === "completed")).toBe(true);
   }, 30000);