Roo to main

.changeset/config.json

@@ -7,5 +7,5 @@
 	"access": "restricted",
 	"baseBranch": "main",
 	"updateInternalDependencies": "patch",
-	"ignore": []
+	"ignore": ["@roo-code/cli"]
 }

apps/cli/src/extension-host.ts

@@ -329,7 +329,7 @@ export class ExtensionHost extends EventEmitter {
 
 		// Update vscode-shim runtime configuration so
 		// vscode.workspace.getConfiguration() returns correct values.
-		setRuntimeConfigValues("roo-cline", settings as Record<string, unknown>)
+		setRuntimeConfigValues("zgsm", settings as Record<string, unknown>)
 	}
 
 	/**

apps/cli/src/index.ts

@@ -21,7 +21,7 @@ import { getEnvVarName, getApiKeyFromEnv, getDefaultExtensionPath } from "./util
 const DEFAULTS = {
 	mode: "code",
 	reasoningEffort: "medium" as const,
-	model: "anthropic/claude-sonnet-4.5",
+	model: "anthropic/claude-opus-4.5",
 }
 
 const REASONING_EFFORTS = [...reasoningEffortsExtended, "unspecified", "disabled"]

packages/evals/Dockerfile.runner

@@ -163,7 +163,7 @@ RUN pnpm vsix -- --out ../bin/roo-code.vsix \
 
 # Build the extension bundle and CLI (for CLI execution method)
 # The CLI requires the extension bundle (src/dist/extension.js) and the CLI build (apps/cli/dist/index.js)
-RUN pnpm --filter roo-cline bundle \
+RUN pnpm --filter zgsm bundle \
     && pnpm --filter @roo-code/cli build
 
 # Copy entrypoint script

packages/evals/src/cli/utils.ts

@@ -157,7 +157,7 @@ export async function copyConversationHistory({
 	logger: Logger
 }): Promise<void> {
 	// VS Code extension global storage path within the container
-	const extensionStoragePath = "/roo/.vscode/User/globalStorage/rooveterinaryinc.roo-cline"
+	const extensionStoragePath = "/roo/.vscode/User/globalStorage/zgsm-ai.zgsm"
 	const taskStoragePath = path.join(extensionStoragePath, "tasks", rooTaskId)
 
 	const filesToCopy = ["api_conversation_history.json", "ui_messages.json"]

src/api/providers/gemini.ts

@@ -456,7 +456,10 @@ export class GeminiHandler extends BaseProvider implements SingleCompletionHandl
 	}
 
 	public getThoughtSignature(): string | undefined {
-		return this.lastThoughtSignature
+		// Disabled to prevent "Corrupted thought signature" errors on task resumption.
+		// Gemini thought signatures are session-specific and cannot be reliably reused
+		// across API calls or after task resumption from history.
+		return undefined
 	}
 
 	public getResponseId(): string | undefined {

src/api/providers/utils/response-render-config.ts

@@ -1,24 +1,30 @@
 import * as vscode from "vscode"
 import { isJetbrainsPlatform } from "../../../utils/platform"
 
+const isJetbrains = isJetbrainsPlatform()
+
 export const renderModes = {
 	noLimit: {
+		limit: 0,
 		interval: 5,
 	},
 	fast: {
-		interval: 10,
+		limit: isJetbrains ? 2 : 1,
+		interval: isJetbrains ? 30 : 15,
 	},
 	medium: {
-		interval: 20,
+		limit: isJetbrains ? 3 : 1,
+		interval: isJetbrains ? 60 : 30,
 	},
 	slow: {
-		interval: 40,
+		limit: isJetbrains ? 4 : 1,
+		interval: isJetbrains ? 100 : 60,
 	},
 }
 
 export function getApiResponseRenderMode() {
-	if (isJetbrainsPlatform()) {
-		return renderModes.slow
+	if (isJetbrains) {
+		return renderModes.medium
 	}
 	const apiResponseRenderMode = vscode.workspace
 		.getConfiguration("zgsm")

src/api/providers/zgsm.ts

@@ -600,7 +600,10 @@ export class ZgsmAiHandler extends BaseProvider implements SingleCompletionHandl
 				}
 				const now = Date.now()
 				// Process in batch when threshold is reached
-				if (time + this.apiResponseRenderModeInfo.interval <= now) {
+				if (
+					time + this.apiResponseRenderModeInfo.interval <= now &&
+					contentBuffer.length >= this.apiResponseRenderModeInfo.limit
+				) {
 					const batchedContent = contentBuffer.join("")
 					for (const processedChunk of matcher.update(batchedContent)) {
 						if (this.abortController?.signal.aborted) {

src/api/transform/__tests__/openai-format.spec.ts

@@ -225,6 +225,36 @@ describe("convertToOpenAiMessages", () => {
 		expect(assistantMessage.tool_calls![0].id).toBe("custom_toolu_123")
 	})
 
+	it("should use empty string for content when assistant message has only tool calls (Gemini compatibility)", () => {
+		// This test ensures that assistant messages with only tool_use blocks (no text)
+		// have content set to "" instead of undefined. Gemini (via OpenRouter) requires
+		// every message to have at least one "parts" field, which fails if content is undefined.
+		// See: ROO-425
+		const anthropicMessages: Anthropic.Messages.MessageParam[] = [
+			{
+				role: "assistant",
+				content: [
+					{
+						type: "tool_use",
+						id: "tool-123",
+						name: "read_file",
+						input: { path: "test.ts" },
+					},
+				],
+			},
+		]
+
+		const openAiMessages = convertToOpenAiMessages(anthropicMessages)
+		expect(openAiMessages).toHaveLength(1)
+
+		const assistantMessage = openAiMessages[0] as OpenAI.Chat.ChatCompletionAssistantMessageParam
+		expect(assistantMessage.role).toBe("assistant")
+		// Content should be an empty string, NOT undefined
+		expect(assistantMessage.content).toBe("")
+		expect(assistantMessage.tool_calls).toHaveLength(1)
+		expect(assistantMessage.tool_calls![0].id).toBe("tool-123")
+	})
+
 	describe("mergeToolResultText option", () => {
 		it("should merge text content into last tool message when mergeToolResultText is true", () => {
 			const anthropicMessages: Anthropic.Messages.MessageParam[] = [

src/api/transform/openai-format.ts

@@ -223,7 +223,9 @@ export function convertToOpenAiMessages(
 					reasoning_details?: any[]
 				} = {
 					role: "assistant",
-					content,
+					// Use empty string instead of undefined for providers like Gemini (via OpenRouter)
+					// that require every message to have content in the "parts" field
+					content: content ?? "",
 				}
 
 				// Pass through reasoning_details to preserve the original shape from the API.

src/core/costrict/activate.ts

@@ -88,14 +88,15 @@ export async function activate(
 	outputChannel: vscode.OutputChannel,
 ) {
 	const isJetbrains = isJetbrainsPlatform()
+	const isVscodePlatform = !isJetbrains && !isCliPatform()
 	const logger = createLogger(Package.outputChannel)
 	initErrorCodeManager(provider)
 	initGitCheckoutDetector(context, logger)
 	await initialize(provider, logger)
 	startIPCServer()
 	connectIPC()
 
-	if (!isJetbrains && !isCliPatform()) {
+	if (isVscodePlatform) {
 		registerAutoCompletionProvider(context, provider)
 	}
 	const completionStatusBar = CompletionStatusBar.getInstance()
@@ -180,7 +181,7 @@ export async function activate(
 		)
 	}
 
-	if (!isJetbrains) {
+	if (isVscodePlatform) {
 		context.subscriptions.push(
 			// Register function header menu
 			vscode.languages.registerCodeLensProvider("*", new CostrictCodeLensProvider()),
@@ -203,7 +204,7 @@ export async function activate(
 
 	// Get zgsmRefreshToken without webview resolve
 	const tokens = await ZgsmAuthStorage.getInstance().getTokens()
-	if (!isJetbrains) {
+	if (isVscodePlatform) {
 		if (tokens?.access_token) {
 			// CompletionStatusBar.initByConfig()
 			completionStatusBar.setEnableState()

src/core/prompts/tools/native-tools/ask_followup_question.ts

@@ -51,7 +51,7 @@ export default {
 						required: ["text", "mode"],
 						additionalProperties: false,
 					},
-					minItems: 2,
+					minItems: 1,
 					maxItems: 4,
 				},
 			},

src/core/tools/ApplyDiffTool.ts

@@ -292,6 +292,7 @@ export class ApplyDiffTool extends BaseTool<"apply_diff"> {
 			}
 
 			await task.diffViewProvider.reset()
+			this.resetPartialState()
 
 			// Process any queued messages after file edit completes
 			task.processQueuedMessages()
@@ -300,6 +301,7 @@ export class ApplyDiffTool extends BaseTool<"apply_diff"> {
 		} catch (error) {
 			await handleError("applying diff", error as Error)
 			await task.diffViewProvider.reset()
+			this.resetPartialState()
 			task.processQueuedMessages()
 			return
 		}
@@ -309,9 +311,14 @@ export class ApplyDiffTool extends BaseTool<"apply_diff"> {
 		const relPath: string | undefined = block.params.path
 		const diffContent: string | undefined = block.params.diff
 
+		// Wait for path to stabilize before showing UI (prevents truncated paths)
+		if (!this.hasPathStabilized(relPath)) {
+			return
+		}
+
 		const sharedMessageProps: ClineSayTool = {
 			tool: "appliedDiff",
-			path: getReadablePath(task.cwd, relPath || ""),
+			path: getReadablePath(task.cwd, relPath),
 			diff: diffContent,
 		}
 

src/core/tools/BaseTool.ts

@@ -47,6 +47,12 @@ export abstract class BaseTool<TName extends ToolName> {
 	 */
 	abstract readonly name: TName
 
