Howdy! Since you requested a review, I looked over your changes here. Looks great to me, no issues! Just a side note that the access=* tag is understood to apply to all other access-type tags like bicycle=*. So in this case, since we have access=no, bicycle=no is redundant. But it doesn't hurt to be extra explicit.
Thanks for your contributions!