@mohasinac/appkit 4.53.0 → 4.53.1

package/dist/_internal/server/features/checkout/supplier-identity.d.ts

@@ -0,0 +1,55 @@
+/**
+ * Who is the supplier of record on this order's invoice.
+ *
+ * ── The rule ──────────────────────────────────────────────────────────────
+ *
+ * A seller WITH their own GSTIN invoices under it. A seller WITHOUT one
+ * invoices under LetItRip's. The official store carries its own, so it is the
+ * first case rather than a special case.
+ *
+ * ── Why this is one function ──────────────────────────────────────────────
+ *
+ * 🛑 It would be two lines to write `store?.gstin ?? settings.gst?.gstin` at
+ * the call site, and that is exactly how Root Cause #75 happened: a money rule
+ * described as "the single source of truth" that was reachable by copy-paste,
+ * got copied four times, and two copies never learned about a branch added
+ * later — so Razorpay captured the list price for an accepted offer. Grepping
+ * for a FUNCTION name finds every caller; grepping for the SHAPE of an
+ * expression does not.
+ *
+ * The three fields also have to move TOGETHER. A GSTIN printed under the wrong
+ * legal name is a worse invoice than no GSTIN at all, so this never mixes a
+ * seller's GSTIN with the platform's name — see `resolve` below.
+ *
+ * ── Why it is snapshot, not looked up ─────────────────────────────────────
+ *
+ * The caller writes the result onto `OrderDocument.supplierGstin` /
+ * `supplierLegalName` / `supplierAddress` at creation and never recomputes it.
+ * `OrderDocument`'s own comment is explicit that these are "OUR GST
+ * registration as it stood at the moment of supply" — an admin correcting a
+ * typo in the GSTIN must not silently reissue every invoice ever produced.
+ */
+import type { StoreDocument } from "../../../../features/stores/schemas/firestore";
+import type { SiteSettingsDocument } from "../../../../features/admin/schemas/firestore";
+export interface SupplierIdentity {
+    gstin?: string;
+    legalName?: string;
+    address?: string;
+    /**
+     * Which party the invoice is issued by.
+     *
+     * Not decoration: the invoice renderers use it to decide whether to say
+     * "Tax Invoice", and an operator reading an order needs to know which
+     * registration was applied without re-deriving the rule.
+     */
+    source: "seller" | "platform" | "none";
+}
+/**
+ * Resolve the supplier identity for an order from `store`.
+ *
+ * Returns `source: "none"` when GST is disabled or the platform has no GSTIN
+ * configured and the seller has none either. A caller must treat that as "this
+ * is not a tax invoice" rather than as an error — GST is genuinely optional
+ * and `siteSettings.gst.enabled` defaults to false.
+ */
+export declare function resolveSupplierIdentity(store: Pick<StoreDocument, "gstin" | "legalName" | "registeredAddress"> | null | undefined, settings: Pick<SiteSettingsDocument, "gst"> | null | undefined): SupplierIdentity;

package/dist/_internal/server/features/checkout/supplier-identity.js

