Delete non-interning, non-singleton @AutoCodec.

PiperOrigin-RevId: 412045358

Author: Googler

src/main/java/com/google/devtools/build/lib/packages/SelectorList.java

@@ -17,7 +17,6 @@
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.Iterables;
 import com.google.devtools.build.lib.collect.nestedset.Depset;
-import com.google.devtools.build.lib.skyframe.serialization.autocodec.AutoCodec;
 import java.util.Arrays;
 import java.util.List;
 import java.util.Objects;
@@ -51,7 +50,6 @@
     name = "select",
     doc = "A selector between configuration-dependent entities.",
     documented = false)
-@AutoCodec
 public final class SelectorList implements StarlarkValue, HasBinary {
 
   // TODO(adonovan): combine Selector{List,Value} and BuildType.SelectorList.
@@ -60,8 +58,7 @@ public final class SelectorList implements StarlarkValue, HasBinary {
   private final Class<?> type;
   private final List<Object> elements;
 
-  @AutoCodec.VisibleForSerialization
-  SelectorList(Class<?> type, List<Object> elements) {
+  private SelectorList(Class<?> type, List<Object> elements) {
     this.type = type;
     this.elements = elements;
   }

src/main/java/com/google/devtools/build/lib/packages/SelectorValue.java

@@ -16,7 +16,6 @@
 import com.google.common.base.Preconditions;
 import com.google.common.collect.ImmutableMap;
 import com.google.common.collect.Iterables;
-import com.google.devtools.build.lib.skyframe.serialization.autocodec.AutoCodec;
 import java.util.Map;
 import net.starlark.java.annot.StarlarkBuiltin;
 import net.starlark.java.eval.EvalException;
@@ -42,7 +41,6 @@
     name = "selector",
     doc = "A selector between configuration-dependent values.",
     documented = false)
-@AutoCodec
 public final class SelectorValue implements StarlarkValue, HasBinary {
 
   // TODO(adonovan): combine Selector{List,Value} and BuildType.SelectorList.

src/main/java/com/google/devtools/build/lib/packages/StarlarkAspectClass.java

@@ -16,11 +16,9 @@
 
 import com.google.devtools.build.lib.cmdline.Label;
 import com.google.devtools.build.lib.concurrent.ThreadSafety.Immutable;
-import com.google.devtools.build.lib.skyframe.serialization.autocodec.AutoCodec;
 import java.util.Objects;
 
 /** {@link AspectClass} for aspects defined in Starlark. */
-@AutoCodec
 @Immutable
 public final class StarlarkAspectClass implements AspectClass {
   private final Label extensionLabel;

src/main/java/com/google/devtools/build/lib/packages/StarlarkCallbackHelper.java

@@ -17,7 +17,6 @@
 import com.google.common.collect.ImmutableMap;
 import com.google.devtools.build.lib.events.Event;
 import com.google.devtools.build.lib.events.EventHandler;
-import com.google.devtools.build.lib.skyframe.serialization.autocodec.AutoCodec;
 import net.starlark.java.eval.EvalException;
 import net.starlark.java.eval.Mutability;
 import net.starlark.java.eval.Starlark;
@@ -34,8 +33,7 @@
  * <p>TODO(adonovan): eliminate the need for this class by making the Starlark calls in the same
  * Starlark thread that instantiated the rule.
  */
-@AutoCodec
-public class StarlarkCallbackHelper {
+public final class StarlarkCallbackHelper {
 
   private final StarlarkFunction callback;
 

src/main/java/com/google/devtools/build/lib/packages/StarlarkDefinedAspect.java

@@ -21,17 +21,14 @@
 import com.google.devtools.build.lib.analysis.config.transitions.ConfigurationTransition;
 import com.google.devtools.build.lib.cmdline.Label;
 import com.google.devtools.build.lib.events.EventHandler;
-import com.google.devtools.build.lib.skyframe.serialization.autocodec.AutoCodec;
-import com.google.devtools.build.lib.skyframe.serialization.autocodec.AutoCodec.VisibleForSerialization;
 import java.util.Objects;
 import net.starlark.java.eval.EvalException;
 import net.starlark.java.eval.Printer;
 import net.starlark.java.eval.Starlark;
 import net.starlark.java.eval.StarlarkCallable;
 
 /** A Starlark value that is a result of an 'aspect(..)' function call. */
-@AutoCodec
-public class StarlarkDefinedAspect implements StarlarkExportable, StarlarkAspect {
+public final class StarlarkDefinedAspect implements StarlarkExportable, StarlarkAspect {
   private final StarlarkCallable implementation;
   private final ImmutableList<String> attributeAspects;
   private final ImmutableList<Attribute> attributes;
@@ -81,43 +78,6 @@ public StarlarkDefinedAspect(
     this.applyToGeneratingRules = applyToGeneratingRules;
   }
 
-  /** Constructor for post export reconstruction for serialization. */
-  @VisibleForSerialization
-  @AutoCodec.Instantiator
-  StarlarkDefinedAspect(
-      StarlarkCallable implementation,
-      ImmutableList<String> attributeAspects,
-      ImmutableList<Attribute> attributes,
-      ImmutableList<ImmutableSet<StarlarkProviderIdentifier>> requiredProviders,
-      ImmutableList<ImmutableSet<StarlarkProviderIdentifier>> requiredAspectProviders,
-      ImmutableSet<StarlarkProviderIdentifier> provides,
-      ImmutableSet<String> paramAttributes,
-      ImmutableSet<StarlarkAspect> requiredAspects,
-      ImmutableSet<String> fragments,
-      // The host transition is in lib.analysis, so we can't reference it directly here.
-      ConfigurationTransition hostTransition,
-      ImmutableSet<String> hostFragments,
-      ImmutableList<Label> requiredToolchains,
-      boolean useToolchainTransition,
-      boolean applyToGeneratingRules,
-      StarlarkAspectClass aspectClass) {
-    this(
-        implementation,
-        attributeAspects,
-        attributes,
-        requiredProviders,
-        requiredAspectProviders,
-        provides,
-        paramAttributes,
-        requiredAspects,
-        fragments,
-        hostTransition,
-        hostFragments,
-        requiredToolchains,
-        useToolchainTransition,
-        applyToGeneratingRules);
-    this.aspectClass = aspectClass;
-  }
 
   public StarlarkCallable getImplementation() {
     return implementation;

src/main/java/com/google/devtools/build/lib/packages/StarlarkNativeAspect.java

@@ -17,13 +17,14 @@
 import com.google.common.base.Function;
 import com.google.common.collect.ImmutableSet;
 import com.google.devtools.build.lib.skyframe.serialization.autocodec.AutoCodec;
+import com.google.devtools.build.lib.skyframe.serialization.autocodec.SerializationConstant;
 import net.starlark.java.eval.EvalException;
 import net.starlark.java.eval.Printer;
 import net.starlark.java.eval.Starlark;
 
 /** A natively-defined aspect that is may be referenced by Starlark attribute definitions. */
 public abstract class StarlarkNativeAspect extends NativeAspectClass implements StarlarkAspect {
-  @AutoCodec @AutoCodec.VisibleForSerialization
+  @SerializationConstant @AutoCodec.VisibleForSerialization
   static final Function<Rule, AspectParameters> EMPTY_FUNCTION = input -> AspectParameters.EMPTY;
 
   @Override

src/main/java/com/google/devtools/build/lib/packages/StarlarkProvider.java

@@ -18,7 +18,6 @@
 import com.google.common.collect.ImmutableList;
 import com.google.devtools.build.lib.cmdline.Label;
 import com.google.devtools.build.lib.events.EventHandler;
-import com.google.devtools.build.lib.skyframe.serialization.autocodec.AutoCodec;
 import com.google.devtools.build.lib.util.Fingerprint;
 import java.util.Collection;
 import java.util.Objects;
@@ -221,8 +220,7 @@ public String toString() {
    * A serializable representation of Starlark-defined {@link StarlarkProvider} that uniquely
    * identifies all {@link StarlarkProvider}s that are exposed to SkyFrame.
    */
-  @AutoCodec
-  public static class Key extends Provider.Key {
+  public static final class Key extends Provider.Key {
     private final Label extensionLabel;
     private final String exportedName;
 

src/main/java/com/google/devtools/build/lib/pkgcache/BUILD

@@ -44,7 +44,6 @@ java_library(
         "//third_party:auto_value",
         "//third_party:guava",
         "//third_party:jsr305",
-        "//third_party/protobuf:protobuf_java",
     ],
 )
 

src/main/java/com/google/devtools/build/lib/pkgcache/FilteringPolicies.java

@@ -18,7 +18,6 @@
 import com.google.devtools.build.lib.packages.Rule;
 import com.google.devtools.build.lib.packages.Target;
 import com.google.devtools.build.lib.packages.TargetUtils;
-import com.google.devtools.build.lib.skyframe.serialization.autocodec.AutoCodec;
 import java.util.Objects;
 
 /**
@@ -108,7 +107,6 @@ public boolean shouldRetain(Target target, boolean explicit) {
 
   /** FilteringPolicy that only matches a specific rule name. */
   @AutoValue
-  @AutoCodec
   abstract static class RuleTypeFilter implements FilteringPolicy {
     abstract String ruleName();
 
@@ -127,8 +125,7 @@ public boolean shouldRetain(Target target, boolean explicit) {
       return false;
     }
 
-    @AutoCodec.Instantiator
-    static RuleTypeFilter create(String ruleName, boolean keepExplicit) {
+    private static RuleTypeFilter create(String ruleName, boolean keepExplicit) {
       return new AutoValue_FilteringPolicies_RuleTypeFilter(ruleName, keepExplicit);
     }
   }

src/main/java/com/google/devtools/build/lib/rules/BUILD

@@ -167,7 +167,6 @@ java_library(
         "//src/main/java/com/google/devtools/build/lib/concurrent",
         "//src/main/java/com/google/devtools/build/lib/packages",
         "//src/main/java/com/google/devtools/build/lib/skyframe:build_configuration",
-        "//src/main/java/com/google/devtools/build/lib/skyframe/serialization/autocodec",
         "//src/main/java/com/google/devtools/build/lib/util:filetype",
         "//src/main/java/net/starlark/java/eval",
         "//third_party:guava",