@cosmicdrift/kumiko-dev-server 0.77.1 → 0.79.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-dev-server",
-  "version": "0.77.1",
+  "version": "0.79.0",
   "description": "Development server bootstrap for Kumiko apps. Bundles the client, mints dev-JWTs, injects the resolved AppSchema, and seeds an admin. Not for production.",
   "license": "BUSL-1.1",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -50,8 +50,8 @@
     "kumiko-schema-check": "./bin/kumiko-schema-check.ts"
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-bundled-features": "0.77.1",
-    "@cosmicdrift/kumiko-framework": "0.77.1",
+    "@cosmicdrift/kumiko-bundled-features": "0.79.0",
+    "@cosmicdrift/kumiko-framework": "0.79.0",
     "ts-morph": "^28.0.0"
   },
   "publishConfig": {

package/src/__tests__/run-prod-app.integration.test.ts

@@ -30,6 +30,7 @@ import {
   createProjectionStateTable,
 } from "@cosmicdrift/kumiko-framework/pipeline";
 import { unsafeEnsureEntityTable } from "@cosmicdrift/kumiko-framework/stack";
+import { Queue } from "bullmq";
 import postgres from "postgres";
 import { z } from "zod";
 import { type ProdAppHandle, runProdApp } from "../run-prod-app";
@@ -118,6 +119,29 @@ const widgetFeature = defineFeature("prod-probe", (r) => {
   });
 });
 
+// Worker-lane cron — the lane the data-export job (run-export-jobs) lives on.
+// runProdApp must schedule it (single-instance runs both lanes); on the old
+// createApiEntrypoint path it was silently never registered → exports hung.
+const cronProbeFeature = defineFeature("cron-probe", (r) => {
+  r.job("worker-lane-cron", { trigger: { cron: "0 0 1 1 *" }, runIn: "worker" }, async () => {});
+});
+
+async function workerLaneSchedulers(prefix: string): Promise<{ name?: string; key?: string }[]> {
+  const url = new URL(process.env["REDIS_URL"] ?? "redis://localhost:16379");
+  const queue = new Queue(`${prefix}-worker`, {
+    connection: { host: url.hostname, port: Number(url.port) },
+  });
+  // Read-only: do NOT obliterate — the running all-in-one worker consumes this
+  // same queue, and deleting its keys mid-flight aborts the worker's blocking
+  // Redis read with "Connection is closed". The unique prefix isolates the
+  // leftover scheduler; the test Redis is ephemeral.
+  try {
+    return await queue.getJobSchedulers();
+  } finally {
+    await queue.close();
+  }
+}
+
 const TENANT_ID = "00000000-0000-4000-8000-000000000001";
 
 // Per-suite DB so reboots can be tested without conflicting with other
@@ -747,3 +771,33 @@ describe("runProdApp — auth allowedOrigins forwarding", () => {
     }
   });
 });
+
+describe("runProdApp job-lane wiring (runSingleInstance)", () => {
+  // Red-then-green for the export bug: on createApiEntrypoint (old default) the
+  // worker-lane cron was never registered. createAllInOneEntrypoint (new
+  // single-instance default) runs two runners, so it IS registered.
+  test("default single-instance schedules the WORKER-lane cron", async () => {
+    const prefix = `test-rsi-${Date.now().toString(36)}`;
+    const handle = await boot(undefined, {
+      features: [cronProbeFeature],
+      jobs: { queueNamePrefix: prefix },
+    });
+    expect(handle.entrypoint.mode).toBe("all-in-one");
+    const schedulers = await workerLaneSchedulers(prefix);
+    expect(schedulers.some((s) => (s.name ?? s.key ?? "").includes("worker-lane-cron"))).toBe(true);
+  });
+
+  test("runSingleInstance:false → api-only, worker lane left to a dedicated worker", async () => {
+    const prefix = `test-rsi-api-${Date.now().toString(36)}`;
+    const handle = await boot(undefined, {
+      features: [cronProbeFeature],
+      jobs: { queueNamePrefix: prefix },
+      runSingleInstance: false,
+    });
+    expect(handle.entrypoint.mode).toBe("api");
+    const schedulers = await workerLaneSchedulers(prefix);
+    expect(schedulers.some((s) => (s.name ?? s.key ?? "").includes("worker-lane-cron"))).toBe(
+      false,
+    );
+  });
+});

