include "local" facts in facts output

lib/puppet/indirector/facts/facter.rb

@@ -49,7 +49,7 @@ def find(request)
                Puppet::Node::Facts.new(request.key, facts.to_h)
              end
 
-    result.add_local_facts unless request.options[:resolve_options]
+    result.add_local_facts(request.options[:user_query])
     result.sanitize
     result
   end

lib/puppet/node/facts.rb

@@ -27,11 +27,12 @@ def save(instance, key = nil, options = {})
 
   attr_accessor :name, :values, :timestamp
 
-  def add_local_facts
-    values["implementation"] = Puppet.implementation
-    values["clientcert"] = Puppet.settings[:certname]
-    values["clientversion"] = Puppet.version.to_s
-    values["clientnoop"] = Puppet.settings[:noop]
+  def add_local_facts(query = [])
+    query = Array(query) # some use cases result in a nil being passed in
+    values["implementation"] = Puppet.implementation if query.empty? or query.include? 'implementation'
+    values["clientcert"] = Puppet.settings[:certname] if query.empty? or query.include? 'clientcert'
+    values["clientversion"] = Puppet.version.to_s if query.empty? or query.include? 'clientversion'
+    values["clientnoop"] = Puppet.settings[:noop] if query.empty? or query.include? 'clientnoop'
   end
 
   def initialize(name, values = {})

spec/unit/application/facts_spec.rb

@@ -55,7 +55,11 @@
     let(:expected) { <<~END }
       {
         "filesystems": "apfs,autofs,devfs",
-        "macaddress": "64:52:11:22:03:2e"
+        "macaddress": "64:52:11:22:03:2e",
+        "implementation": "#{Puppet.implementation}",
+        "clientcert": "#{Puppet.settings[:certname]}",
+        "clientversion": "#{Puppet.version}",
+        "clientnoop": false
       }
     END
 
@@ -81,7 +85,7 @@
     end
 
     it "warns and ignores value-only when multiple fact names are specified" do
-      app.command_line.args << 'filesystems' << 'macaddress' << '--value-only'
+      app.command_line.args << 'filesystems' << 'macaddress' << 'implementation' << 'clientcert' << 'clientversion' << 'clientnoop' << '--value-only'
       expect {
         app.run
       }.to exit_with(0)
@@ -121,6 +125,10 @@
       ---
       filesystems: apfs,autofs,devfs
       macaddress: 64:52:11:22:03:2e
+      implementation: #{Puppet.implementation}
+      clientcert: #{Puppet.settings[:certname]}
+      clientversion: #{Puppet.version}
+      clientnoop: false
     END
 
     before :each do

spec/unit/indirector/facts/facter_spec.rb

@@ -192,12 +192,6 @@ class Puppet::Node::Facts::Facter::MyCollection < Hash; end
       @facter.find(@request)
     end
 
-    it 'should NOT add local facts' do
-      expect(facts).not_to receive(:add_local_facts)
-
-      @facter.find(@request)
-    end
-
     context 'when --show-legacy flag is present' do
       let(:options) { { resolve_options: true, user_query: ["os", "timezone"], show_legacy: true } }