@mohasinac/appkit 4.30.19 → 4.31.0

package/dist/features/shell/field-ui-meta.js

@@ -300,15 +300,22 @@ export function inferFieldUiMeta(name, schema) {
         meta.derived = true;
         meta.tier = meta.tier ?? "t2-server";
     }
-    // >5 options is the standing PaginatedSelect threshold. Recorded as `help`
-    // rather than a separate flag so the renderer can decide without a second
-    // source of truth about what "many options" means.
-    const options = enumOptions(inner);
-    if (meta.kind === "select" && options != null && options.length > 5) {
-        meta.help = meta.help ?? "paginated";
-    }
     return meta;
 }
+/**
+ * Above this many options a select becomes a searchable `<PaginatedSelect>`.
+ * The standing UI rule (CLAUDE.md § UI Primitive Rules): more than five options
+ * and a plain dropdown is unscannable.
+ *
+ * 🛑 This is the WHOLE mechanism — there is deliberately no `paginated` flag on
+ * `FieldUiMeta`. It used to be smuggled through `help` as the literal string
+ * `"paginated"`, which broke in both directions: the assignment was
+ * `meta.help ?? "paginated"`, so a select that had real help text never became
+ * paginated; and the renderer had to blank `help` to avoid printing the
+ * sentinel, so one that DID become paginated could never show help. Counting
+ * the options at render time needs neither.
+ */
+export const PAGINATED_SELECT_THRESHOLD = 5;
 /**
  * The field's final metadata: inference first, explicit registration on top.
  *

package/dist/features/shell/form-sections.d.ts

@@ -0,0 +1,46 @@
+/**
+ * The canonical section ORDER vocabulary.
+ *
+ * ## Why this is a `group`, not a rename of `section`
+ *
+ * The plan called for collapsing the 58 free-form section ids in use across 30
+ * schemas onto a closed 12-value set. Measured before doing it, that merges
+ * sections in 7 files and 8 places — the scam report's `who` + `what` +
+ * `declare` become one blob, coupons' `limits` + `validity` collapse together,
+ * blog's `content` + `seo` merge, and payout methods lose the split between
+ * `method` and `bank`.
+ *
+ * The mistake was treating one identifier as two things. A section id is the
+ * RENDERED UNIT — fine-grained, form-specific, and the thing whose heading the
+ * user reads (`build-sections` falls back to `humaniseFieldName(sectionId)`, so
+ * 144 of the 228 annotations get their visible heading straight from the id).
+ * What the ordering rule actually needs is a PRIORITY BAND: required first,
+ * danger last, everything else in a predictable middle.
+ *
+ * So the id stays free-form and every section declares a `group` from this
+ * closed union. Order and default-open derive from the group; identity and
+ * heading stay with the id. Nothing merges, and the audit still has a closed
+ * set to check against.
+ *
+ * ## Ordering
+ *
+ * Array order IS render order. `required` is index 0 and is the section left
+ * open by default; `danger` is always last. That gives "1. Required → 2.
+ * Additional → … → least important last" mechanically, with no per-form
+ * `sectionOrder` to maintain.
+ */
+export declare const FORM_SECTION_GROUP: readonly ["required", "identity", "classification", "pricing", "inventory", "media", "logistics", "relationships", "schedule", "visibility", "advanced", "danger"];
+export type FormSectionGroup = (typeof FORM_SECTION_GROUP)[number];
+/** Position of a group in render order; unknown groups sort just before danger. */
+export declare function groupRank(group: FormSectionGroup | undefined): number;
+/**
+ * Default group for each section id currently in use, so 30 schemas do not each
+ * need a hand-written `group`. A section id absent here must declare its own —
+ * `audit-form-sectionised` fails otherwise, which is what stops a new id
+ * silently defaulting into the middle of the form.
+ *
+ * Derived from the 58 ids actually present, counted rather than guessed.
+ */
+export declare const SECTION_GROUP_DEFAULTS: Record<string, FormSectionGroup>;
+/** The group for a section id — its declared group, else the default, else advanced. */
+export declare function resolveSectionGroup(sectionId: string, declared?: FormSectionGroup): FormSectionGroup;

package/dist/features/shell/form-sections.js

