@mohasinac/appkit 4.71.0 → 4.72.0

package/dist/features/products/constants/preorder-window.d.ts

@@ -0,0 +1,101 @@
+/**
+ * The pre-order delivery window — ONE predicate, four enforcement points.
+ *
+ * ## Why this module exists
+ *
+ * The 45-day rule has to hold at four places with three different input
+ * shapes, and every one of them used to be unbounded:
+ *
+ *   | Point                                   | Input shape          |
+ *   |-----------------------------------------|----------------------|
+ *   | `preOrderInputSchema` (create)          | ISO string           |
+ *   | `productUpdateSchema` (update)          | ISO string \| Date   |
+ *   | the seller form's section schema        | `<input>` string     |
+ *   | `preOrderRule.preflightChecks` (buy)    | stored `Date`        |
+ *
+ * Four hand-written comparisons is how a rule becomes four rules that
+ * disagree — the shape Root Cause #73 and #75 are both about. So the parse,
+ * the comparison and the message all live here and the call sites only ask for
+ * a verdict.
+ *
+ * ## 🛑 The window closes at the END of the 45th calendar day (UTC)
+ *
+ * Not at `now + 45 × 24h`. The seller picks a calendar DAY in a date control
+ * whose `max` attribute is a calendar day, so an instant-based cap would
+ * refuse the very day the control offered — `now` at 10:00 makes day 45 at
+ * 23:59 "46 days away" by instants. Comparing to the end of that day is what
+ * makes the client's `max` and the server's refusal the same rule rather than
+ * two that nearly agree.
+ *
+ * `PREORDER_MAX_DELIVERY_DAYS` lives in `_internal/shared/checkout/rules/_limits.ts`
+ * with the reason 45 is 45. Import it from there; this module deliberately
+ * does not re-export it (Root Cause #18).
+ *
+ * ## 🛑 Why `features/products/constants/` and not `_internal/shared/`
+ *
+ * The CONSUMER needs it: `src/validation/request-schemas.ts` is the schema the
+ * seller form and `POST /api/admin/products` actually parse through, so the
+ * rule has to be importable from outside appkit. `_internal/**` has no package
+ * export key (only six hand-picked paths are mapped), while `./features/*` is
+ * a wildcard — so this is the layer a shared rule can live in without adding
+ * an export key for one module. Same reasoning as the public `features/<x>/
+ * actions/` layer in CLAUDE.md's duplication framework: two layers, not two
+ * homes for one layer.
+ */
+/**
+ * What a candidate delivery date is, against the rule.
+ *
+ * 🛑 Four cases, not a boolean. A boolean collapses "you typed something that
+ * is not a date", "that date has already passed" and "that is further out than
+ * we can settle" into one message, and the seller then cannot tell which of
+ * three different fixes to apply. `absent` is separate again because an
+ * OPTIONAL field being absent is not a failure on the update path — only on
+ * create, where the caller decides.
+ */
+export type PreOrderDeliveryVerdict = "ok" | "absent" | "unparseable" | "past" | "beyond-window";
+/**
+ * The last instant a pre-order created at `now` may promise delivery.
+ *
+ * End of the 45th day in UTC — see the 🛑 in the header for why the boundary
+ * is a day and not an instant.
+ */
+export declare function preOrderDeliveryWindowEnd(now?: Date): Date;
+/**
+ * The same boundary as a `YYYY-MM-DD` string, for an `<input type="date" max>`.
+ *
+ * 🛑 Derived from `preOrderDeliveryWindowEnd`, never formatted independently.
+ * A `max` attribute computed one way and a server refusal computed another is
+ * a control that offers a value the save then rejects, which reads as a broken
+ * form rather than as a rule.
+ */
+export declare function preOrderDeliveryMaxInputValue(now?: Date): string;
+/**
+ * Anything a delivery date can be by the time it reaches a check: an ISO
+ * string from a request body, a `Date` from Firestore through `mapDoc`, a
+ * Firestore `Timestamp` on a raw read, or absent.
+ *
+ * `Timestamp` cannot be imported here — this module is client-safe and
+ * firebase-admin is not — hence the structural `{ toDate() }` arm, the same
+ * shape `_internal/shared/listing-types/availability.ts` uses.
+ */
+export type PreOrderDeliveryDateLike = string | number | Date | {
+    toDate: () => Date;
+} | null | undefined;
+/**
+ * The one comparison.
+ *
+ * `now` is a parameter rather than a `new Date()` inside, so the negative
+ * control can pin it and a test for "day 46 is refused" cannot pass by
+ * accident on a machine whose clock is a few hours off a boundary.
+ */
+export declare function checkPreOrderDeliveryDate(value: PreOrderDeliveryDateLike, now?: Date): PreOrderDeliveryVerdict;
+export declare const PREORDER_DELIVERY_BEYOND_WINDOW_MESSAGE: string;
+export declare const PREORDER_DELIVERY_PAST_MESSAGE: string;
+export declare const PREORDER_DELIVERY_UNPARSEABLE_MESSAGE = "Enter the estimated delivery date as a real calendar date.";
+export declare const PREORDER_DELIVERY_REQUIRED_MESSAGE = "An estimated delivery date is required for a pre-order.";
+export declare const PREORDER_BUYER_DELIVERY_PAST_MESSAGE: string;
+export declare const PREORDER_BUYER_DELIVERY_BEYOND_WINDOW_MESSAGE: string;
+/** The buyer-facing message for a verdict, or `null` when it is acceptable. */
+export declare function preOrderDeliveryBuyerMessage(verdict: PreOrderDeliveryVerdict): string | null;
+/** The seller-facing message for a verdict, or `null` when it is acceptable. */
+export declare function preOrderDeliveryMessage(verdict: PreOrderDeliveryVerdict): string | null;