package/src/run-prod-app.ts

@@ -58,8 +58,9 @@ import {
   validateBoot,
 } from "@cosmicdrift/kumiko-framework/engine";
 import {
+  type AllInOneEntrypoint,
   type ApiEntrypoint,
-  type ApiEntrypointOptions,
+  createAllInOneEntrypoint,
   createApiEntrypoint,
 } from "@cosmicdrift/kumiko-framework/entrypoint";
 import {
@@ -416,17 +417,21 @@ export type RunProdAppOptions = {
    *  weiterhin automatisch ergänzt; Factory-Result + auto-resolver
    *  werden gemerged (Factory-Werte überschreiben). */
   readonly extraContext?: ExtraContextOption;
-  /** Job-Block. Wenn das Feature `r.job(...)` registriert, MUSS dieser
-   *  Block gesetzt sein — sonst wirft createApiEntrypoint mit dem
-   *  expliziten "registry declares N job(s)..."-Fehler. Default-Pattern
-   *  für Single-Container-Deployments (publicstatus, kleine SaaS):
-   *  `{ runLocalJobs: true }` — der API-Process consumiert auch die
-   *  Worker-Lane, kein separates worker-Image nötig. Für skalierende
-   *  Setups (mehrere API-Replicas + dezidierte Worker): runLocalJobs
-   *  weglassen + workers via separatem `runWorkerApp` (kommt Phase 4). */
+  /** Deploy-Topologie. Default `true` (Single-Container): dieser Prozess
+   *  fährt HTTP + BEIDE Job-Lanes (api + worker) + den Event-Dispatcher
+   *  (MSP-Anwendung) inline — via `createAllInOneEntrypoint`. Damit laufen
+   *  worker-Lane-Crons (z.B. der Daten-Export `run-export-jobs`, default
+   *  `runIn:"worker"`) und r.multiStreamProjection ohne separaten Worker.
+   *
+   *  `false` NUR mit einem dezidierten Worker-Deployment setzen: dann fährt
+   *  dieser Prozess API-only (`createApiEntrypoint`), und worker-Lane-Jobs
+   *  + MSPs werden NICHT mehr lokal angewandt — der Worker muss sie
+   *  übernehmen, sonst bleiben Export-Jobs pending und die Read-Side leer
+   *  (2026-06-11-Incident-Klasse). */
+  readonly runSingleInstance?: boolean;
+  /** Job-Block. Wenn das Feature `r.job(...)` registriert, wird er
+   *  automatisch verdrahtet (siehe runSingleInstance). */
   readonly jobs?: {
-    /** Default true (Single-Container). */
-    readonly runLocalJobs?: boolean;
     /** BullMQ-Queue-Prefix (default "kumiko"). */
     readonly queueNamePrefix?: string;
   };
@@ -495,11 +500,12 @@ export type RunProdAppOptions = {
 };
 
 export type ProdAppHandle = {
-  /** The composed ApiEntrypoint. In KUMIKO_DRY_RUN_ENV mode WITH
+  /** The composed entrypoint — AllInOne (runSingleInstance, default) or
+   *  Api-only (runSingleInstance:false). In KUMIKO_DRY_RUN_ENV mode WITH
    *  `envSource` injected (test path), no boot ran and this slot is an
    *  undefined-cast — do not access. Production dry-run hits
    *  `process.exit(0)` before returning a handle. */
-  readonly entrypoint: ApiEntrypoint;
+  readonly entrypoint: ApiEntrypoint | AllInOneEntrypoint;
   /** The fetch-handler — wired into Bun.serve in production, called
    *  directly in tests. Composes Hono + static-fallback. */
   readonly fetch: (req: Request) => Promise<Response> | Response;
@@ -707,7 +713,7 @@ export async function runProdApp(options: RunProdAppOptions): Promise<ProdAppHan
     ? buildProdSessionAuth(db, options.auth.sessions)
     : undefined;
 
-  const entrypoint = createApiEntrypoint({
+  const baseEntrypointOptions = {
     registry,
     context: {
       db,
@@ -780,32 +786,48 @@ export async function runProdApp(options: RunProdAppOptions): Promise<ProdAppHan
       },
     }),
     ...(resolvedAnonymousAccess && { anonymousAccess: resolvedAnonymousAccess }),
-    // Auto-Pass-Through für r.job-Wiring: wenn das Registry Jobs
-    // deklariert, MUSS der jobs-Block gesetzt sein — sonst stoppt
-    // createApiEntrypoint mit explizitem Fehler. Default für Single-
-    // Container-Deployments: runLocalJobs=true (API-Process consumiert
-    // auch worker-Lane). Caller kann override'n via options.jobs.
-    ...(registry.getAllJobs().size > 0 && {
-      jobs: {
+  };
+
+  // Deploy-Topologie. Default (Single-Container): createAllInOneEntrypoint
+  // fährt HTTP + BEIDE Job-Lanes (zwei Runner, jeder schedult seine eigene
+  // Lane-Crons → kein Double-Fire) + Event-Dispatcher inline. So laufen
+  // worker-Lane-Crons (run-export-jobs default runIn:"worker") UND MSPs ohne
+  // separaten Worker-Process — die Asymmetrie, an der der Daten-Export hing.
+  // runSingleInstance:false → API-only; ein dezidierter Worker MUSS dann die
+  // worker-Lane + MSPs übernehmen (api-Lane-Jobs laufen weiter lokal).
+  // Default single-instance. eventDispatcher.disabled ist die Alt-Art, MSPs
+  // diesem Prozess wegzunehmen (dezidierter Worker) — als runSingleInstance:
+  // false honorieren (api-only, kein lokaler Dispatcher), damit der explizite
+  // Flag Vorrang behält aber Bestands-Caller nicht brechen.
+  const runSingleInstance = options.runSingleInstance ?? options.eventDispatcher?.disabled !== true;
+  const hasJobs = registry.getAllJobs().size > 0;
+  const queueNamePrefix = options.jobs?.queueNamePrefix;
+  const dispatcherTunables =
+    options.eventDispatcher?.pollIntervalMs !== undefined
+      ? { pollIntervalMs: options.eventDispatcher.pollIntervalMs }
+      : {};
+
+  const entrypoint: ApiEntrypoint | AllInOneEntrypoint = runSingleInstance
+    ? createAllInOneEntrypoint({
+        ...baseEntrypointOptions,
+        // Worker-Seite liest die JobsBlock TOP-LEVEL (nicht nested `jobs` wie
+        // die api-Seite); beide Lane-Runner ziehen redisUrl/prefix von hier.
         redisUrl,
-        runLocalJobs: options.jobs?.runLocalJobs ?? true,
-        ...(options.jobs?.queueNamePrefix !== undefined && {
-          queueNamePrefix: options.jobs.queueNamePrefix,
-        }),
-      },
-    }),
-    // Single-Container: MSPs laufen im API-Process (Default an, analog
-    // runLocalJobs). Ohne lokalen Dispatcher würden r.multiStreamProjection-
-    // Projektionen nie anwenden — es gibt hier keinen Worker-Process.
-    ...(!options.eventDispatcher?.disabled && {
-      eventDispatcher: {
-        runLocal: true,
-        ...(options.eventDispatcher?.pollIntervalMs !== undefined && {
-          pollIntervalMs: options.eventDispatcher.pollIntervalMs,
+        ...(queueNamePrefix !== undefined && { queueNamePrefix }),
+        ...(!options.eventDispatcher?.disabled && { eventDispatcher: dispatcherTunables }),
+      })
+    : createApiEntrypoint({
+        ...baseEntrypointOptions,
+        // API-only: api-Lane-Jobs laufen lokal, ein dezidierter Worker fährt
+        // worker-Lane + MSPs. createApiEntrypoint liest den nested `jobs`-Block.
+        ...(hasJobs && {
+          jobs: {
+            redisUrl,
+            runLocalJobs: true,
+            ...(queueNamePrefix !== undefined && { queueNamePrefix }),
+          },
         }),
-      },
-    }),
-  } satisfies ApiEntrypointOptions);
+      });
 
   // 8. Build the AppSchema once + serialize. Wird beim Static-Fallback
   //    in die index.html injiziert damit createKumikoApp() im Browser