Hello,
This is a comment about your review request here.
It looks fine but if roads are no longer open to vehicles and will be removed in the future, they might be better tagged as "demolished:highway=*" (or another lifecycle prefix).
This is because "highway=construction" is only for future roads (i.e. a road which is built for the first time).  The tag is frequently abused for temporary closures during reconstruction, but I don’t think it makes sense if the road will be entirely removed from the map in a few months.
Have a nice day.