@mohasinac/appkit 4.32.6 → 4.33.1

package/dist/features/homepage/constants/section-copy.d.ts

@@ -0,0 +1,107 @@
+/**
+ * Category-neutral default copy for homepage sections.
+ *
+ * Every `cleanTitle(cfg?.title) || "…"` fallback in `lib/section-renderer.tsx`
+ * resolves here. Before this file the renderer carried ~25 hardcoded English
+ * strings, several of them brand- or catalogue-specific — "India's #1
+ * Marketplace", "Why Buyers Trust LetItRip", and an announcement-bar fallback
+ * naming a Pokémon TCG promotion. appkit is a library; a consumer's brand and
+ * catalogue must not be compiled into it.
+ *
+ * Three tiers own the copy on this page, and the split is deliberate:
+ *
+ *   Tier A — THIS FILE. Generic defaults that read correctly for any
+ *            marketplace regardless of what it sells.
+ *   Tier B — `MarketplaceHomepageViewProps.brand`, passed by the consumer from
+ *            its own single source of site identity.
+ *   Tier C — optional `*SectionConfig` fields, so an admin can override any of
+ *            these per section instance without a deploy.
+ *
+ * Not `appkit.config.js`: the renderer runs inside appkit's own RSC, and that
+ * file lives on the CONSUMER side of the seam. Importing it from here would
+ * invert the library dependency — the Duplication Framework's "Lane / API
+ * boundary" clause, and what `audit-ssr-in-appkit` Rule 3 protects.
+ *
+ * Keep every string here catalogue-neutral. "Shop by Category" survives the
+ * catalogue changing; "Burst & X Series" does not.
+ */
+/** Section headings. Overridable per instance via `config.title`. */
+export declare const SECTION_TITLE: {
+    readonly welcome: "Discover Amazing Products";
+    readonly categories: "Shop by Category";
+    readonly products: "Featured Products";
+    readonly auctions: "Live Auctions";
+    readonly preOrders: "Reserve Before It Ships";
+    readonly stores: "Featured Stores";
+    readonly events: "Events & Offers";
+    readonly reviews: "What Our Customers Say";
+    readonly banner: "Thousands of collectibles. One marketplace.";
+    readonly trustIndicators: "Why Buyers Trust Us";
+    readonly features: "Security You Can Trust";
+    readonly whatsappCommunity: "Join Our Community";
+    readonly faq: "Frequently Asked Questions";
+    readonly blogArticles: "From Our Blog";
+    readonly newsletter: "Stay Updated";
+    readonly brands: "Top Brands";
+    readonly featuredBundles: "Curated Bundles";
+};
+/**
+ * "View all →" links. Overridable per instance via `config.viewMoreLabel`.
+ *
+ * These were hardcoded at every call site with no config path at all, so an
+ * admin could rename a section's heading and not its own link.
+ */
+export declare const VIEW_MORE_LABEL: {
+    readonly categories: "All categories →";
+    readonly products: "View all products →";
+    readonly auctions: "View all auctions →";
+    readonly preOrders: "View all pre-orders →";
+    readonly stores: "View all stores →";
+    readonly events: "View all events →";
+    readonly reviews: "See all reviews →";
+    readonly blogArticles: "View all posts →";
+    readonly brands: "All brands →";
+    readonly faq: "View all FAQs →";
+};
+/** Supporting copy that has no per-section config field of its own. */
+export declare const SECTION_COPY: {
+    readonly welcomePill: "Buy, sell & collect";
+    readonly welcomePrimaryCta: "Shop Now";
+    readonly welcomeSecondaryCta: "Browse All";
+    readonly featuresPill: "Built for trust";
+    readonly featuresLearnMore: "Learn about our security →";
+    readonly bannerPrimaryCta: "Shop All Products →";
+    readonly bannerSecondaryCta: "Browse Auctions →";
+    readonly whatsappDescription: "Get deal alerts, auction updates, and exclusive drops before anyone else.";
+    /**
+     * Shown only when `siteSettings.announcementBar` has no message. Was a
+     * hardcoded seasonal Pokémon TCG discount, which would have gone out as
+     * live promotional copy on any install that left the bar unconfigured.
+     */
+    readonly announcementFallback: "Free shipping on eligible orders — shop the latest arrivals.";
+};
+/**
+ * Trust chips under the hero headline.
+ *
+ * Kept as a fixed generic default rather than a config field: making it
+ * editable needs a repeater UI in the section builder, which is
+ * disproportionate to four short labels that are true of any marketplace.
+ * Revisit if a consumer needs different ones.
+ */
+export declare const WELCOME_TRUST_CHIPS: readonly [{
+    readonly key: "delivery";
+    readonly emoji: "🚀";
+    readonly label: "Fast Delivery";
+}, {
+    readonly key: "secure";
+    readonly emoji: "🔒";
+    readonly label: "Secure Payments";
+}, {
+    readonly key: "rating";
+    readonly emoji: "⭐";
+    readonly label: "Buyer Rated";
+}, {
+    readonly key: "returns";
+    readonly emoji: "↩️";
+    readonly label: "Easy Returns";
+}];

