@mohasinac/appkit 3.3.3 → 3.3.4

package/dist/ui/columns/column-renderers.js

@@ -7,7 +7,7 @@
  * For JSX renderers (badge, thumbnail, action buttons) see `column-renderers.tsx`.
  */
 import { formatCurrency } from "../../utils/number.formatter";
-import { getDefaultCurrency, } from "../../core/baseline-resolver";
+import { getDefaultCurrency } from "../../core/baseline-resolver";
 /** Render a boolean as "Yes" / "No" (or custom labels). */
 export function renderBoolean(value, opts) {
     const trueLabel = opts?.trueLabel ?? "Yes";

package/dist/ui/components/Accordion.js

@@ -15,6 +15,16 @@ function renderLegacyAccordion({ title, children, open, className = "", titleCla
     return (_jsxs("details", { open: open, className: `appkit-accordion appkit-accordion--legacy ${className}`, children: [_jsxs("summary", { className: `appkit-accordion__summary ${titleClassName}`, children: [title, _jsx("span", { className: "appkit-accordion__chevron", "aria-hidden": "true", children: "\u25BE" })] }), _jsx("div", { className: `appkit-accordion__content ${contentClassName}`, "data-section": "accordion-div-447", children: children })] }));
 }
 export function Accordion({ title, children, open = false, className = "", titleClassName = "", contentClassName = "", type = "single", defaultValue, value, onChange, }) {
+    // useMemo/useState must run unconditionally on every render — they were
+    // previously declared after the legacy-mode early return below, which
+    // skipped both hook calls whenever `title` was passed, violating the
+    // Rules of Hooks. Their result is simply unused in legacy mode.
+    const initialOpen = useMemo(() => {
+        if (value !== undefined)
+            return toArray(value);
+        return toArray(defaultValue);
+    }, [defaultValue, value]);
+    const [internalOpenValues, setInternalOpenValues] = useState(initialOpen);
     // Backward-compatible simple details mode.
     if (title !== undefined) {
         return renderLegacyAccordion({
@@ -26,12 +36,6 @@ export function Accordion({ title, children, open = false, className = "", title
             contentClassName,
         });
     }
-    const initialOpen = useMemo(() => {
-        if (value !== undefined)
-            return toArray(value);
-        return toArray(defaultValue);
-    }, [defaultValue, value]);
-    const [internalOpenValues, setInternalOpenValues] = useState(initialOpen);
     const activeOpenValues = value !== undefined ? toArray(value) : internalOpenValues;
     const toggle = (itemValue) => {
         const nextOpenValues = type === "single"

package/dist/ui/components/ListingLayout.js

@@ -36,6 +36,12 @@ const DEFAULT_LABELS = {
     filterActiveCount: (count) => `${count} active`,
 };
 export function ListingLayout({ headerSlot, statusTabsSlot, filterContent, filterActiveCount = 0, onFilterApply, onFilterClear, filterTitle, activeFiltersSlot, resultCountSlot, searchSlot, sortSlot, viewToggleSlot, actionsSlot, selectedCount = 0, onClearSelection, bulkActionItems, toolbarPaginationSlot, paginationSlot, children, isDashboard, defaultSidebarOpen = false, filterPendingCount, className = "", loading = false, errorSlot, labels, portal, overlays, detailView, }) {
+    // useState must run unconditionally on every render — these were
+    // previously declared after the detail-view early return below, which
+    // skipped both hook calls whenever `detailView` was passed, violating the
+    // Rules of Hooks.
+    const [sidebarOpen, setSidebarOpen] = useState(defaultSidebarOpen);
+    const [mobileFilterOpen, setMobileFilterOpen] = useState(false);
     // Detail view short-circuit (folded from ListingViewShell)
     if (detailView) {
         return _jsx(_Fragment, { children: detailView });
@@ -43,8 +49,6 @@ export function ListingLayout({ headerSlot, statusTabsSlot, filterContent, filte
     // Portal-derived isDashboard default (folded from ListingViewShell)
     const effectiveIsDashboard = isDashboard ?? (portal === "admin" || portal === "seller");
     const l = { ...DEFAULT_LABELS, ...labels };
-    const [sidebarOpen, setSidebarOpen] = useState(defaultSidebarOpen);
-    const [mobileFilterOpen, setMobileFilterOpen] = useState(false);
     const hasFilter = Boolean(filterContent);
     const panelTitle = filterTitle ?? l.filtersTitle;
     const handleMobileApply = () => {

package/package.json

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