stacklok / toolhive / 25228591736

65%

Build Type

push

github

Committed by web-flow

Commit Message

Fix golangci-lint failures from Go 1.26 linter upgrade (#5161)

- Replace deprecated reflect.Ptr with reflect.Pointer in four files
  (govet inline check now enforces this)
- Exclude gosec G710 (open redirect) globally; it fires on an
  intentional OAuth test mock with no attack surface
- Configure goconst to ignore test files and require 25+ occurrences
  before flagging, and suppress reporting in test/ and deploy/ paths;
  avoids touching test files while keeping the linter meaningful
- Extract httpTransportLabel and skillsDirName constants in
  pkg/client/config.go and use the existing defaultURLFieldName
  constant, replacing 130+ repeated string literals that triggered
  the raised goconst threshold

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Coverage Stats

2 of 2 new or added lines in 2 files covered. (100.0%)

7 existing lines in 2 files now uncovered.

61859 of 95862 relevant lines covered (64.53%)

59.76 hits per line