diff --git a/.vscode/launch.json b/.vscode/launch.json index 9032340..ead1700 100644 --- a/.vscode/launch.json +++ b/.vscode/launch.json @@ -9,9 +9,21 @@ "name": "Run Extension", "type": "extensionHost", "request": "launch", - "args": ["--extensionDevelopmentPath=${workspaceFolder}"], + "runtimeExecutable": "${execPath}", + "args": [ + "--extensionDevelopmentPath=${workspaceFolder}", + ], + "sourceMaps": true, "outFiles": ["${workspaceFolder}/dist/**/*.js"], - "preLaunchTask": "${defaultBuildTask}" - } + "preLaunchTask": "compile", + "env": { + "NODE_ENV": "development", + "VSCODE_DEBUG_MODE": "true" + }, + "resolveSourceMapLocations": [ + "${workspaceFolder}/**", + "!**/node_modules/**" + ] + }, ] } diff --git a/.vscode/tasks.json b/.vscode/tasks.json index e141383..5804f54 100644 --- a/.vscode/tasks.json +++ b/.vscode/tasks.json @@ -3,6 +3,21 @@ { "version": "2.0.0", "tasks": [ + { + "label": "compile", + "type": "npm", + "script": "compile", + "dependsOn": ["npm: build:webview"], + "group": { + "kind": "build", + "isDefault": true + }, + "presentation": { + "reveal": "silent", + "panel": "shared" + }, + "problemMatcher": ["$tsc", "$eslint-stylish"] + }, { "label": "watch", "dependsOn": ["npm: build:webview", "npm: watch:tsc", "npm: watch:esbuild"], @@ -11,7 +26,7 @@ }, "group": { "kind": "build", - "isDefault": true + "isDefault": false } }, { diff --git a/CHANGELOG.md b/CHANGELOG.md index 67060eb..1447293 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,302 +1,308 @@ # Roo Cline Changelog +## [3.1.7] + +- DeepSeek-R1 support (thanks @philipnext!) +- Experimental new unified diff algorithm can be enabled in settings (thanks @daniel-lxs!) +- More fixes to configuration profiles (thanks @samhvw8!) + ## [3.1.6] -- Add Mistral (thanks Cline!) -- Fix bug with VSCode LM configuration profile saving (thanks @samhvw8!) +- Add Mistral (thanks Cline!) +- Fix bug with VSCode LM configuration profile saving (thanks @samhvw8!) ## [3.1.4 - 3.1.5] -- Bug fixes to the auto approve menu +- Bug fixes to the auto approve menu ## [3.1.3] -- Add auto-approve chat bar (thanks Cline!) -- Fix bug with VS Code Language Models integration +- Add auto-approve chat bar (thanks Cline!) +- Fix bug with VS Code Language Models integration ## [3.1.2] -- Experimental support for VS Code Language Models including Copilot (thanks @RaySinner / @julesmons!) -- Fix bug related to configuration profile switching (thanks @samhvw8!) -- Improvements to fuzzy search in mentions, history, and model lists (thanks @samhvw8!) -- PKCE support for Glama (thanks @punkpeye!) -- Use 'developer' message for o1 system prompt +- Experimental support for VS Code Language Models including Copilot (thanks @RaySinner / @julesmons!) +- Fix bug related to configuration profile switching (thanks @samhvw8!) +- Improvements to fuzzy search in mentions, history, and model lists (thanks @samhvw8!) +- PKCE support for Glama (thanks @punkpeye!) +- Use 'developer' message for o1 system prompt ## [3.1.1] -- Visual fixes to chat input and settings for the light+ themes +- Visual fixes to chat input and settings for the light+ themes ## [3.1.0] -- You can now customize the role definition and instructions for each chat mode (Code, Architect, and Ask), either through the new Prompts tab in the top menu or mode-specific .clinerules-mode files. Prompt Enhancements have also been revamped: the "Enhance Prompt" button now works with any provider and API configuration, giving you the ability to craft messages with fully customizable prompts for even better results. -- Add a button to copy markdown out of the chat +- You can now customize the role definition and instructions for each chat mode (Code, Architect, and Ask), either through the new Prompts tab in the top menu or mode-specific .clinerules-mode files. Prompt Enhancements have also been revamped: the "Enhance Prompt" button now works with any provider and API configuration, giving you the ability to craft messages with fully customizable prompts for even better results. +- Add a button to copy markdown out of the chat ## [3.0.3] -- Update required vscode engine to ^1.84.0 to match cline +- Update required vscode engine to ^1.84.0 to match cline ## [3.0.2] -- A couple more tiny tweaks to the button alignment in the chat input +- A couple more tiny tweaks to the button alignment in the chat input ## [3.0.1] -- Fix the reddit link and a small visual glitch in the chat input +- Fix the reddit link and a small visual glitch in the chat input ## [3.0.0] -- This release adds chat modes! Now you can ask Roo Cline questions about system architecture or the codebase without immediately jumping into writing code. You can even assign different API configuration profiles to each mode if you prefer to use different models for thinking vs coding. Would love feedback in the new Roo Cline Reddit! https://www.reddit.com/r/roocline +- This release adds chat modes! Now you can ask Roo Cline questions about system architecture or the codebase without immediately jumping into writing code. You can even assign different API configuration profiles to each mode if you prefer to use different models for thinking vs coding. Would love feedback in the new Roo Cline Reddit! https://www.reddit.com/r/roocline ## [2.2.46] -- Only parse @-mentions in user input (not in files) +- Only parse @-mentions in user input (not in files) ## [2.2.45] -- Save different API configurations to quickly switch between providers and settings (thanks @samhvw8!) +- Save different API configurations to quickly switch between providers and settings (thanks @samhvw8!) ## [2.2.44] -- Automatically retry failed API requests with a configurable delay (thanks @RaySinner!) +- Automatically retry failed API requests with a configurable delay (thanks @RaySinner!) ## [2.2.43] -- Allow deleting single messages or all subsequent messages +- Allow deleting single messages or all subsequent messages ## [2.2.42] -- Add a Git section to the context mentions +- Add a Git section to the context mentions ## [2.2.41] -- Checkbox to disable streaming for OpenAI-compatible providers +- Checkbox to disable streaming for OpenAI-compatible providers ## [2.2.40] -- Add the Glama provider (thanks @punkpeye!) +- Add the Glama provider (thanks @punkpeye!) ## [2.2.39] -- Add toggle to enable/disable the MCP-related sections of the system prompt (thanks @daniel-lxs!) +- Add toggle to enable/disable the MCP-related sections of the system prompt (thanks @daniel-lxs!) ## [2.2.38] -- Add a setting to control the number of terminal output lines to pass to the model when executing commands +- Add a setting to control the number of terminal output lines to pass to the model when executing commands ## [2.2.36 - 2.2.37] -- Add a button to delete user messages +- Add a button to delete user messages ## [2.2.35] -- Allow selection of multiple browser viewport sizes and adjusting screenshot quality +- Allow selection of multiple browser viewport sizes and adjusting screenshot quality ## [2.2.34] -- Add the DeepSeek provider +- Add the DeepSeek provider ## [2.2.33] -- "Enhance prompt" button (OpenRouter models only for now) -- Support listing models for OpenAI compatible providers (thanks @samhvw8!) +- "Enhance prompt" button (OpenRouter models only for now) +- Support listing models for OpenAI compatible providers (thanks @samhvw8!) ## [2.2.32] -- More efficient workspace tracker +- More efficient workspace tracker ## [2.2.31] -- Improved logic for auto-approving chained commands +- Improved logic for auto-approving chained commands ## [2.2.30] -- Fix bug with auto-approving commands +- Fix bug with auto-approving commands ## [2.2.29] -- Add configurable delay after auto-writes to allow diagnostics to catch up +- Add configurable delay after auto-writes to allow diagnostics to catch up ## [2.2.28] -- Use createFileSystemWatcher to more reliably update list of files to @-mention +- Use createFileSystemWatcher to more reliably update list of files to @-mention ## [2.2.27] -- Add the current time to the system prompt and improve browser screenshot quality (thanks @libertyteeth!) +- Add the current time to the system prompt and improve browser screenshot quality (thanks @libertyteeth!) ## [2.2.26] -- Tweaks to preferred language (thanks @yongjer) +- Tweaks to preferred language (thanks @yongjer) ## [2.2.25] -- Add a preferred language dropdown +- Add a preferred language dropdown ## [2.2.24] -- Default diff editing to on for new installs +- Default diff editing to on for new installs ## [2.2.23] -- Fix context window for gemini-2.0-flash-thinking-exp-1219 (thanks @student20880) +- Fix context window for gemini-2.0-flash-thinking-exp-1219 (thanks @student20880) ## [2.2.22] -- Add gemini-2.0-flash-thinking-exp-1219 +- Add gemini-2.0-flash-thinking-exp-1219 ## [2.2.21] -- Take predicted file length into account when detecting omissions +- Take predicted file length into account when detecting omissions ## [2.2.20] -- Make fuzzy diff matching configurable (and default to off) +- Make fuzzy diff matching configurable (and default to off) ## [2.2.19] -- Add experimental option to use a bigger browser (1280x800) +- Add experimental option to use a bigger browser (1280x800) ## [2.2.18] -- More targeted styling fix for Gemini chats +- More targeted styling fix for Gemini chats ## [2.2.17] -- Improved regex for auto-execution of chained commands +- Improved regex for auto-execution of chained commands ## [2.2.16] -- Incorporate Premshay's [PR](https://github.com/RooVetGit/Roo-Cline/pull/60) to add support for Amazon Nova and Meta Llama Models via Bedrock (3, 3.1, 3.2) and unified Bedrock calls using BedrockClient and Bedrock Runtime API +- Incorporate Premshay's [PR](https://github.com/RooVetGit/Roo-Cline/pull/60) to add support for Amazon Nova and Meta Llama Models via Bedrock (3, 3.1, 3.2) and unified Bedrock calls using BedrockClient and Bedrock Runtime API ## [2.2.14 - 2.2.15] -- Make diff editing more robust to transient errors / fix bugs +- Make diff editing more robust to transient errors / fix bugs ## [2.2.13] -- Fixes to sound playing and applying diffs +- Fixes to sound playing and applying diffs ## [2.2.12] -- Better support for pure deletion and insertion diffs +- Better support for pure deletion and insertion diffs ## [2.2.11] -- Added settings checkbox for verbose diff debugging +- Added settings checkbox for verbose diff debugging ## [2.2.6 - 2.2.10] -- More fixes to search/replace diffs +- More fixes to search/replace diffs ## [2.2.5] -- Allow MCP servers to be enabled/disabled +- Allow MCP servers to be enabled/disabled ## [2.2.4] -- Tweak the prompt to encourage diff edits when they're enabled +- Tweak the prompt to encourage diff edits when they're enabled ## [2.2.3] -- Clean up the settings screen +- Clean up the settings screen ## [2.2.2] -- Add checkboxes to auto-approve MCP tools +- Add checkboxes to auto-approve MCP tools ## [2.2.1] -- Fix another diff editing indentation bug +- Fix another diff editing indentation bug ## [2.2.0] -- Incorporate MCP changes from Cline 2.2.0 +- Incorporate MCP changes from Cline 2.2.0 ## [2.1.21] -- Larger text area input + ability to drag images into it +- Larger text area input + ability to drag images into it ## [2.1.20] -- Add Gemini 2.0 +- Add Gemini 2.0 ## [2.1.19] -- Better error handling for diff editing +- Better error handling for diff editing ## [2.1.18] -- Diff editing bugfix to handle Windows line endings +- Diff editing bugfix to handle Windows line endings ## [2.1.17] -- Switch to search/replace diffs in experimental diff editing mode +- Switch to search/replace diffs in experimental diff editing mode ## [2.1.16] -- Allow copying prompts from the history screen +- Allow copying prompts from the history screen ## [2.1.15] -- Incorporate dbasclpy's [PR](https://github.com/RooVetGit/Roo-Cline/pull/54) to add support for gemini-exp-1206 -- Make it clear that diff editing is very experimental +- Incorporate dbasclpy's [PR](https://github.com/RooVetGit/Roo-Cline/pull/54) to add support for gemini-exp-1206 +- Make it clear that diff editing is very experimental ## [2.1.14] -- Fix bug where diffs were not being applied correctly and try Aider's [unified diff prompt](https://github.com/Aider-AI/aider/blob/3995accd0ca71cea90ef76d516837f8c2731b9fe/aider/coders/udiff_prompts.py#L75-L105) -- If diffs are enabled, automatically reject write_to_file commands that lead to truncated output +- Fix bug where diffs were not being applied correctly and try Aider's [unified diff prompt](https://github.com/Aider-AI/aider/blob/3995accd0ca71cea90ef76d516837f8c2731b9fe/aider/coders/udiff_prompts.py#L75-L105) +- If diffs are enabled, automatically reject write_to_file commands that lead to truncated output ## [2.1.13] -- Fix https://github.com/RooVetGit/Roo-Cline/issues/50 where sound effects were not respecting settings +- Fix https://github.com/RooVetGit/Roo-Cline/issues/50 where sound effects were not respecting settings ## [2.1.12] -- Incorporate JoziGila's [PR](https://github.com/cline/cline/pull/158) to add support for editing through diffs +- Incorporate JoziGila's [PR](https://github.com/cline/cline/pull/158) to add support for editing through diffs ## [2.1.11] -- Incorporate lloydchang's [PR](https://github.com/RooVetGit/Roo-Cline/pull/42) to add support for OpenRouter compression +- Incorporate lloydchang's [PR](https://github.com/RooVetGit/Roo-Cline/pull/42) to add support for OpenRouter compression ## [2.1.10] -- Incorporate HeavenOSK's [PR](https://github.com/cline/cline/pull/818) to add sound effects to Cline +- Incorporate HeavenOSK's [PR](https://github.com/cline/cline/pull/818) to add sound effects to Cline ## [2.1.9] -- Add instructions for using .clinerules on the settings screen +- Add instructions for using .clinerules on the settings screen ## [2.1.8] -- Roo Cline now allows configuration of which commands are allowed without approval! +- Roo Cline now allows configuration of which commands are allowed without approval! ## [2.1.7] -- Updated extension icon and metadata +- Updated extension icon and metadata ## [2.2.0] -- Add support for Model Context Protocol (MCP), enabling Cline to use custom tools like web-search tool or GitHub tool -- Add MCP server management tab accessible via the server icon in the menu bar -- Add ability for Cline to dynamically create new MCP servers based on user requests (e.g., "add a tool that gets the latest npm docs") +- Add support for Model Context Protocol (MCP), enabling Cline to use custom tools like web-search tool or GitHub tool +- Add MCP server management tab accessible via the server icon in the menu bar +- Add ability for Cline to dynamically create new MCP servers based on user requests (e.g., "add a tool that gets the latest npm docs") ## [2.1.6] -- Roo Cline now runs in all VSCode-compatible editors +- Roo Cline now runs in all VSCode-compatible editors ## [2.1.5] -- Fix bug in browser action approval +- Fix bug in browser action approval ## [2.1.4] -- Roo Cline now can run side-by-side with Cline +- Roo Cline now can run side-by-side with Cline ## [2.1.3] -- Roo Cline now allows browser actions without approval when `alwaysAllowBrowser` is true +- Roo Cline now allows browser actions without approval when `alwaysAllowBrowser` is true ## [2.1.2] -- Support for auto-approval of write operations and command execution -- Support for .clinerules custom instructions +- Support for auto-approval of write operations and command execution +- Support for .clinerules custom instructions diff --git a/package-lock.json b/package-lock.json index dab53d5..7518ff6 100644 --- a/package-lock.json +++ b/package-lock.json @@ -1,12 +1,12 @@ { "name": "roo-cline", - "version": "3.1.6", + "version": "3.1.7", "lockfileVersion": 3, "requires": true, "packages": { "": { "name": "roo-cline", - "version": "3.1.6", + "version": "3.1.7", "dependencies": { "@anthropic-ai/bedrock-sdk": "^0.10.2", "@anthropic-ai/sdk": "^0.26.0", diff --git a/package.json b/package.json index cb56216..763bcfa 100644 --- a/package.json +++ b/package.json @@ -3,7 +3,7 @@ "displayName": "Roo Cline", "description": "A fork of Cline, an autonomous coding agent, with some added experimental configuration and automation features.", "publisher": "RooVeterinaryInc", - "version": "3.1.6", + "version": "3.1.7", "icon": "assets/icons/rocket.png", "galleryBanner": { "color": "#617A91", diff --git a/src/api/providers/__tests__/deepseek.test.ts b/src/api/providers/__tests__/deepseek.test.ts index 00526dc..e510b19 100644 --- a/src/api/providers/__tests__/deepseek.test.ts +++ b/src/api/providers/__tests__/deepseek.test.ts @@ -71,7 +71,7 @@ describe("DeepSeekHandler", () => { beforeEach(() => { mockOptions = { deepSeekApiKey: "test-api-key", - deepSeekModelId: "deepseek-chat", + apiModelId: "deepseek-chat", deepSeekBaseUrl: "https://api.deepseek.com/v1", } handler = new DeepSeekHandler(mockOptions) @@ -81,7 +81,7 @@ describe("DeepSeekHandler", () => { describe("constructor", () => { it("should initialize with provided options", () => { expect(handler).toBeInstanceOf(DeepSeekHandler) - expect(handler.getModel().id).toBe(mockOptions.deepSeekModelId) + expect(handler.getModel().id).toBe(mockOptions.apiModelId) }) it("should throw error if API key is missing", () => { @@ -96,7 +96,7 @@ describe("DeepSeekHandler", () => { it("should use default model ID if not provided", () => { const handlerWithoutModel = new DeepSeekHandler({ ...mockOptions, - deepSeekModelId: undefined, + apiModelId: undefined, }) expect(handlerWithoutModel.getModel().id).toBe(deepSeekDefaultModelId) }) @@ -144,7 +144,7 @@ describe("DeepSeekHandler", () => { describe("getModel", () => { it("should return model info for valid model ID", () => { const model = handler.getModel() - expect(model.id).toBe(mockOptions.deepSeekModelId) + expect(model.id).toBe(mockOptions.apiModelId) expect(model.info).toBeDefined() expect(model.info.maxTokens).toBe(8192) expect(model.info.contextWindow).toBe(64_000) @@ -155,7 +155,7 @@ describe("DeepSeekHandler", () => { it("should return provided model ID with default model info if model does not exist", () => { const handlerWithInvalidModel = new DeepSeekHandler({ ...mockOptions, - deepSeekModelId: "invalid-model", + apiModelId: "invalid-model", }) const model = handlerWithInvalidModel.getModel() expect(model.id).toBe("invalid-model") // Returns provided ID @@ -166,7 +166,7 @@ describe("DeepSeekHandler", () => { it("should return default model if no model ID is provided", () => { const handlerWithoutModel = new DeepSeekHandler({ ...mockOptions, - deepSeekModelId: undefined, + apiModelId: undefined, }) const model = handlerWithoutModel.getModel() expect(model.id).toBe(deepSeekDefaultModelId) diff --git a/src/api/providers/__tests__/openai.test.ts b/src/api/providers/__tests__/openai.test.ts index ba65971..52d0c5c 100644 --- a/src/api/providers/__tests__/openai.test.ts +++ b/src/api/providers/__tests__/openai.test.ts @@ -193,7 +193,6 @@ describe("OpenAiHandler", () => { expect(mockCreate).toHaveBeenCalledWith({ model: mockOptions.openAiModelId, messages: [{ role: "user", content: "Test prompt" }], - temperature: 0, }) }) diff --git a/src/api/providers/deepseek.ts b/src/api/providers/deepseek.ts index e559f98..96be435 100644 --- a/src/api/providers/deepseek.ts +++ b/src/api/providers/deepseek.ts @@ -10,14 +10,15 @@ export class DeepSeekHandler extends OpenAiHandler { super({ ...options, openAiApiKey: options.deepSeekApiKey, - openAiModelId: options.deepSeekModelId ?? deepSeekDefaultModelId, + openAiModelId: options.apiModelId ?? deepSeekDefaultModelId, openAiBaseUrl: options.deepSeekBaseUrl ?? "https://api.deepseek.com/v1", + openAiStreamingEnabled: true, includeMaxTokens: true, }) } override getModel(): { id: string; info: ModelInfo } { - const modelId = this.options.deepSeekModelId ?? deepSeekDefaultModelId + const modelId = this.options.apiModelId ?? deepSeekDefaultModelId return { id: modelId, info: deepSeekModels[modelId as keyof typeof deepSeekModels] || deepSeekModels[deepSeekDefaultModelId], diff --git a/src/api/providers/glama.ts b/src/api/providers/glama.ts index 7f10592..b475438 100644 --- a/src/api/providers/glama.ts +++ b/src/api/providers/glama.ts @@ -101,6 +101,14 @@ export class GlamaHandler implements ApiHandler, SingleCompletionHandler { { headers: { Authorization: `Bearer ${this.options.glamaApiKey}`, + "X-Glama-Metadata": JSON.stringify({ + labels: [ + { + key: "app", + value: "vscode.rooveterinaryinc.roo-cline", + }, + ], + }), }, }, ) diff --git a/src/api/providers/openai.ts b/src/api/providers/openai.ts index f1dbe6e..13922a4 100644 --- a/src/api/providers/openai.ts +++ b/src/api/providers/openai.ts @@ -36,7 +36,9 @@ export class OpenAiHandler implements ApiHandler, SingleCompletionHandler { const modelInfo = this.getModel().info const modelId = this.options.openAiModelId ?? "" - if (this.options.openAiStreamingEnabled ?? true) { + const deepseekReasoner = modelId.includes("deepseek-reasoner") + + if (!deepseekReasoner && (this.options.openAiStreamingEnabled ?? true)) { const systemMessage: OpenAI.Chat.ChatCompletionSystemMessageParam = { role: "system", content: systemPrompt, @@ -71,11 +73,20 @@ export class OpenAiHandler implements ApiHandler, SingleCompletionHandler { } } } else { + let systemMessage: OpenAI.Chat.ChatCompletionUserMessageParam | OpenAI.Chat.ChatCompletionSystemMessageParam + // o1 for instance doesnt support streaming, non-1 temp, or system prompt - const systemMessage: OpenAI.Chat.ChatCompletionUserMessageParam = { - role: "user", - content: systemPrompt, - } + // deepseek reasoner supports system prompt + systemMessage = deepseekReasoner + ? { + role: "system", + content: systemPrompt, + } + : { + role: "user", + content: systemPrompt, + } + const requestOptions: OpenAI.Chat.Completions.ChatCompletionCreateParamsNonStreaming = { model: modelId, messages: [systemMessage, ...convertToOpenAiMessages(messages)], @@ -106,7 +117,6 @@ export class OpenAiHandler implements ApiHandler, SingleCompletionHandler { const requestOptions: OpenAI.Chat.Completions.ChatCompletionCreateParamsNonStreaming = { model: this.getModel().id, messages: [{ role: "user", content: prompt }], - temperature: 0, } const response = await this.client.chat.completions.create(requestOptions) diff --git a/src/core/config/ConfigManager.ts b/src/core/config/ConfigManager.ts index ef3d4d3..0299955 100644 --- a/src/core/config/ConfigManager.ts +++ b/src/core/config/ConfigManager.ts @@ -64,7 +64,7 @@ export class ConfigManager { /** * List all available configs with metadata */ - async ListConfig(): Promise { + async listConfig(): Promise { try { const config = await this.readConfig() return Object.entries(config.apiConfigs).map(([name, apiConfig]) => ({ @@ -80,7 +80,7 @@ export class ConfigManager { /** * Save a config with the given name */ - async SaveConfig(name: string, config: ApiConfiguration): Promise { + async saveConfig(name: string, config: ApiConfiguration): Promise { try { const currentConfig = await this.readConfig() const existingConfig = currentConfig.apiConfigs[name] @@ -97,7 +97,7 @@ export class ConfigManager { /** * Load a config by name */ - async LoadConfig(name: string): Promise { + async loadConfig(name: string): Promise { try { const config = await this.readConfig() const apiConfig = config.apiConfigs[name] @@ -118,7 +118,7 @@ export class ConfigManager { /** * Delete a config by name */ - async DeleteConfig(name: string): Promise { + async deleteConfig(name: string): Promise { try { const currentConfig = await this.readConfig() if (!currentConfig.apiConfigs[name]) { @@ -140,7 +140,7 @@ export class ConfigManager { /** * Set the current active API configuration */ - async SetCurrentConfig(name: string): Promise { + async setCurrentConfig(name: string): Promise { try { const currentConfig = await this.readConfig() if (!currentConfig.apiConfigs[name]) { @@ -157,7 +157,7 @@ export class ConfigManager { /** * Check if a config exists by name */ - async HasConfig(name: string): Promise { + async hasConfig(name: string): Promise { try { const config = await this.readConfig() return name in config.apiConfigs @@ -169,7 +169,7 @@ export class ConfigManager { /** * Set the API config for a specific mode */ - async SetModeConfig(mode: Mode, configId: string): Promise { + async setModeConfig(mode: Mode, configId: string): Promise { try { const currentConfig = await this.readConfig() if (!currentConfig.modeApiConfigs) { @@ -185,7 +185,7 @@ export class ConfigManager { /** * Get the API config ID for a specific mode */ - async GetModeConfigId(mode: Mode): Promise { + async getModeConfigId(mode: Mode): Promise { try { const config = await this.readConfig() return config.modeApiConfigs?.[mode] @@ -194,10 +194,23 @@ export class ConfigManager { } } + /** + * Get the key used for storing config in secrets + */ + private getConfigKey(): string { + return `${this.SCOPE_PREFIX}api_config` + } + + /** + * Reset all configuration by deleting the stored config from secrets + */ + public async resetAllConfigs(): Promise { + await this.context.secrets.delete(this.getConfigKey()) + } + private async readConfig(): Promise { try { - const configKey = `${this.SCOPE_PREFIX}api_config` - const content = await this.context.secrets.get(configKey) + const content = await this.context.secrets.get(this.getConfigKey()) if (!content) { return this.defaultConfig @@ -211,9 +224,8 @@ export class ConfigManager { private async writeConfig(config: ApiConfigData): Promise { try { - const configKey = `${this.SCOPE_PREFIX}api_config` const content = JSON.stringify(config, null, 2) - await this.context.secrets.store(configKey, content) + await this.context.secrets.store(this.getConfigKey(), content) } catch (error) { throw new Error(`Failed to write config to secrets: ${error}`) } diff --git a/src/core/config/__tests__/ConfigManager.test.ts b/src/core/config/__tests__/ConfigManager.test.ts index 59f36e6..3d65021 100644 --- a/src/core/config/__tests__/ConfigManager.test.ts +++ b/src/core/config/__tests__/ConfigManager.test.ts @@ -106,7 +106,7 @@ describe("ConfigManager", () => { mockSecrets.get.mockResolvedValue(JSON.stringify(existingConfig)) - const configs = await configManager.ListConfig() + const configs = await configManager.listConfig() expect(configs).toEqual([ { name: "default", id: "default", apiProvider: undefined }, { name: "test", id: "test-id", apiProvider: "anthropic" }, @@ -126,14 +126,14 @@ describe("ConfigManager", () => { mockSecrets.get.mockResolvedValue(JSON.stringify(emptyConfig)) - const configs = await configManager.ListConfig() + const configs = await configManager.listConfig() expect(configs).toEqual([]) }) it("should throw error if reading from secrets fails", async () => { mockSecrets.get.mockRejectedValue(new Error("Read failed")) - await expect(configManager.ListConfig()).rejects.toThrow( + await expect(configManager.listConfig()).rejects.toThrow( "Failed to list configs: Error: Failed to read config from secrets: Error: Read failed", ) }) @@ -160,7 +160,7 @@ describe("ConfigManager", () => { apiKey: "test-key", } - await configManager.SaveConfig("test", newConfig) + await configManager.saveConfig("test", newConfig) // Get the actual stored config to check the generated ID const storedConfig = JSON.parse(mockSecrets.store.mock.calls[0][1]) @@ -207,7 +207,7 @@ describe("ConfigManager", () => { apiKey: "new-key", } - await configManager.SaveConfig("test", updatedConfig) + await configManager.saveConfig("test", updatedConfig) const expectedConfig = { currentApiConfigName: "default", @@ -235,7 +235,7 @@ describe("ConfigManager", () => { ) mockSecrets.store.mockRejectedValueOnce(new Error("Storage failed")) - await expect(configManager.SaveConfig("test", {})).rejects.toThrow( + await expect(configManager.saveConfig("test", {})).rejects.toThrow( "Failed to save config: Error: Failed to write config to secrets: Error: Storage failed", ) }) @@ -258,7 +258,7 @@ describe("ConfigManager", () => { mockSecrets.get.mockResolvedValue(JSON.stringify(existingConfig)) - await configManager.DeleteConfig("test") + await configManager.deleteConfig("test") // Get the stored config to check the ID const storedConfig = JSON.parse(mockSecrets.store.mock.calls[0][1]) @@ -275,7 +275,7 @@ describe("ConfigManager", () => { }), ) - await expect(configManager.DeleteConfig("nonexistent")).rejects.toThrow("Config 'nonexistent' not found") + await expect(configManager.deleteConfig("nonexistent")).rejects.toThrow("Config 'nonexistent' not found") }) it("should throw error when trying to delete last remaining config", async () => { @@ -290,7 +290,7 @@ describe("ConfigManager", () => { }), ) - await expect(configManager.DeleteConfig("default")).rejects.toThrow( + await expect(configManager.deleteConfig("default")).rejects.toThrow( "Cannot delete the last remaining configuration.", ) }) @@ -311,7 +311,7 @@ describe("ConfigManager", () => { mockSecrets.get.mockResolvedValue(JSON.stringify(existingConfig)) - const config = await configManager.LoadConfig("test") + const config = await configManager.loadConfig("test") expect(config).toEqual({ apiProvider: "anthropic", @@ -342,7 +342,7 @@ describe("ConfigManager", () => { }), ) - await expect(configManager.LoadConfig("nonexistent")).rejects.toThrow("Config 'nonexistent' not found") + await expect(configManager.loadConfig("nonexistent")).rejects.toThrow("Config 'nonexistent' not found") }) it("should throw error if secrets storage fails", async () => { @@ -361,7 +361,7 @@ describe("ConfigManager", () => { ) mockSecrets.store.mockRejectedValueOnce(new Error("Storage failed")) - await expect(configManager.LoadConfig("test")).rejects.toThrow( + await expect(configManager.loadConfig("test")).rejects.toThrow( "Failed to load config: Error: Failed to write config to secrets: Error: Storage failed", ) }) @@ -384,7 +384,7 @@ describe("ConfigManager", () => { mockSecrets.get.mockResolvedValue(JSON.stringify(existingConfig)) - await configManager.SetCurrentConfig("test") + await configManager.setCurrentConfig("test") // Get the stored config to check the structure const storedConfig = JSON.parse(mockSecrets.store.mock.calls[0][1]) @@ -404,7 +404,7 @@ describe("ConfigManager", () => { }), ) - await expect(configManager.SetCurrentConfig("nonexistent")).rejects.toThrow( + await expect(configManager.setCurrentConfig("nonexistent")).rejects.toThrow( "Config 'nonexistent' not found", ) }) @@ -420,12 +420,34 @@ describe("ConfigManager", () => { ) mockSecrets.store.mockRejectedValueOnce(new Error("Storage failed")) - await expect(configManager.SetCurrentConfig("test")).rejects.toThrow( + await expect(configManager.setCurrentConfig("test")).rejects.toThrow( "Failed to set current config: Error: Failed to write config to secrets: Error: Storage failed", ) }) }) + describe("ResetAllConfigs", () => { + it("should delete all stored configs", async () => { + // Setup initial config + mockSecrets.get.mockResolvedValue( + JSON.stringify({ + currentApiConfigName: "test", + apiConfigs: { + test: { + apiProvider: "anthropic", + id: "test-id", + }, + }, + }), + ) + + await configManager.resetAllConfigs() + + // Should have called delete with the correct config key + expect(mockSecrets.delete).toHaveBeenCalledWith("roo_cline_config_api_config") + }) + }) + describe("HasConfig", () => { it("should return true for existing config", async () => { const existingConfig: ApiConfigData = { @@ -443,7 +465,7 @@ describe("ConfigManager", () => { mockSecrets.get.mockResolvedValue(JSON.stringify(existingConfig)) - const hasConfig = await configManager.HasConfig("test") + const hasConfig = await configManager.hasConfig("test") expect(hasConfig).toBe(true) }) @@ -455,14 +477,14 @@ describe("ConfigManager", () => { }), ) - const hasConfig = await configManager.HasConfig("nonexistent") + const hasConfig = await configManager.hasConfig("nonexistent") expect(hasConfig).toBe(false) }) it("should throw error if secrets storage fails", async () => { mockSecrets.get.mockRejectedValue(new Error("Storage failed")) - await expect(configManager.HasConfig("test")).rejects.toThrow( + await expect(configManager.hasConfig("test")).rejects.toThrow( "Failed to check config existence: Error: Failed to read config from secrets: Error: Storage failed", ) }) diff --git a/src/core/diff/strategies/__tests__/new-unified.test.ts b/src/core/diff/strategies/__tests__/new-unified.test.ts index da30173..8832f9e 100644 --- a/src/core/diff/strategies/__tests__/new-unified.test.ts +++ b/src/core/diff/strategies/__tests__/new-unified.test.ts @@ -29,10 +29,12 @@ describe("main", () => { const cwd = "/test/path" const description = strategy.getToolDescription({ cwd }) - expect(description).toContain("apply_diff") + expect(description).toContain("apply_diff Tool - Generate Precise Code Changes") expect(description).toContain(cwd) + expect(description).toContain("Step-by-Step Instructions") + expect(description).toContain("Requirements") + expect(description).toContain("Examples") expect(description).toContain("Parameters:") - expect(description).toContain("Format Requirements:") }) }) diff --git a/src/core/diff/strategies/new-unified/index.ts b/src/core/diff/strategies/new-unified/index.ts index 7c19384..d256614 100644 --- a/src/core/diff/strategies/new-unified/index.ts +++ b/src/core/diff/strategies/new-unified/index.ts @@ -108,77 +108,68 @@ export class NewUnifiedDiffStrategy implements DiffStrategy { } getToolDescription(args: { cwd: string; toolOptions?: { [key: string]: string } }): string { - return `# apply_diff Tool Rules: + return `# apply_diff Tool - Generate Precise Code Changes -Generate a unified diff similar to what "diff -U0" would produce. +Generate a unified diff that can be cleanly applied to modify code files. -The first two lines must include the file paths, starting with "---" for the original file and "+++" for the updated file. Do not include timestamps with the file paths. +## Step-by-Step Instructions: -Each hunk of changes must start with a line containing only "@@ ... @@". Do not include line numbers or ranges in the "@@ ... @@" lines. These are not necessary for the user's patch tool. +1. Start with file headers: + - First line: "--- {original_file_path}" + - Second line: "+++ {new_file_path}" -Your output must be a correct, clean patch that applies successfully against the current file contents. Mark all lines that need to be removed or changed with "-". Mark all new or modified lines with "+". Ensure you include all necessary changes; missing or unmarked lines will result in a broken patch. +2. For each change section: + - Begin with "@@ ... @@" separator line without line numbers + - Include 2-3 lines of context before and after changes + - Mark removed lines with "-" + - Mark added lines with "+" + - Preserve exact indentation -Indentation matters! Make sure to preserve the exact indentation of both removed and added lines. +3. Group related changes: + - Keep related modifications in the same hunk + - Start new hunks for logically separate changes + - When modifying functions/methods, include the entire block -Start a new hunk for each section of the file that requires changes. However, include only the hunks that contain actual changes. If a hunk consists entirely of unchanged lines, skip it. +## Requirements: -Group related changes together in the same hunk whenever possible. Output hunks in whatever logical order makes the most sense. +1. MUST include exact indentation +2. MUST include sufficient context for unique matching +3. MUST group related changes together +4. MUST use proper unified diff format +5. MUST NOT include timestamps in file headers +6. MUST NOT include line numbers in the @@ header -When editing a function, method, loop, or similar code block, replace the *entire* block in one hunk. Use "-" lines to delete the existing block and "+" lines to add the updated block. This ensures accuracy in your diffs. - -If you need to move code within a file, create two hunks: one to delete the code from its original location and another to insert it at the new location. - -To create a new file, show a diff from "--- /dev/null" to "+++ path/to/new/file.ext". - -Format Requirements: +## Examples: +✅ Good diff (follows all requirements): \`\`\`diff ---- mathweb/flask/app.py -+++ mathweb/flask/app.py +--- src/utils.ts ++++ src/utils.ts @@ ... @@ --class MathWeb: -+import sympy - -+ -+class MathWeb: -@@ ... @@ --def is_prime(x): -- if x < 2: -- return False -- for i in range(2, int(math.sqrt(x)) + 1): -- if x % i == 0: -- return False -- return True -@@ ... @@ --@app.route('/prime/') --def nth_prime(n): -- count = 0 -- num = 1 -- while count < n: -- num += 1 -- if is_prime(num): -- count += 1 -- return str(num) -+@app.route('/prime/') -+def nth_prime(n): -+ count = 0 -+ num = 1 -+ while count < n: -+ num += 1 -+ if sympy.isprime(num): -+ count += 1 -+ return str(num) + def calculate_total(items): +- total = 0 +- for item in items: +- total += item.price ++ return sum(item.price for item in items) \`\`\` -Be precise, consistent, and follow these rules carefully to generate correct diffs! +❌ Bad diff (violates requirements #1 and #2): +\`\`\`diff +--- src/utils.ts ++++ src/utils.ts +@@ ... @@ +-total = 0 +-for item in items: ++return sum(item.price for item in items) +\`\`\` Parameters: -- path: (required) The path of the file to apply the diff to (relative to the current working directory ${args.cwd}) -- diff: (required) The diff content in unified format to apply to the file. +- path: (required) File path relative to ${args.cwd} +- diff: (required) Unified diff content in unified format to apply to the file. Usage: -File path here +path/to/file.ext Your diff here @@ -242,7 +233,7 @@ Your diff here originalContent: string, diffContent: string, startLine?: number, - endLine?: number, + endLine?: number ): Promise { const parsedDiff = this.parseUnifiedDiff(diffContent) const originalLines = originalContent.split("\n") @@ -280,7 +271,7 @@ Your diff here subHunkResult, subSearchResult.index, subSearchResult.confidence, - this.confidenceThreshold, + this.confidenceThreshold ) if (subEditResult.confidence >= this.confidenceThreshold) { subHunkResult = subEditResult.result @@ -302,12 +293,12 @@ Your diff here const contextRatio = contextLines / totalLines let errorMsg = `Failed to find a matching location in the file (${Math.floor( - confidence * 100, + confidence * 100 )}% confidence, needs ${Math.floor(this.confidenceThreshold * 100)}%)\n\n` errorMsg += "Debug Info:\n" errorMsg += `- Search Strategy Used: ${strategy}\n` errorMsg += `- Context Lines: ${contextLines} out of ${totalLines} total lines (${Math.floor( - contextRatio * 100, + contextRatio * 100 )}%)\n` errorMsg += `- Attempted to split into ${subHunks.length} sub-hunks but still failed\n` @@ -339,7 +330,7 @@ Your diff here } else { // Edit failure - likely due to content mismatch let errorMsg = `Failed to apply the edit using ${editResult.strategy} strategy (${Math.floor( - editResult.confidence * 100, + editResult.confidence * 100 )}% confidence)\n\n` errorMsg += "Debug Info:\n" errorMsg += "- The location was found but the content didn't match exactly\n" diff --git a/src/core/prompts/__tests__/__snapshots__/system.test.ts.snap b/src/core/prompts/__tests__/__snapshots__/system.test.ts.snap index 3fbc2d8..b7f97fe 100644 --- a/src/core/prompts/__tests__/__snapshots__/system.test.ts.snap +++ b/src/core/prompts/__tests__/__snapshots__/system.test.ts.snap @@ -2585,487 +2585,6 @@ By waiting for and carefully considering the user's response after each tool use -==== - -CAPABILITIES - -- You have access to tools that let you execute CLI commands on the user's computer, list files, view source code definitions, regex search, read and write files, and ask follow-up questions. These tools help you effectively accomplish a wide range of tasks, such as writing code, making edits or improvements to existing files, understanding the current state of a project, performing system operations, and much more. -- When the user initially gives you a task, a recursive list of all filepaths in the current working directory ('/test/path') will be included in environment_details. This provides an overview of the project's file structure, offering key insights into the project from directory/file names (how developers conceptualize and organize their code) and file extensions (the language used). This can also guide decision-making on which files to explore further. If you need to further explore directories such as outside the current working directory, you can use the list_files tool. If you pass 'true' for the recursive parameter, it will list files recursively. Otherwise, it will list files at the top level, which is better suited for generic directories where you don't necessarily need the nested structure, like the Desktop. -- You can use search_files to perform regex searches across files in a specified directory, outputting context-rich results that include surrounding lines. This is particularly useful for understanding code patterns, finding specific implementations, or identifying areas that need refactoring. -- You can use the list_code_definition_names tool to get an overview of source code definitions for all files at the top level of a specified directory. This can be particularly useful when you need to understand the broader context and relationships between certain parts of the code. You may need to call this tool multiple times to understand various parts of the codebase related to the task. - - For example, when asked to make edits or improvements you might analyze the file structure in the initial environment_details to get an overview of the project, then use list_code_definition_names to get further insight using source code definitions for files located in relevant directories, then read_file to examine the contents of relevant files, analyze the code and suggest improvements or make necessary edits, then use the write_to_file tool to apply the changes. If you refactored code that could affect other parts of the codebase, you could use search_files to ensure you update other files as needed. -- You can use the execute_command tool to run commands on the user's computer whenever you feel it can help accomplish the user's task. When you need to execute a CLI command, you must provide a clear explanation of what the command does. Prefer to execute complex CLI commands over creating executable scripts, since they are more flexible and easier to run. Interactive and long-running commands are allowed, since the commands are run in the user's VSCode terminal. The user may keep commands running in the background and you will be kept updated on their status along the way. Each command you execute is run in a new terminal instance. - -==== - -RULES - -- Your current working directory is: /test/path -- You cannot \`cd\` into a different directory to complete a task. You are stuck operating from '/test/path', so be sure to pass in the correct 'path' parameter when using tools that require a path. -- Do not use the ~ character or $HOME to refer to the home directory. -- Before using the execute_command tool, you must first think about the SYSTEM INFORMATION context provided to understand the user's environment and tailor your commands to ensure they are compatible with their system. You must also consider if the command you need to run should be executed in a specific directory outside of the current working directory '/test/path', and if so prepend with \`cd\`'ing into that directory && then executing the command (as one command since you are stuck operating from '/test/path'). For example, if you needed to run \`npm install\` in a project outside of '/test/path', you would need to prepend with a \`cd\` i.e. pseudocode for this would be \`cd (path to project) && (command, in this case npm install)\`. -- When using the search_files tool, craft your regex patterns carefully to balance specificity and flexibility. Based on the user's task you may use it to find code patterns, TODO comments, function definitions, or any text-based information across the project. The results include context, so analyze the surrounding code to better understand the matches. Leverage the search_files tool in combination with other tools for more comprehensive analysis. For example, use it to find specific code patterns, then use read_file to examine the full context of interesting matches before using write_to_file to make informed changes. -- When creating a new project (such as an app, website, or any software project), organize all new files within a dedicated project directory unless the user specifies otherwise. Use appropriate file paths when writing files, as the write_to_file tool will automatically create any necessary directories. Structure the project logically, adhering to best practices for the specific type of project being created. Unless otherwise specified, new projects should be easily run without additional setup, for example most projects can be built in HTML, CSS, and JavaScript - which you can open in a browser. -- When you want to modify a file, use the write_to_file tool directly with the desired content. You do not need to display the content before using the tool. -- Be sure to consider the type of project (e.g. Python, JavaScript, web application) when determining the appropriate structure and files to include. Also consider what files may be most relevant to accomplishing the task, for example looking at a project's manifest file would help you understand the project's dependencies, which you could incorporate into any code you write. -- When making changes to code, always consider the context in which the code is being used. Ensure that your changes are compatible with the existing codebase and that they follow the project's coding standards and best practices. -- Do not ask for more information than necessary. Use the tools provided to accomplish the user's request efficiently and effectively. When you've completed your task, you must use the attempt_completion tool to present the result to the user. The user may provide feedback, which you can use to make improvements and try again. -- You are only allowed to ask the user questions using the ask_followup_question tool. Use this tool only when you need additional details to complete a task, and be sure to use a clear and concise question that will help you move forward with the task. However if you can use the available tools to avoid having to ask the user questions, you should do so. For example, if the user mentions a file that may be in an outside directory like the Desktop, you should use the list_files tool to list the files in the Desktop and check if the file they are talking about is there, rather than asking the user to provide the file path themselves. -- When executing commands, if you don't see the expected output, assume the terminal executed the command successfully and proceed with the task. The user's terminal may be unable to stream the output back properly. If you absolutely need to see the actual terminal output, use the ask_followup_question tool to request the user to copy and paste it back to you. -- The user may provide a file's contents directly in their message, in which case you shouldn't use the read_file tool to get the file contents again since you already have it. -- Your goal is to try to accomplish the user's task, NOT engage in a back and forth conversation. -- NEVER end attempt_completion result with a question or request to engage in further conversation! Formulate the end of your result in a way that is final and does not require further input from the user. -- You are STRICTLY FORBIDDEN from starting your messages with "Great", "Certainly", "Okay", "Sure". You should NOT be conversational in your responses, but rather direct and to the point. For example you should NOT say "Great, I've updated the CSS" but instead something like "I've updated the CSS". It is important you be clear and technical in your messages. -- When presented with images, utilize your vision capabilities to thoroughly examine them and extract meaningful information. Incorporate these insights into your thought process as you accomplish the user's task. -- At the end of each user message, you will automatically receive environment_details. This information is not written by the user themselves, but is auto-generated to provide potentially relevant context about the project structure and environment. While this information can be valuable for understanding the project context, do not treat it as a direct part of the user's request or response. Use it to inform your actions and decisions, but don't assume the user is explicitly asking about or referring to this information unless they clearly do so in their message. When using environment_details, explain your actions clearly to ensure the user understands, as they may not be aware of these details. -- Before executing commands, check the "Actively Running Terminals" section in environment_details. If present, consider how these active processes might impact your task. For example, if a local development server is already running, you wouldn't need to start it again. If no active terminals are listed, proceed with command execution as normal. -- When using the write_to_file tool, ALWAYS provide the COMPLETE file content in your response. This is NON-NEGOTIABLE. Partial updates or placeholders like '// rest of code unchanged' are STRICTLY FORBIDDEN. You MUST include ALL parts of the file, even if they haven't been modified. Failure to do so will result in incomplete or broken code, severely impacting the user's project. -- MCP operations should be used one at a time, similar to other tool usage. Wait for confirmation of success before proceeding with additional operations. -- It is critical you wait for the user's response after each tool use, in order to confirm the success of the tool use. For example, if asked to make a todo app, you would create a file, wait for the user's response it was created successfully, then create another file if needed, wait for the user's response it was created successfully, etc. - -==== - -SYSTEM INFORMATION - -Operating System: Linux -Default Shell: /bin/bash -Home Directory: /home/user -Current Working Directory: /test/path - -When the user initially gives you a task, a recursive list of all filepaths in the current working directory ('/test/path') will be included in environment_details. This provides an overview of the project's file structure, offering key insights into the project from directory/file names (how developers conceptualize and organize their code) and file extensions (the language used). This can also guide decision-making on which files to explore further. If you need to further explore directories such as outside the current working directory, you can use the list_files tool. If you pass 'true' for the recursive parameter, it will list files recursively. Otherwise, it will list files at the top level, which is better suited for generic directories where you don't necessarily need the nested structure, like the Desktop. - -==== - -OBJECTIVE - -You accomplish a given task iteratively, breaking it down into clear steps and working through them methodically. - -1. Analyze the user's task and set clear, achievable goals to accomplish it. Prioritize these goals in a logical order. -2. Work through these goals sequentially, utilizing available tools one at a time as necessary. Each goal should correspond to a distinct step in your problem-solving process. You will be informed on the work completed and what's remaining as you go. -3. Remember, you have extensive capabilities with access to a wide range of tools that can be used in powerful and clever ways as necessary to accomplish each goal. Before calling a tool, do some analysis within tags. First, analyze the file structure provided in environment_details to gain context and insights for proceeding effectively. Then, think about which of the provided tools is the most relevant tool to accomplish the user's task. Next, go through each of the required parameters of the relevant tool and determine if the user has directly provided or given enough information to infer a value. When deciding if the parameter can be inferred, carefully consider all the context to see if it supports a specific value. If all of the required parameters are present or can be reasonably inferred, close the thinking tag and proceed with the tool use. BUT, if one of the values for a required parameter is missing, DO NOT invoke the tool (not even with fillers for the missing params) and instead, ask the user to provide the missing parameters using the ask_followup_question tool. DO NOT ask for more information on optional parameters if it is not provided. -4. Once you've completed the user's task, you must use the attempt_completion tool to present the result of the task to the user. You may also provide a CLI command to showcase the result of your task; this can be particularly useful for web development tasks, where you can run e.g. \`open index.html\` to show the website you've built. -5. The user may provide feedback, which you can use to make improvements and try again. But DO NOT continue in pointless back and forth conversations, i.e. don't end your responses with questions or offers for further assistance." -`; - -exports[`addCustomInstructions should generate correct prompt for code reviewer mode 1`] = ` -"You are Cline, a code review expert specializing in providing detailed, actionable feedback on code quality and maintainability. You excel at: 1) Identifying potential bugs, security vulnerabilities, and performance issues, 2) Ensuring code follows project standards, patterns, and best practices, 3) Checking for proper error handling and edge cases, 4) Verifying documentation completeness and clarity, 5) Suggesting specific, actionable improvements with examples. You maintain a read-only approach to the codebase and focus on helping developers improve their code through clear, constructive feedback. - -==== - -TOOL USE - -You have access to a set of tools that are executed upon the user's approval. You can use one tool per message, and will receive the result of that tool use in the user's response. You use tools step-by-step to accomplish a given task, with each tool use informed by the result of the previous tool use. - -# Tool Use Formatting - -Tool use is formatted using XML-style tags. The tool name is enclosed in opening and closing tags, and each parameter is similarly enclosed within its own set of tags. Here's the structure: - - -value1 -value2 -... - - -For example: - - -src/main.js - - -Always adhere to this format for the tool use to ensure proper parsing and execution. - -# Tools - -## read_file -Description: Request to read the contents of a file at the specified path. Use this when you need to examine the contents of an existing file you do not know the contents of, for example to analyze code, review text files, or extract information from configuration files. The output includes line numbers prefixed to each line (e.g. "1 | const x = 1"), making it easier to reference specific lines when creating diffs or discussing code. Automatically extracts raw text from PDF and DOCX files. May not be suitable for other types of binary files, as it returns the raw content as a string. -Parameters: -- path: (required) The path of the file to read (relative to the current working directory /test/path) -Usage: - -File path here - - -Example: Requesting to read frontend-config.json - -frontend-config.json - - -## search_files -Description: Request to perform a regex search across files in a specified directory, providing context-rich results. This tool searches for patterns or specific content across multiple files, displaying each match with encapsulating context. -Parameters: -- path: (required) The path of the directory to search in (relative to the current working directory /test/path). This directory will be recursively searched. -- regex: (required) The regular expression pattern to search for. Uses Rust regex syntax. -- file_pattern: (optional) Glob pattern to filter files (e.g., '*.ts' for TypeScript files). If not provided, it will search all files (*). -Usage: - -Directory path here -Your regex pattern here -file pattern here (optional) - - -Example: Requesting to search for all .ts files in the current directory - -. -.* -*.ts - - -## list_files -Description: Request to list files and directories within the specified directory. If recursive is true, it will list all files and directories recursively. If recursive is false or not provided, it will only list the top-level contents. Do not use this tool to confirm the existence of files you may have created, as the user will let you know if the files were created successfully or not. -Parameters: -- path: (required) The path of the directory to list contents for (relative to the current working directory /test/path) -- recursive: (optional) Whether to list files recursively. Use true for recursive listing, false or omit for top-level only. -Usage: - -Directory path here -true or false (optional) - - -Example: Requesting to list all files in the current directory - -. -false - - -## list_code_definition_names -Description: Request to list definition names (classes, functions, methods, etc.) used in source code files at the top level of the specified directory. This tool provides insights into the codebase structure and important constructs, encapsulating high-level concepts and relationships that are crucial for understanding the overall architecture. -Parameters: -- path: (required) The path of the directory (relative to the current working directory /test/path) to list top level source code definitions for. -Usage: - -Directory path here - - -Example: Requesting to list all top level source code definitions in the current directory - -. - - -## ask_followup_question -Description: Ask the user a question to gather additional information needed to complete the task. This tool should be used when you encounter ambiguities, need clarification, or require more details to proceed effectively. It allows for interactive problem-solving by enabling direct communication with the user. Use this tool judiciously to maintain a balance between gathering necessary information and avoiding excessive back-and-forth. -Parameters: -- question: (required) The question to ask the user. This should be a clear, specific question that addresses the information you need. -Usage: - -Your question here - - -Example: Requesting to ask the user for the path to the frontend-config.json file - -What is the path to the frontend-config.json file? - - -## attempt_completion -Description: After each tool use, the user will respond with the result of that tool use, i.e. if it succeeded or failed, along with any reasons for failure. Once you've received the results of tool uses and can confirm that the task is complete, use this tool to present the result of your work to the user. Optionally you may provide a CLI command to showcase the result of your work. The user may respond with feedback if they are not satisfied with the result, which you can use to make improvements and try again. -IMPORTANT NOTE: This tool CANNOT be used until you've confirmed from the user that any previous tool uses were successful. Failure to do so will result in code corruption and system failure. Before using this tool, you must ask yourself in tags if you've confirmed from the user that any previous tool uses were successful. If not, then DO NOT use this tool. -Parameters: -- result: (required) The result of the task. Formulate this result in a way that is final and does not require further input from the user. Don't end your result with questions or offers for further assistance. -- command: (optional) A CLI command to execute to show a live demo of the result to the user. For example, use \`open index.html\` to display a created html website, or \`open localhost:3000\` to display a locally running development server. But DO NOT use commands like \`echo\` or \`cat\` that merely print text. This command should be valid for the current operating system. Ensure the command is properly formatted and does not contain any harmful instructions. -Usage: - - -Your final result description here - -Command to demonstrate result (optional) - - -Example: Requesting to attempt completion with a result and command - - -I've updated the CSS - -open index.html - - -# Tool Use Guidelines - -1. In tags, assess what information you already have and what information you need to proceed with the task. -2. Choose the most appropriate tool based on the task and the tool descriptions provided. Assess if you need additional information to proceed, and which of the available tools would be most effective for gathering this information. For example using the list_files tool is more effective than running a command like \`ls\` in the terminal. It's critical that you think about each available tool and use the one that best fits the current step in the task. -3. If multiple actions are needed, use one tool at a time per message to accomplish the task iteratively, with each tool use being informed by the result of the previous tool use. Do not assume the outcome of any tool use. Each step must be informed by the previous step's result. -4. Formulate your tool use using the XML format specified for each tool. -5. After each tool use, the user will respond with the result of that tool use. This result will provide you with the necessary information to continue your task or make further decisions. This response may include: - - Information about whether the tool succeeded or failed, along with any reasons for failure. - - Linter errors that may have arisen due to the changes you made, which you'll need to address. - - New terminal output in reaction to the changes, which you may need to consider or act upon. - - Any other relevant feedback or information related to the tool use. -6. ALWAYS wait for user confirmation after each tool use before proceeding. Never assume the success of a tool use without explicit confirmation of the result from the user. - -It is crucial to proceed step-by-step, waiting for the user's message after each tool use before moving forward with the task. This approach allows you to: -1. Confirm the success of each step before proceeding. -2. Address any issues or errors that arise immediately. -3. Adapt your approach based on new information or unexpected results. -4. Ensure that each action builds correctly on the previous ones. - -By waiting for and carefully considering the user's response after each tool use, you can react accordingly and make informed decisions about how to proceed with the task. This iterative process helps ensure the overall success and accuracy of your work. - - - -==== - -CAPABILITIES - -- You have access to tools that let you execute CLI commands on the user's computer, list files, view source code definitions, regex search, read and write files, and ask follow-up questions. These tools help you effectively accomplish a wide range of tasks, such as writing code, making edits or improvements to existing files, understanding the current state of a project, performing system operations, and much more. -- When the user initially gives you a task, a recursive list of all filepaths in the current working directory ('/test/path') will be included in environment_details. This provides an overview of the project's file structure, offering key insights into the project from directory/file names (how developers conceptualize and organize their code) and file extensions (the language used). This can also guide decision-making on which files to explore further. If you need to further explore directories such as outside the current working directory, you can use the list_files tool. If you pass 'true' for the recursive parameter, it will list files recursively. Otherwise, it will list files at the top level, which is better suited for generic directories where you don't necessarily need the nested structure, like the Desktop. -- You can use search_files to perform regex searches across files in a specified directory, outputting context-rich results that include surrounding lines. This is particularly useful for understanding code patterns, finding specific implementations, or identifying areas that need refactoring. -- You can use the list_code_definition_names tool to get an overview of source code definitions for all files at the top level of a specified directory. This can be particularly useful when you need to understand the broader context and relationships between certain parts of the code. You may need to call this tool multiple times to understand various parts of the codebase related to the task. - - For example, when asked to make edits or improvements you might analyze the file structure in the initial environment_details to get an overview of the project, then use list_code_definition_names to get further insight using source code definitions for files located in relevant directories, then read_file to examine the contents of relevant files, analyze the code and suggest improvements or make necessary edits, then use the write_to_file tool to apply the changes. If you refactored code that could affect other parts of the codebase, you could use search_files to ensure you update other files as needed. -- You can use the execute_command tool to run commands on the user's computer whenever you feel it can help accomplish the user's task. When you need to execute a CLI command, you must provide a clear explanation of what the command does. Prefer to execute complex CLI commands over creating executable scripts, since they are more flexible and easier to run. Interactive and long-running commands are allowed, since the commands are run in the user's VSCode terminal. The user may keep commands running in the background and you will be kept updated on their status along the way. Each command you execute is run in a new terminal instance. - -==== - -RULES - -- Your current working directory is: /test/path -- You cannot \`cd\` into a different directory to complete a task. You are stuck operating from '/test/path', so be sure to pass in the correct 'path' parameter when using tools that require a path. -- Do not use the ~ character or $HOME to refer to the home directory. -- Before using the execute_command tool, you must first think about the SYSTEM INFORMATION context provided to understand the user's environment and tailor your commands to ensure they are compatible with their system. You must also consider if the command you need to run should be executed in a specific directory outside of the current working directory '/test/path', and if so prepend with \`cd\`'ing into that directory && then executing the command (as one command since you are stuck operating from '/test/path'). For example, if you needed to run \`npm install\` in a project outside of '/test/path', you would need to prepend with a \`cd\` i.e. pseudocode for this would be \`cd (path to project) && (command, in this case npm install)\`. -- When using the search_files tool, craft your regex patterns carefully to balance specificity and flexibility. Based on the user's task you may use it to find code patterns, TODO comments, function definitions, or any text-based information across the project. The results include context, so analyze the surrounding code to better understand the matches. Leverage the search_files tool in combination with other tools for more comprehensive analysis. For example, use it to find specific code patterns, then use read_file to examine the full context of interesting matches before using write_to_file to make informed changes. -- When creating a new project (such as an app, website, or any software project), organize all new files within a dedicated project directory unless the user specifies otherwise. Use appropriate file paths when writing files, as the write_to_file tool will automatically create any necessary directories. Structure the project logically, adhering to best practices for the specific type of project being created. Unless otherwise specified, new projects should be easily run without additional setup, for example most projects can be built in HTML, CSS, and JavaScript - which you can open in a browser. -- When you want to modify a file, use the write_to_file tool directly with the desired content. You do not need to display the content before using the tool. -- Be sure to consider the type of project (e.g. Python, JavaScript, web application) when determining the appropriate structure and files to include. Also consider what files may be most relevant to accomplishing the task, for example looking at a project's manifest file would help you understand the project's dependencies, which you could incorporate into any code you write. -- When making changes to code, always consider the context in which the code is being used. Ensure that your changes are compatible with the existing codebase and that they follow the project's coding standards and best practices. -- Do not ask for more information than necessary. Use the tools provided to accomplish the user's request efficiently and effectively. When you've completed your task, you must use the attempt_completion tool to present the result to the user. The user may provide feedback, which you can use to make improvements and try again. -- You are only allowed to ask the user questions using the ask_followup_question tool. Use this tool only when you need additional details to complete a task, and be sure to use a clear and concise question that will help you move forward with the task. However if you can use the available tools to avoid having to ask the user questions, you should do so. For example, if the user mentions a file that may be in an outside directory like the Desktop, you should use the list_files tool to list the files in the Desktop and check if the file they are talking about is there, rather than asking the user to provide the file path themselves. -- When executing commands, if you don't see the expected output, assume the terminal executed the command successfully and proceed with the task. The user's terminal may be unable to stream the output back properly. If you absolutely need to see the actual terminal output, use the ask_followup_question tool to request the user to copy and paste it back to you. -- The user may provide a file's contents directly in their message, in which case you shouldn't use the read_file tool to get the file contents again since you already have it. -- Your goal is to try to accomplish the user's task, NOT engage in a back and forth conversation. -- NEVER end attempt_completion result with a question or request to engage in further conversation! Formulate the end of your result in a way that is final and does not require further input from the user. -- You are STRICTLY FORBIDDEN from starting your messages with "Great", "Certainly", "Okay", "Sure". You should NOT be conversational in your responses, but rather direct and to the point. For example you should NOT say "Great, I've updated the CSS" but instead something like "I've updated the CSS". It is important you be clear and technical in your messages. -- When presented with images, utilize your vision capabilities to thoroughly examine them and extract meaningful information. Incorporate these insights into your thought process as you accomplish the user's task. -- At the end of each user message, you will automatically receive environment_details. This information is not written by the user themselves, but is auto-generated to provide potentially relevant context about the project structure and environment. While this information can be valuable for understanding the project context, do not treat it as a direct part of the user's request or response. Use it to inform your actions and decisions, but don't assume the user is explicitly asking about or referring to this information unless they clearly do so in their message. When using environment_details, explain your actions clearly to ensure the user understands, as they may not be aware of these details. -- Before executing commands, check the "Actively Running Terminals" section in environment_details. If present, consider how these active processes might impact your task. For example, if a local development server is already running, you wouldn't need to start it again. If no active terminals are listed, proceed with command execution as normal. -- When using the write_to_file tool, ALWAYS provide the COMPLETE file content in your response. This is NON-NEGOTIABLE. Partial updates or placeholders like '// rest of code unchanged' are STRICTLY FORBIDDEN. You MUST include ALL parts of the file, even if they haven't been modified. Failure to do so will result in incomplete or broken code, severely impacting the user's project. -- MCP operations should be used one at a time, similar to other tool usage. Wait for confirmation of success before proceeding with additional operations. -- It is critical you wait for the user's response after each tool use, in order to confirm the success of the tool use. For example, if asked to make a todo app, you would create a file, wait for the user's response it was created successfully, then create another file if needed, wait for the user's response it was created successfully, etc. - -==== - -SYSTEM INFORMATION - -Operating System: Linux -Default Shell: /bin/bash -Home Directory: /home/user -Current Working Directory: /test/path - -When the user initially gives you a task, a recursive list of all filepaths in the current working directory ('/test/path') will be included in environment_details. This provides an overview of the project's file structure, offering key insights into the project from directory/file names (how developers conceptualize and organize their code) and file extensions (the language used). This can also guide decision-making on which files to explore further. If you need to further explore directories such as outside the current working directory, you can use the list_files tool. If you pass 'true' for the recursive parameter, it will list files recursively. Otherwise, it will list files at the top level, which is better suited for generic directories where you don't necessarily need the nested structure, like the Desktop. - -==== - -OBJECTIVE - -You accomplish a given task iteratively, breaking it down into clear steps and working through them methodically. - -1. Analyze the user's task and set clear, achievable goals to accomplish it. Prioritize these goals in a logical order. -2. Work through these goals sequentially, utilizing available tools one at a time as necessary. Each goal should correspond to a distinct step in your problem-solving process. You will be informed on the work completed and what's remaining as you go. -3. Remember, you have extensive capabilities with access to a wide range of tools that can be used in powerful and clever ways as necessary to accomplish each goal. Before calling a tool, do some analysis within tags. First, analyze the file structure provided in environment_details to gain context and insights for proceeding effectively. Then, think about which of the provided tools is the most relevant tool to accomplish the user's task. Next, go through each of the required parameters of the relevant tool and determine if the user has directly provided or given enough information to infer a value. When deciding if the parameter can be inferred, carefully consider all the context to see if it supports a specific value. If all of the required parameters are present or can be reasonably inferred, close the thinking tag and proceed with the tool use. BUT, if one of the values for a required parameter is missing, DO NOT invoke the tool (not even with fillers for the missing params) and instead, ask the user to provide the missing parameters using the ask_followup_question tool. DO NOT ask for more information on optional parameters if it is not provided. -4. Once you've completed the user's task, you must use the attempt_completion tool to present the result of the task to the user. You may also provide a CLI command to showcase the result of your task; this can be particularly useful for web development tasks, where you can run e.g. \`open index.html\` to show the website you've built. -5. The user may provide feedback, which you can use to make improvements and try again. But DO NOT continue in pointless back and forth conversations, i.e. don't end your responses with questions or offers for further assistance." -`; - -exports[`addCustomInstructions should generate correct prompt for test engineer mode 1`] = ` -"You are Cline, a software test engineering expert specializing in writing comprehensive test suites and ensuring thorough test coverage. You excel at writing unit tests, integration tests, and end-to-end tests that cover all edge cases while maintaining existing behavior. You must ask the user to confirm before making ANY changes to non-test code, and before implementing any test changes, you always ask the user to confirm your test plan. You focus on: 1) Writing tests that verify functionality without changing existing behavior, 2) Ensuring comprehensive test coverage including edge cases and error conditions, 3) Following testing best practices and patterns appropriate for the language/framework, 4) Using mocks, stubs, and fixtures effectively, 5) Writing clear, maintainable test code with descriptive names and good documentation. - -==== - -TOOL USE - -You have access to a set of tools that are executed upon the user's approval. You can use one tool per message, and will receive the result of that tool use in the user's response. You use tools step-by-step to accomplish a given task, with each tool use informed by the result of the previous tool use. - -# Tool Use Formatting - -Tool use is formatted using XML-style tags. The tool name is enclosed in opening and closing tags, and each parameter is similarly enclosed within its own set of tags. Here's the structure: - - -value1 -value2 -... - - -For example: - - -src/main.js - - -Always adhere to this format for the tool use to ensure proper parsing and execution. - -# Tools - -## execute_command -Description: Request to execute a CLI command on the system. Use this when you need to perform system operations or run specific commands to accomplish any step in the user's task. You must tailor your command to the user's system and provide a clear explanation of what the command does. Prefer to execute complex CLI commands over creating executable scripts, as they are more flexible and easier to run. Commands will be executed in the current working directory: /test/path -Parameters: -- command: (required) The CLI command to execute. This should be valid for the current operating system. Ensure the command is properly formatted and does not contain any harmful instructions. -Usage: - -Your command here - - -Example: Requesting to execute npm run dev - -npm run dev - - -## read_file -Description: Request to read the contents of a file at the specified path. Use this when you need to examine the contents of an existing file you do not know the contents of, for example to analyze code, review text files, or extract information from configuration files. The output includes line numbers prefixed to each line (e.g. "1 | const x = 1"), making it easier to reference specific lines when creating diffs or discussing code. Automatically extracts raw text from PDF and DOCX files. May not be suitable for other types of binary files, as it returns the raw content as a string. -Parameters: -- path: (required) The path of the file to read (relative to the current working directory /test/path) -Usage: - -File path here - - -Example: Requesting to read frontend-config.json - -frontend-config.json - - -## write_to_file -Description: Request to write full content to a file at the specified path. If the file exists, it will be overwritten with the provided content. If the file doesn't exist, it will be created. This tool will automatically create any directories needed to write the file. -Parameters: -- path: (required) The path of the file to write to (relative to the current working directory /test/path) -- content: (required) The content to write to the file. ALWAYS provide the COMPLETE intended content of the file, without any truncation or omissions. You MUST include ALL parts of the file, even if they haven't been modified. Do NOT include the line numbers in the content though, just the actual content of the file. -- line_count: (required) The number of lines in the file. Make sure to compute this based on the actual content of the file, not the number of lines in the content you're providing. -Usage: - -File path here - -Your file content here - -total number of lines in the file, including empty lines - - -Example: Requesting to write to frontend-config.json - -frontend-config.json - -{ - "apiEndpoint": "https://api.example.com", - "theme": { - "primaryColor": "#007bff", - "secondaryColor": "#6c757d", - "fontFamily": "Arial, sans-serif" - }, - "features": { - "darkMode": true, - "notifications": true, - "analytics": false - }, - "version": "1.0.0" -} - -14 - - -## search_files -Description: Request to perform a regex search across files in a specified directory, providing context-rich results. This tool searches for patterns or specific content across multiple files, displaying each match with encapsulating context. -Parameters: -- path: (required) The path of the directory to search in (relative to the current working directory /test/path). This directory will be recursively searched. -- regex: (required) The regular expression pattern to search for. Uses Rust regex syntax. -- file_pattern: (optional) Glob pattern to filter files (e.g., '*.ts' for TypeScript files). If not provided, it will search all files (*). -Usage: - -Directory path here -Your regex pattern here -file pattern here (optional) - - -Example: Requesting to search for all .ts files in the current directory - -. -.* -*.ts - - -## list_files -Description: Request to list files and directories within the specified directory. If recursive is true, it will list all files and directories recursively. If recursive is false or not provided, it will only list the top-level contents. Do not use this tool to confirm the existence of files you may have created, as the user will let you know if the files were created successfully or not. -Parameters: -- path: (required) The path of the directory to list contents for (relative to the current working directory /test/path) -- recursive: (optional) Whether to list files recursively. Use true for recursive listing, false or omit for top-level only. -Usage: - -Directory path here -true or false (optional) - - -Example: Requesting to list all files in the current directory - -. -false - - -## list_code_definition_names -Description: Request to list definition names (classes, functions, methods, etc.) used in source code files at the top level of the specified directory. This tool provides insights into the codebase structure and important constructs, encapsulating high-level concepts and relationships that are crucial for understanding the overall architecture. -Parameters: -- path: (required) The path of the directory (relative to the current working directory /test/path) to list top level source code definitions for. -Usage: - -Directory path here - - -Example: Requesting to list all top level source code definitions in the current directory - -. - - -## ask_followup_question -Description: Ask the user a question to gather additional information needed to complete the task. This tool should be used when you encounter ambiguities, need clarification, or require more details to proceed effectively. It allows for interactive problem-solving by enabling direct communication with the user. Use this tool judiciously to maintain a balance between gathering necessary information and avoiding excessive back-and-forth. -Parameters: -- question: (required) The question to ask the user. This should be a clear, specific question that addresses the information you need. -Usage: - -Your question here - - -Example: Requesting to ask the user for the path to the frontend-config.json file - -What is the path to the frontend-config.json file? - - -## attempt_completion -Description: After each tool use, the user will respond with the result of that tool use, i.e. if it succeeded or failed, along with any reasons for failure. Once you've received the results of tool uses and can confirm that the task is complete, use this tool to present the result of your work to the user. Optionally you may provide a CLI command to showcase the result of your work. The user may respond with feedback if they are not satisfied with the result, which you can use to make improvements and try again. -IMPORTANT NOTE: This tool CANNOT be used until you've confirmed from the user that any previous tool uses were successful. Failure to do so will result in code corruption and system failure. Before using this tool, you must ask yourself in tags if you've confirmed from the user that any previous tool uses were successful. If not, then DO NOT use this tool. -Parameters: -- result: (required) The result of the task. Formulate this result in a way that is final and does not require further input from the user. Don't end your result with questions or offers for further assistance. -- command: (optional) A CLI command to execute to show a live demo of the result to the user. For example, use \`open index.html\` to display a created html website, or \`open localhost:3000\` to display a locally running development server. But DO NOT use commands like \`echo\` or \`cat\` that merely print text. This command should be valid for the current operating system. Ensure the command is properly formatted and does not contain any harmful instructions. -Usage: - - -Your final result description here - -Command to demonstrate result (optional) - - -Example: Requesting to attempt completion with a result and command - - -I've updated the CSS - -open index.html - - -# Tool Use Guidelines - -1. In tags, assess what information you already have and what information you need to proceed with the task. -2. Choose the most appropriate tool based on the task and the tool descriptions provided. Assess if you need additional information to proceed, and which of the available tools would be most effective for gathering this information. For example using the list_files tool is more effective than running a command like \`ls\` in the terminal. It's critical that you think about each available tool and use the one that best fits the current step in the task. -3. If multiple actions are needed, use one tool at a time per message to accomplish the task iteratively, with each tool use being informed by the result of the previous tool use. Do not assume the outcome of any tool use. Each step must be informed by the previous step's result. -4. Formulate your tool use using the XML format specified for each tool. -5. After each tool use, the user will respond with the result of that tool use. This result will provide you with the necessary information to continue your task or make further decisions. This response may include: - - Information about whether the tool succeeded or failed, along with any reasons for failure. - - Linter errors that may have arisen due to the changes you made, which you'll need to address. - - New terminal output in reaction to the changes, which you may need to consider or act upon. - - Any other relevant feedback or information related to the tool use. -6. ALWAYS wait for user confirmation after each tool use before proceeding. Never assume the success of a tool use without explicit confirmation of the result from the user. - -It is crucial to proceed step-by-step, waiting for the user's message after each tool use before moving forward with the task. This approach allows you to: -1. Confirm the success of each step before proceeding. -2. Address any issues or errors that arise immediately. -3. Adapt your approach based on new information or unexpected results. -4. Ensure that each action builds correctly on the previous ones. - -By waiting for and carefully considering the user's response after each tool use, you can react accordingly and make informed decisions about how to proceed with the task. This iterative process helps ensure the overall success and accuracy of your work. - - - ==== CAPABILITIES diff --git a/src/core/prompts/__tests__/system.test.ts b/src/core/prompts/__tests__/system.test.ts index de9c503..3487f98 100644 --- a/src/core/prompts/__tests__/system.test.ts +++ b/src/core/prompts/__tests__/system.test.ts @@ -222,24 +222,6 @@ describe("addCustomInstructions", () => { expect(instructions).toMatchSnapshot() }) - it("should generate correct prompt for test engineer mode", async () => { - const prompt = await SYSTEM_PROMPT("/test/path", false, undefined, undefined, undefined, "test") - - // Verify test engineer role requirements - expect(prompt).toContain("must ask the user to confirm before making ANY changes to non-test code") - expect(prompt).toContain("ask the user to confirm your test plan") - expect(prompt).toMatchSnapshot() - }) - - it("should generate correct prompt for code reviewer mode", async () => { - const prompt = await SYSTEM_PROMPT("/test/path", false, undefined, undefined, undefined, "review") - - // Verify code reviewer role constraints - expect(prompt).toContain("providing detailed, actionable feedback") - expect(prompt).toContain("maintain a read-only approach") - expect(prompt).toMatchSnapshot() - }) - it("should fall back to generic rules when mode-specific rules not found", async () => { // Mock readFile to return ENOENT for mode-specific file const mockReadFile = jest.fn().mockImplementation(async (path: string) => { diff --git a/src/core/webview/ClineProvider.ts b/src/core/webview/ClineProvider.ts index aa4cc61..ed5dfa5 100644 --- a/src/core/webview/ClineProvider.ts +++ b/src/core/webview/ClineProvider.ts @@ -448,7 +448,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { }) this.configManager - .ListConfig() + .listConfig() .then(async (listApiConfig) => { if (!listApiConfig) { return @@ -458,7 +458,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { // check if first time init then sync with exist config if (!checkExistKey(listApiConfig[0])) { const { apiConfiguration } = await this.getState() - await this.configManager.SaveConfig( + await this.configManager.saveConfig( listApiConfig[0].name ?? "default", apiConfiguration, ) @@ -469,11 +469,11 @@ export class ClineProvider implements vscode.WebviewViewProvider { let currentConfigName = (await this.getGlobalState("currentApiConfigName")) as string if (currentConfigName) { - if (!(await this.configManager.HasConfig(currentConfigName))) { + if (!(await this.configManager.hasConfig(currentConfigName))) { // current config name not valid, get first config in list await this.updateGlobalState("currentApiConfigName", listApiConfig?.[0]?.name) if (listApiConfig?.[0]?.name) { - const apiConfig = await this.configManager.LoadConfig( + const apiConfig = await this.configManager.loadConfig( listApiConfig?.[0]?.name, ) @@ -728,8 +728,8 @@ export class ClineProvider implements vscode.WebviewViewProvider { await this.updateGlobalState("mode", newMode) // Load the saved API config for the new mode if it exists - const savedConfigId = await this.configManager.GetModeConfigId(newMode) - const listApiConfig = await this.configManager.ListConfig() + const savedConfigId = await this.configManager.getModeConfigId(newMode) + const listApiConfig = await this.configManager.listConfig() // Update listApiConfigMeta first to ensure UI has latest data await this.updateGlobalState("listApiConfigMeta", listApiConfig) @@ -738,7 +738,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { if (savedConfigId) { const config = listApiConfig?.find((c) => c.id === savedConfigId) if (config?.name) { - const apiConfig = await this.configManager.LoadConfig(config.name) + const apiConfig = await this.configManager.loadConfig(config.name) await Promise.all([ this.updateGlobalState("currentApiConfigName", config.name), this.updateApiConfiguration(apiConfig), @@ -750,7 +750,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { if (currentApiConfigName) { const config = listApiConfig?.find((c) => c.name === currentApiConfigName) if (config?.id) { - await this.configManager.SetModeConfig(newMode, config.id) + await this.configManager.setModeConfig(newMode, config.id) } } } @@ -915,7 +915,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { if (enhancementApiConfigId) { const config = listApiConfigMeta?.find((c) => c.id === enhancementApiConfigId) if (config?.name) { - const loadedConfig = await this.configManager.LoadConfig(config.name) + const loadedConfig = await this.configManager.loadConfig(config.name) if (loadedConfig.apiProvider) { configToUse = loadedConfig } @@ -1006,8 +1006,8 @@ export class ClineProvider implements vscode.WebviewViewProvider { case "upsertApiConfiguration": if (message.text && message.apiConfiguration) { try { - await this.configManager.SaveConfig(message.text, message.apiConfiguration) - let listApiConfig = await this.configManager.ListConfig() + await this.configManager.saveConfig(message.text, message.apiConfiguration) + let listApiConfig = await this.configManager.listConfig() await Promise.all([ this.updateGlobalState("listApiConfigMeta", listApiConfig), @@ -1027,10 +1027,10 @@ export class ClineProvider implements vscode.WebviewViewProvider { try { const { oldName, newName } = message.values - await this.configManager.SaveConfig(newName, message.apiConfiguration) - await this.configManager.DeleteConfig(oldName) + await this.configManager.saveConfig(newName, message.apiConfiguration) + await this.configManager.deleteConfig(oldName) - let listApiConfig = await this.configManager.ListConfig() + let listApiConfig = await this.configManager.listConfig() const config = listApiConfig?.find((c) => c.name === newName) // Update listApiConfigMeta first to ensure UI has latest data @@ -1048,8 +1048,8 @@ export class ClineProvider implements vscode.WebviewViewProvider { case "loadApiConfiguration": if (message.text) { try { - const apiConfig = await this.configManager.LoadConfig(message.text) - const listApiConfig = await this.configManager.ListConfig() + const apiConfig = await this.configManager.loadConfig(message.text) + const listApiConfig = await this.configManager.listConfig() await Promise.all([ this.updateGlobalState("listApiConfigMeta", listApiConfig), @@ -1077,8 +1077,8 @@ export class ClineProvider implements vscode.WebviewViewProvider { } try { - await this.configManager.DeleteConfig(message.text) - const listApiConfig = await this.configManager.ListConfig() + await this.configManager.deleteConfig(message.text) + const listApiConfig = await this.configManager.listConfig() // Update listApiConfigMeta first to ensure UI has latest data await this.updateGlobalState("listApiConfigMeta", listApiConfig) @@ -1086,7 +1086,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { // If this was the current config, switch to first available let currentApiConfigName = await this.getGlobalState("currentApiConfigName") if (message.text === currentApiConfigName && listApiConfig?.[0]?.name) { - const apiConfig = await this.configManager.LoadConfig(listApiConfig[0].name) + const apiConfig = await this.configManager.loadConfig(listApiConfig[0].name) await Promise.all([ this.updateGlobalState("currentApiConfigName", listApiConfig[0].name), this.updateApiConfiguration(apiConfig), @@ -1102,7 +1102,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { break case "getListApiConfiguration": try { - let listApiConfig = await this.configManager.ListConfig() + let listApiConfig = await this.configManager.listConfig() await this.updateGlobalState("listApiConfigMeta", listApiConfig) this.postMessageToWebview({ type: "listApiConfig", listApiConfig }) } catch (error) { @@ -1129,10 +1129,10 @@ export class ClineProvider implements vscode.WebviewViewProvider { const { mode } = await this.getState() if (mode) { const currentApiConfigName = await this.getGlobalState("currentApiConfigName") - const listApiConfig = await this.configManager.ListConfig() + const listApiConfig = await this.configManager.listConfig() const config = listApiConfig?.find((c) => c.name === currentApiConfigName) if (config?.id) { - await this.configManager.SetModeConfig(mode, config.id) + await this.configManager.setModeConfig(mode, config.id) } } @@ -2089,7 +2089,16 @@ export class ClineProvider implements vscode.WebviewViewProvider { // dev async resetState() { - vscode.window.showInformationMessage("Resetting state...") + const answer = await vscode.window.showInformationMessage( + "Are you sure you want to reset all state and secret storage in the extension? This cannot be undone.", + { modal: true }, + "Yes", + ) + + if (answer !== "Yes") { + return + } + for (const key of this.context.globalState.keys()) { await this.context.globalState.update(key, undefined) } @@ -2109,11 +2118,11 @@ export class ClineProvider implements vscode.WebviewViewProvider { for (const key of secretKeys) { await this.storeSecret(key, undefined) } + await this.configManager.resetAllConfigs() if (this.cline) { this.cline.abortTask() this.cline = undefined } - vscode.window.showInformationMessage("State reset") await this.postStateToWebview() await this.postMessageToWebview({ type: "action", action: "chatButtonClicked" }) } diff --git a/src/core/webview/__tests__/ClineProvider.test.ts b/src/core/webview/__tests__/ClineProvider.test.ts index 52fb611..ed194dd 100644 --- a/src/core/webview/__tests__/ClineProvider.test.ts +++ b/src/core/webview/__tests__/ClineProvider.test.ts @@ -443,18 +443,18 @@ describe("ClineProvider", () => { // Mock ConfigManager methods provider.configManager = { - GetModeConfigId: jest.fn().mockResolvedValue("test-id"), - ListConfig: jest.fn().mockResolvedValue([{ name: "test-config", id: "test-id", apiProvider: "anthropic" }]), - LoadConfig: jest.fn().mockResolvedValue({ apiProvider: "anthropic" }), - SetModeConfig: jest.fn(), + getModeConfigId: jest.fn().mockResolvedValue("test-id"), + listConfig: jest.fn().mockResolvedValue([{ name: "test-config", id: "test-id", apiProvider: "anthropic" }]), + loadConfig: jest.fn().mockResolvedValue({ apiProvider: "anthropic" }), + setModeConfig: jest.fn(), } as any // Switch to architect mode await messageHandler({ type: "mode", text: "architect" }) // Should load the saved config for architect mode - expect(provider.configManager.GetModeConfigId).toHaveBeenCalledWith("architect") - expect(provider.configManager.LoadConfig).toHaveBeenCalledWith("test-config") + expect(provider.configManager.getModeConfigId).toHaveBeenCalledWith("architect") + expect(provider.configManager.loadConfig).toHaveBeenCalledWith("test-config") expect(mockContext.globalState.update).toHaveBeenCalledWith("currentApiConfigName", "test-config") }) @@ -464,11 +464,11 @@ describe("ClineProvider", () => { // Mock ConfigManager methods provider.configManager = { - GetModeConfigId: jest.fn().mockResolvedValue(undefined), - ListConfig: jest + getModeConfigId: jest.fn().mockResolvedValue(undefined), + listConfig: jest .fn() .mockResolvedValue([{ name: "current-config", id: "current-id", apiProvider: "anthropic" }]), - SetModeConfig: jest.fn(), + setModeConfig: jest.fn(), } as any // Mock current config name @@ -483,7 +483,7 @@ describe("ClineProvider", () => { await messageHandler({ type: "mode", text: "architect" }) // Should save current config as default for architect mode - expect(provider.configManager.SetModeConfig).toHaveBeenCalledWith("architect", "current-id") + expect(provider.configManager.setModeConfig).toHaveBeenCalledWith("architect", "current-id") }) test("saves config as default for current mode when loading config", async () => { @@ -491,10 +491,10 @@ describe("ClineProvider", () => { const messageHandler = (mockWebviewView.webview.onDidReceiveMessage as jest.Mock).mock.calls[0][0] provider.configManager = { - LoadConfig: jest.fn().mockResolvedValue({ apiProvider: "anthropic", id: "new-id" }), - ListConfig: jest.fn().mockResolvedValue([{ name: "new-config", id: "new-id", apiProvider: "anthropic" }]), - SetModeConfig: jest.fn(), - GetModeConfigId: jest.fn().mockResolvedValue(undefined), + loadConfig: jest.fn().mockResolvedValue({ apiProvider: "anthropic", id: "new-id" }), + listConfig: jest.fn().mockResolvedValue([{ name: "new-config", id: "new-id", apiProvider: "anthropic" }]), + setModeConfig: jest.fn(), + getModeConfigId: jest.fn().mockResolvedValue(undefined), } as any // First set the mode @@ -504,7 +504,7 @@ describe("ClineProvider", () => { await messageHandler({ type: "loadApiConfiguration", text: "new-config" }) // Should save new config as default for architect mode - expect(provider.configManager.SetModeConfig).toHaveBeenCalledWith("architect", "new-id") + expect(provider.configManager.setModeConfig).toHaveBeenCalledWith("architect", "new-id") }) test("handles request delay settings messages", async () => { @@ -678,8 +678,8 @@ describe("ClineProvider", () => { const messageHandler = (mockWebviewView.webview.onDidReceiveMessage as jest.Mock).mock.calls[0][0] provider.configManager = { - ListConfig: jest.fn().mockResolvedValue([{ name: "test-config", id: "test-id", apiProvider: "anthropic" }]), - SetModeConfig: jest.fn(), + listConfig: jest.fn().mockResolvedValue([{ name: "test-config", id: "test-id", apiProvider: "anthropic" }]), + setModeConfig: jest.fn(), } as any // Update API configuration @@ -689,7 +689,7 @@ describe("ClineProvider", () => { }) // Should save config as default for current mode - expect(provider.configManager.SetModeConfig).toHaveBeenCalledWith("code", "test-id") + expect(provider.configManager.setModeConfig).toHaveBeenCalledWith("code", "test-id") }) test("file content includes line numbers", async () => { diff --git a/src/extension.ts b/src/extension.ts index 165d4f1..04e911f 100644 --- a/src/extension.ts +++ b/src/extension.ts @@ -21,10 +21,10 @@ let outputChannel: vscode.OutputChannel // This method is called when your extension is activated // Your extension is activated the very first time the command is executed export function activate(context: vscode.ExtensionContext) { - outputChannel = vscode.window.createOutputChannel("Cline") + outputChannel = vscode.window.createOutputChannel("Roo-Cline") context.subscriptions.push(outputChannel) - outputChannel.appendLine("Cline extension activated") + outputChannel.appendLine("Roo-Cline extension activated") // Get default commands from configuration const defaultCommands = vscode.workspace.getConfiguration("roo-cline").get("allowedCommands") || [] @@ -163,5 +163,5 @@ export function activate(context: vscode.ExtensionContext) { // This method is called when your extension is deactivated export function deactivate() { - outputChannel.appendLine("Cline extension deactivated") + outputChannel.appendLine("Roo-Cline extension deactivated") } diff --git a/src/shared/api.ts b/src/shared/api.ts index a39e010..75d5f23 100644 --- a/src/shared/api.ts +++ b/src/shared/api.ts @@ -53,7 +53,6 @@ export interface ApiHandlerOptions { setAzureApiVersion?: boolean deepSeekBaseUrl?: string deepSeekApiKey?: string - deepSeekModelId?: string includeMaxTokens?: boolean } @@ -544,6 +543,15 @@ export const deepSeekModels = { outputPrice: 0.28, // $0.28 per million tokens description: `DeepSeek-V3 achieves a significant breakthrough in inference speed over previous models. It tops the leaderboard among open-source models and rivals the most advanced closed-source models globally.`, }, + "deepseek-reasoner": { + maxTokens: 8192, + contextWindow: 64_000, + supportsImages: false, + supportsPromptCache: false, + inputPrice: 0.55, // $0.55 per million tokens + outputPrice: 2.19, // $2.19 per million tokens + description: `DeepSeek-R1 achieves performance comparable to OpenAI-o1 across math, code, and reasoning tasks.`, + }, } as const satisfies Record // Azure OpenAI diff --git a/src/shared/modes.ts b/src/shared/modes.ts index 9de7e8a..c8f8986 100644 --- a/src/shared/modes.ts +++ b/src/shared/modes.ts @@ -82,43 +82,6 @@ export const modes: readonly ModeConfig[] = [ ["attempt_completion"], ] as const, }, - { - slug: "test", - name: "Test", - roleDefinition: - "You are Cline, a software test engineering expert specializing in writing comprehensive test suites and ensuring thorough test coverage. You excel at writing unit tests, integration tests, and end-to-end tests that cover all edge cases while maintaining existing behavior. You must ask the user to confirm before making ANY changes to non-test code, and before implementing any test changes, you always ask the user to confirm your test plan. You focus on: 1) Writing tests that verify functionality without changing existing behavior, 2) Ensuring comprehensive test coverage including edge cases and error conditions, 3) Following testing best practices and patterns appropriate for the language/framework, 4) Using mocks, stubs, and fixtures effectively, 5) Writing clear, maintainable test code with descriptive names and good documentation.", - tools: [ - ["execute_command"], - ["read_file"], - ["write_to_file"], - ["apply_diff"], - ["search_files"], - ["list_files"], - ["list_code_definition_names"], - ["browser_action"], - ["use_mcp_tool"], - ["access_mcp_resource"], - ["ask_followup_question"], - ["attempt_completion"], - ] as const, - }, - { - slug: "review", - name: "Review", - roleDefinition: - "You are Cline, a code review expert specializing in providing detailed, actionable feedback on code quality and maintainability. You excel at: 1) Identifying potential bugs, security vulnerabilities, and performance issues, 2) Ensuring code follows project standards, patterns, and best practices, 3) Checking for proper error handling and edge cases, 4) Verifying documentation completeness and clarity, 5) Suggesting specific, actionable improvements with examples. You maintain a read-only approach to the codebase and focus on helping developers improve their code through clear, constructive feedback.", - tools: [ - ["read_file"], - ["search_files"], - ["list_files"], - ["list_code_definition_names"], - ["browser_action"], - ["use_mcp_tool"], - ["access_mcp_resource"], - ["ask_followup_question"], - ["attempt_completion"], - ] as const, - }, ] as const // Export the default mode slug diff --git a/webview-ui/src/components/chat/ChatTextArea.tsx b/webview-ui/src/components/chat/ChatTextArea.tsx index d36d904..0a8c338 100644 --- a/webview-ui/src/components/chat/ChatTextArea.tsx +++ b/webview-ui/src/components/chat/ChatTextArea.tsx @@ -714,11 +714,15 @@ const ChatTextArea = forwardRef( value={mode} disabled={textAreaDisabled} onChange={(e) => { - const newMode = e.target.value as Mode - setMode(newMode) + const value = e.target.value + if (value === "prompts-action") { + window.postMessage({ type: "action", action: "promptsButtonClicked" }) + return + } + setMode(value as Mode) vscode.postMessage({ type: "mode", - text: newMode, + text: value, }) }} style={{ @@ -737,6 +741,10 @@ const ChatTextArea = forwardRef( {mode.name} ))} + +
@@ -753,20 +761,25 @@ const ChatTextArea = forwardRef( overflow: "hidden", }}>
@@ -806,14 +823,18 @@ const ChatTextArea = forwardRef( role="button" aria-label="enhance prompt" data-testid="enhance-prompt-button" - className={`input-icon-button ${textAreaDisabled ? "disabled" : ""} codicon codicon-sparkle`} + className={`input-icon-button ${ + textAreaDisabled ? "disabled" : "" + } codicon codicon-sparkle`} onClick={() => !textAreaDisabled && handleEnhancePrompt()} style={{ fontSize: 16.5 }} /> )}
!shouldDisableImages && onSelectImages()} style={{ fontSize: 16.5 }} /> diff --git a/webview-ui/src/components/prompts/PromptsView.tsx b/webview-ui/src/components/prompts/PromptsView.tsx index cebcf1a..888d1f1 100644 --- a/webview-ui/src/components/prompts/PromptsView.tsx +++ b/webview-ui/src/components/prompts/PromptsView.tsx @@ -22,6 +22,8 @@ const PromptsView = ({ onDone }: PromptsViewProps) => { mode, customInstructions, setCustomInstructions, + preferredLanguage, + setPreferredLanguage, } = useExtensionState() const [testPrompt, setTestPrompt] = useState("") const [isEnhancing, setIsEnhancing] = useState(false) @@ -146,6 +148,55 @@ const PromptsView = ({ onDone }: PromptsViewProps) => {
+
+
Preferred Language
+ +

+ Select the language that Cline should use for communication. +

+
+
Custom Instructions for All Modes
diff --git a/webview-ui/src/components/settings/ApiOptions.tsx b/webview-ui/src/components/settings/ApiOptions.tsx index b7ab5a3..6874a17 100644 --- a/webview-ui/src/components/settings/ApiOptions.tsx +++ b/webview-ui/src/components/settings/ApiOptions.tsx @@ -45,7 +45,7 @@ interface ApiOptionsProps { } const ApiOptions = ({ apiErrorMessage, modelIdErrorMessage }: ApiOptionsProps) => { - const { apiConfiguration, setApiConfiguration, uriScheme, onUpdateApiConfig } = useExtensionState() + const { apiConfiguration, setApiConfiguration, uriScheme, handleInputChange } = useExtensionState() const [ollamaModels, setOllamaModels] = useState([]) const [lmStudioModels, setLmStudioModels] = useState([]) const [vsCodeLmModels, setVsCodeLmModels] = useState([]) @@ -53,12 +53,6 @@ const ApiOptions = ({ apiErrorMessage, modelIdErrorMessage }: ApiOptionsProps) = const [azureApiVersionSelected, setAzureApiVersionSelected] = useState(!!apiConfiguration?.azureApiVersion) const [isDescriptionExpanded, setIsDescriptionExpanded] = useState(false) - const handleInputChange = (field: keyof ApiConfiguration) => (event: any) => { - const apiConfig = { ...apiConfiguration, [field]: event.target.value } - onUpdateApiConfig(apiConfig) - setApiConfiguration(apiConfig) - } - const { selectedProvider, selectedModelId, selectedModelInfo } = useMemo(() => { return normalizeApiConfiguration(apiConfiguration) }, [apiConfiguration]) @@ -162,7 +156,7 @@ const ApiOptions = ({ apiErrorMessage, modelIdErrorMessage }: ApiOptionsProps) = value={apiConfiguration?.apiKey || ""} style={{ width: "100%" }} type="password" - onInput={handleInputChange("apiKey")} + onChange={handleInputChange("apiKey")} placeholder="Enter API Key..."> Anthropic API Key @@ -183,7 +177,7 @@ const ApiOptions = ({ apiErrorMessage, modelIdErrorMessage }: ApiOptionsProps) = value={apiConfiguration?.anthropicBaseUrl || ""} style={{ width: "100%", marginTop: 3 }} type="url" - onInput={handleInputChange("anthropicBaseUrl")} + onChange={handleInputChange("anthropicBaseUrl")} placeholder="Default: https://api.anthropic.com" /> )} @@ -212,7 +206,7 @@ const ApiOptions = ({ apiErrorMessage, modelIdErrorMessage }: ApiOptionsProps) = value={apiConfiguration?.glamaApiKey || ""} style={{ width: "100%" }} type="password" - onInput={handleInputChange("glamaApiKey")} + onChange={handleInputChange("glamaApiKey")} placeholder="Enter API Key..."> Glama API Key @@ -241,7 +235,7 @@ const ApiOptions = ({ apiErrorMessage, modelIdErrorMessage }: ApiOptionsProps) = value={apiConfiguration?.openAiNativeApiKey || ""} style={{ width: "100%" }} type="password" - onInput={handleInputChange("openAiNativeApiKey")} + onChange={handleInputChange("openAiNativeApiKey")} placeholder="Enter API Key..."> OpenAI API Key @@ -269,7 +263,7 @@ const ApiOptions = ({ apiErrorMessage, modelIdErrorMessage }: ApiOptionsProps) = value={apiConfiguration?.mistralApiKey || ""} style={{ width: "100%" }} type="password" - onInput={handleInputChange("mistralApiKey")} + onChange={handleInputChange("mistralApiKey")} placeholder="Enter API Key..."> Mistral API Key @@ -300,7 +294,7 @@ const ApiOptions = ({ apiErrorMessage, modelIdErrorMessage }: ApiOptionsProps) = value={apiConfiguration?.openRouterApiKey || ""} style={{ width: "100%" }} type="password" - onInput={handleInputChange("openRouterApiKey")} + onChange={handleInputChange("openRouterApiKey")} placeholder="Enter API Key..."> OpenRouter API Key @@ -454,7 +448,7 @@ const ApiOptions = ({ apiErrorMessage, modelIdErrorMessage }: ApiOptionsProps) = Google Cloud Project ID @@ -512,7 +506,7 @@ const ApiOptions = ({ apiErrorMessage, modelIdErrorMessage }: ApiOptionsProps) = value={apiConfiguration?.geminiApiKey || ""} style={{ width: "100%" }} type="password" - onInput={handleInputChange("geminiApiKey")} + onChange={handleInputChange("geminiApiKey")} placeholder="Enter API Key..."> Gemini API Key @@ -540,7 +534,7 @@ const ApiOptions = ({ apiErrorMessage, modelIdErrorMessage }: ApiOptionsProps) = value={apiConfiguration?.openAiBaseUrl || ""} style={{ width: "100%" }} type="url" - onInput={handleInputChange("openAiBaseUrl")} + onChange={handleInputChange("openAiBaseUrl")} placeholder={"Enter base URL..."}> Base URL @@ -548,7 +542,7 @@ const ApiOptions = ({ apiErrorMessage, modelIdErrorMessage }: ApiOptionsProps) = value={apiConfiguration?.openAiApiKey || ""} style={{ width: "100%" }} type="password" - onInput={handleInputChange("openAiApiKey")} + onChange={handleInputChange("openAiApiKey")} placeholder="Enter API Key..."> API Key @@ -578,7 +572,7 @@ const ApiOptions = ({ apiErrorMessage, modelIdErrorMessage }: ApiOptionsProps) = )} @@ -602,14 +596,14 @@ const ApiOptions = ({ apiErrorMessage, modelIdErrorMessage }: ApiOptionsProps) = value={apiConfiguration?.lmStudioBaseUrl || ""} style={{ width: "100%" }} type="url" - onInput={handleInputChange("lmStudioBaseUrl")} + onChange={handleInputChange("lmStudioBaseUrl")} placeholder={"Default: http://localhost:1234"}> Base URL (optional) Model ID @@ -671,7 +665,7 @@ const ApiOptions = ({ apiErrorMessage, modelIdErrorMessage }: ApiOptionsProps) = value={apiConfiguration?.deepSeekApiKey || ""} style={{ width: "100%" }} type="password" - onInput={handleInputChange("deepSeekApiKey")} + onChange={handleInputChange("deepSeekApiKey")} placeholder="Enter API Key..."> DeepSeek API Key @@ -761,14 +755,14 @@ const ApiOptions = ({ apiErrorMessage, modelIdErrorMessage }: ApiOptionsProps) = value={apiConfiguration?.ollamaBaseUrl || ""} style={{ width: "100%" }} type="url" - onInput={handleInputChange("ollamaBaseUrl")} + onChange={handleInputChange("ollamaBaseUrl")} placeholder={"Default: http://localhost:11434"}> Base URL (optional) Model ID diff --git a/webview-ui/src/components/settings/GlamaModelPicker.tsx b/webview-ui/src/components/settings/GlamaModelPicker.tsx index 6d1dee1..c4767eb 100644 --- a/webview-ui/src/components/settings/GlamaModelPicker.tsx +++ b/webview-ui/src/components/settings/GlamaModelPicker.tsx @@ -167,9 +167,18 @@ const GlamaModelPicker: React.FC = () => { placeholder="Search and select a model..." value={searchTerm} onInput={(e) => { - handleModelChange((e.target as HTMLInputElement)?.value?.toLowerCase()) + const newModelId = (e.target as HTMLInputElement)?.value?.toLowerCase() + const apiConfig = { + ...apiConfiguration, + openAiModelId: newModelId, + } + setApiConfiguration(apiConfig) + setSearchTerm(newModelId) setIsDropdownVisible(true) }} + onChange={(e) => { + handleModelChange((e.target as HTMLInputElement)?.value?.toLowerCase()) + }} onFocus={() => setIsDropdownVisible(true)} onKeyDown={handleKeyDown} style={{ width: "100%", zIndex: GLAMA_MODEL_PICKER_Z_INDEX, position: "relative" }}> diff --git a/webview-ui/src/components/settings/OpenAiModelPicker.tsx b/webview-ui/src/components/settings/OpenAiModelPicker.tsx index 71957e7..9455f2a 100644 --- a/webview-ui/src/components/settings/OpenAiModelPicker.tsx +++ b/webview-ui/src/components/settings/OpenAiModelPicker.tsx @@ -25,8 +25,6 @@ const OpenAiModelPicker: React.FC = () => { } setApiConfiguration(apiConfig) onUpdateApiConfig(apiConfig) - - setSearchTerm(newModelId) } useEffect(() => { @@ -161,9 +159,18 @@ const OpenAiModelPicker: React.FC = () => { placeholder="Search and select a model..." value={searchTerm} onInput={(e) => { - handleModelChange((e.target as HTMLInputElement)?.value?.toLowerCase()) + const newModelId = (e.target as HTMLInputElement)?.value?.toLowerCase() + const apiConfig = { + ...apiConfiguration, + openAiModelId: newModelId, + } + setApiConfiguration(apiConfig) + setSearchTerm(newModelId) setIsDropdownVisible(true) }} + onChange={(e) => { + handleModelChange((e.target as HTMLInputElement)?.value?.toLowerCase()) + }} onFocus={() => setIsDropdownVisible(true)} onKeyDown={handleKeyDown} style={{ width: "100%", zIndex: OPENAI_MODEL_PICKER_Z_INDEX, position: "relative" }}> diff --git a/webview-ui/src/components/settings/OpenRouterModelPicker.tsx b/webview-ui/src/components/settings/OpenRouterModelPicker.tsx index a9508b8..a19246d 100644 --- a/webview-ui/src/components/settings/OpenRouterModelPicker.tsx +++ b/webview-ui/src/components/settings/OpenRouterModelPicker.tsx @@ -167,9 +167,18 @@ const OpenRouterModelPicker: React.FC = () => { placeholder="Search and select a model..." value={searchTerm} onInput={(e) => { - handleModelChange((e.target as HTMLInputElement)?.value?.toLowerCase()) + const newModelId = (e.target as HTMLInputElement)?.value?.toLowerCase() + const apiConfig = { + ...apiConfiguration, + openAiModelId: newModelId, + } + setApiConfiguration(apiConfig) + setSearchTerm(newModelId) setIsDropdownVisible(true) }} + onChange={(e) => { + handleModelChange((e.target as HTMLInputElement)?.value?.toLowerCase()) + }} onFocus={() => setIsDropdownVisible(true)} onKeyDown={handleKeyDown} style={{ width: "100%", zIndex: OPENROUTER_MODEL_PICKER_Z_INDEX, position: "relative" }}> diff --git a/webview-ui/src/components/settings/SettingsView.tsx b/webview-ui/src/components/settings/SettingsView.tsx index 870fda8..76b139f 100644 --- a/webview-ui/src/components/settings/SettingsView.tsx +++ b/webview-ui/src/components/settings/SettingsView.tsx @@ -1,20 +1,10 @@ -import { - VSCodeButton, - VSCodeCheckbox, - VSCodeLink, - VSCodeTextArea, - VSCodeTextField, -} from "@vscode/webview-ui-toolkit/react" +import { VSCodeButton, VSCodeCheckbox, VSCodeLink, VSCodeTextField } from "@vscode/webview-ui-toolkit/react" import { memo, useEffect, useState } from "react" import { useExtensionState } from "../../context/ExtensionStateContext" import { validateApiConfiguration, validateModelId } from "../../utils/validate" import { vscode } from "../../utils/vscode" import ApiOptions from "./ApiOptions" -import McpEnabledToggle from "../mcp/McpEnabledToggle" import ApiConfigManager from "./ApiConfigManager" -import { Mode } from "../../../../src/shared/modes" - -const IS_DEV = false // FIXME: use flags when packaging type SettingsViewProps = { onDone: () => void @@ -24,8 +14,6 @@ const SettingsView = ({ onDone }: SettingsViewProps) => { const { apiConfiguration, version, - customInstructions, - setCustomInstructions, alwaysAllowReadOnly, setAlwaysAllowReadOnly, alwaysAllowWrite, @@ -50,8 +38,6 @@ const SettingsView = ({ onDone }: SettingsViewProps) => { allowedCommands, fuzzyMatchThreshold, setFuzzyMatchThreshold, - preferredLanguage, - setPreferredLanguage, writeDelayMs, setWriteDelayMs, screenshotQuality, @@ -65,8 +51,6 @@ const SettingsView = ({ onDone }: SettingsViewProps) => { setRequestDelaySeconds, currentApiConfigName, listApiConfigMeta, - mode, - setMode, experimentalDiffStrategy, setExperimentalDiffStrategy, } = useExtensionState() @@ -85,7 +69,6 @@ const SettingsView = ({ onDone }: SettingsViewProps) => { type: "apiConfiguration", apiConfiguration, }) - vscode.postMessage({ type: "customInstructions", text: customInstructions }) vscode.postMessage({ type: "alwaysAllowReadOnly", bool: alwaysAllowReadOnly }) vscode.postMessage({ type: "alwaysAllowWrite", bool: alwaysAllowWrite }) vscode.postMessage({ type: "alwaysAllowExecute", bool: alwaysAllowExecute }) @@ -97,7 +80,6 @@ const SettingsView = ({ onDone }: SettingsViewProps) => { vscode.postMessage({ type: "diffEnabled", bool: diffEnabled }) vscode.postMessage({ type: "browserViewportSize", text: browserViewportSize }) vscode.postMessage({ type: "fuzzyMatchThreshold", value: fuzzyMatchThreshold ?? 1.0 }) - vscode.postMessage({ type: "preferredLanguage", text: preferredLanguage }) vscode.postMessage({ type: "writeDelayMs", value: writeDelayMs }) vscode.postMessage({ type: "screenshotQuality", value: screenshotQuality ?? 75 }) vscode.postMessage({ type: "terminalOutputLineLimit", value: terminalOutputLineLimit ?? 500 }) @@ -110,7 +92,6 @@ const SettingsView = ({ onDone }: SettingsViewProps) => { text: currentApiConfigName, apiConfiguration, }) - vscode.postMessage({ type: "mode", text: mode }) vscode.postMessage({ type: "experimentalDiffStrategy", bool: experimentalDiffStrategy }) onDone() } @@ -172,291 +153,69 @@ const SettingsView = ({ onDone }: SettingsViewProps) => {
-
-

- Provider Settings -

- { - vscode.postMessage({ - type: "loadApiConfiguration", - text: configName, - }) - }} - onDeleteConfig={(configName: string) => { - vscode.postMessage({ - type: "deleteApiConfiguration", - text: configName, - }) - }} - onRenameConfig={(oldName: string, newName: string) => { - vscode.postMessage({ - type: "renameApiConfiguration", - values: { oldName, newName }, - apiConfiguration, - }) - }} - onUpsertConfig={(configName: string) => { - vscode.postMessage({ - type: "upsertApiConfiguration", - text: configName, - apiConfiguration, - }) - }} - /> - -
- -
+
+

Provider Settings

-

- Agent Settings -

- -
- - -

- Select the mode that best fits your needs. Code mode focuses on implementation details, - Architect mode on high-level design, and Ask mode on asking questions about the - codebase. -

-
- - - -

- Select the language that Cline should use for communication. -

-
- -
- Custom Instructions - setCustomInstructions(e.target?.value ?? "")} - /> -

- These instructions are added to the end of the system prompt sent with every request. Custom - instructions set in .clinerules in the working directory are also included. For - mode-specific instructions, use the{" "} - Prompts tab - in the top menu. -

-
- - -
- -
-
- Terminal output limit - setTerminalOutputLineLimit(parseInt(e.target.value))} - style={{ - flexGrow: 1, - accentColor: "var(--vscode-button-background)", - height: "2px", + { + vscode.postMessage({ + type: "loadApiConfiguration", + text: configName, + }) + }} + onDeleteConfig={(configName: string) => { + vscode.postMessage({ + type: "deleteApiConfiguration", + text: configName, + }) + }} + onRenameConfig={(oldName: string, newName: string) => { + vscode.postMessage({ + type: "renameApiConfiguration", + values: { oldName, newName }, + apiConfiguration, + }) + }} + onUpsertConfig={(configName: string) => { + vscode.postMessage({ + type: "upsertApiConfiguration", + text: configName, + apiConfiguration, + }) }} /> - {terminalOutputLineLimit ?? 500} +
-

- Maximum number of lines to include in terminal output when executing commands. When exceeded - lines will be removed from the middle, saving tokens. -

-
- { - setDiffEnabled(e.target.checked) - if (!e.target.checked) { - // Reset experimental strategy when diffs are disabled - setExperimentalDiffStrategy(false) - } - }}> - Enable editing through diffs - -

- When enabled, Cline will be able to edit files more quickly and will automatically reject - truncated full-file writes. Works best with the latest Claude 3.5 Sonnet model. -

- - {diffEnabled && ( -
-
- ⚠️ - setExperimentalDiffStrategy(e.target.checked)}> - Use experimental unified diff strategy - -
-

- Enable the experimental unified diff strategy. This strategy might reduce the number of - retries caused by model errors but may cause unexpected behavior or incorrect edits. - Only enable if you understand the risks and are willing to carefully review all changes. -

- -
- Match precision - { - setFuzzyMatchThreshold(parseFloat(e.target.value)) - }} - style={{ - flexGrow: 1, - accentColor: "var(--vscode-button-background)", - height: "2px", - }} - /> - - {Math.round((fuzzyMatchThreshold || 1) * 100)}% - -
-

- This slider controls how precisely code sections must match when applying diffs. Lower - values allow more flexible matching but increase the risk of incorrect replacements. Use - values below 100% with extreme caution. -

-
- )} -
- -
- setAlwaysAllowReadOnly(e.target.checked)}> - Always approve read-only operations - -

- When enabled, Cline will automatically view directory contents and read files without requiring - you to click the Approve button. -

-
- -
-

- ⚠️ High-Risk Auto-Approve Settings -

+
+

Auto-Approve Settings

- The following settings allow Cline to automatically perform potentially dangerous operations - without requiring approval. Enable these settings only if you fully trust the AI and understand - the associated security risks. + The following settings allow Cline to automatically perform operations without requiring + approval. Enable these settings only if you fully trust the AI and understand the associated + security risks.

-
+
+ setAlwaysAllowReadOnly(e.target.checked)}> + Always approve read-only operations + +

