Sync MCP managed auth options - #173
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 3595eeb. Configure here.
rgarcia
left a comment
There was a problem hiding this comment.
reviewed — overall looks solid. two input-contract issues worth addressing:
Findings
-
src/lib/mcp/tools/auth-connections.ts:78-100— credential updates are supported by the newupdatehandler, but all four credential parameter descriptions still say(create). change them to(create, update)so MCP clients know the update capability exists. -
src/lib/mcp/tools/auth-connections.ts:477-527— submit validation is partial: canonical input can still be combined with legacy parameters, andinteraction_idcan accompany legacy-only input, both of which the API rejects. complete these canonical-vs-legacy shape checks and add parameterized tests; leave flow-state and value validation to the API.
|
addressed both review findings in 1a53248: credential inputs now advertise update support, and MCP submit validation mirrors the API’s canonical/legacy shape contract with parameterized coverage. full suite passes (277 tests), and BugBot is green on the latest commit. |

Summary
manage_auth_connectionsVerification
CLERK_SECRET_KEY=test-secret-key bun test(266 passing)bunx tsc --noEmit --incremental falsebun run check:managed-auth-appbun run buildwith local placeholder build-time OAuth/Clerk valuesbun run format:checkstill reports the pre-existing formatting mismatch inAGENTS.md; this PR does not modify that file.Note
Medium Risk
Dependency upgrades on the managed-auth stack can change API behavior for login flows and telemetry; risk is mainly integration drift rather than broad refactors in this diff.
Overview
Bumps
@onkernel/sdkto ^0.97.0 and@onkernel/managed-auth-reactto 0.5.1 (with matchingbun.lockentries) so the MCP server tracks the current Kernel managed-auth APIs.README now describes the expanded
manage_auth_connectionssurface—update, timeline inspection, health-check and automatic re-auth settings, managed-auth browser configuration, and canonical interaction-bound submissions—and notes that programmatic create, update, and login route browser telemetry through the nestedbrowser.telemetryAPI shape while keeping defaults when MCP omits those parameters.Reviewed by Cursor Bugbot for commit 5e5976c. Bugbot is set up for automated code reviews on this repo. Configure here.