fix: address review feedback — remove operator agentStatus writes and fix tooltip accessibility

- Change SessionStatusDot tooltip trigger from span to button for proper accessibility (fixes frontend lint error)
- Remove all agentStatus writes from operator (reconciler.go and sessions.go) — agentStatus is now derived at query time from event logs, not stored in CR

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: Ambient Code Bot

components/frontend/src/components/session-status-dot.tsx

@@ -37,9 +37,10 @@ export function SessionStatusDot({ phase, className }: SessionStatusDotProps) {
     <TooltipProvider delayDuration={300}>
       <Tooltip>
         <TooltipTrigger asChild>
-          <span
+          <button
+            type="button"
             className={cn(
-              "inline-block w-2.5 h-2.5 rounded-full flex-shrink-0",
+              "inline-block w-2.5 h-2.5 rounded-full flex-shrink-0 border-0 p-0 cursor-default",
               color,
               animation,
               className

components/operator/internal/handlers/reconciler.go

@@ -84,7 +84,6 @@ func TransitionToStopped(ctx context.Context, session *unstructured.Unstructured
 	statusPatch.SetField("phase", "Stopped")
 	statusPatch.SetField("completionTime", time.Now().UTC().Format(time.RFC3339))
 	statusPatch.SetField("stoppedReason", stopReason)
-	statusPatch.SetField("agentStatus", "idle")
 	statusPatch.AddCondition(conditionUpdate{
 		Type:    conditionReady,
 		Status:  "False",
@@ -130,7 +129,6 @@ func TransitionToFailed(ctx context.Context, session *unstructured.Unstructured,
 	statusPatch := NewStatusPatch(namespace, name)
 	statusPatch.SetField("phase", "Failed")
 	statusPatch.SetField("completionTime", time.Now().UTC().Format(time.RFC3339))
-	statusPatch.SetField("agentStatus", "idle")
 	statusPatch.AddCondition(conditionUpdate{
 		Type:    conditionReady,
 		Status:  "False",
@@ -241,7 +239,6 @@ func UpdateSessionFromPodStatus(ctx context.Context, session *unstructured.Unstr
 	case corev1.PodSucceeded:
 		statusPatch.SetField("phase", "Completed")
 		statusPatch.SetField("completionTime", time.Now().UTC().Format(time.RFC3339))
-		statusPatch.SetField("agentStatus", "idle")
 		statusPatch.AddCondition(conditionUpdate{
 			Type:    conditionReady,
 			Status:  "False",
@@ -258,7 +255,6 @@ func UpdateSessionFromPodStatus(ctx context.Context, session *unstructured.Unstr
 		log.Printf("Pod %s failed: %s", podName, errorMsg)
 		statusPatch.SetField("phase", "Failed")
 		statusPatch.SetField("completionTime", time.Now().UTC().Format(time.RFC3339))
-		statusPatch.SetField("agentStatus", "idle")
 		statusPatch.AddCondition(conditionUpdate{
 			Type:    conditionReady,
 			Status:  "False",

components/operator/internal/handlers/sessions.go

@@ -1908,7 +1908,6 @@ func monitorPod(podName, sessionName, sessionNamespace string) {
 		if pod.Status.Phase == corev1.PodSucceeded {
 			statusPatch.SetField("phase", "Completed")
 			statusPatch.SetField("completionTime", time.Now().UTC().Format(time.RFC3339))
-			statusPatch.SetField("agentStatus", "idle")
 			statusPatch.AddCondition(conditionUpdate{Type: conditionReady, Status: "False", Reason: "Completed", Message: "Session finished"})
 			_ = statusPatch.Apply()
 			_ = deletePodAndPerPodService(sessionNamespace, podName, sessionName)
@@ -1972,7 +1971,6 @@ func monitorPod(podName, sessionName, sessionNamespace string) {
 			log.Printf("Pod %s failed: %s", podName, errorMsg)
 			statusPatch.SetField("phase", "Failed")
 			statusPatch.SetField("completionTime", time.Now().UTC().Format(time.RFC3339))
-			statusPatch.SetField("agentStatus", "idle")
 			statusPatch.AddCondition(conditionUpdate{Type: conditionReady, Status: "False", Reason: "PodFailed", Message: errorMsg})
 			_ = statusPatch.Apply()
 			_ = deletePodAndPerPodService(sessionNamespace, podName, sessionName)
@@ -2013,7 +2011,6 @@ func monitorPod(podName, sessionName, sessionNamespace string) {
 			now := time.Now().UTC().Format(time.RFC3339)
 
 			statusPatch.SetField("completionTime", now)
-			statusPatch.SetField("agentStatus", "idle")
 			switch term.ExitCode {
 			case 0:
 				statusPatch.SetField("phase", "Completed")