fix(cli): document WithSkipPolicyEvaluation behavior

migmartri · migmartri · commit a5c4d4dd9e87 · 2026-02-19T22:06:55.000+01:00
Signed-off-by: Miguel Martinez &lt;miguel@chainloop.dev&gt;
diff --git a/app/cli/pkg/action/attestation_status.go b/app/cli/pkg/action/attestation_status.go
@@ -93,6 +93,9 @@ func NewAttestationStatus(cfg *AttestationStatusOpts) (*AttestationStatus, error
 	}, nil
 }
 
+// WithSkipPolicyEvaluation skips re-evaluating policies against the attestation statement.
+// Existing policy evaluations already stored in the crafting state (e.g. from material additions)
+// will still be included in the result.
 func WithSkipPolicyEvaluation() func(*AttestationStatus) {
 	return func(opts *AttestationStatus) {
 		opts.skipPolicyEvaluation = true

Original file line number	Diff line number	Diff line change
`@@ -93,6 +93,9 @@ func NewAttestationStatus(cfg AttestationStatusOpts) (AttestationStatus, error`
`93`	`93`	`}, nil`
`94`	`94`	`}`
`95`	`95`
	`96`	`+// WithSkipPolicyEvaluation skips re-evaluating policies against the attestation statement.`
	`97`	`+// Existing policy evaluations already stored in the crafting state (e.g. from material additions)`
	`98`	`+// will still be included in the result.`
`96`	`99`	`func WithSkipPolicyEvaluation() func(*AttestationStatus) {`
`97`	`100`	`return func(opts *AttestationStatus) {`
`98`	`101`	`opts.skipPolicyEvaluation = true`