@mohasinac/appkit 4.30.17 → 4.30.19

package/dist/features/scams/actions/scam-actions.js

@@ -1,6 +1,7 @@
 "use server";
 import { sieveFilter, SIEVE_OP } from "@mohasinac/appkit";
 import { scammerRepository } from "../repository/scammer.repository";
+import { safeRead } from "../../../errors/safe-read";
 import { userRepository } from "../../auth/repository/user.repository";
 import { storeRepository } from "../../stores/repository/store.repository";
 import { safeFireAndForget } from "../../../utils/safe-fire-forget";
@@ -39,9 +40,12 @@ export async function listVerifiedScammers(searchParams) {
  */
 export async function getPublicScammerById(id) {
     // Try direct id lookup first; fall back to seoSlug query.
-    let doc = await scammerRepository.findById(id).catch(() => null);
+    // The profile IS the caller's subject, and both lookups already return null
+    // for "no such id/slug" — swallowing a read failure into that same null would
+    // report a real outage as a missing profile.
+    let doc = await scammerRepository.findById(id);
     if (!doc) {
-        doc = await scammerRepository.findBySeoSlug(id).catch(() => null);
+        doc = await scammerRepository.findBySeoSlug(id);
     }
     if (!doc || doc.status !== "verified")
         return null;
@@ -66,27 +70,63 @@ export async function getScammerProfilePageData(id) {
     return { scammer, incidents, comments, relatedScammers, similarScamReports };
 }
 const TRUST_STATUS_CLEAR = { status: "clear", matchedProfileSlugs: [] };
+const TRUST_STATUS_UNKNOWN = { status: "unknown", matchedProfileSlugs: [] };
+/**
+ * Wrap a read so a FAILURE is distinguishable from a legitimate empty result.
+ *
+ * `safeRead` alone cannot express this: its fallback has the same type as a
+ * successful read, so `null` from a failed store lookup and `null` from a store
+ * that does not exist are the same value. Tagging the outcome is what lets the
+ * caller answer "unknown" instead of "clear".
+ */
+async function readTagged(fn, key) {
+    return safeRead(async () => ({ ok: true, value: await fn() }), {
+        route: "sellerTrust",
+        key,
+        fallback: { ok: false, value: null },
+    });
+}
 /**
  * P-12 — resolves a store owner's phone/email against the verified (admin-published)
  * scammer registry only. Never surfaces "under review" reports to buyers — a report
  * that hasn't been verified must not damage a seller's storefront reputation.
  */
 export async function getSellerTrustStatus(storeId) {
-    const store = await storeRepository.findById(storeId).catch(() => null);
-    if (!store?.ownerId)
+    // Every read below fails OPEN to "clear" — a deliberate choice, because a
+    // registry outage must not brand a legitimate seller a scammer. But "clear"
+    // is a positive safety claim, so a failure that produces one has to be
+    // recorded rather than inferred from a silent null.
+    const store = await readTagged(() => storeRepository.findById(storeId), "sellerTrust.store");
+    if (!store.ok)
+        return TRUST_STATUS_UNKNOWN;
+    if (!store.value?.ownerId)
         return TRUST_STATUS_CLEAR;
-    const owner = await userRepository.findById(store.ownerId).catch(() => null);
-    if (!owner)
+    const ownerId = store.value.ownerId;
+    const owner = await readTagged(() => userRepository.findById(ownerId), "sellerTrust.owner");
+    if (!owner.ok)
+        return TRUST_STATUS_UNKNOWN;
+    if (!owner.value)
         return TRUST_STATUS_CLEAR;
+    const ownerDoc = owner.value;
     const lookups = [];
-    if (owner.phoneNumber)
-        lookups.push(scammerRepository.findByContactField("phones", owner.phoneNumber));
-    if (owner.email)
-        lookups.push(scammerRepository.findByContactField("emails", owner.email));
+    if (ownerDoc.phoneNumber) {
+        const phone = ownerDoc.phoneNumber;
+        lookups.push(() => scammerRepository.findByContactField("phones", phone));
+    }
+    if (ownerDoc.email) {
+        const email = ownerDoc.email;
+        lookups.push(() => scammerRepository.findByContactField("emails", email));
+    }
     if (lookups.length === 0)
         return TRUST_STATUS_CLEAR;
-    const results = await Promise.all(lookups.map((p) => p.catch(() => [])));
-    const verified = results.flat().filter((s) => s.status === "verified");
+    const results = await Promise.all(lookups.map((fn) => readTagged(fn, "sellerTrust.registryLookup")));
+    // A partial answer is not an answer: if ANY registry lookup failed we cannot
+    // say the seller is clear, only that we could not check.
+    if (results.some((r) => !r.ok))
+        return TRUST_STATUS_UNKNOWN;
+    const verified = results
+        .flatMap((r) => r.value ?? [])
+        .filter((s) => s.status === "verified");
     if (verified.length === 0)
         return TRUST_STATUS_CLEAR;
     return {

package/dist/features/scams/components/SellerTrustBadge.d.ts

@@ -13,4 +13,4 @@ export interface SellerTrustBadgeProps {
  * unrelated homepage-section `TrustBadge` config type in
  * features/homepage/index.ts (marketing trust-badge icons row).
  */
-export declare function SellerTrustBadge({ trust, className }: SellerTrustBadgeProps): import("react").JSX.Element;
+export declare function SellerTrustBadge({ trust, className }: SellerTrustBadgeProps): import("react").JSX.Element | null;

package/dist/features/scams/components/SellerTrustBadge.js

@@ -13,6 +13,19 @@ import { ROUTES } from "../../../next";
  * features/homepage/index.ts (marketing trust-badge icons row).
  */
 export function SellerTrustBadge({ trust, className = "" }) {
+    // 🛑 `unknown` MUST be handled before the flagged fallthrough below, and must
+    // never render as "Verified Safe".
+    //
+    // Every read in getSellerTrustStatus used to fail open to "clear", so a
+    // scam-registry outage produced an explicit safety endorsement for a seller
+    // nobody had checked. Failing open is still right — an outage must not brand
+    // a legitimate seller a scammer — but the honest failure state is "we could
+    // not check", which is neither of the two claims this badge otherwise makes.
+    // Rendering nothing is deliberate: a neutral badge would still be a statement,
+    // and the absence of a badge already means "no verified match" everywhere the
+    // registry is not consulted.
+    if (trust.status === "unknown")
+        return null;
     if (trust.status === "clear") {
         return (_jsx(Badge, { variant: "success", className: className, children: "\u2713 Verified Safe" }));
     }

package/dist/features/scams/repository/scammer.repository.js

@@ -122,10 +122,11 @@ class ScammerRepository extends BaseRepository {
             await this.getCollection()
                 .doc(id)
                 .update({ [SCAMMER_FIELDS.VIEWS]: increment(1) });
+            // Fire-and-forget analytics: a lost view increment changes no rendered
+            // content, so it must never break the profile read it rides along with.
         }
         catch (_err) {
             void normalizeError(_err);
-            // Fire-and-forget by design.
         }
     }
     /**

package/dist/features/seller/actions/offer-actions.js

@@ -15,6 +15,7 @@ import { cartRepository } from "../../cart/repository/cart.repository";
 import { maskOfferForSeller } from "../../../security";
 import { ERROR_MESSAGES, AuthorizationError, ValidationError, NotFoundError, OFFER_ERROR_CODES, } from "../../../errors";
 import { siteSettingsRepository } from "../../admin/repository/site-settings.repository";
+import { safeRead } from "../../../errors/safe-read";
 import { canMakeOffer } from "../../../_internal/shared/listing-types/capabilities";
 import { normalizeListingType } from "../../products/utils/listing-type";
 import { OfferStatusValues } from "../schemas";
@@ -36,7 +37,11 @@ export async function makeOffer(userId, userEmail, input) {
      * without stranding in-flight ones, so accept/decline/counter/withdraw and
      * `checkoutOffer` deliberately stay open.
      */
-    const settings = await siteSettingsRepository.getSingleton().catch(() => null);
+    // Fails OPEN — a settings read that dies must not block offers site-wide. But
+    // `null` here is indistinguishable from "the flag is on", so it is recorded.
+    const settings = await safeRead(() => siteSettingsRepository.getSingleton(), {
+        route: "offers.makeOffer", key: "makeOffer.siteSettings", fallback: null,
+    });
     if (settings?.featureFlags?.offers === false) {
         throw new ValidationError(ERROR_MESSAGES.OFFER.DISABLED, {
             code: OFFER_ERROR_CODES.DISABLED,

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

@@ -37,12 +37,15 @@ export function FulfillmentView({ currentUserId, }) {
             setOrders(res.orders ?? []);
         }
         catch (err) {
-            void normalizeError(err);
+            // An empty fulfilment queue is a normal, expected state, so a swallowed
+            // failure here reads as "nothing to pack" — matching the rest of this
+            // file, surface it instead.
+            showToast(normalizeError(err).message, "error");
         }
         finally {
             setIsLoadingOrders(false);
         }
-    }, []);
+    }, [showToast]);
     useEffect(() => {
         void fetchOrders();
     }, [fetchOrders]);

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

@@ -22,7 +22,7 @@ export function PhysicalLocationModal({ count, onSave, onClose }) {
         }
         catch (e) {
             void normalizeError(e);
-            setError(e instanceof Error ? e.message : "Failed to save location");
+            setError("Failed to save location");
         }
         finally {
             setSaving(false);

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

@@ -20,10 +20,12 @@ function ProductLabel({ product }) {
                     fontSize: 12,
                     margin: 8,
                 });
+                // JsBarcode throws on a value CODE128 cannot encode. The <svg> is left
+                // empty and the label still prints its title and price — there is no
+                // recovery beyond fixing the barcodeId on the product itself.
             }
             catch (_err) {
                 void normalizeError(_err);
-                // Invalid barcode value — render nothing
             }
         }
     }, [product.barcodeId]);

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

