@cosmicdrift/kumiko-bundled-features 0.385.0 → 0.387.0

package/dist/form-draft/handlers/discard.write.js

@@ -63,7 +63,7 @@ export const discardDraftWrite = defineWriteHandler({
             // Fail-closed erase of file_refs rows FIRST (outside the best-effort
             // storage-hygiene loop). releaseDraftFileRefs swallows per-key errors
             // by design — routing forget through it would report discarded:true
-            // while leaving the row behind (#1922 / review #2009).
+            // while leaving the row behind.
             for (const ref of ownedRefs) {
                 const forgetResult = await fileRefExecutor.forget({ id: ref.id }, event.user, ctx.db);
                 if (!forgetResult.isSuccess)

package/dist/form-draft/release-file-refs.js

@@ -1,5 +1,5 @@
 import { isPlainObject } from "@cosmicdrift/kumiko-types/type-guards";
-// The draft blob's `values` is free-form by design (issue #1889: form-draft
+// The draft blob's `values` is free-form by design: form-draft
 // never knows the consuming form's field schema, only { values, stepIndex,
 // savedAt }). A FileRef pointer is therefore recognised purely by shape —
 // any object anywhere in `values` carrying a non-empty string `storageKey`
@@ -45,11 +45,11 @@ export function collectDraftFileRefKeys(draft) {
 //
 // Takes an already-decided key list, not the draft blob — callers must run
 // collectDraftFileRefKeys() output through db/queries' owned-storage-key
-// filter first. `values` is free-form, client-supplied JSON (issue #1889);
-// releasing a key straight out of it without verifying a real, owned
-// file_refs row exists would let a crafted `{ storageKey: "<victim's key>" }`
-// delete someone else's file — the same ownership check file-routes.ts runs
-// before every read/delete (loadFileForTenant + FileAccessGuard).
+// filter first. `values` is free-form, client-supplied JSON; releasing a key
+// straight out of it without verifying a real, owned file_refs row exists
+// would let a crafted `{ storageKey: "<victim's key>" }` delete someone
+// else's file — the same ownership check file-routes.ts runs before every
+// read/delete (loadFileForTenant + FileAccessGuard).
 export async function releaseDraftFileRefs(keys, deleteBinary, log) {
     for (const key of keys) {
         try {

package/dist/form-draft/schemas.js

@@ -10,9 +10,9 @@ function jsonByteLength(value) {
 const draftValuesSchema = z
     .record(z.string(), z.unknown())
     .refine((values) => jsonByteLength(values) <= FORM_DRAFT_VALUES_MAX_BYTES, `values exceeds the ${FORM_DRAFT_VALUES_MAX_BYTES}-byte limit`);
-// The draft blob shape — fixed by issue #1889 here, not left for consumers
+// The draft blob shape — fixed here, not left for consumers
 // to invent per-app. `stepIndex` is in from the start so wizard-mode resume
-// (kumiko-framework#1884) doesn't need a later migration.
+// doesn't need a later migration.
 export const formDraftBlobSchema = z.object({
     values: z.record(z.string(), z.unknown()),
     stepIndex: z.number().int().min(0),
@@ -30,7 +30,7 @@ export const saveDraftPayloadSchema = z.object({
 // row right after the domain entity write already carries the same
 // storageKeys forward — releasing them here would destroy files the entity
 // now depends on. Only an actual abandon/abort flow should pass
-// releaseFiles: true (issue #1915's "explicit abort" case).
+// releaseFiles: true (the "explicit abort" case).
 export const discardDraftPayloadSchema = z.object({
     draftKey: draftKeySchema,
     releaseFiles: z.boolean().optional(),

package/dist/form-draft/web/constants.js

@@ -1,6 +1,4 @@
 // form-draft bundle constants — feature-name + qualified handler/query names.
-//
-// Part of CosmicDriftGameStudio/kumiko-framework#1883 (wizard mode), issue #1889.
 export const FORM_DRAFT_FEATURE_NAME = "form-draft";
 // Qualified handler/query names (QN format: scope:type:name). Clients
 // reference the object instead of magic strings.

package/dist/inbound-mail-foundation/aggregate-id.d.ts

@@ -1,20 +1,20 @@
 /**
- * Deterministic aggregate-id für eine inbound-message. Key ist
- * (accountId, providerMessageId): derselbe Provider-Fetch der zweimal
- * dieselbe Message liefert (IDLE-Doppel-Notify, Cursor-Overlap,
- * Watch-Restart-Replay) landet auf demselben Stream — der Idempotency-
- * Check im ingest-handler sieht den bereits appendeten Event.
+ * Deterministic aggregate ID for an inbound message. The key is
+ * (accountId, providerMessageId): if the provider delivers the same message
+ * twice (IDLE duplicate notification, cursor overlap, watch-restart replay),
+ * both deliveries use the same stream, so the idempotency check in the
+ * ingest handler sees the event that was already appended.
  *
- * accountId (nicht tenantId) im Key: zwei Accounts desselben Tenants
- * können dieselbe Mail empfangen (CC an beide Postfächer) — das sind
- * fachlich ZWEI inbound-messages.
+ * accountId, rather than tenantId, is part of the key because two accounts in
+ * the same tenant can receive the same email (CC'd to both inboxes); these
+ * are two distinct inbound messages.
  */
 export declare function inboundMessageAggregateId(accountId: string, providerMessageId: string): string;
 /**
- * Deterministic aggregate-id für einen mail-thread. Key ist
- * (tenantId, threadKey) — Threads sind tenant-weit, nicht account-weit:
- * eine Konversation die über zwei verbundene Postfächer läuft ist EIN
- * Thread. threadKey kommt normalisiert vom Provider (References/
- * In-Reply-To-Chain bzw. Provider-Thread-ID).
+ * Deterministic aggregate ID for a mail thread. The key is
+ * (tenantId, threadKey): threads are tenant-wide, not account-specific, so a
+ * conversation spanning two connected inboxes is one thread. threadKey is
+ * normalized by the provider (References/In-Reply-To chain or provider
+ * thread ID).
  */
 export declare function mailThreadAggregateId(tenantId: string, threadKey: string): string;

package/dist/inbound-mail-foundation/aggregate-id.js

@@ -1,33 +1,33 @@
 import { v5 as uuidv5 } from "uuid";
-// Fixed UUID-namespaces für deterministic aggregate-id-Ableitung.
-// Generiert einmalig (2026-07-12), in Stein gemeißelt: ein Wechsel würde
-// jeden existing aggregate-Stream re-keyen → kaputte Message-Historie,
-// kaputte Idempotency. Drift-Pins in __tests__/feature.test.ts.
-/** Namespace für inbound-message-Streams. */
+// Fixed UUID namespaces for deterministic aggregate-ID derivation.
+// Generated once (2026-07-12) and immutable: changing one would re-key
+// every existing aggregate stream, breaking message history and idempotency.
+// Drift is pinned in __tests__/feature.test.ts.
+/** Namespace for inbound-message streams. */
 const INBOUND_MESSAGE_NAMESPACE = "7f2a9c41-3d6b-4e0f-9c58-2b1e7a4d8f63";
-/** Namespace für mail-thread-Streams. */
+/** Namespace for mail-thread streams. */
 const MAIL_THREAD_NAMESPACE = "b48d1e72-5c9a-4f36-8e0d-6a3f9b2c7e15";
 /**
- * Deterministic aggregate-id für eine inbound-message. Key ist
- * (accountId, providerMessageId): derselbe Provider-Fetch der zweimal
- * dieselbe Message liefert (IDLE-Doppel-Notify, Cursor-Overlap,
- * Watch-Restart-Replay) landet auf demselben Stream — der Idempotency-
- * Check im ingest-handler sieht den bereits appendeten Event.
+ * Deterministic aggregate ID for an inbound message. The key is
+ * (accountId, providerMessageId): if the provider delivers the same message
+ * twice (IDLE duplicate notification, cursor overlap, watch-restart replay),
+ * both deliveries use the same stream, so the idempotency check in the
+ * ingest handler sees the event that was already appended.
  *
- * accountId (nicht tenantId) im Key: zwei Accounts desselben Tenants
- * können dieselbe Mail empfangen (CC an beide Postfächer) — das sind
- * fachlich ZWEI inbound-messages.
+ * accountId, rather than tenantId, is part of the key because two accounts in
+ * the same tenant can receive the same email (CC'd to both inboxes); these
+ * are two distinct inbound messages.
  */
 // @wrapper-known uuid-domain
 export function inboundMessageAggregateId(accountId, providerMessageId) {
     return uuidv5(`${accountId}:${providerMessageId}`, INBOUND_MESSAGE_NAMESPACE);
 }
 /**
- * Deterministic aggregate-id für einen mail-thread. Key ist
- * (tenantId, threadKey) — Threads sind tenant-weit, nicht account-weit:
- * eine Konversation die über zwei verbundene Postfächer läuft ist EIN
- * Thread. threadKey kommt normalisiert vom Provider (References/
- * In-Reply-To-Chain bzw. Provider-Thread-ID).
+ * Deterministic aggregate ID for a mail thread. The key is
+ * (tenantId, threadKey): threads are tenant-wide, not account-specific, so a
+ * conversation spanning two connected inboxes is one thread. threadKey is
+ * normalized by the provider (References/In-Reply-To chain or provider
+ * thread ID).
  */
 // @wrapper-known uuid-domain
 export function mailThreadAggregateId(tenantId, threadKey) {

package/dist/inbound-mail-foundation/connect-routes.js

@@ -1,5 +1,5 @@
-// createInboundMailConnectRoutes — declarative extraRoutes (kumiko-
-// framework#3050) the app owner mounts via `extraRoutes` in buildServer.
+// createInboundMailConnectRoutes — declarative extraRoutes the app owner mounts
+// via `extraRoutes` in buildServer.
 //
 //   GET /api/inbound-mail/connect      (entry:"user" — signed-in caller)
 //   GET /inbound-mail/oauth/callback   (entry:"signature" — anonymous,
@@ -10,6 +10,7 @@
 // resolved via resolveInboundProviderForKey. The callback trusts ONLY the
 // HMAC-verified state (CSRF-/foreign-claiming lock, see oauth-state.ts).
 import { ExtraRouteRejection, signatureRoute } from "@cosmicdrift/kumiko-framework/api";
+import { propertyOf } from "@cosmicdrift/kumiko-types/type-guards";
 import { INBOUND_MAIL_FOUNDATION_FEATURE, InboundMailAuthMethods, InboundMailFoundationHandlers, inboundCredentialSecretKey, } from "./constants.js";
 import { inboundRefreshTokenSecretOptions, } from "./oauth-access-token.js";
 import { signOAuthState, verifyOAuthState } from "./oauth-state.js";
@@ -130,7 +131,7 @@ export function createInboundMailConnectRoutes(options) {
             if (!dispatched.isSuccess) {
                 return errorJson(c, 502, "account_create_failed", JSON.stringify(dispatched.error ?? {}));
             }
-            const accountId = dispatched.data?.accountId;
+            const accountId = propertyOf(dispatched.data, "accountId");
             if (typeof accountId !== "string") {
                 return errorJson(c, 502, "account_create_failed", "connect-account returned no accountId");
             }

package/dist/inbound-mail-foundation/constants.d.ts

@@ -1,53 +1,51 @@
 export declare const INBOUND_MAIL_FOUNDATION_FEATURE: "inbound-mail-foundation";
 export declare const INBOUND_MAIL_PROVIDER_EXTENSION: "inboundMailProvider";
 export declare const InboundMailFoundationHandlers: {
-    /** Programmatic entry-point für Provider-Plugins (watch-callback +
-     *  fetchSince-Sync): normalisierte Message rein, Idempotency +
-     *  PII-encrypt + event-append + Thread-Update atomic. */
+    /** Programmatic entry point for provider plugins (watch callback +
+     *  fetchSince sync): accepts a normalized message and atomically performs
+     *  idempotency handling, PII encryption, event append, and thread update. */
     readonly ingestMessage: "inbound-mail-foundation:write:ingest-message";
-    /** Tenant-Admin verbindet ein Postfach (IMAP-Credentials via secrets
-     *  oder OAuth-Redirect via oauth-routes). Legt den mail-account-Stream
-     *  an. */
+    /** Tenant admin connects a mailbox (IMAP credentials via secrets or an
+     *  OAuth redirect via oauth-routes). Creates the mail-account stream. */
     readonly connectAccount: "inbound-mail-foundation:write:connect-account";
-    /** Account-Status-Übergänge (watch error/backoff, oauth-refresh
-     *  failed, re-enabled). */
+    /** Account status transitions (watch error/backoff, OAuth refresh
+     *  failure, re-enabled). */
     readonly updateAccount: "inbound-mail-foundation:write:update-account";
-    /** Tenant-Admin trennt ein Postfach. Stream bleibt (Audit), Status
-     *  wird disconnected, Watch stoppt. */
+    /** Tenant admin disconnects a mailbox. The stream remains for auditing;
+     *  the status becomes disconnected and the watch stops. */
     readonly disconnectAccount: "inbound-mail-foundation:write:disconnect-account";
 };
 export declare const InboundMailFoundationQueries: {
-    /** Tenant-scoped Liste der verbundenen Postfächer. */
+    /** Tenant-scoped list of connected mailboxes. */
     readonly listAccounts: "inbound-mail-foundation:query:account:list";
-    /** Tenant-scoped Message-Liste (Inbox-Cockpit). PII wird beim Read
-     *  decrypted. */
+    /** Tenant-scoped message list (inbox cockpit). PII is decrypted on read.
+     */
     readonly listMessages: "inbound-mail-foundation:query:message:list";
 };
 export declare const InboundMailAccountStatuses: {
-    /** Verbunden, Watch/Sync läuft (oder ist startbereit). */
+    /** Connected; watch/sync is running or ready to start. */
     readonly active: "active";
-    /** Credentials/Token ungültig — Tenant-Admin muss re-connecten. */
+    /** Credentials or token are invalid; the tenant admin must reconnect. */
     readonly authError: "auth_error";
     /** Transienter Fehler, Supervisor backoff-retried. */
     readonly degraded: "degraded";
-    /** Vom Tenant-Admin getrennt. Keine Syncs mehr. */
+    /** Disconnected by the tenant admin; no further syncs run. */
     readonly disconnected: "disconnected";
 };
 export type InboundMailAccountStatus = (typeof InboundMailAccountStatuses)[keyof typeof InboundMailAccountStatuses];
 /**
- * Tenant-Secret-Slot eines Accounts: IMAP-App-Passwort ODER OAuth-
- * Refresh-Token — der Provider interpretiert den Inhalt gemäß
- * `authMethod` (Plan §4). Dynamisches Keying pro Account ist vom
- * secrets-Feature gedeckt: SecretsContext.get/set nehmen beliebige
- * String-Keys (resolveKey in secrets-context.ts) — Plan-Risiko 4
- * verifiziert, kein Framework-Gap.
+ * Tenant secret slot for an account: IMAP app password OR OAuth refresh
+ * token. The provider interprets the contents according to `authMethod`.
+ * Dynamic per-account keying is supported by the secrets feature:
+ * SecretsContext.get/set accept arbitrary string keys (resolveKey in
+ * secrets-context.ts), so no framework change is required.
  */
 export declare function inboundCredentialSecretKey(accountId: string): string;
 export declare const InboundMailAuthMethods: {
     /** Formular-Connect: Config (host/port/secure) + Tenant-Secret. */
     readonly password: "password";
-    /** SASL XOAUTH2 über IMAP — Fallback für M365/Gmail-Konten ohne
-     *  gemounteten Enterprise-Provider (gleicher Secret-Slot). */
+    /** SASL XOAUTH2 over IMAP — fallback for M365/Gmail accounts without a
+     *  mounted enterprise provider (same secret slot). */
     readonly xoauth2: "xoauth2";
     /** Nativer OAuth-Provider (m365-graph, gmail-rest). */
     readonly oauth: "oauth";

package/dist/inbound-mail-foundation/constants.js

@@ -1,69 +1,66 @@
 // Feature name
 export const INBOUND_MAIL_FOUNDATION_FEATURE = "inbound-mail-foundation";
-// Extension-point name für Provider-Plugins (inbound-provider-imap,
+// Extension-point name for provider plugins (inbound-provider-imap,
 // inbound-provider-m365-graph, inbound-provider-gmail-rest, ...).
 export const INBOUND_MAIL_PROVIDER_EXTENSION = "inboundMailProvider";
 // Qualified write handler names (QN format: scope:type:name).
 export const InboundMailFoundationHandlers = {
-    /** Programmatic entry-point für Provider-Plugins (watch-callback +
-     *  fetchSince-Sync): normalisierte Message rein, Idempotency +
-     *  PII-encrypt + event-append + Thread-Update atomic. */
+    /** Programmatic entry point for provider plugins (watch callback +
+     *  fetchSince sync): accepts a normalized message and atomically performs
+     *  idempotency handling, PII encryption, event append, and thread update. */
     ingestMessage: "inbound-mail-foundation:write:ingest-message",
-    /** Tenant-Admin verbindet ein Postfach (IMAP-Credentials via secrets
-     *  oder OAuth-Redirect via oauth-routes). Legt den mail-account-Stream
-     *  an. */
+    /** Tenant admin connects a mailbox (IMAP credentials via secrets or an
+     *  OAuth redirect via oauth-routes). Creates the mail-account stream. */
     connectAccount: "inbound-mail-foundation:write:connect-account",
-    /** Account-Status-Übergänge (watch error/backoff, oauth-refresh
-     *  failed, re-enabled). */
+    /** Account status transitions (watch error/backoff, OAuth refresh
+     *  failure, re-enabled). */
     updateAccount: "inbound-mail-foundation:write:update-account",
-    /** Tenant-Admin trennt ein Postfach. Stream bleibt (Audit), Status
-     *  wird disconnected, Watch stoppt. */
+    /** Tenant admin disconnects a mailbox. The stream remains for auditing;
+     *  the status becomes disconnected and the watch stops. */
     disconnectAccount: "inbound-mail-foundation:write:disconnect-account",
 };
 // Qualified query handler names.
 export const InboundMailFoundationQueries = {
-    /** Tenant-scoped Liste der verbundenen Postfächer. */
+    /** Tenant-scoped list of connected mailboxes. */
     listAccounts: "inbound-mail-foundation:query:account:list",
-    /** Tenant-scoped Message-Liste (Inbox-Cockpit). PII wird beim Read
-     *  decrypted. */
+    /** Tenant-scoped message list (inbox cockpit). PII is decrypted on read.
+     */
     listMessages: "inbound-mail-foundation:query:message:list",
 };
 // Normalized account status values — provider-agnostic.
 export const InboundMailAccountStatuses = {
-    /** Verbunden, Watch/Sync läuft (oder ist startbereit). */
+    /** Connected; watch/sync is running or ready to start. */
     active: "active",
-    /** Credentials/Token ungültig — Tenant-Admin muss re-connecten. */
+    /** Credentials or token are invalid; the tenant admin must reconnect. */
     authError: "auth_error",
     /** Transienter Fehler, Supervisor backoff-retried. */
     degraded: "degraded",
-    /** Vom Tenant-Admin getrennt. Keine Syncs mehr. */
+    /** Disconnected by the tenant admin; no further syncs run. */
     disconnected: "disconnected",
 };
 /**
- * Tenant-Secret-Slot eines Accounts: IMAP-App-Passwort ODER OAuth-
- * Refresh-Token — der Provider interpretiert den Inhalt gemäß
- * `authMethod` (Plan §4). Dynamisches Keying pro Account ist vom
- * secrets-Feature gedeckt: SecretsContext.get/set nehmen beliebige
- * String-Keys (resolveKey in secrets-context.ts) — Plan-Risiko 4
- * verifiziert, kein Framework-Gap.
+ * Tenant secret slot for an account: IMAP app password OR OAuth refresh
+ * token. The provider interprets the contents according to `authMethod`.
+ * Dynamic per-account keying is supported by the secrets feature:
+ * SecretsContext.get/set accept arbitrary string keys (resolveKey in
+ * secrets-context.ts), so no framework change is required.
  */
 export function inboundCredentialSecretKey(accountId) {
     return `${INBOUND_MAIL_FOUNDATION_FEATURE}:inbound.credential.${accountId}`;
 }
-// Auth-Methode eines Accounts — vom connect-Flow gesetzt, von Providern
-// gelesen (imap unterscheidet hierüber Passwort-Formular vs. XOAUTH2-
-// Fallback, Plan §2).
+// Authentication method for an account, set by the connect flow and read
+// by providers (IMAP uses it to distinguish the password form from the
+// XOAUTH2 fallback).
 export const InboundMailAuthMethods = {
     /** Formular-Connect: Config (host/port/secure) + Tenant-Secret. */
     password: "password",
-    /** SASL XOAUTH2 über IMAP — Fallback für M365/Gmail-Konten ohne
-     *  gemounteten Enterprise-Provider (gleicher Secret-Slot). */
+    /** SASL XOAUTH2 over IMAP — fallback for M365/Gmail accounts without a
+     *  mounted enterprise provider (same secret slot). */
     xoauth2: "xoauth2",
     /** Nativer OAuth-Provider (m365-graph, gmail-rest). */
     oauth: "oauth",
 };
-// **Multi-Provider von Tag 1:** KEIN globaler `provider`-config-key
-// (anders als mail-foundation, gleich wie billing-foundation). Ein Tenant
-// kann parallel ein IMAP-Postfach UND ein M365-Postfach verbinden — der
-// Provider steht pro MailAccount-Row (`provider`-Feld), gesetzt beim
-// connect-Flow.
+// **Multi-provider:** no global `provider` config key
+// (unlike mail-foundation, as with billing-foundation). A tenant can connect
+// an IMAP mailbox and an M365 mailbox in parallel; the provider is stored per
+// MailAccount row (`provider` field) and set by the connect flow.

package/dist/inbound-mail-foundation/entities.d.ts

@@ -1,5 +1,5 @@
-/** Ein verbundenes Postfach. Eine Row pro Account, PK = aggregateId
- *  (= accountId, random uuid beim connect). */
+/** A connected mailbox. One row per account, with PK = aggregateId
+ *  (= accountId, random uuid on connect). */
 export declare const mailAccountEntity: import("@cosmicdrift/kumiko-types/fields").EntityDefinition<{
     readonly provider: import("@cosmicdrift/kumiko-types/fields").EntityTextFieldDef & {
         required: true;
@@ -38,8 +38,8 @@ export declare const mailAccountEntity: import("@cosmicdrift/kumiko-types/fields
         required: true;
     };
 }, "tenant">;
-/** Eine eingegangene Mail (Envelope + Snippet, KEIN Body — bodyRef
- *  zeigt auf file-foundation). PK = aggregateId =
+/** An incoming mail (envelope + snippet, NO body — bodyRef
+ *  points to file-foundation). PK = aggregateId =
  *  inboundMessageAggregateId(accountId, providerMessageId). */
 export declare const inboundMessageEntity: import("@cosmicdrift/kumiko-types/fields").EntityDefinition<{
     readonly accountId: import("@cosmicdrift/kumiko-types/fields").EntityTextFieldDef & {
@@ -138,10 +138,10 @@ export declare const mailThreadEntity: import("@cosmicdrift/kumiko-types/fields"
 export declare const MAIL_ACCOUNT_PII_FIELDS: readonly string[];
 export declare const INBOUND_MESSAGE_PII_FIELDS: readonly string[];
 export declare const MAIL_THREAD_PII_FIELDS: readonly string[];
-/** Provider-Cursor pro Account ("wo war ich?"): IMAP UIDVALIDITY:UIDNEXT,
- *  Graph deltaLink, Gmail historyId. Eine Row pro (accountId, scope).
- *  Write-locked auf privileged — nur Foundation-Handler/Supervisor
- *  schreiben, kein Tenant-Request. */
+/** Provider cursor per account ("where was I?"): IMAP UIDVALIDITY:UIDNEXT,
+ *  Graph deltaLink, Gmail historyId. One row per (accountId, scope).
+ *  Write-locked to privileged — only Foundation handlers/supervisors
+ *  write it; tenant requests cannot. */
 export declare const syncCursorEntity: import("@cosmicdrift/kumiko-types/fields").EntityDefinition<{
     readonly accountId: import("@cosmicdrift/kumiko-types/fields").EntityTextFieldDef & {
         required: true;
@@ -168,10 +168,10 @@ export declare const syncCursorEntity: import("@cosmicdrift/kumiko-types/fields"
         required: true;
     };
 }, "tenant">;
-/** Dedup-Anchor für O(1)-Idempotency ohne Stream-Scan bei sehr langen
- *  Message-Historien: (accountId, providerMessageId) UNIQUE. Der
- *  ingest-handler prüft zuerst hier (cheap), der Stream-Scan bleibt
- *  Defense-in-depth. */
+/** Deduplication anchor for O(1) idempotency without scanning the stream
+ *  for very long message histories: (accountId, providerMessageId) UNIQUE.
+ *  The ingest handler checks here first (cheap); the stream scan remains
+ *  defense-in-depth. */
 export declare const seenMessageEntity: import("@cosmicdrift/kumiko-types/fields").EntityDefinition<{
     readonly accountId: import("@cosmicdrift/kumiko-types/fields").EntityTextFieldDef & {
         required: true;

package/dist/inbound-mail-foundation/entities.js

@@ -2,27 +2,26 @@ import { collectPiiSubjectFields } from "@cosmicdrift/kumiko-framework/crypto";
 import { deriveEntityTableMeta } from "@cosmicdrift/kumiko-framework/db";
 import { access, createEntity, createNumberField, createTextField, createTimestampField, } from "@cosmicdrift/kumiko-framework/engine";
 // ============================================================================
-// Read-Models der drei event-sourced Aggregates
+// Read models for the three event-sourced aggregates
 // ============================================================================
 //
-// Inline-Projection-Targets. Geschrieben AUSSCHLIESSLICH von den
-// projection-applies (siehe feature.ts), nie direkt vom handler.
-// Source-of-truth sind die event-store-Streams `mail-account` /
-// `inbound-message` / `mail-thread` — die Tabellen sind rebuild-fähig.
+// Inline projection targets. Written EXCLUSIVELY by projection applies
+// (see feature.ts), never directly by the handler.
+// The event-store streams `mail-account` / `inbound-message` / `mail-thread`
+// are the source of truth, so these tables can be rebuilt.
 //
 // **PII convention:** `personal: "tenant"` (not `encrypted`): fields must
-// crypto-shred when eraseSubjectKeys deletes the tenant subject key on
-// tenant-destroy (#800 pattern from billing-foundation).
-// maxLength jeweils Ciphertext-budgetiert (`kumiko-pii:v1:<subject>:
-// <blob>` — Plaintext×~2.3 + Envelope-Overhead).
+// be crypto-shredded when eraseSubjectKeys deletes the tenant subject key on
+// tenant destruction (pattern from billing-foundation).
+// maxLength is budgeted for ciphertext (`kumiko-pii:v1:<subject>:
+// <blob>` — Plaintext×~2.3 + envelope overhead).
 //
-// **Offene Design-Frage aus dem Plan (§3.4):** PII-Subject einer
-// Inbound-Mail ist vorerst der TENANT (Postfach-Inhaber), nicht der
-// externe Absender — ein Absender-Forget-Flow bräuchte per-Sender-Keys
-// und ist bewusst NICHT Teil von Phase 1 (dokumentierte Entscheidung,
-// Erasure-Pfad ist tenant-destroy).
-/** Ein verbundenes Postfach. Eine Row pro Account, PK = aggregateId
- *  (= accountId, random uuid beim connect). */
+// **Design decision:** The PII subject of an inbound mail is currently the
+// TENANT (mailbox owner), not the external sender. A sender-forget flow would
+// require per-sender keys and is deliberately not included; the erasure path
+// is tenant destruction.
+/** A connected mailbox. One row per account, with PK = aggregateId
+ *  (= accountId, random uuid on connect). */
 export const mailAccountEntity = createEntity({
     table: "read_mail_accounts",
     fields: {
@@ -38,10 +37,10 @@ export const mailAccountEntity = createEntity({
             personal: false,
             reason: "technical_reference",
         }),
-        // null = tenant-geteiltes Postfach (info@), gesetzt = persönliches
-        // Postfach dieses Users. Sichtbarkeits-Filter in den list-queries:
-        // Owner + TenantAdmin (Compliance); KEIN Crypto-Subject-Wechsel in
-        // V1 (Subject bleibt Tenant, siehe Header).
+        // null = tenant-shared mailbox (info@), set = this user's personal
+        // mailbox. Visibility filters in the list queries:
+        // Owner + TenantAdmin (compliance); NO crypto-subject change in
+        // V1 (subject remains Tenant, see header).
         ownerUserId: createTextField({ maxLength: 36, personal: false, reason: "pseudonymous_fk" }),
         // Tenant-authored mailbox label, not a person's name.
         displayName: createTextField({
@@ -49,7 +48,7 @@ export const mailAccountEntity = createEntity({
             personal: false,
             reason: "tenant_config_not_personal",
         }),
-        // Postfach-Adresse — PII des Tenants.
+        // Mailbox address — tenant PII.
         address: createTextField({ required: true, maxLength: 1000, personal: "tenant", find: "none" }),
         status: createTextField({
             required: true,
@@ -61,8 +60,8 @@ export const mailAccountEntity = createEntity({
         connectedAt: createTimestampField({ required: true }),
     },
 });
-/** Eine eingegangene Mail (Envelope + Snippet, KEIN Body — bodyRef
- *  zeigt auf file-foundation). PK = aggregateId =
+/** An incoming mail (envelope + snippet, NO body — bodyRef
+ *  points to file-foundation). PK = aggregateId =
  *  inboundMessageAggregateId(accountId, providerMessageId). */
 export const inboundMessageEntity = createEntity({
     table: "read_inbound_messages",
@@ -73,9 +72,9 @@ export const inboundMessageEntity = createEntity({
             personal: false,
             reason: "technical_reference",
         }),
-        // Scope-Vererbung vom Account zum Ingest-Zeitpunkt — Messages eines
-        // persönlichen Postfachs sind nur für den Owner (+ TenantAdmin)
-        // sichtbar, ohne Join auf read_mail_accounts.
+        // Scope is inherited from the account at ingest time: messages from a
+        // personal mailbox are visible only to the owner (+ TenantAdmin),
+        // without joining read_mail_accounts.
         ownerUserId: createTextField({ maxLength: 36, personal: false, reason: "pseudonymous_fk" }),
         messageIdHeader: createTextField({
             required: true,
@@ -116,27 +115,27 @@ export const mailThreadEntity = createEntity({
         messageCount: createNumberField({ required: true, integer: true }),
     },
 });
-// Single source of truth für die PII-Feld-Listen — encrypt (ingest/
-// connect-handler) und decrypt (list-queries) teilen dieselbe Liste,
-// damit ein künftiges weiteres tenantOwned-Feld nicht an einer der
-// Stellen vergessen wird (Muster billing-foundation entities.ts).
+// Single source of truth for the PII field lists — encrypt (ingest/
+// connect-handler) and decrypt (list-queries) share the same list,
+// so a future tenantOwned field cannot be omitted in either location
+// (pattern from billing-foundation entities.ts).
 export const MAIL_ACCOUNT_PII_FIELDS = collectPiiSubjectFields(mailAccountEntity);
 export const INBOUND_MESSAGE_PII_FIELDS = collectPiiSubjectFields(inboundMessageEntity);
 export const MAIL_THREAD_PII_FIELDS = collectPiiSubjectFields(mailThreadEntity);
 // ============================================================================
-// Unmanaged Direct-Write-Stores — Sync-Maschinerie, NICHT event-sourced
+// Unmanaged direct-write stores — sync machinery, NOT event-sourced
 // ============================================================================
 //
-// SyncCursor + SeenMessage sind technischer Tick-State des Watch/Sync-
-// Loops: hochfrequente Writes ohne Business-Fakt-Charakter. Als r.entity
-// würden sie (a) das Event-Log fluten und (b) beim Projection-Rebuild
-// gewischt (Direct-Write ohne Event → Rebuild findet nichts, #494/#498-
-// Klasse). Deshalb r.unmanagedTable — Migration-DDL ja, Rebuild-Target
-// nein (Muster sessions/feature.ts).
-/** Provider-Cursor pro Account ("wo war ich?"): IMAP UIDVALIDITY:UIDNEXT,
- *  Graph deltaLink, Gmail historyId. Eine Row pro (accountId, scope).
- *  Write-locked auf privileged — nur Foundation-Handler/Supervisor
- *  schreiben, kein Tenant-Request. */
+// SyncCursor + SeenMessage are technical tick state for the watch/sync
+// loops: high-frequency writes without business-fact semantics. As
+// r.entity, they would (a) flood the event log and (b) be wiped during a
+// projection rebuild (direct write without an event means the rebuild finds
+// nothing). Therefore r.unmanagedTable — migration DDL, but not a rebuild
+// target (pattern from sessions/feature.ts).
+/** Provider cursor per account ("where was I?"): IMAP UIDVALIDITY:UIDNEXT,
+ *  Graph deltaLink, Gmail historyId. One row per (accountId, scope).
+ *  Write-locked to privileged — only Foundation handlers/supervisors
+ *  write it; tenant requests cannot. */
 export const syncCursorEntity = createEntity({
     table: "store_mail_sync_cursors",
     softDelete: false,
@@ -168,10 +167,10 @@ export const syncCursorEntity = createEntity({
         }),
     },
 });
-/** Dedup-Anchor für O(1)-Idempotency ohne Stream-Scan bei sehr langen
- *  Message-Historien: (accountId, providerMessageId) UNIQUE. Der
- *  ingest-handler prüft zuerst hier (cheap), der Stream-Scan bleibt
- *  Defense-in-depth. */
+/** Deduplication anchor for O(1) idempotency without scanning the stream
+ *  for very long message histories: (accountId, providerMessageId) UNIQUE.
+ *  The ingest handler checks here first (cheap); the stream scan remains
+ *  defense-in-depth. */
 export const seenMessageEntity = createEntity({
     table: "store_mail_seen_messages",
     softDelete: false,
@@ -196,9 +195,9 @@ export const seenMessageEntity = createEntity({
         }),
     },
 });
-// Plain EntityTableMeta (kein branded EntityTable) — unmanaged Direct-
-// Write-Stores, Handler schreiben via ctx.db (siehe user-session.ts-
-// Rationale).
+// Plain EntityTableMeta (not a branded EntityTable) — unmanaged direct-
+// write stores; handlers write via ctx.db (see user-session.ts-
+// rationale).
 export const syncCursorTable = deriveEntityTableMeta("mail-sync-cursor", syncCursorEntity, {
     source: "unmanaged",
 });

package/dist/inbound-mail-foundation/events.d.ts

@@ -52,13 +52,14 @@ export type MailThreadEventPayload = z.infer<typeof mailThreadEventPayloadSchema
 export type InboundMessageEventHeaders = {
     readonly providerMessageId: string;
     readonly providerName: string;
-    /** Provider-Cursor-Snapshot zum Ingest-Zeitpunkt (UIDVALIDITY:UID,
-     *  deltaLink, historyId ...) — Debug/Replay-Hilfe, kein Payload. */
+    /** Provider cursor snapshot at ingest time (UIDVALIDITY:UID, deltaLink,
+     *  historyId ...) — useful for debugging and replay, not part of the
+     *  payload. */
     readonly providerCursor: string;
 };
 export type MailAccountEventHeaders = {
     readonly providerName: string;
-    /** Was den Übergang ausgelöst hat ("connect_flow", "watch_supervisor",
+    /** What triggered the transition ("connect_flow", "watch_supervisor",
      *  "oauth_refresh", "tenant_admin"). */
     readonly reason: string;
 };