test: update getApiMetrics tests for inverted API protocol logic

Author: daniel-lxs

src/shared/__tests__/getApiMetrics.spec.ts

@@ -61,7 +61,7 @@ describe("getApiMetrics", () => {
 			expect(result.totalCacheWrites).toBe(5)
 			expect(result.totalCacheReads).toBe(10)
 			expect(result.totalCost).toBe(0.005)
-			expect(result.contextTokens).toBe(315) // 100 + 200 + 5 + 10 (includes cache tokens)
+			expect(result.contextTokens).toBe(300) // 100 + 200 (OpenAI default, no cache tokens)
 		})
 
 		it("should calculate metrics from multiple api_req_started messages", () => {
@@ -83,7 +83,7 @@ describe("getApiMetrics", () => {
 			expect(result.totalCacheWrites).toBe(8) // 5 + 3
 			expect(result.totalCacheReads).toBe(17) // 10 + 7
 			expect(result.totalCost).toBe(0.008) // 0.005 + 0.003
-			expect(result.contextTokens).toBe(210) // 50 + 150 + 3 + 7 (from the last message, includes cache tokens)
+			expect(result.contextTokens).toBe(200) // 50 + 150 (OpenAI default, no cache tokens)
 		})
 
 		it("should calculate metrics from condense_context messages", () => {
@@ -123,7 +123,7 @@ describe("getApiMetrics", () => {
 			expect(result.totalCacheWrites).toBe(8) // 5 + 3
 			expect(result.totalCacheReads).toBe(17) // 10 + 7
 			expect(result.totalCost).toBe(0.01) // 0.005 + 0.002 + 0.003
-			expect(result.contextTokens).toBe(210) // 50 + 150 + 3 + 7 (from the last api_req_started message, includes cache tokens)
+			expect(result.contextTokens).toBe(200) // 50 + 150 (OpenAI default, no cache tokens)
 		})
 	})
 
@@ -243,8 +243,8 @@ describe("getApiMetrics", () => {
 			expect(result.totalCost).toBe(0.005)
 
 			// The implementation will use the last message that has any tokens
-			// In this case, it's the cacheReads message (10)
-			expect(result.contextTokens).toBe(10) // Only cacheReads from the last message
+			// In this case, it's the message with tokensOut:200 (since the last few messages have no tokensIn/Out)
+			expect(result.contextTokens).toBe(200) // 0 + 200 (from the tokensOut message)
 		})
 
 		it("should handle non-number values in api_req_started message", () => {
@@ -265,7 +265,7 @@ describe("getApiMetrics", () => {
 			expect(result.totalCost).toBe(0)
 
 			// The implementation concatenates all token values including cache tokens
-			expect(result.contextTokens).toBe("not-a-numbernot-a-numbernot-a-numbernot-a-number") // tokensIn + tokensOut + cacheWrites + cacheReads
+			expect(result.contextTokens).toBe("not-a-numbernot-a-number") // tokensIn + tokensOut (OpenAI default)
 		})
 	})
 
@@ -279,7 +279,7 @@ describe("getApiMetrics", () => {
 			const result = getApiMetrics(messages)
 
 			// Should use the values from the last api_req_started message
-			expect(result.contextTokens).toBe(210) // 50 + 150 + 3 + 7 (includes cache tokens)
+			expect(result.contextTokens).toBe(200) // 50 + 150 (OpenAI default, no cache tokens)
 		})
 
 		it("should calculate contextTokens from the last condense_context message", () => {
@@ -305,7 +305,7 @@ describe("getApiMetrics", () => {
 			const result = getApiMetrics(messages)
 
 			// Should use the values from the last api_req_started message
-			expect(result.contextTokens).toBe(210) // 50 + 150 + 3 + 7 (includes cache tokens)
+			expect(result.contextTokens).toBe(200) // 50 + 150 (OpenAI default, no cache tokens)
 		})
 
 		it("should handle missing values when calculating contextTokens", () => {
@@ -320,7 +320,7 @@ describe("getApiMetrics", () => {
 			const result = getApiMetrics(messages)
 
 			// Should handle missing or invalid values
-			expect(result.contextTokens).toBe(15) // 0 + 0 + 5 + 10 (only cache tokens)
+			expect(result.contextTokens).toBe(0) // 0 + 0 (OpenAI default, no cache tokens)
 
 			// Restore console.error
 			console.error = originalConsoleError