@cosmicdrift/kumiko-renderer 0.386.0 → 0.387.0

package/dist/app/projection-list-shim.js

@@ -1,18 +1,17 @@
-// Shim für ProjectionList-Renderer (analog action-form-shim.ts / config-edit-shim.ts).
+// Shim for the ProjectionList renderer (analogous to action-form-shim.ts / config-edit-shim.ts).
 //
-// RenderList + computeListViewModel verlangen ein `entity: EntityDefinition` +
-// ein `screen: EntityListScreenDefinition`-Pair, nutzen davon aber nur
-// `entity.fields` (Spalten-Typ/Label) und die List-Felder des Screens
-// (columns/rowActions/…). Ein projectionList-Screen ist query-getrieben und hat
-// KEINE Entity — die zwei Helper hier shapen den Input ad-hoc um, damit die
-// bestehende List-Maschinerie reused werden kann. Die Query selbst wird NICHT
-// hierüber aufgelöst (der Body nimmt `screen.query` direkt).
+// RenderList + computeListViewModel require an `entity: EntityDefinition` +
+// `screen: EntityListScreenDefinition` pair, but only use `entity.fields`
+// (column type/label) and the screen's list fields (columns/rowActions/…). A
+// projectionList screen is query-driven and has NO entity, so these two helpers
+// reshape the input ad hoc to reuse the existing list machinery. The query is
+// NOT resolved here (the body reads `screen.query` directly).
 //
-// Selbe Schulden-Reservation wie bei den anderen Shims: greift die List-
-// Maschinerie künftig auf weitere EntityDefinition-Felder (transitions, idType,
-// derivedFields) oder cross-referenziert ein Boot-Validator die schema.entities-
-// Map, brechen die Type-Lies hier silent — dann ist Zeit für eine echte
-// query-native ListView-Komponente.
+// This preserves the same debt reservation as the other shims: if the list
+// machinery later accesses additional EntityDefinition fields (transitions,
+// idType, derivedFields), or a boot validator cross-references the
+// schema.entities map, these type lies will fail silently. At that point, a
+// query-native ListView component is needed.
 import { normalizeListColumn } from "@cosmicdrift/kumiko-framework/ui-types";
 const PROJECTION_PSEUDO_ENTITY = "__projection__";
 // Satisfies the text-field PII stance of the entity field type; these synthesized
