> but because we have this data (file name, etc.) passed back and forth between different layers in the code. we were doing lots of .clone() and db.open() at different layers to fetch the same data. The interesting part for me was that, this change reduced our

Reduced your what? The article seems to be cut off.

▲

taeric 3 hours ago | parent | next [-]

I'm assuming it was going to say lines of code. It is highlighting that more was deleted than added.

▲

larpingscholar 2 hours ago | parent [-]

I agree. Though the line count is meaningless as the vibe coded diff has pointless formatting and whitespace changes.

	▲	suriya-ganesh 2 hours ago \| parent [-]
		I wrote that PR and can guarantee you that it was not vibe coded. the reason why I thought this PR was interesting is that, it only needed ~ 4 lines of real change, everything else was cascades due to that change. and the performance did improve with that "pointless formatting and whitespace changes"

▲

suriya-ganesh 2 hours ago | parent | prev [-]

ah! sorry. I meant to say, "the interesting part for me was that we removed code to improve performance."