fix: update comment to clarify XML is returned immediately when experiment enabled but model unsupported

roomote · roomote · commit e13c18043f91 · 2025-11-17T21:37:57.000Z
diff --git a/webview-ui/src/components/settings/ApiOptions.tsx b/webview-ui/src/components/settings/ApiOptions.tsx
@@ -425,7 +425,8 @@ const ApiOptions = ({
 			if (selectedModelInfo?.supportsNativeTools === true) {
 				return TOOL_PROTOCOL.NATIVE
 			}
-			// If experiment is enabled but model doesn't support it, fall back to XML
+			// If experiment is enabled but model doesn't support it, return XML immediately
+			// This matches resolveToolProtocol.ts behavior (lines 53-57)
 			return TOOL_PROTOCOL.XML
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -425,7 +425,8 @@ const ApiOptions = ({`
`425`	`425`	`if (selectedModelInfo?.supportsNativeTools === true) {`
`426`	`426`	`return TOOL_PROTOCOL.NATIVE`
`427`	`427`	`}`
`428`		`- // If experiment is enabled but model doesn't support it, fall back to XML`
	`428`	`+ // If experiment is enabled but model doesn't support it, return XML immediately`
	`429`	`+ // This matches resolveToolProtocol.ts behavior (lines 53-57)`
`429`	`430`	`return TOOL_PROTOCOL.XML`
`430`	`431`	`}`
`431`	`432`