@fern-api/fdr-sdk 1.2.44-2a040afc4c → 1.2.44-dd364964d5

package/dist/js/navigation/ledger-root-builder.mjs

@@ -1813,6 +1813,9 @@ function buildSidebarRootChildren(ctx, scopeSegs, scopePrefix, changelogScopePre
   const hasRootSection = ordered.some(
     (s) => metaStr(s.seg.metadata, "type") === "section" && s.seg.section === "" && (metaNum(s.seg.metadata, "childIndex") == null || siblingBoundaries.has(s))
   );
+  const hasTrueRootWrapper = ordered.some(
+    (s) => metaStr(s.seg.metadata, "type") === "section" && s.seg.section === "" && metaNum(s.seg.metadata, "childIndex") == null
+  );
   const depth0SectionPaths = /* @__PURE__ */ new Set();
   for (const s of ordered) {
     if (metaStr(s.seg.metadata, "type") === "section" && metaNum(s.seg.metadata, "sidebarDepth") === 0) {
@@ -1847,7 +1850,15 @@ function buildSidebarRootChildren(ctx, scopeSegs, scopePrefix, changelogScopePre
     }
     if (type === "section" && sec === "" && hasRootSection) {
       const bounds = siblingBoundaries.get(s);
-      const ownedSegs = bounds != null ? scopeSegs.filter((seg) => seg.seg.sortOrder >= bounds.lo && seg.seg.sortOrder < bounds.hi) : scopeSegs;
+      let ownedSegs = bounds != null ? scopeSegs.filter((seg) => seg.seg.sortOrder >= bounds.lo && seg.seg.sortOrder < bounds.hi) : scopeSegs;
+      if (!hasTrueRootWrapper) {
+        ownedSegs = ownedSegs.filter((o) => {
+          if (o.seg.section === "") {
+            return true;
+          }
+          return metaNum(o.seg.metadata, "sidebarDepth") != null;
+        });
+      }
       const sectionChildren = buildSectionTree(ctx, ownedSegs, "", scopePrefix, changelogScopePrefix);
       const node = sectionNode(ctx, s, sectionChildren, scopePrefix);
       annotated.push({ node, pos: metaNum(s.seg.metadata, "childIndex"), isLeaf: false });
@@ -1856,7 +1867,7 @@ function buildSidebarRootChildren(ctx, scopeSegs, scopePrefix, changelogScopePre
     const depth = metaNum(s.seg.metadata, "sidebarDepth");
     const parent = parentSectionPath(sec);
     const isRootLevel = depth === 0 || parent === "" || !sectionPaths.has(parent) && nearestExistingAncestor(sec, sectionPaths) === "";
-    if (isRootLevel && hasRootSection && sec !== "" && depth !== 0) {
+    if (isRootLevel && hasRootSection && sec !== "" && (depth != null ? depth !== 0 : hasTrueRootWrapper)) {
       continue;
     }
     if (isRootLevel && depth != null && depth > 0 && depth0SectionPaths.has(sec)) {
@@ -1890,7 +1901,11 @@ function buildSidebarRootChildren(ctx, scopeSegs, scopePrefix, changelogScopePre
   const hasLeafPos = annotated.some((a) => a.isLeaf && a.pos != null);
   const hasSegmentPos = annotated.some((a) => !a.isLeaf && a.pos != null);
   const flat = hasLeafPos && hasSegmentPos ? annotated.map((a, seq) => ({ ...a, seq })).sort(
-    (x, y) => (x.pos ?? Number.MAX_SAFE_INTEGER) - (y.pos ?? Number.MAX_SAFE_INTEGER) || x.seq - y.seq
+    (x, y) => (x.pos ?? Number.MAX_SAFE_INTEGER) - (y.pos ?? Number.MAX_SAFE_INTEGER) || // When positions tie, prefer structural nodes (sections) over
+    // leaves (pages). This handles stale tabRoot nav entries whose
+    // displaySortOrder wasn't updated due to content-addressing
+    // (the segment hash doesn't include childPosition).
+    (x.isLeaf === y.isLeaf ? 0 : x.isLeaf ? 1 : -1) || x.seq - y.seq
   ).map((a) => a.node) : annotated.map((a) => a.node);
   return groupSidebarRootChildren(ctx, flat);
 }