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<GlobalSettings> type union
This commit is contained in:
daniel-lxs 2025-10-27 16:27:14 -05:00
parent 0c83ee59df
commit 72ef4d30bf
No known key found for this signature in database
GPG key ID: 21C74479048B3AA6
2 changed files with 1 additions and 3 deletions

View file

@ -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)"}`
}

View file

@ -354,8 +354,6 @@ export type ExtensionState = Pick<
openRouterImageApiKey?: string
openRouterUseMiddleOutTransform?: boolean
messageQueue?: QueuedMessage[]
includeCurrentTime?: boolean
includeCurrentCost?: boolean
lastShownAnnouncementId?: string
apiModelId?: string
mcpServers?: McpServer[]