@@ -0,0 +1,81 @@
+/**
+ * Who is the supplier of record on this order's invoice.
+ *
+ * ── The rule ──────────────────────────────────────────────────────────────
+ *
+ * A seller WITH their own GSTIN invoices under it. A seller WITHOUT one
+ * invoices under LetItRip's. The official store carries its own, so it is the
+ * first case rather than a special case.
+ *
+ * ── Why this is one function ──────────────────────────────────────────────
+ *
+ * 🛑 It would be two lines to write `store?.gstin ?? settings.gst?.gstin` at
+ * the call site, and that is exactly how Root Cause #75 happened: a money rule
+ * described as "the single source of truth" that was reachable by copy-paste,
+ * got copied four times, and two copies never learned about a branch added
+ * later — so Razorpay captured the list price for an accepted offer. Grepping
+ * for a FUNCTION name finds every caller; grepping for the SHAPE of an
+ * expression does not.
+ *
+ * The three fields also have to move TOGETHER. A GSTIN printed under the wrong
+ * legal name is a worse invoice than no GSTIN at all, so this never mixes a
+ * seller's GSTIN with the platform's name — see `resolve` below.
+ *
+ * ── Why it is snapshot, not looked up ─────────────────────────────────────
+ *
+ * The caller writes the result onto `OrderDocument.supplierGstin` /
+ * `supplierLegalName` / `supplierAddress` at creation and never recomputes it.
+ * `OrderDocument`'s own comment is explicit that these are "OUR GST
+ * registration as it stood at the moment of supply" — an admin correcting a
+ * typo in the GSTIN must not silently reissue every invoice ever produced.
+ */
+const trimmed = (v) => {
+    const s = (v ?? "").trim();
+    return s.length > 0 ? s : undefined;
+};
+/**
+ * Resolve the supplier identity for an order from `store`.
+ *
+ * Returns `source: "none"` when GST is disabled or the platform has no GSTIN
+ * configured and the seller has none either. A caller must treat that as "this
+ * is not a tax invoice" rather than as an error — GST is genuinely optional
+ * and `siteSettings.gst.enabled` defaults to false.
+ */
+export function resolveSupplierIdentity(store, settings) {
+    const sellerGstin = trimmed(store?.gstin);
+    /*
+     * Seller branch. Taken on the GSTIN alone, and the legal name and address
+     * come from the SELLER too — falling back to the platform's for a missing
+     * name would print LetItRip's legal entity beside the seller's registration,
+     * which identifies neither party correctly.
+     *
+     * A seller with a GSTIN and no legal name is a data gap worth seeing as a
+     * blank on the invoice, not one worth papering over with the wrong name.
+     */
+    if (sellerGstin) {
+        return {
+            gstin: sellerGstin,
+            legalName: trimmed(store?.legalName),
+            address: trimmed(store?.registeredAddress),
+            source: "seller",
+        };
+    }
+    /*
+     * Platform branch. Gated on `gst.enabled`, matching what the checkout
+     * already did before sellers could have a GSTIN at all — disabling GST must
+     * continue to produce plain invoices rather than tax invoices.
+     */
+    const gst = settings?.gst;
+    if (gst?.enabled) {
+        const platformGstin = trimmed(gst.gstin);
+        if (platformGstin) {
+            return {
+                gstin: platformGstin,
+                legalName: trimmed(gst.legalName),
+                address: trimmed(gst.address),
+                source: "platform",
+            };
+        }
+    }
+    return { source: "none" };
+}

package/dist/_internal/server/features/classified/metadata.js