@@ -0,0 +1,114 @@
+/**
+ * The canonical section ORDER vocabulary.
+ *
+ * ## Why this is a `group`, not a rename of `section`
+ *
+ * The plan called for collapsing the 58 free-form section ids in use across 30
+ * schemas onto a closed 12-value set. Measured before doing it, that merges
+ * sections in 7 files and 8 places — the scam report's `who` + `what` +
+ * `declare` become one blob, coupons' `limits` + `validity` collapse together,
+ * blog's `content` + `seo` merge, and payout methods lose the split between
+ * `method` and `bank`.
+ *
+ * The mistake was treating one identifier as two things. A section id is the
+ * RENDERED UNIT — fine-grained, form-specific, and the thing whose heading the
+ * user reads (`build-sections` falls back to `humaniseFieldName(sectionId)`, so
+ * 144 of the 228 annotations get their visible heading straight from the id).
+ * What the ordering rule actually needs is a PRIORITY BAND: required first,
+ * danger last, everything else in a predictable middle.
+ *
+ * So the id stays free-form and every section declares a `group` from this
+ * closed union. Order and default-open derive from the group; identity and
+ * heading stay with the id. Nothing merges, and the audit still has a closed
+ * set to check against.
+ *
+ * ## Ordering
+ *
+ * Array order IS render order. `required` is index 0 and is the section left
+ * open by default; `danger` is always last. That gives "1. Required → 2.
+ * Additional → … → least important last" mechanically, with no per-form
+ * `sectionOrder` to maintain.
+ */
+export const FORM_SECTION_GROUP = [
+    /** Cannot save without it. Open by default; a required field of ANY kind
+     *  belongs here, including a mandatory cover image. */
+    "required",
+    /** What the record is called and says — name, slug, description, SEO. */
+    "identity",
+    /** How it is filed — category, type, scope, tags, rules. */
+    "classification",
+    /** Money — price, rates, fees, tax. */
+    "pricing",
+    /** How many and until when — stock, limits, validity windows, slots. */
+    "inventory",
+    /** Optional media — gallery, video, creative, proof. */
+    "media",
+    /** Getting it there — address, shipping, delivery, routing. */
+    "logistics",
+    /** Links to other records — owner, members, linked accounts, sync. */
+    "relationships",
+    /** When — start/end dates, announcements, alerts. */
+    "schedule",
+    /** Who can see it and do what — publish state, placement, access, permissions. */
+    "visibility",
+    /** Everything a normal user never touches — config, defaults, layout. */
+    "advanced",
+    /** Destructive and irreversible. Always last, never open by default. */
+    "danger",
+];
+/** Position of a group in render order; unknown groups sort just before danger. */
+export function groupRank(group) {
+    if (!group)
+        return FORM_SECTION_GROUP.length - 2;
+    const i = FORM_SECTION_GROUP.indexOf(group);
+    return i === -1 ? FORM_SECTION_GROUP.length - 2 : i;
+}
+/**
+ * Default group for each section id currently in use, so 30 schemas do not each
+ * need a hand-written `group`. A section id absent here must declare its own —
+ * `audit-form-sectionised` fails otherwise, which is what stops a new id
+ * silently defaulting into the middle of the form.
+ *
+ * Derived from the 58 ids actually present, counted rather than guessed.
+ */
+export const SECTION_GROUP_DEFAULTS = {
+    // required
+    basics: "required", what: "required", who: "required", entry: "required",
+    declare: "required", ticket: "required", method: "required", card: "required",
+    bank: "required", connection: "required", member: "required",
+    // identity
+    seo: "identity", profile: "identity", details: "identity", detail: "identity",
+    content: "identity", message: "identity",
+    // classification
+    scope: "classification", target: "classification", rule: "classification",
+    flags: "classification",
+    // pricing
+    rates: "pricing", pricing: "pricing", payment: "pricing",
+    // inventory
+    limits: "inventory", validity: "inventory", slots: "inventory",
+    standing: "inventory",
+    // media
+    media: "media", creative: "media", proof: "media",
+    // logistics
+    address: "logistics", shipping: "logistics", delivery: "logistics",
+    routing: "logistics",
+    // relationships
+    owner: "relationships", members: "relationships", link: "relationships",
+    google: "relationships", sync: "relationships", subscribe: "relationships",
+    review: "relationships",
+    // schedule
+    schedule: "schedule", announcement: "schedule", alert: "schedule",
+    // visibility
+    visibility: "visibility", publish: "visibility", placement: "visibility",
+    status: "visibility", access: "visibility", permissions: "visibility",
+    capabilities: "visibility",
+    // advanced
+    settings: "advanced", config: "advanced", defaults: "advanced",
+    layout: "advanced",
+    // danger
+    ban: "danger", cancel: "danger",
+};
+/** The group for a section id — its declared group, else the default, else advanced. */
+export function resolveSectionGroup(sectionId, declared) {
+    return declared ?? SECTION_GROUP_DEFAULTS[sectionId] ?? "advanced";
+}

package/dist/features/shipments/components/AdminShipmentEditorView.js

@@ -1,9 +1,13 @@
 "use client";
-import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
+import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import React from "react";
 import { useQueryClient } from "@tanstack/react-query";
 import { useApiMutation } from "@mohasinac/appkit/client";
-import { Alert, Button, Div, Form, FieldInput, FieldSelect, FieldTextarea, Grid, Heading, Stack, Table, Thead, Tbody, Tr, Th, Td, Text, TextLink, FormErrorSummary, } from "../../../ui";
+import { Alert, Button, Div, FieldInput, Grid, Heading, Stack, Table, Thead, Tbody, Tr, Th, Td, Text, TextLink, FormErrorSummary, } from "../../../ui";
+import { FormShellContext, useFormShellState } from "../../../ui/forms/FormShell";
+import { applyZodIssues } from "../../../ui/forms/apply-zod-issues";
+import { buildSectionsFromSchema, visibleValues } from "../../shell/build-sections";
+import { SectionForm, useSectionFormNav } from "../../shell/SectionForm";
 import { apiClient } from "../../../http";
 import { ADMIN_ENDPOINTS } from "../../../constants/api-endpoints";
 import { ROUTES } from "../../../next/routing/route-map";
