Merge develop into main

Consolidate all feature and fix branches into main:
- feat: orchestrator observability + mosaic rails integration (#422)
- fix: post-422 CI and compose env follow-up (#423)
- fix: orchestrator startup provider-key requirements (#425)
- fix: BetterAuth OAuth2 flow and compose wiring (#426)
- fix: BetterAuth UUID ID generation (#427)
- test: web vitest localStorage/file warnings (#428)
- fix: auth frontend remediation + review hardening (#421)
- Plus numerous Docker, deploy, and auth fixes from develop

Lockfile conflict resolved by regenerating from merged package.json.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

apps/web/src/components/knowledge/LinkAutocomplete.tsx

@@ -56,15 +56,6 @@ export function LinkAutocomplete({
   const mirrorRef = useRef<HTMLDivElement | null>(null);
   const cursorSpanRef = useRef<HTMLSpanElement | null>(null);
 
-  // Refs for event handler to avoid stale closures when effects re-attach listeners
-  const stateRef = useRef(state);
-  const resultsRef = useRef(results);
-  const selectedIndexRef = useRef(selectedIndex);
-  const insertLinkRef = useRef<((result: SearchResult) => void) | null>(null);
-  stateRef.current = state;
-  resultsRef.current = results;
-  selectedIndexRef.current = selectedIndex;
-
   /**
    * Search for knowledge entries matching the query.
    * Accepts an AbortSignal to allow cancellation of in-flight requests,
@@ -263,32 +254,29 @@ export function LinkAutocomplete({
   }, [textareaRef, state.isOpen, calculateDropdownPosition, debouncedSearch]);
 
   /**
-   * Handle keyboard navigation in the dropdown.
+   * Handle keyboard navigation in the dropdown
-   * Reads from refs to avoid stale closures when the effect
-   * that attaches this listener hasn't re-run yet.
    */
-  const handleKeyDown = useCallback((e: KeyboardEvent): void => {
+  const handleKeyDown = useCallback(
-    if (!stateRef.current.isOpen) return;
+    (e: KeyboardEvent): void => {
-
+      if (!state.isOpen) return;
-    const currentResults = resultsRef.current;
 
       switch (e.key) {
         case "ArrowDown":
           e.preventDefault();
-        setSelectedIndex((prev) => (prev + 1) % currentResults.length);
+          setSelectedIndex((prev) => (prev + 1) % results.length);
           break;
 
         case "ArrowUp":
           e.preventDefault();
-        setSelectedIndex((prev) => (prev - 1 + currentResults.length) % currentResults.length);
+          setSelectedIndex((prev) => (prev - 1 + results.length) % results.length);
           break;
 
         case "Enter":
           e.preventDefault();
-        if (currentResults.length > 0 && selectedIndexRef.current >= 0) {
+          if (results.length > 0 && selectedIndex >= 0) {
-          const selected = currentResults[selectedIndexRef.current];
+            const selected = results[selectedIndex];
             if (selected) {
-            insertLinkRef.current?.(selected);
+              insertLink(selected);
             }
           }
           break;
@@ -304,7 +292,9 @@ export function LinkAutocomplete({
           setResults([]);
           break;
       }
-  }, []);
+    },
+    [state.isOpen, results, selectedIndex]
+  );
 
   /**
    * Insert the selected link into the textarea
@@ -340,7 +330,6 @@ export function LinkAutocomplete({
     },
     [textareaRef, state.triggerIndex, onInsert]
   );
-  insertLinkRef.current = insertLink;
 
   /**
    * Handle click on a result