Skip to content

Commit cfc6099

Browse files
authored
fix(telemetry): bump toolkit telemetry generator version (#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
1 parent 0d49535 commit cfc6099

File tree

6 files changed

+10
-7
lines changed

6 files changed

+10
-7
lines changed

package-lock.json

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

packages/amazonq/test/unit/codewhisperer/tracker/codewhispererTracker.test.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -115,6 +115,8 @@ describe('codewhispererTracker', function () {
115115
codewhispererLanguage: 'java',
116116
credentialStartUrl: testStartUrl,
117117
codewhispererUserGroup: 'CrossFile',
118+
codewhispererCharactersAccepted: suggestion.originalString.length,
119+
codewhispererCharactersModified: 0,
118120
})
119121
})
120122
})

packages/core/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4044,7 +4044,7 @@
40444044
},
40454045
"devDependencies": {
40464046
"@aws-sdk/types": "^3.13.1",
4047-
"@aws-toolkits/telemetry": "^1.0.216",
4047+
"@aws-toolkits/telemetry": "^1.0.236",
40484048
"@aws/fully-qualified-names": "^2.1.4",
40494049
"@cspotcode/source-map-support": "^0.8.1",
40504050
"@sinonjs/fake-timers": "^10.0.2",

packages/core/src/codewhisperer/service/transformByQ/humanInTheLoopManager.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,6 @@ export class HumanInTheLoopManager {
110110
codeTransformJobId: transformByQState.getJobId(),
111111
result: MetadataResult.Fail,
112112
reason: errorMessage,
113-
codeTransformApiErrorMessage: errorMessage,
114113
})
115114
}
116115

packages/core/src/codewhisperer/service/transformByQ/transformApiHandler.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -407,7 +407,6 @@ export async function zipCode({ dependenciesFolder, humanInTheLoopFlag, modulePa
407407
} catch (e: any) {
408408
telemetry.codeTransform_logGeneralError.emit({
409409
codeTransformSessionId: CodeTransformTelemetryState.instance.getSessionId(),
410-
codeTransformApiErrorMessage: 'Failed to zip project',
411410
result: MetadataResult.Fail,
412411
reason: 'ZipCreationFailed',
413412
})

packages/core/src/codewhisperer/tracker/codewhispererTracker.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -138,6 +138,8 @@ export class CodeWhispererTracker {
138138
codewhispererLanguage: suggestion.language,
139139
credentialStartUrl: AuthUtil.instance.startUrl,
140140
codewhispererUserGroup: CodeWhispererUserGroupSettings.getUserGroup().toString(),
141+
codewhispererCharactersAccepted: suggestion.originalString.length,
142+
codewhispererCharactersModified: 0, // TODO: currently we don't have an accurate number for this field with existing implementation
141143
})
142144
// TODO:
143145
// Temperary comment out user modification event, need further discussion on how to calculate this metric

0 commit comments

Comments
 (0)