@mohasinac/appkit 4.27.0 → 4.29.0

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

@@ -15,6 +15,7 @@ const addressFormSchema = z.object({
     phone: z.string().min(1, "Phone number is required"),
     addressLine1: z.string().min(1, "Address is required"),
     addressLine2: z.string().optional(),
+    landmark: z.string().max(100, "Keep the landmark under 100 characters.").optional(),
     city: z.string().min(1, "City is required"),
     state: z.string().min(1, "State is required"),
     postalCode: z.string().min(1, "Postal code is required"),
@@ -27,6 +28,7 @@ const DEFAULT_LABELS = {
     phone: "Phone Number",
     addressLine1: "Address Line 1",
     addressLine2: "Address Line 2",
+    landmark: "Landmark",
     city: "City",
     state: "State",
     postalCode: "Postal Code",
@@ -42,6 +44,7 @@ const DEFAULT_PLACEHOLDERS = {
     phone: "Enter phone number",
     addressLine1: "Street address, P.O. box",
     addressLine2: "Apartment, suite, unit, building, floor, etc.",
+    landmark: "Opposite the Metro station, next to the temple, etc.",
     city: "City",
     state: "State/Province",
     postalCode: "Postal/ZIP code",
@@ -58,6 +61,7 @@ export function AddressForm({ initialData, onSubmit, onCancel, isLoading = false
         phone: initialData?.phone || "",
         addressLine1: initialData?.addressLine1 || "",
         addressLine2: initialData?.addressLine2 || "",
+        landmark: initialData?.landmark || "",
         city: initialData?.city || "",
         state: initialData?.state || "",
         postalCode: initialData?.postalCode || "",
@@ -114,5 +118,5 @@ export function AddressForm({ initialData, onSubmit, onCancel, isLoading = false
             submitLabel: effectiveSubmitLabel,
             cancelLabel: mergedLabels.cancel,
             isLoading,
-        }, children: ({ setFieldError, clearErrors }) => (_jsxs(_Fragment, { children: [_jsx(FormField, { label: mergedLabels.label, name: "label", type: "text", value: formData.label, onChange: (value) => handleChange("label", value), placeholder: mergedPlaceholders.label, required: true }), _jsx(FormField, { label: mergedLabels.fullName, name: "fullName", type: "text", value: formData.fullName, onChange: (value) => handleChange("fullName", value), placeholder: mergedPlaceholders.fullName, required: true }), _jsx(FormField, { label: mergedLabels.phone, name: "phone", type: "tel", value: formData.phone, onChange: (value) => handleChange("phone", value), placeholder: mergedPlaceholders.phone, required: true }), _jsx(FormField, { label: mergedLabels.addressLine1, name: "addressLine1", type: "text", value: formData.addressLine1, onChange: (value) => handleChange("addressLine1", value), placeholder: mergedPlaceholders.addressLine1, required: true }), _jsx(FormField, { label: mergedLabels.addressLine2, name: "addressLine2", type: "text", value: formData.addressLine2, onChange: (value) => handleChange("addressLine2", value), placeholder: mergedPlaceholders.addressLine2 }), _jsxs(FormGroup, { columns: 3, children: [_jsx(FormField, { label: mergedLabels.city, name: "city", type: "text", value: formData.city, onChange: (value) => handleChange("city", value), placeholder: mergedPlaceholders.city, required: true }), _jsx(FormField, { label: mergedLabels.state, name: "state", type: "text", value: formData.state, onChange: (value) => handleChange("state", value), placeholder: mergedPlaceholders.state, required: true }), _jsx(FormField, { label: mergedLabels.postalCode, name: "postalCode", type: "text", value: formData.postalCode, onChange: (value) => handleChange("postalCode", value), placeholder: mergedPlaceholders.postalCode, required: true })] }), _jsx(FormField, { label: mergedLabels.country, name: "country", type: "text", value: formData.country, onChange: (value) => handleChange("country", value), placeholder: mergedPlaceholders.country, required: true }), _jsx(Checkbox, { checked: formData.isDefault, onChange: (e) => handleChange("isDefault", e.target.checked), label: mergedLabels.setDefault }), _jsx(FormErrorSummary, {}), _jsxs(Row, { padding: "t-xs", align: "center", justify: "start", gap: "xs", children: [_jsx(Button, { type: "button", variant: "outline", onClick: onCancel, disabled: isLoading, children: mergedLabels.cancel }), _jsx(Button, { type: "submit", variant: "primary", disabled: isLoading, isLoading: isLoading, onClick: () => void submitAddress({ setFieldError, clearErrors }), children: isLoading ? mergedLabels.loading : effectiveSubmitLabel })] })] })) }));
+        }, children: ({ setFieldError, clearErrors }) => (_jsxs(_Fragment, { children: [_jsx(FormField, { label: mergedLabels.label, name: "label", type: "text", value: formData.label, onChange: (value) => handleChange("label", value), placeholder: mergedPlaceholders.label, required: true }), _jsx(FormField, { label: mergedLabels.fullName, name: "fullName", type: "text", value: formData.fullName, onChange: (value) => handleChange("fullName", value), placeholder: mergedPlaceholders.fullName, required: true }), _jsx(FormField, { label: mergedLabels.phone, name: "phone", type: "tel", value: formData.phone, onChange: (value) => handleChange("phone", value), placeholder: mergedPlaceholders.phone, required: true }), _jsx(FormField, { label: mergedLabels.addressLine1, name: "addressLine1", type: "text", value: formData.addressLine1, onChange: (value) => handleChange("addressLine1", value), placeholder: mergedPlaceholders.addressLine1, required: true }), _jsx(FormField, { label: mergedLabels.addressLine2, name: "addressLine2", type: "text", value: formData.addressLine2, onChange: (value) => handleChange("addressLine2", value), placeholder: mergedPlaceholders.addressLine2 }), _jsx(FormField, { label: mergedLabels.landmark, name: "landmark", type: "text", value: formData.landmark ?? "", onChange: (value) => handleChange("landmark", value), placeholder: mergedPlaceholders.landmark }), _jsxs(FormGroup, { columns: 3, children: [_jsx(FormField, { label: mergedLabels.city, name: "city", type: "text", value: formData.city, onChange: (value) => handleChange("city", value), placeholder: mergedPlaceholders.city, required: true }), _jsx(FormField, { label: mergedLabels.state, name: "state", type: "text", value: formData.state, onChange: (value) => handleChange("state", value), placeholder: mergedPlaceholders.state, required: true }), _jsx(FormField, { label: mergedLabels.postalCode, name: "postalCode", type: "text", value: formData.postalCode, onChange: (value) => handleChange("postalCode", value), placeholder: mergedPlaceholders.postalCode, required: true })] }), _jsx(FormField, { label: mergedLabels.country, name: "country", type: "text", value: formData.country, onChange: (value) => handleChange("country", value), placeholder: mergedPlaceholders.country, required: true }), _jsx(Checkbox, { checked: formData.isDefault, onChange: (e) => handleChange("isDefault", e.target.checked), label: mergedLabels.setDefault }), _jsx(FormErrorSummary, {}), _jsxs(Row, { padding: "t-xs", align: "center", justify: "start", gap: "xs", children: [_jsx(Button, { type: "button", variant: "outline", onClick: onCancel, disabled: isLoading, children: mergedLabels.cancel }), _jsx(Button, { type: "submit", variant: "primary", disabled: isLoading, isLoading: isLoading, onClick: () => void submitAddress({ setFieldError, clearErrors }), children: isLoading ? mergedLabels.loading : effectiveSubmitLabel })] })] })) }));
 }

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

