ai newsDi., 20. Okt.
EilmeldungUS bans use of China’s DeepSeek AI tools by government agencies

I benchmarked AI code-review skills on 18 seeded-bug PRs. Detection was a tie. Trust wasn't.

I build correctness-critical software for a living (hospital integrations: HL7, DICOM, FHIR, under EU medical-device rules). When I started using AI agents to review code, the question I cared about was not "can it find bugs?" but "can I act on what it says without re-checking everything myself?" S

dev.toDienstag 13:41 · 20.10.2 Min. Lesezeit
Bild: dev.to

I build correctness-critical software for a living (hospital integrations: HL7, DICOM, FHIR, under EU medical-device rules). When I started using AI agents to review code, the question I cared about was not "can it find bugs?" but "can I act on what it says without re-checking everything myself?"

So I built a small benchmark and measured it.

18 pull requests, each a real git repository with a main and a feature branch:

Two reviewers, same model, same instructions ("review this PR"), each on a fresh copy of every repo:

A third agent judged every pair of reviews blind (anonymised as X and Y, randomised per case) against a written answer key.

Both reviewers found 15/15 seeded bugs, on Claude Opus and on Claude Haiku, and both gave 18/18 correct verdicts.

I did not expect that. Current models find this class of bug no matter which skill wraps them. If someone tells you their review prompt "finds more bugs", ask for the benchmark.

On the fast, cheap model the gap is about 5x on both noise and proof. That is the setting most people actually run reviews in, at scale, in CI.

What "proof" looks like in practice: on the open-redirect PR, the review did not say "the regex might be too permissive". It wrote a test, ran it, and pasted four URLs that land the user on evil.test after login, plus a second bypass ( https://app.example.com//evil.test ) that was not even in my answer key.

On Haiku it reported one false positive (it called deliberate, already-tested cent rounding a Critical under-billing bug) and inflated one severity. Version 1.1 adds two rules: behaviour that a test explicitly asserts is intended unless the PR or spec says otherwise, and when torn between two severities, pick the lower one.

The thing I would take away even if you never use my skill: stop grading review tools on what they find and start grading them on what you can act on without re-checking. Proof is cheap now. Ask for it.

Quelle: dev.to (dev.to). Original öffnen