Title: improvement(mcp): make the OAuth experience visible and verbally consistent by waleedlatif1 · Pull Request #5829 · simstudioai/sim · GitHub
Open Graph Title: improvement(mcp): make the OAuth experience visible and verbally consistent by waleedlatif1 · Pull Request #5829 · simstudioai/sim
X Title: improvement(mcp): make the OAuth experience visible and verbally consistent by waleedlatif1 · Pull Request #5829 · simstudioai/sim
Description: Summary From a full UX-consistency audit of the MCP settings surfaces (design-system + copy + state-walkthrough vs sibling settings pages): One name for one action — 'Authorize' / 'Reopen authorization' everywhere; previously the same action read 'Connect with OAuth', 'Reopen authorization window', 'Authorize', or 'Reopen authorization' depending on surface. Connecting state is visible — while a popup is open the row subtitle shows muted 'Waiting for authorization...' instead of continuing to show the red 'OAuth authorization required'. Visible Authorize chip on the list row (was buried in the ⋯ menu) — so the popup-blocked toast's 'retry' has an obvious target. Removed the redundant aggregate discovery banner — every failing row already reports its specific error; the banner double-reported and out-generalized it. Refresh chip stops rendering an error sentence as a button label — short 'Failed'; the Status field explains. Sentence-case + copy sweep: Unnamed server, Not connected, Server name, Add MCP server / Edit MCP server, 'Add server' submit, Test connection, Edit form, Delete MCP server, 'Search servers...' placeholder. Token compliance: row subtitle → text-caption, inline error → text-small (per sim-settings-pages); Loading empty state instead of a blank body flash. Type of Change Improvement (UX consistency) Testing 12/12 component tests green (label + refresh-state tests updated to the new copy), tsc + biome clean. Checklist Code follows project style guidelines Self-reviewed my changes Tests added/updated and passing No new warnings introduced I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)
Open Graph Description: Summary From a full UX-consistency audit of the MCP settings surfaces (design-system + copy + state-walkthrough vs sibling settings pages): One name for one action — 'Authorize' / 'Reo...
X Description: Summary From a full UX-consistency audit of the MCP settings surfaces (design-system + copy + state-walkthrough vs sibling settings pages): One name for one action — 'Authorize' / ...
Opengraph URL: https://github.com/simstudioai/sim/pull/5829
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:3d245049-6710-0a98-fd41-9c607818933a |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | A428:167705:5FF217:89FA35:6A6375CA |
| html-safe-nonce | c7527a5d4d032d98847ea1ba44d94147ee243f923d79fe6f15bd5ffc6edacfca |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJBNDI4OjE2NzcwNTo1RkYyMTc6ODlGQTM1OjZBNjM3NUNBIiwidmlzaXRvcl9pZCI6IjIwMDk5NTg0NjYyNTQ1MDEzMjIiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 90d3a411738ee621b9f17e7baff3a082a0ee4837a03f697ffd70e449ea78993b |
| hovercard-subject-tag | pull_request:4104074385 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/simstudioai/sim/pull/5829/files |
| twitter:image | https://avatars.githubusercontent.com/u/40672544?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/40672544?s=400&v=4 |
| og:image:alt | Summary From a full UX-consistency audit of the MCP settings surfaces (design-system + copy + state-walkthrough vs sibling settings pages): One name for one action — 'Authorize' / 'Reo... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 15e78334bb345f3864e70c5a376ed0383374123629b9e3022398d9431f6cd8bf |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/simstudioai/sim git https://github.com/simstudioai/sim.git |
| octolytics-dimension-user_id | 199344406 |
| octolytics-dimension-user_login | simstudioai |
| octolytics-dimension-repository_id | 912559512 |
| octolytics-dimension-repository_nwo | simstudioai/sim |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 912559512 |
| octolytics-dimension-repository_network_root_nwo | simstudioai/sim |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 51da2d8d0a841a1393b806a6559fa9ad757e1666 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width