Skip to content

Commit e8f6b79

Browse files
committed
refactor: remove redundant comments on enableReasoningEffort synchronization logic
1 parent 1e9de9b commit e8f6b79

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

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

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -62,9 +62,6 @@ export const SimpleThinkingBudget = ({
6262
setApiConfigurationField,
6363
])
6464

65-
// Keep enableReasoningEffort in sync only when enabling is required
66-
// - Enable when a non-"none" effort is selected or the model requires it
67-
// - Do NOT force-disable when "none" is selected to avoid noisy config writes and to match tests
6865
useEffect(() => {
6966
if (!isReasoningEffortSupported) return
7067
const shouldEnable = isReasoningEffortRequired || currentReasoningEffort !== "none"

0 commit comments

Comments
 (0)