@@ -20,7 +19,7 @@ const PROJECTION_PSEUDO_ENTITY = "__projection__";
 export const SYNTHESIZED_FIELD_PII_REASON = "synthesized display field, not stored entity data";
 /** Minimal EntityDefinition from the column list: every field is a text
  *  field. `sortable` applies to all columns — buildAppSchema derives it from
- *  the query's Zod schema (`screen.sortable`, fw#2165); the query itself has
+ *  the query's Zod schema (`screen.sortable`); the query itself has
  *  no per-field server-sort guarantee, so a column opts out with
  *  `sortable: false` when the handler cannot order by it.
  *  computeListViewModel only reads `fields[<col>].type` — text is enough,
@@ -37,10 +36,10 @@ export function synthesizeProjectionEntity(columns, sortable) {
     }
     return { fields };
 }
-/** Wandelt ein ProjectionListScreenDefinition in die EntityListScreen-Shape die
- *  RenderList erwartet. `type` = "entityList" + Pseudo-Entity halten den Type-
- *  Constraint; RenderList branched nicht auf `type` und liest `entity` nur für
- *  Fehlermeldungen. Die List-Felder werden 1:1 durchgereicht. */
+/** Converts a ProjectionListScreenDefinition to the EntityListScreen shape
+ *  expected by RenderList. `type` = "entityList" plus a pseudo-entity satisfy
+ *  the type constraint; RenderList does not branch on `type` and reads `entity`
+ *  only for error messages. The list fields are passed through unchanged. */
 export function synthesizeProjectionScreen(screen) {
     return {
         id: screen.id,

package/dist/app/qn.js

@@ -1,21 +1,19 @@
-// Qualified-Name-Helper. Kumiko's Registry stempelt beim Boot den
-// Feature-Prefix auf jede Screen/Nav/Workspace-id ein:
+// Qualified-name helper. At boot, Kumiko's Registry adds the feature prefix to every Screen/Nav/Workspace ID:
 //
 //   r.screen({ id: "task-list", ... })
 //     → registry-stored.id === "tasks:screen:task-list"
-//     → schema.screens (ans Browser) hat id === "tasks:screen:task-list"
+//     → schema.screens (sent to the browser) has id === "tasks:screen:task-list"
 //
-// Der Renderer arbeitet aber mit Short-Form ids — formatPath schreibt
-// sie 1:1 in die URL, parsePath liest sie 1:1 raus. Beim Übergang von
-// Schema (QN-Form) → nav.navigate (Short-Form) muss der Prefix weg,
-// sonst landet die URL doppelt-qualifiziert
-// ("/tasks:screen:task-list" + Re-Lookup → "tasks:screen:tasks:screen:
+// The renderer uses short-form IDs: formatPath writes them to the URL unchanged,
+// and parsePath reads them back unchanged. When converting from the schema (QN form)
+// to nav.navigate (short form), the prefix must be removed; otherwise the URL becomes
+// doubly qualified
+// ("/tasks:screen:task-list" + re-lookup → "tasks:screen:tasks:screen:
 // task-list" → 404).
 //
-// `lastSegment` ist die Inverse von qualifyScreenId/Nav/Workspace —
-// nimmt den letzten ":"-getrennten Teil. Robust gegen Strings ohne
-// ":" (returnt sie unverändert, damit App-Author-Code mit Short-Form-
-// ids in eigenen Stellen weiter passt).
+// `lastSegment` is the inverse of qualifyScreenId/Nav/Workspace: it takes the final
+// ":"-separated part. Strings without ":" are returned unchanged so app-author code
+// using short-form IDs continues to work.
 export function lastSegment(qn) {
     const idx = qn.lastIndexOf(":");
     return idx < 0 ? qn : qn.slice(idx + 1);

package/dist/app/qualify-screen-id.js

@@ -7,7 +7,7 @@ export function qualifyScreenId(featureName, screenId) {
 // Inverse of qualifyScreenId. A short id ("task-list") has no feature
 // segment and returns undefined — callers need that to tell "this redirect
 // names a feature" apart from "this redirect is feature-less, resolve by
-// short id alone" (fw#2485).
+// short id alone".
 export function featureNameFromQualifiedScreenId(id) {
     const parts = id.split(":");
     return parts.length === 3 && parts[1] === "screen" ? parts[0] : undefined;

package/dist/app/screen-access.js

@@ -1,7 +1,7 @@
 import { isUiAccessGranted } from "@cosmicdrift/kumiko-framework/ui-types";
 import { findDetailForScreen } from "./nav.js";
 import { lastSegment } from "./qn.js";
-// Shared UI-visibility predicate for the screen-render path (#1203 — nav
+// Shared UI-visibility predicate for the screen-render path — nav
 // filtering via filterByAccess in workspace-shell.tsx hid role-gated
 // screens from the menu, but a direct URL/screenQn hit reached
 // KumikoScreen unchecked). Public name kept as an alias of the shared
@@ -9,7 +9,7 @@ import { lastSegment } from "./qn.js";
 export { isUiAccessGranted as screenAccessAllows };
 // A navigate target (metric click) declares no access rule of its own — the
 // destination screen already does, so the jump is offered exactly when the
-// user may open where it lands (solon#424). No second copy of the rule to
+// user may open where it lands. No second copy of the rule to
 // drift from the screen's own.
 export function navigateTargetAllows(navigate, appFeatures, userRoles) {
     const { screen, entity } = navigate;

package/dist/app/screen-shared.js

@@ -166,7 +166,7 @@ export function entityQueryCommand(featureName, entity, verb) {
 // search/sort/sortDirection/limit + offset/totalCount for pager mode OR
 // cursor for infinite scroll. Shared by EntityListBody and
 // ProjectionListBody so the two branches can't drift on this shape
-// (fw#2165) — entity-only additions (screen.filter, faceted filters) are
+// — entity-only additions (screen.filter, faceted filters) are
 // layered on top by the caller instead of living in here.
 export function buildListQueryPayload(state) {
     const payload = { limit: state.limit };

package/dist/app/secret-mint-body.js

@@ -40,10 +40,10 @@ function isBlank(value) {
 // Mint form → one-time reveal → optional confirm → done. The revealed values
 // live ONLY in this component's `revealed` state — never in the URL, a query
 // cache, or nav — the write-handler's success payload is the only place the
-// secret ever exists (fw#2548). onSubmit copies exclusively the fields
+// secret ever exists. onSubmit copies exclusively the fields
 // declared in `screen.reveal.fields` out of that payload (a whitelist, never
 // the payload as a whole) so an unrelated field (e.g. an internal "id") can
-// never leak into the reveal. `screen.confirm` (fw#2838, e.g. TOTP enroll:
+// never leak into the reveal. `screen.confirm` (e.g. TOTP enroll:
 // scan the code, then enter one) works the same way for its own carried
 // mint-payload fields — those live only in `carriedRef`, never in React state
 // that gets rendered, never in the confirm form's own values.
@@ -63,7 +63,7 @@ export function SecretMintBody({ schema, screen, translate }) {
         urlPrefillFields: screen.urlPrefillFields,
         renderableFields: layoutFieldNames(synthScreen),
         // A mint form has no entity either, so its money fields name their own
-        // currency source (fw#2839, fw#2933).
+        // currency source.
         ...(mintCurrencyOverrides !== undefined && {
             moneyCurrencyOverrides: mintCurrencyOverrides,
         }),
@@ -188,7 +188,7 @@ export function SecretMintBody({ schema, screen, translate }) {
         // submit state) across phases.
         if (confirm !== undefined && confirmEntity !== undefined && confirmScreen !== undefined) {
             // Hold until a tenant-declared currency lands, else the form would seed (and
-            // on a fast click submit) the fallback currency (fw#2933).
+            // on a fast click submit) the fallback currency.
             if (confirmCurrencyLoading) {
                 return _jsx(QueryLoadingBanner, {});
             }

package/dist/app/write-failed-error.js

@@ -1,8 +1,8 @@
-// dispatcher.write wirft bei Server-Fehlern NICHT — es returnt
-// { isSuccess: false, error }. Action-Wiring das das Result verwirft,
-// macht Fehler unsichtbar ("Klick tut nichts"-Prod-Bug 2026-06-07).
-// Diese Klasse trägt den strukturierten DispatcherError dahin, wo eine
-// UI ihn anzeigen kann (Toast mit docsUrl).
+// dispatcher.write does NOT throw on server errors; it returns
+// { isSuccess: false, error }. Action wiring that discards the result
+// makes failures invisible ("click does nothing" in production).
+// This class carries the structured DispatcherError to a layer that can
+// display it in the UI, such as a toast with docsUrl.
 export class WriteFailedError extends Error {
     dispatcherError;
     constructor(error, message) {
@@ -11,8 +11,9 @@ export class WriteFailedError extends Error {
         this.dispatcherError = error;
     }
 }
-// i18nKey gewinnt wenn das Bundle ihn kennt (translate returnt den Key
-// unverändert wenn nicht — Renderer-Convention), sonst message, zuletzt code.
+// i18nKey takes precedence when the bundle knows it (translate returns
+// the key unchanged when it does not, per the renderer convention),
+// followed by message and then code.
 export function dispatcherErrorText(error, translate) {
     const translated = translate(error.i18nKey, error.i18nParams);
     if (translated !== error.i18nKey)

package/dist/components/embedded-list-field.js

@@ -71,11 +71,11 @@ function coerceCellValue(column, text, currency) {
             // (renderCellControl's MoneyInput, which scales by the same
             // currencyDecimals) — a hardcoded ×100 here diverged for zero-/three-
             // decimal currencies (JPY, BHD, ...), landing a pasted value 100x off
-            // from the same value typed by hand (kumiko-framework#1972).
+            // from the same value typed by hand.
             // parseLocaleAwareDecimal handles both "1.234,56" (de) and
             // "1,234.56" (en-US) thousands/decimal conventions — a naive
-            // `replace(",", ".")` mangled DE-locale pastes with thousands
-            // separators into NaN (kumiko-framework#1838).
+            // `replace(",", ".")` mangled German-locale pastes with thousands
+            // separators into NaN.
             if (text.trim() === "")
                 return undefined;
             const n = parseLocaleAwareDecimal(text);
@@ -259,7 +259,7 @@ function EmbeddedListFieldInner({ field, id, onChange, allIssues, featureName, }
                 const coerced = coerceCellValue(column, text, field.embeddedListCurrency ?? "EUR");
                 // Also covers an unparseable money paste (e.g. mangled thousands
                 // separators) — the cell must stay unchanged instead of silently
-                // clearing to undefined (kumiko-framework#1838).
+                // clearing to undefined.
                 const isUnmatchedOrUnparseable = (column.type === "select" || column.type === "reference" || column.type === "money") &&
                     coerced === undefined &&
                     text.trim() !== "";

package/dist/components/grid-cell-for-field.d.ts

@@ -7,7 +7,7 @@ export type GridCellForFieldProps = {
     readonly issues: readonly FieldIssue[] | undefined;
     readonly onChange: (value: unknown) => void;
     readonly GridCell: ReturnType<typeof usePrimitives>["GridCell"];
-    /** Tier 2.7e-3: passed through so Reference fields can build the correct
+    /** Passed through so Reference fields can build the correct
      *  lookup query QN (`<feature>:query:<refEntity>:list`). */
     readonly featureName: string;
     readonly labelAppendix?: ReactNode;

package/dist/components/reference-create-dialog.js

@@ -8,7 +8,7 @@ import { usePrimitives } from "../primitives.js";
 import { RenderEdit } from "./render-edit.js";
 // Hosts a target entity's create-form inside a bare Modal so a reference
 // field can create a missing record without leaving the current form
-// (kumiko-framework#1681) — same create wiring as EntityEditCreateBody
+// — same create wiring as EntityEditCreateBody
 // (kumiko-screen.tsx), but reports the new id back via a callback instead
 // of navigating to the entity's list screen.
 function entityWriteCommand(featureName, entity) {
@@ -17,7 +17,7 @@ function entityWriteCommand(featureName, entity) {
 // The create handler's success payload is `{ kind: "save", id, ... }`
 // (see event-store-executor-write.ts) but RenderEdit's onSubmit only
 // types it as `unknown` — narrow defensively instead of casting through it.
-// `field` is author-declared (ActionFormRedirect.idFrom, fw#2670), never
+// `field` is author-declared (ActionFormRedirect.idFrom), never
 // user input; the own-property check keeps a prototype key like
 // "constructor" from resolving to anything.
 export function extractIdField(data, field) {
@@ -36,7 +36,7 @@ export function ReferenceCreateDialog({ open, onClose, onCreated, featureName, s
     const t = useTranslation();
     const entityDefaultCurrency = entity.defaultCurrency ?? "EUR";
     const tenantCurrencyFieldNames = useMemo(() => (open ? tenantCurrencyMoneyFieldNames(entity.fields) : []), [open, entity.fields]);
-    // Same tenant-currency resolution as the entityEdit create screen (fw#2933),
+    // Same tenant-currency resolution as the entityEdit create screen,
     // so a `currency: { kind: "tenant" }` field seeds identically via this dialog.
     const { overrides: moneyCurrencyOverrides, loading: currencyLoading } = useMoneyCurrencyOverrides(entity.fields, tenantCurrencyFieldNames, entityDefaultCurrency);
     const initial = useMemo(() => buildInitialValues(entity.fields, entityDefaultCurrency, moneyCurrencyOverrides), [entity.fields, entityDefaultCurrency, moneyCurrencyOverrides]);

package/dist/components/related-list-section.d.ts

@@ -13,7 +13,7 @@ export declare function RelatedListSection({ section, parentId, record, featureN
     readonly hideTitle?: boolean;
     /** Under a fixed-height screen the tab panel chain stretches so the count footer sits at the bottom. */
     readonly grow?: boolean;
-    /** Opens a drawer-kind rowAction (fw#2710). Supplied by the parent
+    /** Opens a drawer-kind rowAction. Supplied by the parent
      *  (ProjectionDetailBody), which owns schema + the actual Drawer render —
      *  this component only ever invokes the callback. */
     readonly onOpenDrawer?: (action: RowActionDrawer, initialValues: Readonly<Record<string, unknown>> | undefined) => void;

package/dist/components/related-list-section.js

@@ -338,7 +338,7 @@ export function RelatedListSection({ section, parentId, record, featureName, tra
     // A truncated fetch means `sortedRows` is a sort of a partial set, not of
     // the full related-row set — the client-side sort above (or even plain
     // unsorted display) would silently claim "these are the top N" when they
-    // are really just "the first N in the server's own order" (fw#2722 review).
+    // are really just "the first N in the server's own order".
     // `nextCursor` is exactly how the paged envelope marks that: non-null means
     // more rows exist server-side beyond what was fetched.
     // A handler that omits `nextCursor` (or reports a `total` the fetched rows

package/dist/components/render-edit-logic.js

@@ -1,6 +1,6 @@
 // Hides the Save button when no field is editable and no extension section
 // contributes to the composed form submit. Extensions that persist via their
-// own dispatcher writes must not opt in (fw#2359).
+// own dispatcher writes must not opt in.
 // Explicit-positive per kind (653/1), not `s.kind !== "fields"` — a future
 // third EditSectionViewModel member would otherwise default to "editable"
 // without anyone deciding that on purpose.

package/dist/components/render-edit-types.d.ts

@@ -56,7 +56,7 @@ export type RenderEditProps<TValues extends FormValues, TCtx = unknown> = {
     /** Fires when the form gains or loses unsaved input (field changes or a
      *  dirty extension section). Lets a host such as a drawer guard closing. */
     readonly onDirtyChange?: (dirty: boolean) => void;
-    /** Copy-link action (issue #912) — only set in update mode (create mode
+    /** Copy-link action — only set in update mode (create mode
      *  has no entity id yet, hence no permalink). The callback is already fully
      *  bound (URL building + clipboard happen outside, in
      *  `@cosmicdrift/kumiko-renderer/web`'s RoutedScreen — this platform-neutral
@@ -74,7 +74,7 @@ export type RenderEditProps<TValues extends FormValues, TCtx = unknown> = {
      *  header menu (drawer, card form, shell without a header slot) they fall
      *  back to buttons next to `actions`. */
     readonly menuActions?: readonly RenderEditAction[];
-    /** Opens a relatedList section row's drawer-kind action (fw#2710).
+    /** Opens a relatedList section row's drawer-kind action.
      *  RenderEdit has no `schema` to resolve the target actionForm itself —
      *  the caller (ProjectionDetailBody, which does have schema) supplies the
      *  opener and owns the actual Drawer state/rendering. Only projectionDetail
@@ -117,7 +117,7 @@ export type RenderEditProps<TValues extends FormValues, TCtx = unknown> = {
     /** Edit-mode "changed" markers also for screens without an entity id
      *  (configEdit loads its values from a query, not a record). */
     readonly markChangedFields?: boolean;
-    /** Controlled mode (issue #1887): fires on every values-snapshot change
+    /** Controlled mode: fires on every values-snapshot change
      *  (typing, `patch(...)` from outside) with the current values. `changes`
      *  is the delta against the initial values — same semantics as
      *  `payloadMode: "changes"` — so a caller never overwrites unseen fields.
@@ -134,7 +134,7 @@ export type RenderEditProps<TValues extends FormValues, TCtx = unknown> = {
      *  happens, so a caller patching dependent fields from inside `onChange`
      *  never has to guard against `controls` being undefined. */
     readonly onChange?: (state: RenderEditChangeState<TValues>) => void;
-    /** Controlled mode (issue #1887): called once after mount, hands the
+    /** Controlled mode: called once after mount, hands the
      *  caller `patch`/`validate`/`getValues` bound to this RenderEdit
      *  instance — addressable from outside without a remount. `patch` merges
      *  only the given keys (existing `controller.setValues` semantics),
@@ -153,7 +153,7 @@ export type RenderEditProps<TValues extends FormValues, TCtx = unknown> = {
      *  validation scope (the underlying `useForm` controller is mount-lived);
      *  rendering and `controls.validate()` do stay reactive to later changes. */
     readonly fields?: readonly string[];
-    /** Locked state (issue #1896): every rendered field and the submit button
+    /** Locked state: every rendered field and the submit button
      *  go visibly inactive, no write possible. For cases where input becomes
      *  moot — e.g. Solon's editor pointing at an existing record instead of
      *  creating a new one. Extension sections are out of scope: RenderEdit has
@@ -177,7 +177,7 @@ export type RenderEditProps<TValues extends FormValues, TCtx = unknown> = {
     /** "form" (default) — every field renders as its Input widget, disabled
      *  when `field.readOnly`, unchanged behavior. "text" renders a
      *  `field.readOnly` field as plain text instead of a disabled Input
-     *  (ProjectionDetailBody's read view, fw#2245) — editable fields are
+     *  (ProjectionDetailBody's read view) — editable fields are
      *  unaffected either way, so this only changes forms that already have
      *  readOnly fields. */
     readonly valueDisplay?: "form" | "text";