@mohasinac/appkit 4.42.1 → 4.42.2

package/dist/features/orders/utils/order-ref.js

@@ -0,0 +1,58 @@
+/**
+ * The short, buyer-facing reference for an order.
+ *
+ * 🛑 WHY THIS EXISTS. Four sites independently rendered
+ * `order.id.slice(-8).toUpperCase()` — the buyer's order view (twice), the
+ * invoice, and `OrdersList`. A fixed-width tail is the wrong shape for these
+ * ids: `generateOrderId` produces
+ *
+ *     order-{itemCount}-{YYYYMMDD}-{rand6}
+ *
+ * whose final segment is **6** characters, so `slice(-8)` takes seven of them
+ * plus one extra — and that extra is the last digit of the DATE. Measured on a
+ * real order, `order-2-20260929-ezq0pc` rendered as **`#9-EZQ0PC`**.
+ *
+ * Three things are wrong with that, in increasing order of cost:
+ *
+ * 1. It straddles a delimiter, so the reference is not a substring of anything
+ *    meaningful — the leading `9` looks like part of a structured reference and
+ *    is a fragment of `20260929`.
+ * 2. Every order placed on the 9th, 19th or 29th of any month renders the same
+ *    leading digit, and the 1st/11th/21st/31st share another. The digit adds no
+ *    distinguishing information while looking like it does.
+ * 3. It is the number a buyer reads off their own screen and quotes to support,
+ *    and it does not appear anywhere in the real document id. The payment page
+ *    shows the full `order-2-20260929-ezq0pc`, so the same order presents two
+ *    unrelated-looking references depending on which page you are on.
+ *
+ * The random suffix alone is the intended unique part, so that is what this
+ * returns. Taking the last hyphen-delimited SEGMENT rather than a character
+ * count also means the format can change without silently re-slicing.
+ *
+ * @tag domain:orders
+ * @tag layer:utils
+ * @tag access:isomorphic
+ * @tag sideEffects:none — pure
+ */
+/**
+ * `order-2-20260929-ezq0pc` → `EZQ0PC`.
+ *
+ * Falls back to the whole id, uppercased, for anything that is not in the
+ * generator's shape (a legacy Firestore auto-id, say) — a reference that is
+ * merely long is far better than one that is wrong, and an empty string would
+ * render as a bare `#`.
+ */
+export function shortOrderRef(orderId) {
+    if (!orderId)
+        return "";
+    const segments = orderId.split("-").filter(Boolean);
+    const last = segments[segments.length - 1] ?? orderId;
+    /*
+     * A single trailing segment that is purely numeric is a date, not a suffix —
+     * that would be an id shaped `order-2-20260929` with no random part. Prefer
+     * the whole thing over presenting a date as a reference number.
+     */
+    if (segments.length > 1 && /^\d+$/.test(last))
+        return orderId.toUpperCase();
+    return last.toUpperCase();
+}

package/dist/features/products/components/MakeOfferButton.d.ts

@@ -15,8 +15,25 @@ export interface MakeOfferButtonProps {
      * form accepts, `makeOffer` accepts.
      */
     bounds: OfferBounds;
+    /**
+     * The listing's owning store SLUG (`product.storeId`), so the owning seller
+     * is not offered a control for negotiating with themselves.
+     *
+     * 🛑 Optional only because the two consumer pages must both keep compiling;
+     * pass it. Omitting it restores the old behaviour, which was to show the full
+     * offer form to the very seller who set the price — measured on production as
+     * tyson@beybladearena.in on a listing sold by his own store, complete with
+     * the buyer-facing line "The seller will accept, decline, or suggest a
+     * counter price".
+     *
+     * This is a UI courtesy, NOT the authorization boundary — `makeOffer` refuses
+     * a self-offer server-side, which is where the actual rule lives (Root Cause
+     * #46: a hidden button is not a gate). Hiding it here only stops a seller
+     * filling in a form that was always going to be rejected.
+     */
+    listingStoreId?: string;
     /** Called with (productId, offerAmount, note?). Must return void or throw on error. */
     onMakeOffer: (productId: string, amount: number, note?: string) => Promise<void>;
     className?: string;
 }
-export declare function MakeOfferButton({ productId, listedPrice, currency, bounds, onMakeOffer, className, }: MakeOfferButtonProps): React.JSX.Element;
+export declare function MakeOfferButton({ productId, listedPrice, currency, bounds, listingStoreId, onMakeOffer, className, }: MakeOfferButtonProps): React.JSX.Element | null;

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

@@ -2,6 +2,7 @@
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { useState, useTransition } from "react";
 import { useCanSeePrices } from "../../../react/hooks/useCanSeePrices";
+import { useOptionalSession } from "../../../react/contexts/SessionContext";
 import { Button, Div, LoginRequiredModal, Modal, Span, Stack, Text, } from "../../../ui";
 import { Form } from "../../../ui/components/Form";
 import { FieldInput } from "../../../ui/forms/FieldInput";
@@ -28,11 +29,22 @@ const CLS_PENDING_BODY = "text-[length:var(--appkit-text-xs)] text-warning dark:
 function isActiveOfferError(msg) {
     return msg.includes("active offer") || msg.includes("ACTIVE_OFFER");
 }
-export function MakeOfferButton({ productId, listedPrice, currency, bounds, onMakeOffer, className = "", }) {
+export function MakeOfferButton({ productId, listedPrice, currency, bounds, listingStoreId, onMakeOffer, className = "", }) {
     const [state, setState] = useState("idle");
     const [showLoginModal, setShowLoginModal] = useState(false);
     const { canSeePrices } = useCanSeePrices();
+    const session = useOptionalSession();
     const [isPending, startTransition] = useTransition();
+    /*
+     * `UserDocument.storeId` and `ProductDocument.storeId` are both the store
+     * SLUG, so this is a same-namespace comparison (unlike the uid-vs-slug trap
+     * that made `getSellerProductAction`'s ownership check always fail — Root
+     * Cause #98). `useOptionalSession` rather than `useSession` because library
+     * code that adjusts for the viewer must not require a provider, and no
+     * provider must fail CLOSED — no session means no owner, i.e. show the
+     * button, which is correct for the signed-out visitor it describes.
+     */
+    const viewerOwnsListing = Boolean(listingStoreId) && session?.user?.storeId === listingStoreId;
     const fmt = (n) => (currency ? formatCurrency(n, currency) : `₹${n.toLocaleString()}`);
     // The SAME bounds the server enforces — passed in, never recomputed, so the
     // input can neither seed nor accept an amount `makeOffer` would reject.
@@ -80,6 +92,16 @@ export function MakeOfferButton({ productId, listedPrice, currency, bounds, onMa
             }
         });
     }
+    /*
+     * Checked AFTER the success/pending panels below would have been, but before
+     * the idle button — placed here so it cannot hide a panel reporting an offer
+     * this viewer already sent. In practice an owner never reaches those states
+     * (the server refuses), so ordering only matters if `listingStoreId` is ever
+     * wrong: then the worst outcome is a visible panel rather than a silently
+     * swallowed one.
+     */
+    if (viewerOwnsListing)
+        return null;
     if (state === "success") {
         return (_jsxs(Div, { className: `${CLS_SUCCESS_BOX} ${className}`, children: [_jsx(Span, { size: "lg", children: "\uD83C\uDF89" }), _jsx(Text, { className: "text-success", size: "sm", weight: "medium", children: bounds.isBuyRequest ? "Request sent!" : "Offer sent!" }), _jsxs(Text, { className: "text-success", size: "xs", children: ["The seller will review your ", bounds.isBuyRequest ? "request" : "offer", " and respond shortly."] })] }));
     }

