Skip to content

fix(dashboard): restore external OAuth management - #4569

Merged
qstearns merged 2 commits into
mainfrom
cursor/restore-external-oauth-ui-0b37
Jul 30, 2026
Merged

fix(dashboard): restore external OAuth management#4569
qstearns merged 2 commits into
mainfrom
cursor/restore-external-oauth-ui-0b37

Conversation

@disintegrator

@disintegrator disintegrator commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • restore the external OAuth setup action on the rewritten toolset authentication surface
  • open the existing OAuth wizard directly on the external OAuth form
  • preserve the existing eligibility rules for enabled public MCP servers
  • cover the eligibility matrix, setup-action plumbing, and direct wizard entry with tests
  • add a patch changeset for the dashboard package

Testing

  • pnpm -F dashboard test toolsetAuthSurface OAuthWizard AuthenticationSection (32 tests passed)
  • pnpm -F dashboard type-check
  • pnpm -F dashboard lint
  • pnpm exec changeset status --since=main
  • manually verified the eligible toolset flow: action appears, opens the external OAuth form directly, validates placeholder metadata, and enables submission without submitting

Demo

external_oauth_management_stable.webm

Linear: AIS-364

Linear Issue: AIS-364

Open in Web Open in Cursor 

Summary by cubic

Restores external OAuth management on the MCP toolset Authentication screen. Adds a “Configure External OAuth” action that opens the OAuth wizard directly on the external form. Aligns with Linear AIS-364.

  • Bug Fixes

    • Show “Configure External OAuth” only for eligible servers: mcpEnabled + mcpIsPublic and either an OAuth2 auth code flow or an attached external MCP source requiring OAuth.
    • Wizard accepts initialPath="external" and jumps to the external form; pre-fills when discovered metadata is v2.1.
    • Adds an additionalSetupAction path so the button appears before a session issuer exists.
    • Keeps legacy eligibility rules to avoid exposing unusable config on the serve path.
    • Adds tests for the new entry point and gating (OAuthWizard, toolsetAuthSurface, AuthenticationSection).
  • Dependencies

    • Adds a dashboard changeset to release this as a patch.

Written for commit ef81a9d. Summary will update on new commits.

Review in cubic

Co-authored-by: Georges Haidar <disintegrator@users.noreply.github.com>
@linear-code

linear-code Bot commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

AIS-364

@changeset-bot

changeset-bot Bot commented Jul 26, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: ef81a9d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
dashboard Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@simplesagar
simplesagar marked this pull request as ready for review July 26, 2026 19:39
@simplesagar
simplesagar requested review from a team as code owners July 26, 2026 19:39

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 10 files

Re-trigger cubic

Co-authored-by: Georges Haidar <disintegrator@users.noreply.github.com>
@qstearns
qstearns added this pull request to the merge queue Jul 30, 2026
Merged via the queue into main with commit 1a74d9d Jul 30, 2026
41 checks passed
@qstearns
qstearns deleted the cursor/restore-external-oauth-ui-0b37 branch July 30, 2026 15:33
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants