@mohasinac/appkit 4.30.18 → 4.31.0

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

@@ -0,0 +1,277 @@
+"use client";
+import { jsx as _jsx, Fragment as _Fragment, jsxs as _jsxs } from "react/jsx-runtime";
+/*
+ * WHY: Five seller views — Art, Stickers, Classified, Digital Codes, Live Items
+ *      — were 1,033 lines of the same component. Identical props, identical
+ *      delete scaffolding, identical row actions, identical bulk-action map,
+ *      identical `getTotal`, identical availability scope. What actually
+ *      differed was a listing-type string, four lines of copy, two routes, and
+ *      one to three extra columns.
+ *
+ *      Five copies is well past the Duplication Framework's Rule of Three, and
+ *      it had already drifted: Digital Codes offered four sort options where
+ *      the other four offered five, and every one hand-wrote a sort array the
+ *      plugin registry already publishes.
+ *
+ *      🛑 The deeper point is that this consolidation had ALREADY been done —
+ *      `buildListingTypeListingConfig` exists, is tagged `pattern:factory`, its
+ *      header describes this exact problem, and it already accepts
+ *      `portal: "seller"`. It had FIVE admin callers and ZERO seller ones. The
+ *      seller half was written by hand next to it. That is the unadopted-layer
+ *      shape this whole plan exists to close, so this file adopts it rather
+ *      than starting a third implementation.
+ *
+ * WHAT: One component. Everything shared lives here; everything per-type lives
+ *       in `SELLER_LISTING_TYPE_SPECS` below, which is the complete inventory
+ *       of what genuinely differs.
+ *
+ * EXPORTS: SellerListingRow, SellerListingTypeSpec, SELLER_LISTING_TYPE_SPECS,
+ *          SellerListingTypeView, SellerListingTypeViewProps
+ *
+ * @tag domain:seller,products
+ * @tag layer:view
+ * @tag pattern:factory
+ * @tag access:client
+ * @tag consumers:SellerArtView,SellerStickersView,SellerClassifiedView,SellerDigitalCodesView,SellerLiveView
+ * @tag sideEffects:none
+ */
+import React, { useCallback, useState } from "react";
+import { Badge, ConfirmDeleteModal, RowActionMenu, Span, Text } from "../../../ui";
+import { useEntityDelete } from "../../../react/hooks/useEntityDelete";
+import { SELLER_ENDPOINTS } from "../../../constants/api-endpoints";
+import { ACTIONS } from "../../../_internal/shared/actions/action-registry";
+import { ROUTES } from "../../../next/routing/route-map";
+import { sortBy } from "../../../constants/sort";
+import { PRODUCT_FIELDS } from "../../../constants/field-names";
+import { SIEVE_OP, sieveFilter } from "../../../utils/sieve-builder";
+import { pluginFor } from "../../../_internal/shared/listing-types/_registry";
+import { toCurrency, toRecordArray, toRelativeDate, toStringValue, } from "../../admin/hooks/useAdminListingData";
+import { DataListingView } from "../../admin/components/DataListingView";
+import { ROW_ACTION_META, SELLER_BULK_ACTIONS } from "../../products/constants/action-defs";
+import { useAvailabilityScope } from "../../products/hooks/useAvailabilityScope";
+/* ── Shared column renderers ─────────────────────────────────────────────── */
+const textCell = (key, header) => ({
+    key,
+    header,
+    render: (row) => (_jsx(Text, { className: "text-[var(--appkit-color-text-muted)]", size: "sm", children: row[key] || "—" })),
+});
+const numberCell = (key, header) => ({
+    key,
+    header,
+    render: (row) => (_jsx(Text, { className: "tabular-nums", size: "sm", children: row[key] || "0" })),
+});
+export const SELLER_LISTING_TYPE_SPECS = {
+    art: {
+        editRoute: (id) => String(ROUTES.STORE.ART_EDIT(id)),
+        newRoute: String(ROUTES.STORE.ART_NEW),
+        createLabel: "New Art Print",
+        bulkKey: "art",
+        deleteLabel: "Art Listing",
+        searchPlaceholder: "Search art prints...",
+        emptyLabel: "No art listings yet — list your first print",
+        extraFields: (item) => ({
+            size: toStringValue(item.printMeta?.size, ""),
+        }),
+        extraColumns: [textCell("size", "Size")],
+    },
+    stickers: {
+        editRoute: (id) => String(ROUTES.STORE.STICKERS_EDIT(id)),
+        newRoute: String(ROUTES.STORE.STICKERS_NEW),
+        createLabel: "New Sticker Listing",
+        bulkKey: "stickers",
+        deleteLabel: "Sticker Listing",
+        searchPlaceholder: "Search sticker listings...",
+        emptyLabel: "No sticker listings yet — list your first pack",
+        extraFields: (item) => ({
+            size: toStringValue(item.printMeta?.size, ""),
+        }),
+        extraColumns: [textCell("size", "Size")],
+    },
+    classified: {
+        editRoute: (id) => String(ROUTES.STORE.CLASSIFIED_EDIT(id)),
+        newRoute: String(ROUTES.STORE.CLASSIFIED_NEW),
+        createLabel: "New Classified",
+        bulkKey: "classified",
+        deleteLabel: "Classified Listing",
+        searchPlaceholder: "Search classified listings...",
+        emptyLabel: "No classified listings yet — post your first buy/sell/trade ad",
+        // `title` pinned: the plugin's pluralLabel is "Classifieds", and the page
+        // heading has always read "Classified". Kept rather than silently renamed.
+        title: "Classified",
+        extraFields: (item) => {
+            const meta = (item.classified ?? {});
+            return {
+                location: toStringValue(meta.meetupArea?.city ?? meta.city, ""),
+                shipping: meta.acceptsShipping ? "Ships" : "Meetup only",
+            };
+        },
+        extraColumns: [
+            textCell("location", "Location"),
+            {
+                key: "shipping",
+                header: "Shipping",
+                render: (row) => (_jsx(Badge, { variant: row.shipping === "Ships" ? "active" : "inactive", size: "xs", children: row.shipping })),
+            },
+        ],
+    },
+    "digital-code": {
+        editRoute: (id) => String(ROUTES.STORE.DIGITAL_CODES_EDIT(id)),
+        newRoute: String(ROUTES.STORE.DIGITAL_CODES_NEW),
+        createLabel: "New Digital Code",
+        bulkKey: "digitalCodes",
+        deleteLabel: "Digital Code Listing",
+        searchPlaceholder: "Search digital code listings...",
+        emptyLabel: "No digital code listings yet",
+        titleHeader: "Product",
+        deleteSuccess: "Code deleted.",
+        extraFields: (item) => {
+            const meta = (item.digitalCode ?? {});
+            return {
+                totalCodes: toStringValue(meta.codePoolSize, "0"),
+                available: toStringValue(meta.codesAvailable, "0"),
+                delivery: toStringValue(meta.codeDeliveryMethod, "").replace(/-/g, " "),
+            };
+        },
+        extraColumns: [
+            numberCell("totalCodes", "Total Codes"),
+            numberCell("available", "Available"),
+            {
+                key: "delivery",
+                header: "Delivery",
+                render: (row) => (_jsx(Span, { size: "xs", className: "capitalize", padding: "pill-sm", rounded: "full", surface: "subtle", color: "muted", children: row.delivery || "—" })),
+            },
+        ],
+    },
+    live: {
+        editRoute: (id) => String(ROUTES.STORE.LIVE_ITEMS_EDIT(id)),
+        newRoute: String(ROUTES.STORE.LIVE_ITEMS_NEW),
+        createLabel: "New Live Item",
+        bulkKey: "live",
+        deleteLabel: "Live Item Listing",
+        searchPlaceholder: "Search live item listings...",
+        emptyLabel: "No live item listings yet",
+        titleHeader: "Item",
+        createdHeader: "Listed",
+        extraFields: (item) => {
+            const meta = (item.liveItem ?? {});
+            return {
+                species: toStringValue(meta.species, ""),
+                age: meta.ageMonths ? `${meta.ageMonths} mo` : "",
+                verified: meta.vendorVerified ? "Verified" : "Unverified",
+            };
+        },
+        extraColumns: [
+            textCell("age", "Age"),
+            {
+                key: "verified",
+                header: "Verified",
+                render: (row) => (_jsx(Span, { size: "xs", color: row.verified === "Verified" ? "success" : "warning", children: row.verified })),
+            },
+        ],
+    },
+};
+export function SellerListingTypeView({ type, onCreateClick, onEditClick, onDelete, onBulkDelete, }) {
+    const spec = SELLER_LISTING_TYPE_SPECS[type];
+    const plugin = pluginFor(type);
+    const [deleteTargetId, setDeleteTargetId] = useState(null);
+    const { deletingId, handleDelete: performDelete } = useEntityDelete({
+        endpoint: SELLER_ENDPOINTS.PRODUCT_BY_ID,
+        deleteFn: onDelete,
+        successMessage: spec.deleteSuccess ?? "Listing deleted.",
+        fetchOptions: { credentials: "include" },
+    });
+    const handleDelete = useCallback(async (id) => {
+        await performDelete(id);
+        setDeleteTargetId(null);
+    }, [performDelete]);
+    const handleEdit = useCallback((id) => {
+        if (onEditClick)
+            onEditClick(id);
+        else
+            window.location.href = spec.editRoute(id);
+    }, [onEditClick, spec]);
+    const handleCreate = useCallback(() => {
+        if (onCreateClick)
+            onCreateClick();
+        else
+            window.location.href = spec.newRoute;
+    }, [onCreateClick, spec]);
+    /*
+     * 🛑 Unconditional, and before any early return. `SellerPreOrdersView` called
+     * this AFTER a conditional return — a rules-of-hooks violation that survived
+     * because that view had no page rendering it.
+     */
+    const scope = useAvailabilityScope(React.useMemo(() => [type], [type]));
+    const columns = [
+        {
+            key: "title",
+            header: spec.titleHeader ?? "Listing",
+            render: (row) => (_jsxs(_Fragment, { children: [_jsx(Text, { size: "sm", weight: "medium", children: row.title }), row.species ? (_jsx(Text, { className: "text-[var(--appkit-color-text-muted)]", size: "xs", children: row.species })) : null] })),
+        },
+        { key: "price", header: "Price", render: (row) => _jsx(Text, { className: "tabular-nums", size: "sm", children: row.price }) },
+        ...(spec.extraColumns ?? []),
+        {
+            key: "status",
+            header: "Status",
+            render: (row) => (_jsx(Badge, { variant: row.status === "active" ? "active" : "inactive", size: "xs", className: "capitalize", children: row.status })),
+        },
+        textCell("createdAt", spec.createdHeader ?? "Created"),
+    ];
+    const config = {
+        portal: "seller",
+        title: spec.title ?? plugin.pluralLabel,
+        searchPlaceholder: spec.searchPlaceholder,
+        emptyLabel: spec.emptyLabel,
+        filterKeys: [],
+        defaultSort: sortBy(PRODUCT_FIELDS.CREATED_AT, "DESC"),
+        queryKey: ["seller", plugin.tabSlug],
+        endpoint: SELLER_ENDPOINTS.PRODUCTS,
+        // The registry's set, not a hand-copied array. Digital Codes had drifted to
+        // four options where its four siblings offered five.
+        sortOptions: [...plugin.sortOptions],
+        columns,
+        mapRows: (response) => toRecordArray(response.products ?? response.items).map((item, index) => ({
+            id: toStringValue(item.id, `${type}-${index}`),
+            title: toStringValue(item.productTitle ?? item.title, "Untitled"),
+            price: toCurrency(item.price),
+            status: toStringValue(item.status, "draft"),
+            createdAt: toRelativeDate(item.createdAt),
+            ...(spec.extraFields?.(item) ?? {}),
+        })),
+        getTotal: (response, rows) => typeof response.total === "number"
+            ? response.total
+            : typeof response.meta?.total === "number"
+                ? response.meta.total
+                : rows.length,
+        // `sieveFilter`, not the raw `"listingType==art"` string literal all five
+        // originals wrote — a hand-built clause is invisible to every audit that
+        // checks field names against the schema.
+        buildFilters: () => sieveFilter(PRODUCT_FIELDS.LISTING_TYPE, SIEVE_OP.EQ, type),
+        buildExtraParams: () => scope.extraParams,
+        renderAboveContent: scope.renderAboveContent,
+        primaryAction: { label: spec.createLabel, onClick: handleCreate },
+        onRowClick: (row) => handleEdit(row.id),
+        // Rule #7 — the preset, never an inline action object.
+        buildBulkActions: onBulkDelete
+            ? (selection) => SELLER_BULK_ACTIONS[spec.bulkKey].map((id) => ({
+                id,
+                label: ROW_ACTION_META[id].label,
+                destructive: ROW_ACTION_META[id].destructive,
+                onClick: async () => {
+                    await onBulkDelete(selection.selectedIds);
+                    selection.clearSelection();
+                },
+            }))
+            : 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,
+                },
+            ] })),
+    };
+    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/SellerLiveView.d.ts

