fix: reset policy evaluations on repeated additions (#1732)

Signed-off-by: Miguel Martinez <[email protected]>

Author: migmartri

app/cli/cmd/attestation_status.go

@@ -17,6 +17,8 @@ package cmd
 
 import (
 	"fmt"
+	"slices"
+	"strings"
 	"time"
 
 	"github.com/jedib0t/go-pretty/v6/table"
@@ -173,6 +175,11 @@ func materialsTable(status *action.AttestationStatusResult, full bool) error {
 		return nil
 	}
 
+	// Sort materials by name for consistent output
+	slices.SortFunc(status.Materials, func(a, b action.AttestationStatusMaterial) int {
+		return strings.Compare(a.Name, b.Name)
+	})
+
 	mt := newTableWriter()
 	mt.SetTitle("Materials")
 

pkg/attestation/crafter/crafter.go

@@ -21,6 +21,7 @@ import (
 	"fmt"
 	"net/url"
 	"os"
+	"slices"
 	"strings"
 	"time"
 
@@ -558,6 +559,12 @@ func (c *Crafter) addMaterial(ctx context.Context, m *schemaapi.CraftingSchema_M
 		return nil, fmt.Errorf("validation error: %w", err)
 	}
 
+	// Remove existing policy evaluations for this material
+	// since the value might have changed
+	c.CraftingState.Attestation.PolicyEvaluations = slices.DeleteFunc(c.CraftingState.Attestation.PolicyEvaluations, func(i *api.PolicyEvaluation) bool {
+		return i.MaterialName == m.Name
+	})
+
 	// Validate policy groups
 	pgv := policies.NewPolicyGroupVerifier(c.CraftingState.InputSchema, c.attClient, c.Logger)
 	policyGroupResults, err := pgv.VerifyMaterial(ctx, mt, value)