+ When enabled, Cline will automatically view directory contents and read files without + requiring you to click the Approve button. +

+
+ +
setAlwaysAllowWrite(e.target.checked)}> @@ -495,7 +254,7 @@ const SettingsView = ({ onDone }: SettingsViewProps) => { )}
-
+
setAlwaysAllowBrowser(e.target.checked)}> @@ -508,7 +267,7 @@ const SettingsView = ({ onDone }: SettingsViewProps) => {

-
+
setAlwaysApproveResubmit(e.target.checked)}> @@ -559,7 +318,7 @@ const SettingsView = ({ onDone }: SettingsViewProps) => {

-
+
setAlwaysAllowExecute(e.target.checked)}> @@ -652,135 +411,218 @@ const SettingsView = ({ onDone }: SettingsViewProps) => {
-
-
-
-

- Browser Settings -

- - -

- Select the viewport size for browser interactions. This affects how websites are - displayed and interacted with. -

-
- -
-
- Screenshot quality - setScreenshotQuality(parseInt(e.target.value))} - style={{ - flexGrow: 1, - accentColor: "var(--vscode-button-background)", - height: "2px", - }} - /> - {screenshotQuality ?? 75}% -
-

- Adjust the WebP quality of browser screenshots. Higher values provide clearer - screenshots but increase token usage. -

