release: v2.7.229

Author: robertpelloni

CHANGELOG.md

@@ -4,6 +4,12 @@
 
 All notable changes to this project will be documented in this file.
 
+## [2.7.229] — 2026-03-15
+
+- changed(web/navigation): added shared `comparePaletteRoutes(...)` in `apps/web/src/components/mcp/nav-validation.ts` so quick-switch route ranking now has one reusable contract for favorite priority, recency priority, and stable title fallback ordering.
+- refactor(web/sidebar): `apps/web/src/components/Sidebar.tsx` now delegates palette route sorting to the shared comparator instead of keeping the ranking rules embedded inline inside the palette memo.
+- test(web/navigation): expanded `apps/web/src/components/mcp/nav-validation.test.ts` with explicit palette ranking coverage for favorites, recency, and title fallback ordering; focused nav suites now pass with `40` total tests.
+
 ## [2.7.228] — 2026-03-15
 
 - refactor(web/sidebar): added shared `safeStorageGetJson(...)` in `apps/web/src/components/Sidebar.tsx` so stored navigation preference hydration now reads and parses localStorage payloads through one helper instead of repeating inline `safeStorageGet(...)` + `JSON.parse(...)` flows.

VERSION

@@ -1 +1 @@
-2.7.228
+2.7.229

VERSION.md

@@ -1 +1 @@
-# Borg Project Version: 2.7.228
+# Borg Project Version: 2.7.229

apps/web/src/components/Sidebar.tsx

@@ -10,7 +10,7 @@ import { SortableContext, arrayMove, rectSortingStrategy, useSortable } from "@d
 import { CSS } from "@dnd-kit/utilities";
 import { cn } from "@/lib/utils";
 import { SIDEBAR_SECTIONS } from "./mcp/nav-config";
-import { buildExportedNavPreferences, buildNavItemsByNormalizedHref, buildRecentRouteHistory, buildRecentSearchHistory, getNavDescription, hasNavValidationIssues, isNavHrefActive, matchesNavQuery, normalizeNavHref, sanitizeCollapsedSections, sanitizeFavoriteRoutes, sanitizeNavPreferences, sanitizeRecentRoutes, sanitizeRecentSearches, validateSidebarSections } from "./mcp/nav-validation";
+import { buildExportedNavPreferences, buildNavItemsByNormalizedHref, buildRecentRouteHistory, buildRecentSearchHistory, comparePaletteRoutes, getNavDescription, hasNavValidationIssues, isNavHrefActive, matchesNavQuery, normalizeNavHref, sanitizeCollapsedSections, sanitizeFavoriteRoutes, sanitizeNavPreferences, sanitizeRecentRoutes, sanitizeRecentSearches, validateSidebarSections } from "./mcp/nav-validation";
 
 interface SidebarProps extends React.HTMLAttributes<HTMLDivElement> { }
 
@@ -250,21 +250,7 @@ export function Sidebar({ className }: SidebarProps) {
             return matchesNavQuery(q, item, item.section);
         });
 
-        rows.sort((a, b) => {
-            const aFav = favoriteSet.has(a.href) ? 1 : 0;
-            const bFav = favoriteSet.has(b.href) ? 1 : 0;
-            if (aFav !== bFav) {
-                return bFav - aFav;
-            }
-
-            const aRecent = recencyRank.has(a.href) ? recencyRank.get(a.href)! : Number.MAX_SAFE_INTEGER;
-            const bRecent = recencyRank.has(b.href) ? recencyRank.get(b.href)! : Number.MAX_SAFE_INTEGER;
-            if (aRecent !== bRecent) {
-                return aRecent - bRecent;
-            }
-
-            return a.title.localeCompare(b.title);
-        });
+        rows.sort((a, b) => comparePaletteRoutes(a, b, favoriteSet, recencyRank));
 
         const routeItems: PaletteItem[] = rows.map((item) => ({
             kind: 'route',

apps/web/src/components/mcp/nav-validation.test.ts

@@ -8,6 +8,7 @@ import {
     buildNavItemsByNormalizedHref,
     buildRecentRouteHistory,
     buildRecentSearchHistory,
+    comparePaletteRoutes,
     extractStringArray,
     filterNavHrefsByAllowedSet,
     getNavDescription,
@@ -255,6 +256,56 @@ describe('buildRecentSearchHistory', () => {
     });
 });
 
+describe('comparePaletteRoutes', () => {
+    const library = {
+        title: 'Library',
+        href: '/dashboard/library',
+        section: 'Knowledge',
+    };
+
+    const tools = {
+        title: 'Tools',
+        href: '/dashboard/tools',
+        section: 'Operations',
+    };
+
+    const audit = {
+        title: 'Audit',
+        href: '/dashboard/audit',
+        section: 'Security',
+    };
+
+    it('ranks favorited routes ahead of non-favorited routes', () => {
+        expect(comparePaletteRoutes(
+            library,
+            tools,
+            new Set(['/dashboard/library']),
+            new Map(),
+        )).toBeLessThan(0);
+    });
+
+    it('ranks more recent routes ahead when favorite status is equal', () => {
+        expect(comparePaletteRoutes(
+            tools,
+            audit,
+            new Set(),
+            new Map([
+                ['/dashboard/tools', 0],
+                ['/dashboard/audit', 2],
+            ]),
+        )).toBeLessThan(0);
+    });
+
+    it('falls back to title ordering when favorite and recency status are equal', () => {
+        expect(comparePaletteRoutes(
+            audit,
+            library,
+            new Set(),
+            new Map(),
+        )).toBeLessThan(0);
+    });
+});
+
 describe('getNavDescription', () => {
     it('returns trimmed explicit descriptions before fallback text', () => {
         expect(getNavDescription({

apps/web/src/components/mcp/nav-validation.ts

@@ -14,6 +14,10 @@ export interface NavDescriptionCandidate {
     description?: string;
 }
 
+export interface NavPaletteRouteCandidate extends NavSearchCandidate {
+    section: string;
+}
+
 export interface NavDuplicateIssue {
     href: string;
     sections: string[];
@@ -210,6 +214,27 @@ export function buildRecentSearchHistory(currentSearches: string[], nextQuery: s
     ], limit);
 }
 
+export function comparePaletteRoutes(
+    a: NavPaletteRouteCandidate,
+    b: NavPaletteRouteCandidate,
+    favoriteHrefs: ReadonlySet<string>,
+    recencyRank: ReadonlyMap<string, number>,
+): number {
+    const aFav = favoriteHrefs.has(a.href) ? 1 : 0;
+    const bFav = favoriteHrefs.has(b.href) ? 1 : 0;
+    if (aFav !== bFav) {
+        return bFav - aFav;
+    }
+
+    const aRecent = recencyRank.get(a.href) ?? Number.MAX_SAFE_INTEGER;
+    const bRecent = recencyRank.get(b.href) ?? Number.MAX_SAFE_INTEGER;
+    if (aRecent !== bRecent) {
+        return aRecent - bRecent;
+    }
+
+    return a.title.localeCompare(b.title);
+}
+
 export function buildFallbackNavDescription(title: string, section: string, href?: string): string {
     const normalized = title.toLowerCase();