@mohasinac/appkit 4.30.18 → 4.31.0

package/dist/features/corporate/schemas/index.js

@@ -1,4 +1,5 @@
 import { z } from "zod";
+import { annotate } from "../../shell/field-ui-meta";
 export const corporateInquiryStatusSchema = z.enum([
     "pending",
     "contacted",
@@ -37,23 +38,30 @@ export const corporateInquirySchema = z.object({
  * Form schema for submitting a corporate inquiry.
  */
 export const submitCorporateInquirySchema = z.object({
-    companyName: z.string().trim().min(2, "A company name is required.").max(160),
-    contactPerson: z.string().trim().min(2, "Who should we reply to?").max(120),
-    designation: z.string().trim().max(120).optional(),
-    email: z
+    companyName: annotate(z.string().trim().min(2, "A company name is required.").max(160), {
+        section: "who", sectionLabel: "Your company", sectionRequired: true,
+        order: 1, row: "pair", label: "Company name",
+    }),
+    contactPerson: annotate(z.string().trim().min(2, "Who should we reply to?").max(120), {
+        section: "who", order: 2, row: "pair", label: "Contact person",
+    }),
+    designation: annotate(z.string().trim().max(120).optional(), {
+        section: "who", order: 3, row: "pair", label: "Designation",
+    }),
+    email: annotate(z
         .string()
         .trim()
         .toLowerCase()
         .min(1, "An email address is required.")
-        .email("That doesn't look like an email address."),
+        .email("That doesn't look like an email address."), { section: "who", order: 4, row: "pair", inputType: "email", label: "Email" }),
     // Indian mobile, tolerant of spacing and a country code. Optional — a
     // reachable email is enough, and demanding a phone number loses enquiries.
-    phone: z
+    phone: annotate(z
         .string()
         .trim()
         .regex(/^(\+?91[-\s]?)?[6-9]\d{9}$/, "Enter a 10-digit Indian mobile number.")
         .optional()
-        .or(z.literal("")),
+        .or(z.literal("")), { section: "who", order: 5, row: "pair", inputType: "tel", label: "Phone" }),
     /*
      * 🛑 COERCED. These were `z.number()`, and this schema's own form sends
      * strings — so every submission carrying a unit count failed to parse the
@@ -62,19 +70,32 @@ export const submitCorporateInquirySchema = z.object({
      *
      * A sales reply is priced from these two, and until now both were free text.
      */
-    units: z.coerce
+    units: annotate(z.coerce
         .number({ invalid_type_error: "Enter the number of units as a number." })
         .int("Units must be a whole number.")
         .positive("At least one unit.")
         .max(1000000, "That looks like a typo — contact us directly for orders this large.")
-        .optional(),
-    budgetPerUnit: z.coerce
+        .optional(), { section: "pricing", sectionLabel: "What you need", order: 1, row: "pair", kind: "number", label: "Units" }),
+    budgetPerUnit: annotate(z.coerce
         .number({ invalid_type_error: "Enter a per-unit budget as a number." })
         .min(0, "A budget cannot be negative.")
         .max(10000000, "That looks like a typo.")
-        .optional(),
-    totalBudget: z.coerce.number().min(0).optional(),
-    deliveryDateRequired: z.string().optional(),
-    customBranding: z.boolean().default(false),
-    message: z.string().trim().max(4000).optional(),
+        .optional(), { section: "pricing", order: 2, row: "pair", kind: "number", label: "Budget per unit (₹)" }),
+    // Computed from units × budgetPerUnit on the client, so it is shown as a
+    // live read-only preview rather than asked for twice.
+    totalBudget: annotate(z.coerce.number().min(0).optional(), {
+        section: "pricing", order: 3, row: "pair", kind: "number", label: "Total budget (₹)",
+        derived: true, tier: "t1-derive",
+    }),
+    deliveryDateRequired: annotate(z.string().optional(), {
+        section: "schedule", sectionLabel: "Timeline", order: 1, row: "pair", kind: "date",
+        label: "Delivery date required by",
+    }),
+    customBranding: annotate(z.boolean().default(false), {
+        section: "schedule", order: 2, row: "full", kind: "toggle", label: "We need custom branding",
+    }),
+    message: annotate(z.string().trim().max(4000).optional(), {
+        section: "detail", sectionLabel: "Anything else", order: 1, row: "full", kind: "textarea",
+        label: "Message",
+    }),
 });

package/dist/features/digital-codes/components/DigitalCodeDetailPageView.js

@@ -30,7 +30,10 @@ function toDescriptionHtml(raw) {
 export async function DigitalCodeDetailPageView({ slug, initialProduct, renderPrimaryActions }) {
     const product = initialProduct !== undefined
         ? (initialProduct ?? undefined)
-        : await getDigitalCodeForDetail(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 getDigitalCodeForDetail(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: "Digital Code Listing Not Found" }), _jsx(Text, { color: "muted", children: "The digital code listing you are looking for may have been removed." }), _jsx(Link, { href: String(ROUTES.PUBLIC.DIGITAL_CODES), className: "text-[length:var(--appkit-text-sm)] font-medium text-primary-600 hover:underline", children: "Browse Digital Codes" })] }) }) }) }));
     }

package/dist/features/events/components/EventBanner.js

