use experimental setting instead

Author: canrobins13

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

@@ -297,7 +297,7 @@ export type CommandExecutionStatus = z.infer<typeof commandExecutionStatusSchema
  * ExperimentId
  */
 
-export const experimentIds = ["powerSteering"] as const
+export const experimentIds = ["autoCondenseContext", "powerSteering"] as const
 
 export const experimentIdsSchema = z.enum(experimentIds)
 
@@ -308,6 +308,7 @@ export type ExperimentId = z.infer<typeof experimentIdsSchema>
  */
 
 const experimentsSchema = z.object({
+	autoCondenseContext: z.boolean(),
 	powerSteering: z.boolean(),
 })
 

src/core/task/Task.ts

@@ -1421,7 +1421,6 @@ export class Task extends EventEmitter<ClineEvents> {
 			enableMcpServerCreation,
 			browserToolEnabled,
 			language,
-			enableAutoContextCondensing,
 		} = (await this.providerRef.deref()?.getState()) ?? {}
 
 		const { customModes } = (await this.providerRef.deref()?.getState()) ?? {}
@@ -1484,7 +1483,7 @@ export class Task extends EventEmitter<ClineEvents> {
 			const contextWindow = modelInfo.contextWindow
 
 			let condensedMessages
-			if (enableAutoContextCondensing) {
+			if (experiments?.autoCondenseContext) {
 				condensedMessages = await summarizeConversationIfNeeded(
 					this.apiConversationHistory,
 					totalTokens,

src/exports/roo-code.d.ts

@@ -100,6 +100,7 @@ type GlobalSettings = {
 	fuzzyMatchThreshold?: number | undefined
 	experiments?:
 		| {
+				autoCondenseContext: boolean
 				powerSteering: boolean
 		  }
 		| undefined
@@ -786,6 +787,7 @@ type IpcMessage =
 								terminalCompressProgressBar?: boolean | undefined
 								experiments?:
 									| {
+											autoCondenseContext: boolean
 											powerSteering: boolean
 									  }
 									| undefined
@@ -1248,6 +1250,7 @@ type TaskCommand =
 					terminalCompressProgressBar?: boolean | undefined
 					experiments?:
 						| {
+								autoCondenseContext: boolean
 								powerSteering: boolean
 						  }
 						| undefined

src/exports/types.ts

@@ -100,6 +100,7 @@ type GlobalSettings = {
 	fuzzyMatchThreshold?: number | undefined
 	experiments?:
 		| {
+				autoCondenseContext: boolean
 				powerSteering: boolean
 		  }
 		| undefined
@@ -798,6 +799,7 @@ type IpcMessage =
 								terminalCompressProgressBar?: boolean | undefined
 								experiments?:
 									| {
+											autoCondenseContext: boolean
 											powerSteering: boolean
 									  }
 									| undefined
@@ -1262,6 +1264,7 @@ type TaskCommand =
 					terminalCompressProgressBar?: boolean | undefined
 					experiments?:
 						| {
+								autoCondenseContext: boolean
 								powerSteering: boolean
 						  }
 						| undefined

src/schemas/index.ts

@@ -312,7 +312,7 @@ export type CommandExecutionStatus = z.infer<typeof commandExecutionStatusSchema
  * ExperimentId
  */
 
-export const experimentIds = ["powerSteering"] as const
+export const experimentIds = ["autoCondenseContext", "powerSteering"] as const
 
 export const experimentIdsSchema = z.enum(experimentIds)
 
@@ -323,6 +323,7 @@ export type ExperimentId = z.infer<typeof experimentIdsSchema>
  */
 
 const experimentsSchema = z.object({
+	autoCondenseContext: z.boolean(),
 	powerSteering: z.boolean(),
 })
 

src/shared/__tests__/experiments.test.ts

@@ -10,26 +10,54 @@ describe("experiments", () => {
 		})
 	})
 
+	describe("AUTO_CONDENSE_CONTEXT", () => {
+		it("is configured correctly", () => {
+			expect(EXPERIMENT_IDS.AUTO_CONDENSE_CONTEXT).toBe("autoCondenseContext")
+			expect(experimentConfigsMap.AUTO_CONDENSE_CONTEXT).toMatchObject({
+				enabled: false,
+			})
+		})
+	})
+
 	describe("isEnabled", () => {
-		it("returns false when experiment is not enabled", () => {
+		it("returns false when POWER_STEERING experiment is not enabled", () => {
 			const experiments: Record<ExperimentId, boolean> = {
 				powerSteering: false,
+				autoCondenseContext: false,
 			}
 			expect(Experiments.isEnabled(experiments, EXPERIMENT_IDS.POWER_STEERING)).toBe(false)
 		})
 
-		it("returns true when experiment is enabled", () => {
+		it("returns true when experiment POWER_STEERING is enabled", () => {
 			const experiments: Record<ExperimentId, boolean> = {
 				powerSteering: true,
+				autoCondenseContext: false,
 			}
 			expect(Experiments.isEnabled(experiments, EXPERIMENT_IDS.POWER_STEERING)).toBe(true)
 		})
 
 		it("returns false when experiment is not present", () => {
 			const experiments: Record<ExperimentId, boolean> = {
 				powerSteering: false,
+				autoCondenseContext: false,
 			}
 			expect(Experiments.isEnabled(experiments, EXPERIMENT_IDS.POWER_STEERING)).toBe(false)
 		})
+
+		it("returns false when AUTO_CONDENSE_CONTEXT experiment is not enabled", () => {
+			const experiments: Record<ExperimentId, boolean> = {
+				powerSteering: false,
+				autoCondenseContext: false,
+			}
+			expect(Experiments.isEnabled(experiments, EXPERIMENT_IDS.AUTO_CONDENSE_CONTEXT)).toBe(false)
+		})
+
+		it("returns true when AUTO_CONDENSE_CONTEXT experiment is enabled", () => {
+			const experiments: Record<ExperimentId, boolean> = {
+				powerSteering: false,
+				autoCondenseContext: true,
+			}
+			expect(Experiments.isEnabled(experiments, EXPERIMENT_IDS.AUTO_CONDENSE_CONTEXT)).toBe(true)
+		})
 	})
 })

src/shared/experiments.ts

@@ -4,6 +4,7 @@ import { AssertEqual, Equals, Keys, Values } from "../utils/type-fu"
 export type { ExperimentId }
 
 export const EXPERIMENT_IDS = {
+	AUTO_CONDENSE_CONTEXT: "autoCondenseContext",
 	POWER_STEERING: "powerSteering",
 } as const satisfies Record<string, ExperimentId>
 
@@ -16,6 +17,7 @@ interface ExperimentConfig {
 }
 
 export const experimentConfigsMap: Record<ExperimentKey, ExperimentConfig> = {
+	AUTO_CONDENSE_CONTEXT: { enabled: false },
 	POWER_STEERING: { enabled: false },
 }
 

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

@@ -215,6 +215,7 @@ describe("mergeExtensionState", () => {
 			apiConfiguration: { modelMaxThinkingTokens: 456, modelTemperature: 0.3 },
 			experiments: {
 				powerSteering: true,
+				autoCondenseContext: true,
 			} as Record<ExperimentId, boolean>,
 		}
 
@@ -227,6 +228,7 @@ describe("mergeExtensionState", () => {
 
 		expect(result.experiments).toEqual({
 			powerSteering: true,
+			autoCondenseContext: true,
 		})
 	})
 })

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

@@ -396,6 +396,10 @@
 	},
 	"experimental": {
 		"warning": "⚠️",
+		"AUTO_CONDENSE_CONTEXT": {
+			"name": "Automatically condense the context window",
+			"description": "Uses an LLM call to summarize the past conversation when half of the task's context window is used."
+		},
 		"DIFF_STRATEGY_UNIFIED": {
 			"name": "Use experimental unified diff strategy",
 			"description": "Enable the experimental unified diff strategy. This strategy might reduce the number of retries caused by model errors but may cause unexpected behavior or incorrect edits. Only enable if you understand the risks and are willing to carefully review all changes."