@eifi1/ui-kit 0.52.0 → 0.54.0

package/src/components/treemap.tsx

@@ -129,13 +129,26 @@ export interface TreemapProps extends Omit<ComponentPropsWithoutRef<"div">, "chi
    * A tile's and a group's accessible name (`labels.tile`, `labels.group`, 0.52) says the
    * value too, asked of the formatter as the tooltip asks it, share and all, and its text
    * is used; markup is said as the bare number. The name says the share itself, so a
-   * formatter that writes it into its text has it said twice.
+   * formatter that writes it into its text has it said twice — say so with
+   * {@link TreemapProps.shareInValue}.
    *
    * Default: the bare number through `Intl.NumberFormat` in the provider's locale (0.52,
    * DOC-10: before, the runtime's). A host with a currency passes this — the kit knows
    * none.
    */
   valueFormatter?: (value: number, share: number | undefined) => ReactNode;
+  /**
+   * The `valueFormatter`'s text already says the share ("CHF 412.30 · 12%"), so a tile's
+   * and a group's accessible name leave their own " (12%)" out (0.53, keksdose after 0.52):
+   * the map hands `labels.tile` and `labels.group` the share as `""`, and every catalogue's
+   * template drops it there, in its own typography. Before, the only way out was to replace
+   * both templates, and with them the kit's seven catalogues. The tooltip is the same either
+   * way: it prints what the formatter prints. A formatter whose value is markup is said as
+   * the bare number, so its share is then not said at all.
+   *
+   * Default `false`: the name says the share.
+   */
+  shareInValue?: boolean;
   /**
    * What the map SHOWS, in a sentence — "Rent is three fifths of the month." — the
    * figure's accessible description (0.52, DSP-5). Not drawn; a caption the eye should
@@ -185,8 +198,18 @@ export interface TreemapProps extends Omit<ComponentPropsWithoutRef<"div">, "chi
    * Every tile is tagged as a whole too, clickable or not (0.52): its accessible name
    * (`labels.tile`) says the name and the value, in an attribute a host's redaction
    * reaches only through the tag.
+   *
+   * - `true`: every node — the tiles, and a group's frame (its `<g>` and its band's name)
+   *   and tooltip bubble too.
+   * - `"tiles"` (0.53, keksdose after 0.52): the tiles as `true` tags them, and no group —
+   *   for groups that are public names over private ones, as chains (Migros, Coop) over
+   *   the shops a user paid. Its frame's band, its frame and its bubble's name are
+   *   untagged, and so is the frame's accessible name (`labels.group`), which says the
+   *   group's TOTAL: use it only where the groups' names and totals are not the private
+   *   part. The bubble's value cell is tagged whatever this says, as in every chart.
+   *   A tile's name still says its group's (`labels.inGroup`), under the tile's tag.
    */
-  redactNames?: boolean;
+  redactNames?: boolean | "tiles";
   /**
    * A second, smaller line under a tile's label — one extra figure per node that the
    * name cannot carry (a year-on-year `+12%`, `new`). Return `undefined` for a node
@@ -436,7 +459,9 @@ export interface TreemapCellProps {
   /** Filled in by recharts for a GROUP: the nodes laid out inside it. A cell with
    *  children draws a frame, not a tile (0.52). */
   children?: readonly object[] | null;
