Apply suggestions from code review

Co-authored-by: Sarah French <15078782+SarahFrench@users.noreply.github.com>

Author: dsa0x

internal/plans/plan.go

@@ -171,7 +171,7 @@ type Plan struct {
 	FunctionResults []lang.FunctionResultHash
 
 	// PolicyResults stores the results of policy evaluations that were performed
-	// while making this plan.
+	// while making this plan. These policy results will not be serialized in the plan file.
 	PolicyResults *PolicyResults
 }
 

internal/terraform/context_apply.go

@@ -49,11 +49,13 @@ type ApplyOpts struct {
 	// the actual root modules.
 	AllowRootEphemeralOutputs bool
 
-	// Locks is a read-only snapshot of provider locks (from the dependency lock
-	// file).
-	Locks map[addrs.Provider]*depsfile.ProviderLock
+	// Locks is a read-only snapshot of provider locks (from the dependency lock
+	// file). This is required by policy evaluations against providers to access version information.
+	Locks map[addrs.Provider]*depsfile.ProviderLock
 
-	// Optional policy client to enable live policy evaluations.
+	// Optional policy client.
+	// When set, policy evaluation logic will be executed in the graph.
+	// When nil, that logic will be skipped.
 	PolicyClient  policy.Client
 	PolicyResults *plans.PolicyResults
 }