@@ -88,7 +88,7 @@ export function SellerBidsView({ endpoint = SELLER_ENDPOINTS.BIDS }) {
         }
         catch (err) {
             void normalizeError(err);
-            showToast(err instanceof Error ? err.message : "Failed to cancel bids.", "error");
+            showToast("Failed to cancel bids.", "error");
         }
     }, [showToast]);
     const config = {

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

@@ -32,7 +32,7 @@ export function SellerCouponsView({ onCreateClick, onEditClick, onToggle, onDele
         }
         catch (err) {
             void normalizeError(err);
-            showToast(err instanceof Error ? err.message : "Failed to update coupon.", "error");
+            showToast("Failed to update coupon.", "error");
         }
         finally {
             setTogglingId(null);

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

@@ -76,7 +76,7 @@ export function SellerGoogleReviewsView({ onSave, onSync, }) {
         catch (err) {
             void normalizeError(err);
             setSaveMessage({ text: "Save failed. Please try again.", ok: false });
-            showToast(err instanceof Error ? err.message : "Failed to save settings.", "error");
+            showToast("Failed to save settings.", "error");
         }
         finally {
             setSaving(false);
@@ -101,7 +101,7 @@ export function SellerGoogleReviewsView({ onSave, onSync, }) {
         catch (err) {
             void normalizeError(err);
             setSaveMessage({ text: "Sync failed. Please try again.", ok: false });
-            showToast(err instanceof Error ? err.message : "Failed to sync reviews.", "error");
+            showToast("Failed to sync reviews.", "error");
         }
         finally {
             setSyncing(false);

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 = {

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

@@ -3,7 +3,7 @@ import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-run
 import { normalizeError } from "../../../errors/normalize";
 import { useCallback, useState } from "react";
 import { useQueryClient } from "@tanstack/react-query";
-import { Alert, Badge, Button, Checkbox, Div, Modal, Row, Select, SideDrawer, Span, Stack, Text, Textarea } from "../../../ui";
+import { Alert, Badge, Button, Checkbox, Div, Modal, Row, Select, SideDrawer, Span, Stack, Text, Textarea, useToast } from "../../../ui";
 import { ACTIONS } from "../../../_internal/shared/actions/action-registry";
 import { SELLER_BULK_ACTIONS, ROW_ACTION_META } from "../../products/constants/action-defs";
 import { SELLER_ENDPOINTS } from "../../../constants/api-endpoints";
@@ -34,6 +34,7 @@ function statusBadge(status) {
 }
 export function SellerReviewsView({ reviewsApiBase = SELLER_ENDPOINTS.REVIEWS, replyApiBase = SELLER_ENDPOINTS.REVIEWS, }) {
     const queryClient = useQueryClient();
+    const { showToast } = useToast();
     const refetchReviews = useCallback(() => queryClient.invalidateQueries({ queryKey: ["seller", "reviews", "listing"] }), [queryClient]);
     // Reply drawer
     const [replyTarget, setReplyTarget] = useState(null);
@@ -80,45 +81,75 @@ export function SellerReviewsView({ reviewsApiBase = SELLER_ENDPOINTS.REVIEWS, r
             setReplySaving(false);
         }
     };
+    // The three submits below all closed their modal unconditionally, so a
+    // rejected or unsent request looked exactly like a successful one — the
+    // seller believed the contest/feedback/reply had been filed.
     const submitContest = useCallback(async (onDone) => {
         if (!contestTarget)
             return;
-        await fetch(SELLER_ENDPOINTS.REVIEW_CONTEST(contestTarget.id), {
-            method: "POST",
-            headers: { "Content-Type": "application/json" },
-            body: JSON.stringify({ reason: contestReason }),
-        }).catch(() => null);
+        try {
+            const res = await fetch(SELLER_ENDPOINTS.REVIEW_CONTEST(contestTarget.id), {
+                method: "POST",
+                headers: { "Content-Type": "application/json" },
+                body: JSON.stringify({ reason: contestReason }),
+            });
+            if (!res.ok)
+                throw new Error(`Could not submit the contest (${res.status})`);
+        }
+        catch (err) {
+            showToast(normalizeError(err).message, "error");
+            return;
+        }
         setContestTarget(null);
         setContestReason("");
         onDone();
-    }, [contestTarget, contestReason]);
+    }, [contestTarget, contestReason, showToast]);
     const submitFeedback = useCallback(async () => {
         if (!feedbackTarget)
             return;
-        await fetch(SELLER_ENDPOINTS.REVIEW_FEEDBACK(feedbackTarget.id), {
-            method: "POST",
-            headers: { "Content-Type": "application/json" },
-            body: JSON.stringify({ feedback: feedbackText }),
-        }).catch(() => null);
+        try {
+            const res = await fetch(SELLER_ENDPOINTS.REVIEW_FEEDBACK(feedbackTarget.id), {
+                method: "POST",
+                headers: { "Content-Type": "application/json" },
+                body: JSON.stringify({ feedback: feedbackText }),
+            });
+            if (!res.ok)
+                throw new Error(`Could not send the feedback (${res.status})`);
+        }
+        catch (err) {
+            showToast(normalizeError(err).message, "error");
+            return;
+        }
         setFeedbackTarget(null);
         setFeedbackText("");
-    }, [feedbackTarget, feedbackText]);
+    }, [feedbackTarget, feedbackText, showToast]);
     const submitBulkReply = useCallback(async () => {
         if (!bulkReplyText.trim() || bulkTargetIds.length === 0)
             return;
         setBulkSaving(true);
-        await Promise.all(bulkTargetIds.map((id) => fetch(`${replyApiBase}/${id}/reply`, {
+        // Partial success is the normal outcome of a bulk write, so report the
+        // count that failed rather than throwing away the whole batch's result.
+        const outcomes = await Promise.all(bulkTargetIds.map((id) => fetch(`${replyApiBase}/${id}/reply`, {
             method: "POST",
             headers: { "Content-Type": "application/json" },
             body: JSON.stringify({ reply: bulkReplyText }),
-        }).catch(() => null)));
+        })
+            .then((res) => res.ok)
+            .catch((err) => {
+            void normalizeError(err);
+            return false;
+        })));
+        const failed = outcomes.filter((ok) => !ok).length;
+        if (failed > 0) {
+            showToast(`${failed} of ${outcomes.length} replies could not be saved.`, "error");
+        }
         setBulkSaving(false);
         setBulkReplyOpen(false);
         setBulkReplyText("");
         bulkClearSelection?.();
         setBulkTargetIds([]);
         await refetchReviews();
-    }, [bulkReplyText, bulkTargetIds, replyApiBase, bulkClearSelection, refetchReviews]);
+    }, [bulkReplyText, bulkTargetIds, replyApiBase, bulkClearSelection, refetchReviews, showToast]);
     const config = {
         portal: "seller",
         title: "Reviews",

package/dist/features/seller/components/SellerShippingConfigsView.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 { sortBy } from "@mohasinac/appkit/client";
 import { useState, useCallback } from "react";
 import { useEntityDelete } from "../../../react/hooks/useEntityDelete";
@@ -53,8 +54,8 @@ export function SellerShippingConfigsView({ onCreateClick, onEditClick, onDelete
             await performDelete(id);
         }
         catch (err) {
-            void normalizeError(err);
-            showToast(err instanceof Error ? err.message : "Failed to delete shipping config", "error");
+            const e = normalizeError(err);
+            showToast(toUserMessage(e.code, undefined, { fallback: "Failed to delete shipping config." }), "error");
         }
         setDeleteTargetId(null);
     }, [performDelete, showToast]);
@@ -75,8 +76,8 @@ export function SellerShippingConfigsView({ onCreateClick, onEditClick, onDelete
             showToast("Default shipping config updated.", "success");
         }
         catch (err) {
-            void normalizeError(err);
-            showToast(err instanceof Error ? err.message : "Failed to update default.", "error");
+            const e = normalizeError(err);
+            showToast(toUserMessage(e.code, undefined, { fallback: "Failed to update default." }), "error");
         }
         finally {
             setSettingDefaultId(null);

package/dist/features/seller/components/SellerStoreCategoriesView.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 { Badge, sortBy } from "@mohasinac/appkit/client";
 import { useState, useCallback } from "react";
 import { useEntityDelete } from "../../../react/hooks/useEntityDelete";
@@ -49,8 +50,8 @@ export function SellerStoreCategoriesView({ onCreateClick, onEditClick, onDelete
             await performDelete(id);
         }
         catch (err) {
-            void normalizeError(err);
-            showToast(err instanceof Error ? err.message : "Failed to delete category", "error");
+            const e = normalizeError(err);
+            showToast(toUserMessage(e.code, undefined, { fallback: "Failed to delete category." }), "error");
         }
         setDeleteTargetId(null);
     }, [onDelete, performDelete, showToast]);

package/dist/features/stores/components/StoreDetailLayoutView.d.ts

@@ -1,6 +1,6 @@
 import React from "react";
 import type { ReactNode } from "react";
-export declare const getStoreBySlug: (slug: string) => Promise<import("@mohasinac/appkit").StoreDocument | null | undefined>;
+export declare const getStoreBySlug: (slug: string) => Promise<import("@mohasinac/appkit").StoreDocument | null>;
 export interface StoreDetailLayoutViewProps {
     storeSlug: string;
     /** The current active tab value: "products" | "auctions" | "reviews" | "about" */

package/dist/features/stores/components/StoreDetailLayoutView.js

@@ -2,6 +2,7 @@ import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { sieveFilter, sieveAnd, SIEVE_OP } from "@mohasinac/appkit";
 import { cache } from "react";
 import { storeRepository, siteSettingsRepository, reviewRepository, couponsRepository } from "../../../repositories";
+import { safeRead } from "../../../errors/safe-read";
 import { ROUTES } from "../../../next";
 import { Container, Main, Section, Text } from "../../../ui";
 import { STORE_PAGE_TABS } from "../../products/constants/listing-tabs";
@@ -26,7 +27,10 @@ const STORE_LISTING_HREF = {
     // a real non-zero count and landed on a page with none of those items.
     art: (slug) => String(ROUTES.PUBLIC.STORE_ART(slug)),
 };
-export const getStoreBySlug = cache((slug) => storeRepository.findBySlug(slug).catch(() => undefined));
+// The store IS this layout's subject, and `findBySlug` already returns null for
+// "no such slug" — swallowing a read failure into the same value would render
+// the "Store not found." branch below for a live store during an outage.
+export const getStoreBySlug = cache((slug) => storeRepository.findBySlug(slug));
 function tabLabel(base, count) {
     if (!count)
         return base;
@@ -39,9 +43,14 @@ export async function StoreDetailLayoutView({ storeSlug, activeTab, children, sc
     }
     const storeId = store?.id;
     const [settings, trust] = await Promise.all([
-        siteSettingsRepository.findById("global").catch(() => null),
+        safeRead(() => siteSettingsRepository.findById("global"), {
+            route: "/stores/[slug]", key: "store.siteSettings", fallback: null,
+        }),
         scamRegistryEnabled && storeId
-            ? getSellerTrustStatus(storeId).catch(() => undefined)
+            // Fails open to "no trust verdict", which hides the badge entirely rather
+            // than asserting a seller is clear — but that has to be recorded, since a
+            // missing badge is exactly what a clean seller looks like.
+            ? safeRead(() => getSellerTrustStatus(storeId), { route: "/stores/[slug]", key: "store.sellerTrust", fallback: undefined })
             : Promise.resolve(undefined),
     ]);
     // One derived batch instead of nine copy-pasted per-type queries. The tabs