Skip to content

Add managed auth events proxy rewrite - #570

Merged
masnwilliams merged 2 commits into
mainfrom
hypeship/add-managed-auth-events-proxy
Sep 15, 2026
Merged

masnwilliams merged 2 commits into
mainfrom
hypeship/add-managed-auth-events-proxy

Conversation

@masnwilliams

@masnwilliams masnwilliams commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

summary

  • include the managed auth SSE events endpoint in the same-origin Next.js proxy example
  • update the endpoint count to match the current React component

testing

  • not run (documentation-only change; the local Mintlify CLI is unavailable)

Note

Low Risk
Documentation-only change to the React auth proxy example; no runtime or API behavior is modified.

Overview
Updates the Backend connectivity same-origin proxy docs for @onkernel/managed-auth-react so they match what the component actually calls.

The prose now says to proxy four endpoints (was three), and the next.config.ts rewrite example adds /auth/connections/:id/events → Kernel’s SSE events URL alongside exchange, connection GET, and submit.

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

@mintlify

mintlify Bot commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

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

Project Status Preview Updated
Kernel 🟢 Ready View Preview Sep 14, 2026, 9:30 PM

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

cursor[bot]
cursor Bot approved these changes Sep 14, 2026
@masnwilliams
masnwilliams requested a review from akxue September 14, 2026 23:27
@masnwilliams
masnwilliams merged commit 99b8a7e into main Sep 15, 2026
4 checks passed
@masnwilliams
masnwilliams deleted the hypeship/add-managed-auth-events-proxy branch September 15, 2026 01:08

@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: Risk is unchanged. Prior approval stands; not re-approving.

This synchronize is a merge of main into the feature branch. The three-dot diff against main is still a single documentation edit.

What changed

  • Single file: auth/react.mdx (+5 / −1)
  • Copy update: "three endpoints" → "four endpoints"
  • Adds a Next.js rewrite example for /auth/connections/:id/events next to the existing exchange, connection, and submit rewrites

Why Very Low

  • Docs-only MDX edit on a Mintlify site; no runtime, API, or auth logic is modified
  • Isolated example update; blast radius is limited to readers of this page
  • Small, easy-to-reason-about diff with no schema, infra, or permission changes
  • No CODEOWNERS file in this repo, so code-owner review is not required

Re-evaluation after update: risk did not increase, so approval is not revoked.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

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.

2 participants