INJECT_FACTS_AS_VARS is defaulting to False in 2.24

I’m not trying to force it, just moving to a saner default as the injection uses a lot more memory and cpu time, aside from creating possible collisions and security issues. This will incentivize new content to not rely on it.

That said, I agree, there should be simpler ways of identifying where the variables are used, your PR goes a long way for that, my fear is warning spam and we are looking at a way to address that.

As an aside, what i really want to do in the end is add better introspection in general, being able to list what vars are used where and from what origin for a play/role. But that is a much taller order.