I have put myself forward to review a technical task. I am going to just write notes, not be harsh.
But what if I see a nice to have, do I raise it directly. Example the app isn't very accessible, despite this not being such a concern in the scope of what we do (I still push for it), no keyboard usage to me is a tell about UX consideration and my test was accessable. But then again it's not a criteria.
I think I will just stick to the scope, that's fair isn't it?
Have you ever done this before, let me know in the comments what your approach was. What do you care about, and does it even matter what we care about?