package/dist/features/homepage/constants/section-copy.js

@@ -0,0 +1,96 @@
+/**
+ * Category-neutral default copy for homepage sections.
+ *
+ * Every `cleanTitle(cfg?.title) || "…"` fallback in `lib/section-renderer.tsx`
+ * resolves here. Before this file the renderer carried ~25 hardcoded English
+ * strings, several of them brand- or catalogue-specific — "India's #1
+ * Marketplace", "Why Buyers Trust LetItRip", and an announcement-bar fallback
+ * naming a Pokémon TCG promotion. appkit is a library; a consumer's brand and
+ * catalogue must not be compiled into it.
+ *
+ * Three tiers own the copy on this page, and the split is deliberate:
+ *
+ *   Tier A — THIS FILE. Generic defaults that read correctly for any
+ *            marketplace regardless of what it sells.
+ *   Tier B — `MarketplaceHomepageViewProps.brand`, passed by the consumer from
+ *            its own single source of site identity.
+ *   Tier C — optional `*SectionConfig` fields, so an admin can override any of
+ *            these per section instance without a deploy.
+ *
+ * Not `appkit.config.js`: the renderer runs inside appkit's own RSC, and that
+ * file lives on the CONSUMER side of the seam. Importing it from here would
+ * invert the library dependency — the Duplication Framework's "Lane / API
+ * boundary" clause, and what `audit-ssr-in-appkit` Rule 3 protects.
+ *
+ * Keep every string here catalogue-neutral. "Shop by Category" survives the
+ * catalogue changing; "Burst & X Series" does not.
+ */
+/** Section headings. Overridable per instance via `config.title`. */
+export const SECTION_TITLE = {
+    welcome: "Discover Amazing Products",
+    categories: "Shop by Category",
+    products: "Featured Products",
+    auctions: "Live Auctions",
+    preOrders: "Reserve Before It Ships",
+    stores: "Featured Stores",
+    events: "Events & Offers",
+    reviews: "What Our Customers Say",
+    banner: "Thousands of collectibles. One marketplace.",
+    trustIndicators: "Why Buyers Trust Us",
+    features: "Security You Can Trust",
+    whatsappCommunity: "Join Our Community",
+    faq: "Frequently Asked Questions",
+    blogArticles: "From Our Blog",
+    newsletter: "Stay Updated",
+    brands: "Top Brands",
+    featuredBundles: "Curated Bundles",
+};
+/**
+ * "View all →" links. Overridable per instance via `config.viewMoreLabel`.
+ *
+ * These were hardcoded at every call site with no config path at all, so an
+ * admin could rename a section's heading and not its own link.
+ */
+export const VIEW_MORE_LABEL = {
+    categories: "All categories →",
+    products: "View all products →",
+    auctions: "View all auctions →",
+    preOrders: "View all pre-orders →",
+    stores: "View all stores →",
+    events: "View all events →",
+    reviews: "See all reviews →",
+    blogArticles: "View all posts →",
+    brands: "All brands →",
+    faq: "View all FAQs →",
+};
+/** Supporting copy that has no per-section config field of its own. */
+export const SECTION_COPY = {
+    welcomePill: "Buy, sell & collect",
+    welcomePrimaryCta: "Shop Now",
+    welcomeSecondaryCta: "Browse All",
+    featuresPill: "Built for trust",
+    featuresLearnMore: "Learn about our security →",
+    bannerPrimaryCta: "Shop All Products →",
+    bannerSecondaryCta: "Browse Auctions →",
+    whatsappDescription: "Get deal alerts, auction updates, and exclusive drops before anyone else.",
+    /**
+     * Shown only when `siteSettings.announcementBar` has no message. Was a
+     * hardcoded seasonal Pokémon TCG discount, which would have gone out as
+     * live promotional copy on any install that left the bar unconfigured.
+     */
+    announcementFallback: "Free shipping on eligible orders — shop the latest arrivals.",
+};
+/**
+ * Trust chips under the hero headline.
+ *
+ * Kept as a fixed generic default rather than a config field: making it
+ * editable needs a repeater UI in the section builder, which is
+ * disproportionate to four short labels that are true of any marketplace.
+ * Revisit if a consumer needs different ones.
+ */
+export const WELCOME_TRUST_CHIPS = [
+    { key: "delivery", emoji: "🚀", label: "Fast Delivery" },
+    { key: "secure", emoji: "🔒", label: "Secure Payments" },
+    { key: "rating", emoji: "⭐", label: "Buyer Rated" },
+    { key: "returns", emoji: "↩️", label: "Easy Returns" },
+];

