@mohasinac/appkit 4.30.17 → 4.30.19

package/dist/features/cart/actions/cart-actions.js

@@ -14,6 +14,7 @@ import { getDefaultCurrency } from "../../../core/baseline-resolver";
 import { CART_LANE, activeLane, canAddNewItems, } from "../../../_internal/shared/checkout/lanes";
 import { CART_MAX_ITEMS } from "../../../constants/limits";
 import { getExpandedDecrements, validateCartItemStock, } from "../../../_internal/server/features/checkout/bundle-expansion";
+import { safeRead } from "../../../errors/safe-read";
 /**
  * Product documents don't always carry a denormalized `storeName` — the
  * create routes only set it when the caller explicitly passes one, so
@@ -26,7 +27,9 @@ import { getExpandedDecrements, validateCartItemStock, } from "../../../_interna
 async function resolveStoreName(storeId, candidate) {
     if (candidate)
         return candidate;
-    const store = await storeRepository.findById(storeId).catch(() => null);
+    const store = await safeRead(() => storeRepository.findById(storeId), {
+        route: "/cart", key: "stores.findById", fallback: null,
+    });
     return store?.storeName ?? storeId;
 }
 /**
@@ -102,7 +105,9 @@ export async function addGroupLineToCart(userId, input) {
     const cart = await assertCanAddNewItems(userId);
     // One batched read for the whole selection — capped at 20 above, so well
     // inside the per-request Firestore budget.
-    const docs = await Promise.all(members.map((m) => productRepository.findById(m.productId).catch(() => null)));
+    const docs = await Promise.all(members.map((m) => safeRead(() => productRepository.findById(m.productId), {
+        route: "/cart", key: "products.findById", fallback: null,
+    })));
     const productById = new Map(members.map((m, i) => [m.productId, docs[i]]));
     const { getListingRule } = await import("../../../_internal/shared/checkout/rules");
     const { ProductStatusValues } = await import("../../products/schemas/firestore");
@@ -209,14 +214,18 @@ export async function addGroupLineToCart(userId, input) {
 /** Is this product listed on that `groupedListings` document? */
 async function isGroupedListingMember(groupId, productId) {
     const { groupedListingsRepository } = await import("../../grouped/repository/grouped-listings.repository");
-    const group = await groupedListingsRepository.findById(groupId).catch(() => null);
+    const group = await safeRead(() => groupedListingsRepository.findById(groupId), {
+        route: "/cart", key: "groupedListings.findById", fallback: null,
+    });
     return Boolean(group?.productIds?.includes(productId));
 }
 /** Display name for the line. Falls back to the parent product's group title. */
 async function resolveGroupTitle(groupId, groupSource, sample) {
     if (groupSource === "grouped-listing") {
         const { groupedListingsRepository } = await import("../../grouped/repository/grouped-listings.repository");
-        const group = await groupedListingsRepository.findById(groupId).catch(() => null);
+        const group = await safeRead(() => groupedListingsRepository.findById(groupId), {
+            route: "/cart", key: "groupedListings.title", fallback: null,
+        });
         if (group?.title)
             return group.title;
     }

package/dist/features/cart/utils/pending-ops.js

@@ -72,10 +72,10 @@ function write(key, items) {
         return;
     try {
         localStorage.setItem(key, JSON.stringify(items));
+        // Quota exceeded or storage blocked. Losing pending ops costs a retry, never data: the cart is canonical in Firestore.
     }
     catch (_err) {
         void normalizeError(_err);
-        // storage quota exceeded — best effort
     }
 }
 function writeCartOps(items) {

package/dist/features/categories/components/BrandDetailPageView.js

@@ -16,24 +16,25 @@ import { listingTabCounts, } from "../../../_internal/server/features/products/l
 import { CATEGORY_PAGE_TABS } from "../../products/constants/listing-tabs";
 import { enabledCategoryTypes, enabledListingTypes } from "../../../_internal/shared/listing-types/feature-flags";
 import { siteSettingsRepository } from "../../../repositories";
+import { safeRead } from "../../../errors/safe-read";
 const __O = {
     hidden: "overflow-hidden",
 };
 export async function BrandDetailPageView({ slug, initialBrand }) {
+    // The brand IS this page's subject — a read failure must surface rather than
+    // render an anonymous, empty brand page. `undefined` still means "no such
+    // brand slug".
     const brand = (initialBrand ?? (await categoriesRepository
-        .getCategoryBySlug(slug)
-        .catch(() => undefined)));
+        .getCategoryBySlug(slug)) ?? undefined);
     const brandName = brand?.name;
     const [productsResult, tabCounts, allBundles, activeBrands, groupedListings, settings] = await Promise.all([
         brandName
-            ? productRepository
-                .list({
+            ? safeRead(() => productRepository.list({
                 filters: sieveAnd(sieveFilter("status", SIEVE_OP.EQ, "published"), sieveFilter("brand", SIEVE_OP.EQ, brandName), sieveFilter("listingType", SIEVE_OP.EQ, "standard")),
                 sorts: sortBy("createdAt", "DESC"),
                 page: 1,
                 pageSize: 24,
-            })
-                .catch(() => null)
+            }), { route: "/brands/[slug]", key: "brand.products", fallback: null })
             : Promise.resolve(null),
         // One count per tab, derived from CATEGORY_PAGE_TABS. Four of the nine tabs
         // (classifieds, digital codes, live, art) had no count at all and could
@@ -47,16 +48,31 @@ export async function BrandDetailPageView({ slug, initialBrand }) {
         // We pull all bundle categories and filter client-side by brandSlug —
         // no dedicated repository query needed at this catalog's bundle volume.
         brand?.slug
-            ? categoriesRepository
-                .listByType("bundle", { activeOnly: true, limit: 50 })
-                .catch(() => [])
+            ? safeRead(() => categoriesRepository.listByType("bundle", { activeOnly: true, limit: 50 }), { route: "/brands/[slug]", key: "brand.bundles", fallback: [] })
             : Promise.resolve([]),
         // Related brands — every other active brand row, excluding this one.
-        categoriesRepository.findActiveBrands().catch(() => []),
-        brand?.slug ? getGroupsForBrand(brand.slug).catch(() => []) : Promise.resolve([]),
+        // `as unknown` is load-bearing only because CategoryDocument.createdAt is a
+        // Date while CategoryItem.createdAt is a string — a pre-existing mismatch
+        // the old `.catch(() => [])` union happened to paper over.
+        safeRead(() => categoriesRepository.findActiveBrands(), {
+            route: "/brands/[slug]",
+            key: "brand.relatedBrands",
+            fallback: [],
+        }),
+        brand?.slug
+            ? safeRead(() => getGroupsForBrand(brand.slug), {
+                route: "/brands/[slug]",
+                key: "brand.groupedListings",
+                fallback: [],
+            })
+            : Promise.resolve([]),
         // Feature flags — accepted by BrandDetailTabs all along but never passed,
         // so a disabled listing type still showed its tab here.
-        siteSettingsRepository.findById("global").catch(() => null),
+        safeRead(() => siteSettingsRepository.findById("global"), {
+            route: "/brands/[slug]",
+            key: "brand.siteSettings",
+            fallback: null,
+        }),
     ]);
     const brandBundles = brand?.slug
         ? allBundles.filter((b) => b.brandSlug === brand.slug)

package/dist/features/categories/components/CategoryDetailPageView.js

@@ -15,14 +15,17 @@ import { listingTabCounts, } from "../../../_internal/server/features/products/l
 import { CATEGORY_PAGE_TABS } from "../../products/constants/listing-tabs";
 import { enabledCategoryTypes, enabledListingTypes } from "../../../_internal/shared/listing-types/feature-flags";
 import { siteSettingsRepository } from "../../../repositories";
+import { safeRead } from "../../../errors/safe-read";
 const __O = {
     hidden: "overflow-hidden",
     xAuto: "overflow-x-auto",
 };
 export async function CategoryDetailPageView({ slug }) {
-    const category = await categoriesRepository
-        .getCategoryBySlug(slug)
-        .catch(() => undefined);
+    // The category IS this page's subject — a read failure must surface rather
+    // than render an anonymous, empty category page. `undefined` still means
+    // "no such category slug".
+    const category = ((await categoriesRepository
+        .getCategoryBySlug(slug)) ?? undefined);
     // Products carry their FULL ancestor chain in `categorySlugs`, so matching on
     // this category's own id already returns the whole subtree — no descendant
     // expansion needed. Expanding it was also a latent outage: `categoriesIn` is
@@ -33,14 +36,12 @@ export async function CategoryDetailPageView({ slug }) {
     const categoriesIn = category?.id ? [category.id] : null;
     const [productsResult, tabCounts, bundlesResult, childCategories, rootSiblingCategories, groupedListings, settings] = await Promise.all([
         categoriesIn
-            ? productRepository
-                .list({
+            ? safeRead(() => productRepository.list({
                 filters: sieveAnd(sieveFilter("status", SIEVE_OP.EQ, "published"), sieveFilter("listingType", SIEVE_OP.EQ, "standard")),
                 sorts: sortBy("createdAt", "DESC"),
                 page: 1,
                 pageSize: 24,
-            }, { categoriesIn })
-                .catch(() => null)
+            }, { categoriesIn }), { route: "/categories/[slug]", key: "category.products", fallback: null })
             : Promise.resolve(null),
         // One count per tab, derived from CATEGORY_PAGE_TABS itself — so every
         // listing type is counted and a type with nothing in this category hides.
@@ -51,24 +52,40 @@ export async function CategoryDetailPageView({ slug }) {
         // SB-UNI-D — bundles fetched from the categories collection. We pull
         // all active bundle rows; the carousel filters by category affinity.
         categoriesIn
-            ? categoriesRepository
-                .listByType("bundle", { activeOnly: true, limit: 50 })
-                .catch(() => [])
+            ? safeRead(() => categoriesRepository.listByType("bundle", { activeOnly: true, limit: 50 }), { route: "/categories/[slug]", key: "category.bundles", fallback: [] })
             : Promise.resolve([]),
         category?.id
-            ? categoriesRepository.getChildren(category.id).catch(() => [])
+            ? safeRead(() => categoriesRepository.getChildren(category.id), {
+                route: "/categories/[slug]",
+                key: "category.children",
+                fallback: [],
+            })
             : Promise.resolve([]),
         // Related categories — every other category sharing this category's root
         // (siblings + cousins across the tree, up to the tier-0/1/2 depth the
         // catalog actually uses), not just direct children.
         category?.rootId
-            ? categoriesRepository.getCategoriesByRootId(category.rootId).catch(() => [])
+            ? safeRead(() => categoriesRepository.getCategoriesByRootId(category.rootId), {
+                route: "/categories/[slug]",
+                key: "category.rootSiblings",
+                fallback: [],
+            })
+            : Promise.resolve([]),
+        category?.slug
+            ? safeRead(() => getGroupsForCategory(category.slug), {
+                route: "/categories/[slug]",
+                key: "category.groupedListings",
+                fallback: [],
+            })
             : Promise.resolve([]),
-        category?.slug ? getGroupsForCategory(category.slug).catch(() => []) : Promise.resolve([]),
         // Listing/category-type feature flags. This page accepted the props all
         // along but never passed them, so an admin disabling a listing type still
         // saw its tab here while the store page correctly hid it.
-        siteSettingsRepository.findById("global").catch(() => null),
+        safeRead(() => siteSettingsRepository.findById("global"), {
+            route: "/categories/[slug]",
+            key: "category.siteSettings",
+            fallback: null,
+        }),
     ]);
     const relatedCategories = rootSiblingCategories.filter((c) => c.id !== category?.id && (!c.categoryType || c.categoryType === "category"));
     // Stores tab — stores whose storeCategory is this category or ANY descendant.
@@ -82,18 +99,18 @@ export async function CategoryDetailPageView({ slug }) {
     // which caps at 30 values, hence the chunking.
     const storeCategorySlugs = [
         slug,
-        ...(await categoriesRepository
-            .getDescendantIds(category?.id ?? "")
-            .catch(() => [])),
+        ...(await safeRead(() => categoriesRepository.getDescendantIds(category?.id ?? ""), {
+            route: "/categories/[slug]",
+            key: "category.descendantIds",
+            fallback: [],
+        })),
     ].filter(Boolean);
     const SLUG_CHUNK = 30;
     const slugChunks = [];
     for (let i = 0; i < storeCategorySlugs.length; i += SLUG_CHUNK) {
         slugChunks.push(storeCategorySlugs.slice(i, i + SLUG_CHUNK));
     }
-    const storeResults = await Promise.all(slugChunks.map((chunk) => storeRepository
-        .listStores({ filters: sieveFilter("storeCategory", SIEVE_OP.EQ, chunk.join("|")), page: 1, pageSize: 50 }, true)
-        .catch(() => null)));
+    const storeResults = await Promise.all(slugChunks.map((chunk) => safeRead(() => storeRepository.listStores({ filters: sieveFilter("storeCategory", SIEVE_OP.EQ, chunk.join("|")), page: 1, pageSize: 50 }, true), { route: "/categories/[slug]", key: "category.stores", fallback: null })));
     const seen = new Set();
     const categoryStores = storeResults
         .flatMap((r) => r?.items ?? [])

package/dist/features/categories/repository/categories.repository.js

@@ -480,10 +480,12 @@ export class CategoriesRepository extends BaseRepository {
                 .collection(this.collection)
                 .doc(categoryId)
                 .update({ [CATEGORY_FIELDS.VIEW_COUNT]: increment(1) });
+            // Fire-and-forget analytics: a lost view increment changes nothing that
+            // renders, so it must never break the category read path it rides along
+            // with — the count is approximate by design.
         }
         catch (_err) {
             void normalizeError(_err);
-            // View analytics must not break category read path.
         }
     }
     generateSublistingId(name) {

package/dist/features/checkout/actions/checkout-value-otp-actions.js

@@ -9,6 +9,7 @@
 import { timingSafeEqual } from "crypto";
 import { ValidationError } from "../../../errors";
 import { normalizeError } from "../../../errors/normalize";
+import { safeRead } from "../../../errors/safe-read";
 import { serverLogger } from "../../../monitoring";
 import { sendEmail } from "../../contact/email";
 import { getAdminDb } from "../../../providers/db-firebase";
@@ -47,7 +48,15 @@ export async function sendCheckoutValueOtp(userId, userEmail, channel = "email",
         if (!addressId) {
             throw new ValidationError("Select a delivery address before sending the code via WhatsApp.");
         }
-        const settings = await siteSettingsRepository.getSingleton().catch(() => null);
+        // A failed settings read makes the WhatsApp channel look deliberately
+        // switched off ("send the code by email instead") — a defensible fallback,
+        // but one an operator has to be able to see, since it is indistinguishable
+        // from an admin having disabled it.
+        const settings = await safeRead(() => siteSettingsRepository.getSingleton(), {
+            route: "checkout/value-otp",
+            key: "checkoutValueOtp.siteSettings",
+            fallback: null,
+        });
         const creds = settings?.credentials ?? {};
         const whatsappOtpEnabled = settings?.notificationChannels?.whatsapp?.otpEnabled === true;
         const phoneNumberId = creds.whatsappPhoneNumberId?.trim() ?? "";

package/dist/features/classified/components/ClassifiedContactSellerPanel.js

@@ -36,7 +36,7 @@ export function ClassifiedContactSellerPanel({ productId, onContactSeller }) {
         }
         catch (e) {
             void normalizeError(e);
-            setError(e instanceof Error ? e.message : "Something went wrong");
+            setError("Something went wrong");
         }
         finally {
             setPending(false);

package/dist/features/classified/components/ClassifiedDetailPageView.js

@@ -22,6 +22,7 @@ import { GroupedListingsCarousel } from "../../grouped/components/GroupedListing
 import { getGroupsWithItemsForProduct } from "../../../_internal/server/features/grouped/data";
 import { ClassifiedContactSellerPanel } from "./ClassifiedContactSellerPanel";
 import { getSiteSettingsGlobal } from "../../admin/utils/getSiteSettingsGlobal";
+import { safeRead } from "../../../errors/safe-read";
 import { canMakeOffer } from "../../../_internal/shared/listing-types/capabilities";
 import { DEFAULT_MIN_OFFER_PERCENT } from "../../../_internal/shared/features/offers/config";
 import { ListingBottomActions } from "../../products/components/ListingBottomActions";
@@ -34,7 +35,10 @@ function toDescriptionHtml(raw) {
 export async function ClassifiedDetailPageView({ slug, initialProduct, renderOfferAction }) {
     const product = initialProduct !== undefined
         ? (initialProduct ?? undefined)
-        : await getClassifiedForDetail(slug).catch(() => undefined);
+        // The listing IS this page's subject — a read failure must surface rather
+        // than become the same `undefined` a removed listing produces, which the
+        // "Not Found" branch below already handles.
+        : await getClassifiedForDetail(slug);
     if (!product) {
         return (_jsx(Main, { children: _jsx(Section, { padding: "y-5xl", children: _jsx(Container, { size: "md", children: _jsxs(Stack, { align: "start", gap: "md", className: "text-left", children: [_jsx(Heading, { level: 1, size: "2xl", weight: "semibold", color: "primary", children: "Classified Listing Not Found" }), _jsx(Text, { color: "muted", children: "The classified listing you are looking for may have been removed." }), _jsx(Link, { href: String(ROUTES.PUBLIC.CLASSIFIED), className: "text-[length:var(--appkit-text-sm)] font-medium text-primary-600 hover:underline", children: "Browse Classifieds" })] }) }) }) }));
     }
@@ -58,7 +62,11 @@ export async function ClassifiedDetailPageView({ slug, initialProduct, renderOff
     const [{ relatedItems, relatedByBrand, relatedByTags, relatedByStore }, groups, siteSettings] = await Promise.all([
         computeRelatedItems(product),
         getGroupsWithItemsForProduct(product.id),
-        getSiteSettingsGlobal().catch(() => null),
+        safeRead(() => getSiteSettingsGlobal(), {
+            route: "/classified/[slug]",
+            key: "classified.siteSettings",
+            fallback: null,
+        }),
     ]);
     // Classifieds are the one non-standard type where haggling is the norm, so
     // `canMakeOffer` is true for them — but the seller still opts in per listing

package/dist/features/digital-codes/components/DigitalCodeDetailPageView.js

@@ -30,7 +30,10 @@ function toDescriptionHtml(raw) {
 export async function DigitalCodeDetailPageView({ slug, initialProduct, renderPrimaryActions }) {
     const product = initialProduct !== undefined
         ? (initialProduct ?? undefined)
-        : await getDigitalCodeForDetail(slug).catch(() => undefined);
+        // The listing IS this page's subject — a read failure must surface rather
+        // than become the same `undefined` a removed listing produces, which the
+        // "Not Found" branch below already handles.
+        : await getDigitalCodeForDetail(slug);
     if (!product) {
         return (_jsx(Main, { children: _jsx(Section, { padding: "y-5xl", children: _jsx(Container, { size: "md", children: _jsxs(Stack, { align: "start", gap: "md", className: "text-left", children: [_jsx(Heading, { level: 1, size: "2xl", weight: "semibold", color: "primary", children: "Digital Code Listing Not Found" }), _jsx(Text, { color: "muted", children: "The digital code listing you are looking for may have been removed." }), _jsx(Link, { href: String(ROUTES.PUBLIC.DIGITAL_CODES), className: "text-[length:var(--appkit-text-sm)] font-medium text-primary-600 hover:underline", children: "Browse Digital Codes" })] }) }) }) }));
     }

package/dist/features/events/components/EventBanner.js

@@ -13,10 +13,12 @@ export function EventBanner({ storageKey = "dismissed_event_banners", className
             const stored = sessionStorage.getItem(storageKey);
             if (stored)
                 setDismissedIds(JSON.parse(stored));
+            // sessionStorage is unavailable in private mode and over quota in some
+            // embedded browsers. Losing the dismissal list only means the banner
+            // reappears — nothing an operator can act on, and nothing the user loses.
         }
         catch (_err) {
             void normalizeError(_err);
-            // ignore session storage failures
         }
         setMounted(true);
     }, [storageKey]);
@@ -38,10 +40,11 @@ export function EventBanner({ storageKey = "dismissed_event_banners", className
         setDismissedIds(next);
         try {
             sessionStorage.setItem(storageKey, JSON.stringify(next));
+            // Same as the read above: the banner is already hidden in local state, so
+            // a failed persist only means it returns on the next page load.
         }
         catch (_err) {
             void normalizeError(_err);
-            // ignore
         }
     };
     return (_jsxs(Section, { tone: "accent-banner", className: [

package/dist/features/events/components/EventOfferCard.js

@@ -24,10 +24,12 @@ export function EventOfferCard({ couponCode, offerDescription, title = "Exclusiv
             await navigator.clipboard.writeText(couponCode);
             setCopied(true);
             setTimeout(() => setCopied(false), 2000);
+            // Clipboard access is denied outright on insecure origins and by user
+            // permission. The button simply never flips to "Copied", and the code is
+            // still visible on screen to select by hand.
         }
         catch (_err) {
             void normalizeError(_err);
-            /* clipboard unavailable — leave button as-is */
         }
     };
     return (_jsx(Div, { className: `${__P.p6}`, rounded: "2xl", border: "default", children: _jsxs(Stack, { gap: "md", children: [_jsxs(Stack, { gap: "xs", children: [_jsx(Heading, { level: 2, weight: "semibold", size: "2xl", children: title }), offerDescription ? (_jsx(Text, { size: "sm", color: "muted", children: offerDescription })) : null] }), _jsxs(Row, { surface: "muted", padding: "inline", align: "center", gap: "3", rounded: "lg", children: [_jsx(Code, { className: "flex-1", weight: "bold", size: "base", color: "primary", children: couponCode }), _jsx(Button, { type: "button", variant: "outline", onClick: copyCode, children: copied ? "Copied!" : "Copy code" })] }), _jsx(Div, { children: _jsx(Link, { href: href, children: _jsx(Button, { variant: "primary", children: "Apply at checkout" }) }) })] }) }));

package/dist/features/events/components/EventPollWidget.js

@@ -53,7 +53,7 @@ export function EventPollWidget({ eventId, pollConfig, eventStatus, totalEntries
         }
         catch (err) {
             void normalizeError(err);
-            setError(err instanceof Error ? err.message : "Something went wrong");
+            setError("Something went wrong");
         }
         finally {
             setIsLoading(false);

package/dist/features/events/components/SpinWheelView.js

@@ -4,6 +4,7 @@ import { useCallback, useMemo, useState } from "react";
 import { Button, ClaimCouponButton, Div, Heading, LoginRequiredModal, Row, Span, Stack, Text, useToast } from "../../../ui";
 import { isAuthError } from "../../../utils/auth-error";
 import { normalizeError } from "../../../errors/normalize";
+import { toUserMessage } from "../../../errors/error-display-map";
 const CLS_WHEEL = "relative mx-auto aspect-square w-64 overflow-hidden rounded-full border-4 border-warning bg-[image:var(--appkit-gradient-promotion)]";
 // audit-content-alignment-ok: self-contained circular spin-wheel game widget, all text centered under the wheel by design
 const CLS_PRIZE_BOX = "rounded-xl border border-success bg-success-surface dark:border-success p-[var(--appkit-space-4)] text-center";
@@ -55,14 +56,14 @@ export function SpinWheelView({ eventId, prizes, alreadyUsed, initialPrizeId, in
             }, ANIMATION_MS);
         }
         catch (err) {
-            void normalizeError(err);
+            const e = normalizeError(err);
             setSpinning(false);
             if (isAuthError(err)) {
                 setShowLoginModal(true);
             }
             else {
                 setError(l.errorFallback);
-                showToast(err instanceof Error ? err.message : l.errorFallback, "error");
+                showToast(toUserMessage(e.code, undefined, { fallback: l.errorFallback }), "error");
             }
         }
     }, [disabled, eventId, l.errorFallback, onSpin, showToast]);

package/dist/features/grouped/components/GroupedListingEditorView.js

@@ -46,6 +46,7 @@ import { apiClient } from "../../../http";
 import { ADMIN_ENDPOINTS, SELLER_ENDPOINTS } from "../../../constants/api-endpoints";
 import { ROUTES } from "../../../next/routing/route-map";
 import { normalizeError } from "../../../errors/normalize";
+import { toUserMessage } from "../../../errors/error-display-map";
 import { ProductInlineSelect } from "../../seller/components/ProductInlineSelect";
 import { groupedListingFormSchema, } from "../schemas/grouped-listing-form";
 const EMPTY = {
@@ -173,7 +174,7 @@ export function GroupedListingEditorView({ scope, groupId, fixedStoreId, onSaved
         }
         catch (err) {
             const e = normalizeError(err);
-            showToast(e.message || "Failed to save the group.", "error");
+            showToast(toUserMessage(e.code, undefined, { fallback: "Failed to save the group." }), "error");
         }
         finally {
             setSaving(false);

package/dist/features/history/hooks/useHistory.js

@@ -38,35 +38,28 @@ async function postAuthTrack(args) {
             headers: { "Content-Type": "application/json" },
             body: JSON.stringify(args),
         });
+        // Recently-viewed tracking is a side effect of browsing, fired from a
+        // debounced timer with no UI attached. A lost entry costs the user nothing
+        // and there is no caller in a position to retry or report it.
     }
     catch (_err) {
         void normalizeError(_err);
-        // best-effort tracking
     }
 }
+// Both deletes below are user-initiated and deliberately let failures throw:
+// swallowing them made `remove`/`clear` report success for a DELETE that never
+// landed, and the row would silently return on the next load.
 async function deleteAuthItem(productId) {
-    try {
-        await fetch(ACCOUNT_ENDPOINTS.HISTORY_ITEM(productId), {
-            method: "DELETE",
-            credentials: "include",
-        });
-    }
-    catch (_err) {
-        void normalizeError(_err);
-        // ignore
-    }
+    await fetch(ACCOUNT_ENDPOINTS.HISTORY_ITEM(productId), {
+        method: "DELETE",
+        credentials: "include",
+    });
 }
 async function deleteAuthAll() {
-    try {
-        await fetch(ACCOUNT_ENDPOINTS.HISTORY, {
-            method: "DELETE",
-            credentials: "include",
-        });
-    }
-    catch (_err) {
-        void normalizeError(_err);
-        // ignore
-    }
+    await fetch(ACCOUNT_ENDPOINTS.HISTORY, {
+        method: "DELETE",
+        credentials: "include",
+    });
 }
 export function useHistory(userId) {
     const isAuth = !!userId;
@@ -81,6 +74,9 @@ export function useHistory(userId) {
             else {
                 setItems(getGuestHistory());
             }
+            // fetchAuthHistory already degrades to [] on its own, and getGuestHistory
+            // is a localStorage read that fails only in private mode — either way the
+            // list stays as it was, which is the correct visible outcome here.
         }
         catch (err) {
             void normalizeError(err);
@@ -131,14 +127,20 @@ export function useHistory(userId) {
         trackTimers.current.set(key, timer);
     }, [isAuth]);
     const remove = useCallback(async (productId) => {
-        if (isAuth) {
-            await deleteAuthItem(productId);
-            await loadItems();
+        try {
+            if (isAuth) {
+                await deleteAuthItem(productId);
+                await loadItems();
+            }
+            else {
+                setItems(removeGuestHistoryItem(productId));
+            }
         }
-        else {
-            setItems(removeGuestHistoryItem(productId));
+        catch (_err) {
+            void normalizeError(_err);
+            showToast("Failed to remove this item from history.", "error");
         }
-    }, [isAuth, loadItems]);
+    }, [isAuth, loadItems, showToast]);
     const clear = useCallback(async () => {
         try {
             if (isAuth) {

package/dist/features/homepage/components/AdSlot.js

@@ -16,10 +16,12 @@ function AdsenseAd({ config }) {
             return;
         try {
             (window.adsbygoogle = window.adsbygoogle || []).push({});
+            // adsbygoogle is injected by a third-party script that ad blockers and
+            // consent walls routinely prevent from loading. The <ins> below simply
+            // stays unfilled — there is no failure the site can act on.
         }
         catch (_err) {
             void normalizeError(_err);
-            // AdSense not loaded yet — no-op
         }
     }, []);
     return (_jsx(Div, { ref: ref, children: _jsx("ins", { className: "adsbygoogle block", "data-ad-client": config.adsenseClient ?? "", "data-ad-slot": config.adsenseSlot ?? "", "data-ad-format": config.adsenseFormat ?? "auto", "data-full-width-responsive": "true" }) }));

package/dist/features/homepage/components/GoogleReviewsSection.js

@@ -1,5 +1,6 @@
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { normalizeError } from "../../../errors/normalize";
+import { serverLogger } from "../../../monitoring/server-logger";
 import { Anchor, Div, GRID_MAP, Heading, HandModeRow, Row, Section, Span, Stack, Text } from "../../../ui";
 import { MediaImage } from "../../media/MediaImage";
 import { siteSettingsRepository } from "../../admin/repository/site-settings.repository";
@@ -60,8 +61,13 @@ export async function GoogleReviewsSection(config) {
         totalRatings = result.totalRatings;
     }
     catch (_err) {
-        void normalizeError(_err);
-        // Fail silently — empty state shown
+        // The section renders its empty state below, which is indistinguishable
+        // from a place that genuinely has no reviews — and Google Places calls are
+        // billed, so a persistent failure is worth an operator seeing.
+        serverLogger.warn("homepage.googleReviews: Places fetch failed", {
+            placeId,
+            error: normalizeError(_err).message,
+        });
     }
     const gridClass = layout === "carousel"
         ? "flex gap-[var(--appkit-space-4)] overflow-x-auto pb-[var(--appkit-space-2)] snap-x snap-mandatory"