@mohasinac/appkit 4.11.2 → 4.11.3

package/dist/features/products/components/ProductDetailPageView.js

@@ -1,6 +1,6 @@
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import Link from "next/link";
-import { productRepository, reviewRepository } from "../../../repositories";
+import { productRepository } from "../../../repositories";
 const __P = {
     p3: "p-[var(--appkit-space-3)]",
     p5: "p-[var(--appkit-space-5)]",
@@ -25,7 +25,7 @@ import { getDefaultCurrency } from "../../../core/baseline-resolver";
 import { Button, Container, Div, Heading, Main, Nav, PriceDisplay, RichText, Row, Section, Span, Stack, Text, Ul, Ol, Li, Dl, Dt, Dd, } from "../../../ui";
 import { normalizeRichTextHtml } from "../../../utils/string.formatter";
 import { safeDisplayName } from "../../../security";
-import { ReviewsList } from "../../reviews/components/ReviewsList";
+import { ReviewsListingPanel } from "../../reviews/components/ReviewsListingPanel";
 import { ProductDetailView } from "./ProductDetailView";
 import { ProductGalleryClient } from "./ProductGalleryClient";
 import { ProductTabsShell } from "./ProductTabsShell";
@@ -37,7 +37,7 @@ import { SublistingCarouselSection } from "./SublistingCarouselSection";
 import { ShowGroupSection } from "./ShowGroupSection";
 import { HistoryTracker } from "../../history/components/HistoryTracker";
 import { RelatedItemsSection } from "./RelatedItemsSection";
-import { computeRelatedItems, toReview } from "../../../_internal/server/features/products/data";
+import { computeRelatedItems, getReviewPageForProduct } from "../../../_internal/server/features/products/data";
 import { GroupedListingsCarousel } from "../../grouped/components/GroupedListingsCarousel";
 import { getGroupsWithItemsForProduct } from "../../../_internal/server/features/grouped/data";
 // ---------------------------------------------------------------------------
@@ -165,12 +165,12 @@ export async function ProductDetailPageView({ slug, initialProduct, renderOfferA
     const groupTitle = typeof p.groupTitle === "string" ? p.groupTitle : undefined;
     const storeId = typeof p.storeId === "string" ? p.storeId : null;
     // -- Fetch reviews + the shared 4-signal related-items computation + grouped listings in parallel --
-    const [reviewDocs, related, groups] = await Promise.all([
-        reviewRepository.findApprovedByProduct(product.id).catch(() => []),
+    // Only page 1 of reviews is fetched here; <ReviewsListingPanel> pages the rest client-side.
+    const [initialReviews, related, groups] = await Promise.all([
+        getReviewPageForProduct(product.id),
         computeRelatedItems(product),
         getGroupsWithItemsForProduct(product.id),
     ]);
-    const reviews = reviewDocs.map(toReview);
     const { relatedItems, relatedByBrand, relatedByTags, relatedByStore } = related;
     return (_jsxs(Main, { children: [_jsx(HistoryTracker, { productId: product.id, productType: "product", snapshot: {
                     title: product.title,
@@ -231,7 +231,7 @@ export async function ProductDetailPageView({ slug, initialProduct, renderOfferA
                             ? () => (_jsx(SublistingCarouselSection, { sublistingCategoryId: sublistingCategoryId, currentListingId: String(product.id) }))
                             : undefined, renderGroupSection: groupId
                             ? () => (_jsx(ShowGroupSection, { groupId: groupId, currentSlug: String(p.slug ?? product.id), isParent: isGroupParent, groupTitle: groupTitle }))
-                            : undefined, renderTabs: () => (_jsx(ProductTabsShell, { descriptionContent: descriptionHtml ? (_jsx(RichText, { html: descriptionHtml, proseClass: "prose prose-sm sm:prose max-w-none dark:prose-invert", className: "text-[var(--appkit-color-text-muted)]" })) : undefined, specsContent: allSpecs.length > 0 ? (_jsx(Dl, { divide: "subtle", rounded: "xl", border: "subtle", className: "overflow-hidden", children: allSpecs.map((s, i) => (_jsxs(Row, { gap: "md", oddEven: "zebra", surface: "default", padding: "inline", children: [_jsx(Dt, { className: "w-36 flex-shrink-0", color: "primary", weight: "medium", children: s.name }), _jsxs(Dd, { className: "flex-1", color: "muted", children: [s.value, s.unit ? ` ${s.unit}` : ""] })] }, i))) })) : undefined, ingredientsContent: ingredients.length > 0 ? (_jsx(Ul, { spacing: "comfortable", children: ingredients.map((item, i) => (_jsxs(Li, { layout: "flex-start", gap: "2", children: [_jsx(Span, { className: "mt-1 flex-shrink-0 h-1.5 w-1.5 bg-primary-400", rounded: "full" }), item] }, i))) })) : undefined, howToUseContent: howToUse.length > 0 ? (_jsx(Ol, { spacing: "loose", children: howToUse.map((step, i) => (_jsxs(Li, { layout: "flex-start", gap: "3", textSize: "sm", color: "muted", children: [_jsx(Span, { layout: "flex-center", size: "xs", weight: "bold", className: "flex-shrink-0 h-6 w-6 bg-primary-100 dark:bg-primary-900/30 text-primary-700 dark:text-primary-300", rounded: "full", children: i + 1 }), step] }, i))) })) : undefined, reviewsContent: _jsx(ReviewsList, { reviews: reviews, context: "listing", emptyLabel: "No reviews yet \u2014 be the first to review this product." }), customTabs: customSections.map((s) => ({
+                            : undefined, renderTabs: () => (_jsx(ProductTabsShell, { descriptionContent: descriptionHtml ? (_jsx(RichText, { html: descriptionHtml, proseClass: "prose prose-sm sm:prose max-w-none dark:prose-invert", className: "text-[var(--appkit-color-text-muted)]" })) : undefined, specsContent: allSpecs.length > 0 ? (_jsx(Dl, { divide: "subtle", rounded: "xl", border: "subtle", className: "overflow-hidden", children: allSpecs.map((s, i) => (_jsxs(Row, { gap: "md", oddEven: "zebra", surface: "default", padding: "inline", children: [_jsx(Dt, { className: "w-36 flex-shrink-0", color: "primary", weight: "medium", children: s.name }), _jsxs(Dd, { className: "flex-1", color: "muted", children: [s.value, s.unit ? ` ${s.unit}` : ""] })] }, i))) })) : undefined, ingredientsContent: ingredients.length > 0 ? (_jsx(Ul, { spacing: "comfortable", children: ingredients.map((item, i) => (_jsxs(Li, { layout: "flex-start", gap: "2", children: [_jsx(Span, { className: "mt-1 flex-shrink-0 h-1.5 w-1.5 bg-primary-400", rounded: "full" }), item] }, i))) })) : undefined, howToUseContent: howToUse.length > 0 ? (_jsx(Ol, { spacing: "loose", children: howToUse.map((step, i) => (_jsxs(Li, { layout: "flex-start", gap: "3", textSize: "sm", color: "muted", children: [_jsx(Span, { layout: "flex-center", size: "xs", weight: "bold", className: "flex-shrink-0 h-6 w-6 bg-primary-100 dark:bg-primary-900/30 text-primary-700 dark:text-primary-300", rounded: "full", children: i + 1 }), step] }, i))) })) : undefined, reviewsContent: _jsx(ReviewsListingPanel, { source: { kind: "product", productId: product.id }, stateMode: "local", context: "listing", initialData: initialReviews, emptyLabel: "No reviews yet \u2014 be the first to review this product." }), customTabs: customSections.map((s) => ({
                                 id: s.id,
                                 label: s.title,
                                 content: _jsx(CustomSectionTabContent, { section: s }),

package/dist/features/products/constants/listing-tabs.d.ts

@@ -6,16 +6,20 @@
  * filter on the `products` collection or to a separate collection
  * (currently only `bundles`).
  *
- * Add new tab variants here — never inline the tab list inside the view.
+ * DERIVED, NOT HAND-WRITTEN (2026-08-21). Every array below is built from
+ * `ALL_LISTING_TYPES` + the listing-type plugin registry, so a new listing
+ * type appears on every tab bar automatically. They used to be nine separate
+ * hand-maintained literals and had drifted badly: `/products` offered only 4
+ * of 9 types (missing auctions, pre-orders, prize draws, art, stickers — the
+ * bug that started this sweep), the admin type chips used display LABELS as
+ * Sieve filter ids, and several arrays still listed `bundle`, which stopped
+ * being a listingType in SB-UNI-D. `scripts/audit-listing-type-tab-coverage.mjs`
+ * blocks any regression.
+ *
+ * To add a tab that ISN'T a listing type (bundles, stores), append it to the
+ * relevant array explicitly — those are genuinely different collections.
  */
 import type { ListingType } from "../types";
-/**
- * The consolidated generic `/products` browse tab covers these 4 listing
- * types (auctions/pre-orders/bundles/prize-draws/art+stickers each have
- * their own dedicated public page). Shared between the server-rendered
- * page view and the client listing component so they can't drift apart.
- */
-export declare const GENERIC_PRODUCT_LISTING_TYPES: readonly ["standard", "classified", "digital-code", "live"];
 export interface ListingTab {
     id: string;
     label: string;
@@ -28,197 +32,38 @@ export interface ListingTab {
     /** Set when the tab queries a non-product entity (e.g. `stores`). */
     entity?: "stores";
 }
+/**
+ * The generic `/products` browse page spans EVERY listing type (2026-08-21).
+ *
+ * It used to span only `standard|classified|digital-code|live` on the theory
+ * that auctions/pre-orders/prize-draws/art each had a dedicated page — but
+ * that made five of nine types unreachable from the main catalogue, broke
+ * seven legacy `/search/.../tab/<type>` permanent redirects that all point at
+ * `/products`, and made the cross-type Compare + bulk-cart features (which
+ * only exist on this page) unusable for those types. The dedicated pages
+ * remain, and `/products` links to them when a single time-boxed type is
+ * selected.
+ */
+export declare const GENERIC_PRODUCT_LISTING_TYPES: readonly ListingType[];
+/** The listing types the combined `/art` page spans. */
+export declare const ART_STICKERS_LISTING_TYPES: readonly ListingType[];
 /** Tabs shown on `/categories/[slug]` and `/brands/[slug]` detail pages. */
-export declare const CATEGORY_PAGE_TABS: readonly [{
-    readonly id: "products";
-    readonly label: "Products";
-    readonly listingType: "standard";
-}, {
-    readonly id: "auctions";
-    readonly label: "Auctions";
-    readonly listingType: "auction";
-}, {
-    readonly id: "pre-orders";
-    readonly label: "Pre-Orders";
-    readonly listingType: "pre-order";
-}, {
-    readonly id: "prize-draws";
-    readonly label: "Prize Draws";
-    readonly listingType: "prize-draw";
-}, {
-    readonly id: "bundles";
-    readonly label: "Bundles";
-    readonly collection: "bundles";
-}, {
-    readonly id: "classifieds";
-    readonly label: "Classifieds";
-    readonly listingType: "classified";
-}, {
-    readonly id: "digital-codes";
-    readonly label: "Digital Codes";
-    readonly listingType: "digital-code";
-}, {
-    readonly id: "live";
-    readonly label: "Live Items";
-    readonly listingType: "live";
-}, {
-    readonly id: "art";
-    readonly label: "Art & Stickers";
-    readonly listingType: "art|stickers";
-}, {
-    readonly id: "stores";
-    readonly label: "Stores";
-    readonly entity: "stores";
-}];
-export type CategoryTabId = (typeof CATEGORY_PAGE_TABS)[number]["id"];
+export declare const CATEGORY_PAGE_TABS: readonly ListingTab[];
+export type CategoryTabId = string;
 /** Tabs shown on the public `/stores/[slug]` nav bar. */
-export declare const STORE_PAGE_TABS: readonly [{
-    readonly id: "products";
-    readonly label: "Products";
-    readonly listingType: "standard";
-}, {
-    readonly id: "auctions";
-    readonly label: "Auctions";
-    readonly listingType: "auction";
-}, {
-    readonly id: "pre-orders";
-    readonly label: "Pre-Orders";
-    readonly listingType: "pre-order";
-}, {
-    readonly id: "prize-draws";
-    readonly label: "Prize Draws";
-    readonly listingType: "prize-draw";
-}, {
-    readonly id: "bundles";
-    readonly label: "Bundles";
-    readonly collection: "bundles";
-}, {
-    readonly id: "classifieds";
-    readonly label: "Classifieds";
-    readonly listingType: "classified";
-}, {
-    readonly id: "digital-codes";
-    readonly label: "Digital Codes";
-    readonly listingType: "digital-code";
-}, {
-    readonly id: "live";
-    readonly label: "Live Items";
-    readonly listingType: "live";
-}, {
-    readonly id: "art";
-    readonly label: "Art & Stickers";
-    readonly listingType: "art|stickers";
-}];
-export type StoreTabId = (typeof STORE_PAGE_TABS)[number]["id"];
+export declare const STORE_PAGE_TABS: readonly ListingTab[];
+export type StoreTabId = string;
 /** Tabs shown on the seller-dashboard listings view + admin products list. */
-export declare const SELLER_LISTING_TABS: readonly [{
-    readonly id: "all";
-    readonly label: "All";
-}, {
-    readonly id: "products";
-    readonly label: "Products";
-    readonly listingType: "standard";
-}, {
-    readonly id: "auctions";
-    readonly label: "Auctions";
-    readonly listingType: "auction";
-}, {
-    readonly id: "pre-orders";
-    readonly label: "Pre-Orders";
-    readonly listingType: "pre-order";
-}, {
-    readonly id: "prize-draws";
-    readonly label: "Prize Draws";
-    readonly listingType: "prize-draw";
-}, {
-    readonly id: "classifieds";
-    readonly label: "Classifieds";
-    readonly listingType: "classified";
-}, {
-    readonly id: "digital-codes";
-    readonly label: "Digital Codes";
-    readonly listingType: "digital-code";
-}, {
-    readonly id: "live";
-    readonly label: "Live Items";
-    readonly listingType: "live";
-}, {
-    readonly id: "art";
-    readonly label: "Art";
-    readonly listingType: "art";
-}, {
-    readonly id: "stickers";
-    readonly label: "Stickers";
-    readonly listingType: "stickers";
-}];
-export type SellerListingTabId = (typeof SELLER_LISTING_TABS)[number]["id"];
-/** Tabs shown on `/search` results. */
-export declare const SEARCH_RESULT_TABS: readonly [{
-    readonly id: "all";
-    readonly label: "All";
-}, {
-    readonly id: "products";
-    readonly label: "Products";
-    readonly listingType: "standard";
-}, {
-    readonly id: "auctions";
-    readonly label: "Auctions";
-    readonly listingType: "auction";
-}, {
-    readonly id: "pre-orders";
-    readonly label: "Pre-Orders";
-    readonly listingType: "pre-order";
-}, {
-    readonly id: "prize-draws";
-    readonly label: "Prize Draws";
-    readonly listingType: "prize-draw";
-}, {
-    readonly id: "bundles";
-    readonly label: "Bundles";
-    readonly collection: "bundles";
-}, {
-    readonly id: "classifieds";
-    readonly label: "Classifieds";
-    readonly listingType: "classified";
-}, {
-    readonly id: "digital-codes";
-    readonly label: "Digital Codes";
-    readonly listingType: "digital-code";
-}, {
-    readonly id: "live";
-    readonly label: "Live Items";
-    readonly listingType: "live";
-}, {
-    readonly id: "art";
-    readonly label: "Art & Stickers";
-    readonly listingType: "art|stickers";
-}];
-export type SearchTabId = (typeof SEARCH_RESULT_TABS)[number]["id"];
-/** In-page type-filter chips on the generic `/products` browse page. */
-export declare const PRODUCT_TYPE_FILTER_TABS: readonly [{
-    readonly id: "All";
-    readonly label: "All";
-}, {
-    readonly id: "standard";
-    readonly label: "Standard";
-}, {
-    readonly id: "classified";
-    readonly label: "Classified";
-}, {
-    readonly id: "digital-code";
-    readonly label: "Digital Codes";
-}, {
-    readonly id: "live";
-    readonly label: "Live Items";
-}];
+export declare const SELLER_LISTING_TABS: readonly ListingTab[];
+export type SellerListingTabId = string;
+/**
+ * In-page type-filter chips on the generic `/products` browse page.
+ *
+ * Multi-select (checkbox semantics) — no "All" sentinel chip, because "none
+ * checked" already means "every type". A sentinel would be a second way to
+ * express the same state and would have to be cleared whenever a real chip
+ * was ticked.
+ */
+export declare const PRODUCT_TYPE_FILTER_TABS: readonly ListingTab[];
 /** In-page type-filter chips on the combined `/art` (Art & Stickers) page. */
-export declare const ART_STICKERS_TYPE_FILTER_TABS: readonly [{
-    readonly id: "All";
-    readonly label: "All";
-}, {
-    readonly id: "art";
-    readonly label: "Art";
-}, {
-    readonly id: "stickers";
-    readonly label: "Stickers";
-}];
+export declare const ART_STICKERS_TYPE_FILTER_TABS: readonly ListingTab[];

package/dist/features/products/constants/listing-tabs.js

@@ -6,78 +6,91 @@
  * filter on the `products` collection or to a separate collection
  * (currently only `bundles`).
  *
- * Add new tab variants here — never inline the tab list inside the view.
+ * DERIVED, NOT HAND-WRITTEN (2026-08-21). Every array below is built from
+ * `ALL_LISTING_TYPES` + the listing-type plugin registry, so a new listing
+ * type appears on every tab bar automatically. They used to be nine separate
+ * hand-maintained literals and had drifted badly: `/products` offered only 4
+ * of 9 types (missing auctions, pre-orders, prize draws, art, stickers — the
+ * bug that started this sweep), the admin type chips used display LABELS as
+ * Sieve filter ids, and several arrays still listed `bundle`, which stopped
+ * being a listingType in SB-UNI-D. `scripts/audit-listing-type-tab-coverage.mjs`
+ * blocks any regression.
+ *
+ * To add a tab that ISN'T a listing type (bundles, stores), append it to the
+ * relevant array explicitly — those are genuinely different collections.
+ */
+import { ALL_LISTING_TYPES } from "../../../_internal/shared/listing-types/feature-flags";
+import { pluginFor } from "../../../_internal/shared/listing-types/_registry";
+/**
+ * The generic `/products` browse page spans EVERY listing type (2026-08-21).
+ *
+ * It used to span only `standard|classified|digital-code|live` on the theory
+ * that auctions/pre-orders/prize-draws/art each had a dedicated page — but
+ * that made five of nine types unreachable from the main catalogue, broke
+ * seven legacy `/search/.../tab/<type>` permanent redirects that all point at
+ * `/products`, and made the cross-type Compare + bulk-cart features (which
+ * only exist on this page) unusable for those types. The dedicated pages
+ * remain, and `/products` links to them when a single time-boxed type is
+ * selected.
+ */
+export const GENERIC_PRODUCT_LISTING_TYPES = ALL_LISTING_TYPES;
+/** The listing types the combined `/art` page spans. */
+export const ART_STICKERS_LISTING_TYPES = ["art", "stickers"];
+/**
+ * A FILTER chip — `id` is the canonical `listingType`, because it is passed
+ * straight into `sieveFilter("listingType", EQ, id)`. Never a display label.
+ */
+function chipTab(type) {
+    return { id: type, label: pluginFor(type).chipLabel, listingType: type };
+}
+/**
+ * A ROUTE-BACKED tab — `id` is the plugin's `tabSlug`, because it doubles as
+ * the URL segment (`/stores/{slug}/pre-orders`) and is live in bookmarks.
+ * The canonical type still rides along in `listingType` for the query.
  */
+function pluralTab(type) {
+    const plugin = pluginFor(type);
+    return { id: plugin.tabSlug, label: plugin.pluralLabel, listingType: type };
+}
 /**
- * The consolidated generic `/products` browse tab covers these 4 listing
- * types (auctions/pre-orders/bundles/prize-draws/art+stickers each have
- * their own dedicated public page). Shared between the server-rendered
- * page view and the client listing component so they can't drift apart.
+ * Art and stickers share the combined `/art` browse page, so category/brand/
+ * store tab bars show them as ONE tab backed by a two-value OR-group. Derived
+ * from both plugins rather than a hardcoded `"art|stickers"` string so the
+ * pipe-group can't drift from the registry.
  */
-export const GENERIC_PRODUCT_LISTING_TYPES = ["standard", "classified", "digital-code", "live"];
+const ART_STICKERS_COMBINED_TAB = {
+    id: pluginFor("art").tabSlug,
+    label: `${pluginFor("art").pluralLabel} & ${pluginFor("stickers").pluralLabel}`,
+    listingType: ART_STICKERS_LISTING_TYPES.join("|"),
+};
+/** Listing types that get their own tab on a combined surface (art+stickers merge into one). */
+const SEPARATELY_TABBED_TYPES = ALL_LISTING_TYPES.filter((t) => t !== "art" && t !== "stickers");
 /** Tabs shown on `/categories/[slug]` and `/brands/[slug]` detail pages. */
 export const CATEGORY_PAGE_TABS = [
-    { id: "products", label: "Products", listingType: "standard" },
-    { id: "auctions", label: "Auctions", listingType: "auction" },
-    { id: "pre-orders", label: "Pre-Orders", listingType: "pre-order" },
-    { id: "prize-draws", label: "Prize Draws", listingType: "prize-draw" },
+    ...SEPARATELY_TABBED_TYPES.map(pluralTab),
     { id: "bundles", label: "Bundles", collection: "bundles" },
-    { id: "classifieds", label: "Classifieds", listingType: "classified" },
-    { id: "digital-codes", label: "Digital Codes", listingType: "digital-code" },
-    { id: "live", label: "Live Items", listingType: "live" },
-    // Combined — one public browse page (`/art`) spans both listing types.
-    { id: "art", label: "Art & Stickers", listingType: "art|stickers" },
+    ART_STICKERS_COMBINED_TAB,
     { id: "stores", label: "Stores", entity: "stores" },
 ];
 /** Tabs shown on the public `/stores/[slug]` nav bar. */
 export const STORE_PAGE_TABS = [
-    { id: "products", label: "Products", listingType: "standard" },
-    { id: "auctions", label: "Auctions", listingType: "auction" },
-    { id: "pre-orders", label: "Pre-Orders", listingType: "pre-order" },
-    { id: "prize-draws", label: "Prize Draws", listingType: "prize-draw" },
+    ...SEPARATELY_TABBED_TYPES.map(pluralTab),
     { id: "bundles", label: "Bundles", collection: "bundles" },
-    { id: "classifieds", label: "Classifieds", listingType: "classified" },
-    { id: "digital-codes", label: "Digital Codes", listingType: "digital-code" },
-    { id: "live", label: "Live Items", listingType: "live" },
-    { id: "art", label: "Art & Stickers", listingType: "art|stickers" },
+    ART_STICKERS_COMBINED_TAB,
 ];
 /** Tabs shown on the seller-dashboard listings view + admin products list. */
 export const SELLER_LISTING_TABS = [
     { id: "all", label: "All" },
-    { id: "products", label: "Products", listingType: "standard" },
-    { id: "auctions", label: "Auctions", listingType: "auction" },
-    { id: "pre-orders", label: "Pre-Orders", listingType: "pre-order" },
-    { id: "prize-draws", label: "Prize Draws", listingType: "prize-draw" },
-    { id: "classifieds", label: "Classifieds", listingType: "classified" },
-    { id: "digital-codes", label: "Digital Codes", listingType: "digital-code" },
-    { id: "live", label: "Live Items", listingType: "live" },
-    { id: "art", label: "Art", listingType: "art" },
-    { id: "stickers", label: "Stickers", listingType: "stickers" },
-];
-/** Tabs shown on `/search` results. */
-export const SEARCH_RESULT_TABS = [
-    { id: "all", label: "All" },
-    { id: "products", label: "Products", listingType: "standard" },
-    { id: "auctions", label: "Auctions", listingType: "auction" },
-    { id: "pre-orders", label: "Pre-Orders", listingType: "pre-order" },
-    { id: "prize-draws", label: "Prize Draws", listingType: "prize-draw" },
-    { id: "bundles", label: "Bundles", collection: "bundles" },
-    { id: "classifieds", label: "Classifieds", listingType: "classified" },
-    { id: "digital-codes", label: "Digital Codes", listingType: "digital-code" },
-    { id: "live", label: "Live Items", listingType: "live" },
-    { id: "art", label: "Art & Stickers", listingType: "art|stickers" },
-];
-/** In-page type-filter chips on the generic `/products` browse page. */
-export const PRODUCT_TYPE_FILTER_TABS = [
-    { id: "All", label: "All" },
-    { id: "standard", label: "Standard" },
-    { id: "classified", label: "Classified" },
-    { id: "digital-code", label: "Digital Codes" },
-    { id: "live", label: "Live Items" },
+    ...ALL_LISTING_TYPES.map(pluralTab),
 ];
+/**
+ * In-page type-filter chips on the generic `/products` browse page.
+ *
+ * Multi-select (checkbox semantics) — no "All" sentinel chip, because "none
+ * checked" already means "every type". A sentinel would be a second way to
+ * express the same state and would have to be cleared whenever a real chip
+ * was ticked.
+ */
+export const PRODUCT_TYPE_FILTER_TABS = ALL_LISTING_TYPES.map(chipTab);
 /** In-page type-filter chips on the combined `/art` (Art & Stickers) page. */
-export const ART_STICKERS_TYPE_FILTER_TABS = [
-    { id: "All", label: "All" },
-    { id: "art", label: "Art" },
-    { id: "stickers", label: "Stickers" },
-];
+export const ART_STICKERS_TYPE_FILTER_TABS = ART_STICKERS_LISTING_TYPES.map(chipTab);

package/dist/features/products/constants/sieve.d.ts

@@ -141,6 +141,90 @@ export declare const PREORDER_SORT_OPTIONS: readonly [{
     readonly value: string;
     readonly label: "Lowest Deposit First";
 }];
+/**
+ * Public pre-order browse subset. Drops "Lowest Deposit First" — deposit
+ * amount is a seller-side pricing lever buyers don't shop by — but keeps
+ * both delivery-date directions, which are the primary browse intent.
+ */
+export declare const PREORDER_PUBLIC_SORT_OPTIONS: readonly [{
+    readonly value: string;
+    readonly label: "Earliest Delivery";
+}, {
+    readonly value: string;
+    readonly label: "Latest Delivery";
+}, {
+    readonly value: string;
+    readonly label: "Newest First";
+} | {
+    readonly value: string;
+    readonly label: "Oldest First";
+} | {
+    readonly value: string;
+    readonly label: "Price: Low to High";
+} | {
+    readonly value: string;
+    readonly label: "Price: High to Low";
+} | {
+    readonly value: string;
+    readonly label: "Fewest Slots Left";
+} | {
+    readonly value: string;
+    readonly label: "Lowest Deposit First";
+}, {
+    readonly value: string;
+    readonly label: "Newest First";
+} | {
+    readonly value: string;
+    readonly label: "Oldest First";
+} | {
+    readonly value: string;
+    readonly label: "Price: Low to High";
+} | {
+    readonly value: string;
+    readonly label: "Price: High to Low";
+} | {
+    readonly value: string;
+    readonly label: "Fewest Slots Left";
+} | {
+    readonly value: string;
+    readonly label: "Lowest Deposit First";
+}, {
+    readonly value: string;
+    readonly label: "Newest First";
+} | {
+    readonly value: string;
+    readonly label: "Oldest First";
+} | {
+    readonly value: string;
+    readonly label: "Price: Low to High";
+} | {
+    readonly value: string;
+    readonly label: "Price: High to Low";
+} | {
+    readonly value: string;
+    readonly label: "Fewest Slots Left";
+} | {
+    readonly value: string;
+    readonly label: "Lowest Deposit First";
+}, {
+    readonly value: string;
+    readonly label: "Newest First";
+} | {
+    readonly value: string;
+    readonly label: "Oldest First";
+} | {
+    readonly value: string;
+    readonly label: "Price: Low to High";
+} | {
+    readonly value: string;
+    readonly label: "Price: High to Low";
+} | {
+    readonly value: string;
+    readonly label: "Fewest Slots Left";
+} | {
+    readonly value: string;
+    readonly label: "Lowest Deposit First";
+}];
 export declare const BUNDLE_SORT_OPTIONS: readonly [...({
     readonly value: string;
     readonly label: "Newest First";
@@ -173,4 +257,23 @@ export declare const PRIZE_DRAW_SORT_OPTIONS: readonly [{
     readonly value: string;
     readonly label: "Entry: High to Low";
 }];
-export declare const SORT_OPTIONS_BY_LISTING_TYPE: Record<string, readonly SortOption[]>;
+/** Public prize-draw browse subset — every option is buyer-meaningful. */
+export declare const PRIZE_DRAW_PUBLIC_SORT_OPTIONS: readonly [{
+    readonly value: string;
+    readonly label: "Newest First";
+}, {
+    readonly value: string;
+    readonly label: "Oldest First";
+}, {
+    readonly value: string;
+    readonly label: "Reveal: Soonest";
+}, {
+    readonly value: string;
+    readonly label: "Reveal: Furthest";
+}, {
+    readonly value: string;
+    readonly label: "Entry: Low to High";
+}, {
+    readonly value: string;
+    readonly label: "Entry: High to Low";
+}];

package/dist/features/products/constants/sieve.js

@@ -73,6 +73,19 @@ export const PREORDER_SORT_OPTIONS = [
     { value: sortBy(PRODUCT_FIELDS.PRE_ORDER_CURRENT_COUNT), label: "Fewest Slots Left" },
     { value: sortBy(PRODUCT_FIELDS.PRE_ORDER_DEPOSIT_AMOUNT, "ASC"), label: "Lowest Deposit First" },
 ];
+/**
+ * Public pre-order browse subset. Drops "Lowest Deposit First" — deposit
+ * amount is a seller-side pricing lever buyers don't shop by — but keeps
+ * both delivery-date directions, which are the primary browse intent.
+ */
+export const PREORDER_PUBLIC_SORT_OPTIONS = [
+    PREORDER_SORT_OPTIONS[0],
+    PREORDER_SORT_OPTIONS[1],
+    PREORDER_SORT_OPTIONS[2],
+    PREORDER_SORT_OPTIONS[3],
+    PREORDER_SORT_OPTIONS[4],
+    PREORDER_SORT_OPTIONS[5],
+];
 // ---------------------------------------------------------------------------
 // Bundles
 // ---------------------------------------------------------------------------
@@ -94,13 +107,20 @@ export const PRIZE_DRAW_SORT_OPTIONS = [
     { value: sortBy(PRODUCT_FIELDS.PRICE, "ASC"), label: "Entry: Low to High" },
     { value: sortBy(PRODUCT_FIELDS.PRICE), label: "Entry: High to Low" },
 ];
+/** Public prize-draw browse subset — every option is buyer-meaningful. */
+export const PRIZE_DRAW_PUBLIC_SORT_OPTIONS = PRIZE_DRAW_SORT_OPTIONS;
 // ---------------------------------------------------------------------------
 // Lookup by listing type
 // ---------------------------------------------------------------------------
-export const SORT_OPTIONS_BY_LISTING_TYPE = {
-    standard: STANDARD_SORT_OPTIONS,
-    auction: AUCTION_SORT_OPTIONS,
-    "pre-order": PREORDER_SORT_OPTIONS,
-    bundle: BUNDLE_SORT_OPTIONS,
-    "prize-draw": PRIZE_DRAW_SORT_OPTIONS,
-};
+//
+// The per-type lookup used to live here as a hand-written map keyed on plain
+// strings. It carried a `bundle` key long after SB-UNI-D stopped bundles from
+// being a listingType, and had no entry at all for classified / digital-code /
+// live / art / stickers — five of nine types. It is now DERIVED from the
+// listing-type plugin registry (`sortOptionsFor()` in
+// `_internal/shared/listing-types/_registry.ts`), so a type can never be
+// missing from it again.
+//
+// `BUNDLE_SORT_OPTIONS` above stays a standalone export on purpose — bundles
+// are a `categoryType` on the `categories` collection, not a listingType, so
+// they must not be reachable through a ListingType-keyed lookup.