@mohasinac/appkit 4.42.7 → 4.42.9

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

@@ -8,5 +8,15 @@ export interface AdminStoreEditorViewProps {
     currentIsVerified?: boolean;
     currentIsFeatured?: boolean;
     currentCapabilities?: string[];
+    /**
+     * 🛑 Required for the note to DISPLAY. Without it `adminNotes` was hardcoded
+     * `""` at both the initial-state and the reopen-reset sites, so a stored note
+     * could never appear — and because the payload sent `parsed.adminNotes ||
+     * undefined`, an always-empty box meant `"" || undefined === undefined`, the
+     * route skipped the key, and an existing note could never be CLEARED either.
+     * Verified on production 2026-10-02: the document held `adminNotes:
+     * "RT3-probe"` while `textarea[name="adminNotes"]` read `""`.
+     */
+    currentAdminNotes?: string;
 }
-export declare function AdminStoreEditorView({ open, onClose, storeId, storeName, currentStatus, currentIsVerified, currentIsFeatured, currentCapabilities, }: AdminStoreEditorViewProps): React.JSX.Element;
+export declare function AdminStoreEditorView({ open, onClose, storeId, storeName, currentStatus, currentIsVerified, currentIsFeatured, currentCapabilities, currentAdminNotes, }: AdminStoreEditorViewProps): React.JSX.Element;

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

