From 72ef4d30bf762333b5d3f020c742604f541b0fd2 Mon Sep 17 00:00:00 2001 From: daniel-lxs Date: Mon, 27 Oct 2025 16:27:14 -0500 Subject: [PATCH] fix: address review comments from roomote - Remove unused contextTokens variable in getEnvironmentDetails.ts - Remove duplicate includeCurrentTime/Cost properties in ExtensionMessage.ts - Properties are already defined in the Pick type union --- src/core/environment/getEnvironmentDetails.ts | 2 +- src/shared/ExtensionMessage.ts | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/src/core/environment/getEnvironmentDetails.ts b/src/core/environment/getEnvironmentDetails.ts index aceae059a9..30d9cd0b0d 100644 --- a/src/core/environment/getEnvironmentDetails.ts +++ b/src/core/environment/getEnvironmentDetails.ts @@ -207,7 +207,7 @@ export async function getEnvironmentDetails(cline: Task, includeFileDetails: boo // Add context tokens information (if enabled). if (includeCurrentCost) { - const { contextTokens, totalCost } = getApiMetrics(cline.clineMessages) + const { totalCost } = getApiMetrics(cline.clineMessages) details += `\n\n# Current Cost\n${totalCost !== null ? `$${totalCost.toFixed(2)}` : "(Not available)"}` } diff --git a/src/shared/ExtensionMessage.ts b/src/shared/ExtensionMessage.ts index 43097d4dc2..5929e7a950 100644 --- a/src/shared/ExtensionMessage.ts +++ b/src/shared/ExtensionMessage.ts @@ -354,8 +354,6 @@ export type ExtensionState = Pick< openRouterImageApiKey?: string openRouterUseMiddleOutTransform?: boolean messageQueue?: QueuedMessage[] - includeCurrentTime?: boolean - includeCurrentCost?: boolean lastShownAnnouncementId?: string apiModelId?: string mcpServers?: McpServer[]