address comment: telemetry enabled check

Author: blva

src/telemetry/telemetry.ts

@@ -11,23 +11,6 @@ type EventResult = {
     error?: Error;
 };
 
-/**
- * Checks if telemetry is currently enabled
- * This is a method rather than a constant to capture runtime config changes
- *
- * Follows the Console Do Not Track standard (https://consoledonottrack.com/)
- * by respecting the DO_NOT_TRACK environment variable
- */
-export function isTelemetryEnabled(): boolean {
-    // Check if telemetry is explicitly disabled in config
-    if (config.telemetry === "disabled") {
-        return false;
-    }
-
-    const doNotTrack = "DO_NOT_TRACK" in process.env;
-    return !doNotTrack;
-}
-
 export class Telemetry {
     private readonly commonProperties: CommonProperties;
 
@@ -46,7 +29,7 @@ export class Telemetry {
      */
     public async emitEvents(events: BaseEvent[]): Promise<void> {
         try {
-            if (!isTelemetryEnabled()) {
+            if (!this.isTelemetryEnabled()) {
                 logger.info(LogId.telemetryEmitFailure, "telemetry", `Telemetry is disabled.`);
                 return;
             }
@@ -72,6 +55,23 @@ export class Telemetry {
         };
     }
 
+    /**
+     * Checks if telemetry is currently enabled
+     * This is a method rather than a constant to capture runtime config changes
+     *
+     * Follows the Console Do Not Track standard (https://consoledonottrack.com/)
+     * by respecting the DO_NOT_TRACK environment variable
+     */
+    public isTelemetryEnabled(): boolean {
+        // Check if telemetry is explicitly disabled in config
+        if (config.telemetry === "disabled") {
+            return false;
+        }
+
+        const doNotTrack = "DO_NOT_TRACK" in process.env;
+        return !doNotTrack;
+    }
+
     /**
      * Attempts to emit events through authenticated and unauthenticated clients
      * Falls back to caching if both attempts fail

src/tools/tool.ts

@@ -3,7 +3,7 @@ import type { McpServer, RegisteredTool, ToolCallback } from "@modelcontextproto
 import type { CallToolResult } from "@modelcontextprotocol/sdk/types.js";
 import { Session } from "../session.js";
 import logger, { LogId } from "../logger.js";
-import { Telemetry, isTelemetryEnabled } from "../telemetry/telemetry.js";
+import { Telemetry } from "../telemetry/telemetry.js";
 import { type ToolEvent } from "../telemetry/types.js";
 import { UserConfig } from "../config.js";
 
@@ -185,7 +185,7 @@ export abstract class ToolBase {
         result: CallToolResult,
         ...args: Parameters<ToolCallback<typeof this.argsShape>>
     ): Promise<void> {
-        if (!isTelemetryEnabled()) {
+        if (!this.telemetry.isTelemetryEnabled()) {
             return;
         }
         const duration = Date.now() - startTime;