@mohasinac/appkit 4.30.18 → 4.31.0

package/dist/features/layout/BottomActions.js

@@ -33,7 +33,7 @@ import { useState, useRef, useEffect, useId } from "react";
 import { X, ChevronUp, ChevronDown, Check } from "lucide-react";
 import { useBottomActionsContext } from "./BottomActionsContext";
 import { useClickOutside } from "../../react";
-import { Div, Row, Span, Text, Button } from "../../ui";
+import { ActionRow, Div, Row, Span, Text, Button } from "../../ui";
 // Token values inlined from @mohasinac/appkit/tokens
 const BOTTOM_NAV_BG = "bg-[color-mix(in_srgb,var(--appkit-color-bg)_90%,transparent)] backdrop-blur-md border-t border-[var(--appkit-color-border)]";
 const BOTTOM_NAV_HEIGHT = "h-14";
@@ -88,21 +88,94 @@ function InfoPanel({ id, open, children, }) {
             .filter(Boolean)
             .join(" "), "aria-hidden": !open, children: _jsx(Div, { className: "max-h-[60vh]", overflow: "y-auto", padding: "md", children: children }) }));
 }
-function PageActionsRow({ pageActions, dispatchAction, }) {
-    return (_jsx(_Fragment, { children: pageActions.map((action) => {
+/**
+ * How many actions fit on one line before the row splits in two.
+ *
+ * 🛑 ROW COUNT is a function of `pageActions.length` and NOTHING else — never
+ * of validation state, and never of measured width. `BottomChrome`'s
+ * ResizeObserver faithfully republishes `--bottom-chrome-height` on any height
+ * change, and every consumer (BackToTop, the pagination bar, the footer's
+ * clearance) reflows with it. A bar that grew a row when an error appeared
+ * would shift the whole page while the user was typing. The error count lives
+ * in `infoLabel`, which sits in the absolutely-positioned panel and is excluded
+ * from the published height by construction.
+ *
+ * The invariant is about ROW COUNT, not pixel height. `ActionRow` lets a long
+ * label wrap to a second line, which changes the bar's height by ~16px,
+ * monotonically, and the observer handles that correctly. What must never
+ * happen is a row count that FLIPS: that is a ~48px step and it is reversible
+ * mid-interaction.
+ *
+ * Which is exactly why the count split stays primary and `flex-wrap` is only an
+ * overflow net. In this app LABELS ARE STATE — "Add to Cart" becomes "Adding…",
+ * "Buy Now" becomes "Out of Stock", "Apply" becomes "Apply (3)". If width alone
+ * decided rows, tapping Add to Cart would shorten the label, free ~40px,
+ * un-wrap a row and shift the page under the user's thumb at the exact moment
+ * they committed to the tap.
+ *
+ * 🛑 This constant is COUPLED to `--appkit-size-action-min` (tokens.css).
+ * Two actions per line is safe only while `2 × action-min + gap` fits the
+ * narrowest supported viewport (280px). Raising either without re-checking that
+ * sum reintroduces device-dependent row flipping.
+ */
+const SINGLE_ROW_MAX_ACTIONS = 2;
+function ActionButtons({ actions, dispatchAction, }) {
+    return (_jsx(_Fragment, { children: actions.map((action) => {
             const isIconOnly = !action.label;
-            const growClass = isIconOnly || action.grow === false
-                ? "flex-shrink-0 w-11"
-                : "flex-1 min-w-0";
+            /*
+             * Width is ActionRow's job now, not this component's — every action is
+             * sized from its own label by `.appkit-action-row__group > *`, and only
+             * the leftover is shared out. There is deliberately NO per-action width
+             * class here: the old `grow` ternary is what produced a 44px stub
+             * reading "Ba" next to a full-width "Continue to payment".
+             *
+             * The one thing a stylesheet cannot infer is that a child has no label,
+             * so icon-only opts into its square tap target explicitly.
+             *
+             * 🛑 Never add `flex-1`, `flex-shrink-0`, `basis-*`, `w-*`, `h-*`,
+             * `truncate` or `leading-none` below. Both tailwind configs set
+             * `important: true`, so any of them beats ActionRow's CSS and silently
+             * reverts this bar to the squeezed behaviour with no visible error.
+             */
             return (_jsxs(Button, { type: "button", variant: action.variant ?? "primary", size: "sm", isLoading: action.loading, disabled: action.disabled, onClick: () => dispatchAction(action.id), className: [
-                    "h-10 relative",
-                    growClass,
-                    isIconOnly ? "px-[var(--appkit-space-0)] justify-center" : "",
+                    "relative",
+                    isIconOnly ? "appkit-action-row__icon-only justify-center" : "",
                 ]
                     .filter(Boolean)
-                    .join(" "), children: [action.icon && (_jsx(Span, { className: ["flex-shrink-0", action.label ? "mr-1.5" : ""].join(" "), "aria-hidden": "true", children: action.icon })), action.label && _jsx(Span, { className: "truncate leading-none", children: action.label }), action.badge !== undefined && (_jsx(Span, { className: `absolute -top-1.5 -right-1.5 min-w-[18px] h-[18px] ${CLS_COUNT_BADGE} text-[10px] ${FLEX_CENTER} px-[var(--appkit-space-1)] pointer-events-none select-none`, rounded: "full", weight: "bold", "aria-hidden": "true", children: action.badge }))] }, action.id));
+                    .join(" "), children: [action.icon && (_jsx(Span, { className: ["flex-shrink-0", action.label ? "mr-1.5" : ""].join(" "), "aria-hidden": "true", children: action.icon })), action.label && _jsx(Span, { children: action.label }), action.badge !== undefined && (_jsx(Span, { className: `absolute -top-1.5 -right-1.5 min-w-[18px] h-[18px] ${CLS_COUNT_BADGE} text-[10px] ${FLEX_CENTER} px-[var(--appkit-space-1)] pointer-events-none select-none`, rounded: "full", weight: "bold", "aria-hidden": "true", children: action.badge }))] }, action.id));
         }) }));
 }
+/**
+ * Lays a form's or a page's actions out across one or two lines.
+ *
+ * | n    | layout                                                          |
+ * |------|-----------------------------------------------------------------|
+ * | 1    | one row, full-width primary                                     |
+ * | 2    | one row, each sized from its own label                          |
+ * | 3+   | two rows — secondaries above, primary full-width below           |
+ *
+ * The LAST action is treated as the primary. That holds for every producer:
+ * `useFormBottomActions` appends submit last, and the checkout/cart bars build
+ * `[back, primary]`. Putting it on its own line keeps the thumb target the same
+ * size no matter how many secondary actions a screen has, which is the whole
+ * reason a 5-action form (3 forms + soft-ban + hard-ban on the admin user
+ * editor) was unrepresentable before.
+ *
+ * Secondaries WRAP rather than overflowing into a menu. Wrapping keeps the
+ * height a pure function of the action count — the invariant above — whereas a
+ * popover would add a second interaction to reach an action that is already
+ * only one line away. No current caller exceeds five.
+ */
+function PageActionsRow({ pageActions, dispatchAction, }) {
+    if (pageActions.length === 0)
+        return null;
+    if (pageActions.length <= SINGLE_ROW_MAX_ACTIONS) {
+        return (_jsx(ActionRow, { children: _jsx(ActionButtons, { actions: pageActions, dispatchAction: dispatchAction }) }));
+    }
+    const secondaries = pageActions.slice(0, -1);
+    const primary = pageActions[pageActions.length - 1];
+    return (_jsxs(Div, { className: "flex w-full flex-col gap-[var(--appkit-space-2)]", children: [_jsx(ActionRow, { children: _jsx(ActionButtons, { actions: secondaries, dispatchAction: dispatchAction }) }), _jsx(ActionRow, { children: _jsx(ActionButtons, { actions: [primary], dispatchAction: dispatchAction }) })] }));
+}
 export default function BottomActions() {
     const { state, actionCallbacksRef, bulkCallbacksRef, bulkClearRef } = useBottomActionsContext();
     const [pickerOpen, setPickerOpen] = useState(false);
@@ -204,7 +277,7 @@ export default function BottomActions() {
                             isVisible ? "" : "pointer-events-none",
                         ]
                             .filter(Boolean)
-                            .join(" "), children: [isBulkMode && (_jsx(Div, { className: "h-[3px] w-full bg-[image:var(--appkit-gradient-brand-tri)]" })), secondaryLabel && !isBulkMode && (_jsx(Div, { border: "subtle", className: "pt-[var(--appkit-space-2)] pb-[var(--appkit-space-0)] border-b /80", padding: "x-md", children: _jsx(Text, { className: "leading-5 truncate", color: "muted", size: "xs", weight: "semibold", children: secondaryLabel }) })), infoLabel && !isBulkMode && !hasInfoPanel && (_jsx(Div, { border: "subtle", className: "pt-[var(--appkit-space-2)] pb-[var(--appkit-space-0)] border-b /80", padding: "x-md", children: _jsx(Text, { className: "leading-5 truncate", color: "muted", size: "xs", weight: "semibold", children: infoLabel }) })), infoLabel && !isBulkMode && hasInfoPanel && (_jsxs(Button, { type: "button", variant: "ghost", onClick: () => setInfoOpen((o) => !o), "aria-expanded": infoOpen, "aria-controls": infoPanelId, className: "w-full flex items-center justify-between gap-[var(--appkit-space-2)] rounded-none border-b border-[var(--appkit-color-border)]/80 px-[var(--appkit-space-4)] pt-[var(--appkit-space-2)] pb-[var(--appkit-space-1)] min-h-0", children: [_jsx(Text, { className: "leading-5 truncate", color: "muted", size: "xs", weight: "semibold", children: infoLabel }), _jsxs(Span, { className: "flex-shrink-0 flex items-center gap-[var(--appkit-space-1)]", children: [_jsx(Text, { as: "span", size: "xs", color: "muted", children: infoOpen ? "Hide" : "Details" }), infoOpen ? (_jsx(ChevronDown, { className: "w-4 h-4 text-zinc-400", "aria-hidden": "true" })) : (_jsx(ChevronUp, { className: "w-4 h-4 text-zinc-400", "aria-hidden": "true" }))] })] })), _jsx(Row, { className: `${BOTTOM_NAV_HEIGHT}`, gap: "sm", padding: "x-sm", children: isBulkMode && bulk ? (_jsxs(_Fragment, { children: [_jsxs(Button, { rounded: "full", gap: "xs", type: "button", variant: "ghost", onClick: dispatchBulkClear, className: "inline- flex-shrink-0 bg-primary-50 hover:bg-primary-surface active:bg-primary-200 dark:bg-primary-950/30 text-primary-700 dark:text-primary-300 pl-2 pr-3 h-8 border border-primary-200/70 dark:border-primary-800/50 transition-colors min-h-0", "aria-label": "Clear selection", children: [_jsx(X, { className: "w-3.5 h-3.5 flex-shrink-0", "aria-hidden": "true" }), _jsx(Span, { size: "xs", weight: "semibold", className: "tabular-nums whitespace-nowrap leading-none", children: bulk.noun
+                            .join(" "), children: [isBulkMode && (_jsx(Div, { className: "h-[3px] w-full bg-[image:var(--appkit-gradient-brand-tri)]" })), secondaryLabel && !isBulkMode && (_jsx(Div, { className: "pt-[var(--appkit-space-2)] pb-[var(--appkit-space-0)] border-b border-[var(--appkit-color-border-subtle)]", padding: "x-md", children: _jsx(Text, { className: "leading-5 truncate", color: "muted", size: "xs", weight: "semibold", children: secondaryLabel }) })), infoLabel && !isBulkMode && !hasInfoPanel && (_jsx(Div, { className: "pt-[var(--appkit-space-2)] pb-[var(--appkit-space-0)] border-b border-[var(--appkit-color-border-subtle)]", padding: "x-md", children: _jsx(Text, { className: "leading-5 truncate", color: "muted", size: "xs", weight: "semibold", children: infoLabel }) })), infoLabel && !isBulkMode && hasInfoPanel && (_jsxs(Button, { type: "button", variant: "ghost", onClick: () => setInfoOpen((o) => !o), "aria-expanded": infoOpen, "aria-controls": infoPanelId, className: "w-full flex items-center justify-between gap-[var(--appkit-space-2)] rounded-none border-b border-[var(--appkit-color-border)]/80 px-[var(--appkit-space-4)] pt-[var(--appkit-space-2)] pb-[var(--appkit-space-1)] min-h-0", children: [_jsx(Text, { className: "leading-5 truncate", color: "muted", size: "xs", weight: "semibold", children: infoLabel }), _jsxs(Span, { className: "flex-shrink-0 flex items-center gap-[var(--appkit-space-1)]", children: [_jsx(Text, { as: "span", size: "xs", color: "muted", children: infoOpen ? "Hide" : "Details" }), infoOpen ? (_jsx(ChevronDown, { className: "w-4 h-4 text-zinc-400", "aria-hidden": "true" })) : (_jsx(ChevronUp, { className: "w-4 h-4 text-zinc-400", "aria-hidden": "true" }))] })] })), _jsx(Row, { className: `min-${BOTTOM_NAV_HEIGHT} py-[var(--appkit-space-2)]`, gap: "sm", padding: "x-sm", children: isBulkMode && bulk ? (_jsxs(_Fragment, { children: [_jsxs(Button, { rounded: "full", gap: "xs", type: "button", variant: "ghost", onClick: dispatchBulkClear, className: "inline- flex-shrink-0 bg-primary-50 hover:bg-primary-surface active:bg-primary-200 dark:bg-primary-950/30 text-primary-700 dark:text-primary-300 pl-2 pr-3 h-8 border border-primary-200/70 dark:border-primary-800/50 transition-colors min-h-0", "aria-label": "Clear selection", children: [_jsx(X, { className: "w-3.5 h-3.5 flex-shrink-0", "aria-hidden": "true" }), _jsx(Span, { size: "xs", weight: "semibold", className: "tabular-nums whitespace-nowrap leading-none", children: bulk.noun
                                                         ? `${bulk.selectedCount} ${bulk.noun}`
                                                         : `${bulk.selectedCount} selected` })] }), bulkActions.length > 0 && (_jsxs(Button, { type: "button", variant: "ghost", onClick: () => setPickerOpen((o) => !o), "aria-haspopup": "listbox", "aria-expanded": pickerOpen, className: [
                                                 "flex-1 min-w-0 h-10 flex items-center gap-[var(--appkit-space-2)] px-[var(--appkit-space-3)] rounded-lg border text-[length:var(--appkit-text-sm)] font-medium transition-colors",

package/dist/features/layout/BottomActionsContext.d.ts

@@ -14,7 +14,7 @@
  * ```tsx
  * useBottomActions({
  * actions: [
- * { id: "wishlist", icon: <Heart className="w-4 h-4" />, label: t("wishlist"), variant: "ghost", grow: false, onClick: handleWishlist },
+ * { id: "wishlist", icon: <Heart className="w-4 h-4" />, label: t("wishlist"), variant: "ghost", onClick: handleWishlist },
  * { id: "cart", label: t("addToCart"), variant: "outline", onClick: handleAddToCart },
  * { id: "buy", label: t("buyNow"), variant: "primary", onClick: handleBuyNow },
  * ],
@@ -53,11 +53,6 @@ export interface BottomAction {
     disabled?: boolean;
     /** Show spinner inside the button. */
     loading?: boolean;
-    /**
-     * When true (default) the button grows to fill remaining space.
-     * Set to false for compact icon-only buttons.
-     */
-    grow?: boolean;
 }
 export interface BottomBulkConfig {
     /** Number of currently selected items. Bar activates when > 0. */

package/dist/features/layout/BottomActionsContext.js

@@ -16,7 +16,7 @@ import { jsx as _jsx } from "react/jsx-runtime";
  * ```tsx
  * useBottomActions({
  * actions: [
- * { id: "wishlist", icon: <Heart className="w-4 h-4" />, label: t("wishlist"), variant: "ghost", grow: false, onClick: handleWishlist },
+ * { id: "wishlist", icon: <Heart className="w-4 h-4" />, label: t("wishlist"), variant: "ghost", onClick: handleWishlist },
  * { id: "cart", label: t("addToCart"), variant: "outline", onClick: handleAddToCart },
  * { id: "buy", label: t("buyNow"), variant: "primary", onClick: handleBuyNow },
  * ],

package/dist/features/layout/hooks/useBottomActions.js

@@ -28,10 +28,10 @@ export function useBottomActions(options = {}) {
     const bulkRef = useRef(options.bulk);
     bulkRef.current = options.bulk;
     const actionKey = (options.actions ?? [])
-        .map(({ id, label, variant, badge, disabled, loading, grow }) => `${id}|${label}|${variant}|${badge}|${disabled}|${loading}|${grow}`)
+        .map(({ id, label, variant, badge, disabled, loading }) => `${id}|${label}|${variant}|${badge}|${disabled}|${loading}`)
         .join(",");
     const bulkCountKey = options.bulk
-        ? `${options.bulk.selectedCount}|${options.bulk.actions.map((a) => `${a.id}|${a.label}|${a.variant}|${a.badge}|${a.disabled}|${a.loading}|${a.grow}`).join(",")}`
+        ? `${options.bulk.selectedCount}|${options.bulk.actions.map((a) => `${a.id}|${a.label}|${a.variant}|${a.badge}|${a.disabled}|${a.loading}`).join(",")}`
         : "";
     useEffect(() => {
         if (!isTop)

package/dist/features/layout/hooks/useFormBottomActions.d.ts

@@ -1,4 +1,5 @@
 import { type FormShellContextValue } from "../../../ui/forms/FormShell";
+import type { BottomAction } from "../BottomActionsContext";
 export interface UseFormBottomActionsOptions {
     /** Runs when the pinned primary button is pressed. */
     onSubmit: () => void | Promise<void>;
@@ -26,6 +27,16 @@ export interface UseFormBottomActionsOptions {
         onClick: () => void;
         disabled?: boolean;
     };
+    /**
+     * Further actions this form offers, spliced between the destructive one and
+     * Submit so the ordering rule survives: destructive furthest from the thumb,
+     * submit under it.
+     *
+     * Exists because the bar was hard-capped at three. `/admin/users` needs five
+     * (three forms plus soft-ban and hard-ban), so its extra actions had nowhere
+     * to go and stayed as an inline row the pinned bar then competed with.
+     */
+    extraActions?: BottomAction[];
     ctx?: FormShellContextValue;
 }
 /**
@@ -63,5 +74,5 @@ export interface UseFormBottomActionsOptions {
  *    overlay tells us, because neither portal leaves a detectable trace in
  *    the DOM at the form's position.
  */
-export declare function useFormBottomActions({ onSubmit, onCancel, submitLabel, cancelLabel, isLoading, disabled, enabled, destructiveAction, ctx: ctxProp, }: UseFormBottomActionsOptions): void;
+export declare function useFormBottomActions({ onSubmit, onCancel, submitLabel, cancelLabel, isLoading, disabled, enabled, destructiveAction, extraActions, ctx: ctxProp, }: UseFormBottomActionsOptions): void;
 export default useFormBottomActions;

package/dist/features/layout/hooks/useFormBottomActions.js

@@ -40,7 +40,7 @@ import { useBottomActions } from "./useBottomActions";
  *    overlay tells us, because neither portal leaves a detectable trace in
  *    the DOM at the form's position.
  */
-export function useFormBottomActions({ onSubmit, onCancel, submitLabel = "Save", cancelLabel = "Cancel", isLoading = false, disabled = false, enabled = true, destructiveAction, ctx: ctxProp, }) {
+export function useFormBottomActions({ onSubmit, onCancel, submitLabel = "Save", cancelLabel = "Cancel", isLoading = false, disabled = false, enabled = true, destructiveAction, extraActions, ctx: ctxProp, }) {
     const ambient = useContext(FormShellContext);
     const ctx = ctxProp ?? ambient;
     const insideOverlay = useIsInsideOverlay();
@@ -53,14 +53,18 @@ export function useFormBottomActions({ onSubmit, onCancel, submitLabel = "Save",
         const list = [];
         /*
          * Destructive FIRST, so it is furthest from the thumb's resting position
-         * on the primary action. It never grows, so Save keeps the width.
+         * on the primary action, which BottomActions renders last.
+         *
+         * None of these set a width. `grow` no longer exists on BottomAction —
+         * ActionRow sizes every action from its own label and shares only the
+         * leftover, so "Cancel" and "Save" each get a width that fits their text
+         * instead of Cancel collapsing to a 44px square while Save took the rest.
          */
         if (destructiveAction) {
             list.push({
                 id: "form-destructive",
                 label: destructiveAction.label,
                 variant: "danger",
-                grow: false,
                 onClick: destructiveAction.onClick,
                 disabled: destructiveAction.disabled || isLoading,
             });
@@ -70,22 +74,24 @@ export function useFormBottomActions({ onSubmit, onCancel, submitLabel = "Save",
                 id: "form-cancel",
                 label: cancelLabel,
                 variant: "outline",
-                grow: false,
                 onClick: onCancel,
                 disabled: isLoading,
             });
         }
+        if (extraActions?.length)
+            list.push(...extraActions);
+        // Submit LAST: BottomActions treats the final action as the primary and
+        // gives it its own full-width line once the bar holds three or more.
         list.push({
             id: "form-submit",
             label: submitLabel,
             variant: "primary",
-            grow: true,
             loading: isLoading,
             disabled: disabled || isLoading,
             onClick: () => void onSubmit(),
         });
         return list;
-    }, [active, onCancel, cancelLabel, submitLabel, isLoading, disabled, onSubmit, destructiveAction]);
+    }, [active, onCancel, cancelLabel, submitLabel, isLoading, disabled, onSubmit, destructiveAction, extraActions]);
     useBottomActions({
         actions,
         // Encodes the count on purpose — see (1) in the header.

package/dist/features/live/components/LiveItemDetailPageView.js

@@ -33,7 +33,10 @@ function toDescriptionHtml(raw) {
 export async function LiveItemDetailPageView({ slug, initialProduct, renderActions }) {
     const product = initialProduct !== undefined
         ? (initialProduct ?? undefined)
-        : await getLiveItemForDetail(slug).catch(() => undefined);
+        // The listing IS this page's subject — a read failure must surface rather
+        // than become the same `undefined` a removed listing produces, which the
+        // "Not Found" branch below already handles.
+        : await getLiveItemForDetail(slug);
     if (!product) {
         return (_jsx(Main, { children: _jsx(Section, { padding: "y-5xl", children: _jsx(Container, { size: "md", children: _jsxs(Stack, { align: "start", gap: "md", className: "text-left", children: [_jsx(Heading, { level: 1, size: "2xl", weight: "semibold", color: "primary", children: "Live Listing Not Found" }), _jsx(Text, { color: "muted", children: "The live listing you are looking for may have been removed." }), _jsx(Link, { href: String(ROUTES.PUBLIC.LIVE), className: "text-[length:var(--appkit-text-sm)] font-medium text-primary-600 hover:underline", children: "Browse Live Listings" })] }) }) }) }));
     }

package/dist/features/lottery/schemas/config-write.d.ts

@@ -20,7 +20,16 @@ export declare const lotterySlotWriteSchema: z.ZodObject<{
     slotNumber: number;
     image?: string | undefined;
 }>;
+/**
+ * 🛑 `annotate()` must be the OUTERMOST call on each field — it keys a WeakMap
+ * by schema instance and every zod wrapper returns a new one.
+ */
 export declare const lotteryConfigWriteSchema: z.ZodEffects<z.ZodEffects<z.ZodObject<{
+    pricingMode: z.ZodEnum<["uniform", "variable"]>;
+    uniformPrice: z.ZodOptional<z.ZodNumber>;
+    drawWindowDurationMinutes: z.ZodNumber;
+    maxPullsPerTransaction: z.ZodNumber;
+    maxPullsPerUser: z.ZodNumber;
     slots: z.ZodArray<z.ZodObject<{
         slotNumber: z.ZodNumber;
         name: z.ZodString;
@@ -37,11 +46,6 @@ export declare const lotteryConfigWriteSchema: z.ZodEffects<z.ZodEffects<z.ZodOb
         slotNumber: number;
         image?: string | undefined;
     }>, "many">;
-    pricingMode: z.ZodEnum<["uniform", "variable"]>;
-    uniformPrice: z.ZodOptional<z.ZodNumber>;
-    drawWindowDurationMinutes: z.ZodNumber;
-    maxPullsPerTransaction: z.ZodNumber;
-    maxPullsPerUser: z.ZodNumber;
 }, "strict", z.ZodTypeAny, {
     slots: {
         name: string;

package/dist/features/lottery/schemas/config-write.js

@@ -23,6 +23,17 @@
  * `weight` is absent for the same reason: it is derived from price
  * server-side and is never client input.
  *
+ * ## One schema, not two
+ *
+ * There was a second one — `lotteryConfigFormSchema`, in
+ * `_internal/shared/features/lottery/config-form.ts`. `LotteryAdminEditView`
+ * passed THAT to `<Form>` and parsed THIS in its submit handler, so the form
+ * schema's rules (the duplicate-slot check, the free-lottery check) had never
+ * once executed. It was deleted 2026-08-30 and its annotations moved here:
+ * `annotate()` keys a WeakMap by schema instance and the server ignores the
+ * registry entirely, so carrying layout metadata on the write contract costs
+ * the route nothing and removes the opportunity for the two to disagree.
+ *
  * ## Merging is BY `slotNumber`, not by index
  *
  * Array position is not identity. An admin who deletes slot 3 shifts every
@@ -46,6 +57,7 @@
  * @tag sideEffects:none
  */
 import { z } from "zod";
+import { annotate } from "../../shell/field-ui-meta";
 /** Mirrors `LotteryPricingMode`. */
 const pricingModeSchema = z.enum(["uniform", "variable"]);
 /**
@@ -60,22 +72,70 @@ export const lotterySlotWriteSchema = z
     price: z.coerce.number().min(0, "A price cannot be negative."),
 })
     .strict();
+/**
+ * 🛑 `annotate()` must be the OUTERMOST call on each field — it keys a WeakMap
+ * by schema instance and every zod wrapper returns a new one.
+ */
 export const lotteryConfigWriteSchema = z
     .object({
-    slots: z
+    pricingMode: annotate(pricingModeSchema, {
+        section: "pricing", sectionLabel: "Pricing", sectionRequired: true,
+        quick: true, order: 1, row: "pair", label: "Pricing mode",
+    }),
+    /*
+     * Hidden under variable pricing, and therefore not sent — `visibleValues`
+     * drops it, and the refine below only demands it in uniform mode. The two
+     * halves have to agree or a hidden field fails validation invisibly.
+     */
+    uniformPrice: annotate(z.coerce.number().min(0).optional(), {
+        section: "pricing", order: 2, row: "pair", kind: "number",
+        label: "Price per slot (₹)",
+        when: (v) => v.pricingMode === "uniform",
+    }),
+    drawWindowDurationMinutes: annotate(z.coerce.number().int().min(1).max(1440), {
+        section: "limits", sectionLabel: "Draw rules", order: 1, row: "third",
+        kind: "number", label: "Draw window (minutes)",
+        help: "How long a buyer has to complete a pull once it starts.",
+    }),
+    maxPullsPerTransaction: annotate(z.coerce.number().int().min(1).max(200), {
+        section: "limits", order: 2, row: "third", kind: "number",
+        label: "Max pulls per transaction",
+    }),
+    maxPullsPerUser: annotate(z.coerce.number().int().min(1).max(200), {
+        section: "limits", order: 3, row: "third", kind: "number",
+        label: "Max pulls per user",
+    }),
+    slots: annotate(z
         .array(lotterySlotWriteSchema)
         .min(1, "A lottery needs at least one slot.")
-        .max(200, "A lottery cannot exceed 200 slots."),
-    pricingMode: pricingModeSchema,
-    uniformPrice: z.coerce.number().min(0).optional(),
-    drawWindowDurationMinutes: z.coerce.number().int().min(1).max(1440),
-    maxPullsPerTransaction: z.coerce.number().int().min(1).max(200),
-    maxPullsPerUser: z.coerce.number().int().min(1).max(200),
+        .max(200, "A lottery cannot exceed 200 slots."), {
+        section: "slots", sectionLabel: "Slots", sectionRequired: true,
+        order: 1, row: "full", kind: "list", label: "Prize slots",
+        /*
+         * The slot table holds uncommitted rows — a name half-typed, an image
+         * URL pasted but not yet saved. `<Collapse>` unmounts its children, so
+         * without this a collapse discards them.
+         */
+        sectionKeepMounted: true,
+    }),
 })
     .strict()
     .refine((c) => new Set(c.slots.map((s) => s.slotNumber)).size === c.slots.length, { message: "Two slots share a slot number.", path: ["slots"] })
-    .refine((c) => c.pricingMode !== "uniform" || typeof c.uniformPrice === "number", {
-    message: "Uniform pricing needs a price.",
+    /*
+     * ABOVE zero, not merely present.
+     *
+     * This read `typeof c.uniformPrice === "number"`, so `0` passed — and the
+     * editor's payload builds the value as
+     * `Math.round(parseFloat(uniformPrice) * 100) / 100 || 0`, which turns
+     * "12,00" or "twelve" into exactly that. A typo became a FREE lottery,
+     * indistinguishable from deliberately pricing one at zero.
+     *
+     * `lotteryConfigFormSchema` has carried the correct `> 0` rule all along and
+     * has never executed: the editor passes it to `<Form>` and parses THIS schema
+     * instead. The rule belongs on the schema that runs.
+     */
+    .refine((c) => c.pricingMode !== "uniform" || (typeof c.uniformPrice === "number" && c.uniformPrice > 0), {
+    message: "Set a slot price above zero, or switch to variable pricing.",
     path: ["uniformPrice"],
 });
 /**

package/dist/features/media/finalize.js

@@ -44,10 +44,12 @@ export async function finalizeStagedMediaUrl(url) {
             const finalPath = await moveToFinalPath(asset.storagePath);
             await mediaAssetsRepository.promoteToFinalized(shortId, finalPath);
         }
+        // Non-fatal by construction: the returned URL is the same before and after
+        // the move, so a failed promotion leaves the asset in tmp/ and the proxy
+        // keeps serving it from there until the tmp-cleanup job reclaims it.
     }
     catch (_err) {
         void normalizeError(_err);
-        // Non-fatal: URL is still valid; proxy will serve whatever storagePath points to.
     }
     return url; // URL is stable — same before and after the file move
 }

package/dist/features/media/modals/ImageEditor.js

@@ -63,7 +63,7 @@ export function ImageEditor({ isOpen, imageUrl, onClose, onSave, aspectRatio, ou
         }
         catch (err) {
             void normalizeError(err);
-            showToast(err instanceof Error ? err.message : "Failed to save image.", "error");
+            showToast("Failed to save image.", "error");
         }
         finally {
             setSaving(false);

package/dist/features/media/modals/VideoThumbnailSelector.js

@@ -10,6 +10,7 @@ import { useState, useRef } from "react";
 import { useTranslations } from "next-intl";
 import { Alert, Button, Div, Modal, Row, Span, Spinner, Stack, Text } from "../../../ui";
 import { normalizeError } from "../../../errors/normalize";
+import { toUserMessage } from "../../../errors/error-display-map";
 const __O = {
     hidden: "overflow-hidden",
 };
@@ -45,8 +46,10 @@ export function VideoThumbnailSelector({ isOpen, videoUrl, onClose, onSelect, on
             onClose();
         }
         catch (err) {
-            void normalizeError(err);
-            setError(err instanceof Error ? err.message : t("thumbnailError"));
+            const e = normalizeError(err);
+            // `t` is namespaced ("mediaEditor"), so it cannot resolve the absolute
+            // `errors.codes.*` keys — omit it and let the authored copy stand.
+            setError(toUserMessage(e.code, undefined, { fallback: t("thumbnailError") }));
         }
         finally {
             setIsCapturing(false);

package/dist/features/media/modals/VideoTrimModal.js

@@ -11,6 +11,7 @@ import { Alert, Button, Div, Modal, Row, Span, Stack, Text } from "../../../ui";
 import { MediaSlider } from "../components/MediaSlider";
 import { useMediaTrim } from "../hooks/useMedia";
 import { normalizeError } from "../../../errors/normalize";
+import { toUserMessage } from "../../../errors/error-display-map";
 function formatTime(seconds) {
     const m = Math.floor(seconds / 60);
     const s = Math.floor(seconds % 60);
@@ -62,8 +63,10 @@ export function VideoTrimModal({ isOpen, videoUrl, onClose, onSave, endpoint, })
             onClose();
         }
         catch (err) {
-            void normalizeError(err);
-            setError(err instanceof Error ? err.message : t("trimError"));
+            const e = normalizeError(err);
+            // `t` is namespaced ("mediaEditor"), so it cannot resolve the absolute
+            // `errors.codes.*` keys — omit it and let the authored copy stand.
+            setError(toUserMessage(e.code, undefined, { fallback: t("trimError") }));
         }
     };
     const qualityLabels = {

package/dist/features/media/upload/MediaUploadField.js

@@ -320,7 +320,7 @@ kind, accept: acceptProp, maxSizeMB: maxSizeMBProp, disabled = false, helperText
         }
         catch (err) {
             void normalizeError(err);
-            setError(err instanceof Error ? err.message : "Upload failed");
+            setError("Upload failed");
         }
         finally {
             setIsLoading(false);
@@ -362,7 +362,7 @@ kind, accept: acceptProp, maxSizeMB: maxSizeMBProp, disabled = false, helperText
         }
         catch (err) {
             void normalizeError(err);
-            setError(err instanceof Error ? err.message : "Upload failed");
+            setError("Upload failed");
         }
         finally {
             setIsLoading(false);

package/dist/features/messages/hooks/useConversation.js

@@ -10,8 +10,18 @@
  * canonical Firestore message ordering wins.
  */
 import { useCallback, useEffect, useState } from "react";
-import { getClientRealtimeProvider } from "../../../contracts/client-realtime";
 import { acquireRealtimeSession } from "../../../contracts/realtime-session";
+// One scope for both message hooks, deliberately: /api/realtime/token issues a
+// SINGLE token covering all of this user's conversationIds, so a per-id scope
+// would make the list and the detail hook re-sign against each other on the
+// same page for no gain.
+//
+// 🛑 Known limitation, not fixed here: because the scope is constant, a
+// conversation created AFTER sign-in is not in the cached token's claims and
+// its subscription is denied until the token's expiry skew forces a re-sign.
+// The denial is now surfaced (see the error callback below) rather than
+// swallowed. The durable fix is to re-acquire on a permission denial.
+const MESSAGES_SCOPE = "messages";
 import { conversationPingPath, userConversationsPingPath, } from "../realtime";
 import { normalizeError } from "../../../errors/normalize";
 import { apiClient, ApiClientError } from "../../../http/ApiClient";
@@ -83,12 +93,16 @@ export function useConversation(conversationId) {
         // because `send()` calls `refetch()` directly.
         void (async () => {
             try {
-                release = await acquireRealtimeSession("messages", () => apiClient.post(ADMIN_ENDPOINTS.REALTIME_TOKEN, {}));
+                const lease = await acquireRealtimeSession(MESSAGES_SCOPE, () => apiClient.post(ADMIN_ENDPOINTS.REALTIME_TOKEN, {}));
+                release = lease.release;
                 if (cancelled) {
-                    release?.();
+                    release();
                     return;
                 }
-                unsubscribe = getClientRealtimeProvider().subscribe(conversationPingPath(conversationId), () => {
+                // Subscribe through the LEASE's provider — it is bound to the app this
+                // scope authenticated on. The global provider may be on a different app
+                // carrying another channel's claims.
+                unsubscribe = lease.provider.subscribe(conversationPingPath(conversationId), () => {
                     setIsConnected(true);
                     void refetch();
                 }, (err) => {

package/dist/features/messages/hooks/useConversations.js

@@ -5,8 +5,18 @@
  * conversations gets a new message.
  */
 import { useCallback, useEffect, useState } from "react";
-import { getClientRealtimeProvider } from "../../../contracts/client-realtime";
 import { acquireRealtimeSession } from "../../../contracts/realtime-session";
+// One scope for both message hooks, deliberately: /api/realtime/token issues a
+// SINGLE token covering all of this user's conversationIds, so a per-id scope
+// would make the list and the detail hook re-sign against each other on the
+// same page for no gain.
+//
+// 🛑 Known limitation, not fixed here: because the scope is constant, a
+// conversation created AFTER sign-in is not in the cached token's claims and
+// its subscription is denied until the token's expiry skew forces a re-sign.
+// The denial is now surfaced (see the error callback below) rather than
+// swallowed. The durable fix is to re-acquire on a permission denial.
+const MESSAGES_SCOPE = "messages";
 import { userConversationsPingPath } from "../realtime";
 import { normalizeError } from "../../../errors/normalize";
 import { apiClient } from "../../../http/ApiClient";
@@ -57,23 +67,30 @@ export function useConversations(userId, opts = {}) {
         // `// ignore`, so the whole live-list channel was inert while looking wired.
         void (async () => {
             try {
-                release = await acquireRealtimeSession("messages", () => apiClient.post(ADMIN_ENDPOINTS.REALTIME_TOKEN, {}));
+                const lease = await acquireRealtimeSession(MESSAGES_SCOPE, () => apiClient.post(ADMIN_ENDPOINTS.REALTIME_TOKEN, {}));
+                release = lease.release;
                 if (cancelled) {
-                    release?.();
+                    release();
                     return;
                 }
-                unsubscribe = getClientRealtimeProvider().subscribe(userConversationsPingPath(userId), () => {
+                // Subscribe through the LEASE's provider — see useConversation.
+                unsubscribe = lease.provider.subscribe(userConversationsPingPath(userId), () => {
                     void refetch();
                 }, (err) => {
-                    // Surfaced, not ignored. A denied subscription is a real defect and
-                    // silence here is what hid it for months.
+                    // A denied subscription is a real defect — silence is what hid this
+                    // channel being dead for months — so it is logged, not swallowed.
+                    // (This comment previously claimed "Surfaced, not ignored" while the
+                    // body only called normalizeError, which classifies without
+                    // reporting. That was the same documentation-that-lies pattern this
+                    // workstream exists to remove, so it now actually logs.)
                     void normalizeError(err);
+                    console.warn("[messages] conversations live-update subscription denied; list falls back to fetch-on-load");
                 });
+                // Provider unregistered or token fetch failed — non-fatal; the list
+                // still works via the one-shot fetch above, just without live updates.
             }
             catch (_err) {
                 void normalizeError(_err);
-                // Provider unregistered or token fetch failed — non-fatal; the list
-                // still works via the one-shot fetch above, just without live updates.
             }
         })();
         return () => {

package/dist/features/orders/components/RefundRequestView.js

@@ -16,6 +16,7 @@ import { Button, Checkbox, Div, Span, Stack, Text, Textarea } from "../../../ui"
 import { REFUND_COPY } from "../../../_internal/shared/features/orders/refund-copy";
 import { formatCurrency } from "../../../utils/number.formatter";
 import { normalizeError } from "../../../errors/normalize";
+import { toUserMessage } from "../../../errors/error-display-map";
 const ACK_COUNT = REFUND_COPY.request.acknowledgments.length;
 const CLS_WARN_PANEL = "rounded-xl border border-warning bg-warning-surface p-[var(--appkit-space-4)] dark:border-warning";
 export function RefundRequestView({ order, onSubmitRequest, isLoading = false, className = "", }) {
@@ -37,8 +38,10 @@ export function RefundRequestView({ order, onSubmitRequest, isLoading = false, c
             await onSubmitRequest(reason.trim());
         }
         catch (err) {
-            void normalizeError(err);
-            setError(err instanceof Error ? err.message : REFUND_COPY.request.errorFallback);
+            const e = normalizeError(err);
+            setError(toUserMessage(e.code, undefined, {
+                fallback: REFUND_COPY.request.errorFallback,
+            }));
         }
     };
     return (_jsxs(Stack, { gap: "md", className: className, children: [_jsxs(Text, { size: "sm", color: "muted", children: [REFUND_COPY.request.orderTotalLabel, " ", _jsx(Span, { weight: "bold", children: formatCurrency(order.totalPrice, order.currency ?? "INR") })] }), _jsxs(Stack, { gap: "xs", children: [_jsxs(Text, { size: "sm", weight: "semibold", children: [REFUND_COPY.request.reasonLabel, " ", _jsx(Span, { "aria-hidden": true, children: "*" })] }), _jsx(Textarea, { value: reason, onChange: (e) => setReason(e.target.value), placeholder: REFUND_COPY.request.reasonPlaceholder, rows: 3, "aria-required": "true", "aria-label": REFUND_COPY.request.reasonLabel })] }), _jsxs(Stack, { gap: "sm", children: [_jsx(Text, { size: "sm", weight: "semibold", children: REFUND_COPY.request.acknowledgeHeading }), REFUND_COPY.request.acknowledgments.map((text, i) => (_jsx(Checkbox, { checked: acknowledged[i], onChange: () => handleToggle(i), label: _jsx(Text, { size: "sm", className: "leading-snug", children: text }), "aria-label": `Acknowledgment ${i + 1}` }, i)))] }), error && (_jsx(Text, { size: "sm", color: "danger", role: "alert", children: error })), _jsx(Button, { variant: "danger", onClick: handleSubmit, disabled: !canSubmit, "aria-busy": isLoading, children: isLoading ? REFUND_COPY.request.submittingLabel : REFUND_COPY.request.submitLabel })] }));