@@ -13,10 +13,12 @@ export function EventBanner({ storageKey = "dismissed_event_banners", className
             const stored = sessionStorage.getItem(storageKey);
             if (stored)
                 setDismissedIds(JSON.parse(stored));
+            // sessionStorage is unavailable in private mode and over quota in some
+            // embedded browsers. Losing the dismissal list only means the banner
+            // reappears — nothing an operator can act on, and nothing the user loses.
         }
         catch (_err) {
             void normalizeError(_err);
-            // ignore session storage failures
         }
         setMounted(true);
     }, [storageKey]);
@@ -38,10 +40,11 @@ export function EventBanner({ storageKey = "dismissed_event_banners", className
         setDismissedIds(next);
         try {
             sessionStorage.setItem(storageKey, JSON.stringify(next));
+            // Same as the read above: the banner is already hidden in local state, so
+            // a failed persist only means it returns on the next page load.
         }
         catch (_err) {
             void normalizeError(_err);
-            // ignore
         }
     };
     return (_jsxs(Section, { tone: "accent-banner", className: [

package/dist/features/events/components/EventOfferCard.js

@@ -24,10 +24,12 @@ export function EventOfferCard({ couponCode, offerDescription, title = "Exclusiv
             await navigator.clipboard.writeText(couponCode);
             setCopied(true);
             setTimeout(() => setCopied(false), 2000);
+            // Clipboard access is denied outright on insecure origins and by user
+            // permission. The button simply never flips to "Copied", and the code is
+            // still visible on screen to select by hand.
         }
         catch (_err) {
             void normalizeError(_err);
-            /* clipboard unavailable — leave button as-is */
         }
     };
     return (_jsx(Div, { className: `${__P.p6}`, rounded: "2xl", border: "default", children: _jsxs(Stack, { gap: "md", children: [_jsxs(Stack, { gap: "xs", children: [_jsx(Heading, { level: 2, weight: "semibold", size: "2xl", children: title }), offerDescription ? (_jsx(Text, { size: "sm", color: "muted", children: offerDescription })) : null] }), _jsxs(Row, { surface: "muted", padding: "inline", align: "center", gap: "3", rounded: "lg", children: [_jsx(Code, { className: "flex-1", weight: "bold", size: "base", color: "primary", children: couponCode }), _jsx(Button, { type: "button", variant: "outline", onClick: copyCode, children: copied ? "Copied!" : "Copy code" })] }), _jsx(Div, { children: _jsx(Link, { href: href, children: _jsx(Button, { variant: "primary", children: "Apply at checkout" }) }) })] }) }));

package/dist/features/events/components/EventPollWidget.js

@@ -53,7 +53,7 @@ export function EventPollWidget({ eventId, pollConfig, eventStatus, totalEntries
         }
         catch (err) {
             void normalizeError(err);
-            setError(err instanceof Error ? err.message : "Something went wrong");
+            setError("Something went wrong");
         }
         finally {
             setIsLoading(false);

package/dist/features/events/components/SpinWheelView.js

@@ -4,6 +4,7 @@ import { useCallback, useMemo, useState } from "react";
 import { Button, ClaimCouponButton, Div, Heading, LoginRequiredModal, Row, Span, Stack, Text, useToast } from "../../../ui";
 import { isAuthError } from "../../../utils/auth-error";
 import { normalizeError } from "../../../errors/normalize";
+import { toUserMessage } from "../../../errors/error-display-map";
 const CLS_WHEEL = "relative mx-auto aspect-square w-64 overflow-hidden rounded-full border-4 border-warning bg-[image:var(--appkit-gradient-promotion)]";
 // audit-content-alignment-ok: self-contained circular spin-wheel game widget, all text centered under the wheel by design
 const CLS_PRIZE_BOX = "rounded-xl border border-success bg-success-surface dark:border-success p-[var(--appkit-space-4)] text-center";
@@ -55,14 +56,14 @@ export function SpinWheelView({ eventId, prizes, alreadyUsed, initialPrizeId, in
             }, ANIMATION_MS);
         }
         catch (err) {
-            void normalizeError(err);
+            const e = normalizeError(err);
             setSpinning(false);
             if (isAuthError(err)) {
                 setShowLoginModal(true);
             }
             else {
                 setError(l.errorFallback);
-                showToast(err instanceof Error ? err.message : l.errorFallback, "error");
+                showToast(toUserMessage(e.code, undefined, { fallback: l.errorFallback }), "error");
             }
         }
     }, [disabled, eventId, l.errorFallback, onSpin, showToast]);

package/dist/features/filters/FilterPanel.js

@@ -5,6 +5,7 @@ import { Div } from "../../ui";
 import { FilterFacetSection } from "./FilterFacetSection";
 import { RangeFilter } from "./RangeFilter";
 import { SwitchFilter } from "./SwitchFilter";
+import { joinFilterValues, splitFilterValues } from "../../constants/table-keys";
 export function FilterPanel({ config, table, className = "", }) {
     const [expandedIndex, setExpandedIndex] = useState(() => config.findIndex((c) => !c.defaultCollapsed));
     const handleToggle = (idx) => setExpandedIndex((prev) => (prev === idx ? null : idx));
@@ -17,9 +18,8 @@ export function FilterPanel({ config, table, className = "", }) {
                     return (_jsx(FilterFacetSection, { title: item.title, options: item.options, selectionMode: "single", selected: selected, defaultCollapsed: item.defaultCollapsed, searchable: item.searchable, isOpen: isOpen, onToggle: () => handleToggle(idx), onClear: () => table.set(item.key, ""), onChange: (vals) => table.set(item.key, vals[0] ?? "") }, `${item.type}-${item.key}`));
                 }
                 case "facet-multi": {
-                    const raw = table.get(item.key);
-                    const selected = raw ? raw.split(",").filter(Boolean) : [];
-                    return (_jsx(FilterFacetSection, { title: item.title, options: item.options, selectionMode: "multi", selected: selected, defaultCollapsed: item.defaultCollapsed, searchable: item.searchable, isOpen: isOpen, onToggle: () => handleToggle(idx), onClear: () => table.set(item.key, ""), onChange: (vals) => table.set(item.key, vals.join(",")) }, `${item.type}-${item.key}`));
+                    const selected = splitFilterValues(table.get(item.key));
+                    return (_jsx(FilterFacetSection, { title: item.title, options: item.options, selectionMode: "multi", selected: selected, defaultCollapsed: item.defaultCollapsed, searchable: item.searchable, isOpen: isOpen, onToggle: () => handleToggle(idx), onClear: () => table.set(item.key, ""), onChange: (vals) => table.set(item.key, joinFilterValues(vals)) }, `${item.type}-${item.key}`));
                 }
                 case "switch": {
                     const checked = table.get(item.key) === "true";

package/dist/features/filters/RangeFilter.d.ts

@@ -18,6 +18,16 @@ export interface RangeFilterProps {
     className?: string;
     isOpen?: boolean;
     onToggle?: () => void;
+    /**
+     * Override the clear behaviour. **Rarely needed** — clearing a range is
+     * `onMinChange("") + onMaxChange("")`, which is now the default, so the X
+     * appears on every range filter without a call site opting in.
+     *
+     * It used to be required-in-practice: 18 of the 19 call sites passed nothing
+     * and therefore rendered no clear button at all, leaving a range you could
+     * set and not unset. Pass this only when clearing must ALSO reset something
+     * outside the two values (`FilterPanel` clears its own pending map).
+     */
     onClear?: () => void;
 }
 export declare function RangeFilter({ title, minLabel, maxLabel, minValue, maxValue, onMinChange, onMaxChange, type, prefix, minPlaceholder, maxPlaceholder, defaultCollapsed, showSlider, minBound, maxBound, step, className, isOpen: controlledOpen, onToggle, onClear, }: RangeFilterProps): import("react").JSX.Element;

package/dist/features/filters/RangeFilter.js

@@ -5,10 +5,41 @@ import { Button, Div, Input, Row, Slider, Span, Stack, Text } from "../../ui";
 import { cn } from "./filterUtils";
 const CLS_BADGE_COUNT = "inline-flex items-center justify-center w-5 h-5 rounded-full bg-success-surface dark:bg-success-surface text-success dark:text-success ring-1 ring-success/20";
 const CLS_CLEAR_BTN = "inline-flex items-center justify-center w-5 h-5 text-[var(--appkit-color-text-muted)] hover:text-error dark:hover:text-error rounded-full transition-colors";
+/**
+ * Parse a filter value, distinguishing "not set" from a legitimate zero.
+ *
+ * 🛑 NOT `parseFloat(v) || bound`. `parseFloat("")` is NaN and falsy, which is
+ * the intended case — but so is `parseFloat("0")`, so a user asking for a max
+ * of 0 got the 500000 bound instead, and the filter they could see in the
+ * inputs was not the one being applied.
+ */
+function rawNum(value) {
+    const n = Number.parseFloat(value);
+    return Number.isFinite(n) ? n : null;
+}
 function DualSlider({ minValue, maxValue, onMinChange, onMaxChange, minBound, maxBound, step = 1, prefix = "", }) {
-    const minNum = Math.max(minBound, Math.min(parseFloat(minValue) || minBound, maxBound));
-    const maxNum = Math.min(maxBound, Math.max(parseFloat(maxValue) || maxBound, minBound));
-    return (_jsxs(Stack, { gap: "sm", className: "w-full", children: [_jsxs(Row, { align: "center", justify: "between", children: [_jsxs(Span, { size: "sm", weight: "semibold", className: "tabular-nums text-primary-600 dark:text-secondary-400", children: [prefix, minNum] }), _jsx(Span, { size: "xs", color: "muted", children: "-" }), _jsxs(Span, { size: "sm", weight: "semibold", className: "tabular-nums text-primary-600 dark:text-secondary-400", children: [prefix, maxNum] })] }), _jsxs(Stack, { gap: "xs", "aria-hidden": "true", children: [_jsx(Slider, { value: minNum, min: minBound, max: Math.max(minBound, maxNum - step), step: step, onChange: (v) => onMinChange(String(v)) }), _jsx(Slider, { value: maxNum, min: Math.min(maxBound, minNum + step), max: maxBound, step: step, onChange: (v) => onMaxChange(String(v)) })] }), _jsxs(Row, { align: "center", justify: "between", children: [_jsxs(Span, { size: "xs", color: "muted", children: [prefix, minBound] }), _jsxs(Span, { size: "xs", color: "muted", children: [prefix, maxBound] })] })] }));
+    const minRaw = rawNum(minValue);
+    const maxRaw = rawNum(maxValue);
+    /*
+     * Display only. An unset side shows its bound so the track reads sensibly —
+     * but the bound is never what gets WRITTEN, which is the distinction the old
+     * `|| bound` collapsed.
+     */
+    const minNum = Math.min(Math.max(minRaw ?? minBound, minBound), maxBound);
+    const maxNum = Math.max(Math.min(maxRaw ?? maxBound, maxBound), minBound);
+    /*
+     * Each slider spans the FULL range and clamps itself on write.
+     *
+     * It used to derive its opposite end from the other thumb — the min slider's
+     * `max` was `maxNum - step` — so with max unset, `maxNum` collapsed to the
+     * bound, and any later change to one side silently re-scaled and appeared to
+     * reset the other. Clamping on write instead of on range means dragging one
+     * thumb can never move or clear the other, and a value that would cross is
+     * pinned rather than the whole control rescaling underneath the user.
+     */
+    const writeMin = (v) => onMinChange(String(Math.min(v, maxRaw ?? maxBound)));
+    const writeMax = (v) => onMaxChange(String(Math.max(v, minRaw ?? minBound)));
+    return (_jsxs(Stack, { gap: "sm", className: "w-full", children: [_jsxs(Row, { align: "center", justify: "between", children: [_jsxs(Span, { size: "sm", weight: "semibold", className: "tabular-nums text-primary-600 dark:text-secondary-400", children: [prefix, minNum] }), _jsx(Span, { size: "xs", color: "muted", children: "-" }), _jsxs(Span, { size: "sm", weight: "semibold", className: "tabular-nums text-primary-600 dark:text-secondary-400", children: [prefix, maxNum] })] }), _jsxs(Stack, { gap: "xs", "aria-hidden": "true", children: [_jsx(Slider, { value: minNum, min: minBound, max: maxBound, step: step, onChange: writeMin }), _jsx(Slider, { value: maxNum, min: minBound, max: maxBound, step: step, onChange: writeMax })] }), _jsxs(Row, { align: "center", justify: "between", children: [_jsxs(Span, { size: "xs", color: "muted", children: [prefix, minBound] }), _jsxs(Span, { size: "xs", color: "muted", children: [prefix, maxBound] })] })] }));
 }
 export function RangeFilter({ title, minLabel, maxLabel, minValue, maxValue, onMinChange, onMaxChange, type = "number", prefix, minPlaceholder, maxPlaceholder, defaultCollapsed = true, showSlider = false, minBound, maxBound, step = 1, className = "", isOpen: controlledOpen, onToggle, onClear, }) {
     const isControlled = controlledOpen !== undefined;
@@ -25,6 +56,12 @@ export function RangeFilter({ title, minLabel, maxLabel, minValue, maxValue, onM
         minBound !== undefined &&
         maxBound !== undefined;
     const hasValue = !!(minValue || maxValue);
+    // Clearing a range needs nothing a caller has to supply. See `onClear` above.
+    const handleClear = onClear ??
+        (() => {
+            onMinChange("");
+            onMaxChange("");
+        });
     const inputClass = "w-full rounded-md border border-[var(--appkit-color-border)] bg-[var(--appkit-color-surface-input)] px-[var(--appkit-space-2-5)] py-[var(--appkit-space-1-5)] text-[length:var(--appkit-text-xs)] text-[var(--appkit-color-text)] focus:outline-none focus:ring-2 focus:ring-primary-500/20 ";
-    return (_jsxs(Div, { role: "group", "aria-labelledby": `rf-${title}`, className: cn("p-[var(--appkit-space-4)] border-b border-[var(--appkit-color-border)] last:border-b-0", className), children: [_jsxs(Row, { align: "center", gap: "sm", children: [_jsxs(Button, { type: "button", id: `rf-${title}`, onClick: handleToggle, variant: "ghost", size: "sm", className: "flex-1 justify-[space-between] text-[0.875rem] font-[600] text-[var(--appkit-color-text)] py-[0.25rem] hover:opacity-80 transition-opacity", "aria-expanded": !isCollapsed, children: [_jsxs(Span, { layout: "flex", gap: "md", children: [title, hasValue && (_jsx(Span, { size: "xs", className: CLS_BADGE_COUNT, children: "1" }))] }), _jsx("svg", { className: cn("w-4 h-4 shrink-0 text-[var(--appkit-color-text-muted)] transition-transform duration-200", isCollapsed ? "" : "rotate-180"), fill: "none", viewBox: "0 0 24 24", stroke: "currentColor", strokeWidth: 2, "aria-hidden": "true", children: _jsx("path", { strokeLinecap: "round", strokeLinejoin: "round", d: "M19 9l-7 7-7-7" }) })] }), onClear && hasValue && (_jsx(Button, { type: "button", onClick: onClear, variant: "ghost", size: "sm", className: CLS_CLEAR_BTN, "aria-label": "Clear", children: _jsx("svg", { className: "w-3 h-3 shrink-0", fill: "none", viewBox: "0 0 24 24", stroke: "currentColor", strokeWidth: 2.5, "aria-hidden": "true", children: _jsx("path", { strokeLinecap: "round", strokeLinejoin: "round", d: "M6 18L18 6M6 6l12 12" }) }) }))] }), !isCollapsed && (_jsxs(Stack, { gap: "3", className: "mt-3", children: [canShowSlider && (_jsx(DualSlider, { minValue: minValue, maxValue: maxValue, onMinChange: onMinChange, onMaxChange: onMaxChange, minBound: minBound, maxBound: maxBound, step: step, prefix: prefix })), _jsxs(Row, { align: "end", gap: "sm", children: [_jsxs(Div, { className: "flex-1 min-w-0", children: [minLabel && (_jsx(Text, { className: "mb-1", color: "muted", size: "xs", children: minLabel })), _jsxs(Div, { className: "relative", children: [prefix && (_jsx(Span, { size: "xs", className: "absolute left-2 top-1/2 -translate-y-1/2 pointer-events-none", color: "faint", children: prefix })), _jsx(Input, { type: type, value: minValue, onChange: (e) => onMinChange(e.target.value), placeholder: minPlaceholder ?? (type === "date" ? "From" : "Min"), className: cn(inputClass, prefix ? "pl-5" : "") })] })] }), _jsx(Span, { size: "xs", className: "flex-shrink-0 pb-[0.375rem]", color: "faint", children: "-" }), _jsxs(Div, { className: "flex-1 min-w-0", children: [maxLabel && (_jsx(Text, { className: "mb-1", color: "muted", size: "xs", children: maxLabel })), _jsxs(Div, { className: "relative", children: [prefix && (_jsx(Span, { size: "xs", className: "absolute left-2 top-1/2 -translate-y-1/2 pointer-events-none", color: "faint", children: prefix })), _jsx(Input, { type: type, value: maxValue, onChange: (e) => onMaxChange(e.target.value), placeholder: maxPlaceholder ?? (type === "date" ? "To" : "Max"), className: cn(inputClass, prefix ? "pl-5" : "") })] })] })] })] }))] }));
+    return (_jsxs(Div, { role: "group", "aria-labelledby": `rf-${title}`, className: cn("p-[var(--appkit-space-4)] border-b border-[var(--appkit-color-border)] last:border-b-0", className), children: [_jsxs(Row, { align: "center", gap: "sm", children: [_jsxs(Button, { type: "button", id: `rf-${title}`, onClick: handleToggle, variant: "ghost", size: "sm", className: "flex-1 justify-[space-between] text-[0.875rem] font-[600] text-[var(--appkit-color-text)] py-[0.25rem] hover:opacity-80 transition-opacity", "aria-expanded": !isCollapsed, children: [_jsxs(Span, { layout: "flex", gap: "md", children: [title, hasValue && (_jsx(Span, { size: "xs", className: CLS_BADGE_COUNT, children: "1" }))] }), _jsx("svg", { className: cn("w-4 h-4 shrink-0 text-[var(--appkit-color-text-muted)] transition-transform duration-200", isCollapsed ? "" : "rotate-180"), fill: "none", viewBox: "0 0 24 24", stroke: "currentColor", strokeWidth: 2, "aria-hidden": "true", children: _jsx("path", { strokeLinecap: "round", strokeLinejoin: "round", d: "M19 9l-7 7-7-7" }) })] }), hasValue && (_jsx(Button, { type: "button", onClick: handleClear, variant: "ghost", size: "sm", className: CLS_CLEAR_BTN, "aria-label": "Clear", children: _jsx("svg", { className: "w-3 h-3 shrink-0", fill: "none", viewBox: "0 0 24 24", stroke: "currentColor", strokeWidth: 2.5, "aria-hidden": "true", children: _jsx("path", { strokeLinecap: "round", strokeLinejoin: "round", d: "M6 18L18 6M6 6l12 12" }) }) }))] }), !isCollapsed && (_jsxs(Stack, { gap: "3", className: "mt-3", children: [canShowSlider && (_jsx(DualSlider, { minValue: minValue, maxValue: maxValue, onMinChange: onMinChange, onMaxChange: onMaxChange, minBound: minBound, maxBound: maxBound, step: step, prefix: prefix })), _jsxs(Row, { align: "end", gap: "sm", children: [_jsxs(Div, { className: "flex-1 min-w-0", children: [minLabel && (_jsx(Text, { className: "mb-1", color: "muted", size: "xs", children: minLabel })), _jsxs(Div, { className: "relative", children: [prefix && (_jsx(Span, { size: "xs", className: "absolute left-2 top-1/2 -translate-y-1/2 pointer-events-none", color: "faint", children: prefix })), _jsx(Input, { type: type, value: minValue, onChange: (e) => onMinChange(e.target.value), placeholder: minPlaceholder ?? (type === "date" ? "From" : "Min"), className: cn(inputClass, prefix ? "pl-5" : "") })] })] }), _jsx(Span, { size: "xs", className: "flex-shrink-0 pb-[0.375rem]", color: "faint", children: "-" }), _jsxs(Div, { className: "flex-1 min-w-0", children: [maxLabel && (_jsx(Text, { className: "mb-1", color: "muted", size: "xs", children: maxLabel })), _jsxs(Div, { className: "relative", children: [prefix && (_jsx(Span, { size: "xs", className: "absolute left-2 top-1/2 -translate-y-1/2 pointer-events-none", color: "faint", children: prefix })), _jsx(Input, { type: type, value: maxValue, onChange: (e) => onMaxChange(e.target.value), placeholder: maxPlaceholder ?? (type === "date" ? "To" : "Max"), className: cn(inputClass, prefix ? "pl-5" : "") })] })] })] })] }))] }));
 }

package/dist/features/grouped/components/GroupedListingEditorView.js

@@ -46,6 +46,7 @@ import { apiClient } from "../../../http";
 import { ADMIN_ENDPOINTS, SELLER_ENDPOINTS } from "../../../constants/api-endpoints";
 import { ROUTES } from "../../../next/routing/route-map";
 import { normalizeError } from "../../../errors/normalize";
+import { toUserMessage } from "../../../errors/error-display-map";
 import { ProductInlineSelect } from "../../seller/components/ProductInlineSelect";
 import { groupedListingFormSchema, } from "../schemas/grouped-listing-form";
 const EMPTY = {
@@ -173,7 +174,7 @@ export function GroupedListingEditorView({ scope, groupId, fixedStoreId, onSaved
         }
         catch (err) {
             const e = normalizeError(err);
-            showToast(e.message || "Failed to save the group.", "error");
+            showToast(toUserMessage(e.code, undefined, { fallback: "Failed to save the group." }), "error");
         }
         finally {
             setSaving(false);

package/dist/features/history/hooks/useHistory.js

@@ -38,35 +38,28 @@ async function postAuthTrack(args) {
             headers: { "Content-Type": "application/json" },
             body: JSON.stringify(args),
         });
+        // Recently-viewed tracking is a side effect of browsing, fired from a
+        // debounced timer with no UI attached. A lost entry costs the user nothing
+        // and there is no caller in a position to retry or report it.
     }
     catch (_err) {
         void normalizeError(_err);
-        // best-effort tracking
     }
 }
+// Both deletes below are user-initiated and deliberately let failures throw:
+// swallowing them made `remove`/`clear` report success for a DELETE that never
+// landed, and the row would silently return on the next load.
 async function deleteAuthItem(productId) {
-    try {
-        await fetch(ACCOUNT_ENDPOINTS.HISTORY_ITEM(productId), {
-            method: "DELETE",
-            credentials: "include",
-        });
-    }
-    catch (_err) {
-        void normalizeError(_err);
-        // ignore
-    }
+    await fetch(ACCOUNT_ENDPOINTS.HISTORY_ITEM(productId), {
+        method: "DELETE",
+        credentials: "include",
+    });
 }
 async function deleteAuthAll() {
-    try {
-        await fetch(ACCOUNT_ENDPOINTS.HISTORY, {
-            method: "DELETE",
-            credentials: "include",
-        });
-    }
-    catch (_err) {
-        void normalizeError(_err);
-        // ignore
-    }
+    await fetch(ACCOUNT_ENDPOINTS.HISTORY, {
+        method: "DELETE",
+        credentials: "include",
+    });
 }
 export function useHistory(userId) {
     const isAuth = !!userId;
@@ -81,6 +74,9 @@ export function useHistory(userId) {
             else {
                 setItems(getGuestHistory());
             }
+            // fetchAuthHistory already degrades to [] on its own, and getGuestHistory
+            // is a localStorage read that fails only in private mode — either way the
+            // list stays as it was, which is the correct visible outcome here.
         }
         catch (err) {
             void normalizeError(err);
@@ -131,14 +127,20 @@ export function useHistory(userId) {
         trackTimers.current.set(key, timer);
     }, [isAuth]);
     const remove = useCallback(async (productId) => {
-        if (isAuth) {
-            await deleteAuthItem(productId);
-            await loadItems();
+        try {
+            if (isAuth) {
+                await deleteAuthItem(productId);
+                await loadItems();
+            }
+            else {
+                setItems(removeGuestHistoryItem(productId));
+            }
         }
-        else {
-            setItems(removeGuestHistoryItem(productId));
+        catch (_err) {
+            void normalizeError(_err);
+            showToast("Failed to remove this item from history.", "error");
         }
-    }, [isAuth, loadItems]);
+    }, [isAuth, loadItems, showToast]);
     const clear = useCallback(async () => {
         try {
             if (isAuth) {

package/dist/features/homepage/components/AdSlot.js

@@ -16,10 +16,12 @@ function AdsenseAd({ config }) {
             return;
         try {
             (window.adsbygoogle = window.adsbygoogle || []).push({});
+            // adsbygoogle is injected by a third-party script that ad blockers and
+            // consent walls routinely prevent from loading. The <ins> below simply
+            // stays unfilled — there is no failure the site can act on.
         }
         catch (_err) {
             void normalizeError(_err);
-            // AdSense not loaded yet — no-op
         }
     }, []);
     return (_jsx(Div, { ref: ref, children: _jsx("ins", { className: "adsbygoogle block", "data-ad-client": config.adsenseClient ?? "", "data-ad-slot": config.adsenseSlot ?? "", "data-ad-format": config.adsenseFormat ?? "auto", "data-full-width-responsive": "true" }) }));

package/dist/features/homepage/components/GoogleReviewsSection.js

@@ -1,5 +1,6 @@
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { normalizeError } from "../../../errors/normalize";
+import { serverLogger } from "../../../monitoring/server-logger";
 import { Anchor, Div, GRID_MAP, Heading, HandModeRow, Row, Section, Span, Stack, Text } from "../../../ui";
 import { MediaImage } from "../../media/MediaImage";
 import { siteSettingsRepository } from "../../admin/repository/site-settings.repository";
@@ -60,8 +61,13 @@ export async function GoogleReviewsSection(config) {
         totalRatings = result.totalRatings;
     }
     catch (_err) {
-        void normalizeError(_err);
-        // Fail silently — empty state shown
+        // The section renders its empty state below, which is indistinguishable
+        // from a place that genuinely has no reviews — and Google Places calls are
+        // billed, so a persistent failure is worth an operator seeing.
+        serverLogger.warn("homepage.googleReviews: Places fetch failed", {
+            placeId,
+            error: normalizeError(_err).message,
+        });
     }
     const gridClass = layout === "carousel"
         ? "flex gap-[var(--appkit-space-4)] overflow-x-auto pb-[var(--appkit-space-2)] snap-x snap-mandatory"

package/dist/features/homepage/components/MarketplaceHomepageView.js

@@ -3,10 +3,21 @@ import { sieveFilter, SIEVE_OP } from "@mohasinac/appkit";
 import { sortBy } from "@mohasinac/appkit";
 import { Div, Main } from "../../../ui";
 import { carouselRepository, faqsRepository, siteSettingsRepository } from "../../../repositories";
+import { safeRead } from "../../../errors/safe-read";
 import { fetchLiveStats } from "../lib/live-stats";
 import { renderSection, AnnouncementBar } from "../lib/section-renderer";
 import { homepageSectionsRepository } from "../repository/homepage-sections.repository";
-import { filterSectionsByFeatureFlags } from "../../../_internal/shared/features/homepage/section-gate";
+import { isListingTypeEnabled } from "../../../_internal/shared/listing-types/feature-flags";
+/**
+ * Homepage sections that advertise a specific listing type, and must therefore
+ * disappear when the site stops offering it. A section type absent from this
+ * map always renders.
+ */
+const HOMEPAGE_SECTION_LISTING_TYPE = {
+    auctions: "auction",
+    "pre-orders": "pre-order",
+    "prize-draws": "prize-draw",
+};
 import { getFeaturedProducts, getFeaturedAuctions, getFeaturedPreOrders } from "../../products/actions/product-actions";
 import { listTopLevelCategories, listBrandCategories } from "../../categories/actions/category-actions";
 import { listFeaturedBundles } from "../../../_internal/server/features/bundles/data";
@@ -28,19 +39,39 @@ function toCarouselSlides(docs) {
     }));
 }
 export async function MarketplaceHomepageView({ adSlots, newsletterFormSlot, onBannerDismiss, } = {}) {
-    const slides = await carouselRepository.getActiveSlides().catch(() => []);
-    const siteSettings = await siteSettingsRepository.getSingleton().catch(() => null);
+    // Every read on this page is a RAIL: the homepage must still render when one
+    // fails, but a failure has to be distinguishable from an empty catalogue —
+    // that is exactly Root Cause #59, and safeRead is what records it.
+    const slides = await safeRead(() => carouselRepository.getActiveSlides(), {
+        route: "/", key: "homepage.carouselSlides", fallback: [],
+    });
+    const siteSettings = await safeRead(() => siteSettingsRepository.getSingleton(), {
+        route: "/", key: "homepage.siteSettings", fallback: null,
+    });
     const announcementMessage = siteSettings?.announcementBar?.message?.trim() ||
         "🎉 Up to 15% Off on Pokémon TCG this week — Use code SAVE15";
     const showAnnouncement = siteSettings?.announcementBar?.enabled ?? true;
     const [allSections, rawFaqItems] = await Promise.all([
-        homepageSectionsRepository.getEnabledSections().catch(() => []),
-        faqsRepository.getHomepageFAQs().catch(() => []),
+        safeRead(() => homepageSectionsRepository.getEnabledSections(), {
+            route: "/", key: "homepage.enabledSections", fallback: [],
+        }),
+        safeRead(() => faqsRepository.getHomepageFAQs(), {
+            route: "/", key: "homepage.faqs", fallback: [],
+        }),
     ]);
-    // Respect the feature flags. This gate existed but lived inside a function
-    // with no call sites, so until 2026-08-24 turning off `auctions` hid auctions
-    // everywhere EXCEPT the homepage, which went on advertising them.
-    const enabledSections = filterSectionsByFeatureFlags(allSections, siteSettings?.featureFlags);
+    // A homepage section must not advertise a listing type the site does not
+    // offer. This gate previously read `featureFlags.{auctions,preOrders,events,
+    // blog}`; that group was deleted on 2026-08-29, and only the listing-type
+    // half of it had a real successor (`settings.listings.listingTypes`).
+    //
+    // So auctions / pre-orders / prize-draws are gated by the same control that
+    // hides them on every other surface, and the `events` / `blog` sections are
+    // now unconditional — there is no product control for "we don't run events",
+    // and there never was one that anything but this line read.
+    const enabledSections = allSections.filter((section) => {
+        const gated = HOMEPAGE_SECTION_LISTING_TYPE[section.type];
+        return !gated || isListingTypeEnabled(gated, siteSettings);
+    });
     // Collect live metric requests from all enabled stats sections
     const liveStatRequests = [];
     for (const section of enabledSections) {
@@ -87,12 +118,24 @@ export async function MarketplaceHomepageView({ adSlots, newsletterFormSlot, onB
         activeTypes.has("products") ? getFeaturedProducts(12) : null,
         activeTypes.has("auctions") ? getFeaturedAuctions(12) : null,
         activeTypes.has("pre-orders") ? getFeaturedPreOrders(12) : null,
-        activeTypes.has("categories") ? listTopLevelCategories(12).catch(() => null) : null,
-        activeTypes.has("brands") ? listBrandCategories(12).catch(() => null) : null,
-        activeTypes.has("featured-bundles") ? listFeaturedBundles(8).catch(() => null) : null,
-        activeTypes.has("stores") ? listStores({ pageSize: 8, sorts: sortBy("averageRating", "DESC") }).catch(() => null) : null,
-        activeTypes.has("blog-articles") ? getFeaturedBlogPosts(6).catch(() => null) : null,
-        activeTypes.has("events") ? listPublicEvents({ filters: sieveFilter("status", SIEVE_OP.EQ, "active"), pageSize: 6 }).catch(() => null) : null,
+        activeTypes.has("categories")
+            ? safeRead(() => listTopLevelCategories(12), { route: "/", key: "homepage.categories", fallback: null })
+            : null,
+        activeTypes.has("brands")
+            ? safeRead(() => listBrandCategories(12), { route: "/", key: "homepage.brands", fallback: null })
+            : null,
+        activeTypes.has("featured-bundles")
+            ? safeRead(() => listFeaturedBundles(8), { route: "/", key: "homepage.featuredBundles", fallback: null })
+            : null,
+        activeTypes.has("stores")
+            ? safeRead(() => listStores({ pageSize: 8, sorts: sortBy("averageRating", "DESC") }), { route: "/", key: "homepage.stores", fallback: null })
+            : null,
+        activeTypes.has("blog-articles")
+            ? safeRead(() => getFeaturedBlogPosts(6), { route: "/", key: "homepage.blogPosts", fallback: null })
+            : null,
+        activeTypes.has("events")
+            ? safeRead(() => listPublicEvents({ filters: sieveFilter("status", SIEVE_OP.EQ, "active"), pageSize: 6 }), { route: "/", key: "homepage.events", fallback: null })
+            : null,
     ]);
     // ProductDocument / BlogDocument / EventDocument have Date fields and extra Firestore-only
     // fields absent from the corresponding *Item types. The casts below are safe: section

package/dist/features/homepage/components/SocialFeedSection.js

@@ -1,5 +1,6 @@
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { normalizeError } from "../../../errors/normalize";
+import { safeRead } from "../../../errors/safe-read";
 import { Anchor, Div, GRID_MAP, Heading, HandModeRow, Section, Stack, Text } from "../../../ui";
 import { SocialPostCard } from "./SocialPostCard";
 import { fetchInstagramPosts, fetchFacebookPosts, fetchTikTokPosts, fetchDeviantArtPosts, } from "../lib/social-feed-fetcher";
@@ -47,7 +48,9 @@ async function loadPosts(config) {
     if (!handle)
         return { posts: [], error: `A handle is required for ${PLATFORM_LABELS[platform]}.` };
     try {
-        const credentials = await siteSettingsRepository.getDecryptedCredentials().catch(() => null);
+        // A failed credentials read is reported below as "token not configured in
+        // Site Settings", which is actionable but wrong — record the real cause.
+        const credentials = await safeRead(() => siteSettingsRepository.getDecryptedCredentials(), { route: "/", key: "homepage.socialFeed.credentials", fallback: null });
         switch (platform) {
             case "instagram": {
                 const token = credentials?.metaPageAccessToken;

package/dist/features/homepage/hooks/useBlogArticles.js

@@ -23,7 +23,7 @@ export function useBlogArticles(options) {
                 return featuredResult;
             }
             const remaining = MIN_BLOG_COUNT - featured.length;
-            const latestResult = await apiClient.get(`${BLOG_ENDPOINTS.SORTED("-publishedAt")}&perPage=${MIN_BLOG_COUNT + remaining}`);
+            const latestResult = await apiClient.get(`${BLOG_ENDPOINTS.SORTED("-publishedAt")}&pageSize=${MIN_BLOG_COUNT + remaining}`);
             const existingIds = new Set(featured.map((p) => p.id));
             const filler = latestResult.posts
                 .filter((p) => !existingIds.has(p.id))

package/dist/features/homepage/lib/live-stats.js

@@ -81,7 +81,14 @@ export async function fetchLiveStats(requests) {
             }
             catch (_err) {
                 void normalizeError(_err);
-                // silently fall back to static value
+                // Falling back to the admin-authored static value is correct — but a
+                // stale number that looks live is exactly the failure this must not
+                // hide, so name the collection that could not be counted.
+                serverLogger.warn("homepage.liveStats: collection count failed", {
+                    statKey: key,
+                    collection,
+                    error: normalizeError(_err).message,
+                });
             }
         })());
     }

package/dist/features/homepage/repository/carousel.repository.js

@@ -234,10 +234,11 @@ export class CarouselRepository extends BaseRepository {
                 "stats.views": increment(1),
                 "stats.lastViewed": new Date(),
             });
+            // Fire-and-forget analytics: a lost view increment changes nothing that
+            // renders, and it must never break the carousel read it rides along with.
         }
         catch (_e) {
             void normalizeError(_e);
-            // Swallow errors — analytics failures must not break the carousel response
         }
     }
 }