Don't populate plan.ActionTargetAddrs

internal/stacks/stackruntime/internal/stackeval/applying.go

@@ -6,7 +6,6 @@ package stackeval
 import (
 	"context"
 	"fmt"
-	"log"
 
 	"github.com/hashicorp/hcl/v2"
 
@@ -175,7 +174,6 @@ func ApplyComponentPlan(ctx context.Context, main *Main, plan *plans.Plan, requi
 		hooks: hooksFromContext(ctx),
 		addr:  inst.Addr(),
 	}
-	log.Printf("[DEBUG] ApplyComponentPlan: Created hook for %s", inst.Addr())
 	tfCtx, err := terraform.NewContext(&terraform.ContextOpts{
 		Hooks: []terraform.Hook{
 			tfHook,
@@ -229,23 +227,9 @@ func ApplyComponentPlan(ctx context.Context, main *Main, plan *plans.Plan, requi
 		// works, and so code after this point should not make any further use
 		// of either "modifiedPlan" or "plan" (since they share lots of the same
 		// pointers to mutable objects and so both can get modified together.)
-
-		// Populate ActionTargetAddrs with all action invocations so they will
-		// be added to the apply graph by the ActionInvokeApplyTransformer
-		if len(plan.Changes.ActionInvocations) > 0 {
-			plan.ActionTargetAddrs = make([]addrs.Targetable, 0, len(plan.Changes.ActionInvocations))
-			for _, actionInvocation := range plan.Changes.ActionInvocations {
-				plan.ActionTargetAddrs = append(plan.ActionTargetAddrs, actionInvocation.Addr)
-			}
-			log.Printf("[DEBUG] ApplyComponentPlan: Populated ActionTargetAddrs with %d action invocations for %s", len(plan.ActionTargetAddrs), inst.Addr())
-		}
-
-		log.Printf("[DEBUG] ApplyComponentPlan: About to call tfCtx.Apply() for %s", inst.Addr())
 		newState, moreDiags = tfCtx.Apply(plan, moduleTree, &terraform.ApplyOpts{
-			ExternalProviders:         providerClients,
-			AllowRootEphemeralOutputs: false, // TODO(issues/37822): Enable this.
+			ExternalProviders: providerClients,
 		})
-		log.Printf("[DEBUG] ApplyComponentPlan: tfCtx.Apply() completed for %s", inst.Addr())
 		diags = diags.Append(moreDiags)
 	} else {
 		// For a non-applyable plan, we just skip trying to apply it altogether