@mohasinac/appkit 4.30.19 → 4.31.1

package/dist/features/seller/components/SellerSupportView.js

@@ -0,0 +1,100 @@
+"use client";
+import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
+/**
+ * SellerSupportView — the tickets raised about this seller's store.
+ *
+ * ## Why there was nothing here before
+ *
+ * Support has always had a buyer surface and an admin surface and no seller
+ * one — no page, no API route, not even a `ROUTES.STORE.SUPPORT` constant.
+ * The reason was structural rather than an oversight: `SupportTicketDocument`
+ * had no queryable `storeId`, so "the tickets about my store" was not a
+ * question the data could answer.
+ *
+ * There is a category called `store_change_request` — sellers asking an admin
+ * to change a store field they cannot edit themselves — which a seller could
+ * only ever file through the BUYER surface, and then never see again from the
+ * store dashboard.
+ *
+ * ## Read-only, deliberately
+ *
+ * A seller can see and follow a ticket about their store; replying goes
+ * through the ticket thread the buyer and admin already share, and status is
+ * an admin decision. Giving a seller a status control here would let them
+ * close a complaint made against them.
+ */
+import React from "react";
+import { Badge, Span, Stack, Text } from "../../../ui";
+import { DataListingView } from "../../admin/components/DataListingView";
+import { SELLER_ENDPOINTS } from "../../../constants/api-endpoints";
+import { SUPPORT_TICKET_FIELDS } from "../../../constants/field-names";
+import { sieveFilter, SIEVE_OP } from "../../../utils/sieve-builder";
+import { toRelativeDate, toStringValue } from "../../admin/hooks/useAdminListingData";
+/** Status → badge tone. Mirrors `TicketStatusValues`. */
+const STATUS_VARIANT = {
+    open: "info",
+    in_progress: "warning",
+    waiting_on_user: "warning",
+    resolved: "success",
+    closed: "default",
+};
+export function SellerSupportView({ labels = {} }) {
+    const mapRows = React.useCallback((response) => {
+        const items = Array.isArray(response?.items) ? response.items : [];
+        return items.map((raw) => {
+            const t = (raw ?? {});
+            return {
+                id: toStringValue(t.id),
+                primary: toStringValue(t.subject),
+                secondary: toStringValue(t.description).slice(0, 140),
+                status: toStringValue(t.status),
+                category: toStringValue(t.category).replace(/_/g, " "),
+                updated: toRelativeDate(t.updatedAt ?? t.createdAt),
+            };
+        });
+    }, []);
+    return (_jsxs(Stack, { gap: "md", children: [_jsx(DataListingView, { config: {
+                    portal: "seller",
+                    title: labels.title ?? "Support",
+                    subtitle: "Tickets buyers and staff have raised about your store.",
+                    /*
+                     * The box is declared only because the endpoint now serves it —
+                     * `supportTickets` gained a `searchTxt` corpus in this same commit.
+                     * `fields` names what is actually matched, and it is SUBJECT ONLY:
+                     * the description and every message body are PII-encrypted at rest,
+                     * so they are not in the index and a placeholder promising them
+                     * would be a promise the backend cannot keep.
+                     */
+                    search: {
+                        placeholder: "Search by subject…",
+                        mode: "partial",
+                        fields: ["subject"],
+                        commit: "debounce",
+                    },
+                    emptyLabel: "No tickets have been raised about your store.",
+                    queryKey: ["store", "support-tickets"],
+                    endpoint: SELLER_ENDPOINTS.SUPPORT,
+                    filterKeys: ["status"],
+                    defaultSort: "",
+                    sortOptions: [],
+                    mapRows,
+                    getTotal: (response, rows) => response?.total ?? rows.length,
+                    buildFilters: (filterState) => filterState.status
+                        ? sieveFilter(SUPPORT_TICKET_FIELDS.STATUS, SIEVE_OP.EQ, filterState.status)
+                        : undefined,
+                    columns: [
+                        { key: "primary", header: "Subject" },
+                        {
+                            key: "status",
+                            header: "Status",
+                            render: (row) => (_jsx(Badge, { variant: STATUS_VARIANT[row.status] ?? "default", children: row.status.replace(/_/g, " ") })),
+                        },
+                        {
+                            key: "category",
+                            header: "About",
+                            render: (row) => (_jsx(Span, { size: "xs", color: "muted", children: row.category })),
+                        },
+                        { key: "updated", header: "Updated" },
+                    ],
+                } }), _jsx(Text, { size: "xs", color: "muted", children: "Replies and status changes happen in the ticket thread with our support team \u2014 a ticket raised about your store is not one you can close." })] }));
+}

