perf: remove useless code

Author: wenytang-ms

src/copilot/contextProvider.ts

@@ -74,14 +74,14 @@ function createJavaContextResolver(): ContextResolverFunction {
 async function resolveJavaContext(request: ResolveRequest, copilotCancel: vscode.CancellationToken): Promise<SupportedContextItem[]> {
     const items: SupportedContextItem[] = [];
     const start = performance.now();
-    
+
     let dependenciesResult: CopilotHelper.IResolveResult | undefined;
     let importsResult: CopilotHelper.IResolveResult | undefined;
-    
+
     try {
         // Check for cancellation before starting
         JavaContextProviderUtils.checkCancellation(copilotCancel);
-        
+
         // Resolve project dependencies and convert to context items
         dependenciesResult = await CopilotHelper.resolveAndConvertProjectDependencies(
             vscode.window.activeTextEditor,
@@ -104,9 +104,9 @@ async function resolveJavaContext(request: ResolveRequest, copilotCancel: vscode
     } catch (error: any) {
         if (error instanceof CopilotCancellationError) {
             sendContextResolutionTelemetry(
-                request, 
-                start, 
-                items, 
+                request,
+                start,
+                items,
                 "cancelled_by_copilot",
                 undefined,
                 dependenciesResult?.emptyReason,
@@ -118,9 +118,9 @@ async function resolveJavaContext(request: ResolveRequest, copilotCancel: vscode
         }
         if (error instanceof vscode.CancellationError || error.message === CancellationError.CANCELED) {
             sendContextResolutionTelemetry(
-                request, 
-                start, 
-                items, 
+                request,
+                start,
+                items,
                 "cancelled_internally",
                 undefined,
                 dependenciesResult?.emptyReason,
@@ -133,9 +133,9 @@ async function resolveJavaContext(request: ResolveRequest, copilotCancel: vscode
 
         // Send telemetry for general errors (but continue with partial results)
         sendContextResolutionTelemetry(
-            request, 
-            start, 
-            items, 
+            request,
+            start,
+            items,
             "error_partial_results",
             error.message || "unknown_error",
             dependenciesResult?.emptyReason,
@@ -150,9 +150,9 @@ async function resolveJavaContext(request: ResolveRequest, copilotCancel: vscode
 
     // Send telemetry data once at the end for success case
     sendContextResolutionTelemetry(
-        request, 
-        start, 
-        items, 
+        request,
+        start,
+        items,
         "succeeded",
         undefined,
         dependenciesResult?.emptyReason,

src/copilot/copilotHelper.ts

@@ -73,7 +73,7 @@ export namespace CopilotHelper {
                 Commands.JAVA_PROJECT_GET_IMPORT_CLASS_CONTENT,
                 normalizedUri
             ) as Promise<IImportClassContentResult>;
-            
+
             // Build promises array for race condition
             // Note: Client-side timeout is NECESSARY even if backend has timeout because:
             // 1. Network delays may prevent backend response from arriving
@@ -87,7 +87,7 @@ export namespace CopilotHelper {
                     }, 80); // 80ms client-side timeout (independent of backend timeout)
                 })
             ];
-            
+
             // Add cancellation promise if token provided
             if (cancellationToken) {
                 promises.push(
@@ -98,7 +98,7 @@ export namespace CopilotHelper {
                     })
                 );
             }
-            
+
             const result = await Promise.race(promises);
             if (!result) {
                 return {
@@ -171,7 +171,7 @@ export namespace CopilotHelper {
                     }, 40); // 40ms client-side timeout (independent of backend timeout)
                 })
             ];
-            
+
             // Add cancellation promise if token provided
             if (cancellationToken) {
                 promises.push(
@@ -182,7 +182,7 @@ export namespace CopilotHelper {
                     })
                 );
             }
-            
+
             const result = await Promise.race(promises);
             if (!result) {
                 return {
@@ -238,7 +238,7 @@ export namespace CopilotHelper {
         checkCancellation: (token: CancellationToken) => void
     ): Promise<IResolveResult> {
         const items: any[] = [];
-        
+
         // Check if active editor exists
         if (!activeEditor) {
             return { items: [], emptyReason: EmptyReason.NoActiveEditor, itemCount: 0 };
@@ -287,7 +287,7 @@ export namespace CopilotHelper {
         checkCancellation: (token: CancellationToken) => void
     ): Promise<IResolveResult> {
         const items: any[] = [];
-        
+
         // Check if there's an active editor with a Java document
         if (!activeEditor) {
             return { items: [], emptyReason: EmptyReason.NoActiveEditor, itemCount: 0 };
@@ -310,7 +310,7 @@ export namespace CopilotHelper {
         if (importClassResult.isEmpty && importClassResult.emptyReason) {
             return { items: [], emptyReason: importClassResult.emptyReason, itemCount: 0 };
         }
-        
+
         // Check for cancellation before processing results
         checkCancellation(copilotCancel);
         if (importClassResult.classInfoList && importClassResult.classInfoList.length > 0) {

src/copilot/utils.ts

@@ -248,7 +248,7 @@ export function sendContextResolutionTelemetry(
         "dependenciesCount": dependenciesCount ?? 0,
         "importsCount": importsCount ?? 0
     };
-    
+
     // Add empty reasons if present
     if (dependenciesEmptyReason) {
         telemetryData.dependenciesEmptyReason = dependenciesEmptyReason;
@@ -259,6 +259,6 @@ export function sendContextResolutionTelemetry(
     if (error) {
         telemetryData.error = error;
     }
-    
+
     sendInfo("", telemetryData);
 }