Skip to content
This repository was archived by the owner on May 15, 2026. It is now read-only.

Commit 5d93905

Browse files
committed
chore: remove POWER_STEERING experiment remnants
Remove leftover powerSteering references that were missed during the original POWER_STEERING feature removal in PR #10926. - Remove from packages/types/src/experiment.ts (experimentIds, experimentsSchema) - Remove from src/shared/experiments.ts (EXPERIMENT_IDS, experimentConfigsMap) - Update tests to use PREVENT_FOCUS_DISRUPTION instead - Clean up stale experiment IDs from test mocks
1 parent 953c777 commit 5d93905

4 files changed

Lines changed: 9 additions & 24 deletions

File tree

packages/types/src/experiment.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ import type { Keys, Equals, AssertEqual } from "./type-fu.js"
77
*/
88

99
export const experimentIds = [
10-
"powerSteering",
1110
"preventFocusDisruption",
1211
"imageGeneration",
1312
"runSlashCommand",
@@ -24,7 +23,6 @@ export type ExperimentId = z.infer<typeof experimentIdsSchema>
2423
*/
2524

2625
export const experimentsSchema = z.object({
27-
powerSteering: z.boolean().optional(),
2826
preventFocusDisruption: z.boolean().optional(),
2927
imageGeneration: z.boolean().optional(),
3028
runSlashCommand: z.boolean().optional(),

src/shared/__tests__/experiments.spec.ts

Lines changed: 9 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -5,50 +5,47 @@ import type { ExperimentId } from "@roo-code/types"
55
import { EXPERIMENT_IDS, experimentConfigsMap, experiments as Experiments } from "../experiments"
66

77
describe("experiments", () => {
8-
describe("POWER_STEERING", () => {
8+
describe("PREVENT_FOCUS_DISRUPTION", () => {
99
it("is configured correctly", () => {
10-
expect(EXPERIMENT_IDS.POWER_STEERING).toBe("powerSteering")
11-
expect(experimentConfigsMap.POWER_STEERING).toMatchObject({
10+
expect(EXPERIMENT_IDS.PREVENT_FOCUS_DISRUPTION).toBe("preventFocusDisruption")
11+
expect(experimentConfigsMap.PREVENT_FOCUS_DISRUPTION).toMatchObject({
1212
enabled: false,
1313
})
1414
})
1515
})
1616

1717
describe("isEnabled", () => {
18-
it("returns false when POWER_STEERING experiment is not enabled", () => {
18+
it("returns false when experiment is not enabled", () => {
1919
const experiments: Record<ExperimentId, boolean> = {
20-
powerSteering: false,
2120
preventFocusDisruption: false,
2221
imageGeneration: false,
2322
runSlashCommand: false,
2423
multipleNativeToolCalls: false,
2524
customTools: false,
2625
}
27-
expect(Experiments.isEnabled(experiments, EXPERIMENT_IDS.POWER_STEERING)).toBe(false)
26+
expect(Experiments.isEnabled(experiments, EXPERIMENT_IDS.PREVENT_FOCUS_DISRUPTION)).toBe(false)
2827
})
2928

30-
it("returns true when experiment POWER_STEERING is enabled", () => {
29+
it("returns true when experiment is enabled", () => {
3130
const experiments: Record<ExperimentId, boolean> = {
32-
powerSteering: true,
33-
preventFocusDisruption: false,
31+
preventFocusDisruption: true,
3432
imageGeneration: false,
3533
runSlashCommand: false,
3634
multipleNativeToolCalls: false,
3735
customTools: false,
3836
}
39-
expect(Experiments.isEnabled(experiments, EXPERIMENT_IDS.POWER_STEERING)).toBe(true)
37+
expect(Experiments.isEnabled(experiments, EXPERIMENT_IDS.PREVENT_FOCUS_DISRUPTION)).toBe(true)
4038
})
4139

4240
it("returns false when experiment is not present", () => {
4341
const experiments: Record<ExperimentId, boolean> = {
44-
powerSteering: false,
4542
preventFocusDisruption: false,
4643
imageGeneration: false,
4744
runSlashCommand: false,
4845
multipleNativeToolCalls: false,
4946
customTools: false,
5047
}
51-
expect(Experiments.isEnabled(experiments, EXPERIMENT_IDS.POWER_STEERING)).toBe(false)
48+
expect(Experiments.isEnabled(experiments, EXPERIMENT_IDS.PREVENT_FOCUS_DISRUPTION)).toBe(false)
5249
})
5350
})
5451
})

src/shared/experiments.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import type { AssertEqual, Equals, Keys, Values, ExperimentId, Experiments } from "@roo-code/types"
22

33
export const EXPERIMENT_IDS = {
4-
POWER_STEERING: "powerSteering",
54
PREVENT_FOCUS_DISRUPTION: "preventFocusDisruption",
65
IMAGE_GENERATION: "imageGeneration",
76
RUN_SLASH_COMMAND: "runSlashCommand",
@@ -18,7 +17,6 @@ interface ExperimentConfig {
1817
}
1918

2019
export const experimentConfigsMap: Record<ExperimentKey, ExperimentConfig> = {
21-
POWER_STEERING: { enabled: false },
2220
PREVENT_FOCUS_DISRUPTION: { enabled: false },
2321
IMAGE_GENERATION: { enabled: false },
2422
RUN_SLASH_COMMAND: { enabled: false },

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

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -233,10 +233,6 @@ describe("mergeExtensionState", () => {
233233
...baseState,
234234
apiConfiguration: { modelMaxThinkingTokens: 456, modelTemperature: 0.3 },
235235
experiments: {
236-
powerSteering: true,
237-
marketplace: false,
238-
disableCompletionCommand: false,
239-
concurrentFileReads: true,
240236
preventFocusDisruption: false,
241237
imageGeneration: false,
242238
runSlashCommand: false,
@@ -254,10 +250,6 @@ describe("mergeExtensionState", () => {
254250
})
255251

256252
expect(result.experiments).toEqual({
257-
powerSteering: true,
258-
marketplace: false,
259-
disableCompletionCommand: false,
260-
concurrentFileReads: true,
261253
preventFocusDisruption: false,
262254
imageGeneration: false,
263255
runSlashCommand: false,

0 commit comments

Comments
 (0)