Skip to content

Retire the @better-auth/oauth-provider pnpm patch once upstream ships the redirect fixes #889

Description

@zachdunn

Context

PRs #885#887 worked around two upstream Better Auth 1.7.x bugs to unblock MCP clients (opencode, Cursor):

When upstream releases include these fixes

  1. Bump @better-auth/oauth-provider (and siblings) past the fixed release.
  2. Delete patches/@better-auth__oauth-provider.patch and its patchedDependencies entry.
  3. Delete the pinned "retire the patch" test in apps/auth/src/oauth.test.ts (explicit native + cursor:// registration).
  4. Revisit the pinned "explicit web + http loopback → 400" test — if the release fixes #10913, web+loopback may start succeeding; the test failing is the intended signal, not a regression.
  5. Keep the hook coercion/defaulting: clients that omit application_type or send Cursor's erroneous web still need it regardless of upstream (MCP spec 2026-07-28 requires clients to send it; several don't).

Sentinel: the pinned tests above fail on the offending bump, pointing here.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions