Skip to content

fix: align browser OAuth client parameters#534

Merged
tmustier merged 1 commit intomainfrom
fix/oauth-login-regressions
Apr 24, 2026
Merged

fix: align browser OAuth client parameters#534
tmustier merged 1 commit intomainfrom
fix/oauth-login-regressions

Conversation

@tmustier
Copy link
Copy Markdown
Owner

@tmustier tmustier commented Apr 24, 2026

Summary

  • Fix Anthropic browser OAuth client ID typo (44d5 -> 44d9), matching upstream Pi/Claude Code client ID.
  • Revert OpenAI Codex browser OAuth to identity-only scopes and originator=pi, matching upstream/current Codex token shape.
  • Add/adjust browser OAuth tests for both authorize URLs.

Verification

  • node --test --experimental-strip-types --import ./scripts/register-test-ts-loader.mjs tests/browser-oauth.test.ts
  • npm run check
  • npm run build

Notes

  • The Anthropic typo directly caused Claude's web UI: Invalid client id provided.
  • The OpenAI connector scopes worked for a personal account but failed for the Nexcade ChatGPT Team account; official Codex-authenticated Nexcade tokens use the identity-only scope shape.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 24, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
pi-for-excel Ignored Ignored Apr 24, 2026 1:43pm

@tmustier tmustier merged commit 97b7b0a into main Apr 24, 2026
9 checks passed
@tmustier tmustier deleted the fix/oauth-login-regressions branch April 24, 2026 13:47
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