from Hacker News

Show HN: Tool to Automatically Create Organized Commits for PRs

by edverma2 on 6/20/25, 3:22 AM with 50 comments

I've found it helps PR reviewers when they can look through a set of commits with clear messages and logically organized changes. Typically reviewers prefer a larger quantity of smaller changes versus a smaller quantity of larger changes. Sometimes it gets really messy to break up a change into sufficiently small PRs, so thoughtful commits are a great way of further subdividing changes in PRs. It can be pretty time consuming to do this though, so this tool automates the process with the help of AI.

The tool sends the diff of your git branch against a base branch to an LLM provider. The LLM provider responds with a set of suggested commits with sensible commit messages, change groupings, and descriptions. When you explicitly accept the proposed changes, the tool re-writes the commit history on your branch to match the LLM's suggestion. Then you can force push your branch to your remote to make it match.

The default AI provider is your locally running Ollama server. Cloud providers can be explicitly configured via CLI argument or in a config file, but keeping local models as the default helps to protect against unintentional data sharing. The tool always creates a backup branch in case you need to easily revert in case of changing your mind or an error in commit re-writing. Note that re-writing commit history to a remote branch requires a force push, which is something your team/org will need to be ok with. As long as you are working on a feature branch this is usually fine, but it's always worth checking if you are not sure.

  • by 9dev on 6/20/25, 7:40 AM

    The idea in itself seems good, but I have a lot of hesitation due to the prompt used to rewrite the commit messages. Even looking at the example in the repo there’s this sycophantic, pompous way of describing mundane things that adds nothing, but only makes it harder to understand what has changed. The commits mentioned don’t "implement a complete auth system" and did not add "comprehensive test coverage". They added parts of an authentication system and some tests.

    I’m all for proper commit messages, but only if they add clarity, not take it away.

  • by scottgg on 6/20/25, 5:14 AM

    Since moving to jj[1] as a git-compatible alternative, I’ve found it so easy to make clean commits I do it by default for everything - usually 1/ refactor 2/impl, 3/ docs. Because you can always just “jj new” on top of an existing change then squash it down and get automatic rebase past that point it’s quick to keep things organised and makes review life suck less.

    [1] https://github.com/jj-vcs/jj

  • by amenghra on 6/20/25, 7:18 AM

    https://graphite.dev/ provides a way to stack PRs, it's been discussed on HN in the past (e.g. https://news.ycombinator.com/item?id=30681308).
  • by stpedgwdgfhgdd on 6/20/25, 1:53 PM

    Probably not a popular take on HN; valuable information should not be hidden in commits but in comments. Especially the WHY is crucial to write down in comments.

    I’m fine that people squash as long the reasoning is recorded in comments and reflected through automated tests (unit AND system/api).

    This is also crucial information for AI coding tools.

  • by bredren on 6/20/25, 5:16 AM

    This is a cool idea, though part of what keeps my work organized and my understanding of my own changes is to do the manual preparation of a series of logical commits.

    I make use of interactive partial commits using Pycharm when a single file has changes related to different ideas and rewrite history for clarity.

    It does matter to me if someone else has gone to this trouble. And it is sometimes a tip off if a person is seemingly sloppy in commit history.

    It makes sense that this project exists but I’m also glad it didn’t when I was learning to work in professional environments.

  • by CityOfThrowaway on 6/20/25, 4:30 AM

    I haven't tried this yet (though I plan to).

    One thing I would love is if I could give it a hint and have it extract out certain types of changes into its own branch that could split into a new PR.

    I often find myself adding a new, re-usable component or doing a small refactor in the middle of a project. When you're a few commits into a project and start doing side-quests, it's super annoying to untangle that work.

    The options are one of:

    1. A mega PR (which everybody hates) 2. Methodologically untangling the side quest post-hoc 3. Not doing it

    In principle, the "right" thing to do would be to go checkout main, do the side quest, get it merged and then continue.

    But that's annoying and I'd rather just jam through, have AI untangle it, and then stack the commits (ala Graphite).

    It's easy to verbally explain what stuff is side-quest vs. main quest but it's super annoying to actually do the untangling.

    Maybe this tool magically can do that... but I do wonder if some context hints from the dev would help / make it more effective.

  • by pabs3 on 6/21/25, 4:34 AM

    Reminds me of git-absorb, which is a non-AI version of part of this.

    https://github.com/tummychow/git-absorb

  • by aaronbrethorst on 6/20/25, 5:45 AM

    Very cool, I’ve been looking for something like this, but I’d love to see this flipped around and become an MCP tool that can be consumed by an LLM instead of requiring an API key.
  • by esafak on 6/20/25, 5:27 AM

    I want something that takes a big commit and splits it up!
  • by jaredsohn on 6/20/25, 6:22 AM

    I didn't look at things too closely, but it would be nice if this each commit would include a ticket number from the branch (such as a linear id) and/or pr id in each commit for people who do not squash.

    One huge advantage of squashing branches is if you see a commit in a `git blame` you might have an idea of where it came from within GitHub/Linear/other systems.

  • by bananapub on 6/20/25, 7:56 AM

    I had a look through the tests, but it doesn’t seem you do any testing of of this does a good job or not?

    How did you collate hundreds or thousands of examples of commits being split up and how did you score the results LLMs gave you? Or did it take more than that?

  • by iandanforth on 6/20/25, 4:28 AM

    Can I suggest, don't do this? The sustainable unit of code modification is the ticket, not the commit. When you're ready to merge to main, squash all commits into one that takes the ticket title as its commit message and appends the ticket description as its description. This aligns code changes with planned, scoped, and documented units of work. Anything more granular than that quickly becomes noise. By following the above pattern your main commit history becomes a clean, consistent log of tickets being completed, each linking directly back to your ticket management system.
  • by kelseydh on 6/20/25, 7:00 AM

    I would love Github to integrate this, as that is typically where I am writing my squash commit messages (when merging Pull Requests).
  • by Cthulhu_ on 6/20/25, 6:12 AM

    In our org we squash all commits into one anyway, the main commit title is based on the title of the merge request. We also have an AI code review tool set up (which I usually ignore because there's a lot of extraneous information) that suggests a new title, given that the people making the MR often don't consider that the title ends up in the changelog, becoming the one line that will be used by people using the library to decide whether they need to do something.
  • by 3D39739091 on 6/20/25, 12:36 PM

    Why not just do your work in an organized way in the first place?
  • by bckr on 6/20/25, 4:31 AM

    I was just talking about writing a tool like this. Bookmarked. Thanks.
  • by maomaomiumiu on 6/20/25, 9:02 AM

    Interesting tool. Automating commit cleanup could save time, especially before reviews. Curious to see how well it handles larger or more complex diffs.
  • by nrvn on 6/20/25, 9:43 AM

    In my org I have enforced linear history, squashing all commits into one in PRs and roughly following the rule from [1]:

    > If the request is accepted, all commits will be squashed, and the final commit description will be composed by concatenating the pull request's title and description.

    One less thing to think about.

    Less is more, not vice versa.

    [1]: https://go.dev/doc/contribute#review

  • by h1fra on 6/20/25, 8:56 AM

    I'll never understand people caring about commit in PRs, just push whatever and squash at the end. If commit matters that means you should have done multiple PRs