refactor: remove redundant experimentConfigs array, use experimentConfigsMap directly

Author: samhvw8

src/core/webview/ClineProvider.ts

@@ -40,13 +40,7 @@ import { singleCompletionHandler } from "../../utils/single-completion-handler"
 import { getCommitInfo, searchCommits, getWorkingState } from "../../utils/git"
 import { ConfigManager } from "../config/ConfigManager"
 import { CustomModesManager } from "../config/CustomModesManager"
-import {
-	EXPERIMENT_IDS,
-	experimentConfigs,
-	experiments as Experiments,
-	experimentDefault,
-	ExperimentId,
-} from "../../shared/experiments"
+import { EXPERIMENT_IDS, experiments as Experiments, experimentDefault, ExperimentId } from "../../shared/experiments"
 import { CustomSupportPrompts, supportPrompt } from "../../shared/support-prompt"
 
 import { ACTION_NAMES } from "../CodeActionProvider"

src/shared/experiments.ts

@@ -41,8 +41,6 @@ export const experimentConfigsMap: Record<ExperimentKey, ExperimentConfig> = {
 	},
 }
 
-// Keep the array version for backward compatibility
-export const experimentConfigs = Object.values(experimentConfigsMap)
 export const experimentDefault = Object.fromEntries(
 	Object.entries(experimentConfigsMap).map(([_, config]) => [config.id, config.enabled]),
 ) as Record<ExperimentId, boolean>

webview-ui/src/components/settings/SettingsView.tsx

@@ -5,7 +5,7 @@ import { validateApiConfiguration, validateModelId } from "../../utils/validate"
 import { vscode } from "../../utils/vscode"
 import ApiOptions from "./ApiOptions"
 import ExperimentalFeature from "./ExperimentalFeature"
-import { experimentConfigs, EXPERIMENT_IDS, experimentConfigsMap } from "../../../../src/shared/experiments"
+import { EXPERIMENT_IDS, experimentConfigsMap } from "../../../../src/shared/experiments"
 import ApiConfigManager from "./ApiConfigManager"
 
 type SettingsViewProps = {
@@ -632,7 +632,7 @@ const SettingsView = ({ onDone }: SettingsViewProps) => {
 								</p>
 							</div>
 						)}
-						{experimentConfigs
+						{Object.values(experimentConfigsMap)
 							.filter((config) => config.id !== EXPERIMENT_IDS.DIFF_STRATEGY)
 							.map((config) => (
 								<ExperimentalFeature