fix(telemetry): bump toolkit telemetry generator version (aws#5450)

* bump toolkit telemetry version * patch * patch * patch lock file * fix test compile error * use 0 instead of -1 as the default value before we figureout the modified character length
Bit-Quill · Aug 14, 2024 · cfc6099 · cfc6099
1 parent 0d49535
commit cfc6099
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 7 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/amazonq/test/unit/codewhisperer/tracker/codewhispererTracker.test.ts b/packages/amazonq/test/unit/codewhisperer/tracker/codewhispererTracker.test.ts
@@ -115,6 +115,8 @@ describe('codewhispererTracker', function () {
                 codewhispererLanguage: 'java',
                 credentialStartUrl: testStartUrl,
                 codewhispererUserGroup: 'CrossFile',
+                codewhispererCharactersAccepted: suggestion.originalString.length,
+                codewhispererCharactersModified: 0,
             })
         })
     })

diff --git a/packages/core/package.json b/packages/core/package.json
@@ -4044,7 +4044,7 @@
     },
     "devDependencies": {
         "@aws-sdk/types": "^3.13.1",
-        "@aws-toolkits/telemetry": "^1.0.216",
+        "@aws-toolkits/telemetry": "^1.0.236",
         "@aws/fully-qualified-names": "^2.1.4",
         "@cspotcode/source-map-support": "^0.8.1",
         "@sinonjs/fake-timers": "^10.0.2",

diff --git a/packages/core/src/codewhisperer/service/transformByQ/humanInTheLoopManager.ts b/packages/core/src/codewhisperer/service/transformByQ/humanInTheLoopManager.ts
@@ -110,7 +110,6 @@ export class HumanInTheLoopManager {
             codeTransformJobId: transformByQState.getJobId(),
             result: MetadataResult.Fail,
             reason: errorMessage,
-            codeTransformApiErrorMessage: errorMessage,
         })
     }
 

diff --git a/packages/core/src/codewhisperer/service/transformByQ/transformApiHandler.ts b/packages/core/src/codewhisperer/service/transformByQ/transformApiHandler.ts
@@ -407,7 +407,6 @@ export async function zipCode({ dependenciesFolder, humanInTheLoopFlag, modulePa
     } catch (e: any) {
         telemetry.codeTransform_logGeneralError.emit({
             codeTransformSessionId: CodeTransformTelemetryState.instance.getSessionId(),
-            codeTransformApiErrorMessage: 'Failed to zip project',
             result: MetadataResult.Fail,
             reason: 'ZipCreationFailed',
         })

diff --git a/packages/core/src/codewhisperer/tracker/codewhispererTracker.ts b/packages/core/src/codewhisperer/tracker/codewhispererTracker.ts
@@ -138,6 +138,8 @@ export class CodeWhispererTracker {
                     codewhispererLanguage: suggestion.language,
                     credentialStartUrl: AuthUtil.instance.startUrl,
                     codewhispererUserGroup: CodeWhispererUserGroupSettings.getUserGroup().toString(),
+                    codewhispererCharactersAccepted: suggestion.originalString.length,
+                    codewhispererCharactersModified: 0, // TODO: currently we don't have an accurate number for this field with existing implementation
                 })
                 // TODO:
                 // Temperary comment out user modification event, need further discussion on how to calculate this metric