‹ BackHN Continuity

Thread

Jev Based Code Review

48 points · 57 comments · namanbhulawat

  1. lm2s · · focus · HN ↗
    I think your engineering process is fundamentally broken if you are generating PRs with 230 files changed so regularly that you need to bolt on more AI. You’re solving the wrong problem.
    1. stuaxo · · focus · HN ↗
      That really stood out.

      Code is there to be read and understood by the human developers who come later.

      The git history is a similar record, that's why the commits that make it to main (the squashed PRs) should cover one(ish) thing each and be self contained.

      Something covering 230 files should be a mechanical change like running a linter or the AI is moving an API from one signature to another.

      If an LLM generated a 230 file change they are also capable of going back and breaking it up.

      One thing they are bad at is comments that are succinct since they almost only ever add words.

      1. moffkalast · · focus · HN ↗
        > human developers who come later

        Tbf, once the codebase is slopped enough that becomes impossible and only LLM can come later.

        1. latentsea · · focus · HN ↗
          Yeah, we should push back on this. We don't have to accept this outcome as if it's inevitable.
          1. invader · · focus · HN ↗
            Sadly, the hype is real and that's what's happening in many places. Businesses are eager to bet on AI cause they'd been promised x100 productivity = fire 99 or 100 devs = huge profit.

            I expect one day some slopware will succumb to one of these weird production bugs, no LLM will be able to fix it, and when the biz guys ask me for a fix estimate, I'm going to say "3 years".

            1. latentsea · · focus · HN ↗
              I went all in on it about a year ago, and I've gotten the hype out of my system now. It's now officially a love/hate relationship. Like everything else that is a mainstay in my life.
Open on Hacker News to reply ↗

Unofficial Hacker News client; not affiliated with Y Combinator.