‹ BackHN Continuity

Thread

Commit description as a thinking tool

130 points · 78 comments · yedhukrishnan

  1. seunosewa · · focus · HN ↗
    I use a different LLM family to review commits and write detailed descriptions. If a commit was written with Fable/Opus, I use Sol/Astra to write a well reasoned commit message. If the message doesn't match my intent, then that triggers a manual review.
    1. bigmadshoe · · focus · HN ↗
      This makes no sense to me. The code is already self-documenting if written well, and all you need is a one line commit message to summarize that.

      Doesn't the original conversation at least retain the context about why the change was made? A different LLM literally has no way to tell why you made this change besides guessing from the codebase and git history.

      1. sigbottle · · focus · HN ↗
        the mechanism is self documenting; context is not unless you pollute all your files with an ADR's worth of alternatives.
      2. seunosewa · · focus · HN ↗
        If a change makes sense, a different frontier model can usually figure out why it was made from the code alone. I take that as a signal that that the commit is good.

        I believe they can do this due to having millions of public pull requests and github issues in their training data.

        1. cerved · · focus · HN ↗
          There's pretty much always several plausible reasons for why a change was made. What's interesting is knowing exactly which one, especially when it later turns out to be wrong!
        2. bigmadshoe · · focus · HN ↗
          What a change does is self documenting. Why it was made and why that specific approach was chosen could be due to many things, such as:

          * business objectives,

          * the result of experimentation,

          * the result of an offline conversation

          etc.

          This cannot be captured in code alone. This is why we write commit messages.

Open on Hacker News to reply ↗

Unofficial Hacker News client; not affiliated with Y Combinator.