Skip to content

API keys scopes#2970

Merged
atharvadeosthale merged 1 commit intomainfrom
atharva/keys-scope
Apr 10, 2026
Merged

API keys scopes#2970
atharvadeosthale merged 1 commit intomainfrom
atharva/keys-scope

Conversation

@atharvadeosthale
Copy link
Copy Markdown
Member

No description provided.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 10, 2026

Greptile Summary

This PR adds ScopeDefinition entries for keys.read and keys.write to the scopes array in constants.ts, making API key management scopes visible and selectable in the API keys scopes UI. The additions follow the identical pattern used in the prior webhook scopes PR and are consistent with the existing entries in defaultScopes.

Confidence Score: 5/5

Safe to merge — additive change with no logic or data-flow risk.

The change is purely additive: two ScopeDefinition objects appended to the scopes array. They are already present in defaultScopes and the UI in scopes.svelte already handles the 'Other' category. No existing logic is modified, no deprecation edge cases are introduced, and the descriptions and icon/category values are consistent with the adjacent webhook scope entries.

No files require special attention.

Important Files Changed

Filename Overview
src/lib/constants.ts Adds ScopeDefinition entries for keys.read and keys.write to the scopes array, mirroring the pattern used for the recently added webhook scopes. Both keys are already present in defaultScopes, so UI and scope-catalog are now consistent.

Reviews (1): Last reviewed commit: "API keys scopes" | Re-trigger Greptile

@atharvadeosthale atharvadeosthale merged commit d1729ef into main Apr 10, 2026
3 of 4 checks passed
@atharvadeosthale atharvadeosthale deleted the atharva/keys-scope branch April 10, 2026 14:16
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