Skip to content

Commit a37a12d

Browse files
telemetry(amazonq): update to new module metric name
Problem: Updated the telemetry types and had to change and 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. Signed-off-by: nkomonen-amazon <[email protected]>
1 parent ccd21b2 commit a37a12d

File tree

3 files changed

+6
-4
lines changed

3 files changed

+6
-4
lines changed

package-lock.json

Lines changed: 4 additions & 2 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.296",
44+
"@aws-toolkits/telemetry": "^1.0.303",
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
@@ -57,7 +57,7 @@ export function dispatchWebViewMessagesToApps(
5757
}
5858
case 'send-telemetry': {
5959
if (isOpenAgentTelemetry(msg)) {
60-
telemetry.toolkit_openModule.emit({
60+
telemetry.toolkit_didLoadModule.emit({
6161
module: msg.module,
6262
source: msg.trigger,
6363
result: 'Succeeded',

0 commit comments

Comments
 (0)