That's already available via

$server_facts['environment']

, so I can't see much of a reason to do that https://help.puppet.com/core/current/Content/PuppetCore/lang_facts_builtin_variables.htm