Skip to content

Commit 8aac27b

Browse files
telemetry(amazonq): update to new module metric name (#6717)
## Problem: Updated the telemetry types and had to change an existing name to match the new one. ## Solution: use `toolkit_didLoadModule` to indicate that the module successfully opened. In certain cases we will want to use `toolkit_willOpenModule` to indicate the start of it, but because it doesn't look like that was used already we can ignore it for now and revisit later. --- - Treat all work as PUBLIC. Private `feature/x` branches will not be squash-merged at release time. - Your code changes must meet the guidelines in [CONTRIBUTING.md](https://github.com/aws/aws-toolkit-vscode/blob/master/CONTRIBUTING.md#guidelines). - License: I confirm that my contribution is made under the terms of the Apache 2.0 license. Signed-off-by: nkomonen-amazon <[email protected]>
1 parent df6ebb0 commit 8aac27b

File tree

3 files changed

+8
-7
lines changed

3 files changed

+8
-7
lines changed

package-lock.json

Lines changed: 6 additions & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@
4141
"skippedTestReport": "ts-node ./scripts/skippedTestReport.ts ./packages/amazonq/test/e2e/"
4242
},
4343
"devDependencies": {
44-
"@aws-toolkits/telemetry": "^1.0.301",
44+
"@aws-toolkits/telemetry": "^1.0.304",
4545
"@playwright/browser-chromium": "^1.43.1",
4646
"@stylistic/eslint-plugin": "^2.11.0",
4747
"@types/he": "^1.2.3",

packages/core/src/amazonq/webview/messages/messageDispatcher.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ export function dispatchWebViewMessagesToApps(
5959
}
6060
case 'send-telemetry': {
6161
if (isOpenAgentTelemetry(msg)) {
62-
telemetry.toolkit_openModule.emit({
62+
telemetry.toolkit_didLoadModule.emit({
6363
module: msg.module,
6464
source: msg.trigger,
6565
result: 'Succeeded',

0 commit comments

Comments
 (0)