Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(participant): Report errors to telemetry VSCODE-605 #823

Merged

Conversation

nirinchev
Copy link
Contributor

Description

This reworks a little the error handling in the participant to streamline error reporting. It's now wrapping the entire chat handler rather than only the code that invokes the model.

Additionally, we're now showing the welcome message at all times, even if the prompt was empty.

Checklist

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

@nirinchev nirinchev merged commit 48f4a67 into VSCODE-528-mongodb-copilot Sep 21, 2024
3 checks passed
@nirinchev nirinchev deleted the ni/paritcipant-telemetry-errors branch September 21, 2024 12:42
@nirinchev nirinchev changed the title feat(participant): Report errors to telemetry feat(participant): Report errors to telemetry VSCODE-605 Sep 21, 2024
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