[log] Add debug logging to five pkg/linters analyzers

[github-actions]

Overview

Adds namespaced debug logging (via pkg/logger) to five pkg/linters analyzers, improving observability without changing analyzer behavior or public API.

Changes

Each of the following files gains a package-level logger (logger.New("linters:<name>")) and Printf calls at analysis entry and at each diagnostic-reporting site:

• pkg/linters/seenmapbool/seenmapbool.go — logs package analysis start and each map[string]bool-as-set flag.
• pkg/linters/sortslice/sortslice.go — logs package analysis start and each sort.Slice/sort.SliceStable flag.
• pkg/linters/stringsconcatloop/stringsconcatloop.go — logs package analysis start and each string-concatenation-in-loop flag.
• pkg/linters/tolowerequalfold/tolowerequalfold.go — logs package analysis start and each case-insensitive comparison flag.
• pkg/linters/trimleftright/trimleftright.go — logs package analysis start and each suspicious strings.TrimLeft/TrimRight flag.

Impact

• Non-breaking: no changes to analyzer names, diagnostics, or public APIs.
• Debug output is gated by the DEBUG env var per repo convention (e.g. DEBUG=linters:* or DEBUG=*), so default behavior is unaffected.
• Enables targeted troubleshooting of these five analyzers during gh aw runs or local go vet/golangci-lint-style invocations.

Generated by PR Description Updater for #49707 · auto · 44.3 AIC · ⌖ 5.43 AIC · ⊞ 6.9K · ◷

[github-actions]

Hey @github-actions[bot] 👋 — thanks for adding debug logging to the linters! This is a focused and consistent set of changes across five analyzers.

A few items to help finalize this:

• Tests — while these are side-effect-free logging additions, the PR should include unit tests exercising the log output or at minimum verify the changed files compile and run with the new logger calls. The test plan notes the sandbox could not run make build due to Go version constraints — consider running this locally or in a fresh environment to confirm all changes pass linting and tests.
• Complete test plan — the test plan checklist is currently unchecked. Please verify or update the CI status once the Go version constraint is resolved.

Once tests are confirmed passing, this PR should be ready for review.

Verify that pkg/linters changes compile and pass unit tests:
1. Ensure all five modified files (seenmapbool, sortslice, stringsconcatloop, tolowerequalfold, trimleftright) build without errors
2. Run the linters package tests to confirm no regressions
3. Spot-check one or two of the new log statements are reachable and execute correctly
Report success/failure of make build and make test commands.

Generated by ✅ Contribution Check · auto · 46.6 AIC · ⌖ 8.19 AIC · ⊞ 8.7K · ◷

[github-actions]

🎉 This pull request is included in a new release.

Release: v0.84.3