It's dangerous, and maybe get unexpected result when refactoring. In following example, if we change foo to async, or inside foo somehow reenter foo, cache may got change.

Fail

let cache; 

function foo() {
	cache = {};
	// ...
}

export default foo;

Pass

function foo() {
	let cache;

	// ...
}