Allow toggling setting rule inputs/outputs in query --output=proto

If users don't need them they can save a significant amount of query
output/storage space by omitting them.

RELNOTES: None
PiperOrigin-RevId: 259848294

src/main/java/com/google/devtools/build/lib/query2/CommonQueryOptions.java

@@ -153,6 +153,14 @@ public Set<Setting> toSettings() {
   )
   public List<String> protoOutputRuleAttributes = ImmutableList.of("all");
 
+  @Option(
+      name = "proto:rule_inputs_and_outputs",
+      defaultValue = "true",
+      documentationCategory = OptionDocumentationCategory.QUERY,
+      effectTags = {OptionEffectTag.TERMINAL_OUTPUT},
+      help = "Whether or not to populate the rule_input and rule_output fields.")
+  public boolean protoIncludeRuleInputsAndOutputs;
+
   /** An enum converter for {@code  AspectResolver.Mode} . Should be used internally only. */
   public static class AspectResolutionModeConverter extends EnumConverter<Mode> {
     public AspectResolutionModeConverter() {

src/main/java/com/google/devtools/build/lib/query2/query/output/ProtoOutputFormatter.java

@@ -93,6 +93,7 @@ public class ProtoOutputFormatter extends AbstractUnorderedFormatter {
   private Predicate<String> ruleAttributePredicate = Predicates.alwaysTrue();
   private boolean flattenSelects = true;
   private boolean includeLocations = true;
+  private boolean includeRuleInputsAndOutputs = true;
 
   protected void setDependencyFilter(QueryOptions options) {
     this.dependencyFilter = OutputFormatter.getDependencyFilter(options);
@@ -111,6 +112,7 @@ public void setOptions(CommonQueryOptions options, AspectResolver aspectResolver
     this.ruleAttributePredicate = newAttributePredicate(options.protoOutputRuleAttributes);
     this.flattenSelects = options.protoFlattenSelects;
     this.includeLocations = options.protoIncludeLocations;
+    this.includeRuleInputsAndOutputs = options.protoIncludeRuleInputsAndOutputs;
   }
 
   private static Predicate<String> newAttributePredicate(List<String> outputAttributes) {
@@ -402,7 +404,7 @@ protected boolean includeRuleDefinitionEnvironment() {
   }
 
   protected boolean includeRuleInputsAndOutputs() {
-    return true;
+    return includeRuleInputsAndOutputs;
   }
 
   protected boolean includeLocation() {

src/test/java/com/google/devtools/build/lib/query2/cquery/ProtoOutputFormatterCallbackTest.java

@@ -61,6 +61,7 @@ public final void setUpCqueryOptions() {
     this.options = new CqueryOptions();
     options.aspectDeps = Mode.OFF;
     options.protoIncludeConfigurations = true;
+    options.protoIncludeRuleInputsAndOutputs = true;
 
     this.reporter = new Reporter(new EventBus(), events::add);
   }