@eifi1/ui-kit 0.54.0 → 0.55.0

package/src/feedback/feedback-row-detail.tsx

@@ -1,871 +0,0 @@
-import { useEffect, useRef, useState } from "react";
-import type { ReactNode } from "react";
-import { RotateCcw } from "lucide-react";
-import { AuthedImage } from "../components/authed-image";
-import { CopyButton } from "../components/copy-button";
-import { TextLink } from "../components/text-link";
-import { toast } from "../components/toast";
-import { Button } from "../components/button";
-import type { AuthedFetcher } from "../hooks/use-authed-src";
-import type { PathPattern } from "../landing/routing";
-import { useKitLabels, type LabelOverride } from "../i18n/kit-labels";
-import { useDayFormatter } from "../lib/day-format";
-import { DEFAULT_MAX_ATTACHMENT_BYTES } from "./feedback-attachment";
-import { DEFAULT_FEEDBACK_DIALOG_LABELS } from "./feedback-dialog";
-import {
-  FeedbackCategoryBadge,
-  FeedbackDetail,
-  FeedbackDetailSection,
-  FeedbackNoteEditor,
-  FeedbackProse,
-  FeedbackStatusTransitions,
-  selectableFeedbackStatuses,
-  visibleFeedbackStatuses,
-} from "./feedback-inbox";
-import type { FeedbackNoteAttachment, FeedbackStatus } from "./feedback-inbox";
-import { useFeedbackToastLabels } from "./feedback-labels";
-import {
-  FEEDBACK_ATTACHMENT_ACCEPT,
-  FEEDBACK_AUTHOR_EDITABLE_STATUSES,
-  FEEDBACK_REWORKABLE_STATUSES,
-  appendRework,
-  isImageAttachment,
-  replaceDescription,
-  splitBodyAttachments,
-  splitDescription,
-} from "./feedback-record";
-import type { FeedbackContext, FeedbackRecord } from "./feedback-record";
-import { diagnosticsEntries, FeedbackDiagnosticsList } from "./feedback-diagnostics";
-import { runMutation } from "./feedback-mutation";
-import { feedbackPage, feedbackPageLinkTarget } from "./feedback-page-link";
-import { useFeedbackStatusUndo } from "./feedback-status-undo";
-import type { FeedbackStatusChange } from "./feedback-status-undo";
-import type { CommitScope } from "../components/write-lock";
-import { DEFAULT_FEEDBACK_DETAIL_LABELS } from "./feedback-row-detail-labels";
-import { FeedbackReplyComposer, FeedbackReplyTimeline, feedbackReplyNudge } from "./feedback-replies";
-import { FeedbackAttachmentDownload, plainFeedbackFetch } from "./feedback-attachment-download";
-import { FeedbackPublicSwitch } from "./feedback-public-switch";
-import { FeedbackRepliesDrawnAboveContext } from "./feedback-replies-context";
-import { DEFAULT_FEEDBACK_COMMENTS_LABELS } from "./feedback-thread-parts-labels";
-
-/**
- * One report, opened — the panel a feedback table unfolds under its row (§4.4 of
- * docs/feedback-harmonization.md), lifted from keksdose's `FeedbackRow`
- * (`frontend/src/features/feedback/feedback-page.tsx:329-612`) with the contract's
- * changes applied.
- *
- * Until 0.27 each app wrote this panel itself out of the kit's shell
- * (`FeedbackDetail` / `FeedbackDetailSection` / `FeedbackNoteEditor`), and the three
- * copies had drifted in exactly the places a user notices:
- *
- * - **the description editor held the raw body** — keksdose `initial={fb.body}`
- *   (`:420`), kastlan the same (`pages/feedback-page.tsx:226`) — so an author fixing a
- *   typo could delete the `--- REWORK … ---` rounds and the `[screenshot]` file lines
- *   with it, and the history was gone. Here the editor holds the ORIGINAL description
- *   only ({@link splitDescription}) and saves through {@link replaceDescription}, which
- *   puts every appended block and file line back unchanged;
- * - **kastlan's rework sent `status: "OPEN"`** from the client (`:189-195`) and offered it
- *   on three statuses; keksdose sends the body alone and lets the server reopen the row
- *   on five (live #331). The contract is keksdose's (§3.4) — {@link appendRework}, one
- *   file, the server reopens;
- * - **kastlan showed only `screenshot_url`** (`:265-276`) and no downloads; keksdose the
- *   screenshot, the other files and the rework pictures, a PDF as a download (dev #578);
- * - **the URL fell away on Kurvenschmiede's rows**, whose backfilled `context.url` is a
- *   path (`/accounts?p=2`): keksdose's `new URL(…)` throws on one and the section
- *   vanished. kastlan already fell back to `context.route` (`feedbackRaw`, `:97`).
- *
- * The sections, in the contract's order: **Subject** (the category and the title, §8.1;
- * `showSubject={false}` leaves it out), **Description** (the body; the author's
- * Edit while OPEN / READY / IN_PROGRESS), **URL**, **Attachment**, **Replies** (0.38,
- * §9.7: the timeline, the admin's reply composer with its status picker and the nudge,
- * Rework — on a row without `replies`, a server before 0.38, the frozen
- * **Outcome** read-only in its place), **Comments** (0.38: the app's
- * `FeedbackCommentThread`, when passed), **Send for rework** (only while reworking) and
- * **Status** (with the **Visible to others** switch when `onPublicChange` is passed).
- *
- * The app keeps its data and its API: the row it renders, the PATCH (`onUpdate`), the
- * upload (`onUpload`), the authenticated GET (`fetcher`) and who is looking (`canEdit`,
- * `viewerId`). Every word is the kit's `feedbackDetail` namespace.
- */
-
-/* ── Labels ───────────────────────────────────────────────────────────────── */
-
-/**
- * `feedbackDetail` — the row detail's words (§4.4). English: keksdose's `en.json`
- * (`feedback.*`, `common.*`, `more.copy_url`) with the contract's changes; the de-CH
- * canon (keksdose `de-CH.json`, ss never ß, "Nacharbeit" throughout — §7.14) on each key.
- */
-export interface FeedbackDetailLabels {
-  /** The first section's heading: what the report is called (feedback contract §8.1,
-   *  keksdose live #395 — with the row expanded and the table scrolled, nothing said
-   *  which entry was being answered). de-CH: "Betreff" */
-  subject: string;
-  /** The description section's heading — keksdose's `feedback.body` (§7.3). The dialog's field asks
-   *  "What happened? (optional)"; the description, once filed, is headed as what it is
-   *  — and as the author's "Edit description" calls it. de-CH: "Beschreibung" */
-  body: string;
-  /** The author's button on that heading. de-CH: "Bearbeiten" */
-  edit: string;
-  /** Its accessible name — says WHAT is edited; contains the visible word.
-   *  de-CH: "Beschreibung bearbeiten" */
-  editDescription: string;
-  /** The editors' save button (description, outcome). de-CH: "Speichern" */
-  save: string;
-  /** Every editor's cancel button. de-CH: "Abbrechen" */
-  cancel: string;
-  /** Section 2's heading. de-CH: "URL" */
-  url: string;
-  /** The copy button beside the link — copies the FULL `context.url`.
-   *  de-CH: "URL kopieren" */
-  copyUrl: string;
-  /** The heading over `context.diagnostics`, between the URL and the attachment (keksdose
-   *  live #481). de-CH: "Diagnose" */
-  diagnostics: string;
-  /** Section 3's heading, and each picture's alt text. de-CH: "Anhang" */
-  attachment: string;
-  /** A PDF's / text file's download button, given the file's name.
-   *  de-CH: "{{name}} herunterladen" */
-  download: (name: string) => string;
-  /** The toast when that download failed. de-CH: "Der Anhang konnte nicht
-   *  heruntergeladen werden." */
-  downloadFailed: string;
-  /** Section 4's heading on a row without `replies` (a server before 0.38), where the
-   *  frozen outcome is shown read-only. de-CH: "Ergebnis" */
-  outcome: string;
-  /** Section 4's heading since 0.38: the replies' timeline (§9.7). de-CH: "Antworten" */
-  replies: string;
-  /** On section 4's heading line once the row is settled, given the formatted
-   *  `resolved_at` date. de-CH: "Erledigt am: {{date}}" */
-  resolvedAt: (date: string) => string;
-  /** The button that opens section 5 on an answered row. de-CH: "Nacharbeit" */
-  rework: string;
-  /** Section 5's heading. de-CH: "Zur Nacharbeit senden" */
-  reworkTitle: string;
-  /** Section 5's send button. de-CH: "Nacharbeit senden" */
-  reworkSend: string;
-  /** The line above the rework note. de-CH: "Was muss noch angepasst werden? Neue
-   *  Anforderungen oder Richtungswechsel." */
-  reworkPlaceholder: string;
-  /** The toast when the rework's file did not upload — and so nothing was sent. Names
-   *  no screenshot: it fires for a PDF or a log too (§7.14; keksdose's said
-   *  "screenshot"). de-CH: "Die Datei konnte nicht hochgeladen werden. Die Nacharbeit
-   *  wurde nicht gesendet." */
-  reworkUploadFailed: string;
-  /** Section 6's heading. de-CH: "Status" */
-  status: string;
-  /** A `FeedbackNoteEditor`'s box, named when no `placeholder` line names it and the
-   *  caller gives no `label` — without it the box was read out as just "edit text"
-   *  (round 0.34, the showcase's axe check). de-CH: "Notiz" */
-  note: string;
-}
-
-// Defined in ./feedback-row-detail-labels, which holds no component code (0.34, KS U8).
-export { DEFAULT_FEEDBACK_DETAIL_LABELS };
-
-/** `feedbackDetail` resolved: English, then `<UiKitProvider labels>`, then `labels`. */
-export function useFeedbackDetailLabels(labels?: LabelOverride<FeedbackDetailLabels>): FeedbackDetailLabels {
-  return useKitLabels("feedbackDetail", DEFAULT_FEEDBACK_DETAIL_LABELS, labels);
-}
-
-/* ── The page a report was filed on (§3.2, §4.4.2) ────────────────────────── */
-
-// The page as the reporter had it (path + query + hash) and the full URL "Copy URL"
-// copies were public too, as `feedbackPageHref` and `feedbackPageUrl`, until 0.53 (deprecated
-// in 0.52, API-4): `feedbackPage` (./feedback-page-link) is the one reading, and the kit
-// links only its path.
-
-/** The page's path alone — the compact text a URL column shows (§4.3), and the only part
- *  of the page the kit links. */
-export function feedbackPagePath(context: FeedbackContext | null | undefined): string | null {
-  return feedbackPage(context)?.path ?? null;
-}
-
-/* ── Attachments (§3.5, §4.4.3) ───────────────────────────────────────────── */
-
-/**
- * Every file a report carries, in the order the detail shows them (§4.4.3): the
- * screenshot it was filed with, then its other files, then whatever the rework rounds
- * brought, in body order (keksdose dev #578, live #291). One list rather than three —
- * a reader looking for "the picture" should not have to know at which point in the
- * conversation it was taken.
- *
- * Each URL once: keksdose and Kurvenschmiede content-address their keys, so the same
- * screenshot pasted twice is one URL — shown twice it is the same picture twice (and,
- * keyed by URL, a React key clash keksdose had).
- */
-export function feedbackAttachmentUrls(
-  row: Pick<FeedbackRecord, "screenshot_url" | "attachment_urls" | "body"> & Partial<Pick<FeedbackRecord, "replies">>,
-): string[] {
-  const urls = [
-    ...(row.screenshot_url ? [row.screenshot_url] : []),
-    // `?? []`: NULL on rows filed before the column, and no key at all from a server
-    // that predates it (keksdose dev #578).
-    ...(row.attachment_urls ?? []),
-    ...splitBodyAttachments(row.body).urls,
-    // 0.40 (§10.3): a rework entry's one file, in entry order — where the rework
-    // pictures of the body were before.
-    ...(row.replies ?? []).flatMap((reply) =>
-      reply.kind === "rework" && reply.attachment_url ? [reply.attachment_url] : [],
-    ),
-  ];
-  return [...new Set(urls)];
-}
-
-/* ── Who may do what (§3.4, §4.3, §4.4) ───────────────────────────────────── */
-
-/** What the person looking at a row may do with it — {@link feedbackRowAccess}. */
-export interface FeedbackRowAccess {
-  /** The viewer filed it (`viewerId === row.user_id`; never for an erased author). */
-  isAuthor: boolean;
-  /** "Edit description": the author, while the row is OPEN, READY or IN_PROGRESS — on either
-   *  page (keksdose `canAuthorEdit`, `feedback-page.tsx:398`, is not gated on `mine`). */
-  canEditDescription: boolean;
-  /** The admin on the inbox (`canEdit`): since 0.38 the reply composer (the outcome
-   *  itself is frozen, §9.7). */
-  canEditOutcome: boolean;
-  /** "Rework": the author, on an answered row ({@link FEEDBACK_REWORKABLE_STATUSES}).
-   *  Since 0.40 (§10.2 decision 6) the author's only — an admin sends a report back with
-   *  a reply whose status is `REWORK`; the server answers anyone else 403. */
-  canRework: boolean;
-  /** The status pills: the admin on the inbox; everyone else sees them read-only. */
-  canChangeStatus: boolean;
-}
-
-/**
- * The detail's permission rules in one place — the same answers the server gives
- * (§3.4), so the panel never offers what the PATCH would refuse.
- *
- * `canEdit` is the app's "admin on `/feedback`": an admin on `/my-feedback` edits no
- * status and no outcome there (§4.3), but as the author still edits the description
- * and sends for rework. `viewerId` is the signed-in user's id.
- */
-export function feedbackRowAccess(
-  row: Pick<FeedbackRecord, "user_id" | "status">,
-  { canEdit, viewerId }: { canEdit: boolean; viewerId?: number | null },
-): FeedbackRowAccess {
-  // `user_id: null` is an erased author (keksdose live #275): nobody is that author,
-  // least of all a viewer whose id happens to be missing too.
-  const isAuthor = row.user_id !== null && viewerId !== undefined && viewerId !== null && viewerId === row.user_id;
-  return {
-    isAuthor,
-    canEditDescription: isAuthor && FEEDBACK_AUTHOR_EDITABLE_STATUSES.includes(row.status),
-    canEditOutcome: canEdit,
-    canRework: isAuthor && FEEDBACK_REWORKABLE_STATUSES.includes(row.status),
-    canChangeStatus: canEdit,
-  };
-}
-
-/* ── The PATCH ─────────────────────────────────────────────────────────────── */
-
-/** What the detail sends as `PATCH /feedback/{id}` (§3.4): the body (a description
- *  edit, a rework append) or the outcome — and, through {@link useFeedbackStatusUndo}
- *  when no `statusChange` is given, the status. Never two at once: a rework PATCH must
- *  carry the body ALONE for the server to read it as an append (keksdose live #331). */
-export interface FeedbackDetailPatch {
-  id: number;
-  body?: string;
-  /** `null` clears it — a real edit. **Frozen since 0.38** (§9.7): server-kit 0.11
-   *  answers 422; the detail no longer sends it (a reply is written instead). */
-  outcome?: string | null;
-  status?: FeedbackStatus;
-  /** 0.38.1: the **Visible to others** flip (§9.3) — what `onPublicChange`'s
-   *  `PATCH /feedback/{id} {"public": next}` carries, alone, so an app can send it
-   *  through the same `onUpdate` mutation. */
-  public?: boolean;
-}
-
-/**
- * The page's update mutation (0.52): **TanStack Query's `mutateAsync`** — pass
- * `update.mutateAsync` (and `pending={update.isPending}`), the same function
- * `useFeedbackStatusUndo` takes. An editor closes when the promise resolves, which it does
- * for every call — also when a status pill was pressed while the save was in flight
- * (refactor audit 2026-10-30, FLW-3). A hand-written one resolves once the PATCH has
- * landed. A failure is the mutation's own to toast — the server's `detail`, else
- * `feedbackToast.updateFailed` (§4.4) — and leaves the editor open with its draft.
- */
-export type FeedbackDetailUpdateAsync = (patch: FeedbackDetailPatch) => Promise<unknown>;
-
-/** `POST /feedback/attachments` (§3.3): one file up, its URL
- *  (`/api/v1/feedback/attachments/<key>`) back. Reject on failure. */
-export type FeedbackUpload = (file: File) => Promise<string>;
-
-/* ── Send for rework (§4.4.5) ─────────────────────────────────────────────── */
-
-/** 0.40 (§10.3): `POST /feedback/{id}/rework` — the trimmed note and the uploaded file's
- *  URL. Return the promise: the section closes when it resolves and keeps the note when
- *  it rejects (the toast is the app's: the server's `detail`, else
- *  `feedbackToast.updateFailed`). */
-export type FeedbackReworkSend = (note: string, attachmentUrl?: string) => void | Promise<unknown>;
-
-export interface FeedbackReworkSectionProps {
-  /** The row being sent back: its id, and the body the round is appended to. */
-  row: Pick<FeedbackRecord, "id" | "body">;
-  /** 0.40: the rework as an entry of its own — see {@link FeedbackReworkSend}. With it,
-   *  `onUpdate` is not called. */
-  onRework?: FeedbackReworkSend;
-  /** The PATCH — see {@link FeedbackDetailUpdateAsync}. Called with `{ id, body }` only,
-   *  and only without `onRework`: the legacy append (§3.4), which server-kit 0.13 still
-   *  turns into an entry for one release. Optional since 0.40; pass one of the two. */
-  onUpdate?: FeedbackDetailUpdateAsync;
-  /** The PATCH is in flight (`update.isPending`). The upload adds its own. */
-  pending?: boolean;
-  /** The upload. Left out, the note goes without a file field. */
-  onUpload?: FeedbackUpload;
-  /** Snapshot the app view as a File — the kit's `captureAppScreenshot`, or the app's
-   *  own. A "Capture screenshot" button appears only with it; a rejection (no
-   *  `modern-screenshot`, a canvas that would not draw) toasts
-   *  `feedbackToast.captureFailed` and adds nothing. */
-  onCaptureScreenshot?: () => Promise<File | null>;
-  /** The round landed — the server has the note and, on an answered row, reopened it. */
-  onSent?: () => void;
-  onCancel: () => void;
-  /** Over the field's defaults: the contract's types and 10 MB (§3.5), the
-   *  `feedbackToast` refusals, the `feedbackAttachment` words. */
-  attachment?: Partial<FeedbackNoteAttachment>;
-  /** The send COMMITS — see `FeedbackNoteEditor`'s `commit`. */
-  commit?: CommitScope;
-  labels?: LabelOverride<FeedbackDetailLabels>;
-}
-
-/**
- * The rework editor (§4.4.5): the submitter sends an answered report
- * back with a note and at most one file, and the server reopens it (§3.4, keksdose
- * `submitRework`, `feedback-page.tsx:350`).
- *
- * - **The note is required** and trimmed: Send stays disabled on a blank box (a file
- *   alone cannot be sent; keksdose `:577` dropped that click silently).
- * - **One file**, added, captured or pasted — the dialog's types and 10 MB. It is
- *   uploaded FIRST and awaited: a body naming a file that never arrived would be a
- *   rework whose evidence is missing, and the upload is the half that fails on its
- *   own. If it fails nothing is sent and the toast says so; the note stays in the box.
- * - **0.40: an entry of its own** (§10.3) — with `onRework`, the note and the file's URL
- *   go to `POST /feedback/{id}/rework` and the body is not touched. Without it, the
- *   legacy append below, which server-kit 0.13 still converts (one minor).
- * - **The body alone goes in the PATCH** — {@link appendRework}'s strict extension of
- *   the current body, which is how the server recognises the append and reopens the
- *   row. Never a `status` (kastlan's client sent `OPEN`; an author may not write one,
- *   so the server refused the whole PATCH, note and all — keksdose live #331).
- *
- * Exported on its own for an app that offers rework somewhere other than the row
- * detail. Kurvenschmiede's author verdict ("That is it, done") is NOT here: an answered
- * item stays answered unless its author sends it back (§7.2).
- */
-export function FeedbackReworkSection({
-  row,
-  onRework,
-  onUpdate,
-  pending = false,
-  onUpload,
-  onCaptureScreenshot,
-  onSent,
-  onCancel,
-  attachment,
-  commit,
-  labels: labelsProp,
-}: FeedbackReworkSectionProps) {
-  const labels = useFeedbackDetailLabels(labelsProp);
-  const toasts = useFeedbackToastLabels();
-  const [uploading, setUploading] = useState(false);
-  const [sending, setSending] = useState(false);
-  // The body as it is NOW, not as it was when Send was pressed: the list can refresh
-  // while the file uploads, and an append to a stale body is no longer a strict
-  // extension of the stored one — the server would read it as an edit, and refuse it.
-  const latestBody = useRef(row.body);
-  useEffect(() => {
-    latestBody.current = row.body;
-  }, [row.body]);
-
-  const capture = onCaptureScreenshot
-    ? async () => {
-        try {
-          return await onCaptureScreenshot();
-        } catch {
-          toast.error(toasts.captureFailed);
-          return null;
-        }
-      }
-    : undefined;
-
-  const send = async (value: string, file?: File | null) => {
-    const note = value.trim();
-    if (!note) return;
-    let url: string | null = null;
-    if (file && onUpload) {
-      setUploading(true);
-      try {
-        url = await onUpload(file);
-      } catch {
-        toast.error(labels.reworkUploadFailed);
-        return;
-      } finally {
-        setUploading(false);
-      }
-    }
-    if (onRework) {
-      // 0.40 (§10.3): the entry, not the body — the server checks the URL (a 422 outside
-      // the app's attachment prefix).
-      setSending(true);
-      try {
-        await onRework(note, url ?? undefined);
-      } catch {
-        return; // The app's toast; the note stays in the box.
-      } finally {
-        setSending(false);
-      }
-      onSent?.();
-      return;
-    }
-    if (!onUpdate) return;
-    let body: string | null;
-    try {
-      body = appendRework(latestBody.current, note, url);
-    } catch {
-      // The upload answered with something that is not an attachment URL of this
-      // contract — written anyway it would sit in the body as prose no reader knows to
-      // be a file. As far as the person is concerned, the file did not arrive.
-      toast.error(labels.reworkUploadFailed);
-      return;
-    }
-    if (body === null) return;
-    runMutation(onUpdate, { id: row.id, body }, () => onSent?.());
-  };
-
-  return (
-    <FeedbackDetailSection title={labels.reworkTitle}>
-      <FeedbackNoteEditor
-        key={row.id}
-        initial=""
-        required
-        commit={commit}
-        pending={pending || uploading || sending}
-        onSave={(value, file) => void send(value, file)}
-        onCancel={onCancel}
-        saveLabel={labels.reworkSend}
-        cancelLabel={labels.cancel}
-        placeholder={labels.reworkPlaceholder}
-        attachment={
-          onUpload
-            ? {
-                accept: FEEDBACK_ATTACHMENT_ACCEPT,
-                maxBytes: DEFAULT_MAX_ATTACHMENT_BYTES,
-                onError: (kind) => {
-                  toast.error(kind === "type" ? toasts.attachmentUnsupported : toasts.attachmentTooLarge);
-                },
-                onCaptureScreenshot: capture,
-                ...attachment,
-              }
-            : undefined
-        }
-      />
-    </FeedbackDetailSection>
-  );
-}
-
-/* ── The detail (§4.4) ─────────────────────────────────────────────────────── */
-
-export interface FeedbackRowDetailProps {
-  /** The report — the list's own row, as `GET /feedback` / `GET /feedback/my` sent it. */
-  row: FeedbackRecord;
-  /** The admin on the inbox (`/feedback`): status pills and the outcome editor. `false`
-   *  on `/my-feedback`, for an admin too (§4.3). */
-  canEdit: boolean;
-  /** The signed-in user's id — who the AUTHOR is: the description editor and rework. */
-  viewerId?: number | null;
-  /** The page's PATCH — `update.mutateAsync` (see {@link FeedbackDetailUpdateAsync}). The
-   *  callback form, `update.mutate`, went in 0.53 (deprecated in 0.52). */
-  onUpdate: FeedbackDetailUpdateAsync;
-  /** `update.isPending`: the editors' save buttons wait for it. */
-  pending?: boolean;
-  /** `POST /feedback/attachments` for the rework's one file and, since 0.47 (§12) with
-   *  `replyAttachments`, the admin reply's. Left out, a rework or a reply is text alone. */
-  onUpload?: FeedbackUpload;
-  /** The app's authenticated GET (`AuthedImage`'s) — the pictures and the downloads go
-   *  through it, since the attachment route wants the bearer token (§3.5). Left out,
-   *  the browser's own fetch with same-origin cookies. */
-  fetcher?: AuthedFetcher | null;
-  /**
-   * The undoable status change — what `useFeedbackStatusUndo(update.mutateAsync)` returns,
-   * the SAME one the table's status cell and the swipes use (§4.5). Left out, the
-   * detail builds one from `onUpdate`, which is equivalent as long as `onUpdate` is the
-   * page's mutation (its observer outlives the row a status filter removes).
-   */
-  statusChange?: FeedbackStatusChange;
-  /** For the rework's and (0.47, with `replyAttachments`) the reply composer's "Capture screenshot" — see
-   *  {@link FeedbackReworkSectionProps}. */
-  onCaptureScreenshot?: () => Promise<File | null>;
-  /** The rework's and (0.47, with `replyAttachments`) the reply composer's file field,
-   *  over the contract's defaults. */
-  reworkAttachment?: Partial<FeedbackNoteAttachment>;
-  /** How the `resolved_at` date reads — the app's date cell, as a string. Default
-   *  (0.52): the `<UiKitProvider formatDate>` when the app set one, else the kit's
-   *  `formatDate` in the provider's locale and time zone, `medium` style. */
-  renderDate?: (iso: string) => string;
-  /**
-   * 0.52 (FLW-7): the app's one-shot routes, ON TOP of the kit's — `/join`,
-   * `/register`, `/verify-email`, `/reset-password` (a string is that path and everything
-   * under it, a RegExp is tested on the path, as `useLastVisitedPage`'s `exclude`). The
-   * URL section shows such a page as text with Copy, never as a link: it acts on load,
-   * and a report's page is the reporter's to write. Any other page links its PATH alone;
-   * the query, where tokens and `?next=` targets travel, is text. keksdose:
-   * `["/banks/callback", "/share-target"]`. Keep it stable (module scope).
-   */
-  pageLinkExclude?: readonly PathPattern[];
-  /**
-   * The detail's saves COMMIT (description, outcome, rework, status): under a locked
-   * `WriteLockProvider` each is `aria-disabled` with the lock's reason, and nothing
-   * reaches `onUpdate`. Off by default — keksdose's shell lock is a read-only demo
-   * BUDGET, and a person viewing one still owns their own reports.
-   */
-  commit?: CommitScope;
-  /**
-   * Open with the subject: the category badge and the title (feedback contract §8.1,
-   * keksdose live #395). On by default; `false` for a page that already shows the title
-   * beside the detail.
-   */
-  showSubject?: boolean;
-  /**
-   * 0.38: `POST /feedback/{id}/replies` (§9.7) — the body and the status picked
-   * (`undefined`: "No change"). With `canEdit` and a row that carries `replies`, the
-   * reply composer stands under the timeline. Return the promise: the box clears when it
-   * resolves. A failure is the app's to toast — the server's `detail`, else
-   * `feedbackToast.replyFailed` (0.38.1).
-   *
-   * 0.47 (§12, keksdose live #447): with `replyAttachments` (and `onUpload`) the
-   * composer takes one file as the rework note does — pasted into the box, captured or
-   * picked — uploaded first, and its URL is the third argument (`attachment_url` in the
-   * body; server-kit 0.18).
-   */
-  onReply?: (body: string, status?: FeedbackStatus, attachmentUrl?: string) => void | Promise<unknown>;
-  /**
-   * 0.47 (§12): the reply composer takes one file — through `onUpload`,
-   * `onCaptureScreenshot` and `reworkAttachment`, the rework note's. Off by default:
-   * `onUpload` alone stays the rework's, so a page that passes it and an `onReply` of
-   * `(body, status)` does not grow a file field whose file its reply would drop (round
-   * 0.47 build reviews, keksdose and kastlan B1). Turn it on in the commit that makes
-   * `onReply` send its third argument as `attachment_url`, with server-kit 0.18 (an
-   * older server answers the field with a 422).
-   */
-  replyAttachments?: boolean;
-  /** 0.38: the reply is in flight. */
-  replying?: boolean;
-  /**
-   * 0.40 (§10.3): `POST /feedback/{id}/rework` — the author's rework as an entry of its
-   * own (see {@link FeedbackReworkSend}). The **Rework** button shows only to the author
-   * on an answered row, and only where the reply composer is not shown (§10.2 decision 6:
-   * an admin answering a report sends it back with a reply whose status is `REWORK`).
-   * Left out, the rework goes through `onUpdate` as the legacy body append — for one
-   * minor, while the server converts it.
-   */
-  onRework?: FeedbackReworkSend;
-  /** 0.38: the report's comments — the app's `FeedbackCommentThread` — in a section of
-   *  their own under the replies. A thread here draws the comments alone (0.38.1): the
-   *  Replies section above has the replies, so whatever `replies` the thread is handed
-   *  are dropped, and nothing is drawn twice. */
-  comments?: ReactNode;
-  /** 0.38: `PATCH /feedback/{id} {"public": next}` — draws the **Visible to others**
-   *  switch (`FeedbackPublicSwitch`) in the Status section, for the admin (`canEdit`)
-   *  and, within §9.3's window, the author. The page's own mutation fits:
-   *  `onPublicChange={(next) => update.mutate({ id: row.id, public: next })}` —
-   *  {@link FeedbackDetailPatch}'s `public` (0.38.1). */
-  onPublicChange?: (next: boolean) => void;
-  labels?: LabelOverride<FeedbackDetailLabels>;
-  className?: string;
-}
-
-/**
- * The expanded row of a feedback table (§4.4): what was reported, where, with which
- * files, what the answer was, and where it stands — with the edits each viewer may
- * make. Hand it to `DataTable`'s `expandedRow`:
- *
- * ```tsx
- * const update = useMutation({ mutationFn: ({ id, ...patch }) => api.updateFeedback(id, patch),
- *   onSuccess: invalidate, onError: (e) => toast.error(detailOf(e) ?? toastLabels.updateFailed) });
- * const changeStatus = useFeedbackStatusUndo(update.mutateAsync);
- * <DataTable
- *   isExpanded={(row) => expandedId === row.id}
- *   expandedRow={(row) => (
- *     <FeedbackRowDetail row={row} canEdit={isAdmin && !mine} viewerId={user?.id}
- *       onUpdate={update.mutateAsync} pending={update.isPending} statusChange={changeStatus}
- *       onUpload={api.uploadFeedbackAttachment} fetcher={authedImageFetcher}
- *       onCaptureScreenshot={captureAppScreenshot} />
- *   )} … />
- * ```
- *
- * Mounted only while its row is expanded, so which editor is open lives here, and
- * collapsing the row closes it. Each editor is keyed by the row's id, so a draft is
- * never thrown away by a list refresh under it.
- */
-export function FeedbackRowDetail({
-  row,
-  canEdit,
-  viewerId,
-  onUpdate,
-  pending = false,
-  onUpload,
-  fetcher,
-  statusChange,
-  onCaptureScreenshot,
-  reworkAttachment,
-  renderDate,
-  pageLinkExclude,
-  commit,
-  showSubject = true,
-  onReply,
-  replyAttachments = false,
-  replying = false,
-  onRework,
-  comments,
-  onPublicChange,
-  labels: labelsProp,
-  className,
-}: FeedbackRowDetailProps) {
-  const labels = useFeedbackDetailLabels(labelsProp);
-  // "placeholder = the dialog's body label" (§4.4.1): read from the dialog's own
-  // namespace, so the two can never say different things.
-  const dialogText = useKitLabels("feedbackDialog", DEFAULT_FEEDBACK_DIALOG_LABELS);
-  const ownStatusChange = useFeedbackStatusUndo(onUpdate);
-  const changeStatus = statusChange ?? ownStatusChange;
-  const [editingBody, setEditingBody] = useState(false);
-  const [reworking, setReworking] = useState(false);
-
-  const access = feedbackRowAccess(row, { canEdit, viewerId });
-  // The body as written, without its file lines (they are shown as files below) — the
-  // rework rounds stay in the prose: they are what was said after the answer.
-  const bodyText = splitBodyAttachments(row.body).text;
-  const { description } = splitDescription(row.body);
-  const page = feedbackPage(row.context);
-  const pageLink = page && feedbackPageLinkTarget(page, pageLinkExclude);
-  // What the app attached is for whoever handles the report and whoever sent it, never
-  // for another viewer of a shared row (keksdose live #481).
-  const diagnostics = canEdit || access.isAuthor ? diagnosticsEntries(row.context?.diagnostics) : [];
-  // 0.47 (§12): with the entries, each entry's file is drawn inline in the Replies
-  // timeline, so the Attachment section holds the report's own files.
-  const files = feedbackAttachmentUrls(row.replies ? { ...row, replies: undefined } : row);
-  const fetchFile = fetcher ?? plainFeedbackFetch;
-  // A date in a sentence ("Resolved: …"), so no weekday — through the kit's one day
-  // ladder, which reaches the provider's `formatDate` (0.52, refactor audit 2026-10-30,
-  // DUP-2).
-  const day = useDayFormatter<string>(renderDate);
-  const dateText = (iso: string) => day(iso);
-
-  const saveDescription = (value: string) => {
-    // Only a real change makes a round trip; clearing it is one ("" — the description
-    // is optional, §7.4), and the rounds and file lines go back unchanged.
-    if (value.trim() === description.trim()) {
-      setEditingBody(false);
-      return;
-    }
-    runMutation(onUpdate, { id: row.id, body: replaceDescription(row.body, value) }, () => setEditingBody(false));
-  };
-
-  // 0.38 (§9.7): the replies replace the outcome; a row without them is a server before
-  // 0.38, whose frozen outcome is shown as it is, read-only.
-  const replies = row.replies;
-  const commentLabels = useKitLabels("feedbackComments", DEFAULT_FEEDBACK_COMMENTS_LABELS);
-  // 0.40 (§10.2 decision 6): whoever has the reply composer answers with it — REWORK is
-  // one of its statuses — so the Rework button is for an author without it.
-  const composerShown = Boolean(replies && access.canEditOutcome && onReply);
-  const canRework = access.canRework && !composerShown;
-  const showRework = canRework && reworking;
-
-  return (
-    <FeedbackDetail className={className}>
-      {showSubject && (
-        <FeedbackDetailSection title={labels.subject}>
-          <p className="flex flex-wrap items-center gap-x-2 gap-y-1 text-sm" data-feedback-subject="">
-            <FeedbackCategoryBadge category={row.category} />
-            <span className="min-w-0 font-medium text-primary [overflow-wrap:anywhere]">{row.title}</span>
-          </p>
-        </FeedbackDetailSection>
-      )}
-      <FeedbackDetailSection
-        title={labels.body}
-        action={
-          access.canEditDescription && !editingBody ? (
-            <Button
-              type="button"
-              variant="ghost"
-              size="sm"
-              onClick={(event) => {
-                event.stopPropagation();
-                setEditingBody(true);
-              }}
-              aria-label={labels.editDescription}
-            >
-              {labels.edit}
-            </Button>
-          ) : undefined
-        }
-      >
-        {access.canEditDescription && editingBody ? (
-          <FeedbackNoteEditor
-            key={row.id}
-            // The ORIGINAL description only — see the module note.
-            initial={description}
-            pending={pending}
-            commit={commit}
-            onSave={saveDescription}
-            onCancel={() => setEditingBody(false)}
-            saveLabel={labels.save}
-            cancelLabel={labels.cancel}
-            placeholder={dialogText.bodyOptional ?? dialogText.body}
-          />
-        ) : (
-          <FeedbackProse>{bodyText.trim() ? bodyText : undefined}</FeedbackProse>
-        )}
-      </FeedbackDetailSection>
-
-      {page && (
-        // The page it was filed on, with the full URL a click away (keksdose #278). Only
-        // the path is a link; the query and hash, the reporter's to write, are text — and
-        // a one-shot path is text too (FLW-7, 0.52 code review: see feedback-page-link).
-        <FeedbackDetailSection title={labels.url}>
-          <div className="flex min-w-0 items-center gap-1.5">
-            <code
-              className="min-w-0 break-all text-xs text-secondary"
-              data-feedback-page=""
-              data-feedback-page-unlinked={pageLink ? undefined : ""}
-            >
-              {pageLink ? (
-                <TextLink href={pageLink} tone="secondary" underline="hover" stopPropagation>
-                  {page.path}
-                </TextLink>
-              ) : (
-                page.path
-              )}
-              {page.href.slice(page.path.length)}
-            </code>
-            <CopyButton size="xs" className="shrink-0" text={page.url} label={labels.copyUrl} stopPropagation />
-          </div>
-        </FeedbackDetailSection>
-      )}
-
-      {diagnostics.length > 0 && (
-        <FeedbackDetailSection title={labels.diagnostics}>
-          <FeedbackDiagnosticsList entries={diagnostics} />
-        </FeedbackDetailSection>
-      )}
-
-      {files.length > 0 && (
-        <FeedbackDetailSection title={labels.attachment}>
-          <div className="flex min-w-0 flex-wrap items-start gap-2">
-            {files.map((url) =>
-              isImageAttachment(url) ? (
-                <AuthedImage
-                  key={url}
-                  src={url}
-                  alt={labels.attachment}
-                  fetcher={fetchFile}
-                  link
-                  stopPropagation
-                  // A broken attachment is simply not in the row (keksdose's
-                  // AuthedImage default) rather than a grey tile per failure.
-                  errorFallback={null}
-                  wrapperClassName="inline-block max-w-full"
-                  className="max-h-64 max-w-full rounded border border-subtle"
-                />
-              ) : (
-                <FeedbackAttachmentDownload key={url} url={url} fetcher={fetchFile} />
-              ),
-            )}
-          </div>
-        </FeedbackDetailSection>
-      )}
-
-      <FeedbackDetailSection
-        title={replies ? labels.replies : labels.outcome}
-        // When the row entered DONE / WONT_DO (keksdose feedback #96).
-        action={
-          row.resolved_at ? (
-            <span className="text-xs text-muted">{labels.resolvedAt(dateText(row.resolved_at))}</span>
-          ) : undefined
-        }
-      >
-        <div className="space-y-3">
-          {replies ? (
-            <FeedbackReplyTimeline replies={replies} fetcher={fetchFile} />
-          ) : (
-            <FeedbackProse>{row.outcome || undefined}</FeedbackProse>
-          )}
-          {composerShown && replies && onReply && (
-            <FeedbackReplyComposer
-              key={row.id}
-              status={row.status}
-              onReply={onReply}
-              pending={replying}
-              onUpload={replyAttachments ? onUpload : undefined}
-              onCaptureScreenshot={replyAttachments ? onCaptureScreenshot : undefined}
-              attachment={replyAttachments ? reworkAttachment : undefined}
-              suggestedStatus={feedbackReplyNudge(row)}
-              commit={commit}
-            />
-          )}
-          {/* No "Open page" button here since 0.46 (keksdose live #449): the URL section
-              above already links the page, with the path on screen. */}
-          {canRework && !reworking && (
-            <div className="flex flex-wrap gap-2">
-              <Button
-                type="button"
-                variant="secondary"
-                onClick={(event) => {
-                  event.stopPropagation();
-                  setReworking(true);
-                }}
-              >
-                <RotateCcw className="size-3.5" aria-hidden /> {labels.rework}
-              </Button>
-            </div>
-          )}
-        </div>
-      </FeedbackDetailSection>
-
-      {comments != null && (
-        <FeedbackDetailSection title={commentLabels.comments}>
-          <FeedbackRepliesDrawnAboveContext.Provider value>{comments}</FeedbackRepliesDrawnAboveContext.Provider>
-        </FeedbackDetailSection>
-      )}
-
-      {showRework && (
-        <FeedbackReworkSection
-          row={row}
-          onRework={onRework}
-          onUpdate={onUpdate}
-          pending={pending}
-          onUpload={onUpload}
-          onCaptureScreenshot={onCaptureScreenshot}
-          onSent={() => setReworking(false)}
-          onCancel={() => setReworking(false)}
-          attachment={reworkAttachment}
-          commit={commit}
-          labels={labelsProp}
-        />
-      )}
-
-      <FeedbackDetailSection title={labels.status}>
-        {/* Expanded: the whole enum in one click for the admin (keksdose feedback #421);
-            everyone else the narrow set, read-only — a picture of the workflow. */}
-        <FeedbackStatusTransitions
-          status={row.status}
-          statuses={
-            access.canChangeStatus ? selectableFeedbackStatuses(row.status) : visibleFeedbackStatuses(row.status)
-          }
-          canEdit={access.canChangeStatus}
-          commit={commit}
-          onPick={(status) => changeStatus(row, status)}
-          variant="pill"
-        />
-        {onPublicChange && (
-          <FeedbackPublicSwitch
-            row={row}
-            isAdmin={canEdit}
-            viewerId={viewerId}
-            onChange={onPublicChange}
-            pending={pending}
-            commit={commit}
-            className="mt-3"
-          />
-        )}
-      </FeedbackDetailSection>
-    </FeedbackDetail>
-  );
-}