fix(agent): make Inbox recovery choices visible and unambiguous (#1498)

* fix(agent): make Inbox recovery choices explicit

* fix(agent): invalidate stale recovery actions

* fix(test): preserve agent recovery launch chronology

* fix(sidebar): repair stale accessibility focus

* fix(sidebar): route AX row keyboard navigation

* fix(sidebar): preserve semantic focus for AX rows

* fix(sidebar): sync SwiftUI keyboard focus

* revert(sidebar): drop unrelated accessibility focus changes

These four commits were not part of #1496 (Agent Inbox recovery
choices). They made SidebarAccessibilityRowView accept first responder
status, which stole arrow keys from the sidebar's own key handler and
broke Finder-style keyboard navigation:
SidebarFolderClickTests.testKeyboardNavigationTypeSelectAndOutlineSemantics
failed on all three CI retries while main stayed green.

Sidebar files are now byte-identical to main; only the Agent Inbox work
from #1496 remains.

Author: batonogov

Pine/AccessibilityIdentifiers.swift

@@ -223,6 +223,12 @@ nonisolated enum AccessibilityID {
     static let agentInboxEmpty = "agentInboxEmpty"
     static let agentInboxNavigationStatus = "agentInboxNavigationStatus"
     static let agentInboxHelpButton = "agentInboxHelpButton"
+    static let agentInboxRecoveryActions = "agentInboxRecoveryActions"
+    static let agentInboxResumeSession = "agentInboxResumeSession"
+    static let agentInboxNewSession = "agentInboxNewSession"
+    static let agentInboxMarkReviewed = "agentInboxMarkReviewed"
+    static let agentInboxCopyObjective = "agentInboxCopyObjective"
+    static let agentInboxDismissTask = "agentInboxDismissTask"
     static func agentInboxRow(_ id: UUID) -> String {
         "agentInboxRow_\(id.uuidString)"
     }

Pine/Agent/AgentInboxModels.swift

@@ -48,6 +48,67 @@ nonisolated struct AgentInboxRow: Identifiable, Equatable, Sendable {
     }
 }
 
+/// Minimal state that controls whether an already-presented recovery surface
+/// may remain visible. The task ID alone is insufficient because liveness and
+/// lifecycle can change in place while the same durable task remains listed.
+nonisolated struct AgentInboxRecoveryState: Equatable, Sendable {
+    let id: UUID
+    let canRecover: Bool
+
+    init(row: AgentInboxRow) {
+        id = row.id
+        canRecover = row.canRecover
+    }
+
+    init(id: UUID, canRecover: Bool) {
+        self.id = id
+        self.canRecover = canRecover
+    }
+}
+
+/// Pure presentation policy for activating an Inbox row with Return.
+///
+/// Recovery deliberately has two keyboard steps: the first Return exposes the
+/// available actions, and only a later Return can invoke the visible primary
+/// action. This prevents a restored task from silently opening a fresh session
+/// when a vendor resume path is available.
+nonisolated enum AgentInboxActivation: Equatable, Sendable {
+    case navigate
+    case presentRecoveryActions
+    case recover(AgentTaskRecoveryAction)
+
+    static func resolve(
+        row: AgentInboxRow,
+        recoveryActionsArePresented: Bool,
+        canResumeVendorSession: Bool
+    ) -> AgentInboxActivation {
+        guard row.canRecover else { return .navigate }
+        guard recoveryActionsArePresented else {
+            return .presentRecoveryActions
+        }
+        return .recover(primaryRecoveryAction(
+            canResumeVendorSession: canResumeVendorSession
+        ))
+    }
+
+    static func primaryRecoveryAction(
+        canResumeVendorSession: Bool
+    ) -> AgentTaskRecoveryAction {
+        canResumeVendorSession ? .resumeVendorSession : .startNewSession
+    }
+
+    static func normalizedPresentedTaskID(
+        _ taskID: UUID?,
+        states: [AgentInboxRecoveryState]
+    ) -> UUID? {
+        guard let taskID,
+              states.contains(where: {
+                  $0.id == taskID && $0.canRecover
+              }) else { return nil }
+        return taskID
+    }
+}
+
 nonisolated struct AgentInboxSection: Identifiable, Equatable, Sendable {
     let id: AgentInboxSectionID
     let rows: [AgentInboxRow]