Is this the simplest solution (can you remove code or complexity)?
Are functions pure and testable?
Are concerns properly separated?
Are files / modules properly separated and organized?
Has static analysis been run? (linting, dependency audits, etc.)
Can you explain what it does / the design / the architecture in plain and simple terms? (while this seems unrelated to code, it can be an indication of too much complexity)
Do names say what they are/do? Are arrays pluralized? Are objects singular? Are there single letter variables? Do function names relate what they return?
Check again if anything can be removed or simplified
These are some steps I think through during code review
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
These are some steps I think through during code review