package/dist/features/seller/components/index.d.ts

@@ -1,13 +1,9 @@
 export { SellerDashboardView } from "./SellerDashboardView";
 export type { SellerDashboardViewProps } from "./SellerDashboardView";
 export { SellerProductsView } from "./SellerProductsView";
+export { SellerListingTypeView, SELLER_LISTING_TYPE_SPECS } from "./SellerListingTypeView";
+export type { SellerListingTypeViewProps, SellerListingTypeSpec, SellerListingTypeKey, SellerListingRow } from "./SellerListingTypeView";
 export type { SellerProductsViewProps } from "./SellerProductsView";
-export { SellerAuctionsView } from "./SellerAuctionsView";
-export type { SellerAuctionsViewProps } from "./SellerAuctionsView";
-export { SellerPreOrdersView } from "./SellerPreOrdersView";
-export type { SellerPreOrdersViewProps } from "./SellerPreOrdersView";
-export { SellerPrizeDrawsView } from "./SellerPrizeDrawsView";
-export type { SellerPrizeDrawsViewProps } from "./SellerPrizeDrawsView";
 export { PrintCenterView } from "./PrintCenterView";
 export { SellerOrdersView, SellerOrderDetailPanel } from "./SellerOrdersView";
 export type { SellerOrdersViewProps } from "./SellerOrdersView";
@@ -62,8 +58,6 @@ export { SellerDashboardView as StoreDashboardView } from "./SellerDashboardView
 export type { SellerDashboardViewProps as StoreDashboardViewProps } from "./SellerDashboardView";
 export { SellerProductsView as StoreProductListingsView } from "./SellerProductsView";
 export type { SellerProductsViewProps as StoreProductListingsViewProps } from "./SellerProductsView";
-export { SellerAuctionsView as StoreAuctionsView } from "./SellerAuctionsView";
-export type { SellerAuctionsViewProps as StoreAuctionsViewProps } from "./SellerAuctionsView";
 export { SellerOrdersView as StoreOrdersView } from "./SellerOrdersView";
 export type { SellerOrdersViewProps as StoreOrdersViewProps } from "./SellerOrdersView";
 export { SellerOffersView as StoreOffersView } from "./SellerOffersView";
@@ -132,3 +126,5 @@ export { BarcodeField } from "./BarcodeField";
 export type { BarcodeFieldProps } from "./BarcodeField";
 export { FulfillmentView } from "./FulfillmentView";
 export type { FulfillmentViewProps } from "./FulfillmentView";
+export { SellerSupportView } from "./SellerSupportView";
+export type { SellerSupportViewProps } from "./SellerSupportView";

package/dist/features/seller/components/index.js

@@ -1,8 +1,6 @@
 export { SellerDashboardView } from "./SellerDashboardView";
 export { SellerProductsView } from "./SellerProductsView";
-export { SellerAuctionsView } from "./SellerAuctionsView";
-export { SellerPreOrdersView } from "./SellerPreOrdersView";
-export { SellerPrizeDrawsView } from "./SellerPrizeDrawsView";
+export { SellerListingTypeView, SELLER_LISTING_TYPE_SPECS } from "./SellerListingTypeView";
 export { PrintCenterView } from "./PrintCenterView";
 export { SellerOrdersView, SellerOrderDetailPanel } from "./SellerOrdersView";
 export { SellerOffersView } from "./SellerOffersView";
@@ -34,7 +32,6 @@ export { StoreSidebar } from "./SellerSidebar";
 export { SellerDashboardView as StoreDashboardView } from "./SellerDashboardView";
 // SellerProductsView aliased as StoreProductListingsView (StoreProductsView is the public customer-facing store listing)
 export { SellerProductsView as StoreProductListingsView } from "./SellerProductsView";
-export { SellerAuctionsView as StoreAuctionsView } from "./SellerAuctionsView";
 export { SellerOrdersView as StoreOrdersView } from "./SellerOrdersView";
 export { SellerOffersView as StoreOffersView } from "./SellerOffersView";
 export { SellerCouponsView as StoreCouponsView } from "./SellerCouponsView";
