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

ai commit summary creation using chat-gpt #1979

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

Conversation

extrawurst
Copy link
Owner

@extrawurst extrawurst commented Dec 15, 2023

Right now this is not optimized for long diffs.

TODO:

  • feed it less context if diff gets >4k
  • feed list of files touched first and then diffs and cut at 4k point
  • put behind feature flag
  • other rust crate for openai?
  • changelog
  • readme on how to configure

@extrawurst extrawurst marked this pull request as draft December 15, 2023 23:30
@vladejs
Copy link

vladejs commented Jun 17, 2024

Hi, I tested this on my Mac and noticed it works pretty good . How can we prioritise this? Also, I might help by adding docs on usage or helping trying different prompts.

@vladejs
Copy link

vladejs commented Jun 17, 2024

Current message output gets bigger with current prompt.

Screenshot 2024-06-17 at 1 30 26 p m

This is the default prompt used at jetbrains products which we can modify to our needs:

Avoid overly verbose descriptions or unnecessary details.
Start with a short sentence in imperative form, no more than 50 characters long.
Then leave an empty line and continue with a more detailed explanation.
Write only one sentence for the first part, and two sentences at most for the detailed explanation.

@richgrov
Copy link

Any plans to continue work on this? I'd be open to helping if not

@extrawurst
Copy link
Owner Author

I would appreciate that

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.

3 participants