Upgraded golangci-lint to v2.4.0

.github/workflows/lint.yml

@@ -7,7 +7,7 @@ on:
   pull_request:
   workflow_dispatch:
 env:
-  GOLANGCI_LINT_VERSION: v2.1.6
+  GOLANGCI_LINT_VERSION: v2.4.0
   GO_VERSION: '1.25'
 jobs:
   golangci:

.golangci.yml

@@ -45,6 +45,10 @@ linters:
     - iface
     - gosmopolitan
     - funcorder
+
+    # introduced v2.4.0
+    - noinlineerr
+    - wsl_v5
   settings:
     errcheck:
       check-type-assertions: false
@@ -117,6 +121,10 @@ linters:
         - name: if-return
         - name: increment-decrement
         - name: var-naming
+          arguments:
+            - [] # AllowList
+            - [] # DenyList
+            - - skip-package-name-checks: true
         - name: var-declaration
         - name: package-comments
         - name: range

internal/pool/pool_test.go

@@ -41,6 +41,7 @@ type (
 	}
 	testWaitChPool struct {
 		xsync.Pool[chan *testItem]
+
 		testHookGetWaitCh func()
 	}
 )

internal/query/client.go

@@ -128,6 +128,7 @@ func (c *Client) FetchScriptResults(ctx context.Context,
 
 type executeScriptSettings struct {
 	executeSettings
+
 	ttl             time.Duration
 	operationParams *Ydb_Operations.OperationParams
 }

internal/query/options/retry.go

@@ -33,6 +33,7 @@ type (
 
 	doTxSettings struct {
 		doSettings
+
 		txSettings tx.Settings
 	}
 

internal/query/result_set.go

@@ -40,6 +40,7 @@ type (
 	}
 	resultSetWithClose struct {
 		*resultSet
+
 		close func(ctx context.Context) error
 	}
 )

internal/topic/configs.go

@@ -9,6 +9,7 @@ import (
 
 type Config struct {
 	config.Common
+
 	Trace              *trace.Topic
 	MaxGrpcMessageSize int
 }

internal/topic/topiclistenerinternal/partition_worker_test.go

@@ -28,6 +28,7 @@ import (
 // syncMessageSender extends mockMessageSender with synchronization capabilities
 type syncMessageSender struct {
 	*mockMessageSender
+
 	messageReceived empty.Chan
 }
 

internal/topic/topicmock/grpc_topic_mock.go

@@ -89,6 +89,7 @@ func newGrpcMock(listener net.Listener, topicServiceImpl Ydb_Topic_V1.TopicServi
 
 type mockDiscoveryService struct {
 	Ydb_Discovery_V1.UnimplementedDiscoveryServiceServer
+
 	host string
 	port uint32
 }

internal/topic/topicreaderinternal/reader.go

@@ -248,10 +248,10 @@ func (r *Reader) CommitRanges(ctx context.Context, ranges []topicreadercommon.Pu
 
 type ReaderConfig struct {
 	config.Common
+	topicStreamReaderConfig
 
 	RetrySettings      topic.RetrySettings
 	DefaultBatchConfig ReadMessageBatchOptions
-	topicStreamReaderConfig
 }
 
 type PublicReaderOption func(cfg *ReaderConfig)

internal/topic/topicreaderinternal/transaction_mock_test.go

@@ -18,6 +18,7 @@ func newMockTransactionWrapper(sessionID, transactinID string) *mockTransaction
 
 type mockTransaction struct {
 	tx.Identifier
+
 	materializedID tx.Identifier
 	materialized   bool
 	sessionID      string

internal/xerrors/stacktrace.go

@@ -62,6 +62,7 @@ func (e *stackError) Unwrap() error {
 
 type stackTransportError struct {
 	stackError
+
 	status *grpcStatus.Status
 }
 

internal/xresolver/xresolver.go

@@ -12,12 +12,14 @@ import (
 
 type dnsBuilder struct {
 	resolver.Builder
+
 	scheme string
 	trace  *trace.Driver
 }
 
 type clientConn struct {
 	resolver.ClientConn
+
 	target resolver.Target
 	trace  *trace.Driver
 }

operation/client.go

@@ -26,6 +26,7 @@ type (
 	// Experimental: https://github.com/ydb-platform/ydb-go-sdk/blob/master/VERSIONING.md#experimental
 	listOperationsWithNextToken[PT metadata.Constraint[T], T metadata.TypesConstraint] struct {
 		listOperations[PT, T]
+
 		NextToken string
 	}
 	// Experimental: https://github.com/ydb-platform/ydb-go-sdk/blob/master/VERSIONING.md#experimental
@@ -41,6 +42,7 @@ type (
 	// Experimental: https://github.com/ydb-platform/ydb-go-sdk/blob/master/VERSIONING.md#experimental
 	typedOperation[PT metadata.Constraint[T], T metadata.TypesConstraint] struct {
 		operation
+
 		Metadata *T
 	}
 )

pkg/xtest/grpclogger.go

@@ -68,6 +68,7 @@ func (l GrpcLogger) StreamClientInterceptor(
 
 type grpcLoggerStream struct {
 	grpc.ClientStream
+
 	streamID int64
 	t        testing.TB
 }

pkg/xtest/logger.go

@@ -12,8 +12,9 @@ func MakeSyncedTest(t *testing.T) *SyncedTest {
 }
 
 type SyncedTest struct {
-	m sync.Mutex
 	*testing.T
+
+	m sync.Mutex
 }
 
 func (s *SyncedTest) Cleanup(f func()) {

scheme/scheme.go

@@ -20,6 +20,7 @@ type EntryType uint
 
 type Directory struct {
 	Entry
+
 	Children []Entry
 }
 

topic/topicsugar/iterators.go

@@ -136,6 +136,7 @@ func IteratorFunc[T any](
 
 type TypedTopicMessage[T any] struct {
 	*topicreader.Message
+
 	Data T
 }