From 25cd7cc8e2ec9dfce1a60307e8d978865eb8116a Mon Sep 17 00:00:00 2001 From: Saoud Rizwan <7799382+saoudrizwan@users.noreply.github.com> Date: Mon, 6 Jan 2025 12:41:33 -0800 Subject: [PATCH] chore: Prettier fix formatting --- .prettierrc.json | 2 +- .vscode/extensions.json | 6 +- .vscode/tasks.json | 6 +- esbuild.js | 27 +- src/api/index.ts | 5 +- src/api/providers/anthropic.ts | 58 +- src/api/providers/bedrock.ts | 25 +- src/api/providers/deepseek.ts | 18 +- src/api/providers/gemini.ts | 13 +- src/api/providers/lmstudio.ts | 15 +- src/api/providers/ollama.ts | 15 +- src/api/providers/openai-native.ts | 15 +- src/api/providers/openai.ts | 16 +- src/api/providers/openrouter.ts | 53 +- src/api/providers/vertex.ts | 13 +- src/api/transform/gemini-format.ts | 62 +- src/api/transform/o1-format.ts | 40 +- src/api/transform/openai-format.ts | 125 +- src/core/Cline.ts | 1942 ++++------------- src/core/assistant-message/diff.ts | 56 +- src/core/assistant-message/index.ts | 22 +- .../parse-assistant-message.ts | 61 +- src/core/mentions/index.ts | 50 +- src/core/prompts/responses.ts | 44 +- src/core/prompts/system.ts | 23 +- src/core/webview/ClineProvider.ts | 447 +--- src/core/webview/getNonce.ts | 3 +- src/core/webview/getUri.ts | 6 +- src/exports/README.md | 8 +- src/exports/index.ts | 9 +- src/extension.ts | 80 +- .../checkpoints/CheckpointTracker.ts | 91 +- src/integrations/diagnostics/index.ts | 9 +- .../editor/DecorationController.ts | 34 +- src/integrations/editor/DiffViewProvider.ts | 136 +- src/integrations/editor/detect-omission.ts | 20 +- src/integrations/misc/export-markdown.ts | 41 +- src/integrations/misc/extract-text.ts | 9 +- src/integrations/misc/open-file.ts | 32 +- src/integrations/notifications/index.ts | 16 +- src/integrations/terminal/TerminalManager.ts | 58 +- src/integrations/terminal/TerminalProcess.ts | 53 +- src/integrations/terminal/TerminalRegistry.ts | 4 +- .../theme/default-themes/dark_plus.json | 5 +- .../theme/default-themes/dark_vs.json | 11 +- .../theme/default-themes/hc_black.json | 19 +- .../theme/default-themes/hc_light.json | 27 +- .../theme/default-themes/light_plus.json | 5 +- .../theme/default-themes/light_vs.json | 16 +- src/integrations/theme/getTheme.ts | 54 +- .../workspace/WorkspaceTracker.ts | 38 +- src/integrations/workspace/get-python-env.ts | 4 +- src/services/browser/BrowserSession.ts | 12 +- src/services/glob/list-files.ts | 13 +- src/services/mcp/McpHub.ts | 200 +- src/services/ripgrep/index.ts | 28 +- src/services/tree-sitter/index.ts | 21 +- src/services/tree-sitter/languageParser.ts | 12 +- src/shared/ExtensionMessage.ts | 16 +- src/shared/WebviewMessage.ts | 5 +- src/shared/api.ts | 12 +- src/shared/array.ts | 10 +- src/shared/combineApiRequests.ts | 18 +- src/shared/combineCommandSequences.ts | 34 +- src/shared/context-mentions.ts | 3 +- src/shared/getApiMetrics.ts | 16 +- src/utils/cost.ts | 12 +- src/utils/fs.test.ts | 22 +- src/utils/fs.ts | 4 +- src/utils/path.test.ts | 4 +- src/utils/path.ts | 5 +- webview-ui/public/index.html | 4 +- webview-ui/src/App.tsx | 16 +- .../src/components/chat/Announcement.tsx | 32 +- .../src/components/chat/AutoApproveMenu.tsx | 75 +- .../src/components/chat/BrowserSessionRow.tsx | 183 +- webview-ui/src/components/chat/ChatRow.tsx | 350 +-- .../src/components/chat/ChatTextArea.tsx | 221 +- webview-ui/src/components/chat/ChatView.tsx | 216 +- .../src/components/chat/ContextMenu.tsx | 60 +- webview-ui/src/components/chat/TaskHeader.tsx | 180 +- .../components/common/CheckpointControls.tsx | 46 +- .../src/components/common/CodeAccordian.tsx | 13 +- .../src/components/common/CodeBlock.tsx | 16 +- webview-ui/src/components/common/Demo.tsx | 48 +- .../src/components/common/MarkdownBlock.tsx | 9 +- .../src/components/common/SuccessButton.tsx | 3 +- .../src/components/common/Thumbnails.tsx | 10 +- .../components/common/VSCodeButtonLink.tsx | 6 +- .../src/components/history/HistoryPreview.tsx | 25 +- .../src/components/history/HistoryView.tsx | 146 +- .../src/components/mcp/McpResourceRow.tsx | 9 +- webview-ui/src/components/mcp/McpToolRow.tsx | 11 +- webview-ui/src/components/mcp/McpView.tsx | 88 +- .../src/components/settings/ApiOptions.tsx | 383 +--- .../settings/OpenRouterModelPicker.tsx | 74 +- .../src/components/settings/SettingsView.tsx | 64 +- .../src/components/settings/TabNavbar.tsx | 28 +- .../src/components/welcome/WelcomeView.tsx | 24 +- .../src/context/ExtensionStateContext.tsx | 43 +- webview-ui/src/index.css | 24 +- webview-ui/src/reportWebVitals.ts | 16 +- webview-ui/src/utils/context-mentions.ts | 34 +- webview-ui/src/utils/mcp.ts | 5 +- webview-ui/src/utils/textMateToHljs.ts | 27 +- webview-ui/src/utils/validate.ts | 29 +- 106 files changed, 1629 insertions(+), 5283 deletions(-) diff --git a/.prettierrc.json b/.prettierrc.json index cd4329335c..f2cbe27cf6 100644 --- a/.prettierrc.json +++ b/.prettierrc.json @@ -1,7 +1,7 @@ { "tabWidth": 4, "useTabs": true, - "printWidth": 120, + "printWidth": 130, "semi": false, "bracketSameLine": true } diff --git a/.vscode/extensions.json b/.vscode/extensions.json index 5be3c32fb1..a41ce3c1e1 100644 --- a/.vscode/extensions.json +++ b/.vscode/extensions.json @@ -1,9 +1,5 @@ { // See http://go.microsoft.com/fwlink/?LinkId=827846 // for the documentation about the extensions.json format - "recommendations": [ - "dbaeumer.vscode-eslint", - "connor4312.esbuild-problem-matchers", - "ms-vscode.extension-test-runner" - ] + "recommendations": ["dbaeumer.vscode-eslint", "connor4312.esbuild-problem-matchers", "ms-vscode.extension-test-runner"] } diff --git a/.vscode/tasks.json b/.vscode/tasks.json index 6878c4156c..e1413836d1 100644 --- a/.vscode/tasks.json +++ b/.vscode/tasks.json @@ -5,11 +5,7 @@ "tasks": [ { "label": "watch", - "dependsOn": [ - "npm: build:webview", - "npm: watch:tsc", - "npm: watch:esbuild" - ], + "dependsOn": ["npm: build:webview", "npm: watch:tsc", "npm: watch:esbuild"], "presentation": { "reveal": "never" }, diff --git a/esbuild.js b/esbuild.js index f4ddcc9f9f..8b203076e4 100644 --- a/esbuild.js +++ b/esbuild.js @@ -18,9 +18,7 @@ const esbuildProblemMatcherPlugin = { build.onEnd((result) => { result.errors.forEach(({ text, location }) => { console.error(`✘ [ERROR] ${text}`) - console.error( - ` ${location.file}:${location.line}:${location.column}:`, - ) + console.error(` ${location.file}:${location.line}:${location.column}:`) }) console.log("[watch] build finished") }) @@ -32,26 +30,14 @@ const copyWasmFiles = { setup(build) { build.onEnd(() => { // tree sitter - const sourceDir = path.join( - __dirname, - "node_modules", - "web-tree-sitter", - ) + const sourceDir = path.join(__dirname, "node_modules", "web-tree-sitter") const targetDir = path.join(__dirname, "dist") // Copy tree-sitter.wasm - fs.copyFileSync( - path.join(sourceDir, "tree-sitter.wasm"), - path.join(targetDir, "tree-sitter.wasm"), - ) + fs.copyFileSync(path.join(sourceDir, "tree-sitter.wasm"), path.join(targetDir, "tree-sitter.wasm")) // Copy language-specific WASM files - const languageWasmDir = path.join( - __dirname, - "node_modules", - "tree-sitter-wasms", - "out", - ) + const languageWasmDir = path.join(__dirname, "node_modules", "tree-sitter-wasms", "out") const languages = [ "typescript", "tsx", @@ -70,10 +56,7 @@ const copyWasmFiles = { languages.forEach((lang) => { const filename = `tree-sitter-${lang}.wasm` - fs.copyFileSync( - path.join(languageWasmDir, filename), - path.join(targetDir, filename), - ) + fs.copyFileSync(path.join(languageWasmDir, filename), path.join(targetDir, filename)) }) }) }, diff --git a/src/api/index.ts b/src/api/index.ts index ce75fecd68..287f843642 100644 --- a/src/api/index.ts +++ b/src/api/index.ts @@ -13,10 +13,7 @@ import { ApiStream } from "./transform/stream" import { DeepSeekHandler } from "./providers/deepseek" export interface ApiHandler { - createMessage( - systemPrompt: string, - messages: Anthropic.Messages.MessageParam[], - ): ApiStream + createMessage(systemPrompt: string, messages: Anthropic.Messages.MessageParam[]): ApiStream getModel(): { id: string; info: ModelInfo } } diff --git a/src/api/providers/anthropic.ts b/src/api/providers/anthropic.ts index 944883ff9b..6fbe1f2509 100644 --- a/src/api/providers/anthropic.ts +++ b/src/api/providers/anthropic.ts @@ -1,12 +1,6 @@ import { Anthropic } from "@anthropic-ai/sdk" import { Stream as AnthropicStream } from "@anthropic-ai/sdk/streaming" -import { - anthropicDefaultModelId, - AnthropicModelId, - anthropicModels, - ApiHandlerOptions, - ModelInfo, -} from "../../shared/api" +import { anthropicDefaultModelId, AnthropicModelId, anthropicModels, ApiHandlerOptions, ModelInfo } from "../../shared/api" import { ApiHandler } from "../index" import { ApiStream } from "../transform/stream" @@ -22,10 +16,7 @@ export class AnthropicHandler implements ApiHandler { }) } - async *createMessage( - systemPrompt: string, - messages: Anthropic.Messages.MessageParam[], - ): ApiStream { + async *createMessage(systemPrompt: string, messages: Anthropic.Messages.MessageParam[]): ApiStream { let stream: AnthropicStream const modelId = this.getModel().id switch (modelId) { @@ -38,14 +29,11 @@ export class AnthropicHandler implements ApiHandler { The latest message will be the new user message, one before will be the assistant message from a previous request, and the user message before that will be a previously cached user message. So we need to mark the latest user message as ephemeral to cache it for the next request, and mark the second to last user message as ephemeral to let the server know the last message to retrieve from the cache for the current request.. */ const userMsgIndices = messages.reduce( - (acc, msg, index) => - msg.role === "user" ? [...acc, index] : acc, + (acc, msg, index) => (msg.role === "user" ? [...acc, index] : acc), [] as number[], ) - const lastUserMsgIndex = - userMsgIndices[userMsgIndices.length - 1] ?? -1 - const secondLastMsgUserIndex = - userMsgIndices[userMsgIndices.length - 2] ?? -1 + const lastUserMsgIndex = userMsgIndices[userMsgIndices.length - 1] ?? -1 + const secondLastMsgUserIndex = userMsgIndices[userMsgIndices.length - 2] ?? -1 stream = await this.client.beta.promptCaching.messages.create( { model: modelId, @@ -59,10 +47,7 @@ export class AnthropicHandler implements ApiHandler { }, ], // setting cache breakpoint for system prompt so new tasks can reuse it messages: messages.map((message, index) => { - if ( - index === lastUserMsgIndex || - index === secondLastMsgUserIndex - ) { + if (index === lastUserMsgIndex || index === secondLastMsgUserIndex) { return { ...message, content: @@ -76,19 +61,15 @@ export class AnthropicHandler implements ApiHandler { }, }, ] - : message.content.map( - (content, contentIndex) => - contentIndex === - message.content.length - - 1 - ? { - ...content, - cache_control: - { - type: "ephemeral", - }, - } - : content, + : message.content.map((content, contentIndex) => + contentIndex === message.content.length - 1 + ? { + ...content, + cache_control: { + type: "ephemeral", + }, + } + : content, ), } } @@ -110,8 +91,7 @@ export class AnthropicHandler implements ApiHandler { case "claude-3-haiku-20240307": return { headers: { - "anthropic-beta": - "prompt-caching-2024-07-31", + "anthropic-beta": "prompt-caching-2024-07-31", }, } default: @@ -145,10 +125,8 @@ export class AnthropicHandler implements ApiHandler { type: "usage", inputTokens: usage.input_tokens || 0, outputTokens: usage.output_tokens || 0, - cacheWriteTokens: - usage.cache_creation_input_tokens || undefined, - cacheReadTokens: - usage.cache_read_input_tokens || undefined, + cacheWriteTokens: usage.cache_creation_input_tokens || undefined, + cacheReadTokens: usage.cache_read_input_tokens || undefined, } break case "message_delta": diff --git a/src/api/providers/bedrock.ts b/src/api/providers/bedrock.ts index b02fbfa9e6..a82a71b8ef 100644 --- a/src/api/providers/bedrock.ts +++ b/src/api/providers/bedrock.ts @@ -1,13 +1,7 @@ import AnthropicBedrock from "@anthropic-ai/bedrock-sdk" import { Anthropic } from "@anthropic-ai/sdk" import { ApiHandler } from "../" -import { - ApiHandlerOptions, - bedrockDefaultModelId, - BedrockModelId, - bedrockModels, - ModelInfo, -} from "../../shared/api" +import { ApiHandlerOptions, bedrockDefaultModelId, BedrockModelId, bedrockModels, ModelInfo } from "../../shared/api" import { ApiStream } from "../transform/stream" // https://docs.anthropic.com/en/api/claude-on-amazon-bedrock @@ -20,15 +14,9 @@ export class AwsBedrockHandler implements ApiHandler { this.client = new AnthropicBedrock({ // Authenticate by either providing the keys below or use the default AWS credential providers, such as // using ~/.aws/credentials or the "AWS_SECRET_ACCESS_KEY" and "AWS_ACCESS_KEY_ID" environment variables. - ...(this.options.awsAccessKey - ? { awsAccessKey: this.options.awsAccessKey } - : {}), - ...(this.options.awsSecretKey - ? { awsSecretKey: this.options.awsSecretKey } - : {}), - ...(this.options.awsSessionToken - ? { awsSessionToken: this.options.awsSessionToken } - : {}), + ...(this.options.awsAccessKey ? { awsAccessKey: this.options.awsAccessKey } : {}), + ...(this.options.awsSecretKey ? { awsSecretKey: this.options.awsSecretKey } : {}), + ...(this.options.awsSessionToken ? { awsSessionToken: this.options.awsSessionToken } : {}), // awsRegion changes the aws region to which the request is made. By default, we read AWS_REGION, // and if that's not present, we default to us-east-1. Note that we do not read ~/.aws/config for the region. @@ -36,10 +24,7 @@ export class AwsBedrockHandler implements ApiHandler { }) } - async *createMessage( - systemPrompt: string, - messages: Anthropic.Messages.MessageParam[], - ): ApiStream { + async *createMessage(systemPrompt: string, messages: Anthropic.Messages.MessageParam[]): ApiStream { // cross region inference requires prefixing the model id with the region let modelId: string if (this.options.awsUseCrossRegionInference) { diff --git a/src/api/providers/deepseek.ts b/src/api/providers/deepseek.ts index 9601ca2242..a903ce2dd9 100644 --- a/src/api/providers/deepseek.ts +++ b/src/api/providers/deepseek.ts @@ -1,13 +1,7 @@ import { Anthropic } from "@anthropic-ai/sdk" import OpenAI from "openai" import { ApiHandler } from "../" -import { - ApiHandlerOptions, - DeepSeekModelId, - ModelInfo, - deepSeekDefaultModelId, - deepSeekModels, -} from "../../shared/api" +import { ApiHandlerOptions, DeepSeekModelId, ModelInfo, deepSeekDefaultModelId, deepSeekModels } from "../../shared/api" import { convertToOpenAiMessages } from "../transform/openai-format" import { ApiStream } from "../transform/stream" @@ -23,18 +17,12 @@ export class DeepSeekHandler implements ApiHandler { }) } - async *createMessage( - systemPrompt: string, - messages: Anthropic.Messages.MessageParam[], - ): ApiStream { + async *createMessage(systemPrompt: string, messages: Anthropic.Messages.MessageParam[]): ApiStream { const stream = await this.client.chat.completions.create({ model: this.getModel().id, max_completion_tokens: this.getModel().info.maxTokens, temperature: 0, - messages: [ - { role: "system", content: systemPrompt }, - ...convertToOpenAiMessages(messages), - ], + messages: [{ role: "system", content: systemPrompt }, ...convertToOpenAiMessages(messages)], stream: true, stream_options: { include_usage: true }, }) diff --git a/src/api/providers/gemini.ts b/src/api/providers/gemini.ts index de6bf394ec..39c55548d1 100644 --- a/src/api/providers/gemini.ts +++ b/src/api/providers/gemini.ts @@ -1,13 +1,7 @@ import { Anthropic } from "@anthropic-ai/sdk" import { GoogleGenerativeAI } from "@google/generative-ai" import { ApiHandler } from "../" -import { - ApiHandlerOptions, - geminiDefaultModelId, - GeminiModelId, - geminiModels, - ModelInfo, -} from "../../shared/api" +import { ApiHandlerOptions, geminiDefaultModelId, GeminiModelId, geminiModels, ModelInfo } from "../../shared/api" import { convertAnthropicMessageToGemini } from "../transform/gemini-format" import { ApiStream } from "../transform/stream" @@ -23,10 +17,7 @@ export class GeminiHandler implements ApiHandler { this.client = new GoogleGenerativeAI(options.geminiApiKey) } - async *createMessage( - systemPrompt: string, - messages: Anthropic.Messages.MessageParam[], - ): ApiStream { + async *createMessage(systemPrompt: string, messages: Anthropic.Messages.MessageParam[]): ApiStream { const model = this.client.getGenerativeModel({ model: this.getModel().id, systemInstruction: systemPrompt, diff --git a/src/api/providers/lmstudio.ts b/src/api/providers/lmstudio.ts index 37fa67cea7..868ef7da13 100644 --- a/src/api/providers/lmstudio.ts +++ b/src/api/providers/lmstudio.ts @@ -1,11 +1,7 @@ import { Anthropic } from "@anthropic-ai/sdk" import OpenAI from "openai" import { ApiHandler } from "../" -import { - ApiHandlerOptions, - ModelInfo, - openAiModelInfoSaneDefaults, -} from "../../shared/api" +import { ApiHandlerOptions, ModelInfo, openAiModelInfoSaneDefaults } from "../../shared/api" import { convertToOpenAiMessages } from "../transform/openai-format" import { ApiStream } from "../transform/stream" @@ -16,17 +12,12 @@ export class LmStudioHandler implements ApiHandler { constructor(options: ApiHandlerOptions) { this.options = options this.client = new OpenAI({ - baseURL: - (this.options.lmStudioBaseUrl || "http://localhost:1234") + - "/v1", + baseURL: (this.options.lmStudioBaseUrl || "http://localhost:1234") + "/v1", apiKey: "noop", }) } - async *createMessage( - systemPrompt: string, - messages: Anthropic.Messages.MessageParam[], - ): ApiStream { + async *createMessage(systemPrompt: string, messages: Anthropic.Messages.MessageParam[]): ApiStream { const openAiMessages: OpenAI.Chat.ChatCompletionMessageParam[] = [ { role: "system", content: systemPrompt }, ...convertToOpenAiMessages(messages), diff --git a/src/api/providers/ollama.ts b/src/api/providers/ollama.ts index 01d4f73fe8..7668bd395f 100644 --- a/src/api/providers/ollama.ts +++ b/src/api/providers/ollama.ts @@ -1,11 +1,7 @@ import { Anthropic } from "@anthropic-ai/sdk" import OpenAI from "openai" import { ApiHandler } from "../" -import { - ApiHandlerOptions, - ModelInfo, - openAiModelInfoSaneDefaults, -} from "../../shared/api" +import { ApiHandlerOptions, ModelInfo, openAiModelInfoSaneDefaults } from "../../shared/api" import { convertToOpenAiMessages } from "../transform/openai-format" import { ApiStream } from "../transform/stream" @@ -16,17 +12,12 @@ export class OllamaHandler implements ApiHandler { constructor(options: ApiHandlerOptions) { this.options = options this.client = new OpenAI({ - baseURL: - (this.options.ollamaBaseUrl || "http://localhost:11434") + - "/v1", + baseURL: (this.options.ollamaBaseUrl || "http://localhost:11434") + "/v1", apiKey: "ollama", }) } - async *createMessage( - systemPrompt: string, - messages: Anthropic.Messages.MessageParam[], - ): ApiStream { + async *createMessage(systemPrompt: string, messages: Anthropic.Messages.MessageParam[]): ApiStream { const openAiMessages: OpenAI.Chat.ChatCompletionMessageParam[] = [ { role: "system", content: systemPrompt }, ...convertToOpenAiMessages(messages), diff --git a/src/api/providers/openai-native.ts b/src/api/providers/openai-native.ts index 3e7dc86b2b..d11481add4 100644 --- a/src/api/providers/openai-native.ts +++ b/src/api/providers/openai-native.ts @@ -22,20 +22,14 @@ export class OpenAiNativeHandler implements ApiHandler { }) } - async *createMessage( - systemPrompt: string, - messages: Anthropic.Messages.MessageParam[], - ): ApiStream { + async *createMessage(systemPrompt: string, messages: Anthropic.Messages.MessageParam[]): ApiStream { switch (this.getModel().id) { case "o1-preview": case "o1-mini": { // o1 doesnt support streaming, non-1 temp, or system prompt const response = await this.client.chat.completions.create({ model: this.getModel().id, - messages: [ - { role: "user", content: systemPrompt }, - ...convertToOpenAiMessages(messages), - ], + messages: [{ role: "user", content: systemPrompt }, ...convertToOpenAiMessages(messages)], }) yield { type: "text", @@ -53,10 +47,7 @@ export class OpenAiNativeHandler implements ApiHandler { model: this.getModel().id, // max_completion_tokens: this.getModel().info.maxTokens, temperature: 0, - messages: [ - { role: "system", content: systemPrompt }, - ...convertToOpenAiMessages(messages), - ], + messages: [{ role: "system", content: systemPrompt }, ...convertToOpenAiMessages(messages)], stream: true, stream_options: { include_usage: true }, }) diff --git a/src/api/providers/openai.ts b/src/api/providers/openai.ts index 241f621831..58e4ba0250 100644 --- a/src/api/providers/openai.ts +++ b/src/api/providers/openai.ts @@ -1,11 +1,6 @@ import { Anthropic } from "@anthropic-ai/sdk" import OpenAI, { AzureOpenAI } from "openai" -import { - ApiHandlerOptions, - azureOpenAiDefaultApiVersion, - ModelInfo, - openAiModelInfoSaneDefaults, -} from "../../shared/api" +import { ApiHandlerOptions, azureOpenAiDefaultApiVersion, ModelInfo, openAiModelInfoSaneDefaults } from "../../shared/api" import { ApiHandler } from "../index" import { convertToOpenAiMessages } from "../transform/openai-format" import { ApiStream } from "../transform/stream" @@ -21,9 +16,7 @@ export class OpenAiHandler implements ApiHandler { this.client = new AzureOpenAI({ baseURL: this.options.openAiBaseUrl, apiKey: this.options.openAiApiKey, - apiVersion: - this.options.azureApiVersion || - azureOpenAiDefaultApiVersion, + apiVersion: this.options.azureApiVersion || azureOpenAiDefaultApiVersion, }) } else { this.client = new OpenAI({ @@ -33,10 +26,7 @@ export class OpenAiHandler implements ApiHandler { } } - async *createMessage( - systemPrompt: string, - messages: Anthropic.Messages.MessageParam[], - ): ApiStream { + async *createMessage(systemPrompt: string, messages: Anthropic.Messages.MessageParam[]): ApiStream { const openAiMessages: OpenAI.Chat.ChatCompletionMessageParam[] = [ { role: "system", content: systemPrompt }, ...convertToOpenAiMessages(messages), diff --git a/src/api/providers/openrouter.ts b/src/api/providers/openrouter.ts index 85b27f0df1..3b9d7a354a 100644 --- a/src/api/providers/openrouter.ts +++ b/src/api/providers/openrouter.ts @@ -2,12 +2,7 @@ import { Anthropic } from "@anthropic-ai/sdk" import axios from "axios" import OpenAI from "openai" import { ApiHandler } from "../" -import { - ApiHandlerOptions, - ModelInfo, - openRouterDefaultModelId, - openRouterDefaultModelInfo, -} from "../../shared/api" +import { ApiHandlerOptions, ModelInfo, openRouterDefaultModelId, openRouterDefaultModelInfo } from "../../shared/api" import { convertToOpenAiMessages } from "../transform/openai-format" import { ApiStream } from "../transform/stream" import delay from "delay" @@ -28,10 +23,7 @@ export class OpenRouterHandler implements ApiHandler { }) } - async *createMessage( - systemPrompt: string, - messages: Anthropic.Messages.MessageParam[], - ): ApiStream { + async *createMessage(systemPrompt: string, messages: Anthropic.Messages.MessageParam[]): ApiStream { // Convert Anthropic messages to OpenAI format const openAiMessages: OpenAI.Chat.ChatCompletionMessageParam[] = [ { role: "system", content: systemPrompt }, @@ -66,18 +58,14 @@ export class OpenRouterHandler implements ApiHandler { } // Add cache_control to the last two user messages // (note: this works because we only ever add one user message at a time, but if we added multiple we'd need to mark the user message before the last assistant message) - const lastTwoUserMessages = openAiMessages - .filter((msg) => msg.role === "user") - .slice(-2) + const lastTwoUserMessages = openAiMessages.filter((msg) => msg.role === "user").slice(-2) lastTwoUserMessages.forEach((msg) => { if (typeof msg.content === "string") { msg.content = [{ type: "text", text: msg.content }] } if (Array.isArray(msg.content)) { // NOTE: this is fine since env details will always be added at the end. but if it weren't there, and the user added a image_url type message, it would pop a text part before it and then move it after to the end. - let lastTextPart = msg.content - .filter((part) => part.type === "text") - .pop() + let lastTextPart = msg.content.filter((part) => part.type === "text").pop() if (!lastTextPart) { lastTextPart = { type: "text", text: "..." } @@ -109,8 +97,7 @@ export class OpenRouterHandler implements ApiHandler { } // Removes messages in the middle when close to context window limit. Should not be applied to models that support prompt caching since it would continuously break the cache. - let shouldApplyMiddleOutTransform = - !this.getModel().info.supportsPromptCache + let shouldApplyMiddleOutTransform = !this.getModel().info.supportsPromptCache // except for deepseek (which we set supportsPromptCache to true for), where because the context window is so small our truncation algo might miss and we should use openrouter's middle-out transform as a fallback to ensure we don't exceed the context window (FIXME: once we have a more robust token estimator we should not rely on this) if (this.getModel().id === "deepseek/deepseek-chat") { shouldApplyMiddleOutTransform = true @@ -123,9 +110,7 @@ export class OpenRouterHandler implements ApiHandler { temperature: 0, messages: openAiMessages, stream: true, - transforms: shouldApplyMiddleOutTransform - ? ["middle-out"] - : undefined, + transforms: shouldApplyMiddleOutTransform ? ["middle-out"] : undefined, }) let genId: string | undefined @@ -134,12 +119,8 @@ export class OpenRouterHandler implements ApiHandler { // openrouter returns an error object instead of the openai sdk throwing an error if ("error" in chunk) { const error = chunk.error as { message?: string; code?: number } - console.error( - `OpenRouter API Error: ${error?.code} - ${error?.message}`, - ) - throw new Error( - `OpenRouter API Error ${error?.code}: ${error?.message}`, - ) + console.error(`OpenRouter API Error: ${error?.code} - ${error?.message}`) + throw new Error(`OpenRouter API Error ${error?.code}: ${error?.message}`) } if (!genId && chunk.id) { @@ -165,15 +146,12 @@ export class OpenRouterHandler implements ApiHandler { await delay(500) // FIXME: necessary delay to ensure generation endpoint is ready try { - const response = await axios.get( - `https://openrouter.ai/api/v1/generation?id=${genId}`, - { - headers: { - Authorization: `Bearer ${this.options.openRouterApiKey}`, - }, - timeout: 5_000, // this request hangs sometimes + const response = await axios.get(`https://openrouter.ai/api/v1/generation?id=${genId}`, { + headers: { + Authorization: `Bearer ${this.options.openRouterApiKey}`, }, - ) + timeout: 5_000, // this request hangs sometimes + }) const generation = response.data?.data console.log("OpenRouter generation details:", response.data) @@ -188,10 +166,7 @@ export class OpenRouterHandler implements ApiHandler { } } catch (error) { // ignore if fails - console.error( - "Error fetching OpenRouter generation details:", - error, - ) + console.error("Error fetching OpenRouter generation details:", error) } } diff --git a/src/api/providers/vertex.ts b/src/api/providers/vertex.ts index 304a491859..c8f1efd873 100644 --- a/src/api/providers/vertex.ts +++ b/src/api/providers/vertex.ts @@ -1,13 +1,7 @@ import { Anthropic } from "@anthropic-ai/sdk" import { AnthropicVertex } from "@anthropic-ai/vertex-sdk" import { ApiHandler } from "../" -import { - ApiHandlerOptions, - ModelInfo, - vertexDefaultModelId, - VertexModelId, - vertexModels, -} from "../../shared/api" +import { ApiHandlerOptions, ModelInfo, vertexDefaultModelId, VertexModelId, vertexModels } from "../../shared/api" import { ApiStream } from "../transform/stream" // https://docs.anthropic.com/en/api/claude-on-vertex-ai @@ -24,10 +18,7 @@ export class VertexHandler implements ApiHandler { }) } - async *createMessage( - systemPrompt: string, - messages: Anthropic.Messages.MessageParam[], - ): ApiStream { + async *createMessage(systemPrompt: string, messages: Anthropic.Messages.MessageParam[]): ApiStream { const stream = await this.client.messages.create({ model: this.getModel().id, max_tokens: this.getModel().info.maxTokens || 8192, diff --git a/src/api/transform/gemini-format.ts b/src/api/transform/gemini-format.ts index 16332c3640..0a783dc183 100644 --- a/src/api/transform/gemini-format.ts +++ b/src/api/transform/gemini-format.ts @@ -62,20 +62,10 @@ export function convertAnthropicContentToGemini( } as FunctionResponsePart } else { // The only case when tool_result could be array is when the tool failed and we're providing ie user feedback potentially with images - const textParts = block.content.filter( - (part) => part.type === "text", - ) - const imageParts = block.content.filter( - (part) => part.type === "image", - ) - const text = - textParts.length > 0 - ? textParts.map((part) => part.text).join("\n\n") - : "" - const imageText = - imageParts.length > 0 - ? "\n\n(See next part for image)" - : "" + const textParts = block.content.filter((part) => part.type === "text") + const imageParts = block.content.filter((part) => part.type === "image") + const text = textParts.length > 0 ? textParts.map((part) => part.text).join("\n\n") : "" + const imageText = imageParts.length > 0 ? "\n\n(See next part for image)" : "" return [ { functionResponse: { @@ -98,40 +88,32 @@ export function convertAnthropicContentToGemini( ] } default: - throw new Error( - `Unsupported content block type: ${(block as any).type}`, - ) + throw new Error(`Unsupported content block type: ${(block as any).type}`) } }) } -export function convertAnthropicMessageToGemini( - message: Anthropic.Messages.MessageParam, -): Content { +export function convertAnthropicMessageToGemini(message: Anthropic.Messages.MessageParam): Content { return { role: message.role === "assistant" ? "model" : "user", parts: convertAnthropicContentToGemini(message.content), } } -export function convertAnthropicToolToGemini( - tool: Anthropic.Messages.Tool, -): FunctionDeclaration { +export function convertAnthropicToolToGemini(tool: Anthropic.Messages.Tool): FunctionDeclaration { return { name: tool.name, description: tool.description || "", parameters: { type: SchemaType.OBJECT, properties: Object.fromEntries( - Object.entries(tool.input_schema.properties || {}).map( - ([key, value]) => [ - key, - { - type: (value as any).type.toUpperCase(), - description: (value as any).description || "", - }, - ], - ), + Object.entries(tool.input_schema.properties || {}).map(([key, value]) => [ + key, + { + type: (value as any).type.toUpperCase(), + description: (value as any).description || "", + }, + ]), ), required: (tool.input_schema.required as string[]) || [], }, @@ -142,17 +124,10 @@ export function convertAnthropicToolToGemini( It looks like gemini likes to double escape certain characters when writing file contents: https://discuss.ai.google.dev/t/function-call-string-property-is-double-escaped/37867 */ export function unescapeGeminiContent(content: string) { - return content - .replace(/\\n/g, "\n") - .replace(/\\'/g, "'") - .replace(/\\"/g, '"') - .replace(/\\r/g, "\r") - .replace(/\\t/g, "\t") + return content.replace(/\\n/g, "\n").replace(/\\'/g, "'").replace(/\\"/g, '"').replace(/\\r/g, "\r").replace(/\\t/g, "\t") } -export function convertGeminiResponseToAnthropic( - response: EnhancedGenerateContentResponse, -): Anthropic.Messages.Message { +export function convertGeminiResponseToAnthropic(response: EnhancedGenerateContentResponse): Anthropic.Messages.Message { const content: Anthropic.Messages.ContentBlock[] = [] // Add the main text response @@ -165,10 +140,7 @@ export function convertGeminiResponseToAnthropic( const functionCalls = response.functionCalls() if (functionCalls) { functionCalls.forEach((call, index) => { - if ( - "content" in call.args && - typeof call.args.content === "string" - ) { + if ("content" in call.args && typeof call.args.content === "string") { call.args.content = unescapeGeminiContent(call.args.content) } content.push({ diff --git a/src/api/transform/o1-format.ts b/src/api/transform/o1-format.ts index 16e4de4b6d..b7577cf903 100644 --- a/src/api/transform/o1-format.ts +++ b/src/api/transform/o1-format.ts @@ -272,9 +272,7 @@ function parseToolCalls(toolCallsText: string): ToolCall[] { let remainingText = toolCallsText while (remainingText.length > 0) { - const toolMatch = toolNames.find((tool) => - new RegExp(`<${tool}`, "i").test(remainingText), - ) + const toolMatch = toolNames.find((tool) => new RegExp(`<${tool}`, "i").test(remainingText)) if (!toolMatch) { break // No more tool calls found @@ -289,10 +287,7 @@ function parseToolCalls(toolCallsText: string): ToolCall[] { break // Malformed XML, no closing tag found } - const toolCallContent = remainingText.slice( - startIndex, - endIndex + endTag.length, - ) + const toolCallContent = remainingText.slice(startIndex, endIndex + endTag.length) remainingText = remainingText.slice(endIndex + endTag.length).trim() const toolCall = parseToolCall(toolMatch, toolCallContent) @@ -308,9 +303,7 @@ function parseToolCall(toolName: string, content: string): ToolCall | null { const tool_input: Record = {} // Remove the outer tool tags - const innerContent = content - .replace(new RegExp(`^<${toolName}>|$`, "g"), "") - .trim() + const innerContent = content.replace(new RegExp(`^<${toolName}>|$`, "g"), "").trim() // Parse nested XML elements const paramRegex = /<(\w+)>([\s\S]*?)<\/\1>/gs @@ -331,10 +324,7 @@ function parseToolCall(toolName: string, content: string): ToolCall | null { return { tool: toolName, tool_input } } -function validateToolInput( - toolName: string, - tool_input: Record, -): boolean { +function validateToolInput(toolName: string, tool_input: Record): boolean { switch (toolName) { case "execute_command": return "command" in tool_input @@ -376,9 +366,7 @@ export function convertO1ResponseToAnthropicMessage( completion: OpenAI.Chat.Completions.ChatCompletion, ): Anthropic.Messages.Message { const openAiMessage = completion.choices[0].message - const { normalText, toolCalls } = parseAIResponse( - openAiMessage.content || "", - ) + const { normalText, toolCalls } = parseAIResponse(openAiMessage.content || "") const anthropicMessage: Anthropic.Messages.Message = { id: completion.id, @@ -413,16 +401,14 @@ export function convertO1ResponseToAnthropicMessage( if (toolCalls.length > 0) { anthropicMessage.content.push( - ...toolCalls.map( - (toolCall: ToolCall, index: number): Anthropic.ToolUseBlock => { - return { - type: "tool_use", - id: `call_${index}_${Date.now()}`, // Generate a unique ID for each tool call - name: toolCall.tool, - input: toolCall.tool_input, - } - }, - ), + ...toolCalls.map((toolCall: ToolCall, index: number): Anthropic.ToolUseBlock => { + return { + type: "tool_use", + id: `call_${index}_${Date.now()}`, // Generate a unique ID for each tool call + name: toolCall.tool, + input: toolCall.tool_input, + } + }), ) } diff --git a/src/api/transform/openai-format.ts b/src/api/transform/openai-format.ts index 51ba165b58..0b134ad4ac 100644 --- a/src/api/transform/openai-format.ts +++ b/src/api/transform/openai-format.ts @@ -22,27 +22,20 @@ export function convertToOpenAiMessages( { role: "tool", tool_call_id: "", content: ""} */ if (anthropicMessage.role === "user") { - const { nonToolMessages, toolMessages } = - anthropicMessage.content.reduce<{ - nonToolMessages: ( - | Anthropic.TextBlockParam - | Anthropic.ImageBlockParam - )[] - toolMessages: Anthropic.ToolResultBlockParam[] - }>( - (acc, part) => { - if (part.type === "tool_result") { - acc.toolMessages.push(part) - } else if ( - part.type === "text" || - part.type === "image" - ) { - acc.nonToolMessages.push(part) - } // user cannot send tool_use messages - return acc - }, - { nonToolMessages: [], toolMessages: [] }, - ) + const { nonToolMessages, toolMessages } = anthropicMessage.content.reduce<{ + nonToolMessages: (Anthropic.TextBlockParam | Anthropic.ImageBlockParam)[] + toolMessages: Anthropic.ToolResultBlockParam[] + }>( + (acc, part) => { + if (part.type === "tool_result") { + acc.toolMessages.push(part) + } else if (part.type === "text" || part.type === "image") { + acc.nonToolMessages.push(part) + } // user cannot send tool_use messages + return acc + }, + { nonToolMessages: [], toolMessages: [] }, + ) // Process tool result messages FIRST since they must follow the tool use messages let toolResultImages: Anthropic.Messages.ImageBlockParam[] = [] @@ -105,27 +98,20 @@ export function convertToOpenAiMessages( }) } } else if (anthropicMessage.role === "assistant") { - const { nonToolMessages, toolMessages } = - anthropicMessage.content.reduce<{ - nonToolMessages: ( - | Anthropic.TextBlockParam - | Anthropic.ImageBlockParam - )[] - toolMessages: Anthropic.ToolUseBlockParam[] - }>( - (acc, part) => { - if (part.type === "tool_use") { - acc.toolMessages.push(part) - } else if ( - part.type === "text" || - part.type === "image" - ) { - acc.nonToolMessages.push(part) - } // assistant cannot send tool_result messages - return acc - }, - { nonToolMessages: [], toolMessages: [] }, - ) + const { nonToolMessages, toolMessages } = anthropicMessage.content.reduce<{ + nonToolMessages: (Anthropic.TextBlockParam | Anthropic.ImageBlockParam)[] + toolMessages: Anthropic.ToolUseBlockParam[] + }>( + (acc, part) => { + if (part.type === "tool_use") { + acc.toolMessages.push(part) + } else if (part.type === "text" || part.type === "image") { + acc.nonToolMessages.push(part) + } // assistant cannot send tool_result messages + return acc + }, + { nonToolMessages: [], toolMessages: [] }, + ) // Process non-tool messages let content: string | undefined @@ -141,16 +127,15 @@ export function convertToOpenAiMessages( } // Process tool use messages - let tool_calls: OpenAI.Chat.ChatCompletionMessageToolCall[] = - toolMessages.map((toolMessage) => ({ - id: toolMessage.id, - type: "function", - function: { - name: toolMessage.name, - // json string - arguments: JSON.stringify(toolMessage.input), - }, - })) + let tool_calls: OpenAI.Chat.ChatCompletionMessageToolCall[] = toolMessages.map((toolMessage) => ({ + id: toolMessage.id, + type: "function", + function: { + name: toolMessage.name, + // json string + arguments: JSON.stringify(toolMessage.input), + }, + })) openAiMessages.push({ role: "assistant", @@ -166,9 +151,7 @@ export function convertToOpenAiMessages( } // Convert OpenAI response to Anthropic format -export function convertToAnthropicMessage( - completion: OpenAI.Chat.Completions.ChatCompletion, -): Anthropic.Messages.Message { +export function convertToAnthropicMessage(completion: OpenAI.Chat.Completions.ChatCompletion): Anthropic.Messages.Message { const openAiMessage = completion.choices[0].message const anthropicMessage: Anthropic.Messages.Message = { id: completion.id, @@ -203,24 +186,20 @@ export function convertToAnthropicMessage( if (openAiMessage.tool_calls && openAiMessage.tool_calls.length > 0) { anthropicMessage.content.push( - ...openAiMessage.tool_calls.map( - (toolCall): Anthropic.ToolUseBlock => { - let parsedInput = {} - try { - parsedInput = JSON.parse( - toolCall.function.arguments || "{}", - ) - } catch (error) { - console.error("Failed to parse tool arguments:", error) - } - return { - type: "tool_use", - id: toolCall.id, - name: toolCall.function.name, - input: parsedInput, - } - }, - ), + ...openAiMessage.tool_calls.map((toolCall): Anthropic.ToolUseBlock => { + let parsedInput = {} + try { + parsedInput = JSON.parse(toolCall.function.arguments || "{}") + } catch (error) { + console.error("Failed to parse tool arguments:", error) + } + return { + type: "tool_use", + id: toolCall.id, + name: toolCall.function.name, + input: parsedInput, + } + }), ) } return anthropicMessage diff --git a/src/core/Cline.ts b/src/core/Cline.ts index de94b62f89..e3f46e93a1 100644 --- a/src/core/Cline.ts +++ b/src/core/Cline.ts @@ -9,14 +9,8 @@ import { serializeError } from "serialize-error" import * as vscode from "vscode" import { ApiHandler, buildApiHandler } from "../api" import { ApiStream } from "../api/transform/stream" -import { - DIFF_VIEW_URI_SCHEME, - DiffViewProvider, -} from "../integrations/editor/DiffViewProvider" -import { - findToolName, - formatContentBlockToMarkdown, -} from "../integrations/misc/export-markdown" +import { DIFF_VIEW_URI_SCHEME, DiffViewProvider } from "../integrations/editor/DiffViewProvider" +import { findToolName, formatContentBlockToMarkdown } from "../integrations/misc/export-markdown" import { extractTextFromFile } from "../integrations/misc/extract-text" import { TerminalManager } from "../integrations/terminal/TerminalManager" import { BrowserSession } from "../services/browser/BrowserSession" @@ -28,10 +22,7 @@ import { ApiConfiguration } from "../shared/api" import { findLast, findLastIndex } from "../shared/array" import { AutoApprovalSettings } from "../shared/AutoApprovalSettings" import { combineApiRequests } from "../shared/combineApiRequests" -import { - combineCommandSequences, - COMMAND_REQ_APP_STRING, -} from "../shared/combineCommandSequences" +import { combineCommandSequences, COMMAND_REQ_APP_STRING } from "../shared/combineCommandSequences" import { BrowserAction, BrowserActionResult, @@ -48,19 +39,11 @@ import { } from "../shared/ExtensionMessage" import { getApiMetrics } from "../shared/getApiMetrics" import { HistoryItem } from "../shared/HistoryItem" -import { - ClineAskResponse, - ClineCheckpointRestore, -} from "../shared/WebviewMessage" +import { ClineAskResponse, ClineCheckpointRestore } from "../shared/WebviewMessage" import { calculateApiCost } from "../utils/cost" import { fileExistsAtPath } from "../utils/fs" import { arePathsEqual, getReadablePath } from "../utils/path" -import { - AssistantMessageContent, - parseAssistantMessage, - ToolParamName, - ToolUseName, -} from "./assistant-message" +import { AssistantMessageContent, parseAssistantMessage, ToolParamName, ToolUseName } from "./assistant-message" import { constructNewFileContent } from "./assistant-message/diff" import { parseMentions } from "./mentions" import { formatResponse } from "./prompts/responses" @@ -74,19 +57,11 @@ import { OpenAiHandler } from "../api/providers/openai" import CheckpointTracker from "../integrations/checkpoints/CheckpointTracker" import getFolderSize from "get-folder-size" -const cwd = - vscode.workspace.workspaceFolders - ?.map((folder) => folder.uri.fsPath) - .at(0) ?? path.join(os.homedir(), "Desktop") // may or may not exist but fs checking existence would immediately ask for permission which would be bad UX, need to come up with a better solution +const cwd = vscode.workspace.workspaceFolders?.map((folder) => folder.uri.fsPath).at(0) ?? path.join(os.homedir(), "Desktop") // may or may not exist but fs checking existence would immediately ask for permission which would be bad UX, need to come up with a better solution -type ToolResponse = - | string - | Array +type ToolResponse = string | Array type UserContent = Array< - | Anthropic.TextBlockParam - | Anthropic.ImageBlockParam - | Anthropic.ToolUseBlockParam - | Anthropic.ToolResultBlockParam + Anthropic.TextBlockParam | Anthropic.ImageBlockParam | Anthropic.ToolUseBlockParam | Anthropic.ToolResultBlockParam > export class Cline { @@ -122,10 +97,7 @@ export class Cline { private assistantMessageContent: AssistantMessageContent[] = [] private presentAssistantMessageLocked = false private presentAssistantMessageHasPendingUpdates = false - private userMessageContent: ( - | Anthropic.TextBlockParam - | Anthropic.ImageBlockParam - )[] = [] + private userMessageContent: (Anthropic.TextBlockParam | Anthropic.ImageBlockParam)[] = [] private userMessageContentReady = false private didRejectTool = false private didAlreadyUseTool = false @@ -150,24 +122,20 @@ export class Cline { this.autoApprovalSettings = autoApprovalSettings if (historyItem) { this.taskId = historyItem.id - this.conversationHistoryDeletedRange = - historyItem.conversationHistoryDeletedRange + this.conversationHistoryDeletedRange = historyItem.conversationHistoryDeletedRange this.resumeTaskFromHistory() } else if (task || images) { this.taskId = Date.now().toString() this.startTask(task, images) } else { - throw new Error( - "Either historyItem or task/images must be provided", - ) + throw new Error("Either historyItem or task/images must be provided") } } // Storing task to disk for history private async ensureTaskDirectoryExists(): Promise { - const globalStoragePath = - this.providerRef.deref()?.context.globalStorageUri.fsPath + const globalStoragePath = this.providerRef.deref()?.context.globalStorageUri.fsPath if (!globalStoragePath) { throw new Error("Global storage uri is invalid") } @@ -176,13 +144,8 @@ export class Cline { return taskDir } - private async getSavedApiConversationHistory(): Promise< - Anthropic.MessageParam[] - > { - const filePath = path.join( - await this.ensureTaskDirectoryExists(), - GlobalFileNames.apiConversationHistory, - ) + private async getSavedApiConversationHistory(): Promise { + const filePath = path.join(await this.ensureTaskDirectoryExists(), GlobalFileNames.apiConversationHistory) const fileExists = await fileExistsAtPath(filePath) if (fileExists) { return JSON.parse(await fs.readFile(filePath, "utf8")) @@ -195,23 +158,15 @@ export class Cline { await this.saveApiConversationHistory() } - private async overwriteApiConversationHistory( - newHistory: Anthropic.MessageParam[], - ) { + private async overwriteApiConversationHistory(newHistory: Anthropic.MessageParam[]) { this.apiConversationHistory = newHistory await this.saveApiConversationHistory() } private async saveApiConversationHistory() { try { - const filePath = path.join( - await this.ensureTaskDirectoryExists(), - GlobalFileNames.apiConversationHistory, - ) - await fs.writeFile( - filePath, - JSON.stringify(this.apiConversationHistory), - ) + const filePath = path.join(await this.ensureTaskDirectoryExists(), GlobalFileNames.apiConversationHistory) + await fs.writeFile(filePath, JSON.stringify(this.apiConversationHistory)) } catch (error) { // in the off chance this fails, we don't want to stop the task console.error("Failed to save API conversation history:", error) @@ -219,18 +174,12 @@ export class Cline { } private async getSavedClineMessages(): Promise { - const filePath = path.join( - await this.ensureTaskDirectoryExists(), - GlobalFileNames.uiMessages, - ) + const filePath = path.join(await this.ensureTaskDirectoryExists(), GlobalFileNames.uiMessages) if (await fileExistsAtPath(filePath)) { return JSON.parse(await fs.readFile(filePath, "utf8")) } else { // check old location - const oldPath = path.join( - await this.ensureTaskDirectoryExists(), - "claude_messages.json", - ) + const oldPath = path.join(await this.ensureTaskDirectoryExists(), "claude_messages.json") if (await fileExistsAtPath(oldPath)) { const data = JSON.parse(await fs.readFile(oldPath, "utf8")) await fs.unlink(oldPath) // remove old file @@ -243,10 +192,8 @@ export class Cline { private async addToClineMessages(message: ClineMessage) { // these values allow us to reconstruct the conversation history at the time this cline message was created // it's important that apiConversationHistory is initialized before we add cline messages - message.conversationHistoryIndex = - this.apiConversationHistory.length - 1 // NOTE: this is the index of the last added message which is the user message, and once the clinemessages have been presented we update the apiconversationhistory with the completed assistant message. This means when reseting to a message, we need to +1 this index to get the correct assistant message that this tool use corresponds to - message.conversationHistoryDeletedRange = - this.conversationHistoryDeletedRange + message.conversationHistoryIndex = this.apiConversationHistory.length - 1 // NOTE: this is the index of the last added message which is the user message, and once the clinemessages have been presented we update the apiconversationhistory with the completed assistant message. This means when reseting to a message, we need to +1 this index to get the correct assistant message that this tool use corresponds to + message.conversationHistoryDeletedRange = this.conversationHistoryDeletedRange this.clineMessages.push(message) await this.saveClineMessages() } @@ -262,22 +209,11 @@ export class Cline { const filePath = path.join(taskDir, GlobalFileNames.uiMessages) await fs.writeFile(filePath, JSON.stringify(this.clineMessages)) // combined as they are in ChatView - const apiMetrics = getApiMetrics( - combineApiRequests( - combineCommandSequences(this.clineMessages.slice(1)), - ), - ) + const apiMetrics = getApiMetrics(combineApiRequests(combineCommandSequences(this.clineMessages.slice(1)))) const taskMessage = this.clineMessages[0] // first message is always the task say const lastRelevantMessage = this.clineMessages[ - findLastIndex( - this.clineMessages, - (m) => - !( - m.ask === "resume_task" || - m.ask === "resume_completed_task" - ), - ) + findLastIndex(this.clineMessages, (m) => !(m.ask === "resume_task" || m.ask === "resume_completed_task")) ] let taskDirSize = 0 try { @@ -285,11 +221,7 @@ export class Cline { // returns # of bytes, size/1000/1000 = MB taskDirSize = await getFolderSize.loose(taskDir) } catch (error) { - console.error( - "Failed to get task directory size:", - taskDir, - error, - ) + console.error("Failed to get task directory size:", taskDir, error) } await this.providerRef.deref()?.updateTaskHistory({ id: this.taskId, @@ -301,23 +233,16 @@ export class Cline { cacheReads: apiMetrics.totalCacheReads, totalCost: apiMetrics.totalCost, size: taskDirSize, - shadowGitConfigWorkTree: - await this.checkpointTracker?.getShadowGitConfigWorkTree(), - conversationHistoryDeletedRange: - this.conversationHistoryDeletedRange, + shadowGitConfigWorkTree: await this.checkpointTracker?.getShadowGitConfigWorkTree(), + conversationHistoryDeletedRange: this.conversationHistoryDeletedRange, }) } catch (error) { console.error("Failed to save cline messages:", error) } } - async restoreCheckpoint( - messageTs: number, - restoreType: ClineCheckpointRestore, - ) { - const messageIndex = this.clineMessages.findIndex( - (m) => m.ts === messageTs, - ) + async restoreCheckpoint(messageTs: number, restoreType: ClineCheckpointRestore) { + const messageIndex = this.clineMessages.findIndex((m) => m.ts === messageTs) const message = this.clineMessages[messageIndex] if (!message) { console.error("Message not found", this.clineMessages) @@ -333,20 +258,11 @@ export class Cline { case "workspace": if (!this.checkpointTracker) { try { - this.checkpointTracker = await CheckpointTracker.create( - this.taskId, - this.providerRef.deref(), - ) + this.checkpointTracker = await CheckpointTracker.create(this.taskId, this.providerRef.deref()) this.checkpointTrackerErrorMessage = undefined } catch (error) { - const errorMessage = - error instanceof Error - ? error.message - : "Unknown error" - console.error( - "Failed to initialize checkpoint tracker:", - errorMessage, - ) + const errorMessage = error instanceof Error ? error.message : "Unknown error" + console.error("Failed to initialize checkpoint tracker:", errorMessage) this.checkpointTrackerErrorMessage = errorMessage await this.providerRef.deref()?.postStateToWebview() vscode.window.showErrorMessage(errorMessage) @@ -355,17 +271,10 @@ export class Cline { } if (message.lastCheckpointHash && this.checkpointTracker) { try { - await this.checkpointTracker.resetHead( - message.lastCheckpointHash, - ) + await this.checkpointTracker.resetHead(message.lastCheckpointHash) } catch (error) { - const errorMessage = - error instanceof Error - ? error.message - : "Unknown error" - vscode.window.showErrorMessage( - "Failed to restore checkpoint: " + errorMessage, - ) + const errorMessage = error instanceof Error ? error.message : "Unknown error" + vscode.window.showErrorMessage("Failed to restore checkpoint: " + errorMessage) didWorkspaceRestoreFail = true } } @@ -376,31 +285,18 @@ export class Cline { switch (restoreType) { case "task": case "taskAndWorkspace": - this.conversationHistoryDeletedRange = - message.conversationHistoryDeletedRange - const newConversationHistory = - this.apiConversationHistory.slice( - 0, - (message.conversationHistoryIndex || 0) + 2, - ) // +1 since this index corresponds to the last user message, and another +1 since slice end index is exclusive - await this.overwriteApiConversationHistory( - newConversationHistory, - ) + this.conversationHistoryDeletedRange = message.conversationHistoryDeletedRange + const newConversationHistory = this.apiConversationHistory.slice( + 0, + (message.conversationHistoryIndex || 0) + 2, + ) // +1 since this index corresponds to the last user message, and another +1 since slice end index is exclusive + await this.overwriteApiConversationHistory(newConversationHistory) // aggregate deleted api reqs info so we don't lose costs/tokens - const deletedMessages = this.clineMessages.slice( - messageIndex + 1, - ) - const deletedApiReqsMetrics = getApiMetrics( - combineApiRequests( - combineCommandSequences(deletedMessages), - ), - ) + const deletedMessages = this.clineMessages.slice(messageIndex + 1) + const deletedApiReqsMetrics = getApiMetrics(combineApiRequests(combineCommandSequences(deletedMessages))) - const newClineMessages = this.clineMessages.slice( - 0, - messageIndex + 1, - ) + const newClineMessages = this.clineMessages.slice(0, messageIndex + 1) await this.overwriteClineMessages(newClineMessages) // calls saveClineMessages which saves historyItem await this.say( @@ -420,48 +316,31 @@ export class Cline { switch (restoreType) { case "task": - vscode.window.showInformationMessage( - "Task messages have been restored to the checkpoint", - ) + vscode.window.showInformationMessage("Task messages have been restored to the checkpoint") break case "workspace": - vscode.window.showInformationMessage( - "Workspace files have been restored to the checkpoint", - ) + vscode.window.showInformationMessage("Workspace files have been restored to the checkpoint") break case "taskAndWorkspace": - vscode.window.showInformationMessage( - "Task and workspace have been restored to the checkpoint", - ) + vscode.window.showInformationMessage("Task and workspace have been restored to the checkpoint") break } - await this.providerRef - .deref() - ?.postMessageToWebview({ type: "relinquishControl" }) + await this.providerRef.deref()?.postMessageToWebview({ type: "relinquishControl" }) this.providerRef.deref()?.cancelTask() // the task is already cancelled by the provider beforehand, but we need to re-init to get the updated messages } else { - await this.providerRef - .deref() - ?.postMessageToWebview({ type: "relinquishControl" }) + await this.providerRef.deref()?.postMessageToWebview({ type: "relinquishControl" }) } } - async presentMultifileDiff( - messageTs: number, - seeNewChangesSinceLastTaskCompletion: boolean, - ) { + async presentMultifileDiff(messageTs: number, seeNewChangesSinceLastTaskCompletion: boolean) { const relinquishButton = () => { - this.providerRef - .deref() - ?.postMessageToWebview({ type: "relinquishControl" }) + this.providerRef.deref()?.postMessageToWebview({ type: "relinquishControl" }) } console.log("presentMultifileDiff", messageTs) - const messageIndex = this.clineMessages.findIndex( - (m) => m.ts === messageTs, - ) + const messageIndex = this.clineMessages.findIndex((m) => m.ts === messageTs) const message = this.clineMessages[messageIndex] if (!message) { console.error("Message not found") @@ -478,18 +357,11 @@ export class Cline { // TODO: handle if this is called from outside original workspace, in which case we need to show user error message we cant show diff outside of workspace? if (!this.checkpointTracker) { try { - this.checkpointTracker = await CheckpointTracker.create( - this.taskId, - this.providerRef.deref(), - ) + this.checkpointTracker = await CheckpointTracker.create(this.taskId, this.providerRef.deref()) this.checkpointTrackerErrorMessage = undefined } catch (error) { - const errorMessage = - error instanceof Error ? error.message : "Unknown error" - console.error( - "Failed to initialize checkpoint tracker:", - errorMessage, - ) + const errorMessage = error instanceof Error ? error.message : "Unknown error" + console.error("Failed to initialize checkpoint tracker:", errorMessage) this.checkpointTrackerErrorMessage = errorMessage await this.providerRef.deref()?.postStateToWebview() vscode.window.showErrorMessage(errorMessage) @@ -540,11 +412,8 @@ export class Cline { } } } catch (error) { - const errorMessage = - error instanceof Error ? error.message : "Unknown error" - vscode.window.showErrorMessage( - "Failed to retrieve diff set: " + errorMessage, - ) + const errorMessage = error instanceof Error ? error.message : "Unknown error" + vscode.window.showErrorMessage("Failed to retrieve diff set: " + errorMessage) relinquishButton() return } @@ -563,19 +432,13 @@ export class Cline { // Open multi-diff editor await vscode.commands.executeCommand( "vscode.changes", - seeNewChangesSinceLastTaskCompletion - ? "New changes" - : "Changes since snapshot", + seeNewChangesSinceLastTaskCompletion ? "New changes" : "Changes since snapshot", changedFiles.map((file) => [ vscode.Uri.file(file.absolutePath), - vscode.Uri.parse( - `${DIFF_VIEW_URI_SCHEME}:${file.relativePath}`, - ).with({ + vscode.Uri.parse(`${DIFF_VIEW_URI_SCHEME}:${file.relativePath}`).with({ query: Buffer.from(file.before ?? "").toString("base64"), }), - vscode.Uri.parse( - `${DIFF_VIEW_URI_SCHEME}:${file.relativePath}`, - ).with({ + vscode.Uri.parse(`${DIFF_VIEW_URI_SCHEME}:${file.relativePath}`).with({ query: Buffer.from(file.after ?? "").toString("base64"), }), ]), @@ -584,10 +447,7 @@ export class Cline { } async doesLatestTaskCompletionHaveNewChanges() { - const messageIndex = findLastIndex( - this.clineMessages, - (m) => m.say === "completion_result", - ) + const messageIndex = findLastIndex(this.clineMessages, (m) => m.say === "completion_result") const message = this.clineMessages[messageIndex] if (!message) { console.error("Completion message not found") @@ -601,27 +461,17 @@ export class Cline { if (!this.checkpointTracker) { try { - this.checkpointTracker = await CheckpointTracker.create( - this.taskId, - this.providerRef.deref(), - ) + this.checkpointTracker = await CheckpointTracker.create(this.taskId, this.providerRef.deref()) this.checkpointTrackerErrorMessage = undefined } catch (error) { - const errorMessage = - error instanceof Error ? error.message : "Unknown error" - console.error( - "Failed to initialize checkpoint tracker:", - errorMessage, - ) + const errorMessage = error instanceof Error ? error.message : "Unknown error" + console.error("Failed to initialize checkpoint tracker:", errorMessage) return false } } // Get last task completed - const lastTaskCompletedMessage = findLast( - this.clineMessages.slice(0, messageIndex), - (m) => m.say === "completion_result", - ) + const lastTaskCompletedMessage = findLast(this.clineMessages.slice(0, messageIndex), (m) => m.say === "completion_result") try { // Get changed files between current state and commit @@ -661,10 +511,7 @@ export class Cline { if (partial !== undefined) { const lastMessage = this.clineMessages.at(-1) const isUpdatingPreviousPartial = - lastMessage && - lastMessage.partial && - lastMessage.type === "ask" && - lastMessage.ask === type + lastMessage && lastMessage.partial && lastMessage.type === "ask" && lastMessage.ask === type if (partial) { if (isUpdatingPreviousPartial) { // existing partial message, so update it @@ -673,12 +520,10 @@ export class Cline { // todo be more efficient about saving and posting only new data or one whole message at a time so ignore partial for saves, and only post parts of partial message instead of whole array in new listener // await this.saveClineMessages() // await this.providerRef.deref()?.postStateToWebview() - await this.providerRef - .deref() - ?.postMessageToWebview({ - type: "partialMessage", - partialMessage: lastMessage, - }) + await this.providerRef.deref()?.postMessageToWebview({ + type: "partialMessage", + partialMessage: lastMessage, + }) throw new Error("Current ask promise was ignored 1") } else { // this is a new partial message, so add it with partial state @@ -718,12 +563,10 @@ export class Cline { lastMessage.partial = false await this.saveClineMessages() // await this.providerRef.deref()?.postStateToWebview() - await this.providerRef - .deref() - ?.postMessageToWebview({ - type: "partialMessage", - partialMessage: lastMessage, - }) + await this.providerRef.deref()?.postMessageToWebview({ + type: "partialMessage", + partialMessage: lastMessage, + }) } else { // this is a new partial=false message, so add it like normal this.askResponse = undefined @@ -757,11 +600,7 @@ export class Cline { await this.providerRef.deref()?.postStateToWebview() } - await pWaitFor( - () => - this.askResponse !== undefined || this.lastMessageTs !== askTs, - { interval: 100 }, - ) + await pWaitFor(() => this.askResponse !== undefined || this.lastMessageTs !== askTs, { interval: 100 }) if (this.lastMessageTs !== askTs) { throw new Error("Current ask promise was ignored") // could happen if we send multiple asks in a row i.e. with command_output. It's important that when we know an ask could fail, it is handled gracefully } @@ -776,22 +615,13 @@ export class Cline { return result } - async handleWebviewAskResponse( - askResponse: ClineAskResponse, - text?: string, - images?: string[], - ) { + async handleWebviewAskResponse(askResponse: ClineAskResponse, text?: string, images?: string[]) { this.askResponse = askResponse this.askResponseText = text this.askResponseImages = images } - async say( - type: ClineSay, - text?: string, - images?: string[], - partial?: boolean, - ): Promise { + async say(type: ClineSay, text?: string, images?: string[], partial?: boolean): Promise { if (this.abort) { throw new Error("Cline instance aborted") } @@ -799,22 +629,17 @@ export class Cline { if (partial !== undefined) { const lastMessage = this.clineMessages.at(-1) const isUpdatingPreviousPartial = - lastMessage && - lastMessage.partial && - lastMessage.type === "say" && - lastMessage.say === type + lastMessage && lastMessage.partial && lastMessage.type === "say" && lastMessage.say === type if (partial) { if (isUpdatingPreviousPartial) { // existing partial message, so update it lastMessage.text = text lastMessage.images = images lastMessage.partial = partial - await this.providerRef - .deref() - ?.postMessageToWebview({ - type: "partialMessage", - partialMessage: lastMessage, - }) + await this.providerRef.deref()?.postMessageToWebview({ + type: "partialMessage", + partialMessage: lastMessage, + }) } else { // this is a new partial message, so add it with partial state const sayTs = Date.now() @@ -842,12 +667,10 @@ export class Cline { // instead of streaming partialMessage events, we do a save and post like normal to persist to disk await this.saveClineMessages() // await this.providerRef.deref()?.postStateToWebview() - await this.providerRef - .deref() - ?.postMessageToWebview({ - type: "partialMessage", - partialMessage: lastMessage, - }) // more performant than an entire postStateToWebview + await this.providerRef.deref()?.postMessageToWebview({ + type: "partialMessage", + partialMessage: lastMessage, + }) // more performant than an entire postStateToWebview } else { // this is a new partial=false message, so add it like normal const sayTs = Date.now() @@ -877,32 +700,19 @@ export class Cline { } } - async sayAndCreateMissingParamError( - toolName: ToolUseName, - paramName: string, - relPath?: string, - ) { + async sayAndCreateMissingParamError(toolName: ToolUseName, paramName: string, relPath?: string) { await this.say( "error", `Cline tried to use ${toolName}${ relPath ? ` for '${relPath.toPosix()}'` : "" } without value for required parameter '${paramName}'. Retrying...`, ) - return formatResponse.toolError( - formatResponse.missingToolParameterError(paramName), - ) + return formatResponse.toolError(formatResponse.missingToolParameterError(paramName)) } - async removeLastPartialMessageIfExistsWithType( - type: "ask" | "say", - askOrSay: ClineAsk | ClineSay, - ) { + async removeLastPartialMessageIfExistsWithType(type: "ask" | "say", askOrSay: ClineAsk | ClineSay) { const lastMessage = this.clineMessages.at(-1) - if ( - lastMessage?.partial && - lastMessage.type === type && - (lastMessage.ask === askOrSay || lastMessage.say === askOrSay) - ) { + if (lastMessage?.partial && lastMessage.type === type && (lastMessage.ask === askOrSay || lastMessage.say === askOrSay)) { this.clineMessages.pop() await this.saveClineMessages() await this.providerRef.deref()?.postStateToWebview() @@ -922,8 +732,7 @@ export class Cline { this.isInitialized = true - let imageBlocks: Anthropic.ImageBlockParam[] = - formatResponse.imageBlocks(images) + let imageBlocks: Anthropic.ImageBlockParam[] = formatResponse.imageBlocks(images) await this.initiateTaskLoop( [ { @@ -948,8 +757,7 @@ export class Cline { // Remove any resume messages that may have been added before const lastRelevantMessageIndex = findLastIndex( modifiedClineMessages, - (m) => - !(m.ask === "resume_task" || m.ask === "resume_completed_task"), + (m) => !(m.ask === "resume_task" || m.ask === "resume_completed_task"), ) if (lastRelevantMessageIndex !== -1) { modifiedClineMessages.splice(lastRelevantMessageIndex + 1) @@ -961,11 +769,8 @@ export class Cline { (m) => m.type === "say" && m.say === "api_req_started", ) if (lastApiReqStartedIndex !== -1) { - const lastApiReqStarted = - modifiedClineMessages[lastApiReqStartedIndex] - const { cost, cancelReason }: ClineApiReqInfo = JSON.parse( - lastApiReqStarted.text || "{}", - ) + const lastApiReqStarted = modifiedClineMessages[lastApiReqStartedIndex] + const { cost, cancelReason }: ClineApiReqInfo = JSON.parse(lastApiReqStarted.text || "{}") if (cost === undefined && cancelReason === undefined) { modifiedClineMessages.splice(lastApiReqStartedIndex, 1) } @@ -976,19 +781,12 @@ export class Cline { // Now present the cline messages to the user and ask if they want to resume (NOTE: we ran into a bug before where the apiconversationhistory wouldnt be initialized when opening a old task, and it was because we were waiting for resume) // This is important in case the user deletes messages without resuming the task first - this.apiConversationHistory = - await this.getSavedApiConversationHistory() + this.apiConversationHistory = await this.getSavedApiConversationHistory() const lastClineMessage = this.clineMessages .slice() .reverse() - .find( - (m) => - !( - m.ask === "resume_task" || - m.ask === "resume_completed_task" - ), - ) // could be multiple resume tasks + .find((m) => !(m.ask === "resume_task" || m.ask === "resume_completed_task")) // could be multiple resume tasks // const lastClineMessage = this.clineMessages[lastClineMessageIndex] // could be a completion result with a command // const secondLastClineMessage = this.clineMessages @@ -1020,55 +818,39 @@ export class Cline { // need to make sure that the api conversation history can be resumed by the api, even if it goes out of sync with cline messages - let existingApiConversationHistory: Anthropic.Messages.MessageParam[] = - await this.getSavedApiConversationHistory() + let existingApiConversationHistory: Anthropic.Messages.MessageParam[] = await this.getSavedApiConversationHistory() // v2.0 xml tags refactor caveat: since we don't use tools anymore, we need to replace all tool use blocks with a text block since the API disallows conversations with tool uses and no tool schema - const conversationWithoutToolBlocks = - existingApiConversationHistory.map((message) => { - if (Array.isArray(message.content)) { - const newContent = message.content.map((block) => { - if (block.type === "tool_use") { - // it's important we convert to the new tool schema format so the model doesn't get confused about how to invoke tools - const inputAsXml = Object.entries( - block.input as Record, - ) - .map( - ([key, value]) => - `<${key}>\n${value}\n`, - ) - .join("\n") - return { - type: "text", - text: `<${block.name}>\n${inputAsXml}\n`, - } as Anthropic.Messages.TextBlockParam - } else if (block.type === "tool_result") { - // Convert block.content to text block array, removing images - const contentAsTextBlocks = Array.isArray( - block.content, - ) - ? block.content.filter( - (item) => item.type === "text", - ) - : [{ type: "text", text: block.content }] - const textContent = contentAsTextBlocks - .map((item) => item.text) - .join("\n\n") - const toolName = findToolName( - block.tool_use_id, - existingApiConversationHistory, - ) - return { - type: "text", - text: `[${toolName} Result]\n\n${textContent}`, - } as Anthropic.Messages.TextBlockParam - } - return block - }) - return { ...message, content: newContent } - } - return message - }) + const conversationWithoutToolBlocks = existingApiConversationHistory.map((message) => { + if (Array.isArray(message.content)) { + const newContent = message.content.map((block) => { + if (block.type === "tool_use") { + // it's important we convert to the new tool schema format so the model doesn't get confused about how to invoke tools + const inputAsXml = Object.entries(block.input as Record) + .map(([key, value]) => `<${key}>\n${value}\n`) + .join("\n") + return { + type: "text", + text: `<${block.name}>\n${inputAsXml}\n`, + } as Anthropic.Messages.TextBlockParam + } else if (block.type === "tool_result") { + // Convert block.content to text block array, removing images + const contentAsTextBlocks = Array.isArray(block.content) + ? block.content.filter((item) => item.type === "text") + : [{ type: "text", text: block.content }] + const textContent = contentAsTextBlocks.map((item) => item.text).join("\n\n") + const toolName = findToolName(block.tool_use_id, existingApiConversationHistory) + return { + type: "text", + text: `[${toolName} Result]\n\n${textContent}`, + } as Anthropic.Messages.TextBlockParam + } + return block + }) + return { ...message, content: newContent } + } + return message + }) existingApiConversationHistory = conversationWithoutToolBlocks // FIXME: remove tool use blocks altogether @@ -1082,60 +864,38 @@ export class Cline { let modifiedOldUserContent: UserContent // either the last message if its user message, or the user message before the last (assistant) message let modifiedApiConversationHistory: Anthropic.Messages.MessageParam[] // need to remove the last user message to replace with new modified user message if (existingApiConversationHistory.length > 0) { - const lastMessage = - existingApiConversationHistory[ - existingApiConversationHistory.length - 1 - ] + const lastMessage = existingApiConversationHistory[existingApiConversationHistory.length - 1] if (lastMessage.role === "assistant") { const content = Array.isArray(lastMessage.content) ? lastMessage.content : [{ type: "text", text: lastMessage.content }] - const hasToolUse = content.some( - (block) => block.type === "tool_use", - ) + const hasToolUse = content.some((block) => block.type === "tool_use") if (hasToolUse) { const toolUseBlocks = content.filter( (block) => block.type === "tool_use", ) as Anthropic.Messages.ToolUseBlock[] - const toolResponses: Anthropic.ToolResultBlockParam[] = - toolUseBlocks.map((block) => ({ - type: "tool_result", - tool_use_id: block.id, - content: - "Task was interrupted before this tool call could be completed.", - })) - modifiedApiConversationHistory = [ - ...existingApiConversationHistory, - ] // no changes + const toolResponses: Anthropic.ToolResultBlockParam[] = toolUseBlocks.map((block) => ({ + type: "tool_result", + tool_use_id: block.id, + content: "Task was interrupted before this tool call could be completed.", + })) + modifiedApiConversationHistory = [...existingApiConversationHistory] // no changes modifiedOldUserContent = [...toolResponses] } else { - modifiedApiConversationHistory = [ - ...existingApiConversationHistory, - ] + modifiedApiConversationHistory = [...existingApiConversationHistory] modifiedOldUserContent = [] } } else if (lastMessage.role === "user") { - const previousAssistantMessage: - | Anthropic.Messages.MessageParam - | undefined = - existingApiConversationHistory[ - existingApiConversationHistory.length - 2 - ] - - const existingUserContent: UserContent = Array.isArray( - lastMessage.content, - ) + const previousAssistantMessage: Anthropic.Messages.MessageParam | undefined = + existingApiConversationHistory[existingApiConversationHistory.length - 2] + + const existingUserContent: UserContent = Array.isArray(lastMessage.content) ? lastMessage.content : [{ type: "text", text: lastMessage.content }] - if ( - previousAssistantMessage && - previousAssistantMessage.role === "assistant" - ) { - const assistantContent = Array.isArray( - previousAssistantMessage.content, - ) + if (previousAssistantMessage && previousAssistantMessage.role === "assistant") { + const assistantContent = Array.isArray(previousAssistantMessage.content) ? previousAssistantMessage.content : [ { @@ -1153,43 +913,26 @@ export class Cline { (block) => block.type === "tool_result", ) as Anthropic.ToolResultBlockParam[] - const missingToolResponses: Anthropic.ToolResultBlockParam[] = - toolUseBlocks - .filter( - (toolUse) => - !existingToolResults.some( - (result) => - result.tool_use_id === - toolUse.id, - ), - ) - .map((toolUse) => ({ - type: "tool_result", - tool_use_id: toolUse.id, - content: - "Task was interrupted before this tool call could be completed.", - })) - - modifiedApiConversationHistory = - existingApiConversationHistory.slice(0, -1) // removes the last user message - modifiedOldUserContent = [ - ...existingUserContent, - ...missingToolResponses, - ] + const missingToolResponses: Anthropic.ToolResultBlockParam[] = toolUseBlocks + .filter((toolUse) => !existingToolResults.some((result) => result.tool_use_id === toolUse.id)) + .map((toolUse) => ({ + type: "tool_result", + tool_use_id: toolUse.id, + content: "Task was interrupted before this tool call could be completed.", + })) + + modifiedApiConversationHistory = existingApiConversationHistory.slice(0, -1) // removes the last user message + modifiedOldUserContent = [...existingUserContent, ...missingToolResponses] } else { - modifiedApiConversationHistory = - existingApiConversationHistory.slice(0, -1) + modifiedApiConversationHistory = existingApiConversationHistory.slice(0, -1) modifiedOldUserContent = [...existingUserContent] } } else { - modifiedApiConversationHistory = - existingApiConversationHistory.slice(0, -1) + modifiedApiConversationHistory = existingApiConversationHistory.slice(0, -1) modifiedOldUserContent = [...existingUserContent] } } else { - throw new Error( - "Unexpected: Last message is not a user or assistant message", - ) + throw new Error("Unexpected: Last message is not a user or assistant message") } } else { throw new Error("Unexpected: No existing API conversation history") @@ -1220,8 +963,7 @@ export class Cline { return "just now" })() - const wasRecent = - lastClineMessage?.ts && Date.now() - lastClineMessage.ts < 30_000 + const wasRecent = lastClineMessage?.ts && Date.now() - lastClineMessage.ts < 30_000 newUserContent.push({ type: "text", @@ -1240,24 +982,15 @@ export class Cline { newUserContent.push(...formatResponse.imageBlocks(responseImages)) } - await this.overwriteApiConversationHistory( - modifiedApiConversationHistory, - ) + await this.overwriteApiConversationHistory(modifiedApiConversationHistory) await this.initiateTaskLoop(newUserContent, false) } - private async initiateTaskLoop( - userContent: UserContent, - isNewTask: boolean, - ): Promise { + private async initiateTaskLoop(userContent: UserContent, isNewTask: boolean): Promise { let nextUserContent = userContent let includeFileDetails = true while (!this.abort) { - const didEndLoop = await this.recursivelyMakeClineRequests( - nextUserContent, - includeFileDetails, - isNewTask, - ) + const didEndLoop = await this.recursivelyMakeClineRequests(nextUserContent, includeFileDetails, isNewTask) includeFileDetails = false // we only need file details the first time // The way this agentic loop works is that cline will be given a task that he then calls tools to complete. unless there's an attempt_completion call, we keep responding back to him with his tool's responses until he either attempt_completion or does not use anymore tools. If he does not use anymore tools, we ask him to consider if he's completed the task and then call attempt_completion, otherwise proceed with completing the task. @@ -1333,9 +1066,7 @@ export class Cline { // Tools - async executeCommandTool( - command: string, - ): Promise<[boolean, ToolResponse]> { + async executeCommandTool(command: string): Promise<[boolean, ToolResponse]> { const terminalInfo = await this.terminalManager.getOrCreateTerminal(cwd) terminalInfo.terminal.show() // weird visual bug when creating new terminals (even manually) where there's an empty space at the top. const process = this.terminalManager.runCommand(terminalInfo, command) @@ -1344,10 +1075,7 @@ export class Cline { let didContinue = false const sendCommandOutput = async (line: string): Promise => { try { - const { response, text, images } = await this.ask( - "command_output", - line, - ) + const { response, text, images } = await this.ask("command_output", line) if (response === "yesButtonClicked") { // proceed while running } else { @@ -1391,18 +1119,12 @@ export class Cline { result = result.trim() if (userFeedback) { - await this.say( - "user_feedback", - userFeedback.text, - userFeedback.images, - ) + await this.say("user_feedback", userFeedback.text, userFeedback.images) return [ true, formatResponse.toolResult( `Command is still running in the user's terminal.${ - result.length > 0 - ? `\nHere's the output so far:\n${result}` - : "" + result.length > 0 ? `\nHere's the output so far:\n${result}` : "" }\n\nThe user provided the following feedback:\n\n${userFeedback.text}\n`, userFeedback.images, ), @@ -1410,17 +1132,12 @@ export class Cline { } if (completed) { - return [ - false, - `Command executed.${result.length > 0 ? `\nOutput:\n${result}` : ""}`, - ] + return [false, `Command executed.${result.length > 0 ? `\nOutput:\n${result}` : ""}`] } else { return [ false, `Command is still running in the user's terminal.${ - result.length > 0 - ? `\nHere's the output so far:\n${result}` - : "" + result.length > 0 ? `\nHere's the output so far:\n${result}` : "" }\n\nYou will be updated on the terminal status and new output in the future.`, ] } @@ -1451,10 +1168,7 @@ export class Cline { async *attemptApiRequest(previousApiReqIndex: number): ApiStream { // Wait for MCP servers to be connected before generating system prompt - await pWaitFor( - () => this.providerRef.deref()?.mcpHub?.isConnecting !== true, - { timeout: 10_000 }, - ).catch(() => { + await pWaitFor(() => this.providerRef.deref()?.mcpHub?.isConnecting !== true, { timeout: 10_000 }).catch(() => { console.error("MCP servers failed to connect in time") }) @@ -1463,59 +1177,35 @@ export class Cline { throw new Error("MCP hub not available") } - let systemPrompt = await SYSTEM_PROMPT( - cwd, - this.api.getModel().info.supportsComputerUse ?? false, - mcpHub, - ) + let systemPrompt = await SYSTEM_PROMPT(cwd, this.api.getModel().info.supportsComputerUse ?? false, mcpHub) let settingsCustomInstructions = this.customInstructions?.trim() const clineRulesFilePath = path.resolve(cwd, GlobalFileNames.clineRules) let clineRulesFileInstructions: string | undefined if (await fileExistsAtPath(clineRulesFilePath)) { try { - const ruleFileContent = ( - await fs.readFile(clineRulesFilePath, "utf8") - ).trim() + const ruleFileContent = (await fs.readFile(clineRulesFilePath, "utf8")).trim() if (ruleFileContent) { clineRulesFileInstructions = `# .clinerules\n\nThe following is provided by a root-level .clinerules file where the user has specified instructions for this working directory (${cwd.toPosix()})\n\n${ruleFileContent}` } } catch { - console.error( - `Failed to read .clinerules file at ${clineRulesFilePath}`, - ) + console.error(`Failed to read .clinerules file at ${clineRulesFilePath}`) } } if (settingsCustomInstructions || clineRulesFileInstructions) { // altering the system prompt mid-task will break the prompt cache, but in the grand scheme this will not change often so it's better to not pollute user messages with it the way we have to with - systemPrompt += addUserInstructions( - settingsCustomInstructions, - clineRulesFileInstructions, - ) + systemPrompt += addUserInstructions(settingsCustomInstructions, clineRulesFileInstructions) } // If the previous API request's total token usage is close to the context window, truncate the conversation history to free up space for the new request if (previousApiReqIndex >= 0) { const previousRequest = this.clineMessages[previousApiReqIndex] if (previousRequest && previousRequest.text) { - const { - tokensIn, - tokensOut, - cacheWrites, - cacheReads, - }: ClineApiReqInfo = JSON.parse(previousRequest.text) - const totalTokens = - (tokensIn || 0) + - (tokensOut || 0) + - (cacheWrites || 0) + - (cacheReads || 0) - let contextWindow = - this.api.getModel().info.contextWindow || 128_000 + const { tokensIn, tokensOut, cacheWrites, cacheReads }: ClineApiReqInfo = JSON.parse(previousRequest.text) + const totalTokens = (tokensIn || 0) + (tokensOut || 0) + (cacheWrites || 0) + (cacheReads || 0) + let contextWindow = this.api.getModel().info.contextWindow || 128_000 // FIXME: hack to get anyone using openai compatible with deepseek to have the proper context window instead of the default 128k. We need a way for the user to specify the context window for models they input through openai compatible - if ( - this.api instanceof OpenAiHandler && - this.api.getModel().id.toLowerCase().includes("deepseek") - ) { + if (this.api instanceof OpenAiHandler && this.api.getModel().id.toLowerCase().includes("deepseek")) { contextWindow = 64_000 } let maxAllowedSize: number @@ -1530,20 +1220,16 @@ export class Cline { maxAllowedSize = contextWindow - 40_000 break default: - maxAllowedSize = Math.max( - contextWindow - 40_000, - contextWindow * 0.8, - ) // for deepseek, 80% of 64k meant only ~10k buffer which was too small and resulted in users getting context window errors. + maxAllowedSize = Math.max(contextWindow - 40_000, contextWindow * 0.8) // for deepseek, 80% of 64k meant only ~10k buffer which was too small and resulted in users getting context window errors. } // This is the most reliable way to know when we're close to hitting the context window. if (totalTokens >= maxAllowedSize) { // NOTE: it's okay that we overwriteConversationHistory in resume task since we're only ever removing the last user message and not anything in the middle which would affect this range - this.conversationHistoryDeletedRange = - getNextTruncationRange( - this.apiConversationHistory, - this.conversationHistoryDeletedRange, - ) + this.conversationHistoryDeletedRange = getNextTruncationRange( + this.apiConversationHistory, + this.conversationHistoryDeletedRange, + ) await this.saveClineMessages() // saves task history item which we use to keep track of conversation history deleted range // await this.overwriteApiConversationHistory(truncatedMessages) } @@ -1556,10 +1242,7 @@ export class Cline { this.conversationHistoryDeletedRange, ) - const stream = this.api.createMessage( - systemPrompt, - truncatedConversationHistory, - ) + const stream = this.api.createMessage(systemPrompt, truncatedConversationHistory) const iterator = stream[Symbol.asyncIterator]() try { @@ -1568,10 +1251,7 @@ export class Cline { yield firstChunk.value } catch (error) { // note that this api_req_failed ask is unique in that we only present this option if the api hasn't streamed any content yet (ie it fails on the first chunk due), as it would allow them to hit a retry button. However if the api failed mid-stream, it could be in any arbitrary state where some tools may have executed, so that error is handled differently and requires cancelling the task entirely. - const { response } = await this.ask( - "api_req_failed", - error.message ?? JSON.stringify(serializeError(error), null, 2), - ) + const { response } = await this.ask("api_req_failed", error.message ?? JSON.stringify(serializeError(error), null, 2)) if (response !== "yesButtonClicked") { // this will never happen since if noButtonClicked, we will clear current task, aborting this instance throw new Error("API request failed") @@ -1600,10 +1280,7 @@ export class Cline { this.presentAssistantMessageLocked = true this.presentAssistantMessageHasPendingUpdates = false - if ( - this.currentStreamingContentIndex >= - this.assistantMessageContent.length - ) { + if (this.currentStreamingContentIndex >= this.assistantMessageContent.length) { // this may happen if the last content block was completed before streaming could finish. if streaming is finished, and we're out of bounds then this means we already presented/executed the last content block and are ready to continue to next request if (this.didCompleteReadingStream) { this.userMessageContentReady = true @@ -1614,9 +1291,7 @@ export class Cline { //throw new Error("No more content blocks to stream! This shouldn't happen...") // remove and just return after testing } - const block = cloneDeep( - this.assistantMessageContent[this.currentStreamingContentIndex], - ) // need to create copy bc while stream is updating the array, it could be updating the reference block properties too + const block = cloneDeep(this.assistantMessageContent[this.currentStreamingContentIndex]) // need to create copy bc while stream is updating the array, it could be updating the reference block properties too switch (block.type) { case "text": { if (this.didRejectTool || this.didAlreadyUseTool) { @@ -1650,17 +1325,12 @@ export class Cline { tagContent = possibleTag.slice(1).trim() } // Check if tagContent is likely an incomplete tag name (letters and underscores only) - const isLikelyTagName = /^[a-zA-Z_]+$/.test( - tagContent, - ) + const isLikelyTagName = /^[a-zA-Z_]+$/.test(tagContent) // Preemptively remove < or { - const { response, text, images } = await this.ask( - type, - partialMessage, - false, - ) + const askApproval = async (type: ClineAsk, partialMessage?: string) => { + const { response, text, images } = await this.ask(type, partialMessage, false) if (response !== "yesButtonClicked") { if (response === "messageResponse") { await this.say("user_feedback", text, images) - pushToolResult( - formatResponse.toolResult( - formatResponse.toolDeniedWithFeedback(text), - images, - ), - ) + pushToolResult(formatResponse.toolResult(formatResponse.toolDeniedWithFeedback(text), images)) // this.userMessageContent.push({ // type: "text", // text: `${toolDescription()}`, @@ -1801,13 +1457,8 @@ export class Cline { return true } - const showNotificationForApprovalIfAutoApprovalEnabled = ( - message: string, - ) => { - if ( - this.autoApprovalSettings.enabled && - this.autoApprovalSettings.enableNotifications - ) { + const showNotificationForApprovalIfAutoApprovalEnabled = (message: string) => { + if (this.autoApprovalSettings.enabled && this.autoApprovalSettings.enableNotifications) { showSystemNotification({ subtitle: "Approval Required", message, @@ -1817,9 +1468,7 @@ export class Cline { const handleError = async (action: string, error: Error) => { if (this.abandoned) { - console.log( - "Ignoring error since task was abandoned (i.e. from task cancellation after resetting)", - ) + console.log("Ignoring error since task was abandoned (i.e. from task cancellation after resetting)") return } const errorString = `Error ${action}: ${JSON.stringify(serializeError(error))}` @@ -1836,10 +1485,7 @@ export class Cline { } // If block is partial, remove partial closing tag so its not presented to user - const removeClosingTag = ( - tag: ToolParamName, - text?: string, - ) => { + const removeClosingTag = (tag: ToolParamName, text?: string) => { if (!block.partial) { return text || "" } @@ -1877,23 +1523,18 @@ export class Cline { // Check if file exists using cached map or fs.access let fileExists: boolean if (this.diffViewProvider.editType !== undefined) { - fileExists = - this.diffViewProvider.editType === "modify" + fileExists = this.diffViewProvider.editType === "modify" } else { const absolutePath = path.resolve(cwd, relPath) fileExists = await fileExistsAtPath(absolutePath) - this.diffViewProvider.editType = fileExists - ? "modify" - : "create" + this.diffViewProvider.editType = fileExists ? "modify" : "create" } try { // Construct newContent from diff let newContent: string if (diff) { - if ( - !this.api.getModel().id.includes("claude") - ) { + if (!this.api.getModel().id.includes("claude")) { // deepseek models tend to use unescaped html entities in diffs diff = fixModelHtmlEscaping(diff) diff = removeInvalidChars(diff) @@ -1901,8 +1542,7 @@ export class Cline { try { newContent = await constructNewFileContent( diff, - this.diffViewProvider.originalContent || - "", + this.diffViewProvider.originalContent || "", !block.partial, ) } catch (error) { @@ -1926,26 +1566,15 @@ export class Cline { // pre-processing newContent for cases where weaker models might add artifacts like markdown codeblock markers (deepseek/llama) or extra escape characters (gemini) if (newContent.startsWith("```")) { // this handles cases where it includes language specifiers like ```python ```js - newContent = newContent - .split("\n") - .slice(1) - .join("\n") - .trim() + newContent = newContent.split("\n").slice(1).join("\n").trim() } if (newContent.endsWith("```")) { - newContent = newContent - .split("\n") - .slice(0, -1) - .join("\n") - .trim() + newContent = newContent.split("\n").slice(0, -1).join("\n").trim() } - if ( - !this.api.getModel().id.includes("claude") - ) { + if (!this.api.getModel().id.includes("claude")) { // it seems not just llama models are doing this, but also gemini and potentially others - newContent = - fixModelHtmlEscaping(newContent) + newContent = fixModelHtmlEscaping(newContent) newContent = removeInvalidChars(newContent) } } else { @@ -1956,41 +1585,20 @@ export class Cline { newContent = newContent.trimEnd() // remove any trailing newlines, since it's automatically inserted by the editor const sharedMessageProps: ClineSayTool = { - tool: fileExists - ? "editedExistingFile" - : "newFileCreated", - path: getReadablePath( - cwd, - removeClosingTag("path", relPath), - ), + tool: fileExists ? "editedExistingFile" : "newFileCreated", + path: getReadablePath(cwd, removeClosingTag("path", relPath)), content: diff || content, } if (block.partial) { // update gui message - const partialMessage = - JSON.stringify(sharedMessageProps) + const partialMessage = JSON.stringify(sharedMessageProps) if (this.shouldAutoApproveTool(block.name)) { - this.removeLastPartialMessageIfExistsWithType( - "ask", - "tool", - ) // in case the user changes auto-approval settings mid stream - await this.say( - "tool", - partialMessage, - undefined, - block.partial, - ) + this.removeLastPartialMessageIfExistsWithType("ask", "tool") // in case the user changes auto-approval settings mid stream + await this.say("tool", partialMessage, undefined, block.partial) } else { - this.removeLastPartialMessageIfExistsWithType( - "say", - "tool", - ) - await this.ask( - "tool", - partialMessage, - block.partial, - ).catch(() => {}) + this.removeLastPartialMessageIfExistsWithType("say", "tool") + await this.ask("tool", partialMessage, block.partial).catch(() => {}) } // update editor if (!this.diffViewProvider.isEditing) { @@ -1998,47 +1606,26 @@ export class Cline { await this.diffViewProvider.open(relPath) } // editor is open, stream content in - await this.diffViewProvider.update( - newContent, - false, - ) + await this.diffViewProvider.update(newContent, false) break } else { if (!relPath) { this.consecutiveMistakeCount++ - pushToolResult( - await this.sayAndCreateMissingParamError( - block.name, - "path", - ), - ) + pushToolResult(await this.sayAndCreateMissingParamError(block.name, "path")) await this.diffViewProvider.reset() await this.saveCheckpoint() break } if (block.name === "replace_in_file" && !diff) { this.consecutiveMistakeCount++ - pushToolResult( - await this.sayAndCreateMissingParamError( - "replace_in_file", - "diff", - ), - ) + pushToolResult(await this.sayAndCreateMissingParamError("replace_in_file", "diff")) await this.diffViewProvider.reset() await this.saveCheckpoint() break } - if ( - block.name === "write_to_file" && - !content - ) { + if (block.name === "write_to_file" && !content) { this.consecutiveMistakeCount++ - pushToolResult( - await this.sayAndCreateMissingParamError( - "write_to_file", - "content", - ), - ) + pushToolResult(await this.sayAndCreateMissingParamError("write_to_file", "content")) await this.diffViewProvider.reset() await this.saveCheckpoint() break @@ -2050,19 +1637,11 @@ export class Cline { // in other words, you must always repeat the block.partial logic here if (!this.diffViewProvider.isEditing) { // show gui message before showing edit animation - const partialMessage = - JSON.stringify(sharedMessageProps) - await this.ask( - "tool", - partialMessage, - true, - ).catch(() => {}) // sending true for partial even though it's not a partial, this shows the edit row before the content is streamed into the editor + const partialMessage = JSON.stringify(sharedMessageProps) + await this.ask("tool", partialMessage, true).catch(() => {}) // sending true for partial even though it's not a partial, this shows the edit row before the content is streamed into the editor await this.diffViewProvider.open(relPath) } - await this.diffViewProvider.update( - newContent, - true, - ) + await this.diffViewProvider.update(newContent, true) await delay(300) // wait for diff view to update this.diffViewProvider.scrollToFirstDiff() // showOmissionWarning(this.diffViewProvider.originalContent || "", newContent) @@ -2079,16 +1658,8 @@ export class Cline { } satisfies ClineSayTool) if (this.shouldAutoApproveTool(block.name)) { - this.removeLastPartialMessageIfExistsWithType( - "ask", - "tool", - ) - await this.say( - "tool", - completeMessage, - undefined, - false, - ) + this.removeLastPartialMessageIfExistsWithType("ask", "tool") + await this.say("tool", completeMessage, undefined, false) this.consecutiveAutoApprovedRequestsCount++ // we need an artificial delay to let the diagnostics catch up to the changes @@ -2098,31 +1669,19 @@ export class Cline { showNotificationForApprovalIfAutoApprovalEnabled( `Cline wants to ${fileExists ? "edit" : "create"} ${path.basename(relPath)}`, ) - this.removeLastPartialMessageIfExistsWithType( - "say", - "tool", - ) + this.removeLastPartialMessageIfExistsWithType("say", "tool") // const didApprove = await askApproval("tool", completeMessage) // Need a more customized tool response for file edits to highlight the fact that the file was not updated (particularly important for deepseek) let didApprove = true - const { response, text, images } = - await this.ask( - "tool", - completeMessage, - false, - ) + const { response, text, images } = await this.ask("tool", completeMessage, false) if (response !== "yesButtonClicked") { // TODO: add similar context for other tool denial responses, to emphasize ie that a command was not run const fileDeniedNote = fileExists ? "The file was not updated, and maintains its original contents." : "The file was not created." if (response === "messageResponse") { - await this.say( - "user_feedback", - text, - images, - ) + await this.say("user_feedback", text, images) pushToolResult( formatResponse.toolResult( `The user denied this operation. ${fileDeniedNote}\nThe user provided the following feedback:\n\n${text}\n`, @@ -2132,9 +1691,7 @@ export class Cline { this.didRejectTool = true didApprove = false } else { - pushToolResult( - `The user denied this operation. ${fileDeniedNote}`, - ) + pushToolResult(`The user denied this operation. ${fileDeniedNote}`) this.didRejectTool = true didApprove = false } @@ -2147,20 +1704,14 @@ export class Cline { } } - const { - newProblemsMessage, - userEdits, - autoFormattingEdits, - finalContent, - } = await this.diffViewProvider.saveChanges() + const { newProblemsMessage, userEdits, autoFormattingEdits, finalContent } = + await this.diffViewProvider.saveChanges() this.didEditFile = true // used to determine if we should wait for busy terminal to update before sending api request if (userEdits) { await this.say( "user_feedback_diff", JSON.stringify({ - tool: fileExists - ? "editedExistingFile" - : "newFileCreated", + tool: fileExists ? "editedExistingFile" : "newFileCreated", path: getReadablePath(cwd, relPath), diff: userEdits, } satisfies ClineSayTool), @@ -2207,10 +1758,7 @@ export class Cline { const relPath: string | undefined = block.params.path const sharedMessageProps: ClineSayTool = { tool: "readFile", - path: getReadablePath( - cwd, - removeClosingTag("path", relPath), - ), + path: getReadablePath(cwd, removeClosingTag("path", relPath)), } try { if (block.partial) { @@ -2219,37 +1767,17 @@ export class Cline { content: undefined, } satisfies ClineSayTool) if (this.shouldAutoApproveTool(block.name)) { - this.removeLastPartialMessageIfExistsWithType( - "ask", - "tool", - ) - await this.say( - "tool", - partialMessage, - undefined, - block.partial, - ) + this.removeLastPartialMessageIfExistsWithType("ask", "tool") + await this.say("tool", partialMessage, undefined, block.partial) } else { - this.removeLastPartialMessageIfExistsWithType( - "say", - "tool", - ) - await this.ask( - "tool", - partialMessage, - block.partial, - ).catch(() => {}) + this.removeLastPartialMessageIfExistsWithType("say", "tool") + await this.ask("tool", partialMessage, block.partial).catch(() => {}) } break } else { if (!relPath) { this.consecutiveMistakeCount++ - pushToolResult( - await this.sayAndCreateMissingParamError( - "read_file", - "path", - ), - ) + pushToolResult(await this.sayAndCreateMissingParamError("read_file", "path")) await this.saveCheckpoint() break } @@ -2260,37 +1788,22 @@ export class Cline { content: absolutePath, } satisfies ClineSayTool) if (this.shouldAutoApproveTool(block.name)) { - this.removeLastPartialMessageIfExistsWithType( - "ask", - "tool", - ) - await this.say( - "tool", - completeMessage, - undefined, - false, - ) // need to be sending partialValue bool, since undefined has its own purpose in that the message is treated neither as a partial or completion of a partial, but as a single complete message + this.removeLastPartialMessageIfExistsWithType("ask", "tool") + await this.say("tool", completeMessage, undefined, false) // need to be sending partialValue bool, since undefined has its own purpose in that the message is treated neither as a partial or completion of a partial, but as a single complete message this.consecutiveAutoApprovedRequestsCount++ } else { showNotificationForApprovalIfAutoApprovalEnabled( `Cline wants to read ${path.basename(absolutePath)}`, ) - this.removeLastPartialMessageIfExistsWithType( - "say", - "tool", - ) - const didApprove = await askApproval( - "tool", - completeMessage, - ) + this.removeLastPartialMessageIfExistsWithType("say", "tool") + const didApprove = await askApproval("tool", completeMessage) if (!didApprove) { await this.saveCheckpoint() break } } // now execute the tool like normal - const content = - await extractTextFromFile(absolutePath) + const content = await extractTextFromFile(absolutePath) pushToolResult(content) await this.saveCheckpoint() break @@ -2303,17 +1816,11 @@ export class Cline { } case "list_files": { const relDirPath: string | undefined = block.params.path - const recursiveRaw: string | undefined = - block.params.recursive + const recursiveRaw: string | undefined = block.params.recursive const recursive = recursiveRaw?.toLowerCase() === "true" const sharedMessageProps: ClineSayTool = { - tool: !recursive - ? "listFilesTopLevel" - : "listFilesRecursive", - path: getReadablePath( - cwd, - removeClosingTag("path", relDirPath), - ), + tool: !recursive ? "listFilesTopLevel" : "listFilesRecursive", + path: getReadablePath(cwd, removeClosingTag("path", relDirPath)), } try { if (block.partial) { @@ -2322,83 +1829,38 @@ export class Cline { content: "", } satisfies ClineSayTool) if (this.shouldAutoApproveTool(block.name)) { - this.removeLastPartialMessageIfExistsWithType( - "ask", - "tool", - ) - await this.say( - "tool", - partialMessage, - undefined, - block.partial, - ) + this.removeLastPartialMessageIfExistsWithType("ask", "tool") + await this.say("tool", partialMessage, undefined, block.partial) } else { - this.removeLastPartialMessageIfExistsWithType( - "say", - "tool", - ) - await this.ask( - "tool", - partialMessage, - block.partial, - ).catch(() => {}) + this.removeLastPartialMessageIfExistsWithType("say", "tool") + await this.ask("tool", partialMessage, block.partial).catch(() => {}) } break } else { if (!relDirPath) { this.consecutiveMistakeCount++ - pushToolResult( - await this.sayAndCreateMissingParamError( - "list_files", - "path", - ), - ) + pushToolResult(await this.sayAndCreateMissingParamError("list_files", "path")) await this.saveCheckpoint() break } this.consecutiveMistakeCount = 0 - const absolutePath = path.resolve( - cwd, - relDirPath, - ) - const [files, didHitLimit] = await listFiles( - absolutePath, - recursive, - 200, - ) - const result = formatResponse.formatFilesList( - absolutePath, - files, - didHitLimit, - ) + const absolutePath = path.resolve(cwd, relDirPath) + const [files, didHitLimit] = await listFiles(absolutePath, recursive, 200) + const result = formatResponse.formatFilesList(absolutePath, files, didHitLimit) const completeMessage = JSON.stringify({ ...sharedMessageProps, content: result, } satisfies ClineSayTool) if (this.shouldAutoApproveTool(block.name)) { - this.removeLastPartialMessageIfExistsWithType( - "ask", - "tool", - ) - await this.say( - "tool", - completeMessage, - undefined, - false, - ) + this.removeLastPartialMessageIfExistsWithType("ask", "tool") + await this.say("tool", completeMessage, undefined, false) this.consecutiveAutoApprovedRequestsCount++ } else { showNotificationForApprovalIfAutoApprovalEnabled( `Cline wants to view directory ${path.basename(absolutePath)}/`, ) - this.removeLastPartialMessageIfExistsWithType( - "say", - "tool", - ) - const didApprove = await askApproval( - "tool", - completeMessage, - ) + this.removeLastPartialMessageIfExistsWithType("say", "tool") + const didApprove = await askApproval("tool", completeMessage) if (!didApprove) { await this.saveCheckpoint() break @@ -2418,10 +1880,7 @@ export class Cline { const relDirPath: string | undefined = block.params.path const sharedMessageProps: ClineSayTool = { tool: "listCodeDefinitionNames", - path: getReadablePath( - cwd, - removeClosingTag("path", relDirPath), - ), + path: getReadablePath(cwd, removeClosingTag("path", relDirPath)), } try { if (block.partial) { @@ -2430,77 +1889,37 @@ export class Cline { content: "", } satisfies ClineSayTool) if (this.shouldAutoApproveTool(block.name)) { - this.removeLastPartialMessageIfExistsWithType( - "ask", - "tool", - ) - await this.say( - "tool", - partialMessage, - undefined, - block.partial, - ) + this.removeLastPartialMessageIfExistsWithType("ask", "tool") + await this.say("tool", partialMessage, undefined, block.partial) } else { - this.removeLastPartialMessageIfExistsWithType( - "say", - "tool", - ) - await this.ask( - "tool", - partialMessage, - block.partial, - ).catch(() => {}) + this.removeLastPartialMessageIfExistsWithType("say", "tool") + await this.ask("tool", partialMessage, block.partial).catch(() => {}) } break } else { if (!relDirPath) { this.consecutiveMistakeCount++ - pushToolResult( - await this.sayAndCreateMissingParamError( - "list_code_definition_names", - "path", - ), - ) + pushToolResult(await this.sayAndCreateMissingParamError("list_code_definition_names", "path")) await this.saveCheckpoint() break } this.consecutiveMistakeCount = 0 - const absolutePath = path.resolve( - cwd, - relDirPath, - ) - const result = - await parseSourceCodeForDefinitionsTopLevel( - absolutePath, - ) + const absolutePath = path.resolve(cwd, relDirPath) + const result = await parseSourceCodeForDefinitionsTopLevel(absolutePath) const completeMessage = JSON.stringify({ ...sharedMessageProps, content: result, } satisfies ClineSayTool) if (this.shouldAutoApproveTool(block.name)) { - this.removeLastPartialMessageIfExistsWithType( - "ask", - "tool", - ) - await this.say( - "tool", - completeMessage, - undefined, - false, - ) + this.removeLastPartialMessageIfExistsWithType("ask", "tool") + await this.say("tool", completeMessage, undefined, false) this.consecutiveAutoApprovedRequestsCount++ } else { showNotificationForApprovalIfAutoApprovalEnabled( `Cline wants to view source code definitions in ${path.basename(absolutePath)}/`, ) - this.removeLastPartialMessageIfExistsWithType( - "say", - "tool", - ) - const didApprove = await askApproval( - "tool", - completeMessage, - ) + this.removeLastPartialMessageIfExistsWithType("say", "tool") + const didApprove = await askApproval("tool", completeMessage) if (!didApprove) { await this.saveCheckpoint() break @@ -2511,10 +1930,7 @@ export class Cline { break } } catch (error) { - await handleError( - "parsing source code definitions", - error, - ) + await handleError("parsing source code definitions", error) await this.saveCheckpoint() break } @@ -2522,19 +1938,12 @@ export class Cline { case "search_files": { const relDirPath: string | undefined = block.params.path const regex: string | undefined = block.params.regex - const filePattern: string | undefined = - block.params.file_pattern + const filePattern: string | undefined = block.params.file_pattern const sharedMessageProps: ClineSayTool = { tool: "searchFiles", - path: getReadablePath( - cwd, - removeClosingTag("path", relDirPath), - ), + path: getReadablePath(cwd, removeClosingTag("path", relDirPath)), regex: removeClosingTag("regex", regex), - filePattern: removeClosingTag( - "file_pattern", - filePattern, - ), + filePattern: removeClosingTag("file_pattern", filePattern), } try { if (block.partial) { @@ -2543,90 +1952,43 @@ export class Cline { content: "", } satisfies ClineSayTool) if (this.shouldAutoApproveTool(block.name)) { - this.removeLastPartialMessageIfExistsWithType( - "ask", - "tool", - ) - await this.say( - "tool", - partialMessage, - undefined, - block.partial, - ) + this.removeLastPartialMessageIfExistsWithType("ask", "tool") + await this.say("tool", partialMessage, undefined, block.partial) } else { - this.removeLastPartialMessageIfExistsWithType( - "say", - "tool", - ) - await this.ask( - "tool", - partialMessage, - block.partial, - ).catch(() => {}) + this.removeLastPartialMessageIfExistsWithType("say", "tool") + await this.ask("tool", partialMessage, block.partial).catch(() => {}) } break } else { if (!relDirPath) { this.consecutiveMistakeCount++ - pushToolResult( - await this.sayAndCreateMissingParamError( - "search_files", - "path", - ), - ) + pushToolResult(await this.sayAndCreateMissingParamError("search_files", "path")) await this.saveCheckpoint() break } if (!regex) { this.consecutiveMistakeCount++ - pushToolResult( - await this.sayAndCreateMissingParamError( - "search_files", - "regex", - ), - ) + pushToolResult(await this.sayAndCreateMissingParamError("search_files", "regex")) await this.saveCheckpoint() break } this.consecutiveMistakeCount = 0 - const absolutePath = path.resolve( - cwd, - relDirPath, - ) - const results = await regexSearchFiles( - cwd, - absolutePath, - regex, - filePattern, - ) + const absolutePath = path.resolve(cwd, relDirPath) + const results = await regexSearchFiles(cwd, absolutePath, regex, filePattern) const completeMessage = JSON.stringify({ ...sharedMessageProps, content: results, } satisfies ClineSayTool) if (this.shouldAutoApproveTool(block.name)) { - this.removeLastPartialMessageIfExistsWithType( - "ask", - "tool", - ) - await this.say( - "tool", - completeMessage, - undefined, - false, - ) + this.removeLastPartialMessageIfExistsWithType("ask", "tool") + await this.say("tool", completeMessage, undefined, false) this.consecutiveAutoApprovedRequestsCount++ } else { showNotificationForApprovalIfAutoApprovalEnabled( `Cline wants to search files in ${path.basename(absolutePath)}/`, ) - this.removeLastPartialMessageIfExistsWithType( - "say", - "tool", - ) - const didApprove = await askApproval( - "tool", - completeMessage, - ) + this.removeLastPartialMessageIfExistsWithType("say", "tool") + const didApprove = await askApproval("tool", completeMessage) if (!didApprove) { await this.saveCheckpoint() break @@ -2643,23 +2005,16 @@ export class Cline { } } case "browser_action": { - const action: BrowserAction | undefined = block.params - .action as BrowserAction + const action: BrowserAction | undefined = block.params.action as BrowserAction const url: string | undefined = block.params.url - const coordinate: string | undefined = - block.params.coordinate + const coordinate: string | undefined = block.params.coordinate const text: string | undefined = block.params.text if (!action || !browserActions.includes(action)) { // checking for action to ensure it is complete and valid if (!block.partial) { // if the block is complete and we don't have a valid action this is a mistake this.consecutiveMistakeCount++ - pushToolResult( - await this.sayAndCreateMissingParamError( - "browser_action", - "action", - ), - ) + pushToolResult(await this.sayAndCreateMissingParamError("browser_action", "action")) await this.browserSession.closeBrowser() } break @@ -2668,13 +2023,8 @@ export class Cline { try { if (block.partial) { if (action === "launch") { - if ( - this.shouldAutoApproveTool(block.name) - ) { - this.removeLastPartialMessageIfExistsWithType( - "ask", - "browser_action_launch", - ) + if (this.shouldAutoApproveTool(block.name)) { + this.removeLastPartialMessageIfExistsWithType("ask", "browser_action_launch") await this.say( "browser_action_launch", removeClosingTag("url", url), @@ -2682,10 +2032,7 @@ export class Cline { block.partial, ) } else { - this.removeLastPartialMessageIfExistsWithType( - "say", - "browser_action_launch", - ) + this.removeLastPartialMessageIfExistsWithType("say", "browser_action_launch") await this.ask( "browser_action_launch", removeClosingTag("url", url), @@ -2697,14 +2044,8 @@ export class Cline { "browser_action", JSON.stringify({ action: action as BrowserAction, - coordinate: removeClosingTag( - "coordinate", - coordinate, - ), - text: removeClosingTag( - "text", - text, - ), + coordinate: removeClosingTag("coordinate", coordinate), + text: removeClosingTag("text", text), } satisfies ClineSayBrowserAction), undefined, block.partial, @@ -2716,45 +2057,23 @@ export class Cline { if (action === "launch") { if (!url) { this.consecutiveMistakeCount++ - pushToolResult( - await this.sayAndCreateMissingParamError( - "browser_action", - "url", - ), - ) + pushToolResult(await this.sayAndCreateMissingParamError("browser_action", "url")) await this.browserSession.closeBrowser() await this.saveCheckpoint() break } this.consecutiveMistakeCount = 0 - if ( - this.shouldAutoApproveTool(block.name) - ) { - this.removeLastPartialMessageIfExistsWithType( - "ask", - "browser_action_launch", - ) - await this.say( - "browser_action_launch", - url, - undefined, - false, - ) - this - .consecutiveAutoApprovedRequestsCount++ + if (this.shouldAutoApproveTool(block.name)) { + this.removeLastPartialMessageIfExistsWithType("ask", "browser_action_launch") + await this.say("browser_action_launch", url, undefined, false) + this.consecutiveAutoApprovedRequestsCount++ } else { showNotificationForApprovalIfAutoApprovalEnabled( `Cline wants to use a browser and launch ${url}`, ) - this.removeLastPartialMessageIfExistsWithType( - "say", - "browser_action_launch", - ) - const didApprove = await askApproval( - "browser_action_launch", - url, - ) + this.removeLastPartialMessageIfExistsWithType("say", "browser_action_launch") + const didApprove = await askApproval("browser_action_launch", url) if (!didApprove) { await this.saveCheckpoint() break @@ -2766,19 +2085,13 @@ export class Cline { await this.say("browser_action_result", "") // starts loading spinner await this.browserSession.launchBrowser() - browserActionResult = - await this.browserSession.navigateToUrl( - url, - ) + browserActionResult = await this.browserSession.navigateToUrl(url) } else { if (action === "click") { if (!coordinate) { this.consecutiveMistakeCount++ pushToolResult( - await this.sayAndCreateMissingParamError( - "browser_action", - "coordinate", - ), + await this.sayAndCreateMissingParamError("browser_action", "coordinate"), ) await this.browserSession.closeBrowser() await this.saveCheckpoint() @@ -2788,12 +2101,7 @@ export class Cline { if (action === "type") { if (!text) { this.consecutiveMistakeCount++ - pushToolResult( - await this.sayAndCreateMissingParamError( - "browser_action", - "text", - ), - ) + pushToolResult(await this.sayAndCreateMissingParamError("browser_action", "text")) await this.browserSession.closeBrowser() await this.saveCheckpoint() break @@ -2812,28 +2120,19 @@ export class Cline { ) switch (action) { case "click": - browserActionResult = - await this.browserSession.click( - coordinate!, - ) + browserActionResult = await this.browserSession.click(coordinate!) break case "type": - browserActionResult = - await this.browserSession.type( - text!, - ) + browserActionResult = await this.browserSession.type(text!) break case "scroll_down": - browserActionResult = - await this.browserSession.scrollDown() + browserActionResult = await this.browserSession.scrollDown() break case "scroll_up": - browserActionResult = - await this.browserSession.scrollUp() + browserActionResult = await this.browserSession.scrollUp() break case "close": - browserActionResult = - await this.browserSession.closeBrowser() + browserActionResult = await this.browserSession.closeBrowser() break } } @@ -2844,21 +2143,13 @@ export class Cline { case "type": case "scroll_down": case "scroll_up": - await this.say( - "browser_action_result", - JSON.stringify(browserActionResult), - ) + await this.say("browser_action_result", JSON.stringify(browserActionResult)) pushToolResult( formatResponse.toolResult( `The browser action has been executed. The console logs and screenshot have been captured for your analysis.\n\nConsole logs:\n${ - browserActionResult.logs || - "(No new logs)" + browserActionResult.logs || "(No new logs)" }\n\n(REMEMBER: if you need to proceed to using non-\`browser_action\` tools or launch a new browser, you MUST first close this browser. For example, if after analyzing the logs and screenshot you need to edit a file, you must first close the browser before you can use the write_to_file tool.)`, - browserActionResult.screenshot - ? [ - browserActionResult.screenshot, - ] - : [], + browserActionResult.screenshot ? [browserActionResult.screenshot] : [], ), ) await this.saveCheckpoint() @@ -2885,10 +2176,8 @@ export class Cline { } case "execute_command": { const command: string | undefined = block.params.command - const requiresApprovalRaw: string | undefined = - block.params.requires_approval - const requiresApproval = - requiresApprovalRaw?.toLowerCase() === "true" + const requiresApprovalRaw: string | undefined = block.params.requires_approval + const requiresApproval = requiresApprovalRaw?.toLowerCase() === "true" try { if (block.partial) { @@ -2902,32 +2191,20 @@ export class Cline { // ).catch(() => {}) } else { // don't need to remove last partial since we couldn't have streamed a say - await this.ask( - "command", - removeClosingTag("command", command), - block.partial, - ).catch(() => {}) + await this.ask("command", removeClosingTag("command", command), block.partial).catch(() => {}) } break } else { if (!command) { this.consecutiveMistakeCount++ - pushToolResult( - await this.sayAndCreateMissingParamError( - "execute_command", - "command", - ), - ) + pushToolResult(await this.sayAndCreateMissingParamError("execute_command", "command")) await this.saveCheckpoint() break } if (!requiresApprovalRaw) { this.consecutiveMistakeCount++ pushToolResult( - await this.sayAndCreateMissingParamError( - "execute_command", - "requires_approval", - ), + await this.sayAndCreateMissingParamError("execute_command", "requires_approval"), ) await this.saveCheckpoint() break @@ -2936,20 +2213,9 @@ export class Cline { let didAutoApprove = false - if ( - !requiresApproval && - this.shouldAutoApproveTool(block.name) - ) { - this.removeLastPartialMessageIfExistsWithType( - "ask", - "command", - ) - await this.say( - "command", - command, - undefined, - false, - ) + if (!requiresApproval && this.shouldAutoApproveTool(block.name)) { + this.removeLastPartialMessageIfExistsWithType("ask", "command") + await this.say("command", command, undefined, false) this.consecutiveAutoApprovedRequestsCount++ didAutoApprove = true } else { @@ -2969,24 +2235,18 @@ export class Cline { } let timeoutId: NodeJS.Timeout | undefined - if ( - didAutoApprove && - this.autoApprovalSettings - .enableNotifications - ) { + if (didAutoApprove && this.autoApprovalSettings.enableNotifications) { // if the command was auto-approved, and it's long running we need to notify the user after some time has passed without proceeding timeoutId = setTimeout(() => { showSystemNotification({ - subtitle: - "Command is still running", + subtitle: "Command is still running", message: "An auto-approved command has been running for 30s, and may need your attention.", }) }, 30_000) } - const [userRejected, result] = - await this.executeCommandTool(command) + const [userRejected, result] = await this.executeCommandTool(command) if (timeoutId) { clearTimeout(timeoutId) } @@ -3004,74 +2264,37 @@ export class Cline { } } case "use_mcp_tool": { - const server_name: string | undefined = - block.params.server_name - const tool_name: string | undefined = - block.params.tool_name - const mcp_arguments: string | undefined = - block.params.arguments + const server_name: string | undefined = block.params.server_name + const tool_name: string | undefined = block.params.tool_name + const mcp_arguments: string | undefined = block.params.arguments try { if (block.partial) { const partialMessage = JSON.stringify({ type: "use_mcp_tool", - serverName: removeClosingTag( - "server_name", - server_name, - ), - toolName: removeClosingTag( - "tool_name", - tool_name, - ), - arguments: removeClosingTag( - "arguments", - mcp_arguments, - ), + serverName: removeClosingTag("server_name", server_name), + toolName: removeClosingTag("tool_name", tool_name), + arguments: removeClosingTag("arguments", mcp_arguments), } satisfies ClineAskUseMcpServer) if (this.shouldAutoApproveTool(block.name)) { - this.removeLastPartialMessageIfExistsWithType( - "ask", - "use_mcp_server", - ) - await this.say( - "use_mcp_server", - partialMessage, - undefined, - block.partial, - ) + this.removeLastPartialMessageIfExistsWithType("ask", "use_mcp_server") + await this.say("use_mcp_server", partialMessage, undefined, block.partial) } else { - this.removeLastPartialMessageIfExistsWithType( - "say", - "use_mcp_server", - ) - await this.ask( - "use_mcp_server", - partialMessage, - block.partial, - ).catch(() => {}) + this.removeLastPartialMessageIfExistsWithType("say", "use_mcp_server") + await this.ask("use_mcp_server", partialMessage, block.partial).catch(() => {}) } break } else { if (!server_name) { this.consecutiveMistakeCount++ - pushToolResult( - await this.sayAndCreateMissingParamError( - "use_mcp_tool", - "server_name", - ), - ) + pushToolResult(await this.sayAndCreateMissingParamError("use_mcp_tool", "server_name")) await this.saveCheckpoint() break } if (!tool_name) { this.consecutiveMistakeCount++ - pushToolResult( - await this.sayAndCreateMissingParamError( - "use_mcp_tool", - "tool_name", - ), - ) + pushToolResult(await this.sayAndCreateMissingParamError("use_mcp_tool", "tool_name")) await this.saveCheckpoint() break } @@ -3081,13 +2304,10 @@ export class Cline { // pushToolResult(await this.sayAndCreateMissingParamError("use_mcp_tool", "arguments")) // break // } - let parsedArguments: - | Record - | undefined + let parsedArguments: Record | undefined if (mcp_arguments) { try { - parsedArguments = - JSON.parse(mcp_arguments) + parsedArguments = JSON.parse(mcp_arguments) } catch (error) { this.consecutiveMistakeCount++ await this.say( @@ -3096,10 +2316,7 @@ export class Cline { ) pushToolResult( formatResponse.toolError( - formatResponse.invalidMcpToolArgumentError( - server_name, - tool_name, - ), + formatResponse.invalidMcpToolArgumentError(server_name, tool_name), ), ) await this.saveCheckpoint() @@ -3115,29 +2332,15 @@ export class Cline { } satisfies ClineAskUseMcpServer) if (this.shouldAutoApproveTool(block.name)) { - this.removeLastPartialMessageIfExistsWithType( - "ask", - "use_mcp_server", - ) - await this.say( - "use_mcp_server", - completeMessage, - undefined, - false, - ) + this.removeLastPartialMessageIfExistsWithType("ask", "use_mcp_server") + await this.say("use_mcp_server", completeMessage, undefined, false) this.consecutiveAutoApprovedRequestsCount++ } else { showNotificationForApprovalIfAutoApprovalEnabled( `Cline wants to use ${tool_name} on ${server_name}`, ) - this.removeLastPartialMessageIfExistsWithType( - "say", - "use_mcp_server", - ) - const didApprove = await askApproval( - "use_mcp_server", - completeMessage, - ) + this.removeLastPartialMessageIfExistsWithType("say", "use_mcp_server") + const didApprove = await askApproval("use_mcp_server", completeMessage) if (!didApprove) { await this.saveCheckpoint() break @@ -3148,11 +2351,7 @@ export class Cline { await this.say("mcp_server_request_started") // same as browser_action_result const toolResult = await this.providerRef .deref() - ?.mcpHub?.callTool( - server_name, - tool_name, - parsedArguments, - ) + ?.mcpHub?.callTool(server_name, tool_name, parsedArguments) // TODO: add progress indicator and ability to parse images and non-text responses const toolResultPretty = @@ -3163,25 +2362,15 @@ export class Cline { return item.text } if (item.type === "resource") { - const { blob, ...rest } = - item.resource - return JSON.stringify( - rest, - null, - 2, - ) + const { blob, ...rest } = item.resource + return JSON.stringify(rest, null, 2) } return "" }) .filter(Boolean) .join("\n\n") || "(No response)" - await this.say( - "mcp_server_response", - toolResultPretty, - ) - pushToolResult( - formatResponse.toolResult(toolResultPretty), - ) + await this.say("mcp_server_response", toolResultPretty) + pushToolResult(formatResponse.toolResult(toolResultPretty)) await this.saveCheckpoint() break } @@ -3192,64 +2381,35 @@ export class Cline { } } case "access_mcp_resource": { - const server_name: string | undefined = - block.params.server_name + const server_name: string | undefined = block.params.server_name const uri: string | undefined = block.params.uri try { if (block.partial) { const partialMessage = JSON.stringify({ type: "access_mcp_resource", - serverName: removeClosingTag( - "server_name", - server_name, - ), + serverName: removeClosingTag("server_name", server_name), uri: removeClosingTag("uri", uri), } satisfies ClineAskUseMcpServer) if (this.shouldAutoApproveTool(block.name)) { - this.removeLastPartialMessageIfExistsWithType( - "ask", - "use_mcp_server", - ) - await this.say( - "use_mcp_server", - partialMessage, - undefined, - block.partial, - ) + this.removeLastPartialMessageIfExistsWithType("ask", "use_mcp_server") + await this.say("use_mcp_server", partialMessage, undefined, block.partial) } else { - this.removeLastPartialMessageIfExistsWithType( - "say", - "use_mcp_server", - ) - await this.ask( - "use_mcp_server", - partialMessage, - block.partial, - ).catch(() => {}) + this.removeLastPartialMessageIfExistsWithType("say", "use_mcp_server") + await this.ask("use_mcp_server", partialMessage, block.partial).catch(() => {}) } break } else { if (!server_name) { this.consecutiveMistakeCount++ - pushToolResult( - await this.sayAndCreateMissingParamError( - "access_mcp_resource", - "server_name", - ), - ) + pushToolResult(await this.sayAndCreateMissingParamError("access_mcp_resource", "server_name")) await this.saveCheckpoint() break } if (!uri) { this.consecutiveMistakeCount++ - pushToolResult( - await this.sayAndCreateMissingParamError( - "access_mcp_resource", - "uri", - ), - ) + pushToolResult(await this.sayAndCreateMissingParamError("access_mcp_resource", "uri")) await this.saveCheckpoint() break } @@ -3261,29 +2421,15 @@ export class Cline { } satisfies ClineAskUseMcpServer) if (this.shouldAutoApproveTool(block.name)) { - this.removeLastPartialMessageIfExistsWithType( - "ask", - "use_mcp_server", - ) - await this.say( - "use_mcp_server", - completeMessage, - undefined, - false, - ) + this.removeLastPartialMessageIfExistsWithType("ask", "use_mcp_server") + await this.say("use_mcp_server", completeMessage, undefined, false) this.consecutiveAutoApprovedRequestsCount++ } else { showNotificationForApprovalIfAutoApprovalEnabled( `Cline wants to access ${uri} on ${server_name}`, ) - this.removeLastPartialMessageIfExistsWithType( - "say", - "use_mcp_server", - ) - const didApprove = await askApproval( - "use_mcp_server", - completeMessage, - ) + this.removeLastPartialMessageIfExistsWithType("say", "use_mcp_server") + const didApprove = await askApproval("use_mcp_server", completeMessage) if (!didApprove) { await this.saveCheckpoint() break @@ -3292,9 +2438,7 @@ export class Cline { // now execute the tool await this.say("mcp_server_request_started") - const resourceResult = await this.providerRef - .deref() - ?.mcpHub?.readResource(server_name, uri) + const resourceResult = await this.providerRef.deref()?.mcpHub?.readResource(server_name, uri) const resourceResultPretty = resourceResult?.contents .map((item) => { @@ -3305,15 +2449,8 @@ export class Cline { }) .filter(Boolean) .join("\n\n") || "(Empty response)" - await this.say( - "mcp_server_response", - resourceResultPretty, - ) - pushToolResult( - formatResponse.toolResult( - resourceResultPretty, - ), - ) + await this.say("mcp_server_response", resourceResultPretty) + pushToolResult(formatResponse.toolResult(resourceResultPretty)) await this.saveCheckpoint() break } @@ -3324,57 +2461,30 @@ export class Cline { } } case "ask_followup_question": { - const question: string | undefined = - block.params.question + const question: string | undefined = block.params.question try { if (block.partial) { - await this.ask( - "followup", - removeClosingTag("question", question), - block.partial, - ).catch(() => {}) + await this.ask("followup", removeClosingTag("question", question), block.partial).catch(() => {}) break } else { if (!question) { this.consecutiveMistakeCount++ - pushToolResult( - await this.sayAndCreateMissingParamError( - "ask_followup_question", - "question", - ), - ) + pushToolResult(await this.sayAndCreateMissingParamError("ask_followup_question", "question")) await this.saveCheckpoint() break } this.consecutiveMistakeCount = 0 - if ( - this.autoApprovalSettings.enabled && - this.autoApprovalSettings - .enableNotifications - ) { + if (this.autoApprovalSettings.enabled && this.autoApprovalSettings.enableNotifications) { showSystemNotification({ subtitle: "Cline has a question...", message: question.replace(/\n/g, " "), }) } - const { text, images } = await this.ask( - "followup", - question, - false, - ) - await this.say( - "user_feedback", - text ?? "", - images, - ) - pushToolResult( - formatResponse.toolResult( - `\n${text}\n`, - images, - ), - ) + const { text, images } = await this.ask("followup", question, false) + await this.say("user_feedback", text ?? "", images) + pushToolResult(formatResponse.toolResult(`\n${text}\n`, images)) await this.saveCheckpoint() break } @@ -3408,28 +2518,20 @@ export class Cline { const result: string | undefined = block.params.result const command: string | undefined = block.params.command - const addNewChangesFlagToLastCompletionResultMessage = - async () => { - // Add newchanges flag if there are new changes to the workspace - - const hasNewChanges = - await this.doesLatestTaskCompletionHaveNewChanges() - const lastCompletionResultMessage = findLast( - this.clineMessages, - (m) => m.say === "completion_result", - ) - if ( - lastCompletionResultMessage && - hasNewChanges && - !lastCompletionResultMessage.text?.endsWith( - COMPLETION_RESULT_CHANGES_FLAG, - ) - ) { - lastCompletionResultMessage.text += - COMPLETION_RESULT_CHANGES_FLAG - } - await this.saveClineMessages() + const addNewChangesFlagToLastCompletionResultMessage = async () => { + // Add newchanges flag if there are new changes to the workspace + + const hasNewChanges = await this.doesLatestTaskCompletionHaveNewChanges() + const lastCompletionResultMessage = findLast(this.clineMessages, (m) => m.say === "completion_result") + if ( + lastCompletionResultMessage && + hasNewChanges && + !lastCompletionResultMessage.text?.endsWith(COMPLETION_RESULT_CHANGES_FLAG) + ) { + lastCompletionResultMessage.text += COMPLETION_RESULT_CHANGES_FLAG } + await this.saveClineMessages() + } try { const lastMessage = this.clineMessages.at(-1) @@ -3440,38 +2542,20 @@ export class Cline { // const secondLastMessage = this.clineMessages.at(-2) // NOTE: we do not want to auto approve a command run as part of the attempt_completion tool - if ( - lastMessage && - lastMessage.ask === "command" - ) { + if (lastMessage && lastMessage.ask === "command") { // update command - await this.ask( - "command", - removeClosingTag( - "command", - command, - ), - block.partial, - ).catch(() => {}) + await this.ask("command", removeClosingTag("command", command), block.partial).catch( + () => {}, + ) } else { // last message is completion_result // we have command string, which means we have the result as well, so finish it (doesnt have to exist yet) - await this.say( - "completion_result", - removeClosingTag("result", result), - undefined, - false, - ) + await this.say("completion_result", removeClosingTag("result", result), undefined, false) await this.saveCheckpoint() await addNewChangesFlagToLastCompletionResultMessage() - await this.ask( - "command", - removeClosingTag( - "command", - command, - ), - block.partial, - ).catch(() => {}) + await this.ask("command", removeClosingTag("command", command), block.partial).catch( + () => {}, + ) } } else { // no command, still outputting partial result @@ -3486,22 +2570,13 @@ export class Cline { } else { if (!result) { this.consecutiveMistakeCount++ - pushToolResult( - await this.sayAndCreateMissingParamError( - "attempt_completion", - "result", - ), - ) + pushToolResult(await this.sayAndCreateMissingParamError("attempt_completion", "result")) await this.saveCheckpoint() break } this.consecutiveMistakeCount = 0 - if ( - this.autoApprovalSettings.enabled && - this.autoApprovalSettings - .enableNotifications - ) { + if (this.autoApprovalSettings.enabled && this.autoApprovalSettings.enableNotifications) { showSystemNotification({ subtitle: "Task Completed", message: result.replace(/\n/g, " "), @@ -3510,17 +2585,9 @@ export class Cline { let commandResult: ToolResponse | undefined if (command) { - if ( - lastMessage && - lastMessage.ask !== "command" - ) { + if (lastMessage && lastMessage.ask !== "command") { // havent sent a command message yet so first send completion_result then command - await this.say( - "completion_result", - result, - undefined, - false, - ) + await this.say("completion_result", result, undefined, false) await this.saveCheckpoint() await addNewChangesFlagToLastCompletionResultMessage() } else { @@ -3529,16 +2596,12 @@ export class Cline { } // complete command message - const didApprove = await askApproval( - "command", - command, - ) + const didApprove = await askApproval("command", command) if (!didApprove) { await this.saveCheckpoint() break } - const [userRejected, execCommandResult] = - await this.executeCommandTool(command!) + const [userRejected, execCommandResult] = await this.executeCommandTool(command!) if (userRejected) { this.didRejectTool = true pushToolResult(execCommandResult) @@ -3548,37 +2611,20 @@ export class Cline { // user didn't reject, but the command may have output commandResult = execCommandResult } else { - await this.say( - "completion_result", - result, - undefined, - false, - ) + await this.say("completion_result", result, undefined, false) await this.saveCheckpoint() await addNewChangesFlagToLastCompletionResultMessage() } // we already sent completion_result says, an empty string asks relinquishes control over button and field - const { response, text, images } = - await this.ask( - "completion_result", - "", - false, - ) + const { response, text, images } = await this.ask("completion_result", "", false) if (response === "yesButtonClicked") { pushToolResult("") // signals to recursive loop to stop (for now this never happens since yesButtonClicked will trigger a new task) break } - await this.say( - "user_feedback", - text ?? "", - images, - ) - - const toolResults: ( - | Anthropic.TextBlockParam - | Anthropic.ImageBlockParam - )[] = [] + await this.say("user_feedback", text ?? "", images) + + const toolResults: (Anthropic.TextBlockParam | Anthropic.ImageBlockParam)[] = [] if (commandResult) { if (typeof commandResult === "string") { toolResults.push({ @@ -3593,9 +2639,7 @@ export class Cline { type: "text", text: `The user has provided feedback on the results. Consider their input to continue the task, and then attempt completion again.\n\n${text}\n`, }) - toolResults.push( - ...formatResponse.imageBlocks(images), - ) + toolResults.push(...formatResponse.imageBlocks(images)) this.userMessageContent.push({ type: "text", text: `${toolDescription()} Result:`, @@ -3623,10 +2667,7 @@ export class Cline { // NOTE: when tool is rejected, iterator stream is interrupted and it waits for userMessageContentReady to be true. Future calls to present will skip execution since didRejectTool and iterate until contentIndex is set to message length and it sets userMessageContentReady to true itself (instead of preemptively doing it in iterator) if (!block.partial || this.didRejectTool || this.didAlreadyUseTool) { // block is finished streaming and executing - if ( - this.currentStreamingContentIndex === - this.assistantMessageContent.length - 1 - ) { + if (this.currentStreamingContentIndex === this.assistantMessageContent.length - 1) { // its okay that we increment if !didCompleteReadingStream, it'll just return bc out of bounds and as streaming continues it will call presentAssitantMessage if a new block is ready. if streaming is finished then we set userMessageContentReady to true when out of bounds. This gracefully allows the stream to continue on and all potential content blocks be presented. // last block is complete and it is finished executing this.userMessageContentReady = true // will allow pwaitfor to continue @@ -3635,10 +2676,7 @@ export class Cline { // call next block if it exists (if not then read stream will call it when its ready) this.currentStreamingContentIndex++ // need to increment regardless, so when read stream calls this function again it will be streaming the next block - if ( - this.currentStreamingContentIndex < - this.assistantMessageContent.length - ) { + if (this.currentStreamingContentIndex < this.assistantMessageContent.length) { // there are already more content blocks to stream, so we'll call this function ourselves // await this.presentAssistantContent() @@ -3662,14 +2700,10 @@ export class Cline { } if (this.consecutiveMistakeCount >= 3) { - if ( - this.autoApprovalSettings.enabled && - this.autoApprovalSettings.enableNotifications - ) { + if (this.autoApprovalSettings.enabled && this.autoApprovalSettings.enableNotifications) { showSystemNotification({ subtitle: "Error", - message: - "Cline is having trouble. Would you like to continue the task?", + message: "Cline is having trouble. Would you like to continue the task?", }) } const { response, text, images } = await this.ask( @@ -3694,8 +2728,7 @@ export class Cline { if ( this.autoApprovalSettings.enabled && - this.consecutiveAutoApprovedRequestsCount >= - this.autoApprovalSettings.maxRequests + this.consecutiveAutoApprovedRequestsCount >= this.autoApprovalSettings.maxRequests ) { if (this.autoApprovalSettings.enableNotifications) { showSystemNotification({ @@ -3712,20 +2745,14 @@ export class Cline { } // get previous api req's index to check token usage and determine if we need to truncate conversation history - const previousApiReqIndex = findLastIndex( - this.clineMessages, - (m) => m.say === "api_req_started", - ) + const previousApiReqIndex = findLastIndex(this.clineMessages, (m) => m.say === "api_req_started") // getting verbose details is an expensive operation, it uses globby to top-down build file structure of project which for large projects can take a few seconds // for the best UX we show a placeholder api_req_started message with a loading spinner as this happens await this.say( "api_req_started", JSON.stringify({ - request: - userContent - .map((block) => formatContentBlockToMarkdown(block)) - .join("\n\n") + "\n\nLoading...", + request: userContent.map((block) => formatContentBlockToMarkdown(block)).join("\n\n") + "\n\nLoading...", }), ) @@ -3734,26 +2761,16 @@ export class Cline { // isNewTask && if (!this.checkpointTracker) { try { - this.checkpointTracker = await CheckpointTracker.create( - this.taskId, - this.providerRef.deref(), - ) + this.checkpointTracker = await CheckpointTracker.create(this.taskId, this.providerRef.deref()) this.checkpointTrackerErrorMessage = undefined } catch (error) { - const errorMessage = - error instanceof Error ? error.message : "Unknown error" - console.error( - "Failed to initialize checkpoint tracker:", - errorMessage, - ) + const errorMessage = error instanceof Error ? error.message : "Unknown error" + console.error("Failed to initialize checkpoint tracker:", errorMessage) this.checkpointTrackerErrorMessage = errorMessage // will be displayed right away since we saveClineMessages next which posts state to webview } } - const [parsedUserContent, environmentDetails] = await this.loadContext( - userContent, - includeFileDetails, - ) + const [parsedUserContent, environmentDetails] = await this.loadContext(userContent, includeFileDetails) userContent = parsedUserContent // add environment details as its own text block, separate from tool results userContent.push({ type: "text", text: environmentDetails }) @@ -3764,14 +2781,9 @@ export class Cline { }) // since we sent off a placeholder api_req_started message to update the webview while waiting to actually start the API request (to load potential details for example), we need to update the text of that message - const lastApiReqIndex = findLastIndex( - this.clineMessages, - (m) => m.say === "api_req_started", - ) + const lastApiReqIndex = findLastIndex(this.clineMessages, (m) => m.say === "api_req_started") this.clineMessages[lastApiReqIndex].text = JSON.stringify({ - request: userContent - .map((block) => formatContentBlockToMarkdown(block)) - .join("\n\n"), + request: userContent.map((block) => formatContentBlockToMarkdown(block)).join("\n\n"), } satisfies ClineApiReqInfo) await this.saveClineMessages() await this.providerRef.deref()?.postStateToWebview() @@ -3786,36 +2798,22 @@ export class Cline { // update api_req_started. we can't use api_req_finished anymore since it's a unique case where it could come after a streaming message (ie in the middle of being updated or executed) // fortunately api_req_finished was always parsed out for the gui anyways, so it remains solely for legacy purposes to keep track of prices in tasks from history // (it's worth removing a few months from now) - const updateApiReqMsg = ( - cancelReason?: ClineApiReqCancelReason, - streamingFailedMessage?: string, - ) => { + const updateApiReqMsg = (cancelReason?: ClineApiReqCancelReason, streamingFailedMessage?: string) => { this.clineMessages[lastApiReqIndex].text = JSON.stringify({ - ...JSON.parse( - this.clineMessages[lastApiReqIndex].text || "{}", - ), + ...JSON.parse(this.clineMessages[lastApiReqIndex].text || "{}"), tokensIn: inputTokens, tokensOut: outputTokens, cacheWrites: cacheWriteTokens, cacheReads: cacheReadTokens, cost: totalCost ?? - calculateApiCost( - this.api.getModel().info, - inputTokens, - outputTokens, - cacheWriteTokens, - cacheReadTokens, - ), + calculateApiCost(this.api.getModel().info, inputTokens, outputTokens, cacheWriteTokens, cacheReadTokens), cancelReason, streamingFailedMessage, } satisfies ClineApiReqInfo) } - const abortStream = async ( - cancelReason: ClineApiReqCancelReason, - streamingFailedMessage?: string, - ) => { + const abortStream = async (cancelReason: ClineApiReqCancelReason, streamingFailedMessage?: string) => { if (this.diffViewProvider.isEditing) { await this.diffViewProvider.revertChanges() // closes diff view } @@ -3883,13 +2881,9 @@ export class Cline { case "text": assistantMessage += chunk.text // parse raw assistant message into content blocks - const prevLength = - this.assistantMessageContent.length - this.assistantMessageContent = - parseAssistantMessage(assistantMessage) - if ( - this.assistantMessageContent.length > prevLength - ) { + const prevLength = this.assistantMessageContent.length + this.assistantMessageContent = parseAssistantMessage(assistantMessage) + if (this.assistantMessageContent.length > prevLength) { this.userMessageContentReady = false // new content we need to present, reset to false in case previous content set this to true } // present content to user @@ -3908,8 +2902,7 @@ export class Cline { if (this.didRejectTool) { // userContent has a tool rejection, so interrupt the assistant's response to present the user's feedback - assistantMessage += - "\n\n[Response interrupted by user feedback]" + assistantMessage += "\n\n[Response interrupted by user feedback]" // this.userMessageContentReady = true // instead of setting this premptively, we allow the present iterator to finish and set userMessageContentReady when its ready break } @@ -3926,18 +2919,10 @@ export class Cline { // abandoned happens when extension is no longer waiting for the cline instance to finish aborting (error is thrown here when any function in the for loop throws due to this.abort) if (!this.abandoned) { this.abortTask() // if the stream failed, there's various states the task could be in (i.e. could have streamed some tools the user may have executed), so we just resort to replicating a cancel task - await abortStream( - "streaming_failed", - error.message ?? - JSON.stringify(serializeError(error), null, 2), - ) - const history = await this.providerRef - .deref() - ?.getTaskWithId(this.taskId) + await abortStream("streaming_failed", error.message ?? JSON.stringify(serializeError(error), null, 2)) + const history = await this.providerRef.deref()?.getTaskWithId(this.taskId) if (history) { - await this.providerRef - .deref() - ?.initClineWithHistoryItem(history.historyItem) + await this.providerRef.deref()?.initClineWithHistoryItem(history.historyItem) // await this.providerRef.deref()?.postStateToWebview() } } @@ -3954,9 +2939,7 @@ export class Cline { // set any blocks to be complete to allow presentAssistantMessage to finish and set userMessageContentReady to true // (could be a text block that had no subsequent tool uses, or a text block at the very end, or an invalid tool use, etc. whatever the case, presentAssistantMessage relies on these blocks either to be completed or the user to reject a block in order to proceed and eventually set userMessageContentReady to true) - const partialBlocks = this.assistantMessageContent.filter( - (block) => block.partial, - ) + const partialBlocks = this.assistantMessageContent.filter((block) => block.partial) partialBlocks.forEach((block) => { block.partial = false }) @@ -3989,9 +2972,7 @@ export class Cline { await pWaitFor(() => this.userMessageContentReady) // if the model did not tool use, then we need to tell it to either use a tool or attempt_completion - const didToolUse = this.assistantMessageContent.some( - (block) => block.type === "tool_use", - ) + const didToolUse = this.assistantMessageContent.some((block) => block.type === "tool_use") if (!didToolUse) { this.userMessageContent.push({ type: "text", @@ -4000,9 +2981,7 @@ export class Cline { this.consecutiveMistakeCount++ } - const recDidEndLoop = await this.recursivelyMakeClineRequests( - this.userMessageContent, - ) + const recDidEndLoop = await this.recursivelyMakeClineRequests(this.userMessageContent) didEndLoop = recDidEndLoop } else { // if there's no assistant_responses, that means we got no text or tool_use content blocks from API which we should assume is an error @@ -4028,10 +3007,7 @@ export class Cline { } } - async loadContext( - userContent: UserContent, - includeFileDetails: boolean = false, - ) { + async loadContext(userContent: UserContent, includeFileDetails: boolean = false) { return await Promise.all([ // Process userContent array, which contains various block types: // TextBlockParam, ImageBlockParam, ToolUseBlockParam, and ToolResultBlockParam. @@ -4043,42 +3019,22 @@ export class Cline { if (block.type === "text") { return { ...block, - text: await parseMentions( - block.text, - cwd, - this.urlContentFetcher, - ), + text: await parseMentions(block.text, cwd, this.urlContentFetcher), } } else if (block.type === "tool_result") { - const isUserMessage = (text: string) => - text.includes("") || - text.includes("") - if ( - typeof block.content === "string" && - isUserMessage(block.content) - ) { + const isUserMessage = (text: string) => text.includes("") || text.includes("") + if (typeof block.content === "string" && isUserMessage(block.content)) { return { ...block, - content: await parseMentions( - block.content, - cwd, - this.urlContentFetcher, - ), + content: await parseMentions(block.content, cwd, this.urlContentFetcher), } } else if (Array.isArray(block.content)) { const parsedContent = await Promise.all( block.content.map(async (contentBlock) => { - if ( - contentBlock.type === "text" && - isUserMessage(contentBlock.text) - ) { + if (contentBlock.type === "text" && isUserMessage(contentBlock.text)) { return { ...contentBlock, - text: await parseMentions( - contentBlock.text, - cwd, - this.urlContentFetcher, - ), + text: await parseMentions(contentBlock.text, cwd, this.urlContentFetcher), } } return contentBlock @@ -4139,16 +3095,10 @@ export class Cline { if (busyTerminals.length > 0) { // wait for terminals to cool down // terminalWasBusy = allTerminals.some((t) => this.terminalManager.isProcessHot(t.id)) - await pWaitFor( - () => - busyTerminals.every( - (t) => !this.terminalManager.isProcessHot(t.id), - ), - { - interval: 100, - timeout: 15_000, - }, - ).catch(() => {}) + await pWaitFor(() => busyTerminals.every((t) => !this.terminalManager.isProcessHot(t.id)), { + interval: 100, + timeout: 15_000, + }).catch(() => {}) } // we want to get diagnostics AFTER terminal cools down for a few reasons: terminal could be scaffolding a project, dev servers (compilers like webpack) will first re-compile and then send diagnostics, etc @@ -4177,9 +3127,7 @@ export class Cline { terminalDetails += "\n\n# Actively Running Terminals" for (const busyTerminal of busyTerminals) { terminalDetails += `\n## Original command: \`${busyTerminal.lastCommand}\`` - const newOutput = this.terminalManager.getUnretrievedOutput( - busyTerminal.id, - ) + const newOutput = this.terminalManager.getUnretrievedOutput(busyTerminal.id) if (newOutput) { terminalDetails += `\n### New Output\n${newOutput}` } else { @@ -4191,9 +3139,7 @@ export class Cline { if (inactiveTerminals.length > 0) { const inactiveTerminalOutputs = new Map() for (const inactiveTerminal of inactiveTerminals) { - const newOutput = this.terminalManager.getUnretrievedOutput( - inactiveTerminal.id, - ) + const newOutput = this.terminalManager.getUnretrievedOutput(inactiveTerminal.id) if (newOutput) { inactiveTerminalOutputs.set(inactiveTerminal.id, newOutput) } @@ -4201,9 +3147,7 @@ export class Cline { if (inactiveTerminalOutputs.size > 0) { terminalDetails += "\n\n# Inactive Terminals" for (const [terminalId, newOutput] of inactiveTerminalOutputs) { - const inactiveTerminal = inactiveTerminals.find( - (t) => t.id === terminalId, - ) + const inactiveTerminal = inactiveTerminals.find((t) => t.id === terminalId) if (inactiveTerminal) { terminalDetails += `\n## ${inactiveTerminal.lastCommand}` terminalDetails += `\n### New Output\n${newOutput}` @@ -4225,21 +3169,13 @@ export class Cline { if (includeFileDetails) { details += `\n\n# Current Working Directory (${cwd.toPosix()}) Files\n` - const isDesktop = arePathsEqual( - cwd, - path.join(os.homedir(), "Desktop"), - ) + const isDesktop = arePathsEqual(cwd, path.join(os.homedir(), "Desktop")) if (isDesktop) { // don't want to immediately access desktop since it would show permission popup - details += - "(Desktop files not shown automatically. Use list_files to explore if needed.)" + details += "(Desktop files not shown automatically. Use list_files to explore if needed.)" } else { const [files, didHitLimit] = await listFiles(cwd, true, 200) - const result = formatResponse.formatFilesList( - cwd, - files, - didHitLimit, - ) + const result = formatResponse.formatFilesList(cwd, files, didHitLimit) details += result } } diff --git a/src/core/assistant-message/diff.ts b/src/core/assistant-message/diff.ts index cd4a3b04ca..4d577cd77a 100644 --- a/src/core/assistant-message/diff.ts +++ b/src/core/assistant-message/diff.ts @@ -6,11 +6,7 @@ * * Returns [matchIndexStart, matchIndexEnd] if found, or false if not found. */ -function lineTrimmedFallbackMatch( - originalContent: string, - searchContent: string, - startIndex: number, -): [number, number] | false { +function lineTrimmedFallbackMatch(originalContent: string, searchContent: string, startIndex: number): [number, number] | false { // Split both contents into lines const originalLines = originalContent.split("\n") const searchLines = searchContent.split("\n") @@ -29,11 +25,7 @@ function lineTrimmedFallbackMatch( } // For each possible starting position in original content - for ( - let i = startLineNum; - i <= originalLines.length - searchLines.length; - i++ - ) { + for (let i = startLineNum; i <= originalLines.length - searchLines.length; i++) { let matches = true // Try to match all search lines from this position @@ -95,11 +87,7 @@ function lineTrimmedFallbackMatch( * @param startIndex - The character index in originalContent where to start searching * @returns A tuple of [startIndex, endIndex] if a match is found, false otherwise */ -function blockAnchorFallbackMatch( - originalContent: string, - searchContent: string, - startIndex: number, -): [number, number] | false { +function blockAnchorFallbackMatch(originalContent: string, searchContent: string, startIndex: number): [number, number] | false { const originalLines = originalContent.split("\n") const searchLines = searchContent.split("\n") @@ -126,11 +114,7 @@ function blockAnchorFallbackMatch( } // Look for matching start and end anchors - for ( - let i = startLineNum; - i <= originalLines.length - searchBlockSize; - i++ - ) { + for (let i = startLineNum; i <= originalLines.length - searchBlockSize; i++) { // Check if first line matches if (originalLines[i].trim() !== firstLineSearch) { continue @@ -216,11 +200,7 @@ function blockAnchorFallbackMatch( * - If the search block cannot be matched using any of the available matching strategies, * an error is thrown. */ -export async function constructNewFileContent( - diffContent: string, - originalContent: string, - isFinal: boolean, -): Promise { +export async function constructNewFileContent(diffContent: string, originalContent: string, isFinal: boolean): Promise { let result = "" let lastProcessedIndex = 0 @@ -239,9 +219,7 @@ export async function constructNewFileContent( const lastLine = lines[lines.length - 1] if ( lines.length > 0 && - (lastLine.startsWith("<") || - lastLine.startsWith("=") || - lastLine.startsWith(">")) && + (lastLine.startsWith("<") || lastLine.startsWith("=") || lastLine.startsWith(">")) && lastLine !== "<<<<<<< SEARCH" && lastLine !== "=======" && lastLine !== ">>>>>>> REPLACE" @@ -290,29 +268,18 @@ export async function constructNewFileContent( // } // Exact search match scenario - const exactIndex = originalContent.indexOf( - currentSearchContent, - lastProcessedIndex, - ) + const exactIndex = originalContent.indexOf(currentSearchContent, lastProcessedIndex) if (exactIndex !== -1) { searchMatchIndex = exactIndex searchEndIndex = exactIndex + currentSearchContent.length } else { // Attempt fallback line-trimmed matching - const lineMatch = lineTrimmedFallbackMatch( - originalContent, - currentSearchContent, - lastProcessedIndex, - ) + const lineMatch = lineTrimmedFallbackMatch(originalContent, currentSearchContent, lastProcessedIndex) if (lineMatch) { ;[searchMatchIndex, searchEndIndex] = lineMatch } else { // Try block anchor fallback for larger blocks - const blockMatch = blockAnchorFallbackMatch( - originalContent, - currentSearchContent, - lastProcessedIndex, - ) + const blockMatch = blockAnchorFallbackMatch(originalContent, currentSearchContent, lastProcessedIndex) if (blockMatch) { ;[searchMatchIndex, searchEndIndex] = blockMatch } else { @@ -325,10 +292,7 @@ export async function constructNewFileContent( } // Output everything up to the match location - result += originalContent.slice( - lastProcessedIndex, - searchMatchIndex, - ) + result += originalContent.slice(lastProcessedIndex, searchMatchIndex) continue } diff --git a/src/core/assistant-message/index.ts b/src/core/assistant-message/index.ts index ed03120a32..7ad2c27d7b 100644 --- a/src/core/assistant-message/index.ts +++ b/src/core/assistant-message/index.ts @@ -60,9 +60,7 @@ export interface ToolUse { export interface ExecuteCommandToolUse extends ToolUse { name: "execute_command" // Pick, "command"> makes "command" required, but Partial<> makes it optional - params: Partial< - Pick, "command" | "requires_approval"> - > + params: Partial, "command" | "requires_approval">> } export interface ReadFileToolUse extends ToolUse { @@ -82,9 +80,7 @@ export interface ReplaceInFileToolUse extends ToolUse { export interface SearchFilesToolUse extends ToolUse { name: "search_files" - params: Partial< - Pick, "path" | "regex" | "file_pattern"> - > + params: Partial, "path" | "regex" | "file_pattern">> } export interface ListFilesToolUse extends ToolUse { @@ -99,22 +95,12 @@ export interface ListCodeDefinitionNamesToolUse extends ToolUse { export interface BrowserActionToolUse extends ToolUse { name: "browser_action" - params: Partial< - Pick< - Record, - "action" | "url" | "coordinate" | "text" - > - > + params: Partial, "action" | "url" | "coordinate" | "text">> } export interface UseMcpToolToolUse extends ToolUse { name: "use_mcp_tool" - params: Partial< - Pick< - Record, - "server_name" | "tool_name" | "arguments" - > - > + params: Partial, "server_name" | "tool_name" | "arguments">> } export interface AccessMcpResourceToolUse extends ToolUse { diff --git a/src/core/assistant-message/parse-assistant-message.ts b/src/core/assistant-message/parse-assistant-message.ts index 9c6a2308c8..2c37a650b6 100644 --- a/src/core/assistant-message/parse-assistant-message.ts +++ b/src/core/assistant-message/parse-assistant-message.ts @@ -1,12 +1,4 @@ -import { - AssistantMessageContent, - TextContent, - ToolUse, - ToolParamName, - toolParamNames, - toolUseNames, - ToolUseName, -} from "." +import { AssistantMessageContent, TextContent, ToolUse, ToolParamName, toolParamNames, toolUseNames, ToolUseName } from "." export function parseAssistantMessage(assistantMessage: string) { let contentBlocks: AssistantMessageContent[] = [] @@ -24,15 +16,11 @@ export function parseAssistantMessage(assistantMessage: string) { // there should not be a param without a tool use if (currentToolUse && currentParamName) { - const currentParamValue = accumulator.slice( - currentParamValueStartIndex, - ) + const currentParamValue = accumulator.slice(currentParamValueStartIndex) const paramClosingTag = `` if (currentParamValue.endsWith(paramClosingTag)) { // end of param value - currentToolUse.params[currentParamName] = currentParamValue - .slice(0, -paramClosingTag.length) - .trim() + currentToolUse.params[currentParamName] = currentParamValue.slice(0, -paramClosingTag.length).trim() currentParamName = undefined continue } else { @@ -53,16 +41,11 @@ export function parseAssistantMessage(assistantMessage: string) { currentToolUse = undefined continue } else { - const possibleParamOpeningTags = toolParamNames.map( - (name) => `<${name}>`, - ) + const possibleParamOpeningTags = toolParamNames.map((name) => `<${name}>`) for (const paramOpeningTag of possibleParamOpeningTags) { if (accumulator.endsWith(paramOpeningTag)) { // start of a new parameter - currentParamName = paramOpeningTag.slice( - 1, - -1, - ) as ToolParamName + currentParamName = paramOpeningTag.slice(1, -1) as ToolParamName currentParamValueStartIndex = accumulator.length break } @@ -72,28 +55,14 @@ export function parseAssistantMessage(assistantMessage: string) { // special case for write_to_file where file contents could contain the closing tag, in which case the param would have closed and we end up with the rest of the file contents here. To work around this, we get the string between the starting content tag and the LAST content tag. const contentParamName: ToolParamName = "content" - if ( - currentToolUse.name === "write_to_file" && - accumulator.endsWith(``) - ) { - const toolContent = accumulator.slice( - currentToolUseStartIndex, - ) + if (currentToolUse.name === "write_to_file" && accumulator.endsWith(``)) { + const toolContent = accumulator.slice(currentToolUseStartIndex) const contentStartTag = `<${contentParamName}>` const contentEndTag = `` - const contentStartIndex = - toolContent.indexOf(contentStartTag) + - contentStartTag.length - const contentEndIndex = - toolContent.lastIndexOf(contentEndTag) - if ( - contentStartIndex !== -1 && - contentEndIndex !== -1 && - contentEndIndex > contentStartIndex - ) { - currentToolUse.params[contentParamName] = toolContent - .slice(contentStartIndex, contentEndIndex) - .trim() + const contentStartIndex = toolContent.indexOf(contentStartTag) + contentStartTag.length + const contentEndIndex = toolContent.lastIndexOf(contentEndTag) + if (contentStartIndex !== -1 && contentEndIndex !== -1 && contentEndIndex > contentStartIndex) { + currentToolUse.params[contentParamName] = toolContent.slice(contentStartIndex, contentEndIndex).trim() } } @@ -105,9 +74,7 @@ export function parseAssistantMessage(assistantMessage: string) { // no currentToolUse let didStartToolUse = false - const possibleToolUseOpeningTags = toolUseNames.map( - (name) => `<${name}>`, - ) + const possibleToolUseOpeningTags = toolUseNames.map((name) => `<${name}>`) for (const toolUseOpeningTag of possibleToolUseOpeningTags) { if (accumulator.endsWith(toolUseOpeningTag)) { // start of a new tool use @@ -151,9 +118,7 @@ export function parseAssistantMessage(assistantMessage: string) { // stream did not complete tool call, add it as partial if (currentParamName) { // tool call has a parameter that was not completed - currentToolUse.params[currentParamName] = accumulator - .slice(currentParamValueStartIndex) - .trim() + currentToolUse.params[currentParamName] = accumulator.slice(currentParamValueStartIndex).trim() } contentBlocks.push(currentToolUse) } diff --git a/src/core/mentions/index.ts b/src/core/mentions/index.ts index aad682e94e..1c9c122d1d 100644 --- a/src/core/mentions/index.ts +++ b/src/core/mentions/index.ts @@ -15,18 +15,13 @@ export function openMention(mention?: string): void { if (mention.startsWith("/")) { const relPath = mention.slice(1) - const cwd = vscode.workspace.workspaceFolders - ?.map((folder) => folder.uri.fsPath) - .at(0) + const cwd = vscode.workspace.workspaceFolders?.map((folder) => folder.uri.fsPath).at(0) if (!cwd) { return } const absPath = path.resolve(cwd, relPath) if (mention.endsWith("/")) { - vscode.commands.executeCommand( - "revealInExplorer", - vscode.Uri.file(absPath), - ) + vscode.commands.executeCommand("revealInExplorer", vscode.Uri.file(absPath)) // vscode.commands.executeCommand("vscode.openFolder", , { forceNewWindow: false }) opens in new window } else { openFile(absPath) @@ -38,11 +33,7 @@ export function openMention(mention?: string): void { } } -export async function parseMentions( - text: string, - cwd: string, - urlContentFetcher: UrlContentFetcher, -): Promise { +export async function parseMentions(text: string, cwd: string, urlContentFetcher: UrlContentFetcher): Promise { const mentions: Set = new Set() let parsedText = text.replace(mentionRegexGlobal, (match, mention) => { mentions.add(mention) @@ -59,18 +50,14 @@ export async function parseMentions( return match }) - const urlMention = Array.from(mentions).find((mention) => - mention.startsWith("http"), - ) + const urlMention = Array.from(mentions).find((mention) => mention.startsWith("http")) let launchBrowserError: Error | undefined if (urlMention) { try { await urlContentFetcher.launchBrowser() } catch (error) { launchBrowserError = error - vscode.window.showErrorMessage( - `Error fetching content for ${urlMention}: ${error.message}`, - ) + vscode.window.showErrorMessage(`Error fetching content for ${urlMention}: ${error.message}`) } } @@ -81,13 +68,10 @@ export async function parseMentions( result = `Error fetching content: ${launchBrowserError.message}` } else { try { - const markdown = - await urlContentFetcher.urlToMarkdown(mention) + const markdown = await urlContentFetcher.urlToMarkdown(mention) result = markdown } catch (error) { - vscode.window.showErrorMessage( - `Error fetching content for ${mention}: ${error.message}`, - ) + vscode.window.showErrorMessage(`Error fetching content for ${mention}: ${error.message}`) result = `Error fetching content: ${error.message}` } } @@ -129,10 +113,7 @@ export async function parseMentions( return parsedText } -async function getFileOrFolderContent( - mentionPath: string, - cwd: string, -): Promise { +async function getFileOrFolderContent(mentionPath: string, cwd: string): Promise { const absPath = path.resolve(cwd, mentionPath) try { @@ -160,14 +141,11 @@ async function getFileOrFolderContent( fileContentPromises.push( (async () => { try { - const isBinary = await isBinaryFile( - absoluteFilePath, - ).catch(() => false) + const isBinary = await isBinaryFile(absoluteFilePath).catch(() => false) if (isBinary) { return undefined } - const content = - await extractTextFromFile(absoluteFilePath) + const content = await extractTextFromFile(absoluteFilePath) return `\n${content}\n` } catch (error) { return undefined @@ -181,17 +159,13 @@ async function getFileOrFolderContent( folderContent += `${linePrefix}${entry.name}\n` } }) - const fileContents = ( - await Promise.all(fileContentPromises) - ).filter((content) => content) + const fileContents = (await Promise.all(fileContentPromises)).filter((content) => content) return `${folderContent}\n${fileContents.join("\n\n")}`.trim() } else { return `(Failed to read contents of ${mentionPath})` } } catch (error) { - throw new Error( - `Failed to access path "${mentionPath}": ${error.message}`, - ) + throw new Error(`Failed to access path "${mentionPath}": ${error.message}`) } } diff --git a/src/core/prompts/responses.ts b/src/core/prompts/responses.ts index 15e0ced3b5..e932f4042c 100644 --- a/src/core/prompts/responses.ts +++ b/src/core/prompts/responses.ts @@ -8,8 +8,7 @@ export const formatResponse = { toolDeniedWithFeedback: (feedback?: string) => `The user denied this operation and provided the following feedback:\n\n${feedback}\n`, - toolError: (error?: string) => - `The tool execution failed with the following error:\n\n${error}\n`, + toolError: (error?: string) => `The tool execution failed with the following error:\n\n${error}\n`, noToolsUsed: () => `[ERROR] You did not use a tool in your previous response! Please retry with a tool use. @@ -32,14 +31,10 @@ Otherwise, if you have not completed the task and do not need additional informa invalidMcpToolArgumentError: (serverName: string, toolName: string) => `Invalid JSON argument used with ${serverName} for ${toolName}. Please retry with a properly formatted JSON argument.`, - toolResult: ( - text: string, - images?: string[], - ): string | Array => { + toolResult: (text: string, images?: string[]): string | Array => { if (images && images.length > 0) { const textBlock: Anthropic.TextBlockParam = { type: "text", text } - const imageBlocks: Anthropic.ImageBlockParam[] = - formatImagesIntoBlocks(images) + const imageBlocks: Anthropic.ImageBlockParam[] = formatImagesIntoBlocks(images) // Placing images after text leads to better results return [textBlock, ...imageBlocks] } else { @@ -51,11 +46,7 @@ Otherwise, if you have not completed the task and do not need additional informa return formatImagesIntoBlocks(images) }, - formatFilesList: ( - absolutePath: string, - files: string[], - didHitLimit: boolean, - ): string => { + formatFilesList: (absolutePath: string, files: string[], didHitLimit: boolean): string => { const sorted = files .map((file) => { // convert absolute path to relative path @@ -66,11 +57,7 @@ Otherwise, if you have not completed the task and do not need additional informa .sort((a, b) => { const aParts = a.split("/") // only works if we use toPosix first const bParts = b.split("/") - for ( - let i = 0; - i < Math.min(aParts.length, bParts.length); - i++ - ) { + for (let i = 0; i < Math.min(aParts.length, bParts.length); i++) { if (aParts[i] !== bParts[i]) { // If one is a directory and the other isn't at this level, sort the directory first if (i + 1 === aParts.length && i + 1 < bParts.length) { @@ -94,27 +81,16 @@ Otherwise, if you have not completed the task and do not need additional informa return `${sorted.join( "\n", )}\n\n(File list truncated. Use list_files on specific subdirectories if you need to explore further.)` - } else if ( - sorted.length === 0 || - (sorted.length === 1 && sorted[0] === "") - ) { + } else if (sorted.length === 0 || (sorted.length === 1 && sorted[0] === "")) { return "No files found." } else { return sorted.join("\n") } }, - createPrettyPatch: ( - filename = "file", - oldStr?: string, - newStr?: string, - ) => { + createPrettyPatch: (filename = "file", oldStr?: string, newStr?: string) => { // strings cannot be undefined or diff throws exception - const patch = diff.createPatch( - filename.toPosix(), - oldStr || "", - newStr || "", - ) + const patch = diff.createPatch(filename.toPosix(), oldStr || "", newStr || "") const lines = patch.split("\n") const prettyPatchLines = lines.slice(4) return prettyPatchLines.join("\n") @@ -122,9 +98,7 @@ Otherwise, if you have not completed the task and do not need additional informa } // to avoid circular dependency -const formatImagesIntoBlocks = ( - images?: string[], -): Anthropic.ImageBlockParam[] => { +const formatImagesIntoBlocks = (images?: string[]): Anthropic.ImageBlockParam[] => { return images ? images.map((dataUrl) => { // data:image/png;base64,base64string diff --git a/src/core/prompts/system.ts b/src/core/prompts/system.ts index e9722b20f8..1e39799303 100644 --- a/src/core/prompts/system.ts +++ b/src/core/prompts/system.ts @@ -362,17 +362,11 @@ ${ .join("\n\n") const templates = server.resourceTemplates - ?.map( - (template) => - `- ${template.uriTemplate} (${template.name}): ${template.description}`, - ) + ?.map((template) => `- ${template.uriTemplate} (${template.name}): ${template.description}`) .join("\n") const resources = server.resources - ?.map( - (resource) => - `- ${resource.uri} (${resource.name}): ${resource.description}`, - ) + ?.map((resource) => `- ${resource.uri} (${resource.name}): ${resource.description}`) .join("\n") const config = JSON.parse(server.config) @@ -380,12 +374,8 @@ ${ return ( `## ${server.name} (\`${config.command}${config.args && Array.isArray(config.args) ? ` ${config.args.join(" ")}` : ""}\`)` + (tools ? `\n\n### Available Tools\n${tools}` : "") + - (templates - ? `\n\n### Resource Templates\n${templates}` - : "") + - (resources - ? `\n\n### Direct Resources\n${resources}` - : "") + (templates ? `\n\n### Resource Templates\n${templates}` : "") + + (resources ? `\n\n### Direct Resources\n${resources}` : "") ) }) .join("\n\n")}` @@ -899,10 +889,7 @@ You accomplish a given task iteratively, breaking it down into clear steps and w 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.` -export function addUserInstructions( - settingsCustomInstructions?: string, - clineRulesFileInstructions?: string, -) { +export function addUserInstructions(settingsCustomInstructions?: string, clineRulesFileInstructions?: string) { let customInstructions = "" if (settingsCustomInstructions) { customInstructions += settingsCustomInstructions + "\n\n" diff --git a/src/core/webview/ClineProvider.ts b/src/core/webview/ClineProvider.ts index 69847a1b29..7acb225e70 100644 --- a/src/core/webview/ClineProvider.ts +++ b/src/core/webview/ClineProvider.ts @@ -16,19 +16,13 @@ import { ApiProvider, ModelInfo } from "../../shared/api" import { findLast } from "../../shared/array" import { ExtensionMessage, ExtensionState } from "../../shared/ExtensionMessage" import { HistoryItem } from "../../shared/HistoryItem" -import { - ClineCheckpointRestore, - WebviewMessage, -} from "../../shared/WebviewMessage" +import { ClineCheckpointRestore, WebviewMessage } from "../../shared/WebviewMessage" import { fileExistsAtPath } from "../../utils/fs" import { Cline } from "../Cline" import { openMention } from "../mentions" import { getNonce } from "./getNonce" import { getUri } from "./getUri" -import { - AutoApprovalSettings, - DEFAULT_AUTO_APPROVAL_SETTINGS, -} from "../../shared/AutoApprovalSettings" +import { AutoApprovalSettings, DEFAULT_AUTO_APPROVAL_SETTINGS } from "../../shared/AutoApprovalSettings" /* https://github.com/microsoft/vscode-webview-ui-toolkit-samples/blob/main/default/weather-webview/src/providers/WeatherViewProvider.ts @@ -125,10 +119,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { } public static getVisibleInstance(): ClineProvider | undefined { - return findLast( - Array.from(this.activeInstances), - (instance) => instance.view?.visible === true, - ) + return findLast(Array.from(this.activeInstances), (instance) => instance.view?.visible === true) } resolveWebviewView( @@ -219,22 +210,13 @@ export class ClineProvider implements vscode.WebviewViewProvider { async initClineWithTask(task?: string, images?: string[]) { await this.clearTask() // ensures that an exising task doesn't exist before starting a new one, although this shouldn't be possible since user must clear task before starting a new one - const { apiConfiguration, customInstructions, autoApprovalSettings } = - await this.getState() - this.cline = new Cline( - this, - apiConfiguration, - autoApprovalSettings, - customInstructions, - task, - images, - ) + const { apiConfiguration, customInstructions, autoApprovalSettings } = await this.getState() + this.cline = new Cline(this, apiConfiguration, autoApprovalSettings, customInstructions, task, images) } async initClineWithHistoryItem(historyItem: HistoryItem) { await this.clearTask() - const { apiConfiguration, customInstructions, autoApprovalSettings } = - await this.getState() + const { apiConfiguration, customInstructions, autoApprovalSettings } = await this.getState() this.cline = new Cline( this, apiConfiguration, @@ -267,21 +249,9 @@ export class ClineProvider implements vscode.WebviewViewProvider { // then convert it to a uri we can use in the webview. // The CSS file from the React build output - const stylesUri = getUri(webview, this.context.extensionUri, [ - "webview-ui", - "build", - "static", - "css", - "main.css", - ]) + const stylesUri = getUri(webview, this.context.extensionUri, ["webview-ui", "build", "static", "css", "main.css"]) // The JS file from the React build output - const scriptUri = getUri(webview, this.context.extensionUri, [ - "webview-ui", - "build", - "static", - "js", - "main.js", - ]) + const scriptUri = getUri(webview, this.context.extensionUri, ["webview-ui", "build", "static", "js", "main.js"]) // The codicon font from the React build output // https://github.com/microsoft/vscode-extension-samples/blob/main/webview-codicons-sample/src/extension.ts @@ -369,25 +339,19 @@ export class ClineProvider implements vscode.WebviewViewProvider { // gui relies on model info to be up-to-date to provide the most accurate pricing, so we need to fetch the latest details on launch. // we do this for all users since many users switch between api providers and if they were to switch back to openrouter it would be showing outdated model info if we hadn't retrieved the latest at this point // (see normalizeApiConfiguration > openrouter) - this.refreshOpenRouterModels().then( - async (openRouterModels) => { - if (openRouterModels) { - // update model info in state (this needs to be done here since we don't want to update state while settings is open, and we may refresh models there) - const { apiConfiguration } = - await this.getState() - if (apiConfiguration.openRouterModelId) { - await this.updateGlobalState( - "openRouterModelInfo", - openRouterModels[ - apiConfiguration - .openRouterModelId - ], - ) - await this.postStateToWebview() - } + this.refreshOpenRouterModels().then(async (openRouterModels) => { + if (openRouterModels) { + // update model info in state (this needs to be done here since we don't want to update state while settings is open, and we may refresh models there) + const { apiConfiguration } = await this.getState() + if (apiConfiguration.openRouterModelId) { + await this.updateGlobalState( + "openRouterModelInfo", + openRouterModels[apiConfiguration.openRouterModelId], + ) + await this.postStateToWebview() } - }, - ) + } + }) break case "newTask": // Code that should run in response to the hello message command @@ -398,10 +362,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { // Could also do this in extension .ts //this.postMessageToWebview({ type: "text", text: `Extension: ${Date.now()}` }) // initializing new instance of Cline will make sure that any agentically running promises in old instance don't affect our new task. this essentially creates a fresh slate for the new task - await this.initClineWithTask( - message.text, - message.images, - ) + await this.initClineWithTask(message.text, message.images) break case "apiConfiguration": if (message.apiConfiguration) { @@ -432,92 +393,33 @@ export class ClineProvider implements vscode.WebviewViewProvider { openRouterModelId, openRouterModelInfo, } = message.apiConfiguration - await this.updateGlobalState( - "apiProvider", - apiProvider, - ) - await this.updateGlobalState( - "apiModelId", - apiModelId, - ) + await this.updateGlobalState("apiProvider", apiProvider) + await this.updateGlobalState("apiModelId", apiModelId) await this.storeSecret("apiKey", apiKey) - await this.storeSecret( - "openRouterApiKey", - openRouterApiKey, - ) + await this.storeSecret("openRouterApiKey", openRouterApiKey) await this.storeSecret("awsAccessKey", awsAccessKey) await this.storeSecret("awsSecretKey", awsSecretKey) - await this.storeSecret( - "awsSessionToken", - awsSessionToken, - ) + await this.storeSecret("awsSessionToken", awsSessionToken) await this.updateGlobalState("awsRegion", awsRegion) - await this.updateGlobalState( - "awsUseCrossRegionInference", - awsUseCrossRegionInference, - ) - await this.updateGlobalState( - "vertexProjectId", - vertexProjectId, - ) - await this.updateGlobalState( - "vertexRegion", - vertexRegion, - ) - await this.updateGlobalState( - "openAiBaseUrl", - openAiBaseUrl, - ) + await this.updateGlobalState("awsUseCrossRegionInference", awsUseCrossRegionInference) + await this.updateGlobalState("vertexProjectId", vertexProjectId) + await this.updateGlobalState("vertexRegion", vertexRegion) + await this.updateGlobalState("openAiBaseUrl", openAiBaseUrl) await this.storeSecret("openAiApiKey", openAiApiKey) - await this.updateGlobalState( - "openAiModelId", - openAiModelId, - ) - await this.updateGlobalState( - "ollamaModelId", - ollamaModelId, - ) - await this.updateGlobalState( - "ollamaBaseUrl", - ollamaBaseUrl, - ) - await this.updateGlobalState( - "lmStudioModelId", - lmStudioModelId, - ) - await this.updateGlobalState( - "lmStudioBaseUrl", - lmStudioBaseUrl, - ) - await this.updateGlobalState( - "anthropicBaseUrl", - anthropicBaseUrl, - ) + await this.updateGlobalState("openAiModelId", openAiModelId) + await this.updateGlobalState("ollamaModelId", ollamaModelId) + await this.updateGlobalState("ollamaBaseUrl", ollamaBaseUrl) + await this.updateGlobalState("lmStudioModelId", lmStudioModelId) + await this.updateGlobalState("lmStudioBaseUrl", lmStudioBaseUrl) + await this.updateGlobalState("anthropicBaseUrl", anthropicBaseUrl) await this.storeSecret("geminiApiKey", geminiApiKey) - await this.storeSecret( - "openAiNativeApiKey", - openAiNativeApiKey, - ) - await this.storeSecret( - "deepSeekApiKey", - deepSeekApiKey, - ) - await this.updateGlobalState( - "azureApiVersion", - azureApiVersion, - ) - await this.updateGlobalState( - "openRouterModelId", - openRouterModelId, - ) - await this.updateGlobalState( - "openRouterModelInfo", - openRouterModelInfo, - ) + await this.storeSecret("openAiNativeApiKey", openAiNativeApiKey) + await this.storeSecret("deepSeekApiKey", deepSeekApiKey) + await this.updateGlobalState("azureApiVersion", azureApiVersion) + await this.updateGlobalState("openRouterModelId", openRouterModelId) + await this.updateGlobalState("openRouterModelInfo", openRouterModelInfo) if (this.cline) { - this.cline.api = buildApiHandler( - message.apiConfiguration, - ) + this.cline.api = buildApiHandler(message.apiConfiguration) } } await this.postStateToWebview() @@ -527,23 +429,15 @@ export class ClineProvider implements vscode.WebviewViewProvider { break case "autoApprovalSettings": if (message.autoApprovalSettings) { - await this.updateGlobalState( - "autoApprovalSettings", - message.autoApprovalSettings, - ) + await this.updateGlobalState("autoApprovalSettings", message.autoApprovalSettings) if (this.cline) { - this.cline.autoApprovalSettings = - message.autoApprovalSettings + this.cline.autoApprovalSettings = message.autoApprovalSettings } await this.postStateToWebview() } break case "askResponse": - this.cline?.handleWebviewAskResponse( - message.askResponse!, - message.text, - message.images, - ) + this.cline?.handleWebviewAskResponse(message.askResponse!, message.text, message.images) break case "clearTask": // newTask will start a new task with a given task text, while clear task resets the current session and allows for a new task to be started @@ -551,10 +445,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { await this.postStateToWebview() break case "didShowAnnouncement": - await this.updateGlobalState( - "lastShownAnnouncementId", - this.latestAnnouncementId, - ) + await this.updateGlobalState("lastShownAnnouncementId", this.latestAnnouncementId) await this.postStateToWebview() break case "selectImages": @@ -583,18 +474,14 @@ export class ClineProvider implements vscode.WebviewViewProvider { await this.resetState() break case "requestOllamaModels": - const ollamaModels = await this.getOllamaModels( - message.text, - ) + const ollamaModels = await this.getOllamaModels(message.text) this.postMessageToWebview({ type: "ollamaModels", ollamaModels, }) break case "requestLmStudioModels": - const lmStudioModels = await this.getLmStudioModels( - message.text, - ) + const lmStudioModels = await this.getLmStudioModels(message.text) this.postMessageToWebview({ type: "lmStudioModels", lmStudioModels, @@ -614,10 +501,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { break case "checkpointDiff": { if (message.number) { - await this.cline?.presentMultifileDiff( - message.number, - false, - ) + await this.cline?.presentMultifileDiff(message.number, false) } break } @@ -626,30 +510,19 @@ export class ClineProvider implements vscode.WebviewViewProvider { // cancel task waits for any open editor to be reverted and starts a new cline instance if (message.number) { // wait for messages to be loaded - await pWaitFor( - () => this.cline?.isInitialized === true, - { - timeout: 3_000, - }, - ).catch(() => { - console.error( - "Failed to init new cline instance", - ) + await pWaitFor(() => this.cline?.isInitialized === true, { + timeout: 3_000, + }).catch(() => { + console.error("Failed to init new cline instance") }) // NOTE: cancelTask awaits abortTask, which awaits diffViewProvider.revertChanges, which reverts any edited files, allowing us to reset to a checkpoint rather than running into a state where the revertChanges function is called alongside or after the checkpoint reset - await this.cline?.restoreCheckpoint( - message.number, - message.text! as ClineCheckpointRestore, - ) + await this.cline?.restoreCheckpoint(message.number, message.text! as ClineCheckpointRestore) } break } case "taskCompletionViewChanges": { if (message.number) { - await this.cline?.presentMultifileDiff( - message.number, - true, - ) + await this.cline?.presentMultifileDiff(message.number, true) } break } @@ -657,8 +530,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { this.cancelTask() break case "openMcpSettings": { - const mcpSettingsFilePath = - await this.mcpHub?.getMcpSettingsFilePath() + const mcpSettingsFilePath = await this.mcpHub?.getMcpSettingsFilePath() if (mcpSettingsFilePath) { openFile(mcpSettingsFilePath) } @@ -668,10 +540,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { try { await this.mcpHub?.restartConnection(message.text!) } catch (error) { - console.error( - `Failed to retry connection for ${message.text}:`, - error, - ) + console.error(`Failed to retry connection for ${message.text}:`, error) } break } @@ -693,10 +562,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { console.error("Failed to abort task", error) } await pWaitFor( - () => - this.cline === undefined || - this.cline.isStreaming === false || - this.cline.didFinishAbortingStream, + () => this.cline === undefined || this.cline.isStreaming === false || this.cline.didFinishAbortingStream, { timeout: 3_000, }, @@ -714,10 +580,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { async updateCustomInstructions(instructions?: string) { // User may be clearing the field - await this.updateGlobalState( - "customInstructions", - instructions || undefined, - ) + await this.updateGlobalState("customInstructions", instructions || undefined) if (this.cline) { this.cline.customInstructions = instructions || undefined } @@ -727,12 +590,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { // MCP async ensureMcpServersDirectoryExists(): Promise { - const mcpServersDir = path.join( - os.homedir(), - "Documents", - "Cline", - "MCP", - ) + const mcpServersDir = path.join(os.homedir(), "Documents", "Cline", "MCP") try { await fs.mkdir(mcpServersDir, { recursive: true }) } catch (error) { @@ -742,10 +600,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { } async ensureSettingsDirectoryExists(): Promise { - const settingsDir = path.join( - this.context.globalStorageUri.fsPath, - "settings", - ) + const settingsDir = path.join(this.context.globalStorageUri.fsPath, "settings") await fs.mkdir(settingsDir, { recursive: true }) return settingsDir } @@ -761,8 +616,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { return [] } const response = await axios.get(`${baseUrl}/api/tags`) - const modelsArray = - response.data?.models?.map((model: any) => model.name) || [] + const modelsArray = response.data?.models?.map((model: any) => model.name) || [] const models = [...new Set(modelsArray)] return models } catch (error) { @@ -781,8 +635,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { return [] } const response = await axios.get(`${baseUrl}/v1/models`) - const modelsArray = - response.data?.data?.map((model: any) => model.id) || [] + const modelsArray = response.data?.data?.map((model: any) => model.id) || [] const models = [...new Set(modelsArray)] return models } catch (error) { @@ -795,10 +648,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { async handleOpenRouterCallback(code: string) { let apiKey: string try { - const response = await axios.post( - "https://openrouter.ai/api/v1/auth/keys", - { code }, - ) + const response = await axios.post("https://openrouter.ai/api/v1/auth/keys", { code }) if (response.data && response.data.key) { apiKey = response.data.key } else { @@ -823,43 +673,27 @@ export class ClineProvider implements vscode.WebviewViewProvider { } private async ensureCacheDirectoryExists(): Promise { - const cacheDir = path.join( - this.context.globalStorageUri.fsPath, - "cache", - ) + const cacheDir = path.join(this.context.globalStorageUri.fsPath, "cache") await fs.mkdir(cacheDir, { recursive: true }) return cacheDir } - async readOpenRouterModels(): Promise< - Record | undefined - > { - const openRouterModelsFilePath = path.join( - await this.ensureCacheDirectoryExists(), - GlobalFileNames.openRouterModels, - ) + async readOpenRouterModels(): Promise | undefined> { + const openRouterModelsFilePath = path.join(await this.ensureCacheDirectoryExists(), GlobalFileNames.openRouterModels) const fileExists = await fileExistsAtPath(openRouterModelsFilePath) if (fileExists) { - const fileContents = await fs.readFile( - openRouterModelsFilePath, - "utf8", - ) + const fileContents = await fs.readFile(openRouterModelsFilePath, "utf8") return JSON.parse(fileContents) } return undefined } async refreshOpenRouterModels() { - const openRouterModelsFilePath = path.join( - await this.ensureCacheDirectoryExists(), - GlobalFileNames.openRouterModels, - ) + const openRouterModelsFilePath = path.join(await this.ensureCacheDirectoryExists(), GlobalFileNames.openRouterModels) let models: Record = {} try { - const response = await axios.get( - "https://openrouter.ai/api/v1/models", - ) + const response = await axios.get("https://openrouter.ai/api/v1/models") /* { "id": "anthropic/claude-3.5-sonnet", @@ -898,8 +732,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { const modelInfo: ModelInfo = { maxTokens: rawModel.top_provider?.max_completion_tokens, contextWindow: rawModel.context_length, - supportsImages: - rawModel.architecture?.modality?.includes("image"), + supportsImages: rawModel.architecture?.modality?.includes("image"), supportsPromptCache: false, inputPrice: parsePrice(rawModel.pricing?.prompt), outputPrice: parsePrice(rawModel.pricing?.completion), @@ -981,32 +814,15 @@ export class ClineProvider implements vscode.WebviewViewProvider { uiMessagesFilePath: string apiConversationHistory: Anthropic.MessageParam[] }> { - const history = - ((await this.getGlobalState("taskHistory")) as - | HistoryItem[] - | undefined) || [] + const history = ((await this.getGlobalState("taskHistory")) as HistoryItem[] | undefined) || [] const historyItem = history.find((item) => item.id === id) if (historyItem) { - const taskDirPath = path.join( - this.context.globalStorageUri.fsPath, - "tasks", - id, - ) - const apiConversationHistoryFilePath = path.join( - taskDirPath, - GlobalFileNames.apiConversationHistory, - ) - const uiMessagesFilePath = path.join( - taskDirPath, - GlobalFileNames.uiMessages, - ) - const fileExists = await fileExistsAtPath( - apiConversationHistoryFilePath, - ) + const taskDirPath = path.join(this.context.globalStorageUri.fsPath, "tasks", id) + const apiConversationHistoryFilePath = path.join(taskDirPath, GlobalFileNames.apiConversationHistory) + const uiMessagesFilePath = path.join(taskDirPath, GlobalFileNames.uiMessages) + const fileExists = await fileExistsAtPath(apiConversationHistoryFilePath) if (fileExists) { - const apiConversationHistory = JSON.parse( - await fs.readFile(apiConversationHistoryFilePath, "utf8"), - ) + const apiConversationHistory = JSON.parse(await fs.readFile(apiConversationHistoryFilePath, "utf8")) return { historyItem, taskDirPath, @@ -1035,8 +851,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { } async exportTaskWithId(id: string) { - const { historyItem, apiConversationHistory } = - await this.getTaskWithId(id) + const { historyItem, apiConversationHistory } = await this.getTaskWithId(id) await downloadTask(historyItem.ts, apiConversationHistory) } @@ -1045,18 +860,12 @@ export class ClineProvider implements vscode.WebviewViewProvider { await this.clearTask() } - const { - taskDirPath, - apiConversationHistoryFilePath, - uiMessagesFilePath, - } = await this.getTaskWithId(id) + const { taskDirPath, apiConversationHistoryFilePath, uiMessagesFilePath } = await this.getTaskWithId(id) await this.deleteTaskFromState(id) // Delete the task files - const apiConversationHistoryFileExists = await fileExistsAtPath( - apiConversationHistoryFilePath, - ) + const apiConversationHistoryFileExists = await fileExistsAtPath(apiConversationHistoryFilePath) if (apiConversationHistoryFileExists) { await fs.unlink(apiConversationHistoryFilePath) } @@ -1064,10 +873,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { if (uiMessagesFileExists) { await fs.unlink(uiMessagesFilePath) } - const legacyMessagesFilePath = path.join( - taskDirPath, - "claude_messages.json", - ) + const legacyMessagesFilePath = path.join(taskDirPath, "claude_messages.json") if (await fileExistsAtPath(legacyMessagesFilePath)) { await fs.unlink(legacyMessagesFilePath) } @@ -1078,10 +884,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { try { await fs.rm(checkpointsDir, { recursive: true, force: true }) } catch (error) { - console.error( - `Failed to delete checkpoints directory for task ${id}:`, - error, - ) + console.error(`Failed to delete checkpoints directory for task ${id}:`, error) // Continue with deletion of task directory - don't throw since this is a cleanup operation } } @@ -1091,10 +894,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { async deleteTaskFromState(id: string) { // Remove the task from history - const taskHistory = - ((await this.getGlobalState("taskHistory")) as - | HistoryItem[] - | undefined) || [] + const taskHistory = ((await this.getGlobalState("taskHistory")) as HistoryItem[] | undefined) || [] const updatedTaskHistory = taskHistory.filter((task) => task.id !== id) await this.updateGlobalState("taskHistory", updatedTaskHistory) @@ -1108,31 +908,18 @@ export class ClineProvider implements vscode.WebviewViewProvider { } async getStateToPostToWebview(): Promise { - const { - apiConfiguration, - lastShownAnnouncementId, - customInstructions, - taskHistory, - autoApprovalSettings, - } = await this.getState() + const { apiConfiguration, lastShownAnnouncementId, customInstructions, taskHistory, autoApprovalSettings } = + await this.getState() return { version: this.context.extension?.packageJSON?.version ?? "", apiConfiguration, customInstructions, uriScheme: vscode.env.uriScheme, - currentTaskItem: this.cline?.taskId - ? (taskHistory || []).find( - (item) => item.id === this.cline?.taskId, - ) - : undefined, - checkpointTrackerErrorMessage: - this.cline?.checkpointTrackerErrorMessage, + currentTaskItem: this.cline?.taskId ? (taskHistory || []).find((item) => item.id === this.cline?.taskId) : undefined, + checkpointTrackerErrorMessage: this.cline?.checkpointTrackerErrorMessage, clineMessages: this.cline?.clineMessages || [], - taskHistory: (taskHistory || []) - .filter((item) => item.ts && item.task) - .sort((a, b) => b.ts - a.ts), - shouldShowAnnouncement: - lastShownAnnouncementId !== this.latestAnnouncementId, + taskHistory: (taskHistory || []).filter((item) => item.ts && item.task).sort((a, b) => b.ts - a.ts), + shouldShowAnnouncement: lastShownAnnouncementId !== this.latestAnnouncementId, autoApprovalSettings, } } @@ -1220,9 +1007,7 @@ export class ClineProvider implements vscode.WebviewViewProvider { taskHistory, autoApprovalSettings, ] = await Promise.all([ - this.getGlobalState("apiProvider") as Promise< - ApiProvider | undefined - >, + this.getGlobalState("apiProvider") as Promise, this.getGlobalState("apiModelId") as Promise, this.getSecret("apiKey") as Promise, this.getSecret("openRouterApiKey") as Promise, @@ -1230,51 +1015,27 @@ export class ClineProvider implements vscode.WebviewViewProvider { this.getSecret("awsSecretKey") as Promise, this.getSecret("awsSessionToken") as Promise, this.getGlobalState("awsRegion") as Promise, - this.getGlobalState("awsUseCrossRegionInference") as Promise< - boolean | undefined - >, - this.getGlobalState("vertexProjectId") as Promise< - string | undefined - >, + this.getGlobalState("awsUseCrossRegionInference") as Promise, + this.getGlobalState("vertexProjectId") as Promise, this.getGlobalState("vertexRegion") as Promise, this.getGlobalState("openAiBaseUrl") as Promise, this.getSecret("openAiApiKey") as Promise, this.getGlobalState("openAiModelId") as Promise, this.getGlobalState("ollamaModelId") as Promise, this.getGlobalState("ollamaBaseUrl") as Promise, - this.getGlobalState("lmStudioModelId") as Promise< - string | undefined - >, - this.getGlobalState("lmStudioBaseUrl") as Promise< - string | undefined - >, - this.getGlobalState("anthropicBaseUrl") as Promise< - string | undefined - >, + this.getGlobalState("lmStudioModelId") as Promise, + this.getGlobalState("lmStudioBaseUrl") as Promise, + this.getGlobalState("anthropicBaseUrl") as Promise, this.getSecret("geminiApiKey") as Promise, this.getSecret("openAiNativeApiKey") as Promise, this.getSecret("deepSeekApiKey") as Promise, - this.getGlobalState("azureApiVersion") as Promise< - string | undefined - >, - this.getGlobalState("openRouterModelId") as Promise< - string | undefined - >, - this.getGlobalState("openRouterModelInfo") as Promise< - ModelInfo | undefined - >, - this.getGlobalState("lastShownAnnouncementId") as Promise< - string | undefined - >, - this.getGlobalState("customInstructions") as Promise< - string | undefined - >, - this.getGlobalState("taskHistory") as Promise< - HistoryItem[] | undefined - >, - this.getGlobalState("autoApprovalSettings") as Promise< - AutoApprovalSettings | undefined - >, + this.getGlobalState("azureApiVersion") as Promise, + this.getGlobalState("openRouterModelId") as Promise, + this.getGlobalState("openRouterModelInfo") as Promise, + this.getGlobalState("lastShownAnnouncementId") as Promise, + this.getGlobalState("customInstructions") as Promise, + this.getGlobalState("taskHistory") as Promise, + this.getGlobalState("autoApprovalSettings") as Promise, ]) let apiProvider: ApiProvider @@ -1322,14 +1083,12 @@ export class ClineProvider implements vscode.WebviewViewProvider { lastShownAnnouncementId, customInstructions, taskHistory, - autoApprovalSettings: - autoApprovalSettings || DEFAULT_AUTO_APPROVAL_SETTINGS, // default value can be 0 or empty string + autoApprovalSettings: autoApprovalSettings || DEFAULT_AUTO_APPROVAL_SETTINGS, // default value can be 0 or empty string } } async updateTaskHistory(item: HistoryItem): Promise { - const history = - ((await this.getGlobalState("taskHistory")) as HistoryItem[]) || [] + const history = ((await this.getGlobalState("taskHistory")) as HistoryItem[]) || [] const existingItemIndex = history.findIndex((h) => h.id === item.id) if (existingItemIndex !== -1) { history[existingItemIndex] = item diff --git a/src/core/webview/getNonce.ts b/src/core/webview/getNonce.ts index 7409b500a0..b92871b93d 100644 --- a/src/core/webview/getNonce.ts +++ b/src/core/webview/getNonce.ts @@ -8,8 +8,7 @@ */ export function getNonce() { let text = "" - const possible = - "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789" + const possible = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789" for (let i = 0; i < 32; i++) { text += possible.charAt(Math.floor(Math.random() * possible.length)) } diff --git a/src/core/webview/getUri.ts b/src/core/webview/getUri.ts index db065dd138..13f90af516 100644 --- a/src/core/webview/getUri.ts +++ b/src/core/webview/getUri.ts @@ -10,10 +10,6 @@ import { Uri, Webview } from "vscode" * @param pathList An array of strings representing the path to a file/resource * @returns A URI pointing to the file/resource */ -export function getUri( - webview: Webview, - extensionUri: Uri, - pathList: string[], -) { +export function getUri(webview: Webview, extensionUri: Uri, pathList: string[]) { return webview.asWebviewUri(Uri.joinPath(extensionUri, ...pathList)) } diff --git a/src/exports/README.md b/src/exports/README.md index ed688facf9..40f909a217 100644 --- a/src/exports/README.md +++ b/src/exports/README.md @@ -7,9 +7,7 @@ The Cline extension exposes an API that can be used by other extensions. To use 3. Get access to the API with the following code: ```ts - const clineExtension = vscode.extensions.getExtension( - "saoudrizwan.claude-dev", - ) + const clineExtension = vscode.extensions.getExtension("saoudrizwan.claude-dev") if (!clineExtension?.isActive) { throw new Error("Cline extension is not activated") @@ -31,9 +29,7 @@ The Cline extension exposes an API that can be used by other extensions. To use await cline.startNewTask("Hello, Cline! Let's make a new project...") // Start a new task with an initial message and images - await cline.startNewTask("Use this design language", [ - "data:image/webp;base64,...", - ]) + await cline.startNewTask("Use this design language", ["data:image/webp;base64,..."]) // Send a message to the current task await cline.sendMessage("Can you fix the @problems?") diff --git a/src/exports/index.ts b/src/exports/index.ts index d87bed9d6c..9432f81fa9 100644 --- a/src/exports/index.ts +++ b/src/exports/index.ts @@ -2,10 +2,7 @@ import * as vscode from "vscode" import { ClineProvider } from "../core/webview/ClineProvider" import { ClineAPI } from "./cline" -export function createClineAPI( - outputChannel: vscode.OutputChannel, - sidebarProvider: ClineProvider, -): ClineAPI { +export function createClineAPI(outputChannel: vscode.OutputChannel, sidebarProvider: ClineProvider): ClineAPI { const api: ClineAPI = { setCustomInstructions: async (value: string) => { await sidebarProvider.updateCustomInstructions(value) @@ -13,9 +10,7 @@ export function createClineAPI( }, getCustomInstructions: async () => { - return (await sidebarProvider.getGlobalState( - "customInstructions", - )) as string | undefined + return (await sidebarProvider.getGlobalState("customInstructions")) as string | undefined }, startNewTask: async (task?: string, images?: string[]) => { diff --git a/src/extension.ts b/src/extension.ts index 0ebb78dd29..35dda8b588 100644 --- a/src/extension.ts +++ b/src/extension.ts @@ -29,13 +29,9 @@ export function activate(context: vscode.ExtensionContext) { const sidebarProvider = new ClineProvider(context, outputChannel) context.subscriptions.push( - vscode.window.registerWebviewViewProvider( - ClineProvider.sideBarId, - sidebarProvider, - { - webviewOptions: { retainContextWhenHidden: true }, - }, - ), + vscode.window.registerWebviewViewProvider(ClineProvider.sideBarId, sidebarProvider, { + webviewOptions: { retainContextWhenHidden: true }, + }), ) context.subscriptions.push( @@ -65,48 +61,25 @@ export function activate(context: vscode.ExtensionContext) { // https://github.com/microsoft/vscode-extension-samples/blob/main/webview-sample/src/extension.ts const tabProvider = new ClineProvider(context, outputChannel) //const column = vscode.window.activeTextEditor ? vscode.window.activeTextEditor.viewColumn : undefined - const lastCol = Math.max( - ...vscode.window.visibleTextEditors.map( - (editor) => editor.viewColumn || 0, - ), - ) + const lastCol = Math.max(...vscode.window.visibleTextEditors.map((editor) => editor.viewColumn || 0)) // Check if there are any visible text editors, otherwise open a new group to the right const hasVisibleEditors = vscode.window.visibleTextEditors.length > 0 if (!hasVisibleEditors) { - await vscode.commands.executeCommand( - "workbench.action.newGroupRight", - ) + await vscode.commands.executeCommand("workbench.action.newGroupRight") } - const targetCol = hasVisibleEditors - ? Math.max(lastCol + 1, 1) - : vscode.ViewColumn.Two - - const panel = vscode.window.createWebviewPanel( - ClineProvider.tabPanelId, - "Cline", - targetCol, - { - enableScripts: true, - retainContextWhenHidden: true, - localResourceRoots: [context.extensionUri], - }, - ) + const targetCol = hasVisibleEditors ? Math.max(lastCol + 1, 1) : vscode.ViewColumn.Two + + const panel = vscode.window.createWebviewPanel(ClineProvider.tabPanelId, "Cline", targetCol, { + enableScripts: true, + retainContextWhenHidden: true, + localResourceRoots: [context.extensionUri], + }) // TODO: use better svg icon with light and dark variants (see https://stackoverflow.com/questions/58365687/vscode-extension-iconpath) panel.iconPath = { - light: vscode.Uri.joinPath( - context.extensionUri, - "assets", - "icons", - "robot_panel_light.png", - ), - dark: vscode.Uri.joinPath( - context.extensionUri, - "assets", - "icons", - "robot_panel_dark.png", - ), + light: vscode.Uri.joinPath(context.extensionUri, "assets", "icons", "robot_panel_light.png"), + dark: vscode.Uri.joinPath(context.extensionUri, "assets", "icons", "robot_panel_dark.png"), } tabProvider.resolveWebviewView(panel) @@ -115,18 +88,8 @@ export function activate(context: vscode.ExtensionContext) { await vscode.commands.executeCommand("workbench.action.lockEditorGroup") } - context.subscriptions.push( - vscode.commands.registerCommand( - "cline.popoutButtonClicked", - openClineInNewTab, - ), - ) - context.subscriptions.push( - vscode.commands.registerCommand( - "cline.openInNewTab", - openClineInNewTab, - ), - ) + context.subscriptions.push(vscode.commands.registerCommand("cline.popoutButtonClicked", openClineInNewTab)) + context.subscriptions.push(vscode.commands.registerCommand("cline.openInNewTab", openClineInNewTab)) context.subscriptions.push( vscode.commands.registerCommand("cline.settingsButtonClicked", () => { @@ -154,19 +117,12 @@ export function activate(context: vscode.ExtensionContext) { - Note how the provider doesn't create uris for virtual documents - its role is to provide contents given such an uri. In return, content providers are wired into the open document logic so that providers are always considered. https://code.visualstudio.com/api/extension-guides/virtual-documents */ - const diffContentProvider = new (class - implements vscode.TextDocumentContentProvider - { + const diffContentProvider = new (class implements vscode.TextDocumentContentProvider { provideTextDocumentContent(uri: vscode.Uri): string { return Buffer.from(uri.query, "base64").toString("utf-8") } })() - context.subscriptions.push( - vscode.workspace.registerTextDocumentContentProvider( - DIFF_VIEW_URI_SCHEME, - diffContentProvider, - ), - ) + context.subscriptions.push(vscode.workspace.registerTextDocumentContentProvider(DIFF_VIEW_URI_SCHEME, diffContentProvider)) // URI Handler const handleUri = async (uri: vscode.Uri) => { diff --git a/src/integrations/checkpoints/CheckpointTracker.ts b/src/integrations/checkpoints/CheckpointTracker.ts index 9592c9309d..36a0c5a52e 100644 --- a/src/integrations/checkpoints/CheckpointTracker.ts +++ b/src/integrations/checkpoints/CheckpointTracker.ts @@ -21,15 +21,10 @@ class CheckpointTracker { this.cwd = cwd } - public static async create( - taskId: string, - provider?: ClineProvider, - ): Promise { + public static async create(taskId: string, provider?: ClineProvider): Promise { try { if (!provider) { - throw new Error( - "Provider is required to create a checkpoint tracker", - ) + throw new Error("Provider is required to create a checkpoint tracker") } // Check if git is installed by attempting to get version @@ -50,13 +45,9 @@ class CheckpointTracker { } private static async getWorkingDirectory(): Promise { - const cwd = vscode.workspace.workspaceFolders - ?.map((folder) => folder.uri.fsPath) - .at(0) + const cwd = vscode.workspace.workspaceFolders?.map((folder) => folder.uri.fsPath).at(0) if (!cwd) { - throw new Error( - "No workspace detected. Please open Cline in a workspace to use checkpoints.", - ) + throw new Error("No workspace detected. Please open Cline in a workspace to use checkpoints.") } const homedir = os.homedir() const desktopPath = path.join(homedir, "Desktop") @@ -78,37 +69,22 @@ class CheckpointTracker { } private async getShadowGitPath(): Promise { - const globalStoragePath = - this.providerRef.deref()?.context.globalStorageUri.fsPath + const globalStoragePath = this.providerRef.deref()?.context.globalStorageUri.fsPath if (!globalStoragePath) { throw new Error("Global storage uri is invalid") } - const checkpointsDir = path.join( - globalStoragePath, - "tasks", - this.taskId, - "checkpoints", - ) + const checkpointsDir = path.join(globalStoragePath, "tasks", this.taskId, "checkpoints") await fs.mkdir(checkpointsDir, { recursive: true }) const gitPath = path.join(checkpointsDir, ".git") return gitPath } - public static async doesShadowGitExist( - taskId: string, - provider?: ClineProvider, - ): Promise { + public static async doesShadowGitExist(taskId: string, provider?: ClineProvider): Promise { const globalStoragePath = provider?.context.globalStorageUri.fsPath if (!globalStoragePath) { return false } - const gitPath = path.join( - globalStoragePath, - "tasks", - taskId, - "checkpoints", - ".git", - ) + const gitPath = path.join(globalStoragePath, "tasks", taskId, "checkpoints", ".git") return await fileExistsAtPath(gitPath) } @@ -118,10 +94,7 @@ class CheckpointTracker { // Make sure it's the same cwd as the configured worktree const worktree = await this.getShadowGitConfigWorkTree() if (worktree !== this.cwd) { - throw new Error( - "Checkpoints can only be used in the original workspace: " + - worktree, - ) + throw new Error("Checkpoints can only be used in the original workspace: " + worktree) } return gitPath @@ -250,8 +223,7 @@ class CheckpointTracker { const gitPath = await this.getShadowGitPath() const git = simpleGit(path.dirname(gitPath)) const worktree = await git.getConfig("core.worktree") - this.lastRetrievedShadowGitConfigWorkTree = - worktree.value || undefined + this.lastRetrievedShadowGitConfigWorkTree = worktree.value || undefined return this.lastRetrievedShadowGitConfigWorkTree } catch (error) { console.error("Failed to get shadow git config worktree:", error) @@ -323,11 +295,7 @@ class CheckpointTracker { // If lhsHash is missing, use the initial commit of the repo let baseHash = lhsHash if (!baseHash) { - const rootCommit = await git.raw([ - "rev-list", - "--max-parents=0", - "HEAD", - ]) + const rootCommit = await git.raw(["rev-list", "--max-parents=0", "HEAD"]) baseHash = rootCommit.trim() } @@ -336,14 +304,11 @@ class CheckpointTracker { await git.add(".") await this.renameNestedGitRepos(false) - const diffSummary = rhsHash - ? await git.diffSummary([`${baseHash}..${rhsHash}`]) - : await git.diffSummary([baseHash]) + const diffSummary = rhsHash ? await git.diffSummary([`${baseHash}..${rhsHash}`]) : await git.diffSummary([baseHash]) // For each changed file, gather before/after content const result = [] - const cwdPath = - (await this.getShadowGitConfigWorkTree()) || this.cwd || "" + const cwdPath = (await this.getShadowGitConfigWorkTree()) || this.cwd || "" for (const file of diffSummary.files) { const filePath = file.file @@ -387,16 +352,13 @@ class CheckpointTracker { // Since we use git to track checkpoints, we need to temporarily disable nested git repos to work around git's requirement of using submodules for nested repos. async renameNestedGitRepos(disable: boolean) { // Find all .git directories that are not at the root level - const gitPaths = await globby( - "**/.git" + (disable ? "" : GIT_DISABLED_SUFFIX), - { - cwd: this.cwd, - onlyDirectories: true, - ignore: [".git"], // Ignore root level .git - dot: true, - markDirectories: false, - }, - ) + const gitPaths = await globby("**/.git" + (disable ? "" : GIT_DISABLED_SUFFIX), { + cwd: this.cwd, + onlyDirectories: true, + ignore: [".git"], // Ignore root level .git + dot: true, + markDirectories: false, + }) // For each nested .git directory, rename it based on operation for (const gitPath of gitPaths) { @@ -405,21 +367,14 @@ class CheckpointTracker { if (disable) { newPath = fullPath + GIT_DISABLED_SUFFIX } else { - newPath = fullPath.endsWith(GIT_DISABLED_SUFFIX) - ? fullPath.slice(0, -GIT_DISABLED_SUFFIX.length) - : fullPath + newPath = fullPath.endsWith(GIT_DISABLED_SUFFIX) ? fullPath.slice(0, -GIT_DISABLED_SUFFIX.length) : fullPath } try { await fs.rename(fullPath, newPath) - console.log( - `CheckpointTracker ${disable ? "disabled" : "enabled"} nested git repo ${gitPath}`, - ) + console.log(`CheckpointTracker ${disable ? "disabled" : "enabled"} nested git repo ${gitPath}`) } catch (error) { - console.error( - `CheckpointTracker failed to ${disable ? "disable" : "enable"} nested git repo ${gitPath}:`, - error, - ) + console.error(`CheckpointTracker failed to ${disable ? "disable" : "enable"} nested git repo ${gitPath}:`, error) } } } diff --git a/src/integrations/diagnostics/index.ts b/src/integrations/diagnostics/index.ts index 037529301c..ad4ee7755c 100644 --- a/src/integrations/diagnostics/index.ts +++ b/src/integrations/diagnostics/index.ts @@ -11,10 +11,7 @@ export function getNewDiagnostics( for (const [uri, newDiags] of newDiagnostics) { const oldDiags = oldMap.get(uri) || [] - const newProblemsForUri = newDiags.filter( - (newDiag) => - !oldDiags.some((oldDiag) => deepEqual(oldDiag, newDiag)), - ) + const newProblemsForUri = newDiags.filter((newDiag) => !oldDiags.some((oldDiag) => deepEqual(oldDiag, newDiag))) if (newProblemsForUri.length > 0) { newProblems.push([uri, newProblemsForUri]) @@ -80,9 +77,7 @@ export function diagnosticsToProblemsString( ): string { let result = "" for (const [uri, fileDiagnostics] of diagnostics) { - const problems = fileDiagnostics.filter((d) => - severities.includes(d.severity), - ) + const problems = fileDiagnostics.filter((d) => severities.includes(d.severity)) if (problems.length > 0) { result += `\n\n${path.relative(cwd, uri.fsPath).toPosix()}` for (const diagnostic of problems) { diff --git a/src/integrations/editor/DecorationController.ts b/src/integrations/editor/DecorationController.ts index f99646ea46..6622131e13 100644 --- a/src/integrations/editor/DecorationController.ts +++ b/src/integrations/editor/DecorationController.ts @@ -1,12 +1,10 @@ import * as vscode from "vscode" -const fadedOverlayDecorationType = vscode.window.createTextEditorDecorationType( - { - backgroundColor: "rgba(255, 255, 0, 0.1)", - opacity: "0.4", - isWholeLine: true, - }, -) +const fadedOverlayDecorationType = vscode.window.createTextEditorDecorationType({ + backgroundColor: "rgba(255, 255, 0, 0.1)", + opacity: "0.4", + isWholeLine: true, +}) const activeLineDecorationType = vscode.window.createTextEditorDecorationType({ backgroundColor: "rgba(255, 255, 0, 0.3)", @@ -44,20 +42,10 @@ export class DecorationController { const lastRange = this.ranges[this.ranges.length - 1] if (lastRange && lastRange.end.line === startIndex - 1) { - this.ranges[this.ranges.length - 1] = lastRange.with( - undefined, - lastRange.end.translate(numLines), - ) + this.ranges[this.ranges.length - 1] = lastRange.with(undefined, lastRange.end.translate(numLines)) } else { const endLine = startIndex + numLines - 1 - this.ranges.push( - new vscode.Range( - startIndex, - 0, - endLine, - Number.MAX_SAFE_INTEGER, - ), - ) + this.ranges.push(new vscode.Range(startIndex, 0, endLine, Number.MAX_SAFE_INTEGER)) } this.editor.setDecorations(this.getDecoration(), this.ranges) @@ -75,13 +63,7 @@ export class DecorationController { // Add a new range for all lines after the current line if (line < totalLines - 1) { this.ranges.push( - new vscode.Range( - new vscode.Position(line + 1, 0), - new vscode.Position( - totalLines - 1, - Number.MAX_SAFE_INTEGER, - ), - ), + new vscode.Range(new vscode.Position(line + 1, 0), new vscode.Position(totalLines - 1, Number.MAX_SAFE_INTEGER)), ) } diff --git a/src/integrations/editor/DiffViewProvider.ts b/src/integrations/editor/DiffViewProvider.ts index 1f5fe56cc8..e416f99de6 100644 --- a/src/integrations/editor/DiffViewProvider.ts +++ b/src/integrations/editor/DiffViewProvider.ts @@ -33,9 +33,7 @@ export class DiffViewProvider { this.isEditing = true // if the file is already open, ensure it's not dirty before getting its contents if (fileExists) { - const existingDocument = vscode.workspace.textDocuments.find( - (doc) => arePathsEqual(doc.uri.fsPath, absolutePath), - ) + const existingDocument = vscode.workspace.textDocuments.find((doc) => arePathsEqual(doc.uri.fsPath, absolutePath)) if (existingDocument && existingDocument.isDirty) { await existingDocument.save() } @@ -61,11 +59,7 @@ export class DiffViewProvider { const tabs = vscode.window.tabGroups.all .map((tg) => tg.tabs) .flat() - .filter( - (tab) => - tab.input instanceof vscode.TabInputText && - arePathsEqual(tab.input.uri.fsPath, absolutePath), - ) + .filter((tab) => tab.input instanceof vscode.TabInputText && arePathsEqual(tab.input.uri.fsPath, absolutePath)) for (const tab of tabs) { if (!tab.isDirty) { await vscode.window.tabGroups.close(tab) @@ -73,29 +67,16 @@ export class DiffViewProvider { this.documentWasOpen = true } this.activeDiffEditor = await this.openDiffEditor() - this.fadedOverlayController = new DecorationController( - "fadedOverlay", - this.activeDiffEditor, - ) - this.activeLineController = new DecorationController( - "activeLine", - this.activeDiffEditor, - ) + this.fadedOverlayController = new DecorationController("fadedOverlay", this.activeDiffEditor) + this.activeLineController = new DecorationController("activeLine", this.activeDiffEditor) // Apply faded overlay to all lines initially - this.fadedOverlayController.addLines( - 0, - this.activeDiffEditor.document.lineCount, - ) + this.fadedOverlayController.addLines(0, this.activeDiffEditor.document.lineCount) this.scrollEditorToLine(0) // will this crash for new files? this.streamedLines = [] } async update(accumulatedContent: string, isFinal: boolean) { - if ( - !this.relPath || - !this.activeLineController || - !this.fadedOverlayController - ) { + if (!this.relPath || !this.activeLineController || !this.fadedOverlayController) { throw new Error("Required values not set") } this.newContent = accumulatedContent @@ -113,10 +94,7 @@ export class DiffViewProvider { // Place cursor at the beginning of the diff editor to keep it out of the way of the stream animation const beginningOfDocument = new vscode.Position(0, 0) - diffEditor.selection = new vscode.Selection( - beginningOfDocument, - beginningOfDocument, - ) + diffEditor.selection = new vscode.Selection(beginningOfDocument, beginningOfDocument) for (let i = 0; i < diffLines.length; i++) { const currentLine = this.streamedLines.length + i @@ -124,16 +102,12 @@ export class DiffViewProvider { // This is necessary (as compared to inserting one line at a time) to handle cases where html tags on previous lines are auto closed for example const edit = new vscode.WorkspaceEdit() const rangeToReplace = new vscode.Range(0, 0, currentLine + 1, 0) - const contentToReplace = - accumulatedLines.slice(0, currentLine + 1).join("\n") + "\n" + const contentToReplace = accumulatedLines.slice(0, currentLine + 1).join("\n") + "\n" edit.replace(document.uri, rangeToReplace, contentToReplace) await vscode.workspace.applyEdit(edit) // Update decorations this.activeLineController.setActiveLine(currentLine) - this.fadedOverlayController.updateOverlayAfterLine( - currentLine, - document.lineCount, - ) + this.fadedOverlayController.updateOverlayAfterLine(currentLine, document.lineCount) // Scroll to the current line this.scrollEditorToLine(currentLine) } @@ -143,15 +117,7 @@ export class DiffViewProvider { // Handle any remaining lines if the new content is shorter than the original if (this.streamedLines.length < document.lineCount) { const edit = new vscode.WorkspaceEdit() - edit.delete( - document.uri, - new vscode.Range( - this.streamedLines.length, - 0, - document.lineCount, - 0, - ), - ) + edit.delete(document.uri, new vscode.Range(this.streamedLines.length, 0, document.lineCount, 0)) await vscode.workspace.applyEdit(edit) } // Add empty last line if original content had one @@ -227,31 +193,19 @@ export class DiffViewProvider { this.cwd, ) // will be empty string if no errors const newProblemsMessage = - newProblems.length > 0 - ? `\n\nNew problems detected after saving the file:\n${newProblems}` - : "" + newProblems.length > 0 ? `\n\nNew problems detected after saving the file:\n${newProblems}` : "" // If the edited content has different EOL characters, we don't want to show a diff with all the EOL differences. const newContentEOL = this.newContent.includes("\r\n") ? "\r\n" : "\n" - const normalizedPreSaveContent = - preSaveContent.replace(/\r\n|\n/g, newContentEOL).trimEnd() + - newContentEOL // trimEnd to fix issue where editor adds in extra new line automatically - const normalizedPostSaveContent = - postSaveContent.replace(/\r\n|\n/g, newContentEOL).trimEnd() + - newContentEOL // this is the final content we return to the model to use as the new baseline for future edits + const normalizedPreSaveContent = preSaveContent.replace(/\r\n|\n/g, newContentEOL).trimEnd() + newContentEOL // trimEnd to fix issue where editor adds in extra new line automatically + const normalizedPostSaveContent = postSaveContent.replace(/\r\n|\n/g, newContentEOL).trimEnd() + newContentEOL // this is the final content we return to the model to use as the new baseline for future edits // just in case the new content has a mix of varying EOL characters - const normalizedNewContent = - this.newContent.replace(/\r\n|\n/g, newContentEOL).trimEnd() + - newContentEOL + const normalizedNewContent = this.newContent.replace(/\r\n|\n/g, newContentEOL).trimEnd() + newContentEOL let userEdits: string | undefined if (normalizedPreSaveContent !== normalizedNewContent) { // user made changes before approving edit. let the model know about user made changes (not including post-save auto-formatting changes) - userEdits = formatResponse.createPrettyPatch( - this.relPath.toPosix(), - normalizedNewContent, - normalizedPreSaveContent, - ) + userEdits = formatResponse.createPrettyPatch(this.relPath.toPosix(), normalizedNewContent, normalizedPreSaveContent) // return { newProblemsMessage, userEdits, finalContent: normalizedPostSaveContent } } else { // no changes to cline's edits @@ -292,9 +246,7 @@ export class DiffViewProvider { // Remove only the directories we created, in reverse order for (let i = this.createdDirs.length - 1; i >= 0; i--) { await fs.rmdir(this.createdDirs[i]) - console.log( - `Directory ${this.createdDirs[i]} has been deleted.`, - ) + console.log(`Directory ${this.createdDirs[i]} has been deleted.`) } console.log(`File ${absolutePath} has been deleted.`) } else { @@ -304,24 +256,15 @@ export class DiffViewProvider { updatedDocument.positionAt(0), updatedDocument.positionAt(updatedDocument.getText().length), ) - edit.replace( - updatedDocument.uri, - fullRange, - this.originalContent ?? "", - ) + edit.replace(updatedDocument.uri, fullRange, this.originalContent ?? "") // Apply the edit and save, since contents shouldnt have changed this wont show in local history unless of course the user made changes and saved during the edit await vscode.workspace.applyEdit(edit) await updatedDocument.save() - console.log( - `File ${absolutePath} has been reverted to its original content.`, - ) + console.log(`File ${absolutePath} has been reverted to its original content.`) if (this.documentWasOpen) { - await vscode.window.showTextDocument( - vscode.Uri.file(absolutePath), - { - preview: false, - }, - ) + await vscode.window.showTextDocument(vscode.Uri.file(absolutePath), { + preview: false, + }) } await this.closeAllDiffViews() } @@ -333,11 +276,7 @@ export class DiffViewProvider { private async closeAllDiffViews() { const tabs = vscode.window.tabGroups.all .flatMap((tg) => tg.tabs) - .filter( - (tab) => - tab.input instanceof vscode.TabInputTextDiff && - tab.input?.original?.scheme === DIFF_VIEW_URI_SCHEME, - ) + .filter((tab) => tab.input instanceof vscode.TabInputTextDiff && tab.input?.original?.scheme === DIFF_VIEW_URI_SCHEME) for (const tab of tabs) { // trying to close dirty views results in save popup if (!tab.isDirty) { @@ -361,32 +300,23 @@ export class DiffViewProvider { arePathsEqual(tab.input.modified.fsPath, uri.fsPath), ) if (diffTab && diffTab.input instanceof vscode.TabInputTextDiff) { - const editor = await vscode.window.showTextDocument( - diffTab.input.modified, - ) + const editor = await vscode.window.showTextDocument(diffTab.input.modified) return editor } // Open new diff editor return new Promise((resolve, reject) => { const fileName = path.basename(uri.fsPath) const fileExists = this.editType === "modify" - const disposable = vscode.window.onDidChangeActiveTextEditor( - (editor) => { - if ( - editor && - arePathsEqual(editor.document.uri.fsPath, uri.fsPath) - ) { - disposable.dispose() - resolve(editor) - } - }, - ) + const disposable = vscode.window.onDidChangeActiveTextEditor((editor) => { + if (editor && arePathsEqual(editor.document.uri.fsPath, uri.fsPath)) { + disposable.dispose() + resolve(editor) + } + }) vscode.commands.executeCommand( "vscode.diff", vscode.Uri.parse(`${DIFF_VIEW_URI_SCHEME}:${fileName}`).with({ - query: Buffer.from(this.originalContent ?? "").toString( - "base64", - ), + query: Buffer.from(this.originalContent ?? "").toString("base64"), }), uri, `${fileName}: ${fileExists ? "Original ↔ Cline's Changes" : "New File"} (Editable)`, @@ -394,11 +324,7 @@ export class DiffViewProvider { // This may happen on very slow machines ie project idx setTimeout(() => { disposable.dispose() - reject( - new Error( - "Failed to open diff editor, please try again...", - ), - ) + reject(new Error("Failed to open diff editor, please try again...")) }, 10_000) }) } diff --git a/src/integrations/editor/detect-omission.ts b/src/integrations/editor/detect-omission.ts index 7655131c1c..32de0aac72 100644 --- a/src/integrations/editor/detect-omission.ts +++ b/src/integrations/editor/detect-omission.ts @@ -6,21 +6,10 @@ import * as vscode from "vscode" * @param newFileContent The new content of the file to check. * @returns True if a potential omission is detected, false otherwise. */ -function detectCodeOmission( - originalFileContent: string, - newFileContent: string, -): boolean { +function detectCodeOmission(originalFileContent: string, newFileContent: string): boolean { const originalLines = originalFileContent.split("\n") const newLines = newFileContent.split("\n") - const omissionKeywords = [ - "remain", - "remains", - "unchanged", - "rest", - "previous", - "existing", - "...", - ] + const omissionKeywords = ["remain", "remains", "unchanged", "rest", "previous", "existing", "..."] const commentPatterns = [ /^\s*\/\//, // Single-line comment for most languages @@ -49,10 +38,7 @@ function detectCodeOmission( * @param originalFileContent The original content of the file. * @param newFileContent The new content of the file to check. */ -export function showOmissionWarning( - originalFileContent: string, - newFileContent: string, -): void { +export function showOmissionWarning(originalFileContent: string, newFileContent: string): void { if (detectCodeOmission(originalFileContent, newFileContent)) { vscode.window .showWarningMessage( diff --git a/src/integrations/misc/export-markdown.ts b/src/integrations/misc/export-markdown.ts index 8dca7b759c..76e95e36b7 100644 --- a/src/integrations/misc/export-markdown.ts +++ b/src/integrations/misc/export-markdown.ts @@ -3,10 +3,7 @@ import os from "os" import * as path from "path" import * as vscode from "vscode" -export async function downloadTask( - dateTs: number, - conversationHistory: Anthropic.MessageParam[], -) { +export async function downloadTask(dateTs: number, conversationHistory: Anthropic.MessageParam[]) { // File name const date = new Date(dateTs) const month = date.toLocaleString("en-US", { month: "short" }).toLowerCase() @@ -23,12 +20,9 @@ export async function downloadTask( // Generate markdown const markdownContent = conversationHistory .map((message) => { - const role = - message.role === "user" ? "**User:**" : "**Assistant:**" + const role = message.role === "user" ? "**User:**" : "**Assistant:**" const content = Array.isArray(message.content) - ? message.content - .map((block) => formatContentBlockToMarkdown(block)) - .join("\n") + ? message.content.map((block) => formatContentBlockToMarkdown(block)).join("\n") : message.content return `${role}\n\n${content}\n\n` }) @@ -37,27 +31,18 @@ export async function downloadTask( // Prompt user for save location const saveUri = await vscode.window.showSaveDialog({ filters: { Markdown: ["md"] }, - defaultUri: vscode.Uri.file( - path.join(os.homedir(), "Downloads", fileName), - ), + defaultUri: vscode.Uri.file(path.join(os.homedir(), "Downloads", fileName)), }) if (saveUri) { // Write content to the selected location - await vscode.workspace.fs.writeFile( - saveUri, - Buffer.from(markdownContent), - ) + await vscode.workspace.fs.writeFile(saveUri, Buffer.from(markdownContent)) vscode.window.showTextDocument(saveUri, { preview: true }) } } export function formatContentBlockToMarkdown( - block: - | Anthropic.TextBlockParam - | Anthropic.ImageBlockParam - | Anthropic.ToolUseBlockParam - | Anthropic.ToolResultBlockParam, + block: Anthropic.TextBlockParam | Anthropic.ImageBlockParam | Anthropic.ToolUseBlockParam | Anthropic.ToolResultBlockParam, // messages: Anthropic.MessageParam[] ): string { switch (block.type) { @@ -69,10 +54,7 @@ export function formatContentBlockToMarkdown( let input: string if (typeof block.input === "object" && block.input !== null) { input = Object.entries(block.input) - .map( - ([key, value]) => - `${key.charAt(0).toUpperCase() + key.slice(1)}: ${value}`, - ) + .map(([key, value]) => `${key.charAt(0).toUpperCase() + key.slice(1)}: ${value}`) .join("\n") } else { input = String(block.input) @@ -86,9 +68,7 @@ export function formatContentBlockToMarkdown( return `[${toolName}${block.is_error ? " (Error)" : ""}]\n${block.content}` } else if (Array.isArray(block.content)) { return `[${toolName}${block.is_error ? " (Error)" : ""}]\n${block.content - .map((contentBlock) => - formatContentBlockToMarkdown(contentBlock), - ) + .map((contentBlock) => formatContentBlockToMarkdown(contentBlock)) .join("\n")}` } else { return `[${toolName}${block.is_error ? " (Error)" : ""}]` @@ -98,10 +78,7 @@ export function formatContentBlockToMarkdown( } } -export function findToolName( - toolCallId: string, - messages: Anthropic.MessageParam[], -): string { +export function findToolName(toolCallId: string, messages: Anthropic.MessageParam[]): string { for (const message of messages) { if (Array.isArray(message.content)) { for (const block of message.content) { diff --git a/src/integrations/misc/extract-text.ts b/src/integrations/misc/extract-text.ts index 83644d04b9..67a580af9b 100644 --- a/src/integrations/misc/extract-text.ts +++ b/src/integrations/misc/extract-text.ts @@ -24,9 +24,7 @@ export async function extractTextFromFile(filePath: string): Promise { if (!isBinary) { return await fs.readFile(filePath, "utf8") } else { - throw new Error( - `Cannot read text for file type: ${fileExtension}`, - ) + throw new Error(`Cannot read text for file type: ${fileExtension}`) } } } @@ -48,10 +46,7 @@ async function extractTextFromIPYNB(filePath: string): Promise { let extractedText = "" for (const cell of notebook.cells) { - if ( - (cell.cell_type === "markdown" || cell.cell_type === "code") && - cell.source - ) { + if ((cell.cell_type === "markdown" || cell.cell_type === "code") && cell.source) { extractedText += cell.source.join("\n") + "\n" } } diff --git a/src/integrations/misc/open-file.ts b/src/integrations/misc/open-file.ts index 45eafe00ab..9cd88708d3 100644 --- a/src/integrations/misc/open-file.ts +++ b/src/integrations/misc/open-file.ts @@ -11,19 +11,10 @@ export async function openImage(dataUri: string) { } const [, format, base64Data] = matches const imageBuffer = Buffer.from(base64Data, "base64") - const tempFilePath = path.join( - os.tmpdir(), - `temp_image_${Date.now()}.${format}`, - ) + const tempFilePath = path.join(os.tmpdir(), `temp_image_${Date.now()}.${format}`) try { - await vscode.workspace.fs.writeFile( - vscode.Uri.file(tempFilePath), - imageBuffer, - ) - await vscode.commands.executeCommand( - "vscode.open", - vscode.Uri.file(tempFilePath), - ) + await vscode.workspace.fs.writeFile(vscode.Uri.file(tempFilePath), imageBuffer) + await vscode.commands.executeCommand("vscode.open", vscode.Uri.file(tempFilePath)) } catch (error) { vscode.window.showErrorMessage(`Error opening image: ${error}`) } @@ -37,21 +28,12 @@ export async function openFile(absolutePath: string) { try { for (const group of vscode.window.tabGroups.all) { const existingTab = group.tabs.find( - (tab) => - tab.input instanceof vscode.TabInputText && - arePathsEqual(tab.input.uri.fsPath, uri.fsPath), + (tab) => tab.input instanceof vscode.TabInputText && arePathsEqual(tab.input.uri.fsPath, uri.fsPath), ) if (existingTab) { - const activeColumn = - vscode.window.activeTextEditor?.viewColumn - const tabColumn = vscode.window.tabGroups.all.find( - (group) => group.tabs.includes(existingTab), - )?.viewColumn - if ( - activeColumn && - activeColumn !== tabColumn && - !existingTab.isDirty - ) { + const activeColumn = vscode.window.activeTextEditor?.viewColumn + const tabColumn = vscode.window.tabGroups.all.find((group) => group.tabs.includes(existingTab))?.viewColumn + if (activeColumn && activeColumn !== tabColumn && !existingTab.isDirty) { await vscode.window.tabGroups.close(existingTab) } break diff --git a/src/integrations/notifications/index.ts b/src/integrations/notifications/index.ts index 7faf2d7359..722df87e32 100644 --- a/src/integrations/notifications/index.ts +++ b/src/integrations/notifications/index.ts @@ -7,9 +7,7 @@ interface NotificationOptions { message: string } -async function showMacOSNotification( - options: NotificationOptions, -): Promise { +async function showMacOSNotification(options: NotificationOptions): Promise { const { title, subtitle = "", message } = options const script = `display notification "${message}" with title "${title}" subtitle "${subtitle}" sound name "Tink"` @@ -21,9 +19,7 @@ async function showMacOSNotification( } } -async function showWindowsNotification( - options: NotificationOptions, -): Promise { +async function showWindowsNotification(options: NotificationOptions): Promise { const { subtitle, message } = options const script = ` @@ -54,9 +50,7 @@ async function showWindowsNotification( } } -async function showLinuxNotification( - options: NotificationOptions, -): Promise { +async function showLinuxNotification(options: NotificationOptions): Promise { const { title = "", subtitle = "", message } = options // Combine subtitle and message if subtitle exists @@ -69,9 +63,7 @@ async function showLinuxNotification( } } -export async function showSystemNotification( - options: NotificationOptions, -): Promise { +export async function showSystemNotification(options: NotificationOptions): Promise { try { const { title = "Cline", message } = options diff --git a/src/integrations/terminal/TerminalManager.ts b/src/integrations/terminal/TerminalManager.ts index 8ae4605c9c..81e91ab6b8 100644 --- a/src/integrations/terminal/TerminalManager.ts +++ b/src/integrations/terminal/TerminalManager.ts @@ -1,11 +1,7 @@ import pWaitFor from "p-wait-for" import * as vscode from "vscode" import { arePathsEqual } from "../../utils/path" -import { - mergePromise, - TerminalProcess, - TerminalProcessResultPromise, -} from "./TerminalProcess" +import { mergePromise, TerminalProcess, TerminalProcessResultPromise } from "./TerminalProcess" import { TerminalInfo, TerminalRegistry } from "./TerminalRegistry" /* @@ -101,9 +97,7 @@ export class TerminalManager { constructor() { let disposable: vscode.Disposable | undefined try { - disposable = ( - vscode.window as vscode.Window - ).onDidStartTerminalShellExecution?.(async (e) => { + disposable = (vscode.window as vscode.Window).onDidStartTerminalShellExecution?.(async (e) => { // Creating a read stream here results in a more consistent output. This is most obvious when running the `date` command. e?.execution?.read() }) @@ -115,10 +109,7 @@ export class TerminalManager { } } - runCommand( - terminalInfo: TerminalInfo, - command: string, - ): TerminalProcessResultPromise { + runCommand(terminalInfo: TerminalInfo, command: string): TerminalProcessResultPromise { terminalInfo.busy = true terminalInfo.lastCommand = command const process = new TerminalProcess() @@ -130,9 +121,7 @@ export class TerminalManager { // if shell integration is not available, remove terminal so it does not get reused as it may be running a long-running process process.once("no_shell_integration", () => { - console.log( - `no_shell_integration received for terminal ${terminalInfo.id}`, - ) + console.log(`no_shell_integration received for terminal ${terminalInfo.id}`) // Remove the terminal so we can't reuse it (in case it's running a long-running process) TerminalRegistry.removeTerminal(terminalInfo.id) this.terminalIds.delete(terminalInfo.id) @@ -155,15 +144,9 @@ export class TerminalManager { process.run(terminalInfo.terminal, command) } else { // docs recommend waiting 3s for shell integration to activate - pWaitFor( - () => terminalInfo.terminal.shellIntegration !== undefined, - { timeout: 4000 }, - ).finally(() => { + pWaitFor(() => terminalInfo.terminal.shellIntegration !== undefined, { timeout: 4000 }).finally(() => { const existingProcess = this.processes.get(terminalInfo.id) - if ( - existingProcess && - existingProcess.waitForShellIntegration - ) { + if (existingProcess && existingProcess.waitForShellIntegration) { existingProcess.waitForShellIntegration = false existingProcess.run(terminalInfo.terminal, command) } @@ -175,21 +158,16 @@ export class TerminalManager { async getOrCreateTerminal(cwd: string): Promise { // Find available terminal from our pool first (created for this task) - const availableTerminal = TerminalRegistry.getAllTerminals().find( - (t) => { - if (t.busy) { - return false - } - const terminalCwd = t.terminal.shellIntegration?.cwd // one of cline's commands could have changed the cwd of the terminal - if (!terminalCwd) { - return false - } - return arePathsEqual( - vscode.Uri.file(cwd).fsPath, - terminalCwd.fsPath, - ) - }, - ) + const availableTerminal = TerminalRegistry.getAllTerminals().find((t) => { + if (t.busy) { + return false + } + const terminalCwd = t.terminal.shellIntegration?.cwd // one of cline's commands could have changed the cwd of the terminal + if (!terminalCwd) { + return false + } + return arePathsEqual(vscode.Uri.file(cwd).fsPath, terminalCwd.fsPath) + }) if (availableTerminal) { this.terminalIds.add(availableTerminal.id) return availableTerminal @@ -203,9 +181,7 @@ export class TerminalManager { getTerminals(busy: boolean): { id: number; lastCommand: string }[] { return Array.from(this.terminalIds) .map((id) => TerminalRegistry.getTerminal(id)) - .filter( - (t): t is TerminalInfo => t !== undefined && t.busy === busy, - ) + .filter((t): t is TerminalInfo => t !== undefined && t.busy === busy) .map((t) => ({ id: t.id, lastCommand: t.lastCommand })) } diff --git a/src/integrations/terminal/TerminalProcess.ts b/src/integrations/terminal/TerminalProcess.ts index 30554ce722..5597350db3 100644 --- a/src/integrations/terminal/TerminalProcess.ts +++ b/src/integrations/terminal/TerminalProcess.ts @@ -27,10 +27,7 @@ export class TerminalProcess extends EventEmitter { // super() async run(terminal: vscode.Terminal, command: string) { - if ( - terminal.shellIntegration && - terminal.shellIntegration.executeCommand - ) { + if (terminal.shellIntegration && terminal.shellIntegration.executeCommand) { const execution = terminal.shellIntegration.executeCommand(command) const stream = execution.read() // todo: need to handle errors @@ -63,9 +60,7 @@ export class TerminalProcess extends EventEmitter { // Once we've retrieved any potential output between sequences, we can remove everything up to end of the last sequence // https://code.visualstudio.com/docs/terminal/shell-integration#_vs-code-custom-sequences-osc-633-st const vscodeSequenceRegex = /\x1b\]633;.[^\x07]*\x07/g - const lastMatch = [ - ...data.matchAll(vscodeSequenceRegex), - ].pop() + const lastMatch = [...data.matchAll(vscodeSequenceRegex)].pop() if (lastMatch && lastMatch.index !== undefined) { data = data.slice(lastMatch.index + lastMatch[0].length) } @@ -82,11 +77,7 @@ export class TerminalProcess extends EventEmitter { lines[0] = lines[0].replace(/[^\x20-\x7E]/g, "") } // Check if first two characters are the same, if so remove the first character - if ( - lines.length > 0 && - lines[0].length >= 2 && - lines[0][0] === lines[0][1] - ) { + if (lines.length > 0 && lines[0].length >= 2 && lines[0][0] === lines[0][1]) { lines[0] = lines[0].slice(1) } // Remove everything up to the first alphanumeric character for first two lines @@ -129,14 +120,7 @@ export class TerminalProcess extends EventEmitter { clearTimeout(this.hotTimer) } // these markers indicate the command is some kind of local dev server recompiling the app, which we want to wait for output of before sending request to cline - const compilingMarkers = [ - "compiling", - "building", - "bundling", - "transpiling", - "generating", - "starting", - ] + const compilingMarkers = ["compiling", "building", "bundling", "transpiling", "generating", "starting"] const markerNullifiers = [ "compiled", "success", @@ -152,19 +136,13 @@ export class TerminalProcess extends EventEmitter { "fail", ] const isCompiling = - compilingMarkers.some((marker) => - data.toLowerCase().includes(marker.toLowerCase()), - ) && - !markerNullifiers.some((nullifier) => - data.toLowerCase().includes(nullifier.toLowerCase()), - ) + compilingMarkers.some((marker) => data.toLowerCase().includes(marker.toLowerCase())) && + !markerNullifiers.some((nullifier) => data.toLowerCase().includes(nullifier.toLowerCase())) this.hotTimer = setTimeout( () => { this.isHot = false }, - isCompiling - ? PROCESS_HOT_TIMEOUT_COMPILING - : PROCESS_HOT_TIMEOUT_NORMAL, + isCompiling ? PROCESS_HOT_TIMEOUT_COMPILING : PROCESS_HOT_TIMEOUT_NORMAL, ) // For non-immediately returning commands we want to show loading spinner right away but this wouldnt happen until it emits a line break, so as soon as we get any output we emit "" to let webview know to show spinner @@ -176,8 +154,7 @@ export class TerminalProcess extends EventEmitter { this.fullOutput += data if (this.isListening) { this.emitIfEol(data) - this.lastRetrievedIndex = - this.fullOutput.length - this.buffer.length + this.lastRetrievedIndex = this.fullOutput.length - this.buffer.length } } @@ -260,20 +237,10 @@ export class TerminalProcess extends EventEmitter { export type TerminalProcessResultPromise = TerminalProcess & Promise // Similar to execa's ResultPromise, this lets us create a mixin of both a TerminalProcess and a Promise: https://github.com/sindresorhus/execa/blob/main/lib/methods/promise.js -export function mergePromise( - process: TerminalProcess, - promise: Promise, -): TerminalProcessResultPromise { +export function mergePromise(process: TerminalProcess, promise: Promise): TerminalProcessResultPromise { const nativePromisePrototype = (async () => {})().constructor.prototype const descriptors = ["then", "catch", "finally"].map( - (property) => - [ - property, - Reflect.getOwnPropertyDescriptor( - nativePromisePrototype, - property, - ), - ] as const, + (property) => [property, Reflect.getOwnPropertyDescriptor(nativePromisePrototype, property)] as const, ) for (const [property, descriptor] of descriptors) { if (descriptor) { diff --git a/src/integrations/terminal/TerminalRegistry.ts b/src/integrations/terminal/TerminalRegistry.ts index f7edf6d507..ac0ed30f0d 100644 --- a/src/integrations/terminal/TerminalRegistry.ts +++ b/src/integrations/terminal/TerminalRegistry.ts @@ -50,9 +50,7 @@ export class TerminalRegistry { } static getAllTerminals(): TerminalInfo[] { - this.terminals = this.terminals.filter( - (t) => !this.isTerminalClosed(t.terminal), - ) + this.terminals = this.terminals.filter((t) => !this.isTerminalClosed(t.terminal)) return this.terminals } diff --git a/src/integrations/theme/default-themes/dark_plus.json b/src/integrations/theme/default-themes/dark_plus.json index df757b7170..3a45b1eb83 100644 --- a/src/integrations/theme/default-themes/dark_plus.json +++ b/src/integrations/theme/default-themes/dark_plus.json @@ -154,10 +154,7 @@ } }, { - "scope": [ - "keyword.operator.or.regexp", - "keyword.control.anchor.regexp" - ], + "scope": ["keyword.operator.or.regexp", "keyword.control.anchor.regexp"], "settings": { "foreground": "#DCDCAA" } diff --git a/src/integrations/theme/default-themes/dark_vs.json b/src/integrations/theme/default-themes/dark_vs.json index 2d6daa713c..2abaefadfc 100644 --- a/src/integrations/theme/default-themes/dark_vs.json +++ b/src/integrations/theme/default-themes/dark_vs.json @@ -345,10 +345,7 @@ } }, { - "scope": [ - "punctuation.section.embedded.begin.php", - "punctuation.section.embedded.end.php" - ], + "scope": ["punctuation.section.embedded.begin.php", "punctuation.section.embedded.end.php"], "settings": { "foreground": "#569cd6" } @@ -367,11 +364,7 @@ }, { "name": "coloring of the Java import and package identifiers", - "scope": [ - "storage.modifier.import.java", - "variable.language.wildcard.java", - "storage.modifier.package.java" - ], + "scope": ["storage.modifier.import.java", "variable.language.wildcard.java", "storage.modifier.package.java"], "settings": { "foreground": "#d4d4d4" } diff --git a/src/integrations/theme/default-themes/hc_black.json b/src/integrations/theme/default-themes/hc_black.json index 6acbdb5894..b88b869d47 100644 --- a/src/integrations/theme/default-themes/hc_black.json +++ b/src/integrations/theme/default-themes/hc_black.json @@ -57,12 +57,7 @@ } }, { - "scope": [ - "constant.numeric", - "constant.other.color.rgb-value", - "constant.other.rgb-value", - "support.constant.color" - ], + "scope": ["constant.numeric", "constant.other.color.rgb-value", "constant.other.rgb-value", "support.constant.color"], "settings": { "foreground": "#b5cea8" } @@ -320,11 +315,7 @@ }, { "name": "coloring of the Java import and package identifiers", - "scope": [ - "storage.modifier.import.java", - "variable.language.wildcard.java", - "storage.modifier.package.java" - ], + "scope": ["storage.modifier.import.java", "variable.language.wildcard.java", "storage.modifier.package.java"], "settings": { "foreground": "#d4d4d4" } @@ -410,11 +401,7 @@ }, { "name": "Variable and parameter name", - "scope": [ - "variable", - "meta.definition.variable.name", - "support.variable" - ], + "scope": ["variable", "meta.definition.variable.name", "support.variable"], "settings": { "foreground": "#9CDCFE" } diff --git a/src/integrations/theme/default-themes/hc_light.json b/src/integrations/theme/default-themes/hc_light.json index 35f36229cb..5310173d9c 100644 --- a/src/integrations/theme/default-themes/hc_light.json +++ b/src/integrations/theme/default-themes/hc_light.json @@ -3,11 +3,7 @@ "name": "Light High Contrast", "tokenColors": [ { - "scope": [ - "meta.embedded", - "source.groovy.embedded", - "variable.legacy.builtin.python" - ], + "scope": ["meta.embedded", "source.groovy.embedded", "variable.legacy.builtin.python"], "settings": { "foreground": "#292929" } @@ -150,10 +146,7 @@ } }, { - "scope": [ - "punctuation.definition.quote.begin.markdown", - "punctuation.definition.list.begin.markdown" - ], + "scope": ["punctuation.definition.quote.begin.markdown", "punctuation.definition.list.begin.markdown"], "settings": { "foreground": "#0451A5" } @@ -335,10 +328,7 @@ } }, { - "scope": [ - "punctuation.section.embedded.begin.php", - "punctuation.section.embedded.end.php" - ], + "scope": ["punctuation.section.embedded.begin.php", "punctuation.section.embedded.end.php"], "settings": { "foreground": "#0F4A85" } @@ -356,11 +346,7 @@ } }, { - "scope": [ - "storage.modifier.import.java", - "variable.language.wildcard.java", - "storage.modifier.package.java" - ], + "scope": ["storage.modifier.import.java", "variable.language.wildcard.java", "storage.modifier.package.java"], "settings": { "foreground": "#000000" } @@ -519,10 +505,7 @@ } }, { - "scope": [ - "keyword.operator.or.regexp", - "keyword.control.anchor.regexp" - ], + "scope": ["keyword.operator.or.regexp", "keyword.control.anchor.regexp"], "settings": { "foreground": "#EE0000" } diff --git a/src/integrations/theme/default-themes/light_plus.json b/src/integrations/theme/default-themes/light_plus.json index 51fa251202..e103f48349 100644 --- a/src/integrations/theme/default-themes/light_plus.json +++ b/src/integrations/theme/default-themes/light_plus.json @@ -160,10 +160,7 @@ } }, { - "scope": [ - "keyword.operator.or.regexp", - "keyword.control.anchor.regexp" - ], + "scope": ["keyword.operator.or.regexp", "keyword.control.anchor.regexp"], "settings": { "foreground": "#EE0000" } diff --git a/src/integrations/theme/default-themes/light_vs.json b/src/integrations/theme/default-themes/light_vs.json index 678116e59b..f75e0942fb 100644 --- a/src/integrations/theme/default-themes/light_vs.json +++ b/src/integrations/theme/default-themes/light_vs.json @@ -185,10 +185,7 @@ } }, { - "scope": [ - "punctuation.definition.quote.begin.markdown", - "punctuation.definition.list.begin.markdown" - ], + "scope": ["punctuation.definition.quote.begin.markdown", "punctuation.definition.list.begin.markdown"], "settings": { "foreground": "#0451a5" } @@ -373,10 +370,7 @@ } }, { - "scope": [ - "punctuation.section.embedded.begin.php", - "punctuation.section.embedded.end.php" - ], + "scope": ["punctuation.section.embedded.begin.php", "punctuation.section.embedded.end.php"], "settings": { "foreground": "#800000" } @@ -395,11 +389,7 @@ }, { "name": "coloring of the Java import and package identifiers", - "scope": [ - "storage.modifier.import.java", - "variable.language.wildcard.java", - "storage.modifier.package.java" - ], + "scope": ["storage.modifier.import.java", "variable.language.wildcard.java", "storage.modifier.package.java"], "settings": { "foreground": "#000000" } diff --git a/src/integrations/theme/getTheme.ts b/src/integrations/theme/getTheme.ts index 1e9b8d487e..a5d4493b3d 100644 --- a/src/integrations/theme/getTheme.ts +++ b/src/integrations/theme/getTheme.ts @@ -32,10 +32,7 @@ function parseThemeString(themeString: string | undefined): any { export async function getTheme() { let currentTheme = undefined - const colorTheme = - vscode.workspace - .getConfiguration("workbench") - .get("colorTheme") || "Default Dark Modern" + const colorTheme = vscode.workspace.getConfiguration("workbench").get("colorTheme") || "Default Dark Modern" try { for (let i = vscode.extensions.all.length - 1; i >= 0; i--) { @@ -46,10 +43,7 @@ export async function getTheme() { if (extension.packageJSON?.contributes?.themes?.length > 0) { for (const theme of extension.packageJSON.contributes.themes) { if (theme.label === colorTheme) { - const themePath = path.join( - extension.extensionPath, - theme.path, - ) + const themePath = path.join(extension.extensionPath, theme.path) currentTheme = await fs.readFile(themePath, "utf-8") break } @@ -60,14 +54,7 @@ export async function getTheme() { if (currentTheme === undefined && defaultThemes[colorTheme]) { const filename = `${defaultThemes[colorTheme]}.json` currentTheme = await fs.readFile( - path.join( - getExtensionUri().fsPath, - "src", - "integrations", - "theme", - "default-themes", - filename, - ), + path.join(getExtensionUri().fsPath, "src", "integrations", "theme", "default-themes", filename), "utf-8", ) } @@ -77,14 +64,7 @@ export async function getTheme() { if (parsed.include) { const includeThemeString = await fs.readFile( - path.join( - getExtensionUri().fsPath, - "src", - "integrations", - "theme", - "default-themes", - parsed.include, - ), + path.join(getExtensionUri().fsPath, "src", "integrations", "theme", "default-themes", parsed.include), "utf-8", ) const includeTheme = parseThemeString(includeThemeString) @@ -94,11 +74,7 @@ export async function getTheme() { const converted = convertTheme(parsed) converted.base = ( - ["vs", "hc-black"].includes(converted.base) - ? converted.base - : colorTheme.includes("Light") - ? "vs" - : "vs-dark" + ["vs", "hc-black"].includes(converted.base) ? converted.base : colorTheme.includes("Light") ? "vs" : "vs-dark" ) as any return converted @@ -134,11 +110,7 @@ export function mergeJson( // Merge keys are used to determine whether an item form the second object should override one from the first const keptFromFirst: any[] = [] firstValue.forEach((item: any) => { - if ( - !secondValue.some((item2: any) => - mergeKeys[key](item, item2), - ) - ) { + if (!secondValue.some((item2: any) => mergeKeys[key](item, item2))) { keptFromFirst.push(item) } }) @@ -146,16 +118,9 @@ export function mergeJson( } else { copyOfFirst[key] = [...firstValue, ...secondValue] } - } else if ( - typeof secondValue === "object" && - typeof firstValue === "object" - ) { + } else if (typeof secondValue === "object" && typeof firstValue === "object") { // Object - copyOfFirst[key] = mergeJson( - firstValue, - secondValue, - mergeBehavior, - ) + copyOfFirst[key] = mergeJson(firstValue, secondValue, mergeBehavior) } else { // Other (boolean, number, string) copyOfFirst[key] = secondValue @@ -172,6 +137,5 @@ export function mergeJson( } function getExtensionUri(): vscode.Uri { - return vscode.extensions.getExtension("saoudrizwan.claude-dev")! - .extensionUri + return vscode.extensions.getExtension("saoudrizwan.claude-dev")!.extensionUri } diff --git a/src/integrations/workspace/WorkspaceTracker.ts b/src/integrations/workspace/WorkspaceTracker.ts index 1d390fe0dc..10dfac8f9e 100644 --- a/src/integrations/workspace/WorkspaceTracker.ts +++ b/src/integrations/workspace/WorkspaceTracker.ts @@ -3,9 +3,7 @@ import * as path from "path" import { listFiles } from "../../services/glob/list-files" import { ClineProvider } from "../../core/webview/ClineProvider" -const cwd = vscode.workspace.workspaceFolders - ?.map((folder) => folder.uri.fsPath) - .at(0) +const cwd = vscode.workspace.workspaceFolders?.map((folder) => folder.uri.fsPath).at(0) // Note: this is not a drop-in replacement for listFiles at the start of tasks, since that will be done for Desktops when there is no workspace selected class WorkspaceTracker { @@ -24,28 +22,20 @@ class WorkspaceTracker { return } const [files, _] = await listFiles(cwd, true, 1_000) - files.forEach((file) => - this.filePaths.add(this.normalizeFilePath(file)), - ) + files.forEach((file) => this.filePaths.add(this.normalizeFilePath(file))) this.workspaceDidUpdate() } private registerListeners() { // Listen for file creation // .bind(this) ensures the callback refers to class instance when using this, not necessary when using arrow function - this.disposables.push( - vscode.workspace.onDidCreateFiles(this.onFilesCreated.bind(this)), - ) + this.disposables.push(vscode.workspace.onDidCreateFiles(this.onFilesCreated.bind(this))) // Listen for file deletion - this.disposables.push( - vscode.workspace.onDidDeleteFiles(this.onFilesDeleted.bind(this)), - ) + this.disposables.push(vscode.workspace.onDidDeleteFiles(this.onFilesDeleted.bind(this))) // Listen for file renaming - this.disposables.push( - vscode.workspace.onDidRenameFiles(this.onFilesRenamed.bind(this)), - ) + this.disposables.push(vscode.workspace.onDidRenameFiles(this.onFilesRenamed.bind(this))) /* An event that is emitted when a workspace folder is added or removed. @@ -105,23 +95,16 @@ class WorkspaceTracker { } private normalizeFilePath(filePath: string): string { - const resolvedPath = cwd - ? path.resolve(cwd, filePath) - : path.resolve(filePath) + const resolvedPath = cwd ? path.resolve(cwd, filePath) : path.resolve(filePath) return filePath.endsWith("/") ? resolvedPath + "/" : resolvedPath } private async addFilePath(filePath: string): Promise { const normalizedPath = this.normalizeFilePath(filePath) try { - const stat = await vscode.workspace.fs.stat( - vscode.Uri.file(normalizedPath), - ) + const stat = await vscode.workspace.fs.stat(vscode.Uri.file(normalizedPath)) const isDirectory = (stat.type & vscode.FileType.Directory) !== 0 - const pathWithSlash = - isDirectory && !normalizedPath.endsWith("/") - ? normalizedPath + "/" - : normalizedPath + const pathWithSlash = isDirectory && !normalizedPath.endsWith("/") ? normalizedPath + "/" : normalizedPath this.filePaths.add(pathWithSlash) return pathWithSlash } catch { @@ -133,10 +116,7 @@ class WorkspaceTracker { private async removeFilePath(filePath: string): Promise { const normalizedPath = this.normalizeFilePath(filePath) - return ( - this.filePaths.delete(normalizedPath) || - this.filePaths.delete(normalizedPath + "/") - ) + return this.filePaths.delete(normalizedPath) || this.filePaths.delete(normalizedPath + "/") } public dispose() { diff --git a/src/integrations/workspace/get-python-env.ts b/src/integrations/workspace/get-python-env.ts index f24dc8140c..92575b408a 100644 --- a/src/integrations/workspace/get-python-env.ts +++ b/src/integrations/workspace/get-python-env.ts @@ -33,9 +33,7 @@ export async function getPythonEnvPath(): Promise { return undefined } // Get the active python environment path for the current workspace - const pythonEnv = await pythonApi?.environments?.getActiveEnvironmentPath( - workspaceFolder.uri, - ) + const pythonEnv = await pythonApi?.environments?.getActiveEnvironmentPath(workspaceFolder.uri) if (pythonEnv && pythonEnv.path) { return pythonEnv.path } else { diff --git a/src/services/browser/BrowserSession.ts b/src/services/browser/BrowserSession.ts index e70103f7e5..0b7b0961b5 100644 --- a/src/services/browser/BrowserSession.ts +++ b/src/services/browser/BrowserSession.ts @@ -1,13 +1,7 @@ import * as vscode from "vscode" import * as fs from "fs/promises" import * as path from "path" -import { - Browser, - Page, - ScreenshotOptions, - TimeoutError, - launch, -} from "puppeteer-core" +import { Browser, Page, ScreenshotOptions, TimeoutError, launch } from "puppeteer-core" // @ts-ignore import PCR from "puppeteer-chromium-resolver" import pWaitFor from "p-wait-for" @@ -85,9 +79,7 @@ export class BrowserSession { return {} } - async doAction( - action: (page: Page) => Promise, - ): Promise { + async doAction(action: (page: Page) => Promise): Promise { if (!this.page) { throw new Error( "Browser is not launched. This may occur if the browser was automatically closed by a non-`browser_action` tool.", diff --git a/src/services/glob/list-files.ts b/src/services/glob/list-files.ts index b5c9d3ad6c..8578b914d7 100644 --- a/src/services/glob/list-files.ts +++ b/src/services/glob/list-files.ts @@ -3,15 +3,10 @@ import os from "os" import * as path from "path" import { arePathsEqual } from "../../utils/path" -export async function listFiles( - dirPath: string, - recursive: boolean, - limit: number, -): Promise<[string[], boolean]> { +export async function listFiles(dirPath: string, recursive: boolean, limit: number): Promise<[string[], boolean]> { const absolutePath = path.resolve(dirPath) // Do not allow listing files in root or home directory, which cline tends to want to do when the user's prompt is vague. - const root = - process.platform === "win32" ? path.parse(absolutePath).root : "/" + const root = process.platform === "win32" ? path.parse(absolutePath).root : "/" const isRoot = arePathsEqual(absolutePath, root) if (isRoot) { return [[root], false] @@ -51,9 +46,7 @@ export async function listFiles( onlyFiles: false, // true by default, false means it will list directories on their own too } // * globs all files in one dir, ** globs files in nested directories - const files = recursive - ? await globbyLevelByLevel(limit, options) - : (await globby("*", options)).slice(0, limit) + const files = recursive ? await globbyLevelByLevel(limit, options) : (await globby("*", options)).slice(0, limit) return [files, files.length >= limit] } diff --git a/src/services/mcp/McpHub.ts b/src/services/mcp/McpHub.ts index 4734941cce..59dd3bf38b 100644 --- a/src/services/mcp/McpHub.ts +++ b/src/services/mcp/McpHub.ts @@ -1,8 +1,5 @@ import { Client } from "@modelcontextprotocol/sdk/client/index.js" -import { - StdioClientTransport, - StdioServerParameters, -} from "@modelcontextprotocol/sdk/client/stdio.js" +import { StdioClientTransport, StdioServerParameters } from "@modelcontextprotocol/sdk/client/stdio.js" import { CallToolResultSchema, ListResourcesResultSchema, @@ -17,18 +14,8 @@ import * as fs from "fs/promises" import * as path from "path" import * as vscode from "vscode" import { z } from "zod" -import { - ClineProvider, - GlobalFileNames, -} from "../../core/webview/ClineProvider" -import { - McpResource, - McpResourceResponse, - McpResourceTemplate, - McpServer, - McpTool, - McpToolCallResponse, -} from "../../shared/mcp" +import { ClineProvider, GlobalFileNames } from "../../core/webview/ClineProvider" +import { McpResource, McpResourceResponse, McpResourceTemplate, McpServer, McpTool, McpToolCallResponse } from "../../shared/mcp" import { fileExistsAtPath } from "../../utils/fs" import { arePathsEqual } from "../../utils/path" @@ -81,10 +68,7 @@ export class McpHub { if (!provider) { throw new Error("Provider not available") } - const mcpSettingsFilePath = path.join( - await provider.ensureSettingsDirectoryExists(), - GlobalFileNames.mcpSettings, - ) + const mcpSettingsFilePath = path.join(await provider.ensureSettingsDirectoryExists(), GlobalFileNames.mcpSettings) const fileExists = await fileExistsAtPath(mcpSettingsFilePath) if (!fileExists) { await fs.writeFile( @@ -120,20 +104,11 @@ export class McpHub { return } try { - vscode.window.showInformationMessage( - "Updating MCP servers...", - ) - await this.updateServerConnections( - result.data.mcpServers || {}, - ) - vscode.window.showInformationMessage( - "MCP servers updated", - ) + vscode.window.showInformationMessage("Updating MCP servers...") + await this.updateServerConnections(result.data.mcpServers || {}) + vscode.window.showInformationMessage("MCP servers updated") } catch (error) { - console.error( - "Failed to process MCP settings change:", - error, - ) + console.error("Failed to process MCP settings change:", error) } } }), @@ -151,23 +126,16 @@ export class McpHub { } } - private async connectToServer( - name: string, - config: StdioServerParameters, - ): Promise { + private async connectToServer(name: string, config: StdioServerParameters): Promise { // Remove existing connection if it exists (should never happen, the connection should be deleted beforehand) - this.connections = this.connections.filter( - (conn) => conn.server.name !== name, - ) + this.connections = this.connections.filter((conn) => conn.server.name !== name) try { // Each MCP server requires its own transport connection and has unique capabilities, configurations, and error handling. Having separate clients also allows proper scoping of resources/tools and independent server management like reconnection. const client = new Client( { name: "Cline", - version: - this.providerRef.deref()?.context.extension?.packageJSON - ?.version ?? "1.0.0", + version: this.providerRef.deref()?.context.extension?.packageJSON?.version ?? "1.0.0", }, { capabilities: {}, @@ -187,9 +155,7 @@ export class McpHub { transport.onerror = async (error) => { console.error(`Transport error for "${name}":`, error) - const connection = this.connections.find( - (conn) => conn.server.name === name, - ) + const connection = this.connections.find((conn) => conn.server.name === name) if (connection) { connection.server.status = "disconnected" this.appendErrorMessage(connection, error.message) @@ -198,9 +164,7 @@ export class McpHub { } transport.onclose = async () => { - const connection = this.connections.find( - (conn) => conn.server.name === name, - ) + const connection = this.connections.find((conn) => conn.server.name === name) if (connection) { connection.server.status = "disconnected" } @@ -209,9 +173,7 @@ export class McpHub { // If the config is invalid, show an error if (!StdioConfigSchema.safeParse(config).success) { - console.error( - `Invalid config for "${name}": missing or invalid parameters`, - ) + console.error(`Invalid config for "${name}": missing or invalid parameters`) const connection: McpConnection = { server: { name, @@ -246,9 +208,7 @@ export class McpHub { stderrStream.on("data", async (data: Buffer) => { const errorOutput = data.toString() console.error(`Server "${name}" stderr:`, errorOutput) - const connection = this.connections.find( - (conn) => conn.server.name === name, - ) + const connection = this.connections.find((conn) => conn.server.name === name) if (connection) { // NOTE: we do not set server status to "disconnected" because stderr logs do not necessarily mean the server crashed or disconnected, it could just be informational. In fact when the server first starts up, it immediately logs " server running on stdio" to stderr. this.appendErrorMessage(connection, errorOutput) @@ -293,28 +253,20 @@ export class McpHub { // Initial fetch of tools and resources connection.server.tools = await this.fetchToolsList(name) connection.server.resources = await this.fetchResourcesList(name) - connection.server.resourceTemplates = - await this.fetchResourceTemplatesList(name) + connection.server.resourceTemplates = await this.fetchResourceTemplatesList(name) } catch (error) { // Update status with error - const connection = this.connections.find( - (conn) => conn.server.name === name, - ) + const connection = this.connections.find((conn) => conn.server.name === name) if (connection) { connection.server.status = "disconnected" - this.appendErrorMessage( - connection, - error instanceof Error ? error.message : String(error), - ) + this.appendErrorMessage(connection, error instanceof Error ? error.message : String(error)) } throw error } } private appendErrorMessage(connection: McpConnection, error: string) { - const newError = connection.server.error - ? `${connection.server.error}\n${error}` - : error + const newError = connection.server.error ? `${connection.server.error}\n${error}` : error connection.server.error = newError //.slice(0, 800) } @@ -322,10 +274,7 @@ export class McpHub { try { const response = await this.connections .find((conn) => conn.server.name === serverName) - ?.client.request( - { method: "tools/list" }, - ListToolsResultSchema, - ) + ?.client.request({ method: "tools/list" }, ListToolsResultSchema) return response?.tools || [] } catch (error) { // console.error(`Failed to fetch tools for ${serverName}:`, error) @@ -333,16 +282,11 @@ export class McpHub { } } - private async fetchResourcesList( - serverName: string, - ): Promise { + private async fetchResourcesList(serverName: string): Promise { try { const response = await this.connections .find((conn) => conn.server.name === serverName) - ?.client.request( - { method: "resources/list" }, - ListResourcesResultSchema, - ) + ?.client.request({ method: "resources/list" }, ListResourcesResultSchema) return response?.resources || [] } catch (error) { // console.error(`Failed to fetch resources for ${serverName}:`, error) @@ -350,16 +294,11 @@ export class McpHub { } } - private async fetchResourceTemplatesList( - serverName: string, - ): Promise { + private async fetchResourceTemplatesList(serverName: string): Promise { try { const response = await this.connections .find((conn) => conn.server.name === serverName) - ?.client.request( - { method: "resources/templates/list" }, - ListResourceTemplatesResultSchema, - ) + ?.client.request({ method: "resources/templates/list" }, ListResourceTemplatesResultSchema) return response?.resourceTemplates || [] } catch (error) { // console.error(`Failed to fetch resource templates for ${serverName}:`, error) @@ -368,9 +307,7 @@ export class McpHub { } async deleteConnection(name: string): Promise { - const connection = this.connections.find( - (conn) => conn.server.name === name, - ) + const connection = this.connections.find((conn) => conn.server.name === name) if (connection) { try { // connection.client.removeNotificationHandler("notifications/tools/list_changed") @@ -382,20 +319,14 @@ export class McpHub { } catch (error) { console.error(`Failed to close transport for ${name}:`, error) } - this.connections = this.connections.filter( - (conn) => conn.server.name !== name, - ) + this.connections = this.connections.filter((conn) => conn.server.name !== name) } } - async updateServerConnections( - newServers: Record, - ): Promise { + async updateServerConnections(newServers: Record): Promise { this.isConnecting = true this.removeAllFileWatchers() - const currentNames = new Set( - this.connections.map((conn) => conn.server.name), - ) + const currentNames = new Set(this.connections.map((conn) => conn.server.name)) const newNames = new Set(Object.keys(newServers)) // Delete removed servers @@ -408,9 +339,7 @@ export class McpHub { // Update or add servers for (const [name, config] of Object.entries(newServers)) { - const currentConnection = this.connections.find( - (conn) => conn.server.name === name, - ) + const currentConnection = this.connections.find((conn) => conn.server.name === name) if (!currentConnection) { // New server @@ -418,27 +347,17 @@ export class McpHub { this.setupFileWatcher(name, config) await this.connectToServer(name, config) } catch (error) { - console.error( - `Failed to connect to new MCP server ${name}:`, - error, - ) + console.error(`Failed to connect to new MCP server ${name}:`, error) } - } else if ( - !deepEqual(JSON.parse(currentConnection.server.config), config) - ) { + } else if (!deepEqual(JSON.parse(currentConnection.server.config), config)) { // Existing server with changed config try { this.setupFileWatcher(name, config) await this.deleteConnection(name) await this.connectToServer(name, config) - console.log( - `Reconnected MCP server with updated config: ${name}`, - ) + console.log(`Reconnected MCP server with updated config: ${name}`) } catch (error) { - console.error( - `Failed to reconnect MCP server ${name}:`, - error, - ) + console.error(`Failed to reconnect MCP server ${name}:`, error) } } // If server exists with same config, do nothing @@ -448,9 +367,7 @@ export class McpHub { } private setupFileWatcher(name: string, config: any) { - const filePath = config.args?.find((arg: string) => - arg.includes("build/index.js"), - ) + const filePath = config.args?.find((arg: string) => arg.includes("build/index.js")) if (filePath) { // we use chokidar instead of onDidSaveTextDocument because it doesn't require the file to be open in the editor. The settings config is better suited for onDidSave since that will be manually updated by the user or Cline (and we want to detect save events, not every file change) const watcher = chokidar.watch(filePath, { @@ -460,9 +377,7 @@ export class McpHub { }) watcher.on("change", () => { - console.log( - `Detected change in ${filePath}. Restarting server ${name}...`, - ) + console.log(`Detected change in ${filePath}. Restarting server ${name}...`) this.restartConnection(name) }) @@ -483,14 +398,10 @@ export class McpHub { } // Get existing connection and update its status - const connection = this.connections.find( - (conn) => conn.server.name === serverName, - ) + const connection = this.connections.find((conn) => conn.server.name === serverName) const config = connection?.server.config if (config) { - vscode.window.showInformationMessage( - `Restarting ${serverName} MCP server...`, - ) + vscode.window.showInformationMessage(`Restarting ${serverName} MCP server...`) connection.server.status = "connecting" connection.server.error = "" await this.notifyWebviewOfServerChanges() @@ -499,17 +410,10 @@ export class McpHub { await this.deleteConnection(serverName) // Try to connect again using existing config await this.connectToServer(serverName, JSON.parse(config)) - vscode.window.showInformationMessage( - `${serverName} MCP server connected`, - ) + vscode.window.showInformationMessage(`${serverName} MCP server connected`) } catch (error) { - console.error( - `Failed to restart connection for ${serverName}:`, - error, - ) - vscode.window.showErrorMessage( - `Failed to connect to ${serverName} MCP server`, - ) + console.error(`Failed to restart connection for ${serverName}:`, error) + vscode.window.showErrorMessage(`Failed to connect to ${serverName} MCP server`) } } @@ -537,13 +441,8 @@ export class McpHub { // Using server - async readResource( - serverName: string, - uri: string, - ): Promise { - const connection = this.connections.find( - (conn) => conn.server.name === serverName, - ) + async readResource(serverName: string, uri: string): Promise { + const connection = this.connections.find((conn) => conn.server.name === serverName) if (!connection) { throw new Error(`No connection found for server: ${serverName}`) } @@ -558,14 +457,8 @@ export class McpHub { ) } - async callTool( - serverName: string, - toolName: string, - toolArguments?: Record, - ): Promise { - const connection = this.connections.find( - (conn) => conn.server.name === serverName, - ) + async callTool(serverName: string, toolName: string, toolArguments?: Record): Promise { + const connection = this.connections.find((conn) => conn.server.name === serverName) if (!connection) { throw new Error( `No connection found for server: ${serverName}. Please make sure to use MCP servers available under 'Connected MCP Servers'.`, @@ -589,10 +482,7 @@ export class McpHub { try { await this.deleteConnection(connection.server.name) } catch (error) { - console.error( - `Failed to close connection for ${connection.server.name}:`, - error, - ) + console.error(`Failed to close connection for ${connection.server.name}:`, error) } } this.connections = [] diff --git a/src/services/ripgrep/index.ts b/src/services/ripgrep/index.ts index 7e02a57478..b9ebe68c77 100644 --- a/src/services/ripgrep/index.ts +++ b/src/services/ripgrep/index.ts @@ -122,12 +122,7 @@ async function execRipgrep(bin: string, args: string[]): Promise { }) } -export async function regexSearchFiles( - cwd: string, - directoryPath: string, - regex: string, - filePattern?: string, -): Promise { +export async function regexSearchFiles(cwd: string, directoryPath: string, regex: string, filePattern?: string): Promise { const vscodeAppRoot = vscode.env.appRoot const rgPath = await getBinPath(vscodeAppRoot) @@ -135,16 +130,7 @@ export async function regexSearchFiles( throw new Error("Could not find ripgrep binary") } - const args = [ - "--json", - "-e", - regex, - "--glob", - filePattern || "*", - "--context", - "1", - directoryPath, - ] + const args = ["--json", "-e", regex, "--glob", filePattern || "*", "--context", "1", directoryPath] let output: string try { @@ -173,9 +159,7 @@ export async function regexSearchFiles( } } else if (parsed.type === "context" && currentResult) { if (parsed.data.line_number < currentResult.line!) { - currentResult.beforeContext!.push( - parsed.data.lines.text, - ) + currentResult.beforeContext!.push(parsed.data.lines.text) } else { currentResult.afterContext!.push(parsed.data.lines.text) } @@ -216,11 +200,7 @@ function formatResults(results: SearchResult[], cwd: string): string { output += `${filePath.toPosix()}\n│----\n` fileResults.forEach((result, index) => { - const allLines = [ - ...result.beforeContext, - result.match, - ...result.afterContext, - ] + const allLines = [...result.beforeContext, result.match, ...result.afterContext] allLines.forEach((line) => { output += `│${line?.trimEnd() ?? ""}\n` }) diff --git a/src/services/tree-sitter/index.ts b/src/services/tree-sitter/index.ts index cc5275c641..19d0234f01 100644 --- a/src/services/tree-sitter/index.ts +++ b/src/services/tree-sitter/index.ts @@ -5,9 +5,7 @@ import { LanguageParser, loadRequiredLanguageParsers } from "./languageParser" import { fileExistsAtPath } from "../../utils/fs" // TODO: implement caching behavior to avoid having to keep analyzing project for new tasks. -export async function parseSourceCodeForDefinitionsTopLevel( - dirPath: string, -): Promise { +export async function parseSourceCodeForDefinitionsTopLevel(dirPath: string): Promise { // check if the path exists const dirExists = await fileExistsAtPath(path.resolve(dirPath)) if (!dirExists) { @@ -79,12 +77,8 @@ function separateFiles(allFiles: string[]): { "php", "swift", ].map((e) => `.${e}`) - const filesToParse = allFiles - .filter((file) => extensions.includes(path.extname(file))) - .slice(0, 50) // 50 files max - const remainingFiles = allFiles.filter( - (file) => !filesToParse.includes(file), - ) + const filesToParse = allFiles.filter((file) => extensions.includes(path.extname(file))).slice(0, 50) // 50 files max + const remainingFiles = allFiles.filter((file) => !filesToParse.includes(file)) return { filesToParse, remainingFiles } } @@ -104,10 +98,7 @@ This approach allows us to focus on the most relevant parts of the code (defined - https://github.com/tree-sitter/tree-sitter/blob/master/lib/binding_web/test/helper.js - https://tree-sitter.github.io/tree-sitter/code-navigation-systems */ -async function parseFile( - filePath: string, - languageParsers: LanguageParser, -): Promise { +async function parseFile(filePath: string, languageParsers: LanguageParser): Promise { const fileContent = await fs.readFile(filePath, "utf8") const ext = path.extname(filePath).toLowerCase().slice(1) @@ -127,9 +118,7 @@ async function parseFile( const captures = query.captures(tree.rootNode) // Sort captures by their start position - captures.sort( - (a, b) => a.node.startPosition.row - b.node.startPosition.row, - ) + captures.sort((a, b) => a.node.startPosition.row - b.node.startPosition.row) // Split the file content into individual lines const lines = fileContent.split("\n") diff --git a/src/services/tree-sitter/languageParser.ts b/src/services/tree-sitter/languageParser.ts index b7a5f26817..2d791b39a8 100644 --- a/src/services/tree-sitter/languageParser.ts +++ b/src/services/tree-sitter/languageParser.ts @@ -23,9 +23,7 @@ export interface LanguageParser { } async function loadLanguage(langName: string) { - return await Parser.Language.load( - path.join(__dirname, `tree-sitter-${langName}.wasm`), - ) + return await Parser.Language.load(path.join(__dirname, `tree-sitter-${langName}.wasm`)) } let isParserInitialized = false @@ -59,13 +57,9 @@ Sources: - https://github.com/tree-sitter/tree-sitter/blob/master/lib/binding_web/README.md - https://github.com/tree-sitter/tree-sitter/blob/master/lib/binding_web/test/query-test.js */ -export async function loadRequiredLanguageParsers( - filesToParse: string[], -): Promise { +export async function loadRequiredLanguageParsers(filesToParse: string[]): Promise { await initializeParser() - const extensionsToLoad = new Set( - filesToParse.map((file) => path.extname(file).toLowerCase().slice(1)), - ) + const extensionsToLoad = new Set(filesToParse.map((file) => path.extname(file).toLowerCase().slice(1))) const parsers: LanguageParser = {} for (const ext of extensionsToLoad) { let language: Parser.Language diff --git a/src/shared/ExtensionMessage.ts b/src/shared/ExtensionMessage.ts index 04ab46686d..e4760282a5 100644 --- a/src/shared/ExtensionMessage.ts +++ b/src/shared/ExtensionMessage.ts @@ -21,12 +21,7 @@ export interface ExtensionMessage { | "mcpServers" | "relinquishControl" text?: string - action?: - | "chatButtonClicked" - | "mcpButtonClicked" - | "settingsButtonClicked" - | "historyButtonClicked" - | "didBecomeVisible" + action?: "chatButtonClicked" | "mcpButtonClicked" | "settingsButtonClicked" | "historyButtonClicked" | "didBecomeVisible" invoke?: "sendMessage" | "primaryButtonClick" | "secondaryButtonClick" state?: ExtensionState images?: string[] @@ -118,14 +113,7 @@ export interface ClineSayTool { } // must keep in sync with system prompt -export const browserActions = [ - "launch", - "click", - "type", - "scroll_down", - "scroll_up", - "close", -] as const +export const browserActions = ["launch", "click", "type", "scroll_down", "scroll_up", "close"] as const export type BrowserAction = (typeof browserActions)[number] export interface ClineSayBrowserAction { diff --git a/src/shared/WebviewMessage.ts b/src/shared/WebviewMessage.ts index ff6a991b91..419306ef6c 100644 --- a/src/shared/WebviewMessage.ts +++ b/src/shared/WebviewMessage.ts @@ -38,9 +38,6 @@ export interface WebviewMessage { autoApprovalSettings?: AutoApprovalSettings } -export type ClineAskResponse = - | "yesButtonClicked" - | "noButtonClicked" - | "messageResponse" +export type ClineAskResponse = "yesButtonClicked" | "noButtonClicked" | "messageResponse" export type ClineCheckpointRestore = "task" | "workspace" | "taskAndWorkspace" diff --git a/src/shared/api.ts b/src/shared/api.ts index 3e1681c7d5..d87d13d272 100644 --- a/src/shared/api.ts +++ b/src/shared/api.ts @@ -59,8 +59,7 @@ export interface ModelInfo { // Anthropic // https://docs.anthropic.com/en/docs/about-claude/models export type AnthropicModelId = keyof typeof anthropicModels -export const anthropicDefaultModelId: AnthropicModelId = - "claude-3-5-sonnet-20241022" +export const anthropicDefaultModelId: AnthropicModelId = "claude-3-5-sonnet-20241022" export const anthropicModels = { "claude-3-5-sonnet-20241022": { maxTokens: 8192, @@ -108,8 +107,7 @@ export const anthropicModels = { // AWS Bedrock // https://docs.aws.amazon.com/bedrock/latest/userguide/conversation-inference.html export type BedrockModelId = keyof typeof bedrockModels -export const bedrockDefaultModelId: BedrockModelId = - "anthropic.claude-3-5-sonnet-20241022-v2:0" +export const bedrockDefaultModelId: BedrockModelId = "anthropic.claude-3-5-sonnet-20241022-v2:0" export const bedrockModels = { "anthropic.claude-3-5-sonnet-20241022-v2:0": { maxTokens: 8192, @@ -182,8 +180,7 @@ export const openRouterDefaultModelInfo: ModelInfo = { // Vertex AI // https://cloud.google.com/vertex-ai/generative-ai/docs/partner-models/use-claude export type VertexModelId = keyof typeof vertexModels -export const vertexDefaultModelId: VertexModelId = - "claude-3-5-sonnet-v2@20241022" +export const vertexDefaultModelId: VertexModelId = "claude-3-5-sonnet-v2@20241022" export const vertexModels = { "claude-3-5-sonnet-v2@20241022": { maxTokens: 8192, @@ -240,8 +237,7 @@ export const openAiModelInfoSaneDefaults: ModelInfo = { // Gemini // https://ai.google.dev/gemini-api/docs/models/gemini export type GeminiModelId = keyof typeof geminiModels -export const geminiDefaultModelId: GeminiModelId = - "gemini-2.0-flash-thinking-exp-1219" +export const geminiDefaultModelId: GeminiModelId = "gemini-2.0-flash-thinking-exp-1219" export const geminiModels = { "gemini-2.0-flash-thinking-exp-1219": { maxTokens: 8192, diff --git a/src/shared/array.ts b/src/shared/array.ts index 9a847a7570..b87c458fd3 100644 --- a/src/shared/array.ts +++ b/src/shared/array.ts @@ -6,10 +6,7 @@ * order, until it finds one where predicate returns true. If such an element is found, * findLastIndex immediately returns that element index. Otherwise, findLastIndex returns -1. */ -export function findLastIndex( - array: Array, - predicate: (value: T, index: number, obj: T[]) => boolean, -): number { +export function findLastIndex(array: Array, predicate: (value: T, index: number, obj: T[]) => boolean): number { let l = array.length while (l--) { if (predicate(array[l], l, array)) { @@ -19,10 +16,7 @@ export function findLastIndex( return -1 } -export function findLast( - array: Array, - predicate: (value: T, index: number, obj: T[]) => boolean, -): T | undefined { +export function findLast(array: Array, predicate: (value: T, index: number, obj: T[]) => boolean): T | undefined { const index = findLastIndex(array, predicate) return index === -1 ? undefined : array[index] } diff --git a/src/shared/combineApiRequests.ts b/src/shared/combineApiRequests.ts index 5bac684521..36f318babf 100644 --- a/src/shared/combineApiRequests.ts +++ b/src/shared/combineApiRequests.ts @@ -22,18 +22,12 @@ export function combineApiRequests(messages: ClineMessage[]): ClineMessage[] { const combinedApiRequests: ClineMessage[] = [] for (let i = 0; i < messages.length; i++) { - if ( - messages[i].type === "say" && - messages[i].say === "api_req_started" - ) { + if (messages[i].type === "say" && messages[i].say === "api_req_started") { let startedRequest = JSON.parse(messages[i].text || "{}") let j = i + 1 while (j < messages.length) { - if ( - messages[j].type === "say" && - messages[j].say === "api_req_finished" - ) { + if (messages[j].type === "say" && messages[j].say === "api_req_finished") { let finishedRequest = JSON.parse(messages[j].text || "{}") let combinedRequest = { ...startedRequest, @@ -60,14 +54,10 @@ export function combineApiRequests(messages: ClineMessage[]): ClineMessage[] { // Replace original api_req_started and remove api_req_finished return messages - .filter( - (msg) => !(msg.type === "say" && msg.say === "api_req_finished"), - ) + .filter((msg) => !(msg.type === "say" && msg.say === "api_req_finished")) .map((msg) => { if (msg.type === "say" && msg.say === "api_req_started") { - const combinedRequest = combinedApiRequests.find( - (req) => req.ts === msg.ts, - ) + const combinedRequest = combinedApiRequests.find((req) => req.ts === msg.ts) return combinedRequest || msg } return msg diff --git a/src/shared/combineCommandSequences.ts b/src/shared/combineCommandSequences.ts index 03ee5499ac..3e41cd2df9 100644 --- a/src/shared/combineCommandSequences.ts +++ b/src/shared/combineCommandSequences.ts @@ -20,34 +20,22 @@ import { ClineMessage } from "./ExtensionMessage" * const result = simpleCombineCommandSequences(messages); * // Result: [{ type: 'ask', ask: 'command', text: 'ls\nfile1.txt\nfile2.txt', ts: 1625097600000 }] */ -export function combineCommandSequences( - messages: ClineMessage[], -): ClineMessage[] { +export function combineCommandSequences(messages: ClineMessage[]): ClineMessage[] { const combinedCommands: ClineMessage[] = [] // First pass: combine commands with their outputs for (let i = 0; i < messages.length; i++) { - if ( - messages[i].type === "ask" && - (messages[i].ask === "command" || messages[i].say === "command") - ) { + if (messages[i].type === "ask" && (messages[i].ask === "command" || messages[i].say === "command")) { let combinedText = messages[i].text || "" let didAddOutput = false let j = i + 1 while (j < messages.length) { - if ( - messages[j].type === "ask" && - (messages[j].ask === "command" || - messages[j].say === "command") - ) { + if (messages[j].type === "ask" && (messages[j].ask === "command" || messages[j].say === "command")) { // Stop if we encounter the next command break } - if ( - messages[j].ask === "command_output" || - messages[j].say === "command_output" - ) { + if (messages[j].ask === "command_output" || messages[j].say === "command_output") { if (!didAddOutput) { // Add a newline before the first output combinedText += `\n${COMMAND_OUTPUT_STRING}` @@ -73,18 +61,10 @@ export function combineCommandSequences( // Second pass: remove command_outputs and replace original commands with combined ones return messages - .filter( - (msg) => - !(msg.ask === "command_output" || msg.say === "command_output"), - ) + .filter((msg) => !(msg.ask === "command_output" || msg.say === "command_output")) .map((msg) => { - if ( - msg.type === "ask" && - (msg.ask === "command" || msg.say === "command") - ) { - const combinedCommand = combinedCommands.find( - (cmd) => cmd.ts === msg.ts, - ) + if (msg.type === "ask" && (msg.ask === "command" || msg.say === "command")) { + const combinedCommand = combinedCommands.find((cmd) => cmd.ts === msg.ts) return combinedCommand || msg } return msg diff --git a/src/shared/context-mentions.ts b/src/shared/context-mentions.ts index 1c1c86488e..3912868b10 100644 --- a/src/shared/context-mentions.ts +++ b/src/shared/context-mentions.ts @@ -44,6 +44,5 @@ Mention regex: - `mentionRegexGlobal`: Creates a global version of the `mentionRegex` to find all matches within a given string. */ -export const mentionRegex = - /@((?:\/|\w+:\/\/)[^\s]+?|problems\b)(?=[.,;:!?]?(?=[\s\r\n]|$))/ +export const mentionRegex = /@((?:\/|\w+:\/\/)[^\s]+?|problems\b)(?=[.,;:!?]?(?=[\s\r\n]|$))/ export const mentionRegexGlobal = new RegExp(mentionRegex.source, "g") diff --git a/src/shared/getApiMetrics.ts b/src/shared/getApiMetrics.ts index ee481cb19d..23fa0516d9 100644 --- a/src/shared/getApiMetrics.ts +++ b/src/shared/getApiMetrics.ts @@ -35,16 +35,10 @@ export function getApiMetrics(messages: ClineMessage[]): ApiMetrics { } messages.forEach((message) => { - if ( - message.type === "say" && - (message.say === "api_req_started" || - message.say === "deleted_api_reqs") && - message.text - ) { + if (message.type === "say" && (message.say === "api_req_started" || message.say === "deleted_api_reqs") && message.text) { try { const parsedData = JSON.parse(message.text) - const { tokensIn, tokensOut, cacheWrites, cacheReads, cost } = - parsedData + const { tokensIn, tokensOut, cacheWrites, cacheReads, cost } = parsedData if (typeof tokensIn === "number") { result.totalTokensIn += tokensIn @@ -53,12 +47,10 @@ export function getApiMetrics(messages: ClineMessage[]): ApiMetrics { result.totalTokensOut += tokensOut } if (typeof cacheWrites === "number") { - result.totalCacheWrites = - (result.totalCacheWrites ?? 0) + cacheWrites + result.totalCacheWrites = (result.totalCacheWrites ?? 0) + cacheWrites } if (typeof cacheReads === "number") { - result.totalCacheReads = - (result.totalCacheReads ?? 0) + cacheReads + result.totalCacheReads = (result.totalCacheReads ?? 0) + cacheReads } if (typeof cost === "number") { result.totalCost += cost diff --git a/src/utils/cost.ts b/src/utils/cost.ts index 04caf5a586..f8f5f2b125 100644 --- a/src/utils/cost.ts +++ b/src/utils/cost.ts @@ -10,19 +10,15 @@ export function calculateApiCost( const modelCacheWritesPrice = modelInfo.cacheWritesPrice let cacheWritesCost = 0 if (cacheCreationInputTokens && modelCacheWritesPrice) { - cacheWritesCost = - (modelCacheWritesPrice / 1_000_000) * cacheCreationInputTokens + cacheWritesCost = (modelCacheWritesPrice / 1_000_000) * cacheCreationInputTokens } const modelCacheReadsPrice = modelInfo.cacheReadsPrice let cacheReadsCost = 0 if (cacheReadInputTokens && modelCacheReadsPrice) { - cacheReadsCost = - (modelCacheReadsPrice / 1_000_000) * cacheReadInputTokens + cacheReadsCost = (modelCacheReadsPrice / 1_000_000) * cacheReadInputTokens } - const baseInputCost = - ((modelInfo.inputPrice || 0) / 1_000_000) * inputTokens + const baseInputCost = ((modelInfo.inputPrice || 0) / 1_000_000) * inputTokens const outputCost = ((modelInfo.outputPrice || 0) / 1_000_000) * outputTokens - const totalCost = - cacheWritesCost + cacheReadsCost + baseInputCost + outputCost + const totalCost = cacheWritesCost + cacheReadsCost + baseInputCost + outputCost return totalCost } diff --git a/src/utils/fs.test.ts b/src/utils/fs.test.ts index 32c16ac712..ea9f132d5b 100644 --- a/src/utils/fs.test.ts +++ b/src/utils/fs.test.ts @@ -6,10 +6,7 @@ import "should" import { createDirectoriesForFile, fileExistsAtPath } from "./fs" describe("Filesystem Utilities", () => { - const tmpDir = path.join( - os.tmpdir(), - "cline-test-" + Math.random().toString(36).slice(2), - ) + const tmpDir = path.join(os.tmpdir(), "cline-test-" + Math.random().toString(36).slice(2)) // Clean up after tests after(async () => { @@ -39,13 +36,7 @@ describe("Filesystem Utilities", () => { describe("createDirectoriesForFile", () => { it("should create all necessary directories", async () => { - const deepPath = path.join( - tmpDir, - "deep", - "nested", - "dir", - "file.txt", - ) + const deepPath = path.join(tmpDir, "deep", "nested", "dir", "file.txt") const createdDirs = await createDirectoriesForFile(deepPath) // Verify directories were created @@ -68,14 +59,7 @@ describe("Filesystem Utilities", () => { }) it("should normalize paths", async () => { - const unnormalizedPath = path.join( - tmpDir, - "a", - "..", - "b", - ".", - "file.txt", - ) + const unnormalizedPath = path.join(tmpDir, "a", "..", "b", ".", "file.txt") const createdDirs = await createDirectoriesForFile(unnormalizedPath) // Should create only the necessary directory diff --git a/src/utils/fs.ts b/src/utils/fs.ts index 38c084a1b5..9f7af84e4a 100644 --- a/src/utils/fs.ts +++ b/src/utils/fs.ts @@ -8,9 +8,7 @@ import * as path from "path" * @param filePath - The full path to a file. * @returns A promise that resolves to an array of newly created directories. */ -export async function createDirectoriesForFile( - filePath: string, -): Promise { +export async function createDirectoriesForFile(filePath: string): Promise { const newDirectories: string[] = [] const normalizedFilePath = path.normalize(filePath) // Normalize path for cross-platform compatibility const directoryPath = path.dirname(normalizedFilePath) diff --git a/src/utils/path.test.ts b/src/utils/path.test.ts index 60626ed69c..8efa6e59e2 100644 --- a/src/utils/path.test.ts +++ b/src/utils/path.test.ts @@ -30,9 +30,7 @@ describe("Path Utilities", () => { it("should handle desktop path", () => { const desktop = path.join(os.homedir(), "Desktop") const testPath = path.join(desktop, "test.txt") - getReadablePath(desktop, "test.txt").should.equal( - testPath.replace(/\\/g, "/"), - ) + getReadablePath(desktop, "test.txt").should.equal(testPath.replace(/\\/g, "/")) }) it("should show relative paths within cwd", () => { diff --git a/src/utils/path.ts b/src/utils/path.ts index 5253126cda..b61eb38bed 100644 --- a/src/utils/path.ts +++ b/src/utils/path.ts @@ -72,10 +72,7 @@ function normalizePath(p: string): string { let normalized = path.normalize(p) // however it doesn't remove trailing slashes // remove trailing slash, except for root paths - if ( - normalized.length > 1 && - (normalized.endsWith("/") || normalized.endsWith("\\")) - ) { + if (normalized.length > 1 && (normalized.endsWith("/") || normalized.endsWith("\\"))) { normalized = normalized.slice(0, -1) } return normalized diff --git a/webview-ui/public/index.html b/webview-ui/public/index.html index 202d93d3ef..bd3562a687 100644 --- a/webview-ui/public/index.html +++ b/webview-ui/public/index.html @@ -5,9 +5,7 @@ - +