@@ -51,5 +51,5 @@ export function UserBidsView() {
         },
     };
     const badge = detail ? bidStatusBadge(detail.status) : null;
-    return (_jsxs(_Fragment, { children: [_jsx(DataListingView, { config: config }), _jsx(RecordDetailModal, { isOpen: detail !== null, onClose: () => setDetail(null), title: detail?.productTitle || "Bid", badges: badge ? [{ label: badge.label }] : undefined, fields: detail ? buildBidDetailFields(detail, "buyer") : undefined, footer: detail ? (_jsx(TextLink, { href: String(ROUTES.PUBLIC.AUCTION_DETAIL(detail.productId)), children: "View auction \u2192" })) : undefined })] }));
+    return (_jsxs(_Fragment, { children: [_jsx(DataListingView, { config: config }), _jsx(RecordDetailModal, { isOpen: detail !== null, onClose: () => setDetail(null), title: detail?.productTitle || "Bid", badges: badge ? [{ label: badge.label }] : undefined, fields: detail ? buildBidDetailFields(detail, "buyer") : undefined, footer: detail ? (_jsxs(_Fragment, { children: [_jsx(TextLink, { href: String(ROUTES.USER.BID_DETAIL(detail.id)), children: "Open full page \u2192" }), _jsx(TextLink, { href: String(ROUTES.PUBLIC.AUCTION_DETAIL(detail.productId)), children: "View auction \u2192" })] })) : undefined })] }));
 }

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

