fix(webview): define chevron icon via codicon and add missing isExpanded dep to useMemo; test: remove duplicate GPT-5 Pro background-mode test; chore(core): remove temp debug log

This commit is contained in:
Hannes Rudolph 2025-10-24 16:50:14 -06:00
parent e118846aa5
commit 7cff4db96c
4 changed files with 13 additions and 31 deletions

View file

@ -51,9 +51,8 @@ export const openAiNativeModels = {
maxTokens: 128000,
contextWindow: 400000,
supportsImages: true,
supportsPromptCache: true,
supportsPromptCache: false,
supportsReasoningEffort: false,
reasoningEffort: "high",
inputPrice: 15.0,
outputPrice: 120.0,
description:

View file

@ -1510,26 +1510,6 @@ describe("OpenAI Native background mode behavior", () => {
expect(requestBodyWithOptionFalse.stream).toBe(true)
})
it("auto-enables background mode for gpt-5-pro when no override is specified", async () => {
const handler = new OpenAiNativeHandler({
apiModelId: "gpt-5-pro-2025-10-06",
openAiNativeApiKey: "test",
// no openAiNativeBackgroundMode provided
})
mockResponsesCreate.mockResolvedValueOnce(createMinimalIterable())
const chunks: any[] = []
for await (const chunk of handler.createMessage(systemPrompt, baseMessages, metadataStoreFalse)) {
chunks.push(chunk)
}
expect(chunks).not.toHaveLength(0)
const requestBody = mockResponsesCreate.mock.calls[0][0]
expect(requestBody.background).toBe(true)
expect(requestBody.stream).toBe(true)
expect(requestBody.store).toBe(true)
})
it("forces store:true and includes background:true when falling back to SSE", async () => {
const handler = new OpenAiNativeHandler({
apiModelId: "gpt-5-pro-2025-10-06",

View file

@ -2627,9 +2627,6 @@ export class Task extends EventEmitter<TaskEvents> implements TaskLike {
;(apiReqMsg as any).metadata.responseId = chunk.responseId
}
// Temporary debug to confirm UI metadata updates
console.log(
`[BackgroundMode] status update -> ${chunk.status} (resp=${chunk.responseId ?? "n/a"})`,
)
await this.updateClineMessage(apiReqMsg)
// Force state refresh to ensure UI recomputes derived labels/memos
const provider = this.providerRef.deref()

View file

@ -340,11 +340,7 @@ export const ChatRowContent = ({
)
) : bgDone ? (
bgStatus === "completed" ? (
isExpanded ? (
<ChevronDown className="w-4 shrink-0" />
) : (
<ChevronRight className="w-4 shrink-0" />
)
<span className={`codicon codicon-chevron-${isExpanded ? "down" : "right"}`} />
) : (
getIconSpan("error", bgStatus === "canceled" ? cancelledColor : errorColor)
)
@ -383,7 +379,17 @@ export const ChatRowContent = ({
default:
return [null, null]
}
}, [type, isCommandExecuting, message, isMcpServerResponding, apiReqCancelReason, cost, apiRequestFailedMessage, t])
}, [
type,
isCommandExecuting,
message,
isExpanded,
isMcpServerResponding,
apiReqCancelReason,
cost,
apiRequestFailedMessage,
t,
])
const headerStyle: React.CSSProperties = {
display: "flex",