return if post.blank?
raise Discourse::InvalidParameters.new(:post_id) if @post_id.blank?
This is trivial but usually it’s easier for our brains to parse regular ifs rather than unless/not.
post_changed_elsewhere = (start_raw != post.raw)
raw_changed_here = (raw != post.raw)
Very trivial but I find this easier to read because it makes the one assignment very clear.