fix: resolve lint and staticcheck CI failures

Fix errcheck on Close() calls, remove unused func
firstWordLower, simplify loop to append spread,
and add staticcheck directive for deprecated toml API.

Author: Patel230

internal/compress/caveman_safety.go

@@ -221,15 +221,6 @@ func isUpperLetter(b byte) bool {
 	return b >= 'A' && b <= 'Z'
 }
 
-// firstWordLower returns the first whitespace-delimited word of s,
-// lower-cased. Empty if s has no word.
-func firstWordLower(s string) string {
-	fields := strings.Fields(s)
-	if len(fields) == 0 {
-		return ""
-	}
-	return strings.ToLower(fields[0])
-}
 
 // isCJK reports whether s is primarily CJK characters (Chinese/Japanese/Korean).
 // Used to decide if the caveman rules even apply — CJK text doesn't have

internal/filter/toml_filter.go

@@ -4,6 +4,9 @@
 // command output before it enters the main compression layers. Each
 // filter is defined as a TOML section with these stages:
 //
+// staticcheck:ignore SA1019 — toml.PrimitiveDecode is deprecated but has
+// no direct replacement in the current go-toml API for this decode pattern.
+//
 //  1. match_command   - regex that selects this filter for a given command
 //  2. strip_ansi      - bool, strip ANSI escape sequences
 //  3. replace         - array of {pattern, replacement} regex pairs
@@ -109,11 +112,13 @@ func LoadTOMLFilterFile(path string) (*TOMLFilterFile, error) {
 		return nil, fmt.Errorf("toml_filter: decode %s: %w", path, err)
 	}
 	if v, ok := raw["schema_version"]; ok {
+		//nolint:staticcheck // PrimitiveDecode is deprecated but still the simplest API for this use case
 		_ = toml.PrimitiveDecode(v, &f.Schema)
 		delete(raw, "schema_version")
 	}
 	for name, prim := range raw {
 		cfg := &TOMLFilterConfig{Name: name}
+		//nolint:staticcheck // PrimitiveDecode is deprecated but still the simplest API for this use case
 		if err := toml.PrimitiveDecode(prim, cfg); err != nil {
 			return nil, fmt.Errorf("toml_filter: decode section [%s] in %s: %w", name, path, err)
 		}
@@ -417,9 +422,7 @@ func readDir(dir string) ([]string, error) {
 		return nil, err
 	}
 	names := make([]string, 0, len(entries))
-	for _, e := range entries {
-		names = append(names, e)
-	}
+	names = append(names, entries...)
 	return names, nil
 }
 

internal/tracking/tracking.go

@@ -236,7 +236,7 @@ func (t *Tracker) Recent(ctx context.Context, n int) ([]Event, error) {
 	if err != nil {
 		return nil, fmt.Errorf("tracking: recent: %w", err)
 	}
-	defer rows.Close()
+	defer func() { _ = rows.Close() }()
 	var out []Event
 	for rows.Next() {
 		var ev Event

tracker_test.go

@@ -15,7 +15,7 @@ func TestNewTracker_AtCustomPath(t *testing.T) {
 	if err != nil {
 		t.Fatalf("NewTrackerAt: %v", err)
 	}
-	defer tr.Close()
+	defer func() { _ = tr.Close() }()
 
 	ctx := context.Background()
 	if err := tr.Record(ctx, tok.TrackerEvent{