@mohasinac/appkit 4.33.1 → 4.34.0

package/dist/features/seller/utils/product-draft-mapping.js

@@ -0,0 +1,147 @@
+function pick(v) {
+    return v === undefined || v === null || v === "" ? undefined : v;
+}
+/** Drop keys whose value is undefined so they never reach Firestore. */
+function compact(obj) {
+    const out = {};
+    for (const [k, v] of Object.entries(obj)) {
+        if (v !== undefined)
+            out[k] = v;
+    }
+    return out;
+}
+/**
+ * The flat draft keys this mapper OWNS.
+ *
+ * Stripped from the payload after being folded into their nested homes — the
+ * write schema does not name them, so leaving them in would be harmless today
+ * and confusing forever ("is `classifiedCity` stored or not?").
+ */
+const OWNED_FLAT_KEYS = [
+    "classifiedCity", "classifiedLocality", "classifiedPincode",
+    "classifiedContactMethod", "classifiedAcceptsShipping", "classifiedNegotiable",
+    "digitalCodeDelivery", "digitalCodePoolSize",
+    "digitalCodeRedemptionInstructions", "digitalCodeExpiresAt",
+    "liveSpecies", "liveAgeMonths", "liveSex", "liveCareInfo",
+    "liveTransportMethod", "liveHandlingFee", "liveJurisdictions", "liveCites",
+    "printSize", "printMaterial", "printFinish", "printEditionSize",
+];
+/**
+ * Fold the draft's flat per-type keys into the nested blocks the write schema
+ * expects. Every other key passes through untouched.
+ *
+ * A block is emitted only when the draft actually carries something for it, so
+ * saving a standard listing does not write an empty `classified: {}`.
+ */
+export function draftToProductInput(draft) {
+    const out = { ...draft };
+    for (const k of OWNED_FLAT_KEYS)
+        delete out[k];
+    // ── classified ────────────────────────────────────────────────────────────
+    if (pick(draft.classifiedCity) !== undefined) {
+        out.classified = compact({
+            meetupArea: compact({
+                city: draft.classifiedCity,
+                locality: pick(draft.classifiedLocality),
+                pincode: pick(draft.classifiedPincode),
+            }),
+            acceptsShipping: draft.classifiedAcceptsShipping,
+            negotiable: draft.classifiedNegotiable,
+        });
+    }
+    // ── digital code ──────────────────────────────────────────────────────────
+    if (pick(draft.digitalCodeDelivery) !== undefined) {
+        out.digitalCode = compact({
+            codeDeliveryMethod: draft.digitalCodeDelivery,
+            codePoolSize: pick(draft.digitalCodePoolSize),
+            redemptionInstructions: pick(draft.digitalCodeRedemptionInstructions),
+            expiresAt: pick(draft.digitalCodeExpiresAt),
+        });
+    }
+    // ── live item ─────────────────────────────────────────────────────────────
+    if (pick(draft.liveSpecies) !== undefined) {
+        out.liveItem = compact({
+            species: draft.liveSpecies,
+            ageMonths: pick(draft.liveAgeMonths),
+            sex: pick(draft.liveSex),
+            careInfo: pick(draft.liveCareInfo),
+            /*
+             * `transport` is a REQUIRED object on the schema, and `method` inside it
+             * is required too — so a live listing whose form left the transport
+             * method blank must still produce the object, or the whole `liveItem`
+             * block fails to parse and takes the species with it. "courier" is the
+             * form's own default.
+             */
+            transport: compact({
+                method: draft.liveTransportMethod ?? "courier",
+                handlingFee: pick(draft.liveHandlingFee),
+            }),
+            // Required, min 1. Left empty the schema rejects the listing with a
+            // real message — which is correct: shipping a live animal into a
+            // jurisdiction that forbids it is what the field exists to prevent.
+            jurisdictionAllowed: draft.liveJurisdictions ?? [],
+            // `cites` is a STRING permit number on the document, while the form
+            // carries a boolean "CITES applies" checkbox. Only the affirmative case
+            // means anything, and it means "documented" until a number is entered.
+            cites: draft.liveCites ? "documented" : undefined,
+        });
+    }
+    // ── print meta (art + stickers share one block) ───────────────────────────
+    const printEntries = compact({
+        size: pick(draft.printSize),
+        material: pick(draft.printMaterial),
+        finish: pick(draft.printFinish),
+        editionSize: pick(draft.printEditionSize),
+    });
+    if (Object.keys(printEntries).length > 0)
+        out.printMeta = printEntries;
+    return out;
+}
+/**
+ * The inverse — seed the form from a stored document.
+ *
+ * Tolerant of documents written before the nested blocks existed: a missing
+ * block yields undefined flat keys, which render as empty inputs rather than
+ * throwing.
+ */
+export function productToDraft(product) {
+    const { classified, digitalCode, liveItem, printMeta } = product;
+    return compact({
+        /*
+         * Flat fields the edit pages' hand-written `initialValues` blocks omit.
+         *
+         * Not a nesting mismatch like the rest of this function — just fields
+         * nobody added to ten near-identical literals. `finalSale` matters most:
+         * unseeded it renders as `undefined`, which `isFinalSale` reads as TRUE,
+         * so a listing the seller had opted into returns would open its own edit
+         * form showing "final sale". The stored value survives an untouched save
+         * (an absent key is stripped, and the update is partial), but the form was
+         * telling the seller something untrue about their own listing.
+         */
+        finalSale: product.finalSale,
+        returnPolicy: product.returnPolicy,
+        classifiedCity: classified?.meetupArea?.city,
+        classifiedLocality: classified?.meetupArea?.locality,
+        classifiedPincode: classified?.meetupArea?.pincode,
+        classifiedAcceptsShipping: classified?.acceptsShipping,
+        classifiedNegotiable: classified?.negotiable,
+        digitalCodeDelivery: digitalCode?.codeDeliveryMethod,
+        digitalCodePoolSize: digitalCode?.codePoolSize,
+        digitalCodeRedemptionInstructions: digitalCode?.redemptionInstructions,
+        digitalCodeExpiresAt: digitalCode?.expiresAt,
+        liveSpecies: liveItem?.species,
+        liveAgeMonths: liveItem?.ageMonths,
+        liveSex: liveItem?.sex,
+        liveCareInfo: liveItem?.careInfo,
+        liveTransportMethod: liveItem?.transport?.method,
+        liveHandlingFee: liveItem?.transport?.handlingFee,
+        liveJurisdictions: liveItem?.jurisdictionAllowed,
+        // String -> boolean, mirroring the outbound direction: any recorded permit
+        // means the checkbox is ticked.
+        liveCites: liveItem?.cites ? true : undefined,
+        printSize: printMeta?.size,
+        printMaterial: printMeta?.material,
+        printFinish: printMeta?.finish,
+        printEditionSize: printMeta?.editionSize,
+    });
+}

