telemetry: add fix rate metrics for hover events (#4144)

ctlai95 · web-flow · commit 56fa3c32dcc4 · 2024-02-22T16:13:01.000-08:00
diff --git a/.changes/next-release/feature-71ef1996-ea3e-4f76-bda2-3eb6cc2bcdde.json b/.changes/next-release/feature-71ef1996-ea3e-4f76-bda2-3eb6cc2bcdde.json
@@ -0,0 +1,4 @@
+{
+  "type" : "feature",
+  "description" : "Security issue hover telemetry includes additional metadata"
+}
diff --git a/gradle/libs.versions.toml b/gradle/libs.versions.toml
@@ -21,7 +21,7 @@ mockito = "5.10.0"
 mockitoKotlin = "5.2.1"
 mockk = "1.13.8"
 node-gradle = "7.0.1"
-telemetryGenerator = "1.0.186"
+telemetryGenerator = "1.0.190"
 testLogger = "3.1.0"
 testRetry = "1.5.2"
 # test-only; platform provides slf4j transitively at runtime. <233, 1.7.36; >=233, 2.0.9
diff --git a/jetbrains-core/src/software/aws/toolkits/jetbrains/services/codewhisperer/telemetry/CodeWhispererTelemetryService.kt b/jetbrains-core/src/software/aws/toolkits/jetbrains/services/codewhisperer/telemetry/CodeWhispererTelemetryService.kt
@@ -329,7 +329,8 @@ class CodeWhispererTelemetryService {
         CodewhispererTelemetry.codeScanIssueHover(
             findingId = issue.findingId,
             detectorId = issue.detectorId,
-            ruleId = issue.ruleId
+            ruleId = issue.ruleId,
+            includesFix = issue.suggestedFixes.isNotEmpty()
         )
     }
 

-Original file line number
+Diff line change
@@ @@ -0,0 +1,4 @@ @@
 +{
 +  "type" : "feature",
 +  "description" : "Security issue hover telemetry includes additional metadata"
 +}
Original file line number	Diff line number	Diff line change
`@@ -329,7 +329,8 @@ class CodeWhispererTelemetryService {`
`329`	`329`	`CodewhispererTelemetry.codeScanIssueHover(`
`330`	`330`	`findingId = issue.findingId,`
`331`	`331`	`detectorId = issue.detectorId,`
`332`		`- ruleId = issue.ruleId`
	`332`	`+ ruleId = issue.ruleId,`
	`333`	`+ includesFix = issue.suggestedFixes.isNotEmpty()`
`333`	`334`	`)`
`334`	`335`	`}`
`335`	`336`