@mohasinac/appkit 4.60.1 → 4.62.0

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

@@ -37,6 +37,7 @@ import { jsx as _jsx, Fragment as _Fragment, jsxs as _jsxs } from "react/jsx-run
  */
 import React, { useCallback, useState } from "react";
 import { Badge, ConfirmDeleteModal, RowActionMenu, Span, Text } from "../../../ui/index.js";
+import { apiClient } from "../../../http/index.js";
 import { useEntityDelete } from "../../../react/hooks/useEntityDelete.js";
 import { SELLER_ENDPOINTS } from "../../../constants/api-endpoints.js";
 import { ACTIONS } from "../../../_internal/shared/actions/action-registry.js";
@@ -46,6 +47,7 @@ import { PRODUCT_FIELDS } from "../../../constants/field-names.js";
 import { SIEVE_OP, sieveFilter } from "../../../utils/sieve-builder.js";
 import { pluginFor } from "../../../_internal/shared/listing-types/_registry.js";
 import { toCurrency, toRecordArray, toRelativeDate, toStringValue, } from "../../admin/hooks/useAdminListingData.js";
+import { useOptimisticRowFlag } from "../../admin/hooks/useOptimisticRowFlag.js";
 import { DataListingView } from "../../admin/components/DataListingView.js";
 import { ROW_ACTION_META, SELLER_BULK_ACTIONS } from "../../products/constants/action-defs.js";
 import { useAvailabilityScope } from "../../products/hooks/useAvailabilityScope.js";
@@ -204,6 +206,17 @@ export function SellerListingTypeView({ type, onCreateClick, onEditClick, onDele
         await performDelete(id);
         setDeleteTargetId(null);
     }, [performDelete]);
+    /*
+     * Optimistic status, via the shared hook — see its header for why a
+     * rollback must DELETE the override key rather than write back a captured
+     * value, and why the key is cleared on success too.
+     */
+    const statusFlag = useOptimisticRowFlag({
+        mutate: (id, next) => apiClient.patch(SELLER_ENDPOINTS.PRODUCT_BY_ID(id), { status: next }),
+        invalidate: [["seller", plugin.tabSlug]],
+        successMessage: (next) => next === "published" ? "Listing published." : "Listing unpublished.",
+        errorMessage: "Could not change that listing's status.",
+    });
     const handleEdit = useCallback((id) => {
         if (onEditClick)
             onEditClick(id);
@@ -233,7 +246,15 @@ export function SellerListingTypeView({ type, onCreateClick, onEditClick, onDele
         {
             key: "status",
             header: "Status",
-            render: (row) => (_jsx(Badge, { variant: row.status === "active" ? "active" : "inactive", size: "xs", className: "capitalize", children: row.status })),
+            /*
+             * Reads the optimistic override so the badge and the row action agree
+             * the instant one is used — otherwise the menu would say "Unpublish"
+             * while the badge beside it still said published, until the refetch.
+             */
+            render: (row) => {
+                const status = statusFlag.resolve(row.id, row.status);
+                return (_jsx(Badge, { variant: status === "active" || status === "published" ? "active" : "inactive", size: "xs", className: "capitalize", children: status }));
+            },
         },
         textCell("createdAt", spec.createdHeader ?? "Created"),
     ];
@@ -291,15 +312,40 @@ export function SellerListingTypeView({ type, onCreateClick, onEditClick, onDele
                 },
             }))
             : undefined,
-        renderRowActions: (row) => (_jsx(RowActionMenu, { actions: [
-                { label: ACTIONS.STORE["edit-listing"].label, onClick: () => handleEdit(row.id) },
-                {
-                    label: ACTIONS.STORE["delete-listing"].label,
-                    destructive: true,
-                    onClick: () => setDeleteTargetId(row.id),
-                    disabled: deletingId === row.id,
-                },
-            ] })),
+        renderRowActions: (row) => {
+            const isPublished = statusFlag.resolve(row.id, row.status) === "published";
+            return (_jsx(RowActionMenu, { actions: [
+                    { label: ACTIONS.STORE["edit-listing"].label, onClick: () => handleEdit(row.id) },
+                    /*
+                     * 🛑 This one entry lights up SIX seller pages — art, classified,
+                     * digital codes, live, spares and stickers are each a ~25-line
+                     * wrapper around this component.
+                     *
+                     * `status` was a read-only Badge on all of them, so changing one
+                     * listing from published to draft meant opening the full editor,
+                     * even though `PATCH /api/store/products/[id]` has always taken
+                     * `status` and SellerProductsView already drives that exact call.
+                     *
+                     * `ACTIONS.STORE["publish-listing"]` / `["unpublish-listing"]`
+                     * were defined with ZERO call sites — this is what finally wires
+                     * them, rather than reinventing the labels inline.
+                     */
+                    {
+                        label: isPublished
+                            ? ACTIONS.STORE["unpublish-listing"].label
+                            : ACTIONS.STORE["publish-listing"].label,
+                        separator: true,
+                        disabled: statusFlag.pendingId === row.id,
+                        onClick: () => { void statusFlag.set(row.id, isPublished ? "draft" : "published"); },
+                    },
+                    {
+                        label: ACTIONS.STORE["delete-listing"].label,
+                        destructive: true,
+                        onClick: () => setDeleteTargetId(row.id),
+                        disabled: deletingId === row.id,
+                    },
+                ] }));
+        },
     };
     return (_jsxs(_Fragment, { children: [_jsx(DataListingView, { config: config }), deleteTargetId && (_jsx(ConfirmDeleteModal, { isOpen: true, title: `Delete ${spec.deleteLabel}`, message: `Are you sure you want to delete this ${spec.deleteLabel.toLowerCase()}? This cannot be undone.`, onConfirm: () => handleDelete(deleteTargetId), onClose: () => setDeleteTargetId(null), isDeleting: deletingId === deleteTargetId }))] }));
 }

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

