This part feels odd. I think it makes sense to consider OO design during code review.

It's not quite the place for an in-depth debate, but feedback like "What if we used a Visitor to solve this? It'd would address issue X and Y, at the small cost of tradeoff A" seems valuable.