As to not re-download the whole thing if all we need is to unpack the module.

One test: |make node_modules|. Then delete node_modules.  The |make node_modules|. Without the modules-fetch-stamp target we'd just re-download everything instead of just copying them again.

I have tested the various scenarios and that's all I could come up with.

It that a problem to have this intermediate target?