package/dist/features/products/components/ProductDetailPageView.d.ts

@@ -24,6 +24,11 @@ export interface ProductDetailPageViewProps {
         minOfferPercent: number;
         /** Resolved by `resolveOfferBounds()` — the same rule `makeOffer` enforces. */
         bounds: OfferBounds;
+        /**
+         * The owning store's SLUG, so the control can suppress itself for the
+         * seller who owns the listing. Same namespace as `UserDocument.storeId`.
+         */
+        listingStoreId?: string;
     }) => React.ReactNode;
     /**
      * Renders the primary action buttons (Buy Now / Add to Cart / Wishlist).

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

@@ -204,7 +204,7 @@ export async function ProductDetailPageView({ slug, initialProduct, renderOfferA
                     price: product.price,
                     storeId: product.storeId,
                     storeName: product.storeName,
-                } }), _jsxs(Container, { size: "xl", padding: "y-lg", children: [_jsx(ProductDetailView, { renderBreadcrumb: () => (_jsxs(Row, { className: "mb-4", align: "center", justify: "between", gap: "sm", wrap: true, children: [_jsxs(Nav, { "aria-label": "Breadcrumb", layout: "flex-wrap", gap: "2xs", textSize: "xs", color: "muted", children: [_jsx(Link, { href: String(ROUTES.HOME), className: CLS_BREADCRUMB_LINK, children: "Home" }), _jsx(Span, { "aria-hidden": true, children: "/" }), _jsx(Link, { href: String(ROUTES.PUBLIC.PRODUCTS), className: CLS_BREADCRUMB_LINK, children: "Products" }), (primaryCategorySlug || category) && (_jsxs(_Fragment, { children: [_jsx(Span, { "aria-hidden": true, children: "/" }), _jsx(Link, { href: String(ROUTES.PUBLIC.CATEGORY_DETAIL(primaryCategorySlug ?? category)), className: CLS_BREADCRUMB_LINK, children: categoryName || category })] })), subcategory && (_jsxs(_Fragment, { children: [_jsx(Span, { "aria-hidden": true, children: "/" }), _jsx(Span, { transform: "capitalize", children: subcategory })] }))] }), _jsx(ShareButton, { title: title || undefined })] })), renderGallery: () => (_jsx(ProductGalleryClient, { images: images, video: productVideo, productName: title || undefined })), renderInfo: () => (_jsxs(Stack, { gap: "md", children: [_jsxs(Div, { children: [condition && (_jsx(Span, { size: "xs", weight: "medium", className: "mb-2 inline-block", padding: "pill-sm", rounded: "full", surface: "subtle", color: "muted", transform: "capitalize", children: condition === "new"
+                } }), _jsxs(Container, { size: "xl", padding: "y-lg", children: [_jsx(ProductDetailView, { renderBreadcrumb: () => (_jsxs(Row, { className: "mb-4", align: "center", justify: "between", gap: "sm", wrap: true, children: [_jsxs(Nav, { "aria-label": "Breadcrumb", layout: "flex-wrap", gap: "2xs", textSize: "xs", color: "muted", children: [_jsx(Link, { href: String(ROUTES.HOME), className: CLS_BREADCRUMB_LINK, children: "Home" }), _jsx(Span, { "aria-hidden": true, children: "/" }), _jsx(Link, { href: String(ROUTES.PUBLIC.PRODUCTS), className: CLS_BREADCRUMB_LINK, children: "Products" }), (primaryCategorySlug || category) && (_jsxs(_Fragment, { children: [_jsx(Span, { "aria-hidden": true, children: "/" }), _jsx(Link, { href: String(ROUTES.PUBLIC.CATEGORY_DETAIL(primaryCategorySlug ?? category)), className: CLS_BREADCRUMB_LINK, children: categoryNames[0] || categoryName || category })] })), subcategory && (_jsxs(_Fragment, { children: [_jsx(Span, { "aria-hidden": true, children: "/" }), _jsx(Span, { transform: "capitalize", children: subcategory })] }))] }), _jsx(ShareButton, { title: title || undefined })] })), renderGallery: () => (_jsx(ProductGalleryClient, { images: images, video: productVideo, productName: title || undefined })), renderInfo: () => (_jsxs(Stack, { gap: "md", children: [_jsxs(Div, { children: [condition && (_jsx(Span, { size: "xs", weight: "medium", className: "mb-2 inline-block", padding: "pill-sm", rounded: "full", surface: "subtle", color: "muted", transform: "capitalize", children: condition === "new"
                                                 ? "Brand New"
                                                 : condition === "like_new"
                                                     ? "Like New"
@@ -250,6 +250,7 @@ export async function ProductDetailPageView({ slug, initialProduct, renderOfferA
                                             price,
                                             currency,
                                             minOfferPercent,
+                                            listingStoreId: typeof p.storeId === "string" ? p.storeId : undefined,
                                             bounds: resolveOfferBounds({
                                                 listingType: normalizeListingType(p),
                                                 listedPrice: price,
@@ -272,7 +273,7 @@ export async function ProductDetailPageView({ slug, initialProduct, renderOfferA
                                 content: _jsx(CustomSectionTabContent, { section: s }),
                             })) })), renderBundleSection: bundleMemberships.length > 0
                             ? () => (_jsxs(Div, { className: CLS_BUNDLE_BOX, children: [_jsxs(Row, { className: "mb-3", align: "center", gap: "sm", children: [_jsx(Span, { size: "base", className: CLS_BUNDLE_ICON, "aria-hidden": "true", children: "\uD83D\uDCE6" }), _jsxs(Text, { className: CLS_BUNDLE_TITLE, children: ["Part of ", bundleMemberships.length === 1 ? "a bundle" : `${bundleMemberships.length} bundles`] })] }), _jsx(Stack, { gap: "3", children: bundleMemberships.map((b) => (_jsxs(Div, { className: CLS_BUNDLE_ROW, children: [_jsxs(Div, { className: "min-w-0", children: [_jsx(Text, { className: CLS_BUNDLE_LABEL, children: "Included in bundle" }), _jsx(Text, { className: "truncate", color: "primary", size: "sm", weight: "semibold", children: b.title })] }), _jsx(Link, { href: String(ROUTES.PUBLIC.BUNDLE_DETAIL(b.id)), className: CLS_BUNDLE_CTA, children: "View Bundle \u2192" })] }, b.id))) })] }))
-                            : undefined, renderRelated: () => (_jsxs(Stack, { gap: "xl", children: [_jsx(GroupedListingsCarousel, { groups: groups }), _jsx(RelatedItemsSection, { relatedItems: relatedItems, relatedByBrand: relatedByBrand, relatedByTags: relatedByTags, relatedByStore: relatedByStore, categoryLabel: categoryName || category || undefined, brandLabel: brand || undefined, storeLabel: storeName || undefined })] })) }), renderPrimaryActions
+                            : undefined, renderRelated: () => (_jsxs(Stack, { gap: "xl", children: [_jsx(GroupedListingsCarousel, { groups: groups }), _jsx(RelatedItemsSection, { relatedItems: relatedItems, relatedByBrand: relatedByBrand, relatedByTags: relatedByTags, relatedByStore: relatedByStore, categoryLabel: categoryNames[0] || categoryName || category || undefined, brandLabel: brand || undefined, storeLabel: storeName || undefined })] })) }), renderPrimaryActions
                         ? renderPrimaryActions({
                             productId: product.id,
                             productSlug: product.slug ?? slug,

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

@@ -130,8 +130,35 @@ export class ProductRepository extends BaseRepository {
     async deriveTaxonomy(input) {
         const out = {};
         const cats = this.db.collection(CATEGORIES_COLLECTION);
-        const leaf = (Array.isArray(input.categorySlugs) && input.categorySlugs[0]) ||
-            input.category ||
+        /*
+         * 🛑 THE SCALAR `category` WINS. It used to be the other way round, and a
+         * seller could not re-categorise a listing at all.
+         *
+         * `categorySlugs` is a value this function DERIVES; `category` is the leaf a
+         * category picker actually writes. The seller edit form sends both — its
+         * draft carries the loaded product's `categorySlugs` untouched while the
+         * picker updates `category` — so a payload mid-edit reads, measured off the
+         * real request:
+         *
+         *     "categorySlugs": ["category-burst-superking"]     <- STALE, derived
+         *     "category":      "category-original-plastic-gen"  <- what the user picked
+         *
+         * Preferring `categorySlugs[0]` therefore resolved the OLD leaf, re-derived
+         * the OLD chain from it, and then line `out.category = leaf` wrote the old
+         * value back OVER the user's choice. The save returned success, the form
+         * re-opened on the original category, the product page kept its old chips and
+         * the new category page never listed the item — with nothing logged anywhere,
+         * because from this function's point of view it had resolved a leaf perfectly
+         * well. It was only ever the WRONG leaf.
+         *
+         * Reversing the precedence is safe for every other caller: anyone who passes
+         * `categorySlugs` without a `category` (the seed, an admin PATCH) still falls
+         * through to it, and anyone passing a consistent pair gets the same answer
+         * either way. The two can only disagree when one of them is stale, and the
+         * derived one is the one that goes stale.
+         */
+        const leaf = input.category ||
+            (Array.isArray(input.categorySlugs) && input.categorySlugs[0]) ||
             null;
         /*
          * The brand selector's option value is the brand ROW's id (`brand-beyblade`),

package/dist/features/seller/actions/offer-actions.js

@@ -48,6 +48,38 @@ export async function makeOffer(userId, userEmail, input) {
     const product = await productRepository.findById(productId);
     if (!product)
         throw new NotFoundError(ERROR_MESSAGES.PRODUCT.NOT_FOUND);
+    /*
+     * 🛑 A seller may not offer on their OWN listing. There was no such check.
+     *
+     * Measured on production as tyson@beybladearena.in, who owns
+     * store-beyblade-arena: product-beyblade-burst-valkyrie — sold by that very
+     * store — offered a live 'Make Offer' control, and clicking it opened the
+     * full form ("Listed at ₹999 · Minimum offer: ₹699.3") with a working
+     * 'Send offer of ₹899.1' button and the buyer-facing line "The seller will
+     * accept, decline, or suggest a counter price" shown TO that seller.
+     *
+     * The three gates above it are all real but none of them is about identity:
+     * the type's `canMakeOffer`, the seller's `allowOffers` opt-in, and the
+     * amount bounds. So a self-offer would have been accepted and written, and
+     * the same person would then appear on both sides of a negotiation — able to
+     * accept their own offer and, through the offer lane, mint a locked cart line
+     * at any price down to the minimum floor. That is a pricing bypass, not a
+     * cosmetic one.
+     *
+     * The check is HERE rather than only in the UI on purpose (Root Cause #46): a
+     * hidden button is not an authorization boundary, and the control was already
+     * reachable. `product.storeId` is the store SLUG while `userId` is an Auth
+     * uid — two different namespaces by design — so the owner is resolved through
+     * `findByOwnerId`, exactly as `sellerUpdateProduct` does.
+     *
+     * Admins are not exempted: there is no legitimate reason for anyone to
+     * negotiate with themselves, and an admin wanting to adjust a price has the
+     * listing editor.
+     */
+    const callerStore = await storeRepository.findByOwnerId(userId);
+    if (callerStore && callerStore.id === product.storeId) {
+        throw new ValidationError("You can't make an offer on your own listing — edit its price instead.");
+    }
     const listingType = normalizeListingType(product);
     // Some listing types cannot be negotiated at all — an auction already has
     // bidding, a prize-draw sells fixed-price entries. This was never checked, so

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

