Before this PR, this was dead code. A long time ago, the photoshop.disconnect call returned a promise. Now, photoshop.disconnect is synchronous wrt cleaning up resources. So, we can treat it as synchronous in this function.

The reason this didn't get caught is that we weren't actually calling shutdown anywhere!