Remove experimental tools: insert_content and search_and_replace

- Deleted all references to the insert_content and search_and_replace experimental tools from the codebase, including tests, tool descriptions, and related logic.
- Updated experiment configurations and schemas to reflect the removal of these tools.
- Adjusted tests to ensure they no longer check for the presence of the removed tools.
- Cleaned up related imports and tool groups to maintain code clarity and organization.

Author: samhvw8

evals/packages/types/src/roo-code-defaults.ts

@@ -56,8 +56,6 @@ export const rooCodeDefaults: RooCodeSettings = {
 	diffEnabled: true,
 	fuzzyMatchThreshold: 1.0,
 	experiments: {
-		search_and_replace: false,
-		insert_content: false,
 		powerSteering: false,
 	},
 

evals/packages/types/src/roo-code.ts

@@ -271,7 +271,7 @@ export type CustomSupportPrompts = z.infer<typeof customSupportPromptsSchema>
  * ExperimentId
  */
 
-export const experimentIds = ["search_and_replace", "insert_content", "powerSteering"] as const
+export const experimentIds = ["powerSteering"] as const
 
 export const experimentIdsSchema = z.enum(experimentIds)
 
@@ -282,8 +282,6 @@ export type ExperimentId = z.infer<typeof experimentIdsSchema>
  */
 
 const experimentsSchema = z.object({
-	search_and_replace: z.boolean(),
-	insert_content: z.boolean(),
 	powerSteering: z.boolean(),
 })
 

src/core/Cline.ts

@@ -64,8 +64,6 @@ import { validateToolUse, isToolAllowedForMode, ToolName } from "./mode-validato
 import { getWorkspacePath } from "../utils/path"
 import { writeToFileTool } from "./tools/writeToFileTool"
 import { applyDiffTool } from "./tools/applyDiffTool"
-import { insertContentTool } from "./tools/insertContentTool"
-import { searchAndReplaceTool } from "./tools/searchAndReplaceTool"
 import { listCodeDefinitionNamesTool } from "./tools/listCodeDefinitionNamesTool"
 import { searchFilesTool } from "./tools/searchFilesTool"
 import { browserActionTool } from "./tools/browserActionTool"
@@ -1396,10 +1394,6 @@ export class Cline extends EventEmitter<ClineEvents> {
 							return `[${block.name} for '${block.params.regex}'${
 								block.params.file_pattern ? ` in '${block.params.file_pattern}'` : ""
 							}]`
-						case "insert_content":
-							return `[${block.name} for '${block.params.path}']`
-						case "search_and_replace":
-							return `[${block.name} for '${block.params.path}']`
 						case "list_files":
 							return `[${block.name} for '${block.params.path}']`
 						case "list_code_definition_names":
@@ -1576,19 +1570,6 @@ export class Cline extends EventEmitter<ClineEvents> {
 					case "apply_diff":
 						await applyDiffTool(this, block, askApproval, handleError, pushToolResult, removeClosingTag)
 						break
-					case "insert_content":
-						await insertContentTool(this, block, askApproval, handleError, pushToolResult, removeClosingTag)
-						break
-					case "search_and_replace":
-						await searchAndReplaceTool(
-							this,
-							block,
-							askApproval,
-							handleError,
-							pushToolResult,
-							removeClosingTag,
-						)
-						break
 					case "read_file":
 						await readFileTool(this, block, askApproval, handleError, pushToolResult, removeClosingTag)
 

src/core/assistant-message/index.ts

@@ -13,8 +13,6 @@ export const toolUseNames = [
 	"read_file",
 	"write_to_file",
 	"apply_diff",
-	"insert_content",
-	"search_and_replace",
 	"search_files",
 	"list_files",
 	"list_code_definition_names",
@@ -94,11 +92,6 @@ export interface WriteToFileToolUse extends ToolUse {
 	params: Partial<Pick<Record<ToolParamName, string>, "path" | "content" | "line_count">>
 }
 
-export interface InsertCodeBlockToolUse extends ToolUse {
-	name: "insert_content"
-	params: Partial<Pick<Record<ToolParamName, string>, "path" | "operations">>
-}
-
 export interface SearchFilesToolUse extends ToolUse {
 	name: "search_files"
 	params: Partial<Pick<Record<ToolParamName, string>, "path" | "regex" | "file_pattern">>

src/core/diff/insert-groups.ts

@@ -170,10 +170,7 @@ describe("SYSTEM_PROMPT", () => {
 
 	beforeEach(() => {
 		// Reset experiments before each test to ensure they're disabled by default
-		experiments = {
-			[EXPERIMENT_IDS.SEARCH_AND_REPLACE]: false,
-			[EXPERIMENT_IDS.INSERT_BLOCK]: false,
-		}
+		experiments = {}
 	})
 
 	beforeEach(() => {
@@ -481,10 +478,7 @@ describe("SYSTEM_PROMPT", () => {
 	describe("experimental tools", () => {
 		it("should disable experimental tools by default", async () => {
 			// Set experiments to explicitly disable experimental tools
-			const experimentsConfig = {
-				[EXPERIMENT_IDS.SEARCH_AND_REPLACE]: false,
-				[EXPERIMENT_IDS.INSERT_BLOCK]: false,
-			}
+			const experimentsConfig = {}
 
 			// Reset experiments
 			experiments = experimentsConfig
@@ -508,17 +502,12 @@ describe("SYSTEM_PROMPT", () => {
 			// Check that experimental tool sections are not included
 			const toolSections = prompt.split("\n## ").slice(1)
 			const toolNames = toolSections.map((section) => section.split("\n")[0].trim())
-			expect(toolNames).not.toContain("search_and_replace")
-			expect(toolNames).not.toContain("insert_content")
 			expect(prompt).toMatchSnapshot()
 		})
 
 		it("should enable experimental tools when explicitly enabled", async () => {
 			// Set experiments for testing experimental features
-			const experimentsEnabled = {
-				[EXPERIMENT_IDS.SEARCH_AND_REPLACE]: true,
-				[EXPERIMENT_IDS.INSERT_BLOCK]: true,
-			}
+			const experimentsEnabled = {}
 
 			// Reset default experiments
 			experiments = undefined
@@ -543,18 +532,12 @@ describe("SYSTEM_PROMPT", () => {
 			const toolSections = prompt.split("## ").slice(1) // Split by section headers and remove first non-tool part
 			const toolNames = toolSections.map((section) => section.split("\n")[0].trim())
 
-			// Verify experimental tools are included in the prompt when enabled
-			expect(toolNames).toContain("search_and_replace")
-			expect(toolNames).toContain("insert_content")
 			expect(prompt).toMatchSnapshot()
 		})
 
 		it("should selectively enable experimental tools", async () => {
 			// Set experiments for testing selective enabling
-			const experimentsSelective = {
-				[EXPERIMENT_IDS.SEARCH_AND_REPLACE]: true,
-				[EXPERIMENT_IDS.INSERT_BLOCK]: false,
-			}
+			const experimentsSelective = {}
 
 			// Reset default experiments
 			experiments = undefined
@@ -579,17 +562,11 @@ describe("SYSTEM_PROMPT", () => {
 			const toolSections = prompt.split("## ").slice(1) // Split by section headers and remove first non-tool part
 			const toolNames = toolSections.map((section) => section.split("\n")[0].trim())
 
-			// Verify only enabled experimental tools are included
-			expect(toolNames).toContain("search_and_replace")
-			expect(toolNames).not.toContain("insert_content")
 			expect(prompt).toMatchSnapshot()
 		})
 
 		it("should list all available editing tools in base instruction", async () => {
-			const experiments = {
-				[EXPERIMENT_IDS.SEARCH_AND_REPLACE]: true,
-				[EXPERIMENT_IDS.INSERT_BLOCK]: true,
-			}
+			const experiments = {}
 
 			const prompt = await SYSTEM_PROMPT(
 				mockContext,
@@ -610,14 +587,9 @@ describe("SYSTEM_PROMPT", () => {
 			// Verify base instruction lists all available tools
 			expect(prompt).toContain("apply_diff (for replacing lines in existing files)")
 			expect(prompt).toContain("write_to_file (for creating new files or complete file rewrites)")
-			expect(prompt).toContain("insert_content (for adding lines to existing files)")
-			expect(prompt).toContain("search_and_replace (for finding and replacing individual pieces of text)")
 		})
 		it("should provide detailed instructions for each enabled tool", async () => {
-			const experiments = {
-				[EXPERIMENT_IDS.SEARCH_AND_REPLACE]: true,
-				[EXPERIMENT_IDS.INSERT_BLOCK]: true,
-			}
+			const experiments = {}
 
 			const prompt = await SYSTEM_PROMPT(
 				mockContext,
@@ -639,8 +611,6 @@ describe("SYSTEM_PROMPT", () => {
 			expect(prompt).toContain(
 				"You should always prefer using other editing tools over write_to_file when making changes to existing files since write_to_file is much slower and cannot handle large files.",
 			)
-			expect(prompt).toContain("The insert_content tool adds lines of text to files")
-			expect(prompt).toContain("The search_and_replace tool finds and replaces text or regex in files")
 		})
 	})
 

src/core/prompts/__tests__/__snapshots__/system.test.ts.snap

@@ -13,31 +13,12 @@ function getEditingInstructions(diffStrategy?: DiffStrategy, experiments?: Recor
 	} else {
 		availableTools.push("write_to_file (for creating new files or complete file rewrites)")
 	}
-	if (experiments?.["insert_content"]) {
-		availableTools.push("insert_content (for adding lines to existing files)")
-	}
-	if (experiments?.["search_and_replace"]) {
-		availableTools.push("search_and_replace (for finding and replacing individual pieces of text)")
-	}
 
 	// Base editing instruction mentioning all available tools
 	if (availableTools.length > 1) {
 		instructions.push(`- For editing files, you have access to these tools: ${availableTools.join(", ")}.`)
 	}
 
-	// Additional details for experimental features
-	if (experiments?.["insert_content"]) {
-		instructions.push(
-			"- The insert_content tool adds lines of text to files, such as adding a new function to a JavaScript file or inserting a new route in a Python file. This tool will insert it at the specified line location. It can support multiple operations at once.",
-		)
-	}
-
-	if (experiments?.["search_and_replace"]) {
-		instructions.push(
-			"- The search_and_replace tool finds and replaces text or regex in files. This tool allows you to search for a specific regex pattern or text and replace it with another value. Be cautious when using this tool to ensure you are replacing the correct text. It can support multiple operations at once.",
-		)
-	}
-
 	if (availableTools.length > 1) {
 		instructions.push(
 			"- You should always prefer using other editing tools over write_to_file when making changes to existing files since write_to_file is much slower and cannot handle large files.",

src/core/prompts/__tests__/system.test.ts

@@ -4,8 +4,6 @@ import { getFetchInstructionsDescription } from "./fetch-instructions"
 import { getWriteToFileDescription } from "./write-to-file"
 import { getSearchFilesDescription } from "./search-files"
 import { getListFilesDescription } from "./list-files"
-import { getInsertContentDescription } from "./insert-content"
-import { getSearchAndReplaceDescription } from "./search-and-replace"
 import { getListCodeDefinitionNamesDescription } from "./list-code-definition-names"
 import { getBrowserActionDescription } from "./browser-action"
 import { getAskFollowupQuestionDescription } from "./ask-followup-question"
@@ -36,8 +34,6 @@ const toolDescriptionMap: Record<string, (args: ToolArgs) => string | undefined>
 	access_mcp_resource: (args) => getAccessMcpResourceDescription(args),
 	switch_mode: () => getSwitchModeDescription(),
 	new_task: (args) => getNewTaskDescription(args),
-	insert_content: (args) => getInsertContentDescription(args),
-	search_and_replace: (args) => getSearchAndReplaceDescription(args),
 	apply_diff: (args) =>
 		args.diffStrategy ? args.diffStrategy.getToolDescription({ cwd: args.cwd, toolOptions: args.toolOptions }) : "",
 }
@@ -110,6 +106,4 @@ export {
 	getUseMcpToolDescription,
 	getAccessMcpResourceDescription,
 	getSwitchModeDescription,
-	getInsertContentDescription,
-	getSearchAndReplaceDescription,
 }