@mohasinac/appkit 4.44.0 → 4.45.0

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

@@ -1,35 +1,35 @@
-import { normalizeError } from "../../../errors/normalize";
-import { increment, serverTimestamp } from "../../../contracts/field-ops";
-import { DatabaseError } from "../../../errors";
-import { BaseRepository, prepareForFirestore, parseSieveDateValue, } from "../../../providers/db-firebase";
-import { cacheManager } from "../../../core";
-import { serverLogger } from "../../../monitoring";
-import { generateUniqueId, slugify, generateBarcodeId } from "../../../utils";
-import { parseSearchTxtQuery } from "../../../utils/search-txt";
+import { normalizeError } from "../../../errors/normalize.js";
+import { increment, serverTimestamp } from "../../../contracts/field-ops.js";
+import { DatabaseError } from "../../../errors/index.js";
+import { BaseRepository, prepareForFirestore, parseSieveDateValue, } from "../../../providers/db-firebase/index.js";
+import { cacheManager } from "../../../core/index.js";
+import { serverLogger } from "../../../monitoring/index.js";
+import { generateUniqueId, slugify, generateBarcodeId } from "../../../utils/index.js";
+import { parseSearchTxtQuery } from "../../../utils/search-txt.js";
 // The shared search plan/refine the other eleven migrated collections use.
 // `planSearchTxt` carries the `empty` guard that keeps a punctuation-only query
 // from returning the whole catalogue.
-import { planSearchTxt, refineSearchTxt, emptySearchResult, } from "../../../utils/search-txt-query";
+import { planSearchTxt, SEARCH_REFINE_WINDOW, refineSearchTxt, emptySearchResult, } from "../../../utils/search-txt-query.js";
 // One definition of "which fields feed searchTxt", shared with the seed
 // wrapper. Three copies existed and two were missing the same three fields.
-import { buildProductSearchTxt } from "../../../utils/search-txt-builders";
-import { PRODUCT_COLLECTION, ProductStatusValues } from "../schemas";
+import { buildProductSearchTxt } from "../../../utils/search-txt-builders.js";
+import { PRODUCT_COLLECTION, ProductStatusValues } from "../schemas/index.js";
 // Collection NAMES only — deliberately not `categoriesRepository` or
 // `taxCodesRepository`. See deriveTaxonomy()'s header for why these are raw
 // reads: no repository in this codebase imports another feature's, because of
 // the Turbopack import-chain trap (Root Cause #6/#18).
-import { CATEGORIES_COLLECTION } from "../../categories/schemas/firestore";
-import { TAX_CODES_COLLECTION } from "../../tax-codes/schemas/firestore";
+import { CATEGORIES_COLLECTION } from "../../categories/schemas/firestore.js";
+import { TAX_CODES_COLLECTION } from "../../tax-codes/schemas/firestore.js";
 // Collection id only, never the stores repository — no repository in this
 // codebase imports another feature's (the Turbopack import-chain trap).
-import { STORE_COLLECTION } from "../../stores/schemas/firestore";
+import { STORE_COLLECTION } from "../../stores/schemas/firestore.js";
 // Pure money math — no Firestore, no server-only imports.
-import { DEFAULT_GST_RATE, grossFromTaxable } from "../../../_internal/shared/fees/calculator";
-import { MAX_FEATURES_PER_PRODUCT } from "../schemas/product-features";
+import { DEFAULT_GST_RATE, grossFromTaxable } from "../../../_internal/shared/fees/calculator.js";
+import { MAX_FEATURES_PER_PRODUCT } from "../schemas/product-features.js";
 // The ONE `{{placeholder}}` implementation, promoted out of the FAQ schema in
 // B3. A second regex here would be Root Cause #75's shape.
