@mohasinac/appkit 4.45.0 → 4.47.0

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

@@ -16,6 +16,10 @@ export const PRODUCT_INDEXED_FIELDS = [
     "storeId",
     "status",
     "category",
+    /* Indexed: `array-contains` on the ASSIGNED list is what the category
+     * delete cascade queries to find a product filed here as its SECOND
+     * category — `findBy("category", id)` sees only the default leaf. */
+    "categoryIds",
     "featured",
     "listingType",
     "isPromoted",
@@ -74,6 +78,9 @@ export const PRODUCT_PUBLIC_FIELDS = [
     "title",
     "description",
     "category",
+    /* Public: the chips on a detail page render the assignments. */
+    "categoryIds",
+    "categoryIdNames",
     "subcategory",
     "brand",
     "price",
@@ -92,6 +99,35 @@ export const PRODUCT_PUBLIC_FIELDS = [
     // Buyer-facing: drives the "Final Sale" card pill and the detail-page
     // Delivery & Returns line. Absent means final sale — read via isFinalSale().
     "finalSale",
+    // Buyer-facing by necessity: "lot of 12" is the single most important thing
+    // about a lot listing, and a buyer who cannot see it thinks they are bidding
+    // on one item.
+    "isLot",
+    "lotSize",
+    /* D13/D14 — buyer-facing by definition: a colourway is WHY someone buys that
+     * particular spare, and a unit label is how they tell two identical listings
+     * apart. `assembledFrom` is listed too — it names product ids that are
+     * already public listings, and "built from these parts" is provenance a
+     * buyer should see. */
+    "unitLabel",
+    "partGlossaryId",
+    "colourway",
+    "mouldVariant",
+    "assembledFrom",
+    /* D8 — public because "N people viewed this this week" is the honest social
+     * proof H2 is built on, and it is a figure we measured. 🛑 Deliberately NOT
+     * in PRODUCT_UPDATABLE_FIELDS: a seller who could write their own view count
+     * turns a measurement into a claim. */
+    "views7d",
+    "views30d",
+    /* 🛑 D13/D14. Listed here or an update STRIPS them: the write schema has no
+     * `.passthrough()`, so a field absent from this list is dropped before it
+     * reaches Firestore while the write still returns 200 (Root Cause #101). */
+    "unitLabel",
+    "partGlossaryId",
+    "colourway",
+    "mouldVariant",
+    "assembledFrom",
     "listingType",
     "auctionEndDate",
     "startingBid",
@@ -147,6 +183,12 @@ export const PRODUCT_UPDATABLE_FIELDS = [
     "title",
     "description",
     "category",
+    /* I1 — the ASSIGNED list. 🛑 Named here or an update STRIPS it while still
+     * returning 200 (Root Cause #101), which is exactly how a UI-created
+     * product lost its chain. `categorySlugs`/`categoryNames` stay OUT: they
+     * are derived, and a caller that could set them could desync the closure
+     * from the assignments it is supposed to be the closure OF. */
+    "categoryIds",
     "subcategory",
     "brand",
     "price",
@@ -159,6 +201,13 @@ export const PRODUCT_UPDATABLE_FIELDS = [
     "shippingInfo",
     "returnPolicy",
     "finalSale",
+    /* 🛑 Listed here or an update STRIPS them. The write schema has no
+     * `.passthrough()`, so a field absent from this list is dropped before it
+     * reaches Firestore while the write still returns 200 — exactly how a
+     * UI-created product lost its `categorySlugs` chain (Root Cause #101) and how
+     * `compareAtPrice` is silently discarded today (F1 #25). */
+    "isLot",
+    "lotSize",
     "pickupAddressId",
     "condition",
     "insurance",

package/dist/features/search/components/SearchResultsPageView.d.ts

@@ -0,0 +1,33 @@
+/**
+ * D2 Fix 4 — the cross-resource search RESULTS page.
+ *
+ * Replaces a pure redirect. Before this, `/search` mapped each of 15
+ * `SearchResourceType` values to a browse route and `redirect()`ed, so a buyer
+ * searching "dranzer" could not see that we have a product, a category, a blog
+ * post and an FAQ about it — the dropdown only chose which single listing page
+ * to send them to.
+ *
+ * Measured on production: "burst" returns 21 hits across SIX groups
+ * (listings, categories, brands, stores, articles, FAQs); "zzzznope" and "!!!"
+ * return none.
+ *
+ * 🛑 STILL `noindex`, and that is deliberate rather than inherited. A search
+ * results page is the classic crawl trap: one URL per query string, thin
+ * content, and no canonical answer to index. The redirect it replaces was
+ * `noindex` too, so this changes the page's usefulness to a HUMAN without
+ * changing its relationship to a crawler. The `moreHref` links go to the real
+ * indexable listing pages.
+ *
+ * 🛑 NO PRICES ANYWHERE. Prices are hidden from signed-out visitors and this
+ * is the one surface where gating every row would be most of the markup.
+ * A title plus a link is honest, needs no `<GatedPrice>`, and cannot leak —
+ * there is nothing on the row to leak.
+ */
+import React from "react";
+import type { SearchEverythingResult } from "../../../_internal/server/features/search/search-everything";
+export interface SearchResultsPageViewProps {
+    result: SearchEverythingResult;
+    /** Rendered above the groups — the consumer owns the input control. */
+    renderSearchInput?: () => React.ReactNode;
+}
+export declare function SearchResultsPageView({ result, renderSearchInput, }: SearchResultsPageViewProps): React.JSX.Element;

package/dist/features/search/components/SearchResultsPageView.js

@@ -0,0 +1,21 @@
+import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
+import { Section, Container, Stack, Row, Div, Heading, Text, Span, TextLink } from "../../../ui/index.js";
+export function SearchResultsPageView({ result, renderSearchInput, }) {
+    const { q, terms, groups, totalHits } = result;
+    const hasQuery = q.trim().length > 0;
+    /*
+     * 🛑 THREE states, not two, and the middle one is the honest part.
+     *
+     *   no query          — prompt, no claim about results
+     *   query, 0 terms    — the query normalised away ("!!!", "—"). Saying
+     *                       "no results for !!!" implies we looked; we could
+     *                       not. A2's bug was the inverse of this: a
+     *                       punctuation-only query returned the ENTIRE
+     *                       published catalogue as search results.
+     *   query, 0 hits     — genuinely nothing matched
+     */
+    const normalisedAway = hasQuery && terms.length === 0;
+    return (_jsx(Section, { padding: "page", children: _jsx(Container, { children: _jsxs(Stack, { gap: "loose", children: [_jsxs(Stack, { gap: "xs", children: [_jsx(Heading, { level: 1, children: hasQuery ? `Search: ${q}` : "Search" }), hasQuery && !normalisedAway && (_jsx(Text, { color: "muted", size: "sm", children: totalHits === 0
+                                    ? "Nothing matched."
+                                    : `${totalHits}${groups.some((g) => g.truncated) ? "+" : ""} result${totalHits === 1 ? "" : "s"} across ${groups.length} section${groups.length === 1 ? "" : "s"}.` }))] }), renderSearchInput?.(), !hasQuery && (_jsx(Text, { color: "muted", children: "Search listings, categories, brands, stores, articles and help pages at once." })), normalisedAway && (_jsx(Text, { color: "muted", children: "That search had no searchable words in it. Try a product, part or category name." })), groups.map((group) => (_jsxs(Stack, { gap: "sm", children: [_jsxs(Row, { justify: "between", align: "center", wrap: true, gap: "sm", children: [_jsxs(Heading, { level: 2, size: "lg", children: [group.label, group.truncated && (_jsxs(Span, { size: "sm", color: "muted", children: [" ", "(", group.hits.length, "+)"] }))] }), _jsxs(TextLink, { href: group.moreHref, children: ["See all ", group.label.toLowerCase(), " \u2192"] })] }), _jsx(Stack, { gap: "xs", children: group.hits.map((hit) => (_jsxs(Div, { surface: "card", padding: "sm", rounded: "md", children: [_jsx(TextLink, { href: hit.href, children: hit.title }), hit.detail && (_jsx(Text, { size: "xs", color: "muted", children: hit.detail }))] }, `${group.key}-${hit.id}`))) })] }, group.key))), hasQuery && !normalisedAway && totalHits === 0 && (_jsxs(Stack, { gap: "xs", children: [_jsxs(Text, { color: "muted", children: ["No listings, categories, brands, stores, articles or help pages matched \u201C", q, "\u201D."] }), _jsx(TextLink, { href: "/products", children: "Browse all listings \u2192" })] }))] }) }) }));
+}

package/dist/features/seller/components/CategoryInlineSelect.d.ts

@@ -6,5 +6,21 @@ export interface CategoryInlineSelectProps {
     disabled?: boolean;
     /** Show the "+ Create new category" option. Defaults to true — sellers can create categories inline. */
     allowCreate?: boolean;
+    /**
+     * I2 — the ADDITIONAL assignments, beyond `value`.
+     *
+     * 🛑 Omitting this pair keeps the component exactly single-select, which is
+     * what every existing call site gets. Multi-assignment is opt-in per surface
+     * rather than switched on globally, because `value`/`onChange` is a `string`
+     * contract that five call sites rely on and changing it under them is Root
+     * Cause #20 (a public prop change without its consumers in the same commit).
+     */
+    extraValues?: string[];
+    onExtraValuesChange?: (ids: string[]) => void;
+    /**
+     * How many assignments are allowed in total, `value` included. Defaults to
+     * `MAX_CATEGORY_ASSIGNMENTS`, which is what the repository refuses above.
+     */
+    maxAssignments?: number;
 }
-export declare function CategoryInlineSelect({ value, onChange, placeholder, disabled, allowCreate, }: CategoryInlineSelectProps): React.JSX.Element;
+export declare function CategoryInlineSelect({ value, onChange, placeholder, disabled, allowCreate, extraValues, onExtraValuesChange, maxAssignments, }: CategoryInlineSelectProps): React.JSX.Element;

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

@@ -1,9 +1,12 @@
 "use client";
-import { jsx as _jsx } from "react/jsx-runtime";
+import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { PaginatedSelect } from "../../../ui/components/PaginatedSelect.js";
 import { CategoryQuickCreateForm } from "../../admin/components/CategoryQuickCreateForm.js";
 import { apiClient } from "../../../http/index.js";
 import { ADMIN_ENDPOINTS, CATEGORY_ENDPOINTS } from "../../../constants/api-endpoints.js";
+import { Stack } from "../../../ui/components/Layout.js";
+import { Text } from "../../../ui/components/Typography.js";
+import { MAX_CATEGORY_ASSIGNMENTS } from "../../products/repository/category-assignment.js";
 // Admin: paginated + server-side search via /api/admin/categories (admin/mod only).
 // Response via successResponse({ data: items, total }) → apiClient unwraps to { data, total, ... }.
 async function loadAdminCategoryOptions(query, page) {
@@ -35,9 +38,17 @@ async function loadPublicCategoryOptions(query, _page) {
     const items = q ? all.filter((c) => c.label.toLowerCase().includes(q)) : all;
     return { items, hasMore: false, nextPage: 2 };
 }
-export function CategoryInlineSelect({ value, onChange, placeholder = "Search categories…", disabled, allowCreate = true, }) {
+export function CategoryInlineSelect({ value, onChange, placeholder = "Search categories…", disabled, allowCreate = true, extraValues, onExtraValuesChange, maxAssignments = MAX_CATEGORY_ASSIGNMENTS, }) {
     const loadOptions = allowCreate ? loadAdminCategoryOptions : loadPublicCategoryOptions;
-    return (_jsx(PaginatedSelect, { value: value || null, onChange: (v) => onChange(v ?? ""), loadOptions: loadOptions, placeholder: placeholder, disabled: disabled, createLabel: "category", drawerTitle: "Create Category", renderCreateForm: allowCreate
+    const primary = (_jsx(PaginatedSelect, { value: value || null, onChange: (v) => onChange(v ?? ""), loadOptions: loadOptions, placeholder: placeholder, disabled: disabled, createLabel: "category", drawerTitle: "Create Category", renderCreateForm: allowCreate
             ? ({ onCreated, onCancel }) => (_jsx(CategoryQuickCreateForm, { onSaved: (id, name) => onCreated({ value: id, label: name }), onCancel: onCancel }))
             : undefined }));
+    // Single-select unless the caller opted in — byte-identical to before.
+    if (!onExtraValuesChange)
+        return primary;
+    const extras = extraValues ?? [];
+    const remaining = Math.max(0, maxAssignments - 1 - extras.length);
+    return (_jsxs(Stack, { gap: "sm", children: [primary, _jsx(PaginatedSelect, { multiple: true, value: extras, onChange: (v) => onExtraValuesChange(Array.isArray(v) ? v : []), loadOptions: loadOptions, placeholder: remaining > 0
+                    ? `Also list under… (up to ${remaining} more)`
+                    : "No more categories can be added", disabled: disabled || remaining <= 0 }), _jsx(Text, { size: "xs", color: "muted", children: "Optional. Add a category from a DIFFERENT tree \u2014 a franchise, for example. Two categories from the same tree will be refused, because the broader one is already implied by the narrower." })] }));
 }

package/dist/features/seller/components/SellerProductShell.d.ts

@@ -6,6 +6,16 @@ export interface SellerProductDraft {
     slug?: string;
     description?: string;
     category?: string;
+    /**
+     * I2 — the assignment list, DEFAULT FIRST, with `category` mirroring `[0]`.
+     *
+     * 🛑 Declared on the TYPE as well as the Zod schema. The schema is
+     * `.passthrough()`, so an undeclared key survives at runtime — and stays
+     * untyped, so the picker could not read `values.categoryIds` at all. Both
+     * halves are needed, which is the same trap as a field missing from
+     * `PRODUCT_UPDATABLE_FIELDS` (Root Cause #101) one layer out.
+     */
+    categoryIds?: string[];
     brand?: string;
     condition?: string;
     tags?: string[];
@@ -134,10 +144,25 @@ export interface SellerProductShellProps {
     onDiscard?: () => void;
     isLoading?: boolean;
     storeSlug?: string;
-    /** Render a custom category selector (e.g. PaginatedSelect with createLabel). */
+    /**
+     * Render a custom category selector (e.g. PaginatedSelect with createLabel).
+     *
+     * I2 — `extraValues`/`onExtraValuesChange` carry the ADDITIONAL assignments.
+     * A selector that ignores them stays exactly single-select, which is what
+     * every surface except this one does.
+     */
     renderCategorySelector?: (props: {
         value: string;
         onChange: (v: string) => void;
+        /**
+         * 🛑 OPTIONAL, and that is the whole point. `QuickProductForm` is a second
+         * consumer of this slot and making these required broke it immediately —
+         * Root Cause #20, a public prop change without its consumers, which the
+         * comment above literally warns about. A selector that ignores them stays
+         * exactly single-select.
+         */
+        extraValues?: string[];
+        onExtraValuesChange?: (ids: string[]) => void;
     }) => React.ReactNode;
     /** Render a custom brand selector. */
     renderBrandSelector?: (props: {

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

@@ -46,6 +46,15 @@ const sellerProductSchema = z.object({
     description: z.string().max(10000).optional().or(z.literal("")),
     price: z.number().min(0).optional(),
     category: z.string().optional(),
+    /**
+     * I2 — the assignment list, DEFAULT FIRST. `category` mirrors `[0]`.
+     *
+     * Declared even though this schema is `.passthrough()`, because the draft's
+     * TYPE is what the render site reads `values.categoryIds` off — passthrough
+     * keeps an undeclared key at runtime and still leaves it untyped, so the
+     * picker could not see it.
+     */
+    categoryIds: z.array(z.string()).max(4).optional(),
     brand: z.string().optional(),
     isActive: z.boolean().optional(),
     images: z
@@ -106,7 +115,27 @@ function StepBasic({ values, onChange, renderCategorySelector, renderBrandSelect
         condition: CONDITION_OPTIONS.find((o) => o.value === (values.condition ?? "new"))?.label ?? "",
         siteName: "LetItRip",
     };
-    return (_jsxs(Stack, { gap: "md", children: [_jsx(FormField, { name: "title", label: "Title", type: "text", value: values.title ?? "", onChange: (v) => onChange({ title: v }), placeholder: "e.g. Charizard Base Set PSA 9" }), _jsxs(FormGroup, { columns: 2, children: [renderCategorySelector ? (_jsxs(_Fragment, { children: [_jsx(Text, { className: "text-[var(--appkit-color-text)] mb-1", size: "sm", weight: "medium", children: "Category" }), renderCategorySelector({ value: values.category ?? "", onChange: (v) => onChange({ category: v }) })] })) : (_jsx(FormField, { name: "category", label: "Category", type: "text", value: values.category ?? "", onChange: (v) => onChange({ category: v }), placeholder: "e.g. Trading Cards" })), _jsx(FormField, { name: "condition", label: "Condition", type: "select", value: values.condition ?? "new", onChange: (v) => onChange({ condition: v }), options: CONDITION_OPTIONS })] }), renderTemplateSelector?.({
+    return (_jsxs(Stack, { gap: "md", children: [_jsx(FormField, { name: "title", label: "Title", type: "text", value: values.title ?? "", onChange: (v) => onChange({ title: v }), placeholder: "e.g. Charizard Base Set PSA 9" }), _jsxs(FormGroup, { columns: 2, children: [renderCategorySelector ? (_jsxs(_Fragment, { children: [_jsx(Text, { className: "text-[var(--appkit-color-text)] mb-1", size: "sm", weight: "medium", children: "Category" }), renderCategorySelector({
+                                value: values.category ?? "",
+                                onChange: (v) => onChange({ category: v }),
+                                /*
+                                 * 🛑 `categoryIds[0]` is the DEFAULT assignment and is already
+                                 * rendered by the primary picker above, so the extras list is
+                                 * everything AFTER it. Passing the whole array would show the
+                                 * primary twice and let a seller "remove" it from a control that
+                                 * does not own it.
+                                 */
+                                extraValues: (values.categoryIds ?? []).slice(1),
+                                onExtraValuesChange: (ids) => onChange({
+                                    /*
+                                     * Rebuilt primary-first on every change, because the server
+                                     * derives the default by DEPTH and reorders the stored array
+                                     * anyway — sending it in a different order would make the
+                                     * form and the saved document disagree until the next load.
+                                     */
+                                    categoryIds: [values.category ?? "", ...ids].filter(Boolean),
+                                }),
+                            })] })) : (_jsx(FormField, { name: "category", label: "Category", type: "text", value: values.category ?? "", onChange: (v) => onChange({ category: v }), placeholder: "e.g. Trading Cards" })), _jsx(FormField, { name: "condition", label: "Condition", type: "select", value: values.condition ?? "new", onChange: (v) => onChange({ condition: v }), options: CONDITION_OPTIONS })] }), renderTemplateSelector?.({
                 categoryId: values.category ?? "",
                 condition: values.condition ?? "new",
                 context: templateContext,

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

@@ -2,6 +2,7 @@
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { formatPhysicalLocation } from "../../../utils/physical-location.js";
 import { normalizeError } from "../../../errors/normalize.js";
+import { trackError, ErrorCategory, ErrorSeverity, } from "../../../monitoring/error-tracking.js";
 import { toUserMessage } from "../../../errors/error-display-map.js";
 import { SIEVE_OP, Stack, sieveFilter } from "@mohasinac/appkit/client";
 import { LOW_STOCK_THRESHOLD } from "../../store-extensions/constants/alert-metrics.js";
@@ -9,7 +10,7 @@ import { sortBy } from "@mohasinac/appkit/client";
 import { useState, useCallback } from "react";
 import { useEntityDelete } from "../../../react/hooks/useEntityDelete.js";
 import { useActionDispatch } from "../../../react/hooks/use-action-dispatch.js";
-import { Eye, EyeOff, Pencil, Trash2, Printer, MapPin, Boxes } from "lucide-react";
+import { Eye, EyeOff, Pencil, Trash2, Printer, MapPin, Boxes, Wrench, Scissors } from "lucide-react";
 import { PhysicalLocationModal } from "./PhysicalLocationModal.js";
 import { BulkStockModal } from "./BulkStockModal.js";
 import { useUrlTable } from "../../../react/hooks/useUrlTable.js";
@@ -30,6 +31,7 @@ import { SellerProductsCards } from "./SellerProductsCards.js";
 import { SellerProductsFilterFields } from "./SellerProductsFilterDrawer.js";
 import { listingBadgeVariant } from "../../products/utils/listing-badge-variant.js";
 import { DataListingView } from "../../admin/components/DataListingView.js";
+import { AssemblyDrawer, } from "../../parts/components/AssemblyDrawer.js";
 import { ACTIONS } from "../../../_internal/shared/actions/action-registry.js";
 import { buildBulkAction } from "../../../_internal/shared/actions/bulk-helpers.js";
 const FILTER_KEYS = [
@@ -148,6 +150,29 @@ const PRODUCT_COLUMNS = [
      * Both stock fields have been `canFilter`/`canSort` at the repository layer
      * the whole time; only the view omitted them.
      */
+    /*
+     * D8 — views, which were collected on ~30 surfaces and joined to NOTHING.
+     * A seller could not see how many people looked at their listing: the metric
+     * every competitor platform leads with.
+     *
+     * 🛑 Three states, and the middle one is the whole reason this is a column
+     * rather than a number in a subtitle. `undefined` means NOT YET ROLLED UP —
+     * the nightly pass has not run — and renders an em-dash; `0` means measured
+     * and nobody looked, and renders `0`. Collapsing them with `?? 0` is exactly
+     * what made the admin Top Products table show "0 views" on every row as
+     * though it were a measurement (A5).
+     */
+    {
+        key: "views",
+        header: "7d views",
+        className: "w-24 text-right",
+        render: (row) => {
+            if (typeof row.views7d !== "number") {
+                return _jsx(Span, { size: "xs", color: "faint", children: "\u2014" });
+            }
+            return (_jsx(Span, { size: "xs", color: row.views7d > 0 ? "primary" : "muted", numeric: true, children: row.views7d.toLocaleString() }));
+        },
+    },
     {
         key: "stock",
         header: "Stock",
@@ -227,6 +252,94 @@ export function SellerProductsView({ onDeleteProduct, onCreateClick, }) {
     const [setLocationOpen, setSetLocationOpen] = useState(false);
     const [selectedIdsForLocation, setSelectedIdsForLocation] = useState([]);
     const [setStockOpen, setSetStockOpen] = useState(false);
+    /*
+     * ── D13: assembly ──────────────────────────────────────────────────────
+     *
+     * 🛑 The drawer's data is fetched when it OPENS, not with the page. 50 part
+     * listings and a composition per row would be a read nobody asked for on a
+     * view that renders 50 rows — and the drawer is used on a handful of them.
+     */
+    const [assembleFor, setAssembleFor] = useState(null);
+    const [assembleData, setAssembleData] = useState(null);
+    const [assembleLoading, setAssembleLoading] = useState(false);
+    const openAssembly = useCallback(async (row, direction) => {
+        setAssembleFor({ id: row.id, title: row.primary ?? row.id, direction });
+        setAssembleData(null);
+        setAssembleLoading(true);
+        try {
+            const res = await fetch(SELLER_ENDPOINTS.PRODUCT_ASSEMBLE(row.id));
+            const body = (await res.json().catch(() => null));
+            if (!res.ok)
+                throw new Error(body?.error ?? "Could not load your parts");
+            setAssembleData({
+                compositions: body?.data?.compositions ?? [],
+                options: body?.data?.options ?? [],
+                targetAvailable: body?.data?.targetAvailable ?? 0,
+            });
+        }
+        catch (err) {
+            const e = normalizeError(err);
+            /*
+             * 🛑 Closed and reported, never left open and empty. An empty picker
+             * with no explanation is the state in which a seller concludes they
+             * hold no parts — which may be false.
+             */
+            setAssembleFor(null);
+            /*
+             * 🛑 AUTHORED copy, not `e.message`. Unlike the 409 (which carries
+             * `{needs, has}` and composes a sentence around the real figures),
+             * a load failure has no number worth surfacing — so this is one honest
+             * line, and the raw text goes to the recorder where it belongs.
+             */
+            showToast("Could not load your parts. Please try again.", "error");
+            trackError(e.message, ErrorCategory.API, ErrorSeverity.MEDIUM, {
+                component: "SellerProductsView",
+                action: "openAssembly",
+            });
+        }
+        finally {
+            setAssembleLoading(false);
+        }
+    }, [showToast]);
+    const submitAssembly = useCallback(async (input) => {
+        const id = assembleFor?.id;
+        if (!id)
+            throw new Error("No listing selected");
+        const res = await fetch(SELLER_ENDPOINTS.PRODUCT_ASSEMBLE(id), {
+            method: "POST",
+            headers: { "Content-Type": "application/json" },
+            body: JSON.stringify(input),
+        });
+        const body = (await res.json().catch(() => null));
+        /*
+         * 🛑 THROWS with the server's own message, which the drawer renders. A
+         * 409 reads "Not enough available: needs 6, has 5" — the one sentence
+         * that tells the seller what to do. Swallowing it into a generic failure
+         * would turn "restock" into "retry".
+         */
+        if (!res.ok) {
+            /*
+             * 🛑 Carries the structured DETAIL to the drawer and never a rendered
+             * sentence. The drawer composes its own copy around `{needs, has}`;
+             * `message` here is for the recorder (Root Cause #86 —
+             * `audit-raw-error-text` blocks showing a thrown `.message`).
+             */
+            const err = new Error(body?.error ?? "assembly failed");
+            /*
+             * Typed rather than `unknown`: the 409's shape is OURS — the planner
+             * throws `{ kind, productId, needs, has }` — so declaring it is what
+             * lets the drawer compose a sentence around real figures instead of
+             * casting at the boundary (Root Cause #57/#70: a cast at a boundary is
+             * where this class of bug lives).
+             */
+            err.detail = { code: body?.code, ...(body?.details ?? {}) };
+            throw err;
+        }
+        return {
+            unitsBuilt: body?.data?.unitsBuilt ?? input.units,
+            consumed: body?.data?.consumed ?? [],
+        };
+    }, [assembleFor?.id]);
     const [selectedIdsForStock, setSelectedIdsForStock] = useState([]);
     const [stockBusy, setStockBusy] = useState(false);
     /*
@@ -472,6 +585,9 @@ export function SellerProductsView({ onDeleteProduct, onCreateClick, }) {
                     price: priceRaw ? `₹${priceRaw.toLocaleString("en-IN")}` : "—",
                     stock: stockLabel || "—",
                     stockValue,
+                    // Carried through ONLY when the document really has it — see the
+                    // field's own note on undefined-vs-zero.
+                    ...(typeof item.views7d === "number" ? { views7d: item.views7d } : {}),
                     physicalLocation: item.physicalLocation &&
                         typeof item.physicalLocation.zone === "string"
                         ? item.physicalLocation
@@ -524,8 +640,18 @@ export function SellerProductsView({ onDeleteProduct, onCreateClick, }) {
                 : `/products/${row.id}`,
         renderRowActions: (row) => {
             const isPublished = (statusOverrides.get(row.id) ?? row.status) === "published";
-            return (_jsxs(Row, { gap: "xs", children: [_jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); handleEdit(row); }, "aria-label": ACTIONS.STORE["edit-listing"].ariaLabel, children: _jsx(Pencil, { className: "w-4 h-4" }) }), _jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); void handleTogglePublish(row); }, "aria-label": isPublished ? "Unpublish" : "Publish", title: isPublished ? "Unpublish" : "Publish", disabled: publishingId === row.id, children: isPublished ? _jsx(EyeOff, { className: "w-4 h-4" }) : _jsx(Eye, { className: "w-4 h-4" }) }), _jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); void handleDuplicate(row); }, "aria-label": "Duplicate listing", title: "Duplicate", children: "\u29C9" }), onDeleteProduct && (_jsx(Button, { variant: "ghost", size: "sm", action: ACTIONS.STORE["delete-listing"], onClick: (e) => { e.stopPropagation(); void handleDelete(row); }, disabled: deletingId === row.id, className: "text-[var(--appkit-color-error)] hover:bg-[var(--appkit-color-border-subtle)]", children: _jsx(Trash2, { className: "w-4 h-4" }) }))] }));
+            return (_jsxs(Row, { gap: "xs", children: [_jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); handleEdit(row); }, "aria-label": ACTIONS.STORE["edit-listing"].ariaLabel, children: _jsx(Pencil, { className: "w-4 h-4" }) }), _jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); void openAssembly(row, "assemble"); }, "aria-label": ACTIONS.STORE["assemble-from-parts"].ariaLabel, title: ACTIONS.STORE["assemble-from-parts"].label, disabled: assembleLoading, children: _jsx(Wrench, { className: "w-4 h-4" }) }), _jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); void openAssembly(row, "part-out"); }, "aria-label": ACTIONS.STORE["part-out-listing"].ariaLabel, title: ACTIONS.STORE["part-out-listing"].label, disabled: assembleLoading, children: _jsx(Scissors, { className: "w-4 h-4" }) }), _jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); void handleTogglePublish(row); }, "aria-label": isPublished ? "Unpublish" : "Publish", title: isPublished ? "Unpublish" : "Publish", disabled: publishingId === row.id, children: isPublished ? _jsx(EyeOff, { className: "w-4 h-4" }) : _jsx(Eye, { className: "w-4 h-4" }) }), _jsx(Button, { variant: "ghost", size: "sm", onClick: (e) => { e.stopPropagation(); void handleDuplicate(row); }, "aria-label": "Duplicate listing", title: "Duplicate", children: "\u29C9" }), onDeleteProduct && (_jsx(Button, { variant: "ghost", size: "sm", action: ACTIONS.STORE["delete-listing"], onClick: (e) => { e.stopPropagation(); void handleDelete(row); }, disabled: deletingId === row.id, className: "text-[var(--appkit-color-error)] hover:bg-[var(--appkit-color-border-subtle)]", children: _jsx(Trash2, { className: "w-4 h-4" }) }))] }));
         },
     };
-    return (_jsxs(_Fragment, { children: [_jsx(DataListingView, { config: config }), setLocationOpen && (_jsx(PhysicalLocationModal, { count: selectedIdsForLocation.length, onSave: (loc) => handleSetLocation(loc, selectedIdsForLocation), onClose: () => setSetLocationOpen(false) })), setStockOpen && (_jsx(BulkStockModal, { count: selectedIdsForStock.length, busy: stockBusy, onSave: (qty) => handleSetStock(qty, selectedIdsForStock), onClose: () => setSetStockOpen(false) }))] }));
+    return (_jsxs(_Fragment, { children: [_jsx(DataListingView, { config: config }), setLocationOpen && (_jsx(PhysicalLocationModal, { count: selectedIdsForLocation.length, onSave: (loc) => handleSetLocation(loc, selectedIdsForLocation), onClose: () => setSetLocationOpen(false) })), setStockOpen && (_jsx(BulkStockModal, { count: selectedIdsForStock.length, busy: stockBusy, onSave: (qty) => handleSetStock(qty, selectedIdsForStock), onClose: () => setSetStockOpen(false) })), assembleFor && assembleData && (_jsx(AssemblyDrawer, { isOpen: true, 
+                /*
+                 * 🛑 No refetch on close, and that is a stated limitation rather than
+                 * a silent one: `DataListingView` owns the query and exposes no
+                 * refetch handle to a consumer. The drawer reports the real
+                 * before/after per component, so the seller SEES what moved — but the
+                 * row behind it still shows the pre-build figure until the next
+                 * navigation. Wiring a refetch means giving `ListingViewConfig` a
+                 * handle, which is a shared-surface change and its own commit.
+                 */
+                onClose: () => { setAssembleFor(null); setAssembleData(null); }, direction: assembleFor.direction, targetTitle: assembleFor.title, targetAvailable: assembleData.targetAvailable, compositions: assembleData.compositions, options: assembleData.options, onSubmit: submitAssembly }))] }));
 }

package/dist/features/tester/seed-data/authored/admin__site-system.js

@@ -424,21 +424,6 @@ export const authored = {
         expectedUiState: "The page renders logs, or a readable message naming what is unavailable. It does not show a stack trace, and it does not take the rest of the admin panel with it.",
         endResult: "Read-only; nothing persists.",
     },
-    "checklist-admin-site-system-copilot-admin": {
-        roles: ["admin"],
-        startPage: "/admin",
-        steps: [
-            "Sign in as admin@letitrip.in / TempPass123!.",
-            "Open /admin/copilot and read what it offers.",
-            "Read whether it states which model or provider it uses.",
-            "Submit a simple request and read the response.",
-            "Read whether any error surfaces a provider key or a raw response body.",
-            "Read whether the feature declines cleanly when unconfigured.",
-        ],
-        expectedBehaviour: "The feature works or declines cleanly, and never surfaces a provider credential or a raw upstream error to the browser. Every 5xx message is scrubbed before it reaches a client, with the real one kept server-side for the error recorder.",
-        expectedUiState: "A response renders, or a readable message names what is missing. No provider key, no raw upstream body, no stack trace.",
-        endResult: "Read-only; nothing persists.",
-    },
     "checklist-admin-site-system-team-admin": {
         roles: ["admin"],
         startPage: "/admin/team",

package/dist/features/tester/seed-data/authored/content-discovery__parts-database.d.ts

@@ -0,0 +1,2 @@
+import type { AuthoredCase } from "./_types";
+export declare const authored: Record<string, AuthoredCase>;