Merge integration: tool-call pipeline fixes, browser tools, UI cleanup

Brings in 9 commits from integration branch:
- fix(ui): strip malformed tool-call markup fragments from chat display
- fix(browser): click_browser_by_text context-destroyed in getSummary (Round 2)
- fix(browser): click_browser_by_text 'context destroyed' false error
- fix(tools): generate_image 'None' sentinel + playwright-core bundling
- fix(tools): MiMo serialization, shell selection, duplicate messages + prior WIP
- fix(tools): narrow RecoverableAssistantBlock name access in recovery tests
- fix(tools): execute MiniMax/Hermes XML tool recovery end-to-end
- fix(tools): recover textual tool_calls and harden tool selection
- feat(spec-workspace): selection context intelligence, sanitization boundary, tool reliability

All tools verified working on both Xiaomi built-in and OpenAI Compatible providers.

Co-authored-by: Kilo <kilo@kilocode.com>

Author: CodeKing

.gitignore

@@ -55,3 +55,8 @@ qdrant_storage/
 plans/
 
 roo-cli-*.tar.gz*
+
+# Temporary one-off fix scripts
+fix_spec.js
+fix_spec.mjs
+tmp-*.mjs

packages/types/src/model.ts

@@ -65,7 +65,18 @@ export type ServiceTier = z.infer<typeof serviceTierSchema>
  * ModelParameter
  */
 
-export const modelParameters = ["max_tokens", "temperature", "reasoning", "include_reasoning"] as const
+export const modelParameters = [
+	"max_tokens",
+	"temperature",
+	"reasoning",
+	"include_reasoning",
+	// Tool-calling parameters (OpenRouter-style supported_parameters contract). Presence in
+	// a model's supportedParameters allow-list signals the endpoint accepts these fields;
+	// absence tells generic OpenAI-compatible handlers to omit them for compatibility.
+	"tools",
+	"tool_choice",
+	"parallel_tool_calls",
+] as const
 
 export const modelParametersSchema = z.enum(modelParameters)
 

packages/types/src/roomodes-schema.ts

@@ -12,8 +12,10 @@ import { toolGroups, deprecatedToolGroups } from "./tool.js"
 import { groupOptionsSchema, modeConfigSchema } from "./mode.js"
 
 // Build a ToolGroup enum that includes deprecated groups so existing configs
-// still validate.
-const allToolGroups = [...toolGroups, ...deprecatedToolGroups] as [string, ...string[]]
+// still validate. Deduplicated: a group may be listed as deprecated while still
+// being active (e.g. "browser"), and JSON Schema rejects an enum with duplicate
+// items, which would make the generated schema fail to compile.
+const allToolGroups = [...new Set<string>([...toolGroups, ...deprecatedToolGroups])] as [string, ...string[]]
 const allToolGroupsSchema = z.enum(allToolGroups)
 
 // Build a GroupEntry schema that uses the extended tool group list.

packages/types/src/task.ts

