.golangci.yaml

version: "2"
linters:
  default: none
  enable:
    - asasalint
    - asciicheck
    - bidichk
    - bodyclose
    - canonicalheader
    - containedctx
    - contextcheck
    - durationcheck
    - decorder
    - dogsled
    - durationcheck
    # - err113
    - errcheck
    - errname
    - errorlint
    # - errchkjson
    - exhaustive
    - exptostd
    - fatcontext
    - forbidigo
    - forcetypeassert
    - funcorder
    - gocheckcompilerdirectives
    # - gochecknoinits
    - goconst
    - gocritic
    - gomodguard_v2
    - goprintffuncname
    - gosec
    - govet
    - importas
    - ineffassign
    # - intrange
    - loggercheck
    - makezero
    - mnd
    - musttag
    - nakedret
    - nestif
    - nilerr
    - nilnil
    # - nlreturn
    - noctx
    - nolintlint
    - nonamedreturns
    - nosprintfhostport
    - perfsprint
    - predeclared
    - promlinter
    - protogetter
    - reassign
    - revive
    # - recvcheck
    - rowserrcheck
    - sloglint
    - spancheck
    - sqlclosecheck
    - staticcheck
    # - tagalign
    # - tagliatelle # maybe we want this to force consistent config to ensure consistent tag alignment across the codebase
    - testableexamples
    - testifylint
    - tparallel
    - unconvert
    - unparam
    - unused
    - usestdlibvars
    - usetesting
    - wastedassign
    - whitespace
  settings:
    errcheck:
      check-type-assertions: true
    exhaustive:
      check:
        - switch
        - map
    goconst:
      # Map/composite-literal keys repeat by nature (log attribute names, test
      # tables); they are not extractable constants.
      ignore-map-keys: true
    gocritic:
      settings:
        captLocal:
          paramsOnly: false
        underef:
          skipRecvDeref: false
    gomodguard_v2:
      blocked:
        - module: github.com/golang/protobuf
          recommendations:
            - google.golang.org/protobuf
          reason: see https://developers.google.com/protocol-buffers/docs/reference/go/faq#modules
        - module: github.com/satori/go.uuid
          recommendations:
            - github.com/google/uuid
          reason: satori's package is not maintained
        - module: github.com/gofrs/uuid
          recommendations:
            - github.com/google/uuid
          reason: gofrs' package is not go module
    gosec:
      excludes:
        - G115
    govet:
      disable:
        - fieldalignment
      enable-all: true
      settings:
        shadow:
          strict: true
    mnd:
      ignored-functions:
        - os.Chmod
        - os.Mkdir
        - os.MkdirAll
        - os.OpenFile
        - os.WriteFile
        - prometheus.ExponentialBuckets
        - prometheus.ExponentialBucketsRange
        - prometheus.LinearBuckets
    nolintlint:
      require-explanation: true
      require-specific: true
    rowserrcheck:
      packages:
        - github.com/jackc/pgx/v5
    sloglint:
      kv-only: false
      no-mixed-args: true
      static-msg: true
      attr-only: true
      no-raw-keys: false
      msg-style: lowercased
      key-naming-case: snake
      forbidden-keys:
        - time
        - level
        - msg
        - source
        - request
        - req
        - auth
        - authorization
        - token
      args-on-sep-lines: true
    tagliatelle:
      case:
        rules:
          json: snake
          mapstructure: snake
    usetesting:
      context-background: true
      context-todo: true
  exclusions:
    generated: lax
    presets:
      - comments
      - common-false-positives
      - legacy
      - std-error-handling
    rules:
      - linters:
          - lll
        source: ^//\s*go:generate\s
      - linters:
          - godot
        source: (noinspection|TODO)
      - linters:
          - gocritic
        source: //noinspection
      - linters:
          - errorlint
        source: ^\s+if _, ok := err\.\([^.]+\.InternalError\); ok {
      - linters:
          - bodyclose
          - dupl
          - funlen
          - goconst
          - gosec
          - lll
          - noctx
          - wrapcheck
        path: _test\.go
      - linters:
          - govet
        text: 'shadow: declaration of "(err)" shadows declaration at'
      - linters:
          - goimport
        text: http://www.apache.org/licenses/LICENSE-2.0
      # otdfctl: defer refactoring-level lint fixes to follow-up
      - path: otdfctl/
        linters:
          - contextcheck
        text: should pass the context parameter
      - path: otdfctl/
        linters:
          - revive
        text: unused-parameter
      - path: otdfctl/
        linters:
          - revive
        text: unexported-return
      - path: otdfctl/
        linters:
          - revive
        text: var-naming
      - path: otdfctl/
        linters:
          - nolintlint
        text: "exhaustive"
      # goconst: CLI table-column headers and other presentation strings
      - path: otdfctl/
        linters:
          - goconst
      # goconst: integration-test fixtures that escape the _test.go rule above
      # only because of their filenames
      - path: service/entityresolution/integration/
        linters:
          - goconst
    paths:
      - .*\.pb\.go
      - .*\.pb\.gw.go
      - sdk/sdkconnect/  # generated connect wrappers; forward deprecated RPCs
      - otdfctl/tui/  # excluded during migration, matching original otdfctl lint config
      - third_party$
      - builtin$
issues:
  max-issues-per-linter: 0
  max-same-issues: 0
  uniq-by-line: true
formatters:
  enable:
    - gofumpt
  exclusions:
    generated: lax
    paths:
      - .*\.pb\.go
      - .*\.pb\.gw.go
      - sdk/sdkconnect/  # generated connect wrappers; forward deprecated RPCs
      - otdfctl/tui/
      - third_party$
      - builtin$