Go Linters: コード品質に不可欠なツール

# バイナリインストール（推奨）
curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin

# またはGoでインストール
go install github.com/golangci/golangci-lint/cmd/golangci-lint@latest

linters:
  enable:
    - staticcheck
    - gosimple
    - govet
    - errcheck
    - gosec
    - revive
    - gocyclo
    - misspell
    - unconvert
    - unparam

linters-settings:
  errcheck:
    check-type-assertions: true
    check-blank: true
  
  govet:
    enable-all: true
  
  gocyclo:
    min-complexity: 15
  
  revive:
    severity: warning

run:
  timeout: 5m
  tests: true
  skip-dirs:
    - vendor
    - third_party

issues:
  exclude-use-default: false
  max-issues-per-linter: 0
  max-same-issues: 0

// staticcheckはこの一般的な間違いを検出します
func processData(ctx context.Context) {
    go func() {
        // SA1012: context.Contextは構造体に保存したり、関数が戻った後に渡してはいけません
        // または関数が戻った後に渡してはいけません
        doWork(ctx)  
    }()
}

// staticcheckは非効率的な文字列連結を検出します
func buildString(items []string) string {
    s := ""
    for _, item := range items {
        s += item // SA1024: strings.Builderを使用してください
    }
    return s
}

staticcheck ./...
staticcheck -f stylish ./...  # より見やすい出力
staticcheck -checks SA1*,ST* ./...  # 特定のカテゴリ

# goimportsをインストール
go install golang.org/x/tools/cmd/goimports@latest

# すべてのGoファイルをフォーマット
goimports -w .

# 変更せずに確認
goimports -d .

// goimports前のコード
import (
    "fmt"
    "github.com/pkg/errors"
    "os"
)

// goimports後のコード（自動的にソートおよび整理）
import (
    "fmt"
    "os"
    
    "github.com/pkg/errors"
)

{
  "go.formatTool": "goimports",
  "[go]": {
    "editor.formatOnSave": true
  }
}

go install github.com/securego/gosec/v2/cmd/gosec@latest
gosec ./...

// G201: SQL文字列連結
db.Query("SELECT * FROM users WHERE name = '" + userInput + "'")

// G304: ファイルパスが汚染された入力として提供されている
ioutil.ReadFile(userInput)

// G401: 弱い暗号化プリミティブ
h := md5.New()

// G101: ハードコードされた資格情報
password := "admin123"

linters:
  enable:
    - gosec

linters-settings:
  gosec:
    excludes:
      - G204  # サブプロセスコマンドの監査
    severity: high

linters-settings:
  revive:
    rules:
      - name: var-naming
        severity: warning
        arguments:
          - ["ID", "URL", "HTTP", "API", "JSON", "XML"]  # 允許される頭文字
      - name: cognitive-complexity
        arguments: [15]
      - name: cyclomatic
        arguments: [10]
      - name: line-length-limit
        arguments: [120]
      - name: function-length
        arguments: [50, 0]

// errcheckはこれを検出します
file.Close()  // エラーが無視されています！

// 正しい方法
if err := file.Close(); err != nil {
    log.Printf("ファイルのクローズに失敗しました: %v", err)
}

{
  "gopls": {
    "analyses": {
      "unusedparams": true,
      "shadow": true,
      "nilness": true,
      "unusedwrite": true,
      "fieldalignment": true
    },
    "staticcheck": true
  }
}

name: Lint
on:
  pull_request:
  push:
    branches: [main]

jobs:
  golangci:
    name: lint
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
      
      - uses: actions/setup-go@v5
        with:
          go-version: '1.22'
          cache: true
      
      - name: golangci-lint
        uses: golangci/golangci-lint-action@v4
        with:
          version: latest
          args: --timeout=5m
          # PRでのみ新しい問題を表示
          only-new-issues: true

