diff --git a/webview-ui/src/components/settings/ExperimentalSettings.tsx b/webview-ui/src/components/settings/ExperimentalSettings.tsx index 91f1fa3074..53801232ec 100644 --- a/webview-ui/src/components/settings/ExperimentalSettings.tsx +++ b/webview-ui/src/components/settings/ExperimentalSettings.tsx @@ -7,9 +7,8 @@ import { EXPERIMENT_IDS, experimentConfigsMap } from "@roo/experiments" import { useAppTranslation } from "@src/i18n/TranslationContext" import { cn } from "@src/lib/utils" -import { Input } from "@src/components/ui" -import { SetExperimentEnabled, SetCachedStateField } from "./types" +import { SetExperimentEnabled } from "./types" import { SectionHeader } from "./SectionHeader" import { Section } from "./Section" import { ExperimentalFeature } from "./ExperimentalFeature" @@ -17,15 +16,11 @@ import { ExperimentalFeature } from "./ExperimentalFeature" type ExperimentalSettingsProps = HTMLAttributes & { experiments: Experiments setExperimentEnabled: SetExperimentEnabled - readFileDeduplicationCacheMinutes?: number - setCachedStateField?: SetCachedStateField<"readFileDeduplicationCacheMinutes"> } export const ExperimentalSettings = ({ experiments, setExperimentEnabled, - readFileDeduplicationCacheMinutes, - setCachedStateField, className, ...props }: ExperimentalSettingsProps) => { @@ -70,40 +65,6 @@ export const ExperimentalSettings = ({ /> ) })} - - {/* Show cache time setting when READ_FILE_DEDUPLICATION is enabled */} - {experiments[EXPERIMENT_IDS.READ_FILE_DEDUPLICATION] && ( -
-
- - {t("settings:experimental.READ_FILE_DEDUPLICATION.cacheTimeLabel")} - -
- { - const newValue = parseInt(e.target.value, 10) - if (!isNaN(newValue) && newValue >= 0 && setCachedStateField) { - setCachedStateField("readFileDeduplicationCacheMinutes", newValue) - } - }} - onClick={(e) => e.currentTarget.select()} - data-testid="read-file-deduplication-cache-minutes-input" - /> - - {t("settings:experimental.READ_FILE_DEDUPLICATION.minutes")} - -
-
- {t("settings:experimental.READ_FILE_DEDUPLICATION.cacheTimeDescription")} -
-
-
- )} ) diff --git a/webview-ui/src/components/settings/SettingsView.tsx b/webview-ui/src/components/settings/SettingsView.tsx index 36e6025b3f..a416afd48d 100644 --- a/webview-ui/src/components/settings/SettingsView.tsx +++ b/webview-ui/src/components/settings/SettingsView.tsx @@ -337,10 +337,6 @@ const SettingsView = forwardRef(({ onDone, t vscode.postMessage({ type: "upsertApiConfiguration", text: currentApiConfigName, apiConfiguration }) vscode.postMessage({ type: "telemetrySetting", text: telemetrySetting }) vscode.postMessage({ type: "profileThresholds", values: profileThresholds }) - vscode.postMessage({ - type: "readFileDeduplicationCacheMinutes", - value: cachedState.readFileDeduplicationCacheMinutes ?? 5, - }) setChangeDetected(false) } } @@ -708,12 +704,7 @@ const SettingsView = forwardRef(({ onDone, t {/* Experimental Section */} {activeTab === "experimental" && ( - + )} {/* Language Section */} diff --git a/webview-ui/src/context/ExtensionStateContext.tsx b/webview-ui/src/context/ExtensionStateContext.tsx index a3b4b9af10..cb51fe9c9a 100644 --- a/webview-ui/src/context/ExtensionStateContext.tsx +++ b/webview-ui/src/context/ExtensionStateContext.tsx @@ -236,7 +236,6 @@ export const ExtensionStateContextProvider: React.FC<{ children: React.ReactNode alwaysAllowUpdateTodoList: true, includeDiagnosticMessages: true, maxDiagnosticMessages: 50, - readFileDeduplicationCacheMinutes: 5, // Default to 5 minutes }) const [didHydrateState, setDidHydrateState] = useState(false) diff --git a/webview-ui/src/context/__tests__/ExtensionStateContext.spec.tsx b/webview-ui/src/context/__tests__/ExtensionStateContext.spec.tsx index 7ea1ab1f32..c339cb1b13 100644 --- a/webview-ui/src/context/__tests__/ExtensionStateContext.spec.tsx +++ b/webview-ui/src/context/__tests__/ExtensionStateContext.spec.tsx @@ -209,7 +209,6 @@ describe("mergeExtensionState", () => { sharingEnabled: false, profileThresholds: {}, hasOpenedModeSelector: false, // Add the new required property - readFileDeduplicationCacheMinutes: 5, // Add the new required property } const prevState: ExtensionState = {