package/dist/features/homepage/lib/section-renderer.d.ts

@@ -32,5 +32,20 @@ export type FaqItem = {
     answer: string;
     category: string;
 };
-export declare function renderSection(section: HomepageSectionDocument, adSlots: MarketplaceHomepageViewAdSlots | undefined, newsletterFormSlot: React.ReactNode, faqItems: FaqItem[], slides: CarouselSlide[], liveStats: LiveStatsMap, sectionData?: SectionData): React.ReactNode;
+/**
+ * Consumer's site identity.
+ *
+ * appkit is a library and must not compile a consumer's brand into itself —
+ * the renderer used to hardcode `brandLogoText="LIR"` and
+ * `"Why Buyers Trust LetItRip"`. The consumer passes this from whatever single
+ * source it already uses for site identity, so there is no second definition
+ * to drift.
+ */
+export interface HomepageBrand {
+    /** Full display name, e.g. used in "Why Buyers Trust {name}". */
+    name?: string;
+    /** Short mark shown in the hero panel, e.g. "LT". */
+    shortName?: string;
+}
+export declare function renderSection(section: HomepageSectionDocument, adSlots: MarketplaceHomepageViewAdSlots | undefined, newsletterFormSlot: React.ReactNode, faqItems: FaqItem[], slides: CarouselSlide[], liveStats: LiveStatsMap, sectionData?: SectionData, brand?: HomepageBrand): React.ReactNode;
 export { AnnouncementBar, hashBannerMessage };

package/dist/features/homepage/lib/section-renderer.js

@@ -3,6 +3,8 @@ import React from "react";
 import { ROUTES } from "../../../next";
 import { cleanTitle, parseWelcomeDescription } from "./section-helpers";
 import { DEFAULT_TRUST_FEATURES, DEFAULT_SECURITY_ITEMS } from "./section-defaults";
+import { SECTION_TITLE, SECTION_COPY, VIEW_MORE_LABEL, WELCOME_TRUST_CHIPS, } from "../constants/section-copy";
+import { FAQ_CATEGORY_LABELS } from "../constants/faq-category-labels";
 import { AnnouncementBar, hashBannerMessage } from "../components/AnnouncementBar";
 import { HeroCarousel } from "../components/HeroCarousel";
 import { StatsCounterSection } from "../components/StatsCounterSection";
@@ -38,23 +40,23 @@ const AD_SLOT_MAP = {
     reviews: "afterReviews",
     faq: "afterFAQ",
 };