package/dist/features/shell/SectionForm.js

@@ -9,6 +9,7 @@ import { FormShellContext } from "../../ui/forms/FormShell";
 import { useFormBottomActions } from "../layout/hooks/useFormBottomActions";
 import { useSectionState } from "../account/hooks/useCollapsedSections";
 import { FormSchemaContext } from "./FormShell";
+import { EASE_OUT_MS } from "../../tokens/motion";
 /** Sections in render order: required first, original order preserved within each group. */
 export function orderSections(sections) {
     return [...sections.filter((s) => s.required), ...sections.filter((s) => !s.required)];
@@ -78,8 +79,34 @@ export function scrollToSection(id) {
         return;
     requestAnimationFrame(() => {
         const el = document.getElementById(sectionAnchorId(id));
-        el?.scrollIntoView({ behavior: "smooth", block: "start" });
-        el?.querySelector("[aria-invalid='true']")?.focus();
+        if (!el)
+            return;
+        /*
+         * The header is already at its final position, so scrolling can happen on
+         * the next frame.
+         */
+        el.scrollIntoView({ behavior: "smooth", block: "start" });
+        /*
+         * Focusing the offending field cannot.
+         *
+         * `<Collapse>` animates height 0 -> auto over EASE_OUT_MS, and Framer
+         * drives that with rAF rather than a CSS transition — so there is no
+         * `transitionend` to wait for, and one `requestAnimationFrame` lands while
+         * the panel is still ~0px tall inside `overflow: hidden`. Focusing there
+         * makes the browser scroll-anchor against a box that is about to grow,
+         * which is why "jump to error" could leave the field off-screen.
+         *
+         * A required section renders with no <Collapse> at all, so it is already
+         * at full height and this simply fires a tick later than it needs to.
+         */
+        const reduced = typeof window !== "undefined" &&
+            typeof window.matchMedia === "function" &&
+            window.matchMedia("(prefers-reduced-motion: reduce)").matches;
+        const focusInvalid = () => el.querySelector("[aria-invalid='true']")?.focus();
+        if (reduced)
+            focusInvalid();
+        else
+            window.setTimeout(focusInvalid, EASE_OUT_MS);
     });
 }
 export function SectionForm({ sections, values, onChange, onSubmit, submitLabel = "Save", isLoading = false, schema, openIds: controlledOpenIds, onOpenChange, expandMode = "multi", hideActions = false, destructiveAction, onValidationChange, onCancel, cancelLabel, bottomBar = true, }) {
@@ -112,7 +139,20 @@ export function SectionForm({ sections, values, onChange, onSubmit, submitLabel
             setUncontrolledOpenIds(ids);
         onOpenChange?.(ids);
     }, [controlledOpenIds, onOpenChange]);
-    const fieldToSectionIndex = useMemo(() => buildFieldToSectionIndex(ordered), [ordered]);
+    /*
+     * `ordered` depends on `values`, so it is a NEW array on every keystroke.
+     * Anything derived from it must key on the visible section SET instead, or
+     * the effect below re-fires per character — running a second full-tree
+     * `safeParse` on top of the one `handleFieldChange` already performs, and
+     * rebuilding every downstream callback with it.
+     *
+     * `ordered` itself must keep recomputing: a `when()` predicate can flip
+     * mid-typing and a section can appear or disappear. It is only the DERIVED
+     * values that are stable across that.
+     */
+    const visibleSectionKey = ordered.map((s) => s.id).join("|");
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+    const fieldToSectionIndex = useMemo(() => buildFieldToSectionIndex(ordered), [visibleSectionKey]);
     // Keep the latest callback without making runValidation depend on it — the
     // caller usually passes an inline arrow, which would otherwise rebuild the
     // validator on every render and re-fire the effect below in a loop.
@@ -227,14 +267,29 @@ export function SectionForm({ sections, values, onChange, onSubmit, submitLabel
     }, [fieldErrors]);
     return (_jsxs(Stack, { gap: "md", children: [ordered.map((section) => {
                 const isRequired = section.required === true;
-                // A required section is always open — it holds the fields without
-                // which the record cannot be saved, so hiding it is never useful.
+                /*
+                 * A required section is always open — it holds the fields without
+                 * which the record cannot be saved, so hiding it is never useful.
+                 *
+                 * `collapsible={!isRequired}` below is what makes that VISIBLE: the
+                 * header renders as a plain heading with no control. Previously this
+                 * flag only forced `isCollapsed` false while the header still drew a
+                 * chevron button, so the section was permanently open AND appeared
+                 * broken. `onToggle` no longer needs its `if (!isRequired)` guard,
+                 * because a static header has nothing to click.
+                 */
                 const isCollapsed = isRequired
                     ? false
                     : !(openIds.includes(section.id) || forceOpenIds.includes(section.id));
                 const errorCount = errorCountFor(section);
-                return (_jsx(CollapsibleSection, { id: sectionAnchorId(section.id), title: _jsxs(Row, { align: "center", gap: "xs", children: [_jsx(Span, { weight: "semibold", size: "sm", children: section.heading ?? section.label }), isRequired && _jsx(Span, { size: "xs", color: "muted", children: "Required" })] }), isCollapsed: isCollapsed, onToggle: () => { if (!isRequired)
-                        toggle(section.id); }, keepMounted: section.keepMounted, renderHeaderExtra: errorCount > 0
+                return (_jsx(CollapsibleSection, { id: sectionAnchorId(section.id), title: 
+                    /*
+                     * `as="span"` matters: a collapsible section's header renders
+                     * this INSIDE a <button>, and <Row>'s default <div> made that
+                     * <button><div><span><div> — invalid, since a button may only
+                     * contain phrasing content. Same variant props, legal element.
+                     */
+                    _jsxs(Row, { as: "span", align: "center", gap: "xs", className: "min-w-0", children: [_jsx(Span, { weight: "semibold", size: "sm", children: section.heading ?? section.label }), isRequired && _jsx(Span, { size: "xs", color: "muted", children: "Required" })] }), isCollapsed: isCollapsed, collapsible: !isRequired, onToggle: () => toggle(section.id), keepMounted: section.keepMounted, renderHeaderExtra: errorCount > 0
                         ? () => (_jsxs(Badge, { variant: "danger", children: [errorCount, " ", errorCount === 1 ? "issue" : "issues"] }))
                         : undefined, children: section.render({ values, onChange: handleFieldChange, errors: fieldErrors }) }, section.id));
             }), !hideActions && (

package/dist/features/shell/form-sections.js

@@ -92,6 +92,10 @@ export const SECTION_GROUP_DEFAULTS = {
     // logistics
     address: "logistics", shipping: "logistics", delivery: "logistics",
     routing: "logistics",
+    // Returns sit with logistics rather than with pricing: the seller is
+    // describing what happens to the goods after delivery, and the section
+    // reads directly after Shipping on the listing form.
+    returns: "logistics",
     // relationships
     owner: "relationships", members: "relationships", link: "relationships",
     google: "relationships", sync: "relationships", subscribe: "relationships",

package/dist/features/tester/seed-data/tester-checklist-seed-data.js

@@ -1927,6 +1927,48 @@ const rawTesterChecklistItems = [
                 },
             ],
         },
+        {
+            pageKey: "return-request",
+            pageLabel: "Requesting a return \u2014 and what final sale actually blocks",
+            href: "/user/orders",
+            cases: [
+                {
+                    key: "return-cta-only-when-returnable",
+                    label: "The \"Request Return\" button appears only on a DELIVERED order inside the 7-day window",
+                    description: "Added 2026-09-04. BEFORE there was no buyer-facing return flow at all: the action existed, the CTA was registered in ACTIONS.USER, and nothing anywhere rendered it. AFTER: open a delivered order's detail page and the button is there beside Track Shipment. Open a pending or shipped one and it is not. Open a delivered order older than 7 days and the page explains the window has closed rather than offering a form that would be refused.",
+                },
+                {
+                    key: "return-not-received-accepted-on-final-sale",
+                    label: "\"It never arrived\" is accepted on a FINAL SALE order",
+                    description: "This is the whole rule. Final sale refuses a change of mind; it can never refuse a claim that the buyer did not get what they paid for. On a final-sale order, choose \"It never arrived\", tick the three acknowledgements, submit \u2014 it must go through and the order must move to Return Requested.",
+                },
+                {
+                    key: "return-change-of-mind-refused-by-name",
+                    label: "On a final-sale order the change-of-mind reasons are NOT OFFERED, and the panel says why",
+                    description: "Open the return form for a final-sale order. A warning panel names the situation, and the reason picker lists only the five fault reasons \u2014 never-arrived, not-as-described, damaged, wrong item, counterfeit. \"I changed my mind\" must not be selectable. On a returnable order all ten reasons appear. A generic \"request refused\" with no explanation is the failure this replaced.",
+                },
+                {
+                    key: "return-reason-is-persisted",
+                    label: "The reason the buyer picked is actually STORED and visible to staff",
+                    description: "BEFORE: `requestReturnAction` parsed a free-text reason and then threw it away before writing \u2014 an order reached Return Requested with nothing recorded about why, and the seller had nothing to act on. AFTER: submit a return, then open the same order in /admin/return-requests and confirm the coded reason and any note are there.",
+                },
+                {
+                    key: "return-partial-gates-selected-lines-only",
+                    label: "On a mixed order, a final-sale line does not block a return of the returnable line",
+                    description: "Final sale is snapshotted PER LINE at purchase, not per order. On an order containing one final-sale item and one returnable item, a change-of-mind return scoped to the returnable line must be accepted; the same reason scoped to the final-sale line must not.",
+                },
+                {
+                    key: "return-prize-draw-still-refused-entirely",
+                    label: "A prize-draw order is still refused for EVERY reason \u2014 a different, earlier gate",
+                    description: "`isNonRefundable` is a property of the listing TYPE and admits no exception, including a seller-fault one. Open a prize-draw order's refund surface: it must refuse outright and say so, without offering a reason picker at all. Confusing this with final sale in either direction is the failure.",
+                },
+                {
+                    key: "return-terms-snapshotted-at-purchase",
+                    label: "Changing a listing's return terms does NOT change an order already placed",
+                    description: "Buy a returnable listing. Then, as the seller, switch that listing to final sale. Go back to the order and request a change-of-mind return \u2014 it must still be ACCEPTED, because the buyer agreed to the terms shown at checkout. The order carries its own copy; a seller must not be able to withdraw a right after the sale.",
+                },
+            ],
+        },
     ]),
     ...group("selling", "Selling", [
         {
@@ -2162,6 +2204,30 @@ const rawTesterChecklistItems = [
                     description: "BEFORE: an invalid field produced a banner listing the problem with no way to reach it, and on the site-settings page the summary was mounted outside every form context and so displayed NOTHING at all. AFTER: submit a form with a bad value (a negative refund on an order, a non-https third-party ad URL, an AdSense id that is not ca-pub-…) and the summary must list it, tag it with its section, collapse open that section when clicked, and show a count badge on the section header.",
                     href: "/admin/ads",
                 },
+                {
+                    key: "required-section-has-no-dead-chevron",
+                    label: "A REQUIRED section renders as a plain heading \u2014 no chevron, nothing to click",
+                    description: "Fixed 2026-09-04. BEFORE: SectionDef documented a required section as rendering \"without a collapse control so it can't be hidden\", but CollapsibleSection had no way to say that, so SectionForm passed a no-op handler and the header still drew a full button with a rotating chevron. Clicking it did nothing, on ~73 required sections across the app \u2014 which reads as a broken form. AFTER: on any sectionised form (admin product, admin user, store product), the first section's header has no chevron and no hover state, while every optional section below it still expands and collapses normally.",
+                    href: "/admin/products/new",
+                },
+                {
+                    key: "open-section-does-not-clip-dropdowns",
+                    label: "A dropdown opened inside an expanded section is not clipped at the panel edge",
+                    description: "BEFORE: the collapse animation kept `overflow: hidden` in its SETTLED open state, not just during the transition, so anything escaping the panel box was cut off. AFTER: expand a section containing a select or picker near its bottom edge, open it, and the list must render in full rather than being sliced by the section border.",
+                    href: "/admin/products/new",
+                },
+                {
+                    key: "error-jump-lands-on-the-field",
+                    label: "Jumping to an error scrolls to the section AND lands focus on the offending field",
+                    description: "BEFORE: the scroll fired one animation frame after expanding, while the panel was still ~0px tall inside overflow:hidden \u2014 so focus landed against a box that was about to grow and the field could end up off-screen. AFTER: submit a form with an error inside a COLLAPSED section. It expands, scrolls, and the invalid input is both visible and focused once the animation finishes.",
+                    href: "/admin/products/new",
+                },
+                {
+                    key: "long-form-typing-is-smooth",
+                    label: "Typing in a long form does not stutter",
+                    description: "BEFORE: the section list was rebuilt on every keystroke, which re-fired the validation effect on top of the validation the keystroke already triggered \u2014 two full-tree schema parses per character on a 40-field listing form. AFTER: type a long description into the seller product form. It should keep up. This is a feel test, not a pass/fail assertion; report it if it lags.",
+                    href: "/store/products/new",
+                },
                 {
                     key: "form-mobile-action-bar",
                     label: "Save and Cancel appear in the pinned mobile bar, once",
@@ -2410,6 +2476,122 @@ const rawTesterChecklistItems = [
                 },
             ],
         },
+        {
+            pageKey: "final-sale-authoring",
+            pageLabel: "Final sale \u2014 the seller switch",
+            href: "/store/products/new",
+            cases: [
+                {
+                    key: "final-sale-defaults-on",
+                    label: "A brand-new listing is FINAL SALE by default \u2014 the Returns section shows \"Accept change-of-mind returns\" switched OFF",
+                    description: "Added 2026-09-04. Open the Returns section on a new listing of any type. The toggle is off and the panel explains that not-received / damaged / wrong-item / not-as-described / counterfeit claims are still accepted. Absent means final sale, so a listing saved without ever opening this section is also final sale \u2014 which is the point.",
+                    href: "/store/products/new",
+                },
+                {
+                    key: "final-sale-opt-out-survives-reload",
+                    label: "Turning returns ON, saving, and RELOADING shows it still on",
+                    description: "This is the case that matters, and a reload is the only way to see it. `productBaseSchema` is a plain z.object() with no .passthrough(), so it strips every key it does not name \u2014 a field missing from it saves with a success toast and is gone. BEFORE such a field existed at all. AFTER: toggle Accept returns ON, Save, reload the edit page, and the toggle must still be ON. If it reverts to off, `finalSale` is being stripped on write.",
+                    href: "/store/products",
+                },
+                {
+                    key: "return-policy-authorable-by-seller",
+                    label: "A seller can write their own Return policy text \u2014 for the first time",
+                    description: "BEFORE: `returnPolicy` existed on the document and the ONLY editor for it was the admin product form, so a seller could never state their own terms. AFTER: the Returns section on the seller form has a Return policy textarea. Type into it, save, reload, and confirm it persists and appears on the public listing under Delivery & Returns.",
+                    href: "/store/products",
+                },
+                {
+                    key: "final-sale-admin-override",
+                    label: "An admin can flip the same switch on any listing, and it is the SAME field",
+                    description: "Open a listing in the admin product editor. The \"Accept change-of-mind returns\" toggle sits directly above the Return policy box and reflects whatever the seller chose. Change it, save, then reopen the SELLER edit form for that listing \u2014 it must show the admin's value. Two independent flags would be the failure.",
+                    href: "/admin/products",
+                },
+                {
+                    key: "final-sale-badges-render",
+                    label: "A final-sale listing shows the pill on its card AND the line on its detail page",
+                    description: "Seed data opts one listing per type OUT of final sale, so both states are visible. On /products, a final-sale listing carries a dark \"Final Sale\" pill; the opted-out one does not. On the detail page, the Delivery & Returns block always renders a returns line \u2014 even when the seller wrote no shipping or policy text at all, which previously hid the block entirely and with it the most consequential term of the sale.",
+                    href: "/products",
+                },
+            ],
+        },
+        {
+            pageKey: "media-limits",
+            pageLabel: "Media \u2014 ten images and one video, on every listing type",
+            href: "/store/products/new",
+            cases: [
+                {
+                    key: "media-gallery-accepts-images-and-video",
+                    label: "The gallery takes BOTH images and a video, and shows a live per-type count",
+                    description: "Changed 2026-09-04. BEFORE the single-media control on the listing form was video-only (`kind=\"video\"`), so the one \"add media\" affordance a seller saw refused photos. AFTER: the gallery accepts image files and a video file in the same control, and a counter under it reads \"N/10 images \u00b7 N/1 video\" as you add them.",
+                    href: "/store/products/new",
+                },
+                {
+                    key: "media-tenth-image-uploads",
+                    label: "The 10th gallery image uploads \u2014 the 5th used to fail with a 400",
+                    description: "BEFORE: the gallery was capped at 5 and sent its Nth image as index N+1, while the server's signed-URL guard rejected any index above 5 \u2014 so the LAST slot of every product form returned a 400 that read as a random upload failure. AFTER: add ten images one at a time. All ten must succeed, including the tenth.",
+                    href: "/store/products/new",
+                },
+                {
+                    key: "media-eleventh-image-refused-client-side",
+                    label: "The 11th image is refused by the FORM with a clear message, not by the server",
+                    description: "Try to add an 11th image. The control must refuse it before uploading, naming the limit (\"You can upload up to 10 images.\"). A server-side 400 arriving after the bytes have gone up is the wrong behaviour \u2014 the user has already waited.",
+                    href: "/store/products/new",
+                },
+                {
+                    key: "media-second-video-refused-first-kept",
+                    label: "A second video is refused and the FIRST one is kept",
+                    description: "With one video already in the gallery, try to add another. The message must say only one video is allowed and tell you to remove the current one \u2014 and crucially the existing video must still be there afterwards. Silently replacing it would lose work.",
+                    href: "/store/products/new",
+                },
+                {
+                    key: "media-caps-flat-across-types",
+                    label: "The same 10 + 1 applies to an auction and a pre-order \u2014 there is no per-type table",
+                    description: "Repeat the ten-image check on /store/auctions/new and /store/pre-orders/new. Both must allow ten. BEFORE the server carried separate AUCTION_IMAGE_MAX and PREORDER_IMAGE_MAX constants, both 5, disagreeing with everything else.",
+                    href: "/store/auctions/new",
+                },
+                {
+                    key: "media-live-listing-still-requires-video",
+                    label: "A live listing STILL refuses to publish without a video",
+                    description: "Unchanged behaviour, re-verified because the surrounding control changed. Create a live listing with photos but no video and press Publish. It must be refused, and the Video field must be marked required. That is a requiredness rule, not a count rule, and the media-limit work must not have removed it.",
+                    href: "/store/live/new",
+                },
+            ],
+        },
+        {
+            pageKey: "listing-type-fields-roundtrip",
+            pageLabel: "Per-type fields survive a save \u2014 all nine listing types",
+            href: "/store/products",
+            cases: [
+                {
+                    key: "roundtrip-classified-meetup",
+                    label: "A classified's meetup city, locality and pincode come back after a save",
+                    description: "Fixed 2026-09-04, and this was live data loss. BEFORE: the form held `classifiedCity` while the write schema expected `classified.meetupArea.city`, nothing translated between them, and the schema strips unnamed keys \u2014 so the seller pressed Publish, got a success, and the listing saved with none of the fields that made it a classified. AFTER: fill in the meetup details, save, reopen the edit form, and every value must be there.",
+                    href: "/store/classified/new",
+                },
+                {
+                    key: "roundtrip-live-species-jurisdictions",
+                    label: "A live listing's species, transport method and permitted jurisdictions survive",
+                    description: "Same bug, and the most serious instance of it: `jurisdictionAllowed` is the field that prevents shipping a live animal somewhere that forbids it, and it was being dropped. Create a live listing with species, age, sex, care info, transport method, handling fee and at least one jurisdiction. Save, reopen, confirm all of it returned.",
+                    href: "/store/live/new",
+                },
+                {
+                    key: "roundtrip-digital-code-delivery",
+                    label: "A digital code's delivery method, pool size and redemption instructions survive",
+                    href: "/store/digital-codes/new",
+                },
+                {
+                    key: "roundtrip-print-meta",
+                    label: "An art print's size, material, finish and edition size survive \u2014 and so do a sticker sheet's",
+                    description: "Art and stickers share one `printMeta` block, so check both: create one of each with all four fields filled, save, reopen.",
+                    href: "/store/art/new",
+                },
+                {
+                    key: "roundtrip-edit-form-opens-populated",
+                    label: "Opening the edit form for an EXISTING listing shows its per-type fields already filled",
+                    description: "The inverse half of the same bug. BEFORE: the edit pages hand-built their initial values and listed none of the per-type keys, so the form opened blank for them even when the document held real data \u2014 which read as \"my data was deleted\". AFTER: open the edit page for any seeded classified, live, digital-code, art or sticker listing and its per-type fields are populated on arrival.",
+                    href: "/store/classified",
+                },
+            ],
+        },
     ]),
     ...group("content-discovery", "Content & Discovery", [
         {

package/dist/index.d.ts

@@ -1095,7 +1095,6 @@ export { AdminReviewsView } from "./features/admin/index";
 export { AdminSectionsView } from "./features/admin/index";
 export { AdminSessionsManager } from "./features/admin/index";
 export { AdminSidebar } from "./features/admin/index";
-export { AdminSiteView } from "./features/admin/index";
 export { AdminSublistingCategoriesView } from "./features/admin/index";
 export { AdminSublistingCategoryEditorView } from "./features/admin/index";
 export type { AdminSublistingCategoryEditorViewProps } from "./features/admin/index";
@@ -1198,7 +1197,6 @@ export type { AdminReviewsViewProps } from "./features/admin/index";
 export type { AdminSectionsViewProps } from "./features/admin/index";
 export type { AdminSessionsManagerProps } from "./features/admin/index";
 export type { AdminSidebarProps } from "./features/admin/index";
-export type { AdminSiteViewProps } from "./features/admin/index";
 export type { AdminSiteSettingsViewProps } from "./features/admin/index";
 export type { AdminStatCardProps } from "./features/admin/index";
 export type { AdminStoresViewProps } from "./features/admin/index";
@@ -2175,6 +2173,7 @@ export { RefundHistoryTable } from "./features/orders/index";
 export type { RefundHistoryTableProps } from "./features/orders/index";
 export { RefundRequestView } from "./features/orders/index";
 export type { RefundRequestViewProps } from "./features/orders/index";
+export type { RefundRequestSubmission } from "./features/orders/components/RefundRequestView";
 export { PaymentGatewayValues } from "./features/orders/index";
 export { PaymentMethodValues } from "./features/orders/index";
 export { OutOfStockPolicyValues } from "./features/orders/index";
@@ -2386,6 +2385,7 @@ export { getProductFilterKeys } from "./features/products/index";
 export { getProductSortOptions } from "./features/products/index";
 export { getProductTableColumns } from "./features/products/index";
 export { normalizeListingType, isAuctionListing, isPreOrderListing, isStandardListing, isPrizeDrawListing, isClassifiedListing, isDigitalCodeListing, isLiveListing, isArtListing, isStickersListing, } from "./features/products/index";
+export { isFinalSale, acceptsChangeOfMindReturns, } from "./features/products/constants/final-sale";
 export { isListingTypeEnabled, isCategoryTypeEnabled, enabledListingTypes, enabledCategoryTypes, ALL_LISTING_TYPES, } from "./_internal/shared/listing-types/feature-flags";
 export { actionTracker, setActionTrackerSink, resetActionTrackerSink, type ActionEvent, type ActionTrackerSink, } from "./_internal/shared/listing-types/action-tracker";
 export { cartRequiresShipping, cartIsDigitalOnly, cartIsChatOnly, } from "./_internal/shared/listing-types/cart-shipping";
@@ -3072,6 +3072,8 @@ export type { BundleQueryRule } from "./features/categories/schemas/firestore";
 export { BUNDLE_MIN_ITEMS, BUNDLE_MAX_ITEMS, BUNDLE_MAX_PER_USER_DEFAULT, BUNDLES_PAGE_SIZE, BUNDLES_FEATURED_LIMIT, } from "./_internal/shared/features/categories/bundle-config";
 export { BUNDLE_COPY, BUNDLE_STOCK_VARIANT, } from "./_internal/shared/features/categories/bundle-copy";
 export { REFUND_COPY } from "./_internal/shared/features/orders/refund-copy";
+export { RETURN_REASON, FINAL_SALE_EXEMPT_REASONS, RETURN_REASON_LABEL, RETURN_REASON_GROUP_LABEL, isFinalSaleExempt, isReturnReason, selectableReturnReasons, } from "./_internal/shared/features/orders/return-reasons";
+export type { ReturnReason } from "./_internal/shared/features/orders/return-reasons";
 export { bundleCreateSchema, bundleUpdateSchema, bundleQueryRuleSchema, bundleDisplaySchema, type BundleCreateBody, type BundleUpdateBody, } from "./_internal/shared/features/categories/bundle-schemas";
 export type { GroupedListingDocument, GroupTheme, GroupVisibility } from "./features/grouped/schemas/firestore";
 export { navItemFormSchema, navItemCreateSchema, navItemUpdateSchema, type NavItemFormValues, } from "./features/admin/schemas/nav-item-form";
@@ -3144,7 +3146,7 @@ export { LISTING_TYPE_REGISTRY, pluginFor, detectListingTypeFromSlug } from "./_
 export type { ListingTypePlugin } from "./_internal/shared/listing-types/_registry";
 export { getListingRule, getCategoryRule, pickOrderType, getSplitKey, runSyncPreflight, CHECKOUT_RULES, CATEGORY_CHECKOUT_RULES, CHECKOUT_MAX_ORDERS_PER_TX, PRIZE_DRAW_MAX_REVEALS_PER_ORDER, BUNDLE_MAX_QTY_PER_TX, STANDARD_MAX_QTY_PER_LINE, } from "./_internal/shared/checkout/rules";
 export type { ListingCheckoutRule, CategoryCheckoutRule, CartItemProductPair, OrderItemInput, RefundPolicy, } from "./_internal/shared/checkout/rules";
-export { MEGABYTE, MAX_IMAGE_BYTES, MAX_PDF_BYTES, MAX_VIDEO_BYTES, MAX_LABEL, MAX_BYTES, ALLOWED_IMAGE_MIMES, ALLOWED_VIDEO_MIMES, ALLOWED_DOC_MIMES, ALLOWED_MIMES, ALLOWED_TYPES_LABEL, MIME_TO_EXT, PDF_MAGIC, VIDEO_CONVERSION_HINTS, classifyMime, isAllowedMime, maxBytesFor, getConversionHint, } from "./_internal/shared/media/limits";
+export { MEGABYTE, MAX_IMAGE_BYTES, MAX_PDF_BYTES, MAX_VIDEO_BYTES, MAX_LABEL, MAX_BYTES, ALLOWED_IMAGE_MIMES, ALLOWED_VIDEO_MIMES, ALLOWED_DOC_MIMES, ALLOWED_MIMES, ALLOWED_TYPES_LABEL, MIME_TO_EXT, PDF_MAGIC, VIDEO_CONVERSION_HINTS, classifyMime, isAllowedMime, maxBytesFor, getConversionHint, PRODUCT_MAX_IMAGES, PRODUCT_MAX_VIDEOS, PRODUCT_IMAGE_INDEX_MAX, } from "./_internal/shared/media/limits";
 export type { MediaKind, AllowedImageMime, AllowedVideoMime, AllowedDocMime, AllowedMime, } from "./_internal/shared/media/limits";
 export { PRODUCT_FIELDS, PRODUCT_STATUS_TRANSITIONS, ORDER_FIELDS, REVIEW_FIELDS, BID_FIELDS, AD_FIELDS, AD_TRACKED_FIELDS, BLOG_FIELDS, BRAND_FIELDS, WISHLIST_FIELDS, HISTORY_FIELDS, COUPON_USAGE_FIELDS, CAROUSEL_FIELDS, FAQ_FIELDS, HOMEPAGE_SECTION_FIELDS, SITE_SETTINGS_FIELDS, COMMON_FIELDS, OAUTH_STATE_VALUES, SCHEMA_DEFAULTS, } from "./constants/field-names";
 export { SORT_DIR, sortBy } from "./constants/sort";
@@ -3226,3 +3228,4 @@ export { ROLES_TABS, isRolesTabId, ADMIN_SETTINGS_TABS, isAdminSettingsTabId, }
 export type { AdminPageTab, RolesTabId, AdminSettingsTabId, } from "./features/admin/constants/admin-page-tabs";
 export { validateActionIndexEntry, normaliseRoute, } from "./features/search/action-index/validate-entry";
 export type { ActionIndexValidationContext, ActionIndexValidationIssue, } from "./features/search/action-index/validate-entry";
+export { draftToProductInput, productToDraft, } from "./features/seller/utils/product-draft-mapping";

package/dist/index.js

@@ -2126,8 +2126,6 @@ export { AdminSessionsManager } from "./features/admin/index";
 // AdminSidebar - Shared export for admin sidebar.
 export { AdminSidebar } from "./features/admin/index";
 // [CLIENT-SSR]-Runs in both SSR and browser â€" React component or hook that does not depend on browser-only APIs.
-// AdminSiteView - Component for admin site view.
-export { AdminSiteView } from "./features/admin/index";
 // AdminSublistingCategoriesView - Admin list view for sublisting categories (SC1).
 export { AdminSublistingCategoriesView } from "./features/admin/index";
 // AdminSublistingCategoryEditorView - Admin create/edit form for sublisting categories (SC1).
@@ -4367,6 +4365,9 @@ export { getProductTableColumns } from "./features/products/index";
 // normalizeListingType + predicate helpers â€" canonical SB1-G accessors.
 // SB-UNI-F 2026-05-13 â€" Phase 2 predicates added (classified/digital-code/live).
 export { normalizeListingType, isAuctionListing, isPreOrderListing, isStandardListing, isPrizeDrawListing, isClassifiedListing, isDigitalCodeListing, isLiveListing, isArtListing, isStickersListing, } from "./features/products/index";
+// Final sale — ABSENT MEANS TRUE, so these are the only legal reads of the
+// `finalSale` field. See the module docstring for why `!p.finalSale` is wrong.
+export { isFinalSale, acceptsChangeOfMindReturns, } from "./features/products/constants/final-sale";
 // SB-UNI-X4 2026-05-13 â€" per-type feature-flag helpers.
 export { isListingTypeEnabled, isCategoryTypeEnabled, enabledListingTypes, enabledCategoryTypes, ALL_LISTING_TYPES, } from "./_internal/shared/listing-types/feature-flags";
 // SB-UNI-X5 2026-05-13 â€" action telemetry sink.
@@ -5454,6 +5455,10 @@ export { BUNDLE_MIN_ITEMS, BUNDLE_MAX_ITEMS, BUNDLE_MAX_PER_USER_DEFAULT, BUNDLE
 export { BUNDLE_COPY, BUNDLE_STOCK_VARIANT, } from "./_internal/shared/features/categories/bundle-copy";
 // S-SBUNI-RULES 2026-05-13 â€" shared copy for refund / shipping / sibling-payment UI.
 export { REFUND_COPY } from "./_internal/shared/features/orders/refund-copy";
+// Return / refund reasons — the closed enum the final-sale gate branches on.
+// See the module docstring for why the exempt set is "seller or carrier fault"
+// rather than "serious reasons".
+export { RETURN_REASON, FINAL_SALE_EXEMPT_REASONS, RETURN_REASON_LABEL, RETURN_REASON_GROUP_LABEL, isFinalSaleExempt, isReturnReason, selectableReturnReasons, } from "./_internal/shared/features/orders/return-reasons";
 // S-SBUNI-4 follow-up â€" shared admin bundle zod schemas.
 export { bundleCreateSchema, bundleUpdateSchema, bundleQueryRuleSchema, bundleDisplaySchema, } from "./_internal/shared/features/categories/bundle-schemas";
 export { navItemFormSchema, navItemCreateSchema, navItemUpdateSchema, } from "./features/admin/schemas/nav-item-form";
@@ -5520,7 +5525,7 @@ export { LISTING_TYPE_REGISTRY, pluginFor, detectListingTypeFromSlug } from "./_
 // Checkout rule registry â€" pure TS, safe in both client and server bundles.
 export { getListingRule, getCategoryRule, pickOrderType, getSplitKey, runSyncPreflight, CHECKOUT_RULES, CATEGORY_CHECKOUT_RULES, CHECKOUT_MAX_ORDERS_PER_TX, PRIZE_DRAW_MAX_REVEALS_PER_ORDER, BUNDLE_MAX_QTY_PER_TX, STANDARD_MAX_QTY_PER_LINE, } from "./_internal/shared/checkout/rules";
 // Media upload limits â€" pure constants, safe in both client and server bundles.
-export { MEGABYTE, MAX_IMAGE_BYTES, MAX_PDF_BYTES, MAX_VIDEO_BYTES, MAX_LABEL, MAX_BYTES, ALLOWED_IMAGE_MIMES, ALLOWED_VIDEO_MIMES, ALLOWED_DOC_MIMES, ALLOWED_MIMES, ALLOWED_TYPES_LABEL, MIME_TO_EXT, PDF_MAGIC, VIDEO_CONVERSION_HINTS, classifyMime, isAllowedMime, maxBytesFor, getConversionHint, } from "./_internal/shared/media/limits";
+export { MEGABYTE, MAX_IMAGE_BYTES, MAX_PDF_BYTES, MAX_VIDEO_BYTES, MAX_LABEL, MAX_BYTES, ALLOWED_IMAGE_MIMES, ALLOWED_VIDEO_MIMES, ALLOWED_DOC_MIMES, ALLOWED_MIMES, ALLOWED_TYPES_LABEL, MIME_TO_EXT, PDF_MAGIC, VIDEO_CONVERSION_HINTS, classifyMime, isAllowedMime, maxBytesFor, getConversionHint, PRODUCT_MAX_IMAGES, PRODUCT_MAX_VIDEOS, PRODUCT_IMAGE_INDEX_MAX, } from "./_internal/shared/media/limits";
 // ── Field name constants — canonical source of truth for all Firestore field names ──
 // NOTE: STORE_FIELDS, CATEGORY_FIELDS, EVENT_FIELDS, EVENT_ENTRY_FIELDS, PAYOUT_FIELDS,
 // COUPON_FIELDS, CART_FIELDS, USER_FIELDS, ADDRESS_FIELDS, NOTIFICATION_FIELDS,
@@ -5629,3 +5634,7 @@ export { SITE_SETTINGS_TABS, SITE_SETTINGS_TAB_IDS, isSiteSettingsTabId, } from
 export { PAYOUTS_TABS, isPayoutsTabId, SHIPPING_TABS, isShippingTabId, FULFILLMENT_TABS, isFulfillmentTabId, STOREFRONT_TABS, isStorefrontTabId, } from "./features/seller/constants/store-tabs";
 export { ROLES_TABS, isRolesTabId, ADMIN_SETTINGS_TABS, isAdminSettingsTabId, } from "./features/admin/constants/admin-page-tabs";
 export { validateActionIndexEntry, normaliseRoute, } from "./features/search/action-index/validate-entry";
+// Seller listing form draft <-> product write payload. The flat form keys and
+// the nested document blocks are different shapes, and nothing translated
+// between them — see the module docstring.
+export { draftToProductInput, productToDraft, } from "./features/seller/utils/product-draft-mapping";

package/dist/next/routing/route-map.d.ts

@@ -146,6 +146,7 @@ export declare const DEFAULT_ROUTE_MAP: {
         readonly ORDER_DETAIL: (id: string) => string;
         readonly ORDER_CANCEL: (id: string) => string;
         readonly ORDER_TRACK: (id: string) => string;
+        readonly ORDER_RETURN: (id: string) => string;
         readonly ORDER_INVOICE: (id: string) => string;
         readonly ORDER_PAYMENT: (id: string) => string;
         readonly NOTIFICATIONS: "/user/notifications";
@@ -549,6 +550,7 @@ export declare const ROUTES: {
         readonly ORDER_DETAIL: (id: string) => string;
         readonly ORDER_CANCEL: (id: string) => string;
         readonly ORDER_TRACK: (id: string) => string;
+        readonly ORDER_RETURN: (id: string) => string;
         readonly ORDER_INVOICE: (id: string) => string;
         readonly ORDER_PAYMENT: (id: string) => string;
         readonly NOTIFICATIONS: "/user/notifications";

package/dist/next/routing/route-map.js

@@ -150,6 +150,7 @@ export const DEFAULT_ROUTE_MAP = {
         ORDER_DETAIL: (id) => `/user/orders/view/${id}`,
         ORDER_CANCEL: (id) => `/user/orders/${id}/cancel`,
         ORDER_TRACK: (id) => `/user/orders/${id}/track`,
+        ORDER_RETURN: (id) => `/user/orders/${id}/return`,
         ORDER_INVOICE: (id) => `/user/orders/${id}/invoice`,
         ORDER_PAYMENT: (id) => `/user/orders/${id}/payment`,
         NOTIFICATIONS: "/user/notifications",