@@ -69,10 +69,10 @@ const STATUS_OPTIONS = [
  * right that it is unreadable there.
  */
 function CapabilityPicker({ selected, onToggle, }) {
-    return (_jsxs(Stack, { gap: "sm", children: [_jsxs(Span, { size: "sm", weight: "medium", color: "muted", children: ["Capabilities", _jsxs(Span, { size: "xs", weight: "normal", className: "ml-2", color: "muted", children: ["(", selected.size, " active)"] })] }), _jsx(Div, { className: "divide-y divide-[var(--appkit-color-border)]", rounded: "xl", border: "default", children: CAPABILITY_GROUPS.map((group) => (_jsxs(Details, { className: "group", children: [_jsxs(Summary, { paddingX: "x-sm", paddingY: "y-xs", weight: "semibold", color: "muted", layout: "flex", align: "center", justify: "between", className: "uppercase tracking-wide hover:bg-[var(--appkit-color-bg)] transition-colors", children: [_jsx(Span, { size: "xs", children: group.label }), _jsxs(Span, { size: "xs", weight: "normal", className: "normal-case", color: "faint", children: [group.caps.filter((c) => selected.has(c.key)).length, "/", group.caps.length] })] }), _jsx(Div, { layout: "grid", paddingY: "y-xs-tall", className: "grid-cols-2 gap-x-2 gap-y-1.5", surface: "muted", padding: "x-sm", children: group.caps.map((cap) => (_jsxs("label", { className: "flex items-center gap-[var(--appkit-space-2)] cursor-pointer text-[length:var(--appkit-text-xs)] text-[var(--appkit-color-text-muted)]", children: [_jsx(Checkbox, { bare: true, checked: selected.has(cap.key), onChange: () => onToggle(cap.key), className: "h-3.5 w-3.5 rounded border-[var(--appkit-color-border)] accent-primary" }), cap.label] }, cap.key))) })] }, group.label))) })] }));
+    return (_jsxs(Stack, { gap: "sm", children: [_jsxs(Span, { size: "sm", weight: "medium", color: "muted", children: ["Capabilities", _jsxs(Span, { size: "xs", weight: "normal", className: "ml-2", color: "muted", children: ["(", selected.size, " active)"] })] }), _jsx(Div, { className: "divide-y divide-[var(--appkit-color-border)]", rounded: "xl", border: "default", children: CAPABILITY_GROUPS.map((group) => (_jsxs(Details, { className: "group", children: [_jsxs(Summary, { paddingX: "x-sm", paddingY: "y-xs", weight: "semibold", color: "muted", layout: "flex", align: "center", justify: "between", className: "uppercase tracking-wide hover:bg-[var(--appkit-color-bg)] transition-colors", children: [_jsx(Span, { size: "xs", children: group.label }), _jsxs(Span, { size: "xs", weight: "normal", className: "normal-case", color: "faint", children: [group.caps.filter((c) => selected.has(c.key)).length, "/", group.caps.length] })] }), _jsx(Div, { layout: "grid", paddingY: "y-xs-tall", className: "grid-cols-2 gap-x-2 gap-y-1.5", surface: "muted", padding: "x-sm", children: group.caps.map((cap) => (_jsx(Checkbox, { label: cap.label, checked: selected.has(cap.key), onChange: () => onToggle(cap.key), wrapperClassName: "flex items-center gap-[var(--appkit-space-2)] cursor-pointer text-[length:var(--appkit-text-xs)] text-[var(--appkit-color-text-muted)]" }, cap.key))) })] }, group.label))) })] }));
 }
 // --- Component ---------------------------------------------------------------
-export function AdminStoreEditorView({ open, onClose, storeId, storeName, currentStatus, currentIsVerified, currentIsFeatured, currentCapabilities, }) {
+export function AdminStoreEditorView({ open, onClose, storeId, storeName, currentStatus, currentIsVerified, currentIsFeatured, currentCapabilities, currentAdminNotes, }) {
     const queryClient = useQueryClient();
     const { showToast } = useToast();
     const [values, setValues] = React.useState(() => ({
@@ -81,7 +81,7 @@ export function AdminStoreEditorView({ open, onClose, storeId, storeName, curren
         isFeatured: currentIsFeatured ?? false,
         suspensionReason: "",
         capabilities: currentCapabilities ?? DEFAULT_CAPABILITIES,
-        adminNotes: "",
+        adminNotes: currentAdminNotes ?? "",
     }));
     const handleChange = React.useCallback((partial) => {
         setValues((prev) => Object.assign({}, prev, partial));
@@ -95,9 +95,9 @@ export function AdminStoreEditorView({ open, onClose, storeId, storeName, curren
             isFeatured: currentIsFeatured ?? false,
             suspensionReason: "",
             capabilities: currentCapabilities ?? DEFAULT_CAPABILITIES,
-            adminNotes: "",
+            adminNotes: currentAdminNotes ?? "",
         });
-    }, [open, currentStatus, currentIsVerified, currentIsFeatured, currentCapabilities]);
+    }, [open, currentStatus, currentIsVerified, currentIsFeatured, currentCapabilities, currentAdminNotes]);
     const saveMutation = useApiMutation({
         errorMessage: "Failed to update store.",
         mutationFn: async () => {
@@ -114,7 +114,9 @@ export function AdminStoreEditorView({ open, onClose, storeId, storeName, curren
             const parsed = adminStoreUpdateSchema.parse(values);
             await apiClient.patch(ADMIN_ENDPOINTS.STORE_BY_ID(storeId), {
                 storeStatus: parsed.storeStatus,
-                adminNotes: parsed.adminNotes || undefined,
+                // `?? ""`, NOT `|| undefined` — sending undefined makes the route skip
+                // the key, so clearing a note was impossible.
+                adminNotes: parsed.adminNotes ?? "",
                 isFeatured: parsed.isFeatured,
                 isVerified: parsed.isVerified,
                 suspensionReason: parsed.suspensionReason || undefined,

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

@@ -127,5 +127,15 @@ export function AdminStoresView({ children, ...props }) {
     };
     return (_jsxs(_Fragment, { children: [_jsx(DataListingView, { config: config }), _jsx(AdminStoreEditorView, { open: Boolean(editorRow), onClose: () => setEditorRow(null), storeId: editorRow?.id, storeName: editorRow?.primary, currentStatus: editorRow?.status?.toLowerCase(), currentIsVerified: Boolean(editorRow?._raw?.isVerified), currentIsFeatured: Boolean(editorRow?._raw?.isFeatured), currentCapabilities: Array.isArray(editorRow?._raw?.capabilities)
                     ? editorRow._raw.capabilities
+                    : undefined, 
+                /*
+                 * The list serializer DOES emit `adminNotes`
+                 * (src/app/api/admin/stores/route.ts), so the row already carries it —
+                 * it was simply never passed down, which is why the textbox could not
+                 * display a stored note. Root Cause #38's shape, one hop later: the
+                 * field survives the API and dies at the prop.
+                 */
+                currentAdminNotes: typeof editorRow?._raw?.adminNotes === "string"
+                    ? editorRow._raw.adminNotes
                     : undefined })] }));
 }

package/dist/utils/media-url.js

@@ -129,6 +129,23 @@ const OWN_STORAGE_BUCKET = process.env.NEXT_PUBLIC_FIREBASE_STORAGE_BUCKET ?? ""
 export function resolveMediaUrl(url, opts) {
     if (!url)
         return undefined;
+    /*
+     * 🛑 Same type-guard as `isStoredMediaRef`, and for the same reason: this is a
+     * RENDER-path helper called from `MediaImage`, every avatar, every logo and
+     * every card, with whatever the document happens to hold. `!url` catches
+     * null/undefined/""; an object or array is truthy and falls straight through
+     * to `.startsWith`, throwing `TypeError: e.startsWith is not a function` and
+     * taking out the whole subtree it was rendering.
+     *
+     * Returning undefined is the documented behaviour for a falsy input and is the
+     * right answer here too — the caller renders its placeholder instead of
+     * crashing. The warn names the shape so the real caller stays findable.
+     */
+    if (typeof url !== "string") {
+        // eslint-disable-next-line no-console -- render path; a throw here kills the subtree
+        console.warn("[resolveMediaUrl] expected a string, received", Object.prototype.toString.call(url));
+        return undefined;
+    }
     if (url.startsWith(PROXY_PREFIX))
         return url;
     // A blob:/data: URI is only ever valid in the tab that created it (e.g. a

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mohasinac/appkit",
-  "version": "4.42.7",
+  "version": "4.42.9",
   "license": "MIT",
   "publishConfig": {
     "access": "public"