@mohasinac/appkit 4.23.0 → 4.25.0

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

@@ -23,10 +23,32 @@ export const CART_FIELDS = {
         STORE_ID: "storeId",
         STORE_NAME: "storeName",
         LISTING_TYPE: "listingType",
+        IS_OFFER: "isOffer",
+        OFFER_ID: "offerId",
+        IS_AUCTION_WIN: "isAuctionWin",
+        AUCTION_ID: "auctionId",
+        BID_ID: "bidId",
+        IS_BUYOUT: "isBuyout",
+        LOCKED_PRICE: "lockedPrice",
+        CHECKOUT_DEADLINE: "checkoutDeadline",
+        LOCKED: "locked",
+        BUNDLE_CATEGORY_SLUG: "bundleCategorySlug",
+        /** @deprecated legacy read path — see CartItemDocument.bundleProductIds */
+        BUNDLE_PRODUCT_IDS: "bundleProductIds",
+        LINE_KIND: "lineKind",
+        GROUP_SOURCE: "groupSource",
+        GROUP_ID: "groupId",
+        GROUP_SLUG: "groupSlug",
+        GROUP_TITLE: "groupTitle",
+        GROUP_MEMBERS: "groupMembers",
+        CHOSEN_SHIPPING_PROVIDER_ID: "chosenShippingProviderId",
+        CHOSEN_SHIPPING_FEE: "chosenShippingFee",
         ADDED_AT: "addedAt",
         UPDATED_AT: "updatedAt",
     },
 };
+export const CART_LINE_KIND_VALUES = ["bundle", "group"];
+export const CART_GROUP_SOURCE_VALUES = ["product-group", "grouped-listing"];
 export const DEFAULT_CART_DATA = {
     items: [],
 };

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

@@ -5,5 +5,5 @@
  * `renderToStaticMarkup`. They emit table-based, inline-styled markup that
  * email clients render correctly.
  */
-export { EmailButton, EmailColumn, EmailContainer, EmailDivider, EmailDoc, EmailFooter, EmailHeader, EmailImage, EmailLink, EmailRow, } from "./primitives";
+export { EmailBold, EmailButton, EmailColumn, EmailContainer, EmailDivider, EmailDoc, EmailFooter, EmailHeader, EmailImage, EmailLink, EmailRow, } from "./primitives";
 export type { EmailButtonProps, EmailColumnProps, EmailContainerProps, EmailDividerProps, EmailDocProps, EmailFooterProps, EmailHeaderProps, EmailImageProps, EmailLinkProps, EmailRowProps, EmailTone, } from "./primitives";

package/dist/features/email/index.js

@@ -5,4 +5,8 @@
  * `renderToStaticMarkup`. They emit table-based, inline-styled markup that
  * email clients render correctly.
  */
-export { EmailButton, EmailColumn, EmailContainer, EmailDivider, EmailDoc, EmailFooter, EmailHeader, EmailImage, EmailLink, EmailRow, } from "./primitives";
+export { 
+// Was MISSING from this barrel, so both consumers reached past it into
+// ./primitives directly — a workaround that reads as intentional and
+// quietly says "the barrel is not the public surface after all".
+EmailBold, EmailButton, EmailColumn, EmailContainer, EmailDivider, EmailDoc, EmailFooter, EmailHeader, EmailImage, EmailLink, EmailRow, } from "./primitives";

package/dist/features/email/notification-templates.d.ts

@@ -0,0 +1,35 @@
+import "server-only";
+import type { NotificationType } from "../admin/schemas/firestore";
+export interface NotificationEmailContext {
+    title: string;
+    message: string;
+    siteName: string;
+    /** Absolute URL the CTA opens. Absent → the email renders without a button. */
+    actionUrl?: string;
+}
+export interface NotificationTemplateDef {
+    /** CTA label. Absent → no button, for a type with nowhere to go. */
+    cta?: string;
+    /**
+     * One line of framing above the message, so the reader knows what KIND of
+     * mail this is before reading it. Deliberately not the message itself —
+     * duplicating it is worse than omitting it.
+     */
+    lead: string;
+    /** Terminal-negative types render in the danger tone. */
+    tone?: "default" | "danger" | "success";
+}
+/**
+ * 🛑 Every notification type, no exceptions.
+ *
+ * When adding a type, the compiler will ask for an entry here. Answer three
+ * questions: what does the reader need to know before the message (`lead`),
+ * where are they going (`cta`, or none if nowhere), and is this good news.
+ */
+export declare const NOTIFICATION_EMAIL_TEMPLATES: Record<NotificationType, NotificationTemplateDef>;
+/**
+ * Render one notification as an email.
+ *
+ * Returns the full HTML document, ready for `emailHtml`.
+ */
+export declare function renderNotificationEmail(type: NotificationType, ctx: NotificationEmailContext): string;

package/dist/features/email/notification-templates.js

