Over 150 days, this developer's churn really fluctuates and that's because they work on different things, that requires different amounts of code. And if you look at the following:
you can see they still commit regularly, but as the Reviewability section shows, their changes are mainly small ones, which sort of aligns with what Sid (sytse) mentioned, which is mainly focusing on small changes.
If you look at the bigger picture:
The churn for the project microsoft/vscode fluctuates quite a bit as well.
Based on what I've learned so far, you really need a good baseline (that can vary greatly from one developer to another) to be able to determine if somebody is more/less productive.
Also I would suggest that ad-hoc analyses and other such hacks are unlikely to lead to PRs and other visible artifacts.
Impossible to have 'metric based reviews' at that point. But I think that's fine.
You probably already know quite a few laws, just not that they have names. "Power tends to corrupt; absolute power corrupts absolutely", and "Any sufficiently advanced technology is indistinguishable from magic" come to mind.
One cheap, but mostly invisible mistake that lingers isn't a problem... but they can pile up.
1. The ones that “don’t make mistakes”, but actually they are just unaware. 2. The ones that expect themselves to make mistakes and prepare for that.