-
Notifications
You must be signed in to change notification settings - Fork 61
telemetry(amazonq): add metrics for code symbols, chat history and export #1023
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
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -55,6 +55,11 @@ | |||||||||||
| "type": "double", | ||||||||||||
| "description": "The time it takes to generate code generation response" | ||||||||||||
| }, | ||||||||||||
| { | ||||||||||||
| "name": "amazonqHistoryFileSize", | ||||||||||||
| "type": "int", | ||||||||||||
| "description": "The size of a chat history file (in bytes)" | ||||||||||||
| }, | ||||||||||||
| { | ||||||||||||
| "name": "amazonqNumberOfFilesAccepted", | ||||||||||||
| "type": "double", | ||||||||||||
|
|
@@ -75,6 +80,16 @@ | |||||||||||
| "type": "double", | ||||||||||||
| "description": "Captures the size of the source code" | ||||||||||||
| }, | ||||||||||||
| { | ||||||||||||
| "name": "amazonqTimeToLoadHistory", | ||||||||||||
| "type": "int", | ||||||||||||
| "description": "The time (in milliseconds) it takes to load chat history from filesystem" | ||||||||||||
| }, | ||||||||||||
| { | ||||||||||||
| "name": "amazonqTimeToSearchHistory", | ||||||||||||
| "type": "int", | ||||||||||||
| "description": "The time (in milliseconds) it takes to search through all messages in chat history for a user-provided search string" | ||||||||||||
| }, | ||||||||||||
| { | ||||||||||||
| "name": "amazonqUploadIntent", | ||||||||||||
| "type": "string", | ||||||||||||
|
|
@@ -231,6 +246,17 @@ | |||||||||||
| "type": "boolean", | ||||||||||||
| "description": "Boolean value of whether or not a Cfn parameter file is provided." | ||||||||||||
| }, | ||||||||||||
| { | ||||||||||||
| "name": "chatHistoryAction", | ||||||||||||
| "type": "string", | ||||||||||||
| "allowedValues": [ | ||||||||||||
| "search", | ||||||||||||
| "export", | ||||||||||||
| "open", | ||||||||||||
| "delete" | ||||||||||||
| ], | ||||||||||||
| "description": "The type of action performed on chat history" | ||||||||||||
| }, | ||||||||||||
| { | ||||||||||||
| "name": "checkType", | ||||||||||||
| "type": "string", | ||||||||||||
|
|
@@ -1009,6 +1035,21 @@ | |||||||||||
| "type": "int", | ||||||||||||
| "description": "Index of the code block inside a message in the conversation." | ||||||||||||
| }, | ||||||||||||
| { | ||||||||||||
| "name": "cwsprChatCodeContextCount", | ||||||||||||
| "type": "int", | ||||||||||||
| "description": "Number of code symbols added to context" | ||||||||||||
| }, | ||||||||||||
| { | ||||||||||||
| "name": "cwsprChatCodeContextLength", | ||||||||||||
| "type": "int", | ||||||||||||
| "description": "Total length of code symbols added to context" | ||||||||||||
| }, | ||||||||||||
| { | ||||||||||||
| "name": "cwsprChatCodeContextTruncatedLength", | ||||||||||||
| "type": "int", | ||||||||||||
| "description": "Truncated length of code symbols added to context" | ||||||||||||
| }, | ||||||||||||
| { | ||||||||||||
| "name": "cwsprChatConversationId", | ||||||||||||
| "type": "string", | ||||||||||||
|
|
@@ -1471,6 +1512,15 @@ | |||||||||||
| ], | ||||||||||||
| "description": "The experiment action taken action taken" | ||||||||||||
| }, | ||||||||||||
| { | ||||||||||||
| "name": "exportFormat", | ||||||||||||
| "type": "string", | ||||||||||||
| "allowedValues": [ | ||||||||||||
|
||||||||||||
| { | |
| "name": "filenameExt", | |
| "type": "string", | |
| "description": "Filename extension (examples: .txt, .yml, .yaml, .asl.yaml, ...), or empty string if the filename does not contain dot (.) between two chars." | |
| }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we have action that can be reused here unless we have strong reason for prefix:
aws-toolkit-common/telemetry/definitions/commonDefinitions.json
Lines 18 to 21 in df56506
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no thats fine, i can switch it to the existing action type