feat: Enhance session management by adding session ID handling and re…

[cnjack]

Web Session Management Optimization

Problem

• Clicking "New conversation" immediately creates an empty session file on the backend, resulting in numerous blank sessions cluttering the session list
• Deleting a session requires a secondary confirmation, making the operation cumbersome

Changes

1. Deferred Session Creation (Avoid Empty Sessions)

Backend (internal/web/server.go):

• handleNewSession: When creating a new conversation, no longer immediately creates a recorder; only creates one when restoring an existing session
• submitMessage: Retains existing lazy-loading logic — the recorder and session file are only created when the first message is sent

Frontend (web/src/stores/chat.ts):

• newSession(): If already on the welcome screen (no session ID and no messages), performs no action; otherwise resets the backend state and sets currentSessionId to empty
• agentDone(): After the agent finishes running, refreshes fetchHealth() + fetchSessions() to ensure lazily-created sessions appear in the sidebar

2. Simplified Deletion

Frontend (web/src/components/Sidebar.vue):

• Delete button icon changed from trash can to archive icon
• Removed right-click menu secondary confirmation; click now deletes immediately
• Cleaned up no-longer-needed contextMenuId state and toggleContextMenu function

3. Session Restore & History

Frontend (web/src/stores/chat.ts & App.vue):

• Added restoreCurrentSession(): On page load, fetches the current session content from the backend and replays it into the UI
• loadSession(): Supports restoring session todo snapshots and tool call state matching
• Ctrl+Shift+N keyboard shortcut bound to newSession()

Frontend (web/src/composables/api.ts):

• newSession API now supports passing a sessionId parameter for restoring an existing session

Summary by CodeRabbit

• New Features

  • Added session persistence and restoration—sessions now maintain state across reloads and can be resumed with full chat history and tool execution records restored.
  • Enhanced session tracking with real-time status visibility.

• Improvements

  • Simplified session management UI by streamlining session action controls.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f727494b-cc81-42c5-8cdf-551192107490

📥 Commits

Reviewing files that changed from the base of the PR and between 2977bc1 and 823bfa9.

📒 Files selected for processing (5)

• internal/web/server.go
• web/src/App.vue
• web/src/components/Sidebar.vue
• web/src/composables/api.ts
• web/src/stores/chat.ts

---

📝 Walkthrough

Walkthrough

This pull request implements session restoration and tracking by deferring recorder creation until the first user message, adding session_id and running fields to health responses, tracking the current session in the chat store, and providing a restoreCurrentSession() action to rebuild chat UI state from persisted session files.

Changes

Cohort / File(s)	Summary
Backend Session API internal/web/server.go, web/src/composables/api.ts	Modified /api/health to include session_id and running fields. Deferred recorder creation in handleNewSession until first user message; recorder only created when resuming an existing session (when req.SessionID is provided). handleNewSession now returns session_id in response.
Chat Store State & Restoration web/src/stores/chat.ts	Added currentSessionId reactive state. Enhanced fetchHealth() to synchronize currentSessionId and isRunning from health response. Implemented new restoreCurrentSession() action to load persisted sessions, rebuild timeline, match tool results to tool calls, and finalize unresolved calls. Updated newSession() and loadSession() to track session lifecycle.
Frontend Integration web/src/App.vue	Added restoreCurrentSession() calls in onMounted (after initial state fetch) and onProjectSwitched (after project refresh), sequencing restoration after related data loads.
Session UI Simplification web/src/components/Sidebar.vue	Removed context menu state and confirmation popover for session deletion. Replaced "Delete" control with neutral "Archive" button and icon; clicking now directly invokes delete without intermediate menu.

Sequence Diagram

sequenceDiagram
    participant Client as Client (Browser)
    participant App as App.vue
    participant Store as Chat Store
    participant Server as Backend Server
    participant Storage as Session File

    Client->>App: onMounted
    App->>Store: fetchHealth()
    Store->>Server: GET /api/health
    Server-->>Store: {status, session_id, running, ...}
    Store->>Store: sync currentSessionId, isRunning
    
    App->>Store: restoreCurrentSession()
    
    alt Session exists
        Store->>Server: api.session(currentSessionId)
        Server->>Storage: load session file
        Storage-->>Server: session entries
        Server-->>Store: entries
        Store->>Store: rebuild timeline items
        Store->>Store: match tool results to calls
        Store->>Store: finalize tool calls
        Store-->>Client: restored chat UI
    else No session or agent running
        Store-->>Client: skip restoration
    end

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~50 minutes

Possibly related PRs

• feat: sort fetched sessions by creation date and improve tool result … #20: Modifies web/src/stores/chat.ts to improve tool result resolution by matching on tool_call_id with name-based fallback, directly overlapping with restoration logic.
• fix: deduplicate session ID and fix resume bugs in web/ACP/TUI #28: Modifies internal/web/server.go session-resume and new-session handling, including recorder UUID and API response fields.

Poem

🐰 A recorder waits till first word's said,
Sessions restored from files widespread,
Current IDs tracked with care so light,
Chat timelines bloom—the restore's right! ✨

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feat/web-conversation-logic

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}