-import { interpolate } from "../../../_internal/shared/templating/placeholders";
-import { PRODUCT_FIELDS } from "../../../constants/field-names";
+import { interpolate } from "../../../_internal/shared/templating/placeholders.js";
+import { PRODUCT_FIELDS } from "../../../constants/field-names.js";
 /**
  * Canonical listing-kind tokens used in `listingType`. Note: the public Sieve
  * alias accepts the legacy "preorder" / "product" spelling; we normalise to
@@ -103,6 +103,78 @@ function interpolateDefaults(template, input, derived) {
         siteName: "LetItRip",
     }).text;
 }
+/**
+ * Keep `availableQuantity` in step when a seller edits `stockQuantity`.
+ *
+ * 🛑 THE BUG THIS CLOSES: A SELLER COULD NOT RESTOCK.
+ *
+ * There are two stock numbers. `stockQuantity` is seller-authored and is the
+ * only one any of the three product forms exposes. `availableQuantity` is
+ * system-owned, seeded from `stockQuantity` **once, at create**, and
+ * thereafter driven by checkout decrements and timeout restores.
+ *
+ * Nothing re-synced them. `updateProduct` is a bare spread, and grep for
+ * `availableQuantity` across `src/app/api/store/products/**` and
+ * `seller-actions.ts` returns ZERO. So: a product sells out
+ * (`availableQuantity: 0`), the seller edits `stockQuantity` 0 → 10, and the
+ * product STAYS UNBUYABLE — purchasability gates on `availableQuantity > 0`
+ * in three independent places (`service.ts`, the cart route, and the public
+ * list filter). The only field the seller can reach is the one that does not
+ * control availability.
+ *
+ * With 76–93% of competitor catalogues sold out, restock is the main
+ * lifecycle event, not an edge case.
+ *
+ * WHY A DELTA AND NOT `availableQuantity = stockQuantity`: the difference
+ * between the two IS the outstanding reservation — units held by placed-but-
+ * unpaid orders. Overwriting would hand those units back to the shelf and
+ * oversell them. So carry the reservation forward:
+ *
+ *     reserved = prevStock - prevAvailable        (clamped at 0)
+ *     available = nextStock - reserved            (clamped at 0)
+ *
+ * Examples: 10 stock / 10 available, edit to 12 → 12. 10 stock / 7 available
+ * (3 reserved), edit to 12 → 9. Sold out at 0/0, edit to 10 → 10.
+ *
+ * 🛑 Returns `{}` — not a value — whenever the write does not name
+ * `stockQuantity`, or names `availableQuantity` explicitly. The second case
+ * is what keeps the checkout decrement, the stock-restore job and
+ * `updateAvailableQuantity` authoritative: they write `availableQuantity`
+ * directly and must never have it recomputed from a stale `stockQuantity`
+ * underneath them.
+ *
+ * 🛑 EXPORTED, and not merely for tidiness. A BULK stock edit cannot go
+ * through `update` — that is 50 sequential round trips, and Rule #6 budgets a
+ * request at ~3 — so it writes through `batchUpdateEach`. But the delta is
+ * PER DOCUMENT (each product carries its own reservation), which means a bulk
+ * route has to apply this rule itself. Re-deriving it there is Root Cause
+ * #75's shape on the one field whose whole history is that nothing re-synced
+ * it. So the rule is exported and the bulk route calls THIS.
+ *
+ * 🛑 It cannot become a `BaseRepository` write hook, which was the first
+ * thing I tried. A hook receives `{...doc, ...patch}`, where `stockQuantity`
+ * is already the NEXT value — so `prevStock` is gone and `reserved` is
+ * uncomputable. The rule genuinely needs both documents.
+ */
+export function resolveStockResync(current, data) {
+    if (!current)
+        return {};
+    if (!("stockQuantity" in data))
+        return {};
+    // An explicit availableQuantity write wins — see the 🛑 above.
+    if ("availableQuantity" in data)
+        return {};
+    const nextStock = data.stockQuantity;
+    if (typeof nextStock !== "number" || !Number.isFinite(nextStock))
+        return {};
+    const prevStock = typeof current.stockQuantity === "number" ? current.stockQuantity : 0;
+    const prevAvailable = typeof current.availableQuantity === "number" ? current.availableQuantity : prevStock;
+    const reserved = Math.max(0, prevStock - prevAvailable);
+    const nextAvailable = Math.max(0, nextStock - reserved);
+    if (nextAvailable === prevAvailable)
+        return {};
+    return { availableQuantity: nextAvailable };
+}
 export class ProductRepository extends BaseRepository {
     constructor() {
         super(PRODUCT_COLLECTION);
@@ -419,6 +491,24 @@ export class ProductRepository extends BaseRepository {
             "seoTitleTemplate",
             "seoDescriptionTemplate",
             "seoKeywords",
+            /*
+             * C5a — the PACKED parcel figures.
+             *
+             * 🛑 These are resolved up the ancestor chain like everything else here,
+             * and the field they fill is deliberately OPTIONAL on the product: the
+             * figure a carrier bills on is the packed box, not the item, and a bey
+             * is ~30 g against ~300 g boxed. A required product weight would
+             * therefore be an ITEM weight, systematically under-estimating every
+             * booking — and because the booking is capped at the amount collected
+             * from the buyer, systematic under-estimation means silently absorbing
+             * the difference.
+             *
+             * So the category's packed default drives the checkout ESTIMATE, this
+             * fills it when the leaf declares one, the product may refine it, and
+             * the real parcel figure is entered at shipment time.
+             */
+            "weightG",
+            "dimensionsCm",
         ];
         const isEmpty = (v) => v == null || (Array.isArray(v) && v.length === 0);
         const unresolved = () => WANTED.filter((k) => isEmpty(resolved[k]));
@@ -512,6 +602,22 @@ export class ProductRepository extends BaseRepository {
         if (resolved.defaultAuthenticity && !input.authenticity) {
             out.authenticity = resolved.defaultAuthenticity;
         }
+        /*
+         * C5a — fill-blank, never overwrite. A seller who weighed their own packed
+         * box has measured something the category only estimates.
+         *
+         * 🛑 `input.weightG == null`, not `!input.weightG`. A zero is nonsense for
+         * a parcel but it is an explicit value, and the falsy test would replace
+         * it with the category's figure on every save — the same distinction
+         * `gstRate` needs two blocks up, and for the same reason: "nobody said"
+         * and "somebody said zero" are different facts.
+         */
+        if (resolved.weightG != null && input.weightG == null) {
+            out.weightG = resolved.weightG;
+        }
+        if (resolved.dimensionsCm && !input.dimensionsCm) {
+            out.dimensionsCm = { ...resolved.dimensionsCm };
+        }
         /*
          * Features MERGE rather than replace — the field's own contract, and the
          * reason the blader linkage works: a seller who ticks "Japan Import" keeps
@@ -585,65 +691,6 @@ export class ProductRepository extends BaseRepository {
             out.seoKeywords = merged;
         }
     }
-    /**
-     * Keep `availableQuantity` in step when a seller edits `stockQuantity`.
-     *
-     * 🛑 THE BUG THIS CLOSES: A SELLER COULD NOT RESTOCK.
-     *
-     * There are two stock numbers. `stockQuantity` is seller-authored and is the
-     * only one any of the three product forms exposes. `availableQuantity` is
-     * system-owned, seeded from `stockQuantity` **once, at create**, and
-     * thereafter driven by checkout decrements and timeout restores.
-     *
-     * Nothing re-synced them. `updateProduct` is a bare spread, and grep for
-     * `availableQuantity` across `src/app/api/store/products/**` and
-     * `seller-actions.ts` returns ZERO. So: a product sells out
-     * (`availableQuantity: 0`), the seller edits `stockQuantity` 0 → 10, and the
-     * product STAYS UNBUYABLE — purchasability gates on `availableQuantity > 0`
-     * in three independent places (`service.ts`, the cart route, and the public
-     * list filter). The only field the seller can reach is the one that does not
-     * control availability.
-     *
-     * With 76–93% of competitor catalogues sold out, restock is the main
-     * lifecycle event, not an edge case.
-     *
-     * WHY A DELTA AND NOT `availableQuantity = stockQuantity`: the difference
-     * between the two IS the outstanding reservation — units held by placed-but-
-     * unpaid orders. Overwriting would hand those units back to the shelf and
-     * oversell them. So carry the reservation forward:
-     *
-     *     reserved = prevStock - prevAvailable        (clamped at 0)
-     *     available = nextStock - reserved            (clamped at 0)
-     *
-     * Examples: 10 stock / 10 available, edit to 12 → 12. 10 stock / 7 available
-     * (3 reserved), edit to 12 → 9. Sold out at 0/0, edit to 10 → 10.
-     *
-     * 🛑 Returns `{}` — not a value — whenever the write does not name
-     * `stockQuantity`, or names `availableQuantity` explicitly. The second case
-     * is what keeps the checkout decrement, the stock-restore job and
-     * `updateAvailableQuantity` authoritative: they write `availableQuantity`
-     * directly and must never have it recomputed from a stale `stockQuantity`
-     * underneath them.
-     */
-    static resyncAvailableQuantity(current, data) {
-        if (!current)
-            return {};
-        if (!("stockQuantity" in data))
-            return {};
-        // An explicit availableQuantity write wins — see the 🛑 above.
-        if ("availableQuantity" in data)
-            return {};
-        const nextStock = data.stockQuantity;
-        if (typeof nextStock !== "number" || !Number.isFinite(nextStock))
-            return {};
-        const prevStock = typeof current.stockQuantity === "number" ? current.stockQuantity : 0;
-        const prevAvailable = typeof current.availableQuantity === "number" ? current.availableQuantity : prevStock;
-        const reserved = Math.max(0, prevStock - prevAvailable);
-        const nextAvailable = Math.max(0, nextStock - reserved);
-        if (nextAvailable === prevAvailable)
-            return {};
-        return { availableQuantity: nextAvailable };
-    }
     async update(id, data) {
         this.cacheInvalidateForId(id);
         const current = await super.findById(id);
@@ -657,7 +704,7 @@ export class ProductRepository extends BaseRepository {
         const derived = touchesTaxonomy
             ? await this.deriveTaxonomy({ ...current, ...data })
             : {};
-        const stock = ProductRepository.resyncAvailableQuantity(current, data);
+        const stock = resolveStockResync(current, data);
         const merged = { ...current, ...data, ...derived, ...stock };
         const updated = await super.update(id, {
             ...data,
@@ -991,10 +1038,44 @@ export class ProductRepository extends BaseRepository {
         const plan = planSearchTxt(opts?.search);
         if (plan.empty)
             return emptySearchResult();
-        const result = await this.sieveQuery(model, ProductRepository.SIEVE_FIELDS, {
+        /*
+         * ── D2 Fix 2: a MULTI-TERM search over-fetches a window ────────────────
+         *
+         * 🛑 The bug: `refineSearchTxt` used to filter the already-paginated page,
+         * so a two-word search could only ever return matches that happened to fall
+         * inside one page window — rows 51+ matching both terms were unreachable,
+         * and `hasMore: false` asserted there was no page 2. A search returning 7
+         * of 40 real matches reported "7 results" and disabled Next, which reads as
+         * a working search.
+         *
+         * So for a multi-term query the FETCH asks for a bounded window and the
+         * refine slices the requested page out of it. A single-term query is
+         * untouched: its one `array-contains` clause paginates exactly, there is
+         * nothing to refine, and over-fetching would be pure cost.
+         *
+         * 🛑 Bounded, not unbounded (Rule #6): one read of at most
+         * `SEARCH_REFINE_WINDOW`, and when that saturates the result carries
+         * `truncated` so the total renders as a floor rather than a false total.
+         */
+        const isMultiTerm = plan.rest.length > 0;
+        const requestedPage = Math.max(1, Number(model?.page ?? 1) || 1);
+        const requestedPageSize = Math.max(1, Number(model?.pageSize ?? 20) || 20);
+        const result = await this.sieveQuery(
+        /*
+         * The window is requested through the MODEL, not a new `SieveOptions`
+         * field — `sieveQuery` already derives its page from `model.page` /
+         * `model.pageSize`, so overriding them needs no API change and no second
+         * way to express a page size.
+         */
+        isMultiTerm ? { ...model, page: 1, pageSize: SEARCH_REFINE_WINDOW } : model, ProductRepository.SIEVE_FIELDS, {
             baseQuery: this.buildScopedQuery(opts),
             defaultPageSize: 20,
-            maxPageSize: 100,
+            /*
+             * 🛑 `maxPageSize` has to rise with the window or Sieve clamps the
+             * request back to 100 — and the window would silently be a page again,
+             * which is the original bug with a larger constant.
+             */
+            maxPageSize: isMultiTerm ? SEARCH_REFINE_WINDOW : 100,
             aliases: ProductRepository.FILTER_ALIASES,
         });
         // Only the most selective term became the `array-contains` clause, because
@@ -1009,7 +1090,15 @@ export class ProductRepository extends BaseRepository {
         // this page only. That is the known post-pagination-filter debt, not a new
         // invention — reporting the pre-filter total would promise pages that
         // render empty.
-        return refineSearchTxt(result, plan.rest);
+        return refineSearchTxt(result, plan.rest, isMultiTerm
+            ? {
+                page: requestedPage,
+                pageSize: requestedPageSize,
+                // Saturated means more may exist beyond the window, so `total` is a
+                // FLOOR and `truncated` says so.
+                windowSaturated: (result.items?.length ?? 0) >= SEARCH_REFINE_WINDOW,
+            }
+            : undefined);
     }
     /**
      * How many products of each listing type exist within a scope — the numbers
@@ -1147,6 +1236,29 @@ ProductRepository.SIEVE_FIELDS = {
     isSold: { canFilter: true, canSort: true },
     isPartOfBundle: { canFilter: true, canSort: false },
     isTestData: { canFilter: true, canSort: false },
+    /*
+     * The warehouse location — C4.
+     *
+     * 🛑 `physicalLocation` was absent from this config entirely, so a filter
+     * on it was DROPPED SILENTLY: `sieve.ts` sets `throwExceptions: false`, so
+     * `findField()` returning undefined removes the clause rather than
+     * raising. A seller could set a location and then filter by it to no
+     * effect, with nothing anywhere saying so — Root Cause #62's
+     * EMITTED_BUT_UNFILTERABLE shape.
+     *
+     * Nested paths, because that is what Firestore filters on
+     * (`physicalLocation.zone`), and `canSort: false` on all three: sorting by
+     * bin would pair an orderBy with whatever else is filtered and demand a
+     * composite index per combination, which is the trap next door
+     * (Root Cause #59). Browsing a bin is a FILTER, not a sort.
+     *
+     * The primary way to find a location is still `searchTxt` — see
+     * `physicalLocationTokens`. These exist so a dashboard can ask the precise
+     * question ("zone A exactly") where search answers the loose one.
+     */
+    "physicalLocation.zone": { canFilter: true, canSort: false },
+    "physicalLocation.shelf": { canFilter: true, canSort: false },
+    "physicalLocation.bin": { canFilter: true, canSort: false },
     brandSlug: { canFilter: true, canSort: false },
     sublistingCategoryId: { canFilter: true, canSort: false },
     createdAt: { canFilter: true, canSort: true, parseValue: parseSieveDateValue },

package/dist/features/products/schemas/firestore.d.ts

@@ -437,6 +437,31 @@ export interface ProductDocument extends BaseDocument {
             etaDaysMax?: number;
         }>;
     };
+    /**
+     * The PACKED parcel — C5a. Optional on purpose.
+     *
+     * 🛑 The figure a carrier bills on is the packed box, not the item: a bey is
+     * ~30 g and ~300 g boxed. A REQUIRED product weight would therefore be an
+     * item weight that systematically under-estimates every booking — and
+     * because a booking is capped at the amount collected from the buyer,
+     * systematic under-estimation means silently absorbing the difference on
+     * every order.
+     *
+     * So three tiers: the category's packed default drives the checkout
+     * estimate, this refines it when a seller has actually weighed a box, and
+     * the real figure is entered at shipment time. See
+     * `_internal/shared/features/shipping/parcel.ts`.
+     *
+     * `dimensionsCm` is all three or nothing — a box with a length and no width
+     * is not a dimension, and a volumetric calculation fed two of three yields a
+     * quote for a flat sheet.
+     */
+    weightG?: number;
+    dimensionsCm?: {
+        l: number;
+        w: number;
+        h: number;
+    };
     /** Physical storage location for inventory management (Print & Label Center). */
     physicalLocation?: {
         zone: string;
@@ -480,7 +505,7 @@ export declare const ProductStatusValues: {
     readonly ARCHIVED: "archived";
 };
 export declare const PRODUCT_COLLECTION: "products";
-export declare const PRODUCT_INDEXED_FIELDS: readonly ["storeId", "status", "category", "featured", "listingType", "isPromoted", "isOnSale", "isSold", "searchTxt", "createdAt", "barcodeId"];
+export declare const PRODUCT_INDEXED_FIELDS: readonly ["storeId", "status", "category", "featured", "listingType", "isPromoted", "isOnSale", "isSold", "searchTxt", "createdAt", "barcodeId", "physicalLocation.zone"];
 export declare const DEFAULT_PRODUCT_DATA: Partial<ProductDocument>;
 export declare const PRODUCT_PUBLIC_FIELDS: readonly ["allowOffers", "minOfferPercent", "id", "title", "description", "category", "subcategory", "brand", "price", "currency", "stockQuantity", "availableQuantity", "images", "status", "storeName", "featured", "tags", "specifications", "features", "shippingInfo", "returnPolicy", "finalSale", "listingType", "auctionEndDate", "startingBid", "currentBid", "bidCount", "reservePrice", "buyNowPrice", "minBidIncrement", "autoExtendable", "auctionExtensionMinutes", "auctionShippingPaidBy", "bidsHaveStarted", "grading", "card", "classified", "digitalCode", "liveItem", "catalogProductId", "preOrderDeliveryDate", "preOrderDepositPercent", "preOrderDepositAmount", "preOrderMaxQuantity", "preOrderCurrentCount", "preOrderProductionStatus", "preOrderCancellable", "condition", "insurance", "insuranceCost", "shippingPaidBy", "isPromoted", "isOnSale", "isSold", "slug", "seoTitle", "seoDescription", "seoKeywords", "viewCount", "customFields", "customSections", "sublistingCategoryId", "groupId", "isGroupParent", "groupParentSlug", "groupChildSlugs", "groupTitle", "createdAt"];
 export declare const PRODUCT_UPDATABLE_FIELDS: readonly ["title", "description", "category", "subcategory", "brand", "price", "stockQuantity", "images", "status", "tags", "specifications", "features", "shippingInfo", "returnPolicy", "finalSale", "pickupAddressId", "condition", "insurance", "shippingPaidBy", "autoExtendable", "auctionExtensionMinutes", "auctionShippingPaidBy", "reservePrice", "buyNowPrice", "minBidIncrement", "bidsHaveStarted", "grading", "card", "classified", "digitalCode", "liveItem", "catalogProductId", "listingType", "preOrderDeliveryDate", "preOrderDepositPercent", "preOrderDepositAmount", "preOrderMaxQuantity", "preOrderProductionStatus", "preOrderCancellable", "isOnSale", "isSold", "allowShipBeforeEmiComplete", "gstRate", "hsnCode", "authenticity", "printMeta", "seoTitle", "seoDescription", "seoKeywords", "customFields", "customSections", "sublistingCategoryId", "groupId", "isGroupParent", "groupParentSlug", "groupChildSlugs", "groupTitle", "barcodeId"];

package/dist/features/products/schemas/firestore.js

@@ -1,7 +1,7 @@
 /**
  * Products Firestore Document Types & Constants
  */
-import { generateProductId, generateAuctionId, generatePreOrderId, } from "../../../utils/id-generators";
+import { generateProductId, generateAuctionId, generatePreOrderId, } from "../../../utils/id-generators.js";
 export const MAX_CUSTOM_FIELDS = 50;
 export const MAX_CUSTOM_SECTIONS = 3;
 /** Runtime-accessible product status values — use instead of bare string literals. */
@@ -24,6 +24,23 @@ export const PRODUCT_INDEXED_FIELDS = [
     "searchTxt",
     "createdAt",
     "barcodeId",
+    /*
+     * The warehouse location — C4.
+     *
+     * 🛑 A single-field entry, deliberately, and the distinction matters.
+     * Firestore indexes single fields automatically, so this list is
+     * documentation of what the collection is QUERIED by rather than the
+     * mechanism — and the real answer for location is `searchTxt`, which is
+     * already here. Listing the nested paths records that they are now a
+     * supported query shape (the Sieve config accepts them), so the next person
+     * adding a composite sees them rather than discovering a dropped clause.
+     *
+     * `physicalLocation.zone` is the only part worth naming: a zone is the
+     * coarse bucket a dashboard groups by, while shelf and bin are what search
+     * resolves. Naming all three would imply three query shapes exist when one
+     * does.
+     */
+    "physicalLocation.zone",
 ];
 export const DEFAULT_PRODUCT_DATA = {
     status: "draft",

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

@@ -459,10 +459,10 @@ export declare const productItemSchema: z.ZodObject<{
     }[] | undefined;
     prizeGithubFileUrl?: string | undefined;
     inStock?: boolean | undefined;
-    attributes?: Record<string, string> | undefined;
     originalPrice?: number | undefined;
     sellerAvatar?: string | undefined;
     stockCount?: number | undefined;
+    attributes?: Record<string, string> | undefined;
 }, {
     status: "published" | "draft" | "in_review" | "archived";
     id: string;
@@ -598,10 +598,10 @@ export declare const productItemSchema: z.ZodObject<{
     }[] | undefined;
     prizeGithubFileUrl?: string | undefined;
     inStock?: boolean | undefined;
-    attributes?: Record<string, string> | undefined;
     originalPrice?: number | undefined;
     sellerAvatar?: string | undefined;
     stockCount?: number | undefined;
+    attributes?: Record<string, string> | undefined;
 }>;
 /** Base Zod schema for list-query parameters. */
 export declare const productListParamsSchema: z.ZodObject<{

package/dist/features/products/schemas/index.js

@@ -1,7 +1,7 @@
-export * from "./firestore";
+export * from "./firestore.js";
 import { z } from "zod";
-import { mediaFieldSchema } from "../../media/types/index";
-import { PRIZE_DRAW_DURATION_DAYS_MIN, PRIZE_DRAW_DURATION_DAYS_MAX, } from "../../../_internal/shared/checkout/rules/_limits";
+import { mediaFieldSchema } from "../../media/types/index.js";
+import { PRIZE_DRAW_DURATION_DAYS_MIN, PRIZE_DRAW_DURATION_DAYS_MAX, } from "../../../_internal/shared/checkout/rules/_limits.js";
 // --- Sub-schemas --------------------------------------------------------------
 export const productImageSchema = z.object({
     url: z.string(),

package/dist/features/products/schemas/product-features.validators.js

@@ -6,7 +6,7 @@
  * of hand-rolling parallel schemas.
  */
 import { z } from "zod";
-import { annotate } from "../../shell/field-ui-meta";
+import { annotate } from "../../shell/field-ui-meta.js";
 export const PRODUCT_FEATURE_PRODUCT_TYPE_ENUM = z.enum([
     "product",
     "auction",

package/dist/features/products/server.js

@@ -3,7 +3,7 @@
  *
  * Server-only entry point — repositories and API route handlers.
  */
-export * from "./actions";
-export { ProductRepository, productRepository, } from "./repository/products.repository";
-export { GET, POST } from "./api/route";
-export { GET as productItemGET, PATCH as productItemPATCH, DELETE as productItemDELETE, } from "./api/[id]/route";
+export * from "./actions/index.js";
+export { ProductRepository, productRepository, } from "./repository/products.repository.js";
+export { GET, POST } from "./api/route.js";
+export { GET as productItemGET, PATCH as productItemPATCH, DELETE as productItemDELETE, } from "./api/[id]/route.js";

package/dist/features/products/utils/listing-badge-variant.js

@@ -1,4 +1,4 @@
-import { isListingType } from "./listing-type";
+import { isListingType } from "./listing-type.js";
 /**
  * Listing-kind → Badge variant map.
  *

package/dist/features/products/utils/listing-type.js

@@ -1,4 +1,4 @@
-import { ALL_LISTING_TYPES } from "../../../_internal/shared/listing-types/feature-flags";
+import { ALL_LISTING_TYPES } from "../../../_internal/shared/listing-types/feature-flags.js";
 const LISTING_TYPE_SET = new Set(ALL_LISTING_TYPES);
 /** True when `value` is a member of the canonical `ListingType` union. */
 export function isListingType(value) {

package/dist/features/promotions/actions/coupon-actions.js

@@ -4,8 +4,8 @@
  * Pure business functions — auth, rate-limiting, and Next.js specifics
  * are handled by the calling server action in the consumer.
  */
-import { couponsRepository } from "../repository/coupons.repository";
-import { productRepository } from "../../products/repository/products.repository";
+import { couponsRepository } from "../repository/coupons.repository.js";
+import { productRepository } from "../../products/repository/products.repository.js";
 export async function validateCoupon(userId, code, orderTotal) {
     return couponsRepository.validateCoupon(code, userId, orderTotal);
 }

package/dist/features/promotions/actions/index.js

@@ -1,4 +1,4 @@
-export * from "./coupon-actions";
-export * from "./coupon-stacking";
-export * from "./promotions-actions";
-export * from "./seller-coupon-actions";
+export * from "./coupon-actions.js";
+export * from "./coupon-stacking.js";
+export * from "./promotions-actions.js";
+export * from "./seller-coupon-actions.js";

package/dist/features/promotions/actions/promotions-actions.js

@@ -4,9 +4,9 @@
  * Pure business functions — auth, rate-limiting, and Next.js specifics
  * are handled by the calling server action in the consumer.
  */
-import { productRepository } from "../../../features/products/repository/products.repository";
-import { couponsRepository } from "../repository/coupons.repository";
-import { ProductStatusValues } from "../../../features/products/schemas";
+import { productRepository } from "../../../features/products/repository/products.repository.js";
+import { couponsRepository } from "../repository/coupons.repository.js";
+import { ProductStatusValues } from "../../../features/products/schemas/index.js";
 /**
  * Fetch all promotions data in parallel:
  * promoted products, featured products, and active/valid coupons.

package/dist/features/promotions/actions/seller-coupon-actions.js

@@ -4,11 +4,11 @@
  * Business logic for seller-scoped coupon management.
  * Auth, rate-limiting, and input validation are handled by the calling server action.
  */
-import { serverLogger } from "../../../monitoring";
-import { couponsRepository } from "../repository/coupons.repository";
-import { storeRepository } from "../../stores/repository/store.repository";
-import { buildSellerCouponCode } from "../schemas/firestore";
-import { AuthorizationError, ValidationError, NotFoundError, } from "../../../errors";
+import { serverLogger } from "../../../monitoring/index.js";
+import { couponsRepository } from "../repository/coupons.repository.js";
+import { storeRepository } from "../../stores/repository/store.repository.js";
+import { buildSellerCouponCode } from "../schemas/firestore.js";
+import { AuthorizationError, ValidationError, NotFoundError, } from "../../../errors/index.js";
 // --- Create ----------------------------------------------------------------
 export async function sellerCreateCoupon(userId, input) {
     const store = await storeRepository.findByOwnerId(userId);

package/dist/features/promotions/api/route.js

@@ -18,8 +18,8 @@
  * Collection: "coupons"
  */
 import { NextResponse } from "next/server.js";
-import { getProviders } from "../../../contracts";
-import { normalizeError } from "../../../errors/normalize";
+import { getProviders } from "../../../contracts/index.js";
+import { normalizeError } from "../../../errors/normalize.js";
 function param(url, key) {
     return url.searchParams.get(key);
 }

package/dist/features/promotions/columns/index.js

@@ -1,4 +1,4 @@
-import { buildColumns, renderBoolean } from "../../../ui/columns";
+import { buildColumns, renderBoolean } from "../../../ui/columns/index.js";
 /**
  * Default admin column definitions for a coupons/promotions table.
  *

package/dist/features/promotions/components/CouponCard.js

@@ -1,11 +1,11 @@
 "use client";
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
-import { normalizeError } from "../../../errors/normalize";
+import { normalizeError } from "../../../errors/normalize.js";
 import { useState } from "react";
 import { Pencil, Trash2, ToggleLeft, ToggleRight } from "lucide-react";
-import { Badge, BaseListingCard, Button, ConfirmDeleteModal, Div, IconButton, Row, Span, Stack, Text } from "../../../ui";
-import { useLongPress } from "../../../react/hooks/useLongPress";
-import { ACCOUNT_ENDPOINTS } from "../../../constants/api-endpoints";
+import { Badge, BaseListingCard, Button, ConfirmDeleteModal, Div, IconButton, Row, Span, Stack, Text } from "../../../ui/index.js";
+import { useLongPress } from "../../../react/hooks/useLongPress.js";
+import { ACCOUNT_ENDPOINTS } from "../../../constants/api-endpoints.js";
 const CLS_TOGGLE_ON = "h-4 w-4 text-success dark:text-success";
 const CLS_DELETE_BTN = "p-[var(--appkit-space-1-5)] text-[var(--appkit-color-text-muted)] hover:bg-error-surface dark:hover:bg-error-surface hover:text-error dark:hover:text-error transition-colors disabled:opacity-50";
 const TYPE_COLORS = {

package/dist/features/promotions/components/CouponHelpDetails.js

@@ -1,7 +1,7 @@
 "use client";
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
-import { Details, Summary, Stack, Text, Ul, Li, Span } from "../../../ui";
-import { COUPON_HELP } from "../constants/coupon-help";
+import { Details, Summary, Stack, Text, Ul, Li, Span } from "../../../ui/index.js";
+import { COUPON_HELP } from "../constants/coupon-help.js";
 function HelpSection({ heading, items }) {
     return (_jsxs(Stack, { gap: "dense", children: [_jsx(Text, { size: "sm", weight: "semibold", children: heading }), _jsx(Ul, { marker: "disc", indent: "md", spacing: "tight", size: "sm", children: items.map((item) => (_jsx(Li, { color: "muted", children: item }, item))) })] }));
 }

package/dist/features/promotions/components/CouponsIndexListing.js

@@ -2,14 +2,14 @@
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { useState, useCallback, useMemo } from "react";
 import { Search, SlidersHorizontal, X } from "lucide-react";
-import { useUrlTable } from "../../../react/hooks/useUrlTable";
-import { Button, DateInput, IconButton, Input, ListingFilterDrawer, Pagination, RadioGroup, SortDropdown, Div, Grid, Row, Span, Stack, Text, Heading, StickyToolbar } from "../../../ui";
-import { usePromotions } from "../hooks/usePromotions";
-import { CouponCard } from "./CouponCard";
-import { CouponHelpDetails } from "./CouponHelpDetails";
-import { TABLE_KEYS } from "../../../constants/table-keys";
-import { COUPON_FIELDS } from "../../../constants/field-names";
-import { sortBy } from "../../../constants/sort";
+import { useUrlTable } from "../../../react/hooks/useUrlTable.js";
+import { Button, DateInput, IconButton, Input, ListingFilterDrawer, Pagination, RadioGroup, SortDropdown, Div, Grid, Row, Span, Stack, Text, Heading, StickyToolbar } from "../../../ui/index.js";
+import { usePromotions } from "../hooks/usePromotions.js";
+import { CouponCard } from "./CouponCard.js";
+import { CouponHelpDetails } from "./CouponHelpDetails.js";
+import { TABLE_KEYS } from "../../../constants/table-keys.js";
+import { COUPON_FIELDS } from "../../../constants/field-names.js";
+import { sortBy } from "../../../constants/sort.js";
 const __O = {
     hidden: "overflow-hidden",
 };

package/dist/features/promotions/components/PromotionsView.js

@@ -1,5 +1,5 @@
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
-import { Div, Heading, Section, Stack, Text } from "../../../ui";
+import { Div, Heading, Section, Stack, Text } from "../../../ui/index.js";
 export function PromotionsViewProductSection({ title, subtitle, renderProducts, hasProducts, }) {
     if (!hasProducts)
         return null;