@@ -12,61 +16,72 @@ import { createShipmentSchema } from "../schemas/validation";
 import { useShipment } from "../hooks/useShipments";
 import { formatCurrency } from "../../../utils/number.formatter";
 import { RecordStatusTimeline } from "../../status-history/components/RecordStatusTimeline";
-const STATUS_OPTIONS = [
-    { label: "Planning", value: "planning" },
-    { label: "Ordered", value: "ordered" },
-    { label: "In Transit", value: "in_transit" },
-    { label: "Customs", value: "customs" },
-    { label: "Received", value: "received" },
-    { label: "Processing", value: "processing" },
-    { label: "Completed", value: "completed" },
-    { label: "Cancelled", value: "cancelled" },
-];
+const EMPTY_SHIPMENT = {
+    shipmentNumber: "",
+    supplierName: "",
+    originCountry: "",
+    status: "planning",
+    trackingNumber: "",
+    carrier: "",
+    etaDate: "",
+    receivedDate: "",
+    customsTotal: 0,
+    shippingTotal: 0,
+    laborHoursSpent: 0,
+    laborRatePerHour: 0,
+    notes: "",
+};
+/** A Firestore date, as a `<input type="date">` value. */
+function toDateInput(value) {
+    if (!value)
+        return "";
+    return new Date(value).toISOString().slice(0, 10);
+}
 export function AdminShipmentEditorView({ shipmentId, onSaved }) {
     const isCreate = !shipmentId;
     const queryClient = useQueryClient();
     const { shipment, lots, refetchLots } = useShipment(shipmentId);
-    const [shipmentNumber, setShipmentNumber] = React.useState("");
-    const [supplierName, setSupplierName] = React.useState("");
-    const [originCountry, setOriginCountry] = React.useState("");
-    const [status, setStatus] = React.useState("planning");
-    const [trackingNumber, setTrackingNumber] = React.useState("");
-    const [carrier, setCarrier] = React.useState("");
-    const [etaDate, setEtaDate] = React.useState("");
-    const [notes, setNotes] = React.useState("");
-    const [customsTotalRupees, setCustomsTotalRupees] = React.useState("0");
-    const [shippingTotalRupees, setShippingTotalRupees] = React.useState("0");
-    const [laborHoursSpent, setLaborHoursSpent] = React.useState("0");
+    const [form, setForm] = React.useState(EMPTY_SHIPMENT);
+    const patch = (partial) => setForm((prev) => Object.assign({}, prev, partial));
     React.useEffect(() => {
         if (!shipment)
             return;
-        setShipmentNumber(shipment.shipmentNumber);
-        setSupplierName(shipment.supplierName);
-        setOriginCountry(shipment.originCountry ?? "");
-        setStatus(shipment.status);
-        setTrackingNumber(shipment.trackingNumber ?? "");
-        setCarrier(shipment.carrier ?? "");
-        setEtaDate(shipment.etaDate ? new Date(shipment.etaDate).toISOString().slice(0, 10) : "");
-        setNotes(shipment.notes ?? "");
-        setCustomsTotalRupees(String(shipment.customsTotal));
-        setShippingTotalRupees(String(shipment.shippingTotal));
-        setLaborHoursSpent(String(shipment.laborHoursSpent ?? 0));
+        const doc = shipment;
+        patch({
+            shipmentNumber: doc.shipmentNumber,
+            supplierName: doc.supplierName,
+            originCountry: doc.originCountry ?? "",
+            status: doc.status,
+            trackingNumber: doc.trackingNumber ?? "",
+            carrier: doc.carrier ?? "",
+            etaDate: toDateInput(doc.etaDate),
+            receivedDate: toDateInput(doc.receivedDate),
+            customsTotal: doc.customsTotal,
+            shippingTotal: doc.shippingTotal,
+            laborHoursSpent: doc.laborHoursSpent ?? 0,
+            laborRatePerHour: doc.laborRatePerHour ?? 0,
+            notes: doc.notes ?? "",
+        });
+        // eslint-disable-next-line react-hooks/exhaustive-deps
     }, [shipment]);
     const saveMutation = useApiMutation({
         successMessage: isCreate ? "Shipment created" : "Shipment updated",
         mutationFn: async () => {
+            const draft = visibleValues(createShipmentSchema, form);
             const payload = {
-                shipmentNumber,
-                supplierName,
-                originCountry: originCountry || undefined,
-                status,
-                trackingNumber: trackingNumber || undefined,
-                carrier: carrier || undefined,
-                etaDate: etaDate || undefined,
-                notes: notes || undefined,
-                customsTotal: Math.round(Number(customsTotalRupees) * 100) / 100,
-                shippingTotal: Math.round(Number(shippingTotalRupees) * 100) / 100,
-                laborHoursSpent: Number(laborHoursSpent),
+                shipmentNumber: draft.shipmentNumber,
+                supplierName: draft.supplierName,
+                originCountry: draft.originCountry || undefined,
+                status: draft.status,
+                trackingNumber: draft.trackingNumber || undefined,
+                carrier: draft.carrier || undefined,
+                etaDate: draft.etaDate || undefined,
+                receivedDate: draft.receivedDate || undefined,
+                notes: draft.notes || undefined,
+                customsTotal: draft.customsTotal,
+                shippingTotal: draft.shippingTotal,
+                laborHoursSpent: draft.laborHoursSpent,
+                laborRatePerHour: draft.laborRatePerHour || undefined,
             };
             if (isCreate) {
                 return apiClient.post(ADMIN_ENDPOINTS.SHIPMENTS, payload);
@@ -78,24 +93,32 @@ export function AdminShipmentEditorView({ shipmentId, onSaved }) {
             onSaved?.(saved.id);
         },
     });
-    return (_jsx(Form, { schema: createShipmentSchema, spacing: "md", children: ({ setFieldError, markSubmitAttempted }) => (_jsxs(_Fragment, { children: [_jsxs(Grid, { cols: 2, gap: "md", children: [_jsx(FieldInput, { name: "shipmentNumber", label: "Shipment Number", required: true, value: shipmentNumber, onChange: setShipmentNumber, placeholder: "SH-2026-0001" }), _jsx(FieldInput, { name: "supplierName", label: "Supplier Name", required: true, value: supplierName, onChange: setSupplierName }), _jsx(FieldInput, { name: "originCountry", label: "Origin Country", value: originCountry, onChange: setOriginCountry }), _jsx(FieldSelect, { name: "status", label: "Status", required: true, value: status, onChange: setStatus, options: STATUS_OPTIONS }), _jsx(FieldInput, { name: "trackingNumber", label: "Tracking Number", value: trackingNumber, onChange: setTrackingNumber }), _jsx(FieldInput, { name: "carrier", label: "Carrier", value: carrier, onChange: setCarrier }), _jsx(FieldInput, { name: "etaDate", label: "ETA Date", type: "date", value: etaDate, onChange: setEtaDate }), _jsx(FieldInput, { name: "laborHoursSpent", label: "Labor Hours Spent", type: "number", min: "0", value: laborHoursSpent, onChange: setLaborHoursSpent }), _jsx(FieldInput, { name: "customsTotal", label: "Customs Total (\u20B9)", type: "number", min: "0", value: customsTotalRupees, onChange: setCustomsTotalRupees, hint: "Split across lots by declared value" }), _jsx(FieldInput, { name: "shippingTotal", label: "Shipping Total (\u20B9)", type: "number", min: "0", value: shippingTotalRupees, onChange: setShippingTotalRupees, hint: "Split across lots by weight" })] }), _jsx(FieldTextarea, { name: "notes", label: "Notes", value: notes, onChange: setNotes, rows: 3 }), _jsx(FormErrorSummary, {}), _jsx(Button, { type: "button", isLoading: saveMutation.isPending, onClick: () => {
-                        // type="button" — no native submit, so unhide the summary here.
-                        markSubmitAttempted();
-                        const parsed = createShipmentSchema.safeParse({
-                            shipmentNumber,
-                            supplierName,
-                            status,
-                            customsTotal: Math.round(Number(customsTotalRupees) * 100) / 100,
-                            shippingTotal: Math.round(Number(shippingTotalRupees) * 100) / 100,
-                            laborHoursSpent: Number(laborHoursSpent),
-                        });
-                        if (!parsed.success) {
-                            for (const issue of parsed.error.issues)
-                                setFieldError(String(issue.path[0]), issue.message);
-                            return;
-                        }
-                        saveMutation.mutate();
-                    }, children: isCreate ? "Create Shipment" : "Save Changes" }), !isCreate && shipmentId && (_jsx(ShipmentLotsSection, { shipmentId: shipmentId, lots: lots, onLotsChanged: refetchLots })), !isCreate && (_jsx(RecordStatusTimeline, { entries: shipment?.statusHistory, truncatedCount: shipment?.statusHistoryTruncated }))] })) }));
+    const sections = React.useMemo(() => buildSectionsFromSchema(createShipmentSchema), []);
+    const nav = useSectionFormNav(sections, form, { scope: "admin:shipment-editor" });
+    const { shellCtx, setFieldError, clearErrors } = useFormShellState(createShipmentSchema, {
+        sections: nav.sectionMeta,
+        onGoToSection: nav.goToSection,
+        fieldToSectionIndex: nav.fieldToSectionIndex,
+    });
+    const handleSubmit = () => {
+        clearErrors();
+        /*
+         * The whole draft, not six of its thirteen fields.
+         *
+         * The hand-rolled parse passed a literal covering `shipmentNumber`,
+         * `supplierName`, `status` and the three money fields — so `originCountry`,
+         * `trackingNumber`, `carrier`, `etaDate` and `notes` were never checked by
+         * anything, and `receivedDate` / `laborRatePerHour` had no control at all
+         * although the schema declares them and the route accepts them.
+         */
+        const parsed = createShipmentSchema.safeParse(visibleValues(createShipmentSchema, form));
+        if (!parsed.success) {
+            applyZodIssues(parsed.error.issues, setFieldError);
+            return;
+        }
+        saveMutation.mutate();
+    };
+    return (_jsxs(Stack, { gap: "md", children: [_jsxs(FormShellContext.Provider, { value: shellCtx, children: [_jsx(FormErrorSummary, {}), _jsx(SectionForm, { sections: sections, values: form, onChange: patch, onSubmit: handleSubmit, schema: createShipmentSchema, openIds: nav.openIds, onOpenChange: nav.setOpenIds, isLoading: saveMutation.isPending, submitLabel: isCreate ? "Create shipment" : "Save changes" })] }), !isCreate && shipmentId && (_jsx(ShipmentLotsSection, { shipmentId: shipmentId, lots: lots, onLotsChanged: refetchLots })), !isCreate && (_jsx(RecordStatusTimeline, { entries: shipment?.statusHistory, truncatedCount: shipment?.statusHistoryTruncated }))] }));
 }
 function ShipmentLotsSection({ shipmentId, lots, onLotsChanged, }) {
     const [showAddLot, setShowAddLot] = React.useState(false);
@@ -122,5 +145,5 @@ function ShipmentLotsSection({ shipmentId, lots, onLotsChanged, }) {
         mutationFn: (lotId) => apiClient.delete(ADMIN_ENDPOINTS.SHIPMENT_LOT_BY_ID(shipmentId, lotId)),
         onSuccess: () => onLotsChanged(),
     });
-    return (_jsxs(Stack, { gap: "md", className: "border-t pt-[var(--appkit-space-4)]", children: [_jsxs(Div, { layout: "flex", align: "center", justify: "between", children: [_jsxs(Heading, { level: 4, children: ["Lots (", lots.length, "/10)"] }), lots.length < 10 && (_jsx(Button, { size: "sm", variant: "secondary", type: "button", onClick: () => setShowAddLot((v) => !v), children: showAddLot ? "Cancel" : "+ Add Lot" }))] }), showAddLot && (_jsxs(Grid, { cols: 3, gap: "sm", children: [_jsx(FieldInput, { name: "lotName", label: "Lot Name", value: lotName, onChange: setLotName }), _jsx(FieldInput, { name: "weightGrams", label: "Weight (g)", type: "number", min: "0", value: weightGrams, onChange: setWeightGrams }), _jsx(FieldInput, { name: "purchaseCost", label: "Purchase Cost (\u20B9)", type: "number", min: "0", value: purchaseCostRupees, onChange: setPurchaseCostRupees }), _jsx(Button, { type: "button", size: "sm", isLoading: addLotMutation.isPending, disabled: !lotName, onClick: () => addLotMutation.mutate(), children: "Save Lot" })] })), lots.length === 0 ? (_jsx(Text, { variant: "secondary", children: "No lots yet. Add a lot, then manage its items." })) : (_jsxs(Table, { children: [_jsx(Thead, { children: _jsxs(Tr, { children: [_jsx(Th, { children: "Lot" }), _jsx(Th, { children: "Weight (g)" }), _jsx(Th, { children: "Items" }), _jsx(Th, { children: "Projected Profit" }), _jsx(Th, {})] }) }), _jsx(Tbody, { children: lots.map((lot) => (_jsxs(Tr, { children: [_jsx(Td, { children: lot.lotName }), _jsx(Td, { children: lot.weightGrams }), _jsx(Td, { children: lot.itemCount }), _jsx(Td, { children: formatCurrency(lot.projectedProfit) }), _jsxs(Td, { className: "flex gap-[var(--appkit-space-2)]", children: [_jsx(TextLink, { href: ROUTES.ADMIN.SHIPMENT_LOT_ITEMS(shipmentId, lot.id), children: "Manage Items \u2192" }), _jsx(Button, { size: "sm", variant: "ghost", type: "button", action: ACTIONS.SHIPMENT["delete"], isLoading: deleteLotMutation.isPending, onClick: () => deleteLotMutation.mutate(lot.id), children: "Delete" })] })] }, lot.id))) })] })), lots.length > 0 && (_jsx(Alert, { variant: "info", children: "Customs/shipping allocation and profit totals recompute in the background a few seconds after any lot or item change \u2014 reload if a number looks stale." }))] }));
+    return (_jsxs(Stack, { gap: "md", className: "border-t pt-[var(--appkit-space-4)]", children: [_jsxs(Div, { layout: "flex", align: "center", justify: "between", children: [_jsxs(Heading, { level: 4, children: ["Lots (", lots.length, "/10)"] }), lots.length < 10 && (_jsx(Button, { size: "sm", variant: "secondary", type: "button", onClick: () => setShowAddLot((v) => !v), children: showAddLot ? "Cancel" : "+ Add Lot" }))] }), showAddLot && (_jsxs(Grid, { cols: 3, gap: "sm", children: [_jsx(FieldInput, { name: "lotName", label: "Lot Name", value: lotName, onChange: setLotName }), _jsx(FieldInput, { name: "weightGrams", label: "Weight (g)", type: "number", min: "0", value: weightGrams, onChange: setWeightGrams }), _jsx(FieldInput, { name: "purchaseCost", label: "Purchase Cost (\u20B9)", type: "number", min: "0", value: purchaseCostRupees, onChange: setPurchaseCostRupees }), _jsx(Button, { type: "button", size: "sm", isLoading: addLotMutation.isPending, disabled: !lotName, onClick: () => addLotMutation.mutate(), children: "Save Lot" })] })), lots.length === 0 ? (_jsx(Text, { variant: "secondary", children: "No lots yet. Add a lot, then manage its items." })) : (_jsxs(Table, { children: [_jsx(Thead, { children: _jsxs(Tr, { children: [_jsx(Th, { children: "Lot" }), _jsx(Th, { children: "Weight (g)" }), _jsx(Th, { children: "Items" }), _jsx(Th, { children: "Projected Profit" }), _jsx(Th, {})] }) }), _jsx(Tbody, { children: lots.map((lot) => (_jsxs(Tr, { children: [_jsx(Td, { children: lot.lotName }), _jsx(Td, { children: lot.weightGrams }), _jsx(Td, { children: lot.itemCount }), _jsx(Td, { children: formatCurrency(lot.projectedProfit) }), _jsxs(Td, { className: "flex gap-[var(--appkit-space-2)]", children: [_jsx(TextLink, { href: ROUTES.ADMIN.SHIPMENT_LOT_ITEMS(shipmentId, lot.id), children: "Manage Items \u2192" }), _jsx(Button, { size: "sm", variant: "ghost", type: "button", action: ACTIONS.SHIPMENT["delete-lot"], isLoading: deleteLotMutation.isPending, onClick: () => deleteLotMutation.mutate(lot.id), children: "Delete" })] })] }, lot.id))) })] })), lots.length > 0 && (_jsx(Alert, { variant: "info", children: "Customs/shipping allocation and profit totals recompute in the background a few seconds after any lot or item change \u2014 reload if a number looks stale." }))] }));
 }

package/dist/features/shipments/schemas/validation.d.ts

@@ -9,11 +9,11 @@ export declare const createShipmentSchema: z.ZodObject<{
     carrier: z.ZodOptional<z.ZodString>;
     etaDate: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodDate]>>;
     receivedDate: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodDate]>>;