@@ -13,6 +13,16 @@ function toRupeesString(price) {
 function fromRupeesString(rupeeStr) {
     return Math.round((parseFloat(rupeeStr) || 0) * 100) / 100;
 }
+/**
+ * Mirrors `productBaseSchema.description` (`z.string().min(20).max(5000)`).
+ *
+ * 🛑 This validator already mirrored every other server constraint — title
+ * length, price, image, category — and simply omitted description, so the
+ * server was the first thing to notice and it answered in Zod's own words:
+ * "Invalid input: expected string, received undefined", printed under a field
+ * whose placeholder read "(optional)".
+ */
+const DESCRIPTION_MIN = 20;
 function validate(values) {
     const errors = {};
     if (!values.title?.trim() || (values.title.trim().length < 3)) {
@@ -27,6 +37,10 @@ function validate(values) {
     if (!values.category?.trim()) {
         errors.category = "Category is required";
     }
+    const description = values.description?.trim() ?? "";
+    if (description.length < DESCRIPTION_MIN) {
+        errors.description = `Description must be at least ${DESCRIPTION_MIN} characters`;
+    }
     return errors;
 }
 export function QuickProductForm({ values, onChange, onPublish, onSave, onSwitchToFull, isLoading, renderCategorySelector, onUploadImage, }) {
@@ -51,5 +65,5 @@ export function QuickProductForm({ values, onChange, onPublish, onSave, onSwitch
     return (_jsxs(Stack, { gap: "lg", className: `${__P.p5}`, children: [_jsx(Div, { children: _jsx(Text, { size: "sm", color: "muted", children: "Quick add \u2014 fill the essentials and publish. You can add more details later." }) }), touched && Object.keys(errors).length > 0 && (_jsx(Alert, { variant: "error", children: "Please fix the highlighted fields before publishing." })), _jsx(FormField, { name: "title", label: "Product Name", type: "text", value: values.title ?? "", onChange: (v) => onChange({ title: v }), placeholder: "e.g. Charizard Base Set PSA 9", required: true, error: touched ? errors.title : undefined }), renderCategorySelector ? (_jsxs(Div, { children: [_jsx(Text, { className: "mb-1", color: "muted", size: "sm", weight: "medium", children: "Category" }), renderCategorySelector({
                         value: values.category ?? "",
                         onChange: (v) => onChange({ category: v }),
-                    }), touched && errors.category && (_jsx(Text, { className: "text-[var(--appkit-color-error)] mt-1", size: "xs", children: errors.category }))] })) : (_jsx(FormField, { name: "category", label: "Category", type: "text", value: values.category ?? "", onChange: (v) => onChange({ category: v }), placeholder: "e.g. Trading Cards", error: touched ? errors.category : undefined })), _jsx(FormField, { name: "price", label: "Price (\u20B9)", type: "number", value: toRupeesString(values.price), onChange: (v) => onChange({ price: fromRupeesString(v) }), placeholder: "e.g. 499", required: true, error: touched ? errors.price : undefined }), _jsx(FormField, { name: "mainImage", label: "Product Image", type: "image", value: values.mainImage ?? "", onChange: (v) => onChange({ mainImage: v }), onUpload: onUploadImage, required: true, error: touched ? errors.mainImage : undefined }), _jsx(FormField, { name: "description", label: "Description", type: "textarea", value: values.description ?? "", onChange: (v) => onChange({ description: v }), placeholder: "Brief description (optional)", rows: 3 }), _jsx(FormField, { name: "stockQuantity", label: "Stock Quantity", type: "number", value: String(values.stockQuantity ?? 1), onChange: (v) => onChange({ stockQuantity: Math.max(0, parseInt(v, 10) || 0) }), placeholder: "1", hint: "How many units are available" }), _jsxs(Stack, { gap: "3", padding: "t-xs", children: [_jsx(FormErrorSummary, {}), _jsxs(Row, { gap: "3", children: [_jsx(Button, { variant: "primary", onClick: handlePublish, disabled: isLoading, className: "flex-1", children: isLoading ? "Publishing..." : "Publish" }), _jsx(Button, { variant: "secondary", onClick: handleSave, disabled: isLoading, children: "Save Draft" })] }), _jsx(Button, { variant: "ghost", size: "sm", textSize: "xs", onClick: onSwitchToFull, className: "self-center", children: "Show all fields (advanced)" })] })] }));
+                    }), touched && errors.category && (_jsx(Text, { className: "text-[var(--appkit-color-error)] mt-1", size: "xs", children: errors.category }))] })) : (_jsx(FormField, { name: "category", label: "Category", type: "text", value: values.category ?? "", onChange: (v) => onChange({ category: v }), placeholder: "e.g. Trading Cards", error: touched ? errors.category : undefined })), _jsx(FormField, { name: "price", label: "Price (\u20B9)", type: "number", value: toRupeesString(values.price), onChange: (v) => onChange({ price: fromRupeesString(v) }), placeholder: "e.g. 499", required: true, error: touched ? errors.price : undefined }), _jsx(FormField, { name: "mainImage", label: "Product Image", type: "image", value: values.mainImage ?? "", onChange: (v) => onChange({ mainImage: v }), onUpload: onUploadImage, required: true, error: touched ? errors.mainImage : undefined }), _jsx(FormField, { name: "description", label: "Description", type: "textarea", required: true, value: values.description ?? "", onChange: (v) => onChange({ description: v }), placeholder: "What is it, what condition is it in, what is included? (at least 20 characters)", rows: 3, error: touched ? errors.description : undefined }), _jsx(FormField, { name: "stockQuantity", label: "Stock Quantity", type: "number", value: String(values.stockQuantity ?? 1), onChange: (v) => onChange({ stockQuantity: Math.max(0, parseInt(v, 10) || 0) }), placeholder: "1", hint: "How many units are available" }), _jsxs(Stack, { gap: "3", padding: "t-xs", children: [_jsx(FormErrorSummary, {}), _jsxs(Row, { gap: "3", children: [_jsx(Button, { variant: "primary", onClick: handlePublish, disabled: isLoading, className: "flex-1", children: isLoading ? "Publishing..." : "Publish" }), _jsx(Button, { variant: "secondary", onClick: handleSave, disabled: isLoading, children: "Save Draft" })] }), _jsx(Button, { variant: "ghost", size: "sm", textSize: "xs", onClick: onSwitchToFull, className: "self-center", children: "Show all fields (advanced)" })] })] }));
 }

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

