fix(mcp): clear oauthConnected on deselect; null guard on allowedTools prop

This commit is contained in:
Ishaan Jaffer 2026-03-11 20:46:04 -07:00
parent 9e767ce77f
commit d207c45766
2 changed files with 9 additions and 1 deletions

View file

@ -182,6 +182,14 @@ const MCPAppsPanel: React.FC<Props> = ({ accessToken, selectedServers, onChange
const handleToggle = async (serverName: string, checked: boolean, serverId?: string) => {
if (!checked) {
onChange(selectedServers.filter((s) => s !== serverName));
// Also clear from oauthConnected so the auto-enable effect doesn't re-add it.
if (serverId) {
setOauthConnected((prev) => {
const next = new Set(prev);
next.delete(serverId);
return next;
});
}
return;
}
setTogglingOn((prev) => new Set(prev).add(serverName));

View file

@ -467,7 +467,7 @@ const MCPToolConfiguration: React.FC<MCPToolConfigurationProps> = ({
<McpCrudPermissionPanel
tools={tools}
searchFilter={toolSearchTerm}
value={allowedTools}
value={allowedTools ?? undefined}
onChange={(allowed) => onAllowedToolsChange(allowed)}
/>
)}