fix(url-state): clear log tab on all close paths + trim KB search

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/knowledge/[id]/base.tsx

@@ -298,7 +298,8 @@ export function KnowledgeBase({
   // doesn't refetch on every keystroke. Changing the search resets pagination.
   const handleSearchChange = useCallback(
     (newQuery: string) => {
-      const next = newQuery.length > 0 ? newQuery : null
+      const trimmed = newQuery.trim()
+      const next = trimmed.length > 0 ? trimmed : null
       setDocumentFilters(
         { q: next },
         next === null ? undefined : { limitUrlUpdates: debounce(300) }

apps/sim/app/workspace/[workspaceId]/logs/logs.tsx

@@ -422,10 +422,24 @@ export default function Logs() {
   const handleCloseSidebar = useCallback(() => {
     dispatch({ type: 'CLOSE_SIDEBAR' })
     activeLogTabRef.current = 'overview'
-    // Strip `tab` from the URL (back to the default) so reopening another log
-    // starts on the overview tab instead of inheriting the closed log's tab.
-    setLogDetailsTab(null)
-  }, [setLogDetailsTab])
+  }, [])
+
+  /**
+   * Strip the `tab` param whenever the detail panel transitions from open to
+   * closed — by the X button, toggling the same row, or the keyboard — so
+   * reopening another log starts on overview rather than inheriting the closed
+   * log's tab. Guarded on a prior-open ref so an initial deep-linked `?tab=` is
+   * preserved (the panel isn't open yet on first mount).
+   */
+  const wasSidebarOpenRef = useRef(false)
+  useEffect(() => {
+    if (isSidebarOpen) {
+      wasSidebarOpenRef.current = true
+    } else if (wasSidebarOpenRef.current) {
+      wasSidebarOpenRef.current = false
+      setLogDetailsTab(null)
+    }
+  }, [isSidebarOpen, setLogDetailsTab])
 
   const handleActiveTabChange = useCallback((tab: string) => {
     activeLogTabRef.current = tab