@@ -10,7 +10,9 @@ import { SELLER_ENDPOINTS } from "../../../constants/api-endpoints";
 import { ACTIONS } from "../../../_internal/shared/actions/action-registry";
 import { buildBulkAction } from "../../../_internal/shared/actions/bulk-helpers";
 import { SELLER_BID_STATUS_TABS } from "../../admin/constants/filter-tabs";
-import { toRecordArray, toRelativeDate, toCurrency, toStringValue } from "../hooks/useSellerListingData";
+import { toRecordArray, toCurrency, toStringValue } from "../hooks/useSellerListingData";
+// Defining module (Root Cause #18) — the seller-side duplicate was deleted for drift.
+import { toRelativeDate } from "../../admin/hooks/useAdminListingData";
 import { DataTable } from "../../admin/components/DataTable";
 import { DataListingView } from "../../admin/components/DataListingView";
 import { TextLink } from "../../../ui";

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

@@ -8,7 +8,7 @@ import { QuickFormDrawer } from "../../shell/QuickFormDrawer";
 import { OfferPhaseTimeline } from "./OfferPhaseTimeline";
 import { counterOfferFormSchema } from "../schemas/offer-forms";
 import { SELLER_ENDPOINTS } from "../../../constants/api-endpoints";
-import { SELLER_OFFER_STATUS_TABS } from "../../admin/constants/filter-tabs";
+import { SELLER_OFFER_STATUS_TABS, offerStatusLabel } from "../../admin/constants/filter-tabs";
 import { ACTIONS } from "../../../_internal/shared/actions/action-registry";
 import { toRecordArray, toRelativeDate, toCurrency, toStringValue, } from "../../admin/hooks/useAdminListingData";
 import { DataListingView } from "../../admin/components/DataListingView";
@@ -49,7 +49,7 @@ export function SellerOffersView({ children, onAcceptOffer, onRejectOffer, onCou
                 `Listed: ${toCurrency(item.listedPrice)}`,
                 toStringValue(item.buyerName ?? "Unknown buyer", "Unknown buyer"),
             ].join(" · "),
-            status: toStringValue(item.status, "Pending"),
+            status: offerStatusLabel(toStringValue(item.status, "pending"), "seller"),
             updatedAt: toRelativeDate(item.updatedAt ?? item.createdAt),
             detail: item,
         })),

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

@@ -22,7 +22,9 @@ import { applyZodIssues } from "../../../ui/forms/apply-zod-issues";
 import { FormErrorSummary } from "../../../ui/forms/FormErrorSummary";
 import { PhysicalLocationModal } from "./PhysicalLocationModal";
 import { ROUTES } from "../../../constants";
-import { toRecordArray, toRelativeDate, toCurrency, toStringValue } from "../hooks/useSellerListingData";
+import { toRecordArray, toCurrency, toStringValue } from "../hooks/useSellerListingData";
+// Defining module (Root Cause #18) — the seller-side duplicate was deleted for drift.
+import { toRelativeDate } from "../../admin/hooks/useAdminListingData";
 import { DataListingView } from "../../admin/components/DataListingView";
 import { MediaImage } from "../../media/MediaImage";
 const __O = {
@@ -119,7 +121,22 @@ export function SellerOrderDetailPanel({ orderId, apiBase, onClose, }) {
             setDraft({
                 status: "",
                 trackingNumber: o.trackingNumber ?? "",
-                carrier: o.carrier ?? "",
+                /*
+                 * 🛑 `shippingCarrier` is the stored field. `OrderDocument` declares
+                 * it at line ~422 and declares no `carrier` at all.
+                 *
+                 * The WRITE below already gets this right (`payload.shippingCarrier`),
+                 * so the carrier has been persisting correctly all along — verified
+                 * against the live endpoint after shipping an order: `shippingCarrier:
+                 * "QA Carrier"`, `carrier: undefined`. Only the read-back was wrong, so
+                 * reopening a shipped order showed an EMPTY Carrier box next to a
+                 * populated tracking number. A seller would reasonably retype it, and
+                 * the diff-against-loaded-state payload means retyping the same value
+                 * would then look like no change at all.
+                 *
+                 * Not a lost write — a field the form could not see.
+                 */
+                carrier: o.shippingCarrier ?? o.carrier ?? "",
                 trackingUrl: o.trackingUrl ?? "",
             });
         })
@@ -191,7 +208,10 @@ export function SellerOrderDetailPanel({ orderId, apiBase, onClose, }) {
             setDraft({
                 status: "",
                 trackingNumber: next.trackingNumber ?? "",
-                carrier: next.carrier ?? "",
+                // Same stored-field name as the initial load above — this re-seed is
+                // explicitly "against the saved state", so reading the wrong field here
+                // makes every subsequent diff think the carrier had been cleared.
+                carrier: next.shippingCarrier ?? next.carrier ?? "",
                 trackingUrl: next.trackingUrl ?? "",
             });
         }
