@mohasinac/appkit 4.44.0 → 4.46.0

package/dist/_internal/server/jobs/core/onProductWrite.js

@@ -1,29 +1,61 @@
-import { normalizeError } from "../../../../errors/normalize";
+import { normalizeError } from "../../../../errors/normalize.js";
 /**
  * Core: keep category metrics + store stats in sync when a product document
  * is written. Only published products count.
  */
-import { categoriesRepository, storeRepository, } from "../../../../repositories";
-import { ProductStatusValues } from "../../../../features/products/schemas/firestore";
-import { PRODUCT_FIELDS } from "../../../../constants/field-names";
-import { isListingRowAvailable } from "../../../shared/listing-types/_registry";
-import { notifyRevalidate, productCacheFieldsChanged } from "./revalidateNotify";
+import { categoriesRepository, storeRepository, } from "../../../../repositories/index.js";
+import { ProductStatusValues } from "../../../../features/products/schemas/firestore.js";
+import { PRODUCT_FIELDS } from "../../../../constants/field-names.js";
+import { isListingRowAvailable } from "../../../shared/listing-types/_registry.js";
+import { notifyRevalidate, productCacheFieldsChanged } from "./revalidateNotify.js";
 const UNAVAILABLE_PRODUCT_STATUSES = new Set([
     ProductStatusValues.ARCHIVED,
 ]);
 void UNAVAILABLE_PRODUCT_STATUSES;
