Ok. I personally think these things should clean up after themselves. But if not, there should at least be a very obvious warning in the doc block to make it clear that this class creates a temp directory but doesn't remove it. Or maybe if it makes the most sense for the caller to delete it, we should just force the caller to create it.