.golangci.yml

# GolangCI-Lint v2 Configuration for gogpu/gg - Pure Go 2D Graphics Library
# Documentation: https://golangci-lint.run/docs/configuration/

version: "2"

run:
  timeout: 5m
  tests: true

linters:
  enable:
    # Code quality and complexity
    - gocyclo       # Check cyclomatic complexity
    - gocognit      # Check cognitive complexity
    - funlen        # Check function length
    - maintidx      # Maintainability index
    - cyclop        # Check cyclomatic complexity (alternative)
    - nestif        # Reports deeply nested if statements

    # Bug detection
    - govet         # Standard Go vet
    - staticcheck   # Comprehensive static analysis
    - errcheck      # Check that errors are handled
    - errorlint     # Check error wrapping
    # gosec disabled: v2.23.0 panics on float constants (securego/gosec#1502)
    # Re-enable after golangci-lint ships gosec ≥2.23.1
    # - gosec
    - nilnil        # Check that functions don't return nil both ways
    - nilerr        # Check nil error returns
    - ineffassign   # Detect ineffectual assignments

    # Code style and consistency
    - misspell      # Check for misspelled words
    - whitespace    # Check for trailing whitespace
    - unconvert     # Remove unnecessary type conversions
    - unparam       # Detect unused function parameters

    # Naming conventions
    - errname       # Check error naming conventions
    - revive        # Fast, configurable, extensible linter

    # Performance
    - prealloc      # Find slice declarations that could be preallocated
    - makezero      # Find slice declarations with non-zero initial length

    # Code practices
    - goconst       # Find repeated strings that could be constants
    - gocritic      # Comprehensive code checker
    - goprintffuncname  # Check printf-like function names
    - nolintlint    # Check nolint directives are used correctly
    - nakedret      # Checks for naked returns

    # Additional quality checkers
    - dupl          # Detect duplicate code
    - dogsled       # Check for assignments with too many blank identifiers
    - durationcheck # Check for two durations multiplied together

  settings:
    govet:
      enable:
        - copylocks
      disable:
        - fieldalignment

    gocyclo:
      min-complexity: 20

    cyclop:
      max-complexity: 20

    funlen:
      lines: 120
      statements: 60

    gocognit:
      min-complexity: 30

    misspell:
      locale: US

    nestif:
      min-complexity: 4

    goconst:
      min-occurrences: 10  # enum String() methods share common names ("Auto", "None", "Size") across files
      ignore-string-values:
        - "Unknown"

    revive:
      rules:
        - name: var-naming
        - name: error-return
        - name: error-naming
        - name: if-return
        - name: increment-decrement
        - name: var-declaration
        - name: range
        - name: receiver-naming
        - name: time-naming
        - name: unexported-return
        - name: indent-error-flow
        - name: errorf
        - name: empty-block
        - name: superfluous-else
        - name: unreachable-code
        - name: redefines-builtin-id

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

      disabled-checks:
        - commentFormatting
        - whyNoLint
        - unnamedResult
        - commentedOutCode
        - octalLiteral
        - paramTypeCombine

      settings:
        hugeParam:
          sizeThreshold: 256

  exclusions:
    paths:
      - tmp

    rules:
      # Test files - allow more flexibility
      - path: _test\.go
        linters:
          - gocyclo
          - cyclop
          - funlen
          - maintidx
          - errcheck
          - gosec
          - goconst
          - dupl
          - gocognit

      # Internal font handling
      - path: internal/fontkit/.*\.go
        linters:
          - funlen       # Font parsing is verbose
          - gocyclo      # Font handling can be complex

      # Internal image processing (v0.3.0)
      - path: internal/image/.*\.go
        linters:
          - gocyclo      # Interpolation algorithms are complex
          - funlen       # Image processing is verbose
          - cyclop       # Complex branching in sampling
          - gocognit     # Bilinear/bicubic inherently complex
          - gosec        # Integer conversions are intentional in image processing

      # Internal clipping (v0.3.0)
      - path: internal/clip/.*\.go
        linters:
          - gocyclo      # Cohen-Sutherland, curve clipping
          - funlen       # Clipping algorithms are verbose
          - cyclop       # Complex edge cases
          - gocognit     # Geometric algorithms

      # Internal blend modes (v0.3.0)
      - path: internal/blend/.*\.go
        linters:
          - gocyclo      # Many blend modes = many branches
          - cyclop       # Porter-Duff has many cases
          - funlen       # GetBlendFunc is a large switch statement

      # Internal filter effects (TASK-099b)
      - path: internal/filter/.*\.go
        linters:
          - gocyclo      # Blur convolution loops are complex
          - funlen       # Filter algorithms are verbose
          - cyclop       # Complex bounds checking
          - gocognit     # Separable blur inherently complex
          - gosec        # Integer conversions are intentional in filter processing

      # Text script detection (TEXT-004)
      - path: text/script\.go
        linters:
          - gocyclo      # Unicode script detection requires many branches
          - cyclop       # Many Unicode ranges to check
          - gocognit     # Inherently complex due to Unicode

      # Emoji detection and sequence handling (TASK-050i)
      - path: text/emoji/.*\.go
        linters:
          - gocyclo      # Emoji Unicode ranges require many branches
          - cyclop       # Many emoji ranges to check
          - gocognit     # Inherently complex due to Unicode spec
          - funlen       # Emoji range functions are necessarily verbose
          - maintidx     # Unicode lookups inherently complex

      # Scene package blend mode mapping (TASK-099)
      - path: scene/encoding\.go
        linters:
          - gocyclo      # String() for 28 blend modes is a large switch
          - cyclop       # Same reason - mode classification methods
          - funlen       # String() is necessarily large with 28 cases

      - path: scene/blend_integration\.go
        linters:
          - gocyclo      # Mode mapping switch statements are necessarily large
          - cyclop       # Same reason
          - funlen       # Conversion functions map all 28+ modes

      # Internal packages that intentionally shadow stdlib names.
      # These are INTERNAL packages — renaming would be a massive refactor
      # for no benefit. The names match their domain (color, image, path).
      - path: internal/color/.*\.go
        linters:
          - revive        # var-naming: package name shadows stdlib color

      - path: internal/image/.*\.go
        linters:
          - revive        # var-naming: package name shadows stdlib image

      - path: internal/path/.*\.go
        linters:
          - revive        # var-naming: package name shadows stdlib path

      # TrueType bytecode interpreter - skrifa port (1:1 translation from Rust)
      # Algorithmic complexity is inherent to virtual machine instruction dispatch
      - path: text/tt_.*\.go
        linters:
          - gocognit     # Outline ops (MDRP/MIRP) are algorithmically complex
          - gocyclo      # Error enum switch and dispatch are large
          - cyclop        # Same — faithful port of skrifa round/error functions
          - nestif        # Deep nesting inherent to point movement with fallbacks
          - funlen        # Skrifa port functions are inherently long (1:1 translation)
          - errname       # tt-prefixed error names match subsystem convention
          - maintidx      # Opcode dispatch has inherently low maintainability index

      # CPU rasterization CORE - Skia AAA port (1:1 translation from C++)
      # Algorithmic complexity is inherent to edge-walk rasterization algorithms
      - path: internal/raster/.*\.go
        linters:
          - gocognit     # Skia trapezoid walker and convex walker are complex algorithms
          - gocyclo      # Same — ported Skia edge-walk algorithms
          - cyclop        # Same — complex control flow in rasterization
          - nestif        # Deep nesting inherent to multi-path edge walker (rect/general)
          - funlen        # Skia port functions are inherently long (1:1 translation)
          - maintidx      # Skia port has low maintainability index by design

      # Vello compute pipeline port - 1:1 translation from Rust/WGSL (v0.30.0)
      # Algorithmic complexity is inherent to GPU compute pipeline stages
      - path: internal/gpu/tilecompute/.*\.go
        linters:
          - gocognit     # GPU pipeline stages are algorithmically complex
          - gocyclo      # Same — ported GPU algorithms
          - cyclop        # Same — complex control flow in GPU stages
          - nestif        # Deep nesting inherent to tile walk algorithms
          - funlen        # GPU stages are verbose
          - maintidx      # GPU pipeline stages have inherently low maintainability index
          - gocritic      # Ported patterns (ifElseChain, badCond for degenerate checks)
          - nolintlint    # Stale directives from porting

      # Vello GPU accelerator and compute dispatcher (v0.30.0)
      # Complex diagnostic logging and buffer management for 9-stage pipeline
      - path: internal/gpu/vello_accelerator\.go
        linters:
          - gocognit     # Diagnostic logging inspects all 9 pipeline stages
          - gocyclo      # Same — complex diagnostic function
          - cyclop        # Same — complex cyclomatic complexity
          - funlen        # logPipelineDiagnostics inspects all pipeline buffers
          - maintidx      # Same — diagnostic function has many branches
          - nestif        # Buffer readback error handling requires deep nesting
          - unparam       # numPaths reserved for future per-path diagnostics

      # GPU render pipelines - structurally similar WebGPU pipeline descriptors
      # Each pipeline (SDF, convex, stencil, text, glyph mask, LCD) has its own
      # CreateRenderPipeline call with unique labels, shaders, and vertex layouts.
      # The boilerplate structure is identical but the content differs per tier.
      - path: internal/gpu/.*\.go
        linters:
          - dupl          # Pipeline creation boilerplate is not real duplication

      # Example code - demonstration purposes
      - path: examples?/.*\.go
        linters:
          - errcheck
          - errorlint
          - funlen
          - gocyclo
          - cyclop
          - gocognit
          - godot
          - revive
          - gosec
          - gocritic

      # Demo command
      - path: cmd/.*\.go
        linters:
          - errcheck
          - funlen
          - gocyclo

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