-    notes: z.ZodOptional<z.ZodString>;
     customsTotal: z.ZodNumber;
     shippingTotal: z.ZodNumber;
     laborHoursSpent: z.ZodOptional<z.ZodNumber>;
     laborRatePerHour: z.ZodOptional<z.ZodNumber>;
+    notes: z.ZodOptional<z.ZodString>;
 }, "strip", z.ZodTypeAny, {
     status: "received" | "cancelled" | "processing" | "completed" | "ordered" | "planning" | "in_transit" | "customs";
     shipmentNumber: string;
@@ -52,11 +52,11 @@ export declare const updateShipmentSchema: z.ZodObject<{
     carrier: z.ZodOptional<z.ZodOptional<z.ZodString>>;
     etaDate: z.ZodOptional<z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodDate]>>>;
     receivedDate: z.ZodOptional<z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodDate]>>>;
-    notes: z.ZodOptional<z.ZodOptional<z.ZodString>>;
     customsTotal: z.ZodOptional<z.ZodNumber>;
     shippingTotal: z.ZodOptional<z.ZodNumber>;
     laborHoursSpent: z.ZodOptional<z.ZodOptional<z.ZodNumber>>;
     laborRatePerHour: z.ZodOptional<z.ZodOptional<z.ZodNumber>>;
