@mohasinac/appkit 4.62.0 → 4.63.0

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

@@ -0,0 +1,74 @@
+"use client";
+import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
+import { useCallback, useMemo, useState } from "react";
+import { Alert, Badge, Button, Div, Heading, Row, Stack, Text, } from "../../../ui/index.js";
+import { useToast } from "../../../ui/components/Toast.js";
+import { normalizeError } from "../../../errors/normalize.js";
+import { ACTIONS } from "../../../_internal/shared/actions/action-registry.js";
+import { INTEGRATIONS, INTEGRATION_ORDER, allIntegrationsPatch, integrationPatch, readIntegrationStates, } from "../../../_internal/shared/features/admin/integrations.js";
+/** What the switch currently is, in the admin's vocabulary rather than a boolean. */
+function stateLabel(id, enabled) {
+    /*
+     * 🛑 Shipping is NOT on/off, and this is the one row where saying so matters.
+     *
+     * "Off" selects `provider: "manual"` — the self-ship flow sellers already
+     * use and claim postage back through. Rendering that as "Disabled" beside
+     * three switches that genuinely stop things would invite an admin to read it
+     * as "shipping is broken", and the descriptor's own note says the switch
+     * "selects manual rather than stopping anything".
+     */
+    if (id === "shipping")
+        return enabled ? "We book couriers" : "Sellers self-ship";
+    return enabled ? "On" : "Off";
+}
+export function AdminIntegrationsView({ settings, onSave }) {
+    const { showToast } = useToast();
+    const [pending, setPending] = useState(null);
+    /*
+     * 🛑 Derived from the prop, NOT copied into state.
+     *
+     * A `useState(readIntegrationStates(settings))` would go stale the moment
+     * the page refetched, and the staleness would be invisible: the row would
+     * read "On" while the guard enforcing it read "off". This page's entire job
+     * is to agree with the guard, so it re-derives rather than remembers.
+     */
+    const states = useMemo(() => readIntegrationStates(settings), [settings]);
+    const apply = useCallback(async (key, patch, label) => {
+        setPending(key);
+        try {
+            await onSave(patch);
+            showToast(`${label} saved.`, "success");
+        }
+        catch (err) {
+            void normalizeError(err);
+            /*
+             * 🛑 No optimistic flip anywhere in this component, so a failure needs
+             * no rollback — the row still shows what the server last told us. On a
+             * kill switch the cost of the two mistakes is wildly asymmetric: a
+             * row that briefly lags is a nuisance, while a row claiming "payments
+             * off" over a live gateway is the failure the model's own header warns
+             * about.
+             */
+            showToast(`Couldn't save ${label.toLowerCase()}. Nothing was changed.`, "error");
+        }
+        finally {
+            setPending(null);
+        }
+    }, [onSave, showToast]);
+    return (_jsxs(Stack, { gap: "lg", children: [_jsxs(Stack, { gap: "xs", children: [_jsx(Heading, { level: 1, size: "xl", weight: "semibold", color: "primary", children: "Integrations" }), _jsx(Text, { size: "sm", color: "muted", children: "What the platform talks to, and what happens when it stops. Each switch writes the same settings field the code enforcing it reads." })] }), _jsx(Alert, { variant: "warning", title: "Switch payments off only when nobody is checking out", children: "Turning online payments off also closes the routes that FINISH a payment, including the gateway webhook. A buyer part-way through the gateway can be charged without an order being placed, and the gateway will not retry. There is no safe mid-checkout moment." }), _jsx(Stack, { gap: "md", children: INTEGRATION_ORDER.map((id) => {
+                    const d = INTEGRATIONS[id];
+                    const enabled = states[id].enabled;
+                    const busy = pending === id || pending === "all";
+                    const action = enabled
+                        ? ACTIONS.ADMIN["disable-integration"]
+                        : ACTIONS.ADMIN["enable-integration"];
+                    return (_jsx(Div, { surface: d.severe && enabled ? "warning-surface" : "card", padding: "md", rounded: "lg", border: "default", children: _jsxs(Stack, { gap: "sm", children: [_jsxs(Row, { align: "start", justify: "between", gap: "md", className: "min-w-0", children: [_jsxs(Stack, { gap: "xs", className: "min-w-0", children: [_jsxs(Row, { align: "center", gap: "sm", className: "min-w-0", children: [_jsx(Text, { weight: "semibold", color: "primary", className: "min-w-0", children: d.label }), _jsx(Badge, { variant: enabled ? "success" : "inactive", children: stateLabel(id, enabled) }), d.severe && _jsx(Badge, { variant: "warning", children: "High impact" })] }), _jsx(Text, { size: "sm", color: "muted", children: d.summary })] }), _jsx(Button, { action: action, 
+                                            /*
+                                             * 🛑 The dialog body is composed HERE from the descriptor.
+                                             * The ActionDef ships an empty body on purpose — see its
+                                             * note. A per-integration consequence copied into the
+                                             * registry would be a second source for one sentence.
+                                             */
+                                            confirmationBody: enabled ? d.offConsequence : undefined, isLoading: busy, disabled: busy, onClick: () => apply(id, integrationPatch(id, !enabled), `${d.label} ${enabled ? "off" : "on"}`) })] }), _jsxs(Text, { size: "xs", color: "muted", children: [_jsxs(Text, { as: "span", size: "xs", weight: "semibold", color: "muted", children: ["Off means:", " "] }), d.offConsequence] }), _jsxs(Row, { align: "center", gap: "sm", className: "min-w-0", children: [_jsx(Text, { size: "xs", color: "faint", numeric: true, className: "min-w-0", truncate: 1, children: d.field }), _jsx(Text, { size: "xs", color: "faint", children: "\u00B7" }), _jsx(Text, { size: "xs", color: "faint", className: "min-w-0", truncate: 1, children: d.routeGuard === "gated" ? "API routes close when off" : "no route gate" })] })] }) }, id));
+                }) }), _jsxs(Row, { align: "center", gap: "sm", border: "top-subtle", padding: "t-md", children: [_jsx(Button, { action: ACTIONS.ADMIN["enable-all-integrations"], isLoading: pending === "all", disabled: pending !== null, onClick: () => apply("all", allIntegrationsPatch(true), "All integrations on") }), _jsx(Button, { action: ACTIONS.ADMIN["disable-all-integrations"], isLoading: pending === "all", disabled: pending !== null, onClick: () => apply("all", allIntegrationsPatch(false), "All integrations off") })] })] }));
+}

package/dist/features/admin/components/AdminInvoicingView.d.ts

@@ -0,0 +1,2 @@
+import React from "react";
+export declare function AdminInvoicingView(): React.JSX.Element;

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

