@lerianstudio/sindarian-ui 2.0.0-beta.1 → 2.0.0-beta.3

package/dist/components/ui/file-upload/index.js

@@ -34,15 +34,23 @@ var __importStar = (this && this.__importStar) || (function () {
     };
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.FileUpload = void 0;
+exports.MultipleFileUpload = exports.FileUpload = void 0;
 exports.validateFile = validateFile;
+exports.humanizeSize = humanizeSize;
 const jsx_runtime_1 = require("react/jsx-runtime");
 /**
- * FileUpload — a controlled "pick a file, read it to text, hand back the text"
- * primitive. The defining job: select one file (by click, keyboard, or
- * drag-and-drop), validate it against an accept filter + a byte ceiling, read
- * it as UTF-8 via FileReader.readAsText, and emit `{ file, text }`. Binary/DER
- * (.pfx/.p12) is explicitly OUT of scope — readAsText only.
+ * FileUpload — a controlled "pick a file, hand it back" primitive. The defining
+ * job: select one file (by click, keyboard, or drag-and-drop), validate it
+ * against an accept filter + a byte ceiling, and emit `{ file, text }`.
+ *
+ * `readAs` decides whether the bytes are decoded on the way through. `'text'`
+ * (the default) reads UTF-8 via FileReader.readAsText, which is what a PEM/CSV
+ * host wants. `'none'` hands the `File` over untouched and is the only mode a
+ * BINARY file can use: a PDF, XLSX or PFX put through readAsText decodes into
+ * replacement-character garbage, and that garbage string is then retained for
+ * as long as the host holds the value — 20 MiB of PDF became a useless 20 MiB
+ * string, and the host still had to read the file a second time to get at the
+ * real bytes.
  *
  * The real `<input type="file">` IS the accessible control: it is `sr-only`
  * (visually hidden) but focusable and labelable — never `aria-hidden`, never
@@ -63,6 +71,11 @@ const jsx_runtime_1 = require("react/jsx-runtime");
  * a value, so it cannot masquerade as valid. `accept` is validated against
  * BOTH extension and MIME because the native `accept` attribute is only a
  * browser hint and is bypassable via drag-drop.
+ *
+ * A host that already announces the rejection itself — a toast in its own
+ * locale, driven off `onError` — silences this one by returning nothing from
+ * `labels.error`. Two announcements for one event, in two languages, is the
+ * accessibility defect; the wording is not.
  */
 const React = __importStar(require("react"));
 const lucide_react_1 = require("lucide-react");
@@ -105,7 +118,11 @@ function matchesAccept(file, accept) {
         return mime === token;
     });
 }
-/** Humanize a byte count for the selected-file chip. Binary units, 1 decimal. */
+/**
+ * Humanize a byte count for the selected-file chip. Binary units, 1 decimal.
+ * Exported so a `labels.error` override can format `maxSizeBytes` exactly the
+ * way the chip and the default message do, instead of re-deriving binary units.
+ */
 function humanizeSize(bytes) {
     if (bytes < 1024)
         return `${bytes} B`;
@@ -118,8 +135,8 @@ function humanizeSize(bytes) {
     }
     return `${size.toFixed(1)} ${units[unit]}`;
 }