-  redactNames?: boolean;
+  /** See {@link TreemapProps.redactNames}: `"tiles"` tags a cell with no `children`, and
+   *  leaves a group's frame and band untagged (0.53). */
+  redactNames?: boolean | "tiles";
   nodeNote?: (id: string) => string | undefined;
   /** The page's reading direction, which `Treemap` reads off its own root. In `rtl`
    *  the label stands in the tile's top RIGHT corner and is shaped right-to-left, so a
@@ -578,6 +603,10 @@ function Tile({
   // is a button.
   const roving = path != null && keys ? keys : undefined;
   const focusable = roving !== undefined || activate !== undefined;
+  // `"tiles"` (0.53, keksdose after 0.52) leaves a group's frame and band untagged: its
+  // groups are public chain names, and its blur on `[data-private]` blurred them with the
+  // payees. A truthy test on the union would have tagged them still.
+  const redact = redactNames === true || (redactNames === "tiles" && !group);
   const onKeyDown = focusable
     ? (e: KeyboardEvent<SVGGElement>) => {
         if (roving?.move(path!, e)) {
@@ -606,7 +635,7 @@ function Tile({
       // In a map, -1 as drawn: the map moves its one stop on the elements in place.
       tabIndex={roving ? -1 : activate ? 0 : undefined}
       aria-label={focusable ? (roving?.name(path!) ?? name) : undefined}
-      data-private={focusable && redactNames ? "" : undefined}
+      data-private={focusable && redact ? "" : undefined}
       data-group={group ? "" : undefined}
       cursor={activate ? "pointer" : undefined}
       // CSS beats a presentation attribute, so this recolours the rect's own stroke
@@ -646,7 +675,7 @@ function Tile({
         <text
           // A plain attribute, so the host's `[data-private]` rule reaches an SVG
           // <text> exactly as it reaches a table cell.
-          data-private={redactNames ? "" : undefined}
+          data-private={redact ? "" : undefined}
           x={textX}
           // A band's name centred in its 20px; a tile's where its floor leaves room below.
           y={py + (group ? 14 : 15) * scale}
@@ -697,10 +726,20 @@ const TREEMAP_CONFIG: ChartConfig = {};
  * recharts CLONES the `content` element to inject `active`/`payload`, so this has to BE
  * that element — a `<div>` wrapped around `<ChartTooltipContent />` in the JSX would
  * take those props itself and hand the tooltip nothing.
+ *
+ * `groups: false` is `redactNames="tiles"` (0.53): a group's bubble is left untagged, as its
+ * frame is. The payload says which node is hovered — recharts hands the node itself, with
+ * its laid-out `children` for a group and `null` for a tile, as the cell is told — so
+ * anything else, or no payload, is tagged.
  */