@@ -69,3 +66,4 @@ export { SellerDigitalCodesView } from "./SellerDigitalCodesView";
 export { SellerLiveView } from "./SellerLiveView";
 export { BarcodeField } from "./BarcodeField";
 export { FulfillmentView } from "./FulfillmentView";
+export { SellerSupportView } from "./SellerSupportView";

package/dist/features/seller/index.d.ts

@@ -8,4 +8,3 @@ export * from "./components";
 export * from "./schemas";
 export * from "./columns";
 export { manifest } from "./manifest";
-export { SELLER_PAGE_PERMISSIONS } from "./permission-map";

package/dist/features/seller/index.js

@@ -8,4 +8,3 @@ export * from "./components";
 export * from "./schemas";
 export * from "./columns";
 export { manifest } from "./manifest";
-export { SELLER_PAGE_PERMISSIONS } from "./permission-map";

package/dist/features/seller/schemas/coupon-form.d.ts

@@ -3,6 +3,68 @@ import { z } from "zod";
  * 🛑 `annotate()` must be the OUTERMOST call on each field — it keys a WeakMap
  * by schema instance and every zod wrapper returns a new one.
  */
+/**
+ * The fields both coupon editors share, before either adds its own rules.
+ *
+ * Split out of `sellerCouponFormSchema` so the ADMIN editor can extend it.
+ * It could not before: that schema ends in `.superRefine`, which returns a
+ * ZodEffects with no `.extend()` — which is a large part of why the admin
+ * editor ended up declaring its own five-field `.passthrough()` stub locally
+ * and validating nothing at all.
+ */
+export declare const couponFormBase: z.ZodObject<{
+    code: z.ZodString;
+    type: z.ZodEnum<["percentage", "fixed", "free_shipping"]>;
+    value: z.ZodEffects<z.ZodString, string, string>;
+    minPurchase: z.ZodEffects<z.ZodString, string, string>;
+    maxDiscount: z.ZodEffects<z.ZodString, string, string>;
+    totalLimit: z.ZodEffects<z.ZodString, string, string>;
+    perUserLimit: z.ZodEffects<z.ZodString, string, string>;
+    startDate: z.ZodString;
+    endDate: z.ZodString;
+    isActive: z.ZodBoolean;
+    applicableProducts: z.ZodArray<z.ZodString, "many">;
+    applicableCategories: z.ZodArray<z.ZodString, "many">;
+}, "strip", z.ZodTypeAny, {
+    value: string;
+    code: string;
+    type: "fixed" | "percentage" | "free_shipping";
+    isActive: boolean;
+    endDate: string;
+    maxDiscount: string;
+    minPurchase: string;
+    applicableProducts: string[];
+    applicableCategories: string[];
+    totalLimit: string;
+    perUserLimit: string;
+    startDate: string;
+}, {
+    value: string;
+    code: string;
+    type: "fixed" | "percentage" | "free_shipping";
+    isActive: boolean;
+    endDate: string;
+    maxDiscount: string;
+    minPurchase: string;
+    applicableProducts: string[];
+    applicableCategories: string[];
+    totalLimit: string;
+    perUserLimit: string;
+    startDate: string;
+}>;
+/**
+ * The cross-field rules both editors need. Shared for the same reason the
+ * fields are: a percentage over 100 is the store paying the customer whichever
+ * portal typed it.
+ */
+export declare function refineCouponCommon(v: {
+    type: string;
+    value: string;
+    startDate: string;
+    endDate: string;
+    totalLimit: string;
+    perUserLimit: string;
+}, ctx: z.RefinementCtx): void;
 export declare const sellerCouponFormSchema: z.ZodEffects<z.ZodObject<{
     code: z.ZodString;
     type: z.ZodEnum<["percentage", "fixed", "free_shipping"]>;
@@ -70,3 +132,155 @@ export declare const sellerCouponFormSchema: z.ZodEffects<z.ZodObject<{
     startDate: string;
 }>;
 export type SellerCouponFormValues = z.infer<typeof sellerCouponFormSchema>;
+/**
+ * What `POST /api/store/coupons` accepts.
+ *
+ * The index signature is what lets it pass as a JSON body without a cast at the
+ * callsite — the named fields still constrain what may be set.
+ */
+export interface SellerCouponPayload {
+    [key: string]: string | number | boolean | string[] | undefined;
+    code: string;
+    type: SellerCouponFormValues["type"];
+    value: number;
+    minPurchase?: number;
+    maxDiscount?: number;
+    totalLimit: number;
+    perUserLimit: number;
+    startDate: string;
+    endDate: string;
+    isActive: boolean;
+    applicableProducts?: string[];
+    applicableCategories?: string[];
+}
+/**
+ * The draft, as the route wants it.
+ *
+ * 🛑 One implementation because the conditions have to match the form's, and a
+ * per-callsite conversion is how they stopped matching: the create client
+ * correctly zeroed `value` for a free-shipping coupon and then sent
+ * `maxDiscount` for EVERY type — so a cap typed while the type was
+ * "percentage" survived a switch to fixed or free-shipping, invisibly, because
+ * the input that would have shown it was no longer rendered.
+ *
+ * The rule is now stated once, here, next to the `when` predicates it mirrors:
+ * a field the form hides is a field the payload omits.
+ */
+export declare function couponDraftToPayload(draft: SellerCouponFormValues): SellerCouponPayload;
+/**
+ * A coupon as an ADMIN edits it — the seller's fields plus the ones only the
+ * platform can set.
+ *
+ * `AdminCouponEditorView` declared its own schema locally: five fields, two of
+ * them `.passthrough()`, against nineteen rendered controls. It was passed to
+ * `<Form schema>` and never parsed, so none of the seventeen unlisted controls
+ * was checked by anything — `value` could be 500%, `endDate` could precede
+ * `startDate`, and `buyQuantity` could be zero.
+ *
+ * Extending the shared base rather than restating it is what keeps the two
+ * portals' rules from drifting; that they had already drifted is Root Cause
+ * #59's shape on the validation axis.
+ */
+export declare const adminCouponFormSchema: z.ZodEffects<z.ZodObject<{
+    code: z.ZodString;
+    value: z.ZodEffects<z.ZodString, string, string>;
+    minPurchase: z.ZodEffects<z.ZodString, string, string>;
+    maxDiscount: z.ZodEffects<z.ZodString, string, string>;
+    totalLimit: z.ZodEffects<z.ZodString, string, string>;
+    perUserLimit: z.ZodEffects<z.ZodString, string, string>;
+    startDate: z.ZodString;
+    endDate: z.ZodString;
+    isActive: z.ZodBoolean;
+    applicableProducts: z.ZodArray<z.ZodString, "many">;
+    applicableCategories: z.ZodArray<z.ZodString, "many">;
+} & {
+    type: z.ZodEnum<["percentage", "fixed", "free_shipping", "buy_x_get_y"]>;
+    name: z.ZodString;
+    description: z.ZodUnion<[z.ZodOptional<z.ZodString>, z.ZodLiteral<"">]>;
+    buyQuantity: z.ZodEffects<z.ZodString, string, string>;
+    getQuantity: z.ZodEffects<z.ZodString, string, string>;
+    currentUsage: z.ZodOptional<z.ZodNumber>;
+    firstTimeUserOnly: z.ZodBoolean;
+    appliesToAuctions: z.ZodBoolean;
+}, "strip", z.ZodTypeAny, {
+    value: string;
+    code: string;
+    type: "fixed" | "percentage" | "free_shipping" | "buy_x_get_y";
+    name: string;
+    isActive: boolean;
+    endDate: string;
+    firstTimeUserOnly: boolean;
+    maxDiscount: string;
+    minPurchase: string;
+    buyQuantity: string;
+    getQuantity: string;
+    applicableProducts: string[];
+    applicableCategories: string[];
+    totalLimit: string;
+    perUserLimit: string;
+    startDate: string;
+    appliesToAuctions: boolean;
+    description?: string | undefined;
+    currentUsage?: number | undefined;
+}, {
+    value: string;
+    code: string;
+    type: "fixed" | "percentage" | "free_shipping" | "buy_x_get_y";
+    name: string;
+    isActive: boolean;
+    endDate: string;
+    firstTimeUserOnly: boolean;
+    maxDiscount: string;
+    minPurchase: string;
+    buyQuantity: string;
+    getQuantity: string;
+    applicableProducts: string[];
+    applicableCategories: string[];
+    totalLimit: string;
+    perUserLimit: string;
+    startDate: string;
+    appliesToAuctions: boolean;
+    description?: string | undefined;
+    currentUsage?: number | undefined;
+}>, {
+    value: string;
+    code: string;
+    type: "fixed" | "percentage" | "free_shipping" | "buy_x_get_y";
+    name: string;
+    isActive: boolean;
+    endDate: string;
+    firstTimeUserOnly: boolean;
+    maxDiscount: string;
+    minPurchase: string;
+    buyQuantity: string;
+    getQuantity: string;
+    applicableProducts: string[];
+    applicableCategories: string[];
+    totalLimit: string;
+    perUserLimit: string;
+    startDate: string;
+    appliesToAuctions: boolean;
+    description?: string | undefined;
+    currentUsage?: number | undefined;
+}, {
+    value: string;
+    code: string;
+    type: "fixed" | "percentage" | "free_shipping" | "buy_x_get_y";
+    name: string;
+    isActive: boolean;
+    endDate: string;
+    firstTimeUserOnly: boolean;
+    maxDiscount: string;
+    minPurchase: string;
+    buyQuantity: string;
+    getQuantity: string;
+    applicableProducts: string[];
+    applicableCategories: string[];
+    totalLimit: string;
+    perUserLimit: string;
+    startDate: string;
+    appliesToAuctions: boolean;
+    description?: string | undefined;
+    currentUsage?: number | undefined;
+}>;
+export type AdminCouponFormValues = z.infer<typeof adminCouponFormSchema>;

package/dist/features/seller/schemas/coupon-form.js

@@ -44,7 +44,16 @@ const numericString = (label) => z
  * 🛑 `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 sellerCouponFormSchema = z
+/**
+ * The fields both coupon editors share, before either adds its own rules.
+ *
+ * Split out of `sellerCouponFormSchema` so the ADMIN editor can extend it.
+ * It could not before: that schema ends in `.superRefine`, which returns a
+ * ZodEffects with no `.extend()` — which is a large part of why the admin
+ * editor ended up declaring its own five-field `.passthrough()` stub locally
+ * and validating nothing at all.
+ */
+export const couponFormBase = z
     .object({
     code: annotate(z
         .string()
@@ -57,14 +66,26 @@ export const sellerCouponFormSchema = z
     type: annotate(z.enum(["percentage", "fixed", "free_shipping"]), {
         section: "basics", quick: true, order: 2, row: "pair",
     }),
+    /*
+     * `when` carries what the editor held as JSX conditionals, so a field the
+     * form hides is one the payload cannot carry — see `couponDraftToPayload`.
+     */
     value: annotate(numericString("The discount value"), {
         section: "basics", quick: true, order: 3, row: "pair",
+        when: (v) => v.type !== "free_shipping",
     }),
     minPurchase: annotate(numericString("Minimum purchase"), {
         section: "limits", sectionLabel: "Limits", order: 1, row: "pair",
     }),
+    /*
+     * A percentage cap is meaningless on a fixed or free-shipping coupon. The
+     * input was already hidden for those, and the value was SENT anyway — so a
+     * cap typed while the type was "percentage" persisted after switching to
+     * free shipping, invisibly.
+     */
     maxDiscount: annotate(numericString("Maximum discount"), {
         section: "limits", order: 2, row: "pair",
+        when: (v) => v.type === "percentage",
     }),
     totalLimit: annotate(numericString("Total uses"), {
         section: "limits", order: 3, row: "pair",
@@ -87,30 +108,141 @@ export const sellerCouponFormSchema = z
     applicableCategories: annotate(z.array(z.string()), {
         section: "scope", order: 2, row: "full", kind: "list",
     }),
+});
+/**
+ * The cross-field rules both editors need. Shared for the same reason the
+ * fields are: a percentage over 100 is the store paying the customer whichever
+ * portal typed it.
+ */
+export function refineCouponCommon(v, ctx) {
+    {
+        const issue = (path, message) => ctx.addIssue({ code: z.ZodIssueCode.custom, path: [path], message });
+        // A discount value is required for everything except free shipping, and
+        // a percentage above 100 is not a discount — it is the store paying the
+        // customer. Nothing bounded this before.
+        if (v.type !== "free_shipping") {
+            const amount = Number(v.value);
+            if (v.value.trim() === "")
+                issue("value", "A discount value is required.");
+            else if (amount <= 0)
+                issue("value", "The discount must be greater than zero.");
+            else if (v.type === "percentage" && amount > 100) {
+                issue("value", "A percentage discount cannot exceed 100%.");
+            }
+        }
+        if (v.startDate && v.endDate && v.startDate > v.endDate) {
+            issue("endDate", "The end date must be after the start date.");
+        }
+        // A per-customer limit above the total makes the total unreachable — the
+        // coupon reads as more generous than it can ever be.
+        const total = Number(v.totalLimit);
+        const perUser = Number(v.perUserLimit);
+        if (v.totalLimit.trim() && v.perUserLimit.trim() && perUser > total) {
+            issue("perUserLimit", "Uses per customer cannot exceed the total uses.");
+        }
+    }
+}
+export const sellerCouponFormSchema = couponFormBase.superRefine(refineCouponCommon);
+/**
+ * The draft, as the route wants it.
+ *
+ * 🛑 One implementation because the conditions have to match the form's, and a
+ * per-callsite conversion is how they stopped matching: the create client
+ * correctly zeroed `value` for a free-shipping coupon and then sent
+ * `maxDiscount` for EVERY type — so a cap typed while the type was
+ * "percentage" survived a switch to fixed or free-shipping, invisibly, because
+ * the input that would have shown it was no longer rendered.
+ *
+ * The rule is now stated once, here, next to the `when` predicates it mirrors:
+ * a field the form hides is a field the payload omits.
+ */
+export function couponDraftToPayload(draft) {
+    const isPercentage = draft.type === "percentage";
+    const hasValue = draft.type !== "free_shipping";
+    return {
+        code: draft.code,
+        type: draft.type,
+        value: hasValue ? Number(draft.value) || 0 : 0,
+        minPurchase: draft.minPurchase ? Number(draft.minPurchase) : undefined,
+        maxDiscount: isPercentage && draft.maxDiscount ? Number(draft.maxDiscount) : undefined,
+        totalLimit: Number(draft.totalLimit) || 0,
+        perUserLimit: Number(draft.perUserLimit) || 0,
+        startDate: draft.startDate,
+        endDate: draft.endDate,
+        isActive: draft.isActive,
+        ...(draft.applicableProducts?.length ? { applicableProducts: draft.applicableProducts } : {}),
+        ...(draft.applicableCategories?.length
+            ? { applicableCategories: draft.applicableCategories }
+            : {}),
+    };
+}
+/**
+ * A coupon as an ADMIN edits it — the seller's fields plus the ones only the
+ * platform can set.
+ *
+ * `AdminCouponEditorView` declared its own schema locally: five fields, two of
+ * them `.passthrough()`, against nineteen rendered controls. It was passed to
+ * `<Form schema>` and never parsed, so none of the seventeen unlisted controls
+ * was checked by anything — `value` could be 500%, `endDate` could precede
+ * `startDate`, and `buyQuantity` could be zero.
+ *
+ * Extending the shared base rather than restating it is what keeps the two
+ * portals' rules from drifting; that they had already drifted is Root Cause
+ * #59's shape on the validation axis.
+ */
+export const adminCouponFormSchema = couponFormBase
+    .extend({
+    /*
+     * Widened past the seller's three. Only the platform runs buy-x-get-y —
+     * a seller offering one would be spending another seller's stock.
+     */
+    type: annotate(z.enum(["percentage", "fixed", "free_shipping", "buy_x_get_y"]), {
+        section: "basics", quick: true, order: 2, row: "pair", label: "Coupon type",
+    }),
+    name: annotate(z.string().min(1, "Give the campaign a name.").max(120), {
+        section: "basics", quick: true, order: 4, row: "pair", label: "Campaign name",
+    }),
+    description: annotate(z.string().max(500).optional().or(z.literal("")), {
+        section: "basics", order: 5, row: "full", kind: "textarea",
+    }),
+    buyQuantity: annotate(numericString("Buy quantity"), {
+        section: "basics", order: 6, row: "pair", kind: "number",
+        when: (v) => v.type === "buy_x_get_y",
+    }),
+    getQuantity: annotate(numericString("Get quantity"), {
+        section: "basics", order: 7, row: "pair", kind: "number",
+        when: (v) => v.type === "buy_x_get_y",
+    }),
+    /*
+     * Written by orders, never by this form. `t3-mirror` renders it read-only
+     * instead of as a box an admin could set to whatever they liked — which is
+     * what "Current usage" was, `disabled` by an attribute a programmatic
+     * submit ignores.
+     */
+    currentUsage: annotate(z.number().int().min(0).optional(), {
+        section: "limits", order: 5, row: "pair", derived: true, tier: "t3-mirror",
+        label: "Current usage", help: "Updated by orders.",
+    }),
+    firstTimeUserOnly: annotate(z.boolean(), {
+        section: "scope", order: 3, row: "quarter", label: "First-time users only",
+    }),
+    appliesToAuctions: annotate(z.boolean(), {
+        section: "scope", order: 4, row: "quarter", label: "Applies to auctions",
+    }),
 })
     .superRefine((v, ctx) => {
-    const issue = (path, message) => ctx.addIssue({ code: z.ZodIssueCode.custom, path: [path], message });
-    // A discount value is required for everything except free shipping, and
-    // a percentage above 100 is not a discount — it is the store paying the
-    // customer. Nothing bounded this before.
-    if (v.type !== "free_shipping") {
-        const amount = Number(v.value);
-        if (v.value.trim() === "")
-            issue("value", "A discount value is required.");
-        else if (amount <= 0)
-            issue("value", "The discount must be greater than zero.");
-        else if (v.type === "percentage" && amount > 100) {
-            issue("value", "A percentage discount cannot exceed 100%.");
+    refineCouponCommon(v, ctx);
+    // A buy-x-get-y with a zero on either side is not an offer.
+    if (v.type === "buy_x_get_y") {
+        for (const key of ["buyQuantity", "getQuantity"]) {
+            const n = Number(v[key]);
+            if (v[key].trim() === "" || !Number.isFinite(n) || n < 1) {
+                ctx.addIssue({
+                    code: z.ZodIssueCode.custom,
+                    path: [key],
+                    message: "Enter a whole number of one or more.",
+                });
+            }
         }
     }
-    if (v.startDate && v.endDate && v.startDate > v.endDate) {
-        issue("endDate", "The end date must be after the start date.");
-    }
-    // A per-customer limit above the total makes the total unreachable — the
-    // coupon reads as more generous than it can ever be.
-    const total = Number(v.totalLimit);
-    const perUser = Number(v.perUserLimit);
-    if (v.totalLimit.trim() && v.perUserLimit.trim() && perUser > total) {
-        issue("perUserLimit", "Uses per customer cannot exceed the total uses.");
-    }
 });

package/dist/features/seller/schemas/offer-forms.d.ts

@@ -1,4 +1,5 @@
 import { z } from "zod";
+import type { OfferBounds } from "../../../_internal/shared/features/offers/config";
 export declare const counterOfferFormSchema: z.ZodObject<{
     counterAmount: z.ZodNumber;
     sellerNote: z.ZodOptional<z.ZodString>;
@@ -13,16 +14,24 @@ export type CounterOfferFormValues = z.infer<typeof counterOfferFormSchema>;
 /**
  * Buyer-side Make-an-Offer form.
  *
- * A factory rather than a constant because both bounds depend on the listing:
- * the floor is `minOfferAmount(listedPrice, minOfferPercent)` and the ceiling is
- * the listed price itself. Both are re-checked server-side in `makeOffer` — this
- * exists so the buyer is told before submitting, using the SAME
- * `minOfferAmount()` the server uses, rather than a second rounding rule that
- * could disagree by a rupee.
+ * A factory rather than a constant because both bounds depend on the listing.
+ * It takes the already-resolved `OfferBounds` rather than recomputing them:
+ * `resolveOfferBounds()` is what `makeOffer` enforces, so anything this schema
+ * accepts, the server accepts. Deriving a second floor here is exactly how the
+ * forms came to advertise a 50% minimum the server rejected at 70 (see the
+ * offers config header).
+ *
+ * Three shapes, all from the same bounds object:
+ *
+ * - **Buy request** (`isBuyRequest`) — no cart on this type and no haggling
+ *   opted in, so `min === max` and the only legal amount is the asking price.
+ * - **Offer, no cart** — floor to asking price *inclusive*: with no Add to Cart
+ *   button, refusing a full-price offer would leave the buyer nowhere to go.
+ * - **Offer, cartable** — the original rule, ceiling exclusive of the price.
  */
 export declare function makeOfferFormSchema(opts: {
     listedPrice: number;
-    minOffer: number;
+    bounds: OfferBounds;
     formatAmount: (n: number) => string;
 }): z.ZodObject<{
     offerAmount: z.ZodNumber;