fix(cli): prevent empty sessions from being saved

[uinstinct]

Description

Empty sessions files are created in ~/.continue/sessions even when there are no user messages. This PR fixes that

AI Code Review

• Team members only: AI review runs automatically when PR is opened or marked ready for review
• Team members can also trigger a review by commenting @continue-review

Checklist

• [] I've read the contributing guide
• [] The relevant docs, if any, have been updated or created
• [] The relevant tests, if any, have been updated or created

Screen recording or screenshot

[ When applicable, please include a short screen recording or screenshot - this makes it much easier for us as contributors to review and understand your changes. See this PR as a good example. ]

Tests

[ What tests were added or updated to ensure the changes work as expected? ]

---

Continue Tasks: 🔄 7 running — View all

---

Summary by cubic

Prevents empty CLI chat sessions from being saved and adds a /sessions command to quickly view all sessions.

• Bug Fixes

  • Skip saving sessions with no content or only system messages.
  • Delay session creation; start with sessionId "" and no file until content exists.
  • Updated tests to reflect the new save behavior.

• New Features

  • Added /sessions slash command to open the session selector.

^{Written for commit 4244ee9. Summary will update on new commits.}

[cubic-dev-ai]

2 issues found across 6 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="extensions/cli/src/slashCommands.ts">

<violation number="1" location="extensions/cli/src/slashCommands.ts:177">
P2: `/sessions` emits duplicate `useSlashCommand` telemetry because it is captured globally and again inside `handleSessions`.</violation>
</file>

<file name="extensions/cli/src/services/ChatHistoryService.ts">

<violation number="1" location="extensions/cli/src/services/ChatHistoryService.ts:83">
P2: Unsafe optional chaining in initialization log can throw when `session` exists but `history` is missing.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

P2: /sessions emits duplicate useSlashCommand telemetry because it is captured globally and again inside handleSessions.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At extensions/cli/src/slashCommands.ts, line 177:

<comment>`/sessions` emits duplicate `useSlashCommand` telemetry because it is captured globally and again inside `handleSessions`.</comment>

<file context>
@@ -173,6 +173,11 @@ function handleJobs() {
 }
 
+function handleSessions() {
+  posthogService.capture("useSlashCommand", { name: "sessions" });
+  return { openSessionSelector: true };
+}
</file context>