Skip to content

Commit d2e47ee

Browse files
committed
fix: refactor codebase index handling in ExperimentalSettings and SettingsView components
1 parent b62f29b commit d2e47ee

File tree

2 files changed

+3
-11
lines changed

2 files changed

+3
-11
lines changed

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

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ import { EXPERIMENT_IDS, experimentConfigsMap } from "@roo/experiments"
1010
import { useAppTranslation } from "@src/i18n/TranslationContext"
1111
import { cn } from "@src/lib/utils"
1212
import { buildDocLink } from "@src/utils/docLinks"
13-
import { vscode } from "@src/utils/vscode"
1413

1514
import { SetExperimentEnabled } from "./types"
1615
import { SectionHeader } from "./SectionHeader"
@@ -88,20 +87,12 @@ export const ExperimentalSettings = ({
8887
checked={codebaseIndexEnabled || false}
8988
onChange={(e: any) => {
9089
const newEnabledState = e.target.checked
91-
92-
// Update the local cached state for immediate UI feedback
9390
if (setCachedStateField && codebaseIndexConfig) {
9491
setCachedStateField("codebaseIndexConfig", {
9592
...codebaseIndexConfig,
9693
codebaseIndexEnabled: newEnabledState,
9794
})
9895
}
99-
100-
// Send the message to update the backend state
101-
vscode.postMessage({
102-
type: "codebaseIndexEnabled",
103-
bool: newEnabledState,
104-
})
10596
}}>
10697
<span className="font-medium">{t("settings:codeIndex.enableLabel")}</span>
10798
</VSCodeCheckbox>

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

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -320,8 +320,9 @@ const SettingsView = forwardRef<SettingsViewRef, SettingsViewProps>(({ onDone, t
320320
vscode.postMessage({ type: "updateSupportPrompt", values: customSupportPrompts || {} })
321321
vscode.postMessage({ type: "upsertApiConfiguration", text: currentApiConfigName, apiConfiguration })
322322
vscode.postMessage({ type: "telemetrySetting", text: telemetrySetting })
323-
// Code index config is now handled separately in CodeIndexSettings
324-
// vscode.postMessage({ type: "codebaseIndexConfig", values: codebaseIndexConfig })
323+
if (codebaseIndexConfig) {
324+
vscode.postMessage({ type: "codebaseIndexEnabled", bool: codebaseIndexConfig.codebaseIndexEnabled })
325+
}
325326
vscode.postMessage({ type: "profileThresholds", values: profileThresholds })
326327
setChangeDetected(false)
327328
}

0 commit comments

Comments
 (0)