Nophase linter

Initial commit of a nophase linter. Phases, much like capes simply won't
do!

https://i.redd.it/84209d3ydns91.jpg

Author: theobarberbany

pkg/analysis/nophase/analyzer.go

@@ -0,0 +1,62 @@
+package nophase
+
+import (
+	"errors"
+	"go/ast"
+	"strings"
+
+	"golang.org/x/tools/go/analysis"
+	"golang.org/x/tools/go/analysis/passes/inspect"
+	"golang.org/x/tools/go/ast/inspector"
+)
+
+const name = "nophase"
+
+var errCouldNotGetInspector = errors.New("could not get inspector")
+
+var Analyzer = &analysis.Analyzer{
+	Name:     name,
+	Doc:      "phase fields are deprecated and conditions should be preferred, avoid phase like enum fields",
+	Run:      run,
+	Requires: []*analysis.Analyzer{inspect.Analyzer},
+}
+
+func run(pass *analysis.Pass) (interface{}, error) {
+	inspect, ok := pass.ResultOf[inspect.Analyzer].(*inspector.Inspector)
+	if !ok {
+		return nil, errCouldNotGetInspector
+	}
+
+	// Filter to structs so that we can iterate over fields in a struct.
+	nodeFilter := []ast.Node{
+		(*ast.StructType)(nil),
+	}
+
+	// Preorder visits all the nodes of the AST in depth-first order. It calls
+	// f(n) for each node n before it visits n's children.
+	//
+	// We use the filter defined above, ensuring we only look at struct fields.
+	inspect.Preorder(nodeFilter, func(n ast.Node) {
+		sTyp, ok := n.(*ast.StructType)
+		if !ok {
+			return
+		}
+
+		if sTyp.Fields == nil {
+			return
+		}
+
+		for _, field := range sTyp.Fields.List {
+			fieldName := field.Names[0].Name
+
+			if strings.Contains(strings.ToLower(fieldName), "phase") {
+				pass.Reportf(field.Pos(),
+					"field %s: phase fields are deprecated and conditions should be preferred, avoid phase like enum fields",
+					fieldName,
+				)
+			}
+		}
+	})
+
+	return nil, nil //nolint:nilnil
+}

pkg/analysis/nophase/analyzer_test.go

@@ -0,0 +1,13 @@
+package nophase_test
+
+import (
+	"testing"
+
+	"github.com/JoelSpeed/kal/pkg/analysis/nophase"
+	"golang.org/x/tools/go/analysis/analysistest"
+)
+
+func Test(t *testing.T) {
+	testdata := analysistest.TestData()
+	analysistest.RunWithSuggestedFixes(t, testdata, nophase.Analyzer, "a")
+}

pkg/analysis/nophase/doc.go

@@ -0,0 +1,11 @@
+/*
+nophase provides a linter to ensure that structs do not contain a Phase field.
+
+Phase fields are deprecated and conditions should be preferred. Avoid phase like enum
+fields.
+
+The linter will flag any struct field containing the substring 'phase'. This means both
+Phase and FooPhase will be flagged.
+*/
+
+package nophase

pkg/analysis/nophase/initializer.go

@@ -0,0 +1,30 @@
+package nophase
+
+import (
+	"github.com/JoelSpeed/kal/pkg/config"
+	"golang.org/x/tools/go/analysis"
+)
+
+// Initializer returns the AnalyzerInitializer for this
+// Analyzer so that it can be added to the registry.
+func Initializer() initializer {
+	return initializer{}
+}
+
+// intializer implements the AnalyzerInitializer interface.
+type initializer struct{}
+
+// Name returns the name of the Analyzer.
+func (initializer) Name() string {
+	return name
+}
+
+// Init returns the intialized Analyzer.
+func (initializer) Init(cfg config.LintersConfig) (*analysis.Analyzer, error) {
+	return Analyzer, nil
+}
+
+// Default determines whether this Analyzer is on by default, or not.
+func (initializer) Default() bool {
+	return true
+}

pkg/analysis/nophase/testdata/src/a/a.go

@@ -0,0 +1,13 @@
+package a
+
+type NoPhaseTestStruct struct {
+	// +optional
+	Phase *string `json:"phase,omitempty"` // want "field Phase: phase fields are deprecated and conditions should be preferred, avoid phase like enum fields"
+
+}
+
+type NoSubPhaseTestStruct struct {
+	// +optional
+	FooPhase *string `json:"phase,omitempty"` // want "field FooPhase: phase fields are deprecated and conditions should be preferred, avoid phase like enum fields"
+
+}

pkg/analysis/registry.go

@@ -5,6 +5,7 @@ import (
 
 	"github.com/JoelSpeed/kal/pkg/analysis/commentstart"
 	"github.com/JoelSpeed/kal/pkg/analysis/jsontags"
+	"github.com/JoelSpeed/kal/pkg/analysis/nophase"
 	"github.com/JoelSpeed/kal/pkg/analysis/optionalorrequired"
 	"github.com/JoelSpeed/kal/pkg/analysis/requiredfields"
 	"github.com/JoelSpeed/kal/pkg/config"
@@ -53,6 +54,7 @@ func NewRegistry() Registry {
 			jsontags.Initializer(),
 			optionalorrequired.Initializer(),
 			requiredfields.Initializer(),
+			nophase.Initializer(),
 		},
 	}
 }