[code-simplifier] refactor: simplify recent code changes for clarity and idiomaticity

pkg/cli/audit_report_analysis.go

@@ -105,7 +105,10 @@ func generateFindings(processedRun ProcessedRun, metrics MetricsData, errors []E
 
 	// Missing tool findings
 	if len(processedRun.MissingTools) > 0 {
-		toolNames := sliceutil.Map(processedRun.MissingTools[:min(3, len(processedRun.MissingTools))], func(t MissingToolReport) string { return t.Tool })
+		toolNames := sliceutil.Map(
+			processedRun.MissingTools[:min(3, len(processedRun.MissingTools))],
+			func(t MissingToolReport) string { return t.Tool },
+		)
 		desc := "Missing tools: " + strings.Join(toolNames, ", ")
 		if len(processedRun.MissingTools) > 3 {
 			desc += fmt.Sprintf(" (and %d more)", len(processedRun.MissingTools)-3)

pkg/fileutil/tar.go

@@ -3,6 +3,7 @@ package fileutil
 import (
 	"archive/tar"
 	"bytes"
+	"errors"
 	"fmt"
 	"io"
 
@@ -20,7 +21,7 @@ func ExtractFileFromTar(data []byte, path string) ([]byte, error) {
 	entriesScanned := 0
 	for {
 		header, err := tr.Next()
-		if err == io.EOF {
+		if errors.Is(err, io.EOF) {
 			tarLog.Printf("File not found in tar archive after scanning %d entries: %s", entriesScanned, path)
 			break
 		}

pkg/parser/schema_utilities.go

@@ -29,8 +29,7 @@ func filterIgnoredFields(frontmatter map[string]any) map[string]any {
 	filtered := make(map[string]any)
 	for key, value := range frontmatter {
 		// Skip ignored fields
-		ignored := slices.Contains(constants.IgnoredFrontmatterFields, key)
-		if ignored {
+		if slices.Contains(constants.IgnoredFrontmatterFields, key) {
 			schemaUtilitiesLog.Printf("Removing ignored frontmatter field: %s", key)
 		} else {
 			filtered[key] = value