Adversarial changes
The PR that tries to talk its way past the reviewer.
Most review tools assume the diff is arguing in good faith. This group assumes it isn't. It's the one group that never gets gated away, because the PRs worth catching are exactly the ones that look cheap to skip.
- Instructions aimed at the reviewerText planted for an AI reviewer to obey, forged sign-offs, wording that tries to redefine the rules
- Code hidden from reviewEncoded payloads decoded and run, a string assembled and executed, a minified blob passed off as hand-written source
- New traffic to an outside hostFresh HTTP, socket or DNS calls leaving the service, especially carrying secrets or user data
- Behaviour on a hidden triggerDestructive or auth-related code that only fires on a certain date, tenant, hostname or magic value
- A description that doesn't match the diffA security-relevant change the write-up never mentions, or a high-impact edit buried in a large one
- The guard removed with the thing it guardedA security test deleted alongside the change it covered, an assertion loosened, a snapshot rewritten to bless the new output