@mohasinac/appkit 4.53.1 → 4.53.2

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

@@ -15,6 +15,7 @@ import { PRODUCT_FIELDS } from "../../../constants/field-names.js";
 import { annotate } from "../../shell/field-ui-meta.js";
 import { SectionForm, useSectionFormNav, buildSectionsFromSchema, visibleValues, } from "../../shell/index.js";
 import { applyZodIssues } from "../../../ui/forms/apply-zod-issues.js";
+import { productImageUrls } from "../schemas/firestore.js";
 const groupTitleSchema = z.object({
     groupTitle: annotate(z.string().min(1, "Group title is required"), {
         section: "group", sectionLabel: "Group", sectionRequired: true, quick: true,
@@ -351,7 +352,7 @@ export function GroupSettingsPanel({ productId, productSlug, groupId, isGroupPar
                     setOpen(next);
                     if (next && isGroupParent && children === null)
                         loadChildren();
-                }, paddingX: "none", paddingY: "xs", justify: "between", className: "w-full text-left group", "aria-expanded": open, children: [_jsx(Heading, { level: 3, size: "sm", weight: "semibold", color: "muted", children: "Group Settings" }), _jsx(Span, { size: "xs", className: "group-hover:text-[var(--appkit-color-text-muted)] dark:group-hover:text-[var(--appkit-color-text-faint)] transition-colors", color: "faint", children: open ? "▲" : "▼" })] }), open && (_jsxs(Stack, { gap: "sm", className: `mt-3 ${__P.p4}`, surface: "muted", rounded: "lg", border: "default", children: [!groupId && !groupParentSlug && (_jsxs(Stack, { gap: "xs", children: [_jsx(Text, { size: "xs", color: "muted", children: "Group related listings together \u2014 e.g. a set, bundle, or multi-part item. Parts can be sold individually but shown together." }), _jsx(Button, { type: "button", variant: "secondary", size: "sm", onClick: startGroup, isLoading: loading, children: "Start a group" })] })), isGroupParent && groupId && (_jsxs(Stack, { gap: "md", children: [_jsxs(FormShellContext.Provider, { value: titleShellCtx, children: [_jsx(FormErrorSummary, {}), _jsx(SectionForm, { sections: titleSections, values: titleDraft, onChange: (partial) => setTitleDraft((d) => ({ ...d, ...partial })), onSubmit: () => void saveTitle(), onValidationChange: () => titleForm.validate(titleDraft), schema: groupTitleSchema, openIds: titleNav.openIds, onOpenChange: titleNav.setOpenIds, submitLabel: "Save title", isLoading: loading })] }), _jsxs(Div, { children: [_jsx(Text, { className: "mb-2", color: "muted", size: "xs", weight: "semibold", children: "Selection rules" }), _jsxs(Grid, { cols: 2, gap: "sm", children: [_jsx(FieldInput, { name: "groupMinMembers", label: "Min items in a selection", type: "number", value: ruleDraft.min, onChange: (v) => setRuleDraft((d) => ({ ...d, min: v })), hint: "Blank = no minimum" }), _jsx(FieldInput, { name: "groupMaxMembers", label: "Max items in a selection", type: "number", value: ruleDraft.max, onChange: (v) => setRuleDraft((d) => ({ ...d, max: v })), hint: "Blank = no maximum" }), _jsx(FieldInput, { name: "groupMaxUnitsPerMember", label: "Max units of any one item", type: "number", value: ruleDraft.perMember, onChange: (v) => setRuleDraft((d) => ({ ...d, perMember: v })), hint: "Blank = no per-item cap" }), _jsx(FieldInput, { name: "groupDiscountPercent", label: "Group discount (%)", type: "number", value: ruleDraft.discount, onChange: (v) => setRuleDraft((d) => ({ ...d, discount: v })), hint: "Off the selection subtotal. Applies to new cart lines; a line already in a cart keeps the price it was quoted and is re-checked at checkout." })] }), _jsx(Button, { type: "button", variant: "secondary", size: "sm", className: "mt-2", onClick: () => void saveRules(), isLoading: loading, children: "Save rules" })] }), _jsxs(Div, { children: [_jsxs(Text, { className: "mb-2", color: "muted", size: "xs", weight: "semibold", children: ["Members (", childSlugsCount + 1, " \u2014 including this listing)"] }), _jsx(Text, { className: "mb-2", color: "faint", size: "xs", children: "Each listing has its own stock. Editing here saves them together \u2014 it does not pool them." }), loading && !children ? (_jsx(Text, { size: "xs", color: "faint", children: "Loading\u2026" })) : (_jsxs(Div, { className: "divide-y divide-zinc-100 divide-[var(--appkit-color-border-subtle)]", children: [_jsxs(Row, { align: "center", gap: "sm", padding: "y-xs", children: [_jsx(Span, { padding: "pill-2xs", weight: "semibold", className: "bg-[var(--appkit-color-primary)]/10 text-[var(--appkit-color-primary)] text-[10px]", rounded: "default", children: "Parent" }), _jsx(Text, { className: "flex-1", color: "primary", size: "sm", children: productSlug })] }), (children ?? []).filter((c) => c.id !== productId).map((child) => (_jsxs(Row, { align: "center", gap: "sm", padding: "y-xs", children: [child.images?.[0] ? (_jsx(Div, { className: "relative w-8 h-8 flex-shrink-0 border border-[var(--appkit-color-border)]", rounded: "full", overflow: "hidden", children: _jsx(MediaImage, { src: child.images[0], alt: child.title, size: "thumbnail" }) })) : (_jsx(Div, { className: "w-8 h-8", surface: "subtle", rounded: "full" })), _jsx(Text, { className: "flex-1 truncate", color: "primary", size: "sm", children: child.title }), child.status && child.status !== "published" ? (_jsx(Span, { size: "xs", surface: "warning-surface", className: "text-warning", padding: "pill-2xs", rounded: "default", children: child.status })) : null, (motDraft[child.id] ?? child.madeToOrder) === true ? (_jsx(Text, { size: "xs", color: "muted", children: "Made to order" })) : (_jsx(Input, { type: "number", "aria-label": `Stock for ${child.title}`, className: "w-20", value: stockDraft[child.id] ??
+                }, paddingX: "none", paddingY: "xs", justify: "between", className: "w-full text-left group", "aria-expanded": open, children: [_jsx(Heading, { level: 3, size: "sm", weight: "semibold", color: "muted", children: "Group Settings" }), _jsx(Span, { size: "xs", className: "group-hover:text-[var(--appkit-color-text-muted)] dark:group-hover:text-[var(--appkit-color-text-faint)] transition-colors", color: "faint", children: open ? "▲" : "▼" })] }), open && (_jsxs(Stack, { gap: "sm", className: `mt-3 ${__P.p4}`, surface: "muted", rounded: "lg", border: "default", children: [!groupId && !groupParentSlug && (_jsxs(Stack, { gap: "xs", children: [_jsx(Text, { size: "xs", color: "muted", children: "Group related listings together \u2014 e.g. a set, bundle, or multi-part item. Parts can be sold individually but shown together." }), _jsx(Button, { type: "button", variant: "secondary", size: "sm", onClick: startGroup, isLoading: loading, children: "Start a group" })] })), isGroupParent && groupId && (_jsxs(Stack, { gap: "md", children: [_jsxs(FormShellContext.Provider, { value: titleShellCtx, children: [_jsx(FormErrorSummary, {}), _jsx(SectionForm, { sections: titleSections, values: titleDraft, onChange: (partial) => setTitleDraft((d) => ({ ...d, ...partial })), onSubmit: () => void saveTitle(), onValidationChange: () => titleForm.validate(titleDraft), schema: groupTitleSchema, openIds: titleNav.openIds, onOpenChange: titleNav.setOpenIds, submitLabel: "Save title", isLoading: loading })] }), _jsxs(Div, { children: [_jsx(Text, { className: "mb-2", color: "muted", size: "xs", weight: "semibold", children: "Selection rules" }), _jsxs(Grid, { cols: 2, gap: "sm", children: [_jsx(FieldInput, { name: "groupMinMembers", label: "Min items in a selection", type: "number", value: ruleDraft.min, onChange: (v) => setRuleDraft((d) => ({ ...d, min: v })), hint: "Blank = no minimum" }), _jsx(FieldInput, { name: "groupMaxMembers", label: "Max items in a selection", type: "number", value: ruleDraft.max, onChange: (v) => setRuleDraft((d) => ({ ...d, max: v })), hint: "Blank = no maximum" }), _jsx(FieldInput, { name: "groupMaxUnitsPerMember", label: "Max units of any one item", type: "number", value: ruleDraft.perMember, onChange: (v) => setRuleDraft((d) => ({ ...d, perMember: v })), hint: "Blank = no per-item cap" }), _jsx(FieldInput, { name: "groupDiscountPercent", label: "Group discount (%)", type: "number", value: ruleDraft.discount, onChange: (v) => setRuleDraft((d) => ({ ...d, discount: v })), hint: "Off the selection subtotal. Applies to new cart lines; a line already in a cart keeps the price it was quoted and is re-checked at checkout." })] }), _jsx(Button, { type: "button", variant: "secondary", size: "sm", className: "mt-2", onClick: () => void saveRules(), isLoading: loading, children: "Save rules" })] }), _jsxs(Div, { children: [_jsxs(Text, { className: "mb-2", color: "muted", size: "xs", weight: "semibold", children: ["Members (", childSlugsCount + 1, " \u2014 including this listing)"] }), _jsx(Text, { className: "mb-2", color: "faint", size: "xs", children: "Each listing has its own stock. Editing here saves them together \u2014 it does not pool them." }), loading && !children ? (_jsx(Text, { size: "xs", color: "faint", children: "Loading\u2026" })) : (_jsxs(Div, { className: "divide-y divide-zinc-100 divide-[var(--appkit-color-border-subtle)]", children: [_jsxs(Row, { align: "center", gap: "sm", padding: "y-xs", children: [_jsx(Span, { padding: "pill-2xs", weight: "semibold", className: "bg-[var(--appkit-color-primary)]/10 text-[var(--appkit-color-primary)] text-[10px]", rounded: "default", children: "Parent" }), _jsx(Text, { className: "flex-1", color: "primary", size: "sm", children: productSlug })] }), (children ?? []).filter((c) => c.id !== productId).map((child) => (_jsxs(Row, { align: "center", gap: "sm", padding: "y-xs", children: [productImageUrls(child.images)[0] ? (_jsx(Div, { className: "relative w-8 h-8 flex-shrink-0 border border-[var(--appkit-color-border)]", rounded: "full", overflow: "hidden", children: _jsx(MediaImage, { src: productImageUrls(child.images)[0], alt: child.title, size: "thumbnail" }) })) : (_jsx(Div, { className: "w-8 h-8", surface: "subtle", rounded: "full" })), _jsx(Text, { className: "flex-1 truncate", color: "primary", size: "sm", children: child.title }), child.status && child.status !== "published" ? (_jsx(Span, { size: "xs", surface: "warning-surface", className: "text-warning", padding: "pill-2xs", rounded: "default", children: child.status })) : null, (motDraft[child.id] ?? child.madeToOrder) === true ? (_jsx(Text, { size: "xs", color: "muted", children: "Made to order" })) : (_jsx(Input, { type: "number", "aria-label": `Stock for ${child.title}`, className: "w-20", value: stockDraft[child.id] ??
                                                             (child.availableQuantity ?? 0).toString(), onChange: (e) => setStockDraft((d) => ({ ...d, [child.id]: e.target.value })) })), _jsx(Checkbox, { label: "MTO", "aria-label": `Made to order for ${child.title}`, checked: motDraft[child.id] ?? child.madeToOrder === true, onChange: (e) => setMotDraft((d) => ({ ...d, [child.id]: e.target.checked })) }), _jsx(Button, { type: "button", variant: "ghost", size: "sm", onClick: () => requestUnlinkChild(child.id), isLoading: loading, className: `${CLS_DELETE_LINK} text-[length:var(--appkit-text-xs)]`, children: "Unlink" })] }, child.id)))] })), (children ?? []).length > 0 && (_jsx(Button, { type: "button", variant: "secondary", size: "sm", className: "mt-2", onClick: () => void saveGroupStock(), isLoading: loading, children: "Save group stock" }))] }), _jsxs(Row, { gap: "sm", wrap: true, children: [_jsx(Button, { type: "button", variant: "secondary", size: "sm", onClick: () => setShowAddModal(true), children: "Add child listing" }), _jsx(Button, { type: "button", variant: "ghost", size: "sm", onClick: requestDissolveGroup, isLoading: loading, className: CLS_DELETE_LINK, children: "Dissolve group" })] })] })), !isGroupParent && groupParentSlug && (_jsxs(Stack, { gap: "xs", children: [_jsxs(Text, { size: "sm", color: "muted", children: ["Part of:", " ", _jsx(Span, { weight: "medium", color: "primary", children: groupTitle ?? groupParentSlug })] }), _jsxs(Text, { size: "xs", color: "muted", children: ["Parent listing: ", _jsx(Code, { className: "font-mono", children: groupParentSlug })] }), _jsx(Button, { type: "button", variant: "ghost", size: "sm", onClick: requestLeaveGroup, isLoading: loading, className: `${CLS_DELETE_LINK} w-fit`, children: "Leave group" })] }))] })), (() => {
                 const content = (_jsx(AddChildContent, { addTab: addTab, setAddTab: setAddTab, createForm: createForm, setCreateForm: setCreateForm, productSlug: productSlug, storeProductsEndpoint: storeProductsEndpoint, linkTargets: linkTargets, setLinkTargets: setLinkTargets, onAddCreate: addCreateChild, onAddLink: addLinkChild, loading: loading }));
                 const close = () => setShowAddModal(false);

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

@@ -24,6 +24,7 @@ const CLS_BUNDLE_ROW = "flex items-center justify-between gap-[var(--appkit-spac
 const CLS_BUNDLE_LABEL = "text-[length:var(--appkit-text-xs)] text-success dark:text-success font-medium uppercase tracking-wide mb-0.5";
 const CLS_BUNDLE_CTA = "flex-shrink-0 rounded-lg bg-success-solid hover:brightness-110 px-[var(--appkit-space-3)] py-[var(--appkit-space-1-5)] text-[length:var(--appkit-text-xs)] font-semibold text-success-on-solid transition-colors";
 import { ROUTES } from "../../../next/routing/route-map.js";
+import { productImageUrls } from "../schemas/firestore.js";
 import { PART_SLOT_LABEL } from "../../parts/schemas/firestore.js";
 import { getModelsUsingPart } from "../../../_internal/server/features/parts/data.js";
 import { ctaAnchorProps } from "../../layout/cta-anchor.js";
@@ -135,8 +136,12 @@ export async function ProductDetailPageView({ slug, initialProduct, renderOfferA
     const title = String(p.title ?? p.name ?? "");
     const price = typeof p.price === "number" ? p.price : null;
     const originalPrice = typeof p.originalPrice === "number" ? p.originalPrice : null;
-    const images = Array.isArray(p.images)
-        ? p.images
+    // `images` is `ProductImage[]` since A6 — `productImageUrls` is the one
+    // normaliser (it also accepts the legacy `string[]`). Casting it to
+    // `string[]` handed the gallery objects and rendered a placeholder.
+    const galleryUrls = productImageUrls(p.images);
+    const images = galleryUrls.length > 0
+        ? galleryUrls
         : typeof p.mainImage === "string"
             ? [p.mainImage]
             : [];

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

@@ -9,6 +9,7 @@ import { formatCurrency } from "../../../utils/number.formatter.js";
 import { normalizeListingType } from "../utils/listing-type.js";
 import { pluginFor } from "../../../_internal/shared/listing-types/_registry.js";
 import { SUBLISTING_CATEGORY_ENDPOINTS } from "../../../constants/api-endpoints.js";
+import { productImageUrls } from "../schemas/firestore.js";
 const __O = {
     hidden: "overflow-hidden",
     xAuto: "overflow-x-auto",
@@ -18,7 +19,7 @@ function getHref(listing) {
     return pluginFor(normalizeListingType(listing)).detailRoute(slug);
 }
 function ListingThumb({ listing, isCurrent, }) {
-    const image = listing.images?.[0] ?? listing.mainImage ?? "";
+    const image = productImageUrls(listing.images)[0] || listing.mainImage || "";
     const href = getHref(listing);
     const price = formatCurrency(listing.price, listing.currency ?? "INR");
     return (_jsxs(Link, { href: href, "aria-label": listing.title, className: "flex flex-col items-center gap-[var(--appkit-space-1-5)] flex-shrink-0 w-16 group", children: [_jsx(Div, { rounded: "full", className: `relative w-14 h-14 overflow-hidden border-2 transition-all ${isCurrent

package/dist/features/products/repository/products.repository.d.ts

@@ -694,6 +694,7 @@ export declare class ProductRepository extends BaseRepository<ProductDocument> {
         status?: string;
         categoriesIn?: string[];
         search?: string;
+        brandName?: string;
     }): Promise<FirebaseSieveResult<ProductDocument>>;
     /**
      * How many products of each listing type exist within a scope — the numbers

package/dist/features/products/repository/products.repository.js

@@ -1451,7 +1451,11 @@ export class ProductRepository extends BaseRepository {
         }
         return baseQuery;
     }
-    async list(model, opts) {
+    async list(model, 
+    // `brandName` was already honoured by `buildScopedQuery` and simply not
+    // declared here, so a brand-scoped caller had to express the brand as a
+    // Sieve clause instead — a second spelling of one filter.
+    opts) {
         /*
          * 🛑 THE EMPTY GUARD — a non-blank query that yields NO usable token must
          * return nothing, not everything.

package/dist/features/products/utils/group-member-image.d.ts

@@ -1,3 +1,4 @@
+import type { ProductImage } from "../schemas/firestore";
 /**
  * The thumbnail for one product-group member, from whichever shape the caller
  * supplied.
@@ -33,7 +34,13 @@
 export interface GroupMemberImageSource {
     /** The public projection's single thumbnail. */
     image?: string;
-    /** A raw document's gallery. Only read when `image` is absent. */
-    images?: string[];
+    /**
+     * A raw document's gallery. Only read when `image` is absent.
+     *
+     * 🛑 `ProductImage[]` since A6 — a raw document holds objects, a legacy one
+     * holds strings, and the public projection holds neither. `productImageUrls`
+     * is the one normaliser that spans all three.
+     */
+    images?: readonly (string | ProductImage)[];
 }
 export declare function resolveGroupMemberImage(m: GroupMemberImageSource): string;

package/dist/features/products/utils/group-member-image.js

@@ -1,3 +1,4 @@
+import { productImageUrls } from "../schemas/firestore.js";
 export function resolveGroupMemberImage(m) {
-    return m.image ?? m.images?.[0] ?? "";
+    return m.image || productImageUrls(m.images)[0] || "";
 }

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

@@ -34,6 +34,7 @@ import { DataListingView } from "../../admin/components/DataListingView.js";
 import { AssemblyDrawer, } from "../../parts/components/AssemblyDrawer.js";
 import { ACTIONS } from "../../../_internal/shared/actions/action-registry.js";
 import { buildBulkAction } from "../../../_internal/shared/actions/bulk-helpers.js";
+import { productImageUrls } from "../../products/schemas/firestore.js";
 const FILTER_KEYS = [
     "status",
     "category",
@@ -579,8 +580,8 @@ export function SellerProductsView({ onDeleteProduct, onCreateClick, }) {
                     secondary: [priceLabel, stockLabel, secondaryBase, statusLabel].filter(Boolean).join(" · "),
                     status: statusLabel,
                     updatedAt: toRelativeDate(item.updatedAt ?? item.createdAt),
-                    imageUrl: toStringValue(item.mainImage ?? item.images?.[0], undefined),
-                    image: toStringValue(item.mainImage ?? item.images?.[0], undefined),
+                    imageUrl: toStringValue(item.mainImage, undefined) || productImageUrls(item.images)[0],
+                    image: toStringValue(item.mainImage, undefined) || productImageUrls(item.images)[0],
                     listingKind: kind,
                     price: priceRaw ? `₹${priceRaw.toLocaleString("en-IN")}` : "—",
                     stock: stockLabel || "—",

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

@@ -16,6 +16,7 @@ import { AUCTION_PUBLIC_SORT_OPTIONS } from "../../products/constants/sieve.js";
 import { AvailabilityTabs } from "../../products/components/AvailabilityTabs.js";
 import { AVAILABILITY_VALUES } from "../../../constants/field-names.js";
 import { TABLE_KEYS } from "../../../constants/table-keys.js";
+import { productImageUrls } from "../../products/schemas/firestore.js";
 const STORE_AUCTION_TYPES = ["auction"];
 const __P = {
     p3: "p-[var(--appkit-space-3)]",
@@ -75,7 +76,7 @@ export function StoreAuctionsListing({ storeId, initialData }) {
         description: p.description,
         price: p.price ?? 0,
         currency: p.currency || getDefaultCurrency(),
-        mainImage: p.mainImage ?? p.images?.[0],
+        mainImage: p.mainImage || productImageUrls(p.images)[0],
         images: p.images,
         listingType: "auction",
         auctionEndDate: p.auctionEndDate,

package/dist/features/tester/seed-data/authored/content-discovery__authenticity-checker.js

@@ -1,6 +1,6 @@
 /*
  * WHY: Authored six-part procedures for content-discovery/authenticity-checker.
- * WHAT: 16 cases, keyed by full checklist id.
+ * WHAT: 22 cases, keyed by full checklist id.
  *
  * 🛑 EVERY CASE IS WRITTEN AS BEFORE → AFTER, because this feature's whole
  * failure mode is a confident, plausible, wrong answer returned with HTTP 200. A
@@ -44,7 +44,7 @@ export const authored = {
         startPage: "/tools/authenticity",
         steps: [
             "Open /tools/authenticity signed out.",
-            "Confirm the LOOKUP FORM itself is there and usable — Model name, Code on the box, Brand, Generation, How it was sold, and a 'Check it' button.",
+            "Confirm the LOOKUP FORM itself is there and usable — 'Which bey is it?', 'Code on the box', a collapsed 'More details' holding the name-as-printed, brand, generation and how-it-was-sold fields, and a 'Check it' button.",
             "Confirm there is no sign-in card and no 'Sign in to run a lookup' text anywhere on the page.",
             "Enter A0813 as the code and submit, still signed out.",
             "Confirm a real assessment comes back rather than an error or a prompt to sign in.",
@@ -475,4 +475,116 @@ export const authoredTypeahead = {
         expectedUiState: "The result renders immediately. The network log shows no fetch to the search artifact or any /api/search* endpoint.",
         endResult: "Read-only; nothing persists.",
     },
+    /* ── The model picker and the per-model code list (2026-10-20) ──────────
+       * 🛑 The reported bug these exist for: typing "l drago" returned
+       * "Inconclusive — Nothing contradicts it, and nothing confirms it either",
+       * while the tool held 33 recorded box codes for Lightning L-Drago, 35 for
+       * Meteo and 27 for Destructor and had no way to reach any of them. Every
+       * table was keyed by CODE, so naming a bey could never surface them.
+       *
+       * So each case compares the answer against data the tool DEMONSTRABLY HAS.
+       * "Check the picker works" passes against the bug; "picking Lightning
+       * L-Drago must list its 33 codes" does not.
+       */
+    "checklist-content-discovery-authenticity-checker-model-picker-finds-l-drago": {
+        roles: ["guest"],
+        startPage: "/tools/authenticity",
+        steps: [
+            "Open /tools/authenticity.",
+            "Confirm the FIRST field is 'Which bey is it?' and is a searchable select, not a plain text box.",
+            "Open it and type: l drago",
+            "Confirm all three L-Drago releases are offered — Lightning L-Drago 100HF, Meteo L-Drago LW105LF and L-Drago Destructor F:S.",
+            "Confirm each offered row states how many box codes are on file for it.",
+            "Clear the search and type: pegasis",
+            "Confirm Storm Pegasus still appears, even though the list spells it Pegasus.",
+            "Answer no if 'l drago' offers nothing, or if the field is a free-text input with no list.",
+        ],
+        inputs: { search: "l drago" },
+        expectedBehaviour: "BEFORE: the first field was a free-text 'Model name'. Typing 'l drago' matched four releases, so the engine refused to pick one and returned an inconclusive verdict — correct, and useless, because the three L-Dragos it declined to choose between have 95 recorded box codes between them and the screen never said so. AFTER: the picker resolves exactly one bey, which is what unlocks the per-model code list. It searches aliases as well as labels, because Takara Tomy romanises the line Pegasis and Hasbro prints Pegasus — a label-only filter would find neither from the other.",
+        expectedUiState: "A searchable select labelled 'Which bey is it?' listing three L-Drago rows, each annotated with a box-code count. Beys with codes on file are offered before beys without.",
+        endResult: "Nothing is written. The picker only reads a bundled list.",
+    },
+    "checklist-content-discovery-authenticity-checker-picked-model-lists-its-box-codes": {
+        roles: ["guest"],
+        startPage: "/tools/authenticity",
+        steps: [
+            "Open /tools/authenticity.",
+            "In 'Which bey is it?', search for and pick: Lightning L-Drago 100HF",
+            "Leave 'Code on the box' EMPTY.",
+            "Press 'Check it'.",
+            "Read the verdict line, then look for a panel headed 'Box codes recorded for Lightning L-Drago 100HF'.",
+            "Count the codes it lists.",
+            "Answer no if the verdict is 'Inconclusive — Nothing contradicts it, and nothing confirms it either', or if no code list appears.",
+        ],
+        inputs: { modelSlug: "lightning-l-drago-100hf" },
+        expectedBehaviour: "BEFORE: naming this bey with no code produced 'Inconclusive — Nothing contradicts it, and nothing confirms it either' and a physical checklist. True, in that nothing had been checked — and it withheld the 33 box codes the tables record for this exact bey, which is the single most useful thing the tool holds and the one thing the user had asked for. AFTER: the verdict is 'I know this one. Now find the code stamped on the box.' — still not a judgement, because nothing has been checked, but with the next step named and the 33 codes listed to check against.",
+        expectedUiState: "Verdict badge 'Need a bit more', headline 'I know this one. Now find the code stamped on the box.', a caveat naming the count, and a panel listing 33 codes as badges. A link reading 'Everything we hold on Lightning L-Drago 100HF' points at the model's own category page.",
+        expectedData: { recordedCodes: 33 },
+        endResult: "Read-only.",
+    },
+    "checklist-content-discovery-authenticity-checker-picked-model-plus-its-own-code": {
+        roles: ["guest"],
+        startPage: "/tools/authenticity",
+        steps: [
+            "Pick 'Lightning L-Drago 100HF' in 'Which bey is it?'.",
+            "Enter A0910 in 'Code on the box' — a code recorded against this exact bey.",
+            "Press 'Check it' and read the verdict.",
+            "Expand 'How I got there' and find the row about the code.",
+            "Press 'Start over'. Pick Lightning L-Drago 100HF again, this time enter A0310 — a real code recorded against a DIFFERENT bey.",
+            "Read the verdict and expand the working-out again.",
+            "Answer no if the second case reads as a mismatch, a discrepancy, or anything implying the item is wrong.",
+        ],
+        inputs: { modelSlug: "lightning-l-drago-100hf", recordedCode: "A0910", otherBeyCode: "A0310" },
+        expectedBehaviour: "A0910 is in this bey's own list, so the check reads as agreement — the strongest thing the tool can say from data, and it still says so is not proof. A0310 is recorded, but on Earth Aquila: that is WEAK SUPPORT, not a mismatch, because a production date covers whatever was boxed that day. The distinction is the whole point. Reporting 'that code belongs to a different bey' as a discrepancy would manufacture suspicion out of how the tables are organised, and the panel highlights the code you entered inside the list so you can see where it sits.",
+        expectedUiState: "First run: 'Consistent with an original', and a row reading that the code is recorded on genuine boxes of this exact bey. Second run: still not adverse — a neutral row saying the code is recorded but not against this bey, with the phrase 'weak support rather than a mismatch'.",
+        endResult: "Read-only.",
+    },
+    "checklist-content-discovery-authenticity-checker-missing-code-is-not-an-accusation": {
+        roles: ["guest"],
+        startPage: "/tools/authenticity",
+        steps: [
+            "Pick 'Lightning L-Drago 100HF' in 'Which bey is it?'.",
+            "Enter Z9999 in 'Code on the box' — well-formed, and in none of the tables.",
+            "Press 'Check it'.",
+            "Read the verdict, the code panel and the working-out.",
+            "Answer no if any sentence implies the item is fake, counterfeit or a reproduction, or if the verdict is more severe than 'Inconclusive'.",
+        ],
+        inputs: { modelSlug: "lightning-l-drago-100hf", code: "Z9999" },
+        expectedBehaviour: "A code absent from a model's list means UNRECORDED, never wrong. These tables are what collectors photographed, not a manufacturing record: 38 of 221 canonical beys have any recorded code at all, new genuine codes turn up regularly, and counterfeiters print real codes anyway. The panel must say so where a user will actually read it, not only inside the collapsed working-out — the reading 'my code is not in the list, so it is fake' is the one this whole feature has to defend against.",
+        expectedUiState: "Verdict 'Inconclusive'. A neutral row reading 'Not among the 33 codes recorded for Lightning L-Drago 100HF' whose text says 'unrecorded, not wrong'. The code panel carries a line beginning 'A code missing from this list means unrecorded, not fake.'",
+        endResult: "Read-only.",
+    },
+    "checklist-content-discovery-authenticity-checker-working-out-is-collapsed-by-default": {
+        roles: ["guest"],
+        startPage: "/tools/authenticity",
+        steps: [
+            "Pick 'Lightning L-Drago 100HF' and enter A0910. Press 'Check it'.",
+            "WITHOUT expanding anything, confirm you can read the verdict badge, the headline, the caveat and the recorded-code list.",
+            "Confirm the reasoning is behind a single collapsed control reading 'How I got there — N checks'.",
+            "Expand it and confirm every individual check is still there, each with its own confidence chip.",
+            "Press 'Start over'. Now enter the code A0813 with no model picked — a code the sources record as a midfake batch.",
+            "Confirm the working-out is ALREADY OPEN this time, without being clicked.",
+            "Answer no if the reasoning is expanded by default on the clean run, or collapsed on the midfake run.",
+        ],
+        inputs: { modelSlug: "lightning-l-drago-100hf", cleanCode: "A0910", midfakeCode: "A0813" },
+        expectedBehaviour: "BEFORE: every check rendered expanded under a 'What I checked' heading — up to eleven rows of reasoning between the buyer and the answer they asked for. AFTER: it is collapsed, not deleted, because 'why should I believe this' has to stay one tap away for a tool whose entire posture is that it hedges and cites. A CONCERN forces it open: the one case where the reasoning IS the answer is the case where something is wrong, and the user who never expands it must not be the user who never finds out.",
+        expectedUiState: "Clean run: verdict block, code panel, a collapsed 'How I got there — N checks', then the physical checks. Midfake run: the same control, already open.",
+        endResult: "Read-only.",
+    },
+    "checklist-content-discovery-authenticity-checker-result-scrolls-into-view": {
+        roles: ["guest"],
+        startPage: "/tools/authenticity",
+        steps: [
+            "Resize the browser to a phone viewport (about 390 x 740) or use device emulation.",
+            "Pick 'Lightning L-Drago 100HF' and enter A0910.",
+            "Scroll so the 'Check it' button is visible, and note what is on screen.",
+            "Press 'Check it'.",
+            "Without scrolling yourself, confirm the verdict badge and headline are now on screen.",
+            "Answer no if the page does not move and the verdict is below the fold.",
+        ],
+        inputs: { viewport: "390x740", modelSlug: "lightning-l-drago-100hf", code: "A0910" },
+        expectedBehaviour: "BEFORE: the form is taller than a phone viewport, so the answer rendered below the fold and pressing 'Check it' looked like it had done nothing — at the one moment this tool must not be ambiguous. AFTER: the result is scrolled into view on every fresh assessment, including the re-run a clarifying question triggers, and anchored at the top of the viewport rather than the middle so the badge and headline land where the eye already is.",
+        expectedUiState: "After submitting, the verdict badge and headline are visible without manual scrolling.",
+        endResult: "Read-only.",
+    },
 };

package/dist/features/tester/seed-data/tester-checklist-seed-data.js

@@ -3301,6 +3301,18 @@ const rawTesterChecklistItems = [
                 { key: "regional-code-reads-as-regional", label: "A Korean or Middle-East code names its region instead of coming back unrecorded" },
                 { key: "series-window-names-the-production-run", label: "A derivable date names its Metal-series production window, or names none outside all four" },
                 { key: "temporal-context-for-an-unmatched-code", label: "An unmatched code reports what else the tables record from the same month, framed as context not a match" },
+                /* ── The model picker and the per-model code list (2026-10-20) ───
+                 * The reported bug: naming a bey the tool holds 33 recorded box
+                 * codes for produced "Nothing contradicts it, and nothing confirms
+                 * it either" and never mentioned them. Every case here compares the
+                 * answer against data the tool demonstrably has.
+                 */
+                { key: "model-picker-finds-l-drago", label: "The model picker finds all three L-Dragos by partial name, and each shows how many box codes are on file" },
+                { key: "picked-model-lists-its-box-codes", label: "Picking a model with no code returns the codes recorded for that exact bey, not a bare 'inconclusive'" },
+                { key: "picked-model-plus-its-own-code", label: "A code recorded against the picked bey reads as agreement; one recorded against a different bey reads as weak support, not a mismatch" },
+                { key: "missing-code-is-not-an-accusation", label: "A code absent from the picked model's list says 'unrecorded', never 'fake', and the verdict stays inconclusive" },
+                { key: "working-out-is-collapsed-by-default", label: "The verdict is readable without expanding anything, and a concern auto-opens the working-out" },
+                { key: "result-scrolls-into-view", label: "Submitting scrolls the verdict into view on a phone rather than leaving it below the fold" },
             ],
         },
         {

package/dist/seed/products-beyblade-x-seed-data.js

@@ -51,10 +51,44 @@ import { seedGallery } from "./_helpers/media.js";
 import { PRODUCT_FIELDS } from "../constants/field-names.js";
 import { withFinalSale } from "./_helpers/final-sale-fixtures.js";
 import { withDerivedCategoryNames } from "./_helpers/category-names.js";
+import { buildSearchTxt } from "../utils/search-txt.js";
 const STORE_ID = "store-letitrip-official";
 const STORE_NAME = "LetItRip Official";
 const BRAND_SLUG = "brand-takara-tomy";
 const BRAND = "Takara-Tomy";
+/**
+ * Build the `searchTxt` prefix index for a row.
+ *
+ * 🛑 THIS FILE SHIPPED WITHOUT IT AND EVERY LISTING WAS UNSEARCHABLE.
+ *
+ * `searchTxt` is the ONLY field the public product search queries. A row
+ * without it is not ranked lower — it is absent. Measured on production
+ * before this fix: 12 of the 14 real listings had no `searchTxt` at all, and
+ * the only two that did were the auction and the pre-order, whose own files
+ * already applied a tokeniser. So `?q=aero` returned 1 of the 3 AeroPegasus
+ * products and `?q=beyblade` returned 2 of 14, while a nonsense control
+ * correctly returned 0 — the search was working perfectly against an index
+ * that was almost empty.
+ *
+ * CLAUDE.md documents this exact trap on this exact pipeline: a wrapper
+ * missed when a seed file is created, with no error anywhere, because an
+ * absent field is indistinguishable from "no match" at every layer above.
+ *
+ * 🛑 Applied LAST, after `withDerivedCategoryNames`. The category names are
+ * derived by that wrapper, so tokenising before it indexes `undefined` and
+ * silently loses every category term ("Basic Line", "Beyblade X Tops") from
+ * the index.
+ */
+function withSearchTokens(p) {
+    return {
+        ...p,
+        searchTxt: buildSearchTxt([
+            p.title, p.description, p.brand, p.brandSlug,
+            p.categoryNames, p.tags, p.condition,
+            p.specifications?.map((s) => `${s.name} ${s.value}`),
+        ]),
+    };
+}
 const _raw = [
     // ===== Standalone tops =====
     {
@@ -438,4 +472,5 @@ export const productsBeybladeXSeedData = _raw
     seedPurpose: "permanent",
 }))
     .map(withFinalSale)
-    .map(withDerivedCategoryNames);
+    .map(withDerivedCategoryNames)
+    .map(withSearchTokens);

package/dist/server.d.ts

@@ -289,6 +289,8 @@ export { processRefundAction, type ProcessRefundInput, } from "./_internal/serve
 export { computeCheckoutFees, computePayoutDeduction, computeCodHandlingFee, computeWhatsAppNotifyFee, computeGiftWrapFee, computeShipmentProtectionFee, computeSaleProtectionFee, } from "./_internal/shared/fees/calculator";
 export { searchPriceIndex, getPriceIndexBandsForIds, PRICE_INDEX_SEARCH_LIMIT, PRICE_INDEX_MIN_QUERY_LENGTH } from "./_internal/server/features/price-index/search";
 export type { PriceIndexSearchResponse, PriceIndexSearchResult, PriceIndexSearchResultBand, } from "./_internal/server/features/price-index/search";
+export { getPriceIndexMarketForIds } from "./_internal/server/features/price-index/market";
+export type { PriceIndexMarketRow } from "./_internal/server/features/price-index/market";
 export { protectionAvailability, computeProtectionFees, adjudicateClaim, badgesFor, showsRiskNotice, showsSafetyBadge, returnDestination, rtoDestination, PROTECTION_KINDS, } from "./_internal/shared/features/shipping/protections";
 export type { ProtectionKind, ProtectionAvailability, ProtectionTierInput, ProtectionUnavailableReason, ClaimFinding, ClaimOutcome, } from "./_internal/shared/features/shipping/protections";
 export type { FeeCommissionRates, CheckoutFees, PayoutDeduction, CodHandlingFeeRates, WhatsAppNotifyFeeRates, GiftWrapFeeRates, ShipmentProtectionFeeRates, SaleProtectionFeeRates, } from "./_internal/shared/fees/calculator";

package/dist/server.js

@@ -846,6 +846,14 @@ export { computeCheckoutFees, computePayoutDeduction, computeCodHandlingFee, com
 // client.ts (verify-entries would catch firebase-admin, but the import chain
 // is the thing to keep clean — Root Cause #6).
 export { searchPriceIndex, getPriceIndexBandsForIds, PRICE_INDEX_SEARCH_LIMIT, PRICE_INDEX_MIN_QUERY_LENGTH } from "./_internal/server/features/price-index/search.js";
+/*
+ * The MARKET half of the same tool, and a separate export because it is a
+ * separate AUTHENTICATED endpoint — the competitor-derived band must not ride
+ * the public, edge-cached sold response. The full argument is in that module's
+ * own header; the short version is that merging them would either poison a
+ * public cache with viewer-dependent content or cost the sold half its cache.
+ */
+export { getPriceIndexMarketForIds } from "./_internal/server/features/price-index/market.js";
 export { protectionAvailability, computeProtectionFees, adjudicateClaim, badgesFor, showsRiskNotice, showsSafetyBadge, returnDestination, rtoDestination, PROTECTION_KINDS, } from "./_internal/shared/features/shipping/protections.js";
 // [SERVER-ONLY] Payout refund deduction action
 export { applyRefundDeductionAction, } from "./_internal/server/features/payouts/actions.js";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mohasinac/appkit",
-  "version": "4.53.1",
+  "version": "4.53.2",
   "license": "MIT",
   "publishConfig": {
     "access": "public"

package/scripts/seed-cli.mjs

@@ -387,6 +387,104 @@ const COLLECTION_MAP = {
  * A root with no mapping yields NO franchise (living collectibles are not an
  * IP) — see `FRANCHISE_BY_CATEGORY_ROOT`.
  */
+/**
+ * The real, sellable catalogue — everything else in the product seed is demo
+ * furniture and is DROPPED HERE, before anything is written.
+ *
+ * ── 🛑 WHY A FILTER AND NOT A DELETION ────────────────────────────────────
+ *
+ * The fixtures stay in their files on purpose: they are the only coverage
+ * several features have (every `preOrderProductionStatus` value, a sold-out
+ * reservation, a depleted digital-code pool, an ended-unsold auction), and
+ * the tester checklist deep-links to them by id. Deleting them would silently
+ * remove that coverage. Not LOADING them is the part that was wanted.
+ *
+ * ── 🛑 WHY THIS HAD TO EXIST ──────────────────────────────────────────────
+ *
+ * `appkit-seed load` writes every registered row and `purge` deletes
+ * `seedPurpose: "test"` afterwards — so the demo listings came BACK on every
+ * reseed, and anyone who ran `load` to fix imagery silently republished 76
+ * fake products to the live storefront. That happened twice in one session.
+ * The owner's instruction was "delete them from the seed itself and make sure
+ * they do not return"; this is the point where that can be guaranteed, since
+ * every write path funnels through here.
+ *
+ * ── The discriminator is the STORE, not the listing type ──────────────────
+ *
+ * `store-beyblade-arena` is the fictional demo seller; `store-letitrip-
+ * official` is the real one. Keying on the store rather than on a per-file
+ * allow-list means a fixture added to ANY product seed file later is excluded
+ * automatically, which is the failure mode this is replacing — a list that
+ * has to be remembered is a list that drifts.
+ *
+ * Set `SEED_INCLUDE_DEMO=1` to load them anyway, for local demo work.
+ */
+const REAL_STORE_IDS = new Set(["store-letitrip-official"]);
+
+function realListingsOnly(products) {
+  if (process.env.SEED_INCLUDE_DEMO === "1") {
+    console.log("▌ SEED_INCLUDE_DEMO=1 — demo listings INCLUDED");
+    return products;
+  }
+  /*
+   * 🛑 TWO rules, because the store alone is not enough. Six prize-draw
+   * fixtures live under the REAL store and are still demo furniture — their
+   * imagery is a 460-byte generated colour tile from /images/seed-tiles/,
+   * which is the giveaway: a listing nobody photographed is a listing nobody
+   * intends to sell. Measured before writing this: every one of the six, and
+   * nothing real, matches.
+   */
+  const isPlaceholder = (p) => {
+    const urls = [p?.mainImage, ...(Array.isArray(p?.images) ? p.images : [])]
+      .map((v) => (typeof v === "string" ? v : v?.url))
+      .filter(Boolean);
+    return urls.length > 0 && urls.every((u) => String(u).includes("/images/seed-tiles/"));
+  };
+  const kept = products.filter((p) => REAL_STORE_IDS.has(p?.storeId) && !isPlaceholder(p));
+  const dropped = products.length - kept.length;
+  if (dropped > 0) {
+    console.log(`  • ${dropped} demo listing(s) not loaded (set SEED_INCLUDE_DEMO=1 to include)`);
+  }
+  return kept;
+}
+
+/**
+ * Drop bundle categories whose members are not in the loaded catalogue.
+ *
+ * 🛑 A bundle is ALL-OR-NOTHING — `canAddToCart` buys every member together —
+ * so a bundle missing even one member is unsellable, and one missing ALL of
+ * them is a card that opens an empty page. Measured right after the demo
+ * listings stopped loading: of the five seeded bundles, three had zero
+ * surviving members and the other two had 1-of-4 and 2-of-3. All five were
+ * still `isActive: true`, and all five rendered on the HOMEPAGE as collage
+ * cards built from `display.mosaicImages` — which are baked at seed time from
+ * the members' own imagery, so they kept showing pictures of products that no
+ * longer exist. That is 92 placeholder-tile references on the homepage.
+ *
+ * Derived rather than hand-maintained for the same reason as the product
+ * filter: a bundle that becomes valid later starts loading again on its own,
+ * and one that breaks stops, with nobody having to remember either.
+ *
+ * Only `categoryType: "bundle"` rows are considered. Every other category is
+ * taxonomy and is never touched — rebuilding the tree is expensive and is
+ * explicitly not what this is for.
+ */
+function loadableBundlesOnly(categories, loadedProducts) {
+  const live = new Set(loadedProducts.map((p) => p?.id).filter(Boolean));
+  let dropped = 0;
+  const kept = categories.filter((c) => {
+    if (c?.categoryType !== "bundle") return true;
+    const ids = Array.isArray(c.bundleProductIds) ? c.bundleProductIds : [];
+    const ok = ids.length > 0 && ids.every((id) => live.has(id));
+    if (!ok) dropped++;
+    return ok;
+  });
+  if (dropped > 0) {
+    console.log(`  • ${dropped} bundle(s) not loaded — members missing from the catalogue`);
+  }
+  return kept;
+}
+
 function withFranchise(products) {
   let stamped = 0;
   const out = products.map((p) => {
@@ -469,32 +567,13 @@ function withDerivedCategoryFields(products) {
   return out;
 }
 
-const SEED_DATA_MAP = {
-  users: usersSeedData,
-  /*
-   * store-extensions. COLLECTION_MAP was only HALF the wiring: load() reads
-   * SEED_DATA_MAP and short-circuits on an empty array, so adding the ten
-   * collection names alone produced "no seed data, skipping" for all ten.
-   * Same shape as the art/stickers note further down -- added everywhere
-   * except the map that actually runs.
-   */
-  payoutMethods: payoutMethodsSeedData,
-  shippingConfigs: shippingConfigsSeedData,
-  analyticsCards: analyticsCardsSeedData,
-  analyticsAlerts: analyticsAlertsSeedData,
-  storeCategories: storeCategoriesSeedData,
-  listingTemplates: listingTemplatesSeedData,
-  moderationQueue: moderationQueueSeedData,
-  reports: reportsSeedData,
-  itemRequests: itemRequestsSeedData,
-  storeGoogleConfig: storeGoogleConfigSeedData,
-  addresses: addressesSeedData,
-  storeAddresses: storeAddressesSeedData,
-  // SB-UNI-C: brands were merged into categories (categoryType:"brand") — no
-  // standalone brands collection/seed exists anymore.
-  categories: categoriesSeedData,
-  stores: storesSeedData,
-  products: withFranchise(withDerivedCategoryFields([
+/**
+ * The real catalogue, computed ONCE so the bundle filter and the product
+ * loader cannot disagree about what "loaded" means. Deriving it twice is
+ * Root Cause #75's shape — two copies of one rule, one of which learns about
+ * a change and the other does not.
+ */
+const REAL_PRODUCTS = realListingsOnly([
     ...(productsStandardSeedData || []),
     // Real, wiki-verified Beyblade X listings under store-letitrip-official —
     // added 2026-10-19 for PhonePe merchant registration, which requires at
@@ -536,7 +615,34 @@ const SEED_DATA_MAP = {
      * what should happen to a leaf that now has BOTH a real listing and a
      * generated one.
      */
-  ])),
+  ]);
+
+const SEED_DATA_MAP = {
+  users: usersSeedData,
+  /*
+   * store-extensions. COLLECTION_MAP was only HALF the wiring: load() reads
+   * SEED_DATA_MAP and short-circuits on an empty array, so adding the ten
+   * collection names alone produced "no seed data, skipping" for all ten.
+   * Same shape as the art/stickers note further down -- added everywhere
+   * except the map that actually runs.
+   */
+  payoutMethods: payoutMethodsSeedData,
+  shippingConfigs: shippingConfigsSeedData,
+  analyticsCards: analyticsCardsSeedData,
+  analyticsAlerts: analyticsAlertsSeedData,
+  storeCategories: storeCategoriesSeedData,
+  listingTemplates: listingTemplatesSeedData,
+  moderationQueue: moderationQueueSeedData,
+  reports: reportsSeedData,
+  itemRequests: itemRequestsSeedData,
+  storeGoogleConfig: storeGoogleConfigSeedData,
+  addresses: addressesSeedData,
+  storeAddresses: storeAddressesSeedData,
+  // SB-UNI-C: brands were merged into categories (categoryType:"brand") — no
+  // standalone brands collection/seed exists anymore.
+  categories: loadableBundlesOnly(categoriesSeedData, REAL_PRODUCTS),
+  stores: storesSeedData,
+  products: withFranchise(withDerivedCategoryFields(REAL_PRODUCTS)),
   orders: ordersSeedData,
   reviews: reviewsSeedData,
   bids: bidsSeedData,