-/** A human-readable announcement for each rejection kind. */
-function errorMessage(error) {
+/** The default English announcement for each rejection kind. */
+function defaultErrorMessage(error) {
     switch (error.kind) {
         case 'too-large':
             return `File is too large (max ${humanizeSize(error.maxSizeBytes)}).`;
@@ -129,7 +146,7 @@ function errorMessage(error) {
             return 'Could not read the file.';
     }
 }
-exports.FileUpload = React.forwardRef(function FileUpload({ accept, maxSizeBytes, value, onSelect, onError, disabled = false, id, className, 'aria-invalid': ariaInvalid, 'aria-required': ariaRequired, 'aria-describedby': ariaDescribedby, 'aria-label': ariaLabel, ...rest }, ref) {
+exports.FileUpload = React.forwardRef(function FileUpload({ accept, maxSizeBytes, value, readAs = 'text', labels, onSelect, onError, disabled = false, id, className, 'aria-invalid': ariaInvalid, 'aria-required': ariaRequired, 'aria-describedby': ariaDescribedby, 'aria-label': ariaLabel, ...rest }, ref) {
     const internalRef = React.useRef(null);
     React.useImperativeHandle(ref, () => internalRef.current);
     const reactId = React.useId();
@@ -141,12 +158,21 @@ exports.FileUpload = React.forwardRef(function FileUpload({ accept, maxSizeBytes
     // component's own validation/read error — a self-detected bad pick must read
     // as invalid even outside a form.
     const invalid = ariaInvalid || error !== null;
+    // Resolve the announcement BEFORE deciding whether the alert exists: a
+    // consumer that returns nothing is opting out of this surface entirely,
+    // and an association pointing at an unrendered node is worse than none.
+    const resolved = error
+        ? labels?.error
+            ? labels.error(error)
+            : defaultErrorMessage(error)
+        : null;
+    const errorText = resolved ? resolved : null;
     // Radix Slot OVERWRITES aria-describedby (it does not merge), so merge the
     // primitive's own role=alert error id with the FormControl-injected one so
     // both associations coexist on the input. A plain join, never `cn` —
     // tailwind-merge treats these as class names and would drop an id that
     // happens to look like a conflicting utility.
-    const describedBy = [ariaDescribedby, error ? errorId : undefined]
+    const describedBy = [ariaDescribedby, errorText ? errorId : undefined]
         .filter(Boolean)
         .join(' ') || undefined;
     // Last-resolved-wins: a slow read for pick A must not overwrite a newer
@@ -162,6 +188,15 @@ exports.FileUpload = React.forwardRef(function FileUpload({ accept, maxSizeBytes
             onError?.(validationError);
             return;
         }
+        // Binary path: no decode, no reader, no retained garbage string. The
+        // stale-rejection clear still has to happen here — the alert is this
+        // component's own state and a good pick must retire it.
+        if (readAs === 'none') {
+            readerRef.current = null;
+            setError(null);
+            onSelect({ file, text: '' });
+            return;
+        }
         const reader = new FileReader();
         readerRef.current = reader;
         reader.onload = () => {
@@ -228,10 +263,290 @@ exports.FileUpload = React.forwardRef(function FileUpload({ accept, maxSizeBytes
     return ((0, jsx_runtime_1.jsxs)("div", { className: (0, utils_1.cn)('space-y-2', className), children: [(0, jsx_runtime_1.jsxs)("div", { onClick: openPicker, onDragOver: onDragOver, onDragLeave: onDragLeave, onDrop: onDrop, className: (0, utils_1.cn)('border-input bg-card focus-within:ring-ring focus-within:ring-offset-background aria-[invalid=true]:border-destructive aria-[invalid=true]:focus-within:ring-destructive flex w-full items-center gap-3 rounded-md border px-3 py-4 text-sm shadow-xs transition-colors focus-within:ring-2 focus-within:ring-offset-1 focus-within:outline-none', dragActive &&
                     'border-ring ring-ring ring-offset-background ring-2 ring-offset-1', disabled
                     ? 'border-muted bg-muted/30 cursor-not-allowed shadow-none'
-                    : 'cursor-pointer'), "aria-invalid": invalid || undefined, children: [(0, jsx_runtime_1.jsx)("input", { ref: internalRef, id: inputId, type: "file", accept: accept, disabled: disabled, className: "sr-only", "aria-invalid": invalid || undefined, "aria-required": ariaRequired || undefined, "aria-describedby": describedBy, "aria-label": ariaLabel, onChange: onInputChange, ...rest }), (0, jsx_runtime_1.jsx)(lucide_react_1.Upload, { className: "text-muted-foreground size-4 shrink-0", "aria-hidden": "true" }), value ? ((0, jsx_runtime_1.jsxs)(jsx_runtime_1.Fragment, { children: [(0, jsx_runtime_1.jsxs)("span", { className: "min-w-0 flex-1 truncate", children: [(0, jsx_runtime_1.jsx)("span", { className: "text-foreground font-medium", children: value.file.name }), ' ', (0, jsx_runtime_1.jsx)("span", { className: "text-muted-foreground tabular-nums", children: humanizeSize(value.file.size) })] }), (0, jsx_runtime_1.jsx)(icon_button_1.IconButton, { type: "button", variant: "plain", size: "small", disabled: disabled, "aria-label": "Remove file", onClick: (event) => {
+                    : 'cursor-pointer'), "aria-invalid": invalid || undefined, children: [(0, jsx_runtime_1.jsx)("input", { ref: internalRef, id: inputId, type: "file", accept: accept, disabled: disabled, className: "sr-only", "aria-invalid": invalid || undefined, "aria-required": ariaRequired || undefined, "aria-describedby": describedBy, "aria-label": ariaLabel, onChange: onInputChange, ...rest }), (0, jsx_runtime_1.jsx)(lucide_react_1.Upload, { className: "text-muted-foreground size-4 shrink-0", "aria-hidden": "true" }), value ? ((0, jsx_runtime_1.jsxs)(jsx_runtime_1.Fragment, { children: [(0, jsx_runtime_1.jsxs)("span", { className: "min-w-0 flex-1 truncate", children: [(0, jsx_runtime_1.jsx)("span", { className: "text-foreground font-medium", children: value.file.name }), ' ', (0, jsx_runtime_1.jsx)("span", { className: "text-muted-foreground tabular-nums", children: humanizeSize(value.file.size) })] }), (0, jsx_runtime_1.jsx)(icon_button_1.IconButton, { type: "button", variant: "plain", size: "small", disabled: disabled, "aria-label": labels?.remove?.trim() ? labels.remove : 'Remove file', onClick: (event) => {
                                     // Don't bubble to the zone's openPicker and re-open the dialog.
                                     event.stopPropagation();
                                     clear();
-                                }, children: (0, jsx_runtime_1.jsx)(lucide_react_1.X, { className: "size-4", "aria-hidden": "true" }) })] })) : ((0, jsx_runtime_1.jsxs)("span", { className: "text-muted-foreground", children: [(0, jsx_runtime_1.jsx)("span", { className: "text-foreground font-medium", children: "Choose a file" }), ' ', "or drag and drop"] }))] }), error ? ((0, jsx_runtime_1.jsx)("p", { id: errorId, role: "alert", className: "text-system-error-h1a text-xs font-medium", children: errorMessage(error) })) : null] }));
+                                }, children: (0, jsx_runtime_1.jsx)(lucide_react_1.X, { className: "size-4", "aria-hidden": "true" }) })] })) : ((0, jsx_runtime_1.jsxs)("span", { className: "text-muted-foreground", children: [(0, jsx_runtime_1.jsx)("span", { className: "text-foreground font-medium", children: labels?.action ?? 'Choose a file' }), ' ', labels?.hint ?? 'or drag and drop'] }))] }), errorText ? ((0, jsx_runtime_1.jsx)("p", { id: errorId, role: "alert", className: "text-system-error-h1a text-xs font-medium", children: errorText })) : null] }));
 });
 exports.FileUpload.displayName = 'FileUpload';
+/** The default English announcement for each rejection kind. */
+function defaultMultipleErrorMessage(error) {
+    if (error.kind === 'too-many') {
+        return `Too many files (max ${error.maxFiles}). ${error.file.name} was not added.`;
+    }
+    return defaultErrorMessage(error);
+}
+/** The default zone copy once the cap is reached, in grammatical English. */
+function defaultFullMessage(maxFiles) {
+    return maxFiles === 1
+        ? 'Maximum of 1 file reached.'
+        : `Maximum of ${maxFiles} files reached.`;
+}
+/**
+ * MultipleFileUpload: the plural sibling of FileUpload. Pick SEVERAL files,
+ * accumulate them across repeated picks, validate each one, cap the total, and
+ * hand back `FileUploadResult[]`.
+ *
+ * A SIBLING COMPONENT, not a `multiple` flag. This library already answers
+ * "this one takes many" that way (`Select` / `MultipleSelect`), and FileUpload
+ * strips `'multiple'` from its props on purpose: single-file is its contract,
+ * not a default it happens to have. A boolean would have forced every prop
+ * here into a union that means one thing when the flag is set and another when
+ * it is not: `value` as `Result | Result[] | null`, a `maxFiles` that is
+ * meaningless in half the configurations, and a remove control whose
+ * accessible name is fixed copy in one mode and per-file in the other. The
+ * plural props follow the house shape for plural components: `value?: T[]`
+ * with `onValueChange?: (values: T[]) => void`.
+ *
+ * WHERE THE LINE SITS: this component owns SELECTION and nothing after it.
+ * Choosing, validating, capping, listing and removing are its job; uploading
+ * is not. That split is not squeamishness about scope, it is where the
+ * knowledge actually lives. An upload needs an endpoint, an auth scheme, a
+ * concurrency policy, a retry policy and, very often, a parent id that does
+ * not exist yet when the files are chosen: the motivating host stages evidence
+ * files while a form is being filled and can only upload them against the id
+ * that its create call returns afterwards. None of that is knowable from
+ * inside a library primitive, and a component that guessed would have to be
+ * fought rather than used. So the host keeps its own per-file record with
+ * status and retry, and this component keeps the part a form can hold and
+ * validate: the chosen files. `FileUploadResult[]` is a value; an upload state
+ * machine is not.
+ *
+ * ACCUMULATION is the defining behaviour. A second pick ADDS to the selection
+ * rather than replacing it, because a user assembling five documents does it
+ * in two or three trips to the file dialog, not one. Everything else follows
+ * from that: room is measured against what is already selected, and the batch
+ * that overflows the cap still contributes the files that fit.
+ *
+ * A BATCH SURVIVES ITS OWN CASUALTIES. One file rejected for type, size or a
+ * failed read does not discard the rest of the batch, and it does not consume
+ * a slot either: validation runs over the WHOLE batch before the cap is
+ * applied, so a file that was never eligible cannot cost an eligible one its
+ * place, and `'too-many'` always names a file a slot would genuinely have
+ * taken. The alternative punishes
+ * a user for a mistake in one file by throwing away four good ones, and hands
+ * back no way to tell which was which. Every rejection is reported through
+ * `onError` and announced together in one `role="alert"`.
+ *
+ * Accessibility follows the sibling BELOW THE CAP: the real `<input
+ * type="file">` is `sr-only` but focusable and labelable, and never
+ * `aria-hidden`, so FormControl-injected ARIA and react-hook-form's focus on
+ * error both work while the picker is enabled. At the cap that changes, and
+ * the paragraph below says how.
+ * The file list sits OUTSIDE the click zone, so activating a remove control
+ * cannot also reopen the picker, and each remove control is named after its
+ * own file: a column of identical "Remove file" buttons tells a screen-reader
+ * user nothing about which row they are on.
+ *
+ * AT THE CAP the picker takes the native `disabled` attribute, and that DOES
+ * take it out of the tab order. This is the one state in which the input is
+ * not a focus target, and the one state in which focus-on-error cannot land on
+ * it, so it is a real cost rather than a free win. The cap has no counterpart
+ * in the single-file sibling, so the precedent followed here is
+ * `DateRangePicker`'s trigger: a control whose only job is to open a dialog has
+ * no state worth keeping focusable, and native `disabled` is what both removes
+ * it from the tab order and keeps the dialog shut. The alternative, an enabled
+ * picker that opens the file dialog and then refuses every file with
+ * `too-many`, is a control that lies about being available. What keeps the cap
+ * from being a dead end is the escape hatch: the remove controls answer to
+ * `disabled` alone and NEVER to the cap, so they stay focusable and removing
+ * one file reopens the picker.
+ */
+exports.MultipleFileUpload = React.forwardRef(function MultipleFileUpload({ accept, maxSizeBytes, maxFiles, value = [], readAs = 'text', labels, onValueChange, onError, disabled = false, id, className, 'aria-invalid': ariaInvalid, 'aria-required': ariaRequired, 'aria-describedby': ariaDescribedby, 'aria-label': ariaLabel, ...rest }, ref) {
+    const internalRef = React.useRef(null);
+    React.useImperativeHandle(ref, () => internalRef.current);
+    const reactId = React.useId();
+    const inputId = id ?? reactId;
+    const errorId = `${inputId}-file-upload-error`;
+    const [dragActive, setDragActive] = React.useState(false);
+    const [errors, setErrors] = React.useState([]);
+    // The authoritative base for the next append. Props win on every render, so
+    // the host stays in control; the commit below also writes through, so a
+    // second batch that settles before the host has re-rendered still appends to
+    // the first batch's result instead of overwriting it.
+    const valueRef = React.useRef(value);
+    React.useEffect(() => {
+        valueRef.current = value;
+    });
+    // Superseding is NOT the contract here the way it is in the single-file
+    // sibling — batches accumulate, so an in-flight read is never stale. The
+    // readers are tracked for the one case that does have to stop them: an
+    // unmount. A read that lands afterwards would settle its batch and commit,
+    // calling the host's `onValueChange` for a component that no longer exists.
+    const readersRef = React.useRef(new Set());
+    React.useEffect(() => () => {
+        for (const reader of readersRef.current)
+            reader.abort();
+        readersRef.current.clear();
+    }, []);
+    const invalid = ariaInvalid || errors.length > 0;
+    // Resolve the announcements BEFORE deciding whether the alert exists: a
+    // consumer that returns nothing for every rejection is opting out of this
+    // surface, and an association pointing at an unrendered node is worse than
+    // none.
+    const messages = errors
+        .map((failure) => labels?.error
+        ? labels.error(failure)
+        : defaultMultipleErrorMessage(failure))
+        .filter((message) => Boolean(message));
+    // A plain join, never `cn`: tailwind-merge treats these as class names and
+    // would drop an id that happens to look like a conflicting utility.
+    const describedBy = [ariaDescribedby, messages.length > 0 ? errorId : undefined]
+        .filter(Boolean)
+        .join(' ') || undefined;
+    const full = maxFiles !== undefined && value.length >= maxFiles;
+    // Removing must stay possible at the cap, so only the PICKER closes.
+    const pickerDisabled = disabled || full;
+    const roomFor = (selected) => maxFiles === undefined
+        ? Number.POSITIVE_INFINITY
+        : Math.max(maxFiles - selected, 0);
+    // The cap is enforced HERE, at the only place that appends, because only the
+    // commit knows the base it lands on. `ingest` measures room too, but for an
+    // asynchronous batch it measures it BEFORE any read settles: two overlapping
+    // batches both see the pre-commit selection and would each believe they fit.
+    const commit = (accepted, rejections) => {
+        const room = roomFor(valueRef.current.length);
+        const fitting = accepted.slice(0, room);
+        const overflow = accepted[room];
+        const failures = overflow !== undefined && maxFiles !== undefined
+            ? [
+                ...rejections,
+                { kind: 'too-many', file: overflow.file, maxFiles }
+            ]
+            : rejections;
+        setErrors(failures);
+        for (const rejection of failures)
+            onError?.(rejection);
+        if (fitting.length === 0)
+            return;
+        const next = [...valueRef.current, ...fitting];
+        valueRef.current = next;
+        onValueChange(next);
+    };
+    const ingest = (incoming) => {
+        if (incoming.length === 0)
+            return;
+        const room = roomFor(valueRef.current.length);
+        const rejections = [];
+        // Validate EVERY file BEFORE the cap is applied. Slicing to the remaining
+        // room first would let a file that was never eligible consume a slot a
+        // good file could have used — one bad pick costing a good one, which is
+        // the opposite of a batch surviving its own casualties — and it would
+        // leave every file past the slice window neither validated nor reported.
+        // Validation is pure metadata (size and accept), so running it over files
+        // that may not fit costs nothing.
+        const eligible = [];
+        for (const file of incoming) {
+            const rejection = validateFile(file, { accept, maxSizeBytes });
+            // Continue rather than abort: one bad file must not cost the good ones.
+            if (rejection) {
+                rejections.push(rejection);
+                continue;
+            }
+            eligible.push(file);
+        }
+        // The cap then applies to the SURVIVORS. One rejection for the batch,
+        // naming the FIRST eligible file that did not fit: naming one already
+        // refused for its size or type would blame the cap for the wrong thing,
+        // and repeating it per overflowing file buries the actionable part.
+        if (eligible.length > room && maxFiles !== undefined) {
+            rejections.push({ kind: 'too-many', file: eligible[room], maxFiles });
+        }
+        const candidates = eligible.slice(0, room);
+        // Binary path: no decode, no reader, no retained garbage string.
+        if (readAs === 'none') {
+            commit(candidates.map((file) => ({ file, text: '' })), rejections);
+            return;
+        }
+        if (candidates.length === 0) {
+            commit([], rejections);
+            return;
+        }
+        // Slot-per-candidate so the emitted batch keeps PICK order regardless of
+        // the order the reads settle in. A null slot is a read that failed; the
+        // batch commits once every read has settled, one way or the other.
+        const slots = new Array(candidates.length).fill(null);
+        let remaining = candidates.length;
+        const settle = () => {
+            remaining -= 1;
+            if (remaining > 0)
+                return;
+            const accepted = [];
+            const readFailures = [];
+            slots.forEach((slot, index) => {
+                if (slot)
+                    accepted.push(slot);
+                else
+                    readFailures.push({ kind: 'read-failed', file: candidates[index] });
+            });
+            commit(accepted, [...rejections, ...readFailures]);
+        };
+        candidates.forEach((file, index) => {
+            const reader = new FileReader();
+            readersRef.current.add(reader);
+            // Deliberately no `onabort` handler: an aborted read must NOT settle,
+            // or the batch would commit at exactly the moment we are stopping it.
+            reader.onload = () => {
+                readersRef.current.delete(reader);
+                slots[index] = { file, text: String(reader.result ?? '') };
+                settle();
+            };
+            reader.onerror = () => {
+                readersRef.current.delete(reader);
+                settle();
+            };
+            reader.readAsText(file);
+        });
+    };
+    const onInputChange = (event) => {
+        const files = Array.from(event.target.files ?? []);
+        // Release the FileList the moment it has been read, or re-picking the same
+        // file is silently a no-op: the browser fires `change` only when the
+        // selection DIFFERS from what the input already holds.
+        event.target.value = '';
+        ingest(files);
+    };
+    const onDragOver = (event) => {
+        if (disabled)
+            return;
+        event.preventDefault();
+        setDragActive(true);
+    };
+    const onDragLeave = (event) => {
+        event.preventDefault();
+        setDragActive(false);
+    };
+    const onDrop = (event) => {
+        if (disabled)
+            return;
+        event.preventDefault();
+        setDragActive(false);
+        // Deliberately NOT gated on `full`: a drop onto a full zone is answered
+        // with the too-many rejection, which says why, instead of nothing at all.
+        ingest(Array.from(event.dataTransfer.files ?? []));
+    };
+    // Mouse convenience only. A click that ORIGINATED on the input already opens
+    // the picker natively and bubbles up here, so ignore it or it opens twice.
+    const openPicker = (event) => {
+        if (pickerDisabled || event.target === internalRef.current)
+            return;
+        internalRef.current?.click();
+    };
+    const removeAt = (index) => {
+        // By index, not by name: two files can share a filename and identity is
+        // what the row actually stands for.
+        const next = valueRef.current.filter((_, position) => position !== index);
+        valueRef.current = next;
+        onValueChange(next);
+    };
+    return ((0, jsx_runtime_1.jsxs)("div", { className: (0, utils_1.cn)('space-y-2', className), children: [(0, jsx_runtime_1.jsxs)("div", { onClick: openPicker, onDragOver: onDragOver, onDragLeave: onDragLeave, onDrop: onDrop, className: (0, utils_1.cn)('border-input bg-card focus-within:ring-ring focus-within:ring-offset-background aria-[invalid=true]:border-destructive aria-[invalid=true]:focus-within:ring-destructive flex w-full items-center gap-3 rounded-md border px-3 py-4 text-sm shadow-xs transition-colors focus-within:ring-2 focus-within:ring-offset-1 focus-within:outline-none', dragActive &&
+                    'border-ring ring-ring ring-offset-background ring-2 ring-offset-1', pickerDisabled
+                    ? 'border-muted bg-muted/30 cursor-not-allowed shadow-none'
+                    : 'cursor-pointer'), "aria-invalid": invalid || undefined, children: [(0, jsx_runtime_1.jsx)("input", { ref: internalRef, id: inputId, type: "file", multiple: true, accept: accept, disabled: pickerDisabled, className: "sr-only", "aria-invalid": invalid || undefined, "aria-required": ariaRequired || undefined, "aria-describedby": describedBy, "aria-label": ariaLabel, onChange: onInputChange, ...rest }), (0, jsx_runtime_1.jsx)(lucide_react_1.Upload, { className: "text-muted-foreground size-4 shrink-0", "aria-hidden": "true" }), full && maxFiles !== undefined ? ((0, jsx_runtime_1.jsx)("span", { className: "text-muted-foreground", children: labels?.full
+                            ? labels.full(maxFiles)
+                            : defaultFullMessage(maxFiles) })) : ((0, jsx_runtime_1.jsxs)("span", { className: "text-muted-foreground", children: [(0, jsx_runtime_1.jsx)("span", { className: "text-foreground font-medium", children: labels?.action ?? 'Choose files' }), ' ', labels?.hint ?? 'or drag and drop'] }))] }), value.length > 0 ? ((0, jsx_runtime_1.jsx)("ul", { className: "space-y-1", children: value.map((entry, index) => ((0, jsx_runtime_1.jsxs)("li", { className: "flex items-center gap-3 text-sm", children: [(0, jsx_runtime_1.jsxs)("span", { className: "min-w-0 flex-1 truncate", children: [(0, jsx_runtime_1.jsx)("span", { className: "text-foreground font-medium", children: entry.file.name }), ' ', (0, jsx_runtime_1.jsx)("span", { className: "text-muted-foreground tabular-nums", children: humanizeSize(entry.file.size) })] }), (0, jsx_runtime_1.jsx)(icon_button_1.IconButton, { type: "button", variant: "plain", size: "small", disabled: disabled, "aria-label": labels?.remove
+                                ? labels.remove(entry.file)
+                                : `Remove ${entry.file.name}`, onClick: () => removeAt(index), children: (0, jsx_runtime_1.jsx)(lucide_react_1.X, { className: "size-4", "aria-hidden": "true" }) })] }, `${entry.file.name}-${index}`))) })) : null, messages.length > 0 ? ((0, jsx_runtime_1.jsx)("div", { id: errorId, role: "alert", className: "space-y-1", children: messages.map((message, index) => ((0, jsx_runtime_1.jsx)("p", { className: "text-system-error-h1a text-xs font-medium", children: message }, index))) })) : null] }));
+});
+exports.MultipleFileUpload.displayName = 'MultipleFileUpload';

package/dist/components/ui/form.d.ts

@@ -16,7 +16,11 @@ declare const useFormField: () => {
     formDescriptionId: string;
     formMessageId: string;
     required: boolean | undefined;
+    described: Record<DescribingSlot, boolean> | undefined;
+    onDescribedChange: ((slot: DescribingSlot, rendered: boolean) => void) | undefined;
 };
+/** The two slots that can describe a control, in the order a reader hears them. */
+type DescribingSlot = 'message' | 'description';
 export type FormItemProps = React.HTMLAttributes<HTMLDivElement> & {
     required?: boolean;
 };

package/dist/components/ui/form.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"form.d.ts","sourceRoot":"","sources":["../../../src/components/ui/form.tsx"],"names":[],"mappings":"AAEA,OAAO,KAAK,KAAK,MAAM,OAAO,CAAA;AAC9B,OAAO,KAAK,cAAc,MAAM,uBAAuB,CAAA;AAEvD,OAAO,EAEL,eAAe,EACf,SAAS,EACT,WAAW,EAGZ,MAAM,iBAAiB,CAAA;AAKxB,OAAO,EAAE,oBAAoB,EAAE,MAAM,yBAAyB,CAAA;AAS9D,QAAA,MAAM,IAAI,4MAAe,CAAA;AAazB,QAAA,MAAM,SAAS,GACb,YAAY,SAAS,WAAW,GAAG,WAAW,EAC9C,KAAK,SAAS,SAAS,CAAC,YAAY,CAAC,GAAG,SAAS,CAAC,YAAY,CAAC,EAC/D,cAEC,eAAe,CAAC,YAAY,EAAE,KAAK,CAAC,sBAMtC,CAAA;AAED,QAAA,MAAM,YAAY;;;;;;;;;;;;CAyBjB,CAAA;AAWD,MAAM,MAAM,aAAa,GAAG,KAAK,CAAC,cAAc,CAAC,cAAc,CAAC,GAAG;IACjE,QAAQ,CAAC,EAAE,OAAO,CAAA;CACnB,CAAA;AAED,QAAA,MAAM,QAAQ;eAHD,OAAO;wCAanB,CAAA;AAGD,MAAM,MAAM,cAAc,GAAG,KAAK,CAAC,wBAAwB,CACzD,OAAO,cAAc,CAAC,IAAI,CAC3B,GAAG;IACF,KAAK,CAAC,EAAE,KAAK,CAAC,SAAS,CAAA;CACxB,CAAA;AAED,QAAA,MAAM,SAAS;YAHL,KAAK,CAAC,SAAS;0CA6BvB,CAAA;AAGF,eAAO,MAAM,WAAW,GAAI,yBAAyB,oBAAoB,sBASxE,CAAA;AAED,QAAA,MAAM,WAAW;;gFAmBf,CAAA;AAGF,QAAA,MAAM,eAAe,yHAcnB,CAAA;AAGF,QAAA,MAAM,WAAW,yHAuBf,CAAA;AAGF,OAAO,EACL,YAAY,EACZ,IAAI,EACJ,QAAQ,EACR,SAAS,EACT,WAAW,EACX,eAAe,EACf,WAAW,EACX,SAAS,EACV,CAAA"}
+{"version":3,"file":"form.d.ts","sourceRoot":"","sources":["../../../src/components/ui/form.tsx"],"names":[],"mappings":"AAEA,OAAO,KAAK,KAAK,MAAM,OAAO,CAAA;AAC9B,OAAO,KAAK,cAAc,MAAM,uBAAuB,CAAA;AAEvD,OAAO,EAEL,eAAe,EACf,SAAS,EACT,WAAW,EAGZ,MAAM,iBAAiB,CAAA;AAKxB,OAAO,EAAE,oBAAoB,EAAE,MAAM,yBAAyB,CAAA;AAS9D,QAAA,MAAM,IAAI,4MAAe,CAAA;AAazB,QAAA,MAAM,SAAS,GACb,YAAY,SAAS,WAAW,GAAG,WAAW,EAC9C,KAAK,SAAS,SAAS,CAAC,YAAY,CAAC,GAAG,SAAS,CAAC,YAAY,CAAC,EAC/D,cAEC,eAAe,CAAC,YAAY,EAAE,KAAK,CAAC,sBAMtC,CAAA;AAED,QAAA,MAAM,YAAY;;;;;;;;;;;;;+BA4CW,cAAc,YAAY,OAAO,KAAK,IAAI;CAjBtE,CAAA;AAED,mFAAmF;AACnF,KAAK,cAAc,GAAG,SAAS,GAAG,aAAa,CAAA;AAwC/C,MAAM,MAAM,aAAa,GAAG,KAAK,CAAC,cAAc,CAAC,cAAc,CAAC,GAAG;IACjE,QAAQ,CAAC,EAAE,OAAO,CAAA;CACnB,CAAA;AAED,QAAA,MAAM,QAAQ;eAHD,OAAO;wCA8BnB,CAAA;AAGD,MAAM,MAAM,cAAc,GAAG,KAAK,CAAC,wBAAwB,CACzD,OAAO,cAAc,CAAC,IAAI,CAC3B,GAAG;IACF,KAAK,CAAC,EAAE,KAAK,CAAC,SAAS,CAAA;CACxB,CAAA;AAED,QAAA,MAAM,SAAS;YAHL,KAAK,CAAC,SAAS;0CA6BvB,CAAA;AAGF,eAAO,MAAM,WAAW,GAAI,yBAAyB,oBAAoB,sBASxE,CAAA;AAED,QAAA,MAAM,WAAW;;gFAgCf,CAAA;AAGF,QAAA,MAAM,eAAe,yHAgBnB,CAAA;AAGF,QAAA,MAAM,WAAW,yHA6Bf,CAAA;AAGF,OAAO,EACL,YAAY,EACZ,IAAI,EACJ,QAAQ,EACR,SAAS,EACT,WAAW,EACX,eAAe,EACf,WAAW,EACX,SAAS,EACV,CAAA"}

package/dist/components/ui/form.js

@@ -61,7 +61,7 @@ const useFormField = () => {
     const fieldState = form && fieldContext.name
         ? form.getFieldState(fieldContext.name, form.formState)
         : undefined;
-    const { id, required } = itemContext;
+    const { id, required, described, onDescribedChange } = itemContext;
     return {
         id,
         name: fieldContext.name,
@@ -69,14 +69,38 @@ const useFormField = () => {
         formDescriptionId: `${id}-form-item-description`,
         formMessageId: `${id}-form-item-message`,
         required,
+        described,
+        onDescribedChange,
         ...fieldState
     };
 };
 exports.useFormField = useFormField;
 const FormItemContext = React.createContext({});
+/**
+ * Announce that this slot is on the page, for as long as it is.
+ *
+ * `rendered` is a parameter rather than a caller-side condition because
+ * `FormMessage` returns null when it has nothing to say, and a hook cannot sit
+ * behind that early return.
+ */
+function useDescribingSlot(slot, rendered) {
+    const { onDescribedChange } = React.useContext(FormItemContext);
+    React.useEffect(() => {
+        if (!rendered)
+            return;
+        onDescribedChange?.(slot, true);
+        return () => onDescribedChange?.(slot, false);
+    }, [onDescribedChange, slot, rendered]);
+}
 const FormItem = React.forwardRef(({ className, required, ...props }, ref) => {
     const id = React.useId();
-    return ((0, jsx_runtime_1.jsx)(FormItemContext.Provider, { value: { id, required }, children: (0, jsx_runtime_1.jsx)("div", { ref: ref, className: (0, utils_1.cn)('space-y-2', className), ...props }) }));
+    const [described, setDescribed] = React.useState({ message: false, description: false });
+    // Keeps the object identity when nothing moved, so a slot re-registering
+    // does not re-render the item for no reason.
+    const onDescribedChange = React.useCallback((slot, rendered) => setDescribed((current) => current[slot] === rendered
+        ? current
+        : { ...current, [slot]: rendered }), []);
+    return ((0, jsx_runtime_1.jsx)(FormItemContext.Provider, { value: { id, required, described, onDescribedChange }, children: (0, jsx_runtime_1.jsx)("div", { ref: ref, className: (0, utils_1.cn)('space-y-2', className), ...props }) }));
 });
 exports.FormItem = FormItem;
 FormItem.displayName = 'FormItem';
@@ -92,15 +116,28 @@ FormLabel.displayName = 'FormLabel';
 const FormTooltip = ({ children, ...others }) => ((0, jsx_runtime_1.jsx)(tooltip_1.TooltipProvider, { ...others, children: (0, jsx_runtime_1.jsxs)(tooltip_1.Tooltip, { children: [(0, jsx_runtime_1.jsx)(tooltip_1.TooltipTrigger, { asChild: true, children: (0, jsx_runtime_1.jsx)(lucide_react_1.HelpCircle, { className: "text-container-text ml-2 h-4 w-4" }) }), (0, jsx_runtime_1.jsx)(tooltip_1.TooltipContent, { children: children })] }) }));
 exports.FormTooltip = FormTooltip;
 const FormControl = React.forwardRef(({ ...props }, ref) => {
-    const { error, formItemId, formDescriptionId, formMessageId } = useFormField();
-    return ((0, jsx_runtime_1.jsx)(react_slot_1.Slot, { ref: ref, id: formItemId, "aria-describedby": !error
-            ? `${formDescriptionId}`
-            : `${formDescriptionId} ${formMessageId}`, "aria-invalid": !!error, ...props }));
+    const { error, formItemId, formDescriptionId, formMessageId, described } = useFormField();
+    // Only ids that resolve. A screen reader drops a dangling IDREF silently, so
+    // the barrier this removes is diagnostic rather than operational: a form with
+    // thirteen broken associations is a form where a real one cannot be spotted.
+    //
+    // Message first: a reader hears the error before the hint. `undefined` when
+    // both slots are empty, which drops the attribute — and, because Radix Slot
+    // OVERWRITES rather than merges, still lets the wrapped element carry its own
+    // (`ui/file-upload` merges the injected id with its own error id that way).
+    const describedBy = [
+        described?.message ? formMessageId : undefined,
+        described?.description ? formDescriptionId : undefined
+    ]
+        .filter(Boolean)
+        .join(' ') || undefined;
+    return ((0, jsx_runtime_1.jsx)(react_slot_1.Slot, { ref: ref, id: formItemId, "aria-describedby": describedBy, "aria-invalid": !!error, ...props }));
 });
 exports.FormControl = FormControl;
 FormControl.displayName = 'FormControl';
 const FormDescription = React.forwardRef(({ className, ...props }, ref) => {
     const { formDescriptionId } = useFormField();
+    useDescribingSlot('description', true);
     return ((0, jsx_runtime_1.jsx)("p", { ref: ref, id: formDescriptionId, className: (0, utils_1.cn)('text-muted-foreground text-xs font-medium', className), ...props }));
 });
 exports.FormDescription = FormDescription;
@@ -108,6 +145,11 @@ FormDescription.displayName = 'FormDescription';
 const FormMessage = React.forwardRef(({ className, children, ...props }, ref) => {
     const { error, formMessageId } = useFormField();
     const body = error ? String(error?.message) : children;
+    // Registered on `body`, not on `error`: a message given plain children
+    // renders copy that the control has to point at too. The old expression only
+    // added the message id when react-hook-form reported an error, so a
+    // standalone message was rendered and never associated.
+    useDescribingSlot('message', Boolean(body));
     if (!body) {
         return null;
     }

package/dist/components/ui/toggle-group/index.d.ts

@@ -13,6 +13,6 @@ declare const toggleVariants: (props?: ({
 declare function ToggleGroup({ className, variant, size, spacing, children, style, ...props }: React.ComponentProps<typeof ToggleGroupPrimitive.Root> & VariantProps<typeof toggleVariants> & {
     spacing?: number;
 }): React.JSX.Element;
-declare function ToggleGroupItem({ className, children, variant, size, ...props }: React.ComponentProps<typeof ToggleGroupPrimitive.Item> & VariantProps<typeof toggleVariants>): React.JSX.Element;
+declare function ToggleGroupItem({ className, children, variant, size, onFocus, ...props }: React.ComponentProps<typeof ToggleGroupPrimitive.Item> & VariantProps<typeof toggleVariants>): React.JSX.Element;
 export { ToggleGroup, ToggleGroupItem };
 //# sourceMappingURL=index.d.ts.map

package/dist/components/ui/toggle-group/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../src/components/ui/toggle-group/index.tsx"],"names":[],"mappings":"AAEA,OAAO,KAAK,KAAK,MAAM,OAAO,CAAA;AAC9B,OAAO,KAAK,oBAAoB,MAAM,8BAA8B,CAAA;AACpE,OAAO,EAAO,KAAK,YAAY,EAAE,MAAM,0BAA0B,CAAA;AAIjE;;;;GAIG;AACH,QAAA,MAAM,cAAc;;;8EAoBnB,CAAA;AAYD,iBAAS,WAAW,CAAC,EACnB,SAAS,EACT,OAAO,EACP,IAAI,EACJ,OAAW,EACX,QAAQ,EACR,KAAK,EACL,GAAG,KAAK,EACT,EAAE,KAAK,CAAC,cAAc,CAAC,OAAO,oBAAoB,CAAC,IAAI,CAAC,GACvD,YAAY,CAAC,OAAO,cAAc,CAAC,GAAG;IACpC,OAAO,CAAC,EAAE,MAAM,CAAA;CACjB,qBAwBF;AAED,iBAAS,eAAe,CAAC,EACvB,SAAS,EACT,QAAQ,EACR,OAAO,EACP,IAAI,EACJ,GAAG,KAAK,EACT,EAAE,KAAK,CAAC,cAAc,CAAC,OAAO,oBAAoB,CAAC,IAAI,CAAC,GACvD,YAAY,CAAC,OAAO,cAAc,CAAC,qBAuBpC;AAED,OAAO,EAAE,WAAW,EAAE,eAAe,EAAE,CAAA"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../src/components/ui/toggle-group/index.tsx"],"names":[],"mappings":"AAEA,OAAO,KAAK,KAAK,MAAM,OAAO,CAAA;AAC9B,OAAO,KAAK,oBAAoB,MAAM,8BAA8B,CAAA;AACpE,OAAO,EAAO,KAAK,YAAY,EAAE,MAAM,0BAA0B,CAAA;AAIjE;;;;GAIG;AACH,QAAA,MAAM,cAAc;;;8EAoBnB,CAAA;AA4DD,iBAAS,WAAW,CAAC,EACnB,SAAS,EACT,OAAO,EACP,IAAI,EACJ,OAAW,EACX,QAAQ,EACR,KAAK,EACL,GAAG,KAAK,EACT,EAAE,KAAK,CAAC,cAAc,CAAC,OAAO,oBAAoB,CAAC,IAAI,CAAC,GACvD,YAAY,CAAC,OAAO,cAAc,CAAC,GAAG;IACpC,OAAO,CAAC,EAAE,MAAM,CAAA;CACjB,qBA+BF;AAED,iBAAS,eAAe,CAAC,EACvB,SAAS,EACT,QAAQ,EACR,OAAO,EACP,IAAI,EACJ,OAAO,EACP,GAAG,KAAK,EACT,EAAE,KAAK,CAAC,cAAc,CAAC,OAAO,oBAAoB,CAAC,IAAI,CAAC,GACvD,YAAY,CAAC,OAAO,cAAc,CAAC,qBAiDpC;AAED,OAAO,EAAE,WAAW,EAAE,eAAe,EAAE,CAAA"}

package/dist/components/ui/toggle-group/index.js

@@ -63,12 +63,58 @@ const toggleVariants = (0, class_variance_authority_1.cva)("hover:bg-muted hover
         size: 'default'
     }
 });
+const ARROW_KEYS = ['ArrowUp', 'ArrowDown', 'ArrowLeft', 'ArrowRight'];
+/**
+ * Is an arrow key down right now?
+ *
+ * This is how an item tells keyboard navigation apart from a pointer press or a
+ * programmatic focus, and it is the seam Radix uses for the same job in its own
+ * `RadioGroup`. It works because `RovingFocusGroup` defers the focus move to a
+ * `setTimeout`: the keydown has already bubbled to `document` and set the flag
+ * by the time the item receives `focus`.
+ *
+ * The distinction is not cosmetic. A pointer press focuses BEFORE the click
+ * lands, so selecting on every focus would turn one click into
+ * select-then-deselect and break mouse selection outright.
+ *
+ * Returns null when disabled, so `type="multiple"` carries no flag to read.
+ */
+function useArrowKeyPressed(enabled) {
+    const arrowKeyPressed = React.useRef(false);
+    React.useEffect(() => {
+        if (!enabled)
+            return;
+        const onKeyDown = (event) => {
+            if (ARROW_KEYS.includes(event.key))
+                arrowKeyPressed.current = true;
+        };
+        const onKeyUp = () => {
+            arrowKeyPressed.current = false;
+        };
+        document.addEventListener('keydown', onKeyDown);
+        document.addEventListener('keyup', onKeyUp);
+        window.addEventListener('blur', onKeyUp);
+        return () => {
+            document.removeEventListener('keydown', onKeyDown);
+            document.removeEventListener('keyup', onKeyUp);
+            window.removeEventListener('blur', onKeyUp);
+            // A group unmounted mid-keystroke never sees the keyup.
+            arrowKeyPressed.current = false;
+        };
+    }, [enabled]);
+    return enabled ? arrowKeyPressed : null;
+}
 const ToggleGroupContext = React.createContext({
     size: 'default',
     variant: 'default',
-    spacing: 0
+    spacing: 0,
+    arrowKeyPressed: null
 });
 function ToggleGroup({ className, variant, size, spacing = 0, children, style, ...props }) {
+    // Read, never destructured: `type` is the discriminant of the Radix props
+    // union, and pulling it out then spreading it back widens the union so the
+    // single/multiple props stop resolving.
+    const arrowKeyPressed = useArrowKeyPressed(props.type === 'single');
     return ((0, jsx_runtime_1.jsx)(ToggleGroupPrimitive.Root, { "data-slot": "toggle-group", "data-variant": variant, "data-size": size, "data-spacing": spacing, 
         // Caller style first: `--gap` is ours and must survive, but everything
         // else the caller passes has to reach the element.
@@ -76,12 +122,35 @@ function ToggleGroup({ className, variant, size, spacing = 0, children, style, .
         // Legacy carried a `data-[spacing=default]:...:shadow-xs` rule here that
         // never matched — `data-spacing` is always a number. Removed rather than
         // activated, so the rendered pixels stay identical to sindarian-x.
-        'group/toggle-group flex w-fit items-center gap-[--spacing(var(--gap))] rounded-md', className), ...props, children: (0, jsx_runtime_1.jsx)(ToggleGroupContext.Provider, { value: { variant, size, spacing }, children: children }) }));
+        'group/toggle-group flex w-fit items-center gap-[--spacing(var(--gap))] rounded-md', className), ...props, children: (0, jsx_runtime_1.jsx)(ToggleGroupContext.Provider, { value: { variant, size, spacing, arrowKeyPressed }, children: children }) }));
 }
-function ToggleGroupItem({ className, children, variant, size, ...props }) {
+function ToggleGroupItem({ className, children, variant, size, onFocus, ...props }) {
     const context = React.useContext(ToggleGroupContext);
+    /**
+     * Selection follows focus, which is what `role="radio"` promises.
+     *
+     * With `type="single"` Radix puts the group on the ARIA radio pattern: the
+     * root is a `radiogroup`, each item a `radio` with `aria-checked`. Under that
+     * pattern an arrow key both moves focus and checks the item it lands on.
+     * Radix only moves focus, so a screen-reader operator used to arrow across
+     * "radio, not checked, 2 of 3" with nothing ever selected.
+     *
+     * `type="multiple"` is a `toolbar` of `aria-pressed` buttons and keeps its
+     * current behavior, where moving focus without pressing is correct: the flag
+     * is null there.
+     *
+     * Clicking the element rather than writing the value keeps a single
+     * activation path, so Radix's own toggle and any caller `onClick` both fire
+     * exactly as they do for a real click.
+     */
+    const handleFocus = (event) => {
+        onFocus?.(event);
+        if (context.arrowKeyPressed?.current) {
+            event.currentTarget.click();
+        }
+    };
     return ((0, jsx_runtime_1.jsx)(ToggleGroupPrimitive.Item, { "data-slot": "toggle-group-item", "data-variant": context.variant || variant, "data-size": context.size || size, "data-spacing": context.spacing, className: (0, utils_1.cn)(toggleVariants({
             variant: context.variant || variant,
             size: context.size || size
-        }), 'w-auto min-w-0 shrink-0 px-3 focus:z-10 focus-visible:z-10', 'data-[spacing=0]:rounded-none data-[spacing=0]:shadow-none data-[spacing=0]:first:rounded-l-md data-[spacing=0]:last:rounded-r-md data-[spacing=0]:data-[variant=outline]:border-l-0 data-[spacing=0]:data-[variant=outline]:first:border-l', className), ...props, children: children }));
+        }), 'w-auto min-w-0 shrink-0 px-3 focus:z-10 focus-visible:z-10', 'data-[spacing=0]:rounded-none data-[spacing=0]:shadow-none data-[spacing=0]:first:rounded-l-md data-[spacing=0]:last:rounded-r-md data-[spacing=0]:data-[variant=outline]:border-l-0 data-[spacing=0]:data-[variant=outline]:first:border-l', className), ...props, onFocus: handleFocus, children: children }));
 }