chore: remove POWER_STEERING experimental feature (#10926)

- Remove powerSteering from experimentIds array and schema in packages/types
- Remove POWER_STEERING from EXPERIMENT_IDS and experimentConfigsMap
- Remove power steering conditional block from getEnvironmentDetails
- Remove POWER_STEERING entry from all 18 locale settings.json files
- Update related test files to remove power steering references

Author: hannesrudolph

packages/types/src/experiment.ts

@@ -7,7 +7,6 @@ import type { Keys, Equals, AssertEqual } from "./type-fu.js"
  */
 
 export const experimentIds = [
-	"powerSteering",
 	"multiFileApplyDiff",
 	"preventFocusDisruption",
 	"imageGeneration",
@@ -25,7 +24,6 @@ export type ExperimentId = z.infer<typeof experimentIdsSchema>
  */
 
 export const experimentsSchema = z.object({
-	powerSteering: z.boolean().optional(),
 	multiFileApplyDiff: z.boolean().optional(),
 	preventFocusDisruption: z.boolean().optional(),
 	imageGeneration: z.boolean().optional(),

src/core/environment/__tests__/getEnvironmentDetails.spec.ts

@@ -5,7 +5,6 @@ import delay from "delay"
 import type { Mock } from "vitest"
 
 import { getEnvironmentDetails } from "../getEnvironmentDetails"
-import { EXPERIMENT_IDS, experiments } from "../../../shared/experiments"
 import { getFullModeDetails } from "../../../shared/modes"
 import { isToolAllowedForMode } from "../../tools/validateToolUse"
 import { getApiMetrics } from "../../../shared/getApiMetrics"
@@ -43,7 +42,6 @@ vi.mock("execa", () => ({
 	execa: vi.fn(),
 }))
 
-vi.mock("../../../shared/experiments")
 vi.mock("../../../shared/modes")
 vi.mock("../../../shared/getApiMetrics")
 vi.mock("../../../services/glob/list-files")
@@ -321,16 +319,6 @@ describe("getEnvironmentDetails", () => {
 		expect(mockInactiveTerminal.getCurrentWorkingDirectory).toHaveBeenCalled()
 	})
 
-	it("should include experiment-specific details when Power Steering is enabled", async () => {
-		mockState.experiments = { [EXPERIMENT_IDS.POWER_STEERING]: true }
-		;(experiments.isEnabled as Mock).mockReturnValue(true)
-
-		const result = await getEnvironmentDetails(mockCline as Task)
-
-		expect(result).toContain("<role>You are a code assistant</role>")
-		expect(result).toContain("<custom_instructions>Custom instructions</custom_instructions>")
-	})
-
 	it("should handle missing provider or state", async () => {
 		// Mock provider to return null.
 		mockCline.providerRef!.deref = vi.fn().mockReturnValue(null)

src/core/environment/getEnvironmentDetails.ts

@@ -8,7 +8,6 @@ import delay from "delay"
 import type { ExperimentId } from "@roo-code/types"
 import { DEFAULT_TERMINAL_OUTPUT_CHARACTER_LIMIT } from "@roo-code/types"
 
-import { EXPERIMENT_IDS, experiments as Experiments } from "../../shared/experiments"
 import { formatLanguage } from "../../shared/language"
 import { defaultModeSlug, getFullModeDetails } from "../../shared/modes"
 import { getApiMetrics } from "../../shared/getApiMetrics"
@@ -243,14 +242,6 @@ export async function getEnvironmentDetails(cline: Task, includeFileDetails: boo
 	details += `<model>${modelId}</model>\n`
 	details += `<tool_format>${toolFormat}</tool_format>\n`
 
-	if (Experiments.isEnabled(experiments ?? {}, EXPERIMENT_IDS.POWER_STEERING)) {
-		details += `<role>${modeDetails.roleDefinition}</role>\n`
-
-		if (modeDetails.customInstructions) {
-			details += `<custom_instructions>${modeDetails.customInstructions}</custom_instructions>\n`
-		}
-	}
-
 	// Add browser session status - Only show when active to prevent cluttering context
 	const isBrowserActive = cline.browserSession.isSessionActive()
 

src/shared/__tests__/experiments.spec.ts

@@ -5,15 +5,6 @@ import type { ExperimentId } from "@roo-code/types"
 import { EXPERIMENT_IDS, experimentConfigsMap, experiments as Experiments } from "../experiments"
 
 describe("experiments", () => {
-	describe("POWER_STEERING", () => {
-		it("is configured correctly", () => {
-			expect(EXPERIMENT_IDS.POWER_STEERING).toBe("powerSteering")
-			expect(experimentConfigsMap.POWER_STEERING).toMatchObject({
-				enabled: false,
-			})
-		})
-	})
-
 	describe("MULTI_FILE_APPLY_DIFF", () => {
 		it("is configured correctly", () => {
 			expect(EXPERIMENT_IDS.MULTI_FILE_APPLY_DIFF).toBe("multiFileApplyDiff")
@@ -24,43 +15,40 @@ describe("experiments", () => {
 	})
 
 	describe("isEnabled", () => {
-		it("returns false when POWER_STEERING experiment is not enabled", () => {
+		it("returns false when MULTI_FILE_APPLY_DIFF experiment is not enabled", () => {
 			const experiments: Record<ExperimentId, boolean> = {
-				powerSteering: false,
 				multiFileApplyDiff: false,
 				preventFocusDisruption: false,
 				imageGeneration: false,
 				runSlashCommand: false,
 				multipleNativeToolCalls: false,
 				customTools: false,
 			}
-			expect(Experiments.isEnabled(experiments, EXPERIMENT_IDS.POWER_STEERING)).toBe(false)
+			expect(Experiments.isEnabled(experiments, EXPERIMENT_IDS.MULTI_FILE_APPLY_DIFF)).toBe(false)
 		})
 
-		it("returns true when experiment POWER_STEERING is enabled", () => {
+		it("returns true when experiment MULTI_FILE_APPLY_DIFF is enabled", () => {
 			const experiments: Record<ExperimentId, boolean> = {
-				powerSteering: true,
-				multiFileApplyDiff: false,
+				multiFileApplyDiff: true,
 				preventFocusDisruption: false,
 				imageGeneration: false,
 				runSlashCommand: false,
 				multipleNativeToolCalls: false,
 				customTools: false,
 			}
-			expect(Experiments.isEnabled(experiments, EXPERIMENT_IDS.POWER_STEERING)).toBe(true)
+			expect(Experiments.isEnabled(experiments, EXPERIMENT_IDS.MULTI_FILE_APPLY_DIFF)).toBe(true)
 		})
 
 		it("returns false when experiment is not present", () => {
 			const experiments: Record<ExperimentId, boolean> = {
-				powerSteering: false,
 				multiFileApplyDiff: false,
 				preventFocusDisruption: false,
 				imageGeneration: false,
 				runSlashCommand: false,
 				multipleNativeToolCalls: false,
 				customTools: false,
 			}
-			expect(Experiments.isEnabled(experiments, EXPERIMENT_IDS.POWER_STEERING)).toBe(false)
+			expect(Experiments.isEnabled(experiments, EXPERIMENT_IDS.MULTI_FILE_APPLY_DIFF)).toBe(false)
 		})
 	})
 })

src/shared/experiments.ts

@@ -2,7 +2,6 @@ import type { AssertEqual, Equals, Keys, Values, ExperimentId, Experiments } fro
 
 export const EXPERIMENT_IDS = {
 	MULTI_FILE_APPLY_DIFF: "multiFileApplyDiff",
-	POWER_STEERING: "powerSteering",
 	PREVENT_FOCUS_DISRUPTION: "preventFocusDisruption",
 	IMAGE_GENERATION: "imageGeneration",
 	RUN_SLASH_COMMAND: "runSlashCommand",
@@ -20,7 +19,6 @@ interface ExperimentConfig {
 
 export const experimentConfigsMap: Record<ExperimentKey, ExperimentConfig> = {
 	MULTI_FILE_APPLY_DIFF: { enabled: false },
-	POWER_STEERING: { enabled: false },
 	PREVENT_FOCUS_DISRUPTION: { enabled: false },
 	IMAGE_GENERATION: { enabled: false },
 	RUN_SLASH_COMMAND: { enabled: false },

webview-ui/src/context/__tests__/ExtensionStateContext.spec.tsx

@@ -233,16 +233,10 @@ describe("mergeExtensionState", () => {
 			...baseState,
 			apiConfiguration: { modelMaxThinkingTokens: 456, modelTemperature: 0.3 },
 			experiments: {
-				powerSteering: true,
-				marketplace: false,
-				disableCompletionCommand: false,
-				concurrentFileReads: true,
 				multiFileApplyDiff: true,
 				preventFocusDisruption: false,
-				newTaskRequireTodos: false,
 				imageGeneration: false,
 				runSlashCommand: false,
-				nativeToolCalling: false,
 				multipleNativeToolCalls: false,
 				customTools: false,
 			} as Record<ExperimentId, boolean>,
@@ -257,16 +251,10 @@ describe("mergeExtensionState", () => {
 		})
 
 		expect(result.experiments).toEqual({
-			powerSteering: true,
-			marketplace: false,
-			disableCompletionCommand: false,
-			concurrentFileReads: true,
 			multiFileApplyDiff: true,
 			preventFocusDisruption: false,
-			newTaskRequireTodos: false,
 			imageGeneration: false,
 			runSlashCommand: false,
-			nativeToolCalling: false,
 			multipleNativeToolCalls: false,
 			customTools: false,
 		})

webview-ui/src/i18n/locales/ca/settings.json

@@ -808,10 +808,6 @@
 			"name": "Use experimental insert content tool",
 			"description": "Enable the experimental insert content tool, allowing Roo to insert content at specific line numbers without needing to create a diff."
 		},
-		"POWER_STEERING": {
-			"name": "Use experimental \"power steering\" mode",
-			"description": "When enabled, Roo will remind the model about the details of its current mode definition more frequently. This will lead to stronger adherence to role definitions and custom instructions, but will use more tokens per message."
-		},
 		"CONCURRENT_FILE_READS": {
 			"name": "Enable concurrent file reads",
 			"description": "When enabled, Roo can read multiple files in a single request. When disabled, Roo must read files one at a time. Disabling this can help when working with less capable models or when you want more control over file access."