package/dist/features/products/constants/preorder-window.js

@@ -0,0 +1,172 @@
+/**
+ * The pre-order delivery window — ONE predicate, four enforcement points.
+ *
+ * ## Why this module exists
+ *
+ * The 45-day rule has to hold at four places with three different input
+ * shapes, and every one of them used to be unbounded:
+ *
+ *   | Point                                   | Input shape          |
+ *   |-----------------------------------------|----------------------|
+ *   | `preOrderInputSchema` (create)          | ISO string           |
+ *   | `productUpdateSchema` (update)          | ISO string \| Date   |
+ *   | the seller form's section schema        | `<input>` string     |
+ *   | `preOrderRule.preflightChecks` (buy)    | stored `Date`        |
+ *
+ * Four hand-written comparisons is how a rule becomes four rules that
+ * disagree — the shape Root Cause #73 and #75 are both about. So the parse,
+ * the comparison and the message all live here and the call sites only ask for
+ * a verdict.
+ *
+ * ## 🛑 The window closes at the END of the 45th calendar day (UTC)
+ *
+ * Not at `now + 45 × 24h`. The seller picks a calendar DAY in a date control
+ * whose `max` attribute is a calendar day, so an instant-based cap would
+ * refuse the very day the control offered — `now` at 10:00 makes day 45 at
+ * 23:59 "46 days away" by instants. Comparing to the end of that day is what
+ * makes the client's `max` and the server's refusal the same rule rather than
+ * two that nearly agree.
+ *
+ * `PREORDER_MAX_DELIVERY_DAYS` lives in `_internal/shared/checkout/rules/_limits.ts`
+ * with the reason 45 is 45. Import it from there; this module deliberately
+ * does not re-export it (Root Cause #18).
+ *
+ * ## 🛑 Why `features/products/constants/` and not `_internal/shared/`
+ *
+ * The CONSUMER needs it: `src/validation/request-schemas.ts` is the schema the
+ * seller form and `POST /api/admin/products` actually parse through, so the
+ * rule has to be importable from outside appkit. `_internal/**` has no package
+ * export key (only six hand-picked paths are mapped), while `./features/*` is
+ * a wildcard — so this is the layer a shared rule can live in without adding
+ * an export key for one module. Same reasoning as the public `features/<x>/
+ * actions/` layer in CLAUDE.md's duplication framework: two layers, not two
+ * homes for one layer.
+ */
+import { PREORDER_MAX_DELIVERY_DAYS } from "../../../_internal/shared/checkout/rules/_limits.js";
+const DAY_MS = 24 * 60 * 60 * 1000;
+/**
+ * The last instant a pre-order created at `now` may promise delivery.
+ *
+ * End of the 45th day in UTC — see the 🛑 in the header for why the boundary
+ * is a day and not an instant.
+ */
+export function preOrderDeliveryWindowEnd(now = new Date()) {
+    const end = new Date(now.getTime() + PREORDER_MAX_DELIVERY_DAYS * DAY_MS);
+    end.setUTCHours(23, 59, 59, 999);
+    return end;
+}
+/**
+ * The same boundary as a `YYYY-MM-DD` string, for an `<input type="date" max>`.
+ *
+ * 🛑 Derived from `preOrderDeliveryWindowEnd`, never formatted independently.
+ * A `max` attribute computed one way and a server refusal computed another is
+ * a control that offers a value the save then rejects, which reads as a broken
+ * form rather than as a rule.
+ */
+export function preOrderDeliveryMaxInputValue(now = new Date()) {
+    return preOrderDeliveryWindowEnd(now).toISOString().slice(0, 10);
+}
+function coerce(value) {
+    if (value === null || value === undefined || value === "")
+        return null;
+    if (value instanceof Date)
+        return Number.isNaN(value.getTime()) ? null : value;
+    if (typeof value === "object" && typeof value.toDate === "function") {
+        const d = value.toDate();
+        return d instanceof Date && !Number.isNaN(d.getTime()) ? d : null;
+    }
+    const d = new Date(value);
+    return Number.isNaN(d.getTime()) ? null : d;
+}
+/**
+ * The one comparison.
+ *
+ * `now` is a parameter rather than a `new Date()` inside, so the negative
+ * control can pin it and a test for "day 46 is refused" cannot pass by
+ * accident on a machine whose clock is a few hours off a boundary.
+ */
+export function checkPreOrderDeliveryDate(value, now = new Date()) {
+    if (value === null || value === undefined || value === "")
+        return "absent";
+    const date = coerce(value);
+    if (!date)
+        return "unparseable";
+    if (date.getTime() <= now.getTime())
+        return "past";
+    if (date.getTime() > preOrderDeliveryWindowEnd(now).getTime())
+        return "beyond-window";
+    return "ok";
+}
+/*
+ * 🛑 There is deliberately no `isWritablePreOrderDeliveryDate(value): boolean`
+ * convenience wrapper. One was written and deleted: a boolean cannot carry
+ * WHICH of the four verdicts applied, so the two write schemas would have had
+ * to call `checkPreOrderDeliveryDate` anyway to pick a message — and an
+ * exported helper with no caller is the speculative-infra pattern. Call sites
+ * compare the verdict; the two they accept (`ok`, `absent`) are three
+ * characters longer and say what they mean.
+ */
+/*
+ * ── Messages ────────────────────────────────────────────────────────────────
+ *
+ * 🛑 These name the LIMIT and its CONSEQUENCE, in that order, because the
+ * consequence is the part a seller will otherwise discover as a cancelled
+ * order three weeks later. They interpolate the constant rather than quoting
+ * "45", so a gateway change updates the message instead of contradicting it.
+ */
+export const PREORDER_DELIVERY_BEYOND_WINDOW_MESSAGE = `A pre-order must promise delivery within ${PREORDER_MAX_DELIVERY_DAYS} days of purchase. ` +
+    `Pick a date no more than ${PREORDER_MAX_DELIVERY_DAYS} days away, or list this as an ordinary ` +
+    `product once you can ship it — a pre-order that overruns the window is cancelled and refunded ` +
+    `automatically.`;
+export const PREORDER_DELIVERY_PAST_MESSAGE = "The estimated delivery date has already passed. Set a future date you can meet — " +
+    "a pre-order is cancelled and refunded automatically once its promised date goes by.";
+export const PREORDER_DELIVERY_UNPARSEABLE_MESSAGE = "Enter the estimated delivery date as a real calendar date.";
+export const PREORDER_DELIVERY_REQUIRED_MESSAGE = "An estimated delivery date is required for a pre-order.";
+/*
+ * ── Buyer-facing messages ───────────────────────────────────────────────────
+ *
+ * 🛑 SEPARATE from the seller's, deliberately. The seller's message tells them
+ * which fix to apply ("pick a date no more than 45 days away"), and a buyer
+ * refused at checkout can do none of those things — handing them the seller's
+ * instructions reads as the site blaming them for someone else's listing.
+ * These say what happened and what they can do: wait, or ask the seller.
+ */
+export const PREORDER_BUYER_DELIVERY_PAST_MESSAGE = "Its promised delivery date has already passed, so it is no longer taking pre-orders. " +
+    "The seller needs to set a new date before it can be bought.";
+export const PREORDER_BUYER_DELIVERY_BEYOND_WINDOW_MESSAGE = `Its promised delivery date is more than ${PREORDER_MAX_DELIVERY_DAYS} days away, which we cannot ` +
+    `accept payment for — a pre-order has to be deliverable within ${PREORDER_MAX_DELIVERY_DAYS} days. ` +
+    `The seller needs to bring the date forward before it can be bought.`;
+/** The buyer-facing message for a verdict, or `null` when it is acceptable. */
+export function preOrderDeliveryBuyerMessage(verdict) {
+    switch (verdict) {
+        case "past":
+            return PREORDER_BUYER_DELIVERY_PAST_MESSAGE;
+        case "beyond-window":
+            return PREORDER_BUYER_DELIVERY_BEYOND_WINDOW_MESSAGE;
+        /*
+         * A buyer is never shown anything for these three. `ok` is fine; `absent`
+         * and `unparseable` are deliberately NOT refused at checkout (see the
+         * comment in `preorder.rule.ts`), so a message for them would be a
+         * sentence with no surface to appear on.
+         */
+        case "ok":
+        case "absent":
+        case "unparseable":
+            return null;
+    }
+}
+/** The seller-facing message for a verdict, or `null` when it is acceptable. */
+export function preOrderDeliveryMessage(verdict) {
+    switch (verdict) {
+        case "ok":
+            return null;
+        case "absent":
+            return PREORDER_DELIVERY_REQUIRED_MESSAGE;
+        case "unparseable":
+            return PREORDER_DELIVERY_UNPARSEABLE_MESSAGE;
+        case "past":
+            return PREORDER_DELIVERY_PAST_MESSAGE;
+        case "beyond-window":
+            return PREORDER_DELIVERY_BEYOND_WINDOW_MESSAGE;
+    }
+}

