Skip to content

Conversation

quge009
Copy link
Collaborator

@quge009 quge009 commented Sep 10, 2025

  • Change uuid to uuidv4, 32 bit
  • Log below message into database
    • Question from user
    • Answer generated by Copilot
    • Feedback (like, dislike) of the generated answer from user

Copy link

@Copilot 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

This PR adds analytics data collection functionality to the Copilot system by logging questions, answers, and feedback to a Kusto database. The changes update UUID generation from shortened 8-character strings to full UUIDs and implement threaded data collection to avoid blocking the main conversation flow.

  • Replace shortened UUID generation with full UUID v4 strings for better uniqueness
  • Add threaded data collection system that logs questions, answers, and feedback to Kusto
  • Add configuration support for destination Kusto cluster and database settings

Reviewed Changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
copilot_conversation.py Main implementation of data collection logic with threading and Kusto integration
copilot-chat-deployment.yaml.template Adds environment variables for destination Kusto cluster configuration
copilot-chat.yaml Adds default configuration values for data collection Kusto settings
state.ts Updates UUID generation from random strings to proper UUID v4
ChatBox.tsx Updates client-side UUID generation to use UUID v4
package.json Adds uuid package dependency and type definitions
Files not reviewed (1)
  • contrib/copilot-plugin/package-lock.json: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

quge009 and others added 5 commits September 10, 2025 17:49
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@yukirora yukirora changed the title Copilot: Add feature: collect Question/Answer/Feedback for analytict purpose Copilot: Add feature: collect Question/Answer/Feedback for analytic purpose Sep 11, 2025
@quge009 quge009 requested a review from yukirora September 11, 2025 02:28
@quge009 quge009 merged commit 46f48f0 into dev Sep 11, 2025
1 check passed
@quge009 quge009 deleted the lequ/copilot/feedback-dev2 branch September 11, 2025 05:16
This was referenced Sep 15, 2025
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.

2 participants