@cosmicdrift/kumiko-framework 0.307.0 → 0.309.0

package/src/engine/feature-config-events-jobs.ts

@@ -1,5 +1,6 @@
 import { normalizeEventPiiSubject } from "@cosmicdrift/kumiko-types/handlers";
-import { ZodObject, type ZodType, type z } from "zod";
+import type * as z from "zod";
+import { ZodObject, type ZodType } from "zod";
 import type { FeatureBuilderState } from "./feature-builder-state";
 import { resolveName } from "./handler-helpers";
 import { splitNamedDefinition, unwrapArrayForm } from "./object-form";

package/src/engine/feature-entity-handlers.ts

@@ -1,5 +1,6 @@
 import { ENTITY_CONVENTION_QUERY_BRAND } from "@cosmicdrift/kumiko-types/handlers";
-import type { ZodType, z } from "zod";
+import type * as z from "zod";
+import type { ZodType } from "zod";
 import { toTableName } from "../db/table-builder";
 import {
   isEntityConventionQueryHandler,

package/src/engine/schema-builder.ts

@@ -1,4 +1,4 @@
-import { z } from "zod";
+import * as z from "zod";
 import { moneyPayloadToMinorUnits } from "../db/money";
 import { isValidIanaTimeZone } from "../time";
 import { assertUnreachable } from "../utils";

package/src/entrypoint/__tests__/entrypoint-attach-dispatcher.integration.test.ts

@@ -10,7 +10,7 @@
 // throws the #2043 stub).
 
 import { afterAll, beforeAll, describe, expect, test } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import { type BunTestDb, createTestDb } from "../../bun-db/__tests__/bun-test-db";
 import { asRawClient } from "../../db/query";
 import { createRegistry, defineFeature } from "../../engine";

package/src/entrypoint/__tests__/entrypoint-job-wiring.integration.test.ts

@@ -13,7 +13,7 @@
 // regressing to the Welle-2.5 state.
 
 import { afterAll, beforeAll, describe, expect, test } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import { type BunTestDb, createTestDb } from "../../bun-db/__tests__/bun-test-db";
 import { createRegistry, defineFeature } from "../../engine";
 import { createArchivedStreamsTable } from "../../event-store";

package/src/entrypoint/__tests__/split-deploy.integration.test.ts

@@ -13,7 +13,7 @@
 //      fires with systemConsumers explicitly disabled).
 
 import { afterAll, beforeAll, describe, expect, test } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import { type BunTestDb, createTestDb } from "../../bun-db/__tests__/bun-test-db";
 import { asRawClient } from "../../db/query";
 import { createRegistry, defineFeature } from "../../engine";

package/src/env/__tests__/compose-env-schema.test.ts

@@ -1,5 +1,5 @@
 import { describe, expect, it } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import { defineFeature } from "../../engine/define-feature";
 import {
   camelCase,

package/src/env/__tests__/dry-run.test.ts

@@ -1,5 +1,5 @@
 import { describe, expect, it } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import { defineFeature } from "../../engine/define-feature";
 import { prometheusMetricsEnvSchema } from "../../observability/metrics-wiring";
 import { renderDryRun } from "../dry-run";

package/src/env/__tests__/parse-env-dry-run.test.ts

@@ -2,7 +2,7 @@
 // undefined. parseEnvDryRun liefert stattdessen ein ehrliches Partial.
 
 import { describe, expect, test } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import { parseEnvDryRun } from "../index";
 
 const schema = z.object({

package/src/env/_zod-introspect.ts

@@ -6,7 +6,7 @@
 // All helpers accept the *runtime* Zod instance — TypeScript wrapper
 // (`z.ZodType`) and core (`$ZodType`) are the same object at runtime.
 
-import type { z } from "zod";
+import type * as z from "zod";
 
 export type ZodDef = {
   readonly innerType?: z.ZodType;

package/src/env/dry-run.ts

@@ -8,7 +8,7 @@
 // The same schema introspection (`classifyField`, `readKumikoMeta`) used
 // by parseEnv drives the output here — single source of truth.
 
-import type { z } from "zod";
+import type * as z from "zod";
 import { zodShape } from "./_zod-introspect";
 import {
   type ComposedEnvSchema,

package/src/env/index.ts

@@ -14,7 +14,7 @@
 // Without meta, defaults: `camelCase(envVarName)` for the Pulumi key,
 // `<set-me>` placeholder for the value, no `--secret` flag.
 
-import { z } from "zod";
+import * as z from "zod";
 import type { FeatureDefinition } from "../engine/types/feature";
 import { zodDef, zodDescription, zodMeta, zodShape, zodShapeField } from "./_zod-introspect";
 

package/src/errors/__tests__/classes.test.ts

@@ -1,5 +1,5 @@
 import { describe, expect, test } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import {
   AccessDeniedError,
   buildErrorLog,

package/src/es-ops/__tests__/runner.integration.test.ts

@@ -17,7 +17,7 @@ import { afterAll, beforeAll, beforeEach, describe, expect, test } from "bun:tes
 import { mkdtempSync, rmSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
-import { z } from "zod";
+import * as z from "zod";
 import { createEventStoreExecutor } from "../../db/event-store-executor";
 import { asRawClient, insertOne, selectMany } from "../../db/query";
 import { buildEntityTable } from "../../db/table-builder";

package/src/event-store/__tests__/backfill-pii.integration.test.ts

@@ -12,7 +12,7 @@ import {
   resetBlindIndexKeyForTests,
   resetPiiSubjectKmsForTests,
 } from "@cosmicdrift/kumiko-framework/testing";
-import { z } from "zod";
+import * as z from "zod";
 import {
   configureBlindIndexKey,
   configurePiiSubjectKms,

package/src/event-store/__tests__/event-attribution.integration.test.ts

@@ -15,7 +15,7 @@
 
 import { afterAll, afterEach, beforeAll, describe, expect, test } from "bun:test";
 import { UNATTRIBUTED_ORIGIN, type WriteOrigin } from "@cosmicdrift/kumiko-types/event-store-types";
-import { z } from "zod";
+import * as z from "zod";
 import { createEventStoreExecutor } from "../../db/event-store-executor";
 import { selectMany } from "../../db/query";
 import { buildEntityTable } from "../../db/table-builder";

package/src/event-store/__tests__/upcaster.integration.test.ts

@@ -9,7 +9,7 @@
 //      can never silently hand half-migrated data to consumers.
 
 import { afterAll, beforeAll, beforeEach, describe, expect, test } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import { integer as pgInteger, table as pgTable, text as pgText } from "../../db/dialect";
 import { createEventStoreExecutor } from "../../db/event-store-executor";
 import { asRawClient, insertOne, selectMany } from "../../db/query";

package/src/jobs/__tests__/job-backoff.integration.test.ts

@@ -10,7 +10,7 @@
 // so a >= assertion is flake-free without inflating waitFor's budget.
 
 import { afterAll, beforeAll, beforeEach, describe, expect, test } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import { defineFeature } from "../../engine";
 import { setupTestStack, type TestStack, TestUsers } from "../../stack";
 import { waitFor } from "../../testing";

package/src/jobs/__tests__/job-event-trigger.integration.test.ts

@@ -1,6 +1,6 @@
 import { afterAll, beforeAll, describe, expect, test } from "bun:test";
 import type { Hono } from "hono";
-import { z } from "zod";
+import * as z from "zod";
 import { buildServer, type JwtHelper } from "../../api";
 import { type BunTestDb, createTestDb } from "../../bun-db/__tests__/bun-test-db";
 import { createRegistry, defineFeature, type SessionUser } from "../../engine";

package/src/jobs/__tests__/job-multi-trigger.integration.test.ts

@@ -8,7 +8,7 @@
 
 import { afterAll, beforeAll, describe, expect, test } from "bun:test";
 import type { Hono } from "hono";
-import { z } from "zod";
+import * as z from "zod";
 import { buildServer, type JwtHelper } from "../../api";
 import { type BunTestDb, createTestDb } from "../../bun-db/__tests__/bun-test-db";
 import { createRegistry, defineFeature, type SessionUser } from "../../engine";

package/src/jobs/__tests__/job-public-intake-origin.integration.test.ts

@@ -2,7 +2,7 @@
 
 import { afterAll, beforeAll, beforeEach, describe, expect, test } from "bun:test";
 import { Queue } from "bullmq";
-import { z } from "zod";
+import * as z from "zod";
 import type { SchemaTable } from "../../db";
 import { createEventStoreExecutor } from "../../db/event-store-executor";
 import { asRawClient, selectMany } from "../../db/query";

package/src/jobs/__tests__/job-retention.integration.test.ts

@@ -7,7 +7,7 @@
 
 import { afterAll, beforeAll, describe, expect, test } from "bun:test";
 import { Queue } from "bullmq";
-import { z } from "zod";
+import * as z from "zod";
 import { createRegistry, defineFeature } from "../../engine";
 import type { AppContext, TenantId } from "../../engine/types";
 import { createTestRedis, type TestRedis } from "../../stack";

package/src/jobs/__tests__/job-write-as.integration.test.ts

@@ -9,7 +9,7 @@
 // to the actor rather than to SYSTEM.
 
 import { afterAll, beforeAll, describe, expect, test } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import { ROLES } from "../../auth/roles";
 import { type BunTestDb, createTestDb } from "../../bun-db/__tests__/bun-test-db";
 import { asRawClient } from "../../db/query";

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

@@ -1,6 +1,6 @@
 import { afterAll, beforeAll, describe, expect, test } from "bun:test";
 import { Queue } from "bullmq";
-import { z } from "zod";
+import * as z from "zod";
 import { requestContext } from "../../api/request-context";
 import { tenantDbRunner } from "../../db/tenant-db-runner";
 import { createRegistry, defineFeature } from "../../engine";

package/src/observability/__tests__/metrics-wiring.test.ts

@@ -1,5 +1,5 @@
 import { describe, expect, it } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import { composeEnvSchema, readKumikoMeta } from "../../env";
 import { prometheusMetricsEnvSchema, resolveObservabilityWiring } from "../metrics-wiring";
 

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

@@ -1,5 +1,5 @@
 import { afterAll, afterEach, beforeAll, beforeEach, describe, expect, it } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import type { TableColumns } from "../../db/dialect";
 import { buildEntityTable } from "../../db/table-builder";
 import { createRegistry, defineFeature } from "../../engine";

package/src/observability/metrics-wiring.ts

@@ -1,4 +1,4 @@
-import { z } from "zod";
+import * as z from "zod";
 import { createNoopProvider } from "./noop-provider";
 import { createPrometheusMeter, type PrometheusMeter } from "./prometheus-meter";
 import type { ObservabilityProvider } from "./types";

package/src/pipeline/__tests__/archive-stream.integration.test.ts

@@ -6,7 +6,7 @@
 // streams never pay for extra metadata writes.
 
 import { afterAll, afterEach, beforeAll, describe, expect, test } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import { createEventStoreExecutor } from "../../db/event-store-executor";
 import { buildEntityTable } from "../../db/table-builder";
 import { createEntity, createTextField, defineFeature } from "../../engine";

package/src/pipeline/__tests__/causation-chain.integration.test.ts

@@ -16,7 +16,7 @@
 // msp-multi-hop.integration.ts.
 
 import { afterAll, afterEach, beforeAll, describe, expect, test } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import { requestContext } from "../../api/request-context";
 import { createEventStoreExecutor } from "../../db/event-store-executor";
 import { selectMany } from "../../db/query";

package/src/pipeline/__tests__/ctx-bridge.integration.test.ts

@@ -1,5 +1,5 @@
 import { afterAll, beforeAll, beforeEach, describe, expect, test } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import { createEventStoreExecutor } from "../../db/event-store-executor";
 import { asRawClient, selectMany } from "../../db/query";
 import { buildEntityTable } from "../../db/table-builder";

package/src/pipeline/__tests__/ctx-systemdb.integration.test.ts

@@ -1,5 +1,5 @@
 import { afterAll, beforeAll, describe, expect, test } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import { defineFeature } from "../../engine";
 import { setupTestStack, type TestStack, TestUsers } from "../../stack";
 

package/src/pipeline/__tests__/dispatcher.test.ts

@@ -1,5 +1,5 @@
 import { describe, expect, test } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import type { SseBroker } from "../../api/sse-broker";
 import { createEntity, createRegistry, createTextField, defineFeature } from "../../engine";
 import type { TenantId } from "../../engine/types/identifiers";

package/src/pipeline/__tests__/domain-events-projections.integration.test.ts

@@ -10,7 +10,7 @@
 // projections-runner fires on appendEvent, dispatcher routes appendEvent to
 // the aggregate stream), any of the assertions below go red.
 import { afterAll, afterEach, beforeAll, describe, expect, test } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import {
   integer as pgInteger,
   table as pgTable,

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

@@ -4,7 +4,7 @@
 // system-identity-switch.integration.test.ts.
 
 import { afterAll, beforeAll, describe, expect, test } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import { defineUnmanagedTable } from "../../db/entity-table-meta";
 import { executeRawQuery } from "../../db/queries/raw-sql";
 import { createSystemUser, defineFeature } from "../../engine";

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

@@ -3,7 +3,7 @@
 // Real HTTP calls + setupTestStack — never createTestDispatcher.
 
 import { afterAll, beforeAll, describe, expect, test } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import { defineUnmanagedTable } from "../../db/entity-table-meta";
 import { defineFeature } from "../../engine";
 import { createTestUser, setupTestStack, type TestStack } from "../../stack";

package/src/pipeline/__tests__/event-dedup.integration.test.ts

@@ -1,5 +1,5 @@
 import { afterAll, beforeAll, beforeEach, describe, expect, test } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import { createEventStoreExecutor } from "../../db/event-store-executor";
 import { defineFeature, type SaveContext } from "../../engine";
 import {

package/src/pipeline/__tests__/event-define-event-strict.integration.test.ts

@@ -11,7 +11,7 @@
 //   3. r.defineEvent returns `{ name: qualifiedName, schema }` — callers
 //      pass `def.name` to ctx.appendEvent without building the qn manually.
 import { afterEach, beforeAll, describe, expect, test } from "bun:test";
-import { z } from "zod";
+import * as z from "zod";
 import { selectMany } from "../../db/query";
 import { defineFeature } from "../../engine";
 import { eventsTable } from "../../event-store";

package/src/pipeline/__tests__/event-dispatcher-batch-delivery.test.ts

@@ -0,0 +1,178 @@
+// #3227 — deliverEvents' batchHandler fast-path: batch success must land
+// the exact outcome the per-event loop would produce, a batch throw falls
+// back to per-event delivery without bumping attempts for the failed
+// batch attempt itself, and pending-gap resolution still splits pending
+// vs new ids the same way it does for the per-event path.
+
+import { beforeAll, describe, expect, test } from "bun:test";
+import type { AppContext } from "../../engine/types";
+import { ensureTemporalPolyfill } from "../../time/polyfill";
+import type { EventConsumer } from "../event-dispatcher";
+import {
+  type ConsumerStateRow,
+  deliverEvents,
+  type StoredEventRow,
+} from "../event-dispatcher-delivery";
+
+beforeAll(async () => {
+  await ensureTemporalPolyfill();
+});
+
+function stubContext(): AppContext {
+  return {
+    db: {} as unknown as AppContext["db"],
+    redis: {} as unknown as AppContext["redis"],
+    registry: {} as unknown as AppContext["registry"],
+  } as AppContext;
+}
+
+function stubEvent(id: bigint): StoredEventRow {
+  return {
+    id,
+    aggregateId: "agg-1",
+    aggregateType: "widget",
+    tenantId: "tenant-1",
+    version: 1,
+    type: "widget.created",
+    eventVersion: 1,
+    payload: {},
+    metadata: { userId: "system" },
+    createdAt: Temporal.Now.instant(),
+    createdBy: "system",
+  };
+}
+
+function stubState(overrides: Partial<ConsumerStateRow> = {}): ConsumerStateRow {
+  return {
+    name: "consumer",
+    instanceId: "shared",
+    lastProcessedEventId: 0n,
+    status: "idle",
+    attempts: 0,
+    rearmCount: 0,
+    pendingGaps: [],
+    lastError: null,
+    updatedAt: Temporal.Now.instant(),
+    ...overrides,
+  };
+}
+
+describe("deliverEvents — batchHandler fast-path", () => {
+  test("batch success calls batchHandler once, never handler, and matches the per-event outcome", async () => {
+    const rows = [1n, 2n, 3n].map(stubEvent);
+    const batchCalls: string[][] = [];
+    let handlerCalls = 0;
+
+    const batchConsumer: EventConsumer = {
+      name: "search",
+      batchHandler: async (events) => {
+        batchCalls.push(events.map((e) => e.id));
+      },
+      handler: async () => {
+        handlerCalls += 1;
+      },
+    };
+    const perEventConsumer: EventConsumer = {
+      name: "search",
+      handler: async () => {
+        // no-op — succeeds for every event, same as the batch above
+      },
+    };
+
+    const batchOutcome = await deliverEvents(batchConsumer, rows, stubContext(), 10, stubState());
+    const perEventOutcome = await deliverEvents(
+      perEventConsumer,
+      rows,
+      stubContext(),
+      10,
+      stubState(),
+    );
+
+    expect(batchCalls).toEqual([["1", "2", "3"]]);
+    expect(handlerCalls).toBe(0);
+    expect(batchOutcome).toEqual(perEventOutcome);
+    expect(batchOutcome.cursor).toBe(3n);
+    expect(batchOutcome.attempts).toBe(0);
+    expect(batchOutcome.lastError).toBeNull();
+    expect(batchOutcome.processed).toBe(3);
+    expect(batchOutcome.failed).toBe(0);
+  });
+
+  test("pending-gap ids resolve the same way as per-event delivery", async () => {
+    const rows = [4n, 7n, 11n, 12n].map(stubEvent);
+    const consumer: EventConsumer = {
+      name: "search",
+      batchHandler: async () => {
+        // succeeds
+      },
+      handler: async () => {
+        throw new Error("handler must not run on batch success");
+      },
+    };
+
+    const outcome = await deliverEvents(
+      consumer,
+      rows,
+      stubContext(),
+      10,
+      stubState({ lastProcessedEventId: 10n }),
+    );
+
+    expect(outcome.resolvedPendingIds).toEqual([4n, 7n]);
+    expect(outcome.cursor).toBe(12n);
+  });
+
+  test("batch throws, one poison row: falls back to per-event delivery without bumping attempts for the batch failure itself", async () => {
+    const rows = [1n, 2n, 3n, 4n, 5n].map(stubEvent);
+    const handlerCalls: string[] = [];
+    const consumer: EventConsumer = {
+      name: "search",
+      errorPolicy: { maxAttempts: 2 },
+      batchHandler: async () => {
+        throw new Error("batch failed");
+      },
+      handler: async (event) => {
+        handlerCalls.push(event.id);
+        if (event.id === "3") throw new Error("poison event");
+      },
+    };
+
+    const first = await deliverEvents(consumer, rows, stubContext(), 10, stubState());
+    expect(first.cursor).toBe(2n);
+    expect(first.attempts).toBe(1);
+    expect(first.deadLettered).toBe(false);
+    expect(handlerCalls).toEqual(["1", "2", "3"]);
+
+    const remaining = [3n, 4n, 5n].map(stubEvent);
+    const second = await deliverEvents(
+      consumer,
+      remaining,
+      stubContext(),
+      10,
+      stubState({ lastProcessedEventId: first.cursor, attempts: first.attempts }),
+    );
+    expect(second.attempts).toBe(2);
+    expect(second.deadLettered).toBe(true);
+  });
+
+  test("batch throws with skipApplyErrors: true skips the poison row and delivers the rest via handler", async () => {
+    const rows = [1n, 2n, 3n, 4n, 5n].map(stubEvent);
+    const handlerCalls: string[] = [];
+    const consumer: EventConsumer = {
+      name: "search",
+      errorPolicy: { skipApplyErrors: true },
+      batchHandler: async () => {
+        throw new Error("batch failed");
+      },
+      handler: async (event) => {
+        handlerCalls.push(event.id);
+        if (event.id === "3") throw new Error("poison event");
+      },
+    };
+
+    const outcome = await deliverEvents(consumer, rows, stubContext(), 10, stubState());
+
+    expect(outcome.cursor).toBe(5n);
+    expect(handlerCalls).toEqual(["1", "2", "3", "4", "5"]);
+  });
+});