Ah. I get it. Yes modules will have `facts.d` which is where the external facts will live. Instead of having this logic here, it should be using methods on `Puppet::Module`, which it should get from the current `Puppet::Node::Environment` instance.