golint-sl

Appearance

lifecycle

About 257 wordsLess than 1 minute

2025-01-16

Enforces component lifecycle patterns (Run/Close).

Category

Architecture

What It Checks

This analyzer ensures components that start background work have proper lifecycle methods:

  • Run(ctx context.Context) error for starting
  • Close() error for cleanup

Why It Matters

Components without lifecycle management:

  • Leak goroutines
  • Don't handle shutdown gracefully
  • Can't be properly tested
  • Cause resource leaks

Examples

Bad: No Lifecycle

type Worker struct {
    done chan struct{}
}

func NewWorker() *Worker {
    w := &Worker{done: make(chan struct{})}
    go w.loop()  // Starts goroutine in constructor!
    return w
}

func (w *Worker) loop() {
    for {
        // No way to stop this!
        doWork()
    }
}

Good: Proper Lifecycle

type Worker struct {
    done chan struct{}
}

func NewWorker() *Worker {
    return &Worker{
        done: make(chan struct{}),
    }
}

// Run starts the worker. It blocks until ctx is cancelled or Close is called.
func (w *Worker) Run(ctx context.Context) error {
    for {
        select {
        case <-ctx.Done():
            return ctx.Err()
        case <-w.done:
            return nil
        default:
            doWork()
        }
    }
}

// Close stops the worker gracefully.
func (w *Worker) Close() error {
    close(w.done)
    return nil
}

Usage Pattern

func main() {
    ctx, cancel := context.WithCancel(context.Background())
    defer cancel()

    worker := NewWorker()

    // Handle shutdown
    go func() {
        sigCh := make(chan os.Signal, 1)
        signal.Notify(sigCh, syscall.SIGTERM, syscall.SIGINT)
        <-sigCh
        worker.Close()
    }()

    if err := worker.Run(ctx); err != nil {
        log.Fatal(err)
    }
}

The Lifecycle Interface

type Lifecycle interface {
    Run(ctx context.Context) error
    Close() error
}

Configuration

# .golint-sl.yaml
analyzers:
  lifecycle: true  # enabled by default

When to Disable

  • Simple utilities without background work
  • Stateless functions
analyzers:
  lifecycle: false