Skip to content

fix: preserve deprecated API key scopes#3088

Merged
TorstenDittmann merged 1 commit into
mainfrom
fix-api-key-deprecated-scopes
Jun 18, 2026
Merged

fix: preserve deprecated API key scopes#3088
TorstenDittmann merged 1 commit into
mainfrom
fix-api-key-deprecated-scopes

Conversation

@TorstenDittmann

@TorstenDittmann TorstenDittmann commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Preserve exact API key scope IDs when initializing the scopes selector
  • Keep deprecated scopes like documents.read and documents.write checked after saving
  • Count category state from exact selected scope IDs instead of normalized aliases

Tests

  • bun run check

@TorstenDittmann TorstenDittmann merged commit 0250d7d into main Jun 18, 2026
3 of 4 checks passed
@TorstenDittmann TorstenDittmann deleted the fix-api-key-deprecated-scopes branch June 18, 2026 12:30
@greptile-apps

greptile-apps Bot commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Updates the API key scope selector to preserve deprecated scope IDs exactly.
Initializes checked scopes from saved scope IDs without alias normalization.
Computes category selection state from the exact selected scope IDs.

Confidence Score: 5/5

Safe to merge based on the scoped API key selector change.

The change is limited to preserving exact selected scope identifiers and computing category state from those exact values, matching the intended behavior for deprecated scopes.

T-Rex T-Rex Logs

What T-Rex did

  • Attempted a Playwright render of the documents.read surface, but the execution blocker occurred when the real component mounted in the browser, throwing PAGEERROR Layout is not defined before rendering documents.read.

View all artifacts

T-Rex Ran code and verified through T-Rex

Reviews (1): Last reviewed commit: "fix: preserve deprecated API key scopes" | Re-trigger Greptile

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.

2 participants