Skip to content

fix(cli): resume selected session from cn ls#12938

Open
JSap0914 wants to merge 1 commit into
continuedev:mainfrom
JSap0914:fix/cli-resume-selected-session-12927
Open

fix(cli): resume selected session from cn ls#12938
JSap0914 wants to merge 1 commit into
continuedev:mainfrom
JSap0914:fix/cli-resume-selected-session-12927

Conversation

@JSap0914

@JSap0914 JSap0914 commented Jul 3, 2026

Copy link
Copy Markdown

Summary

  • pass the selected cn ls session id through to chat --resume
  • add loadResumeSession() so targeted resume sets the current session before future saves
  • remove the stale env-var session override path from ls

Fixes #12927

Verification

  • npm test -- src/commands/ls.test.ts src/session.test.ts (38 tests passed)
  • npm run typecheck currently fails before this change due missing local workspace package builds/type declarations such as @continuedev/config-yaml and existing core NodeNext import diagnostics in this checkout.

@JSap0914 JSap0914 requested a review from a team as a code owner July 3, 2026 07:14
@JSap0914 JSap0914 requested review from Copilot and sestinj and removed request for a team July 3, 2026 07:14
@dosubot dosubot Bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jul 3, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@JSap0914 JSap0914 force-pushed the fix/cli-resume-selected-session-12927 branch from 18d10b1 to cfc2ea9 Compare July 3, 2026 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cn ls always resumes the latest session regardless of user selection

2 participants