Skip to content

feat(admin): show scopes for proxy clients#172

Merged
vitramir merged 1 commit intomainfrom
noa/issue-171
Apr 12, 2026
Merged

feat(admin): show scopes for proxy clients#172
vitramir merged 1 commit intomainfrom
noa/issue-171

Conversation

@casey-brooks
Copy link
Copy Markdown
Contributor

Summary

  • show the scopes editor for proxy-mode clients
  • adjust proxy mode messaging around scope enforcement
  • update proxy client detail coverage in unit and e2e tests

Testing

  • pnpm lint
  • pnpm typecheck
  • pnpm test
  • pnpm test:prepare
  • PLAYWRIGHT_BROWSERS_PATH=.playwright-browsers pnpm exec dotenv -e .env.test -o -- playwright test --workers=1

Closes #171

@casey-brooks casey-brooks requested a review from a team as a code owner April 12, 2026 16:50
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 12, 2026

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

Project Deployment Actions Updated (UTC)
mockauth Ready Ready Preview, Comment Apr 12, 2026 4:51pm

Request Review

@casey-brooks
Copy link
Copy Markdown
Contributor Author

Summary

  • show the scopes editor for proxy-mode clients and refresh proxy copy
  • update proxy client detail expectations in unit and e2e coverage

Testing

  • pnpm lint
    • Lint: no errors
  • pnpm typecheck
    • Typecheck: no errors
  • pnpm test
    • Passed: 236, Failed: 0, Skipped: 0
  • pnpm test:prepare
  • PLAYWRIGHT_BROWSERS_PATH=.playwright-browsers pnpm exec dotenv -e .env.test -o -- playwright test --workers=1
    • Passed: 40, Failed: 0, Skipped: 0

Copy link
Copy Markdown

@noa-lucent noa-lucent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes match #171.

  • Proxy-mode client detail page now renders the Allowed scopes card.
  • Proxy-mode copy updated to clarify that scope allowlisting is still enforced.
  • Unit + e2e coverage updated accordingly.

No issues found.

@vitramir vitramir added this pull request to the merge queue Apr 12, 2026
Merged via the queue into main with commit 5d1751a Apr 12, 2026
4 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.

Admin UI: proxy client Allowed scopes + mixed-case scope support

3 participants