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: add report for ai completion #3189

Merged
merged 1 commit into from
Nov 17, 2023

Conversation

pipiiiiii
Copy link
Contributor

@pipiiiiii pipiiiiii commented Nov 17, 2023

Types

  • 🎉 New Features

Background or solution

🤖[deprecated] Generated by Copilot at dd28e6e

  • Add logic for reporting completion requests and user feedback to the AI service using the IAIReporter interface (link, link, link, link, link, link, link, link, link, link)
  • Refactor some variables to use const instead of let to indicate that they are not reassigned and improve readability (link, link, link, link, link, link)
  • Simplify the expression for getting the current timestamp in milliseconds by using +new Date() instead of new Date().getTime() (link)
  • Remove some unused imports from the completeProvider.ts file (link)

Changelog

🤖[deprecated] Generated by Copilot at dd28e6e

This pull request enhances the ai-native module with reporting capabilities for the inline completion feature. It defines a new interface IAIReporter and a new type CompletionEvent for sending metrics and events to the AI service. It also updates the completeProvider.ts file to use the aiReporter instance and the AISerivceType.Completion enum value.

@pipiiiiii pipiiiiii requested a review from Ricbet November 17, 2023 09:52
@opensumi opensumi bot added the 🎨 feature feature required label Nov 17, 2023
@pipiiiiii pipiiiiii added the v3 OpenSumi 3.0 label Nov 17, 2023
@pipiiiiii pipiiiiii merged commit fb99430 into chore/ai-native-module Nov 17, 2023
5 checks passed
@pipiiiiii pipiiiiii deleted the feat/completion-report branch November 17, 2023 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 feature feature required v3 OpenSumi 3.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants