[Code Quality] Fix leftover golint findings: cmd/gh-aw/main.go, stringsconcatloop, actionpins_internal_test

[github-actions]

Description

The Daily lint scan report - 2026-08-02 identified 5 non-shared golint findings that were explicitly not assigned to any of the created execution issues (#49253, #49254) because they fall outside the shared function-length refactoring backlog (#49251/#49252).

Suggested Changes

• cmd/gh-aw/main.go: address 3 largefunc findings by extracting logical sub-steps into smaller helper functions.
• pkg/linters/stringsconcatloop/stringsconcatloop.go: address 1 largefunc finding the same way.
• pkg/actionpins/actionpins_internal_test.go: fix 1 map-set lint finding (likely a gocritic/ineffassign-style map literal or set-construction issue reported by make golint-custom).

Run make golint-custom locally, filter lint-diagnostics.txt for these three files, and resolve each reported diagnostic without changing external behavior.

Files Affected

• cmd/gh-aw/main.go
• pkg/linters/stringsconcatloop/stringsconcatloop.go
• pkg/actionpins/actionpins_internal_test.go

Success Criteria

• make golint-custom reports zero findings for these three files.
• All existing tests continue to pass (go test ./cmd/... ./pkg/linters/... ./pkg/actionpins/...).
• No behavior change; only structural/lint cleanup.

Source

Extracted from Daily lint scan report - 2026-08-02 (listed as "Non-shared leftovers (not assigned in this run)").

Priority

Low-Medium - small, well-scoped lint cleanup; not blocking.

🔍 Task mining by Discussion Task Miner - Code Quality Improvement Agent · auto · 51.6 AIC · ⌖ 11.4 AIC · ⊞ 10.3K · ◷

• expires on Aug 2, 2026, 11:41 PM UTC-08:00