-function renderSectionElement(section, newsletterFormSlot, faqItems, slides, liveStats, sectionData) {
+function renderSectionElement(section, newsletterFormSlot, faqItems, slides, liveStats, sectionData, brand) {
     const { type, config } = section;
     switch (type) {
-        case "carousel":
-            return _jsx(HeroCarousel, { initialSlides: slides });
+        case "carousel": {
+            const cfg = config;
+            return (_jsx(HeroCarousel, { initialSlides: slides, title: cfg?.title, height: cfg?.height, defaultAutoplayDelayMs: cfg?.defaultAutoplayDelayMs, pauseOnHover: cfg?.pauseOnHover, showDots: cfg?.showDots, showArrows: cfg?.showArrows }));
+        }
         case "welcome": {
             const cfg = config;
-            return (_jsx(WelcomeSection, { title: cleanTitle(cfg?.h1) || "Discover Amazing Products", subtitle: parseWelcomeDescription(cfg?.description), pillLabel: "India's #1 Marketplace", showCTA: cfg?.showCTA ?? true, ctaLabel: cfg?.ctaText || "Shop Now", ctaHref: cfg?.ctaLink || ROUTES.PUBLIC.PRODUCTS, secondaryCtaLabel: "Browse All", secondaryCtaHref: ROUTES.PUBLIC.PRODUCTS, trustChips: [
-                    { key: "delivery", emoji: "🚀", label: "Fast Delivery" },
-                    { key: "secure", emoji: "🔒", label: "Secure Payments" },
-                    { key: "rating", emoji: "⭐", label: "4.8+ Rated" },
-                    { key: "returns", emoji: "↩️", label: "Easy Returns" },
-                ], brandLogoText: "LIR" }));
+            return (_jsx(WelcomeSection, { title: cleanTitle(cfg?.h1) || SECTION_TITLE.welcome, 
+                // A real `subtitle` wins; `description` is the legacy source and is
+                // still parsed for documents that only carry it.
+                subtitle: cfg?.subtitle?.trim() || parseWelcomeDescription(cfg?.description), pillLabel: cfg?.pillLabel || SECTION_COPY.welcomePill, showCTA: cfg?.showCTA ?? true, ctaLabel: cfg?.ctaText || SECTION_COPY.welcomePrimaryCta, ctaHref: cfg?.ctaLink || ROUTES.PUBLIC.PRODUCTS, secondaryCtaLabel: cfg?.secondaryCtaText || SECTION_COPY.welcomeSecondaryCta, secondaryCtaHref: cfg?.secondaryCtaLink || ROUTES.PUBLIC.PRODUCTS, trustChips: [...WELCOME_TRUST_CHIPS], brandLogoText: brand?.shortName }));
         }
         case "categories": {
             const cfg = config;
-            return (_jsx(ShopByCategorySection, { title: cleanTitle(cfg?.title) || "Shop by Category", viewMoreHref: ROUTES.PUBLIC.CATEGORIES, viewMoreLabel: "All categories \u2192", initialItems: sectionData.categories, cta: cfg?.cta, filters: cfg?.filters, autoScroll: cfg?.autoScroll, scrollInterval: cfg?.scrollInterval, loop: cfg?.loop }));
+            return (_jsx(ShopByCategorySection, { title: cleanTitle(cfg?.title) || SECTION_TITLE.categories, viewMoreHref: ROUTES.PUBLIC.CATEGORIES, viewMoreLabel: cfg?.viewMoreLabel || VIEW_MORE_LABEL.categories, initialItems: sectionData.categories, limit: cfg?.maxCategories, cta: cfg?.cta, filters: cfg?.filters, autoScroll: cfg?.autoScroll, scrollInterval: cfg?.scrollInterval, loop: cfg?.loop }));
         }
         case "stats": {
             const cfg = config;
@@ -81,47 +83,75 @@ function renderSectionElement(section, newsletterFormSlot, faqItems, slides, liv
                 : [];
             if (stats.length === 0)
                 return null;
-            return _jsx(StatsCounterSection, { stats: stats });
+            return _jsx(StatsCounterSection, { stats: stats, title: cleanTitle(cfg?.title) });
         }
         case "products": {
             const cfg = config;
-            return (_jsx(FeaturedProductsSection, { title: cleanTitle(cfg?.title) || "Featured Products", viewMoreHref: ROUTES.PUBLIC.PRODUCTS, viewMoreLabel: "View all products \u2192", filterByBrand: cfg?.filterByBrand, initialItems: sectionData.products, rows: cfg?.rows, maxItems: cfg?.maxItems, autoScroll: cfg?.autoScroll, scrollInterval: cfg?.scrollInterval, loop: cfg?.loop }));
+            return (_jsx(FeaturedProductsSection, { title: cleanTitle(cfg?.title) || SECTION_TITLE.products, description: cfg?.subtitle, viewMoreHref: ROUTES.PUBLIC.PRODUCTS, viewMoreLabel: cfg?.viewMoreLabel || VIEW_MORE_LABEL.products, filterByBrand: cfg?.filterByBrand, initialItems: sectionData.products, rows: cfg?.rows, 
+                // `maxProducts` is what the admin builder's "Max items" input writes;
+                // `maxItems` is the schema's own name. The two never met, so the
+                // control had no effect. Read both, newest name first.
+                maxItems: cfg?.maxItems ?? cfg?.maxProducts, autoScroll: cfg?.autoScroll, scrollInterval: cfg?.scrollInterval, loop: cfg?.loop }));
         }
         case "auctions": {
             const cfg = config;
-            return (_jsx(FeaturedAuctionsSection, { title: cleanTitle(cfg?.title) || "Live Auctions", viewMoreHref: ROUTES.PUBLIC.AUCTIONS, viewMoreLabel: "View all auctions \u2192", filterByBrand: cfg?.filterByBrand, initialItems: sectionData.auctions, rows: cfg?.rows, autoScroll: cfg?.autoScroll, scrollInterval: cfg?.scrollInterval, loop: cfg?.loop }));
+            return (_jsx(FeaturedAuctionsSection, { title: cleanTitle(cfg?.title) || SECTION_TITLE.auctions, description: cfg?.subtitle, viewMoreHref: ROUTES.PUBLIC.AUCTIONS, viewMoreLabel: cfg?.viewMoreLabel || VIEW_MORE_LABEL.auctions, filterByBrand: cfg?.filterByBrand, initialItems: sectionData.auctions, rows: cfg?.rows, maxItems: cfg?.maxAuctions, autoScroll: cfg?.autoScroll, scrollInterval: cfg?.scrollInterval, loop: cfg?.loop }));
         }
         case "pre-orders": {
             const cfg = config;
-            return (_jsx(FeaturedPreOrdersSection, { title: cleanTitle(cfg?.title) || "Reserve Before It Ships", viewMoreHref: ROUTES.PUBLIC.PRE_ORDERS, viewMoreLabel: "View all pre-orders \u2192", filterByBrand: cfg?.filterByBrand, initialItems: sectionData.preOrders, rows: cfg?.rows, autoScroll: cfg?.autoScroll, scrollInterval: cfg?.scrollInterval, loop: cfg?.loop }));
+            return (_jsx(FeaturedPreOrdersSection, { title: cleanTitle(cfg?.title) || SECTION_TITLE.preOrders, description: cfg?.subtitle, viewMoreHref: ROUTES.PUBLIC.PRE_ORDERS, viewMoreLabel: cfg?.viewMoreLabel || VIEW_MORE_LABEL.preOrders, filterByBrand: cfg?.filterByBrand, initialItems: sectionData.preOrders, rows: cfg?.rows, maxItems: cfg?.maxItems, autoScroll: cfg?.autoScroll, scrollInterval: cfg?.scrollInterval, loop: cfg?.loop }));
         }
         case "stores": {
             const cfg = config;
-            return (_jsx(FeaturedStoresSection, { title: cleanTitle(cfg?.title) || "Featured Stores", viewMoreHref: ROUTES.PUBLIC.STORES, viewMoreLabel: "View all stores \u2192", initialItems: sectionData.stores, autoScroll: cfg?.autoScroll, scrollInterval: cfg?.scrollInterval, loop: cfg?.loop }));
+            return (_jsx(FeaturedStoresSection, { title: cleanTitle(cfg?.title) || SECTION_TITLE.stores, description: cfg?.subtitle, viewMoreHref: ROUTES.PUBLIC.STORES, viewMoreLabel: cfg?.viewMoreLabel || VIEW_MORE_LABEL.stores, initialItems: sectionData.stores, limit: cfg?.maxStores, autoScroll: cfg?.autoScroll, scrollInterval: cfg?.scrollInterval, loop: cfg?.loop }));
         }
         case "events": {
             const cfg = config;
-            return (_jsx(EventsSection, { title: cleanTitle(cfg?.title) || "Events & Offers", viewMoreHref: ROUTES.PUBLIC.EVENTS, viewMoreLabel: "View all events \u2192", initialItems: sectionData.events, autoScroll: cfg?.autoScroll, scrollInterval: cfg?.scrollInterval, loop: cfg?.loop }));
+            return (_jsx(EventsSection, { title: cleanTitle(cfg?.title) || SECTION_TITLE.events, description: cfg?.subtitle, viewMoreHref: ROUTES.PUBLIC.EVENTS, viewMoreLabel: cfg?.viewMoreLabel || VIEW_MORE_LABEL.events, initialItems: sectionData.events, limit: cfg?.maxEvents, autoScroll: cfg?.autoScroll, scrollInterval: cfg?.scrollInterval, loop: cfg?.loop }));
         }
         case "reviews": {
             const cfg = config;
-            return (_jsx(HomepageCustomerReviewsSection, { title: cleanTitle(cfg?.title) || "What Our Customers Say", viewMoreHref: ROUTES.PUBLIC.REVIEWS, viewMoreLabel: "See all reviews \u2192", autoScroll: cfg?.autoScroll, scrollInterval: cfg?.scrollInterval, loop: cfg?.loop }));
+            return (_jsx(HomepageCustomerReviewsSection, { title: cleanTitle(cfg?.title) || SECTION_TITLE.reviews, viewMoreHref: ROUTES.PUBLIC.REVIEWS, viewMoreLabel: cfg?.viewMoreLabel || VIEW_MORE_LABEL.reviews, maxReviews: cfg?.maxReviews, itemsPerView: cfg?.itemsPerView, mobileItemsPerView: cfg?.mobileItemsPerView, autoScroll: cfg?.autoScroll, scrollInterval: cfg?.scrollInterval, loop: cfg?.loop }));
         }
         case "banner": {
             const cfg = config;
-            return (_jsx(CTABannerSection, { title: cfg?.content?.title || "Thousands of collectibles. One marketplace.", primaryLabel: "Shop All Products \u2192", primaryHref: ROUTES.PUBLIC.PRODUCTS, secondaryLabel: "Browse Auctions \u2192", secondaryHref: ROUTES.PUBLIC.AUCTIONS }));
+            // CTAs come from the configured buttons; the component's own neutral
+            // defaults fill in when none are set. Previously both labels and both
+            // hrefs were hardcoded here and `buttons[]` was ignored entirely.
+            const [primaryBtn, secondaryBtn] = cfg?.buttons ?? [];
+            return (_jsx(CTABannerSection, { title: cfg?.content?.title || SECTION_TITLE.banner, subtitle: cfg?.content?.subtitle, description: cfg?.content?.description, height: cfg?.height, backgroundImage: cfg?.backgroundImage, primaryLabel: primaryBtn?.text, primaryHref: primaryBtn?.link, secondaryLabel: secondaryBtn?.text, secondaryHref: secondaryBtn?.link }));
         }
         case "trust-indicators": {
             const cfg = config;
-            return (_jsx(TrustFeaturesSection, { title: cleanTitle(cfg?.title) || "Why Buyers Trust LetItRip", items: DEFAULT_TRUST_FEATURES }));
+            const configured = (cfg?.indicators ?? [])
+                .filter((ind) => ind?.title?.trim())
+                .map((ind) => ({
+                key: ind.id,
+                iconName: ind.icon,
+                title: ind.title,
+                description: ind.description,
+            }));
+            return (_jsx(TrustFeaturesSection, { title: cleanTitle(cfg?.title) ||
+                    (brand?.name ? `Why Buyers Trust ${brand.name}` : SECTION_TITLE.trustIndicators), items: configured.length > 0 ? configured : DEFAULT_TRUST_FEATURES }));
         }
         case "features": {
             const cfg = config;
-            return (_jsx(SecurityHighlightsSection, { title: cleanTitle(cfg?.title) || "Security You Can Trust", pillLabel: "Built for trust", items: DEFAULT_SECURITY_ITEMS, learnMoreHref: ROUTES.PUBLIC.SECURITY, learnMoreLabel: "Learn about our security \u2192" }));
+            // `items` is the real shape; `features: string[]` is the legacy one and
+            // becomes title-only cards. Neither reached the component before — it
+            // always rendered DEFAULT_SECURITY_ITEMS regardless of config.
+            const configured = cfg?.items?.filter((item) => item?.title?.trim()) ??
+                cfg?.features
+                    ?.filter((label) => label?.trim())
+                    .map((label, index) => ({ key: `feature-${index}`, title: label, description: "" })) ??
+                [];
+            return (_jsx(SecurityHighlightsSection, { title: cleanTitle(cfg?.title) || SECTION_TITLE.features, pillLabel: cfg?.pillLabel || SECTION_COPY.featuresPill, items: configured.length > 0 ? configured : DEFAULT_SECURITY_ITEMS, learnMoreHref: cfg?.learnMoreLink || ROUTES.PUBLIC.SECURITY, learnMoreLabel: cfg?.learnMoreLabel || SECTION_COPY.featuresLearnMore }));
         }
         case "whatsapp-community": {
             const cfg = config;
-            return (_jsx(WhatsAppCommunitySection, { title: cleanTitle(cfg?.title) || "Join Our WhatsApp Community", descriptionHtml: cfg?.description || "5,000+ members. Get deal alerts, auction updates, and exclusive drops before anyone else.", memberCount: cfg?.memberCount || 5000, groupLink: cfg?.groupLink || "https://chat.whatsapp.com/", benefits: cfg?.benefits ?? [], testimonial: cfg?.testimonial, buttonText: cfg?.buttonText }));
+            return (_jsx(WhatsAppCommunitySection, { title: cleanTitle(cfg?.title) || SECTION_TITLE.whatsappCommunity, descriptionHtml: cfg?.description || SECTION_COPY.whatsappDescription, 
+                // No `|| 5000` fallback: inventing a member count the site cannot
+                // verify is a fabricated social-proof number.
+                memberCount: cfg?.memberCount, groupLink: cfg?.groupLink || "https://chat.whatsapp.com/", benefits: cfg?.benefits ?? [], testimonial: cfg?.testimonial, buttonText: cfg?.buttonText }));
         }
         case "faq": {
             const cfg = config;
@@ -129,15 +159,6 @@ function renderSectionElement(section, newsletterFormSlot, faqItems, slides, liv
                 return null;
             // Build tabs from configured visibleTabs (or all categories in config)
             const tabCategories = (cfg?.visibleTabs?.length ? cfg.visibleTabs : cfg?.categories) ?? [];
-            const FAQ_CATEGORY_LABELS = {
-                general: "General",
-                orders_payment: "Orders & Payment",
-                shipping_delivery: "Shipping",
-                returns_refunds: "Returns & Refunds",
-                product_information: "Products",
-                account_security: "Account",
-                technical_support: "Support",
-            };
             const tabs = tabCategories.map((cat) => ({
                 value: cat,
                 label: FAQ_CATEGORY_LABELS[cat] ?? cat,
@@ -145,19 +166,19 @@ function renderSectionElement(section, newsletterFormSlot, faqItems, slides, liv
             const slicedItems = cfg?.displayCount ? faqItems.slice(0, cfg.displayCount) : faqItems;
             const totalFaqs = faqItems.length;
             const hasMore = totalFaqs > slicedItems.length;
-            return (_jsx(FAQSection, { title: cleanTitle(cfg?.title) || "Frequently Asked Questions", subtitle: cfg?.subtitle, tabs: tabs, showCategoryTabs: cfg?.showCategoryTabs ?? false, allowMultipleOpen: cfg?.allowMultipleOpen ?? false, defaultOpenCount: cfg?.defaultOpenCount ?? 0, items: slicedItems, viewMoreHref: cfg?.linkToFullPage ? ROUTES.PUBLIC.FAQS : undefined, viewMoreLabel: "View all FAQs \u2192", hasMore: hasMore, moreCount: totalFaqs - slicedItems.length }));
+            return (_jsx(FAQSection, { title: cleanTitle(cfg?.title) || SECTION_TITLE.faq, subtitle: cfg?.subtitle, tabs: tabs, showCategoryTabs: cfg?.showCategoryTabs ?? false, allowMultipleOpen: cfg?.allowMultipleOpen ?? false, defaultOpenCount: cfg?.defaultOpenCount ?? 0, items: slicedItems, viewMoreHref: cfg?.linkToFullPage ? ROUTES.PUBLIC.FAQS : undefined, viewMoreLabel: cfg?.viewMoreLabel || VIEW_MORE_LABEL.faq, hasMore: hasMore, moreCount: totalFaqs - slicedItems.length }));
         }
         case "blog-articles": {
             const cfg = config;
-            return (_jsx(BlogArticlesSection, { title: cleanTitle(cfg?.title) || "From Our Blog", viewMoreHref: ROUTES.BLOG.LIST, viewMoreLabel: "View all posts \u2192", initialItems: sectionData.blog }));
+            return (_jsx(BlogArticlesSection, { title: cleanTitle(cfg?.title) || SECTION_TITLE.blogArticles, viewMoreHref: ROUTES.BLOG.LIST, viewMoreLabel: cfg?.viewMoreLabel || VIEW_MORE_LABEL.blogArticles, initialItems: sectionData.blog, limit: cfg?.maxArticles, showReadTime: cfg?.showReadTime, showAuthor: cfg?.showAuthor, showThumbnails: cfg?.showThumbnails }));
         }
         case "newsletter": {
             const cfg = config;
-            return (_jsx(NewsletterSection, { title: cleanTitle(cfg?.title) || "Stay Updated", subtitle: cfg?.description || "Get the latest drops, auctions, and deals in your inbox.", renderForm: () => newsletterFormSlot ?? null }));
+            return (_jsx(NewsletterSection, { title: cleanTitle(cfg?.title) || SECTION_TITLE.newsletter, subtitle: cfg?.description, privacyLabel: cfg?.privacyText, privacyHref: cfg?.privacyLink, renderForm: () => newsletterFormSlot ?? null }));
         }
         case "brands": {
             const cfg = config;
-            return (_jsx(BrandsSection, { title: cleanTitle(cfg?.title) || "Top Brands", subtitle: cfg?.subtitle, limit: cfg?.maxBrands || 12, viewMoreHref: ROUTES.PUBLIC.CATEGORIES, viewMoreLabel: "All brands \u2192", initialItems: sectionData.brands, cta: cfg?.cta, filters: cfg?.filters, autoScroll: cfg?.autoScroll, scrollInterval: cfg?.scrollInterval, loop: cfg?.loop }));
+            return (_jsx(BrandsSection, { title: cleanTitle(cfg?.title) || SECTION_TITLE.brands, subtitle: cfg?.subtitle, limit: cfg?.maxBrands || 12, viewMoreHref: ROUTES.PUBLIC.CATEGORIES, viewMoreLabel: cfg?.viewMoreLabel || VIEW_MORE_LABEL.brands, initialItems: sectionData.brands, cta: cfg?.cta, filters: cfg?.filters, autoScroll: cfg?.autoScroll, scrollInterval: cfg?.scrollInterval, loop: cfg?.loop }));
         }
         case "social-feed": {
             const cfg = config;
@@ -178,7 +199,7 @@ function renderSectionElement(section, newsletterFormSlot, faqItems, slides, liv
         }
         case "featured-bundles": {
             const cfg = config;
-            return (_jsx(FeaturedBundlesSection, { title: cleanTitle(cfg?.title) || "Curated Bundles", initialItems: sectionData.bundles }));
+            return (_jsx(FeaturedBundlesSection, { title: cleanTitle(cfg?.title) || SECTION_TITLE.featuredBundles, description: cfg?.subtitle, initialItems: sectionData.bundles, maxItems: cfg?.maxItems, showSavingsBadge: cfg?.showSavingsBadge, viewMoreLabel: cfg?.viewMoreLabel }));
         }
         case "prize-draws": {
             const cfg = config;
@@ -196,8 +217,8 @@ function renderSectionElement(section, newsletterFormSlot, faqItems, slides, liv
             return null;
     }
 }
-export function renderSection(section, adSlots, newsletterFormSlot, faqItems, slides, liveStats, sectionData = {}) {
-    const sectionElement = renderSectionElement(section, newsletterFormSlot, faqItems, slides, liveStats, sectionData);
+export function renderSection(section, adSlots, newsletterFormSlot, faqItems, slides, liveStats, sectionData = {}, brand) {
+    const sectionElement = renderSectionElement(section, newsletterFormSlot, faqItems, slides, liveStats, sectionData, brand);
     if (!sectionElement)
         return null;
     const adSlotKey = AD_SLOT_MAP[section.type];

package/dist/features/homepage/repository/homepage-sections.repository.d.ts

@@ -11,6 +11,29 @@ export declare class HomepageSectionsRepository extends BaseRepository<HomepageS
     static readonly SIEVE_FIELDS: FirebaseSieveFields;
     list(model: SieveModel): Promise<FirebaseSieveResult<HomepageSectionDocument>>;
     create(input: HomepageSectionCreateInput): Promise<HomepageSectionDocument>;
+    /**
+     * Merge `config` instead of replacing it.
+     *
+     * `BaseRepository.update` writes `config` as a top-level key, and a Firestore
+     * `update()` REPLACES a map field wholesale rather than merging it. So every
+     * config key a caller omitted was silently deleted — HTTP 200, no error. The
+     * admin section builder emits only the fields it has inputs for, which meant
+     * opening any section and clicking Save destroyed `products.rows`/`maxItems`/
+     * `filterByBrand`, `banner.backgroundImage`, `categories.cta`/`filters` and
+     * every other unbuilt field. Root Cause #76.
+     *
+     * Both write paths — `PATCH /api/admin/sections/[id]` and the
+     * `updateHomepageSection` server action — funnel through here, so the merge
+     * belongs in the write primitive rather than at either call site.
+     *
+     * Costs one extra read, and only when `config` is actually being written:
+     * a reorder or an enable/disable still costs exactly one write.
+     *
+     * Semantics: an omitted key is PRESERVED. Clearing one requires sending it
+     * explicitly (`null` / `""` / `[]`), not omitting it. `resetSectionToDefault`
+     * below deliberately keeps replace semantics — that is its whole purpose.
+     */
+    update(id: string, data: Partial<HomepageSectionDocument>): Promise<HomepageSectionDocument>;
     getEnabledSections(): Promise<HomepageSectionDocument[]>;
     getDisabledSections(): Promise<HomepageSectionDocument[]>;
     getSectionByType(type: SectionType): Promise<HomepageSectionDocument | null>;

package/dist/features/homepage/repository/homepage-sections.repository.js

@@ -28,6 +28,46 @@ export class HomepageSectionsRepository extends BaseRepository {
             .set(prepareForFirestore(sectionData));
         return { id, ...sectionData };
     }
+    /**
+     * Merge `config` instead of replacing it.
+     *
+     * `BaseRepository.update` writes `config` as a top-level key, and a Firestore
+     * `update()` REPLACES a map field wholesale rather than merging it. So every
+     * config key a caller omitted was silently deleted — HTTP 200, no error. The
+     * admin section builder emits only the fields it has inputs for, which meant
+     * opening any section and clicking Save destroyed `products.rows`/`maxItems`/
+     * `filterByBrand`, `banner.backgroundImage`, `categories.cta`/`filters` and
+     * every other unbuilt field. Root Cause #76.
+     *
+     * Both write paths — `PATCH /api/admin/sections/[id]` and the
+     * `updateHomepageSection` server action — funnel through here, so the merge
+     * belongs in the write primitive rather than at either call site.
+     *
+     * Costs one extra read, and only when `config` is actually being written:
+     * a reorder or an enable/disable still costs exactly one write.
+     *
+     * Semantics: an omitted key is PRESERVED. Clearing one requires sending it
+     * explicitly (`null` / `""` / `[]`), not omitting it. `resetSectionToDefault`
+     * below deliberately keeps replace semantics — that is its whole purpose.
+     */
+    async update(id, data) {
+        if (!data.config) {
+            return super.update(id, data);
+        }
+        // A missing doc falls through to super.update, which fails the same way it
+        // did before this override existed — don't invent a new error path here.
+        const existing = await this.findById(id);
+        if (!existing) {
+            return super.update(id, data);
+        }
+        return super.update(id, {
+            ...data,
+            config: {
+                ...(existing.config ?? {}),
+                ...data.config,
+            },
+        });
+    }
     async getEnabledSections() {
         try {
             const snapshot = await this.db