@cosmicdrift/kumiko-bundled-features 0.79.0 → 0.79.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-bundled-features",
-  "version": "0.79.0",
+  "version": "0.79.3",
   "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>",
@@ -84,11 +84,11 @@
     "./step-dispatcher": "./src/step-dispatcher/index.ts"
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-dispatcher-live": "0.79.0",
-    "@cosmicdrift/kumiko-framework": "0.79.0",
-    "@cosmicdrift/kumiko-headless": "0.79.0",
-    "@cosmicdrift/kumiko-renderer": "0.79.0",
-    "@cosmicdrift/kumiko-renderer-web": "0.79.0",
+    "@cosmicdrift/kumiko-dispatcher-live": "0.79.3",
+    "@cosmicdrift/kumiko-framework": "0.79.3",
+    "@cosmicdrift/kumiko-headless": "0.79.3",
+    "@cosmicdrift/kumiko-renderer": "0.79.3",
+    "@cosmicdrift/kumiko-renderer-web": "0.79.3",
     "@mollie/api-client": "^4.5.0",
     "@node-rs/argon2": "^2.0.2",
     "@types/nodemailer": "^8.0.0",

package/src/user-data-rights/__tests__/download.integration.test.ts

@@ -374,19 +374,22 @@ describe("download-by-token :: error paths", () => {
 });
 
 describe("download-by-job :: happy path", () => {
-  test("session-auth: Job-Owner → returns signed URL + audit", async () => {
+  test("session-auth: Job-Owner → returns signed URL + audit (IP aus X-Forwarded-For)", async () => {
     const { jobId } = await seedDoneJobWithToken();
 
-    const result = await stack.http.queryOk<{ url: string }>(
+    // Der UI-Klick laeuft als direkter download-by-job-Query (Client traegt
+    // X-CSRF-Token). Die Audit-IP kommt server-trusted aus dem RequestContext
+    // (X-Forwarded-For, erster Hop), nicht aus einem vom Client mitgeschickten
+    // Feld.
+    const res = await stack.http.queryWithHeaders(
       "user-data-rights:query:download-by-job",
-      {
-        jobId,
-        auditMeta: { ip: "10.0.0.5", userAgent: "Mozilla/5.0" },
-      },
+      { jobId },
       aliceUser,
+      { "x-forwarded-for": "10.0.0.5, 10.0.0.1" },
     );
-
-    expect(result.url).toMatch(/^memory:\/\//);
+    expect(res.status).toBe(200);
+    const body = (await res.json()) as { data: { url: string } };
+    expect(body.data.url).toMatch(/^memory:\/\//);
 
     // UI-Klick zaehlt auch als Use → audit-row updated
     const [row] = (await selectMany(stack.db, exportDownloadTokensTable, { jobId })) as Array<{

package/src/user-data-rights/__tests__/run-export-jobs-cron-context.integration.test.ts

@@ -0,0 +1,148 @@
+// Treibt den ECHTEN registrierten Export-Cron-Job (r.job "run-export-jobs")
+// über seinen Job-Kontext — so wie der Job-Runner ihn in prod aufruft:
+// `ctx.configResolver` gesetzt (App-Override provider=inmemory), aber KEIN
+// per-request `ctx.config` (das baut nur der HTTP-Dispatcher).
+//
+// Der bestehende run-export-jobs-Test reicht `buildStorageProvider` MANUELL —
+// und übersprang damit genau diesen Pfad: der Wrapper baut providerCtx aus dem
+// Job-Kontext. Ohne den configResolver→ConfigAccessor-Bau wirft
+// createFileProviderForTenant "ctx.config is missing" (genau der prod-Bug, der
+// jeden Export auf "failed" setzte).
+
+import { afterAll, beforeAll, beforeEach, describe, expect, test } from "bun:test";
+import { asRawClient, insertOne, selectMany } from "@cosmicdrift/kumiko-framework/bun-db";
+import { SYSTEM_USER_ID } from "@cosmicdrift/kumiko-framework/engine";
+import { createEventsTable } from "@cosmicdrift/kumiko-framework/event-store";
+import {
+  setupTestStack,
+  type TestStack,
+  unsafeCreateEntityTable,
+  unsafePushTables,
+} from "@cosmicdrift/kumiko-framework/stack";
+import { getTemporal } from "@cosmicdrift/kumiko-framework/time";
+import {
+  createComplianceProfilesFeature,
+  tenantComplianceProfileEntity,
+} from "../../compliance-profiles";
+import { configValuesTable, createConfigFeature, createConfigResolver } from "../../config";
+import { createDataRetentionFeature } from "../../data-retention";
+import { fileFoundationFeature } from "../../file-foundation";
+import { fileProviderInMemoryFeature } from "../../file-provider-inmemory";
+import { createSessionsFeature } from "../../sessions";
+import { createUserFeature, USER_STATUS, userEntity, userTable } from "../../user";
+import { createUserDataRightsFeature } from "../feature";
+import { exportDownloadTokenEntity } from "../schema/download-token";
+import { EXPORT_JOB_STATUS, exportJobEntity, exportJobsTable } from "../schema/export-job";
+
+const TENANT = "00000000-0000-4000-8000-0000000009a1";
+const USER_ID = "00000000-0000-4000-8000-0000000009b1";
+const JOB_QN = "user-data-rights:job:run-export-jobs";
+
+// App-weiter Override wie money-horse's cashColtConfigResolver — provider=inmemory
+// ohne per-Tenant-config-Row. Der Job-Kontext trägt DIESEN resolver, kein config.
+const configResolver = createConfigResolver({
+  appOverrides: new Map([["file-foundation:config:provider", "inmemory"]]),
+});
+
+let stack: TestStack;
+
+beforeAll(async () => {
+  stack = await setupTestStack({
+    features: [
+      createConfigFeature(),
+      createUserFeature(),
+      createDataRetentionFeature(),
+      createComplianceProfilesFeature(),
+      fileFoundationFeature,
+      fileProviderInMemoryFeature,
+      createSessionsFeature(),
+      createUserDataRightsFeature(),
+    ],
+  });
+  await createEventsTable(stack.db);
+  await unsafePushTables(stack.db, { configValuesTable });
+  await unsafeCreateEntityTable(stack.db, exportJobEntity);
+  await unsafeCreateEntityTable(stack.db, exportDownloadTokenEntity);
+  await unsafeCreateEntityTable(stack.db, tenantComplianceProfileEntity);
+  await unsafeCreateEntityTable(stack.db, userEntity);
+  await asRawClient(stack.db).unsafe(`
+    CREATE TABLE IF NOT EXISTS read_tenant_memberships (
+      id UUID PRIMARY KEY DEFAULT gen_random_uuid(),
+      tenant_id UUID NOT NULL,
+      user_id TEXT NOT NULL,
+      version INTEGER NOT NULL DEFAULT 0,
+      inserted_at TIMESTAMPTZ NOT NULL DEFAULT now(),
+      modified_at TIMESTAMPTZ NOT NULL DEFAULT now(),
+      inserted_by_id TEXT,
+      modified_by_id TEXT,
+      is_deleted BOOLEAN NOT NULL DEFAULT false,
+      deleted_at TIMESTAMPTZ,
+      deleted_by_id TEXT,
+      roles TEXT NOT NULL DEFAULT '[]',
+      UNIQUE(user_id, tenant_id)
+    )
+  `);
+});
+
+afterAll(async () => {
+  await stack.cleanup();
+});
+
+beforeEach(async () => {
+  const raw = asRawClient(stack.db);
+  await raw.unsafe("DELETE FROM read_export_jobs");
+  await raw.unsafe("DELETE FROM read_users");
+  await raw.unsafe("DELETE FROM read_tenant_memberships");
+  await insertOne(stack.db, userTable, {
+    id: USER_ID,
+    tenantId: TENANT,
+    email: "export-cron@example.test",
+    passwordHash: "hashed",
+    displayName: "Cron Export",
+    locale: "de",
+    emailVerified: true,
+    roles: '["Member"]',
+    status: USER_STATUS.Active,
+  });
+  await raw.unsafe(
+    `INSERT INTO read_tenant_memberships (tenant_id, user_id, roles) VALUES ('${TENANT}', '${USER_ID}', '["Member"]')`,
+  );
+});
+
+// Seedet einen pending Export-Job über den echten request-export-Handler.
+async function seedPendingJob(): Promise<string> {
+  const res = await stack.http.writeOk<{ jobId: string }>(
+    "user-data-rights:write:request-export",
+    {},
+    { id: USER_ID, tenantId: TENANT, roles: ["Member"] },
+  );
+  return res.jobId;
+}
+
+describe("run-export-jobs cron-context", () => {
+  test("Cron-Job-Kontext (configResolver, KEIN config) → Export läuft durch, bytesWritten > 0", async () => {
+    const jobId = await seedPendingJob();
+    const job = stack.registry.getJob(JOB_QN);
+    expect(job).toBeDefined();
+
+    // EXAKT der prod-Job-Kontext: configResolver gesetzt, config undefined.
+    const jobCtx = {
+      db: stack.db,
+      registry: stack.registry,
+      configResolver,
+      _userId: SYSTEM_USER_ID,
+      now: getTemporal().Now.instant(),
+    };
+    // Vor dem Fix wirft der Wrapper hier "ctx.config is missing".
+    await job?.handler({}, jobCtx as never);
+
+    const [row] = (await selectMany(stack.db, exportJobsTable, { id: jobId })) as Array<{
+      status: string;
+      bytesWritten: number | null;
+      errorMessage: string | null;
+    }>;
+    expect(row?.errorMessage).toBeNull();
+    expect(row?.status).toBe(EXPORT_JOB_STATUS.Done);
+    expect(row?.bytesWritten ?? 0).toBeGreaterThan(0);
+  });
+});

package/src/user-data-rights/constants.ts

@@ -13,6 +13,7 @@ export const PRIVACY_CENTER_SCREEN_ID = "privacy-center" as const;
 export const UserDataRightsQueries = {
   exportStatus: "user-data-rights:query:export-status",
   myAuditLog: "user-data-rights:query:my-audit-log",
+  downloadByJob: "user-data-rights:query:download-by-job",
 } as const;
 
 export const UserDataRightsHandlers = {
@@ -28,13 +29,6 @@ export const UserDataRightsHandlers = {
 // Screen-Test vergleicht gegen UserQueries.me.
 export const USER_ME_QUERY = "user:query:user:me" as const;
 
-// Download-Pfad des fertigen Export-Bundles: der dokumentierte UI-Klick-Pfad
-// (r.httpRoute in feature.ts), der per 302 auf die signed Storage-URL
-// weiterleitet. Anchor-navigierbar (Cookie-Auth wird mitgesendet).
-export function userExportByJobPath(jobId: string): string {
-  return `/user-export/by-job/${jobId}`;
-}
-
 // Client-safe Mirror von EXPORT_JOB_STATUS (schema/export-job.ts ist
 // server-only via Drizzle-Import). Drift-Schutz: der Screen-Test vergleicht
 // gegen die Schema-Originale.

package/src/user-data-rights/feature.ts

@@ -4,6 +4,7 @@ import {
   type FeatureDefinition,
   SYSTEM_USER_ID,
 } from "@cosmicdrift/kumiko-framework/engine";
+import { createConfigAccessor } from "../config";
 import { createFileProviderForTenant } from "../file-foundation";
 import { PRIVACY_CENTER_SCREEN_ID } from "./constants";
 import { cancelDeletionWrite } from "./handlers/cancel-deletion.write";
@@ -219,17 +220,18 @@ export function createUserDataRightsFeature(opts: UserDataRightsOptions = {}): F
       access: { openToAll: true },
     });
 
-    // r.httpRoute-Wrapper: Magic-Link-Pfad (anonymous) + UI-Klick-Pfad.
+    // Magic-Link-Pfad (anonymous): GET /user-export/by-token?token=<plain>.
+    // Ruft via app.fetch /api/query → success: 302-Redirect zur signed-URL →
+    // Browser folgt → Download startet beim Object-Store. Bei error:
+    // passthrough (404/410/501) als JSON.
     //
-    // Beide rufen via app.fetch /api/query → wenn success: 302-Redirect
-    // zur signed-URL → Browser folgt → Download startet beim Object-Store.
-    // Bei error: passthrough (404/410/501) als JSON.
+    // Path liegt AUSSERHALB /api/* weil r.httpRoute den /api-namespace nicht
+    // claimen darf (reserved fuer write/query/batch/auth/sse-dispatcher).
     //
-    // **Token-Pfad (anonymous):** GET /user-export/by-token?token=<plain>
-    //
-    // Path liegt AUSSERHALB /api/* weil r.httpRoute den /api-namespace
-    // nicht claimen darf (reserved fuer write/query/batch/auth/sse-
-    // dispatcher).
+    // Der Session-Pfad (eingeloggter Download) braucht KEINEN httpRoute-
+    // Wrapper: der Client ruft download-by-job direkt via Dispatcher (traegt
+    // X-CSRF-Token mit) und navigiert auf die zurueckgegebene signed-URL —
+    // siehe postWithDownload im privacy-center-screen.
     r.httpRoute({
       method: "GET",
       path: "/user-export/by-token",
@@ -254,33 +256,6 @@ export function createUserDataRightsFeature(opts: UserDataRightsOptions = {}): F
       },
     });
 
-    // **Session-Pfad (auth):** GET /user-export/by-job/:jobId
-    r.httpRoute({
-      method: "GET",
-      path: "/user-export/by-job/:jobId",
-      handler: async (c, { app }) => {
-        const url = new URL(c.req.url);
-        const jobId = c.req.param("jobId");
-        if (!jobId) {
-          return c.json({ error: "missing_job_id" }, 400);
-        }
-        const queryRes = await app.fetch(
-          new Request(`${url.origin}/api/query`, {
-            method: "POST",
-            headers: {
-              "content-type": "application/json",
-              ...forwardAuthHeaders(c.req.raw.headers),
-            },
-            body: JSON.stringify({
-              type: "user-data-rights:query:download-by-job",
-              payload: { jobId, auditMeta: extractAuditMeta(c.req.raw.headers) },
-            }),
-          }),
-        );
-        return mapQueryResponseToRedirect(c, queryRes);
-      },
-    });
-
     // S2.U3 Atom 3b — Worker fuer Async Export-Pipeline. Cron-getriggert.
     r.job(
       "run-export-jobs",
@@ -295,17 +270,35 @@ export function createUserDataRightsFeature(opts: UserDataRightsOptions = {}): F
         // SYSTEM_USER_ID ist die framework-weite Konvention. Der job-
         // Discriminator wird via handlerName="user-data-rights:run-export-
         // jobs" im Secret-Read-Audit erfasst.
-        const providerCtx = {
-          config: ctx.config,
-          registry: ctx.registry,
-          secrets: ctx.secrets,
-          _userId: ctx._userId ?? SYSTEM_USER_ID,
-        };
+        const exportUserId = ctx._userId ?? SYSTEM_USER_ID;
+        const exportDb = ctx.db as import("@cosmicdrift/kumiko-framework/db").DbConnection; // @cast-boundary db-operator
+        const exportRegistry = ctx.registry;
         await runExportJobs({
-          db: ctx.db as import("@cosmicdrift/kumiko-framework/db").DbConnection, // @cast-boundary db-operator
-          registry: ctx.registry,
-          buildStorageProvider: async (tenantId) =>
-            createFileProviderForTenant(providerCtx, tenantId, "user-data-rights:run-export-jobs"), // @wrapper-known semantic-alias
+          db: exportDb,
+          registry: exportRegistry,
+          buildStorageProvider: async (tenantId) => {
+            // ctx.config (per-request ConfigAccessor) existiert nur im HTTP-
+            // Dispatcher; der Cron-Job-Kontext trägt ctx.configResolver. Den
+            // per-Tenant-Accessor daraus bauen (wie der HTTP-Pfad via
+            // _configAccessorFactory) — sonst wirft createFileProviderForTenant
+            // "ctx.config is missing" und jeder Export landet auf failed.
+            const config =
+              ctx.config ??
+              (ctx.configResolver
+                ? createConfigAccessor(
+                    exportRegistry,
+                    ctx.configResolver,
+                    tenantId as Parameters<typeof createConfigAccessor>[2],
+                    exportUserId,
+                    exportDb,
+                  )
+                : undefined);
+            return createFileProviderForTenant(
+              { config, registry: exportRegistry, secrets: ctx.secrets, _userId: exportUserId },
+              tenantId,
+              "user-data-rights:run-export-jobs",
+            );
+          },
           now: T.Now.instant(),
           // Atom 5 — App-Author-Callbacks fuer Email-Notification.
           // Optional: wenn nicht gesetzt, kein Email; User pollt
@@ -344,15 +337,6 @@ async function mapQueryResponseToRedirect(
   return c.redirect(body.data.url, 302);
 }
 
-function forwardAuthHeaders(headers: Headers): Record<string, string> {
-  const out: Record<string, string> = {};
-  const auth = headers.get("authorization");
-  if (auth) out["authorization"] = auth;
-  const cookie = headers.get("cookie");
-  if (cookie) out["cookie"] = cookie;
-  return out;
-}
-
 // Extract Audit-Meta (IP + UA) aus den HTTP-Headers + steck es in die
 // query-payload. Der httpRoute-Wrapper ist trusted-source — er hat den
 // raw-request gesehen, nicht der direkter /api/query-Caller. User der

package/src/user-data-rights/handlers/download-by-job.query.ts

@@ -23,6 +23,7 @@
 //   6. Audit-Update: useCount + 1, IP, UA, lastUsedAt (best-effort)
 //   7. Return {url, expiresAt}
 
+import { requestContext } from "@cosmicdrift/kumiko-framework/api";
 import { fetchOne } from "@cosmicdrift/kumiko-framework/bun-db";
 import { defineQueryHandler } from "@cosmicdrift/kumiko-framework/engine";
 import { NotFoundError, UnprocessableError } from "@cosmicdrift/kumiko-framework/errors";
@@ -54,12 +55,6 @@ export const downloadByJobQuery = defineQueryHandler({
   name: "download-by-job",
   schema: z.object({
     jobId: z.string().min(1, "jobId required"),
-    auditMeta: z
-      .object({
-        ip: z.string().nullable(),
-        userAgent: z.string().nullable(),
-      })
-      .optional(),
   }),
   access: { openToAll: true }, // openToAll = auth-required, kein anonymous
   handler: async (query, ctx) => {
@@ -68,8 +63,13 @@ export const downloadByJobQuery = defineQueryHandler({
     const userId = query.user.id;
     const jobId = query.payload.jobId;
     const tenantId = query.user.tenantId;
-    const auditIp = query.payload.auditMeta?.ip ?? null;
-    const auditUa = query.payload.auditMeta?.userAgent ?? null;
+    // IP aus dem request-scoped Kontext (von requestIdMiddleware aus
+    // x-forwarded-for befuellt) — server-trusted, anders als ein vom Client
+    // mitgeschickter Wert. UA steht nicht im RequestContext; der Audit-Row
+    // laesst sie null (best-effort, via requestId in den Server-Logs
+    // cross-referenzierbar).
+    const auditIp = requestContext.get()?.ip ?? null;
+    const auditUa: string | null = null;
 
     // Step 1-2: job-lookup + cross-user-isolation
     // ctx.db.raw weil tenant-agnostisch — Alice in Tenant B sucht den
@@ -179,8 +179,8 @@ export const downloadByJobQuery = defineQueryHandler({
         tokenUseCount: tokenRow.useCount ?? 0,
         tenantId: jobRow.requestedFromTenantId,
         now,
-        ip: query.payload.auditMeta?.ip ?? null,
-        userAgent: query.payload.auditMeta?.userAgent ?? null,
+        ip: auditIp,
+        userAgent: auditUa,
       });
     }
     // Wenn tokenRow fehlt (sollte nicht passieren wenn Atom 4a sauber

package/src/user-data-rights/web/__tests__/privacy-center-screen.test.tsx

@@ -139,7 +139,7 @@ describe.skip("PrivacyCenterScreen", () => {
     expect(view.container.textContent).not.toContain("userDataRights.privacyCenter");
   });
 
-  test("export done: Download-Link auf den by-job-Pfad + Verfügbar-bis-Datum", async () => {
+  test("export done: Download-Button + Verfügbar-bis-Datum", async () => {
     const { view } = renderCenter({
       me: activeMe,
       exportStatus: {
@@ -148,8 +148,7 @@ describe.skip("PrivacyCenterScreen", () => {
       },
     });
     await waitForMount(view);
-    const link = view.getByTestId("privacy-export-download") as HTMLAnchorElement;
-    expect(link.getAttribute("href")).toBe("/user-export/by-job/job-123");
+    expect(view.getByTestId("privacy-export-download")).toBeTruthy();
     const ready = view.getByTestId("privacy-export-ready");
     expect(ready.textContent).toContain("2026-07-11");
     expect(ready.textContent).not.toContain("T00:00");

package/src/user-data-rights/web/privacy-center-screen.tsx

@@ -19,6 +19,7 @@ import {
   useQuery,
   useTranslation,
 } from "@cosmicdrift/kumiko-renderer";
+import { postWithDownload } from "@cosmicdrift/kumiko-renderer-web";
 import { type ReactNode, useEffect, useState } from "react";
 import {
   EXPORT_JOB_STATUS,
@@ -26,7 +27,6 @@ import {
   USER_ME_QUERY,
   UserDataRightsHandlers,
   UserDataRightsQueries,
-  userExportByJobPath,
 } from "../constants";
 
 const STATUS_DELETION_REQUESTED = "deletionRequested";
@@ -104,6 +104,15 @@ function ExportSection(): ReactNode {
     void statusQuery.refetch?.();
   };
 
+  // Download laeuft ueber den Dispatcher (traegt X-CSRF-Token) statt ueber
+  // eine <a>-Navigation: download-by-job liefert eine signed URL zurueck, auf
+  // die postWithDownload den Browser navigiert (content-disposition:
+  // attachment → laedt herunter).
+  const downloadExport = async (jobId: string): Promise<void> => {
+    const err = await postWithDownload(dispatcher, UserDataRightsQueries.downloadByJob, { jobId });
+    if (err) setStatus({ kind: "error", messageKey: failureKey(err) });
+  };
+
   const result = statusQuery.data;
   const job = result && result.hasJob ? result.job : null;
   const submitting = status.kind === "submitting";
@@ -150,13 +159,15 @@ function ExportSection(): ReactNode {
               })}
             </p>
           )}
-          <a
-            href={userExportByJobPath(job.id)}
-            data-testid="privacy-export-download"
-            className="mt-2 inline-block font-medium underline"
-          >
-            {t("userDataRights.privacyCenter.export.download")}
-          </a>
+          <div className="mt-2">
+            <Button
+              variant="secondary"
+              onClick={() => void downloadExport(job.id)}
+              testId="privacy-export-download"
+            >
+              {t("userDataRights.privacyCenter.export.download")}
+            </Button>
+          </div>
         </Banner>
       )}
       <StatusBanner status={status} />