-function getEffectiveCategory(doc) {
+/**
+ * I3 — the ASSIGNED categories, where `own` moves.
+ *
+ * 🛑 The selection order must stay byte-identical to `describeCountable` and to
+ * `countersReconcile.tallyProduct`. All three read the same two fields in the
+ * same order, and if any one of them disagreed the nightly recount would
+ * "correct" a live count that was right — drift that is invisible precisely
+ * because the job overwrites rather than compares.
+ */
+function getEffectiveCategories(doc) {
     if (!doc)
-        return null;
+        return [];
+    const ids = doc.categoryIds;
+    if (Array.isArray(ids) && ids.length > 0)
+        return ids.filter(Boolean);
     const categorySlugs = doc.categorySlugs;
-    if (Array.isArray(categorySlugs) && categorySlugs.length > 0)
-        return categorySlugs[0];
-    return doc.category ?? null;
+    if (Array.isArray(categorySlugs) && categorySlugs.length > 0) {
+        return [categorySlugs[0]].filter(Boolean);
+    }
+    const scalar = doc.category;
+    return scalar ? [scalar] : [];
 }
-async function getParentIds(categoryId) {
-    if (!categoryId)
+/**
+ * Each assignment paired with its ancestor chain, read from the LIVE tree.
+ *
+ * 🛑 From the category documents, never from the product's stored closure —
+ * that is what makes a reparent self-heal here as well as in the nightly job.
+ * One read per assignment, in parallel, and at most
+ * `MAX_CATEGORY_ASSIGNMENTS` of them.
+ */
+async function withParents(categoryIds) {
+    if (categoryIds.length === 0)
         return [];
-    return (await categoriesRepository.findById(categoryId))?.parentIds ?? [];
+    return Promise.all(categoryIds.map(async (id) => ({
+        id,
+        parentIds: (await categoriesRepository.findById(id))?.parentIds ?? [],
+    })));
+}
+/** True when the two assignment lists differ as SETS, not as ordered arrays. */
+function assignmentsMoved(before, after) {
+    if (before.length !== after.length)
+        return true;
+    const a = new Set(after);
+    return before.some((id) => !a.has(id));
 }
 /**
  * 🛑 THIS IS A CROSS-COLLECTION TRIGGER AND THAT IS THE SAFETY ARGUMENT.
@@ -46,17 +78,21 @@ async function getParentIds(categoryId) {
  * own count equals its rollup by construction.
  */
 async function dispatchProductWriteEvent(p) {
-    const { productId, beforeCategory, afterCategory, beforeBrand, afterBrand, beforeStoreId, afterStoreId, isAuction, beforeIsAuction, wasPublished, isPublished, isDelete, ctx } = p;
-    /** Stage one category (and, if given, one brand) at `sign` × 1 item. */
-    const stage = (batch, categoryId, brandId, parentIds, auction, sign) => {
+    const { productId, beforeCategories, afterCategories, beforeBrand, afterBrand, beforeStoreId, afterStoreId, isAuction, beforeIsAuction, wasPublished, isPublished, isDelete, ctx } = p;
+    /** Stage every assignment (and, if given, one brand) at `sign` x 1 item. */
+    const stage = (batch, assignments, brandId, auction, sign) => {
         const productDelta = auction ? 0 : sign;
         const auctionDelta = auction ? sign : 0;
-        if (categoryId) {
-            categoriesRepository.updateMetricsInBatch(batch, categoryId, parentIds, productDelta, auctionDelta, productId);
-        }
-        if (brandId) {
-            categoriesRepository.updateMetricsInBatch(batch, brandId, [], productDelta, auctionDelta, productId);
-        }
+        if (assignments.length === 0 && !brandId)
+            return;
+        /*
+         * ONE call, which aggregates per document id — see
+         * `updateMetricsForAssignmentsInBatch`. Looping the single-category version
+         * instead would stage two writes on a shared ancestor, and Firestore
+         * rejects a batch containing two writes to the same document, losing every
+         * increment in it rather than just the duplicate.
+         */
+        categoriesRepository.updateMetricsForAssignmentsInBatch(batch, assignments, productDelta, auctionDelta, productId, brandId);
     };
     /*
      * `batch.update()` REJECTS a missing document, and a batch is atomic — so one
@@ -71,26 +107,26 @@ async function dispatchProductWriteEvent(p) {
         await batch.commit();
         ctx.logger.info(what, { productId, ...meta });
     };
-    if (isDelete && wasPublished && (beforeCategory || beforeBrand)) {
+    if (isDelete && wasPublished && (beforeCategories.length > 0 || beforeBrand)) {
         const batch = ctx.db.batch();
-        stage(batch, beforeCategory, beforeBrand, await getParentIds(beforeCategory ?? ""), beforeIsAuction, -1);
-        await commit(batch, "Decremented counters on hard-delete", { category: beforeCategory, brand: beforeBrand, storeId: beforeStoreId });
+        stage(batch, await withParents(beforeCategories), beforeBrand, beforeIsAuction, -1);
+        await commit(batch, "Decremented counters on hard-delete", { categories: beforeCategories, brand: beforeBrand, storeId: beforeStoreId });
         if (beforeStoreId)
             await storeRepository.incrementTotalProducts(beforeStoreId, -1);
         return;
     }
-    if (!wasPublished && isPublished && (afterCategory || afterBrand)) {
+    if (!wasPublished && isPublished && (afterCategories.length > 0 || afterBrand)) {
         const batch = ctx.db.batch();
-        stage(batch, afterCategory, afterBrand, await getParentIds(afterCategory ?? ""), isAuction, 1);
-        await commit(batch, "Incremented counters on publish", { category: afterCategory, brand: afterBrand, storeId: afterStoreId });
+        stage(batch, await withParents(afterCategories), afterBrand, isAuction, 1);
+        await commit(batch, "Incremented counters on publish", { categories: afterCategories, brand: afterBrand, storeId: afterStoreId });
         if (afterStoreId)
             await storeRepository.incrementTotalProducts(afterStoreId, 1);
         return;
     }
-    if (wasPublished && !isPublished && (beforeCategory || beforeBrand)) {
+    if (wasPublished && !isPublished && (beforeCategories.length > 0 || beforeBrand)) {
         const batch = ctx.db.batch();
-        stage(batch, beforeCategory, beforeBrand, await getParentIds(beforeCategory ?? ""), beforeIsAuction, -1);
-        await commit(batch, "Decremented counters on unpublish", { category: beforeCategory, brand: beforeBrand, storeId: beforeStoreId });
+        stage(batch, await withParents(beforeCategories), beforeBrand, beforeIsAuction, -1);
+        await commit(batch, "Decremented counters on unpublish", { categories: beforeCategories, brand: beforeBrand, storeId: beforeStoreId });
         if (beforeStoreId)
             await storeRepository.incrementTotalProducts(beforeStoreId, -1);
         return;
@@ -104,7 +140,7 @@ async function dispatchProductWriteEvent(p) {
      * through every branch and silently leave both brand rows wrong.
      */
     if (wasPublished && isPublished) {
-        const categoryMoved = beforeCategory !== afterCategory;
+        const categoryMoved = assignmentsMoved(beforeCategories, afterCategories);
         const brandMoved = beforeBrand !== afterBrand;
         /*
          * A listing can also change WHICH counter it belongs in without moving at
@@ -125,16 +161,10 @@ async function dispatchProductWriteEvent(p) {
          * correctly does not move — the item did not come or go, it changed shape.
          */
         if (!categoryMoved && !brandMoved) {
-            const parents = await getParentIds(afterCategory ?? "");
             const productDelta = isAuction ? -1 : 1;
             const auctionDelta = isAuction ? 1 : -1;
             const batch = ctx.db.batch();
-            if (afterCategory) {
-                categoriesRepository.updateMetricsInBatch(batch, afterCategory, parents, productDelta, auctionDelta, productId);
-            }
-            if (afterBrand) {
-                categoriesRepository.updateMetricsInBatch(batch, afterBrand, [], productDelta, auctionDelta, productId);
-            }
+            categoriesRepository.updateMetricsForAssignmentsInBatch(batch, await withParents(afterCategories), productDelta, auctionDelta, productId, afterBrand);
             await commit(batch, "Listing type changed", {
                 productId,
                 from: beforeIsAuction ? "auction" : "product",
@@ -143,15 +173,35 @@ async function dispatchProductWriteEvent(p) {
             return;
         }
         const [beforeParents, afterParents] = await Promise.all([
-            getParentIds(categoryMoved ? beforeCategory ?? "" : ""),
-            getParentIds(categoryMoved ? afterCategory ?? "" : ""),
+            withParents(categoryMoved ? beforeCategories : []),
+            withParents(categoryMoved ? afterCategories : []),
         ]);
+        /*
+         * TWO BATCHES, not one, and this is the I3 change that matters most.
+         *
+         * The -1 and the +1 both touch the rows that did NOT move: a listing
+         * re-filed from one leaf to a sibling shares every ancestor above them, and
+         * a product gaining a SECOND assignment keeps its first one entirely. In
+         * one batch those are two writes to the same document, which Firestore
+         * rejects outright — so the whole re-file would throw and BOTH counters
+         * would be left wrong.
+         *
+         * It worked before only because a single assignment's -1 and +1 were staged
+         * with ids that happened to be disjoint whenever the branch ran at all.
+         *
+         * Sequential commits are not atomic together: a crash between them leaves
+         * the decrement applied and the increment missing. That is the right trade,
+         * because it is self-healing — `countersReconcile` rewrites from scratch
+         * nightly and reports it as `drifted` — whereas a rejected batch is silent.
+         */
+        const outBatch = ctx.db.batch();
+        stage(outBatch, beforeParents, brandMoved ? beforeBrand : null, beforeIsAuction, -1);
+        await outBatch.commit();
         const batch = ctx.db.batch();
-        stage(batch, categoryMoved ? beforeCategory : null, brandMoved ? beforeBrand : null, beforeParents, beforeIsAuction, -1);
-        stage(batch, categoryMoved ? afterCategory : null, brandMoved ? afterBrand : null, afterParents, isAuction, 1);
+        stage(batch, afterParents, brandMoved ? afterBrand : null, isAuction, 1);
         await commit(batch, "Re-filed product", {
-            fromCategory: categoryMoved ? beforeCategory : undefined,
-            toCategory: categoryMoved ? afterCategory : undefined,
+            fromCategories: categoryMoved ? beforeCategories : undefined,
+            toCategories: categoryMoved ? afterCategories : undefined,
             fromBrand: brandMoved ? beforeBrand : undefined,
             toBrand: brandMoved ? afterBrand : undefined,
         });
@@ -195,8 +245,8 @@ export async function handleProductWrite(input, ctx) {
     // mapDoc() normalizes this on read, but a raw trigger snapshot bypasses that,
     // so a product written via categorySlugs-only paths would otherwise silently
     // no-op category metrics here. Prefer categorySlugs[0], fall back to legacy category.
-    const beforeCategory = getEffectiveCategory(before);
-    const afterCategory = getEffectiveCategory(after);
+    const beforeCategories = getEffectiveCategories(before);
+    const afterCategories = getEffectiveCategories(after);
     /*
      * `brandSlug` is the brand ROW's document id in this same `categories`
      * collection. `brand` is the display NAME and is not a key — CLAUDE.md's
@@ -238,7 +288,7 @@ export async function handleProductWrite(input, ctx) {
     const isDelete = !after;
     try {
         await dispatchProductWriteEvent({
-            productId, beforeCategory, afterCategory, beforeBrand, afterBrand,
+            productId, beforeCategories, afterCategories, beforeBrand, afterBrand,
             beforeStoreId, afterStoreId,
             isAuction, beforeIsAuction, wasPublished, isPublished, isDelete, ctx,
         });

package/dist/_internal/server/jobs/core/onReviewWrite.js

@@ -1,7 +1,7 @@
-import { notifyRevalidate } from "./revalidateNotify";
-import { reviewRepository, storeRepository } from "../../../../repositories";
-import { ReviewStatusValues } from "../../../../features/reviews/schemas/firestore";
-import { PRODUCT_COLLECTION } from "../../../../features/products/schemas/firestore";
+import { notifyRevalidate } from "./revalidateNotify.js";
+import { reviewRepository, storeRepository } from "../../../../repositories/index.js";
+import { ReviewStatusValues } from "../../../../features/reviews/schemas/firestore.js";
+import { PRODUCT_COLLECTION } from "../../../../features/products/schemas/firestore.js";
 export async function handleReviewWrite(input, ctx) {
     const { reviewId, before: beforeData, after: afterData } = input;
     /*

package/dist/_internal/server/jobs/core/onScamReportCreate.js

@@ -1,7 +1,7 @@
-import { normalizeError } from "../../../../errors/normalize";
-import { adminNotificationsRepository, userRepository } from "../../../../repositories";
-import { sendNotification } from "../../../../features/admin/actions/notification-actions";
-import { SCAM_TYPE_LABELS } from "../../../../features/scams/constants/scam-types";
+import { normalizeError } from "../../../../errors/normalize.js";
+import { adminNotificationsRepository, userRepository } from "../../../../repositories/index.js";
+import { sendNotification } from "../../../../features/admin/actions/notification-actions.js";
+import { SCAM_TYPE_LABELS } from "../../../../features/scams/constants/scam-types.js";
 export async function handleScamReportCreate(input, ctx) {
     const { scammerId, report } = input;
     const { reportedBy, displayNames, scamType, scamPlatform, amountLost } = report;

package/dist/_internal/server/jobs/core/onScamReportRejected.js

@@ -1,6 +1,6 @@
-import { normalizeError } from "../../../../errors/normalize";
-import { userRepository } from "../../../../repositories";
-import { sendNotification } from "../../../../features/admin/actions/notification-actions";
+import { normalizeError } from "../../../../errors/normalize.js";
+import { userRepository } from "../../../../repositories/index.js";
+import { sendNotification } from "../../../../features/admin/actions/notification-actions.js";
 export async function handleScamReportRejected(input, ctx) {
     const { scammerId, report } = input;
     const { reportedBy, displayNames, prevStatus, nextStatus } = report;

package/dist/_internal/server/jobs/core/onScamReportVerified.js

@@ -1,6 +1,6 @@
-import { normalizeError } from "../../../../errors/normalize";
-import { userRepository } from "../../../../repositories";
-import { sendNotification } from "../../../../features/admin/actions/notification-actions";
+import { normalizeError } from "../../../../errors/normalize.js";
+import { userRepository } from "../../../../repositories/index.js";
+import { sendNotification } from "../../../../features/admin/actions/notification-actions.js";
 export async function handleScamReportVerified(input, ctx) {
     const { scammerId, report } = input;
     const { reportedBy, displayNames, prevStatus, nextStatus } = report;

package/dist/_internal/server/jobs/core/onStoreWrite.js

@@ -2,7 +2,7 @@
  * Core: stores/{id} onWrite — currently a no-op. Previous implementation
  * synced an external search index that has since been removed.
  */
-import { notifyRevalidate } from "./revalidateNotify";
+import { notifyRevalidate } from "./revalidateNotify.js";
 export async function handleStoreWrite(input, ctx) {
     // The search-index sync this used to perform is gone (Firestore queries handle
     // search now), so the only remaining side-effect is cache invalidation: drop

package/dist/_internal/server/jobs/core/onSupportTicketCreate.js

@@ -1,5 +1,5 @@
-import { normalizeError } from "../../../../errors/normalize";
-import { sendNotification } from "../../../../features/admin/actions/notification-actions";
+import { normalizeError } from "../../../../errors/normalize.js";
+import { sendNotification } from "../../../../features/admin/actions/notification-actions.js";
 export async function handleSupportTicketCreate(input, ctx) {
     const { ticketId, ticket } = input;
     const userId = ticket.userId;

package/dist/_internal/server/jobs/core/onSupportTicketUpdate.js

@@ -1,5 +1,5 @@
-import { normalizeError } from "../../../../errors/normalize";
-import { sendNotification } from "../../../../features/admin/actions/notification-actions";
+import { normalizeError } from "../../../../errors/normalize.js";
+import { sendNotification } from "../../../../features/admin/actions/notification-actions.js";
 const USER_NOTIFY_STATUSES = new Set(["resolved", "closed", "waiting_on_user", "in_progress"]);
 const STATUS_MESSAGES = {
     resolved: {

package/dist/_internal/server/jobs/core/onUserBanChange.js

@@ -1,4 +1,4 @@
-import { normalizeError } from "../../../../errors/normalize";
+import { normalizeError } from "../../../../errors/normalize.js";
 export async function handleUserBanChange(input, ctx) {
     const { uid, before, after } = input;
     if (!after)

package/dist/_internal/server/jobs/core/pageViewPrune.js

@@ -22,8 +22,8 @@
  * @tag consumers:pageViewPrune scheduled function
  * @tag sideEffects:deletes pageViews documents
  */
-import { pageViewsRepository } from "../../../../features/analytics/repository/page-views.repository";
-import { batchDelete } from "../handlers/_helpers";
+import { pageViewsRepository } from "../../../../features/analytics/repository/page-views.repository.js";
+import { batchDelete } from "../handlers/_helpers.js";
 /**
  * 90 days.
  *

package/dist/_internal/server/jobs/core/pageViewRollup.js

@@ -17,8 +17,8 @@
  * @tag consumers:pageViewRollup scheduled function
  * @tag sideEffects:writes analytics/pageViewRollup
  */
-import { pageViewsRepository } from "../../../../features/analytics/repository/page-views.repository";
-import { pageViewRollupRepository } from "../../../../features/admin/repository/analytics-rollup.repository";
+import { pageViewsRepository } from "../../../../features/analytics/repository/page-views.repository.js";
+import { pageViewRollupRepository } from "../../../../features/admin/repository/analytics-rollup.repository.js";
 /** The earliest date this rollup will fold in on a cold start. */
 const EPOCH_DATE = "2020-01-01";
 function dayKey(offsetDays) {

package/dist/_internal/server/jobs/core/paymentReviewAutoApprove.js

@@ -1,7 +1,7 @@
-import { orderRepository } from "../../../../repositories";
-import { sendNotification } from "../../../../features/admin/actions/notification-actions";
-import { normalizeError } from "../../../../errors/normalize";
-import { ORDER_MESSAGES } from "../handlers/messages";
+import { orderRepository } from "../../../../repositories/index.js";
+import { sendNotification } from "../../../../features/admin/actions/notification-actions.js";
+import { normalizeError } from "../../../../errors/normalize.js";
+import { ORDER_MESSAGES } from "../handlers/messages.js";
 const AUTO_APPROVE_HOURS = 2;
 /**
  * Buyout orders get a full day of admin review instead of two hours.

package/dist/_internal/server/jobs/core/paymentWindowTimeout.js

@@ -1,10 +1,10 @@
-import { orderRepository } from "../../../../repositories";
-import { normalizeError } from "../../../../errors/normalize";
-import { sendNotification } from "../../../../features/admin/actions/notification-actions";
-import { restoreStockForOrder } from "../../features/checkout/stock-restore";
-import { OrderStatusValues } from "../../../../features/orders/schemas/firestore";
-import { PAYMENT_WINDOW_EXPIRED_REASON } from "../../../../features/orders/constants/payment-window";
-import { ORDER_MESSAGES } from "../handlers/messages";
+import { orderRepository } from "../../../../repositories/index.js";
+import { normalizeError } from "../../../../errors/normalize.js";
+import { sendNotification } from "../../../../features/admin/actions/notification-actions.js";
+import { restoreStockForOrder } from "../../features/checkout/stock-restore.js";
+import { OrderStatusValues } from "../../../../features/orders/schemas/firestore.js";
+import { PAYMENT_WINDOW_EXPIRED_REASON } from "../../../../features/orders/constants/payment-window.js";
+import { ORDER_MESSAGES } from "../handlers/messages.js";
 /**
  * 15-minute payment window sweep — cancels + restocks any `upi_manual`/
  * `cash`/`emi` order still `pending`/`pending` with no proof uploaded past

package/dist/_internal/server/jobs/core/pendingOrderTimeout.js

@@ -1,9 +1,9 @@
-import { orderRepository } from "../../../../repositories";
-import { normalizeError } from "../../../../errors/normalize";
-import { sendNotification } from "../../../../features/admin/actions/notification-actions";
-import { restoreStockForOrder } from "../../features/checkout/stock-restore";
-import { OrderStatusValues } from "../../../../features/orders/schemas/firestore";
-import { ORDER_MESSAGES } from "../handlers/messages";
+import { orderRepository } from "../../../../repositories/index.js";
+import { normalizeError } from "../../../../errors/normalize.js";
+import { sendNotification } from "../../../../features/admin/actions/notification-actions.js";
+import { restoreStockForOrder } from "../../features/checkout/stock-restore.js";
+import { OrderStatusValues } from "../../../../features/orders/schemas/firestore.js";
+import { ORDER_MESSAGES } from "../handlers/messages.js";
 const DEFAULT_TIMEOUT_HOURS = 24;
 /**
  * COD 24h safety net. Orders with a `paymentDeadline` set (upi_manual/cash/

package/dist/_internal/server/jobs/core/positionsReconcile.js

@@ -3,7 +3,7 @@
  * DFS pre-order from the roots. Heals drift left by the trigger's
  * `positionDirty` flag and any failed real-time updates.
  */
-import { BATCH_LIMIT } from "../handlers/messages";
+import { BATCH_LIMIT } from "../handlers/messages.js";
 const CATEGORIES = "categories";
 function dfsAssign(nodeId, childMap, counter, result) {
     const position = counter;

package/dist/_internal/server/jobs/core/priceIndexRollup.js

@@ -41,8 +41,8 @@
  * pattern that paused this project on Hobby caps. One pre-computed read per
  * row, with a visible "prices as of {date}". Deliberately one day stale.
  */
-import { orderRepository, productRepository } from "../../../../repositories";
-import { priceIndexRepository, } from "../../../../features/categories/repository/price-index.repository";
+import { orderRepository, productRepository } from "../../../../repositories/index.js";
+import { priceIndexRepository, } from "../../../../features/categories/repository/price-index.repository.js";
 /** How far back a band looks. */
 export const PRICE_INDEX_WINDOW_DAYS = 90;
 /**
@@ -100,18 +100,74 @@ function median(sorted) {
     const raw = n % 2 === 1 ? sorted[mid] : (sorted[mid - 1] + sorted[mid]) / 2;
     return Math.round(raw * 100) / 100;
 }
+/**
+ * The arithmetic mean, to the paisa.
+ *
+ * 🛑 Not a substitute for the median — see `PriceIndexBand.mean`. Both are
+ * computed and both are stored, because on a collectibles leaf one grail sale
+ * moves the mean and not the median, and the gap is the thing worth seeing.
+ */
+function mean(amounts) {
+    if (amounts.length === 0)
+        return 0;
+    const total = amounts.reduce((s, a) => s + a, 0);
+    return Math.round((total / amounts.length) * 100) / 100;
+}
+/** The newest sample, or null for an empty set. */
+function latestOf(samples) {
+    return samples.reduce((best, s) => (!best || s.at > best.at ? s : best), null);
+}
+function toConditionBand(samples) {
+    const amounts = samples.map((s) => s.amount).sort((a, b) => a - b);
+    const latest = latestOf(samples);
+    return {
+        min: amounts[0],
+        median: median(amounts),
+        max: amounts[amounts.length - 1],
+        mean: mean(amounts),
+        sampleSize: samples.length,
+        ...(latest ? { lastSold: { amount: latest.amount, at: latest.at } } : {}),
+    };
+}
 function toBand(samples) {
     const amounts = samples.map((s) => s.amount).sort((a, b) => a - b);
     const byPath = {};
     for (const s of samples)
         byPath[s.via] = (byPath[s.via] ?? 0) + 1;
-    const latest = samples.reduce((best, s) => (!best || s.at > best.at ? s : best), null);
+    const latest = latestOf(samples);
+    /*
+     * The per-condition slice.
+     *
+     * 🛑 Samples with no recorded condition are SKIPPED here while still counting
+     * toward the parent band above — their amount is a real sale, but they cannot
+     * tell us what condition sold for what. Bucketing them anywhere would invent
+     * a market.
+     *
+     * 🛑 And the key is omitted entirely rather than written as `{}` when nothing
+     * qualifies. An empty object renders as "we measured conditions and found
+     * none", which is false; absence renders as "no breakdown available", which
+     * is true. Same reasoning as `undefined` vs `0` on a count (Root Cause #72).
+     */
+    const byConditionSamples = new Map();
+    for (const s of samples) {
+        if (!s.condition)
+            continue;
+        const bucket = byConditionSamples.get(s.condition) ?? [];
+        bucket.push(s);
+        byConditionSamples.set(s.condition, bucket);
+    }
+    const byCondition = {};
+    for (const [condition, bucket] of byConditionSamples) {
+        byCondition[condition] = toConditionBand(bucket);
+    }
     return {
         min: amounts[0],
         median: median(amounts),
         max: amounts[amounts.length - 1],
+        mean: mean(amounts),
         sampleSize: samples.length,
         byPath,
+        ...(Object.keys(byCondition).length > 0 ? { byCondition } : {}),
         ...(latest ? { lastSold: { amount: latest.amount, at: latest.at, via: latest.via } } : {}),
     };
 }
@@ -141,6 +197,7 @@ export async function runPriceIndexRollup(ctx) {
     }
     const leafOf = new Map();
     const typeOf = new Map();
+    const conditionOf = new Map();
     for (const p of rows) {
         if (!p.id)
             continue;
@@ -149,6 +206,8 @@ export async function runPriceIndexRollup(ctx) {
             leafOf.set(p.id, leaf);
         if (p.listingType)
             typeOf.set(p.id, p.listingType);
+        if (p.condition)
+            conditionOf.set(p.id, p.condition);
     }
     const delivered = (await orderRepository.findByStatus("delivered"));
     /** leafId → listingType → samples */
@@ -189,8 +248,19 @@ export async function runPriceIndexRollup(ctx) {
             }
             const listingType = item.listingType ?? (item.productId ? typeOf.get(item.productId) : undefined) ?? "standard";
             const types = byLeaf.get(leaf) ?? new Map();
+            /*
+             * 🛑 The product's CURRENT condition is a fallback, not the answer.
+             *
+             * A seller who relists the same slug after refurbishing it changes
+             * `product.condition`, which would retroactively re-file an old sale
+             * under the new condition — the same class of lie as recomputing GST
+             * from a changed tax code, which is why `hsnCode`/`gstRate`/`finalSale`
+             * are all snapshotted on the order item. `condition` now is too, and
+             * the map only answers for orders written before that.
+             */
+            const condition = item.condition ?? (item.productId ? conditionOf.get(item.productId) : undefined);
             const samples = types.get(listingType) ?? [];
-            samples.push({ amount, at, via });
+            samples.push({ amount, at, via, ...(condition ? { condition } : {}) });
             types.set(listingType, samples);
             byLeaf.set(leaf, types);
             considered += 1;

package/dist/_internal/server/jobs/core/prizeDrawAssignWinner.js

@@ -1,6 +1,6 @@
 import crypto from "node:crypto";
-import { normalizeError } from "../../../../errors/normalize";
-import { sendNotification } from "../../../../features/admin/actions/notification-actions";
+import { normalizeError } from "../../../../errors/normalize.js";
+import { sendNotification } from "../../../../features/admin/actions/notification-actions.js";
 const PRODUCT_COLLECTION = "products";
 const ORDER_COLLECTION = "orders";
 /**

package/dist/_internal/server/jobs/core/prizeDrawExpiryReveal.js

@@ -1,6 +1,6 @@
-import { normalizeError } from "../../../../errors/normalize";
-import { PRODUCT_FIELDS, ORDER_FIELDS, COMMON_FIELDS } from "../../../../constants/field-names";
-import { assignPrizeDrawWinner } from "./prizeDrawAssignWinner";
+import { normalizeError } from "../../../../errors/normalize.js";
+import { PRODUCT_FIELDS, ORDER_FIELDS, COMMON_FIELDS } from "../../../../constants/field-names.js";
+import { assignPrizeDrawWinner } from "./prizeDrawAssignWinner.js";
 const PRODUCT_COLLECTION = "products";
 const ORDER_COLLECTION = "orders";
 const PRIZE_DRAW_LISTING_TYPE = "prize-draw";

package/dist/_internal/server/jobs/core/prizeDrawSoldOutReveal.js

@@ -1,6 +1,6 @@
-import { normalizeError } from "../../../../errors/normalize";
-import { PRODUCT_FIELDS, ORDER_FIELDS, COMMON_FIELDS } from "../../../../constants/field-names";
-import { assignPrizeDrawWinner } from "./prizeDrawAssignWinner";
+import { normalizeError } from "../../../../errors/normalize.js";
+import { PRODUCT_FIELDS, ORDER_FIELDS, COMMON_FIELDS } from "../../../../constants/field-names.js";
+import { assignPrizeDrawWinner } from "./prizeDrawAssignWinner.js";
 const ORDER_COLLECTION = "orders";
 function isSoldOut(doc) {
     if (!doc)

package/dist/_internal/server/jobs/core/productStatsSync.js

@@ -4,7 +4,7 @@
  *
  * Concurrency capped at 5 to avoid Firestore read-rate spikes.
  */
-import { productRepository, reviewRepository } from "../../../../repositories";
+import { productRepository, reviewRepository } from "../../../../repositories/index.js";
 const CONCURRENCY = 5;
 async function syncOne(productId) {
     const { count, avgRating } = await reviewRepository.getApprovedRatingAggregate(productId);

package/dist/_internal/server/jobs/core/productViewWindows.d.ts

@@ -0,0 +1,2 @@
+import type { JobContext } from "../runtime/types";
+export declare function runProductViewWindows(ctx: JobContext): Promise<void>;