fix(lint): fix gci import ordering and nolint opaque for staging factory functions

Author: whites11

flow/connectors/clickhouse/clickhouse.go

@@ -14,18 +14,18 @@ import (
 	"strings"
 	"time"
 
-	chproto "github.com/ClickHouse/ch-go/proto"
-	"github.com/ClickHouse/clickhouse-go/v2"
-	"github.com/ClickHouse/clickhouse-go/v2/lib/driver"
-	clickhouseproto "github.com/ClickHouse/clickhouse-go/v2/lib/proto"
-	"go.temporal.io/sdk/log"
-
 	metadataStore "github.com/PeerDB-io/peerdb/flow/connectors/external_metadata"
 	"github.com/PeerDB-io/peerdb/flow/generated/protos"
 	"github.com/PeerDB-io/peerdb/flow/internal"
 	peerdb_clickhouse "github.com/PeerDB-io/peerdb/flow/pkg/clickhouse"
 	"github.com/PeerDB-io/peerdb/flow/shared"
 	"github.com/PeerDB-io/peerdb/flow/shared/types"
+
+	chproto "github.com/ClickHouse/ch-go/proto"
+	"github.com/ClickHouse/clickhouse-go/v2"
+	"github.com/ClickHouse/clickhouse-go/v2/lib/driver"
+	clickhouseproto "github.com/ClickHouse/clickhouse-go/v2/lib/proto"
+	"go.temporal.io/sdk/log"
 )
 
 type ClickHouseConnector struct {

flow/connectors/clickhouse/staging.go

@@ -49,6 +49,7 @@ func createStagingStore(
 	}
 }
 
+//nolint:iface // factory function intentionally returns interface
 func newS3StagingStoreFromConfig(
 	ctx context.Context,
 	config *protos.ClickhouseConfig,
@@ -108,6 +109,7 @@ func newS3StagingStoreFromConfig(
 	return newS3StagingStore(awsBucketPath, credentialsProvider)
 }
 
+//nolint:iface // factory function intentionally returns interface
 func newGCSStagingStore(ctx context.Context, bucketName string) (StagingStore, error) {
 	if bucketName == "" {
 		return nil, errors.New("PEERDB_CLICKHOUSE_STAGING_BUCKET_NAME must be set when staging provider is gcs")