@@ -203,9 +223,32 @@ export function SellerOrderDetailPanel({ orderId, apiBase, onClose, }) {
             setSaving(false);
         }
     };
-    const addr = order?.shippingAddress ?? {};
-    const addrLine = [addr.addressLine1, addr.city, addr.state, addr.pincode].filter(Boolean).join(", ");
-    return (_jsxs(_Fragment, { children: [loading && (_jsx(Row, { align: "center", justify: "center", padding: "y-4xl", children: _jsx(Div, { className: "h-6 w-6 animate-spin border-2 border-[var(--appkit-color-primary)] border-t-transparent", rounded: "full" }) })), fetchError && (_jsx(Div, { textSize: "sm", className: "mx-4 mt-4 border border-error/20", color: "error", surface: "danger-surface", padding: "inline", rounded: "lg", children: fetchError })), order && !loading && (_jsxs(Stack, { gap: "none", children: [_jsxs(Stack, { className: `flex-1 ${__O.yAuto}`, gap: "5", padding: "md", children: [_jsxs(Row, { align: "center", justify: "between", children: [_jsx(Badge, { variant: STATUS_BADGE_VARIANT[order.status ?? ""] ?? "default", children: order.status ?? "Unknown" }), _jsx(Text, { size: "sm", className: "text-[var(--appkit-color-text-secondary)]", children: toRelativeDate(order.createdAt) })] }), (order.items ?? []).length > 0 && (_jsxs(Div, { children: [_jsx(Text, { size: "sm", className: "text-[var(--appkit-color-text-primary)] mb-2", weight: "semibold", children: "Items" }), _jsx(Div, { className: "divide-y divide-[var(--appkit-color-border)] divide-[var(--appkit-color-border)] border border-[var(--appkit-color-border)]", rounded: "lg", children: (order.items ?? []).map((item, i) => (_jsxs(Row, { paddingY: "y-xs-tall", padding: "x-sm", align: "center", justify: "between", gap: "3", children: [_jsxs(Row, { align: "center", gap: "sm", className: "min-w-0", children: [_jsx(Div, { className: "h-10 w-10 shrink-0", rounded: "md", overflow: "hidden", children: _jsx(MediaImage, { src: item.image, alt: item.title ?? "Order item", size: "thumbnail" }) }), _jsxs(Div, { className: "min-w-0", children: [_jsx(Text, { size: "sm", className: "truncate", weight: "medium", children: item.title ?? item.productId ?? "Item" }), _jsxs(Text, { size: "xs", className: "text-[var(--appkit-color-text-secondary)]", children: ["Qty: ", item.quantity ?? 1] })] })] }), _jsx(Text, { size: "sm", className: "shrink-0", weight: "medium", children: toCurrency(item.price ?? 0) })] }, i))) })] })), _jsx(OrderAddonBadges, { order: order, variant: "detail" }), order.giftWrapAddon && (_jsxs(Div, { className: "border border-[var(--appkit-color-primary-200)] dark:border-[var(--appkit-color-primary-800)]", surface: "subtle", padding: "inline", rounded: "lg", children: [_jsx(Text, { size: "sm", weight: "semibold", children: "\uD83C\uDF81 Gift wrap requested" }), order.giftWrapMessage && (_jsxs(Text, { size: "sm", className: "mt-1 text-[var(--appkit-color-text-secondary)]", children: ["\u201C", order.giftWrapMessage, "\u201D"] }))] })), _jsxs(Row, { surface: "muted", padding: "inline", align: "center", justify: "between", rounded: "lg", children: [_jsx(Text, { size: "sm", weight: "semibold", children: "Total" }), _jsx(Text, { size: "sm", className: "text-[var(--appkit-color-primary)]", weight: "bold", children: toCurrency(order.totalAmount ?? 0) })] }), addrLine && (_jsxs(Div, { children: [_jsx(Text, { size: "sm", className: "mb-1", weight: "semibold", children: "Shipping address" }), _jsx(Text, { size: "sm", className: "text-[var(--appkit-color-text-secondary)]", children: [String(addr.fullName ?? ""), addrLine].filter(Boolean).join(" · ") })] })), order.paymentMethod && (_jsxs(Div, { children: [_jsxs(Row, { align: "center", justify: "between", className: "mb-1", gap: "3", children: [_jsx(Text, { size: "sm", weight: "semibold", children: "Payment" }), isManualPaymentMethod(order.paymentMethod) && (_jsx(Badge, { variant: sellerPaymentBadge(order).variant, children: sellerPaymentBadge(order).label }))] }), _jsx(Text, { size: "sm", className: "text-[var(--appkit-color-text-secondary)]", transform: "capitalize", children: order.paymentMethod }), isManualPaymentMethod(order.paymentMethod) && order.paymentTransactionId && (_jsxs(Text, { size: "xs", color: "muted", className: "mt-1", children: ["UTR: ", order.paymentTransactionId] }))] })), order.emiEnabled && (_jsxs(Div, { className: "border-t border-[var(--appkit-color-border)]", padding: "t-md", children: [_jsxs(Row, { align: "center", justify: "between", className: "mb-2", children: [_jsxs(Text, { size: "sm", weight: "semibold", children: ["EMI plan ", order.emiTenureMonths ? `· ${order.emiTenureMonths} months` : ""] }), _jsx(Badge, { variant: order.emiComplete ? "success" : "warning", children: order.emiComplete ? "Fully paid" : "In progress" })] }), _jsxs(Row, { align: "center", justify: "between", className: "mb-2", children: [_jsx(Text, { size: "xs", color: "muted", children: "Token collected" }), _jsx(Text, { size: "xs", weight: "medium", children: toCurrency(order.emiTokenAmount ?? 0) })] }), _jsxs(Row, { align: "center", justify: "between", className: "mb-3", children: [_jsx(Text, { size: "xs", color: "muted", children: "Remaining balance" }), _jsx(Text, { size: "xs", weight: "medium", children: toCurrency(order.emiRemainingBalance ?? 0) })] }), _jsx(Div, { className: "divide-y divide-[var(--appkit-color-border)] border border-[var(--appkit-color-border)]", rounded: "lg", children: (order.emiInstallments ?? []).map((inst) => (_jsxs(Row, { paddingY: "y-xs-tall", padding: "x-sm", align: "center", justify: "between", gap: "3", children: [_jsxs(Div, { className: "min-w-0", children: [_jsxs(Text, { size: "sm", weight: "medium", children: ["Installment ", inst.index, " \u00B7 ", toCurrency(inst.amount)] }), _jsx(Text, { size: "xs", color: "muted", children: inst.status === "paid" && inst.paidAt
+    /*
+     * 🛑 `shippingAddress` is sometimes a STRING, and destructuring one yields
+     * nothing but silence.
+     *
+     * Measured on both a seeded order and one placed through real checkout,
+     * the single-order store endpoint returns it pre-formatted:
+     *
+     *     "Mock User 3, 123 Stadium Lane, Vijay Nagar, Indore, Madhya Pradesh, 452010, India"
+     *
+     * Reading `.addressLine1` / `.city` / `.state` / `.pincode` off a string gives
+     * four undefineds, `filter(Boolean)` drops them all, and `addrLine` became "".
+     * The block below renders only when `addrLine` is truthy — so the seller was
+     * shown NO shipping destination at all for an order they are expected to post,
+     * with no error and no empty state. (`Object.keys()` on it returns "0".."80",
+     * which is the tell: those are string indices, not fields.)
+     *
+     * Handle both shapes rather than picking one: the object form is what the
+     * schema declares and what newer writes produce, and a seller needs the
+     * address either way.
+     */
+    const rawAddr = order?.shippingAddress;
+    const addr = (typeof rawAddr === "object" && rawAddr !== null ? rawAddr : {});
+    const addrLine = typeof rawAddr === "string" && rawAddr.trim()
+        ? rawAddr.trim()
+        : [addr.addressLine1, addr.city, addr.state, addr.pincode].filter(Boolean).join(", ");
+    return (_jsxs(_Fragment, { children: [loading && (_jsx(Row, { align: "center", justify: "center", padding: "y-4xl", children: _jsx(Div, { className: "h-6 w-6 animate-spin border-2 border-[var(--appkit-color-primary)] border-t-transparent", rounded: "full" }) })), fetchError && (_jsx(Div, { textSize: "sm", className: "mx-4 mt-4 border border-error/20", color: "error", surface: "danger-surface", padding: "inline", rounded: "lg", children: fetchError })), order && !loading && (_jsxs(Stack, { gap: "none", children: [_jsxs(Stack, { className: `flex-1 ${__O.yAuto}`, gap: "5", padding: "md", children: [_jsxs(Row, { align: "center", justify: "between", children: [_jsx(Badge, { variant: STATUS_BADGE_VARIANT[order.status ?? ""] ?? "default", children: order.status ?? "Unknown" }), _jsx(Text, { size: "sm", className: "text-[var(--appkit-color-text-secondary)]", children: toRelativeDate(order.createdAt) })] }), (order.items ?? []).length > 0 && (_jsxs(Div, { children: [_jsx(Text, { size: "sm", className: "text-[var(--appkit-color-text-primary)] mb-2", weight: "semibold", children: "Items" }), _jsx(Div, { className: "divide-y divide-[var(--appkit-color-border)] divide-[var(--appkit-color-border)] border border-[var(--appkit-color-border)]", rounded: "lg", children: (order.items ?? []).map((item, i) => (_jsxs(Row, { paddingY: "y-xs-tall", padding: "x-sm", align: "center", justify: "between", gap: "3", children: [_jsxs(Row, { align: "center", gap: "sm", className: "min-w-0", children: [_jsx(Div, { className: "h-10 w-10 shrink-0", rounded: "md", overflow: "hidden", children: _jsx(MediaImage, { src: item.image, alt: item.title ?? "Order item", size: "thumbnail" }) }), _jsxs(Div, { className: "min-w-0", children: [_jsx(Text, { size: "sm", className: "truncate", weight: "medium", children: item.productTitle ?? item.title ?? item.productId ?? "Item" }), _jsxs(Text, { size: "xs", className: "text-[var(--appkit-color-text-secondary)]", children: ["Qty: ", item.quantity ?? 1] })] })] }), _jsx(Text, { size: "sm", className: "shrink-0", weight: "medium", children: toCurrency(item.totalPrice ?? item.unitPrice ?? item.price ?? 0) })] }, i))) })] })), _jsx(OrderAddonBadges, { order: order, variant: "detail" }), order.giftWrapAddon && (_jsxs(Div, { className: "border border-[var(--appkit-color-primary-200)] dark:border-[var(--appkit-color-primary-800)]", surface: "subtle", padding: "inline", rounded: "lg", children: [_jsx(Text, { size: "sm", weight: "semibold", children: "\uD83C\uDF81 Gift wrap requested" }), order.giftWrapMessage && (_jsxs(Text, { size: "sm", className: "mt-1 text-[var(--appkit-color-text-secondary)]", children: ["\u201C", order.giftWrapMessage, "\u201D"] }))] })), _jsxs(Row, { surface: "muted", padding: "inline", align: "center", justify: "between", rounded: "lg", children: [_jsx(Text, { size: "sm", weight: "semibold", children: "Total" }), _jsx(Text, { size: "sm", className: "text-[var(--appkit-color-primary)]", weight: "bold", children: toCurrency(order.totalAmount ?? order.totalPrice ?? 0) })] }), addrLine && (_jsxs(Div, { children: [_jsx(Text, { size: "sm", className: "mb-1", weight: "semibold", children: "Shipping address" }), _jsx(Text, { size: "sm", className: "text-[var(--appkit-color-text-secondary)]", children: [String(addr.fullName ?? ""), addrLine].filter(Boolean).join(" · ") })] })), order.paymentMethod && (_jsxs(Div, { children: [_jsxs(Row, { align: "center", justify: "between", className: "mb-1", gap: "3", children: [_jsx(Text, { size: "sm", weight: "semibold", children: "Payment" }), isManualPaymentMethod(order.paymentMethod) && (_jsx(Badge, { variant: sellerPaymentBadge(order).variant, children: sellerPaymentBadge(order).label }))] }), _jsx(Text, { size: "sm", className: "text-[var(--appkit-color-text-secondary)]", transform: "capitalize", children: order.paymentMethod }), isManualPaymentMethod(order.paymentMethod) && order.paymentTransactionId && (_jsxs(Text, { size: "xs", color: "muted", className: "mt-1", children: ["UTR: ", order.paymentTransactionId] }))] })), order.emiEnabled && (_jsxs(Div, { className: "border-t border-[var(--appkit-color-border)]", padding: "t-md", children: [_jsxs(Row, { align: "center", justify: "between", className: "mb-2", children: [_jsxs(Text, { size: "sm", weight: "semibold", children: ["EMI plan ", order.emiTenureMonths ? `· ${order.emiTenureMonths} months` : ""] }), _jsx(Badge, { variant: order.emiComplete ? "success" : "warning", children: order.emiComplete ? "Fully paid" : "In progress" })] }), _jsxs(Row, { align: "center", justify: "between", className: "mb-2", children: [_jsx(Text, { size: "xs", color: "muted", children: "Token collected" }), _jsx(Text, { size: "xs", weight: "medium", children: toCurrency(order.emiTokenAmount ?? 0) })] }), _jsxs(Row, { align: "center", justify: "between", className: "mb-3", children: [_jsx(Text, { size: "xs", color: "muted", children: "Remaining balance" }), _jsx(Text, { size: "xs", weight: "medium", children: toCurrency(order.emiRemainingBalance ?? 0) })] }), _jsx(Div, { className: "divide-y divide-[var(--appkit-color-border)] border border-[var(--appkit-color-border)]", rounded: "lg", children: (order.emiInstallments ?? []).map((inst) => (_jsxs(Row, { paddingY: "y-xs-tall", padding: "x-sm", align: "center", justify: "between", gap: "3", children: [_jsxs(Div, { className: "min-w-0", children: [_jsxs(Text, { size: "sm", weight: "medium", children: ["Installment ", inst.index, " \u00B7 ", toCurrency(inst.amount)] }), _jsx(Text, { size: "xs", color: "muted", children: inst.status === "paid" && inst.paidAt
                                                                 ? `Paid ${toRelativeDate(inst.paidAt)}`
                                                                 : inst.dueDate
                                                                     ? `Due ${toRelativeDate(inst.dueDate)}`

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

@@ -3,6 +3,7 @@ import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { Badge, Button, Checkbox, Div, Grid, Row, Stack, Text, TextLink } from "../../../ui";
 import { MediaImage } from "../../media/MediaImage";
 import { ROW_ACTION_META, ROW_ACTION_ID } from "../../../features/products/constants/action-defs";
+import { ACTIONS } from "../../../_internal/shared/actions/action-registry";
 import { listingBadgeVariant } from "../../products/utils/listing-badge-variant";
 const __P = {
     p3: "p-[var(--appkit-space-3)]",
@@ -17,6 +18,27 @@ function buildHref(row) {
         return `/pre-orders/${row.id}`;
     return `/products/${row.id}`;
 }
+/*
+ * 🛑 THE DELETE BUTTONS BELOW CARRY `action`, AND THAT IS WHAT CONFIRMS THEM.
+ *
+ * They took their LABEL from `ROW_ACTION_META[ROW_ACTION_ID.DELETE]` and
+ * passed no `action` prop, so `<Button>`'s confirmation gate — which fires
+ * only when `action.confirmation` is set — never engaged. One click on a
+ * row's Delete permanently removed a PUBLISHED listing with no dialog of any
+ * kind; measured on a real seller account, the row was gone immediately and
+ * still gone after a reload.
+ *
+ * Using the registry for the label while bypassing it for the confirmation is
+ * exactly the split Rule #7 describes, on the one action where that rule says
+ * the cost is data loss. `ACTIONS.STORE["delete-listing"]` already carried the
+ * copy ("Delete listing? / This listing will be permanently removed. This
+ * action cannot be undone.") and the bulk bar already resolved it correctly;
+ * only the row path did not.
+ *
+ * `variant="ghost"` stays explicit so the row keeps its quiet styling: Button
+ * resolves the variant as `variant ?? kindDefault`, so the caller wins and the
+ * danger kind drives the dialog without turning the row button red.
+ */
 export function SellerProductsCards({ view, rows, isLoading, listingKind, selectedIds, toggle, onEdit, onDuplicate, onDelete, }) {
     const Container = view === "grid" ? Grid : Stack;
     const containerProps = view === "grid"
@@ -25,6 +47,6 @@ export function SellerProductsCards({ view, rows, isLoading, listingKind, select
     return (_jsxs(Container, { ...containerProps, children: [rows.length === 0 && !isLoading && (_jsx(Text, { className: "text-[var(--appkit-color-text-muted)] col-span-full", size: "sm", children: listingKind !== "all" ? `No ${listingKind} listings found` : "No products listed yet" })), rows.map((row) => {
                 const isSelected = selectedIds.has(row.id);
                 const href = buildHref(row);
-                return view === "grid" ? (_jsxs(Div, { className: "group relative bg-[var(--appkit-color-surface)] overflow-hidden", shadow: "hover-md", rounded: "xl", border: isSelected ? "strong" : "default", children: [_jsx(Div, { className: "absolute top-2 left-2 z-10", children: _jsx(Checkbox, { bare: true, checked: isSelected, onChange: () => toggle(row.id), className: "h-4 w-4 rounded border-[var(--appkit-color-border)]", onClick: (e) => e.stopPropagation(), "aria-label": "Select" }) }), _jsxs(TextLink, { href: href, className: "block", children: [_jsx(Div, { className: "w-full aspect-square", children: _jsx(MediaImage, { src: row.imageUrl, alt: row.primary, size: "card" }) }), _jsxs(Stack, { className: `${__P.p3}`, gap: "xs", children: [_jsx(Text, { className: "line-clamp-1", size: "sm", weight: "medium", children: row.primary }), _jsxs(Row, { gap: "sm", children: [_jsx(Badge, { variant: listingBadgeVariant(row.listingKind), children: row.listingKind }), _jsx(Text, { className: "text-[var(--appkit-color-text-muted)] line-clamp-1", size: "xs", children: row.secondary })] })] })] }), _jsxs(Row, { className: "border-t border-[var(--appkit-color-border)]", justify: "end", gap: "xs", padding: "xs", children: [_jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); onEdit(row); }, "aria-label": "Edit", children: ROW_ACTION_META[ROW_ACTION_ID.EDIT].label }), _jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); onDuplicate(row); }, "aria-label": "Duplicate", children: ROW_ACTION_META[ROW_ACTION_ID.DUPLICATE].label }), onDelete && (_jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); onDelete(row); }, "aria-label": "Delete", children: ROW_ACTION_META[ROW_ACTION_ID.DELETE].label }))] })] }, row.id)) : (_jsxs(Row, { className: "bg-[var(--appkit-color-surface)] hover:bg-[var(--appkit-color-surface-raised)]", align: "center", gap: "3", rounded: "lg", padding: "inlineSm", border: isSelected ? "strong" : "default", children: [_jsx(Checkbox, { bare: true, checked: isSelected, onChange: () => toggle(row.id), className: "h-4 w-4 rounded border-[var(--appkit-color-border)]", "aria-label": "Select" }), _jsx(Div, { className: `w-12 h-12 ${__O.hidden}`, rounded: "default", children: _jsx(MediaImage, { src: row.imageUrl, alt: row.primary, size: "thumbnail" }) }), _jsxs(TextLink, { href: href, className: "flex-1 min-w-0", children: [_jsx(Text, { className: "line-clamp-1", size: "sm", weight: "medium", children: row.primary }), _jsxs(Row, { gap: "sm", children: [_jsx(Badge, { variant: listingBadgeVariant(row.listingKind), children: row.listingKind }), _jsx(Text, { className: "text-[var(--appkit-color-text-muted)] line-clamp-1", size: "xs", children: row.secondary })] })] }), _jsxs(Row, { className: "flex-shrink-0", gap: "xs", children: [_jsx(Button, { variant: "ghost", size: "sm", onClick: () => onEdit(row), "aria-label": "Edit", children: ROW_ACTION_META[ROW_ACTION_ID.EDIT].label }), _jsx(Button, { variant: "ghost", size: "sm", onClick: () => onDuplicate(row), "aria-label": "Duplicate", children: ROW_ACTION_META[ROW_ACTION_ID.DUPLICATE].label }), onDelete && (_jsx(Button, { variant: "ghost", size: "sm", onClick: () => onDelete(row), "aria-label": "Delete", children: ROW_ACTION_META[ROW_ACTION_ID.DELETE].label }))] })] }, row.id));
+                return view === "grid" ? (_jsxs(Div, { className: "group relative bg-[var(--appkit-color-surface)] overflow-hidden", shadow: "hover-md", rounded: "xl", border: isSelected ? "strong" : "default", children: [_jsx(Div, { className: "absolute top-2 left-2 z-10", children: _jsx(Checkbox, { bare: true, checked: isSelected, onChange: () => toggle(row.id), className: "h-4 w-4 rounded border-[var(--appkit-color-border)]", onClick: (e) => e.stopPropagation(), "aria-label": "Select" }) }), _jsxs(TextLink, { href: href, className: "block", children: [_jsx(Div, { className: "w-full aspect-square", children: _jsx(MediaImage, { src: row.imageUrl, alt: row.primary, size: "card" }) }), _jsxs(Stack, { className: `${__P.p3}`, gap: "xs", children: [_jsx(Text, { className: "line-clamp-1", size: "sm", weight: "medium", children: row.primary }), _jsxs(Row, { gap: "sm", children: [_jsx(Badge, { variant: listingBadgeVariant(row.listingKind), children: row.listingKind }), _jsx(Text, { className: "text-[var(--appkit-color-text-muted)] line-clamp-1", size: "xs", children: row.secondary })] })] })] }), _jsxs(Row, { className: "border-t border-[var(--appkit-color-border)]", justify: "end", gap: "xs", padding: "xs", children: [_jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); onEdit(row); }, "aria-label": "Edit", children: ROW_ACTION_META[ROW_ACTION_ID.EDIT].label }), _jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); onDuplicate(row); }, "aria-label": "Duplicate", children: ROW_ACTION_META[ROW_ACTION_ID.DUPLICATE].label }), onDelete && (_jsx(Button, { variant: "ghost", size: "sm", action: ACTIONS.STORE["delete-listing"], onClick: (e) => { e.stopPropagation(); onDelete(row); }, "aria-label": "Delete", children: ROW_ACTION_META[ROW_ACTION_ID.DELETE].label }))] })] }, row.id)) : (_jsxs(Row, { className: "bg-[var(--appkit-color-surface)] hover:bg-[var(--appkit-color-surface-raised)]", align: "center", gap: "3", rounded: "lg", padding: "inlineSm", border: isSelected ? "strong" : "default", children: [_jsx(Checkbox, { bare: true, checked: isSelected, onChange: () => toggle(row.id), className: "h-4 w-4 rounded border-[var(--appkit-color-border)]", "aria-label": "Select" }), _jsx(Div, { className: `w-12 h-12 ${__O.hidden}`, rounded: "default", children: _jsx(MediaImage, { src: row.imageUrl, alt: row.primary, size: "thumbnail" }) }), _jsxs(TextLink, { href: href, className: "flex-1 min-w-0", children: [_jsx(Text, { className: "line-clamp-1", size: "sm", weight: "medium", children: row.primary }), _jsxs(Row, { gap: "sm", children: [_jsx(Badge, { variant: listingBadgeVariant(row.listingKind), children: row.listingKind }), _jsx(Text, { className: "text-[var(--appkit-color-text-muted)] line-clamp-1", size: "xs", children: row.secondary })] })] }), _jsxs(Row, { className: "flex-shrink-0", gap: "xs", children: [_jsx(Button, { variant: "ghost", size: "sm", onClick: () => onEdit(row), "aria-label": "Edit", children: ROW_ACTION_META[ROW_ACTION_ID.EDIT].label }), _jsx(Button, { variant: "ghost", size: "sm", onClick: () => onDuplicate(row), "aria-label": "Duplicate", children: ROW_ACTION_META[ROW_ACTION_ID.DUPLICATE].label }), onDelete && (_jsx(Button, { variant: "ghost", size: "sm", action: ACTIONS.STORE["delete-listing"], onClick: () => onDelete(row), "aria-label": "Delete", children: ROW_ACTION_META[ROW_ACTION_ID.DELETE].label }))] })] }, row.id));
             })] }));
 }

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

@@ -17,7 +17,9 @@ import { SELLER_PRODUCT_STATUS_TABS } from "../../admin/constants/filter-tabs";
 import { ROUTES } from "../../../constants";
 import { normalizeListingType } from "../../products/utils/listing-type";
 import { useAvailabilityScope } from "../../products/hooks/useAvailabilityScope";
-import { toRecordArray, toRelativeDate, toStringValue } from "../hooks/useSellerListingData";
+import { toRecordArray, toStringValue } from "../hooks/useSellerListingData";
+// Defining module (Root Cause #18) — the seller-side duplicate was deleted for drift.
+import { toRelativeDate } from "../../admin/hooks/useAdminListingData";
 import { useListingTypeFlags } from "../../../react/hooks/useListingTypeFlags";
 import { ALL_LISTING_TYPES } from "../../../_internal/shared/listing-types/feature-flags";
 import { pluginFor } from "../../../_internal/shared/listing-types/_registry";
@@ -267,17 +269,40 @@ export function SellerProductsView({ onDeleteProduct, onCreateClick, }) {
                  * the UI simply never presented it.
                  */
                 const statusLabel = toStringValue(item.status, "draft");
+                /*
+                 * 🛑 PRICE AND STOCK GO IN THE VISIBLE LINE TOO — same reason as the
+                 * status note above, one field over.
+                 *
+                 * `price` was already mapped onto the row and correct, and this view
+                 * renders only `primary`, `secondary`, the type chip and the action
+                 * buttons — so it was never displayed. Measured on a real seller
+                 * account with 127 listings: ZERO rupee amounts and zero stock figures
+                 * anywhere in the list. To learn what one of their own listings costs
+                 * or how many are left, a seller had to open it.
+                 *
+                 * Those are the two numbers a seller scans for, so price leads.
+                 * `stockQuantity` was not mapped at all and is added here.
+                 *
+                 * An auction keeps its own summary (reserve / bids / ends) and is not
+                 * given a list price: for an auction the meaningful money is the
+                 * reserve and the current bid, both already in `auctionSecondary`, and
+                 * a "price" beside them would read as a third, competing figure.
+                 */
+                const stockRaw = typeof item.stockQuantity === "number" ? item.stockQuantity : null;
+                const priceLabel = kind !== "auction" && priceRaw ? `₹${priceRaw.toLocaleString("en-IN")}` : "";
+                const stockLabel = stockRaw === null ? "" : `${stockRaw} in stock`;
                 const secondaryBase = auctionSecondary || toStringValue(item.condition, "");
                 return {
                     id: toStringValue(item.id, `product-${index}`),
                     primary: toStringValue(item.title ?? item.name, "Untitled product"),
-                    secondary: [secondaryBase, statusLabel].filter(Boolean).join(" · "),
+                    secondary: [priceLabel, stockLabel, secondaryBase, statusLabel].filter(Boolean).join(" · "),
                     status: statusLabel,
                     updatedAt: toRelativeDate(item.updatedAt ?? item.createdAt),
                     imageUrl: toStringValue(item.mainImage ?? item.images?.[0], undefined),
                     image: toStringValue(item.mainImage ?? item.images?.[0], undefined),
                     listingKind: kind,
                     price: priceRaw ? `₹${priceRaw.toLocaleString("en-IN")}` : "—",
+                    stock: stockLabel || "—",
                     physicalLocation: item.physicalLocation &&
                         typeof item.physicalLocation.zone === "string"
                         ? item.physicalLocation

package/dist/features/seller/hooks/useSellerListingData.d.ts

@@ -14,5 +14,4 @@ type UnknownRecord = Record<string, JsonValue>;
 export declare function toRecordArray(value: unknown): UnknownRecord[];
 export declare function toStringValue(value: unknown, fallback?: string): string;
 export declare function toCurrency(value: unknown): string;
-export declare function toRelativeDate(value: unknown): string;
 export {};

package/dist/features/seller/hooks/useSellerListingData.js

@@ -18,25 +18,23 @@ export function toCurrency(value) {
     }
     return formatCurrency(value);
 }
-export function toRelativeDate(value) {
-    const date = parseDate(value);
-    if (!date) {
-        return "-";
-    }
-    const deltaMs = Date.now() - date.getTime();
-    const minute = 60000;
-    const hour = 60 * minute;
-    const day = 24 * hour;
-    if (deltaMs < minute)
-        return "just now";
-    if (deltaMs < hour)
-        return `${Math.floor(deltaMs / minute)}m ago`;
-    if (deltaMs < day)
-        return `${Math.floor(deltaMs / hour)}h ago`;
-    if (deltaMs < 7 * day)
-        return `${Math.floor(deltaMs / day)}d ago`;
-    return date.toLocaleDateString("en-IN", { year: "numeric", month: "short", day: "numeric" });
-}
+/*
+ * 🛑 `toRelativeDate` WAS HERE AND IS GONE. Import it from
+ * `features/admin/hooks/useAdminListingData` — the defining module now, and
+ * the one 44 files already use (Root Cause #18: import from where a symbol is
+ * defined, never from a convenience copy).
+ *
+ * This copy had drifted from that one and the drift was live: for a FUTURE
+ * date, `deltaMs` is negative, so `deltaMs < minute` matched and this version
+ * answered **"just now"** while the admin version answered **"1m ago"** — two
+ * different wrong answers to "when is this deadline", depending on which file
+ * a view happened to import from. The admin one is fixed to say "in 48h"; a
+ * second implementation is how that fix would have been half-applied.
+ *
+ * The three sibling helpers below are also duplicated there. They are left
+ * alone deliberately: none of them is wrong, and collapsing them is an import
+ * rewrite across both feature trees rather than a bug fix.
+ */
 function parseDate(value) {
     if (value instanceof Date)
         return value;