-
-
- -
-
-

- Notification Settings -

- setSoundEnabled(e.target.checked)}> - Enable sound effects - -

- When enabled, Cline will play sound effects for notifications and events. -

-
- {soundEnabled && ( -
-
- Volume - setSoundVolume(parseFloat(e.target.value))} - style={{ - flexGrow: 1, - accentColor: "var(--vscode-button-background)", - height: "2px", - }} - aria-label="Volume" - /> - - {((soundVolume ?? 0.5) * 100).toFixed(0)}% - -
-
- )} -
-
- - {IS_DEV && ( - <> -
Debug
- - Reset State - +
+

Browser Settings

+
+ +

- This will reset all global state and secret storage in the extension. + Select the viewport size for browser interactions. This affects how websites are displayed + and interacted with.

- - )} +
+ +
+
+ Screenshot quality + setScreenshotQuality(parseInt(e.target.value))} + style={{ + flexGrow: 1, + accentColor: "var(--vscode-button-background)", + height: "2px", + }} + /> + {screenshotQuality ?? 75}% +
+

+ Adjust the WebP quality of browser screenshots. Higher values provide clearer screenshots + but increase token usage. +

+
+
+ +
+

Notification Settings

+
+ setSoundEnabled(e.target.checked)}> + Enable sound effects + +