@@ -0,0 +1,96 @@
+import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
+import "server-only";
+import { EmailBold, EmailButton, EmailContainer, EmailDivider, EmailDoc, EmailFooter, EmailHeader, EmailRow, } from "./primitives";
+/**
+ * A real email per notification type.
+ *
+ * ## What this replaces
+ *
+ * `emailHtml` has existed on `SendNotificationInput` since the channel was
+ * built, and **0 of 40** call sites passed it — so every one of the 28
+ * notification types fell through to `html: \`<p>${message}</p>\``. A bare
+ * paragraph, no subject context, no branding, nothing to click. That is the
+ * reported "emails are sent but I can't understand what is being sent".
+ *
+ * ## Why a Record, not a lookup with a fallback
+ *
+ * `Record<NotificationType, TemplateDef>` — a new notification type cannot
+ * compile without an email. A `Partial` with a default would have let the
+ * next type silently go back to `<p>{message}</p>`, which is how the feature
+ * got here in the first place.
+ *
+ * ## The two structural constraints
+ *
+ * 1. **`renderToStaticMarkup` must be a lazy `require`.** A top-level
+ *    `import` from "react-dom/server" is flagged by Next's static tracer as
+ *    soon as any module in the re-export chain is reachable from a Server
+ *    Component — and this one is, via `@mohasinac/appkit/server`. `require()`
+ *    is opaque to that analyser, so the chain stops here. Same reason
+ *    `features/contact/email.tsx` does it.
+ * 2. **Never a raw `<table>`.** The `Email*` primitives emit the table-based,
+ *    inline-styled markup email clients actually render; hand-rolled markup
+ *    is what `audit-email-raw-html` blocks.
+ */
+// See constraint (1) above — this MUST stay a lazy require.
+function renderToStaticMarkup(el) {
+    const m = require("react-dom/server");
+    return m.renderToStaticMarkup(el);
+}
+/**
+ * 🛑 Every notification type, no exceptions.
+ *
+ * When adding a type, the compiler will ask for an entry here. Answer three
+ * questions: what does the reader need to know before the message (`lead`),
+ * where are they going (`cta`, or none if nowhere), and is this good news.
+ */
+export const NOTIFICATION_EMAIL_TEMPLATES = {
+    order_placed: { lead: "We've received your order.", cta: "View order", tone: "success" },
+    order_confirmed: { lead: "Your order is confirmed and being prepared.", cta: "View order", tone: "success" },
+    order_shipped: { lead: "Your order is on its way.", cta: "Track order", tone: "success" },
+    order_delivered: { lead: "Your order has been delivered.", cta: "View order", tone: "success" },
+    order_cancelled: { lead: "Your order has been cancelled.", cta: "View order", tone: "danger" },
+    refund_initiated: { lead: "A refund is on its way back to you.", cta: "View order" },
+    payment_review: { lead: "There's an update on your payment.", cta: "View payment" },
+    emi_installment_due_soon: { lead: "An EMI installment is due soon.", cta: "View order" },
+    emi_installment_overdue: { lead: "An EMI installment is overdue.", cta: "View order", tone: "danger" },
+    bid_placed: { lead: "Your bid has been placed.", cta: "View bids" },
+    bid_outbid: { lead: "Someone has outbid you.", cta: "View bids", tone: "danger" },
+    bid_won: { lead: "You won the auction.", cta: "Complete checkout", tone: "success" },
+    bid_lost: { lead: "This auction closed without you.", cta: "View bids" },
+    auction_ended: { lead: "Your auction has ended.", cta: "View auction" },
+    offer_received: { lead: "A buyer has made you an offer.", cta: "Review offer" },
+    offer_responded: { lead: "The seller has responded to your offer.", cta: "View offer" },
+    offer_counter_accepted: { lead: "Your counter-offer was accepted.", cta: "Complete checkout", tone: "success" },
+    offer_expired: { lead: "An offer has expired.", cta: "View offers" },
+    prize_won: { lead: "You've won a prize.", cta: "View prize", tone: "success" },
+    prize_reveal_expired: { lead: "A prize reveal window has closed.", cta: "View prize draws" },
+    review_approved: { lead: "A review has been published.", cta: "Read review" },
+    review_replied: { lead: "Someone replied to your review.", cta: "Read reply" },
+    product_available: { lead: "Something you were watching is back in stock.", cta: "View listing" },
+    catalogue_images_stale: { lead: "Your catalogue photos need refreshing.", cta: "Update photos" },
+    welcome: { lead: "Welcome aboard.", cta: "Go to your account", tone: "success" },
+    account_action: { lead: "There's been a change to your account.", cta: "View account" },
+    // No CTA: a promotion's destination is whatever the copy names, and a
+    // generic "View" button on a marketing mail is worse than none.
+    promotion: { lead: "An offer you might like." },
+    // No CTA for the same reason — a system announcement has no record behind
+    // it, and `resolveNotificationActionUrl` correctly returns undefined.
+    system: { lead: "A message from the team." },
+};
+/**
+ * Render one notification as an email.
+ *
+ * Returns the full HTML document, ready for `emailHtml`.
+ */
+export function renderNotificationEmail(type, ctx) {
+    const def = NOTIFICATION_EMAIL_TEMPLATES[type];
+    /*
+     * `EmailButton` takes a narrower tone set than `EmailTone` — brand / accent
+     * / neutral only, because a button is a call to action and there is no
+     * useful "danger" button. So the template's tone drives the BUTTON's
+     * emphasis, not its colour: bad news gets the neutral button ("View order"
+     * on a cancellation should not shout), good news gets the brand one.
+     */
+    const buttonTone = def.tone === "danger" ? "neutral" : "brand";
+    return `<!DOCTYPE html>${renderToStaticMarkup(_jsx(EmailDoc, { title: ctx.title, children: _jsxs(EmailContainer, { children: [_jsx(EmailHeader, { brandName: ctx.siteName }), _jsxs(EmailRow, { children: [_jsx(EmailBold, { children: ctx.title }), _jsx("br", {}), _jsx("br", {}), def.lead] }), _jsx(EmailDivider, {}), _jsx(EmailRow, { children: ctx.message }), def.cta && ctx.actionUrl ? (_jsx(EmailRow, { children: _jsx(EmailButton, { href: ctx.actionUrl, tone: buttonTone, children: def.cta }) })) : null, _jsx(EmailFooter, { copyright: `© ${ctx.siteName}` })] }) }))}`;
+}

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

