I think it's better to not run npm as root user on container.
I would suggest adding --user 1000 to your docker run command.
Good point. Here's the improvement that work for me
https://github.com/ashishb/dotfiles/commit/fe4fb15fe867bf77a...
Would you prefer doing those tricks or exposing everything on your machine to random npm packages?