fix(ambient-ui): address CodeRabbit review findings

- Dialog closes after mutation settles, not before (shows errors)
- Send path honors disabled prop (blocks Enter key when loading)
- Sidebar syncs with browser back/forward via popstate listener
- Drag listeners cleaned up on unmount (no leaked event handlers)
- Normalize tool_use.input to always be an object

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

Author: jsell-rh

components/ambient-ui/src/app/(dashboard)/[projectId]/fleet/[sessionId]/_components/session-header.tsx

@@ -50,17 +50,19 @@ export function SessionHeader({ session }: { session: DomainSession }) {
   const canRestart = RESTARTABLE_PHASES.has(session.phase)
 
   const handleConfirmStop = useCallback(() => {
-    stopSession.mutate(session.id)
-    setStopDialogOpen(false)
+    stopSession.mutate(session.id, {
+      onSettled: () => setStopDialogOpen(false),
+    })
   }, [stopSession, session.id])
 
   const handleConfirmDelete = useCallback(() => {
     deleteSession.mutate(session.id, {
       onSuccess: () => {
+        setDeleteDialogOpen(false)
         router.push(`/${projectId}/fleet`)
       },
+      onError: () => setDeleteDialogOpen(false),
     })
-    setDeleteDialogOpen(false)
   }, [deleteSession, session.id, router, projectId])
 
   const handleExport = useCallback(async () => {

components/ambient-ui/src/components/chat-messages.tsx

@@ -352,7 +352,7 @@ export function ChatInput({ sessionId, phase, disabled }: ChatInputProps) {
 
   const handleSend = useCallback(() => {
     const trimmed = input.trim()
-    if (!trimmed || !isRunning || sendMessage.isPending) return
+    if (!trimmed || !isRunning || disabled || sendMessage.isPending) return
     sendMessage.mutate(trimmed, {
       onSuccess: () => {
         setInput('')

components/ambient-ui/src/components/chat-sidebar-context.tsx

@@ -5,6 +5,7 @@ import {
   useContext,
   useState,
   useCallback,
+  useEffect,
   useMemo,
   type ReactNode,
 } from 'react'
@@ -46,6 +47,12 @@ export function ChatSidebarProvider({ children }: { children: ReactNode }) {
     updateChatParam(null)
   }, [])
 
+  useEffect(() => {
+    const handlePopState = () => setOpenSessionId(readChatParam())
+    window.addEventListener('popstate', handlePopState)
+    return () => window.removeEventListener('popstate', handlePopState)
+  }, [])
+
   const value = useMemo<ChatSidebarState>(
     () => ({
       openSessionId,

components/ambient-ui/src/components/chat-sidebar.tsx

@@ -73,10 +73,28 @@ export function ChatSidebar() {
     hasScrolledOnLoad.current = false
   }, [openSessionId])
 
-  // Drag-to-resize handler
+  // Drag-to-resize handler with cleanup refs
+  const dragListenersRef = useRef<{ move: (e: MouseEvent) => void; up: () => void } | null>(null)
+
+  const cleanupDrag = useCallback(() => {
+    if (dragListenersRef.current) {
+      document.removeEventListener('mousemove', dragListenersRef.current.move)
+      document.removeEventListener('mouseup', dragListenersRef.current.up)
+      dragListenersRef.current = null
+    }
+    isDragging.current = false
+    document.body.style.cursor = ''
+    document.body.style.userSelect = ''
+  }, [])
+
+  useEffect(() => {
+    return cleanupDrag
+  }, [cleanupDrag])
+
   const handleMouseDown = useCallback(
     (e: React.MouseEvent) => {
       e.preventDefault()
+      cleanupDrag()
       isDragging.current = true
       startX.current = e.clientX
       startWidth.current = width
@@ -90,18 +108,13 @@ export function ChatSidebar() {
         setWidth(newWidth)
       }
 
-      const handleMouseUp = () => {
-        isDragging.current = false
-        document.body.style.cursor = ''
-        document.body.style.userSelect = ''
-        document.removeEventListener('mousemove', handleMouseMove)
-        document.removeEventListener('mouseup', handleMouseUp)
-      }
+      const handleMouseUp = () => cleanupDrag()
 
+      dragListenersRef.current = { move: handleMouseMove, up: handleMouseUp }
       document.addEventListener('mousemove', handleMouseMove)
       document.addEventListener('mouseup', handleMouseUp)
     },
-    [width],
+    [width, cleanupDrag],
   )
 
   // Handle Escape key to close

components/runners/ambient-runner/ambient_runner/bridges/claude/operational_events.py

@@ -63,9 +63,12 @@ def _build_payload(
         payload: dict = {"tool": name, "tool_call_id": tool_id}
         if args:
             try:
-                payload["input"] = json.loads(args)
+                parsed = json.loads(args)
+                payload["input"] = (
+                    parsed if isinstance(parsed, dict) else {"value": parsed}
+                )
             except (json.JSONDecodeError, TypeError):
-                payload["input"] = args
+                payload["input"] = {"raw": args}
         return json.dumps(payload)
 
     if event_type_str == "TOOL_CALL_RESULT":