refactor(deepseek): remove unused getReasoningContent() method

- Remove currentReasoningContent field and accumulation logic
- Remove getReasoningContent() method (was never called outside tests)
- Simplify reasoning_content handling to just yield without accumulating
- Remove associated tests for removed functionality

Author: hannesrudolph

src/api/providers/__tests__/deepseek.spec.ts

@@ -424,25 +424,6 @@ describe("DeepSeekHandler", () => {
 			expect(reasoningChunks[1].text).toBe(" I'll analyze step by step.")
 		})
 
-		it("should accumulate reasoning content via getReasoningContent()", async () => {
-			const reasonerHandler = new DeepSeekHandler({
-				...mockOptions,
-				apiModelId: "deepseek-reasoner",
-			})
-
-			// Before any API call, reasoning content should be undefined
-			expect(reasonerHandler.getReasoningContent()).toBeUndefined()
-
-			const stream = reasonerHandler.createMessage(systemPrompt, messages)
-			for await (const _chunk of stream) {
-				// Consume the stream
-			}
-
-			// After streaming, reasoning content should be accumulated
-			const reasoningContent = reasonerHandler.getReasoningContent()
-			expect(reasoningContent).toBe("Let me think about this... I'll analyze step by step.")
-		})
-
 		it("should pass thinking parameter for deepseek-reasoner model", async () => {
 			const reasonerHandler = new DeepSeekHandler({
 				...mockOptions,
@@ -511,36 +492,6 @@ describe("DeepSeekHandler", () => {
 			const toolCallChunks = chunks.filter((chunk) => chunk.type === "tool_call_partial")
 			expect(toolCallChunks.length).toBeGreaterThan(0)
 			expect(toolCallChunks[0].name).toBe("get_weather")
-
-			// Reasoning content should be accumulated for potential continuation
-			const reasoningContent = reasonerHandler.getReasoningContent()
-			expect(reasoningContent).toBeDefined()
-		})
-
-		it("should reset reasoning content for each new request", async () => {
-			const reasonerHandler = new DeepSeekHandler({
-				...mockOptions,
-				apiModelId: "deepseek-reasoner",
-			})
-
-			// First request
-			const stream1 = reasonerHandler.createMessage(systemPrompt, messages)
-			for await (const _chunk of stream1) {
-				// Consume the stream
-			}
-
-			const reasoningContent1 = reasonerHandler.getReasoningContent()
-			expect(reasoningContent1).toBeDefined()
-
-			// Second request should reset the reasoning content
-			const stream2 = reasonerHandler.createMessage(systemPrompt, messages)
-			for await (const _chunk of stream2) {
-				// Consume the stream
-			}
-
-			// The reasoning content should be fresh from the second request
-			const reasoningContent2 = reasonerHandler.getReasoningContent()
-			expect(reasoningContent2).toBe("Let me think about this... I'll analyze step by step.")
 		})
 	})
 })

src/api/providers/deepseek.ts

@@ -24,8 +24,6 @@ type DeepSeekChatCompletionParams = OpenAI.Chat.ChatCompletionCreateParamsStream
 }
 
 export class DeepSeekHandler extends OpenAiHandler {
-	private currentReasoningContent: string = ""
-
 	constructor(options: ApiHandlerOptions) {
 		super({
 			...options,
@@ -37,15 +35,6 @@ export class DeepSeekHandler extends OpenAiHandler {
 		})
 	}
 
-	/**
-	 * Returns the accumulated reasoning content from the last API call.
-	 * This is used for interleaved thinking with tool calls - the reasoning_content
-	 * needs to be passed back to the API in subsequent requests within the same turn.
-	 */
-	getReasoningContent(): string | undefined {
-		return this.currentReasoningContent || undefined
-	}
-
 	override getModel() {
 		const id = this.options.apiModelId ?? deepSeekDefaultModelId
 		const info = deepSeekModels[id as keyof typeof deepSeekModels] || deepSeekModels[deepSeekDefaultModelId]
@@ -64,9 +53,6 @@ export class DeepSeekHandler extends OpenAiHandler {
 		// Check if this is a thinking-enabled model (deepseek-reasoner)
 		const isThinkingModel = modelId.includes("deepseek-reasoner")
 
-		// Reset reasoning content accumulator for this request
-		this.currentReasoningContent = ""
-
 		// Convert messages to R1 format (merges consecutive same-role messages)
 		// This is required for DeepSeek which does not support successive messages with the same role
 		const convertedMessages = convertToR1Format([{ role: "user", content: systemPrompt }, ...messages])
@@ -128,12 +114,9 @@ export class DeepSeekHandler extends OpenAiHandler {
 			// Handle reasoning_content from DeepSeek's interleaved thinking
 			// This is the proper way DeepSeek sends thinking content in streaming
 			if ("reasoning_content" in delta && delta.reasoning_content) {
-				const reasoningText = (delta.reasoning_content as string) || ""
-				// Accumulate reasoning content for potential tool call continuation
-				this.currentReasoningContent += reasoningText
 				yield {
 					type: "reasoning",
-					text: reasoningText,
+					text: (delta.reasoning_content as string) || "",
 				}
 			}