@@ -0,0 +1,65 @@
+"use client";
+import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
+import { useState } from "react";
+import { Button, Card, Disclaimer, Div, Heading, Row, Stack, Text } from "../../../ui/index.js";
+import { FieldInput } from "../../../ui/forms/FieldInput.js";
+import { Icon } from "../../../ui/components/Icon.js";
+import { ADMIN_ENDPOINTS } from "../../../constants/api-endpoints.js";
+/**
+ * Admin → Finance → Invoicing: the GSTR-1 draft export (Phase 7d).
+ *
+ * ## 🛑 Why this page exists at all
+ *
+ * The export is an API route, and an API route with no surface is a dead end —
+ * reachable only by someone who already knows the URL, which is Root Cause
+ * #56's shape (16 of 70 listing views had no way to open a row). The person
+ * who needs this is an accountant at quarter end, and they will not type a
+ * query string.
+ *
+ * ## 🛑 It downloads; it does not file
+ *
+ * Nothing here talks to the GST portal — there is no webhook facility in
+ * existence and no GSP contract. The copy says "draft" everywhere it can,
+ * because a page that implied a return had been FILED when it had not is the
+ * most consequential thing this feature could get wrong.
+ */
+/** Indian FY quarters, which is the unit a GSTR-1 is filed in. */
+const QUARTERS = [
+    { label: "Q1 — Apr to Jun", fromMonth: "04-01", toMonth: "06-30" },
+    { label: "Q2 — Jul to Sep", fromMonth: "07-01", toMonth: "09-30" },
+    { label: "Q3 — Oct to Dec", fromMonth: "10-01", toMonth: "12-31" },
+    /*
+     * 🛑 Q4 SPANS A CALENDAR YEAR BOUNDARY. Jan–Mar of FY 26-27 is calendar
+     * 2027, not 2026 — the same off-by-one the `financialYearLabel` helper
+     * exists to contain, surfacing here because a quarter picker has to know it
+     * too. `yearOffset` is what carries it; dropping it would export the wrong
+     * three months with no visible error.
+     */
+    { label: "Q4 — Jan to Mar", fromMonth: "01-01", toMonth: "03-31", yearOffset: 1 },
+];
+function defaultFyStartYear() {
+    const now = new Date();
+    // Zero-based month: April is 3. Jan–Mar belongs to the FY that started last
+    // calendar year.
+    return now.getMonth() >= 3 ? now.getFullYear() : now.getFullYear() - 1;
+}
+export function AdminInvoicingView() {
+    const [fyStart, setFyStart] = useState(String(defaultFyStartYear()));
+    const [from, setFrom] = useState("");
+    const [to, setTo] = useState("");
+    const applyQuarter = (q) => {
+        const start = Number(fyStart);
+        if (!Number.isFinite(start))
+            return;
+        const offset = "yearOffset" in q ? q.yearOffset : 0;
+        setFrom(`${start + offset}-${q.fromMonth}`);
+        setTo(`${start + offset}-${q.toMonth}`);
+    };
+    const href = from && to
+        ? `${ADMIN_ENDPOINTS.INVOICING_GSTR1}?from=${from}&to=${to}`
+        : undefined;
+    return (_jsxs(Stack, { gap: "lg", children: [_jsxs(Div, { children: [_jsx(Heading, { level: 1, children: "Invoicing" }), _jsx(Text, { color: "muted", size: "sm", children: "Export the invoice and credit-note register for a period, as a draft for your accountant." })] }), _jsx(Card, { padding: "md", children: _jsxs(Stack, { gap: "md", children: [_jsx(Heading, { level: 2, children: "GSTR-1 draft" }), _jsx(Disclaimer, { id: "accounting-output" }), _jsx(Row, { gap: "sm", wrap: true, children: _jsx(Div, { className: "w-40", children: _jsx(FieldInput, { name: "fyStart", label: "FY starting", value: fyStart, onChange: setFyStart, hint: "e.g. 2026 for FY 26-27" }) }) }), _jsxs(Stack, { gap: "xs", children: [_jsx(Text, { size: "sm", weight: "semibold", children: "Quarter" }), _jsx(Row, { gap: "sm", wrap: true, children: QUARTERS.map((q) => (_jsx(Button, { type: "button", variant: "secondary", size: "sm", onClick: () => applyQuarter(q), children: q.label }, q.label))) })] }), _jsxs(Row, { gap: "sm", wrap: true, children: [_jsx(Div, { className: "w-48", children: _jsx(FieldInput, { name: "from", label: "From", value: from, onChange: setFrom }) }), _jsx(Div, { className: "w-48", children: _jsx(FieldInput, { name: "to", label: "To", value: to, onChange: setTo }) })] }), _jsxs(Row, { gap: "sm", align: "center", children: [_jsxs(Button, { type: "button", variant: "primary", disabled: !href, onClick: () => {
+                                        if (href)
+                                            window.location.href = href;
+                                    }, children: [_jsx(Icon, { name: "download", size: "sm" }), "Download draft CSV"] }), !href && (_jsx(Text, { color: "muted", size: "xs", children: "Pick a quarter, or enter both dates as YYYY-MM-DD." }))] }), _jsx(Text, { color: "muted", size: "xs", children: "Credit notes appear as negative rows, so the TOTAL row is net of refunds. Any paid order in the window that carries no invoice number is listed at the bottom of the file as a warning \u2014 those are not included in the totals and need looking at." })] }) })] }));
+}

package/dist/features/admin/components/AdminOrderEditorView.d.ts

@@ -1,4 +1,5 @@
 import { type OrderAddonBadgesOrder } from "../../orders/components/OrderAddonBadges";