+	/**
+	 * Track the last seen path during streaming to detect when the path has stabilized.
+	 * Used by hasPathStabilized() to prevent displaying truncated paths from partial-json parsing.
+	 */
+	protected lastSeenPartialPath: string | undefined = undefined
+
 	/**
 	 * Parse XML/legacy string-based parameters into typed parameters.
 	 *
@@ -120,6 +126,41 @@ export abstract class BaseTool<TName extends ToolName> {
 		return text.replace(tagRegex, "")
 	}
 
+	/**
+	 * Check if a path parameter has stabilized during streaming.
+	 *
+	 * During native tool call streaming, the partial-json library may return truncated
+	 * string values when chunk boundaries fall mid-value. This method tracks the path
+	 * value between consecutive handlePartial() calls and returns true only when the
+	 * path has stopped changing (stabilized).
+	 *
+	 * Usage in handlePartial():
+	 * ```typescript
+	 * if (!this.hasPathStabilized(block.params.path)) {
+	 *     return // Path still changing, wait for it to stabilize
+	 * }
+	 * // Path is stable, proceed with UI updates
+	 * ```
+	 *
+	 * @param path - The current path value from the partial block
+	 * @returns true if path has stabilized (same value seen twice) and is non-empty, false otherwise
+	 */
+	protected hasPathStabilized(path: string | undefined): boolean {
+		const pathHasStabilized = this.lastSeenPartialPath !== undefined && this.lastSeenPartialPath === path
+		this.lastSeenPartialPath = path
+		return pathHasStabilized && !!path
+	}
+
+	/**
+	 * Reset the partial state tracking.
+	 *
+	 * Should be called at the end of execute() (both success and error paths)
+	 * to ensure clean state for the next tool invocation.
+	 */
+	resetPartialState(): void {
+		this.lastSeenPartialPath = undefined
+	}
+
 	/**
 	 * Main entry point for tool execution.
 	 *

src/core/tools/EditFileTool.ts

@@ -327,19 +327,26 @@ export class EditFileTool extends BaseTool<"edit_file"> {
 			// Record successful tool usage and cleanup
 			task.recordToolUsage("edit_file")
 			await task.diffViewProvider.reset()
+			this.resetPartialState()
 
 			// Process any queued messages after file edit completes
 			task.processQueuedMessages()
 		} catch (error) {
 			await handleError("edit_file", error as Error)
 			await task.diffViewProvider.reset()
+			this.resetPartialState()
 		}
 	}
 
 	override async handlePartial(task: Task, block: ToolUse<"edit_file">): Promise<void> {
 		const filePath: string | undefined = block.params.file_path
 		const oldString: string | undefined = block.params.old_string
 
+		// Wait for path to stabilize before showing UI (prevents truncated paths)
+		if (!this.hasPathStabilized(filePath)) {
+			return
+		}
+
 		let operationPreview: string | undefined
 		if (oldString !== undefined) {
 			if (oldString === "") {
@@ -350,14 +357,14 @@ export class EditFileTool extends BaseTool<"edit_file"> {
 			}
 		}
 
-		// Determine relative path for display
-		let relPath = filePath || ""
-		if (filePath && path.isAbsolute(filePath)) {
-			relPath = path.relative(task.cwd, filePath)
+		// Determine relative path for display (filePath is guaranteed non-null after hasPathStabilized)
+		let relPath = filePath!
+		if (path.isAbsolute(relPath)) {
+			relPath = path.relative(task.cwd, relPath)
 		}
 
-		const absolutePath = relPath ? path.resolve(task.cwd, relPath) : ""
-		const isOutsideWorkspace = absolutePath ? isPathOutsideWorkspace(absolutePath) : false
+		const absolutePath = path.resolve(task.cwd, relPath)
+		const isOutsideWorkspace = isPathOutsideWorkspace(absolutePath)
 
 		const sharedMessageProps: ClineSayTool = {
 			tool: "appliedDiff",

src/core/tools/SearchAndReplaceTool.ts

@@ -259,17 +259,25 @@ export class SearchAndReplaceTool extends BaseTool<"search_and_replace"> {
 			// Record successful tool usage and cleanup
 			task.recordToolUsage("search_and_replace")
 			await task.diffViewProvider.reset()
+			this.resetPartialState()
 
 			// Process any queued messages after file edit completes
 			task.processQueuedMessages()
 		} catch (error) {
 			await handleError("search and replace", error as Error)
 			await task.diffViewProvider.reset()
+			this.resetPartialState()
 		}
 	}
 
 	override async handlePartial(task: Task, block: ToolUse<"search_and_replace">): Promise<void> {
 		const relPath: string | undefined = block.params.path
+
+		// Wait for path to stabilize before showing UI (prevents truncated paths)
+		if (!this.hasPathStabilized(relPath)) {
+			return
+		}
+
 		const operationsStr: string | undefined = block.params.operations
 
 		let operationsPreview: string | undefined
@@ -284,12 +292,13 @@ export class SearchAndReplaceTool extends BaseTool<"search_and_replace"> {
 			}
 		}
 
-		const absolutePath = relPath ? path.resolve(task.cwd, relPath) : ""
-		const isOutsideWorkspace = absolutePath ? isPathOutsideWorkspace(absolutePath) : false
+		// relPath is guaranteed non-null after hasPathStabilized
+		const absolutePath = path.resolve(task.cwd, relPath!)
+		const isOutsideWorkspace = isPathOutsideWorkspace(absolutePath)
 
 		const sharedMessageProps: ClineSayTool = {
 			tool: "appliedDiff",
-			path: getReadablePath(task.cwd, relPath || ""),
+			path: getReadablePath(task.cwd, relPath!),
 			diff: operationsPreview,
 			isOutsideWorkspace,
 		}