Add Rule Skill

Add Rule Skill

This skill helps you add new rules to the attgo-linter project.

Usage

When asked to add a new linter rule, follow these steps:

1. Gather Information

Ask the user for:

• Rule name: e.g., "no-global-vars"
• Description: What the rule checks
• Priority: HIGH (enabled by default), MEDIUM, or LOW (disabled by default)
• Bad example: Code that should trigger the warning
• Good example: Code that passes the check
• Configuration options: Any customizable settings

2. Create Analyzer Package

Create the following files:

analyzers/​{rulename}/
├── analyzer.go
└── analyzer_test.go

The analyzer name must use underscores (not hyphens): attgo_{rule_name}

3. Create Test Fixtures

analyzers/​{rulename}/​testdata/​src/​{rulename}/
└── {rulename}.go

Use // want \regex pattern`` comments for expected diagnostics.

4. Register in Plugin

Update plugin.go:

• Add import
• Add to BuildAnalyzers() with config check

Update config.go:

• Add Enable{RuleName} field to Config struct
• Set default in DefaultConfig()

Update plugin.go config handling:

• Add explicit check for the setting in New()

5. Update Documentation

• Add rule section to README.md
• Create docs/​rules/​attgo-{rule-name}.md
• Update CHANGELOG.md

6. Verify

Run tests:

go test ./​...

Template: analyzer.go

// Copyright © 2026 Attestant Limited.
// Licensed under the Apache License, Version 2.0 (the "License");
// ...

// Package {rulename} provides an analyzer that {description}.
package {rulename}

import (
    "golang.org/​x/​tools/​go/​analysis"
)

const (
    analyzerName = "attgo_{rule_name}"
    doc          = `{short description}

{detailed explanation}

Bad:
    {bad example}

Good:
    {good example}`
)

// Analyzer is the {description} analyzer.
var Analyzer = &analysis.Analyzer{
    Name: analyzerName,
    Doc:  doc,
    Run:  run,
}

func run(pass *analysis.Pass) (any, error) {
    for _, file := range pass.Files {
        // Implementation
    }
    return nil, nil
}

Template: analyzer_test.go

// Copyright © 2025 Attestant Limited.
// ...

package {rulename}_test

import (
    "testing"

    "github.com/​attestantio/​attgo-linter/​analyzers/​{rulename}"
    "golang.org/​x/​tools/​go/​analysis/​analysistest"
)

func TestAnalyzer(t *testing.T) {
    testdata := analysistest.TestData()
    analysistest.Run(t, testdata, {rulename}.Analyzer, "{rulename}")
}

Template: testdata

// Copyright © 2025 Attestant Limited.
// ...

package {rulename}

// Bad case.
var bad = something // want `expected error`

// Good case.
var good = something // No warning

Checklist

• Created analyzers/​{rulename}/​analyzer.go
• Created analyzers/​{rulename}/​analyzer_test.go
• Created analyzers/​{rulename}/​testdata/​src/​{rulename}/​{rulename}.go
• Added import to plugin.go
• Added to BuildAnalyzers() in plugin.go
• Added config field to config.go
• Added default value in DefaultConfig()
• Added config handling in New()
• Tests pass: go test ./​...
• Added documentation to README.md
• Updated CHANGELOG.md