refactor: remove telemetry logging from documentation links in various components

Author: hannesrudolph

webview-ui/src/components/chat/ChatView.tsx

@@ -1259,35 +1259,7 @@ const ChatViewComponent: React.ForwardRefRenderFunction<ChatViewRef, ChatViewPro
 								i18nKey="chat:about"
 								components={{
 									DocsLink: (
-										<a
-											href={buildDocLink("", "welcome")}
-											target="_blank"
-											rel="noopener noreferrer"
-											onClick={() => {
-												const url = buildDocLink("", "welcome")
-												const event = "docs_link_clicked"
-												const properties = { campaign: "welcome", page: "/" }
-												// TEMP LOGGING: UTM Telemetry Event
-												// eslint-disable-next-line no-console
-												console.log(
-													"%c[Telemetry]%c Event: %c%s%c | Properties: %c%o%c | URL: %c%s",
-													"background: #222; color: #fff; padding:2px 4px; border-radius:2px;",
-													"",
-													"color: #4FC3F7; font-weight:bold;",
-													event,
-													"",
-													"color: #81C784;",
-													properties,
-													"",
-													"color: #FFD54F;",
-													url,
-												)
-												vscode.postMessage({
-													type: "telemetry",
-													event,
-													properties,
-												} as any)
-											}}>
+										<a href={buildDocLink("", "welcome")} target="_blank" rel="noopener noreferrer">
 											the docs
 										</a>
 									),

webview-ui/src/components/chat/CommandExecutionError.tsx

@@ -2,7 +2,6 @@ import { useCallback } from "react"
 import { useTranslation, Trans } from "react-i18next"
 import { VSCodeLink } from "@vscode/webview-ui-toolkit/react"
 import { buildDocLink } from "../../utils/docLinks"
-import { vscode } from "@/utils/vscode"
 
 export const CommandExecutionError = () => {
 	const { t } = useTranslation()
@@ -34,32 +33,7 @@ export const CommandExecutionError = () => {
 					className="underline"
 					style={{ color: "inherit" }}
 					target="_blank"
-					rel="noopener noreferrer"
-					onClick={() => {
-						const url = buildDocLink("troubleshooting/shell-integration/", "error_tooltip")
-						const event = "docs_link_clicked"
-						const properties = { campaign: "error_tooltip", page: "/troubleshooting/shell-integration/" }
-						// TEMP LOGGING: UTM Telemetry Event
-						// eslint-disable-next-line no-console
-						console.log(
-							"%c[Telemetry]%c Event: %c%s%c | Properties: %c%o%c | URL: %c%s",
-							"background: #222; color: #fff; padding:2px 4px; border-radius:2px;",
-							"",
-							"color: #4FC3F7; font-weight:bold;",
-							event,
-							"",
-							"color: #81C784;",
-							properties,
-							"",
-							"color: #FFD54F;",
-							url,
-						)
-						vscode.postMessage({
-							type: "telemetry",
-							event,
-							properties,
-						} as any)
-					}}>
+					rel="noopener noreferrer">
 					{t("chat:shellIntegration.troubleshooting")}
 				</a>
 			</div>

webview-ui/src/components/settings/ApiOptions.tsx

@@ -288,35 +288,7 @@ const ApiOptions = ({
 					<label className="block font-medium mb-1">{t("settings:providers.apiProvider")}</label>
 					{docs && (
 						<div className="text-xs text-vscode-descriptionForeground">
-							<VSCodeLink
-								href={docs.url}
-								className="hover:text-vscode-foreground"
-								target="_blank"
-								onClick={() => {
-									const url = docs.url
-									const event = "docs_link_clicked"
-									const properties = { campaign: "provider_docs", page: docs.page }
-									// TEMP LOGGING: UTM Telemetry Event
-									// eslint-disable-next-line no-console
-									console.log(
-										"%c[Telemetry]%c Event: %c%s%c | Properties: %c%o%c | URL: %c%s",
-										"background: #222; color: #fff; padding:2px 4px; border-radius:2px;",
-										"",
-										"color: #4FC3F7; font-weight:bold;",
-										event,
-										"",
-										"color: #81C784;",
-										properties,
-										"",
-										"color: #FFD54F;",
-										url,
-									)
-									vscode.postMessage({
-										type: "telemetry",
-										event,
-										properties,
-									} as any)
-								}}>
+							<VSCodeLink href={docs.url} className="hover:text-vscode-foreground" target="_blank">
 								{t("settings:providers.providerDocumentation", { provider: docs.name })}
 							</VSCodeLink>
 						</div>

webview-ui/src/components/welcome/RooTips.tsx

@@ -4,7 +4,6 @@ import { useState, useEffect } from "react"
 import clsx from "clsx"
 
 import { buildDocLink } from "@src/utils/docLinks"
-import { vscode } from "@/utils/vscode"
 
 const tips = [
 	{
@@ -67,21 +66,8 @@ const RooTips = ({ cycle = false }: RooTipsProps) => {
 						{" "}
 						<span className={`codicon ${currentTip.icon}`}></span>
 						<span>
-							<VSCodeLink
-								href={currentTip.href}
-								onClick={() =>
-									vscode.postMessage({
-										type: "telemetry",
-										event: "docs_link_clicked",
-										properties: {
-											campaign: "tips",
-											page: currentTip.page,
-										},
-									} as any)
-								}>
-								{t(currentTip.titleKey)}
-							</VSCodeLink>
-							: {t(currentTip.descriptionKey)}
+							<VSCodeLink href={currentTip.href}>{t(currentTip.titleKey)}</VSCodeLink>:{" "}
+							{t(currentTip.descriptionKey)}
 						</span>
 					</div>
 				</>
@@ -92,21 +78,7 @@ const RooTips = ({ cycle = false }: RooTipsProps) => {
 						className="flex items-center gap-2 text-vscode-editor-foreground font-vscode max-w-[250px]">
 						<span className={`codicon ${tip.icon}`}></span>
 						<span>
-							<VSCodeLink
-								href={tip.href}
-								onClick={() =>
-									vscode.postMessage({
-										type: "telemetry",
-										event: "docs_link_clicked",
-										properties: {
-											campaign: "tips",
-											page: tip.page,
-										},
-									} as any)
-								}>
-								{t(tip.titleKey)}
-							</VSCodeLink>
-							: {t(tip.descriptionKey)}
+							<VSCodeLink href={tip.href}>{t(tip.titleKey)}</VSCodeLink>: {t(tip.descriptionKey)}
 						</span>
 					</div>
 				))