+import type { OrderSourceContext } from "../../orders/schemas/firestore";
 import React from "react";
 export interface AdminOrderItemRow {
     productId: string;
@@ -7,6 +8,10 @@ export interface AdminOrderItemRow {
     quantity: number;
     unitPrice: number;
     totalPrice: number;
+    groupSlug?: string;
+    groupTitle?: string;
+    bundleCategorySlug?: string;
+    bundleProductIds?: string[];
 }
 export interface AdminOrderEditorViewProps {
     open: boolean;
@@ -36,5 +41,32 @@ export interface AdminOrderEditorViewProps {
      * coupon is what an admin needs when reconciling a discounted order.
      */
     addons?: OrderAddonBadgesOrder;
+    /**
+     * How the buyer earned this price — a won auction, a buyout, an agreed
+     * offer, a pre-order deposit, prize-draw entries.
+     *
+     * Support is the surface that needs this most: a buyer disputing "I agreed
+     * ₹X" is otherwise unanswerable from the admin side, because the negotiated
+     * amount lives only in `sourceContext` and nothing rendered it anywhere.
+     * `GET /api/admin/orders/[id]` already returns it via `maskedOrder`.
+     */
+    sourceContext?: OrderSourceContext;
+    /** For the provenance card's money formatting; falls back to the default. */
+    currency?: string;
+    /**
+     * D55 — the pre-order balance block, present once a seller has opened it.
+     *
+     * 🛑 Admin is the ONLY role that sees `proofUrl` here, matching the deposit:
+     * the seller's own endpoint strips it, because a bank screenshot is the
+     * buyer's and verification is not the seller's job. Reviewing a payment to
+     * yourself is not a review.
+     */
+    balance?: {
+        amount: number;
+        status: "awaiting" | "submitted" | "paid" | "rejected";
+        proofUrl?: string;
+        transactionId?: string;
+        reviewNote?: string;
+    };
 }
-export declare function AdminOrderEditorView({ open, onClose, orderId, orderLabel, currentStatus, items, paymentProofUrl, paymentTransactionId, paymentMethod, paymentStatus, displayedUpiId, buyerReportedUpiId, paymentUpiMismatch, buyerMarkedPaid, buyerFraudAgreementAccepted, paymentReviewOutcome, addons, }: AdminOrderEditorViewProps): React.JSX.Element;
+export declare function AdminOrderEditorView({ open, onClose, orderId, orderLabel, currentStatus, items, sourceContext, currency, paymentProofUrl, paymentTransactionId, paymentMethod, paymentStatus, displayedUpiId, buyerReportedUpiId, paymentUpiMismatch, buyerMarkedPaid, buyerFraudAgreementAccepted, paymentReviewOutcome, addons, balance, }: AdminOrderEditorViewProps): React.JSX.Element;

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

@@ -1,6 +1,8 @@
 "use client";
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { OrderAddonBadges } from "../../orders/components/OrderAddonBadges.js";
+import { OrderProvenanceCard } from "../../orders/components/OrderProvenanceCard.js";
+import { groupOrderItemsByLine } from "../../orders/utils/bundle-grouping.js";
 import { isManualPaymentMethod } from "../../orders/constants/payment-window.js";
 import { useApiMutation } from "@mohasinac/appkit/client";
 import React, { useState } from "react";
@@ -18,6 +20,13 @@ import { useFormShellState, FormShellContext } from "../../../ui/forms/index.js"
 import { applyZodIssues } from "../../../ui/forms/apply-zod-issues.js";
 import { FormErrorSummary } from "../../../ui/forms/FormErrorSummary.js";
 import { ValidationError } from "../../../errors/validation-error.js";
+/**
+ * One item row. Extracted so the single and grouped render paths share exactly
+ * one piece of markup rather than two that can drift.
+ */
+function adminOrderItemRow(item, key) {
+    return (_jsxs(Div, { layout: "flex", align: "center", gap: "3", padding: "sm", children: [_jsx(Div, { className: "h-10 w-10 shrink-0", rounded: "md", overflow: "hidden", children: _jsx(MediaImage, { src: item.image, alt: item.title, size: "thumbnail" }) }), _jsxs(Div, { className: "min-w-0 flex-1", children: [_jsx(Text, { size: "sm", className: "truncate", weight: "medium", children: item.title }), _jsxs(Text, { size: "xs", color: "muted", children: ["Qty: ", item.quantity, " \u00D7 ", toCurrency(item.unitPrice)] })] }), _jsx(Text, { size: "sm", className: "shrink-0", weight: "semibold", children: toCurrency(item.totalPrice) })] }, key));
+}
 const STATUS_OPTIONS = [
     { label: "Pending", value: "pending" },
     { label: "Processing", value: "processing" },
@@ -37,7 +46,7 @@ const CARRIER_OPTIONS = [
     { label: "Other", value: "Other" },
 ];
 // --- Component ---------------------------------------------------------------
-export function AdminOrderEditorView({ open, onClose, orderId, orderLabel, currentStatus, items, paymentProofUrl, paymentTransactionId, paymentMethod, paymentStatus, displayedUpiId, buyerReportedUpiId, paymentUpiMismatch, buyerMarkedPaid, buyerFraudAgreementAccepted, paymentReviewOutcome, addons, }) {
+export function AdminOrderEditorView({ open, onClose, orderId, orderLabel, currentStatus, items, sourceContext, currency, paymentProofUrl, paymentTransactionId, paymentMethod, paymentStatus, displayedUpiId, buyerReportedUpiId, paymentUpiMismatch, buyerMarkedPaid, buyerFraudAgreementAccepted, paymentReviewOutcome, addons, balance, }) {
     const queryClient = useQueryClient();
     const { showToast } = useToast();
     const [draft, setDraft] = React.useState({
@@ -74,6 +83,19 @@ export function AdminOrderEditorView({ open, onClose, orderId, orderLabel, curre
     const [reviewNoteError, setReviewNoteError] = React.useState(null);
     const [isRequestingReupload, setIsRequestingReupload] = useState(false);
     const [isRejectingFraud, setIsRejectingFraud] = useState(false);
+    /*
+     * D55 — the balance review's OWN note and its own pending flags.
+     *
+     * 🛑 Not shared with `reviewNote` above, which belongs to the deposit's
+     * three actions. One note box feeding two reviews is the bug where an admin
+     * types "blurry screenshot" meaning the balance and sends it as a
+     * fraud-rejection reason on the deposit — and the deposit's rejection bans
+     * the account for 7 days.
+     */
+    const [balanceNote, setBalanceNote] = React.useState("");
+    const [balanceNoteError, setBalanceNoteError] = React.useState(null);
+    const [isAcceptingBalance, setIsAcceptingBalance] = useState(false);
+    const [isRejectingBalance, setIsRejectingBalance] = useState(false);
     // This used to inline `paymentMethod === "cash" || === "upi_manual"`, which
     // omitted `emi` — so the whole proof panel never rendered for an EMI order and
     // an admin had no way to verify one, even though the list row said "Awaiting
@@ -96,6 +118,10 @@ export function AdminOrderEditorView({ open, onClose, orderId, orderLabel, curre
                 notes: "",
             });
             setReviewNote("");
+            // Reset with its sibling — a note typed for one order must not reappear
+            // under the next one the drawer opens.
+            setBalanceNote("");
+            setBalanceNoteError(null);
         }
     }, [open, currentStatus]);
     const saveMutation = useApiMutation({
@@ -207,7 +233,49 @@ export function AdminOrderEditorView({ open, onClose, orderId, orderLabel, curre
             setIsRejectingFraud(false);
         }
     };
-    return (_jsx(SideDrawer, { isOpen: open, onClose: onClose, title: orderLabel ? `Order: ${orderLabel}` : "Update Order", children: _jsx(FormShellContext.Provider, { value: form.shellCtx, children: _jsxs(Stack, { gap: "md", padding: "md", children: [_jsx(FormErrorSummary, {}), addons && _jsx(OrderAddonBadges, { order: addons, variant: "detail" }), items && items.length > 0 && (_jsxs(Stack, { gap: "xs", children: [_jsxs(Label, { size: "sm", weight: "medium", color: "primary", children: ["Items (", items.length, ")"] }), _jsx(Div, { className: "divide-y divide-[var(--appkit-color-border)] border border-[var(--appkit-color-border)]", rounded: "lg", children: items.map((item, i) => (_jsxs(Div, { layout: "flex", align: "center", gap: "3", padding: "sm", children: [_jsx(Div, { className: "h-10 w-10 shrink-0", rounded: "md", overflow: "hidden", children: _jsx(MediaImage, { src: item.image, alt: item.title, size: "thumbnail" }) }), _jsxs(Div, { className: "min-w-0 flex-1", children: [_jsx(Text, { size: "sm", className: "truncate", weight: "medium", children: item.title }), _jsxs(Text, { size: "xs", color: "muted", children: ["Qty: ", item.quantity, " \u00D7 ", toCurrency(item.unitPrice)] })] }), _jsx(Text, { size: "sm", className: "shrink-0", weight: "semibold", children: toCurrency(item.totalPrice) })] }, item.productId || i))) })] })), _jsx(SectionForm, { sections: sections, values: draft, onChange: (partial) => setDraft((d) => ({ ...d, ...partial })), onSubmit: () => {
+    /**
+     * D55 — accept or reject the pre-order BALANCE proof.
+     *
+     * 🛑 One handler, because the two verdicts differ by a boolean and a note
+     * and nothing else. The deposit has three handlers because its three
+     * outcomes are genuinely different operations (verify / re-ask / cancel +
+     * ban); mirroring that shape here would be three copies of one request.
+     *
+     * The note requirement is checked in BOTH places on purpose: here so the
+     * admin is told before the round trip, and in `reviewBalanceProof` because
+     * the route is reachable without this component.
+     */
+    const handleBalanceReview = async (approve) => {
+        if (!orderId)
+            return;
+        if (!approve && !balanceNote.trim()) {
+            setBalanceNoteError("Say what was wrong — without it the buyer will send the same proof again.");
+            return;
+        }
+        setBalanceNoteError(null);
+        const setPending = approve ? setIsAcceptingBalance : setIsRejectingBalance;
+        setPending(true);
+        try {
+            await apiClient.post(ADMIN_ENDPOINTS.ORDER_BALANCE_REVIEW(orderId), {
+                approve,
+                ...(balanceNote.trim() ? { note: balanceNote.trim() } : {}),
+            });
+            showToast(approve
+                ? "Balance accepted. The order is clear to ship."
+                : "Proof returned to the buyer with your note.", "success");
+            queryClient.invalidateQueries({ queryKey: ["admin", "orders"] });
+            onClose();
+        }
+        catch (err) {
+            void normalizeError(err);
+            // Authored copy — `err.message` is server text (Root Cause #86).
+            showToast("Could not record that decision.", "error");
+        }
+        finally {
+            setPending(false);
+        }
+    };
+    return (_jsx(SideDrawer, { isOpen: open, onClose: onClose, title: orderLabel ? `Order: ${orderLabel}` : "Update Order", children: _jsx(FormShellContext.Provider, { value: form.shellCtx, children: _jsxs(Stack, { gap: "md", padding: "md", children: [_jsx(FormErrorSummary, {}), addons && _jsx(OrderAddonBadges, { order: addons, variant: "detail" }), _jsx(OrderProvenanceCard, { sourceContext: sourceContext, currency: currency }), items && items.length > 0 && (_jsxs(Stack, { gap: "xs", children: [_jsxs(Label, { size: "sm", weight: "medium", color: "primary", children: ["Items (", items.length, ")"] }), _jsx(Div, { className: "divide-y divide-[var(--appkit-color-border)] border border-[var(--appkit-color-border)]", rounded: "lg", children: groupOrderItemsByLine(items).map((group, gi) => group.kind === "single" ? (adminOrderItemRow(group.item, `s-${group.index}`)) : (_jsxs(Div, { children: [_jsxs(Div, { layout: "flex", align: "center", gap: "2", padding: "x-sm", surface: "muted", children: [_jsx(Text, { size: "xs", weight: "semibold", children: group.groupTitle ?? "Grouped purchase" }), _jsx(Text, { size: "xs", color: "muted", children: `${group.memberCount} ${group.memberCount === 1 ? "item" : "items"} · one line` })] }), _jsx(Div, { paddingX: "l-md", children: group.items.map(({ item, index }) => adminOrderItemRow(item, `m-${index}`)) })] }, `g-${gi}`))) })] })), _jsx(SectionForm, { sections: sections, values: draft, onChange: (partial) => setDraft((d) => ({ ...d, ...partial })), onSubmit: () => {
                             // SectionForm surfaces errors and then submits regardless, so the
                             // guard is here.
                             form.clearErrors();
@@ -217,5 +285,15 @@ export function AdminOrderEditorView({ open, onClose, orderId, orderLabel, curre
                                 return;
                             }
                             saveMutation.mutate();
-                        }, onValidationChange: () => form.validate(draft), schema: adminOrderUpdateSchema, openIds: nav.openIds, onOpenChange: nav.setOpenIds, submitLabel: "Save changes", cancelLabel: "Cancel", onCancel: onClose, isLoading: saveMutation.isPending }), isManualPayment && (_jsxs(Stack, { gap: "xs", children: [_jsx(Label, { size: "sm", weight: "medium", color: "primary", children: "Payment Proof" }), isRejected && (_jsx(Text, { size: "xs", className: "text-error", weight: "medium", children: "Payment rejected as fraud \u2014 no further action available here." })), isReuploadRequested && !isVerified && (_jsx(Text, { size: "xs", className: "text-warning", weight: "medium", children: "Re-upload requested \u2014 waiting for the buyer to submit a new proof." })), paymentProofUrl ? (_jsxs(Stack, { gap: "xs", children: [_jsx(Div, { border: "default", rounded: "lg", overflow: "hidden", children: _jsx(MediaImage, { src: paymentProofUrl, alt: "Payment screenshot", size: "card" }) }), paymentTransactionId && (_jsxs(Text, { size: "xs", color: "muted", children: ["UTR: ", _jsx(Text, { as: "span", size: "xs", weight: "medium", children: paymentTransactionId })] })), needsVerification && (_jsxs(Stack, { gap: "xs", children: [(displayedUpiId || buyerReportedUpiId) && (_jsxs(Stack, { gap: "xs", children: [_jsxs(Text, { size: "xs", color: "muted", children: ["Expected UPI: ", _jsx(Text, { as: "span", size: "xs", weight: "medium", children: displayedUpiId || "—" }), " · ", "Buyer reported: ", _jsx(Text, { as: "span", size: "xs", weight: "medium", children: buyerReportedUpiId || "—" })] }), paymentUpiMismatch && (_jsx(Div, { rounded: "lg", padding: "inlineSm", className: "border border-error/20", surface: "danger-surface", children: _jsx(Text, { size: "xs", className: "text-error", weight: "semibold", children: "UPI mismatch \u2014 buyer-reported ID doesn't match the one shown for this order." }) }))] })), _jsxs(Text, { size: "xs", color: "muted", children: ["Buyer marked as paid: ", _jsx(Text, { as: "span", size: "xs", weight: "medium", children: buyerMarkedPaid ? "Yes" : "No" }), " · ", "Fraud agreement accepted: ", _jsx(Text, { as: "span", size: "xs", weight: "medium", children: buyerFraudAgreementAccepted ? "Yes" : "No" })] }), _jsxs(Stack, { gap: "xs", children: [_jsx(Label, { size: "sm", weight: "medium", color: "primary", children: "Review note" }), _jsxs(Text, { size: "xs", color: "muted", children: ["Sent with ", _jsx(Text, { as: "span", size: "xs", weight: "medium", children: "Request re-upload" }), " or", " ", _jsx(Text, { as: "span", size: "xs", weight: "medium", children: "Reject as fraud" }), " below. Not saved by \u201CSave changes\u201D."] }), _jsx(Textarea, { value: reviewNote, onChange: (e) => handleReviewNoteChange(e.target.value), rows: 2, "aria-invalid": reviewNoteError ? true : undefined, placeholder: "e.g. Screenshot is blurry \u2014 amount not readable / UPI ID doesn't match order" }), reviewNoteError && (_jsx(Text, { size: "xs", color: "error", role: "alert", children: reviewNoteError }))] }), _jsx(Button, { type: "button", action: ACTIONS.ADMIN["verify-payment"], onClick: handleVerifyPayment, isLoading: isVerifyingPayment, disabled: isVerifyingPayment, variant: "primary", className: "w-full" }), _jsx(Button, { type: "button", action: ACTIONS.ADMIN["request-payment-reupload"], onClick: handleRequestReupload, isLoading: isRequestingReupload, disabled: isRequestingReupload, variant: "secondary", className: "w-full" }), _jsx(Button, { type: "button", action: ACTIONS.ADMIN["reject-payment-fraud"], onClick: handleRejectFraud, isLoading: isRejectingFraud, disabled: isRejectingFraud, variant: "danger", className: "w-full" })] })), isVerified && (_jsx(Div, { rounded: "lg", padding: "inlineSm", className: "border border-success/20", surface: "success-surface", children: _jsx(Text, { size: "xs", className: "text-success", weight: "medium", children: "Payment verified" }) }))] })) : (_jsx(Text, { size: "xs", color: "faint", children: "No proof uploaded yet." }))] }))] }) }) }));
+                        }, onValidationChange: () => form.validate(draft), schema: adminOrderUpdateSchema, openIds: nav.openIds, onOpenChange: nav.setOpenIds, submitLabel: "Save changes", cancelLabel: "Cancel", onCancel: onClose, isLoading: saveMutation.isPending }), isManualPayment && (_jsxs(Stack, { gap: "xs", children: [_jsx(Label, { size: "sm", weight: "medium", color: "primary", children: "Payment Proof" }), isRejected && (_jsx(Text, { size: "xs", className: "text-error", weight: "medium", children: "Payment rejected as fraud \u2014 no further action available here." })), isReuploadRequested && !isVerified && (_jsx(Text, { size: "xs", className: "text-warning", weight: "medium", children: "Re-upload requested \u2014 waiting for the buyer to submit a new proof." })), paymentProofUrl ? (_jsxs(Stack, { gap: "xs", children: [_jsx(Div, { border: "default", rounded: "lg", overflow: "hidden", children: _jsx(MediaImage, { src: paymentProofUrl, alt: "Payment screenshot", size: "card" }) }), paymentTransactionId && (_jsxs(Text, { size: "xs", color: "muted", children: ["UTR: ", _jsx(Text, { as: "span", size: "xs", weight: "medium", children: paymentTransactionId })] })), needsVerification && (_jsxs(Stack, { gap: "xs", children: [(displayedUpiId || buyerReportedUpiId) && (_jsxs(Stack, { gap: "xs", children: [_jsxs(Text, { size: "xs", color: "muted", children: ["Expected UPI: ", _jsx(Text, { as: "span", size: "xs", weight: "medium", children: displayedUpiId || "—" }), " · ", "Buyer reported: ", _jsx(Text, { as: "span", size: "xs", weight: "medium", children: buyerReportedUpiId || "—" })] }), paymentUpiMismatch && (_jsx(Div, { rounded: "lg", padding: "inlineSm", className: "border border-error/20", surface: "danger-surface", children: _jsx(Text, { size: "xs", className: "text-error", weight: "semibold", children: "UPI mismatch \u2014 buyer-reported ID doesn't match the one shown for this order." }) }))] })), _jsxs(Text, { size: "xs", color: "muted", children: ["Buyer marked as paid: ", _jsx(Text, { as: "span", size: "xs", weight: "medium", children: buyerMarkedPaid ? "Yes" : "No" }), " · ", "Fraud agreement accepted: ", _jsx(Text, { as: "span", size: "xs", weight: "medium", children: buyerFraudAgreementAccepted ? "Yes" : "No" })] }), _jsxs(Stack, { gap: "xs", children: [_jsx(Label, { size: "sm", weight: "medium", color: "primary", children: "Review note" }), _jsxs(Text, { size: "xs", color: "muted", children: ["Sent with ", _jsx(Text, { as: "span", size: "xs", weight: "medium", children: "Request re-upload" }), " or", " ", _jsx(Text, { as: "span", size: "xs", weight: "medium", children: "Reject as fraud" }), " below. Not saved by \u201CSave changes\u201D."] }), _jsx(Textarea, { value: reviewNote, onChange: (e) => handleReviewNoteChange(e.target.value), rows: 2, "aria-invalid": reviewNoteError ? true : undefined, placeholder: "e.g. Screenshot is blurry \u2014 amount not readable / UPI ID doesn't match order" }), reviewNoteError && (_jsx(Text, { size: "xs", color: "error", role: "alert", children: reviewNoteError }))] }), _jsx(Button, { type: "button", action: ACTIONS.ADMIN["verify-payment"], onClick: handleVerifyPayment, isLoading: isVerifyingPayment, disabled: isVerifyingPayment, variant: "primary", className: "w-full" }), _jsx(Button, { type: "button", action: ACTIONS.ADMIN["request-payment-reupload"], onClick: handleRequestReupload, isLoading: isRequestingReupload, disabled: isRequestingReupload, variant: "secondary", className: "w-full" }), _jsx(Button, { type: "button", action: ACTIONS.ADMIN["reject-payment-fraud"], onClick: handleRejectFraud, isLoading: isRejectingFraud, disabled: isRejectingFraud, variant: "danger", className: "w-full" })] })), isVerified && (_jsx(Div, { rounded: "lg", padding: "inlineSm", className: "border border-success/20", surface: "success-surface", children: _jsx(Text, { size: "xs", className: "text-success", weight: "medium", children: "Payment verified" }) }))] })) : (_jsx(Text, { size: "xs", color: "faint", children: "No proof uploaded yet." }))] })), balance && (_jsxs(Stack, { gap: "xs", children: [_jsx(Label, { size: "sm", weight: "medium", color: "primary", children: "Pre-order balance" }), _jsxs(Text, { size: "xs", color: "muted", children: [toCurrency(balance.amount), " \u00B7", " ", balance.status === "paid"
+                                        ? "accepted"
+                                        : balance.status === "submitted"
+                                            ? "proof submitted, awaiting your decision"
+                                            : balance.status === "rejected"
+                                                ? "returned to the buyer"
+                                                : "open — the buyer has not paid yet", balance.transactionId ? ` · Ref ${balance.transactionId}` : ""] }), balance.status === "rejected" && balance.reviewNote && (_jsxs(Text, { size: "xs", color: "muted", children: ["Last note: ", balance.reviewNote] })), balance.status === "submitted" && (_jsxs(Stack, { gap: "xs", children: [balance.proofUrl && (_jsx(Div, { border: "default", rounded: "lg", overflow: "hidden", children: _jsx(MediaImage, { src: balance.proofUrl, alt: "Balance payment screenshot", size: "card" }) })), _jsx(Text, { size: "xs", color: "muted", children: "Check the reference against the bank before accepting. Accepting does NOT post a ledger entry \u2014 the sale was recognised in full when the order was created, so a second entry would double it." }), _jsx(Textarea, { value: balanceNote, onChange: (e) => {
+                                            setBalanceNote(e.target.value);
+                                            if (balanceNoteError)
+                                                setBalanceNoteError(null);
+                                        }, rows: 2, "aria-invalid": balanceNoteError ? true : undefined, placeholder: "Required when rejecting \u2014 e.g. reference does not match any credit we received" }), balanceNoteError && (_jsx(Text, { size: "xs", color: "error", role: "alert", children: balanceNoteError })), _jsx(Button, { type: "button", action: ACTIONS.ADMIN["verify-balance"], onClick: () => void handleBalanceReview(true), isLoading: isAcceptingBalance, disabled: isAcceptingBalance || isRejectingBalance, variant: "primary", className: "w-full" }), _jsx(Button, { type: "button", action: ACTIONS.ADMIN["reject-balance"], onClick: () => void handleBalanceReview(false), isLoading: isRejectingBalance, disabled: isAcceptingBalance || isRejectingBalance, variant: "secondary", className: "w-full" })] }))] }))] }) }) }));
 }

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

@@ -2,6 +2,16 @@ import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-run
 import { ShoppingBag, Hash, Scale, Wallet, RotateCcw, DollarSign, Gavel } from "lucide-react";
 import { Code, Div, Heading, Li, Row, Section, Span, Stack, Text, Ul } from "../../../ui/index.js";
 import { GC } from "../../_guide-cls.js";
+import { resolvePublishedValues } from "../../../_internal/shared/content/published-values.js";
+/*
+ * Phase 9 — read from the published-value registry rather than restated.
+ *
+ * 🛑 No settings argument, deliberately: this is the OBLIGATION-derived window
+ * and it has no settings override, so passing settings would imply an
+ * admin-editable number that does not exist (the 18-dead-settings pattern,
+ * from the "control that appears to do something the code ignores" direction).
+ */
+const { returnRespondHours: RETURN_RESPOND_HOURS, refundBusinessDaysMin: REFUND_DAYS_MIN, refundBusinessDaysMax: REFUND_DAYS_MAX, } = resolvePublishedValues();
 export function AdminOrdersGuideView() {
     return (_jsxs(Stack, { className: "max-w-3xl mx-auto", padding: "b-2xl", gap: "xl", children: [_jsxs(Section, { children: [_jsxs(Row, { className: "mb-2", align: "center", gap: "3", children: [_jsx(Row, { className: "flex-shrink-0 w-10 h-10 [background:linear-gradient(135deg,var(--appkit-color-primary-700)_0%,var(--appkit-color-cobalt)_100%)]", align: "center", justify: "center", rounded: "xl", children: _jsx(ShoppingBag, { className: "w-5 h-5 text-white" }) }), _jsx(Text, { className: "text-[var(--appkit-color-text-muted)] tracking-widest", size: "sm", weight: "semibold", transform: "uppercase", children: "Admin Guide" })] }), _jsx(Heading, { level: 1, className: "text-[var(--appkit-color-text)] mb-2", mdSize: "3xl", size: "2xl", weight: "bold", children: "Orders & Finance" }), _jsx(Text, { className: "text-[var(--appkit-color-text-muted)]", children: "Order statuses, payouts, disputes, returns, and commission math on LetItRip." })] }), [
                 {
@@ -11,7 +21,18 @@ export function AdminOrdersGuideView() {
                                 ["PROCESSING", "Seller has acknowledged and is packing."],
                                 ["SHIPPED", "Dispatched. Tracking number provided by seller."],
                                 ["DELIVERED", "Carrier confirmed delivery."],
-                                ["RETURN_REQUESTED", "Buyer opened a return. Seller has 3 days to respond."],
+                                /*
+                                 * 🛑 Phase 9, decision 23 — this said "3 days" while the FAQ
+                                 * said 48 hours, and the FAQ WINS: it is the buyer-facing
+                                 * promise and the one a buyer has already read. The number now
+                                 * comes from `OBLIGATION_POLICIES["return-respond"]`, which is
+                                 * what actually closes the window — so a staff-facing guide can
+                                 * no longer quietly contradict what the buyer was told.
+                                 */
+                                [
+                                    "RETURN_REQUESTED",
+                                    `Buyer opened a return. Seller has ${RETURN_RESPOND_HOURS} hours to respond.`,
+                                ],
                                 ["REFUNDED", "Return accepted and refund issued via Razorpay."],
                                 ["CANCELLED", "Cancelled before shipping. Payment returned."],
                             ].map(([status, note]) => (_jsxs(Row, { gap: "3", children: [_jsx(Text, { className: "flex-shrink-0 font-mono text-[var(--appkit-color-primary)] w-40", size: "xs", weight: "semibold", children: status }), _jsx(Text, { className: "text-[var(--appkit-color-text-muted)]", children: note })] }, status))), _jsx(Text, { className: "text-[var(--appkit-color-text-muted)] pt-[0.5rem]", size: "sm", children: "Admin-only transitions: force-cancel (any status before DELIVERED), force-refund (post-DELIVERED dispute)." })] })),
@@ -30,7 +51,7 @@ export function AdminOrdersGuideView() {
                 },
                 {
                     Icon: RotateCcw, title: "Return Requests",
-                    content: (_jsxs(Ul, { className: GC.listMuted, children: [_jsx(Li, { children: "Admin can view the return reason and attached photos on the order detail page." }), _jsxs(Li, { children: [_jsx(Span, { weight: "bold", children: "When to override a seller rejection" }), ": Only when photographic evidence clearly supports the buyer's claim of a significant not-as-described defect. Do not override for buyer's remorse."] }), _jsx(Li, { children: "After admin override: refund is issued automatically via Razorpay (3\u20137 business days for card, 1\u20132 for UPI). Seller payout for that order is withheld." })] })),
+                    content: (_jsxs(Ul, { className: GC.listMuted, children: [_jsx(Li, { children: "Admin can view the return reason and attached photos on the order detail page." }), _jsxs(Li, { children: [_jsx(Span, { weight: "bold", children: "When to override a seller rejection" }), ": Only when photographic evidence clearly supports the buyer's claim of a significant not-as-described defect. Do not override for buyer's remorse."] }), _jsxs(Li, { children: ["After admin override: the refund is issued automatically (", REFUND_DAYS_MIN, "\u2013", REFUND_DAYS_MAX, " business days for card, 1\u20132 for UPI). Seller payout for that order is withheld."] })] })),
                 },
                 {
                     Icon: DollarSign, title: "Commission & Settlement Math",

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

@@ -1,5 +1,6 @@
 "use client";
 import { jsx as _jsx, Fragment as _Fragment, jsxs as _jsxs } from "react/jsx-runtime";
+import { groupOrderItemsByLine } from "../../orders/utils/bundle-grouping.js";
 import { sieveFilter, SIEVE_OP } from "@mohasinac/appkit/client";
 import { sortBy } from "@mohasinac/appkit/client";
 import React, { useState, useCallback } from "react";
@@ -61,14 +62,70 @@ function toOrderItemRows(item) {
         quantity: typeof entry.quantity === "number" ? entry.quantity : 1,
         unitPrice: typeof entry.unitPrice === "number" ? entry.unitPrice : 0,
         totalPrice: typeof entry.totalPrice === "number" ? entry.totalPrice : 0,
+        /*
+         * The collapse keys, carried so the editor can group a bundle or
+         * buyer-assembled group back into one line the way the buyer's receipt
+         * does. Narrowed by `typeof` like every field above rather than cast —
+         * this function's whole job is turning a loose record into a typed row,
+         * and a cast here is how `title`/`price` once slipped through (RC#45).
+         */
+        groupSlug: typeof entry.groupSlug === "string" ? entry.groupSlug : undefined,
+        groupTitle: typeof entry.groupTitle === "string" ? entry.groupTitle : undefined,
+        bundleCategorySlug: typeof entry.bundleCategorySlug === "string" ? entry.bundleCategorySlug : undefined,
+        bundleProductIds: Array.isArray(entry.bundleProductIds)
+            ? entry.bundleProductIds.filter((v) => typeof v === "string")
+            : undefined,
     }));
 }
+/*
+ * The row's headline, counted in LINES rather than item rows.
+ *
+ * 🛑 `extraCount` deliberately changed meaning when grouping landed: it was
+ * `items.length - 1`, so a bundle of four read "Dranzer S +3 more" — naming an
+ * arbitrary member and implying four separate things were bought. It is now
+ * `groups.length - 1`, so the same order reads "Starter Bundle +0" and a
+ * bundle plus one standalone reads "+1 more". The number a buyer or admin
+ * reads on a row should match the number of lines they will see inside it.
+ */
+/*
+ * `sourceContext` arrives as JsonValue off the raw admin-list doc, so it can
+ * be a string, an array or null — none of which `OrderProvenanceCard` can
+ * switch on. This narrows to "an object carrying a `path` tag" and nothing
+ * further: that is deliberately NOT the card's own switch duplicated, it only
+ * establishes that there is a tag to switch on, which is what makes the single
+ * cast true instead of hopeful. `as unknown as OrderSourceContext` would
+ * assert exactly the same thing while checking nothing (Root Cause #98).
+ */
+function toSourceContext(value) {
+    if (!value || typeof value !== "object" || Array.isArray(value))
+        return undefined;
+    /*
+     * Via `object` rather than straight off the JsonValue union, for two
+     * reasons that pull the same way. `Array.isArray` does not narrow
+     * `JsonArray | JsonObject` apart, so `value.path` does not resolve; and
+     * `OrderSourceContext` is not assignable to `JsonObject` (its optional
+     * members admit `undefined`, which is not a JsonValue), so TS refuses that
+     * conversion outright. Probing as `{ path?: string }` keeps both the probe
+     * and the assertion legal without an `unknown` anywhere.
+     */
+    const obj = value;
+    return typeof obj.path === "string"
+        ? obj
+        : undefined;
+}
 function firstOrderItem(item) {
-    const items = toOrderItemRows(item);
+    const groups = groupOrderItemsByLine(toOrderItemRows(item));
+    const first = groups[0];
+    const extraCount = Math.max(0, groups.length - 1);
+    if (!first)
+        return { extraCount: 0 };
+    if (first.kind === "single") {
+        return { image: first.item.image, title: first.item.title, extraCount };
+    }
     return {
-        image: items[0]?.image,
-        title: items[0]?.title,
-        extraCount: Math.max(0, items.length - 1),
+        image: first.items[0]?.item.image,
+        title: first.groupTitle ?? "Grouped purchase",
+        extraCount,
     };
 }
 /**
@@ -206,6 +263,30 @@ export function AdminOrdersView({ children, ...props }) {
                     couponCode: toStringValue(item.couponCode) || undefined,
                     couponDiscount: Number(item.couponDiscount ?? 0) || undefined,
                 },
+                // Raw doc in, so no serializer entry — but unlike `addons` above,
+                // which coerces every field by hand, this one needs a tag check
+                // before it can be trusted. See `toSourceContext`.
+                sourceContext: toSourceContext(item.sourceContext),
+                /*
+                 * D55. Coerced field by field like `addons` above, deliberately
+                 * NOT `as AdminOrderEditorViewProps["balance"]` — a cast at a
+                 * raw-document boundary is where Root Cause #98 lives, and main
+                 * had just removed the same cast from `sourceContext` one line up.
+                 *
+                 * `proofUrl` is omitted ON PURPOSE: this is the LIST payload, and
+                 * the screenshot is admin-detail only. A list row never needs it.
+                 */
+                balance: item.balance
+                    ? {
+                        amount: Number(item.balance.amount ?? 0),
+                        status: toStringValue(item.balance.status, "awaiting"),
+                        transactionId: toStringValue(item.balance.transactionId) ||
+                            undefined,
+                        reviewNote: toStringValue(item.balance.reviewNote) ||
+                            undefined,
+                    }
+                    : undefined,
+                currency: toStringValue(item.currency, "") || undefined,
             };
         }),
         getTotal: (response, mappedRows) => typeof response.meta?.total === "number" ? response.meta.total : mappedRows.length,
@@ -289,5 +370,5 @@ export function AdminOrdersView({ children, ...props }) {
             ] })),
         renderFilterPanel: ({ pendingFilters, setPendingFilters }) => (_jsxs(_Fragment, { children: [_jsx(FilterChipGroup, { label: "Status", tabs: ADMIN_ORDER_STATUS_TABS, value: pendingFilters.status ?? "", onChange: (id) => setPendingFilters((p) => ({ ...p, status: id, paymentReview: "" })) }), _jsx(FilterChipGroup, { label: "Manual payment", tabs: ADMIN_ORDER_PAYMENT_REVIEW_TABS, value: pendingFilters.paymentReview ?? "", onChange: (id) => setPendingFilters((p) => ({ ...p, paymentReview: id, status: "" })) })] })),
     };
-    return (_jsxs(_Fragment, { children: [_jsx(DataListingView, { config: config }), _jsx(AdminOrderEditorView, { open: drawerOpen, onClose: () => setDrawerOpen(false), orderId: selectedRow?.id, orderLabel: selectedRow?.primary, currentStatus: selectedRow?.status, items: selectedRow?.items, paymentProofUrl: selectedRow?.paymentProofUrl, paymentTransactionId: selectedRow?.paymentTransactionId, paymentMethod: selectedRow?.paymentMethod, paymentStatus: selectedRow?.paymentStatus, displayedUpiId: selectedRow?.displayedUpiId, buyerReportedUpiId: selectedRow?.buyerReportedUpiId, paymentUpiMismatch: selectedRow?.paymentUpiMismatch, buyerMarkedPaid: selectedRow?.buyerMarkedPaid, buyerFraudAgreementAccepted: selectedRow?.buyerFraudAgreementAccepted, paymentReviewOutcome: selectedRow?.paymentReviewOutcome, addons: selectedRow?.addons })] }));
+    return (_jsxs(_Fragment, { children: [_jsx(DataListingView, { config: config }), _jsx(AdminOrderEditorView, { open: drawerOpen, onClose: () => setDrawerOpen(false), orderId: selectedRow?.id, orderLabel: selectedRow?.primary, currentStatus: selectedRow?.status, items: selectedRow?.items, paymentProofUrl: selectedRow?.paymentProofUrl, paymentTransactionId: selectedRow?.paymentTransactionId, paymentMethod: selectedRow?.paymentMethod, paymentStatus: selectedRow?.paymentStatus, displayedUpiId: selectedRow?.displayedUpiId, buyerReportedUpiId: selectedRow?.buyerReportedUpiId, paymentUpiMismatch: selectedRow?.paymentUpiMismatch, buyerMarkedPaid: selectedRow?.buyerMarkedPaid, buyerFraudAgreementAccepted: selectedRow?.buyerFraudAgreementAccepted, paymentReviewOutcome: selectedRow?.paymentReviewOutcome, addons: selectedRow?.addons, sourceContext: selectedRow?.sourceContext, balance: selectedRow?.balance, currency: selectedRow?.currency })] }));
 }

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

@@ -236,6 +236,28 @@ export function OwnerQuickActions({ resource, editHref, onDelete, onClone, owner
      * 🛑 `inline-flex`, never `w-full`. In normal flow this must shrink to its
      * own content — a full-width bar would read as a section divider and would
      * stretch across a card body it is only a corner of.
+     *
+     * 🛑 `shrink-0` IS THE MOBILE FIX, and it is one word because the bug was
+     * one missing word.
+     *
+     * A flex item's default is `flex-shrink: 1`, so in the title/actions row
+     * this toolbar lives in — `<Row justify="between">` with an `<h1>` on the
+     * left — the heading and the toolbar compete for width, and at ~390px the
+     * toolbar LOSES. It compresses below its own content: the icons slide under
+     * each other, the padding collapses, and the 44px tap targets stop being
+     * 44px wide. That is the "works incorrectly on mobile" report.
+     *
+     * With `shrink-0` the toolbar keeps its intrinsic size and the HEADING gives
+     * way instead — which is right, because a heading wraps gracefully onto a
+     * second line and a row of hit targets does not.
+     *
+     * 🛑 Note what this is NOT: the desktop fix (`d94496c42`) gave the toolbar a
+     * container and added an audit whose two rules are about DOM ORDER — must
+     * come after the `<h1>`, must not share the heading's row unless that row is
+     * `justify="between"`. Both pass here. R2's exemption actively WHITELISTS
+     * the layout that breaks, so the surfaces that satisfy that audit are
+     * precisely the ones most likely to be wrong at phone width. A source-order
+     * rule cannot see a width problem.
      */
     const boxClass = onDark
         ? /*
@@ -244,12 +266,12 @@ export function OwnerQuickActions({ resource, editHref, onDelete, onClone, owner
            * LIGHT background in every light theme, so a themed box on a dark hero
            * photo is a white rectangle over the artwork.
            */
-            "inline-flex rounded-[var(--appkit-radius-lg)] bg-black/55 backdrop-blur-sm p-[var(--appkit-space-1)] shadow-md"
+            "inline-flex shrink-0 rounded-[var(--appkit-radius-lg)] bg-black/55 backdrop-blur-sm p-[var(--appkit-space-1)] shadow-md"
         : /*
            * Off media: the real surface tokens, so it inverts with the theme
            * instead of being a fixed near-white that disappears in dark mode.
            */
-            "inline-flex rounded-[var(--appkit-radius-lg)] bg-[var(--appkit-color-surface-elevated)] border border-[var(--appkit-color-border)] p-[var(--appkit-space-1)] shadow-sm";
+            "inline-flex shrink-0 rounded-[var(--appkit-radius-lg)] bg-[var(--appkit-color-surface-elevated)] border border-[var(--appkit-color-border)] p-[var(--appkit-space-1)] shadow-sm";
     return (_jsxs(_Fragment, { children: [_jsxs(Row, { gap: "xs", align: "center", className: `${boxClass} ${className}`.trim(), 
                 /*
                  * Stops the click reaching an ancestor `<Link>` or the card's own

package/dist/features/admin/constants/filter-tabs.d.ts

@@ -457,6 +457,9 @@ export declare const ADMIN_COUPON_TYPE_TABS: readonly [{
 }, {
     readonly id: "buy_x_get_y";
     readonly label: "Buy X Get Y";
+}, {
+    readonly id: "points_earn";
+    readonly label: "Points Back";
 }];
 /** Empty-sentinel variant for views (like SellerBidsView) that drive the
  *  filter URL with `""` instead of `"All"` so the `"" == no filter` invariant
@@ -757,6 +760,9 @@ export declare const ADMIN_PAYMENT_CASE_KIND_TABS: readonly [{
 }, {
     readonly id: "shipping_unpriced";
     readonly label: "Shipping unpriced";
+}, {
+    readonly id: "courier_unavailable";
+    readonly label: "Courier unavailable";
 }, {
     readonly id: "dead_job";
     readonly label: "Dead job";

package/dist/features/admin/constants/filter-tabs.js

@@ -245,6 +245,11 @@ export const ADMIN_COUPON_TYPE_TABS = [
     { id: "fixed", label: "Fixed" },
     { id: "free_shipping", label: "Free Shipping" },
     { id: "buy_x_get_y", label: "Buy X Get Y" },
+    // D51. `audit-filter-tab-enums`' COVERAGE rule fails on a real union value
+    // with no chip — which is what it was extended to catch after ten such gaps
+    // were found at once (Root Cause #72), including sellers being unable to
+    // filter their own `in_review` listings.
+    { id: "points_earn", label: "Points Back" },
 ];
 // ---------------------------------------------------------------------------
 // Seller-dashboard filter-chip tab sets
@@ -441,6 +446,7 @@ export const ADMIN_PAYMENT_CASE_KIND_TABS = [
     { id: "protection_claim", label: "Protection claim" },
     { id: "grievance", label: "Grievance" },
     { id: "shipping_unpriced", label: "Shipping unpriced" },
+    { id: "courier_unavailable", label: "Courier unavailable" },
     { id: "dead_job", label: "Dead job" },
     { id: "ledger_drift", label: "Points drift" },
 ];