feat: added .golangci.yml config file

Author: ralvarezdev

.golangci.yml

@@ -0,0 +1,278 @@
+# yaml-language-server: $schema=https://golangci-lint.run/jsonschema/golangci.jsonschema.json
+# golangci-lint configuration for github.com/ralvarezdev/go-loader
+# Based on v2.5.0 best practices
+# Last updated: 2025-10-21
+
+version: "2"
+
+run:
+  timeout: 5m
+  concurrency: 0  # Auto-detect CPU cores
+  tests: true
+  modules-download-mode: readonly
+  go: '1.25.1'  # Match your project's Go version
+  relative-path-mode: gomod
+
+output:
+  formats:
+    text:
+      path: stdout
+      print-linter-name: true
+      colors: true
+  sort-order:
+    - severity
+    - linter
+    - file
+  show-stats: true
+
+formatters:
+  enable:
+    - goimports
+    - golines
+  
+  settings:
+    goimports:
+      local-prefixes:
+        - github.com/ralvarezdev/go-loader
+    
+    golines:
+      max-len: 120
+      shorten-comments: true
+  
+  exclusions:
+    generated: lax
+    paths:
+      - "vendor/"
+      - "third_party/"
+
+linters:
+  enable:
+    # Essential Linters
+    - govet
+    - staticcheck
+    - unused
+    - goconst
+    - godoclint
+    - sloglint
+    - misspell
+    
+    # Security Linters
+    - gosec
+    - bidichk
+    
+    # Resource Management
+    - bodyclose
+    - sqlclosecheck
+    - rowserrcheck
+    
+    # Error Handling Linters
+    - errorlint
+    - errcheck
+    - nilerr
+    - nilnil
+    
+    # Code Quality
+    - dupl
+    - nestif
+    - ineffassign
+    - revive
+    - gocritic
+    - unconvert
+    - wastedassign
+    - usestdlibvars
+    
+    # Context & Concurrency
+    - contextcheck
+    - noctx
+    
+    # OPTIONAL - Testing (if using testify)
+    # - testifylint
+    # - thelper
+
+  settings:
+    govet:
+      enable:
+        - shadow
+        # - fieldalignment  # Enable for performance-critical code
+      settings:
+        shadow:
+          strict: true
+    
+    errcheck:
+      check-type-assertions: true
+      check-blank: true
+      exclude-functions:
+        - io.Copy(*bytes.Buffer)
+        - io.Copy(os.Stdout)
+        - (*net/http.ResponseWriter).Write
+    
+    staticcheck:
+      checks:
+        - "all"
+        - "-SA1019"  # Deprecation warnings (noisy during refactoring)
+        - "-ST1000"  # Package comment requirements
+        - "-ST1003"  # Underscore naming (conflicts with protobuf)
+        - "-ST1016"  # Receiver naming rules
+    
+    goconst:
+      min-len: 3
+      min-occurrences: 3
+      ignore-string-values:
+        - 'http.*'
+        - '^test.*'
+        - '.*\.proto$'
+    
+    godoclint:
+      default: basic
+      disable:
+        - require-pkg-doc
+    
+    sloglint:
+      no-global: "all"
+      static-msg: true
+      key-naming-case: "snake"
+      forbidden-keys:
+        - time
+        - level
+        - msg
+        - source
+    
+    misspell:
+      locale: US
+      mode: restricted
+      ignore-rules:
+        - referer
+        - kubernetes
+        - grpc
+        - protobuf
+    
+    dupl:
+      threshold: 100
+    
+    nestif:
+      min-complexity: 4
+    
+    nilnil:
+      only-two: true
+      detect-opposite: true
+    
+    gosec:
+      excludes:
+        - G104  # Duplicate of errcheck
+      includes:
+        - G201  # SQL injection
+        - G202  # SQL injection
+        - G204  # Command injection
+        - G301  # File permissions
+        - G302  # File permissions
+        - G304  # File inclusion
+        - G401  # Weak crypto
+        - G402  # TLS InsecureSkipVerify
+    
+    errorlint:
+      errorf: true
+      asserts: true
+      comparison: true
+    
+    revive:
+      rules:
+        # Error handling
+        - name: error-return
+        - name: error-strings
+        - name: error-naming
+        
+        # Code quality
+        - name: if-return
+        - name: increment-decrement
+        - name: indent-error-flow
+        - name: superfluous-else
+        - name: unreachable-code
+        
+        # Style
+        - name: var-naming
+        - name: range
+        - name: receiver-naming
+        - name: blank-imports
+        - name: context-as-argument
+        - name: dot-imports
+        - name: empty-block
+        
+        # Disable noisy rules
+        - name: exported
+          disabled: true
+        - name: package-comments
+          disabled: true
+        - name: unused-parameter
+          disabled: true
+    
+    gocritic:
+      enabled-tags:
+        - diagnostic
+        - style
+        - performance
+      disabled-checks:
+        - commentFormatting
+        - whyNoLint
+    
+  exclusions:
+    warn-unused: true
+      
+    presets:
+      - common-false-positives
+      - std-error-handling
+      
+    rules:
+      # Test file exclusions
+      - path: '_test\.go'
+        linters:
+          - dupl
+          - errcheck
+          - nestif
+          - goconst
+          - gosec
+          - funlen
+          - gocyclo
+        
+      # Generated code
+      - path: '\.pb\.go$'
+        linters:
+          - all
+        
+      - path: 'mock_.+\.go$'
+        linters:
+          - all
+          
+      # Long lines in go:generate
+      - linters:
+          - lll
+        source: '^//go:generate '
+        
+      # SQL linter false positives for repository pattern
+      - text: "Rows/Stmt was not closed"
+        path: _repository\.go$
+        linters:
+          - sqlclosecheck
+  
+issues:
+  max-issues-per-linter: 0
+  max-same-issues: 50
+  uniq-by-line: true
+    
+  # For gradual adoption on legacy codebases
+  # new-from-rev: HEAD
+  # new-from-merge-base: main
+    
+severity:
+  default: error
+  
+  rules:
+    # Informational linters
+    - linters:
+        - dupl
+        - goconst
+      severity: info
+    
+    # Security linters use their own severity
+    - linters:
+        - gosec
+      severity: "@linter"

