Ask HN: How to deal with long vibe-coded PRs?

2•philippta•3h ago

Today I came across a PR for a (in theory) relatively simple service.

It span across 9000 LOC, 63 new files, including a DSL parser and much more.

How would you go about reviewing a PR like this?

Comments

zigcBenx•3h ago

In my opinion no PR should have so much changes. It's impossible to review such things.

The only exception is some large migration or version upgrade that required lots of files to change.

As far it goes for Vibe coded gigantic PRs It's a straight reject from me.

aaronrobinson•3h ago

Reject it

wengo314•2h ago

reject outright. ask to split it into reasonable chain of changesets.

White-Collar Jobs Are Disappearing as AI Starts to Bite

It's time for Germany to admit its mistake on nuclear energy (2024)

Looking for feedback on this AI code review benchmark

Money to Help Nations Cope with Climate Disasters Is Declining, U.N. Says

Links Between Sky Transients, Nuclear Blasts, Unidentified Anomalous Phenomena

A satellite runs Doom from orbit, using Ubuntu on Arm

Beatriz Villaroel says we're not alone, we have company [video]

Moving Back to a Dynamic Website

Collatz-Weyl Generators: Pseudorandom Number Generators

Kudos to Python Software Foundation. I just made my first donation

A modern C++ wrapper for the Firebird database API

Show HN: Learn German with Games

US startup Substrate announces chipmaking tool that it says will rival ASML

U.S. attempted to capture Venezuela's Maduro by bribing his pilot

Show HN: See all the AI chaos in one place – 24/7

Iframe with Hot-Relod

Jamf to be Acquired by Francisco Partners for $2.2B

The Data Centers That Train A.I. and Drain the Electrical Grid

I built a benchmark to score the 'architectural intelligence' of neural nets

Tons of steel: Half the heart of SPARC fusion machine arrives [video]

Show HN: We built a tool that helps founders automate their LinkedIn

Solar-powered QR reading postboxes being rolled out across UK

The Doom Spenders

Arctic town Kiruna is colder after the move

When `-O3` is 2x slower than `-O2` – profiling binary search in Rust

Autodesk's John Walker Explained HP and IBM in 1991

Have the environmental benefits of insect farming been overstated? A review

Free traffic estimation API based on open page rank

GRC Tool Vanta Forces Insecure CAA Configuration

React Native Upgrade: How to Tackle the Mission