@@ -85,6 +85,8 @@ export type TaskProviderEvents = {
 
 export interface CreateTaskOptions {
 	taskId?: string
+	/** Internal context sent to the model for the initial request only; never rendered in chat. */
+	initialHiddenContext?: string
 	enableCheckpoints?: boolean
 	consecutiveMistakeLimit?: number
 	experiments?: Record<string, boolean>

packages/types/src/tool.ts

@@ -89,6 +89,11 @@ export const toolNames = [
 	"toggle_mcp_server",
 	"delete_mcp_server",
 	"refresh_mcp_servers",
+	// Virtual Spec Workspace (F-004 / F-022) — globalStorage only, never project files
+	"list_specs",
+	"read_spec",
+	"write_spec",
+	"delete_spec",
 ] as const
 
 export const toolNamesSchema = z.enum(toolNames)

packages/types/src/vscode-extension-host.ts

@@ -108,6 +108,10 @@ export interface ExtensionMessage {
 		| "fileContent"
 		| "rooHistoryImportProgress"
 	text?: string
+	/** Opaque, one-use context token associated with a visible chat prefill. */
+	selectionContextToken?: string
+	/** The AI selection action type (rewrite/improve/remove/custom) for pill rendering. */
+	selectionContextAction?: string
 	/** For fileContent: { path, content, error? } */
 	fileContent?: { path: string; content: string | null; error?: string }
 	payload?: any // eslint-disable-line @typescript-eslint/no-explicit-any
@@ -668,6 +672,8 @@ export interface WebviewMessage {
 		| "renameTask"
 	text?: string
 	taskId?: string
+	/** Opaque, one-use context token associated with a visible chat prefill. */
+	selectionContextToken?: string
 	editedMessageContent?: string
 	tab?: "settings" | "history" | "mcp" | "modes" | "chat" | "marketplace" | "cloud"
 	disabled?: boolean
@@ -878,7 +884,25 @@ export interface ClineSayTool {
 		| "listMcpConfig"
 		| "getMcpServer"
 		| "refreshMcpServers"
+		// Virtual Spec Workspace tools (F-004 / F-019 / F-022 activity mapping)
+		| "list_specs"
+		| "read_spec"
+		| "write_spec"
+		| "delete_spec"
 	path?: string
+	/** write_spec / read_spec document kind */
+	doc?: string
+	/** write_spec: create|write; delete_spec: delete|delete_bulk|delete_progress */
+	action?: string
+	/** write_spec pack title (create) / delete_spec title */
+	title?: string
+	/** write_spec / delete_spec pack id */
+	specId?: string
+	/** F-022b bulk delete progress */
+	index?: number
+	total?: number
+	count?: number
+	explicitBulk?: boolean
 	// For readCommandOutput
 	readStart?: number
 	readEnd?: number

packages/types/src/vscode.ts

@@ -48,6 +48,9 @@ export const commandIds = [
 	"toggleAutoApprove",
 
 	"showRipgrepDiagnostic",
+
+	/** Open virtual Spec Workspace panel (F-002) */
+	"openSpecWorkspace",
 ] as const
 
 export type CommandId = (typeof commandIds)[number]

schemas/roomodes.json

@@ -44,7 +44,16 @@
 							"anyOf": [
 								{
 									"type": "string",
-									"enum": ["read", "edit", "command", "mcp", "modes", "browser"]
+									"enum": [
+										"read",
+										"edit",
+										"command",
+										"mcp",
+										"modes",
+										"browser",
+										"provider_manage",
+										"mcp_manage"
+									]
 								},
 								{
 									"type": "array",
@@ -53,7 +62,16 @@
 									"items": [
 										{
 											"type": "string",
-											"enum": ["read", "edit", "command", "mcp", "modes", "browser"]
+											"enum": [
+												"read",
+												"edit",
+												"command",
+												"mcp",
+												"modes",
+												"browser",
+												"provider_manage",
+												"mcp_manage"
+											]
 										},
 										{
 											"type": "object",

src/activate/registerCommands.ts

@@ -157,6 +157,16 @@ const getCommandsMap = ({
 		const { promptForCustomStoragePath } = await import("../utils/storage")
 		await promptForCustomStoragePath()
 	},
+	openSpecWorkspace: async () => {
+		const { SpecWorkspacePanel } = await import("../core/specs/ui/SpecWorkspacePanel")
+		const globalStoragePath = provider.contextProxy.globalStorageUri.fsPath
+		SpecWorkspacePanel.createOrShow({
+			context,
+			globalStoragePath,
+			getWorkspaceRoot: () => provider.cwd,
+			outputChannel,
+		})
+	},
 	importSettings: async (filePath?: string) => {
 		const visibleProvider = getVisibleProviderOrLog(outputChannel)
 		if (!visibleProvider) {

src/api/providers/__tests__/base-provider.spec.ts

@@ -22,14 +22,24 @@ class TestProvider extends BaseProvider {
 		}
 	}
 
-	// Expose protected method for testing
+	// Expose protected method for testing (schema rewrite used when enableStrict: true)
 	public testConvertToolSchemaForOpenAI(schema: any): any {
-		return this.convertToolsForOpenAI(schema)
+		return this.convertToolSchemaForOpenAI(schema)
 	}
 
 	// Expose protected method for testing
-	public testConvertToolsForOpenAI(tools: any[] | undefined): any[] | undefined {
-		return this.convertToolsForOpenAI(tools)
+	public testConvertToolsForOpenAI(
+		tools: any[] | undefined,
+		options?: { enableStrict?: boolean },
+	): any[] | undefined {
+		return this.convertToolsForOpenAI(tools, options)
+	}
+
+	public convertToolsForOpenAIPublic(
+		tools: any[] | undefined,
+		options?: { enableStrict?: boolean },
+	): any[] | undefined {
+		return this.convertToolsForOpenAI(tools, options)
 	}
 
 	// Expose private method for testing via any cast (private methods cannot be overridden)
@@ -166,21 +176,56 @@ describe("BaseProvider", () => {
 			expect(result).toBeUndefined()
 		})
 
-		it("should set strict: true for non-MCP tools", () => {
+		it("defaults to strict: false for non-MCP tools (third-party gateway safe)", () => {
 			const tools = [
 				{
 					type: "function",
 					function: {
 						name: "read_file",
 						description: "Read a file",
-						parameters: { type: "object", properties: {} },
+						parameters: {
+							type: "object",
+							properties: {
+								path: { type: "string" },
+								limit: { type: "number" },
+							},
+							required: ["path"],
+						},
 					},
 				},
 			]
 
 			const result = provider.testConvertToolsForOpenAI(tools)
 
+			expect(result?.[0].function.strict).toBe(false)
+			// Optional params must stay optional (not rewritten to required: all keys)
+			expect(result?.[0].function.parameters.required).toEqual(["path"])
+		})
+
+		it("should set strict: true when enableStrict is true", () => {
+			const tools = [
+				{
+					type: "function",
+					function: {
+						name: "read_file",
+						description: "Read a file",
+						parameters: {
+							type: "object",
+							properties: {
+								path: { type: "string" },
+								limit: { type: "number" },
+							},
+							required: ["path"],
+						},
+					},
+				},
+			]
+
+			const result = provider.convertToolsForOpenAIPublic(tools, { enableStrict: true })
+
 			expect(result?.[0].function.strict).toBe(true)
+			expect(result?.[0].function.parameters.required).toEqual(["path", "limit"])
+			expect(result?.[0].function.parameters.additionalProperties).toBe(false)
 		})
 
 		it("should set strict: false for MCP tools (mcp-- prefix)", () => {