@opendocsdev/cli 0.2.3 → 0.2.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@opendocsdev/cli",
-  "version": "0.2.3",
+  "version": "0.2.5",
   "publishConfig": {
     "access": "public"
   },

package/src/engine/src/components/Analytics.astro

@@ -30,28 +30,23 @@ const shouldTrack = backend && siteId && analyticsEnabled;
     is:inline
   >
     (function() {
-      // Get configuration from script tag
-      const script = document.currentScript;
-      const backend = script.getAttribute('data-backend');
-      const siteId = script.getAttribute('data-site-id');
-      const pathFromProp = script.getAttribute('data-path');
+      var script = document.currentScript;
+      var backend = script.getAttribute('data-backend');
+      var siteId = script.getAttribute('data-site-id');
 
-      // Use provided path or current pathname
-      const path = pathFromProp || window.location.pathname;
+      function trackPageview() {
+        fetch(backend + '/api/analytics/pv', {
+          method: 'POST',
+          headers: { 'Content-Type': 'application/json' },
+          body: JSON.stringify({
+            siteId: siteId,
+            path: window.location.pathname,
+          }),
+        }).catch(function() {});
+      }
 
-      // Send pageview to backend (short path to avoid ad blockers)
-      fetch(`${backend}/api/analytics/pv`, {
-        method: 'POST',
-        headers: {
-          'Content-Type': 'application/json',
-        },
-        body: JSON.stringify({
-          siteId: siteId,
-          path: path,
-        }),
-      }).catch(function(err) {
-        console.debug('[opendocs] Analytics error:', err);
-      });
+      trackPageview();
+      document.addEventListener('astro:after-swap', trackPageview);
     })();
   </script>
 )}

package/src/engine/src/components/react/SearchModal.tsx

@@ -80,13 +80,13 @@ export function SearchModal({ isOpen, onClose, backend, siteId }: SearchModalPro
   const resultsRef = useRef<HTMLDivElement>(null);
   const pagefindRef = useRef<any>(null);
 
-  // Track the last search for analytics (only sent on meaningful actions)
-  const lastSearchRef = useRef<{ query: string; resultCount: number } | null>(null);
   const hasTrackedRef = useRef(false);
+  const lastResultCountRef = useRef(0);
+  const queryRef = useRef(query);
+  queryRef.current = query;
 
   const loadPagefind = async () => {
     try {
-      // Use Function constructor to create a truly dynamic import that Vite won't analyze
       const importPagefind = new Function(
         'return import("/pagefind/pagefind.js")'
       ) as () => Promise<any>;
@@ -100,17 +100,15 @@ export function SearchModal({ isOpen, onClose, backend, siteId }: SearchModalPro
     }
   };
 
-  // Track search query for analytics - only tracks once per search session
-  const trackSearch = useCallback(() => {
+  // Fire-and-forget analytics using current query state
+  // Tracks on close or result click — captures the query text at that moment,
+  // including searches with 0 results (failed searches)
+  const trackSearch = useCallback((searchQuery: string, resultCount: number) => {
     if (!backend || !siteId) return;
     if (hasTrackedRef.current) return;
-    if (!lastSearchRef.current) return;
-
-    const { query: searchQuery, resultCount } = lastSearchRef.current;
     if (!searchQuery.trim()) return;
 
     hasTrackedRef.current = true;
-
     const blob = new Blob(
       [JSON.stringify({ siteId, query: searchQuery, resultCount })],
       { type: "application/json" }
@@ -118,15 +116,12 @@ export function SearchModal({ isOpen, onClose, backend, siteId }: SearchModalPro
     navigator.sendBeacon(`${backend}/api/analytics/sq`, blob);
   }, [backend, siteId]);
 
-  // Handle close with analytics tracking
-  const handleClose = useCallback(() => {
-    trackSearch();
-    onClose();
-  }, [onClose, trackSearch]);
+  // Ref so cleanup effects always call the latest trackSearch without re-registering
+  const trackSearchRef = useRef(trackSearch);
+  trackSearchRef.current = trackSearch;
 
-  // Handle result selection with analytics tracking
-  const handleResultSelect = useCallback((url: string) => {
-    trackSearch();
+  const handleResultSelect = useCallback((url: string, searchQuery: string, resultCount: number) => {
+    trackSearch(searchQuery, resultCount);
     window.location.href = url;
   }, [trackSearch]);
 
@@ -155,7 +150,7 @@ export function SearchModal({ isOpen, onClose, backend, siteId }: SearchModalPro
         })
       );
       dispatch({ type: "SEARCH_SUCCESS", payload: formattedResults });
-      lastSearchRef.current = { query: searchQuery, resultCount: searchResults.results.length };
+      lastResultCountRef.current = searchResults.results.length;
     } catch (e) {
       console.error("Search error:", e);
       dispatch({ type: "SEARCH_ERROR", payload: "Search failed" });
@@ -169,15 +164,18 @@ export function SearchModal({ isOpen, onClose, backend, siteId }: SearchModalPro
     }
   }, [isOpen, pagefindLoaded]);
 
-  // Reset state and focus input when modal opens
+  // Track analytics when modal closes, reset state when it opens
   useEffect(() => {
     if (isOpen) {
       dispatch({ type: "RESET" });
-      lastSearchRef.current = null;
       hasTrackedRef.current = false;
-      // Small delay to ensure modal is rendered before focusing
+      lastResultCountRef.current = 0;
       const timerId = setTimeout(() => inputRef.current?.focus(), 50);
-      return () => clearTimeout(timerId);
+      return () => {
+        clearTimeout(timerId);
+        // Fires when isOpen flips to false — track the search session
+        trackSearchRef.current(queryRef.current, lastResultCountRef.current);
+      };
     }
   }, [isOpen]);
 
@@ -187,12 +185,12 @@ export function SearchModal({ isOpen, onClose, backend, siteId }: SearchModalPro
     const handleEscape = (e: KeyboardEvent) => {
       if (e.key === "Escape") {
         e.preventDefault();
-        handleClose();
+        onClose();
       }
     };
     document.addEventListener("keydown", handleEscape);
     return () => document.removeEventListener("keydown", handleEscape);
-  }, [isOpen, handleClose]);
+  }, [isOpen, onClose]);
 
   // Scroll selected item into view
   useEffect(() => {
@@ -224,7 +222,7 @@ export function SearchModal({ isOpen, onClose, backend, siteId }: SearchModalPro
       case "Enter":
         e.preventDefault();
         if (results[selectedIndex]) {
-          handleResultSelect(results[selectedIndex].url);
+          handleResultSelect(results[selectedIndex].url, query, lastResultCountRef.current);
         }
         break;
     }
@@ -243,7 +241,7 @@ export function SearchModal({ isOpen, onClose, backend, siteId }: SearchModalPro
       {/* Backdrop */}
       <div
         className="absolute inset-0 bg-black/50 backdrop-blur-sm"
-        onClick={handleClose}
+        onClick={onClose}
         aria-hidden="true"
       />
 
@@ -288,7 +286,7 @@ export function SearchModal({ isOpen, onClose, backend, siteId }: SearchModalPro
                     href={result.url}
                     onClick={(e) => {
                       e.preventDefault();
-                      handleResultSelect(result.url);
+                      handleResultSelect(result.url, query, lastResultCountRef.current);
                     }}
                     className={cn(
                       "block px-4 py-3 transition-colors",