@@ -1,8 +1,15 @@
+/**
+ * SellerLiveView — seller browse/manage of live item listings.
+ *
+ * A named wrapper, kept so the page, the barrel and every existing import path
+ * stay unchanged. All behaviour is `SellerListingTypeView`; everything this
+ * type does differently is one entry in `SELLER_LISTING_TYPE_SPECS`.
+ *
+ * This file was 224 lines of a component that four siblings also contained
+ * verbatim — see that file's header for why the duplication mattered rather
+ * than merely being untidy.
+ */
 import React from "react";
-export interface SellerLiveViewProps {
-    onCreateClick?: () => void;
-    onEditClick?: (id: string) => void;
-    onDelete?: (id: string) => Promise<void>;
-    onBulkDelete?: (ids: string[]) => Promise<void>;
-}
-export declare function SellerLiveView({ onCreateClick, onEditClick, onDelete, onBulkDelete, }: SellerLiveViewProps): React.JSX.Element;
+import { type SellerListingTypeViewProps } from "./SellerListingTypeView";
+export type SellerLiveViewProps = SellerListingTypeViewProps;
+export declare function SellerLiveView(props: SellerLiveViewProps): React.JSX.Element;

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

@@ -1,128 +1,6 @@
 "use client";
-import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
-import { Badge, Span, sortBy } from "@mohasinac/appkit/client";
-import { useState, useCallback } from "react";
-import { useEntityDelete } from "../../../react/hooks/useEntityDelete";
-import { ConfirmDeleteModal, Div, RowActionMenu, Text } from "../../../ui";
-import { SELLER_ENDPOINTS } from "../../../constants/api-endpoints";
-import { ACTIONS } from "../../../_internal/shared/actions/action-registry";
-import { ROUTES } from "../../../constants/index";
-import { toRecordArray, toRelativeDate, toCurrency, toStringValue, } from "../../admin/hooks/useAdminListingData";
-import { DataListingView } from "../../admin/components/DataListingView";
-import { SELLER_BULK_ACTIONS, ROW_ACTION_META } from "../../products/constants/action-defs";
-import { useAvailabilityScope } from "../../products/hooks/useAvailabilityScope";
-const LISTING_TYPES = ["live"];
-const COLUMNS = [
-    {
-        key: "title",
-        header: "Item",
-        render: (row) => (_jsxs(Div, { children: [_jsx(Text, { size: "sm", weight: "medium", children: row.title }), row.species && (_jsx(Text, { className: "text-[var(--appkit-color-text-muted)] italic", size: "xs", children: row.species }))] })),
-    },
-    { key: "price", header: "Price", render: (row) => _jsx(Text, { className: "tabular-nums", size: "sm", children: row.price }) },
-    {
-        key: "ageMonths",
-        header: "Age",
-        render: (row) => (_jsx(Text, { className: "text-[var(--appkit-color-text-muted)]", size: "sm", children: row.ageMonths !== null ? `${row.ageMonths}mo` : "—" })),
-    },
-    {
-        key: "vendorVerified",
-        header: "Verified",
-        render: (row) => (_jsx(Span, { layout: "inline-flex", surface: row.vendorVerified ? "success-surface" : "warning-surface", color: row.vendorVerified ? "success" : "warning", size: "xs", weight: "medium", rounded: "full", padding: "pill-xs", children: row.vendorVerified ? "Verified" : "Pending" })),
-    },
-    {
-        key: "status",
-        header: "Status",
-        render: (row) => (_jsx(Badge, { variant: row.status === "active" ? "active" : "inactive", size: "xs", className: "capitalize", children: row.status })),
-    },
-    {
-        key: "createdAt",
-        header: "Listed",
-        render: (row) => (_jsx(Text, { className: "text-[var(--appkit-color-text-muted)]", size: "sm", children: row.createdAt })),
-    },
-];
-export function SellerLiveView({ onCreateClick, onEditClick, onDelete, onBulkDelete, }) {
-    const [deleteTargetId, setDeleteTargetId] = useState(null);
-    const { deletingId, handleDelete: performDelete } = useEntityDelete({
-        endpoint: SELLER_ENDPOINTS.PRODUCT_BY_ID,
-        deleteFn: onDelete,
-        successMessage: "Listing deleted.",
-        fetchOptions: { credentials: "include" },
-    });
-    const handleDelete = useCallback(async (id) => {
-        await performDelete(id);
-        setDeleteTargetId(null);
-    }, [performDelete]);
-    const handleEdit = useCallback((id) => {
-        if (onEditClick)
-            onEditClick(id);
-        else
-            window.location.href = String(ROUTES.STORE.LIVE_ITEMS_EDIT(id));
-    }, [onEditClick]);
-    const handleCreate = useCallback(() => {
-        if (onCreateClick)
-            onCreateClick();
-        else
-            window.location.href = String(ROUTES.STORE.LIVE_ITEMS_NEW);
-    }, [onCreateClick]);
-    const scope = useAvailabilityScope(LISTING_TYPES);
-    const config = {
-        portal: "seller",
-        title: "Live Items",
-        searchPlaceholder: "Search live item listings...",
-        emptyLabel: "No live item listings yet",
-        filterKeys: [],
-        defaultSort: sortBy("createdAt", "DESC"),
-        queryKey: ["seller", "live-items"],
-        endpoint: SELLER_ENDPOINTS.PRODUCTS,
-        sortOptions: [
-            { value: sortBy("createdAt", "DESC"), label: "Newest" },
-            { value: sortBy("createdAt", "ASC"), label: "Oldest" },
-            { value: "title", label: "Name A–Z" },
-            { value: sortBy("price", "ASC"), label: "Price: Low–High" },
-            { value: sortBy("price", "DESC"), label: "Price: High–Low" },
-        ],
-        columns: COLUMNS,
-        mapRows: (response) => toRecordArray(response.products).map((item, index) => {
-            const live = (item.liveItem ?? {});
-            return {
-                id: toStringValue(item.id, `live-${index}`),
-                title: toStringValue(item.productTitle ?? item.title, "Untitled"),
-                price: toCurrency(item.price),
-                species: toStringValue(live.species, ""),
-                ageMonths: typeof live.ageMonths === "number" ? live.ageMonths : null,
-                vendorVerified: Boolean(live.vendorVerified),
-                status: toStringValue(item.status, "draft"),
-                createdAt: toRelativeDate(item.createdAt),
-            };
-        }),
-        getTotal: (response, mappedRows) => typeof response.meta?.total === "number" ? response.meta.total : mappedRows.length,
-        buildFilters: () => "listingType==live",
-        buildExtraParams: () => scope.extraParams,
-        renderAboveContent: scope.renderAboveContent,
-        primaryAction: { label: "New Live Item", onClick: () => handleCreate() },
-        // Mirrors the "Edit" row action below so table rows and cards both navigate on click.
-        onRowClick: (row) => handleEdit(row.id),
-        // Rule #7: bulk-action array sourced from the SELLER_BULK_ACTIONS preset.
-        buildBulkActions: onBulkDelete
-            ? (selection) => SELLER_BULK_ACTIONS.live.map((id) => ({
-                id,
-                label: ROW_ACTION_META[id].label,
-                destructive: ROW_ACTION_META[id].destructive,
-                onClick: async () => {
-                    await onBulkDelete(selection.selectedIds);
-                    selection.clearSelection();
-                },
-            }))
-            : 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,
-                },
-            ] })),
-    };
-    return (_jsxs(_Fragment, { children: [_jsx(DataListingView, { config: config }), deleteTargetId && (_jsx(ConfirmDeleteModal, { isOpen: true, title: "Delete Live Item Listing", message: "Are you sure you want to delete this live item listing? This cannot be undone.", onConfirm: () => handleDelete(deleteTargetId), onClose: () => setDeleteTargetId(null), isDeleting: deletingId === deleteTargetId }))] }));
+import { jsx as _jsx } from "react/jsx-runtime";
+import { SellerListingTypeView, } from "./SellerListingTypeView";
+export function SellerLiveView(props) {
+    return _jsx(SellerListingTypeView, { type: "live", ...props });
 }

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

