node/.golangci.yml at main · luxfi/node · GitHub

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
# https://golangci-lint.run/usage/configuration/
version: "2"

run:
  timeout: 10m

linters:
  default: none
  enable:
    - govet
    - ineffassign
    - staticcheck
    # Note: errcheck and unused disabled until codebase is cleaned up
    # - errcheck
    # - unused
  exclusions:
    # Use lax mode for generated files - excludes files with "autogenerated", "code generated", etc.
    generated: lax
    # Preset exclusions for common false positives
    presets:
      - comments
      - std-error-handling
    # Path patterns to exclude from linting
    paths:
      - ".*\\.pb\\.go$"
      - ".*_mock\\.go$"
      - ".*mock.*\\.go$"
      - "third_party/"
      - "testdata/"
      - "examples/"
      - "Godeps/"
      - "builtin/"
      - "vendor/"
    # Per-linter exclusion rules
    rules:
      # Ignore staticcheck deprecation warnings (too many in codebase)
      - linters:
          - staticcheck
        text: "SA1019:"
      # Ignore staticcheck quickfix suggestions (not errors)
      - linters:
          - staticcheck
        text: "QF"
      # Ignore staticcheck empty branch (common in benchmarks)
      - linters:
          - staticcheck
        text: "SA9003:"
      # Ignore unused append results (common pattern)
      - linters:
          - staticcheck
        text: "SA4010:"
      # Ignore nil context warnings (legacy code)
      - linters:
          - staticcheck
        text: "SA1012:"
      # Ignore efficiency suggestions (not errors)
      - linters:
          - staticcheck
        text: "SA6001:"
      # Ignore loop replacement suggestions (not errors)
      - linters:
          - staticcheck
        text: "S1011:"
      # Ignore unconditionally terminated loop (design patterns)
      - linters:
          - staticcheck
        text: "SA4004:"
      # Ignore duplicate imports (aliasing is intentional)
      - linters:
          - staticcheck
        text: "ST1019"
      # Ignore error string capitalization (many errors intentionally capitalized)
      - linters:
          - staticcheck
        text: "ST1005:"
      # Ignore dot imports (intentional in some packages)
      - linters:
          - staticcheck
        text: "ST1001:"
      # Ignore type inference suggestions (explicit types can improve readability)
      - linters:
          - staticcheck
        text: "ST1023:"
      # Ignore nil check for len suggestions (explicit nil checks can be clearer)
      - linters:
          - staticcheck
        text: "S1009:"
      # Ignore pointer-like allocation suggestions (performance optimization, not critical)
      - linters:
          - staticcheck
        text: "SA6002:"
      # Ignore possible nil dereference in vendored/complex code
      - linters:
          - staticcheck
        text: "SA5011"
      # Ignore unused value warnings (common in tests)
      - linters:
          - staticcheck
        text: "SA4006:"
      # Ignore same type assertion (sometimes used for interface validation)
      - linters:
          - staticcheck
        text: "S1040:"
      # Ignore unnecessary Sprintf (readability preference)
      - linters:
          - staticcheck
        text: "S1039:"
      # Ignore String() vs Sprintf preference
      - linters:
          - staticcheck
        text: "S1025:"
      # Ignore variable declaration merge suggestions
      - linters:
          - staticcheck
        text: "S1021:"
      # Ignore govet shadow warnings (too many false positives)
      - linters:
          - govet
        text: "shadow:"
      # Ignore govet copylocks warnings (architectural tech debt)
      - linters:
          - govet
        text: "copylocks:"
      # Ignore govet unreachable code (sometimes intentional for safety)
      - linters:
          - govet
        text: "unreachable:"
      # Ignore ineffassign in test files
      - linters:
          - ineffassign
        path: "_test\\.go$"

issues:
  max-issues-per-linter: 0
  max-same-issues: 0

linters-settings:
  staticcheck:
    checks:
      - "all"
      - "-ST1000" # Package comments
      - "-ST1003" # Naming convention