Skip to content

Add issue reporting for failed chat session loads in Copilot CLI#4378

Merged
DonJayamanne merged 3 commits intomainfrom
don/sympathetic-kingfisher
Mar 12, 2026
Merged

Add issue reporting for failed chat session loads in Copilot CLI#4378
DonJayamanne merged 3 commits intomainfrom
don/sympathetic-kingfisher

Conversation

@DonJayamanne
Copy link
Collaborator

No description provided.

@DonJayamanne DonJayamanne self-assigned this Mar 12, 2026
@DonJayamanne DonJayamanne marked this pull request as ready for review March 12, 2026 20:12
Copilot AI review requested due to automatic review settings March 12, 2026 20:12
@DonJayamanne DonJayamanne enabled auto-merge March 12, 2026 20:12
@vs-code-engineering vs-code-engineering bot added this to the 1.112.0 milestone Mar 12, 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

Adds a more actionable warning for Copilot CLI chat sessions that fail to load by including a pre-filled “report an issue” link with environment details, and updates the unit test vscode mock to support the new runtime info gathering.

Changes:

  • Generate runtime/environment info (platform, VS Code version, extension version) for session-load failure reporting.
  • Replace the plain invalid-session warning with a Markdown warning containing a “report an issue” link.
  • Extend the Vitest vscode mock to include env, version, and extensions.getExtension().

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/extension/chatSessions/vscode-node/copilotCLIChatSessionsContribution.ts Builds a GitHub issue URL/body for session-load failures and surfaces it via a Markdown warning link.
src/extension/chatSessions/vscode-node/test/copilotCLIChatSessionParticipant.spec.ts Updates the mocked vscode module to provide the new env/version/extensions fields used by the contribution code.

@DonJayamanne DonJayamanne marked this pull request as draft March 12, 2026 20:20
auto-merge was automatically disabled March 12, 2026 20:20

Pull request was converted to draft

@DonJayamanne DonJayamanne marked this pull request as ready for review March 12, 2026 22:24
@DonJayamanne DonJayamanne enabled auto-merge March 12, 2026 22:24
@DonJayamanne DonJayamanne added this pull request to the merge queue Mar 12, 2026
Merged via the queue into main with commit 44a8c81 Mar 12, 2026
19 checks passed
@DonJayamanne DonJayamanne deleted the don/sympathetic-kingfisher branch March 12, 2026 22:52
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