lint:
  image: golangci/golangci-lint:latest
  stage: test
  script:
    - golangci-lint run --timeout=5m --out-format colored-line-number
  cache:
    paths:
      - .golangci.cache
  only:
    - merge_requests
    - main

docker run --rm -v $(pwd):/app -w /app golangci/golangci-lint:latest golangci-lint run -v

.PHONY: lint
lint:
	golangci-lint run --timeout=5m

.PHONY: lint-fix
lint-fix:
	golangci-lint run --fix --timeout=5m

.PHONY: format
format:
	goimports -w .
	gofmt -s -w .

.PHONY: check
check: format lint
	go test -race -coverprofile=coverage.out ./...
	go vet ./...

# 修正可能なものを自動修正
golangci-lint run --fix

# 特定のリンターのみを修正
golangci-lint run --fix --disable-all --enable=goimports,gofmt

# 変更を適用せずにプレビュー
golangci-lint run --fix --out-format=json | jq '.Issues[] | select(.Fixed == true)'

// ineffassign: 無効な割り当て
x := 5  // 使用されていない
x = 10

// 修正：使用されていない変数を削除

// nilaway: 未初期化のポインタの dereference の可能性
var user *User
fmt.Println(user.Name)  // user が nil の場合にクラッシュ

// 修正：nilチェックを追加
if user != nil {
    fmt.Println(user.Name)
}

// prealloc: スライスの事前確保を提案
var results []string
for _, item := range items {
    results = append(results, process(item))
}

// 修正：事前確保
results := make([]string, 0, len(items))

// 特定のリンターを無効にする
//nolint:errcheck
file.Close()

// 複数のリンターを無効にし、理由を記載
//nolint:gosec,G304 // ユーザー提供のパスは以前に検証済み
ioutil.ReadFile(trustedPath)

// 全ファイルで無効にする
//nolint:stylecheck
package main

run:
  # より多くのCPUコアを使用
  concurrency: 4
  
  # 分析結果をキャッシュ
  build-cache: true
  modules-download-mode: readonly
  
  # 生成されたファイルをスキップ
  skip-files:
    - ".*\\.pb\\.go$"
    - ".*_generated\\.go$"

# GitHub Actions
- uses: actions/cache@v3
  with:
    path: ~/.cache/golangci-lint
    key: ${{ runner.os }}-golangci-lint-${{ hashFiles('**/go.sum') }}

linters:
  enable:
    - staticcheck
    - govet
    - errcheck
    - gosec
    - gosimple
    - ineffassign
    - revive
    - typecheck
    - unused
    - misspell
    - gocyclo
    - dupl
    - goconst
    - gofmt
    - goimports
    
linters-settings:
  gocyclo:
    min-complexity: 10
  errcheck:
    check-type-assertions: true
    check-blank: true
  gosec:
    severity: medium

linters:
  enable:
    - staticcheck
    - govet
    - errcheck
    - unparam
    - unconvert
    - misspell
    - gofmt
    - goimports
    - nakedret
    - gocognit

linters-settings:
  nakedret:
    max-func-lines: 30
  gocognit:
    min-complexity: 20

linters:
  enable:
    - govet
    - errcheck
    - staticcheck
    - gofmt
    - ineffassign
    
run:
  tests: false  # 速度のためテストのリントをスキップ

issues:
  exclude-rules:
    - path: _test\.go
      linters:
        - errcheck

go install go.uber.org/nilaway/cmd/nilaway@latest
nilaway ./...

go install github.com/segmentio/golines@latest
golines -w --max-len=120 .

go install golang.org/x/vuln/cmd/govulncheck@latest
govulncheck ./...

run:
  tests: true  # テストファイルを分析
  
issues:
  exclude-rules:
    # ただしテストではある程度の柔軟性を許可
    - path: _test\.go
      linters:
        - funlen
        - gocyclo

# プリコミットフック
cat > .git/hooks/pre-commit << 'EOF'
#!/bin/sh
make lint
EOF
chmod +x .git/hooks/pre-commit