-function RedactedTooltipContent(props: ComponentProps<typeof ChartTooltipContent>) {
+function RedactedTooltipContent({
+  groups = true,
+  ...props
+}: ComponentProps<typeof ChartTooltipContent> & { groups?: boolean }) {
+  const children = props.payload?.[0]?.payload?.children;
+  const group = Array.isArray(children) && children.length > 0;
   return (
-    <div data-private>
+    <div data-private={groups || !group ? true : undefined}>
       <ChartTooltipContent {...props} />
     </div>
   );
@@ -895,6 +934,7 @@ export function Treemap({
   onNodeClick,
   onGroupClick,
   redactNames = false,
+  shareInValue = false,
   nodeNote,
   maxTiles,
   colors,
@@ -1066,10 +1106,13 @@ export function Treemap({
     };
     // A group's name by its path; a node's groups by theirs, its own path's prefixes.
     const groupName = new Map(order.filter((n) => n.children).map((n) => [n.path, n.name]));
+    // `""` when the formatter's text says the share already, which every template drops
+    // (0.53, `shareInValue`).
+    const shareOf = (v: number) => (shareInValue ? "" : shareText(total > 0 ? v / total : 0));
     return new Map(
       order.map((n) => {
         const say = n.children ? labels.group : labels.tile;
-        const own = say(n.name, valueText(n.size), shareText(total > 0 ? n.size / total : 0));
+        const own = say(n.name, valueText(n.size), shareOf(n.size));
         // The innermost group first: "Rent: …, in Housing, in Fixed costs".
         const named = n.groups
           .map((g, i) => groupName.get(pathOf(n.groups.slice(0, i), g)) ?? g)
@@ -1077,7 +1120,7 @@ export function Treemap({
         return [n.path, named] as const;
       }),
     );
-  }, [order, labels, format, numberFormat, shareText, total]);
+  }, [order, labels, format, numberFormat, shareText, shareInValue, total]);
   const keys = useMemo<TileKeys>(
     () => ({ name: (path) => names.get(path), onFocus, register, move }),
     [names, onFocus, register, move],
@@ -1086,7 +1129,6 @@ export function Treemap({
     () => ({ tokens: tokenHex, edgeFor, edges: edgesInForce, keys, layout, hover: anchor.set }),
     [tokenHex, edgeFor, edgesInForce, keys, layout, anchor],
   );
-  const Content = redactNames ? RedactedTooltipContent : ChartTooltipContent;
   const cell = useMemo(
     () => (
       <TreemapCell
@@ -1100,8 +1142,19 @@ export function Treemap({
     [onNodeClick, onGroupClick, redactNames, nodeNote, dir],
   );
   const tooltip = useMemo(
-    () => <TreemapTooltip content={<Content hideLabel valueFormatter={format} />} anchor={anchor} />,
-    [Content, format, anchor],
+    () => (
+      <TreemapTooltip
+        content={
+          redactNames ? (
+            <RedactedTooltipContent hideLabel valueFormatter={format} groups={redactNames === true} />
+          ) : (
+            <ChartTooltipContent hideLabel valueFormatter={format} />
+          )
+        }
+        anchor={anchor}
+      />
+    ),
+    [redactNames, format, anchor],
   );
 
   if (nodes.length === 0) return null;

package/src/currencies.ts

@@ -12,6 +12,11 @@
 // CurrencySelect, AmountInput's currency chip and NumberInput's code detection read this
 // same array (components/currency-option-list re-exports it, and currency-select as it
 // always has), so the list the pickers show and the flags an app ships cannot drift.
+//
+// A currency that was REPLACED stays in the array, marked `retired`: the pickers stop
+// offering it, and whatever reads a value an app already holds — the trigger's flag and
+// code, `getCurrency`, the flag subset, NumberInput's code detection — keeps knowing it
+// (keksdose audit follow-up, 2026-11-01: the lev, after Bulgaria's euro).
 
 export interface CurrencyOption {
   code: string;
@@ -19,6 +24,19 @@ export interface CurrencyOption {
   symbol: string;
   /** ISO 3166-1 alpha-2 country code (lowercase) used by flag-icons. */
   country: string;
+  /**
+   * The day ("YYYY-MM-DD") the currency was replaced as its country's money — set only on
+   * one that was. The pickers leave it out of their default list. A field that already
+   * holds it still shows it (the trigger's flag and code, and its row in the list,
+   * checked), and an explicit `options` list that names it still offers it: that list is
+   * the caller's.
+   *
+   * Marked rather than deleted, because an app may hold amounts in it from before. Gone
+   * from the array, `getCurrency` would answer `undefined` for a stored value, the field
+   * would show its placeholder over a real amount, and an app's flag subset
+   * (`CURRENCY_COUNTRIES`) would ship that value's flag as an empty box.
+   */
+  retired?: string;
 }
 
 export const CURRENCIES: CurrencyOption[] = [
@@ -37,7 +55,8 @@ export const CURRENCIES: CurrencyOption[] = [
   { code: "CZK", name: "Czech Koruna", symbol: "Kč", country: "cz" },
   { code: "HUF", name: "Hungarian Forint", symbol: "Ft", country: "hu" },
   { code: "RON", name: "Romanian Leu", symbol: "lei", country: "ro" },
-  { code: "BGN", name: "Bulgarian Lev", symbol: "лв", country: "bg" },
+  // Bulgaria adopted the euro on 2026-01-01 (keksdose audit follow-up, 2026-11-01).
+  { code: "BGN", name: "Bulgarian Lev", symbol: "лв", country: "bg", retired: "2026-01-01" },
   { code: "TRY", name: "Turkish Lira", symbol: "₺", country: "tr" },
   { code: "RUB", name: "Russian Ruble", symbol: "₽", country: "ru" },
   { code: "CNY", name: "Chinese Yuan", symbol: "¥", country: "cn" },
@@ -56,8 +75,9 @@ export const CURRENCIES: CurrencyOption[] = [
  * code flag-icons names it by (`fi-ch`, `flags/4x3/ch.svg`). `CHF` → `"ch"`, `EUR` →
  * `"eu"` (flag-icons has the EU's flag under that code).
  *
- * Derived from {@link CURRENCIES}, so a currency the pickers gain is a flag here too. An
- * app that ships only the flags it renders builds its subset from these values:
+ * Derived from {@link CURRENCIES}, so a currency the pickers gain is a flag here too — and
+ * a `retired` one keeps its flag, for the values an app still holds in it. An app that
+ * ships only the flags it renders builds its subset from these values:
  *
  * ```ts
  * // vite.config.ts — plain Node, no DOM

package/src/data-table.ts

@@ -27,22 +27,12 @@ export type {
   DataTableRowAction,
   DataTableDesktopSwipeActions,
 } from "./components/data-table";
-// 0.46.1 (keksdose N3): a swipe plan as `RowActions`' actions — the desktop column's own
-// derivation, for an app's hand-built actions column.
-export {
-  /** @deprecated 0.52 — `DataTable`'s `desktopSwipeActions`, which builds its column with it (API-4); removed in 0.53 */
-  swipePlanRowActions,
-} from "./components/data-table-row-actions";
 // 0.48 (keksdose dev #605): a `groupBy` level (also `mobileGroupBy`'s) and the `loadMore` footer.
 export type { DataTableGroupBy, DataTableLoadMore } from "./components/data-table-groups";
 // 0.48.2 (keksdose dev #632): `virtualize`'s options.
 export type { DataTableVirtualize } from "./components/data-table-virtual";
 export * from "./components/data-table-cells";
-export {
-  useTableUrlState,
-  /** @deprecated 0.52 — `useTableUrlState`, which reads the URL with it (API-4); removed in 0.53 */
-  readTableUrlState,
-} from "./components/use-table-state";
+export { useTableUrlState } from "./components/use-table-state";
 export type {
   TableUrlState,
   TableUrlSync,

package/src/demo/demo-countdown.ts

@@ -50,14 +50,9 @@ function minutesAt(expiry: number, now: number): number | null {
   return Math.max(0, Math.ceil((expiry - now) / MINUTE));
 }
 
-/**
- * The countdown at `now` — pure, for a test or a server-rendered value. See
- * {@link DemoCountdown} for the rounding and the thresholds.
- */
-export function demoCountdown(expiresAt: DemoExpiry | null | undefined, now: number): DemoCountdown {
-  return countdownOf(minutesAt(expiryMs(expiresAt), now));
-}
-
+/** The countdown for `minutesLeft` — see {@link DemoCountdown} for the rounding and the
+ *  thresholds. (Public as `demoCountdown(expiresAt, now)` until 0.53, deprecated in 0.52,
+ *  API-4: `useDemoCountdown` with its `now` option is the same countdown.) */
 function countdownOf(minutesLeft: number | null): DemoCountdown {
   if (minutesLeft === null) {
     return { minutesLeft, hours: 0, minutes: 0, lastHour: false, warning: false, ended: false };

package/src/feedback/feedback-context-labels.ts

@@ -7,4 +7,5 @@ import type { FeedbackContextLabels } from "./feedback-context";
 export const DEFAULT_FEEDBACK_CONTEXT_LABELS: FeedbackContextLabels = {
   user: "User",
   attachUrl: "Attach current page URL",
+  diagnostics: "Diagnostics sent with the report",
 };

package/src/feedback/feedback-context.tsx

@@ -1,8 +1,10 @@
+import { useId } from "react";
 import { Checkbox } from "../components/checkbox";
 import { DEFAULT_COMMON_LABELS, useKitLabels, type LabelOverride } from "../i18n/kit-labels";
 import { cn } from "../lib/cn";
 import type { FeedbackContext, FeedbackEnvironment } from "./feedback-record";
 import { DEFAULT_FEEDBACK_CONTEXT_LABELS } from "./feedback-context-labels";
+import { diagnosticsEntries, FeedbackDiagnosticsList } from "./feedback-diagnostics";
 
 /**
  * The reporter context of a feedback report: the box the submit dialog shows, and the
@@ -70,6 +72,10 @@ export interface FeedbackContextInput {
   viewport?: string;
   /** Default `navigator.userAgent`. */
   ua?: string;
+  /** What the app attached about the thing reported — `useFeedbackSubmit().open(category,
+   *  { diagnostics })` (keksdose live #481). Sent whatever `attachUrl` says, as the box
+   *  shows it; left out of the object when it has no key to show. */
+  diagnostics?: Readonly<Record<string, unknown>> | null;
 }
 
 /** The pathname of a full or path-only URL; `""` for something that is neither. */
@@ -85,12 +91,13 @@ function pathOf(url: string, base: string | undefined): string {
 /**
  * The `context` object of `POST /feedback` (§3.2), keys in keksdose's order: `url`, `route`,
  * `origin`, `environment`, `user_id`, `user_email`, `user_display_name`, `viewport`, `ua`,
- * `version`.
+ * `version`, then the app's `diagnostics`.
  *
  * With `attachUrl: false`, `url` and `route` are `""` — present and empty, as keksdose
  * sends them, so a reader can tell "withheld" from "an old row without the key".
  * `environment` and `version` are omitted when the app passes none (kastlan had no
- * version), rather than sent as `undefined`.
+ * version), rather than sent as `undefined`; `diagnostics` when the box would show no
+ * line of it (none, `{}`, not a record).
  *
  * Reads `window` / `navigator` only for the defaults it was not given, so it is a pure
  * function in a test that passes everything.
@@ -111,6 +118,8 @@ export function feedbackContext(input: FeedbackContextInput): FeedbackContext {
   context.viewport = input.viewport ?? (win ? `${win.innerWidth}x${win.innerHeight}` : "");
   context.ua = input.ua ?? (typeof navigator === "undefined" ? "" : navigator.userAgent);
   if (input.version !== undefined) context.version = input.version;
+  // The box's own test: what it draws no line of is not sent either.
+  if (input.diagnostics && diagnosticsEntries(input.diagnostics).length) context.diagnostics = input.diagnostics;
   return context;
 }
 
@@ -127,6 +136,9 @@ export interface FeedbackContextLabels {
   user: string;
   /** The checkbox. The URL it is about is the line beneath it. */
   attachUrl: string;
+  /** The heading over what the app attached — `open(category, { diagnostics })`, shown
+   *  read-only under the URL (keksdose live #481). de-CH: "Mitgesendete Diagnose" */
+  diagnostics: string;
 }
 
 // Defined in ./feedback-context-labels, which holds no component code (0.34, KS U8).
@@ -140,6 +152,10 @@ export interface FeedbackContextBoxProps {
   /** The checkbox, on by default in the dialog (§4.2) — the same `attachUrl`. */
   attachUrl: boolean;
   onAttachUrlChange: (next: boolean) => void;
+  /** What the app attached — the same `diagnostics` handed to {@link feedbackContext}.
+   *  Shown read-only, each key and its value as the row detail draws them; no block when
+   *  there is no line to show. */
+  diagnostics?: Readonly<Record<string, unknown>> | null;
   /** Prop > `<UiKitProvider labels={{ feedbackContext }}>` > English. */
   labels?: LabelOverride<FeedbackContextLabels>;
   className?: string;
@@ -157,6 +173,10 @@ export interface FeedbackContextBoxProps {
  * against whatever screen happened to be open, the queue reads as if it were about that
  * screen.
  *
+ * Below the URL, what the app attached (`diagnostics`, keksdose live #481: a stuck receipt
+ * scan's bookkeeping), read-only and in the row detail's own lines: the reporter sees what
+ * goes with the report, and that the URL checkbox does not withhold it.
+ *
  * Nothing in here is the app's: hand it the values handed to {@link feedbackContext}, and
  * what the box shows is what is sent.
  */
@@ -165,11 +185,14 @@ export function FeedbackContextBox({
   url,
   attachUrl,
   onAttachUrlChange,
+  diagnostics,
   labels: labelsProp,
   className,
 }: FeedbackContextBoxProps) {
   const labels = useKitLabels("feedbackContext", DEFAULT_FEEDBACK_CONTEXT_LABELS, labelsProp);
   const common = useKitLabels("common", DEFAULT_COMMON_LABELS);
+  const diagnosticsId = useId();
+  const attached = diagnosticsEntries(diagnostics);
   const submitter = submitterLine(user);
   // "User: Ada …" through `common.fieldValue`, so French gets its space before the colon
   // and Chinese its full-width one; the label and its punctuation are drawn bold, as
@@ -208,6 +231,15 @@ export function FeedbackContextBox({
           ) : undefined
         }
       />
+      {attached.length > 0 && (
+        // A named group, so a screen reader hears what the pairs are before it reads them.
+        <div role="group" aria-labelledby={diagnosticsId} className="space-y-0.5 pt-1">
+          <div id={diagnosticsId} className="font-medium text-secondary">
+            {labels.diagnostics}
+          </div>
+          <FeedbackDiagnosticsList entries={attached} />
+        </div>
+      )}
     </div>
   );
 }

package/src/feedback/feedback-diagnostics.tsx

@@ -0,0 +1,41 @@
+/**
+ * `context.diagnostics` (§3.2), drawn one way wherever it shows: in the submit dialog's
+ * context box, to the reporter before it goes (§4.2), and in the row detail, to the admin
+ * and the author afterwards (§4.4.2). One module for both, so the reporter is shown the
+ * very lines the admin will read (keksdose live #481).
+ *
+ * Internal: neither `@eifi1/ui-kit/feedback` nor the barrel names it.
+ */
+
+/** `diagnostics` as [key, text] pairs, in the app's order: a string, number or boolean as
+ *  written, an object or a list as JSON, nothing as "—". Empty for no diagnostics, and for
+ *  a value that is not a plain object — the server stores whatever the client sent. */
+export function diagnosticsEntries(diagnostics: unknown): Array<[string, string]> {
+  if (!diagnostics || typeof diagnostics !== "object" || Array.isArray(diagnostics)) return [];
+  return Object.entries(diagnostics).map(([key, v]) => [
+    key,
+    v === null || v === undefined || v === ""
+      ? "—"
+      : typeof v === "object"
+        ? JSON.stringify(v)
+        : String(v as string | number | boolean | bigint),
+  ]);
+}
+
+/** The pairs as a definition list: each key beside its value, in monospace. Read-only in
+ *  both places; the heading over it is the caller's. */
+export function FeedbackDiagnosticsList({ entries }: { entries: ReadonlyArray<readonly [string, string]> }) {
+  return (
+    <dl
+      className="grid grid-cols-[auto_minmax(0,1fr)] gap-x-3 gap-y-0.5 font-mono text-xs"
+      data-feedback-diagnostics=""
+    >
+      {entries.map(([key, text]) => (
+        <div key={key} className="contents">
+          <dt className="text-muted">{key}</dt>
+          <dd className="min-w-0 break-words text-secondary">{text}</dd>
+        </div>
+      ))}
+    </dl>
+  );
+}

package/src/feedback/feedback-dialog.tsx

@@ -146,6 +146,16 @@ interface FeedbackDialogBaseProps<C extends string = string> {
    *  English. Optional since 0.12.0; a whole `FeedbackDialogLabels` still fits. */
   labels?: LabelOverride<FeedbackDialogLabels>;
   submitting?: boolean;
+  /**
+   * The subject and the body each opening starts with, in place of empty fields — the
+   * reporter edits them like anything typed. `useFeedbackSubmit().open(category, { title,
+   * body })` passes them (keksdose live #481: "Report this problem" opens on words the
+   * screen already knows). Read when the dialog opens; a change while it is open leaves
+   * the draft alone.
+   */
+  defaultTitle?: string;
+  /** The body's, as `defaultTitle` is the subject's. */
+  defaultBody?: string;
   contextSlot?: ReactNode;
   attachmentAccept?: string[];
   maxAttachmentBytes?: number;
@@ -245,6 +255,8 @@ export function FeedbackDialog<C extends string = string>(props: FeedbackDialogP
     onCategoryChange,
     labels: labelsProp,
     submitting = false,
+    defaultTitle = "",
+    defaultBody = "",
     contextSlot,
     attachmentAccept = DEFAULT_ATTACHMENT_ACCEPT,
     maxAttachmentBytes = DEFAULT_MAX_ATTACHMENT_BYTES,
@@ -254,8 +266,8 @@ export function FeedbackDialog<C extends string = string>(props: FeedbackDialogP
     attachmentButtonSize,
     publicNotice,
   } = props;
-  const [title, setTitle] = useState("");
-  const [body, setBody] = useState("");
+  const [title, setTitle] = useState(defaultTitle);
+  const [body, setBody] = useState(defaultBody);
   const [attachment, setAttachment] = useState<File | null>(null);
   // `attachments="multiple"` only: the capture, apart from the picked/pasted files.
   const [screenshot, setScreenshot] = useState<File | null>(null);
@@ -283,8 +295,8 @@ export function FeedbackDialog<C extends string = string>(props: FeedbackDialogP
   if (open !== wasOpen) {
     setWasOpen(open);
     if (open) {
-      setTitle("");
-      setBody("");
+      setTitle(defaultTitle);
+      setBody(defaultBody);
       setAttachment(null);
       setScreenshot(null);
       setFiles([]);

package/src/feedback/feedback-mutation.ts

@@ -1,41 +1,20 @@
-import { isThenable } from "../lib/thenable";
-
 /**
- * A page's mutation in either of the forms the feedback parts take, run once (refactor
- * audit 2026-10-30, FLW-3).
+ * A page's mutation, run once (refactor audit 2026-10-30, FLW-3): TanStack Query's
+ * `mutateAsync`, or any `(patch) => Promise` that resolves once the PATCH has landed.
  *
- * The parts used to take TanStack Query's `mutate` with a per-call `onSuccess`. TanStack
- * v5 calls per-call callbacks for the LATEST `mutate()` on an observer only — it
- * overwrites the options and detaches from the earlier mutation — so a second change sent
- * before the first PATCH answered (another swipe, another pill, the Undo itself) lost the
- * first change's Undo toast, and the detail's editor stayed open. The promise
- * `mutateAsync` returns settles for every call, whatever the observer does next.
+ * The parts took TanStack Query's `mutate` with a per-call `onSuccess` until 0.53
+ * (deprecated in 0.52). TanStack v5 calls per-call callbacks for the LATEST `mutate()` on
+ * an observer only — it overwrites the options and detaches from the earlier mutation —
+ * so a second change sent before the first PATCH answered (another swipe, another pill,
+ * the Undo itself) lost the first change's Undo toast, and the detail's editor stayed
+ * open. The promise `mutateAsync` returns settles for every call, whatever the observer
+ * does next.
  *
- * Both forms are called the same way — the patch and `{ onSuccess }` — and the answer is
- * taken from whichever comes first: the callback, or the promise the call returned. A
- * rejected promise is the mutation's own to report (its `onError`), so it is caught here
+ * A rejected promise is the mutation's own to report (its `onError`), so it is caught here
  * and nothing more happens.
  *
  * @internal Not part of `/feedback` or the barrel.
  */
-
-/**
- * A mutation in either form: the promise form (0.52) — TanStack's `mutateAsync`, any
- * `(patch) => Promise` — or the callback form before it — TanStack's `mutate`, a function
- * that calls `onSuccess` once the PATCH has landed, and only then. One signature for
- * both, so an inline function handed to a prop of this type gets its parameter types (a
- * union of the two leaves `async (patch) => …` an implicit `any`).
- */
-export type FeedbackMutationCall<P> = (patch: P, callbacks: { onSuccess: () => void }) => void | Promise<unknown>;
-
-/** Send `patch` through `call`, and run `landed` once when it has landed. */
-export function runMutation<P>(call: FeedbackMutationCall<P>, patch: P, landed: () => void): void {
-  let done = false;
-  const once = () => {
-    if (done) return;
-    done = true;
-    landed();
-  };
-  const answer: unknown = call(patch, { onSuccess: once });
-  if (isThenable(answer)) answer.then(once, () => undefined);
+export function runMutation<P>(call: (patch: P) => Promise<unknown>, patch: P, landed: () => void): void {
+  call(patch).then(landed, () => undefined);
 }

package/src/feedback/feedback-public-list.tsx

@@ -174,11 +174,6 @@ export interface FeedbackPublicListProps<T extends FeedbackPublicListItem = Feed
    *  what it carries: every other `onOpenChange` in the kit is a boolean (refactor audit
    *  2026-10-30, API-5). */
   onOpenIdChange?: (id: number | null) => void;
-  /**
-   * @deprecated 0.52 — `onOpenIdChange`, the same call: an `onOpenChange` is a boolean
-   * everywhere else in the kit; removed in 0.53
-   */
-  onOpenChange?: (id: number | null) => void;
   /**
    * 0.38.1: the opened report when it is NOT on this page — a `?row=` link to an older
    * one (Kurvenschmiede). The app fetches it (`GET /feedback/public/{id}`, a list item's
@@ -249,7 +244,6 @@ export function FeedbackPublicList<T extends FeedbackPublicListItem = FeedbackPu
   loading = false,
   openId: openIdProp,
   onOpenIdChange,
-  onOpenChange,
   openRow,
   page: pageIndex = 0,
   hasMore = false,
@@ -269,7 +263,7 @@ export function FeedbackPublicList<T extends FeedbackPublicListItem = FeedbackPu
   const openId = openIdProp !== undefined ? openIdProp : ownOpenId;
   const setOpen = (id: number | null) => {
     if (openIdProp === undefined) setOwnOpenId(id);
-    (onOpenIdChange ?? onOpenChange)?.(id);
+    onOpenIdChange?.(id);
   };
   // A `?row=` report from another page: first, while it is the open one, and only when
   // the page does not have it itself.

package/src/feedback/feedback-record.ts

@@ -59,6 +59,11 @@ export interface FeedbackContext {
   ua?: string;
   /** The app build (`__APP_VERSION__`). */
   version?: string;
+  /** What the app attached about the thing reported (keksdose's stuck receipt scan, live
+   *  #481): a flat record of short values, in the order the app wrote them. The row
+   *  detail shows it to the admin and the author, never to anyone else; erasure drops it
+   *  (it is not on server-kit's `FEEDBACK_CONTEXT_KEEP`). */
+  diagnostics?: Readonly<Record<string, unknown>>;
   // Crash rows (server-written, keksdose `feedback_service.py:346`).
   fingerprint?: string;
   /** Which error boundary caught it. */

package/src/feedback/feedback-row-detail-labels.ts

@@ -14,20 +14,13 @@ export const DEFAULT_FEEDBACK_DETAIL_LABELS: FeedbackDetailLabels = {
   cancel: "Cancel",
   url: "URL",
   copyUrl: "Copy URL",
+  diagnostics: "Diagnostics",
   attachment: "Attachment",
   download: (name) => `Download ${name}`,
   downloadFailed: "The attachment could not be downloaded.",
   outcome: "Outcome",
   replies: "Replies",
   resolvedAt: (date) => `Resolved: ${date}`,
-  // The outcome editor's three words, read by nothing since 0.38 and "kept for one
-  // release" for 13 minors: no tag, so the deprecations test never listed them (refactor
-  // audit 2026-10-30, DOC-9). Deprecated on `FeedbackDetailLabels`, the type an app reads
-  // them through — a tag on these keys reached no reader (0.52 code review). The keys
-  // leave the type and all seven catalogues in 0.53.
-  outcomeAdd: "Add outcome",
-  outcomeUpdate: "Update",
-  outcomePlaceholder: "What was done, decided, or why this won't be addressed.",
   rework: "Rework",
   reworkTitle: "Send for rework",
   reworkSend: "Send rework",