@@ -27,7 +27,7 @@ function NavLink({ item, isActive, onClick }) {
     };
     const linkClass = `flex items-center justify-end gap-[var(--appkit-space-2-5)] rounded-lg px-[var(--appkit-space-3)] py-[var(--appkit-space-2)] text-[0.8125rem] font-medium leading-tight transition-colors ${isActive
         ? "bg-primary-50 dark:bg-primary-900/25 text-primary-700 dark:text-primary-300"
-        : "text-[var(--appkit-color-text-muted)] hover:bg-zinc-50 hover:bg-[var(--appkit-color-surface-elevated)] hover:text-zinc-900 dark:hover:text-zinc-100"}`;
+        : "text-[var(--appkit-color-text-muted)] hover:bg-surface-hover hover:text-zinc-900 dark:hover:text-zinc-100"}`;
     return (_jsxs(_Fragment, { children: [_jsxs(Link, { href: item.href, onClick: handleClick, className: linkClass, children: [item.icon && _jsx(Span, { size: "base", className: "shrink-0 opacity-70", children: item.icon }), _jsx(Span, { className: "truncate", children: item.label })] }), showConfirm && item.confirm && (_jsx(ConfirmDeleteModal, { isOpen: true, title: item.confirm.title ?? "Confirm", message: item.confirm.message, confirmText: "Continue", variant: "warning", onConfirm: () => {
                     setShowConfirm(false);
                     onClick?.();

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

@@ -7,6 +7,8 @@ import { Button, Div, FormActions, Row, Select, SideDrawer, Span, Stack, Text, U
 import { FieldInput } from "../../../ui/forms/FieldInput";
 import { FieldTextarea } from "../../../ui/forms/FieldTextarea";
 import { apiClient } from "../../../http";
+import { supportTicketCreateSchema } from "../../support/schemas/ticket-create-form";
+import { ValidationError } from "../../../errors/validation-error";
 import { SUPPORT_ENDPOINTS } from "../../../constants/api-endpoints";
 const __P = {
     p3: "p-[var(--appkit-space-3)]",
@@ -33,7 +35,7 @@ const CATEGORY_OPTIONS = [
     { label: "Appeal a ban (unban request)", value: "unban_request" },
     { label: "General", value: "general" },
 ];
-const CLS_MSG_USER = "bg-zinc-50 border border-zinc-200 bg-[var(--appkit-color-surface)]/40 border-[var(--appkit-color-border)]";
+const CLS_MSG_USER = "border bg-[var(--appkit-color-surface)]/40 border-[var(--appkit-color-border)]";
 const CLS_MSG_STAFF = "bg-info-surface border border-info dark:bg-info-surface dark:border-info";
 const STATUS_BADGE = {
     open: "bg-info-surface text-info",
@@ -68,12 +70,32 @@ export function UserSupportView(_props) {
     const invalidate = () => queryClient.invalidateQueries({ queryKey: ["user", "support-tickets"] });
     const createMutation = useApiMutation({
         mutationFn: async () => {
-            await apiClient.post(SUPPORT_ENDPOINTS.TICKETS, {
+            /*
+             * This drawer validated NOTHING. Its sibling page
+             * (`/user/support/new`) hand-rolled a `canSubmit` boolean that required
+             * an order id for an `order_issue` ticket — a real rule, since an order
+             * complaint with no order is unactionable — and the route did not
+             * enforce it either. So the same ticket was acceptable or not purely
+             * depending on which surface the user happened to open.
+             *
+             * One schema now, across both surfaces and the route.
+             */
+            const parsed = supportTicketCreateSchema.safeParse({
                 subject: newSubject.trim(),
                 category: newCategory,
                 description: newDescription.trim(),
                 orderId: newOrderId.trim() || undefined,
             });
+            if (!parsed.success) {
+                /*
+                 * ValidationError, not a bare Error — it carries the issue LIST, so
+                 * the failure keeps which field caused it. A plain Error flattens the
+                 * whole thing to one string, which is what `audit-unvalidated-safeparse`
+                 * exists to stop. It caught this in my own code.
+                 */
+                throw new ValidationError(parsed.error.issues[0]?.message ?? "Check the ticket details.", parsed.error.issues);
+            }
+            await apiClient.post(SUPPORT_ENDPOINTS.TICKETS, parsed.data);
         },
         onSuccess: () => {
             showToast("Support ticket created.", "success");

package/dist/features/account/hooks/useAddresses.d.ts

@@ -9,6 +9,18 @@ export interface Address {
     state: string;
     postalCode: string;
     country: string;
+    /**
+     * A nearby reference point ("opposite the Metro station").
+     *
+     * Store addresses have always accepted it — `POST /api/store/addresses`
+     * declares `landmark: z.string().max(100).optional()` and
+     * `SellerAddressesView`'s drawer collects it — while this shape did not.
+     * So an address created in that drawer WITH a landmark, then edited through
+     * any surface built on `AddressForm`, was saved back with `landmark`
+     * absent and lost it. Added here, on the form, and on the `Address` type so
+     * the round trip is lossless.
+     */
+    landmark?: string;
     isDefault?: boolean;
 }
 export interface AddressFormData {
@@ -21,6 +33,8 @@ export interface AddressFormData {
     state: string;
     postalCode: string;
     country: string;
+    /** See Address.landmark — the form must carry it or an edit drops it. */
+    landmark?: string;
     isDefault?: boolean;
 }
 interface SetDefaultAddressData {
@@ -46,8 +60,22 @@ export declare function useCreateAddress(options?: {
     onSuccess?: (data: Address) => void;
     onError?: (error: Error) => void;
 }): import("@tanstack/react-query").UseMutationResult<Address, Error, AddressFormData, unknown>;
-export declare function useUpdateAddress(id: string, options?: {
+/**
+ * Update options.
+ *
+ * `method` exists because the two portals disagree: `/api/user/addresses/[id]`
+ * exports PATCH and `/api/store/addresses/[id]` exports PUT, and
+ * `SellerAddressesView` hardcodes PUT against the latter. Changing the route's
+ * verb would break that working call site, so the hook bends instead.
+ *
+ * Caught by `audit-client-verb-match` before the store pages were written —
+ * this hook would have 405'd the moment they reused it.
+ */
+export interface UpdateAddressOptions {
     byIdEndpoint?: (id: string) => string;
+    method?: "PUT" | "PATCH";
+}
+export declare function useUpdateAddress(id: string, options?: UpdateAddressOptions & {
     onSuccess?: (data: Address) => void;
     onError?: (error: Error) => void;
 }): import("@tanstack/react-query").UseMutationResult<Address, Error, AddressFormData, unknown>;

package/dist/features/account/hooks/useAddresses.js

@@ -48,8 +48,11 @@ export function useCreateAddress(options) {
 export function useUpdateAddress(id, options) {
     const queryClient = useQueryClient();
     const byIdEndpoint = options?.byIdEndpoint ?? DEFAULT_ENDPOINTS.byId;
+    const method = options?.method ?? "PATCH";
     return useMutation({
-        mutationFn: (data) => apiClient.patch(byIdEndpoint(id), data),
+        mutationFn: (data) => method === "PUT"
+            ? apiClient.put(byIdEndpoint(id), data)
+            : apiClient.patch(byIdEndpoint(id), data),
         onSuccess: (data) => {
             queryClient.invalidateQueries({ queryKey: ["addresses"] });
             options?.onSuccess?.(data);

package/dist/features/account/hooks/useProfile.d.ts

@@ -1,4 +1,4 @@
-import { type JsonValue } from "@mohasinac/appkit/client";
+import type { JsonValue } from "../../../schemas/types";
 export interface UserProfile {
     uid: string;
     email: string | null;

package/dist/features/account/hooks/useProfile.js

@@ -1,4 +1,4 @@
-import { useApiMutation } from "@mohasinac/appkit/client";
+import { useApiMutation } from "../../../client/api/useApiMutation";
 import { useQuery, useQueryClient } from "@tanstack/react-query";
 import { apiClient } from "../../../http";
 import { ACCOUNT_ENDPOINTS } from "../../../constants/api-endpoints";

package/dist/features/addresses/repository/addresses.repository.d.ts

@@ -11,6 +11,19 @@ export declare class AddressesRepository extends BaseRepository<AddressDocument>
     listByOwnerType(ownerType: AddressOwnerType, limit?: number): Promise<AddressDocument[]>;
     countByOwner(ownerType: AddressOwnerType, ownerId: string): Promise<number>;
     createForOwner(ownerType: AddressOwnerType, ownerId: string, input: AddressCreateInput): Promise<AddressDocument>;
+    /**
+     * Read one address, scoped to its owner.
+     *
+     * The owner check is `ownerType` AND `ownerId` — the same pair
+     * `updateForOwner` and `deleteForOwner` use, because `addresses` is one
+     * top-level collection holding both buyer and store addresses discriminated
+     * by `ownerType` (SB-UNI-A). Checking only `ownerId` would let a seller read
+     * their own personal address through the store endpoint and vice versa.
+     *
+     * Returns null rather than throwing: the caller turns it into a 404, and a
+     * missing address and a foreign one must be indistinguishable to the client.
+     */
+    getForOwner(ownerType: AddressOwnerType, ownerId: string, addressId: string): Promise<AddressDocument | null>;
     updateForOwner(ownerType: AddressOwnerType, ownerId: string, addressId: string, input: AddressUpdateInput): Promise<AddressDocument>;
     deleteForOwner(ownerType: AddressOwnerType, ownerId: string, addressId: string): Promise<void>;
     setDefault(ownerType: AddressOwnerType, ownerId: string, addressId: string): Promise<AddressDocument>;

package/dist/features/addresses/repository/addresses.repository.js

@@ -115,6 +115,25 @@ export class AddressesRepository extends BaseRepository {
             throw new DatabaseError(`Failed to create address for ${ownerType}:${ownerId}`, error);
         }
     }
+    /**
+     * Read one address, scoped to its owner.
+     *
+     * The owner check is `ownerType` AND `ownerId` — the same pair
+     * `updateForOwner` and `deleteForOwner` use, because `addresses` is one
+     * top-level collection holding both buyer and store addresses discriminated
+     * by `ownerType` (SB-UNI-A). Checking only `ownerId` would let a seller read
+     * their own personal address through the store endpoint and vice versa.
+     *
+     * Returns null rather than throwing: the caller turns it into a 404, and a
+     * missing address and a foreign one must be indistinguishable to the client.
+     */
+    async getForOwner(ownerType, ownerId, addressId) {
+        const existing = await this.findById(addressId);
+        if (!existing || existing.ownerType !== ownerType || existing.ownerId !== ownerId) {
+            return null;
+        }
+        return existing;
+    }
     async updateForOwner(ownerType, ownerId, addressId, input) {
         const existing = await this.findById(addressId);
         if (!existing || existing.ownerType !== ownerType || existing.ownerId !== ownerId) {

package/dist/features/admin/actions/chat-actions.js

@@ -14,6 +14,7 @@ import { getAdminRealtimeDb } from "../../../providers/db-firebase/index";
 import { AuthorizationError, NotFoundError, ValidationError, } from "../../../errors/index";
 import { ERROR_MESSAGES } from "../../../errors/messages";
 import { serverLogger } from "../../../monitoring/index";
+import { normalizeError } from "../../../errors/normalize";
 export async function getChatRooms(userId, chatEnabled) {
     if (!chatEnabled)
         throw new AuthorizationError("Chat is temporarily unavailable.");
@@ -77,7 +78,10 @@ export async function sendChatMessage(userId, chatId, message) {
             : (room.buyerName ?? "Member");
     await msgRef.set({ userId, userName, message, timestamp: Date.now() });
     chatRepository.updateLastMessage(chatId, message).catch((err) => {
-        serverLogger.warn("Failed to update chat lastMessage", { chatId, err });
+        serverLogger.warn("Failed to update chat lastMessage", {
+            chatId,
+            error: normalizeError(err).message,
+        });
     });
     serverLogger.debug("sendChatMessage", { userId, chatId });
     return { messageId: id, timestamp: Date.now() };

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

@@ -116,7 +116,7 @@ export function AdminAdsView({ endpoint = ADMIN_ENDPOINTS.ADS, labels = {}, crea
             render: (row) => {
                 const nextStatus = row.status === "active" ? "paused" : "active";
                 const cannotPublish = nextStatus === "active" && row.publishReady === false;
-                return (_jsxs(Row, { align: "center", gap: "sm", children: [renderEditLink ? renderEditLink(row) : (_jsx(TextLink, { variant: "bare", href: `/admin/ads/${row.id}/edit`, rounded: "md", paddingX: "sm", size: "xs", layout: "inline-flex", align: "center", className: "h-8 border border-neutral-200 border-[var(--appkit-color-border)]", children: "Edit" })), _jsx(Button, { size: "sm", variant: "outline", disabled: statusMutation.isPending || cannotPublish, onClick: () => statusMutation.mutate({ id: row.id, nextStatus }), children: row.status === "active" ? "Pause" : "Publish" })] }));
+                return (_jsxs(Row, { align: "center", gap: "sm", children: [renderEditLink ? renderEditLink(row) : (_jsx(TextLink, { variant: "bare", href: `/admin/ads/${row.id}/edit`, rounded: "md", paddingX: "sm", size: "xs", layout: "inline-flex", align: "center", className: "h-8 border border-[var(--appkit-color-border)]", children: "Edit" })), _jsx(Button, { size: "sm", variant: "outline", disabled: statusMutation.isPending || cannotPublish, onClick: () => statusMutation.mutate({ id: row.id, nextStatus }), children: row.status === "active" ? "Pause" : "Publish" })] }));
             },
         },
     ];

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

@@ -15,6 +15,8 @@ import { toRecordArray, toRelativeDate, toCurrency, toStringValue, } from "../ho
 import { DataListingView } from "./DataListingView";
 import { apiClient } from "../../../http";
 import { RecordStatusTimeline } from "../../status-history/components/RecordStatusTimeline";
+import { TextLink } from "../../../ui";
+import { ROUTES } from "../../../constants/index";
 export function AdminBidsView({ children, ...props }) {
     const queryClient = useQueryClient();
     const { showToast } = useToast();
@@ -99,7 +101,10 @@ export function AdminBidsView({ children, ...props }) {
     const detailBadge = detail ? bidStatusBadge(detail.status) : null;
     return (_jsxs(_Fragment, { children: [_jsx(DataListingView, { config: config }), _jsx(RecordDetailModal, { isOpen: detail !== null, onClose: () => setDetail(null), title: toStringValue(detail?.detail?.productTitle, "Bid"), badges: detailBadge ? [{ label: detailBadge.label }] : undefined, fields: detail
                     ? buildBidDetailFields(detail.detail, "admin")
-                    : undefined, extra: 
+                    : undefined, footer: detail ? (
+                // The full page — a modal cannot be linked, bookmarked or reloaded
+                // into, which is the whole reason the page exists.
+                _jsx(TextLink, { href: String(ROUTES.ADMIN.BID_DETAIL(detail.id)), children: "Open full page \u2192" })) : undefined, extra: 
                 // A bid moves through more states than any record a buyer owns, and
                 // every one of them happened inside a settlement batch with nothing
                 // recording it until W18. Absent history renders the empty label,

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

@@ -2,5 +2,13 @@ import React from "react";
 import type { StackedViewShellProps } from "../../../ui";
 export interface AdminCarouselGroupEditorViewProps extends Omit<StackedViewShellProps, "sections"> {
     onCreated?: (id: string) => void;
+    /**
+     * Present → edit that carousel; absent → create a new one.
+     *
+     * This component was create-only, which is why `/admin/carousels/[id]/edit`
+     * did not exist. Same `featureId?`-style seam `AdminFeatureEditorView` uses.
+     */
+    carouselId?: string;
+    onSaved?: (id: string) => void;
 }
-export declare function AdminCarouselGroupEditorView({ onCreated, ...rest }: AdminCarouselGroupEditorViewProps): React.JSX.Element;
+export declare function AdminCarouselGroupEditorView({ onCreated, carouselId, onSaved, ...rest }: AdminCarouselGroupEditorViewProps): React.JSX.Element;

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

@@ -3,6 +3,7 @@ import { jsx as _jsx, Fragment as _Fragment, jsxs as _jsxs } from "react/jsx-run
 import React from "react";
 import { z } from "zod";
 import { useRouter } from "next/navigation";
+import { useQuery } from "@tanstack/react-query";
 import { Button, Form, Row, StackedViewShell, useToast } from "../../../ui";
 import { FieldInput, FieldSelect, FormErrorSummary } from "../../../ui/forms";
 import { useApiMutation } from "@mohasinac/appkit/client";
@@ -13,28 +14,59 @@ const carouselGroupFormSchema = z.object({
     name: z.string().min(1, "Name is required").max(200),
     status: z.enum(["active", "draft"]),
 });
-export function AdminCarouselGroupEditorView({ onCreated, ...rest }) {
+export function AdminCarouselGroupEditorView({ onCreated, carouselId, onSaved, ...rest }) {
     const { showToast } = useToast();
     const router = useRouter();
+    const isEdit = Boolean(carouselId);
     const [name, setName] = React.useState("");
     const [status, setStatus] = React.useState("draft");
+    // Seed from the single-item GET, never from a cached list row — a list
+    // projection is narrower than the document, and saving would write the
+    // missing fields back as their defaults (Root Cause #38).
+    const existing = useQuery({
+        queryKey: ["admin", "carousel", carouselId],
+        queryFn: () => apiClient.get(ADMIN_ENDPOINTS.CAROUSELS_BY_ID(carouselId)),
+        enabled: isEdit,
+    });
+    React.useEffect(() => {
+        const doc = existing.data?.data;
+        if (!doc)
+            return;
+        setName(doc.name ?? "");
+        setStatus(doc.status === "active" ? "active" : "draft");
+    }, [existing.data]);
     const createMutation = useApiMutation({
-        mutationFn: () => apiClient.post(ADMIN_ENDPOINTS.CAROUSELS, { name, status }),
+        /*
+         * PATCH on the edit path, not PUT. `/api/admin/carousels/[id]` exports
+         * GET/PATCH/DELETE — copying `AdminFeatureEditorView`'s `apiClient.put`
+         * here would have been a 405, which is exactly what
+         * `audit-client-verb-match` exists to catch.
+         */
+        mutationFn: () => isEdit
+            ? apiClient.patch(ADMIN_ENDPOINTS.CAROUSELS_BY_ID(carouselId), { name, status })
+            : apiClient.post(ADMIN_ENDPOINTS.CAROUSELS, { name, status }),
         onSuccess: (res) => {
-            const id = res?.data?.id;
-            showToast("Carousel created.", "success");
-            if (id) {
-                if (onCreated)
-                    onCreated(id);
+            const id = res?.data?.id ?? carouselId;
+            showToast(isEdit ? "Carousel updated." : "Carousel created.", "success");
+            if (!id)
+                return;
+            if (isEdit) {
+                if (onSaved)
+                    onSaved(id);
                 else
                     router.push(String(ROUTES.ADMIN.CAROUSEL_DETAIL(id)));
+                return;
             }
+            if (onCreated)
+                onCreated(id);
+            else
+                router.push(String(ROUTES.ADMIN.CAROUSEL_DETAIL(id)));
         },
         onError: (err) => {
-            showToast(err?.message ?? "Failed to create carousel.", "error");
+            showToast(err?.message ?? `Failed to ${isEdit ? "update" : "create"} carousel.`, "error");
         },
     });
-    return (_jsx(StackedViewShell, { portal: "admin", ...rest, title: "New Named Carousel", sections: [
+    return (_jsx(StackedViewShell, { portal: "admin", ...rest, title: isEdit ? "Edit Carousel" : "New Named Carousel", sections: [
             _jsx(Form, { schema: carouselGroupFormSchema, onSubmit: (e) => e.preventDefault(), spacing: "md", children: ({ setFieldError, clearErrors }) => (_jsxs(_Fragment, { children: [_jsx(FieldInput, { name: "name", label: "Carousel name", value: name, onChange: (v) => setName(v), required: true, placeholder: "e.g. Homepage Hero" }), _jsx(FieldSelect, { name: "status", label: "Status", value: status, onChange: (v) => setStatus(v), options: [
                                 { value: "draft", label: "Draft" },
                                 { value: "active", label: "Active" },
@@ -45,6 +77,6 @@ export function AdminCarouselGroupEditorView({ onCreated, ...rest }) {
                                         return;
                                     }
                                     createMutation.mutate();
-                                }, children: "Create carousel" }) })] })) }, "carousel-group-form"),
+                                }, children: isEdit ? "Save changes" : "Create carousel" }) })] })) }, "carousel-group-form"),
         ] }));
 }

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

@@ -145,5 +145,5 @@ export function AdminCarouselView({ children, onBulkDelete, ...props }) {
                 ] })),
         },
     ];
-    return (_jsxs(Div, { className: "min-h-screen", children: [_jsx(ListingToolbar, { filterCount: activeFilterCount, onFiltersClick: openFilters, searchValue: searchInput, searchPlaceholder: "Search slide titles", onSearchChange: setSearchInput, onSearchCommit: commitSearch, showTableView: true, view: view, onViewChange: (v) => setView(v), onResetAll: resetAll, hasActiveState: hasActiveState }), totalPages > 1 && (_jsx(StickyToolbar, { offset: "header+pagination", tone: "translucent", border: true, padding: "toolbar", children: _jsx(Row, { justify: "center", children: _jsx(Pagination, { currentPage: currentPage, totalPages: totalPages, onPageChange: (p) => table.setPage(p) }) }) })), selection.selectedCount > 0 && bulkActions.length > 0 && (_jsx(BulkActionBar, { selectedCount: selection.selectedCount, actions: bulkActions, onClearSelection: selection.clearSelection })), _jsxs(Div, { paddingX: "x-sm-md", padding: "y-md", children: [errorMessage && (_jsx(Div, { textSize: "sm", className: "mb-4 border border-error/20", color: "error", surface: "danger-surface", padding: "inline", rounded: "xl", children: errorMessage })), _jsx(DataTable, { rows: displayRows, columns: columns, isLoading: isLoading, emptyLabel: "No carousel slides \u2014 use 'New slide' to add one", rowHrefTemplate: "/admin/carousel/{id}/edit" })] }), _jsx(ListingFilterDrawer, { open: filterOpen, onClose: () => setFilterOpen(false), onApply: applyFilters, onClear: clearFilters, activeCount: activeFilterCount, children: _jsxs(Stack, { gap: "sm", children: [_jsx(Text, { className: "tracking-widest", color: "muted", size: "xs", weight: "semibold", transform: "uppercase", children: "Status" }), _jsx(Row, { wrap: true, gap: "sm", children: [{ label: "All", value: "" }, { label: "Active", value: "true" }, { label: "Inactive", value: "false" }].map((opt) => (_jsx(Button, { variant: "ghost", type: "button", onClick: () => setPendingFilters((p) => ({ ...p, active: opt.value })), className: `rounded-full px-[var(--appkit-space-3)] py-[var(--appkit-space-1)] text-[length:var(--appkit-text-xs)] font-medium border transition-colors min-h-0 h-auto ${(pendingFilters.active || "") === opt.value ? "bg-primary text-white border-primary" : "border-[var(--appkit-color-border)] text-[var(--appkit-color-text-muted)] hover:bg-zinc-50 hover:bg-[var(--appkit-color-surface-elevated)]"}`, children: opt.label }, opt.label))) })] }) }), deleteTargetId && (_jsx(ConfirmDeleteModal, { isOpen: true, title: "Delete Carousel Slide", message: "Delete this carousel slide? This cannot be undone.", onConfirm: () => { deleteMutation.mutate(deleteTargetId); setDeleteTargetId(null); }, onClose: () => setDeleteTargetId(null), isDeleting: deleteMutation.isPending }))] }));
+    return (_jsxs(Div, { className: "min-h-screen", children: [_jsx(ListingToolbar, { filterCount: activeFilterCount, onFiltersClick: openFilters, searchValue: searchInput, searchPlaceholder: "Search slide titles", onSearchChange: setSearchInput, onSearchCommit: commitSearch, showTableView: true, view: view, onViewChange: (v) => setView(v), onResetAll: resetAll, hasActiveState: hasActiveState }), totalPages > 1 && (_jsx(StickyToolbar, { offset: "header+pagination", tone: "translucent", border: true, padding: "toolbar", children: _jsx(Row, { justify: "center", children: _jsx(Pagination, { currentPage: currentPage, totalPages: totalPages, onPageChange: (p) => table.setPage(p) }) }) })), selection.selectedCount > 0 && bulkActions.length > 0 && (_jsx(BulkActionBar, { selectedCount: selection.selectedCount, actions: bulkActions, onClearSelection: selection.clearSelection })), _jsxs(Div, { paddingX: "x-sm-md", padding: "y-md", children: [errorMessage && (_jsx(Div, { textSize: "sm", className: "mb-4 border border-error/20", color: "error", surface: "danger-surface", padding: "inline", rounded: "xl", children: errorMessage })), _jsx(DataTable, { rows: displayRows, columns: columns, isLoading: isLoading, emptyLabel: "No carousel slides \u2014 use 'New slide' to add one", rowHrefTemplate: "/admin/carousel/{id}/edit" })] }), _jsx(ListingFilterDrawer, { open: filterOpen, onClose: () => setFilterOpen(false), onApply: applyFilters, onClear: clearFilters, activeCount: activeFilterCount, children: _jsxs(Stack, { gap: "sm", children: [_jsx(Text, { className: "tracking-widest", color: "muted", size: "xs", weight: "semibold", transform: "uppercase", children: "Status" }), _jsx(Row, { wrap: true, gap: "sm", children: [{ label: "All", value: "" }, { label: "Active", value: "true" }, { label: "Inactive", value: "false" }].map((opt) => (_jsx(Button, { variant: "ghost", type: "button", onClick: () => setPendingFilters((p) => ({ ...p, active: opt.value })), className: `rounded-full px-[var(--appkit-space-3)] py-[var(--appkit-space-1)] text-[length:var(--appkit-text-xs)] font-medium border transition-colors min-h-0 h-auto ${(pendingFilters.active || "") === opt.value ? "bg-primary text-white border-primary" : "border-[var(--appkit-color-border)] text-[var(--appkit-color-text-muted)] hover:bg-surface-hover"}`, children: opt.label }, opt.label))) })] }) }), deleteTargetId && (_jsx(ConfirmDeleteModal, { isOpen: true, title: "Delete Carousel Slide", message: "Delete this carousel slide? This cannot be undone.", onConfirm: () => { deleteMutation.mutate(deleteTargetId); setDeleteTargetId(null); }, onClose: () => setDeleteTargetId(null), isDeleting: deleteMutation.isPending }))] }));
 }

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

@@ -14,7 +14,7 @@ export function AdminContentGuideView() {
                 },
                 {
                     Icon: BookOpen, title: "FAQs",
-                    content: (_jsxs(Ul, { className: GC.listMuted, children: [_jsxs(Li, { children: [_jsx(Span, { weight: "bold", children: "Categories" }), " and where they appear: Shipping (help + footer), Returns (help), Payments (help), Auctions (help), Pre-orders (help). Set ", _jsx(Code, { size: "xs", padding: "xs", rounded: "default", surface: "subtle", children: "showOnHomepage: true" }), " for up to 5 FAQs shown on the public home page."] }), _jsxs(Li, { children: [_jsx(Span, { weight: "bold", children: "searchTokens[]" }), ": Must be manually filled \u2014 drives the FAQ search feature. Include synonyms, misspellings, and related keywords."] }), _jsxs(Li, { children: [_jsx(Span, { weight: "bold", children: "priority + order" }), ": ", _jsx(Code, { size: "xs", padding: "xs", rounded: "default", surface: "subtle", children: "priority" }), " sorts FAQs within their category; ", _jsx(Code, { size: "xs", padding: "xs", rounded: "default", surface: "subtle", children: "order" }), " is the global absolute position. ", _jsx(Code, { size: "xs", padding: "xs", rounded: "default", surface: "subtle", children: "isPinned: true" }), " floats the FAQ above non-pinned items in its category."] })] })),
+                    content: (_jsxs(Ul, { className: GC.listMuted, children: [_jsxs(Li, { children: [_jsx(Span, { weight: "bold", children: "Categories" }), " and where they appear: Shipping (help + footer), Returns (help), Payments (help), Auctions (help), Pre-orders (help). Set ", _jsx(Code, { size: "xs", padding: "xs", rounded: "default", surface: "subtle", children: "showOnHomepage: true" }), " for up to 5 FAQs shown on the public home page."] }), _jsxs(Li, { children: [_jsx(Span, { weight: "bold", children: "Search" }), ": handled automatically \u2014 search terms are derived from the question, answer, category and tags when you save. There is nothing to fill in."] }), _jsxs(Li, { children: [_jsx(Span, { weight: "bold", children: "priority + order" }), ": ", _jsx(Code, { size: "xs", padding: "xs", rounded: "default", surface: "subtle", children: "priority" }), " sorts FAQs within their category; ", _jsx(Code, { size: "xs", padding: "xs", rounded: "default", surface: "subtle", children: "order" }), " is the global absolute position. ", _jsx(Code, { size: "xs", padding: "xs", rounded: "default", surface: "subtle", children: "isPinned: true" }), " floats the FAQ above non-pinned items in its category."] })] })),
                 },
                 {
                     Icon: Layout, title: "Carousel Slides",

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

@@ -19,7 +19,7 @@ const __O = {
 };
 const PILL_BASE_CLASS = "inline-flex items-center gap-[var(--appkit-space-1-5)] rounded-full border px-[var(--appkit-space-3)] py-[var(--appkit-space-1-5)] text-[length:var(--appkit-text-xs)] font-medium cursor-pointer transition-colors";
 const PILL_CHECKED_CLASS = "bg-primary text-white border-primary";
-const PILL_UNCHECKED_CLASS = "border-[var(--appkit-color-border)] text-[var(--appkit-color-text-muted)] hover:bg-zinc-50 hover:bg-[var(--appkit-color-surface-elevated)]";
+const PILL_UNCHECKED_CLASS = "border-[var(--appkit-color-border)] text-[var(--appkit-color-text-muted)] hover:bg-surface-hover";
 const FIELD_LABEL_CLASS = "text-[length:var(--appkit-text-sm)] font-medium text-[var(--appkit-color-text-muted)]";
 const DELETE_CONFIRM_TEXT = "Delete this feature? It will fail if any product still references it.";
 const TOAST = {

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

@@ -8,12 +8,13 @@ import { normalizeError } from "../../../errors/normalize";
 import { sortBy } from "@mohasinac/appkit/client";
 import React, { useState } from "react";
 import { ListingLayout } from "../../../ui";
-import { Button, Row, SideDrawer, Stack, Text, useToast, } from "../../../ui";
+import { Button, Row, SideDrawer, Stack, Text, TextLink, useToast, } from "../../../ui";
 import { toRecordArray, toRelativeDate, toStringValue, } from "../hooks/useAdminListingData";
 import { DataListingView } from "./DataListingView";
 import { ProductInlineSelect } from "../../seller/components/ProductInlineSelect";
 import { apiClient } from "../../../http";
 import { ADMIN_ENDPOINTS } from "../../../constants/api-endpoints";
+import { ROUTES } from "../../../next/routing/route-map";
 function ReassignProductsDrawer({ state, onClose, onSaved, }) {
     const [productIds, setProductIds] = useState(state.currentProductIds);
     const [saving, setSaving] = useState(false);
@@ -75,10 +76,21 @@ export function AdminGroupedListingsView({ children, ...props }) {
         })),
         getTotal: (response, mappedRows) => typeof response.total === "number" ? response.total : mappedRows.length,
         buildFilters: () => "",
+        /*
+         * Admin had NO create path at all — there was no POST route either,
+         * so this could not have been wired before now.
+         *
+         * `toolbarExtra` rather than `primaryAction`: the latter takes an
+         * `onClick(panel)` for opening a create DRAWER, and this create is a
+         * page. Same shape `AdminAdsView` uses, and `audit-create-affordance`
+         * counts both — which is exactly why it has to, or four views with a
+         * perfectly good create link would read as having none.
+         */
+        toolbarExtra: (_jsx(TextLink, { variant: "bare", href: String(ROUTES.ADMIN.GROUPED_LISTINGS_NEW), rounded: "md", paddingX: "sm", size: "sm", weight: "medium", layout: "inline-flex", align: "center", className: "h-9 bg-[var(--appkit-color-surface)] text-[var(--appkit-color-text)]", children: "New group" })),
         // Mirrors the row action button's "Reassign products" drawer so the
         // row itself is clickable, not just the button.
         onRowClick: (row) => openReassign(row),
-        renderRowActions: (row) => (_jsx(Button, { size: "sm", variant: "outline", onClick: () => openReassign(row), children: "Reassign products" })),
+        renderRowActions: (row) => (_jsxs(Row, { gap: "xs", children: [_jsx(TextLink, { href: String(ROUTES.ADMIN.GROUPED_LISTINGS_EDIT(row.id)), children: "Edit" }), _jsx(Button, { size: "sm", variant: "outline", onClick: () => openReassign(row), children: "Reassign products" })] })),
     }), [refreshKey, openReassign]);
     if (React.Children.count(children) > 0) {
         return (_jsx(ListingLayout, { portal: "admin", ...props, children: children }));

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

@@ -17,7 +17,7 @@ const __O = {
 function NavLink({ item, isActive, onClick }) {
     return (_jsxs(Link, { href: item.href, onClick: onClick, className: `flex items-center justify-end gap-[var(--appkit-space-2-5)] rounded-lg px-[var(--appkit-space-3)] py-[var(--appkit-space-2)] text-[0.8125rem] font-medium leading-tight transition-colors ${isActive
             ? "bg-[var(--appkit-color-surface)] bg-[var(--appkit-color-surface-elevated)] text-[var(--appkit-color-text)]"
-            : "text-[var(--appkit-color-text-muted)] hover:bg-zinc-50 hover:bg-[var(--appkit-color-surface-elevated)]/60 hover:text-zinc-800 hover:text-[var(--appkit-color-text-muted)]"}`, children: [item.icon && _jsx(Span, { size: "base", className: "shrink-0 opacity-60", children: item.icon }), _jsx(Span, { className: "truncate", children: item.label })] }));
+            : "text-[var(--appkit-color-text-muted)] hover:bg-surface-hover hover:text-zinc-800 hover:text-[var(--appkit-color-text-muted)]"}`, children: [item.icon && _jsx(Span, { size: "base", className: "shrink-0 opacity-60", children: item.icon }), _jsx(Span, { className: "truncate", children: item.label })] }));
 }
 function GroupsContent({ groups, activePath, onItemClick, }) {
     // Accordion — only one group open at a time. Initial pick: the group

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

@@ -52,7 +52,7 @@ function AdminCardItem({ row, view, selected, onToggleSelect, onRowClick, render
     };
     if (view === "list") {
         return (_jsxs(Row, { gap: "sm", className: [
-                "px-[var(--appkit-space-4)] py-[var(--appkit-space-3)] cursor-pointer transition-colors hover:bg-zinc-50 hover:bg-[var(--appkit-color-surface-elevated)]/50",
+                "px-[var(--appkit-space-4)] py-[var(--appkit-space-3)] cursor-pointer transition-colors hover:bg-surface-hover",
                 selected ? "bg-primary-50/40 dark:bg-primary-900/10" : "",
             ].filter(Boolean).join(" "), onClick: handleClick, role: onRowClick ? "button" : undefined, children: [onToggleSelect && (_jsx(Div, { "data-no-row-click": true, className: "shrink-0", onClick: (e) => { e.stopPropagation(); onToggleSelect(row.id); }, children: _jsx(Checkbox, { bare: true, checked: selected, onChange: () => onToggleSelect(row.id), className: "h-4 w-4 rounded border-zinc-300 text-primary accent-primary cursor-pointer", "aria-label": `Select ${row.primary}` }) })), _jsx(RowAvatar, { image: row.image, alt: row.primary, icon: resourceIcon }), _jsxs(Stack, { gap: "none", className: "flex-1 min-w-0", children: [_jsx(Text, { size: "sm", weight: "semibold", className: "truncate", color: "primary", children: row.primary }), _jsx(Text, { size: "xs", color: "muted", className: "truncate", children: row.secondary }), row.barcodeId && (_jsx(Text, { size: "xs", color: "faint", className: "truncate font-mono", children: row.barcodeId }))] }), flags.length > 0 && (_jsx(Row, { gap: "xs", className: "hidden sm:flex shrink-0", children: flags.map(({ key, label, tone }) => (_jsx(Span, { padding: "pill-2xs", weight: "medium", className: `inline-flex text-[10px] ${STATUS_TONE_CLASSES[tone]}`, rounded: "full", children: label }, key))) })), _jsx(StatusBadge, { status: row.status }), _jsx(Span, { size: "xs", color: "muted", className: "hidden sm:block shrink-0 w-24", align: "end", children: row.updatedAt }), renderRowActions && (_jsx(Div, { "data-no-row-click": true, className: "shrink-0", onClick: (e) => e.stopPropagation(), children: renderRowActions(row) }))] }));
     }
@@ -60,7 +60,7 @@ function AdminCardItem({ row, view, selected, onToggleSelect, onRowClick, render
             "border overflow-hidden cursor-pointer transition-all hover:shadow-md hover:-translate-y-0.5",
             selected
                 ? "border-primary ring-1 ring-primary/20 bg-primary-50/30 dark:bg-primary-900/10"
-                : "border-zinc-100 border-[var(--appkit-color-border)] bg-[var(--appkit-color-surface)]",
+                : " border-[var(--appkit-color-border)] bg-[var(--appkit-color-surface)]",
         ].filter(Boolean).join(" "), onClick: handleClick, role: onRowClick ? "button" : undefined, children: [onToggleSelect && (_jsxs(Row, { gap: "xs", "data-no-row-click": true, paddingY: "t-sm", padding: "x-sm", onClick: (e) => { e.stopPropagation(); onToggleSelect(row.id); }, children: [_jsx(Checkbox, { bare: true, checked: selected, onChange: () => onToggleSelect(row.id), className: "h-4 w-4 rounded border-zinc-300 text-primary accent-primary cursor-pointer", "aria-label": `Select ${row.primary}` }), flags.length > 0 && (_jsx(Row, { gap: "xs", wrap: true, children: flags.map(({ key, label, tone }) => (_jsx(Span, { padding: "pill-2xs", weight: "medium", className: `inline-flex text-[10px] ${STATUS_TONE_CLASSES[tone]}`, rounded: "full", children: label }, key))) }))] })), _jsxs(Stack, { gap: "xs", className: `${__P.p3}.5`, children: [_jsxs(Row, { gap: "sm", align: "center", children: [_jsx(RowAvatar, { image: row.image, alt: row.primary, icon: resourceIcon, size: "12" }), _jsxs(Stack, { gap: "none", className: "min-w-0 flex-1", children: [_jsx(Text, { size: "sm", weight: "semibold", className: "line-clamp-2 leading-snug", color: "primary", children: row.primary }), _jsx(Text, { size: "xs", color: "muted", className: "truncate", children: row.secondary })] })] }), row.barcodeId && (_jsx(Text, { size: "xs", color: "faint", className: "truncate font-mono", children: row.barcodeId })), _jsxs(Row, { justify: "between", gap: "xs", children: [_jsx(StatusBadge, { status: row.status }), _jsx(Span, { color: "muted", className: "text-[11px] shrink-0", children: row.updatedAt })] }), renderRowActions && (_jsx(Row, { "data-no-row-click": true, justify: "end", gap: "xs", className: "border-t border-[var(--appkit-color-border)] -mx-[var(--appkit-space-3)] -mb-[var(--appkit-space-3)] mt-[var(--appkit-space-1)] px-[var(--appkit-space-3)] py-[var(--appkit-space-2)]", onClick: (e) => e.stopPropagation(), children: renderRowActions(row) }))] })] }));
 }
 export function AdminViewCards({ rows, view, isLoading, emptyLabel = "No items found", onRowClick, selectedIdSet, onToggleSelect, renderRowActions, resourceIcon, }) {

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

@@ -39,7 +39,7 @@ function SelectableRow({ row, columns, isSelected, onToggle, renderRowActions, o
             : undefined;
     const handleKeyDown = handleClick
         ? (event) => {
-            if (event.key === "Enter" || event.key === " ")
+            if (event.key === "Enter" || event.key === "")
                 handleClick();
         }
         : undefined;
@@ -54,5 +54,5 @@ export function DataTable({ columns: columnsProp, rows, isLoading, sortKey, sort
     const allRowsSelected = selectionEnabled && rows.length > 0 && rows.every((r) => selectedIds?.has(r.id));
     return (_jsxs(Div, { surface: "card", className: `${__O.hidden}`, children: [_jsx(Div, { className: `${__O.xAuto}`, children: _jsxs(Table, { "data-testid": "data-table", size: "sm", children: [_jsx(Thead, { children: _jsxs(Tr, { className: "bg-[var(--appkit-color-bg)] dark:bg-[var(--appkit-color-surface-elevated)]", border: "default", children: [selectionEnabled && (_jsx(Th, { padding: "xs-3", scope: "col", className: "w-10", children: onToggleSelectAll && (_jsx(Checkbox, { bare: true, "data-testid": "select-all-checkbox", "aria-label": allRowsSelected ? "Deselect all" : "Select all", checked: allRowsSelected, onChange: () => onToggleSelectAll(!allRowsSelected), className: "h-4 w-4 rounded border-[var(--appkit-color-border)] accent-zinc-900 dark:accent-zinc-100" })) })), columns.map((col) => (_jsxs(Th, { scope: "col", onClick: col.sortable && onSort ? () => onSort(col.key) : undefined, className: `text-left text-[var(--appkit-color-text)] ${col.sortable && onSort ? "cursor-pointer select-none hover:text-primary" : ""} ${COLUMN_PRIORITY_CLASS[resolveColumnPriority(col)]} ${col.className ?? ""}`, padding: "md", weight: "semibold", children: [col.header, col.sortable && sortKey === col.key && (_jsx(Span, { className: "ml-1", children: sortDir === "asc" ? "↑" : "↓" }))] }, col.key))), renderRowActions && _jsx(Th, { padding: "xs-3", scope: "col", className: "w-12" })] }) }), _jsx(Tbody, { children: isLoading ? (Array.from({ length: 5 }).map((_, i) => (_jsxs(Tr, { border: "default", children: [selectionEnabled && _jsx(Td, { padding: "xs-3", className: "w-10" }), columns.map((col) => (_jsx(Td, { padding: "md", className: COLUMN_PRIORITY_CLASS[resolveColumnPriority(col)], children: _jsx(Div, { className: "h-4 w-full animate-pulse bg-neutral-200", rounded: "default" }) }, col.key)))] }, i)))) : rows.length === 0 ? (_jsx(Tr, { children: _jsx(Td, { colSpan: columns.length + (selectionEnabled ? 1 : 0) + (renderRowActions ? 1 : 0), align: "center", color: "faint", paddingX: "md", paddingY: "md", children: emptyLabel }) })) : (rows.map((row) => (_jsx(SelectableRow, { row: row, columns: columns, isSelected: selectedIds?.has(row.id) ?? false, onToggle: onToggleSelect, renderRowActions: renderRowActions, onRowClick: onRowClick, rowHref: typeof rowHrefTemplate === "function"
                                     ? rowHrefTemplate(row)
-                                    : rowHrefTemplate?.replace("{id}", encodeURIComponent(row.id)), selectionEnabled: selectionEnabled }, row.id)))) })] }) }), totalPages > 1 && onPageChange && (_jsx(Row, { className: "border-t border-neutral-200", padding: "inline", align: "center", justify: "end", gap: "sm", children: Array.from({ length: totalPages }, (_, i) => i + 1).map((p) => (_jsx(Button, { onClick: () => onPageChange(p), variant: p === currentPage ? "primary" : "ghost", size: "sm", className: `h-8 w-8 rounded text-[length:var(--appkit-text-xs)] font-medium transition ${p === currentPage ? "bg-neutral-900 text-white" : "text-[var(--appkit-color-text-muted)] text-[var(--appkit-color-text-muted)] hover:bg-[var(--appkit-color-surface)] hover:bg-[var(--appkit-color-surface-elevated)]"}`, children: p }, p))) }))] }));
+                                    : rowHrefTemplate?.replace("{id}", encodeURIComponent(row.id)), selectionEnabled: selectionEnabled }, row.id)))) })] }) }), totalPages > 1 && onPageChange && (_jsx(Row, { className: "border-t border-neutral-200", padding: "inline", align: "center", justify: "end", gap: "sm", children: Array.from({ length: totalPages }, (_, i) => i + 1).map((p) => (_jsx(Button, { onClick: () => onPageChange(p), variant: p === currentPage ? "primary" : "ghost", size: "sm", className: `h-8 w-8 rounded text-[length:var(--appkit-text-xs)] font-medium transition ${p === currentPage ? "bg-neutral-900 text-white" : "text-[var(--appkit-color-text-muted)] text-[var(--appkit-color-text-muted)] hover:bg-surface-hover"}`, children: p }, p))) }))] }));
 }

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

@@ -254,11 +254,17 @@ export const DEFAULT_SITE_SETTINGS_DATA = {
         codDepositPercent: 10,
         codHandlingFeeMin: 200,
         codHandlingFeePercent: 10,
-        whatsappNotifyFeeEnabled: false,
+        // The three buyer add-ons ship ENABLED. They were `false`, which made
+        // StoreAddonsPicker return null on both the cart and checkout, so a fresh
+        // install had no add-ons at all and no indication why. `undefined` still
+        // means disabled everywhere (computeWhatsAppNotifyFee et al. read
+        // `if (!rates.XEnabled) return 0`) — only the DEFAULT changed, so an
+        // existing siteSettings document keeps whatever it already stored.
+        whatsappNotifyFeeEnabled: true,
         whatsappNotifyFee: 10,
-        giftWrapFeeEnabled: false,
+        giftWrapFeeEnabled: true,
         giftWrapFee: 49,
-        shipmentProtectionFeeEnabled: false,
+        shipmentProtectionFeeEnabled: true,
         shipmentProtectionFeePercent: 2,
         shipmentProtectionFeeMin: 30,
         sellerShippingFixed: 0,