@mohasinac/appkit 4.13.1 → 4.15.0

package/dist/errors/messages.js

@@ -374,9 +374,12 @@ export const ERROR_MESSAGES = {
         BID_TOO_LOW: "Your bid must be higher than the current bid",
         INCREMENT_TOO_LOW: "Your bid does not meet the minimum increment requirement",
         OWN_AUCTION: "You cannot bid on your own auction",
-        BUY_NOW_UNAVAILABLE: "Buy Now is not available for this auction",
-        BUY_NOW_BIDS_STARTED: "Buy Now is no longer available — bids have already been placed",
+        BUY_NOW_UNAVAILABLE: "Buy Now is unavailable — bidding has already passed the Buy Now price",
         BUY_NOW_NO_PRICE: "This auction does not have a Buy Now price",
+        BUYOUT_LAPSED: "Your Buy Now claim expired. Head back to the auction to bid or try again.",
+        BUYOUT_OUTBID: "Bidding passed the Buy Now price while you were checking out, so this purchase can no longer go through.",
+        AUCTION_ALREADY_SOLD: "Another buyer completed their purchase first, so this auction is already sold.",
+        AUCTION_ENDED_BEFORE_CHECKOUT: "The auction ended before you completed payment, so it went to the winning bidder.",
     },
     CONTACT: {
         SEND_FAILED: "Failed to send your message. Please try again.",
@@ -454,6 +457,11 @@ export const ERROR_MESSAGES = {
     },
     // Offer errors
     OFFER: {
+        /**
+         * The site-wide `featureFlags.offers` kill switch is off. Existing offers
+         * can still be responded to and paid for — only new ones are refused.
+         */
+        DISABLED: "Offers are currently turned off across the site.",
         LIMIT_REACHED: "You have reached the maximum of 3 offers for this product. Wait for the seller to update the listing.",
         COUNTER_RANGE: "Your counter must be within 20% of the seller's counter price.",
         NOT_COUNTERED: "There is no seller counter to respond to.",

package/dist/features/about/components/AboutView.d.ts

@@ -9,6 +9,9 @@ export interface AboutViewProps {
         missionText?: string;
         howItWorksTitle?: string;
         valuesTitle?: string;
+        valuesSubtitle?: string;
+        /** When set, renders a link from the values section through to `/ethics`. */
+        valuesLinkLabel?: string;
         milestonesTitle?: string;
         teamTitle?: string;
         teamSubtitle?: string;

package/dist/features/about/components/AboutView.js

@@ -1,5 +1,6 @@
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
-import { Anchor, Avatar, Badge, Card, Div, Heading, Row, Section, Span, Stack, Text } from "../../../ui";
+import { Anchor, Avatar, Badge, Card, Div, Heading, Row, Section, Span, Stack, Text, TextLink } from "../../../ui";
+import { ROUTES } from "../../../constants";
 export function AboutView({ labels = {}, howItems = [], valueItems = [], milestones = [], teamMembers = [], ctaBannerClass = "", renderCtaButtons, }) {
-    return (_jsxs(Div, { className: "-mx-4 md:-mx-6 lg:-mx-8 -mt-6 sm:-mt-8 lg:-mt-10", children: [_jsx(Section, { color: "inverse", tone: "accent-banner", padding: "y-4xl", children: _jsxs(Div, { paddingX: "x-page", className: "max-w-4xl mx-auto text-center", children: [_jsx(Heading, { color: "inverse", level: 1, variant: "none", className: "mb-6", mdSize: "5xl", size: "4xl", weight: "bold", children: labels.title }), _jsx(Text, { color: "inverse", variant: "none", className: "/80 max-w-2xl mx-auto", size: "xl", children: labels.subtitle })] }) }), _jsxs(Div, { paddingX: "x-page", className: "max-w-5xl mx-auto", padding: "y-4xl", children: [_jsxs(Section, { className: "text-center max-w-3xl mx-auto", children: [_jsx(Heading, { level: 2, className: "mb-4", children: labels.missionTitle }), _jsx(Text, { size: "lg", variant: "secondary", className: "leading-relaxed", children: labels.missionText })] }), _jsxs(Section, { children: [_jsx(Heading, { level: 2, className: "mb-12", align: "center", children: labels.howItWorksTitle }), _jsx(Div, { className: "grid md:grid-cols-3", gap: "6", children: howItems.map(({ title, text, icon, tone }) => (_jsxs(Card, { variant: `gradient-${tone ?? "indigo"}`, padding: "md", spacing: "sm", children: [_jsx(Div, { children: _jsx(Span, { size: "4xl", children: icon }) }), _jsx(Heading, { level: 3, children: title }), _jsx(Text, { size: "sm", variant: "secondary", className: "leading-relaxed", children: text })] }, title))) })] }), _jsxs(Section, { children: [_jsx(Heading, { level: 2, className: "mb-12", align: "center", children: labels.valuesTitle }), _jsx(Div, { className: "grid md:grid-cols-3", gap: "5", children: valueItems.map(({ title, text, icon }) => (_jsxs(Stack, { gap: "3", surface: "default", padding: "lg", rounded: "xl", border: "default", children: [_jsx(Div, { children: _jsx(Span, { size: "3xl", children: icon }) }), _jsx(Heading, { level: 3, children: title }), _jsx(Text, { size: "sm", variant: "secondary", className: "leading-relaxed", children: text })] }, title))) })] }), _jsxs(Section, { children: [_jsx(Heading, { level: 2, className: "mb-10", align: "center", children: labels.milestonesTitle }), _jsx(Stack, { className: "relative border-l-2 border-primary/30 max-w-2xl mx-auto pl-[2rem]", gap: "xl", children: milestones.map(({ year, text }) => (_jsxs(Div, { className: "relative", children: [_jsx(Div, { className: "absolute -left-10 top-1 w-4 h-4 bg-primary border-2 border-white", rounded: "full" }), _jsx(Span, { size: "xs", weight: "bold", className: "text-primary tracking-wide", transform: "uppercase", children: year }), _jsx(Text, { className: "mt-1", children: text })] }, year))) })] }), teamMembers.length > 0 && (_jsxs(Section, { children: [_jsx(Heading, { level: 2, className: "mb-2", align: "center", children: labels.teamTitle }), labels.teamSubtitle && (_jsx(Text, { size: "lg", variant: "secondary", align: "center", className: "mb-12 max-w-2xl mx-auto", children: labels.teamSubtitle })), _jsx(Div, { className: "grid md:grid-cols-3", gap: "6", children: teamMembers.map((member) => (_jsxs(Stack, { gap: "3", surface: "default", padding: "lg", rounded: "xl", border: "default", align: "center", className: "text-center", children: [_jsx(Avatar, { src: member.photoUrl, name: member.name, alt: member.name, size: "lg" }), _jsx(Heading, { level: 3, children: member.name }), _jsxs(Row, { gap: "xs", wrap: true, justify: "center", children: [_jsx(Span, { size: "xs", weight: "bold", className: "text-primary tracking-wide", transform: "uppercase", children: member.role }), member.isFounder && _jsx(Badge, { variant: "primary", size: "xs", children: "Founder" }), member.isDeveloper && _jsx(Badge, { variant: "secondary", size: "xs", children: "Developer" })] }), _jsx(Text, { size: "sm", variant: "secondary", className: "leading-relaxed", children: member.bio }), member.githubUrl && (_jsx(Anchor, { href: member.githubUrl, tone: "muted", size: "sm", underline: "hover", children: "GitHub \u2197" }))] }, member.name))) })] })), _jsxs(Section, { color: "inverse", tone: "accent-banner", className: `text-center ${ctaBannerClass}`, padding: "xl", rounded: "2xl", children: [_jsx(Heading, { color: "inverse", level: 2, variant: "none", className: "mb-8", size: "3xl", weight: "bold", children: labels.ctaTitle }), renderCtaButtons ? (renderCtaButtons()) : (_jsx(Row, { justify: "center", gap: "3", wrap: true, children: _jsxs(Span, { color: "inverse", size: "sm", className: "/70", children: [labels.ctaSell, " \u00B7 ", labels.ctaShop] }) }))] })] })] }));
+    return (_jsxs(Div, { className: "-mx-4 md:-mx-6 lg:-mx-8 -mt-6 sm:-mt-8 lg:-mt-10", children: [_jsx(Section, { color: "inverse", tone: "accent-banner", padding: "y-4xl", children: _jsxs(Div, { paddingX: "x-page", className: "max-w-4xl mx-auto text-center", children: [_jsx(Heading, { color: "inverse", level: 1, variant: "none", className: "mb-6", mdSize: "5xl", size: "4xl", weight: "bold", children: labels.title }), _jsx(Text, { color: "inverse", variant: "none", className: "/80 max-w-2xl mx-auto", size: "xl", children: labels.subtitle })] }) }), _jsxs(Div, { paddingX: "x-page", className: "max-w-5xl mx-auto", padding: "y-4xl", children: [_jsxs(Section, { className: "text-center max-w-3xl mx-auto", children: [_jsx(Heading, { level: 2, className: "mb-4", children: labels.missionTitle }), _jsx(Text, { size: "lg", variant: "secondary", className: "leading-relaxed", children: labels.missionText })] }), _jsxs(Section, { children: [_jsx(Heading, { level: 2, className: "mb-12", align: "center", children: labels.howItWorksTitle }), _jsx(Div, { className: "grid md:grid-cols-3", gap: "6", children: howItems.map(({ title, text, icon, tone }) => (_jsxs(Card, { variant: `gradient-${tone ?? "indigo"}`, padding: "md", spacing: "sm", children: [_jsx(Div, { children: _jsx(Span, { size: "4xl", children: icon }) }), _jsx(Heading, { level: 3, children: title }), _jsx(Text, { size: "sm", variant: "secondary", className: "leading-relaxed", children: text })] }, title))) })] }), _jsxs(Section, { children: [_jsx(Heading, { level: 2, className: labels.valuesSubtitle ? "mb-3" : "mb-12", align: "center", children: labels.valuesTitle }), labels.valuesSubtitle && (_jsx(Text, { size: "base", variant: "secondary", align: "center", className: "mb-12 max-w-2xl mx-auto leading-relaxed", children: labels.valuesSubtitle })), _jsx(Div, { className: "grid md:grid-cols-3", gap: "5", children: valueItems.map(({ title, text, icon, detail }) => (_jsxs(Stack, { gap: "3", surface: "default", padding: "lg", rounded: "xl", border: "default", children: [_jsx(Div, { children: _jsx(Span, { size: "3xl", children: icon }) }), _jsx(Heading, { level: 3, children: title }), _jsx(Text, { size: "sm", variant: "secondary", className: "leading-relaxed", children: text }), detail && (_jsx(Text, { size: "sm", color: "muted", className: "leading-relaxed", children: detail }))] }, title))) }), labels.valuesLinkLabel && (_jsx(Div, { className: "mt-8 text-center", children: _jsx(TextLink, { href: String(ROUTES.PUBLIC.ETHICS), size: "sm", children: labels.valuesLinkLabel }) }))] }), _jsxs(Section, { children: [_jsx(Heading, { level: 2, className: "mb-10", align: "center", children: labels.milestonesTitle }), _jsx(Stack, { className: "relative border-l-2 border-primary/30 max-w-2xl mx-auto pl-[2rem]", gap: "xl", children: milestones.map(({ year, text }) => (_jsxs(Div, { className: "relative", children: [_jsx(Div, { className: "absolute -left-10 top-1 w-4 h-4 bg-primary border-2 border-white", rounded: "full" }), _jsx(Span, { size: "xs", weight: "bold", className: "text-primary tracking-wide", transform: "uppercase", children: year }), _jsx(Text, { className: "mt-1", children: text })] }, year))) })] }), teamMembers.length > 0 && (_jsxs(Section, { children: [_jsx(Heading, { level: 2, className: "mb-2", align: "center", children: labels.teamTitle }), labels.teamSubtitle && (_jsx(Text, { size: "lg", variant: "secondary", align: "center", className: "mb-12 max-w-2xl mx-auto", children: labels.teamSubtitle })), _jsx(Div, { className: "grid md:grid-cols-3", gap: "6", children: teamMembers.map((member) => (_jsxs(Stack, { gap: "3", surface: "default", padding: "lg", rounded: "xl", border: "default", align: "center", className: "text-center", children: [_jsx(Avatar, { src: member.photoUrl, name: member.name, alt: member.name, size: "lg" }), _jsx(Heading, { level: 3, children: member.name }), _jsxs(Row, { gap: "xs", wrap: true, justify: "center", children: [_jsx(Span, { size: "xs", weight: "bold", className: "text-primary tracking-wide", transform: "uppercase", children: member.role }), member.isFounder && _jsx(Badge, { variant: "primary", size: "xs", children: "Founder" }), member.isDeveloper && _jsx(Badge, { variant: "secondary", size: "xs", children: "Developer" })] }), _jsx(Text, { size: "sm", variant: "secondary", className: "leading-relaxed", children: member.bio }), member.githubUrl && (_jsx(Anchor, { href: member.githubUrl, tone: "muted", size: "sm", underline: "hover", children: "GitHub \u2197" }))] }, member.name))) })] })), _jsxs(Section, { color: "inverse", tone: "accent-banner", className: `text-center ${ctaBannerClass}`, padding: "xl", rounded: "2xl", children: [_jsx(Heading, { color: "inverse", level: 2, variant: "none", className: "mb-8", size: "3xl", weight: "bold", children: labels.ctaTitle }), renderCtaButtons ? (renderCtaButtons()) : (_jsx(Row, { justify: "center", gap: "3", wrap: true, children: _jsxs(Span, { color: "inverse", size: "sm", className: "/70", children: [labels.ctaSell, " \u00B7 ", labels.ctaShop] }) }))] })] })] }));
 }

package/dist/features/about/components/PolicyPageView.d.ts

@@ -1,5 +1,6 @@
+export type PolicyKey = "privacy" | "terms" | "cookies" | "refund" | "ethics" | "conduct";
 export interface PolicyPageViewProps {
     /** Which policy to render */
-    policy: "privacy" | "terms" | "cookies" | "refund";
+    policy: PolicyKey;
 }
 export declare function PolicyPageView({ policy, }: PolicyPageViewProps): Promise<import("react").JSX.Element>;

package/dist/features/about/components/PolicyPageView.js

@@ -6,28 +6,69 @@ import { PAGE_CONTAINER } from "../../../_internal/shared/styles/page";
 import { Div, Heading, Text, Section, Stack } from "../../../ui";
 import { TextLink } from "../../../ui";
 import { siteSettingsRepository } from "../../../repositories";
-const namespaceMap = {
-    privacy: "privacy",
-    terms: "terms",
-    cookies: "cookies",
-    refund: "refundPolicy",
-};
-const firestoreFieldMap = {
-    privacy: "privacy",
-    terms: "terms",
-    cookies: "cookies",
-    refund: "refundPolicy",
+/**
+ * Single source of truth for every policy page this shell can render.
+ *
+ * Declared as `Record<PolicyKey, …>` on purpose: adding a member to `PolicyKey`
+ * without adding its entry here is a COMPILE error, not a silent gap. This
+ * replaced three separate hand-maintained enumerations of the same union
+ * (a namespace map, a Firestore-field map, and an inline path lookup inside the
+ * related-links filter) — see Recurrent Root Cause #61.
+ *
+ * - `path`            — used to exclude the current page from its own related list
+ * - `namespace`       — next-intl namespace holding this page's copy
+ * - `firestoreField`  — key inside `siteSettings.legalPages` carrying the admin HTML override
+ * - `relatedLabelKey` — i18n key that OTHER policy namespaces use to label a link to this page
+ */
+const POLICY_META = {
+    privacy: {
+        path: String(ROUTES.PUBLIC.PRIVACY),
+        namespace: "privacy",
+        firestoreField: "privacy",
+        relatedLabelKey: "relatedPrivacy",
+    },
+    terms: {
+        path: String(ROUTES.PUBLIC.TERMS),
+        namespace: "terms",
+        firestoreField: "terms",
+        relatedLabelKey: "relatedTerms",
+    },
+    cookies: {
+        path: String(ROUTES.PUBLIC.COOKIE_POLICY),
+        namespace: "cookies",
+        firestoreField: "cookies",
+        relatedLabelKey: "relatedCookies",
+    },
+    refund: {
+        path: String(ROUTES.PUBLIC.REFUND_POLICY),
+        namespace: "refundPolicy",
+        firestoreField: "refundPolicy",
+        relatedLabelKey: "relatedRefund",
+    },
+    ethics: {
+        path: String(ROUTES.PUBLIC.ETHICS),
+        namespace: "ethics",
+        firestoreField: "ethics",
+        relatedLabelKey: "relatedEthics",
+    },
+    conduct: {
+        path: String(ROUTES.PUBLIC.CODE_OF_CONDUCT),
+        namespace: "codeOfConduct",
+        firestoreField: "codeOfConduct",
+        relatedLabelKey: "relatedConduct",
+    },
 };
+const POLICY_KEYS = Object.keys(POLICY_META);
 export async function PolicyPageView({ policy, }) {
     const page = { container: PAGE_CONTAINER };
+    const meta = POLICY_META[policy];
     const { getTranslations } = await import("next-intl/server");
-    const t = await getTranslations(namespaceMap[policy]);
+    const t = await getTranslations(meta.namespace);
     // Check Firestore for admin-overridden HTML content
     let adminHtml = "";
     try {
         const settings = await siteSettingsRepository.getSingleton();
-        const legalPages = settings.legalPages ?? {};
-        adminHtml = legalPages[firestoreFieldMap[policy]] ?? "";
+        adminHtml = settings.legalPages?.[meta.firestoreField] ?? "";
     }
     catch (_err) {
         void normalizeError(_err);
@@ -36,15 +77,12 @@ export async function PolicyPageView({ policy, }) {
     // i18n fallback sections
     const rawSections = t.raw("sections");
     const sections = Array.isArray(rawSections) ? rawSections : [];
-    const relatedLinks = [
-        { label: t("relatedPrivacy"), href: String(ROUTES.PUBLIC.PRIVACY) },
-        { label: t("relatedTerms"), href: String(ROUTES.PUBLIC.TERMS) },
-        { label: t("relatedCookies"), href: String(ROUTES.PUBLIC.COOKIE_POLICY) },
-        { label: t("relatedRefund"), href: String(ROUTES.PUBLIC.REFUND_POLICY) },
-    ].filter((l) => {
-        const policyPath = { privacy: "/privacy", terms: "/terms", cookies: "/cookies", refund: "/refund-policy" }[policy];
-        return !l.href.endsWith(policyPath ?? "");
-    });
+    // Derived from POLICY_META, so a newly added policy appears in every other
+    // page's related list automatically and can never link to itself.
+    const relatedLinks = POLICY_KEYS.filter((key) => key !== policy).map((key) => ({
+        label: t(POLICY_META[key].relatedLabelKey),
+        href: POLICY_META[key].path,
+    }));
     return (_jsxs(Div, { className: "-mx-4 md:-mx-6 lg:-mx-8 -mt-6 sm:-mt-8 lg:-mt-10", children: [_jsx(Section, { color: "inverse", tone: "accent-banner", padding: "banner", children: _jsxs(Div, { className: `${page.container.sm}`, children: [_jsx(Heading, { color: "inverse", level: 1, variant: "none", className: "mb-3", children: t("title") }), _jsx(Text, { color: "inverse", variant: "none", className: "/80", size: "sm", children: t("lastUpdated") })] }) }), _jsxs(Div, { className: `${page.container.sm}`, padding: "content-banner", children: [adminHtml ? (
                     /* Admin-set HTML takes priority */
                     _jsx(Div, { className: "prose prose-neutral dark:prose-invert max-w-none", dangerouslySetInnerHTML: { __html: adminHtml } })) : (_jsxs(_Fragment, { children: [t("intro") && (_jsx(Text, { variant: "secondary", className: "mb-10 leading-relaxed", size: "base", children: t("intro") })), _jsx(Stack, { gap: "xl", children: sections.map((section, i) => (_jsxs(Section, { children: [_jsx(Heading, { level: 2, className: "mb-3", children: section.heading }), _jsx(Text, { variant: "secondary", className: "leading-relaxed whitespace-pre-line", children: section.body })] }, i))) })] })), relatedLinks.length > 0 && (_jsxs(Div, { className: `mt-14 border-t`, border: "default", padding: "t-xl", children: [_jsx(Heading, { level: 3, className: "mb-3 uppercase tracking-wide", color: "muted", size: "sm", weight: "semibold", children: t("relatedTitle") }), _jsx(Row, { wrap: true, gap: "md", children: relatedLinks.map((l) => (_jsx(TextLink, { href: l.href, variant: "muted", size: "sm", children: l.label }, l.href))) })] }))] })] }));

package/dist/features/about/components/PublicProfileView.js

@@ -48,6 +48,11 @@ function toProductItem(p) {
         currentBid: p.currentBid,
         availableQuantity: p.availableQuantity,
         tags: p.tags,
+        // Feeds ProductCard's "Taking Offers" pill. This is a second, local copy of
+        // the shared `toProductItem` in _internal/shared/features/products/ and it
+        // had the same omission — when a field is added to one, check the other.
+        allowOffers: p.allowOffers,
+        minOfferPercent: p.minOfferPercent,
         createdAt: p.createdAt instanceof Date ? p.createdAt.toISOString() : undefined,
     };
 }

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

@@ -13,6 +13,8 @@ export interface AboutValueItem {
     title: string;
     text: string;
     icon: string;
+    /** Optional second paragraph — how the value is actually enforced, not just stated. */
+    detail?: string;
 }
 export interface AboutMilestone {
     year: string;
@@ -36,6 +38,7 @@ export interface AboutContentDocument {
     howItWorksTitle: string;
     howItems: AboutHowItem[];
     valuesTitle: string;
+    valuesSubtitle?: string;
     valueItems: AboutValueItem[];
     milestonesTitle: string;
     milestones: AboutMilestone[];

package/dist/features/account/components/UserBidsView.js

@@ -1,9 +1,11 @@
 "use client";
-import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
+import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { sortBy, sieveFilter, SIEVE_OP } from "../../../utils/sieve-builder";
 import { ACCOUNT_ENDPOINTS } from "../../../constants/api-endpoints";
 import { ROUTES } from "../../../constants/index";
-import { Div, Span, Stack, TextLink } from "../../../ui";
+import { useState } from "react";
+import { Div, Span, Stack, TextLink, RecordDetailModal } from "../../../ui";
+import { buildBidDetailFields, bidStatusBadge } from "../../auctions/components/bid-detail-fields";
 import { FieldSelect } from "../../../ui/forms";
 import { AuctionBidsTable } from "../../auctions/components/AuctionBidsTable";
 import { DataListingView } from "../../admin/components/DataListingView";
@@ -21,6 +23,11 @@ const STATUS_OPTIONS = [
     { value: "cancelled", label: "Cancelled" },
 ];
 export function UserBidsView() {
+    // A bid could be seen in this list and never opened — no row click, no row
+    // action, no detail page. The "Browse auctions" link in the empty state is
+    // not an affordance: it is not row-scoped, and it only shows when there are
+    // no bids at all.
+    const [detail, setDetail] = useState(null);
     const config = {
         portal: "user",
         title: "My Bids",
@@ -40,8 +47,9 @@ export function UserBidsView() {
             if (isLoading) {
                 return (_jsx(Stack, { gap: "md", children: Array.from({ length: 3 }).map((_, i) => (_jsx(Div, { className: "h-16 animate-pulse border border-[var(--appkit-color-border)]", rounded: "xl" }, i))) }));
             }
-            return (_jsx(AuctionBidsTable, { bids: rows, portal: "buyer", totalPages: 1, currentPage: 1, onPageChange: () => { }, emptyLabel: _jsxs(Span, { children: ["You haven't placed any bids yet.", " ", _jsx(TextLink, { href: String(ROUTES.PUBLIC.AUCTIONS), children: "Browse auctions" })] }) }));
+            return (_jsx(AuctionBidsTable, { bids: rows, portal: "buyer", onRowClick: (bid) => setDetail(bid), totalPages: 1, currentPage: 1, onPageChange: () => { }, emptyLabel: _jsxs(Span, { children: ["You haven't placed any bids yet.", " ", _jsx(TextLink, { href: String(ROUTES.PUBLIC.AUCTIONS), children: "Browse auctions" })] }) }));
         },
     };
-    return _jsx(DataListingView, { config: config });
+    const badge = detail ? bidStatusBadge(detail.status) : null;
+    return (_jsxs(_Fragment, { children: [_jsx(DataListingView, { config: config }), _jsx(RecordDetailModal, { isOpen: detail !== null, onClose: () => setDetail(null), title: detail?.productTitle || "Bid", badges: badge ? [{ label: badge.label }] : undefined, fields: detail ? buildBidDetailFields(detail, "buyer") : undefined, footer: detail ? (_jsx(TextLink, { href: String(ROUTES.PUBLIC.AUCTION_DETAIL(detail.productId)), children: "View auction \u2192" })) : undefined })] }));
 }

package/dist/features/account/components/UserOrdersView.js

@@ -8,6 +8,7 @@ import { Button, Div, FilterChipGroup, Grid, Stack, Text, TextLink } from "../..
 import { PaginatedSelect } from "../../../ui/components/PaginatedSelect";
 import { DataListingView } from "../../admin/components/DataListingView";
 import { OrderCard } from "../../orders/components/OrdersList";
+import { useOrderScope } from "../../orders/components/OrderScopeTabs";
 const CANCELLABLE_STATUSES = new Set(["pending", "confirmed", "processing"]);
 const TRACKABLE_STATUSES = new Set(["shipped"]);
 const SORT_OPTIONS = [
@@ -43,6 +44,7 @@ const STATUS_OPTIONS = [
     { value: "return_requested", label: "Return requested" },
 ];
 export function UserOrdersView({ onOrderClick }) {
+    const orderScope = useOrderScope();
     const config = {
         portal: "user",
         title: "My Orders",
@@ -62,6 +64,11 @@ export function UserOrdersView({ onOrderClick }) {
         ]
             .filter(Boolean)
             .join(",") || undefined,
+        // The lifecycle scope sits alongside the lane tabs as a second,
+        // independent axis: a buyer can ask "which of my auction wins are still
+        // in flight" without those two questions fighting over one control.
+        buildExtraParams: () => orderScope.extraParams,
+        renderAboveContent: orderScope.renderAboveContent,
         renderFilterPanel: ({ pendingFilters, setPendingFilters }) => (_jsxs(Stack, { gap: "md", children: [_jsx(FilterChipGroup, { label: "Type", tabs: ORDER_LANE_TABS, value: pendingFilters.orderType ?? "", onChange: (id) => setPendingFilters((prev) => ({ ...prev, orderType: id })) }), _jsx(Div, { children: _jsx(PaginatedSelect, { value: pendingFilters.status || null, onChange: (v) => setPendingFilters((p) => ({ ...p, status: v ?? "" })), options: STATUS_OPTIONS, placeholder: "All statuses", ariaLabel: "Filter by order status" }) })] })),
         renderCards: (rows, _view, _selection, isLoading) => {
             if (isLoading) {

package/dist/features/account/components/UserReturnsView.d.ts

@@ -1,5 +1,8 @@
 import type { Order } from "../../orders/types";
 export interface UserReturnsViewProps {
+    /**
+     * Optional override. When omitted the card still opens — see below.
+     */
     onOrderClick?: (order: Order) => void;
 }
 export declare function UserReturnsView({ onOrderClick }: UserReturnsViewProps): import("react").JSX.Element;

package/dist/features/account/components/UserReturnsView.js

@@ -3,6 +3,8 @@ import { jsx as _jsx } from "react/jsx-runtime";
 import { sortBy } from "../../../constants/sort";
 import { ACCOUNT_ENDPOINTS } from "../../../constants/api-endpoints";
 import { Div, Grid, Stack, Text } from "../../../ui";
+import { useRouter } from "next/navigation";
+import { ROUTES } from "../../../next/routing/route-map";
 import { DataListingView } from "../../admin/components/DataListingView";
 import { OrderCard } from "../../orders/components/OrdersList";
 const SORT_OPTIONS = [
@@ -11,6 +13,11 @@ const SORT_OPTIONS = [
     { value: sortBy("createdAt", "ASC"), label: "Oldest" },
 ];
 export function UserReturnsView({ onOrderClick }) {
+    const router = useRouter();
+    // Client-side navigation, not a full reload — the row is a link in behaviour
+    // even though the card itself contains its own controls, so wrapping it in
+    // an <a> would nest interactive content inside a link.
+    const openOrder = (order) => router.push(String(ROUTES.USER.ORDER_DETAIL(order.id)));
     const config = {
         portal: "user",
         title: "Returns & Refunds",
@@ -32,7 +39,16 @@ export function UserReturnsView({ onOrderClick }) {
             if (rows.length === 0) {
                 return _jsx(Text, { color: "muted", children: "You have no return or refund requests." });
             }
-            return (_jsx(Grid, { gap: "md", className: "grid-cols-1", children: rows.map((order) => (_jsx(OrderCard, { order: order, onClick: onOrderClick }, order.id))) }));
+            return (_jsx(Grid, { gap: "md", className: "grid-cols-1", children: rows.map((order) => (_jsx(OrderCard, { order: order, 
+                    // A return IS an order, and its detail page already exists — so
+                    // the row opens whether or not a consumer supplies a handler.
+                    //
+                    // It was previously openable ONLY because the single consumer
+                    // happened to pass `onOrderClick`; a second one that forgot
+                    // would have produced a listing whose rows do nothing, and no
+                    // audit could have seen it because the affordance lived in the
+                    // caller. Defaulting here makes it structural.
+                    onClick: onOrderClick ?? openOrder }, order.id))) }));
         },
     };
     return _jsx(DataListingView, { config: config });

package/dist/features/account/components/index.d.ts

@@ -25,8 +25,6 @@ export { UserAddressesView } from "./UserAddressesView";
 export type { UserAddressesViewProps, UserAddressesViewLabels, } from "./UserAddressesView";
 export { UserNotificationsView } from "./UserNotificationsView";
 export type { UserNotificationsViewProps, UserNotificationsViewLabels, } from "./UserNotificationsView";
-export { UserOffersView } from "./UserOffersView";
-export type { UserOffersViewProps, UserOffersViewLabels, } from "./UserOffersView";
 export { UserOffersPanel } from "./UserOffersPanel";
 export type { UserOffersPanelProps } from "./UserOffersPanel";
 export { MessagesView } from "./MessagesView";

package/dist/features/account/components/index.js

@@ -12,7 +12,6 @@ export { UserBidsView } from "./UserBidsView";
 export { OrderDetailView } from "./OrderDetailView";
 export { UserAddressesView } from "./UserAddressesView";
 export { UserNotificationsView } from "./UserNotificationsView";
-export { UserOffersView } from "./UserOffersView";
 export { UserOffersPanel } from "./UserOffersPanel";
 export { MessagesView } from "./MessagesView";
 export { ChatList } from "./ChatList";

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

@@ -144,9 +144,9 @@ export declare const userProfileSchema: z.ZodObject<{
         phone?: string | undefined;
         line2?: string | undefined;
     }[] | undefined;
-    phone?: string | undefined;
     photoURL?: string | undefined;
     bio?: string | undefined;
+    phone?: string | undefined;
     notificationPreferences?: {
         push?: boolean | undefined;
         newsletter?: boolean | undefined;
@@ -172,9 +172,9 @@ export declare const userProfileSchema: z.ZodObject<{
         phone?: string | undefined;
         line2?: string | undefined;
     }[] | undefined;
-    phone?: string | undefined;
     photoURL?: string | undefined;
     bio?: string | undefined;
+    phone?: string | undefined;
     notificationPreferences?: {
         push?: boolean | undefined;
         newsletter?: boolean | undefined;

package/dist/features/account/types/index.d.ts

@@ -1,5 +1,24 @@
-export type OrderStatus = "placed" | "pending" | "confirmed" | "processing" | "shipped" | "delivered" | "cancelled" | "returned" | "refunded";
-export type PaymentStatus = "pending" | "paid" | "failed" | "refunded";
+/**
+ * Root Cause #36 — this module used to declare its OWN `OrderStatus` and
+ * `PaymentStatus`, and `appkit/src/index.ts` re-exported *those* rather than
+ * the real ones. So a bare `import type { OrderStatus } from "@mohasinac/appkit"`
+ * silently resolved to a narrower, wrong union:
+ *
+ *   OrderStatus    had a fake `"placed"` (used nowhere but its own declaration)
+ *                  and was MISSING `"return_requested"` — a real status with
+ *                  29 references across the codebase.
+ *   PaymentStatus  was missing `"processing"` and `"partial_refund"`.
+ *
+ * Assigning a legitimate `OrderStatusValues.RETURN_REQUESTED` to an
+ * `OrderStatus[]` therefore failed to typecheck, which is what pushed callers
+ * onto `Set<string>` and untyped workarounds.
+ *
+ * These are now re-exports of the single source of truth. The account feature
+ * describes the same orders the rest of the app does; it never had grounds for
+ * its own union.
+ */
+import type { OrderStatus, PaymentStatus } from "../../orders/types/index";
+export type { OrderStatus, PaymentStatus };
 export interface OrderItem {
     productId: string;
     title: string;

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

@@ -41,12 +41,12 @@ export declare const addressFirestoreSchema: z.ZodObject<{
         toDate: z.ZodFunction<z.ZodTuple<[], z.ZodUnknown>, z.ZodDate>;
     }, z.ZodTypeAny, "passthrough">;
     isDefault: boolean;
-    phone: string;
-    city: string;
     ownerId: string;
     ownerType: "user" | "store";
     fullName: string;
+    phone: string;
     addressLine1: string;
+    city: string;
     state: string;
     postalCode: string;
     addressLine2?: string | undefined;
@@ -62,12 +62,12 @@ export declare const addressFirestoreSchema: z.ZodObject<{
         toDate: z.ZodFunction<z.ZodTuple<[], z.ZodUnknown>, z.ZodDate>;
     }, z.ZodTypeAny, "passthrough">;
     isDefault: boolean;
-    phone: string;
-    city: string;
     ownerId: string;
     ownerType: "user" | "store";
     fullName: string;
+    phone: string;
     addressLine1: string;
+    city: string;
     state: string;
     postalCode: string;
     addressLine2?: string | undefined;

package/dist/features/admin/components/AdminArtView.js

@@ -13,14 +13,19 @@ import React from "react";
 import { ListingLayout } from "../../../ui";
 import { buildListingTypeListingConfig } from "../../products/config/listing-type-listing-config";
 import { DataListingView } from "./DataListingView";
-const CONFIG = buildListingTypeListingConfig("art", {
+import { useAvailabilityScope } from "../../products/hooks/useAvailabilityScope";
+const LISTING_TYPES = ["art"];
+const CONFIG_OPTS = {
     title: "Art",
     searchPlaceholder: "Search art prints by name or seller",
     emptyLabel: "No art listings",
-});
+};
 export function AdminArtView({ children, ...props }) {
+    const scope = useAvailabilityScope(LISTING_TYPES);
     if (React.Children.count(children) > 0) {
         return (_jsx(ListingLayout, { portal: "admin", ...props, children: children }));
     }
-    return _jsx(DataListingView, { config: CONFIG });
+    // Built per-render rather than at module scope: the availability scope
+    // reads the URL through a hook, which cannot run outside a component.
+    return (_jsx(DataListingView, { config: buildListingTypeListingConfig("art", { ...CONFIG_OPTS, scope }) }));
 }

package/dist/features/admin/components/AdminAuditLogView.js

@@ -23,6 +23,7 @@ const ACTION_LABELS = {
     checkout_bypass: "Admin checkout bypass",
     coupon_update: "Coupon updated",
     payout_mark_paid: "Payout marked paid",
+    offer_cancel: "Offer cancelled",
     store_status_change: "Store status changed",
     user_role_change: "User role changed",
 };

package/dist/features/admin/components/AdminBidsView.js

@@ -5,11 +5,12 @@ import { sieveFilter, SIEVE_OP } from "@mohasinac/appkit/client";
 import { sortBy } from "@mohasinac/appkit/client";
 import React, { useState } from "react";
 import { useQueryClient } from "@tanstack/react-query";
-import { ConfirmDeleteModal, FilterChipGroup, ListingLayout, RowActionMenu, useToast } from "../../../ui";
+import { ConfirmDeleteModal, FilterChipGroup, ListingLayout, RecordDetailModal, RowActionMenu, useToast, } from "../../../ui";
 import { ADMIN_ENDPOINTS } from "../../../constants/api-endpoints";
 import { ADMIN_BID_STATUS_TABS } from "../constants/filter-tabs";
 import { ACTIONS } from "../../../_internal/shared/actions/action-registry";
 import { ROW_ACTION_META, ROW_ACTION_ID } from "../../products/constants/action-defs";
+import { buildBidDetailFields, bidStatusBadge } from "../../auctions/components/bid-detail-fields";
 import { toRecordArray, toRelativeDate, toCurrency, toStringValue, } from "../hooks/useAdminListingData";
 import { DataListingView } from "./DataListingView";
 import { apiClient } from "../../../http";
@@ -18,6 +19,7 @@ export function AdminBidsView({ children, ...props }) {
     const { showToast } = useToast();
     const [cancelOpen, setCancelOpen] = useState(false);
     const [selectedRow, setSelectedRow] = useState(null);
+    const [detail, setDetail] = useState(null);
     const cancelMutation = useApiMutation({
         mutationFn: async (bidId) => {
             await apiClient.patch(ADMIN_ENDPOINTS.BID_BY_ID(bidId), { status: "cancelled" });
@@ -58,6 +60,7 @@ export function AdminBidsView({ children, ...props }) {
             secondary: toStringValue(item.bidderId ?? item.bidderName ?? item.userId, "Unknown bidder"),
             status: toStringValue(item.status, "active"),
             updatedAt: toRelativeDate(item.bidDate ?? item.updatedAt ?? item.createdAt),
+            detail: item,
         })),
         getTotal: (response, mappedRows) => typeof response.total === "number" ? response.total : mappedRows.length,
         buildFilters: (state) => state.status && state.status !== "All" ? sieveFilter("status", SIEVE_OP.EQ, state.status) : undefined,
@@ -72,6 +75,13 @@ export function AdminBidsView({ children, ...props }) {
         renderRowActions: (row) => {
             const isCancelled = row.status === "cancelled" || row.status === "voided";
             return (_jsx(RowActionMenu, { actions: [
+                    // A menu of pure MUTATIONS is not a detail affordance (Root Cause
+                    // #56): it let an admin cancel a bid they had never been able to
+                    // read. View comes first for that reason.
+                    {
+                        label: ROW_ACTION_META[ROW_ACTION_ID.VIEW].label,
+                        onClick: () => setDetail(row),
+                    },
                     {
                         label: ROW_ACTION_META[ROW_ACTION_ID.CANCEL].label,
                         destructive: ROW_ACTION_META[ROW_ACTION_ID.CANCEL].destructive,
@@ -85,7 +95,10 @@ export function AdminBidsView({ children, ...props }) {
         },
         renderFilterPanel: ({ pendingFilters, setPendingFilters }) => (_jsx(FilterChipGroup, { label: "Status", tabs: ADMIN_BID_STATUS_TABS, value: pendingFilters.status ?? "", onChange: (id) => setPendingFilters((p) => ({ ...p, status: id })) })),
     };
-    return (_jsxs(_Fragment, { children: [_jsx(DataListingView, { config: config }), _jsx(ConfirmDeleteModal, { isOpen: cancelOpen, onClose: () => {
+    const detailBadge = detail ? bidStatusBadge(detail.status) : null;
+    return (_jsxs(_Fragment, { children: [_jsx(DataListingView, { config: config }), _jsx(RecordDetailModal, { isOpen: detail !== null, onClose: () => setDetail(null), title: toStringValue(detail?.detail?.productTitle, "Bid"), badges: detailBadge ? [{ label: detailBadge.label }] : undefined, fields: detail
+                    ? buildBidDetailFields(detail.detail, "admin")
+                    : undefined }), _jsx(ConfirmDeleteModal, { isOpen: cancelOpen, onClose: () => {
                     setCancelOpen(false);
                     setSelectedRow(null);
                 }, onConfirm: () => {