Skip to content

ChatGPT Intialization #810

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Mar 25, 2023
Merged

ChatGPT Intialization #810

merged 7 commits into from
Mar 25, 2023

Conversation

tmcdonnell2
Copy link
Contributor

@tmcdonnell2 tmcdonnell2 commented Mar 23, 2023 •
edited by Zabuzard
Loading

Some bare bone implementation of ChatGPT. The main class is the ChatGPTService Class which allows for interaction with the AI. Right now all it can do is accept a question via ask(String prompt). ChatGPTCommand allows for asking the AI in a simple manner through /chatgpt prompt:___ .

Release

Added config entry:

"openaiApiKey": "<check pins in #tjbot_discussion for the key>"

tmcdonnell2 added 2 commits March 22, 2023 18:11
Initial setup for ChatGPT integration. Add ChatGPT Service to communi…
241c504 
…cate with API. Include bare bones ChatGPTCommand for future testing of integration.
Update ChatGPTCommand to update message when response is ready. Updat…
0ab420c 
…e Duration time out and maximum token amount to of ChatGPTService.
@tmcdonnell2 tmcdonnell2 requested review from a team as code owners March 23, 2023 20:37
Turn ChatGPTService to be statically accessed and thus a utility class.
a9115d6 
Update to pass Code Smell of not having package-info.java.
Taz03
Taz03 requested changes Mar 24, 2023
View reviewed changes
Taz03
Taz03 requested changes Mar 24, 2023
View reviewed changes
Taz03
Taz03 requested changes Mar 24, 2023
View reviewed changes
@Zabuzard Zabuzard linked an issue Mar 24, 2023 that may be closed by this pull request
Integrate ChatGPT #809
Closed
@Zabuzard Zabuzard added new command Add a new command or group of commands to the bot priority: major labels Mar 24, 2023
Zabuzard
Zabuzard requested changes Mar 24, 2023
View reviewed changes
Change ChatGptService.ask to return Optional. Clearer user experience…
520505c 
… text.

Variable name fixes throughout files.
@tmcdonnell2
Copy link
Contributor Author

I am getting a weird error now and I am not sure what the issue is:
There was an error using the OpenAI API: Incorrect API key provided: ***************************************. You can find your API key at https://platform.openai.com/account/api-keys. Code: invalid_api_key Type: invalid_request_error Status Code: 401
It shows the apiKey beginning and end but places stars in the middle. This is likely just to obfuscate the real key. I watched it in the debugger and it appears to be passed in and around correct. Did something happen to the key? Do I have the Config added in wrong?

@marko-radosavljevic
Copy link
Contributor

marko-radosavljevic commented Mar 24, 2023 •
edited
Loading

Yup, because the key was leaked, it triggered automatic key rotation. ☺️

No worries though, we can pin it on discord or something similar.

Zabuzard
Zabuzard requested changes Mar 25, 2023
View reviewed changes
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@Zabuzard Zabuzard merged commit 23a97eb into develop Mar 25, 2023
@Zabuzard Zabuzard deleted the ChatGPT-Impl branch March 25, 2023 19:06
@Zabuzard
Copy link
Member

lets gooo

@marko-radosavljevic marko-radosavljevic mentioned this pull request May 18, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Zabuzard Zabuzard Zabuzard approved these changes

@Taz03 Taz03 Taz03 approved these changes

Assignees

@tmcdonnell2 tmcdonnell2

Labels
new command Add a new command or group of commands to the bot priority: major
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Integrate ChatGPT
4 participants
@tmcdonnell2 @marko-radosavljevic @Zabuzard @Taz03