package/dist/features/products/constants/product-features.constants.js

@@ -13,16 +13,16 @@ export const PRODUCT_FEATURE_CATEGORY_OPTIONS = [
     { value: "seller", label: "Seller" },
     { value: "condition", label: "Condition" },
     { value: "platform", label: "Platform" },
-    { value: "auction", label: "Auction" },
+    { value: "collector-drop", label: "Collector Drop" },
     { value: "preorder", label: "Pre-order" },
     { value: "custom", label: "Custom" },
 ];
 export const PRODUCT_FEATURE_PRODUCT_TYPE_OPTIONS = [
     { value: "all", label: "All listing types" },
     { value: "product", label: "Standard product" },
-    { value: "auction", label: "Auction" },
+    { value: "collector-drop", label: "Collector Drop" },
     { value: "preorder", label: "Pre-order" },
-    { value: "prize-draw", label: "Prize draw" },
+    { value: "clearance", label: "Clearance" },
     { value: "classified", label: "Classified" },
     { value: "digital-code", label: "Digital code" },
     { value: "live", label: "Live item" },

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

@@ -77,7 +77,7 @@ export declare const STANDARD_PUBLIC_SORT_OPTIONS: readonly [{
     readonly value: string;
     readonly label: "Name: Z–A";
 }];
-export declare const AUCTION_SORT_OPTIONS: readonly [{
+export declare const COLLECTOR_DROP_SORT_OPTIONS: readonly [{
     readonly value: string;
     readonly label: "Ending Soon";
 }, {
@@ -105,7 +105,7 @@ export declare const AUCTION_SORT_OPTIONS: readonly [{
     readonly value: string;
     readonly label: "Oldest Listed";
 }];
-export declare const AUCTION_PUBLIC_SORT_OPTIONS: readonly [{
+export declare const COLLECTOR_DROP_PUBLIC_SORT_OPTIONS: readonly [{
     readonly value: string;
     readonly label: "Ending Soon";
 }, {
@@ -246,7 +246,7 @@ export declare const BUNDLE_SORT_OPTIONS: readonly [...({
     readonly value: string;
     readonly label: "Price: High to Low";
 }];
-export declare const PRIZE_DRAW_SORT_OPTIONS: readonly [{
+export declare const CLEARANCE_SORT_OPTIONS: readonly [{
     readonly value: string;
     readonly label: "Newest First";
 }, {
@@ -266,7 +266,7 @@ export declare const PRIZE_DRAW_SORT_OPTIONS: readonly [{
     readonly label: "Entry: High to Low";
 }];
 /** Public prize-draw browse subset — every option is buyer-meaningful. */
-export declare const PRIZE_DRAW_PUBLIC_SORT_OPTIONS: readonly [{
+export declare const CLEARANCE_PUBLIC_SORT_OPTIONS: readonly [{
     readonly value: string;
     readonly label: "Newest First";
 }, {

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

@@ -6,7 +6,7 @@ const LABEL_NEWEST = "Newest First";
 const LABEL_OLDEST = "Oldest First";
 // ---------------------------------------------------------------------------
 // W2-6 — Base sort options shared by standard / pre-order / bundle / prize-draw.
-// Auctions get their own base (auctionEndDate primary) because the "Ending Soon"
+// Collector Drops get their own base (collectorDropEndDate primary) because the "Ending Soon"
 // dimension dominates browse intent.
 // ---------------------------------------------------------------------------
 const BASE_TIME_SORTS = [
@@ -49,7 +49,7 @@ export function withoutPriceSorts(options, canSeePrices) {
  * `price` is: "Highest Current Bid" ranks every hidden bid on the page, which
  * is the disclosure the gate exists to prevent. `bidCount` is deliberately
  * ABSENT — the number of bids is not an amount, it is already displayed
- * ungated beside every auction, and hiding it would remove a real signal for
+ * ungated beside every collector drop, and hiding it would remove a real signal for
  * no gain.
  */
 const MONEY_SORT_FIELDS = [
@@ -113,10 +113,10 @@ export const STANDARD_PUBLIC_SORT_OPTIONS = [
      */
 ];
 // ---------------------------------------------------------------------------
-// Auctions — auction-specific base puts auctionEndDate primary
+// Collector Drops — collector-drop-specific base puts collectorDropEndDate primary
 // ---------------------------------------------------------------------------
-export const AUCTION_SORT_OPTIONS = [
-    { value: sortBy(PRODUCT_FIELDS.AUCTION_END_DATE, "ASC"), label: "Ending Soon" },
+export const COLLECTOR_DROP_SORT_OPTIONS = [
+    { value: sortBy(PRODUCT_FIELDS.COLLECTOR_DROP_END_DATE, "ASC"), label: "Ending Soon" },
     { value: sortBy(PRODUCT_FIELDS.CREATED_AT), label: "Just Started" },
     { value: sortBy(PRODUCT_FIELDS.CURRENT_BID, "ASC"), label: "Lowest Current Bid" },
     { value: sortBy(PRODUCT_FIELDS.CURRENT_BID), label: "Highest Current Bid" },
@@ -126,13 +126,13 @@ export const AUCTION_SORT_OPTIONS = [
     { value: sortBy(PRODUCT_FIELDS.BUY_NOW_PRICE, "ASC"), label: "Buy It Now: Low–High" },
     { value: sortBy(PRODUCT_FIELDS.CREATED_AT, "ASC"), label: "Oldest Listed" },
 ];
-export const AUCTION_PUBLIC_SORT_OPTIONS = [
-    AUCTION_SORT_OPTIONS[0],
-    AUCTION_SORT_OPTIONS[1],
-    AUCTION_SORT_OPTIONS[2],
-    AUCTION_SORT_OPTIONS[3],
-    AUCTION_SORT_OPTIONS[5],
-    AUCTION_SORT_OPTIONS[7],
+export const COLLECTOR_DROP_PUBLIC_SORT_OPTIONS = [
+    COLLECTOR_DROP_SORT_OPTIONS[0],
+    COLLECTOR_DROP_SORT_OPTIONS[1],
+    COLLECTOR_DROP_SORT_OPTIONS[2],
+    COLLECTOR_DROP_SORT_OPTIONS[3],
+    COLLECTOR_DROP_SORT_OPTIONS[5],
+    COLLECTOR_DROP_SORT_OPTIONS[7],
 ];
 // ---------------------------------------------------------------------------
 // Pre-Orders
@@ -172,7 +172,7 @@ export const BUNDLE_SORT_OPTIONS = [
 // ---------------------------------------------------------------------------
 // Prize Draws
 // ---------------------------------------------------------------------------
-export const PRIZE_DRAW_SORT_OPTIONS = [
+export const CLEARANCE_SORT_OPTIONS = [
     ...BASE_TIME_SORTS,
     { value: sortBy(PRODUCT_FIELDS.PRIZE_REVEAL_WINDOW_START, "ASC"), label: "Reveal: Soonest" },
     { value: sortBy(PRODUCT_FIELDS.PRIZE_REVEAL_WINDOW_START), label: "Reveal: Furthest" },
@@ -180,7 +180,7 @@ export const PRIZE_DRAW_SORT_OPTIONS = [
     { 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;
+export const CLEARANCE_PUBLIC_SORT_OPTIONS = CLEARANCE_SORT_OPTIONS;
 // ---------------------------------------------------------------------------
 // Lookup by listing type
 // ---------------------------------------------------------------------------

package/dist/features/products/hooks/useProducts.js

@@ -44,7 +44,7 @@ export function useProducts(params = {}, opts) {
      * derived from the other, so they could not be kept in step by anything.
      * Every client-side fetch therefore arrived with a sort the server could not
      * see and fell back to `-createdAt`, while the dropdown displayed the sort
-     * the user had picked. Measured: `?sorts=auctionEndDate` returned byte-identical
+     * the user had picked. Measured: `?sorts=collectorDropEndDate` returned byte-identical
      * row order to sending no sort at all.
      *
      * "We have shared constants and it still broke" is exactly this: the

package/dist/features/products/hooks/useRelatedProducts.d.ts

@@ -1,7 +1,7 @@
 import type { ProductListResponse } from "../types";
 /**
- * Fetch related products in the same category. SB1-G — `isAuction` legacy
+ * Fetch related products in the same category. SB1-G — `isCollectorDrop` legacy
  * boolean translates to the canonical `listingType` clause; new callers can
  * pass the union directly via the same parameter slot in a follow-up.
  */
-export declare function useRelatedProducts(category: string, excludeId: string, limit?: number, isAuction?: boolean): import("@tanstack/react-query").UseQueryResult<ProductListResponse, Error>;
+export declare function useRelatedProducts(category: string, excludeId: string, limit?: number, isCollectorDrop?: boolean): import("@tanstack/react-query").UseQueryResult<ProductListResponse, Error>;

package/dist/features/products/hooks/useRelatedProducts.js

@@ -4,13 +4,13 @@ import { useQuery } from "@tanstack/react-query";
 import { apiClient } from "../../../http/index.js";
 import { PRODUCT_ENDPOINTS } from "../../../constants/api-endpoints.js";
 /**
- * Fetch related products in the same category. SB1-G — `isAuction` legacy
+ * Fetch related products in the same category. SB1-G — `isCollectorDrop` legacy
  * boolean translates to the canonical `listingType` clause; new callers can
  * pass the union directly via the same parameter slot in a follow-up.
  */
-export function useRelatedProducts(category, excludeId, limit = 8, isAuction = false) {
-    const listingTypeClause = isAuction
-        ? sieveFilter("listingType", SIEVE_OP.EQ, "auction")
+export function useRelatedProducts(category, excludeId, limit = 8, isCollectorDrop = false) {
+    const listingTypeClause = isCollectorDrop
+        ? sieveFilter("listingType", SIEVE_OP.EQ, "collector-drop")
         : sieveFilter("listingType", SIEVE_OP.EQ, "standard");
     const params = new URLSearchParams({
         filters: sieveAnd(sieveFilter("status", SIEVE_OP.EQ, "published"), sieveFilter("category", SIEVE_OP.EQ, encodeURIComponent(category)), listingTypeClause),
@@ -18,7 +18,7 @@ export function useRelatedProducts(category, excludeId, limit = 8, isAuction = f
         pageSize: String(limit),
     });
     return useQuery({
-        queryKey: ["related-products", category, excludeId, String(isAuction)],
+        queryKey: ["related-products", category, excludeId, String(isCollectorDrop)],
         queryFn: () => apiClient.get(`${PRODUCT_ENDPOINTS.LIST}?${params.toString()}`),
         enabled: Boolean(category),
         staleTime: 5 * 60 * 1000,

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

@@ -10,5 +10,5 @@ export * from "./components";
 export * from "./schemas";
 export * from "./columns";
 export { manifest } from "./manifest";
-export { normalizeListingType, isAuctionListing, isPreOrderListing, isStandardListing, isPrizeDrawListing, isClassifiedListing, isDigitalCodeListing, isLiveListing, isArtListing, isStickersListing, } from "./utils/listing-type";
+export { normalizeListingType, isCollectorDropListing, isPreOrderListing, isStandardListing, isClearanceListing, isClassifiedListing, isDigitalCodeListing, isLiveListing, isArtAndStickersListing, } from "./utils/listing-type";
 export { sanitizeProductForPublic, sanitizeProductsForPublic, } from "./utils/sanitize";

package/dist/features/products/index.js

@@ -10,9 +10,9 @@ export * from "./components/index.js";
 export * from "./schemas/index.js";
 export * from "./columns/index.js";
 export { manifest } from "./manifest.js";
-export { normalizeListingType, isAuctionListing, isPreOrderListing, isStandardListing, isPrizeDrawListing, 
+export { normalizeListingType, isCollectorDropListing, isPreOrderListing, isStandardListing, isClearanceListing, 
 // SB-UNI-F 2026-05-13 — Phase 2 predicates.
 isClassifiedListing, isDigitalCodeListing, isLiveListing, 
-// Art/stickers session.
-isArtListing, isStickersListing, } from "./utils/listing-type.js";
+// Printed-only physical goods — one merged type (1.34).
+isArtAndStickersListing, } from "./utils/listing-type.js";
 export { sanitizeProductForPublic, sanitizeProductsForPublic, } from "./utils/sanitize.js";

package/dist/features/products/messages/en.json

@@ -3,7 +3,7 @@
   "noProducts": "No products found",
   "addToWishlist": "Add to Wishlist",
   "removeFromWishlist": "Remove from Wishlist",
-  "auction": "Auction",
+  "collectorDrop": "Collector Drop",
   "sortNewest": "Newest",
   "sortPriceLow": "Price: Low to High",
   "sortPriceHigh": "Price: High to Low",

package/dist/features/products/repository/category-assignment.js

@@ -47,7 +47,7 @@
  * @tag sideEffects:none
  */
 import { ConflictError, ValidationError } from "../../../errors/index.js";
-import { LISTING_TYPE_HAS_COMPOSITION } from "../../parts/schemas/firestore.js";
+import { LISTING_TYPE_HAS_COMPOSITION } from "../schemas/part-slots.js";
 /**
  * 🛑 A CAP, because the closure is `array-contains`-indexed and Firestore's
  * `array-contains-any` takes at most 30 comparison values.
@@ -284,7 +284,7 @@ export function assertAssignmentShape(resolved, isLot, opts = {}) {
          * 409, not 400: the request is well formed and the seller is not confused —
          * OUR category tree has not been rebuilt, so we cannot tell whether that
          * node is a leaf. The correct next action is to rebuild the tree, not to
-         * fix a field. Same distinction as removing a booked lottery slot.
+         * fix a field. Same distinction as removing a booked giveaway slot.
          */
         throw new ConflictError(`"${notIndexed.name ?? notIndexed.id}" has not been indexed yet, so we cannot tell whether it is a ` +
             `single-item category. Rebuild the category tree and try again.`);

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

@@ -59,9 +59,12 @@ import type { ProductStatus } from "../types";
 export declare function resolveStockResync(current: ProductDocument | null, data: Partial<ProductDocument>): Partial<ProductDocument>;
 export declare class ProductRepository extends BaseRepository<ProductDocument> {
     private static readonly CACHE_TTL_MS;
+    private static readonly CACHE_PREFIX;
     constructor();
     private cacheKeyById;
     private cacheKeyBySlug;
+    /** Item 8 §18 (overhaul brief) — the one dynamic, per-id/per-slug keyed cache today. */
+    cachePrefix(): string;
     private cacheSet;
     private cacheInvalidateForId;
     findById(id: string): Promise<ProductDocument | null>;
@@ -260,7 +263,7 @@ export declare class ProductRepository extends BaseRepository<ProductDocument> {
      * (CompareOverlay) treat the returned array as "what we could find".
      */
     listByIds(ids: string[]): Promise<ProductDocument[]>;
-    findAuctions(): Promise<ProductDocument[]>;
+    findCollectorDrops(): Promise<ProductDocument[]>;
     findPreOrders(): Promise<ProductDocument[]>;
     findPromoted(): Promise<ProductDocument[]>;
     updateProduct(productId: string, data: ProductUpdateInput): Promise<ProductDocument>;
@@ -514,7 +517,7 @@ export declare class ProductRepository extends BaseRepository<ProductDocument> {
             canSort: boolean;
             parseValue: typeof parseSieveDateValue;
         };
-        auctionEndDate: {
+        collectorDropEndDate: {
             canFilter: boolean;
             canSort: boolean;
             parseValue: typeof parseSieveDateValue;
@@ -668,7 +671,7 @@ export declare class ProductRepository extends BaseRepository<ProductDocument> {
         };
     };
     /**
-     * Virtual filter aliases — callers can pass `f=listingType==auction` for
+     * Virtual filter aliases — callers can pass `f=listingType==collector-drop` for
      * the canonical listing-kind discriminator. The Sieve adapter expands the
      * alias into real clauses before reaching Firestore.
      *
@@ -715,10 +718,10 @@ export declare class ProductRepository extends BaseRepository<ProductDocument> {
         brandName?: string;
     }): Promise<Record<string, number | undefined>>;
     /**
-     * Cloud Functions: find published auctions whose end date has already passed.
-     * Used by the auction settlement job to finalize expired auctions.
+     * Cloud Functions: find published collector drops whose end date has already passed.
+     * Used by the collector drop settlement job to finalize expired collector drops.
      */
-    getExpiredAuctions(now: Date): Promise<Array<{
+    getExpiredCollectorDrops(now: Date): Promise<Array<{
         id: string;
         ref: DocumentReference;
         data: ProductDocument;