+    notes: z.ZodOptional<z.ZodOptional<z.ZodString>>;
 }, "strip", z.ZodTypeAny, {
     status?: "received" | "cancelled" | "processing" | "completed" | "ordered" | "planning" | "in_transit" | "customs" | undefined;
     notes?: string | undefined;

package/dist/features/shipments/schemas/validation.js

@@ -1,4 +1,5 @@
 import { z } from "zod";
+import { annotate } from "../../shell/field-ui-meta";
 import { mediaFieldSchema } from "../../media/types/index";
 import { MAX_ITEMS_PER_LOT } from "./firestore";
 export const shipmentStatusSchema = z.enum([
@@ -36,19 +37,49 @@ const listingTypeSchema = z.enum([
     "stickers",
 ]);
 export const createShipmentSchema = z.object({
-    shipmentNumber: z.string().min(1, "Shipment number is required"),
-    supplierName: z.string().min(1, "Supplier name is required"),
-    originCountry: z.string().optional(),
-    status: shipmentStatusSchema,
-    trackingNumber: z.string().optional(),
-    carrier: z.string().optional(),
-    etaDate: z.union([z.string(), z.date()]).optional(),
-    receivedDate: z.union([z.string(), z.date()]).optional(),
-    notes: z.string().optional(),
-    customsTotal: z.number().min(0),
-    shippingTotal: z.number().min(0),
-    laborHoursSpent: z.number().min(0).optional(),
-    laborRatePerHour: z.number().min(0).optional(),
+    shipmentNumber: annotate(z.string().min(1, "Shipment number is required"), {
+        section: "basics", sectionLabel: "Shipment", sectionRequired: true,
+        order: 1, row: "pair", label: "Shipment number",
+    }),
+    supplierName: annotate(z.string().min(1, "Supplier name is required"), {
+        section: "basics", order: 2, row: "pair", label: "Supplier",
+    }),
+    originCountry: annotate(z.string().optional(), {
+        section: "basics", order: 3, row: "pair", label: "Origin country",
+    }),
+    status: annotate(shipmentStatusSchema, {
+        section: "basics", order: 4, row: "pair", kind: "select", label: "Status",
+    }),
+    trackingNumber: annotate(z.string().optional(), {
+        section: "logistics", sectionLabel: "Tracking & dates",
+        order: 1, row: "pair", label: "Tracking number",
+    }),
+    carrier: annotate(z.string().optional(), {
+        section: "logistics", order: 2, row: "pair", label: "Carrier",
+    }),
+    etaDate: annotate(z.union([z.string(), z.date()]).optional(), {
+        section: "logistics", order: 3, row: "pair", kind: "date", label: "ETA",
+    }),
+    receivedDate: annotate(z.union([z.string(), z.date()]).optional(), {
+        section: "logistics", order: 4, row: "pair", kind: "date", label: "Received on",
+    }),
+    customsTotal: annotate(z.number().min(0), {
+        section: "pricing", sectionLabel: "Landed cost",
+        order: 1, row: "pair", kind: "number", label: "Customs total (₹)",
+    }),
+    shippingTotal: annotate(z.number().min(0), {
+        section: "pricing", order: 2, row: "pair", kind: "number", label: "Shipping total (₹)",
+    }),
+    laborHoursSpent: annotate(z.number().min(0).optional(), {
+        section: "pricing", order: 3, row: "pair", kind: "number", label: "Labour hours",
+    }),
+    laborRatePerHour: annotate(z.number().min(0).optional(), {
+        section: "pricing", order: 4, row: "pair", kind: "number", label: "Labour rate (₹/hr)",
+    }),
+    notes: annotate(z.string().optional(), {
+        section: "detail", sectionLabel: "Notes", order: 1, row: "full", kind: "textarea",
+        label: "Notes",
+    }),
 });
 export const updateShipmentSchema = createShipmentSchema.partial();
 export const createShipmentLotSchema = z.object({

package/dist/features/store-extensions/schemas/custom-role-form.js

@@ -19,8 +19,12 @@
  * it grants nothing — it silently never matches — so a role built from typos
  * looks configured and does nothing, which is worse than being rejected.
  *
- * `"*"` is accepted deliberately: `security/rbac/types.ts` defines it as the
- * wildcard, and rejecting it here would break a legitimate super-role.
+ * `"*"` is accepted deliberately as a super-role wildcard. It used to be
+ * defined by `security/rbac/types.ts`, which was deleted 2026-08-29 as a dead
+ * second permission system — so today NOTHING expands it: `checkPermission`
+ * does a plain `includes()`. A role carrying `"*"` therefore grants exactly
+ * nothing. Kept accepted so existing stored roles still validate; make it
+ * expand, or reject it, before advertising it as a feature.
  *
  * ## What is NOT accepted from the body
  *

package/dist/features/store-extensions/schemas/listing-template-form.d.ts

@@ -18,15 +18,15 @@ export declare const listingTemplateFormSchema: z.ZodObject<{
     name: string;
     listingType: "art" | "stickers" | "classified" | "standard" | "auction" | "pre-order" | "prize-draw" | "digital-code" | "live" | "bundle";
     isActive: boolean;
-    isShared: boolean;
     defaults: Record<string, import("../../..").FirestoreValue>;
+    isShared: boolean;
     description?: string | undefined;
 }, {
     name: string;
     listingType: "art" | "stickers" | "classified" | "standard" | "auction" | "pre-order" | "prize-draw" | "digital-code" | "live" | "bundle";
     isActive: boolean;
-    isShared: boolean;
     defaults: Record<string, import("../../..").FirestoreValue>;
+    isShared: boolean;
     description?: string | undefined;
 }>;
 export type ListingTemplateFormValues = z.infer<typeof listingTemplateFormSchema>;
@@ -40,22 +40,22 @@ export declare const listingTemplateCreateSchema: z.ZodObject<{
     listingType: z.ZodEnum<["standard", "auction", "pre-order", "prize-draw", "bundle", "classified", "digital-code", "live", "art", "stickers"]>;
     description: z.ZodOptional<z.ZodOptional<z.ZodString>>;
     isActive: z.ZodOptional<z.ZodBoolean>;
-    isShared: z.ZodOptional<z.ZodBoolean>;
     defaults: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodType<import("../../..").FirestoreValue, z.ZodTypeDef, import("../../..").FirestoreValue>>>;
+    isShared: z.ZodOptional<z.ZodBoolean>;
 }, "strip", z.ZodTypeAny, {
     name: string;
     listingType: "art" | "stickers" | "classified" | "standard" | "auction" | "pre-order" | "prize-draw" | "digital-code" | "live" | "bundle";
     description?: string | undefined;
     isActive?: boolean | undefined;
-    isShared?: boolean | undefined;
     defaults?: Record<string, import("../../..").FirestoreValue> | undefined;
+    isShared?: boolean | undefined;
 }, {
     name: string;
     listingType: "art" | "stickers" | "classified" | "standard" | "auction" | "pre-order" | "prize-draw" | "digital-code" | "live" | "bundle";
     description?: string | undefined;
     isActive?: boolean | undefined;
-    isShared?: boolean | undefined;
     defaults?: Record<string, import("../../..").FirestoreValue> | undefined;
+    isShared?: boolean | undefined;
 }>;
 /** Update contract — every field optional, and the same ownership fields absent. */
 /**
@@ -84,13 +84,13 @@ export declare const listingTemplateUpdateSchema: z.ZodObject<{
     listingType?: "art" | "stickers" | "classified" | "standard" | "auction" | "pre-order" | "prize-draw" | "digital-code" | "live" | "bundle" | undefined;
     description?: string | undefined;
     isActive?: boolean | undefined;
-    isShared?: boolean | undefined;
     defaults?: Record<string, import("../../..").FirestoreValue> | undefined;
+    isShared?: boolean | undefined;
 }, {
     name?: string | undefined;
     listingType?: "art" | "stickers" | "classified" | "standard" | "auction" | "pre-order" | "prize-draw" | "digital-code" | "live" | "bundle" | undefined;
     description?: string | undefined;
     isActive?: boolean | undefined;
-    isShared?: boolean | undefined;
     defaults?: Record<string, import("../../..").FirestoreValue> | undefined;
+    isShared?: boolean | undefined;
 }>;

package/dist/features/store-extensions/schemas/payout-method-form.js

@@ -56,21 +56,35 @@ export const payoutMethodFormSchema = z
     label: annotate(z.string().min(1, "Give this method a name you'll recognise.").max(80), {
         section: "method", quick: true, order: 2, row: "pair",
     }),
+    /*
+     * `when` carries the branching the PAGE used to hold as JSX conditionals,
+     * so the generated form is the same form.
+     *
+     * Every one of these is already `.optional()` at the type level and made
+     * conditionally required by the `superRefine` below — which is the pairing
+     * `FieldUiMeta.when` demands, because hiding a field does NOT relax its
+     * schema. A required field hidden by `when` fails validation invisibly.
+     */
     upiVpa: annotate(z.string().max(256).optional(), {
         section: "method", order: 3, row: "full",
         help: "For UPI methods, e.g. name@bank.",
+        when: (v) => v.type === "upi",
     }),
     accountHolderName: annotate(z.string().max(120).optional(), {
         section: "bank", sectionLabel: "Bank Account", order: 1, row: "full",
+        when: (v) => v.type === "bank",
     }),
     accountNumber: annotate(z.string().max(34).optional(), {
         section: "bank", order: 2, row: "pair",
+        when: (v) => v.type === "bank",
     }),
     ifscCode: annotate(z.string().max(11).optional(), {
         section: "bank", order: 3, row: "pair",
+        when: (v) => v.type === "bank",
     }),
     bankName: annotate(z.string().max(120).optional(), {
         section: "bank", order: 4, row: "pair",
+        when: (v) => v.type === "bank",
     }),
     isDefault: annotate(z.boolean(), {
         section: "visibility", sectionLabel: "Visibility", order: 1, row: "quarter",

package/dist/features/stores/actions/store-query-actions.js

@@ -15,6 +15,7 @@ import { STORE_FIELDS } from "../schemas";
 import { productRepository } from "../../products/repository/products.repository";
 import { ProductStatusValues } from "../../products/schemas";
 import { reviewRepository } from "../../reviews/repository/reviews.repository";
+import { PRODUCT_FIELDS } from "../../../constants/field-names";
 export async function listStores(params = {}, viewer) {
     const { filters, sorts = "-createdAt", page = 1, pageSize = 24, q } = params;
     const model = { filters, sorts, page, pageSize };
@@ -65,7 +66,7 @@ export async function getStoreProducts(storeSlug, params = {}, viewer) {
     const filtersArr = [
         sieveFilter("storeId", SIEVE_OP.EQ, storeDoc.id),
         "status==published",
-        "listingType==standard",
+        sieveFilter(PRODUCT_FIELDS.LISTING_TYPE, SIEVE_OP.EQ, "standard"),
     ];
     if (filters)
         filtersArr.push(filters);
@@ -87,7 +88,7 @@ export async function getStoreAuctions(storeSlug, params = {}, viewer) {
     const filtersArr = [
         sieveFilter("storeId", SIEVE_OP.EQ, storeDoc.id),
         "status==published",
-        "listingType==auction",
+        sieveFilter(PRODUCT_FIELDS.LISTING_TYPE, SIEVE_OP.EQ, "auction"),
     ];
     if (filters)
         filtersArr.push(filters);