Skip to content

Commit 3edd491

Browse files
authored
fix(telemetry): missing "result" field in amazonq metrics #4484
1 parent e665622 commit 3edd491

File tree

1 file changed

+17
-4
lines changed

1 file changed

+17
-4
lines changed

packages/toolkit/src/codewhispererChat/controllers/chat/telemetryHelper.ts

Lines changed: 17 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ export function logSendTelemetryEventFailure(error: any) {
5151

5252
export function recordTelemetryChatRunCommand(type: CwsprChatCommandType, command?: string) {
5353
telemetry.amazonq_runCommand.emit({
54+
result: 'Succeeded',
5455
cwsprChatCommandType: type,
5556
cwsprChatCommandName: command,
5657
credentialStartUrl: AuthUtil.instance.startUrl,
@@ -91,19 +92,19 @@ export class CWCTelemetryHelper {
9192
}
9293

9394
public recordOpenChat() {
94-
telemetry.amazonq_openChat.emit({ passive: true })
95+
telemetry.amazonq_openChat.emit({ result: 'Succeeded', passive: true })
9596
}
9697

9798
public recordCloseChat() {
98-
telemetry.amazonq_closeChat.emit({ passive: true })
99+
telemetry.amazonq_closeChat.emit({ result: 'Succeeded', passive: true })
99100
}
100101

101102
public recordEnterFocusChat() {
102-
telemetry.amazonq_enterFocusChat.emit({ passive: true })
103+
telemetry.amazonq_enterFocusChat.emit({ result: 'Succeeded', passive: true })
103104
}
104105

105106
public recordExitFocusChat() {
106-
telemetry.amazonq_exitFocusChat.emit({ passive: true })
107+
telemetry.amazonq_exitFocusChat.emit({ result: 'Succeeded', passive: true })
107108
}
108109

109110
public async recordFeedback(message: ChatItemFeedbackMessage) {
@@ -153,6 +154,7 @@ export class CWCTelemetryHelper {
153154
case 'insert_code_at_cursor_position':
154155
message = message as InsertCodeAtCursorPosition
155156
event = {
157+
result: 'Succeeded',
156158
cwsprChatConversationId: conversationId ?? '',
157159
credentialStartUrl: AuthUtil.instance.startUrl,
158160
cwsprChatMessageId: message.messageId,
@@ -165,6 +167,7 @@ export class CWCTelemetryHelper {
165167
case 'code_was_copied_to_clipboard':
166168
message = message as CopyCodeToClipboard
167169
event = {
170+
result: 'Succeeded',
168171
cwsprChatConversationId: conversationId ?? '',
169172
credentialStartUrl: AuthUtil.instance.startUrl,
170173
cwsprChatMessageId: message.messageId,
@@ -177,6 +180,7 @@ export class CWCTelemetryHelper {
177180
case 'follow-up-was-clicked':
178181
message = message as PromptMessage
179182
event = {
183+
result: 'Succeeded',
180184
cwsprChatConversationId: conversationId ?? '',
181185
credentialStartUrl: AuthUtil.instance.startUrl,
182186
cwsprChatMessageId: message.messageId,
@@ -186,6 +190,7 @@ export class CWCTelemetryHelper {
186190
case 'chat-item-voted':
187191
message = message as ChatItemVotedMessage
188192
event = {
193+
result: 'Succeeded',
189194
cwsprChatMessageId: message.messageId,
190195
cwsprChatConversationId: conversationId ?? '',
191196
credentialStartUrl: AuthUtil.instance.startUrl,
@@ -195,6 +200,7 @@ export class CWCTelemetryHelper {
195200
case 'source-link-click':
196201
message = message as SourceLinkClickMessage
197202
event = {
203+
result: 'Succeeded',
198204
cwsprChatMessageId: message.messageId,
199205
cwsprChatConversationId: conversationId ?? '',
200206
credentialStartUrl: AuthUtil.instance.startUrl,
@@ -205,6 +211,7 @@ export class CWCTelemetryHelper {
205211
case 'response-body-link-click':
206212
message = message as ResponseBodyLinkClickMessage
207213
event = {
214+
result: 'Succeeded',
208215
cwsprChatMessageId: message.messageId,
209216
cwsprChatConversationId: conversationId ?? '',
210217
credentialStartUrl: AuthUtil.instance.startUrl,
@@ -215,6 +222,7 @@ export class CWCTelemetryHelper {
215222
case 'footer-info-link-click':
216223
message = message as FooterInfoLinkClick
217224
event = {
225+
result: 'Succeeded',
218226
cwsprChatMessageId: 'footer',
219227
cwsprChatConversationId: conversationId ?? '',
220228
credentialStartUrl: AuthUtil.instance.startUrl,
@@ -291,6 +299,7 @@ export class CWCTelemetryHelper {
291299
const telemetryUserIntent = this.getUserIntentForTelemetry(triggerPayload.userIntent)
292300

293301
telemetry.amazonq_startConversation.emit({
302+
result: 'Succeeded',
294303
cwsprChatConversationId: this.getConversationId(triggerEvent.tabID) ?? '',
295304
cwsprChatTriggerInteraction: this.getTriggerInteractionFromTriggerEvent(triggerEvent),
296305
cwsprChatConversationType: 'Chat',
@@ -305,6 +314,7 @@ export class CWCTelemetryHelper {
305314
const triggerEvent = this.triggerEventsStorage.getLastTriggerEventByTabID(message.tabID)
306315

307316
const event: AmazonqAddMessage = {
317+
result: 'Succeeded',
308318
cwsprChatConversationId: this.getConversationId(message.tabID) ?? '',
309319
cwsprChatMessageId: message.messageID,
310320
cwsprChatTriggerInteraction: this.getTriggerInteractionFromTriggerEvent(triggerEvent),
@@ -356,6 +366,7 @@ export class CWCTelemetryHelper {
356366
const triggerEvent = this.triggerEventsStorage.getLastTriggerEventByTabID(tabID)
357367

358368
telemetry.amazonq_messageResponseError.emit({
369+
result: 'Succeeded',
359370
cwsprChatConversationId: this.getConversationId(tabID) ?? '',
360371
cwsprChatTriggerInteraction: this.getTriggerInteractionFromTriggerEvent(triggerEvent),
361372
cwsprChatUserIntent: this.getUserIntentForTelemetry(triggerPayload.userIntent),
@@ -374,6 +385,7 @@ export class CWCTelemetryHelper {
374385
const conversationId = this.getConversationId(tabID)
375386
if (conversationId) {
376387
telemetry.amazonq_enterFocusConversation.emit({
388+
result: 'Succeeded',
377389
cwsprChatConversationId: conversationId,
378390
})
379391
}
@@ -383,6 +395,7 @@ export class CWCTelemetryHelper {
383395
const conversationId = this.getConversationId(tabID)
384396
if (conversationId) {
385397
telemetry.amazonq_exitFocusConversation.emit({
398+
result: 'Succeeded',
386399
cwsprChatConversationId: conversationId,
387400
})
388401
}

0 commit comments

Comments
 (0)