@@ -33,7 +33,7 @@ export function buildClassifiedMetadata(product, opts) {
             description,
             url: canonical,
             type: "website",
-            images: image ? [{ url: image }] : undefined,
+            images: image ? [{ url: typeof image === "string" ? image : image.url }] : undefined,
         },
         twitter: {
             card: image ? "summary_large_image" : "summary",

package/dist/_internal/server/features/classified/og.d.ts

@@ -10,7 +10,9 @@ interface ClassifiedDocLike {
     price?: number | null;
     currency?: string | null;
     mainImage?: string | null;
-    images?: (string | null | undefined)[] | null;
+    images?: ({
+        url: string;
+    } | string | null | undefined)[] | null;
     classified?: {
         meetupArea?: {
             city?: string;

package/dist/_internal/server/features/classified/og.js

@@ -8,7 +8,9 @@ export function renderClassifiedOg(doc, opts) {
     return renderClassifiedOgImage({
         title: doc?.title ?? "Classified Listing",
         location,
-        imageUrl: resolveOgImageUrl(doc?.mainImage || doc?.images?.[0] || null, opts.baseUrl),
+        imageUrl: resolveOgImageUrl(doc?.mainImage ||
+            (typeof doc?.images?.[0] === "string" ? doc.images[0] : doc?.images?.[0]?.url) ||
+            null, opts.baseUrl),
     }, opts.siteName);
 }
 export function renderClassifiedOgImage(data, siteName) {

package/dist/_internal/server/features/digital-code/metadata.js

@@ -26,7 +26,7 @@ export function buildDigitalCodeMetadata(product, opts) {
             description,
             url: canonical,
             type: "website",
-            images: image ? [{ url: image }] : undefined,
+            images: image ? [{ url: typeof image === "string" ? image : image.url }] : undefined,
         },
         twitter: {
             card: image ? "summary_large_image" : "summary",

package/dist/_internal/server/features/digital-code/og.d.ts

@@ -10,7 +10,9 @@ interface DigitalCodeDocLike {
     price?: number | null;
     currency?: string | null;
     mainImage?: string | null;
-    images?: (string | null | undefined)[] | null;
+    images?: ({
+        url: string;
+    } | string | null | undefined)[] | null;
     digitalCode?: {
         codeDeliveryMethod?: string;
     } | null;

package/dist/_internal/server/features/digital-code/og.js

@@ -9,7 +9,9 @@ export function renderDigitalCodeOg(doc, opts) {
     return renderDigitalCodeOgImage({
         title: doc?.title ?? "Digital Code",
         deliveryMethod,
-        imageUrl: resolveOgImageUrl(doc?.mainImage || doc?.images?.[0] || null, opts.baseUrl),
+        imageUrl: resolveOgImageUrl(doc?.mainImage ||
+            (typeof doc?.images?.[0] === "string" ? doc.images[0] : doc?.images?.[0]?.url) ||
+            null, opts.baseUrl),
     }, opts.siteName);
 }
 export function renderDigitalCodeOgImage(data, siteName) {

package/dist/_internal/server/features/digital-code/pool.d.ts

@@ -20,10 +20,25 @@ import { type ProductCodeContentKind } from "../../../../features/products/schem
 /**
  * Where a delivered asset's bytes live.
  *
- * Deliberately OUTSIDE every prefix the public media proxy knows how to resolve,
- * and no `mediaAssets` row is ever created for it — a `mediaAssets` row is what
- * mints a `/media/{shortId}` slug, and that slug is served without
- * authentication. No row, no slug, no public URL.
+ * No `mediaAssets` row is ever created for one — a row is what mints a
+ * `/media/{shortId}` slug, and that slug is served without authentication. No
+ * row, no slug, no public URL. That half was, and remains, true.
+ *
+ * 🛑 CORRECTED 2026-10-19. This block used to open with "deliberately OUTSIDE
+ * every prefix the public media proxy knows how to resolve", and that was
+ * FALSE from the day the proxy's raw-path fallback shipped: `/media/[...slug]`
+ * treats any multi-segment slug it cannot resolve through `mediaAssets` as a
+ * literal storage path, so `/media/private/digital-content/...` resolved and
+ * served. It did watermark, so no pristine original leaked — but a paid asset
+ * was reachable by anyone who guessed the object name, and the only thing
+ * standing in the way was `randomUUID()`.
+ *
+ * The proxy now refuses a raw path under `private/` outright
+ * (`PRIVATE_PREFIX` in `src/app/api/media/[...slug]/route.ts`), so the
+ * statement is true again — but it is true because of a check, not because of
+ * the prefix, and that distinction is the whole reason this comment was wrong
+ * for so long. An unenforced invariant written as a fact is worse than no
+ * comment: it is the reason nobody looked.
  */
 export declare const DIGITAL_CONTENT_PREFIX = "private/digital-content";
 /**

package/dist/_internal/server/features/digital-code/pool.js

@@ -26,10 +26,25 @@ import { PRODUCT_COLLECTION, PRODUCT_CODES_SUBCOLLECTION, } from "../../../../fe
 /**
  * Where a delivered asset's bytes live.
  *
- * Deliberately OUTSIDE every prefix the public media proxy knows how to resolve,
- * and no `mediaAssets` row is ever created for it — a `mediaAssets` row is what
- * mints a `/media/{shortId}` slug, and that slug is served without
- * authentication. No row, no slug, no public URL.
+ * No `mediaAssets` row is ever created for one — a row is what mints a
+ * `/media/{shortId}` slug, and that slug is served without authentication. No
+ * row, no slug, no public URL. That half was, and remains, true.
+ *
+ * 🛑 CORRECTED 2026-10-19. This block used to open with "deliberately OUTSIDE
+ * every prefix the public media proxy knows how to resolve", and that was
+ * FALSE from the day the proxy's raw-path fallback shipped: `/media/[...slug]`
+ * treats any multi-segment slug it cannot resolve through `mediaAssets` as a
+ * literal storage path, so `/media/private/digital-content/...` resolved and
+ * served. It did watermark, so no pristine original leaked — but a paid asset
+ * was reachable by anyone who guessed the object name, and the only thing
+ * standing in the way was `randomUUID()`.
+ *
+ * The proxy now refuses a raw path under `private/` outright
+ * (`PRIVATE_PREFIX` in `src/app/api/media/[...slug]/route.ts`), so the
+ * statement is true again — but it is true because of a check, not because of
+ * the prefix, and that distinction is the whole reason this comment was wrong
+ * for so long. An unenforced invariant written as a fact is worse than no
+ * comment: it is the reason nobody looked.
  */
 export const DIGITAL_CONTENT_PREFIX = "private/digital-content";
 /**

package/dist/_internal/server/features/live/metadata.js

@@ -29,7 +29,7 @@ export function buildLiveItemMetadata(product, opts) {
             description,
             url: canonical,
             type: "website",
-            images: image ? [{ url: image }] : undefined,
+            images: image ? [{ url: typeof image === "string" ? image : image.url }] : undefined,
         },
         twitter: {
             card: image ? "summary_large_image" : "summary",

package/dist/_internal/server/features/live/og.d.ts

@@ -10,7 +10,9 @@ interface LiveItemDocLike {
     price?: number | null;
     currency?: string | null;
     mainImage?: string | null;
-    images?: (string | null | undefined)[] | null;
+    images?: ({
+        url: string;
+    } | string | null | undefined)[] | null;
     liveItem?: {
         species?: string;
     } | null;

package/dist/_internal/server/features/live/og.js

@@ -4,7 +4,9 @@ export function renderLiveItemOg(doc, opts) {
     return renderLiveItemOgImage({
         title: doc?.title ?? "Live Listing",
         species: doc?.liveItem?.species ?? null,
-        imageUrl: resolveOgImageUrl(doc?.mainImage || doc?.images?.[0] || null, opts.baseUrl),
+        imageUrl: resolveOgImageUrl(doc?.mainImage ||
+            (typeof doc?.images?.[0] === "string" ? doc.images[0] : doc?.images?.[0]?.url) ||
+            null, opts.baseUrl),
     }, opts.siteName);
 }
 export function renderLiveItemOgImage(data, siteName) {

package/dist/_internal/server/features/messages/actions.js

@@ -160,7 +160,7 @@ export async function sendMessage(input) {
             productRef = {
                 productId: product.id,
                 title: product.title,
-                image: product.images?.[0],
+                image: product.images?.[0]?.url,
                 price: product.price,
                 // Without this the card cannot pick the right detail route — see the
                 // field's own comment on `MessageProductRef`.

package/dist/_internal/server/features/parts/og.js

@@ -58,7 +58,7 @@ export function renderPartOg(part, usedByCount, opts) {
 /** One classification's card. */
 export function renderPartClassificationOg(slot, count, opts) {
     const label = PART_SLOT_LABEL[slot] ?? slot;
-    return shell("Parts database", label, 
+    return shell("Beyblade parts gallery", label, 
     // `count` comes from the same bounded read the page renders, so the number
     // on the card is the number on the page. A separate count query would be a
     // second source that could disagree.

package/dist/_internal/server/features/pre-orders/og.d.ts

@@ -8,7 +8,9 @@ interface PreOrderDocLike {
     title?: string | null;
     preOrderReleaseDate?: Date | string | null;
     mainImage?: string | null;
-    images?: (string | null | undefined)[] | null;
+    images?: ({
+        url: string;
+    } | string | null | undefined)[] | null;
 }
 /** High-level OG renderer — accepts the raw pre-order document from `getPreOrderForDetail`. */
 export declare function renderPreOrderOg(doc: PreOrderDocLike | null | undefined, opts: {

package/dist/_internal/server/features/pre-orders/og.js

@@ -12,7 +12,9 @@ export function renderPreOrderOg(doc, opts) {
     return renderPreOrderOgImage({
         title: doc?.title ?? "Pre-Order",
         releaseDateLabel,
-        imageUrl: resolveOgImageUrl(doc?.mainImage || doc?.images?.[0] || null, opts.baseUrl),
+        imageUrl: resolveOgImageUrl(doc?.mainImage ||
+            (typeof doc?.images?.[0] === "string" ? doc.images[0] : doc?.images?.[0]?.url) ||
+            null, opts.baseUrl),
     }, opts.siteName);
 }
 export function renderPreOrderOgImage(data, siteName) {

package/dist/_internal/server/features/price-index/search.d.ts

@@ -117,3 +117,21 @@ export interface PriceIndexSearchResponse {
  * Cost: one `searchTxt` push-down query plus ONE multi-get. Both bounded.
  */
 export declare function searchPriceIndex(rawQuery: string): Promise<PriceIndexSearchResponse>;
+/**
+ * Tier 1 adoption — the client already resolved candidate leaves LOCALLY via
+ * the public search artifact (`useSiteSearch({ types: ["model", "category"] })`),
+ * at zero Firestore reads. This is the ONLY query this path still needs: one
+ * multi-get, bounded by however many ids the client resolved (capped the
+ * same as the search path, so a crafted request cannot demand an unbounded
+ * `getAll`).
+ *
+ * 🛑 Deliberately NOT filtering brand/bundle categoryType here — the CALLER
+ * already did that at index-build time (`PUBLIC_INDEX_SOURCES`'s category
+ * source only emits real listing leaves under the "model"/"category" type
+ * keys), so re-deriving the same filter from a bare id list would be a
+ * second, independently-maintained copy of a rule that already lives in one
+ * place. If that assumption is ever wrong, the cost is a band shown for a
+ * brand/bundle row — cosmetic, not a leak, since no money figure is ever
+ * attached to a document that does not already have sales recorded.
+ */
+export declare function getPriceIndexBandsForIds(rawIds: readonly string[]): Promise<PriceIndexSearchResult[]>;

package/dist/_internal/server/features/price-index/search.js

@@ -154,8 +154,22 @@ export async function searchPriceIndex(rawQuery) {
         return !t || t === "category" || t === "sublisting";
     });
     const page = listingRows.slice(0, PRICE_INDEX_SEARCH_LIMIT);
-    const bandsByLeaf = await priceIndexRepository.getManyForLeaves(page.map((c) => c.id));
-    const results = page.map((c) => {
+    const results = await buildBandsForLeaves(page);
+    return {
+        ...base,
+        results,
+        truncated: listingRows.length > PRICE_INDEX_SEARCH_LIMIT,
+    };
+}
+/**
+ * The ONE multi-get, shared by the search path above and the ids-only path
+ * below — the actual band-shaping logic (sort by sample size, omit an empty
+ * `bands` array, the "no sales recorded" vs "zero sales" distinction) must
+ * answer identically regardless of how the category list was resolved.
+ */
+async function buildBandsForLeaves(leaves) {
+    const bandsByLeaf = await priceIndexRepository.getManyForLeaves(leaves.map((c) => c.id));
+    return leaves.map((c) => {
         const doc = bandsByLeaf.get(c.id);
         const bands = doc
             ? Object.entries(doc.byListingType ?? {})
@@ -180,9 +194,35 @@ export async function searchPriceIndex(rawQuery) {
             ...(doc ? { windowDays: doc.windowDays, computedAt: toIso(doc.computedAt) } : {}),
         };
     });
-    return {
-        ...base,
-        results,
-        truncated: listingRows.length > PRICE_INDEX_SEARCH_LIMIT,
-    };
+}
+/**
+ * Tier 1 adoption — the client already resolved candidate leaves LOCALLY via
+ * the public search artifact (`useSiteSearch({ types: ["model", "category"] })`),
+ * at zero Firestore reads. This is the ONLY query this path still needs: one
+ * multi-get, bounded by however many ids the client resolved (capped the
+ * same as the search path, so a crafted request cannot demand an unbounded
+ * `getAll`).
+ *
+ * 🛑 Deliberately NOT filtering brand/bundle categoryType here — the CALLER
+ * already did that at index-build time (`PUBLIC_INDEX_SOURCES`'s category
+ * source only emits real listing leaves under the "model"/"category" type
+ * keys), so re-deriving the same filter from a bare id list would be a
+ * second, independently-maintained copy of a rule that already lives in one
+ * place. If that assumption is ever wrong, the cost is a band shown for a
+ * brand/bundle row — cosmetic, not a leak, since no money figure is ever
+ * attached to a document that does not already have sales recorded.
+ */
+export async function getPriceIndexBandsForIds(rawIds) {
+    const ids = [...new Set(rawIds.filter((id) => typeof id === "string" && id.length > 0))].slice(0, PRICE_INDEX_SEARCH_LIMIT);
+    if (ids.length === 0)
+        return [];
+    const docs = await categoriesRepository.getManyByIds(ids);
+    const leaves = docs.map((c) => ({
+        id: c.id,
+        name: c.name,
+        slug: c.slug,
+        path: c.path,
+        display: c.display,
+    }));
+    return buildBandsForLeaves(leaves);
 }

package/dist/_internal/server/features/prize-draws/og.d.ts

@@ -17,7 +17,9 @@ interface PrizeDrawDocLike {
     price?: number | null;
     currency?: string | null;
     mainImage?: string | null;
-    images?: (string | null | undefined)[] | null;
+    images?: ({
+        url: string;
+    } | string | null | undefined)[] | null;
     prizeRevealStatus?: "pending" | "open" | "closed" | null;
     prizeMaxEntries?: number | null;
     prizeCurrentEntries?: number | null;

package/dist/_internal/server/features/prize-draws/og.js

@@ -19,7 +19,9 @@ export function renderPrizeDrawOg(doc, opts) {
     return renderPrizeDrawOgImage({
         title: doc?.title ?? doc?.name ?? "Prize Draw",
         pricePerEntryLabel,
-        imageUrl: resolveOgImageUrl(doc?.mainImage || doc?.images?.[0] || null, opts.baseUrl),
+        imageUrl: resolveOgImageUrl(doc?.mainImage ||
+            (typeof doc?.images?.[0] === "string" ? doc.images[0] : doc?.images?.[0]?.url) ||
+            null, opts.baseUrl),
         revealStatus: doc?.prizeRevealStatus ?? null,
         entriesLabel,
     }, opts.siteName);

package/dist/_internal/server/features/products/group-members.d.ts

@@ -42,6 +42,16 @@ export interface PublicGroupMember {
     groupTitle?: string;
     /** Drives the stepper's `max`. */
     availableQuantity: number;
+    /**
+     * Made to order — the stepper is UNCAPPED for this member.
+     *
+     * 🛑 This is an ALLOW-LIST projection: a field absent from this interface and
+     * from `toPublicGroupMember` below simply does not reach the picker, with no
+     * error anywhere. Without it the picker reads `availableQuantity: 0` on a
+     * print-on-demand member and caps the stepper at zero — the member renders,
+     * looks selectable, and cannot be chosen.
+     */
+    madeToOrder?: boolean;
     storeId: string;
     storeName?: string;
     status: string;
@@ -50,6 +60,22 @@ export interface PublicGroupMember {
      *  snapshot it for per-member GST proration at checkout. */
     gstRate?: 0 | 5 | 12 | 18 | 28;
     hsnCode?: string;
+    /**
+     * The group's OWN selection rules — present only on the PARENT row, since
+     * that is the only document that carries them.
+     *
+     * 🛑 Projected so `readGroupRules` has something real to read client-side.
+     * Without these the picker would silently see no rules on every page that
+     * derives them from this endpoint's own response (`ShowGroupSection`),
+     * which is a client-only regression of the exact kind E3 exists to close —
+     * a server-only rule reads as a broken button, and here it would be the
+     * reverse: a client that never finds out a rule exists at all.
+     */
+    groupMinMembers?: number;
+    groupMaxMembers?: number;
+    groupMaxUnitsPerMember?: number;
+    groupRequiredMemberIds?: string[];
+    groupDiscountPercent?: number;
 }
 export declare function toPublicGroupMember(doc: ProductDocument): PublicGroupMember;
 export declare function toPublicGroupMembers(docs: readonly ProductDocument[]): PublicGroupMember[];

package/dist/_internal/server/features/products/group-members.js

@@ -22,18 +22,24 @@ export function toPublicGroupMember(doc) {
         title: doc.title,
         price: doc.price,
         currency: doc.currency,
-        image: doc.images?.[0] ?? doc.mainImage,
+        image: doc.images?.[0]?.url ?? doc.mainImage,
         condition: doc.condition,
         listingType: normalizeListingType(doc),
         isGroupParent: doc.isGroupParent,
         groupTitle: doc.groupTitle,
         availableQuantity: doc.availableQuantity ?? 0,
+        madeToOrder: doc.madeToOrder,
         storeId: doc.storeId,
         storeName: doc.storeName,
         status: doc.status,
         isSold: doc.isSold,
         gstRate: doc.gstRate,
         hsnCode: doc.hsnCode,
+        groupMinMembers: doc.groupMinMembers,
+        groupMaxMembers: doc.groupMaxMembers,
+        groupMaxUnitsPerMember: doc.groupMaxUnitsPerMember,
+        groupRequiredMemberIds: doc.groupRequiredMemberIds,
+        groupDiscountPercent: doc.groupDiscountPercent,
     };
 }
 export function toPublicGroupMembers(docs) {

package/dist/_internal/server/features/products/og.d.ts

@@ -11,7 +11,9 @@ interface ProductDocLike {
     title?: string | null;
     price?: number | null;
     mainImage?: string | null;
-    images?: (string | null | undefined)[] | null;
+    images?: ({
+        url: string;
+    } | string | null | undefined)[] | null;
 }
 /** High-level OG renderer — accepts the raw product document from `getProductForDetail`. */
 export declare function renderProductOg(doc: ProductDocLike | null | undefined, opts: OgOptions & {

package/dist/_internal/server/features/products/og.js

@@ -6,7 +6,9 @@ export function renderProductOg(doc, opts) {
     return renderProductOgImage({
         title: doc?.title ?? "Product",
         price: doc?.price ?? null,
-        imageUrl: resolveOgImageUrl(doc?.mainImage || doc?.images?.[0] || null, opts.baseUrl),
+        imageUrl: resolveOgImageUrl(doc?.mainImage ||
+            (typeof doc?.images?.[0] === "string" ? doc.images[0] : doc?.images?.[0]?.url) ||
+            null, opts.baseUrl),
     }, opts.siteName);
 }
 export function renderProductOgImage(data, siteName) {

package/dist/_internal/server/features/search/scoped-sources.d.ts

@@ -0,0 +1,26 @@
+import { type SearchSource } from "./search-everything";
+/** The viewer this search is scoped TO. Null means signed out. */
+export interface SearchViewer {
+    readonly uid: string;
+    readonly role: string;
+    /** The store slug this viewer owns, if any. See § Store Identity. */
+    readonly storeId: string | null;
+    readonly isAdmin: boolean;
+    readonly isSeller: boolean;
+}
+/**
+ * Build the viewer's sources.
+ *
+ * 🛑 Returns `[]` for a guest, which is what makes "zero authenticated
+ * requests when signed out" structural rather than a caller's discipline.
+ */
+export declare function scopedSourcesFor(viewer: SearchViewer | null): SearchSource[];
+/**
+ * Σ `scanLimit`, for the budget check.
+ *
+ * Exported so `audit-scoped-search-ownership` R4 can print the arithmetic
+ * rather than just a verdict: "over budget" without the per-source figures
+ * tells you nothing about which source to shrink.
+ */
+export declare function scopedScanTotal(sources: readonly SearchSource[]): number;
+export declare function assertWithinScopedBudget(sources: readonly SearchSource[]): void;