@mohasinac/appkit 4.61.0 → 4.62.0

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/shell/SectionForm.js

@@ -325,6 +325,25 @@ export function SectionForm({ sections, values, onChange, onSubmit, submitLabel
         }
         void onSubmit();
     }, [shellCtx, onSubmit, erroringSectionIds, fieldErrors, openIds, setOpenIds, expandMode]);
+    /**
+     * Open a section if it is closed, then scroll to it.
+     *
+     * Named rather than inlined in the rail's `onClick`: inline it sat five
+     * braces deep inside the JSX, and it is the same open-then-scroll the
+     * submit handler above performs for the first erroring section — so the
+     * two now read as the same operation rather than two similar ones.
+     *
+     * 🛑 Opening is CONDITIONAL on purpose. In `single` mode an unconditional
+     * `setOpenIds([id])` would COLLAPSE the section you are already in when you
+     * click its own rail entry, so the rail would read as having closed the
+     * form.
+     */
+    const openAndScrollTo = useCallback((id) => {
+        if (!openIds.includes(id)) {
+            setOpenIds(expandMode === "single" ? [id] : [...openIds, id]);
+        }
+        scrollToSection(id);
+    }, [openIds, setOpenIds, expandMode]);
     useFormBottomActions({
         onSubmit: handleSubmit,
         onCancel,
@@ -391,21 +410,6 @@ export function SectionForm({ sections, values, onChange, onSubmit, submitLabel
      * sections").
      */
     const showRail = ordered.length >= 3;
-    /**
-     * One rail click: open the section if it is not already, then scroll to it.
-     * Named rather than inline in the button's `onClick`, where the `if` body
-     * sat a brace too deep inside the mapped JSX.
-     *
-     * Opening is conditional on purpose — in `single` mode an unconditional
-     * `setOpenIds([id])` would COLLAPSE the section you are already in when you
-     * click its own rail entry, so the rail would look like it closed the form.
-     */
-    const openAndScrollTo = (id) => {
-        if (!openIds.includes(id)) {
-            setOpenIds(expandMode === "single" ? [id] : [...openIds, id]);
-        }
-        scrollToSection(id);
-    };
     /*
      * `sticky` under the measured header, and it publishes NO height: per the
      * three-tier rule a sticky element CONSUMES the tier and never joins it.
@@ -444,7 +448,7 @@ export function SectionForm({ sections, values, onChange, onSubmit, submitLabel
                         forceOpenIds.includes(section.id);
                     return (_jsxs(Button, { type: "button", 
                         /* `ghost`, not `bare` — UI_BUTTON has no `bare` variant. */
-                        variant: "ghost", size: "sm", justify: "between", gap: "sm", className: `w-full text-left ${isOpen ? "appkit-section-rail__item--active" : ""}`, "aria-current": isOpen ? "true" : undefined, onClick: () => openAndScrollTo(section.id), children: [_jsx(Span, { size: "sm", color: isOpen ? "primary" : "muted", children: section.label }), errorCount > 0 && (_jsx(Span, { size: "xs", color: "error", weight: "semibold", children: errorCount }))] }, section.id));
+                        variant: "ghost", size: "sm", justify: "between", gap: "sm", className: `w-full text-left ${isOpen ? "appkit-section-rail__item--active" : ""}`, "aria-current": isOpen ? "true" : undefined, onClick: () => openAndScrollTo(section.id), children: [_jsx(Span, { size: "sm", color: isOpen ? "primary" : "muted", children: section.label }), submitAttempted && errorCount > 0 && (_jsx(Span, { size: "xs", color: "error", weight: "semibold", children: errorCount }))] }, section.id));
                 }) }) }) })) : null;
     const panels = (_jsxs(Stack, { gap: "md", children: [ordered.map((section) => {
                 const isRequired = section.required === true;

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

@@ -288,7 +288,20 @@ function renderField(field, ctx, opts) {
              * case that most needs searching.
              */
             if (options.length > PAGINATED_SELECT_THRESHOLD) {
-                return (_jsxs(Stack, { gap: "xs", children: [_jsxs(Text, { size: "sm", weight: "medium", color: "muted", children: [label, required ? " *" : ""] }), _jsx(PaginatedSelect, { value: typeof value === "string" ? value : null, onChange: (v) => set(v ?? ""), options: options, ariaLabel: label, placeholder: `Select ${label.toLowerCase()}…` }), meta.help && _jsx(Text, { size: "xs", color: "muted", children: meta.help }), ctx.errors[name] && (_jsx(Text, { size: "xs", color: "error", role: "alert", children: ctx.errors[name] }))] }, name));
+                /*
+                 * 🛑 The error block here used to be hand-rolled, and was the only
+                 * field in this generator whose error was not wired like the rest:
+                 * a `<Text>` with no `id`, nothing pointing `aria-describedby` at it,
+                 * no `aria-invalid` on the control, and — unlike every `Field*`
+                 * branch below — the help text was still rendered underneath the
+                 * error rather than replaced by it.
+                 *
+                 * `PaginatedSelect` now takes `name`/`error` and owns all of that,
+                 * so hand it the error and delete the copy. This is the one branch
+                 * that every schema-generated form shares, so the fix lands on all
+                 * of them at once rather than per call site.
+                 */
+                return (_jsxs(Stack, { gap: "xs", children: [_jsxs(Text, { size: "sm", weight: "medium", color: "muted", children: [label, required ? " *" : ""] }), _jsx(PaginatedSelect, { name: name, error: ctx.errors[name], value: typeof value === "string" ? value : null, onChange: (v) => set(v ?? ""), options: options, ariaLabel: label, placeholder: `Select ${label.toLowerCase()}…` }), meta.help && !ctx.errors[name] && (_jsx(Text, { size: "xs", color: "muted", children: meta.help }))] }, name));
             }
             return (_jsx(FieldSelect, { name: name, label: label, hint: meta.help, required: required, value: typeof value === "string" ? value : "", options: options, onChange: set }, name));
         }

package/dist/features/shipments/components/AdminShipmentLotItemsView.js

@@ -1,6 +1,7 @@
 "use client";
 import { jsxs as _jsxs, jsx as _jsx } from "react/jsx-runtime";
 import React from "react";
+import { z } from "zod";
 import { useApiMutation } from "@mohasinac/appkit/client";
 import { Alert, Badge, Button, Div, Heading, Modal, Pagination, Skeleton, Stack, Table, Thead, Tbody, Tr, Th, Td, Text, Textarea, } from "../../../ui/index.js";
 import { apiClient } from "../../../http/index.js";
@@ -11,6 +12,7 @@ import { applyZodIssues } from "../../../ui/forms/apply-zod-issues.js";
 import { ValidationError } from "../../../errors/validation-error.js";
 import { ADMIN_ENDPOINTS } from "../../../constants/api-endpoints.js";
 import { ACTIONS } from "../../../_internal/shared/actions/action-registry.js";
+import { QuickEditMenu } from "../../admin/components/QuickEditMenu.js";
 import { useShipmentItems } from "../hooks/useShipments.js";
 import { ShipmentItemLinkModal } from "./ShipmentItemLinkModal.js";
 import { formatCurrency } from "../../../utils/number.formatter.js";
@@ -32,6 +34,30 @@ function parseBulkText(text) {
     })
         .filter((row) => row.title.length > 0);
 }
+/**
+ * The three fields a lot item can be corrected on after import.
+ *
+ * Deliberately NOT the whole of `shipmentItemBaseSchema`: `isForSelfUse`
+ * decides whether `price` is required at all and flipping it from a
+ * three-field drawer would silently change what the row means to the
+ * projections, and `categorySlugs`/`brandSlug` belong to the link flow that
+ * already owns them.
+ */
+const shipmentItemQuickEditSchema = z.object({
+    title: z.string().trim().min(1, "Item title is required"),
+    quantity: z.coerce.number().int().min(1, "Quantity must be at least 1."),
+    price: z.coerce.number().min(0, "Price cannot be negative."),
+});
+const SHIPMENT_ITEM_QUICK_FIELDS = [
+    { name: "title", label: "Title", type: "text", required: true },
+    { name: "quantity", label: "Quantity", type: "number", required: true },
+    {
+        name: "price",
+        label: "Projected sale price",
+        type: "number",
+        helperText: "Feeds this lot's projected revenue, and every profit figure on the projections screen.",
+    },
+];
 export function AdminShipmentLotItemsView({ shipmentId, lotId }) {
     const [page, setPage] = React.useState(1);
     const [pageSize, setPageSize] = React.useState(50);
@@ -113,6 +139,19 @@ export function AdminShipmentLotItemsView({ shipmentId, lotId }) {
         mutationFn: (itemId) => apiClient.delete(ADMIN_ENDPOINTS.SHIPMENT_LOT_ITEM_BY_ID(shipmentId, lotId, itemId)),
         onSuccess: () => refetch(),
     });
+    const editItemMutation = useApiMutation({
+        successMessage: "Item updated",
+        errorMessage: "Could not update that item.",
+        mutationFn: ({ itemId, values }) => {
+            const parsed = shipmentItemQuickEditSchema.parse(values);
+            return apiClient.patch(ADMIN_ENDPOINTS.SHIPMENT_LOT_ITEM_BY_ID(shipmentId, lotId, itemId), parsed);
+        },
+        onSuccess: () => refetch(),
+    });
+    /** Named so the row-action array stays a flat list of declarations. */
+    const saveItemEdit = React.useCallback(async (itemId, values) => {
+        await editItemMutation.mutateAsync({ itemId, values });
+    }, [editItemMutation]);
     const unlinkItemMutation = useApiMutation({
         successMessage: "Item unlinked",
         mutationFn: (itemId) => apiClient.patch(ADMIN_ENDPOINTS.SHIPMENT_LOT_ITEM_BY_ID(shipmentId, lotId, itemId), {
@@ -134,7 +173,21 @@ export function AdminShipmentLotItemsView({ shipmentId, lotId }) {
                                 return;
                             }
                             addItemMutation.mutate();
-                        }, onValidationChange: () => itemForm.validate(itemDraft), schema: shipmentItemFormSchema, openIds: itemNav.openIds, onOpenChange: itemNav.setOpenIds, submitLabel: "Save Item", cancelLabel: "Cancel", onCancel: () => setShowAddItem(false), isLoading: addItemMutation.isPending })] })), isLoading ? (_jsxs(Stack, { gap: "sm", children: [_jsx(Skeleton, { variant: "rectangular", height: "32px" }), _jsx(Skeleton, { variant: "rectangular", height: "32px" })] })) : items.length === 0 ? (_jsx(Text, { variant: "secondary", children: "No items tracked yet." })) : (_jsxs(Table, { children: [_jsx(Thead, { children: _jsxs(Tr, { children: [_jsx(Th, { children: "Title" }), _jsx(Th, { children: "Qty" }), _jsx(Th, { children: "Price" }), _jsx(Th, { children: "Status" }), _jsx(Th, {})] }) }), _jsx(Tbody, { children: items.map((item) => (_jsxs(Tr, { children: [_jsx(Td, { children: item.title }), _jsx(Td, { children: item.quantity }), _jsx(Td, { children: item.isForSelfUse ? "—" : formatCurrency(item.price ?? 0) }), _jsx(Td, { children: item.isForSelfUse ? (_jsx(Badge, { variant: "secondary", children: "Self use" })) : item.linkedProductId ? (_jsx(Badge, { variant: "success", children: "Linked" })) : (_jsx(Badge, { variant: "default", children: "Unlinked" })) }), _jsxs(Td, { className: "flex gap-[var(--appkit-space-2)]", children: [!item.isForSelfUse &&
+                        }, onValidationChange: () => itemForm.validate(itemDraft), schema: shipmentItemFormSchema, openIds: itemNav.openIds, onOpenChange: itemNav.setOpenIds, submitLabel: "Save Item", cancelLabel: "Cancel", onCancel: () => setShowAddItem(false), isLoading: addItemMutation.isPending })] })), isLoading ? (_jsxs(Stack, { gap: "sm", children: [_jsx(Skeleton, { variant: "rectangular", height: "32px" }), _jsx(Skeleton, { variant: "rectangular", height: "32px" })] })) : items.length === 0 ? (_jsx(Text, { variant: "secondary", children: "No items tracked yet." })) : (_jsxs(Table, { children: [_jsx(Thead, { children: _jsxs(Tr, { children: [_jsx(Th, { children: "Title" }), _jsx(Th, { children: "Qty" }), _jsx(Th, { children: "Price" }), _jsx(Th, { children: "Status" }), _jsx(Th, {})] }) }), _jsx(Tbody, { children: items.map((item) => (_jsxs(Tr, { children: [_jsx(Td, { children: item.title }), _jsx(Td, { children: item.quantity }), _jsx(Td, { children: item.isForSelfUse ? "—" : formatCurrency(item.price ?? 0) }), _jsx(Td, { children: item.isForSelfUse ? (_jsx(Badge, { variant: "secondary", children: "Self use" })) : item.linkedProductId ? (_jsx(Badge, { variant: "success", children: "Linked" })) : (_jsx(Badge, { variant: "default", children: "Unlinked" })) }), _jsxs(Td, { className: "flex gap-[var(--appkit-space-2)]", children: [_jsx(QuickEditMenu, { actions: [
+                                                {
+                                                    label: "Edit item",
+                                                    formTitle: `Edit "${item.title}"`,
+                                                    schema: shipmentItemQuickEditSchema,
+                                                    fields: SHIPMENT_ITEM_QUICK_FIELDS,
+                                                    defaultValues: {
+                                                        title: item.title,
+                                                        quantity: item.quantity,
+                                                        price: item.price ?? 0,
+                                                    },
+                                                    onSubmit: (values) => saveItemEdit(item.id, values),
+                                                    submitLabel: "Save item",
+                                                },
+                                            ] }), !item.isForSelfUse &&
                                             (item.linkedProductId ? (_jsx(Button, { size: "sm", variant: "ghost", isLoading: unlinkItemMutation.isPending, onClick: () => unlinkItemMutation.mutate(item.id), children: "Unlink" })) : (_jsx(Button, { size: "sm", variant: "ghost", onClick: () => setLinkTarget({ itemId: item.id, title: item.title }), children: "Create pre-order link" }))), _jsx(Button, { size: "sm", variant: "ghost", action: ACTIONS.SHIPMENT["delete"], isLoading: deleteItemMutation.isPending, onClick: () => deleteItemMutation.mutate(item.id), children: "Delete" })] })] }, item.id))) })] })), meta && meta.totalPages > 1 && (_jsx(Pagination, { currentPage: page, totalPages: meta.totalPages, onPageChange: setPage, pageSize: pageSize, onPageSizeChange: (size) => {
                     setPageSize(size);
                     setPage(1);