Use RuleConfiguredTarget#getActions everywhere instead of accessing the actions field to piggyback on the null check.

PiperOrigin-RevId: 709344848
Change-Id: Ie9bf68b8a68c6ca0dabe90be03871430bd21e214

Author: jin

src/main/java/com/google/devtools/build/lib/analysis/configuredtargets/RuleConfiguredTarget.java

@@ -82,9 +82,12 @@ public final class RuleConfiguredTarget extends AbstractConfiguredTarget {
   private final RuleClassId ruleClassId;
 
   /**
-   * Operations accessing actions, for example, executing them should be performed in the same Bazel
-   * instance that constructs the {@code RuleConfiguredTarget} instance and not on a Bazel instance
-   * that retrieves it remotely using deserialization.
+   * Operations accessing actions (for example, executing them or looking up metadata in them)
+   * should be performed in the same Bazel instance that constructs the {@code RuleConfiguredTarget}
+   * instance and not on a Bazel instance that retrieves the {@code RuleConfiguredTarget} remotely
+   * using deserialization, because actions will be null then.
+   *
+   * <p>Prefer using {@link #getActions()} which guards against null actions with a clear error.
    */
   @Nullable // Null if deserialized.
   private final transient ImmutableList<ActionAnalysisMetadata> actions;
@@ -254,7 +257,7 @@ protected Object rawGetStarlarkProvider(String providerKey) {
       // Only expose actions which are legitimate Starlark values, otherwise they will later
       // cause a Bazel crash.
       // TODO(cparsons): Expose all actions to Starlark.
-      return actions.stream()
+      return getActions().stream()
           .filter(action -> action instanceof ActionApi)
           .collect(ImmutableList.toImmutableList());
     }
@@ -299,7 +302,7 @@ public Artifact findArtifactByOutputLabel(Label outputLabel) {
         outputLabel,
         this);
     PathFragment relativeOutputPath = outputLabel.toPathFragment();
-    for (ActionAnalysisMetadata action : actions) {
+    for (ActionAnalysisMetadata action : getActions()) {
       for (Artifact output : action.getOutputs()) {
         if (output.getExecPath().endsWith(relativeOutputPath)) {
           return output;