fix: skip prefetching for sidebar top links (#86)

* fix: skip prefetching for sidebar top links

Add data-no-prefetch attribute check in PrefetchProvider to skip
links that shouldn't be prefetched. Apply to content and API top
links in sidebar which are index routes, not doc pages.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* refactor: extract data-no-prefetch attr name to constant

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: use template literal for querySelector interpolation

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: rsbh

packages/chronicle/src/components/ui/PrefetchProvider.tsx

@@ -1,6 +1,8 @@
 import { useEffect } from 'react';
 import { prefetchPageData } from '@/lib/preload';
 
+const NO_PREFETCH_ATTR = 'data-no-prefetch';
+
 function resolvePathname(href: string | null): string | null {
   if (!href) return null;
   try {
@@ -16,14 +18,14 @@ export function PrefetchProvider({ children }: { children: React.ReactNode }) {
   useEffect(() => {
     const handleMouseOver = (e: MouseEvent) => {
       const anchor = (e.target as HTMLElement).closest?.('a[href]');
-      if (!anchor) return;
+      if (!anchor || anchor.hasAttribute(NO_PREFETCH_ATTR)) return;
       const pathname = resolvePathname(anchor.getAttribute('href'));
       if (pathname) prefetchPageData(pathname);
     };
 
     const handleFocusIn = (e: FocusEvent) => {
       const anchor = (e.target as HTMLElement).closest?.('a[href]');
-      if (!anchor) return;
+      if (!anchor || anchor.hasAttribute(NO_PREFETCH_ATTR)) return;
       const pathname = resolvePathname(anchor.getAttribute('href'));
       if (pathname) prefetchPageData(pathname);
     };
@@ -45,7 +47,7 @@ export function PrefetchProvider({ children }: { children: React.ReactNode }) {
     );
 
     const observeLinks = () => {
-      document.querySelectorAll('a[href]:not([data-prefetch-observed])').forEach((link) => {
+      document.querySelectorAll(`a[href]:not([data-prefetch-observed]):not([${NO_PREFETCH_ATTR}])`).forEach((link) => {
         const pathname = resolvePathname(link.getAttribute('href'));
         if (pathname) {
           link.setAttribute('data-prefetch-observed', '');

packages/chronicle/src/themes/default/Layout.tsx

@@ -138,7 +138,7 @@ export function Layout({
                         <DocumentTextIcon width={16} height={16} />
                       )}
                       classNames={{ root: styles.topLinkItem, text: styles.topLinkText }}
-                      render={<RouterLink to={entry.href} />}
+                      render={<RouterLink to={entry.href} data-no-prefetch />}
                     >
                       {entry.label}
                     </Sidebar.Item>
@@ -154,7 +154,7 @@ export function Layout({
                         <CodeBracketSquareIcon width={16} height={16} />
                       )}
                       classNames={{ root: styles.topLinkItem, text: styles.topLinkText }}
-                      render={<RouterLink to={api.basePath} />}
+                      render={<RouterLink to={api.basePath} data-no-prefetch />}
                     >
                       {api.name} API
                     </Sidebar.Item>