Revert targetEngine rename in deployCore to avoid variable shadowing

Co-authored-by: Isaac

Author: shreyas-goenka

bundle/phases/deploy.go

@@ -97,31 +97,25 @@ func approvalForDeploy(ctx context.Context, b *bundle.Bundle, plan *deployplan.P
 	return approved, nil
 }
 
-func deployCore(ctx context.Context, b *bundle.Bundle, plan *deployplan.Plan, engine engine.EngineType) {
-	// Core mutators that CRUD resources and modify deployment state. These
-	// mutators need informed consent if they are potentially destructive.
+func deployCore(ctx context.Context, b *bundle.Bundle, plan *deployplan.Plan, targetEngine engine.EngineType) {
 	cmdio.LogString(ctx, "Deploying resources...")
 
-	if engine.IsDirect() {
+	if targetEngine.IsDirect() {
 		b.DeploymentBundle.Apply(ctx, b.WorkspaceClient(), plan, direct.MigrateMode(false))
 	} else {
 		bundle.ApplyContext(ctx, b, terraform.Apply())
 	}
 
-	// Even if deployment failed, there might be updates in state that we need
-	// to upload. For the filesystem-based state, this uploads the state file to
-	// the workspace. For the metadata service, this is a no-op since operation
-	// results are reported inline during deployment.
-	statemgmt.PushResourcesState(ctx, b, engine)
+	statemgmt.PushResourcesState(ctx, b, targetEngine)
 	if logdiag.HasError(ctx) {
 		return
 	}
 
 	bundle.ApplySeqContext(ctx, b,
-		statemgmt.Load(engine),
+		statemgmt.Load(targetEngine),
 		metadata.Compute(),
 		metadata.Upload(),
-		statemgmt.UploadStateForYamlSync(engine),
+		statemgmt.UploadStateForYamlSync(targetEngine),
 	)
 
 	if !logdiag.HasError(ctx) {