scriptoriumadmin/Roo-Code
1
0
Fork 0
mirror of https://github.com/RooVetGit/Roo-Code.git synced 2026-09-11 22:51:26 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix: preserve openrouter reasoning details and plain-text reasoning history

Browse source
This commit is contained in:
Hannes Rudolph 2025-11-21 15:29:18 -07:00
parent 7f352dacf5
commit f1764e10d3
1 changed files with 55 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

63
src/core/task/Task.ts
View file

@ -651,10 +651,6 @@ export class Task extends EventEmitter<TaskEvents> implements TaskLike {
// We only persist data reported by the current response body.
let metadata = this.api.getGenerationMetadata?.()
if (reasoning) {
metadata = { ...metadata, reasoning }
}
if (message.role === "assistant") {
// Start from the original assistant message
const messageWithTs: any = {
@ -698,6 +694,52 @@ export class Task extends EventEmitter<TaskEvents> implements TaskLike {
}
}
// If we captured plain-text reasoning during streaming, persist it alongside the assistant message.
if (reasoning) {
const modelInfo = this.api.getModel().info
const shouldPreserveReasoning = modelInfo.preserveReasoning === true
// Normalize existing content into an array for manipulation
let contentArray: Anthropic.Messages.ContentBlockParam[]
if (Array.isArray(messageWithTs.content)) {
contentArray = messageWithTs.content as Anthropic.Messages.ContentBlockParam[]
} else if (typeof messageWithTs.content === "string") {
contentArray = [
{ type: "text", text: messageWithTs.content } satisfies Anthropic.Messages.TextBlockParam,
]
} else {
contentArray = []
}
if (shouldPreserveReasoning) {
// For models that preserve reasoning, prepend reasoning in <think> tags to the first text block.
let targetTextBlock = contentArray.find((block) => block.type === "text") as
| Anthropic.Messages.TextBlockParam
| undefined
if (!targetTextBlock) {
targetTextBlock = { type: "text", text: "" }
contentArray.unshift(targetTextBlock)
}
const existingText = targetTextBlock.text ?? ""
targetTextBlock.text = `<think>${reasoning}</think>\n${existingText}`
} else {
// For all other providers, store reasoning as a separate plain-text reasoning block
// before the assistant's visible response. This is kept in history but stripped
// from subsequent API requests unless preserveReasoning is enabled.
const reasoningBlock: any = {
type: "reasoning",
text: reasoning,
summary: [],
}
contentArray = [reasoningBlock, ...contentArray]
}
messageWithTs.content = contentArray
}
this.apiConversationHistory.push(messageWithTs)
} else {
const messageWithTs = { ...message, ts: Date.now() }
@ -2084,14 +2126,14 @@ export class Task extends EventEmitter<TaskEvents> implements TaskLike {
outputTokens,
cacheWriteTokens,
cacheReadTokens,
)
)
: calculateApiCostOpenAI(
streamModelInfo,
inputTokens,
outputTokens,
cacheWriteTokens,
cacheReadTokens,
)
)
this.clineMessages[lastApiReqIndex].text = JSON.stringify({
...existingData,
@ -2520,7 +2562,7 @@ export class Task extends EventEmitter<TaskEvents> implements TaskLike {
currentItem.retryAttempt ?? 0,
error,
streamingFailedMessage,
)
)
// Check if task was aborted during the backoff
if (this.abort) {
@ -3377,7 +3419,12 @@ export class Task extends EventEmitter<TaskEvents> implements TaskLike {
// Legacy support: Handle embedded reasoning blocks from the transitional period
// (where reasoning was a block inside content with type="reasoning")
if (msg.role === "assistant" && Array.isArray(msg.content)) {
const firstBlock = msg.content[0]
const contentArray = msg.content as Anthropic.Messages.ContentBlockParam[]
const [firstBlock, ...rest] = contentArray
const hasPlainTextReasoning =
firstBlock &&
(firstBlock as any).type === "reasoning" &&
typeof (firstBlock as any).text === "string"
if (
firstBlock &&
(firstBlock as any).type === "reasoning" &&