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

Introducing JFrog CLI AI commands assistant #2703

Merged
merged 21 commits into from
Sep 24, 2024

Conversation

sverdlov93
Copy link
Contributor

@sverdlov93 sverdlov93 commented Sep 22, 2024

  • All tests have passed. If this feature is not already covered by the tests, new tests have been added.
  • The pull request is targeting the dev branch.
  • The code has been validated to compile successfully by running go vet ./....
  • The code has been formatted properly using go fmt ./....

The JFrog CLI AI Command Assistant streamlines your workflow by turning natural language inputs into JFrog CLI commands.

Simply describe your desired actions, and the assistant generates commands with all necessary parameters, whether you're uploading artifacts, managing repositories, scanning your code or performing other actions using the JFrog CLI.

Each query is treated individually, and while the interface allows you to refine requests, it doesn’t maintain a chat history.

This tool helps users access the full power of JFrog CLI without needing to remember specific syntax, ensuring efficiency and accuracy.

Full docs - https://docs.jfrog-applications.jfrog.io/jfrog-applications/jfrog-cli/cli-ai

Signed-off-by: Michael Sverdlov <[email protected]>
@sverdlov93 sverdlov93 added the improvement Automatically generated release notes label Sep 22, 2024
Signed-off-by: Michael Sverdlov <[email protected]>
Signed-off-by: Michael Sverdlov <[email protected]>
Signed-off-by: Michael Sverdlov <[email protected]>
Signed-off-by: Michael Sverdlov <[email protected]>
Signed-off-by: Michael Sverdlov <[email protected]>
@sverdlov93 sverdlov93 added the safe to test Approve running integration tests on a pull request label Sep 22, 2024
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Sep 22, 2024
Copy link
Contributor

@eyalbe4 eyalbe4 left a comment

Choose a reason for hiding this comment

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

I didn't review the entire PR yet, but I added my inline comments.
I'd be happy to review the entire PR after you fix or respond to my comments.

general/ai/cli.go Outdated Show resolved Hide resolved
utils/cliutils/persistence.go Outdated Show resolved Hide resolved
utils/cliutils/persistence.go Outdated Show resolved Hide resolved
utils/cliutils/persistence.go Outdated Show resolved Hide resolved
Signed-off-by: Michael Sverdlov <[email protected]>
Signed-off-by: Michael Sverdlov <[email protected]>
Copy link
Contributor

@eyalbe4 eyalbe4 left a comment

Choose a reason for hiding this comment

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

I'm done reviewing all the code.
I'd like to review this PR again when you'er done implementing or responding to my comments.

utils/cliutils/persistence_test.go Outdated Show resolved Hide resolved
utils/cliutils/persistence_test.go Outdated Show resolved Hide resolved
utils/cliutils/utils.go Show resolved Hide resolved
Signed-off-by: Michael Sverdlov <[email protected]>
Signed-off-by: Michael Sverdlov <[email protected]>
@sverdlov93 sverdlov93 added the safe to test Approve running integration tests on a pull request label Sep 22, 2024
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Sep 22, 2024
@sverdlov93
Copy link
Contributor Author

I'm done reviewing all the code. I'd like to review this PR again when you'er done implementing or responding to my comments.

done. alse there is this - jfrog/documentation#159
@eyalbe4

@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Sep 23, 2024
Signed-off-by: Michael Sverdlov <[email protected]>
@sverdlov93 sverdlov93 added the safe to test Approve running integration tests on a pull request label Sep 23, 2024
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Sep 23, 2024
Signed-off-by: Michael Sverdlov <[email protected]>
@sverdlov93 sverdlov93 added the safe to test Approve running integration tests on a pull request label Sep 24, 2024
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Sep 24, 2024
Signed-off-by: Michael Sverdlov <[email protected]>
Signed-off-by: Michael Sverdlov <[email protected]>
@sverdlov93 sverdlov93 added the safe to test Approve running integration tests on a pull request label Sep 24, 2024
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Sep 24, 2024
Signed-off-by: Michael Sverdlov <[email protected]>
@sverdlov93 sverdlov93 added the safe to test Approve running integration tests on a pull request label Sep 24, 2024
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Sep 24, 2024
Signed-off-by: Michael Sverdlov <[email protected]>
@sverdlov93 sverdlov93 added the safe to test Approve running integration tests on a pull request label Sep 24, 2024
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Sep 24, 2024
Signed-off-by: Michael Sverdlov <[email protected]>
@sverdlov93 sverdlov93 changed the title Add persistence file to handle local indicators such as latest version check Introducing JFrog CLI AI commands assistant Sep 24, 2024
@sverdlov93 sverdlov93 added new feature Automatically generated release notes and removed improvement Automatically generated release notes labels Sep 24, 2024
@sverdlov93 sverdlov93 merged commit 2c5f949 into jfrog:dev Sep 24, 2024
29 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature Automatically generated release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants