both ellis and liberty islands are part of the Statue of Liberty National Monument so the polys for these islands should be marked with those tags for a national park. its already been set to be part of the Statue of Liberty National Monument relation.

this is not a lighthouse, rather it should be tagged as a type of seamark osm.wiki/Seamarks/Seamark_Objects

police node: node/368051819 and building: way/278112893 but have a police tag on them but really only one should exist here, perhaps merge the node on to the building and update the name tags accordingly or remove the police tag from the building and instead use a government office tag in favor of keeping the existing node that has the police tag on it. Number of ways to go about it...

not sure that sidewalks should have names on them way/846200669 perhaps at a minimum name should be a note - a lot more sidewalks attached to the empire trail have had names added to them too, its not just this one.

technically the island is no longer here node/357607394 so perhaps tags should be changed to update it? otherwise is it something still visible at extreme low water levels? See description of island here: https://en.wikipedia.org/wiki/Patroon_Island_Bridge

road is closed to public access sunrise to sunset which should be added here: see conditional access time section access=*, and road also allows pedestrian, bike, cross-country skiing, and snow-shoeing access which could be added as well. road is not plowed during winter so additional tag for winter service should be added: winter_service=*. Road access info here: https://www.nps.gov/sara/planyourvisit/hours.htm

think this should not be tagged as an attraction...

there is a parking node and parking area here but with different tags, they should be merged into one but tags need to be harmonized