Revert "fix: hide closed ACP sessions"

This reverts commit 9a1b856.

Author: lsongdev

app.go

@@ -422,9 +422,6 @@ func (a *App) ListAgentSessions() ([]agent.Session, error) {
 			}
 			session.AgentCommand = commandString(endpoint)
 			session.Key = session.AgentID + ":" + session.ID
-			if a.manager.IsSessionClosed(session.Key) {
-				continue
-			}
 			sessions = append(sessions, session)
 		}
 	}

frontend/src/components/SessionList.jsx

@@ -129,17 +129,13 @@ export default function SessionList({ activeSessionId, onSelectSession, onNewSes
   const handleClose = async (session, e) => {
     e.stopPropagation()
     setActionError(null)
-    const key = sessionKey(session)
-    const snapshot = sessions
-    setSessions((prev) => prev.filter((s) => sessionKey(s) !== key))
     try {
       await onBeforeSessionAction?.(session)
       await CloseSession(session.id)
-      onSessionClosed?.(key)
-      fetchSessions()
+      await fetchSessions()
+      onSessionClosed?.(sessionKey(session))
     } catch (err) {
       console.error('Close session error:', err)
-      setSessions(snapshot)
       setActionError(`Close failed: ${err?.toString() || err}`)
     }
   }

internal/agent/manager.go

@@ -56,7 +56,6 @@ type Manager struct {
 	replayMu       sync.Mutex
 	replays        map[string]*sessionReplay
 	loadedSessions map[string]bool
-	closedSessions map[string]struct{}
 }
 
 type sessionReplay struct {
@@ -76,7 +75,6 @@ func New(ctx context.Context, loadConfig agentclient.ConfigLoader, emit EventEmi
 		emit:           emit,
 		replays:        make(map[string]*sessionReplay),
 		loadedSessions: make(map[string]bool),
-		closedSessions: make(map[string]struct{}),
 	}
 }
 
@@ -411,9 +409,6 @@ func (m *Manager) NewSession(cwd string) (*Session, error) {
 	sessionID := string(resp.SessionID)
 	conversationID := m.scopedSessionID(sessionID)
 	log.Printf("[agent] NewSession created: id=%q", sessionID)
-	m.mu.Lock()
-	delete(m.closedSessions, conversationID)
-	m.mu.Unlock()
 	session := &Session{
 		ID:        sessionID,
 		Key:       conversationID,
@@ -497,9 +492,6 @@ func (m *Manager) ListSessions() ([]Session, error) {
 	for _, s := range resp.Sessions {
 		session := Session{ID: string(s.SessionID), Cwd: s.Cwd}
 		session.Key = m.scopedSessionID(session.ID)
-		if m.IsSessionClosed(session.Key) {
-			continue
-		}
 		session.AgentID = m.currentAgentID()
 		session.AgentName = m.currentAgentID()
 		if s.Title != nil {
@@ -530,28 +522,9 @@ func (m *Manager) CloseSession(sessionID string) error {
 	if err != nil {
 		return err
 	}
-	conversationID, acpSessionID := m.splitSessionRef(sessionID)
-	log.Printf("[agent] CloseSession: id=%q acp=%q", conversationID, acpSessionID)
+	_, acpSessionID := m.splitSessionRef(sessionID)
 	_, err = client.CloseSession(&acp.CloseSessionRequest{SessionID: acp.SessionID(acpSessionID)})
-	if err != nil {
-		log.Printf("[agent] CloseSession failed: %v", err)
-		return err
-	}
-	m.mu.Lock()
-	m.closedSessions[conversationID] = struct{}{}
-	delete(m.loadedSessions, conversationID)
-	m.mu.Unlock()
-	return nil
-}
-
-func (m *Manager) IsSessionClosed(sessionID string) bool {
-	if m == nil {
-		return false
-	}
-	m.mu.Lock()
-	defer m.mu.Unlock()
-	_, closed := m.closedSessions[sessionID]
-	return closed
+	return err
 }
 
 func (m *Manager) DeleteSession(sessionID string) error {

internal/agent/manager_test.go

@@ -28,21 +28,6 @@ func TestConnectEndpointRejectsIDsThatBreakSessionKeys(t *testing.T) {
 	}
 }
 
-func TestClosedSessionStateIsScopedAndInMemory(t *testing.T) {
-	manager := New(context.Background(), nil, nil)
-	manager.closedSessions["opencode:session-1"] = struct{}{}
-
-	if !manager.IsSessionClosed("opencode:session-1") {
-		t.Fatal("IsSessionClosed returned false for a closed session")
-	}
-	if manager.IsSessionClosed("builtin:session-1") {
-		t.Fatal("IsSessionClosed mixed sessions from different agents")
-	}
-	if New(context.Background(), nil, nil).IsSessionClosed("opencode:session-1") {
-		t.Fatal("closed session state persisted into a new manager")
-	}
-}
-
 func TestStoreForUpdateUsesReplayStoreAndBatchesProgress(t *testing.T) {
 	manager := New(context.Background(), nil, nil)
 	replayStore := conversation.NewStore()