From f92cef4e5454fbe6cd4bb9792b731404ccec4337 Mon Sep 17 00:00:00 2001 From: Copilot Date: Thu, 5 Mar 2026 19:08:38 +0000 Subject: [PATCH 1/2] refactor: simplify recent code changes for clarity and idiomaticity - pkg/fileutil/tar.go: use errors.Is(err, io.EOF) instead of == comparison (errorlint linter requires idiomatic error comparison) - pkg/parser/schema_utilities.go: inline single-use 'ignored' variable to reduce unnecessary intermediate assignment - pkg/cli/audit_report_analysis.go: split long sliceutil.Map call onto multiple lines for improved readability Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- pkg/cli/audit_report_analysis.go | 5 ++++- pkg/fileutil/tar.go | 3 ++- pkg/parser/schema_utilities.go | 3 +-- 3 files changed, 7 insertions(+), 4 deletions(-) diff --git a/pkg/cli/audit_report_analysis.go b/pkg/cli/audit_report_analysis.go index b47f41941b9..da14929c25e 100644 --- a/pkg/cli/audit_report_analysis.go +++ b/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) diff --git a/pkg/fileutil/tar.go b/pkg/fileutil/tar.go index 601aa064722..7f6be73bfab 100644 --- a/pkg/fileutil/tar.go +++ b/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 } diff --git a/pkg/parser/schema_utilities.go b/pkg/parser/schema_utilities.go index 18dda1cf608..1b9f10ff8a9 100644 --- a/pkg/parser/schema_utilities.go +++ b/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 From e7dbe205536b2f1e622a99a99cc4c244ca592d9b Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" Date: Thu, 5 Mar 2026 19:10:35 +0000 Subject: [PATCH 2/2] ci: trigger checks