Skip to content

Commit 338e868

Browse files
authored
refactor: use isAmazonQ() aws#4858
1 parent 6bbeaf2 commit 338e868

File tree

3 files changed

+10
-17
lines changed

3 files changed

+10
-17
lines changed

packages/core/src/shared/telemetry/activation.ts

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@ import { isAutomation, isReleaseVersion } from '../vscode/env'
1919
import { AWSProduct } from './clienttelemetry'
2020
import { DefaultTelemetryClient } from './telemetryClient'
2121
import { Commands } from '../vscode/commands2'
22-
import { VSCODE_EXTENSION_ID } from '../utilities'
2322

2423
export const noticeResponseViewSettings = localize('AWS.telemetry.notificationViewSettings', 'Settings')
2524
export const noticeResponseOk = localize('AWS.telemetry.notificationOk', 'OK')
@@ -53,17 +52,14 @@ export async function activate(
5352
globals.telemetry.telemetryEnabled = config.isEnabled()
5453

5554
extensionContext.subscriptions.push(
56-
(globals.context.extension.id === VSCODE_EXTENSION_ID.amazonq
57-
? config.amazonQConfig
58-
: config.toolkitConfig
59-
).onDidChange(event => {
55+
(isAmazonQ() ? config.amazonQConfig : config.toolkitConfig).onDidChange(event => {
6056
if (event.key === 'telemetry') {
6157
globals.telemetry.telemetryEnabled = config.isEnabled()
6258
}
6359
})
6460
)
6561

66-
if (extensionContext.extension.id === VSCODE_EXTENSION_ID.amazonq) {
62+
if (isAmazonQ()) {
6763
extensionContext.subscriptions.push(
6864
Commands.register('aws.amazonq.setupTelemetryId', async () => {
6965
await setupTelemetryId(extensionContext)

packages/core/src/shared/telemetry/util.ts

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ import { mapMetadata } from './telemetryLogger'
1515
import { Result } from './telemetry.gen'
1616
import { MetricDatum } from './clienttelemetry'
1717
import { isValidationExemptMetric } from './exemptMetrics'
18-
import { isCloud9, isSageMaker } from '../../shared/extensionUtilities'
18+
import { isAmazonQ, isCloud9, isSageMaker } from '../../shared/extensionUtilities'
1919
import { isExtensionInstalled, VSCODE_EXTENSION_ID } from '../utilities'
2020
import { randomUUID } from '../../common/crypto'
2121
import { activateExtension } from '../utilities/vsCodeUtils'
@@ -55,9 +55,7 @@ export class TelemetryConfig {
5555
}
5656

5757
public isEnabled(): boolean {
58-
return (
59-
globals.context.extension.id === VSCODE_EXTENSION_ID.amazonq ? this.amazonQConfig : this.toolkitConfig
60-
).get(`telemetry`, true)
58+
return (isAmazonQ() ? this.amazonQConfig : this.toolkitConfig).get(`telemetry`, true)
6159
}
6260

6361
public async initAmazonQSetting() {
@@ -119,10 +117,9 @@ export async function getUserAgent(
119117
opt?: { includePlatform?: boolean; includeClientId?: boolean },
120118
globalState = globals.context.globalState
121119
): Promise<string> {
122-
const pairs =
123-
globals.context.extension.id === VSCODE_EXTENSION_ID.amazonq
124-
? [`AmazonQ-For-VSCode/${extensionVersion}`]
125-
: [`AWS-Toolkit-For-VSCode/${extensionVersion}`]
120+
const pairs = isAmazonQ()
121+
? [`AmazonQ-For-VSCode/${extensionVersion}`]
122+
: [`AWS-Toolkit-For-VSCode/${extensionVersion}`]
126123

127124
if (opt?.includePlatform) {
128125
pairs.push(platformPair())
@@ -225,7 +222,7 @@ export async function setupTelemetryId(extensionContext: vscode.ExtensionContext
225222
await vscode.commands.executeCommand('aws.amazonq.setupTelemetryId')
226223
})
227224
}
228-
} else if (extensionContext.extension.id === VSCODE_EXTENSION_ID.amazonq) {
225+
} else if (isAmazonQ()) {
229226
getLogger().debug(`telemetry: Set telemetry client id to ${storedClientId}`)
230227
await globals.context.globalState.update(telemetryClientIdGlobalStatekey, storedClientId)
231228
} else {

packages/core/src/shared/vscode/commands2.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ import { ToolkitError } from '../errors'
1515
import crypto from 'crypto'
1616
import { keysAsInt } from '../utilities/tsUtils'
1717
import { partialClone } from '../utilities/collectionUtils'
18-
import { VSCODE_EXTENSION_ID } from '../utilities'
18+
import { isAmazonQ } from '../extensionUtilities'
1919

2020
type Callback = (...args: any[]) => any
2121
type CommandFactory<T extends Callback, U extends any[]> = (...parameters: U) => T
@@ -636,7 +636,7 @@ async function runCommand<T extends Callback>(fn: T, info: CommandInfo<T>): Prom
636636

637637
try {
638638
if (info.autoconnect === true) {
639-
const prefix = globals.context.extension.id === VSCODE_EXTENSION_ID.amazonq ? 'amazonq' : 'toolkit'
639+
const prefix = isAmazonQ() ? 'amazonq' : 'toolkit'
640640
await vscode.commands.executeCommand(`_aws.${prefix}.auth.autoConnect`)
641641
}
642642

0 commit comments

Comments
 (0)