@@ -612,6 +612,7 @@ export declare const orderFirestoreSchema: z.ZodObject<{
         shelf: string;
         bin: string;
     } | undefined;
+    offerId?: string | undefined;
     orderType?: string | undefined;
     imageUrls?: string[] | undefined;
     refundFeeDeducted?: number | undefined;
@@ -649,7 +650,6 @@ export declare const orderFirestoreSchema: z.ZodObject<{
     }[] | undefined;
     emiRemainingBalance?: number | undefined;
     emiComplete?: boolean | undefined;
-    offerId?: string | undefined;
     isNonRefundable?: boolean | undefined;
     paymentBatchId?: string | undefined;
     refunds?: {
@@ -760,6 +760,7 @@ export declare const orderFirestoreSchema: z.ZodObject<{
         shelf: string;
         bin: string;
     } | undefined;
+    offerId?: string | undefined;
     orderType?: string | undefined;
     imageUrls?: string[] | undefined;
     refundFeeDeducted?: number | undefined;
@@ -797,7 +798,6 @@ export declare const orderFirestoreSchema: z.ZodObject<{
     }[] | undefined;
     emiRemainingBalance?: number | undefined;
     emiComplete?: boolean | undefined;
-    offerId?: string | undefined;
     isNonRefundable?: boolean | undefined;
     paymentBatchId?: string | undefined;
     refunds?: {

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

@@ -0,0 +1,46 @@
+/**
+ * GroupMemberPicker — pick which members of a group you want, and how many of
+ * each, then add the whole selection as ONE cart line.
+ *
+ * This is the "pick as you wish" half of the grouped-purchase model. Bundles
+ * are the other half and deliberately do NOT use this: a bundle is priced as a
+ * whole and is all-or-nothing, so its members render read-only and the only
+ * stepper is the bundle's own copy count.
+ *
+ * Two mount points, one component: the "Part of: …" modal/drawer on a product
+ * detail page, and the public `/groups/{slug}` page.
+ *
+ * The component talks to the API directly rather than taking an `onAdd` server
+ * action prop. That keeps `ShowGroupSection`'s `renderGroupSection` slot
+ * signature unchanged, which in turn leaves both detail-page views and both
+ * consumer pages untouched.
+ */
+import React from "react";
+import type { ListingType } from "../types";
+export interface GroupPickerMember {
+    id: string;
+    title: string;
+    price: number;
+    currency?: string;
+    images?: string[];
+    image?: string;
+    slug?: string;
+    listingType?: ListingType;
+    isGroupParent?: boolean;
+    condition?: string;
+    availableQuantity?: number;
+    storeId?: string;
+    storeName?: string;
+    isSold?: boolean;
+    status?: string;
+}
+export interface GroupMemberPickerProps {
+    groupId: string;
+    groupSource: "product-group" | "grouped-listing";
+    members: GroupPickerMember[];
+    /** Called after a successful add so the host can close its overlay. */
+    onAdded?: () => void;
+    /** Called when the API reports the caller isn't signed in. */
+    onAuthRequired?: () => void;
+}
+export declare function GroupMemberPicker({ groupId, groupSource, members, onAdded, onAuthRequired, }: GroupMemberPickerProps): React.JSX.Element;

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

@@ -0,0 +1,108 @@
+"use client";
+import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
+/**
+ * GroupMemberPicker — pick which members of a group you want, and how many of
+ * each, then add the whole selection as ONE cart line.
+ *
+ * This is the "pick as you wish" half of the grouped-purchase model. Bundles
+ * are the other half and deliberately do NOT use this: a bundle is priced as a
+ * whole and is all-or-nothing, so its members render read-only and the only
+ * stepper is the bundle's own copy count.
+ *
+ * Two mount points, one component: the "Part of: …" modal/drawer on a product
+ * detail page, and the public `/groups/{slug}` page.
+ *
+ * The component talks to the API directly rather than taking an `onAdd` server
+ * action prop. That keeps `ShowGroupSection`'s `renderGroupSection` slot
+ * signature unchanged, which in turn leaves both detail-page views and both
+ * consumer pages untouched.
+ */
+import { useMemo, useState } from "react";
+import { Alert, Button, Div, Row, Span, Stack, Table, Tbody, Td, Text, Th, Thead, Tr, } from "../../../ui";
+import { QuantityStepper } from "../../../ui/components/QuantityStepper";
+import { MediaImage } from "../../media/MediaImage";
+import { formatCurrency } from "../../../utils/number.formatter";
+import { ACTIONS } from "../../../_internal/shared/actions/action-registry";
+import { CART_ENDPOINTS } from "../../../constants/api-endpoints";
+import { normalizeError } from "../../../errors/normalize";
+import { pluginFor } from "../../../_internal/shared/listing-types/_registry";
+import { normalizeListingType } from "../utils/listing-type";
+const __O = { xAuto: "overflow-x-auto", hidden: "overflow-hidden" };
+function memberImage(m) {
+    return m.image ?? m.images?.[0] ?? "";
+}
+function memberHref(m) {
+    return pluginFor(normalizeListingType(m)).detailRoute(m.slug ?? m.id);
+}
+/** A member the buyer cannot choose, and the reason to show instead of a stepper. */
+function blockedReason(m) {
+    if (m.isSold)
+        return "Sold";
+    if (m.status && m.status !== "published")
+        return "Unavailable";
+    if ((m.availableQuantity ?? 0) <= 0)
+        return "Out of stock";
+    // `cartLine: "blocked"` is how a classified / live listing says "contact the
+    // seller instead" — those can never join a cart line.
+    if (pluginFor(normalizeListingType(m)).cartLine === "blocked")
+        return "Not purchasable here";
+    return null;
+}
+export function GroupMemberPicker({ groupId, groupSource, members, onAdded, onAuthRequired, }) {
+    const [qty, setQty] = useState({});
+    const [isSaving, setIsSaving] = useState(false);
+    const [error, setError] = useState(null);
+    // A group whose members span sellers cannot become one cart line: storeId is
+    // the order-splitting key and the key per-store add-ons, coupons, shipping and
+    // payout all hang off. Rather than offer a button that always fails, the
+    // picker stands down and the table stays read-only.
+    const storeIds = useMemo(() => new Set(members.map((m) => m.storeId).filter(Boolean)), [members]);
+    const isCrossStore = storeIds.size > 1;
+    const selected = useMemo(() => members
+        .map((m) => ({ member: m, quantity: qty[m.id] ?? 0 }))
+        .filter((s) => s.quantity > 0), [members, qty]);
+    const selectedTotal = selected.reduce((sum, s) => sum + s.member.price * s.quantity, 0);
+    const selectedUnits = selected.reduce((sum, s) => sum + s.quantity, 0);
+    const currency = members[0]?.currency ?? "INR";
+    const handleAdd = async () => {
+        setError(null);
+        setIsSaving(true);
+        try {
+            const res = await fetch(CART_ENDPOINTS.GROUP, {
+                method: "POST",
+                headers: { "Content-Type": "application/json" },
+                body: JSON.stringify({
+                    groupId,
+                    groupSource,
+                    // Ids and quantities only. Prices and titles are re-resolved on the
+                    // server — anything else would be a price-manipulation hole.
+                    members: selected.map((s) => ({ productId: s.member.id, quantity: s.quantity })),
+                }),
+            });
+            if (res.status === 401) {
+                onAuthRequired?.();
+                return;
+            }
+            const json = (await res.json());
+            if (!res.ok || json.success === false) {
+                setError(json.error ?? "Couldn't add those items to your cart.");
+                return;
+            }
+            setQty({});
+            onAdded?.();
+        }
+        catch (err) {
+            void normalizeError(err);
+            setError("Couldn't reach the server. Please try again.");
+        }
+        finally {
+            setIsSaving(false);
+        }
+    };
+    return (_jsxs(Stack, { gap: "sm", children: [isCrossStore && (_jsx(Alert, { variant: "info", children: "These items are sold by different sellers, so they can't be combined into one cart line. Open an item to buy it on its own." })), _jsx(Div, { className: __O.xAuto, children: _jsxs(Table, { className: "text-left min-w-[440px]", children: [_jsx(Thead, { children: _jsxs(Tr, { border: "default", children: [_jsx(Th, { paddingY: "sm", paddingSide: "pr-sm", color: "muted", size: "xs", weight: "semibold", children: "Image" }), _jsx(Th, { paddingY: "sm", paddingSide: "pr-sm", color: "muted", size: "xs", weight: "semibold", children: "Name" }), _jsx(Th, { paddingY: "sm", paddingSide: "pr-sm", color: "muted", size: "xs", weight: "semibold", children: "Price" }), !isCrossStore && (_jsx(Th, { paddingY: "sm", paddingSide: "pr-sm", color: "muted", size: "xs", weight: "semibold", children: "Qty" })), _jsx(Th, { paddingY: "sm", paddingSide: "pb-sm", color: "muted", size: "xs", weight: "semibold" })] }) }), _jsx(Tbody, { children: members.map((m) => {
+                                const blocked = blockedReason(m);
+                                return (_jsxs(Tr, { className: "last:border-0", border: "subtle", children: [_jsx(Td, { paddingSide: "pr-sm", padding: "xs-tall", children: _jsx(Div, { className: `w-10 h-10 ${__O.hidden}`, rounded: "full", border: "default", children: _jsx(MediaImage, { src: memberImage(m), alt: m.title, size: "thumbnail" }) }) }), _jsxs(Td, { paddingSide: "pr-sm", padding: "xs-tall", children: [_jsx(Text, { className: "line-clamp-2", color: "primary", size: "sm", weight: "medium", children: m.title }), m.isGroupParent && (_jsx(Span, { weight: "semibold", className: "text-[10px] text-[var(--appkit-color-primary)]", children: "Parent" }))] }), _jsx(Td, { paddingSide: "pr-sm", padding: "xs-tall", children: _jsx(Text, { size: "sm", color: "muted", children: formatCurrency(m.price, m.currency ?? currency) }) }), !isCrossStore && (_jsx(Td, { paddingSide: "pr-sm", padding: "xs-tall", children: blocked ? (_jsx(Span, { size: "xs", color: "muted", surface: "muted", rounded: "default", padding: "pill-2xs", children: blocked })) : (_jsx(QuantityStepper, { value: qty[m.id] ?? 0, onChange: (next) => setQty((prev) => ({ ...prev, [m.id]: next })), min: 0, max: m.availableQuantity, ariaLabel: `Quantity for ${m.title}`, decrementLabel: ACTIONS.CART["decrease-quantity"].ariaLabel, incrementLabel: ACTIONS.CART["increase-quantity"].ariaLabel })) })), _jsx(Td, { padding: "xs-tall", children: _jsx("a", { href: memberHref(m), className: "text-[length:var(--appkit-text-xs)] text-[var(--appkit-color-primary)] hover:underline", children: "View \u2192" }) })] }, m.id));
+                            }) })] }) }), !isCrossStore && (_jsxs(_Fragment, { children: [error && _jsx(Alert, { variant: "error", children: error }), _jsxs(Row, { justify: "between", align: "center", gap: "sm", className: "min-w-0", children: [_jsx(Text, { size: "sm", color: "muted", children: selectedUnits > 0
+                                    ? `${selectedUnits} item${selectedUnits === 1 ? "" : "s"} · ${formatCurrency(selectedTotal, currency)}`
+                                    : "Nothing selected yet" }), _jsx(Button, { action: ACTIONS.CART["add-group-to-cart"], disabled: selectedUnits === 0, isLoading: isSaving, onClick: handleAdd })] })] }))] }));
+}

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

@@ -2,8 +2,9 @@
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { useEffect, useState } from "react";
 import Link from "next/link";
-import { Div, Row, Span, Table, Thead, Tbody, Tr, Th, Td, Text, Modal, SideDrawer, Button } from "../../../ui";
+import { Div, Row, Span, Text, Modal, SideDrawer, Button } from "../../../ui";
 import { MediaImage } from "../../media/MediaImage";
+import { GroupMemberPicker } from "./GroupMemberPicker";
 import { formatCurrency } from "../../../utils/number.formatter";
 import { normalizeListingType } from "../utils/listing-type";
 import { pluginFor } from "../../../_internal/shared/listing-types/_registry";
@@ -24,12 +25,6 @@ function MemberThumb({ member, isCurrent }) {
                     ? "border-[var(--appkit-color-primary)] ring-2 ring-[var(--appkit-color-primary)]/30"
                     : "border-[var(--appkit-color-border)] group-hover:border-[var(--appkit-color-primary)]"}`, children: [_jsx(MediaImage, { src: image, alt: member.title, size: "card" }), member.isGroupParent && (_jsx(Span, { color: "inverse", rounded: "sm", className: "absolute bottom-0 right-0 bg-[var(--appkit-color-primary)] text-[8px] leading-none px-[0.25rem] py-[0.125rem]", children: "Set" }))] }), _jsx(Text, { className: "text-[10px] leading-tight line-clamp-2 w-full", color: "muted", align: "start", children: member.title }), _jsx(Text, { className: "text-[10px]", color: "primary", weight: "semibold", children: price })] }));
 }
-function GroupTableRow({ member }) {
-    const href = memberHref(member);
-    const price = formatCurrency(member.price, member.currency ?? "INR");
-    const image = member.images?.[0] ?? "";
-    return (_jsxs(Tr, { className: "last:border-0", border: "subtle", children: [_jsx(Td, { paddingSide: "pr-sm", padding: "xs-tall", children: _jsx(Div, { className: `w-10 h-10 ${__O.hidden}`, rounded: "full", border: "default", children: _jsx(MediaImage, { src: image, alt: member.title, size: "thumbnail" }) }) }), _jsxs(Td, { paddingSide: "pr-sm", padding: "xs-tall", children: [_jsx(Text, { className: "line-clamp-2", color: "primary", size: "sm", weight: "medium", children: member.title }), member.isGroupParent && (_jsx(Span, { weight: "semibold", className: "text-[10px] text-[var(--appkit-color-primary)]", children: "Parent" }))] }), _jsx(Td, { paddingSide: "pr-sm", padding: "xs-tall", children: _jsx(Text, { size: "sm", color: "muted", children: price }) }), _jsx(Td, { paddingSide: "pr-sm", padding: "xs-tall", children: _jsx(Text, { size: "xs", transform: "capitalize", color: "muted", children: member.condition ?? "—" }) }), _jsx(Td, { padding: "xs-tall", children: _jsx(Link, { href: href, className: "text-[length:var(--appkit-text-xs)] text-[var(--appkit-color-primary)] hover:underline", children: "View \u2192" }) })] }));
-}
 export function ShowGroupSection({ groupId, currentSlug, isParent, groupTitle }) {
     const [open, setOpen] = useState(false);
     const [members, setMembers] = useState([]);
@@ -50,11 +45,14 @@ export function ShowGroupSection({ groupId, currentSlug, isParent, groupTitle })
     const label = groupTitle ?? "Product group";
     const parentLabel = isParent ? `Parts in this group: ${label}` : `Part of: ${label}`;
     const useDrawer = members.length >= 5;
-    const tableContent = (_jsx(Div, { className: `${__O.xAuto}`, children: _jsxs(Table, { className: "text-left min-w-[400px]", children: [_jsx(Thead, { children: _jsxs(Tr, { border: "default", children: [_jsx(Th, { paddingY: "sm", paddingSide: "pr-sm", color: "muted", size: "xs", weight: "semibold", children: "Image" }), _jsx(Th, { paddingY: "sm", paddingSide: "pr-sm", color: "muted", size: "xs", weight: "semibold", children: "Name" }), _jsx(Th, { paddingY: "sm", paddingSide: "pr-sm", color: "muted", size: "xs", weight: "semibold", children: "Price" }), _jsx(Th, { paddingY: "sm", paddingSide: "pr-sm", color: "muted", size: "xs", weight: "semibold", children: "Condition" }), _jsx(Th, { paddingY: "sm", paddingSide: "pb-sm", color: "muted", size: "xs", weight: "semibold" })] }) }), _jsx(Tbody, { children: members.map((m) => _jsx(GroupTableRow, { member: m }, m.id)) })] }) }));
+    // The overlay is now a PICKER, not a read-only table: pick how many of each
+    // member you want and add the whole selection as one cart line. The picker
+    // stands itself down (read-only, no CTA) when the group spans sellers.
+    const tableContent = (_jsx(GroupMemberPicker, { groupId: groupId, groupSource: "product-group", members: members, onAdded: () => setShowAll(false), onAuthRequired: () => setShowAll(false) }));
     return (_jsxs(_Fragment, { children: [_jsxs(Div, { className: `${__O.hidden}`, surface: "muted", rounded: "xl", border: "default", children: [_jsxs(Row, { role: "button", tabIndex: 0, onClick: () => setOpen((v) => !v), onKeyDown: (e) => { if (e.key === "Enter" || e.key === " ") {
                             e.preventDefault();
                             setOpen((v) => !v);
-                        } }, justify: "between", paddingX: "x-md", paddingY: "y-sm", className: "w-full cursor-pointer text-left hover:bg-[var(--appkit-color-surface-elevated)]/70 transition-colors", "aria-expanded": open, children: [_jsxs(Row, { align: "center", gap: "xs", children: [_jsx(Span, { size: "xs", className: "mr-1", color: "faint", children: open ? "▼" : "▶" }), _jsx(Text, { size: "sm", weight: "medium", color: "primary", children: parentLabel }), _jsx(Span, { size: "xs", className: "ml-1", rounded: "full", padding: "pill-xs", surface: "subtle", color: "muted", children: members.length })] }), _jsx(Button, { type: "button", variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); setShowAll(true); }, className: "text-[length:var(--appkit-text-xs)] text-[var(--appkit-color-primary)] hover:underline ml-3 flex-shrink-0", children: "View whole group \u2192" })] }), open && (_jsx(Div, { className: `pb-[1rem] pt-[0.25rem] ${__O.xAuto}`, padding: "x-md", children: _jsx(Row, { gap: "3", className: "min-w-0", children: [...members]
+                        } }, justify: "between", paddingX: "x-md", paddingY: "y-sm", className: "w-full cursor-pointer text-left hover:bg-[var(--appkit-color-surface-elevated)]/70 transition-colors", "aria-expanded": open, children: [_jsxs(Row, { align: "center", gap: "xs", children: [_jsx(Span, { size: "xs", className: "mr-1", color: "faint", children: open ? "▼" : "▶" }), _jsx(Text, { size: "sm", weight: "medium", color: "primary", children: parentLabel }), _jsx(Span, { size: "xs", className: "ml-1", rounded: "full", padding: "pill-xs", surface: "subtle", color: "muted", children: members.length })] }), _jsx(Button, { type: "button", variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); setShowAll(true); }, className: "text-[length:var(--appkit-text-xs)] text-[var(--appkit-color-primary)] hover:underline ml-3 flex-shrink-0", children: "Pick items \u2192" })] }), open && (_jsx(Div, { className: `pb-[1rem] pt-[0.25rem] ${__O.xAuto}`, padding: "x-md", children: _jsx(Row, { gap: "3", className: "min-w-0", children: [...members]
                                 .sort((a, b) => (b.isGroupParent ? 1 : 0) - (a.isGroupParent ? 1 : 0))
                                 .map((m) => (_jsx(MemberThumb, { member: m, isCurrent: m.slug === currentSlug || m.id === currentSlug }, m.id))) }) }))] }), useDrawer ? (_jsx(SideDrawer, { isOpen: showAll, onClose: () => setShowAll(false), title: label, children: tableContent })) : (_jsx(Modal, { open: showAll, onClose: () => setShowAll(false), title: label, children: tableContent }))] }));
 }

package/dist/features/tester/seed-data/tester-checklist-seed-data.js

@@ -1156,6 +1156,36 @@ const rawTesterChecklistItems = [
                     description: "The fix had to cover the update path as well as create — a create-only fix would mean the listing saved correctly and then lost its per-type block the first time anyone edited anything else (Root Cause #39).",
                     href: "/store/classified",
                 },
+                {
+                    key: "notification-email-is-a-real-email",
+                    label: "A notification email has a heading, a lead line and a button — not one bare paragraph",
+                    description: "`emailHtml` existed on the input since the channel was built and 0 of 40 call sites passed it, so all 28 notification types shipped `<p>{message}</p>`. Trigger any notification that emails (place an order as a buyer). The mail must carry the site header, a bold title, a one-line lead saying what kind of mail it is, the message, and a CTA button.",
+                    href: "/user/notifications",
+                },
+                {
+                    key: "notification-click-lands-on-the-right-page",
+                    label: "Clicking a notification opens the record it is about, in YOUR portal",
+                    description: "actionUrl was set by 5 of 40 writers, so 35 notifications had nothing to click. It is now resolved centrally from relatedType + relatedId + audience. Check that an order notification opens /user/orders/view/... for the buyer and /store/orders/.../view for the seller — the SAME event, two destinations. A bid or offer lands on the list, deliberately: no per-record page exists in any role yet.",
+                    href: "/user/notifications",
+                },
+                {
+                    key: "admin-can-filter-every-notification-type",
+                    label: "The admin notifications filter offers all 28 types, and no imaginary ones",
+                    description: "The chips came from a hand-written 9-value list while the document held 27 — so 18 real types were unfilterable and two chips (`review_posted`, `payout_processed`) matched zero rows because no notification has ever had those values. The list is derived now. Confirm `offer_received` and `payment_review` are both present.",
+                    href: "/admin/notifications",
+                },
+                {
+                    key: "admin-can-allowlist-every-type-per-channel",
+                    label: "Site Settings can allow-list offer and payment-review notifications for email",
+                    description: "Same 9-value list fed the per-channel allow-list, so an admin literally could not allow-list `offer_received` or `payment_review` — the toggle did not exist. Open Site Settings → Notifications and confirm every type appears in the per-channel type picker.",
+                    href: "/admin/site",
+                },
+                {
+                    key: "notification-optout-is-honoured-for-emi-and-payment",
+                    label: "Turning off order updates actually stops EMI and payment-review emails",
+                    description: "`typeToPrefsKey` was a Partial map missing `emi_installment_due_soon`, `emi_installment_overdue` and `payment_review` — and a missing key meant the opt-out check never ran, so the user's setting was silently ignored while reading as honoured. Turn off Order updates in /user/settings, then trigger a payment-review notification and confirm no email arrives.",
+                    href: "/user/settings",
+                },
                 {
                     key: "ticket-resolution-timestamp-stamped",
                     label: "Resolving a support ticket records WHEN it was resolved",

package/dist/index.d.ts

@@ -678,7 +678,8 @@ export { groupedListingsSeedData } from "./seed/index";
 export type { CouponDiscountType } from "./seed/index";
 export type { FirestoreIndexConfig } from "./seed/index";
 export type { HomepageSectionType } from "./seed/index";
-export type { NotificationType } from "./seed/index";
+export type { NotificationType } from "./features/admin/schemas/firestore";
+export { NOTIFICATION_TYPE_VALUES } from "./features/admin/schemas/firestore";
 export type { PayoutStatus } from "./seed/index";
 export type { SeedAddressDocument } from "./seed/index";
 export type { SeedBaseOrderDocument } from "./seed/index";
@@ -1047,7 +1048,6 @@ export type { UserAccountHubViewProps } from "./features/account/index";
 export type { UserAddress } from "./features/account/index";
 export type { UserAddressesViewLabels } from "./features/account/index";
 export type { UserAddressesViewProps } from "./features/account/index";
-export type { UserNotification } from "./features/account/index";
 export type { UserNotificationsViewLabels } from "./features/account/index";
 export type { UserNotificationsViewProps } from "./features/account/index";
 export type { UserOffer } from "./features/account/index";
@@ -1551,7 +1551,8 @@ export { updateBlogPostSchema } from "./features/blog/server";
 export type { BlogPostDetailResponse } from "./features/blog/server";
 export type { CreateBlogPostInput } from "./features/blog/server";
 export type { UpdateBlogPostInput } from "./features/blog/server";
-export { addItemToCart } from "./features/cart/server";
+export { addItemToCart, assertCanAddNewItems, addGroupLineToCart, updateCartGroupMembers, GROUP_LINE_MAX_MEMBERS, type AddGroupLineToCartInput, } from "./features/cart/server";
+export { toPublicGroupMember, toPublicGroupMembers, type PublicGroupMember, } from "./_internal/server/features/products/group-members";
 export { clearCart } from "./features/cart/server";
 export { getCart } from "./features/cart/server";
 export { mergeGuestCart } from "./features/cart/server";

package/dist/index.js

@@ -1395,6 +1395,7 @@ export { conversationsSeedData } from "./seed/index";
 // SB-UNI-B â€" sublistingCategoriesSeedData merged into categoriesSeedData with categoryType:"sublisting".
 // groupedListingsSeedData - Model for grouped listings seed data.
 export { groupedListingsSeedData } from "./seed/index";
+export { NOTIFICATION_TYPE_VALUES } from "./features/admin/schemas/firestore";
 // ./monitoring/index
 // [UTIL]-Pure utility â€" no framework or runtime dependency; safe to import from any environment.
 // ANALYTICS_EVENTS - Constant used across modules.
@@ -2910,7 +2911,12 @@ export { updateBlogPostSchema } from "./features/blog/server";
 // ./features/cart/server
 // [SERVER-ONLY]-Server-only â€" uses Node.js, Next.js server internals, or third-party server SDKs (auth, email, payment, shipping).
 // addItemToCart - Helper for add item to cart.
-export { addItemToCart } from "./features/cart/server";
+// assertCanAddNewItems - the lane gate every add-to-cart entry point must run.
+// addGroupLineToCart - add a buyer-assembled group selection as ONE line.
+export { addItemToCart, assertCanAddNewItems, addGroupLineToCart, updateCartGroupMembers, GROUP_LINE_MAX_MEMBERS, } from "./features/cart/server";
+// Public allow-list projection for a product-group member — see
+// § "Public Data Projections".
+export { toPublicGroupMember, toPublicGroupMembers, } from "./_internal/server/features/products/group-members";
 // [SERVER-ONLY]-Server-only â€" uses Node.js, Next.js server internals, or third-party server SDKs (auth, email, payment, shipping).
 // clearCart - Shared export for clear cart.
 export { clearCart } from "./features/cart/server";

package/dist/schemas/registry.d.ts

@@ -2658,6 +2658,7 @@ export declare const SCHEMAS: {
                 shelf: string;
                 bin: string;
             } | undefined;
+            offerId?: string | undefined;
             orderType?: string | undefined;
             imageUrls?: string[] | undefined;
             refundFeeDeducted?: number | undefined;
@@ -2695,7 +2696,6 @@ export declare const SCHEMAS: {
             }[] | undefined;
             emiRemainingBalance?: number | undefined;
             emiComplete?: boolean | undefined;
-            offerId?: string | undefined;
             isNonRefundable?: boolean | undefined;
             paymentBatchId?: string | undefined;
             refunds?: {
@@ -2806,6 +2806,7 @@ export declare const SCHEMAS: {
                 shelf: string;
                 bin: string;
             } | undefined;
+            offerId?: string | undefined;
             orderType?: string | undefined;
             imageUrls?: string[] | undefined;
             refundFeeDeducted?: number | undefined;
@@ -2843,7 +2844,6 @@ export declare const SCHEMAS: {
             }[] | undefined;
             emiRemainingBalance?: number | undefined;
             emiComplete?: boolean | undefined;
-            offerId?: string | undefined;
             isNonRefundable?: boolean | undefined;
             paymentBatchId?: string | undefined;
             refunds?: {

package/dist/seed/factories/notification.factory.d.ts

@@ -1,4 +1,5 @@
-export type NotificationType = "order_placed" | "order_shipped" | "order_delivered" | "order_cancelled" | "bid_outbid" | "bid_won" | "review_received" | "payout_completed" | "system";
+import type { NotificationType } from "../../features/admin/schemas/firestore";
+export type { NotificationType };
 export interface SeedNotificationDocument {
     id: string;
     userId: string;