Skip to content

feat: add OAuth PKCE login flow for automatic API key authentication#28

Merged
HeshamHM28 merged 4 commits intomainfrom
add-oauth-pkce-login
Mar 19, 2026
Merged

feat: add OAuth PKCE login flow for automatic API key authentication#28
HeshamHM28 merged 4 commits intomainfrom
add-oauth-pkce-login

Conversation

@HeshamHM28
Copy link
Collaborator

Screen.Recording.2026-03-17.at.8.12.56.AM.mov

@HeshamHM28 HeshamHM28 requested review from KRRT7 and Saga4 March 17, 2026 06:15
@aseembits93
Copy link
Contributor

when is the api key appended to zshrc in this flow? @HeshamHM28

@HeshamHM28
Copy link
Collaborator Author

HeshamHM28 commented Mar 17, 2026

when is the api key appended to zshrc in this flow? @HeshamHM28

After redirecting to the success message, the script performs polling, retrieves the API key, and saves it in the .zshrc file. @aseembits93

@aseembits93 aseembits93 force-pushed the add-oauth-pkce-login branch from 5426697 to fb2d8c3 Compare March 17, 2026 23:53
HeshamHM28 and others added 2 commits March 19, 2026 01:27
Test 27 was failing because the new has_api_key check exits before
reaching the permissions.allow message. Providing a mock API key
lets the test exercise the auto-allow logic as intended.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@HeshamHM28 HeshamHM28 merged commit cc182b1 into main Mar 19, 2026
2 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.

2 participants