Hello.   We discussed this issue on the OSM discord.  It falls into a grey area.  Typically OSM doesn't like to close for construction unless the construction will take 6 months, but we also don't know the schedule.

This is cause items like GPS get updated infrequently and this area may continue to be avoided for years by them.

So as a compromise I had already updated the tags on the sections with the sinkhole to have a conditional tag stating no vehicle access for the next 2-4 months....which are just estimates based off news reports.

Of course this doesn't change any visual representation so it's easy to miss unless you literally know to go looking for it or watch the notes.  Also most of the routers are ignoring this tag.  GraphHopper was the only one respecting it.

I'm not saying what you did is wrong, knowing NJ construction schedules, it's probably right, all I ask you to do is monitor the two notes.