Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat [cli]: Add keyring management for OpenAI API keys #361

Merged
merged 12 commits into from
Nov 28, 2024
Merged

Conversation

ahau-square
Copy link
Collaborator

Remove API key handling from client for OpenAI provider. Instead, core goose will check the keyring, then the env variable for the API key and finally ask the user to input it if in neither of those locations, after which it will be set in the environment.

Copy link
Collaborator

@Kvadratni Kvadratni left a comment

Choose a reason for hiding this comment

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

I generally like this, but can we rename it to something like get_keyring_secret. I am sure this will extend beyond API keys.

@alicehau
Copy link

I generally like this, but can we rename it to something like get_keyring_secret. I am sure this will extend beyond API keys.

updated!

@baxen baxen merged commit 3c6583a into v1.0 Nov 28, 2024
2 checks passed
jsibbison-square added a commit that referenced this pull request Nov 28, 2024
* v1.0:
  feat: Hardened runtime and notarization (#367)
  feat: use mac main menu (#366)
  [app] Turn send button into stop button when goose is responding so user can cancel if needed
  [app] Fix position for MoreMenu and make top region a little smaller
  feat [cli]: Add keyring management for OpenAI API keys (#361)
michaelneale added a commit that referenced this pull request Nov 28, 2024
* v1.0:
  fix: During configure remove 'recommended' from provider descriptions (#372)
  [cli] On interrupt repair messages (#370)
  feat: add low priority user output for bash
  feat: collapse long tool args (#368)
  feat: Hardened runtime and notarization (#367)
  feat: use mac main menu (#366)
  [app] Turn send button into stop button when goose is responding so user can cancel if needed
  [app] Fix position for MoreMenu and make top region a little smaller
  feat [cli]: Add keyring management for OpenAI API keys (#361)
  Interrupt running shell tool commands (#365)
  [GUI multiprocess] tabless multiple goosed processes per window (#355)
  feat: simplify initial session splash (#362)
  fix: hide response form behind flag
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.

4 participants