9 comments

  • munro 9 minutes ago
    IDK lot of noise in report, I need more signal to noise ratio, already reading so much AI output as it is.

    But the example PR... It said code generate with Claude, is it really still this bad? Last I was using SQLAlchemy with Claude was Opus 4.5, and it took some heavy coaching and skill gen not to generate code like this--but why even use SQLAlchemy at all if you're just going to use string interpolation jeez

    https://github.com/erich-oliveira-winnin/full-stack-fastapi-...

        owner = "" if current_user.is_superuser else f"WHERE owner_id = '{current_user.id}'"
        query = f"SELECT status, count(*) FROM item {owner} GROUP BY status"
        rows = session.execute(text(query)).all()
    
    Part of me thinks maybe because it's not popular or well documented, but SQLAlchemy has an eDSL of SQL, so you can skip the whole ORM/table mapping layer, and just use it to programmatically construct SQL syntax
    • benglish11 4 minutes ago
      That’s my favorite part of sqlalchemy. I don’t even use the orm anymore.
  • fcarraldo 15 minutes ago
    I think tools like this are needed and will become standard in the near future.

    However, the backend demo report seems longer and more complex than just reading the code. I’m trying to build something similar as well, and no one has quite nailed this yet.

    The risk map / assisted review needs to be succinct and information dense, to guide your attention. My current philosophy is that we need to be annotating code, not replacing the entire code review interface with a bunch of text.

    • ericholiveira 12 minutes ago
      Thanks... it surely is possible to try to reduce the amount of information
  • alsiola 23 minutes ago
    Looked at the backend demo, and the first lines suggested to look at were importing things. Exactly what I really don't need to be reviewing
    • ericholiveira 16 minutes ago
      It suggested based on the files most used (through graph dependency), for this reason it appears as a important file (as if it breaks, it can break several places in your codebase), but that being said, you can add some files to be ignored (if you have special files, like consts or i18n)
  • ccvannorman 59 minutes ago
    why is the only comment by the OP flagged and dead? Is it because they were supposed to include that data in the description text rather than a comment, was it flagged as AI generated, or..?

    Trying to understand why the poster's own comment was flagged and killed but the original article remains up.

    • BalinKing 43 minutes ago
      AI-edited or -generated comments are disallowed by HN guidelines: https://news.ycombinator.com/newsguidelines.html#generated. I don't think any such restriction exists for submissions themselves, so that's my guess as to why the comment but not the submission has been flagged to death.
    • shiandow 54 minutes ago
      I'm not sure either. The comments don't look wrong to me.
    • rubyn00bie 44 minutes ago
      You can vouch for it, if you view the comment by clicking the time since posted a “vouch” option appears. I did it. I’ve noticed this happening more and more on here for show HN posts especially. I’m not sure if it’s like some sort of troll gang or what…
  • sikozu 49 minutes ago
    I like the concept, a risk map could be useful in some circumstances but I absolutely hate the UI design; it reeks of AI with no human thought or care put into it.
    • ericholiveira 15 minutes ago
      Thats is surely true... i did it with AI and i do lack design skills
  • looperhacks 36 minutes ago
    Or you could, you know, read the changes your approving
    • y42 34 minutes ago
      Sire, wouldst thou perchance elaborate upon this ancient, forgotten technique, long buried in the mists of antiquity and shrouded in mysterious lore?
  • usefulbasis 31 minutes ago
    [flagged]
  • chelseahermes 58 minutes ago
    [flagged]
  • ericholiveira 1 hour ago
    [flagged]