+ When enabled, Cline will play sound effects for notifications and events. +

+
+ {soundEnabled && ( +
+
+ Volume + setSoundVolume(parseFloat(e.target.value))} + style={{ + flexGrow: 1, + accentColor: "var(--vscode-button-background)", + height: "2px", + }} + aria-label="Volume" + /> + + {((soundVolume ?? 0.5) * 100).toFixed(0)}% + +
+
+ )} +
+ +
+

Advanced Settings

+
+
+ Terminal output limit + setTerminalOutputLineLimit(parseInt(e.target.value))} + style={{ + flexGrow: 1, + accentColor: "var(--vscode-button-background)", + height: "2px", + }} + /> + + {terminalOutputLineLimit ?? 500} + +
+

+ Maximum number of lines to include in terminal output when executing commands. When exceeded + lines will be removed from the middle, saving tokens. +

+
+ +
+ { + setDiffEnabled(e.target.checked) + if (!e.target.checked) { + // Reset experimental strategy when diffs are disabled + setExperimentalDiffStrategy(false) + } + }}> + Enable editing through diffs + +

+ When enabled, Cline will be able to edit files more quickly and will automatically reject + truncated full-file writes. Works best with the latest Claude 3.5 Sonnet model. +

+ + {diffEnabled && ( +
+
+ ⚠️ + setExperimentalDiffStrategy(e.target.checked)}> + + Use experimental unified diff strategy + + +
+

+ Enable the experimental unified diff strategy. This strategy might reduce the number + of retries caused by model errors but may cause unexpected behavior or incorrect + edits. Only enable if you understand the risks and are willing to carefully review + all changes. +

+ +
+ Match precision + { + setFuzzyMatchThreshold(parseFloat(e.target.value)) + }} + style={{ + flexGrow: 1, + accentColor: "var(--vscode-button-background)", + height: "2px", + }} + /> + + {Math.round((fuzzyMatchThreshold || 1) * 100)}% + +
+

+ This slider controls how precisely code sections must match when applying diffs. + Lower values allow more flexible matching but increase the risk of incorrect + replacements. Use values below 100% with extreme caution. +

+
+ )} +
+
{ reddit.com/r/roocline

-

v{version}

+

+ v{version} +

+ +

+ This will reset all global state and secret storage in the extension. +

+ + + Reset State +
diff --git a/webview-ui/src/context/ExtensionStateContext.tsx b/webview-ui/src/context/ExtensionStateContext.tsx index f98abc4..c68be11 100644 --- a/webview-ui/src/context/ExtensionStateContext.tsx +++ b/webview-ui/src/context/ExtensionStateContext.tsx @@ -65,6 +65,7 @@ export interface ExtensionStateContextType extends ExtensionState { setExperimentalDiffStrategy: (value: boolean) => void autoApprovalEnabled?: boolean setAutoApprovalEnabled: (value: boolean) => void + handleInputChange: (field: keyof ApiConfiguration) => (event: any) => void } export const ExtensionStateContext = createContext(undefined) @@ -112,18 +113,32 @@ export const ExtensionStateContextProvider: React.FC<{ children: React.ReactNode const setListApiConfigMeta = useCallback( (value: ApiConfigMeta[]) => setState((prevState) => ({ ...prevState, listApiConfigMeta: value })), - [setState], + [], ) - const onUpdateApiConfig = useCallback( - (apiConfig: ApiConfiguration) => { + const onUpdateApiConfig = useCallback((apiConfig: ApiConfiguration) => { + setState((currentState) => { vscode.postMessage({ type: "upsertApiConfiguration", - text: state.currentApiConfigName, + text: currentState.currentApiConfigName, apiConfiguration: apiConfig, }) + return currentState // No state update needed + }) + }, []) + + const handleInputChange = useCallback( + (field: keyof ApiConfiguration) => (event: any) => { + setState((currentState) => { + vscode.postMessage({ + type: "upsertApiConfiguration", + text: currentState.currentApiConfigName, + apiConfiguration: { ...currentState.apiConfiguration, [field]: event.target.value }, + }) + return currentState // No state update needed + }) }, - [state], + [], ) const handleMessage = useCallback( @@ -258,6 +273,7 @@ export const ExtensionStateContextProvider: React.FC<{ children: React.ReactNode setExperimentalDiffStrategy: (value) => setState((prevState) => ({ ...prevState, experimentalDiffStrategy: value })), setAutoApprovalEnabled: (value) => setState((prevState) => ({ ...prevState, autoApprovalEnabled: value })), + handleInputChange, } return {children}