@@ -8,6 +8,8 @@ import { Badge, ConfirmDeleteModal, Div, Row, RowActionMenu, Span, Text, useToas
 import { SELLER_ENDPOINTS } from "../../../constants/api-endpoints.js";
 import { ROUTES } from "../../../constants/index.js";
 import { toRecordArray, toStringValue } from "../hooks/useSellerListingData.js";
+import { useOptimisticRowFlag } from "../../admin/hooks/useOptimisticRowFlag.js";
+import { apiClient } from "../../../http/index.js";
 import { DataListingView } from "../../admin/components/DataListingView.js";
 const CLS_TYPE_PILL = "inline-flex items-center rounded-full px-[var(--appkit-space-2)] py-[var(--appkit-space-0-5)] text-[length:var(--appkit-text-xs)] font-semibold uppercase tracking-wide bg-violet-100 text-violet-700 dark:bg-violet-900/40 dark:text-violet-300";
 const DEFAULT_SORT = "-createdAt";
@@ -42,6 +44,22 @@ const COLUMNS = [
     },
 ];
 export function SellerPayoutMethodsView({ onCreateClick, onEditClick, onDelete, onSetDefault, }) {
+    /*
+     * 🛑 `isActive` was a READ-ONLY badge here, and the update schema was built
+     * to take it on its own — shipping-config's even says so in a comment: "a
+     * PATCH may legitimately carry only `isActive`". Nothing in the client had
+     * ever sent one, so flipping one boolean meant a full-editor round trip.
+     *
+     * The optimistic/rollback logic lives in the hook because eight listings
+     * grew this control at once; see its header for why rollback must DELETE
+     * the override key rather than write back a captured value.
+     */
+    const activeFlag = useOptimisticRowFlag({
+        mutate: (id, next) => apiClient.patch(SELLER_ENDPOINTS.PAYOUT_METHOD_BY_ID(id), { isActive: next }),
+        invalidate: [["seller", "payout-methods", "listing"]],
+        successMessage: (next) => (next ? "Payout method activated." : "Payout method deactivated."),
+        errorMessage: "Could not change that payout method's status.",
+    });
     const [deleteTargetId, setDeleteTargetId] = useState(null);
     const { deletingId, handleDelete: performDelete } = useEntityDelete({
         endpoint: SELLER_ENDPOINTS.PAYOUT_METHOD_BY_ID,
@@ -114,13 +132,27 @@ export function SellerPayoutMethodsView({ onCreateClick, onEditClick, onDelete,
             label: String(item.label ?? ""),
             type: String(item.type ?? ""),
             isDefault: Boolean(item.isDefault),
-            isActive: Boolean(item.isActive),
+            // Resolved through the pending override so the Status badge and the
+            // row toggle cannot disagree between the click and the refetch.
+            isActive: activeFlag.resolve(toStringValue(item.id, `pm-${index}`), Boolean(item.isActive)),
             maskedIdentifier: getMaskedIdentifier(item),
         })),
         getTotal: (response, mappedRows) => (typeof response.total === "number" ? response.total : mappedRows.length),
         buildFilters: () => undefined,
         primaryAction: { label: "New Method", onClick: handleCreate },
         renderRowActions: (row) => (_jsx(RowActionMenu, { actions: [
+                /*
+                 * 🛑 `isActive` was a read-only badge and the update schema was
+                 * built to accept it alone. A menu entry rather than a switch here,
+                 * to match the other actions in this menu — the menu is already
+                 * click-stopped in all three renderers.
+                 */
+                {
+                    label: row.isActive ? "Deactivate" : "Activate",
+                    separator: true,
+                    disabled: activeFlag.pendingId === row.id,
+                    onClick: () => { void activeFlag.set(row.id, !row.isActive); },
+                },
                 ...(!row.isDefault
                     ? [{ label: "Set as Default", disabled: settingDefaultId === row.id, onClick: () => handleSetDefault(row.id) }]
                     : []),

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

@@ -1,5 +1,5 @@
 "use client";
-import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
+import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { useCallback, useEffect, useMemo, useState } from "react";
 import { z } from "zod";
 import { Alert, Badge, Checkbox, Div, FormField, FormGroup, Heading, Label, Row, Span, Stack, Text, Toggle } from "../../../ui/index.js";
@@ -8,6 +8,7 @@ import { FormShellContext, useFormShellState, applyZodIssues, FormErrorSummary }
 import { SectionForm, useSectionFormNav, annotate } from "../../shell/index.js";
 import { SELLER_ENDPOINTS } from "../../../constants/api-endpoints.js";
 import { normalizeError } from "../../../errors/normalize.js";
+import { toUserMessage } from "../../../errors/error-display-map.js";
 const __P = {
     p3: "p-[var(--appkit-space-3)]",
 };
@@ -153,6 +154,20 @@ const DEFAULT_DRAFT = {
      */
     advancedListingTypesEnabled: false,
 };
+/**
+ * One radio group for a single-value field, with ONE error message.
+ *
+ * Extracted because the payout-method and account-type groups were the same
+ * eleven lines twice, and because adding the error handling to both tipped
+ * this view past `audit-code-quality`'s component-size threshold. The error
+ * placement is the part worth sharing: `aria-invalid` goes on every radio so
+ * assistive tech marks the whole group, but the message renders ONCE
+ * underneath — the field is one value, so an `error` per radio would print
+ * and announce the same sentence for every option.
+ */
+function RadioFieldGroup({ name, value, options, error, disabled, onSelect, renderOption, layout: Layout, }) {
+    return (_jsxs(_Fragment, { children: [_jsx(Layout, { gap: "sm", children: options.map((option) => (_jsxs(Label, { layout: "flex", gap: "md", className: "cursor-pointer", children: [_jsx(Checkbox, { bare: true, type: "radio", name: name, value: option, checked: value === option, onChange: () => onSelect(option), className: "accent-[var(--appkit-color-primary)]", disabled: disabled, "aria-invalid": error ? "true" : undefined }), renderOption(option)] }, option))) }), error && (_jsx(Text, { size: "sm", variant: "error", role: "alert", children: error }))] }));
+}
 export function SellerPayoutSettingsView({ apiBase = SELLER_ENDPOINTS.PAYOUT_SETTINGS }) {
     const [draft, setDraft] = useState(DEFAULT_DRAFT);
     const [current, setCurrent] = useState(null);
@@ -253,7 +268,22 @@ export function SellerPayoutSettingsView({ apiBase = SELLER_ENDPOINTS.PAYOUT_SET
                 if (Array.isArray(json?.issues) && json.issues.length > 0) {
                     applyZodIssues(json.issues, setFieldError);
                 }
-                setError(json?.error ?? "Failed to save");
+                /*
+                 * `json.error` is the SERVER's sentence — never shown to a user
+                 * (Rule #9.6). Resolve the stable `code` instead; `toUserMessage`
+                 * terminates in this fallback and can never return server text.
+                 *
+                 * Only a banner when the issues above attached nothing: if they did,
+                 * the fields now carry the detail and a banner repeating a vaguer
+                 * version of it is noise.
+                 */
+                const attached = Array.isArray(json?.issues) &&
+                    json.issues.some((i) => i?.path?.length);
+                if (!attached) {
+                    setError(toUserMessage(json?.code, undefined, {
+                        fallback: "We couldn't save your payout settings.",
+                    }));
+                }
                 return;
             }
             clearErrors();
@@ -263,8 +293,17 @@ export function SellerPayoutSettingsView({ apiBase = SELLER_ENDPOINTS.PAYOUT_SET
             setDraft((prev) => ({ ...prev, accountNumber: "" }));
         }
         catch (err) {
-            void normalizeError(err);
-            setError(err.message);
+            /*
+             * `normalizeError` was already being called and its result DISCARDED,
+             * then the raw `.message` shown anyway — a thrown value's message is
+             * written for a developer, and this is the path a network failure or a
+             * bundler fault takes (Root Cause #86 rendered a require stack this
+             * way). Use the normalized code; the sentence is ours.
+             */
+            const normalized = normalizeError(err);
+            setError(toUserMessage(normalized.code, undefined, {
+                fallback: "We couldn't save your payout settings. Please try again.",
+            }));
         }
         finally {
             setSaving(false);
@@ -283,12 +322,12 @@ export function SellerPayoutSettingsView({ apiBase = SELLER_ENDPOINTS.PAYOUT_SET
             required: true,
             quick: true,
             fields: ["method", "upiId", "accountHolderName", "accountNumber", "ifscCode", "bankName", "accountType"],
-            render: ({ values, onChange }) => (_jsxs(Stack, { gap: "md", children: [_jsx(Heading, { level: 3, className: "mb-2", children: "Payout Method" }), current && (_jsx(Row, { className: "mb-2", align: "center", gap: "sm", children: _jsx(Badge, { variant: current.isConfigured ? "success" : "warning", children: current.isConfigured ? "Payout configured" : "Not configured" }) })), current?.isConfigured && (_jsx(Alert, { variant: "info", children: current.method === "upi"
+            render: ({ values, onChange, errors }) => (_jsxs(Stack, { gap: "md", children: [_jsx(Heading, { level: 3, className: "mb-2", children: "Payout Method" }), current && (_jsx(Row, { className: "mb-2", align: "center", gap: "sm", children: _jsx(Badge, { variant: current.isConfigured ? "success" : "warning", children: current.isConfigured ? "Payout configured" : "Not configured" }) })), current?.isConfigured && (_jsx(Alert, { variant: "info", children: current.method === "upi"
                             ? `Current UPI: ${current.upiId}`
-                            : `Current bank: ${current.bankAccount?.bankName ?? ""} — ••••${current.bankAccount?.accountNumberMasked?.slice(-4) ?? "••••"} (${current.bankAccount?.accountType})` })), _jsx(Stack, { gap: "sm", children: ([
-                            { value: "upi", label: "UPI", desc: "Instant payouts via UPI VPA (e.g. name@upi)." },
-                            { value: "bank_transfer", label: "Bank Transfer", desc: "NEFT/RTGS to your bank account within 2–3 business days." },
-                        ]).map(({ value, label, desc }) => (_jsxs("label", { className: "flex items-center gap-[var(--appkit-space-3)] p-[var(--appkit-space-3)] rounded-lg border border-[var(--appkit-color-border)] cursor-pointer has-[:checked]:border-[var(--appkit-color-primary)] has-[:checked]:bg-[var(--appkit-color-primary)]/5", children: [_jsx(Checkbox, { bare: true, type: "radio", name: "payoutMethod", value: value, checked: values.method === value, onChange: () => onChange({ method: value }), className: "accent-[var(--appkit-color-primary)]", disabled: busy }), _jsxs(Div, { children: [_jsx(Text, { weight: "medium", children: label }), _jsx(Text, { className: "text-[var(--appkit-color-text-muted)]", size: "sm", children: desc })] })] }, value))) }), values.method === "upi" && (_jsx(FormField, { name: "upiId", label: "UPI ID (VPA)", type: "text", value: values.upiId, onChange: (v) => onChange({ upiId: v }), placeholder: "yourname@upi", helpText: "Ensure this VPA is registered and active.", disabled: busy })), values.method === "bank_transfer" && (_jsxs(Stack, { gap: "md", className: "mt-2", children: [_jsx(FormField, { name: "accountHolderName", label: "Account Holder Name", type: "text", value: values.accountHolderName, onChange: (v) => onChange({ accountHolderName: v }), placeholder: "Name as on bank account", disabled: busy }), _jsxs(FormGroup, { columns: 2, children: [_jsx(FormField, { name: "accountNumber", label: "Account Number", type: "text", value: values.accountNumber, onChange: (v) => onChange({ accountNumber: v }), placeholder: "Enter full account number", helpText: current?.bankAccount ? `Saved: ••••${current.bankAccount.accountNumberMasked?.slice(-4) ?? "••••"}` : "Stored securely, never displayed in full.", disabled: busy }), _jsx(FormField, { name: "ifscCode", label: "IFSC Code", type: "text", value: values.ifscCode, onChange: (v) => onChange({ ifscCode: v.toUpperCase() }), placeholder: "e.g. SBIN0001234", disabled: busy })] }), _jsxs(FormGroup, { columns: 2, children: [_jsx(FormField, { name: "bankName", label: "Bank Name", type: "text", value: values.bankName, onChange: (v) => onChange({ bankName: v }), placeholder: "e.g. State Bank of India", disabled: busy }), _jsxs(Div, { children: [_jsx(Text, { className: "mb-1.5", size: "sm", weight: "medium", children: "Account Type" }), _jsx(Row, { gap: "xs", children: ["savings", "current"].map((t) => (_jsxs(Label, { layout: "flex", gap: "md", className: "cursor-pointer", children: [_jsx(Checkbox, { bare: true, type: "radio", name: "accountType", value: t, checked: values.accountType === t, onChange: () => onChange({ accountType: t }), className: "accent-[var(--appkit-color-primary)]", disabled: busy }), _jsx(Text, { size: "sm", transform: "capitalize", children: t })] }, t))) })] })] })] }))] })),
+                            : `Current bank: ${current.bankAccount?.bankName ?? ""} — ••••${current.bankAccount?.accountNumberMasked?.slice(-4) ?? "••••"} (${current.bankAccount?.accountType})` })), _jsxs(Stack, { gap: "sm", children: [([
+                                { value: "upi", label: "UPI", desc: "Instant payouts via UPI VPA (e.g. name@upi)." },
+                                { value: "bank_transfer", label: "Bank Transfer", desc: "NEFT/RTGS to your bank account within 2–3 business days." },
+                            ]).map(({ value, label, desc }) => (_jsxs("label", { className: "flex items-center gap-[var(--appkit-space-3)] p-[var(--appkit-space-3)] rounded-lg border border-[var(--appkit-color-border)] cursor-pointer has-[:checked]:border-[var(--appkit-color-primary)] has-[:checked]:bg-[var(--appkit-color-primary)]/5", children: [_jsx(Checkbox, { bare: true, type: "radio", name: "payoutMethod", value: value, checked: values.method === value, onChange: () => onChange({ method: value }), className: "accent-[var(--appkit-color-primary)]", disabled: busy, "aria-invalid": errors.method ? "true" : undefined }), _jsxs(Div, { children: [_jsx(Text, { weight: "medium", children: label }), _jsx(Text, { className: "text-[var(--appkit-color-text-muted)]", size: "sm", children: desc })] })] }, value))), errors.method && (_jsx(Text, { size: "sm", variant: "error", role: "alert", children: errors.method }))] }), values.method === "upi" && (_jsx(FormField, { name: "upiId", label: "UPI ID (VPA)", type: "text", value: values.upiId, onChange: (v) => onChange({ upiId: v }), placeholder: "yourname@upi", helpText: "Ensure this VPA is registered and active.", disabled: busy })), values.method === "bank_transfer" && (_jsxs(Stack, { gap: "md", className: "mt-2", children: [_jsx(FormField, { name: "accountHolderName", label: "Account Holder Name", type: "text", value: values.accountHolderName, onChange: (v) => onChange({ accountHolderName: v }), placeholder: "Name as on bank account", disabled: busy }), _jsxs(FormGroup, { columns: 2, children: [_jsx(FormField, { name: "accountNumber", label: "Account Number", type: "text", value: values.accountNumber, onChange: (v) => onChange({ accountNumber: v }), placeholder: "Enter full account number", helpText: current?.bankAccount ? `Saved: ••••${current.bankAccount.accountNumberMasked?.slice(-4) ?? "••••"}` : "Stored securely, never displayed in full.", disabled: busy }), _jsx(FormField, { name: "ifscCode", label: "IFSC Code", type: "text", value: values.ifscCode, onChange: (v) => onChange({ ifscCode: v.toUpperCase() }), placeholder: "e.g. SBIN0001234", disabled: busy })] }), _jsxs(FormGroup, { columns: 2, children: [_jsx(FormField, { name: "bankName", label: "Bank Name", type: "text", value: values.bankName, onChange: (v) => onChange({ bankName: v }), placeholder: "e.g. State Bank of India", disabled: busy }), _jsxs(Div, { children: [_jsx(Text, { className: "mb-1.5", size: "sm", weight: "medium", children: "Account Type" }), _jsx(RadioFieldGroup, { name: "accountType", value: values.accountType, options: ["savings", "current"], error: errors.accountType, disabled: busy, onSelect: (t) => onChange({ accountType: t }), layout: Row, renderOption: (t) => (_jsx(Text, { size: "sm", transform: "capitalize", children: t })) })] })] })] }))] })),
         },
         {
             id: "tax",

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

@@ -641,7 +641,11 @@ export function SellerProductsView({ onDeleteProduct, onCreateClick, }) {
                 : `/products/${row.id}`,
         renderRowActions: (row) => {
             const isPublished = (statusOverrides.get(row.id) ?? row.status) === "published";
-            return (_jsxs(Row, { gap: "xs", children: [_jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); handleEdit(row); }, "aria-label": ACTIONS.STORE["edit-listing"].ariaLabel, children: _jsx(Pencil, { className: "w-4 h-4" }) }), _jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); void openAssembly(row, "assemble"); }, "aria-label": ACTIONS.STORE["assemble-from-parts"].ariaLabel, title: ACTIONS.STORE["assemble-from-parts"].label, disabled: assembleLoading, children: _jsx(Wrench, { className: "w-4 h-4" }) }), _jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); void openAssembly(row, "part-out"); }, "aria-label": ACTIONS.STORE["part-out-listing"].ariaLabel, title: ACTIONS.STORE["part-out-listing"].label, disabled: assembleLoading, children: _jsx(Scissors, { className: "w-4 h-4" }) }), _jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); void handleTogglePublish(row); }, "aria-label": isPublished ? "Unpublish" : "Publish", title: isPublished ? "Unpublish" : "Publish", disabled: publishingId === row.id, children: isPublished ? _jsx(EyeOff, { className: "w-4 h-4" }) : _jsx(Eye, { className: "w-4 h-4" }) }), _jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); void handleDuplicate(row); }, "aria-label": "Duplicate listing", title: "Duplicate", children: "\u29C9" }), onDeleteProduct && (_jsx(Button, { variant: "ghost", size: "sm", action: ACTIONS.STORE["delete-listing"], onClick: (e) => { e.stopPropagation(); void handleDelete(row); }, disabled: deletingId === row.id, className: "text-[var(--appkit-color-error)] hover:bg-[var(--appkit-color-border-subtle)]", children: _jsx(Trash2, { className: "w-4 h-4" }) }))] }));
+            return (_jsxs(Row, { gap: "xs", children: [_jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); handleEdit(row); }, "aria-label": ACTIONS.STORE["edit-listing"].ariaLabel, children: _jsx(Pencil, { className: "w-4 h-4" }) }), _jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); void openAssembly(row, "assemble"); }, "aria-label": ACTIONS.STORE["assemble-from-parts"].ariaLabel, title: ACTIONS.STORE["assemble-from-parts"].label, disabled: assembleLoading, children: _jsx(Wrench, { className: "w-4 h-4" }) }), _jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); void openAssembly(row, "part-out"); }, "aria-label": ACTIONS.STORE["part-out-listing"].ariaLabel, title: ACTIONS.STORE["part-out-listing"].label, disabled: assembleLoading, children: _jsx(Scissors, { className: "w-4 h-4" }) }), _jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); void handleTogglePublish(row); }, "aria-label": isPublished
+                            ? ACTIONS.STORE["unpublish-listing"].label
+                            : ACTIONS.STORE["publish-listing"].label, title: isPublished
+                            ? ACTIONS.STORE["unpublish-listing"].description
+                            : ACTIONS.STORE["publish-listing"].description, disabled: publishingId === row.id, children: isPublished ? _jsx(EyeOff, { className: "w-4 h-4" }) : _jsx(Eye, { className: "w-4 h-4" }) }), _jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); void handleDuplicate(row); }, "aria-label": "Duplicate listing", title: "Duplicate", children: "\u29C9" }), onDeleteProduct && (_jsx(Button, { variant: "ghost", size: "sm", action: ACTIONS.STORE["delete-listing"], onClick: (e) => { e.stopPropagation(); void handleDelete(row); }, disabled: deletingId === row.id, className: "text-[var(--appkit-color-error)] hover:bg-[var(--appkit-color-border-subtle)]", children: _jsx(Trash2, { className: "w-4 h-4" }) }))] }));
         },
     };
     return (_jsxs(_Fragment, { children: [_jsx(DataListingView, { config: config }), setLocationOpen && (_jsx(PhysicalLocationModal, { count: selectedIdsForLocation.length, onSave: (loc) => handleSetLocation(loc, selectedIdsForLocation), onClose: () => setSetLocationOpen(false) })), setStockOpen && (_jsx(BulkStockModal, { count: selectedIdsForStock.length, busy: stockBusy, onSave: (qty) => handleSetStock(qty, selectedIdsForStock), onClose: () => setSetStockOpen(false) })), assembleFor && assembleData && (_jsx(AssemblyDrawer, { isOpen: true, 

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

@@ -10,6 +10,8 @@ import { SELLER_ENDPOINTS } from "../../../constants/api-endpoints.js";
 import { ACTIONS } from "../../../_internal/shared/actions/action-registry.js";
 import { ROUTES } from "../../../constants/index.js";
 import { toRecordArray, toStringValue } from "../hooks/useSellerListingData.js";
+import { useOptimisticRowFlag } from "../../admin/hooks/useOptimisticRowFlag.js";
+import { apiClient } from "../../../http/index.js";
 import { DataListingView } from "../../admin/components/DataListingView.js";
 const DEFAULT_SORT = "-createdAt";
 const SORT_OPTIONS = [
@@ -40,6 +42,22 @@ const COLUMNS = [
     },
 ];
 export function SellerShippingConfigsView({ onCreateClick, onEditClick, onDelete, onSetDefault, }) {
+    /*
+     * 🛑 `isActive` was a READ-ONLY badge here, and the update schema was built
+     * to take it on its own — shipping-config's even says so in a comment: "a
+     * PATCH may legitimately carry only `isActive`". Nothing in the client had
+     * ever sent one, so flipping one boolean meant a full-editor round trip.
+     *
+     * The optimistic/rollback logic lives in the hook because eight listings
+     * grew this control at once; see its header for why rollback must DELETE
+     * the override key rather than write back a captured value.
+     */
+    const activeFlag = useOptimisticRowFlag({
+        mutate: (id, next) => apiClient.patch(SELLER_ENDPOINTS.SHIPPING_CONFIG_BY_ID(id), { isActive: next }),
+        invalidate: [["seller", "shipping-configs", "listing"]],
+        successMessage: (next) => (next ? "Shipping option activated." : "Shipping option deactivated."),
+        errorMessage: "Could not change that shipping option's status.",
+    });
     const [deleteTargetId, setDeleteTargetId] = useState(null);
     const { deletingId, handleDelete: performDelete } = useEntityDelete({
         endpoint: SELLER_ENDPOINTS.SHIPPING_CONFIG_BY_ID,
@@ -113,12 +131,26 @@ export function SellerShippingConfigsView({ onCreateClick, onEditClick, onDelete
             method: String(item.method ?? ""),
             estimatedDays: typeof item.estimatedDays === "number" ? item.estimatedDays : null,
             isDefault: Boolean(item.isDefault),
-            isActive: Boolean(item.isActive),
+            // Resolved through the pending override so the Status badge and the
+            // row toggle cannot disagree between the click and the refetch.
+            isActive: activeFlag.resolve(toStringValue(item.id, `sc-${index}`), Boolean(item.isActive)),
         })),
         getTotal: (response, mappedRows) => (typeof response.total === "number" ? response.total : mappedRows.length),
         buildFilters: () => undefined,
         primaryAction: { label: "New Config", onClick: handleCreate },
         renderRowActions: (row) => (_jsx(RowActionMenu, { actions: [
+                /*
+                 * 🛑 `isActive` was a read-only badge and the update schema was
+                 * built to accept it alone. A menu entry rather than a switch here,
+                 * to match the other actions in this menu — the menu is already
+                 * click-stopped in all three renderers.
+                 */
+                {
+                    label: row.isActive ? "Deactivate" : "Activate",
+                    separator: true,
+                    disabled: activeFlag.pendingId === row.id,
+                    onClick: () => { void activeFlag.set(row.id, !row.isActive); },
+                },
                 { label: ACTIONS.STORE["edit-listing"].label, onClick: () => handleEdit(row.id) },
                 ...(!row.isDefault
                     ? [{ label: "Set as Default", disabled: settingDefaultId === row.id, onClick: () => handleSetDefault(row.id) }]

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

@@ -9,6 +9,7 @@ import { StoreAddressSelectorCreate } from "../../stores/components/StoreAddress
 import { SectionForm, useSectionFormNav } from "../../shell/index.js";
 import { SELLER_ENDPOINTS } from "../../../constants/api-endpoints.js";
 import { normalizeError } from "../../../errors/normalize.js";
+import { toUserMessage } from "../../../errors/error-display-map.js";
 const shippingDraftSchema = z.object({
     customCarrierName: z.string().min(1, "Carrier name is required"),
     customShippingPrice: z.string(),
@@ -82,7 +83,7 @@ export function SellerShippingView({ apiBase = SELLER_ENDPOINTS.SHIPPING }) {
                 if (Array.isArray(json?.issues) && json.issues.length > 0) {
                     applyZodIssues(json.issues, setFieldError);
                 }
-                setError(json?.error ?? "Failed to save");
+                setError(toUserMessage(json?.code, undefined, { fallback: "We couldn't save your shipping settings." }));
                 return;
             }
             clearErrors();

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

@@ -11,6 +11,8 @@ import { ACTIONS } from "../../../_internal/shared/actions/action-registry.js";
 import { buildBulkAction } from "../../../_internal/shared/actions/bulk-helpers.js";
 import { ROUTES } from "../../../next/routing/route-map.js";
 import { toRecordArray, toStringValue } from "../hooks/useSellerListingData.js";
+import { useOptimisticRowFlag } from "../../admin/hooks/useOptimisticRowFlag.js";
+import { apiClient } from "../../../http/index.js";
 import { DataListingView } from "../../admin/components/DataListingView.js";
 const DEFAULT_SORT = "displayOrder";
 const SORT_OPTIONS = [
@@ -37,6 +39,22 @@ const COLUMNS = [
     },
 ];
 export function SellerStoreCategoriesView({ onCreateClick, onEditClick, onDelete, onBulkDelete, onBulkActivate, onBulkDeactivate, }) {
+    /*
+     * 🛑 `isActive` was a READ-ONLY badge here, and the update schema was built
+     * to take it on its own — shipping-config's even says so in a comment: "a
+     * PATCH may legitimately carry only `isActive`". Nothing in the client had
+     * ever sent one, so flipping one boolean meant a full-editor round trip.
+     *
+     * The optimistic/rollback logic lives in the hook because eight listings
+     * grew this control at once; see its header for why rollback must DELETE
+     * the override key rather than write back a captured value.
+     */
+    const activeFlag = useOptimisticRowFlag({
+        mutate: (id, next) => apiClient.patch(SELLER_ENDPOINTS.STORE_CATEGORY_BY_ID(id), { isActive: next }),
+        invalidate: [["seller", "store-categories", "listing"]],
+        successMessage: (next) => (next ? "Category activated." : "Category deactivated."),
+        errorMessage: "Could not change that category's status.",
+    });
     const { showToast } = useToast();
     const [deleteTargetId, setDeleteTargetId] = useState(null);
     const { deletingId, handleDelete: performDelete } = useEntityDelete({
@@ -79,7 +97,9 @@ export function SellerStoreCategoriesView({ onCreateClick, onEditClick, onDelete
             label: String(item.label ?? ""),
             slug: String(item.slug ?? ""),
             productCount: Number(item.productIds?.length ?? 0),
-            isActive: Boolean(item.isActive),
+            // Resolved through the pending override so the Status badge and the
+            // row toggle cannot disagree between the click and the refetch.
+            isActive: activeFlag.resolve(toStringValue(item.id, `cat-${index}`), Boolean(item.isActive)),
         })),
         getTotal: (response, rows) => (typeof response.total === "number" ? response.total : rows.length),
         buildFilters: () => undefined,
@@ -93,6 +113,18 @@ export function SellerStoreCategoriesView({ onCreateClick, onEditClick, onDelete
             return actions;
         },
         renderRowActions: (row) => (_jsx(RowActionMenu, { actions: [
+                /*
+                 * 🛑 `isActive` was a read-only badge and the update schema was
+                 * built to accept it alone. A menu entry rather than a switch here,
+                 * to match the other actions in this menu — the menu is already
+                 * click-stopped in all three renderers.
+                 */
+                {
+                    label: row.isActive ? "Deactivate" : "Activate",
+                    separator: true,
+                    disabled: activeFlag.pendingId === row.id,
+                    onClick: () => { void activeFlag.set(row.id, !row.isActive); },
+                },
                 {
                     label: ACTIONS.STORE["edit-listing"].label,
                     onClick: () => onEditClick

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

@@ -24,7 +24,9 @@ import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
  * close a complaint made against them.
  */
 import React from "react";
-import { Badge, Span, Stack, Text } from "../../../ui/index.js";
+import { useQuery } from "@tanstack/react-query";
+import { Badge, RecordDetailModal, Span, Stack, Text } from "../../../ui/index.js";
+import { apiClient } from "../../../http/index.js";
 import { DataListingView } from "../../admin/components/DataListingView.js";
 import { SELLER_ENDPOINTS } from "../../../constants/api-endpoints.js";
 import { SUPPORT_TICKET_FIELDS } from "../../../constants/field-names.js";
@@ -38,7 +40,58 @@ const STATUS_VARIANT = {
     resolved: "success",
     closed: "default",
 };
+/**
+ * One ticket, read-only, in the shared detail modal.
+ *
+ * Its own component so the query only runs once a row has actually been
+ * opened — the list is the common case and nobody should pay a per-ticket
+ * fetch to scroll it.
+ */
+function SellerSupportTicketModal({ ticketId, onClose, }) {
+    const { data, isLoading } = useQuery({
+        queryKey: ["store", "support-ticket", ticketId],
+        queryFn: () => apiClient.get(SELLER_ENDPOINTS.SUPPORT_TICKET_BY_ID(ticketId)),
+        staleTime: 30000,
+    });
+    const ticket = data?.ticket;
+    const messages = Array.isArray(ticket?.messages)
+        ? ticket.messages
+        : [];
+    return (_jsx(RecordDetailModal, { isOpen: true, onClose: onClose, title: toStringValue(ticket?.subject, isLoading ? "Loading…" : "Ticket"), badges: ticket
+            ? [
+                {
+                    label: toStringValue(ticket.status).replace(/_/g, " "),
+                    variant: STATUS_VARIANT[toStringValue(ticket.status)] ?? "default",
+                },
+                { label: toStringValue(ticket.category).replace(/_/g, " "), variant: "default" },
+            ]
+            : undefined, description: toStringValue(ticket?.description), fields: ticket
+            ? [
+                { label: "Ticket", value: toStringValue(ticket.id) },
+                { label: "Raised", value: toRelativeDate(ticket.createdAt) },
+                { label: "Last update", value: toRelativeDate(ticket.updatedAt ?? ticket.createdAt) },
+                { label: "Priority", value: toStringValue(ticket.priority, "—") },
+            ]
+            : undefined, items: messages.length > 0
+            ? {
+                heading: `Thread (${messages.length})`,
+                entries: messages.map((m) => ({
+                    /*
+                     * The author's ROLE, not their name. A seller reading a
+                     * complaint made against them does not need the buyer's
+                     * identity to follow the thread, and those fields are
+                     * PII-encrypted at rest precisely so they are not handed out
+                     * by default.
+                     */
+                    title: m.authorRole === "admin" ? "Support team" : "Buyer",
+                    subtitle: toStringValue(m.body),
+                    trailing: toRelativeDate((m.createdAt ?? null)),
+                })),
+            }
+            : undefined }));
+}
 export function SellerSupportView({ labels = {} }) {
+    const [detailId, setDetailId] = React.useState(null);
     const mapRows = React.useCallback((response) => {
         const items = Array.isArray(response?.items) ? response.items : [];
         return items.map((raw) => {
@@ -96,5 +149,19 @@ export function SellerSupportView({ labels = {} }) {
                         },
                         { 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." })] }));
+                    /*
+                     * 🛑 This list was a DEAD END, and unlike most of the ones fixed
+                     * alongside it that was not a pure UI gap: a seller could read a
+                     * subject and a 140-character slice of the description and never
+                     * open the thread, under the footer below promising the thread is
+                     * where replies happen. `/api/store/support` was GET-list-only with
+                     * no `[id]` sibling at all, so the route had to be built first.
+                     *
+                     * A modal rather than a page: the seller cannot act on these — a
+                     * ticket raised ABOUT a store is not one its owner can reply to or
+                     * close — so there is nothing here a route would add, and this is
+                     * the same `RecordDetailModal` the other dead ends were fixed with.
+                     */
+                    onRowClick: (row) => setDetailId(row.id),
+                } }), detailId && (_jsx(SellerSupportTicketModal, { ticketId: detailId, onClose: () => setDetailId(null) })), _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/schemas/coupon-form.d.ts

@@ -199,6 +199,7 @@ export declare const adminCouponFormSchema: z.ZodEffects<z.ZodObject<{
     description: z.ZodUnion<[z.ZodOptional<z.ZodString>, z.ZodLiteral<"">]>;
     buyQuantity: z.ZodOptional<z.ZodEffects<z.ZodString, string, string>>;
     getQuantity: z.ZodOptional<z.ZodEffects<z.ZodString, string, string>>;
+    maxItemPrice: z.ZodOptional<z.ZodEffects<z.ZodString, string, string>>;
     currentUsage: z.ZodOptional<z.ZodNumber>;
     tiers: z.ZodOptional<z.ZodArray<z.ZodObject<{
         minAmount: z.ZodNumber;
@@ -233,6 +234,7 @@ export declare const adminCouponFormSchema: z.ZodEffects<z.ZodObject<{
         discountValue: number;
     }[] | undefined;
     currentUsage?: number | undefined;
+    maxItemPrice?: string | undefined;
     maxDiscount?: string | undefined;
     buyQuantity?: string | undefined;
     getQuantity?: string | undefined;
@@ -257,6 +259,7 @@ export declare const adminCouponFormSchema: z.ZodEffects<z.ZodObject<{
         discountValue: number;
     }[] | undefined;
     currentUsage?: number | undefined;
+    maxItemPrice?: string | undefined;
     maxDiscount?: string | undefined;
     buyQuantity?: string | undefined;
     getQuantity?: string | undefined;
@@ -281,6 +284,7 @@ export declare const adminCouponFormSchema: z.ZodEffects<z.ZodObject<{
         discountValue: number;
     }[] | undefined;
     currentUsage?: number | undefined;
+    maxItemPrice?: string | undefined;
     maxDiscount?: string | undefined;
     buyQuantity?: string | undefined;
     getQuantity?: string | undefined;
@@ -305,6 +309,7 @@ export declare const adminCouponFormSchema: z.ZodEffects<z.ZodObject<{
         discountValue: number;
     }[] | undefined;
     currentUsage?: number | undefined;
+    maxItemPrice?: string | undefined;
     maxDiscount?: string | undefined;
     buyQuantity?: string | undefined;
     getQuantity?: string | undefined;

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

@@ -236,6 +236,24 @@ export const adminCouponFormSchema = couponFormBase
         section: "basics", order: 7, row: "pair", kind: "number",
         when: (v) => v.type === "buy_x_get_y",
     }),
+    /*
+     * The per-ITEM price ceiling — "buy 3 get 1 free on items under ₹2,000".
+     *
+     * 🛑 Genuinely optional, unlike the buy/get pair: `superRefine` below does
+     * NOT demand it when the type is buy_x_get_y, because a BOGO with no
+     * ceiling is a perfectly ordinary offer and every one written before
+     * 2026-10-22 is exactly that. Leaving it blank means no ceiling.
+     *
+     * 🛑 Not to be confused with `minPurchase`, which is the ORDER total. This
+     * is one unit's price, so a line of three ₹1,500 tops qualifies on all
+     * three units rather than failing a ₹4,500 test.
+     */
+    maxItemPrice: annotate(numericString("Maximum item price").optional(), {
+        section: "basics", order: 8, row: "pair", kind: "number",
+        when: (v) => v.type === "buy_x_get_y",
+        label: "Max item price (₹)",
+        help: "Only items at or below this price qualify. Leave blank for no limit.",
+    }),
     /*
      * 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