@fern-api/fdr-sdk 1.2.44-ef8582fd87 → 1.2.45-1a67633024

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

@@ -1306,13 +1306,19 @@ function mergeChildrenByPosition(ctx, nav, sectionChildren) {
 function sectionNode(ctx, scoped, sectionChildren, scopePrefix) {
   const { seg, nav } = scoped;
   const overview = overviewRoute(nav) ?? overviewRouteFromSegmentMeta(seg, nav);
-  const overviewPageId = overview != null ? metaStr(overview.metadata, "pageId") : void 0;
+  const overviewPageId = metaStr(seg.metadata, "overviewPageId");
   const relSection = stripLeadingScopePrefix(seg.section, scopePrefix);
   let slug = structuralNodeSlug(ctx, overview?.fullPath, relSection, scopePrefix);
   if (relSection === "" && overview?.fullPath == null) {
-    slug = inferSectionSlugFromNav(ctx, nav, scopePrefix) ?? slug;
+    const inferred = inferSectionSlugFromNav(ctx, nav, scopePrefix);
+    if (inferred != null) {
+      const inferredStr = String(inferred);
+      const collides = nav.some((r) => r.fullPath != null && String(ctx.contentSlug(r.fullPath)) === inferredStr);
+      if (!collides) {
+        slug = inferred;
+      }
+    }
   }
-  const metaPointsTo = metaStr(seg.metadata, "pointsTo");
   const defaults = withMetadataDefaults({
     id: ctx.nodeId(`section:${slug}`),
     title: metaStr(seg.metadata, "title") ?? "",
@@ -1333,7 +1339,10 @@ function sectionNode(ctx, scoped, sectionChildren, scopePrefix) {
     collapsible: metaBool(seg.metadata, "collapsible") ?? (metaCollapsed(seg.metadata) === true ? true : void 0),
     collapsedByDefault: metaBool(seg.metadata, "collapsedByDefault") ?? (metaCollapsed(seg.metadata) === true ? true : void 0),
     availability: metaAvailability(seg.metadata),
-    pointsTo: metaPointsTo != null ? pointsToSlug(metaPointsTo) : void 0,
+    // Don't read pointsTo from metadata for sections; let
+    // mutableUpdatePointsTo compute it via followRedirect so that
+    // self-referencing redirects are cleared, matching v2 behavior.
+    pointsTo: void 0,
     children: mergeChildrenByPosition(ctx, nav, sectionChildren)
   };
 }
@@ -1901,7 +1910,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);
 }