@cosmicdrift/kumiko-bundled-features 0.21.0 → 0.22.0

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

@@ -12,10 +12,12 @@
 
 import { afterAll, beforeAll, describe, expect, test } from "bun:test";
 import { asRawClient, insertOne } from "@cosmicdrift/kumiko-framework/bun-db";
+import { fileRefsTable } from "@cosmicdrift/kumiko-framework/files";
 import {
   setupTestStack,
   type TestStack,
   unsafeCreateEntityTable,
+  unsafePushTables,
 } from "@cosmicdrift/kumiko-framework/stack";
 import { createComplianceProfilesFeature } from "../../compliance-profiles";
 import { createDataRetentionFeature } from "../../data-retention";
@@ -53,24 +55,10 @@ beforeAll(async () => {
   // Drizzle-Generated-Queries kollidieren).
   await unsafeCreateEntityTable(stack.db, userEntity);
 
-  // file_refs ist framework-pgTable (nicht entity-getrieben, S1.5 hat
-  // die Schema-Sicht ohne buildEntityTable-Auto-Generation). Manuelle
-  // CREATE matched die Spalten aus framework/src/files/file-ref-table.ts
-  await asRawClient(stack.db).unsafe(`
-    CREATE TABLE IF NOT EXISTS file_refs (
-      id UUID PRIMARY KEY,
-      tenant_id UUID NOT NULL,
-      storage_key TEXT NOT NULL,
-      file_name TEXT NOT NULL,
-      mime_type TEXT NOT NULL,
-      size INTEGER NOT NULL,
-      entity_type TEXT,
-      entity_id TEXT,
-      field_name TEXT,
-      inserted_at TIMESTAMPTZ DEFAULT now() NOT NULL,
-      inserted_by_id TEXT
-    )
-  `);
+  // file_refs ist jetzt das buildEntityTable-getriebene fileRef-Entity
+  // (softDelete → is_deleted/deleted_at/deleted_by_id). Echte Entity-Tabelle
+  // pushen statt hand-CREATE, damit der is_deleted-Filter der Hooks greift.
+  await unsafePushTables(stack.db, { fileRefsTable });
 });
 
 afterAll(async () => {

package/src/user-data-rights-defaults/hooks/file-ref.userdata-hook.ts

@@ -30,9 +30,12 @@ import { fileRefsTable } from "@cosmicdrift/kumiko-framework/files";
 // Cleanup als TODO und faellen das in S2.U5 nochmal an.
 
 export const fileRefExportHook: UserDataExportHook = async (ctx) => {
+  // isDeleted:false — soft-deleted (trashed) Files gehören nicht ins
+  // Auskunfts-Bundle. Forget (delete-Hook unten) erfasst sie trotzdem.
   const rawRows = await selectMany(ctx.db, fileRefsTable, {
     tenantId: ctx.tenantId,
     insertedById: ctx.userId,
+    isDeleted: false,
   });
 
   // @cast-boundary db-row: drizzle liefert insertedAt als Instant

package/src/files/schema/file-ref.ts

@@ -1,58 +0,0 @@
-import {
-  createEntity,
-  createNumberField,
-  createTextField,
-  createTimestampField,
-} from "@cosmicdrift/kumiko-framework/engine";
-
-// fileRef — Schema-Sicht der File-Metadata-Tabelle aus dem Framework.
-//
-// Architektur-Entscheidung (Sprint 1.5):
-//
-// Die DB-Tabelle `file_refs` lebt weiterhin in
-// `framework/src/files/file-ref-table.ts` als drizzle pgTable, weil
-// die Hono-Upload-/Download-Routes (`createFileRoutes` in
-// `framework/src/api/server.ts`) sie direkt nutzen. Multipart-Upload
-// und Binary-Streaming passen nicht in das Write/Query-Handler-Pattern
-// — Routes bleiben framework-internal.
-//
-// Was hier passiert: dieselbe DB-Tabelle wird zusätzlich als
-// `r.entity("fileRef")` in einem bundled-feature deklariert. Das
-// ermoeglicht:
-//   1. r.useExtension(EXT_USER_DATA, "fileRef", { export, delete })
-//      in Sprint 2 — Forget-Flow + Daten-Export erkennen die Entity.
-//   2. r.useExtension(EXT_TENANT_DATA, "fileRef", { destroy })
-//      in Sprint 5 — Tenant-Lifecycle löscht alle FileRefs.
-//   3. Boot-Validation für PII-Annotations greift (fileName, originalName).
-//
-// Kein buildEntityTable hier — die Mapping-Tabelle existiert schon im
-// Framework. Drizzle-Reads in den Sprint-2+-Hooks gehen direkt über
-// `fileRefsTable` aus `@cosmicdrift/kumiko-framework/files`.
-//
-// PII-Annotations (Sprint 0.1+0.7+1.7):
-//   - fileName  → pii: true (Originalname enthält oft Personen-Bezug:
-//                "Marc-Lebenslauf.pdf", "Krankheitsattest-Mai.pdf")
-//
-//   Andere Felder brauchen KEINE Annotation:
-//   - storageKey, mimeType, size, entityType, entityId, fieldName,
-//     insertedById → keine PII-typischen Field-Namen, PII-Heuristik
-//     greift nicht (siehe boot-validator.ts PII_DIRECT_NAME_HINTS).
-//     Ein allowPlaintext-Marker wäre Über-Annotation ohne Effekt.
-//   - insertedAt → Audit-Timestamp, framework-managed.
-//
-// Tabellenname matched die Framework-pgTable damit r.entity-Reads über
-// dieselbe Postgres-Tabelle laufen.
-export const fileRefEntity = createEntity({
-  table: "file_refs",
-  fields: {
-    storageKey: createTextField({ required: true }),
-    fileName: createTextField({ required: true, pii: true }),
-    mimeType: createTextField({ required: true }),
-    size: createNumberField({ required: true }),
-    entityType: createTextField(),
-    entityId: createTextField(),
-    fieldName: createTextField(),
-    insertedAt: createTimestampField({ sortable: true, filterable: true }),
-    insertedById: createTextField(),
-  },
-});