modelcontextprotocol/inspector
Watch
1
0
Fork
You've already forked inspector
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

fix(web): name the Scopes control in the offline_access warning

Browse source 
Copilot review round 9 on #2114. "Scopes above lists offline_access"
reads as a plural subject with a singular verb; naming the control
("The Scopes field above lists ...") fixes the agreement and says which
control to go edit.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013hzwzS8UvBsr4yZm7o7sev
Signed-off-by: cliffhall <cliff@futurescale.com>
This commit is contained in:
cliffhall 2026-08-24 23:17:30 -04:00
commit 40149a2a4b
1 changed files with 5 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

9
clients/web/src/components/groups/ServerSettingsForm/ServerSettingsForm.tsx
View file

@ -919,10 +919,11 @@ export function ServerSettingsForm({
/> />
{refreshTokenOptedOut && scopesIncludeOfflineAccess ? ( {refreshTokenOptedOut && scopesIncludeOfflineAccess ? (
<Alert color="yellow" title="offline_access is still requested"> <Alert color="yellow" title="offline_access is still requested">
Scopes above lists offline_access, and prompt=consent is added The Scopes field above lists offline_access, and
whenever that scope is requested — regardless of this setting. prompt=consent is added whenever that scope is requested —
Remove it from Scopes as well, or the consent prompt this regardless of this setting. Remove it from Scopes as well, or
checkbox is meant to avoid still goes out. the consent prompt this checkbox is meant to avoid still goes
out.
</Alert> </Alert>
) : null} ) : null}
<Stack gap="xs"> <Stack gap="xs">