Skip to content

Avoid continuing in Copilot CLI when already in Copilot CLI#300025

Merged
DonJayamanne merged 3 commits intomainfrom
don/copilot/present-canid
Mar 8, 2026
Merged

Avoid continuing in Copilot CLI when already in Copilot CLI#300025
DonJayamanne merged 3 commits intomainfrom
don/copilot/present-canid

Conversation

@DonJayamanne
Copy link
Contributor

Enhance the actionProvider and ChatSuggestNextWidget to utilize contextKeyService for improved session type checks. Simplify actionProvider by removing unnecessary dependencies. This refactor streamlines session type handling within the chat functionality.

Copilot AI review requested due to automatic review settings March 8, 2026 03:11
@DonJayamanne DonJayamanne self-assigned this Mar 8, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Refactors the “Continue in …” dropdown logic in the chat handoff/suggest-next widget to use context keys for determining the current chat session type, preventing offering a “continue in” target that matches the current session.

Changes:

  • Injects IContextKeyService into ChatSuggestNextWidget and reads the current session type from ChatContextKeys.chatSessionType.
  • Filters out chat session contributions whose type matches the current session type when building the dropdown actions.

@DonJayamanne DonJayamanne merged commit 2c51cfd into main Mar 8, 2026
24 checks passed
@DonJayamanne DonJayamanne deleted the don/copilot/present-canid branch March 8, 2026 06:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants