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

Chat command with gpt-wrapper integration #545

Merged
merged 19 commits into from
Jun 21, 2023
Merged

Chat command with gpt-wrapper integration #545

merged 19 commits into from
Jun 21, 2023

Conversation

diogopcx
Copy link
Contributor

Description

Add chat command

References

Testing

Checklist

  • I have added documentation for new/changed functionality in this PR (if applicable).
  • I have updated the CLI help for new/changed functionality in this PR (if applicable).
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used

@pedrompflopes pedrompflopes requested review from a team, pedrompflopes and tiagobcx and removed request for a team June 16, 2023 13:37
@github-actions
Copy link

github-actions bot commented Jun 16, 2023

Logo
Checkmarx One – Scan Summary & Details4783042e-24f2-4da5-8bff-2ae795168436

No New Or Fixed Issues Found

@diogopcx diogopcx changed the title Gpt Chat command with gpt-wrapper integration Jun 16, 2023
@diogopcx diogopcx marked this pull request as ready for review June 16, 2023 16:35
@pedrompflopes pedrompflopes requested a review from a team June 16, 2023 16:35
@diogopcx diogopcx merged commit 17d5b0f into main Jun 21, 2023
6 checks passed
@diogopcx diogopcx deleted the gpt branch June 21, 2023 11:51
thtri pushed a commit to thtri/ast-cli that referenced this pull request Apr 19, 2024
* Add chat command

* Config git to access checkmarxDev

* Return chat errors as part of the conversation

* Improve error handling, add unit/integration tests

* Add wrapper with mock for unit testing

* Add missing wrapper argument

* Fix linter issues

* Improve assert message

* Fix chat unit test asserts

* Remove unnecessary ToLower call

* Change prompt to use AI Guided Remediation

* Change prompt to use AI Guided Remediation and IaC Security

* Change prompt to use AI Guided Remediation and IaC Security

* Change prompt to use AI Guided Remediation and IaC Security

* Update gpt-wrapper
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