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: support inlinechat report #3178

Merged
merged 1 commit into from
Nov 16, 2023
Merged

Conversation

pipiiiiii
Copy link
Contributor

@pipiiiiii pipiiiiii commented Nov 16, 2023

Types

  • ⏱ Tests

Background or solution

🤖[deprecated] Generated by Copilot at b02d3c4

  • Import and inject IAIReporter service to report AI service events in AiEditorContribution class (link, link)
  • Modify handleDiffEditorResult function to accept and use relationId parameter for reporting (link, link, link)
  • Call this.aiReporter.start and this.aiReporter.end methods with appropriate messages and timings in handleAIEditorResult and handleAIEditorRetry functions (link, link)
  • Change msg parameter type from AISerivceType to string in start method of IAIReporter interface (link)

Changelog

🤖[deprecated] Generated by Copilot at b02d3c4

Improved AI service event reporting in ai-native package. Changed msg parameter type in IAIReporter interface and used it to send more detailed messages in AiEditorContribution.

@opensumi opensumi bot added the 🎨 feature feature required label Nov 16, 2023
@pipiiiiii pipiiiiii merged commit 6518b78 into chore/ai-native-module Nov 16, 2023
5 checks passed
@pipiiiiii pipiiiiii deleted the feat/report-editor branch November 16, 2023 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 feature feature required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant