avito-pr-project/.golangci.yml at main · AidarHaziahmetov/avito-pr-project · 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
143
144
145
146
147
148
149
150
151
152
153
154
155
run:
  timeout: 5m
  tests: true
  modules-download-mode: readonly

linters:
  enable:
    - errcheck      # Проверка обработки ошибок
    - gosimple      # Упрощение кода
    - govet         # Vet проверяет код на проблемы
    - ineffassign   # Неэффективные присваивания
    - staticcheck   # Статический анализ
    - unused        # Неиспользуемые константы, переменные, функции
    - typecheck     # Проверка типов
    - gofmt         # Форматирование кода
    - goimports     # Проверка импортов
    - misspell      # Орфографические ошибки
    - revive        # Замена golint
    - gosec         # Проверка безопасности
    - bodyclose     # Проверка закрытия HTTP body
    - noctx         # Проверка передачи context
    - sqlclosecheck # Проверка закрытия SQL запросов
    - rowserrcheck  # Проверка ошибок SQL rows
    - copyloopvar   # Проверка ссылок в циклах (замена exportloopref)
    - goconst       # Повторяющиеся строки -> константы
    - gocritic      # Мета-линтер с множеством проверок
    - dupl          # Дубликаты кода
    - whitespace    # Лишние пробелы

linters-settings:
  errcheck:
    check-type-assertions: true
    check-blank: true

  govet:
    enable:
      - shadow
      - fieldalignment

  goimports:
    local-prefixes: github.com/aidar/avito-pr-project

  revive:
    rules:
      - name: exported
        disabled: false
      - name: var-naming
        disabled: false
      - name: package-comments
        disabled: true
      - name: indent-error-flow
        disabled: false
      - name: error-strings
        disabled: false
      - name: error-naming
        disabled: false

  gosec:
    excludes:
      - G204 # Subprocess launched with variable
      - G304 # File path provided as taint input

  goconst:
    min-len: 3
    min-occurrences: 3

  dupl:
    threshold: 100

  gocritic:
    enabled-tags:
      - diagnostic
      - style
      - performance
      - experimental
      - opinionated

issues:
  exclude-use-default: false
  max-issues-per-linter: 0
  max-same-issues: 0
  uniq-by-line: true
  exclude-dirs:
    - bin
    - vendor

  exclude-rules:
    # Исключаем некоторые линтеры для тестовых файлов
    - path: _test\.go
      linters:
        - dupl
        - gosec
        - goconst
        - errcheck
        - noctx
        - govet

    # Исключаем линтеры для тестовых helper'ов
    - path: tests/
      linters:
        - errcheck
        - noctx
        - govet
        - gosec

    # Исключаем известные false positives
    - text: "weak cryptographic primitive"
      linters:
        - gosec

    # Разрешаем слабый random для выбора ревьюверов (не критично для безопасности)
    - text: "G404.*weak random"
      linters:
        - gosec

    # Разрешаем blank импорты для embed
    - source: "_ \"embed\""
      linters:
        - revive

    # Игнорируем fieldalignment для config структур (читаемость важнее)
    - path: internal/config/
      text: "fieldalignment"
      linters:
        - govet

    # Разрешаем похожие паттерны в handlers (стандартный код)
    - path: internal/handler/
      text: "lines are duplicate"
      linters:
        - dupl

    # Игнорируем hugeParam для методов config (читаемость)
    - path: internal/config/
      text: "hugeParam"
      linters:
        - gocritic

    # Игнорируем fieldalignment для domain структур (читаемость важнее)
    - path: internal/domain/
      text: "fieldalignment"
      linters:
        - govet

    # Разрешаем игнорировать ошибки в defer rollback (стандартный паттерн)
    - text: "Error return value of.*Rollback.*is not checked"
      linters:
        - errcheck

output:
  formats:
    - format: colored-line-number
  print-issued-lines: true
  print-linter-name: true
  sort-results: true