@@ -210,7 +210,7 @@ export function SellerOrdersView({ orderDetailApiBase = SELLER_ENDPOINTS.ORDERS,
         }
         catch (err) {
             void normalizeError(err);
-            showToast(err instanceof Error ? err.message : "Failed to mark order shipped.", "error");
+            showToast("Failed to mark order shipped.", "error");
         }
         finally {
             setShippingRowId(null);
@@ -232,7 +232,7 @@ export function SellerOrdersView({ orderDetailApiBase = SELLER_ENDPOINTS.ORDERS,
         }
         catch (err) {
             void normalizeError(err);
-            showToast(err instanceof Error ? err.message : "Failed to update location.", "error");
+            showToast("Failed to update location.", "error");
         }
     }, [showToast]);
     const requestPayoutForSelection = useCallback(async (selection) => {
@@ -253,7 +253,7 @@ export function SellerOrdersView({ orderDetailApiBase = SELLER_ENDPOINTS.ORDERS,
         }
         catch (err) {
             void normalizeError(err);
-            showToast(err instanceof Error ? err.message : "Failed to request payout.", "error");
+            showToast("Failed to request payout.", "error");
         }
     }, [showToast]);
     const [selectedIdsForLocation, setSelectedIdsForLocation] = useState([]);

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

@@ -57,7 +57,7 @@ export function SellerPayoutMethodsView({ onCreateClick, onEditClick, onDelete,
         }
         catch (err) {
             void normalizeError(err);
-            showToast(err instanceof Error ? err.message : "Failed to delete payout method", "error");
+            showToast("Failed to delete payout method", "error");
         }
         setDeleteTargetId(null);
     }, [performDelete, showToast]);
@@ -79,7 +79,7 @@ export function SellerPayoutMethodsView({ onCreateClick, onEditClick, onDelete,
         }
         catch (err) {
             void normalizeError(err);
-            showToast(err instanceof Error ? err.message : "Failed to update default.", "error");
+            showToast("Failed to update default.", "error");
         }
         finally {
             setSettingDefaultId(null);

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

@@ -74,6 +74,11 @@ export interface SellerProductDraft {
 /** Shape returned by `onSave`/`onPublish` — structurally compatible with appkit's `ActionResult<T>` failure/success arms, without importing the server-only type into this client component. */
 export interface ProductActionResult {
     ok: boolean;
+    /**
+     * Stable error code. This is what the failure toast resolves — `error` is the
+     * server's own sentence and is reported, never shown (audit-raw-error-text).
+     */
+    code?: string;
     error?: string;
     issues?: unknown[];
 }

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

@@ -13,6 +13,7 @@ import { BarcodeField } from "./BarcodeField";
 import { pluginFor } from "../../../_internal/shared/listing-types/_registry";
 import { DEFAULT_MIN_OFFER_PERCENT } from "../../../_internal/shared/features/offers/config";
 import { normalizeError } from "../../../errors/normalize";
+import { toUserMessage } from "../../../errors/error-display-map";
 /**
  * "bundle" is a categoryType, not a ListingType, so it can't live in
  * LISTING_TYPE_REGISTRY — this is the single, explicit escape hatch that
@@ -346,18 +347,25 @@ export function SellerProductShell({ mode, listingType = "standard", initialValu
                 return;
             }
             applyZodIssues(result.issues ?? [], setFieldError);
-            if (!silent)
-                showToast(result.error || "Fix the highlighted errors and try again.", "error");
+            if (!silent) {
+                showToast(toUserMessage(result.code, undefined, {
+                    fallback: "Fix the highlighted errors and try again.",
+                }), "error");
+            }
         }
         catch (err) {
             void normalizeError(err);
             if (!silent)
-                showToast(err instanceof Error ? err.message : "Failed to save.", "error");
+                showToast("Failed to save.", "error");
         }
     }, [draft, onSave, markClean, clearErrors, setFieldError, showToast]);
     const handleAutoSave = useCallback(async () => {
         try {
             await handleSave(true);
+            // handleSave was called with silent=true, which is the whole point of
+            // auto-save: it has already applied the field errors and populated
+            // FormErrorSummary, so a toast every two seconds while the seller is
+            // still typing would be worse than saying nothing at all here.
         }
         catch (err) {
             void normalizeError(err);
@@ -383,11 +391,13 @@ export function SellerProductShell({ mode, listingType = "standard", initialValu
                 return;
             }
             applyZodIssues(result.issues ?? [], setFieldError);
-            showToast(result.error || "Fix the highlighted errors and try again.", "error");
+            showToast(toUserMessage(result.code, undefined, {
+                fallback: "Fix the highlighted errors and try again.",
+            }), "error");
         }
         catch (err) {
             void normalizeError(err);
-            showToast(err instanceof Error ? err.message : "Failed to publish.", "error");
+            showToast("Failed to publish.", "error");
         }
     }, [draft, onPublish, markClean, clearErrors, setFieldError, showToast]);
     const listingTypeLabel = pluginForMode(listingType).typeLabel;

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

@@ -1,6 +1,7 @@
 "use client";
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { normalizeError } from "../../../errors/normalize";
+import { toUserMessage } from "../../../errors/error-display-map";
 import { SIEVE_OP, Stack, sieveFilter } from "@mohasinac/appkit/client";
 import { sortBy } from "@mohasinac/appkit/client";
 import { useState, useCallback } from "react";
@@ -148,15 +149,23 @@ export function SellerProductsView({ onDeleteProduct, onCreateClick, }) {
         await performDelete(row.id);
     };
     const handleDuplicate = async (row) => {
-        const res = await fetch(SELLER_ENDPOINTS.PRODUCT_DUPLICATE(row.id), {
-            method: "POST",
-        }).catch(() => null);
-        if (res && res.ok) {
+        // A swallowed failure here does nothing at all and says nothing — the
+        // seller clicks Duplicate and no copy appears, with no way to tell whether
+        // the request even left the browser.
+        try {
+            const res = await fetch(SELLER_ENDPOINTS.PRODUCT_DUPLICATE(row.id), {
+                method: "POST",
+            });
+            if (!res.ok)
+                throw new Error(`Duplicate failed (${res.status})`);
             const json = await res.json().catch(() => null);
             const newId = json?.data?.id;
             if (newId)
                 handleEdit({ ...row, id: newId });
         }
+        catch (err) {
+            showToast(normalizeError(err).message, "error");
+        }
     };
     const handleTogglePublish = async (row) => {
         const currentStatus = statusOverrides.get(row.id) ?? row.status;
@@ -167,10 +176,15 @@ export function SellerProductsView({ onDeleteProduct, onCreateClick, }) {
                 method: "PATCH",
                 headers: { "Content-Type": "application/json" },
                 body: JSON.stringify({ status: newStatus }),
-            }).catch(() => null);
-            if (res?.ok) {
-                setStatusOverrides((prev) => new Map([...prev, [row.id, newStatus]]));
-            }
+            });
+            if (!res.ok)
+                throw new Error(`Could not change status (${res.status})`);
+            setStatusOverrides((prev) => new Map([...prev, [row.id, newStatus]]));
+        }
+        catch (err) {
+            // The row silently snapped back to its old status before this — a seller
+            // could not tell a rejected publish from a UI that had not repainted.
+            showToast(normalizeError(err).message, "error");
         }
         finally {
             setPublishingId(null);
@@ -191,8 +205,8 @@ export function SellerProductsView({ onDeleteProduct, onCreateClick, }) {
             setSetLocationOpen(false);
         }
         catch (err) {
-            void normalizeError(err);
-            showToast(err instanceof Error ? err.message : "Failed to update location.", "error");
+            const e = normalizeError(err);
+            showToast(toUserMessage(e.code, undefined, { fallback: "Failed to update location." }), "error");
         }
     }, [showToast]);
     const config = {