jamdesk 1.1.170 → 1.1.171

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jamdesk",
-  "version": "1.1.170",
+  "version": "1.1.171",
   "description": "CLI for Jamdesk — build, preview, and deploy documentation sites from MDX. Dev server with hot reload, 50+ components, OpenAPI support, AI search, and Mintlify migration",
   "keywords": [
     "jamdesk",

package/vendored/components/navigation/Sidebar.tsx

@@ -433,13 +433,28 @@ export function Sidebar({ config, layout = 'header-logo', tabsPosition: tabsPosi
   // Prevent body scroll when sidebar is open on mobile
   useBodyScrollLock(isOpen);
 
-  // Toggle group expansion; if expanding, navigate to first page. Top-level
-  // sections never visually collapse — for them the toggled key only gates
-  // prefetch (see renderGroupContents) while the click still navigates.
+  // Top-level section titles never collapse; clicking one navigates to the
+  // section's first page. The key is still marked expanded so the section's
+  // Links upgrade to prefetch (see renderGroupContents).
   // The router.push must run AFTER the setState commits — calling it inside the
   // updater triggers "Cannot update a component (Router) while rendering Sidebar".
-  const handleGroupClick = useCallback((group: ResolvedGroup, key: string) => {
-    const willExpand = !expandedGroups.has(key);
+  const handleTopLevelGroupClick = useCallback((group: ResolvedGroup, key: string) => {
+    setExpandedGroups(prev => {
+      if (prev.has(key)) return prev;
+      const newSet = new Set(prev);
+      newSet.add(key);
+      return newSet;
+    });
+    const firstPagePath = findFirstPageInGroups([group]);
+    if (firstPagePath) {
+      handleButtonNav(`${linkPrefix}/${firstPagePath}`);
+    }
+  }, [linkPrefix, handleButtonNav]);
+
+  // Toggle a nested group's accordion. Expanding only reveals the pages —
+  // navigation happens when the visitor clicks a page link, never on the
+  // caret itself.
+  const handleGroupClick = useCallback((key: string) => {
     setExpandedGroups(prev => {
       const newSet = new Set(prev);
       if (prev.has(key)) {
@@ -449,13 +464,7 @@ export function Sidebar({ config, layout = 'header-logo', tabsPosition: tabsPosi
       }
       return newSet;
     });
-    if (willExpand) {
-      const firstPagePath = findFirstPageInGroups([group]);
-      if (firstPagePath) {
-        handleButtonNav(`${linkPrefix}/${firstPagePath}`);
-      }
-    }
-  }, [linkPrefix, expandedGroups, handleButtonNav]);
+  }, []);
 
   // Render the pages and nested groups inside a group. Split out from
   // renderGroup so a named group can gate its contents behind an expanded panel
@@ -574,7 +583,7 @@ export function Sidebar({ config, layout = 'header-logo', tabsPosition: tabsPosi
           {group.name && (
             <button
               type="button"
-              onClick={() => handleGroupClick(group, key)}
+              onClick={() => handleTopLevelGroupClick(group, key)}
               className="flex items-center justify-between w-full text-left mb-2 cursor-pointer"
             >
               <span className="text-sm font-semibold text-[var(--color-text-tertiary)] flex items-center gap-2">
@@ -606,7 +615,7 @@ export function Sidebar({ config, layout = 'header-logo', tabsPosition: tabsPosi
             type="button"
             aria-expanded={isExpanded}
             aria-controls={panelId}
-            onClick={() => handleGroupClick(group, key)}
+            onClick={() => handleGroupClick(key)}
             className={buttonClassName}
           >
             {group.icon && (