feat: added .golangci.yml config file and upgraded dependencies

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-databases
+# 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-databases
+    
+    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"

errors.go

@@ -1,4 +1,4 @@
-package go_databases
+package godatabases
 
 import "errors"
 

go.mod

@@ -6,7 +6,7 @@ require (
 	github.com/go-redis/redis/v8 v8.11.5
 	github.com/jackc/pgx/v5 v5.7.6
 	go.mongodb.org/mongo-driver v1.17.4
-	golang.org/x/net v0.44.0
+	golang.org/x/net v0.46.0
 	gorm.io/gorm v1.31.0
 )
 
@@ -19,13 +19,13 @@ require (
 	github.com/jackc/puddle/v2 v2.2.2 // indirect
 	github.com/jinzhu/inflection v1.0.0 // indirect
 	github.com/jinzhu/now v1.1.5 // indirect
-	github.com/klauspost/compress v1.18.0 // indirect
+	github.com/klauspost/compress v1.18.1 // indirect
 	github.com/montanaflynn/stats v0.7.1 // indirect
 	github.com/xdg-go/pbkdf2 v1.0.0 // indirect
 	github.com/xdg-go/scram v1.1.2 // indirect
 	github.com/xdg-go/stringprep v1.0.4 // indirect
 	github.com/youmark/pkcs8 v0.0.0-20240726163527-a2c0da244d78 // indirect
-	golang.org/x/crypto v0.42.0 // indirect
+	golang.org/x/crypto v0.43.0 // indirect
 	golang.org/x/sync v0.17.0 // indirect
-	golang.org/x/text v0.29.0 // indirect
+	golang.org/x/text v0.30.0 // indirect
 )

go.sum

@@ -33,6 +33,8 @@ github.com/klauspost/compress v1.16.7 h1:2mk3MPGNzKyxErAw8YaohYh69+pa4sIQSC0fPGC
 github.com/klauspost/compress v1.16.7/go.mod h1:ntbaceVETuRiXiv4DpjP66DpAtAGkEQskQzEyD//IeE=
 github.com/klauspost/compress v1.18.0 h1:c/Cqfb0r+Yi+JtIEq73FWXVkRonBlf0CRNYc8Zttxdo=
 github.com/klauspost/compress v1.18.0/go.mod h1:2Pp+KzxcywXVXMr50+X0Q/Lsb43OQHYWRCY2AiWywWQ=
+github.com/klauspost/compress v1.18.1 h1:bcSGx7UbpBqMChDtsF28Lw6v/G94LPrrbMbdC3JH2co=
+github.com/klauspost/compress v1.18.1/go.mod h1:ZQFFVG+MdnR0P+l6wpXgIL4NTtwiKIdBnrBd8Nrxr+0=
 github.com/montanaflynn/stats v0.7.1 h1:etflOAAHORrCC44V+aR6Ftzort912ZU+YLiSTuV8eaE=
 github.com/montanaflynn/stats v0.7.1/go.mod h1:etXPPgVO6n31NxCd9KQUMvCM+ve0ruNzt6R8Bnaayow=
 github.com/nxadm/tail v1.4.8 h1:nPr65rt6Y5JFSKQO7qToXr7pePgD6Gwiw05lkbyAQTE=
@@ -75,6 +77,8 @@ golang.org/x/crypto v0.40.0 h1:r4x+VvoG5Fm+eJcxMaY8CQM7Lb0l1lsmjGBQ6s8BfKM=
 golang.org/x/crypto v0.40.0/go.mod h1:Qr1vMER5WyS2dfPHAlsOj01wgLbsyWtFn/aY+5+ZdxY=
 golang.org/x/crypto v0.42.0 h1:chiH31gIWm57EkTXpwnqf8qeuMUi0yekh6mT2AvFlqI=
 golang.org/x/crypto v0.42.0/go.mod h1:4+rDnOTJhQCx2q7/j6rAN5XDw8kPjeaXEUR2eL94ix8=
+golang.org/x/crypto v0.43.0 h1:dduJYIi3A3KOfdGOHX8AVZ/jGiyPa3IbBozJ5kNuE04=
+golang.org/x/crypto v0.43.0/go.mod h1:BFbav4mRNlXJL4wNeejLpWxB7wMbc79PdRGhWKncxR0=
 golang.org/x/mod v0.6.0-dev.0.20220419223038-86c51ed26bb4/go.mod h1:jJ57K6gSWd91VN4djpZkiMVwK6gcyfeH4XE8wZrZaV4=
 golang.org/x/net v0.0.0-20190620200207-3b0461eec859/go.mod h1:z5CRVTTTmAJ677TzLLGU+0bjPO0LkuOLi4/5GtJWs/s=
 golang.org/x/net v0.0.0-20210226172049-e18ecbb05110/go.mod h1:m0MpNAwzfU5UDzcl9v0D8zg8gWTRqZa9RBIspLL5mdg=
