Skip to content

Document Managed Auth tab ownership#462

Merged
masnwilliams merged 1 commit into
mainfrom
hypeship/docs-managed-auth-tabs
Jul 24, 2026
Merged

Document Managed Auth tab ownership#462
masnwilliams merged 1 commit into
mainfrom
hypeship/docs-managed-auth-tabs

Conversation

@masnwilliams

@masnwilliams masnwilliams commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • explain that Managed Auth starts login and re-authentication in a deterministic single tab
  • clarify that successful authentication persists the resulting tab state while failed and canceled sessions do not
  • recommend setting start_url when an automation requires a specific first page

Testing

  • rendered /auth/profiles with the Mintlify local preview

Note

Low Risk
Documentation-only change to profiles/Managed Auth behavior; no runtime or API code modified.

Overview
Adds a Note under Override opening existing tabs in a new session in auth/profiles.mdx that spells out how Managed Auth treats tabs on profiles tied to auth connections.

The note states that each login or automatic re-auth opens a single tab at the configured login URL (or the domain homepage if none is set). Successful auth persists that tab state with updated credentials; failed or canceled sessions do not alter the saved profile. It warns automations not to rely on prior tabs/windows on Managed Auth profiles and to pass start_url when the first page must be explicit.

Reviewed by Cursor Bugbot for commit 5675861. Bugbot is set up for automated code reviews on this repo. Configure here.

@vercel

vercel Bot commented Jul 24, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Error Error Jul 24, 2026 11:00pm

@mintlify

mintlify Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
Kernel 🟢 Ready View Preview Jul 24, 2026, 11:01 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@mintlify

mintlify Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
Kernel 🟡 Building Jul 24, 2026, 11:00 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Risk assessment: Very Low

Verdict: Approve

Evidence (from diff)

  • 1 file changed: auth/profiles.mdx (+6 / −0)
  • Adds a documentation <Note> describing Managed Auth tab ownership (single-tab login/re-auth, persist on success, unchanged on failure/cancel) and recommending start_url when a specific first page is required
  • No application code, config, CI, infrastructure, or API surface changes

Risk factors

Factor Assessment
Codepaths modified Docs only
Blast radius Documentation readers only
Complexity Trivial
Infrastructure impact None
Operational / security risk None (describes existing product behavior in docs)

Safe to approve as a narrowly scoped documentation clarification.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

@masnwilliams
masnwilliams merged commit 6af7fac into main Jul 24, 2026
6 of 7 checks passed
@masnwilliams
masnwilliams deleted the hypeship/docs-managed-auth-tabs branch July 24, 2026 23:18
masnwilliams added a commit that referenced this pull request Jul 25, 2026
Carries main's CLI-reference sync (#463) and managed-auth tab-ownership
docs (#462) onto the migrated content/docs paths. New reference/cli/org
page relocated via directory-rename detection; the other CLI/auth edits
auto-merged, docs.json nav merged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant