Skip to content

[codex] Improve OIDC auth UI and context flow#14

Merged
winetree94 merged 2 commits into
mainfrom
codex/oidc-auth-ui-ux-improvements
Jul 2, 2026
Merged

[codex] Improve OIDC auth UI and context flow#14
winetree94 merged 2 commits into
mainfrom
codex/oidc-auth-ui-ux-improvements

Conversation

@winetree94

Copy link
Copy Markdown
Contributor

Summary

  • Add a validated OAuth authorization-context API for pre-auth login and registration screens.
  • Show the requesting client, redirect origin, and requested scopes across OIDC login/register entry points.
  • Improve auth form accessibility with labels, hints, ARIA wiring, clearer terms controls, and responsive layout polish.
  • Update route, server, and Playwright tests for the new OIDC-aware login flow.

Why

The OIDC provider login and signup screens did not clearly communicate which client application was requesting authentication before the user authenticated. This made the flow harder to trust and less aligned with the post-login consent step.

Impact

Users now see validated client and permission context earlier in the flow, password-only deployments land directly on the password form, and auth forms have stronger accessibility semantics across supported languages.

Validation

  • git diff --cached --check
  • pnpm biome check .
  • pnpm build
  • pnpm test 2>&1 | Select-Object -Last 200
  • Targeted frontend route unit tests for login/password/register and route test utilities
  • Server authorization-context route tests
  • Targeted Playwright minimal OIDC/auth tests
  • Theme/system multilingual Playwright tests

Note: Windows did not have tail, so the documented tail -200 test summary command was run with PowerShell's Select-Object -Last 200 equivalent. Full tests exited successfully; Playwright reported three Firefox account-selection cases as flaky after retry, then passed.

@winetree94 winetree94 marked this pull request as ready for review July 2, 2026 16:59
@winetree94 winetree94 merged commit 7dd276c into main Jul 2, 2026
7 checks passed
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