Skip to content

Commit 0459b0f

Browse files
FrankLiu4138claude
andcommitted
refactor(telemetry): replace extensionOutdated with extensionInstall start/end
Remove separate extensionOutdated event. Now extensionInstall has start (with action: install/update) and end (with action + result). Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
1 parent cf2b8fe commit 0459b0f

1 file changed

Lines changed: 6 additions & 8 deletions

File tree

src/upgrade/utility.ts

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -175,25 +175,23 @@ export async function checkOrInstallAppModExtensionForUpgrade(
175175
if (installedVersion && semver.gte(installedVersion, Upgrade.MIN_APPMOD_VERSION)) {
176176
return;
177177
}
178-
// Extension is installed but too old — update it
179-
sendInfo("", {
180-
operationName: "java.dependency.upgrade.extensionOutdated",
181-
installedVersion,
182-
minRequiredVersion: Upgrade.MIN_APPMOD_VERSION,
183-
});
184178
}
185179

186180
const action = ext ? "update" : "install";
181+
sendInfo("", {
182+
operationName: "java.dependency.upgrade.extensionInstall.start",
183+
action,
184+
});
187185
try {
188186
await commands.executeCommand("workbench.extensions.installExtension", ExtensionName.APP_MODERNIZATION_FOR_JAVA);
189187
sendInfo("", {
190-
operationName: "java.dependency.upgrade.extensionInstall",
188+
operationName: "java.dependency.upgrade.extensionInstall.end",
191189
action,
192190
result: "success",
193191
});
194192
} catch (e) {
195193
sendInfo("", {
196-
operationName: "java.dependency.upgrade.extensionInstall",
194+
operationName: "java.dependency.upgrade.extensionInstall.end",
197195
action,
198196
result: "failure",
199197
error: e instanceof Error ? e.message : String(e),

0 commit comments

Comments
 (0)