filesystem/read_files.go

@@ -75,9 +75,9 @@ func CloseFile(file *os.File) error {
 //   - []string: A slice of strings containing the headers (if readHeaders is true)
 //   - error: An error if something went wrong
 func ReadCSVFile(file *os.File, readHeaders bool) (
-	[][]string,
-	[]string,
-	error,
+	records [][]string,
+	headers []string,
+	err error,
 ) {
 	// Check if the file is nil
 	if file == nil {
@@ -86,17 +86,15 @@ func ReadCSVFile(file *os.File, readHeaders bool) (
 
 	// Defer closing the file
 	defer func(file *os.File) {
-		if err := CloseFile(file); err != nil {
-			fmt.Println(err)
+		if closeErr := CloseFile(file); closeErr != nil {
+			fmt.Println(closeErr)
 		}
 	}(file)
 
 	// Create a new CSV reader
 	reader := csv.NewReader(file)
 
 	// Read the header line
-	var headers []string
-	var err error
 	if readHeaders {
 		headers, err = reader.Read()
 		if err != nil {
@@ -105,7 +103,7 @@ func ReadCSVFile(file *os.File, readHeaders bool) (
 	}
 
 	// Read all records from the CSV file
-	records, err := reader.ReadAll()
+	records, err = reader.ReadAll()
 	if err != nil {
 		return nil, nil, fmt.Errorf("failed to read CSV file: %w", err)
 	}

go.mod

@@ -3,13 +3,13 @@ module github.com/ralvarezdev/go-loader
 go 1.24.0
 
 require (
-	golang.org/x/oauth2 v0.31.0
-	google.golang.org/api v0.251.0
-	google.golang.org/grpc v1.75.1
+	golang.org/x/oauth2 v0.32.0
+	google.golang.org/api v0.252.0
+	google.golang.org/grpc v1.76.0
 )
 
 require (
-	cloud.google.com/go/auth v0.16.5 // indirect
+	cloud.google.com/go/auth v0.17.0 // indirect
 	cloud.google.com/go/auth/oauth2adapt v0.2.8 // indirect
 	cloud.google.com/go/compute/metadata v0.9.0 // indirect
 	github.com/felixge/httpsnoop v1.0.4 // indirect
@@ -18,14 +18,15 @@ require (
 	github.com/google/s2a-go v0.1.9 // indirect
 	github.com/googleapis/enterprise-certificate-proxy v0.3.6 // indirect
 	github.com/googleapis/gax-go/v2 v2.15.0 // indirect
-	go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.61.0 // indirect
-	go.opentelemetry.io/otel v1.37.0 // indirect
-	go.opentelemetry.io/otel/metric v1.37.0 // indirect
-	go.opentelemetry.io/otel/trace v1.37.0 // indirect
-	golang.org/x/crypto v0.42.0 // indirect
-	golang.org/x/net v0.44.0 // indirect
-	golang.org/x/sys v0.36.0 // indirect
-	golang.org/x/text v0.29.0 // indirect
-	google.golang.org/genproto/googleapis/rpc v0.0.0-20251002232023-7c0ddcbb5797 // indirect
+	go.opentelemetry.io/auto/sdk v1.2.1 // indirect
+	go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.63.0 // indirect
+	go.opentelemetry.io/otel v1.38.0 // indirect
+	go.opentelemetry.io/otel/metric v1.38.0 // indirect
+	go.opentelemetry.io/otel/trace v1.38.0 // indirect
+	golang.org/x/crypto v0.43.0 // indirect
+	golang.org/x/net v0.46.0 // indirect
+	golang.org/x/sys v0.37.0 // indirect
+	golang.org/x/text v0.30.0 // indirect
+	google.golang.org/genproto/googleapis/rpc v0.0.0-20251020155222-88f65dc88635 // indirect
 	google.golang.org/protobuf v1.36.10 // indirect
 )