refactor(core): Remove withPerformance from telemetry

Problem:
- withPerformance was an experiment to see if we could correlate cpu/memory with number of files

Solution:
- Cpu usage/memory usage in the extension host is way too volitile so we might as well clean up this code

Author: jpinkney-aws

packages/core/src/shared/performance/performance.ts

@@ -5,7 +5,6 @@
 
 import assert from 'assert'
 import { getLogger } from '../logger'
-import { isWeb } from '../extensionGlobals'
 
 interface PerformanceMetrics {
     /**
@@ -46,10 +45,6 @@ export class PerformanceTracker {
 
     constructor(private readonly name: string) {}
 
-    static enabled(name: string, trackPerformance: boolean): boolean {
-        return name === 'function_call' && trackPerformance && !isWeb()
-    }
-
     start() {
         this.#startPerformance = {
             cpuUsage: process.cpuUsage(),

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

@@ -23,7 +23,6 @@ import {
     getTelemetryResult,
 } from '../errors'
 import { entries, NumericKeys } from '../utilities/tsUtils'
-import { PerformanceTracker } from '../performance/performance'
 import { randomUUID } from '../crypto'
 
 const AsyncLocalStorage: typeof AsyncLocalStorageClass =
@@ -135,10 +134,7 @@ export type SpanOptions = {
  */
 export class TelemetrySpan<T extends MetricBase = MetricBase> {
     #startTime?: Date
-    #options: SpanOptions & {
-        trackPerformance: boolean
-    }
-    #performance?: PerformanceTracker
+    #options: SpanOptions
 
     private readonly state: Partial<T> = {}
     private readonly definition = definitions[this.name] ?? {
@@ -163,10 +159,6 @@ export class TelemetrySpan<T extends MetricBase = MetricBase> {
             // do emit by default
             emit: options?.emit === undefined ? true : options.emit,
             functionId: options?.functionId,
-            trackPerformance: PerformanceTracker.enabled(
-                this.name,
-                this.definition.trackPerformance && (options?.emit ?? false) // only track the performance if we are also emitting
-            ),
         }
 
         this.metricId = randomUUID()
@@ -209,10 +201,6 @@ export class TelemetrySpan<T extends MetricBase = MetricBase> {
      */
     public start(): this {
         this.#startTime = new globals.clock.Date()
-        if (this.#options.trackPerformance) {
-            ;(this.#performance ??= new PerformanceTracker(this.name)).start()
-        }
-
         return this
     }
 
@@ -224,21 +212,6 @@ export class TelemetrySpan<T extends MetricBase = MetricBase> {
     public stop(err?: unknown): void {
         const duration = this.startTime !== undefined ? globals.clock.Date.now() - this.startTime.getTime() : undefined
 
-        if (this.#options.trackPerformance) {
-            // TODO add these to the global metrics, right now it just forces them in the telemetry and ignores the type
-            // if someone enables this action
-            const performanceMetrics = this.#performance?.stop()
-            if (performanceMetrics) {
-                this.record({
-                    userCpuUsage: performanceMetrics.userCpuUsage,
-                    systemCpuUsage: performanceMetrics.systemCpuUsage,
-                    heapTotal: performanceMetrics.heapTotal,
-                    functionName: this.#options.functionId?.name ?? this.name,
-                    architecture: process.arch,
-                } as any)
-            }
-        }
-
         if (this.#options.emit) {
             this.emit({
                 duration,

packages/core/src/shared/telemetry/vscodeTelemetry.json

@@ -307,16 +307,6 @@
             "type": "string",
             "description": "The CPU architecture"
         },
-        {
-            "name": "totalFileSizeInMB",
-            "type": "int",
-            "description": "The total size of all files being sent to Amazon Q"
-        },
-        {
-            "name": "totalFiles",
-            "type": "int",
-            "description": "The total number of files being sent to Amazon Q"
-        },
         {
             "name": "traceId",
             "type": "string",
@@ -1165,14 +1155,6 @@
                     "type": "systemCpuUsage",
                     "required": false
                 },
-                {
-                    "type": "totalFiles",
-                    "required": false
-                },
-                {
-                    "type": "totalFileSizeInMB",
-                    "required": false
-                },
                 {
                     "type": "architecture",
                     "required": false
@@ -1186,8 +1168,7 @@
                     "required": true
                 }
             ],
-            "passive": true,
-            "trackPerformance": true
+            "passive": true
         },
         {
             "name": "webview_load",

packages/core/src/shared/utilities/workspaceUtils.ts

@@ -344,15 +344,13 @@ export async function collectFiles(
             }
 
             let totalSizeBytes = 0
-            let totalFiles = 0
             for (const rootPath of sourcePaths) {
                 const allFiles = await vscode.workspace.findFiles(
                     new vscode.RelativePattern(rootPath, '**'),
                     getExcludePattern()
                 )
 
                 const files = respectGitIgnore ? await filterOutGitignoredFiles(rootPath, allFiles) : allFiles
-                totalFiles += files.length
 
                 for (const file of files) {
                     const relativePath = getWorkspaceRelativePath(file.fsPath, { workspaceFolders })
@@ -385,7 +383,6 @@ export async function collectFiles(
                     })
                 }
             }
-            span.record({ totalFiles, totalFileSizeInMB: totalSizeBytes / (1024 * 1024) })
             return storage
         },
         {

packages/core/src/testInteg/shared/utilities/workspaceUtils.test.ts

@@ -301,8 +301,6 @@ describe('collectFiles', function () {
         )
 
         assertTelemetry('function_call', {
-            totalFiles: 8,
-            totalFileSizeInMB: 0.0002593994140625,
             functionName: 'collectFiles',
         })
     })