أدوات Go Linters: أدوات ضرورية لجودة الكود

# التثبيت من خلال ملف ثنائي (الموصى به)
curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin

# أو عبر Go install
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
          # عرض المشكلات الجديدة فقط في طلبات الدمج
          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: احتمال لاستخدام مؤشر NULL
var user *User
fmt.Println(user.Name)  // يتعطل إذا كان user NULL

// الإصلاح: أضف فحص NULL
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:
  # استخدم المزيد من نوى المعالج
  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