@@ -83,6 +87,8 @@ golang.org/x/net v0.42.0 h1:jzkYrhi3YQWD6MLBJcsklgQsoAcw89EcZbJw8Z614hs=
 golang.org/x/net v0.42.0/go.mod h1:FF1RA5d3u7nAYA4z2TkclSCKh68eSXtiFwcWQpPXdt8=
 golang.org/x/net v0.44.0 h1:evd8IRDyfNBMBTTY5XRF1vaZlD+EmWx6x8PkhR04H/I=
 golang.org/x/net v0.44.0/go.mod h1:ECOoLqd5U3Lhyeyo/QDCEVQ4sNgYsqvCZ722XogGieY=
+golang.org/x/net v0.46.0 h1:giFlY12I07fugqwPuWJi68oOnpfqFnJIJzaIIm2JVV4=
+golang.org/x/net v0.46.0/go.mod h1:Q9BGdFy1y4nkUwiLvT5qtyhAnEHgnQ/zd8PfU6nc210=
 golang.org/x/sync v0.0.0-20190423024810-112230192c58/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
 golang.org/x/sync v0.0.0-20220722155255-886fb9371eb4/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
 golang.org/x/sync v0.16.0 h1:ycBJEhp9p4vXvUZNszeOq0kGTPghopOL8q0fq3vstxw=
@@ -106,6 +112,8 @@ golang.org/x/text v0.27.0 h1:4fGWRpyh641NLlecmyl4LOe6yDdfaYNrGb2zdfo4JV4=
 golang.org/x/text v0.27.0/go.mod h1:1D28KMCvyooCX9hBiosv5Tz/+YLxj0j7XhWjpSUF7CU=
 golang.org/x/text v0.29.0 h1:1neNs90w9YzJ9BocxfsQNHKuAT4pkghyXc4nhZ6sJvk=
 golang.org/x/text v0.29.0/go.mod h1:7MhJOA9CD2qZyOKYazxdYMF85OwPdEr9jTtBpO7ydH4=
+golang.org/x/text v0.30.0 h1:yznKA/E9zq54KzlzBEAWn1NXSQ8DIp/NYMy88xJjl4k=
+golang.org/x/text v0.30.0/go.mod h1:yDdHFIX9t+tORqspjENWgzaCVXgk0yYnYuSZ8UzzBVM=
 golang.org/x/tools v0.0.0-20180917221912-90fa682c2a6e/go.mod h1:n7NCudcB/nEzxVGmLbDWY5pfWTLqBcC2KZ6jyYvM4mQ=
 golang.org/x/tools v0.0.0-20191119224855-298f0cb1881e/go.mod h1:b+2E5dAYhXwXZwtnZ6UAqBI28+e2cm9otk0dWdXHAEo=
 golang.org/x/tools v0.1.12/go.mod h1:hNGJHUnrk76NpqgfD5Aqm5Crs+Hm0VOH/i9J2+nxYbc=

mongodb/collection.go

@@ -47,8 +47,8 @@ func (c Collection) CreateCollection(database *mongo.Database) (
 	collection = database.Collection(c.name)
 
 	// Create the indexes
-	if err = c.createIndexes(collection); err != nil {
-		return nil, err
+	if createErr := c.createIndexes(collection); createErr != nil {
+		return nil, createErr
 	}
 
 	return collection, nil

mongodb/connection.go

@@ -3,10 +3,11 @@ package mongodb
 import (
 	"sync"
 
-	godatabases "github.com/ralvarezdev/go-databases"
 	"go.mongodb.org/mongo-driver/mongo"
 	"go.mongodb.org/mongo-driver/mongo/options"
 	"golang.org/x/net/context"
+
+	godatabases "github.com/ralvarezdev/go-databases"
 )
 
 type (

mongodb/index.go

@@ -22,15 +22,15 @@ func NewFieldIndex(name string, order Order) *FieldIndex {
 // NewUniqueIndex creates a new unique field index model
 func NewUniqueIndex(fieldIndex FieldIndex, unique bool) *mongo.IndexModel {
 	return &mongo.IndexModel{
-		Keys:    bson.D{{fieldIndex.name, fieldIndex.order.OrderInt()}},
+		Keys:    bson.D{{Key: fieldIndex.name, Value: fieldIndex.order.OrderInt()}},
 		Options: options.Index().SetUnique(unique),
 	}
 }
 
 // NewTTLIndex creates a new TTL index model
 func NewTTLIndex(fieldName string, expireAfterSeconds int32) *mongo.IndexModel {
 	return &mongo.IndexModel{
-		Keys:    bson.D{{fieldName, 1}},
+		Keys:    bson.D{{Key: fieldName, Value: 1}},
 		Options: options.Index().SetExpireAfterSeconds(expireAfterSeconds),
 	}
 }